[5.3] Single Notification for User Collection (followers)

Hello!
So for a single notifiable user, a single entry in the notifications table, along with a mail/sms sent is perfectly working.
An important issue I'm facing with this however, is when I have a user collection, a list of 1k users following me, and I post an update. Here is what happens when using the Notifiable trait as suggested for multi user case:

1k mails/sms sent (issue not here)
1k notification entries added to the db notifications table

It seems that adding 1k notifications to the db notifications table is not an optimal solution. Since the toArray data is the same, and everything else in the db notifications table is the same for 1k rows, with the only difference being the notifiable_id of the user notifiable_type.
An optimal solution out of the box would be:

laravel would pick up the fact that it's an array notifiable_type
Save a single notification as notifiable_type user_array or user with notifiable_id 0 (zero would only be used to signify it's a multi notifiable user)
Create/use another table notifications_read using the notification_id it just created as the foreign_key and insert 1k rows, of just these fields:
notification_id notifiable_id notifiable_type read_at

I am hoping there is already a way to do this as I am at this point in my application and would love to use the built in Notifications for this situation, as I am firing off emails/sms notifications, which is fine to repeat 1k times I think, but it's the entry of the same data into the db that is the problem that needs to be optimized.
Any thoughts/ideas how to proceed in this situation?

Please open an issues for the internals repo if you have a RFC, or ask on the forums if you'd just like to talk about implementations with people please. :)

I asked here on the forums for others.

Please open an issues for the internals repo if you have a RFC

@GrahamCampbell Can you point me to a url please so I can check it out?


RFM - peazip.portable

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details

Package URL: https://chocolatey.org/packages/peazip.portable
Package source URL: https://github.com/dtgm/chocolatey-packages/tree/master/automatic/peazip.portable/

Date the maintainer was contacted (in YYYY-MM-DD): 2021-12-13
How the maintainer was contacted: email

The peazip and peazip.install packages are maintained in the community repository. Please open up a discussion there to get the portable variant moved to the same repository.


tests, integ: fix TypeError on dns tests

On some platforms NMCI is getting the following error:
state = {'dns-resolver': {'config': {'search': [], 'server':
MarkDecorator(mark=Mark(name='xfail', args=(['2001:4860:4860::888...:
'eth1'}, {'destination': '::/0', 'metric': 201, 'next-hop-address':
'2001:db8:2::f', 'next-hop-interface': 'eth1'}]}}

    def validate_dns(state):
        """
        Only support at most 2 name servers now:
        https://nmstate.atlassian.net/browse/NMSTATE-220
        """
        dns_servers = (
            state.get(DNS.KEY, {}).get(DNS.CONFIG, {}).get(DNS.SERVER, [])
        )
>       if len(dns_servers) > 3:
E       TypeError: object of type 'MarkDecorator' has no len()

In order to fix this, this patch is splitting the supported and not
supported 3+ nameserver edit tests.
Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

The pytest doc: http://doc.pytest.org/en/latest/skipping.html#skip-xfail-with-parametrize suggests other format. But not sure it is support by our CI environment. Just a FYI.

Done, thanks!


[Proposal] Hotfix into stable: Dart compiler crashes while building for web

Steps to Reproduce

Running flutter build web crashes the dart compiler.

This issues has been fixes in dart-sdk: https://github.com/dart-lang/sdk/issues/47916
Would it be possible to bring this as a hotfix into flutter? I see that the dart version in flutter master channel is 2.17.0-182.0.dev. But this fix mentioned is included in dart versions starting from 2.17.0-188.0.dev.
Or is there a way to change the dart version in flutter sdk?
Note: This issue exists in flutter stable channel as well, but the logs included is with master channel
Basic Repro
abstract class A {
  const factory A() = B;
}

abstract class B implements A {
  const factory B() = C;
}

class C implements B {
  const C();
}

main() {
  A.new;
}

Note: This crashes the flutter dart2js compiler, works fine with android and iOS.
Logs

Logs

 Target dart2js failed: Exception: Warning: The 'dart2js' entrypoint script is deprecated, please use 'dart compile js' instead.
           lib/app/pages/signals/application/signal-actor-bloc/signal-actor-bloc.cg.freezed.dart:5940:17:
           Internal Error: The compiler crashed when compiling this element.
             const factory _ReportInaccurateSignalFailure(SignalsFailure failure) =
                           ^
           The compiler is broken.

           When compiling the above element, the compiler crashed. It is not
           possible to tell if this is caused by a problem in your program or
           not. Regardless, the compiler should not crash.

           The Dart team would greatly appreciate if you would take a moment to
           report this problem at http://dartbug.com/new.

           Please include the following information:

           * the name and version of your operating system,

           * the Dart SDK build number (2.17.0-182.0.dev), and

           * the entire message you see here (including the full stack trace
             below as well as the source location above).

           The compiler crashed: NoSuchMethodError: The getter 'boxedVariables' was called on null.
           Receiver: null
           Tried calling: boxedVariables
           #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:38:5)
           #1      JsKernelToElementMap.makeRecordContainer (package:compiler/src/js_model/element_map_impl.dart:1770:14)
           #2      ClosureDataBuilder.createClosureEntities.processModel (package:compiler/src/js_model/closure.dart:329:23)
           #3      ClosureDataBuilder.createClosureEntities.<anonymous closure>.<anonymous closure>
(package:compiler/src/js_model/closure.dart:389:9)
           #4      CompilerDiagnosticReporter.withCurrentElement (package:compiler/src/compiler.dart:1025:15)
           #5      ClosureDataBuilder.createClosureEntities.<anonymous closure> (package:compiler/src/js_model/closure.dart:388:17)
           #6      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:579:13)
           #7      ClosureDataBuilder.createClosureEntities (package:compiler/src/js_model/closure.dart:387:19)
           #8      JsClosedWorldBuilder.convertClosedWorld (package:compiler/src/js_model/js_world_builder.dart:161:41)
           #9      JsBackendStrategy.createJClosedWorld (package:compiler/src/js_model/js_strategy.dart:174:52)
           #10     Compiler.closeResolution (package:compiler/src/compiler.dart:610:25)
           #11     Compiler.computeClosedWorld (package:compiler/src/compiler.dart:470:9)
           #12     Compiler.compileFromKernel.<anonymous closure>.<anonymous closure> (package:compiler/src/compiler.dart:572:17)
           #13     CompilerTask.measureSubtask (package:compiler/src/common/tasks.dart:181:35)
           #14     Compiler.compileFromKernel.<anonymous closure> (package:compiler/src/compiler.dart:571:44)
           #15     CompilerTask.measureSubtask (package:compiler/src/common/tasks.dart:181:35)
           #16     Compiler.compileFromKernel (package:compiler/src/compiler.dart:570:14)
           #17     Compiler.runInternal (package:compiler/src/compiler.dart:372:17)
           <asynchronous suspension>
           #18     main (package:compiler/src/dart2js.dart:1392:3)
           <asynchronous suspension>

[  +17 ms] Compiling lib/main.dart for the Web... (completed in 30.2s)
[   +1 ms] "flutter web" took 31,363ms.
[   +4 ms] Exception: Failed to compile application for the Web.
[        ]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      buildWeb (package:flutter_tools/src/web/compile.dart:81:5)
           <asynchronous suspension>
           #2      BuildWebCommand.runCommand (package:flutter_tools/src/commands/build_web.dart:112:5)
           <asynchronous suspension>
           #3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1165:27)
           <asynchronous suspension>
           #4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #6      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
           <asynchronous suspension>
           #7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
           <asynchronous suspension>
           #9      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #11     main (package:flutter_tools/executable.dart:94:3)
           <asynchronous suspension>

[  +90 ms] ensureAnalyticsSent: 88ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 1

[✓] Flutter (Channel master, 2.11.0-0.0.pre.871, on macOS 12.2.1 21D62 darwin-x64, locale en-US)
    • Flutter version 2.11.0-0.0.pre.871 at /Users/nithinrao/fvm/versions/beta
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4f4c7816c3 (55 minutes ago), 2022-03-14 07:35:07 +0100
    • Engine revision 6239bfb884
    • Dart version 2.17.0 (build 2.17.0-182.0.dev)
    • DevTools version 2.11.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc5)
    • Android SDK at /Users/nithinrao/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0-rc5
    • ANDROID_SDK_ROOT = /Users/nithinrao/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version 1.65.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.36.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 99.0.4844.51

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @dopecoder, Thanks for filing the issue.

Would it be possible to bring this as a hotfix into flutter? I see that the dart version in the flutter master channel is 2.17.0-182.0.dev. But this fix mentioned is included in dart versions starting from 2.17.0-188.0.dev.

Leaving this issue open for the team members to decide.

Master rolls continuously, and looks like it has the fix now:
Flutter 2.11.0-0.0.pre.886 • channel master • https://github.com/flutter/flutter.git
Framework • revision 2bd3e0d914 (5 hours ago) • 2022-03-15 04:20:15 -0400
Engine • revision 6c02eaea3a
Tools • Dart 2.17.0 (build 2.17.0-202.0.dev) • DevTools 2.11.4

We had some discussions earlier today (before your comment) and we were wondering about that. As with any hotfix, we need to weigh the value added and risk associated with it. In our discussions on the Dart SDk we agreed to cherry-pick this into beta, but didn't conclude about plans for the stable channel yet to get some additional input.
Do you expect most developers affected by this issue are in the stable channel or will it be common for them to be on beta?
@yjbanov @kevmoo - curious to get your input too (you can find more context in https://github.com/dart-lang/sdk/issues/48559)

Hi @sigmundch, really appreciate the work you guys are doing. We usually do production builds in stable, it would be great to get a hot fix in stable. Although fix just in beta should be acceptable.

It seems fix landed in 2.10.4. Can we close this?

@dopecoder can you please confirm if this has landed?

@maheshmnj - I can confirm it has landed. It's subtle, but the second bullet point in the release notes here https://github.com/flutter/flutter/wiki/Hotfixes-to-the-Stable-Channel#2104-march-28-2022 is referring to this issue, but uses the underlying Dart SDK bug instead

Hi @maheshmnj, I can confirm its landed and working. Thanks everyone.


status bar colour in android 4.0

Hi, I'd like to change the status bar color on android 4.0 devices
I saw that the colour change works on jelly bean, but I'd also like to have a specific colour on all my devices (my app is actually targeting v2.3 and above but if it works on android 4.0 that's fine for me.)
So can this library help me do that ??

unfortunately not


(3.0.77) Changes menu item returns changes for Rowan project only

When I ask for changes on projects other than Rowan, I seem to only get Rowan changes ... a look at RowanProjectServices>>changes is a possible clue. Note that Rowan is wired in as a constant:
changes

	| jadeServer |
	jadeServer := Rowan platform jadeServerClassNamed: #JadeServer.
	changes := Array new.
	(Rowan projectTools diff
		patchesForProjectNamed: 'Rowan') do: [:assoc |
			"key is packageName, value is a CypressPatch"
			| patch |
			patch := assoc value.
			changes add:(jadeServer new
				_mcDescriptionOfPatch: patch
				baseName: 'closest ancestor'
				alternateName: nil) ].
	self refresh.
	RowanCommandResult addResult: self.

In 3.0.78, I verified that after I changed a method in RowanSample1, Tonel, Rowan, and Izmir, each of these projects, the changes menu item only returns the set of changes in that correct project.
The contents of RowanProjectService >> changes in my image is
| jadeServer |
jadeServer := Rowan platform jadeServerClassNamed: #JadeServer.
changes := Array new.
self packageNames do:[:packageName |
		| patch |
		patch := Rowan packageTools diff patchForPackageName: packageName.
		changes add:(jadeServer new
			_mcDescriptionOfPatch: patch
			baseName: 'closest ancestor'
			alternateName: nil)].
self refresh.
RowanCommandResult addResult: self.

... just understood that the source for the method in my image is different than the source that you are seeing and that is likely to be a result of a merge mistake on my part when I made changes to how changes were handled for V2.0 ... I'll need to create a patch for V2.0


Adding CMake to Build Canopy-App

add the Cmake build system to canopy-app instead of traditional makefiles.

#75

Remember to update the Binder config when doing this!

@zmoon Moving to CMake would be the other requirement for integrating canopy-app with UFS/CCPP #32, so it would be great to get your assistance or suggestions here as the next developments.


Revert project set to fix continuous build error

This is added in https://github.com/datacommonsorg/tools/pull/162 and causes build error: https://groups.google.com/a/google.com/g/datacommons-alerts/c/dJeTKN-k1Kw/m/6dD9KNJFBAAJ

Thanks for the quick change!
Took a brief look at the script. It seems like the periodic build builds individual repos. But I can't seem to find the build of each repo in Cloud Build. Did you intend to set the project id instead of account so that each individual repo build shows up on datcom-ci?

That's a good point, changed to project.


add noisetorch checksums

It's best not to add checksums for git repos, as they can change

Currently, LURE doesn't use checksums for git repos anyway, as there's no good way to verify them.

Strange, I checked and it works))

It works because LURE ignores checksums for git repos

It works because LURE ignores checksums for git repos
oh) i understud


[BREAKING] Remove deprecated coco binary

A reminder that coco can be removed, to be merged when a major release is planned.

Wrong fork branch 😅


Change MapBox satellite basemap to BCGW base map

[ ] Find BCGW satellite basemap location
[ ] Add BCGW satellite basemap to packages
[ ] Add script to delete MapBox satellite basemap

@bferguso I think this is the basemap we use in HRIA: https://www.arcgis.com/home/item.html?id=10df2279f9684e4a9f6a7f08febac2a9

@bferguso are we able to use this basemap in Arches? https://catalogue.data.gov.bc.ca/dataset/bc-basemap. It has nice hillside shading and park labels and its the same one the BC Geographical Names Offices uses in their app (our Heritage Branch colleagues) https://apps.gov.bc.ca/pub/bcgnws/web/. I have a contact for this basemap so we could ask if it needs to be in a different format for Arches.

Review these for satellite imagery:
https://catalogue.data.gov.bc.ca/dataset/bcad8f9e-274b-4a29-9e06-d05def3a50c2
https://catalogue.data.gov.bc.ca/dataset/73b2a989-df6b-4298-bd38-c60f0254a453
https://catalogue.data.gov.bc.ca/dataset/b590c13e-7b8f-4b1f-a5d0-959e7284b8b7
All three of these records are access only so there are restrictions on using the data. For more information you can consult this link: Satellite imagery access.


The origins menu doesn't appear at all on the server

I wanted to make an origin server for me and my friends to play on, but for whatever reason it just doesn’t work on the server?? The server is 1.20.1 forge and everything seems to be working when I’m in singleplayer but when I log in and check my origin it says “you can not have any origins” and  the origin orb doesn’t work either (the origin choice menu doesn’t appear at all). Any help would be appreciated. I have no mods except for the origins mod itself and its dependencies

Duplicate of #423


lbry-daemon constantly uploading ~1.5mbps

The Issue
Steps to reproduce

Launch LBRY client

Expected behaviour
Catch up with blockchain, load different screens in client; using bandwidth only when loading
Actual behaviour
Catches up with blockchain, constant use of 1.5mbps upload (maximum of my connection) for over a day.
Resource monitor shows lbry-daemon using the majority of the bandwidth. Downloads also increase, but not to the point of constantly maxing out my connection.
System Configuration

LBRY Daemon version: 0.10.0
LBRY App version: 0.10.0rc5
LBRY Installation ID:5K59LZivKh2PmG9tk4gbygqEo9HbQU5ztByhzsSGfC3m1B5ZK9svnbEtJa532zSJ8e
Operating system: Windows 10 (10-10.0.14393)

Anything Else
This is not new to the latest release; launching only lbry-daemon via command line would show the same behavior. I've let it go for over a day with no change. The data directories (listed here: https://lbry.io/faq/lbry-directories) do not appear to increase in size.

I'm working on this, it's happening in the DHT and blob announcer.

If there's anything I can do to get you more data on it, let me know. I'd be happy to help.

Have you downloaded or published any files?
If so, the daemon is serving them to other clients over the network which will use your upload bandwidth.

Yes, I have published and downloaded.  That makes sense, but I never expected LBRY to be that bandwidth intensive yet...is there a way to limit the upload speed?  I can't check because I can only connect remotely at the moment, and LBRY uses all my upload...

There is an ability to limit the rates on the Settings page of the app.
On Tue, Apr 25, 2017 at 1:17 PM, scarint<EMAIL_ADDRESS>wrote:

Yes, I have published and downloaded. That makes sense, but I never
expected LBRY to be that bandwidth intensive yet...is there a way to limit
the upload speed? I can't check because I can only connect remotely at the
moment, and LBRY uses all my upload...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry/issues/622#issuecomment-297102288, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgZVmxh3yDMh35OJuPxZAf-_a5uHI7bks5rziq9gaJpZM4NHjD3
.

--
Jeremy Kauffman, Founder, LBRY http://lbry.io/
(267) 210-4292
Build LBRY: get https://lbry.io/get, follow https://twitter.com/lbryio,
like https://facebook.com/lbryio

(If you're using the daemon directly, I believe the command is settings_set)
On Tue, Apr 25, 2017 at 2:11 PM, Jeremy Kauffman<EMAIL_ADDRESS>wrote:

There is an ability to limit the rates on the Settings page of the app.
On Tue, Apr 25, 2017 at 1:17 PM, scarint<EMAIL_ADDRESS>wrote:

Yes, I have published and downloaded. That makes sense, but I never
expected LBRY to be that bandwidth intensive yet...is there a way to limit
the upload speed? I can't check because I can only connect remotely at the
moment, and LBRY uses all my upload...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry/issues/622#issuecomment-297102288, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgZVmxh3yDMh35OJuPxZAf-_a5uHI7bks5rziq9gaJpZM4NHjD3
.

--
Jeremy Kauffman, Founder, LBRY http://lbry.io/
(267) 210-4292
Build LBRY: get https://lbry.io/get, follow https://twitter.com/lbryio
, like https://facebook.com/lbryio

--
Jeremy Kauffman, Founder, LBRY http://lbry.io/
(267) 210-4292
Build LBRY: get https://lbry.io/get, follow https://twitter.com/lbryio,
like https://facebook.com/lbryio

Changing the upload setting to 0.5Mb/s or 0.25Mb/s appears to have no effect; LBRY still pushes bandwidth for all its worth.
I can throw up some QOS/prioritization on my firewall; is there a set of ports I could limit?

@jackrobison knows about this and is actively working on it. We'll have a fix out soon.

@jackrobison knows about this and is actively working on it. We'll have a fix out soon.

Sounds good; I'll keep running it when I can and try to break it some more.

@jackrobison any updates on this?

@jackrobison still waiting for an update

Hey all.  I think I'm having this issue too.  Just stuck in a screen which says "Network loading...catching up with the blockchain..."
Any solution to this yet?
Running on mac os 10.12.5
Thanks

@trumpomax can you send an email to<EMAIL_ADDRESS>with your log file please https://lbry.io/faq/how-to-find-lbry-log-file
It could be an issue coming from an older version.

Will do,
Thanks.
On Tue, 1 Aug 2017 at 19:17 Thomas Zarebczan<EMAIL_ADDRESS>wrote:

@trumpomax can you send an email to<EMAIL_ADDRESS>with your log file
please https://lbry.io/faq/how-to-find-lbry-log-file
It could be an issue coming from an older version.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry/issues/622#issuecomment-319548915, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGyD2gtKCPNIJ6YziwAnGD-zJON2qy2Iks5sT9wwgaJpZM4NHjD3
.

Jack says this is fixed

I have not been able to replicate the situation in the original post; either enough people have what I shared so it doesn't use my bandwidth, or it is fixed, I cannot confirm one way or another. But I can say that LBRY doesn't use nearly as much bandwidth as it used to.


Not working?

Python version:
2.7.12
Bandcamp-dl version:
0.0.6-01
Bancamp-dl options:
url:
options:
Describe the issue:
C:\Users\admin\Music\bandcamp>bandcamp_dl.py https://virginbabylonrecords.bandca
mp.com/album/laideronnette
Traceback (most recent call last):
File "C:\Users\admin\Music\bandcamp\bandcamp_dl.py", line 75, in
main()
File "C:\Users\admin\Music\bandcamp\bandcamp_dl.py", line 63, in main
album = bandcamp.parse(url)
File "C:\Users\admin\Music\bandcamp\bandcamp.py", line 9, in parse
r = requests.get(url)
File "C:\Python27\lib\site-packages\requests\api.py", line 70, in get
return request('get', url, params=params, **kwargs)
File "C:\Python27\lib\site-packages\requests\api.py", line 56, in request
return session.request(method=method, url=url, **kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 488, in reques
t
resp = self.send(prep, **send_kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 609, in send
r = adapter.send(request, **kwargs)
File "C:\Python27\lib\site-packages\requests\adapters.py", line 485, in send
raise ProxyError(e, request=request)
requests.exceptions.ProxyError: HTTPSConnectionPool(host='virginbabylonrecords.b
andcamp.com', port=443): Max retries exceeded with url: /album/laideronnette (Ca
used by ProxyError('Cannot connect to proxy.', NewConnectionError('<requests.pac
kages.urllib3.connection.VerifiedHTTPSConnection object at 0x027D5510>: Failed t
o establish a new connection: [Errno 11004] getaddrinfo failed',)))

I've turned off my own proxy and i get the same error.
Any other ideas?

Duplicate Issue


TLS "hostname doesn't match exception"

Hello, and thank you for python-etcd.  When I run this code
import etcd

if __name__ == "__main__":
    c = etcd.Client(host="<IP_ADDRESS>", port=2379, protocol="https", ca_cert="cacert.pem")
    c.api_execute("/version", 'GET').data

against a server bearing this certificate
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            7f:ff:bc:cf:b7:c6:dc:a3:3e:4c:76:79:c8:6d:56:3c:54:23:ee:a0
    Signature Algorithm: sha256WithRSAEncryption
        Issuer: C=US, O=Xoom, OU=INF ENG, L=San Francisco, ST=California, CN=Xoom INF ENG CA
        Validity
            Not Before: Jul  6 00:28:00 2016 GMT
            Not After : Jul  4 00:28:00 2026 GMT
        Subject: C=US, ST=California, L=San Francicso, O=INF ENG, OU=etcd cluster, CN=etcd
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                Public-Key: (2048 bit)
                Modulus:
                    00:df:94:2b:6f:4d:f2:e0:97:2d:55:18:f7:a0:10:
                    90:6d:f9:3f:35:03:2f:f1:f8:b8:ad:5e:96:64:f3:
                    f3:1e:30:e1:12:27:72:c6:79:11:b9:68:44:35:19:
                    f0:db:f9:f6:15:a9:eb:92:0c:44:cb:c7:fd:19:dd:
                    a2:c5:9c:10:77:81:e3:54:ea:6f:7d:86:ff:f4:86:
                    b0:13:cb:7d:9f:78:7e:28:6f:3f:45:ca:4d:bb:90:
                    20:00:db:8d:4e:8c:6c:33:de:0f:48:82:47:ec:0d:
                    9b:e9:88:45:3f:c0:2d:86:6a:9d:c3:62:fa:7f:d8:
                    c4:35:5b:36:84:6c:c3:a2:bb:4a:8a:76:cc:6d:f6:
                    92:1a:f6:d1:0a:58:f1:49:14:05:5f:63:78:6f:87:
                    94:20:e8:03:f5:9a:9a:e1:5c:dd:0e:34:98:15:1c:
                    c5:af:24:9e:d1:49:2f:40:72:ce:eb:84:b3:3f:bc:
                    38:3e:94:ee:06:88:78:5b:9a:c3:8c:95:74:66:e6:
                    14:ab:ce:89:7b:c7:cf:ff:3d:47:0b:5b:ad:88:a0:
                    23:78:9d:94:8a:4e:cc:c6:4e:c4:46:d4:de:52:d7:
                    83:5d:07:4e:ea:14:d2:9a:02:c9:d9:24:87:9d:1a:
                    91:2b:89:20:ab:a7:6f:58:61:1e:b6:26:5a:b6:eb:
                    60:77
                Exponent: 65537 (0x10001)
        X509v3 extensions:
            X509v3 Key Usage: critical
                Digital Signature, Key Encipherment
            X509v3 Extended Key Usage:
                TLS Web Server Authentication
            X509v3 Basic Constraints: critical
                CA:FALSE
            X509v3 Subject Key Identifier:
                FD:30:A8:DF:EB:6A:67:30:63:C6:69:65:D0:65:48:85:94:7E:09:D0
            X509v3 Authority Key Identifier:
                keyid:84:1C:C5:89:42:DC:E2:FF:96:15:65:28:D2:A2:17:BD:D1:E6:D5:0C

            X509v3 Subject Alternative Name:
                DNS:localhost, IP Address:<IP_ADDRESS>, IP Address:<IP_ADDRESS>, IP Address:<IP_ADDRESS>, IP Address:<IP_ADDRESS>
    Signature Algorithm: sha256WithRSAEncryption
         75:97:75:40:5d:5f:8b:41:82:ac:94:29:b6:0e:b8:7a:22:43:
         21:2b:a3:20:e4:56:0e:94:5a:b0:2a:76:81:9f:ee:10:fc:b7:
         9d:f0:91:2a:a2:ef:61:65:45:4f:9e:22:3c:9d:00:81:49:bb:
         86:5c:3a:5c:8d:68:25:9a:f4:2a:4c:e9:f8:4a:b7:75:b6:49:
         dc:69:37:d0:72:99:bc:df:b3:18:be:13:39:7b:0f:88:23:70:
         1a:30:8f:30:78:a2:a9:1e:b6:a3:15:12:af:ff:78:e1:b6:18:
         32:a9:99:de:ae:c8:c6:9c:49:c1:1d:0e:4b:33:ef:b6:1d:e6:
         71:52:c0:8a:0a:57:2e:c1:ec:43:18:2a:d5:ee:8f:32:5e:01:
         5c:4b:74:0b:4a:05:96:32:d0:80:cc:50:20:af:8a:b4:69:bc:
         39:8c:6e:c1:6a:28:71:be:ba:12:e9:3d:5a:fc:8c:65:0b:0f:
         a8:ba:0c:9f:52:ab:a4:25:79:92:ab:bb:de:24:ee:b8:1f:a1:
         d9:88:60:bd:26:6d:80:10:db:5c:e9:4a:c1:2e:03:5b:f4:28:
         3a:58:a5:fd:14:ed:34:b5:82:8e:4d:54:24:17:d3:af:3b:54:
         a3:cd:09:ba:e4:be:09:21:ba:9d:6b:0d:a7:b9:73:3f:cf:42:
         bc:21:11:03

I receive
$ python main.py
Traceback (most recent call last):
  File "main.py", line 5, in <module>
    c.api_execute("/version", 'GET').data
  File "/Library/Python/2.7/site-packages/python_etcd-0.4.3-py2.7.egg/etcd/client.py", line 857, in wrapper
    cause=e
etcd.EtcdConnectionFailed: Connection to etcd failed due to SSLError(CertificateError("hostname '<IP_ADDRESS>' doesn't match 'localhost'",),)

I am using python 2.7.10.  It appears to me that client or its dependencies is not availing itself of the Subject Alternative Names in the cert, but I could be wrong.  Do I need to install additional python packages in support of python-etcd to avoid this error?
Thank you.

I bet this is the problem:
https://github.com/shazow/urllib3/issues/258

Should be solved with my patch to use a more modern urllib3


C# Unit Testing Tutorial for Beginners

C# Unit Testing Tutorial for Beginners
Unit Testing C# Code - Tutorial for Beginners.
Link: https://www.youtube.com/watch?v=HYrXogLj7vg
Milestones

[x] What is Automated Testing
[x] Benefits of Automated Testing
[x] Types of Tests
[x] Test Pyramid
[x] The Tooling
[x] Source Code
[x] Writing Your First Unit Test
[x] Testing All the Execution Paths
[x] Refactoring with Confidence
[x] Using NUnit in Visual Studio
[x] What is Test-Driven Development
[x] Course Structure
[x] Summary

This will close #95.


libttng-ust: create compatibility symlink for powershell

Motivation for this change
powershell expects a .so.0 library file.
Cc: @alyssais
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[X] Fits CONTRIBUTING.md.

Presumably the library number was bumped for a reason i.e. backwards
incompatibility?  Is upstream working on this?

Presumably the library number was bumped for a reason i.e. backwards
incompatibility?

One would think so, but powershell at least works fine with this hack in place.

Can this be fixed in powershell instead?

@ofborg eval

Can this be fixed in powershell instead?

Yes
diff --git a/pkgs/shells/powershell/default.nix b/pkgs/shells/powershell/default.nix
index 5c082641e9f24..15ca1b2d910cb 100644
--- a/pkgs/shells/powershell/default.nix
+++ b/pkgs/shells/powershell/default.nix
@@ -15,7 +15,7 @@ let archString = if stdenv.isAarch64 then "arm64"
                      else if stdenv.isLinux then "LD_LIBRARY_PATH"
                      else throw "unsupported platform";
                      libraries = [ libunwind libuuid icu curl openssl_1_1 ] ++
-                       (if stdenv.isLinux then [ pam lttng-ust ] else [ darwin.Libsystem ]);
+                       (if stdenv.isLinux then [ pam ] else [ darwin.Libsystem ]);
 in
 stdenv.mkDerivation rec {
   pname = "powershell";
@@ -34,6 +34,9 @@ stdenv.mkDerivation rec {
   let
     ext = stdenv.hostPlatform.extensions.sharedLibrary;
   in ''
+    mkdir -p $out/lib
+    ln -s ${lttng-ust}/lib/liblttng-ust.so.1 $out/lib/liblttng-ust.so.0
+
     pslibs=$out/share/powershell
     mkdir -p $pslibs

@@ -69,7 +72,7 @@ stdenv.mkDerivation rec {
   meta = with lib; {
     description = "Powerful cross-platform (Windows, Linux, and macOS) shell and scripting language based on .NET";
     homepage = "https://github.com/PowerShell/PowerShell";
-    maintainers = with maintainers; [ yrashk srgom ];
+    maintainers = with maintainers; [ yrashk srgom p3psi ];
     platforms = [ "x86_64-darwin" "x86_64-linux" "aarch64-linux"];
     license = with licenses; [ mit ];
   };

https://github.com/p3psi-boo/nixpkgs/commit/549311a6d19ea2e0a0572169f7637d93673467eb

https://github.com/NixOS/nixpkgs/pull/138299#event-5376620766


No-op simplest example requires Output wrapper

Currently the minimal no-op program looks like this:
@main def main(): Unit = Pulumi.run {
  Output {
    Pulumi.exports(
      name = "name"
    )
  }
}

A theoretical minimum is:
@main def main(): Unit = Pulumi.run {
  Pulumi.exports(
    name = "name"
  )
}

The team decided to not change this due to no practical use case and to avoid implementation complexity


It would be helpful to specify access requirements

I have being following this repo for a few days and checked out some of the books referenced. Some links point to files stored behind subscription barriers (i.e. you must join a list a cop junk mail to read the book). These types of books are rarely 'free' per se. Maybe submitters should flag if a link points to a subscription site or has particular license constraints.

yea i know what you mean, i try to avoid merging these in. @vhf can weigh in about these too.

:+1: for this issue! I think it is important to note when accounts or other requirements are necessary to obtain a book/course. It isn't truly free if you need to provide something on your end. I think we can assume account logins for sites like Coursera, because how else are you to track your progress?
@SimonChristopherCropper I would suggest opening a PR noting which links require logins, etc. We should amend (add info) to the list, at the very least, before removing helpful references.

I agree with @onebree !

@onebree @vhf Is it preferable to separate such links into their own category, or just add a note?
As my focus on documentation is with readability, my concern with notes (granting there might be numerous problem links present) is we'll bloat the existing document vertically, while adding no real value. That is, people will inherently gravitate towards easy access and truly "free" content over paywalls or the like, so it seems more apt to separate these into their own Subscription-based category within the documents.

As I said before, we should change as little of the current links as possible, until we research further. I think an "account required" category would be nice -- the word "subscription" makes me think that money is involved (and not free).
I could create a PR in a couple of days to add this section to all the English docs...

@onebree I lean towards your word choice here. "Account required" or "Signup required" is definitely less suggestive than "Subscription."

Agreed. If you have some free time, I'd gladly accept a PR annotating such links with *(account required)*

@vhf I am unable to submit a PR at this time. Hopefully another person can!

No problem @onebree! You helped this repo a lot already, and it is/was much appreciated. You certainly noticed I closed a bunch of issues today. This one is still standing though, and I won't close it until it's fixed, by any contributor or by me. :)

#1746 made me realise I have not been clear about what I would prefer regarding this issue.
I suggest annotating links with (account required) when it applies, instead of creating a new category "Account Required" and duplicating sub-categories.

No @vhf, my bad!!! I was in a hurry and misunderstood the requirements! Sorry

@vhf  I think this issue should remain open until we put somewhere (maybe CONTRIBUTING.md?) these guidelines.

Oh I thought it was already in CONTRIBUTING.md. I'll add it.

Thank you. What are your thoughts on a table format? The raw markdown may not look as pretty, and added new columns will be a pain... I am not sure how this could be done elegantly, but just an idea.

Would something like www.syncfusion.com be against this then? It requires signing up for an account before access to the book is given.

@johnaoss it's not really against this. I think although syncfusion doesn't require an actual account, we should still annotate all syncfusion links with account required. I'm open to suggestions on this matter.

@onebree I suspect that's something for another issue.


⚠ Tukarcoid has degraded performance

In 74859e1, Tukarcoid (https://tukar.co.id) experienced degraded performance:

HTTP code: 200
Response time: 2637 ms

Resolved: Tukarcoid performance has improved in 4c90428 after 7 minutes.


[Bug] Overflowing section in liked songs playlist

Do you have the latest versions of Spicetify, Spotify and Better Bloom?

[X] I'm using the latest versions of Spicetify, Spotify and Better Bloom

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Is your problem definitely caused by the theme?

[X] I have checked the app behavior without the theme and it is different

Environment & Computer Info
Spotify for Windows (64 bit)
<IP_ADDRESS>0.ga4a70a0e
Spicetify v2.36.11
Theme: Better-Bloom / dark
Extensions: theme.js
Custom apps: marketplace

Description

Go to liked songs playlist
See the overflowing area where the play button and tags rest; right below the album cover

Screenshots

sorry for the issue caused i will fix it as soon as possible

updated the theme !.
as am not able to reproduce the issue i tried a fix for it hope it helps
checkout the theme now is the issue fixed ?

Works great now! Thanks.

Awesome ! thanks for using the theme

commit 19f9ba4fd7b33fee7f3d452befbf758444b146e5 breaks this fix

ohh soory for the issue caused i did it by mistake


generate osgi bundle - branch 1.7.x

There is a problem with you PR (look at the changes and number of commits). You should not create it from your master, but from your own 1.7.x branch with the appropriate changes...


purge dead/obsolete/merged branches from GitHub

removed all branches that were merged or closed PRs.
There were some other branches here that did not have a PR assigned. For each of those, I checked what was inside and saw that they had been part of a different PR already and generally implemented in the code already.


feat(security): disable access for anonymous users, when disablePublicMemos is true

Added check system.disablePublicMemos for non authorized users.
If user is anonymous and disablePublicMemos is on, then redirect to auth page
hide link to explore on sidebar for non authorized users.

Two design points:

The disablePublicMemos is mainly used for new public memos created by users. For those existing old public memos, they will still be displayed on the explore page.
The explore page should always be visible, whether logged in or not, just like someone's blog.

So is there some misunderstanding here, or do you have some other thoughts?

Two design points:

The disablePublicMemos is mainly used for new public memos created by users. For those existing old public memos, they will still be displayed on the explore page.
The explore page should always be visible, whether logged in or not, just like someone's blog.

So is there some misunderstanding here, or do you have some other thoughts?

I think it should be possible to disable public access to all notes with one button. In my opinion, the disablePublicMemos setting can be used just for this. I make disablePublicMemos = true and all public memos become inaccessible to anonymous users. You need to be logged in to view them. At the same time, the visibility state in the notes themselves does not need to be changed.
There are only edits for the frontend, but in the future it will be necessary to finalize the backend. I also think that using id access to users and notes is a bad idea. You can sort through all the notes through brute force. From a security point of view, it is now considered good practice to use uuid identifiers.

I think it should be possible to disable public access to all notes with one button. In my opinion, the disablePublicMemos setting can be used just for this. I make disablePublicMemos = true and all public memos become inaccessible to anonymous users. You need to be logged in to view them. At the same time, the visibility state in the notes themselves does not need to be changed.
There are only edits for the frontend, but in the future it will be necessary to finalize the backend. I also think that using id access to users and notes is a bad idea. You can sort through all the notes through brute force. From a security point of view, it is now considered good practice to use uuid identifiers.

For the uid problem, we can use username instead of id for users in the routers. And add a uuid field to the memo model to replace the id in the router. In the backend, we should still keep using the id. WDYT?
Related issue: https://github.com/usememos/memos/issues/1916

I think it should be possible to disable public access to all notes with one button. In my opinion, the disablePublicMemos setting can be used just for this. I make disablePublicMemos = true and all public memos become inaccessible to anonymous users. You need to be logged in to view them. At the same time, the visibility state in the notes themselves does not need to be changed.
There are only edits for the frontend, but in the future it will be necessary to finalize the backend. I also think that using id access to users and notes is a bad idea. You can sort through all the notes through brute force. From a security point of view, it is now considered good practice to use uuid identifiers.

For the uid problem, we can use username instead of id for users in the routers. And add a uuid field to the memo model to replace the id in the router. In the backend, we should still keep using the id. WDYT?
Related issue: #1916

Yes, I saw issue about username. I agree that username can be used. I haven't studied the code in detail, so I don't know if there is currently a uniqueness limit for username.
If the username is already unique, then there will be a minimum of problems with substitution in the url.
For memos and resources, you can generate a uuid based on id + salt (username or user id). This will greatly complicate brute force.
But all these changes are best done iteratively, with different commits/pull requests. Not in this PR. Many small PRs are better than one big one.


[vimeo] playlist download fails with ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError())

Verbose log
$ youtube-dl --verbose --list-formats https://vimeo.com/392712342
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'--list-formats', u'https://vimeo.com/392712342']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.03.01
[debug] Python version 2.7.6 (CPython) - Linux-3.13.0-85-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg 3.3.3, ffprobe 3.3.3, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[vimeo] 392712342: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2238, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 410, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 523, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 448, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

youtube-dl --verbose --list-formats https://vimeo.com/392712342
[debug] System config: []
[debug] User config: ['-f', 'best[height<=768]/bestvideo[height<=768]+bestaudio/best', '--hls-prefer-ffmpeg', '--external-downloader', 'curl', '--sub-format', 'ass/tt/ttml/dfxp/scc/vtt/srt/sami/best']
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--list-formats', 'https://vimeo.com/392712342']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.03.06
[debug] Python version 3.8.2 (CPython) - Linux-5.5.7-arch1-1-x86_64-with-glibc2.2.5
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2, rtmpdump 2.4
[debug] Proxy map: {}
[vimeo] 392712342: Downloading webpage
[vimeo] 392712342: Extracting information
[vimeo] 392712342: Downloading JSON metadata
[vimeo] 392712342: Downloading JSON metadata
[vimeo] 392712342: Downloading akfire_interconnect_quic m3u8 information
[vimeo] 392712342: Downloading fastly_skyfire m3u8 information
[vimeo] 392712342: Downloading akfire_interconnect_quic MPD information
[vimeo] 392712342: Downloading fastly_skyfire MPD information
[info] Available formats for 392712342:
format code                                      extension  resolution note
dash-akfire_interconnect_quic-audio-07e642ac     m4a        audio only DASH audio   64k , m4a_dash container, mp4a.40.2 (24000Hz)
dash-fastly_skyfire-audio-07e642ac               m4a        audio only DASH audio   64k , m4a_dash container, mp4a.40.2 (24000Hz)
dash-akfire_interconnect_quic-audio-39c4bd40     m4a        audio only DASH audio  128k , m4a_dash container, mp4a.40.2 (48000Hz)
dash-fastly_skyfire-audio-39c4bd40               m4a        audio only DASH audio  128k , m4a_dash container, mp4a.40.2 (48000Hz)
dash-akfire_interconnect_quic-audio-05b2760f     m4a        audio only DASH audio  255k , m4a_dash container, mp4a.40.2 (48000Hz)
dash-fastly_skyfire-audio-05b2760f               m4a        audio only DASH audio  255k , m4a_dash container, mp4a.40.2 (48000Hz)
dash-akfire_interconnect_quic-video-07e642ac     mp4        426x180    DASH video  342k , mp4_dash container, avc1.640015, video only
dash-fastly_skyfire-video-07e642ac               mp4        426x180    DASH video  342k , mp4_dash container, avc1.640015, video only
dash-akfire_interconnect_quic-video-39c4bd40     mp4        640x270    DASH video  568k , mp4_dash container, avc1.64001E, video only
dash-fastly_skyfire-video-39c4bd40               mp4        640x270    DASH video  568k , mp4_dash container, avc1.64001E, video only
dash-akfire_interconnect_quic-video-05b2760f     mp4        720x304    DASH video 1457k , mp4_dash container, avc1.64001F, video only
dash-fastly_skyfire-video-05b2760f               mp4        720x304    DASH video 1457k , mp4_dash container, avc1.64001F, video only
hls-akfire_interconnect_quic-audio-high-audio    mp4        audio only
hls-akfire_interconnect_quic-audio-medium-audio  mp4        audio only
hls-fastly_skyfire-audio-high-audio              mp4        audio only
hls-fastly_skyfire-audio-medium-audio            mp4        audio only
http-240p                                        mp4        426x180    23fps
hls-akfire_interconnect_quic-445                 mp4        426x180     445k , avc1.640015, 23.976fps, video only
hls-fastly_skyfire-445                           mp4        426x180     445k , avc1.640015, 23.976fps, video only
http-360p-0                                      mp4        640x270    23fps
hls-akfire_interconnect_quic-665                 mp4        640x270     665k , avc1.64001E, 23.976fps, video only
hls-fastly_skyfire-665                           mp4        640x270     665k , avc1.64001E, 23.976fps, video only
http-360p-1                                      mp4        720x304    23fps
hls-akfire_interconnect_quic-1060                mp4        720x304    1060k , avc1.64001F, 23.976fps, video only
hls-fastly_skyfire-1060                          mp4        720x304    1060k , avc1.64001F, 23.976fps, video only (best)

Getting the same error using youtube-dl with vimeo
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.03.24
[debug] Python version 2.7.17 (CPython) - Linux-5.3.0-40-generic-x86_64-with-Ubuntu-19.10-eoan
[debug] exe versions: ffmpeg 4.1.4-1build2, ffprobe 4.1.4-1build2, phantomjs ., rtmpdump 2.4
[debug] Proxy map: {}
[vimeo] 267040577: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2238, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 467, in error
result = self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 654, in http_error_302
return self.parent.open(new, timeout=req.timeout)
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 473, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

For me the same situation was caused by country-based access blocking on vimeo side. Proxy is the solution.


Adds not connected icon badge

adds not connected icon

The changes themselves look fine, but I didn't manage to see the badge when using the extension. What conditions do I need to see it on the extension?

cut off internet connection and it should happen when it cannot reach the RPC anymore

Tested in Chrome build fine 👍


Args in json format

have to alter the argID and isDependentOn fields of the JSON object. As for now these two fields contain garbage.

Fixes #27


Update ScreenWrap.json

Version 0.2.0

Added a new behavior for physics objects (Thanks Entropy!)
Updated descriptions

Project files
ScreenWrapWithPhysics_Example.zip
Video
https://user-images.githubusercontent.com/8879811/151092622-8ac56153-3efc-44c1-9531-98c8cfd8165d.mp4

Should we update this extension to have an option to use AABB?  If so, how should it be expose to the user?

New string property called "TriggerSource", with options "Center of AABB bounding box (default)", "Center point of sprite", "Origin point of sprite"

OR

Boolean property called "UseCenterPoint"

OR

Some other method?

Should we update this extension to have an option to use AABB? If so, how should it be expose to the user?

Based on the center is faster and makes more sense to me than the AABB.

FYI - Sorry for the confusion.  This change is complete and ready for review.  My questions were about future improvements.

AngularVelocity should be hidden
Why preEvents is not used in both?
It needs a comment to explain why velocity need to be saved.
The 2 behaviors are not small and there is a lot of duplicated code. This would probably be a nightmare to maintain.
What about creating a new behavior to share all this code?

ManualScreenWrap will give:

4 conditions to check if the object must be moved
4 actions to move it on a screen border
it will handle all the configuration

ScreenWrap and ScreenWrap will require ManualScreenWrap and only contains a doStepPreEvent.

It's not an ideal solution and it's a breaking change but it should avoid headache.

Should we:

accept this PR knowing that works but has a lot of duplicated code
people can start benefiting from using it today

Or:

wait for someone to rewrite it to be more maintainable

I think we should accept this as-is, and we can always improve it later.

I guess I could always resubmit it as a community extension?  Would that be the preferred option?

Actually there is a lot of functions but not as much events as I though. I'm fine to keep it this way. Sorry for the trouble.
But this should still be addressed:

The AngularVelocity property should be hidden
It needs a comment to explain why velocity need to be saved.

@tristanbob There is only to add in 'doStepPreEvents' a comment for why the velocity needs to be saved.
If you know, you can tell me here, and I'll do the changes, then merge this PR.

I fixed the conflicts and moved this extension into the reviewed folder.
@tristanbob There is only to add in 'doStepPreEvents' (for physics behavior) a comment for why the velocity needs to be saved. If you know, you can tell me here, and I'll do the changes, then merge this PR.

It's because Physics2 reset object velocities when an object is moved from the outside of the extension.

Thanks for the detail.
I added the comment and tested the extension.
It can be merged.


yq-go: 4.13.0 -> 4.13.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/yq-go/versions.
meta.description for yq-go is: "Portable command-line YAML processor"
meta.homepage for yq-go is: "https://mikefarah.gitbook.io/yq/"
meta.changelog for yq-go is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2/bin/yq passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 4.13.2 with grep in /nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2

found 4.13.2 in filename of file in /nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2

directory tree listing: https://gist.github.com/db87f983eed5284da1e69e363437aab7

du listing: https://gist.github.com/a16c901039599bb05aca47fecb101587

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
yq-go

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A yq-go https://github.com/r-ryantm/nixpkgs/archive/5d3f726f0bab0fc367b30f6183a80542bdc5bd84.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2
ls -la /nix/store/9xngnb6wvzv5cvprfgg9hmw5xpqd0564-yq-go-4.13.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

yq-go

Maintainer pings
cc @nlewo @kalbasit for testing.

Result of nixpkgs-review pr 138861 run on x86_64-linux 1

1 package built:

yq-go


Questions for the chapter4 Istanbul data

Hi,
I am studying chapter4 and have two questions.

The meaning of levels_to traverse

It looks like the same operation is repeated for the number of levels_to_traverse in the code.
There seems to be no explanation about the traverse.
Would you tell me why we need to specify levels_to_traverse ?

Mistakes in the code?

The line below are needed? It seems that sum_points are not used in other parts.

https://github.com/dipanjanS/learning-social-media-analytics-with-r/blob/master/Chapter 4 - Analyzing Foursquare/Code/B06056_04_02_code.R#L126

At this line, we should specify file_level1 instead of file_level2?

https://github.com/dipanjanS/learning-social-media-analytics-with-r/blob/master/Chapter 4 - Analyzing Foursquare/Code/B06056_04_02_code.R#L129
Thanks.

Hi Dichika,

levels_to_traverse is used to determine how many hops from the city center you want to traverse to collect data. If you do more hops you will cover more geographical area and hence more data. In the code we are looking for new centers in each hop and looking for data around those centers.

a. You are right sum_points is not used in other part. It was sort of a diagnostic variable at development time to keep track of how many points were generated as part of data collection process.
b. This one is a typo and the file should be file_level1 as you pointed out. I will fix these mistakes with a code push.

Thank you for your reply.
As for Q1, I understand the meaning of levels_to_traverse.
Based on your answer, it is expected that the center is updated according to the levels_to_traverse.
However, the information of the levels_to_traverse (that is j) does not seem to be used in the code.
The line below reflects your intension?
https://github.com/dipanjanS/learning-social-media-analytics-with-r/blob/master/Chapter 4 - Analyzing Foursquare/Code/B06056_04_02_code.R#L119

It is not being directly but it governs the number of times the loop is run. Each time the loop is run we will generate new centers based on centers existing in the list. This if repeated 1 times gives you one hop from the center, if repeated 2 times you will get centers which are a result of spanning an arc from each of these centers so on and so forth. The variable level_to_traverse is used to control how many times you want to repeat the process.

Thank you for your concise reply.
I realized the meaning of the repeat with reading your explanation and executing the code.


feat(ecs-patterns): add targetProps to LoadBalancedServiceBase

This can be used to specify a different health check path for the
load balancer.
fixes #2359

Pull Request Checklist

[ ] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[x] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[x] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Closing this because it turns out it's easier/better to just define FargateService and ApplicationTargetGroup by hand than use LoadBalancedFargateService


cli: change docusaurus start/build to protosaurus start/build

This PR migrates the docusaurus start and docusaurus build to protosaurus start or build. Behind it, it will generate the required JSON/MDX files (if necessary).
protosaurus build or protosaurus start can receive 1 optional argument: the path to directory containing buf.gen.yaml. By default it's resolved to its parent directory.
Signed-off-by: Try Ajitiono<EMAIL_ADDRESS>

Thanks @dio for the review!


Require stack & Yarn PnP errors surfaced

Summary

Add a require stack printed relative to <rootDir> for convenience and linking when a MODULE_NOT_FOUND error is thrown by the resolver.
Throw errors thrown by Yarn PnP as-is. Jest was swallowing them and failing later attempting to resolve the module.

Solves #9594
Test plan
E2E tests included in pnp and resolve

It should be all good now.
I've fixed another bug, when there is some kind of nesting of required modules (index.js > module1.js > module2.js). The "However Jest was able to find moduleName" error wiill always show up since the error will bubble up from module2 to module1 and when it looks for a sibling module with similar file extension, it will find the module itself (module1.js) and show the hint for a file that actually exists and is there.
I've fixed it by adding a moduleName property on ModuleNotFoundError to hold a reference to the actual module that could not be found, so when the runtime attempts to find a sibling file with a different extension looks only for a file that was not found. The test should throw module not found error if the module has dependencies that cannot be found at e2e/resolve/__tests__/resolve.test.js covers this scenario.
There should not be any breaking changes AFAIK. In case a ModuleNotFoundError is thrown by a resolver that is not the default one the runtime will still be able tu duck type it, and use the previous behaviour in case the resolver did not specify the module that failed to require (moduleName property).

Codecov Report

Merging #9681 into master will decrease coverage by 0.16%.
The diff coverage is 26.00%.

@@            Coverage Diff             @@
##           master    #9681      +/-   ##
==========================================
- Coverage   65.09%   64.92%   -0.17%
==========================================
  Files         287      288       +1
  Lines       12144    12184      +40
  Branches     3007     3020      +13
==========================================
+ Hits         7905     7911       +6
- Misses       3604     3635      +31
- Partials      635      638       +3

Impacted Files
Coverage Δ

packages/jest-resolve/src/ModuleNotFoundError.ts
0.00% <0.00%> (ø)

packages/jest-resolve/src/index.ts
44.72% <10.00%> (-3.00%)
:arrow_down:

packages/jest-runtime/src/index.ts
64.75% <45.83%> (-1.00%)
:arrow_down:

e2e/pnp/undeclared-dependency/index.js
50.00% <50.00%> (ø)

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 4a59daa...5a805e5. Read the comment docs.


SRC-14 Simple Proxy Standard

Type of change
New Standard
Changes
The following changes have been made:

Add the SRC-14 standard specification, abi and examples

Related Issues
https://github.com/FuelLabs/sway-rfcs/pull/39

Nice work, want a little bit more time to mull on the proposal but from my initial passes the syntax looks well thought out.

Dependent on https://github.com/FuelLabs/sway-standards/pull/95

Are we missing a section about auto generating the proxy contract byte code? I'm not 100% clear how that part is going to be implemented.


Use on virtual machines?

Has anyone been successful using bluepill on virtual machines? Building the app once (build-for-testing) and then sharding out the test execution across multiple VMs?

I don't see any technical limitation not being able to do that. The only limitation for this is - Apple license only allows 2 macOS VM instances in one single hardware.

Cool. Thanks for the response.


Not all results obtained with SPARSE

Set up a new key with set two points:
DROP TEST
SET test 1 OBJECT '{"type":"Point","coordinates":[-122.44502,37.79191]}'
SET test 2 OBJECT '{"type":"Point","coordinates":[-122.44484,37.79193]}'

The NEARBY works correctly with either limit or radius:
NEARBY test DISTANCE POINT 37.79191 -122.44502 20
{
  "objects":[
    {"id":"1","object":{"type":"Point","coordinates":[-122.44502,37.79191]}},
    {"id":"2","object":{"type":"Point","coordinates":-122.44484,37.79193]},"distance":15.97}
  ],
  "count":2

But with SPARSE it only returns one result, the center of the NEARBY search:
NEARBY test SPARSE 8 DISTANCE POINT 37.79191 -122.44502 20
{
  "objects":[
    {"id":"1","object":{"type":"Point","coordinates":-122.44502,37.79191]}}
  ],
  "count":1
}

Similar thing happens with WITHIN and INTERSECT:
INTERSECTS test SPARSE 8 BOUNDS 37.79190 -122.44502 37.79194 -122.44483
{
  "objects":[
      {"id":"1","object":{"type":"Point","coordinates":[-122.44502,37.79191]}}
  ],
  "count":1
}

We actually see this issue in different context, with WHERE and SPARSE combined.  The WITHIN returns 90 results using LIMIT and WHERE, but it returns 0 results using SPARSE and WHERE.
Is SPARSE also applied before the WHERE clause?  If so, would it be possible to revert it and apply WHERE after SPARSE?

SPARSE is unique and I can see where it might need some special care. I'll look into this further and keep you posted.

Hi @tidwall, any news on this issue?

Hello @tidwall, any updates? Facing the same issue. Thank you.

https://github.com/tidwall/tile38/issues/103#issuecomment-268324070
This is an old comment and I haven't looked at the code but maybe the code can loop through all the divisions, picking only one item each time until the sparse limit is reached. That way if there is already a small dataset, all data points are essentially not sparsed


shim away RTCSessionDescription/RTCIcecandidate

basically #266 but separate so it doesn't collide with #288
This is still required in Chrome 51, even though not if promises are used. See here
cc @jan-ivar, sorry for breaking this

LGTM. One thing though, we should have test that catch this. I will start looking at separating the tests into separate files per API since it's way too big now.


Feature: Create/Remove custom delegate Target Roles

What is the feature about?
The user could have the flexibility to create delegated roles from the Targets Role.
The delegated role would require:

Role name (not allowed top-level roles and BINS)
path to make it unique, for example project-name/*/*

the top-level * is reserved for current RSTUF BINS succinct delegate role

Optional succinct delegated roles and the size

The name would be automatically the Role name followed by -bins. For example: MyProjectRole will create also MyProjectRole-bins
It will automatically use the online key

This feature requires the following features as pre-requisite:

[x] https://github.com/repository-service-tuf/repository-service-tuf/issues/23
[ ] https://github.com/repository-service-tuf/repository-service-tuf/issues/327

Services it relates to
repository-service-for-tuf-api, repository-service-for-tuf-worker
Related tasks
No response
References
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Last week we had a couple of meetings with RSTUF contributors where we discussed some general notes about this feature:

We will support only path prefixes.
All Custom Target Roles will use the same 1 online key as the rest of the online roles
a. Add documentation that custom target roles doesn’t support custom keys at the moment. - ISSUE
We are conservative: only artifacts with paths matching one of the prefixes of custom target roles are allowed
If the user wants to add an artifact with path not included in any path prefixes we say we cannot do this.
We need to validate that input of the custom target role name contains specific characters (letters and numbers for example) DO NOT ALLOW wildcards

TODO first iteration defining custom target roles only during initial bootstrap:

[ ] https://github.com/repository-service-tuf/repository-service-tuf-api/issues/517
[ ] https://github.com/repository-service-tuf/repository-service-tuf-worker/issues/438
[ ] https://github.com/repository-service-tuf/repository-service-tuf-cli/issues/491


Add ESM build

Description
Since about Angular 9 or 10 it has been reporting "CommonJS or AMD dependencies can cause optimization bailouts." It has been reporting this for this BugSnag dependency.
Describe the solution you'd like
Add an ESM (ES Module) build that we can use instead of the CommonJS one. I recently did this for one of our libraries that was also reporting the same problem and found it easy to follow along with this article https://blog.logrocket.com/publishing-node-modules-typescript-es-modules/.
Describe alternatives you've considered
Use a different system that supports ESM.

Hi @irahopkinson
This was previously raised here https://github.com/bugsnag/bugsnag-js/issues/1276 and there is a lot of previous discussion on this issue: https://github.com/bugsnag/bugsnag-js/issues/463#issuecomment-454693484
Despite the "CommonJS or AMD dependencies can cause optimization bailouts." warning you get in recent Angular versions, our analysis has shown that moving the Bugsnag library to ES Modules would not provide any size benefit from tree shaking.
We're therefore unlikely to add an ESM build in the near future due to the effort and maintenance overhead involved.
I'll keep this open as a feature request as it's something that we may revisit if any future changes to the library make this worthwhile.

I believe that because of the way Bugsnag-js is built and given to consumers, some optimizations are impossible on their side.
For example, I think the library is not tree shaked properly when @bugsnag/js is imported in a file without being used. (In a situation where the entire library should not be present in the output bundle)

I created a small repo to show the issue I've seen
https://github.com/tleunen/bugsnag-treeshaking-issue
The entry point is this file: https://github.com/tleunen/bugsnag-treeshaking-issue/blob/main/src/index.ts
You'll see that no bugsnag code is ever used. But bugsnag is still present in the output bundle. I built the project twice (with ncc / webpack 5) to get a minified version and an un-minified version of it. You'll see the build files in the dist folder (I pushed them on git to be visible easily as well)
https://github.com/tleunen/bugsnag-treeshaking-issue/tree/main/dist
Hope this can help @mattdyoung

We are also interested in this one. We are moving our Typescript Node project to the NodeNext module resolution and the type-checking fails because @bugsnag/js doesn't comply with Node's conventions.

Hi all,
We are continuing to monitor this thread and feed comments back to our Product Team, as we have an item on our backlog to publish the JS notifier as ES modules. While we don't currently have any further updates on this, we will make sure to keep this thread posted with any developments.


Collapsible Header With Tab View

Motivation
The UI of having a collapsible header above a Tab View is a commonly requested pattern (#411), but it is surprisingly challenging for newer RN devs to implement. Creating a wrapper for this functionality and some examples for using the wrapper will help allow them to have a baseline for how to add this pattern into their own apps.
Prior Examples
@satya164 Provided https://snack.expo.io/@satya164/collapsible-header-with-tabview as an example. It's a valuable start for understanding how to work on this, but the snack itself can't be used without significant additional work due to the additional of large amounts of padding above the content when switching tabs in certain cases #441.
This example appears to be complete, but it's a rather larger example that is challenging to quickly bring into a different projects that just wants the minimum code needed to reproduce the baseline effect.
@JungHsuan Provided a medium article on his approach, which was practically production ready and had the exact UI flow that was buttery smooth and consistent in all cases. I started with this approach, and converted it into something more re-usable in different situations, trying to make it quicker for other developers to pull into their projects.
UI Animation

Current known limitations/issues

Since the collapsible header view becomes absolutely positioned and is animated, you'll need to adjust zIndex of the headers above it (see example code)

In order to support scrolling the TabBar up, we must set a minimum height for the scrollable content. If you have one tab with only a small amount of content in there, it will be allowed to scroll up with large padding below it. This maybe be possible fix via measuring the content, but that would be an enhancement. Edit: Discussed possible fix here.

The CollapsibleHeader prop structure I initially proposed may not be ideal. It basically forces use to create an array of renderItem and data, which is used by each tab separately to draw its content. But the idea behind this even for non-virtualized content, we need a ScrollView or FlatList wrapper in there to drive this behavior, so we want to bake this functionality into the wrapper while still making implementation lightweight. This also has the side effect of fully supporting both tab content that are rendered at once, or using FlatList virtualized rendering.

I attempted to use SectionList to drive this, but challenges with measuring scroll distances shut down my early attempts.

If you scroll tab 1 all the way down, swap to tab 2, scroll it up any amount, and come back to tab 1, you'll notice tab 1 is now at the top, with the collapsible header being set at the same height. This isn't a real 'limitation' as it's simply necessary for the UI to be smooth, but it's good to be aware of.

Test plan
This is an early draft version of the PR. I am submitting in this state to see if there is interest in me polishing it up.
TODO's:

[ ] Stricter typings (I used any in many places where it can be better typed)
[ ] Removal of inline functions
[ ] Adding example of correctly using underlying FlatList
[ ] There is some warning spam right now that needs to be cleaned up

When the header's height > screen height it cant work
because the header cant scroll

You can try the react-native-head-tab-view.
(tabs with collapsible header)

@jehartzog @hejun041 @sagrawalcb @sjmueller @fantattitude
You can try the react-native-head-tab-view.
(tabs with collapsible header)

@zyslife Thanks for putting that together. In this case, I'm hoping to add the functionality as an optional inclusion to react-native-tab-view as I think that will help the larger base of devs who already use react-native-tab-view in their RN project and what to support this behavior.
Is there any interest in merging this in? I'm still happy to clean this up some more, write docs, but I've learned to wait until I hear back from maintainers.

I´m trying to implement this solution with SectionList right now. Regarding the limitations of point 4, are you referring to the fact that the only scrollTo function that is supportted in sectionList is the scrollToLocation one?

@alexco2 Yes, that is what I saw. I only looked for ~10 minutes into it, as I suspected the scrollToLocation would be more complicated due to being dependent on what kind of data was added. I didn't really look into it deeply though.

@jehartzog scrollToLocation expects itemIndex and sectionIndex. Both can be set to 0 though. With viewOffset you can then control the scrolling position with the use of pixel like in your example with the flatlist. Two problems then accrued.
First, if the sectionlist of the inactive route doesn´t have enough rows so that the header can be completely collapsed, the function syncScrollOffset somehow updates the header of the active route with the amount of translateY the inactive route can be scrolled. So, if the active route was scrolled with 50 pixel but the inactive route can just be scrolled 5 pixel because of to little rows, the active collapsible header gets updated with a translateY value of 5. My workaround is to dynamically extend a listfooter so that the list can alwas be scrolled to collapse the header completly.
Also, if the inactive route does not have any data in it, the syncScrollOffset function throws an out of bounds error because the itemIndex = 0 and sectionIndex = 0 do not exist. Weirdly using -1 each does not fix this. My workaround is to just deactivate the second route if there is no data, since it does not have any information for the user anyways.

@zyslife Thanks for putting that together. In this case, I'm hoping to add the functionality as an optional inclusion to react-native-tab-view as I think that will help the larger base of devs who already use react-native-tab-view in their RN project and what to support this behavior.
Is there any interest in merging this in? I'm still happy to clean this up some more, write docs, but I've learned to wait until I hear back from maintainers.

Thanks for your invitation,If I have enough energy, I should be able to react-native-tab-view add this feature. But not right now. I can only make react-native- head-tab-View  work right now ,out of my working hours. I hope I can do something in the future.

Hi folks I'm coming late to the conversation but was also working on this, I've just made a PR, let me know if you guys see how to improve it :)

We've been using the code based on this PR successfully in our prod app for a while, but going forward I'd probably pickup the outside packages published by #1096 rather than continue to maintain our fork. Just throwing this in for anyone looking to add these features.
On the other hand, repo owners decide they want to pull in this PR, I will be happy to spend the time to clean it up, get it back up to date, and get it merged. Just need someone w/ review authority to give thumbs up or down.

@jehartzog thanks for you solve my issue！！

On the other hand, repo owners decide they want to pull in this PR, I will be happy to spend the time to clean it up, get it back up to date, and get it merged. Just need someone w/ review authority to give thumbs up or down.

@satya164

Hey @jehartzog, thanks for your PR. Can you rebase this with the main branch? So we can get back to working on this and hopefully merge it. I will review this once you rebase it.

After consulting this with @satya164 we decided that we want to keep the library as small as possible (and easy to maintain) so I suggest you to publish this as a separate library.

@okwasniewski Would this feature be a huge increase in size to the library? This feature would be extremely useful but i definitely understand the reasoning haha


3719 enable map response terminology for bulk export

Refactored out the logic for mapping codes in ResponseTerminologyTranslationInterceptor into a service called ResponseTerminologyTranslationSvc
added logic in the write to file step of bulk export to call the ResponseTerminologyTranslationSvc to map the codes before export, in the file BaseResourceToFileWriter

The mapping specification will be removed from ResponseTerminologyTranslationInterceptor once the cdr side has been changed to use the new service as a way to set the mappings in terminology translation.
This way cdr will not break during this code change


Delete freeleech (Sonarr CF)

Sonarr does not support freeleech condition
Pull request
Purpose
Detailed description why you created this Pull Request.
Approach
If this Pull Request is created to solve a issue explain how does this change address the problem?
Open Questions and Pre-Merge TODOs
Check all boxes as they are completed

[x] Use github checklists. When solved, check the box and explain the answer.

Learning
If you're adding a new Custom Format make sure you follow the Radarr/Sonarr Custom Format (JSON) Guidelines.
Requirements
Check all boxes as they are completed

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

It also needs to be removed from the table
I will do that when I get home before I merge it.
Thnx


gqa model  runtime > no gqa model runtime

Hi!, I'm trying to replicate your implementation with Llama 2-13B and 7B, but curiously the runtimes didn't make sense (llama 2 gqa > llama 2 WITHOUT gqa) there is a little difference between my code and yours.
I do mean pooling to attention weights to group them as GQA paper, then I load this new attn_wts with load_state_dict method, but as I said, it didn't work.
Then, I tried your implementation out-of-the-box, I simply download the repo and copy-paste the t5 example, but it doesn't seem to work either.
%%time
outputs = t5.generate(input_ids, max_new_tokens=25)
CPU times: user 2.23 s, sys: 0 ns, total: 2.23 s
Wall time: 97 ms

%%time
outputs = t5_gqa.generate(input_ids, max_new_tokens=25)
CPU times: user 9.78 s, sys: 0 ns, total: 9.78 s
Wall time: 414 ms

I use the next packages in python 3.10:

transformers 4.36.2
torch 2.0.1+cu117
accelerate 0.25.0

Hardware: A100 80GB

Hey @Adonai02 and @fkodom, have you been able to figure out what was going on ? Thanks!