Update KSP version numbers for KS3P, PlanetShine, Scatterer

I've tested all three mods by setting the compat mode in CKAN, they work all fine. They're all dependencies of Astronomer's Visual Pack.
Please merge so everyone can benefit from this! :)

Checking the forum threads, most of these changes don't seem to be author supported. Mod authors don't like receiving bug reports caused by using their mods on versions that aren't officially supported, and they like it even less if CKAN falsely represented them as supported, even if a third party claims to have had no problems trying unspecified tests.

https://forum.kerbalspaceprogram.com/index.php?/topic/173138-141-planetshine-0261-mar-28-2018/

https://forum.kerbalspaceprogram.com/index.php?/topic/103963-wip161-scatterer-atmospheric-scattering-v00540-17032019-fixed-tsunami-bug-11/

Thanks for being interested in helping, but the right solution is for users who want this to opt-in via the compatibility pop-up that you used for testing. That way they know that they're doing something that the mod author technically hasn't endorsed, and will (hopefully) act accordingly.

The exception is KS3P, which looks like it should say 1.7 instead of the current 1.7.3, so we can look into making that change:

https://forum.kerbalspaceprogram.com/index.php?/topic/168763-17x-ks3p/page/32/

Hmm, KS3P's forum thread has repeated reports that it does not work on 1.7.3:

This ultimately leads up to the author identifying an uncaught exception from a user's logs and, on August 25, stating he's testing a fix. Since then his attention seems to be concentrated on real life (going by context of earlier statements) and there has not been a new release. Looks like not the right time to make those updates either. Let's give the author time to upload a new release, and presumably he will label it as compatible with 1.7.3.

Oh, ok. I'll see if I can get the other two authors to make a statement if they support 1.7.3 or not. Thanks for taking the time to explain your reasoning!


When developing a strapi plugin, is there a reason why I must use the strapi design-system?

Context
Hello, I have been using strapi for 2 years, but I am a beginner in strapi plugin development.
I am asking this question because I am considering various scenarios for strapi-based plugin development. We look forward to hearing from someone with experience.

When developing strapi plugin, is it absolutely necessary to use strapi design-system? Is there a reason why you must use the strapi design-system?

When developing strapi plugin, can I use a different react UI framework other than strapi design-system?

We are planning to move the previously developed functions using React Material-UI into the strapi plugin. Is this still okay?

When developing strapi plugin, is it okay to use other react UI framework and strapi design-system together?

When developing a site with a standalone react dashboard, is it okay to use strapi design-system (together)?
Our plan is to develop and use the strapi plugin, but if the functionality becomes too extensive, we may migrate to a standalone dashboard.

Proposal
Hello, I have been using strapi for 2 years, but I am a beginner in strapi plugin development.
In addition to design unity with strapi, I am curious whether the strapi design-system has other functional advantages, such as ease of functional integration with strapi.
Reasons
No response

Thanks for asking! It's a great question – if you want your plugin to be listed on our marketplace then yes, you must use the design-system for the UI.
If you're not interested in that then using the design system is not required to develop a plugin 🙂 however, having said that there's a huge benefit to using the design system, namely UI consistency across the product your users are working with.


Creating continuous views from continuous views

I have a stream  from which I am creating a continuous view .
I also have a continuous view which has a structure similar to this:
CREATE CONTINUOUS VIEW cv2
AS SELECT [fields]
FROM s1 AS s1
JOIN cv1 AS cv1 ON s1.id = cv1.id
WHERE [condition]

The problem is that when I insert something in the stream, the first view cv1 is populated, but cv2 is not.
However, cv2 is populated at the second/third insert AFTER the first view has some data on which the second view is dependent upon.

Hey @vighiosif, this would be a little complicated to implement under our current architecture. Could you maybe describe the problem you're trying to solve at a slightly more high-level and maybe we can figure out a workaround?

Closing this out for now, we can revisit at some point if necessary.


Fork ldap-filter package

As with #616, we should fork this package and issue a new release using the fork.
https://github.com/pfmooney/node-ldap-filter
Basically, I want to move all "ldap" dependencies into the org.

Should we see if @pfmooney wants to transfer it? or just clone it?

Either is fine. If we have to clone it, I'd prefer to do a disconnected clone as I did with the asn module:
$ git clone git@github.com:ldapjs/ldap-filter.git
$ cd ldap-filter
$ git remote add original-upstream https://github.com/pfmooney/node-ldap-filter.git
$ git pull original-upstream master
$ git push

Hi, any news on this one?
ldap-filter is now officially deprecated/archived.
Thanks

I expect it to happen as I get the next major prepared.

That's a really good news. Thanks for your work.

@richard-julien<EMAIL_ADDRESS>is available if you need it. I recommend looking through https://github.com/ldapjs/filter/pull/1 before using it, though. And you should expect other drastic changes prior to a final release.

v1.0.0 has been released. There are significant changes in it. See https://github.com/ldapjs/filter/pull/1

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Fixed missing ) in README.md

Coverage remained the same at 99.829% when pulling 16925bfe177966bb3071ae9bc132993906aa47c1 on ilyapuchka:patch-1 into 37e311929e34d8fb1a2fe30ef79644e4ad83c4ab on nicklockwood:master.

Thanks!


Support for BluDevices works only for Gateways in the Shelly Cloud

Hi @Luligu,,
I tried add a Shelly BLU DoorWindow using a GEN3 device as described. I struggled with the fact, that the Gateway needs to be configured in the Shelly-Cloud. Otherwise it seems not to get updates from the BLU device.
I think you should note that in the documentation.

Hi,
I cannot reproduce this issue.
I imagine that the issue is related to a single device. In my experience some BLU gateways are less effective then others.
All my gateway receive updates even if not connected to cloud. It makes sense to me cause all is local.

Discord
Tamer (https://github.com/tammeryousef1006) has created the Matterbridge Discord group: https://discord.gg/QX58CDe6hd.
Feel free to join (the link is now permanent)!

I can confirm that everything works without shelly cloud via matterbridge now. The strange thing is, that the shelly app itself does not update the blu device states as long as the gateway-device is not in the shelly cloud.

Even the process of adding a blu device without having a gateway in the shelly cloud worked now for me. So I can not reproduce that anymore. I think you can close this issue. Thanks for taking time.


Drop v0.4

Drop v0.4 in tests, and require v0.5. This will be required for using views. Note that this means that people on previous versions when using Pkg.add will get the last release before this release. I think this is sensible for such a large change: people looking for backwards compatibility likely won't want such a large change dropping in.

Coverage increased (+0.006%) to 11.062% when pulling 2e0a8cc4e14cfaf4e9878d609d7fba0b3fae87d3 on ChrisRackauckas-patch-1 into c33c56e25c6682a99aca625fb79f2f07aec689bb on master.


context: Breaking change in Go 1.15 not called out in the release notes

What version of Go are you using (go version)?

$ go version
go version go1.15 linux/amd64

Does this issue reproduce with the latest release?
Yes.  This is a new issue introduced in Go 1.15.
What operating system and processor architecture are you using (go env)?
go env Output (using the `golang:1.15` Docker image)
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/root/sandbox/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build266629529=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Run pre-existing code that was passing nil as the parent context to context.WithValue().  Minimal example: https://play.golang.org/p/J0Bxux7Bfs6
What did you expect to see?
The string "didn't panic" written to standard out
What did you see instead?
panic: cannot create context from nil parent

goroutine 1 [running]:
context.WithValue(0x0, 0x0, 0x4afc80, 0xc0000961f0, 0x4afa40, 0x4e7ae0, 0x0, 0x0)
	/usr/local/go-faketime/src/context/context.go:521 +0x187
main.main()
	/tmp/sandbox506034916/prog.go:14 +0x77

Program exited: status 2.
This change in behavior was introduced by https://github.com/golang/go/commit/02057906f7272a4787b8a0b5b7cafff8ad3024f0.  While I agree with the change in principle, I feel like this should have been specifically called out in the release notes for Go 1.15.
Updating context.WithCancel(), context.WithDeadline() and context.WithValue() to panic when they did not in Go 1.14 and prior will likely be a breaking change for users and, I feel, should be a highlighted change for visibility.

It would be nice if passing a nil parent behaved the same as passing a context.Background()


GA TTLAfterFinished

This PR updates the TTLAfterFinish controller to stable as part of 1.23 k8s release.
Related PRs:

https://github.com/kubernetes/kubernetes/pull/105219
https://github.com/kubernetes/enhancements/pull/2841

/sig apps

@sftim @ahg-g reworded to make it explicit that the feature is supported only for jobs. Please let me know if any further changes needed.

I also recommend looking at https://github.com/kubernetes/website/issues/26917, which is about making sure the right pages hyperlink to the TTLAfterFinished docs.

@sftim I have addressed the review feedback given so far. Please take a look. Thanks again!


Error expanding server node when connecting to SQL Server

Server node not expanding when connecting to SQL Server (server is accessible via SQL Management Studio)
[2020-12-17 10:57:14.209] [renderer1] [info] Creating new connection connection:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:57:14.210] [renderer1] [info] Adding connection connection:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:57:14.210] [renderer1] [info] Successfully added connection connection:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:57:27.378] [renderer1] [warning] Could not find ServerInfo for connection 8eb3cdec-40d2-4332-a447-f1b193038a95 when updating icon
Double clicking to open the Server properties tab produces this output:
[2020-12-17 10:59:50.575] [renderer1] [info] Creating new connection dashboard:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:59:50.576] [renderer1] [info] Adding connection dashboard:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:59:50.576] [renderer1] [info] Successfully added connection dashboard:providerName:MSSQL|applicationName:azdata|authenticationType:Integrated|database:MBIMASTER|server:hcs-dev-mstr1|user:|databaseDisplayName:MBIMASTER|group:a42c273a-d048-4ff4-b0b9-df0172a73954
[2020-12-17 10:59:51.236] [renderer1] [warning] Could not find ServerInfo for connection 8eb3cdec-40d2-4332-a447-f1b193038a95 when updating icon
[2020-12-17 10:59:51.892] [renderer1] [error] Error rendering widget explorer-widget {}

I get a similar bug that affects only certain users. Same error in logs "Could not find ServerInfo for connection" ... "when updating icon".
Expanding databases and tables works fine for these users in SSMS just not ADS.
Interestingly I have a very high privileged user on the same server and ADS works fine to expand Databases and Tables in that case. My affected users should have sufficient privileges to perform this action especially given that it works in SSMS. This definitely points to a bug in ADS.
I tried explicitly granting VIEW Definition, EXECUTE, etc. and on master VIEW ANY DATABASE to see if that would change anything for the affected users and nope.
Version is current:
Version: 1.25.2
Commit: 4d4917d328ab7a7526817a0f0ef03237a79aa54d

I have the same issue with one specific server.
I suspect I have limited privileges on this server although not sure exactly what they are as it is monitored/maintained outside of our immediate team and they are reapplied via script on db restore.
Everything works correctly in SSMS.
Please let me know if there is additional info I can provide to help debug this. It is frustrating to have to jump back to SSMS and I have very few reasons to even have SSMS installed currently.

This looks similar to https://github.com/microsoft/azuredatastudio/issues/6450.  I'll close this as a duplicate to consolidate the investigations.  There may be multiple issues showing up with similar buggy behavior, but having related discussions in one issue helps track the various diagnostic logs easier.


Call resolver dynamically upon call to \lyp-require, \lyp-include etc

See also #61.
Currently the resolver scans any dependency declared, then checks all included files. This leads to problems when we wish to do fancier stuff, like \lyp-require #'(a b c d), or invoking the lyp API from scheme code. It also means we need to scan any possible include file before Lilypond starts to process the file.
So, idea is to get rid of the pre-processing scanning of files, and invoke the resolver inside of lyp:require. That means:

The resolver is a bin script that is run from within the lyp:require scheme procedure.
It returns a text representation of a scheme object containing info about the package - name, version, entry-point path.
The resolver selects the best version that satisfies the package ref. If a different version of the package is resolved later, lyp:require should throw an error.
If the dependency is not found, lyp:require should throw an error.

Plan of action:

Provide path to resolver bin script in the dynamically generated wrapper.
Infrastructure to run resolver script with the dependency reference as command line argument.
Receive output from resolver script, eval it into a scheme object and process it.
Handle errors (dependency not found, version ambiguity)
Profit!

Another thing that should be taken into account: if dependencies are all defined in an external file, it might help the resolver to make smarter decisions about versions, as it does now. So we have:

Dynamic resolution of dependencies.
Ahead-of-time resolution of dependencies.

These two things are not mutually exclusive. Let's say for example that dependencies can be defined in a lyp.yaml file. This file is optional - it just allows the resolver to make smarter decisions about versions. So, if such a file containing all dependencies exists, the resolver uses it to resolve the different versions before the file is compiled, and injects the information into the wrapper script handed to Lilypond. If any dependency is not found in lyp:package-refs/lyp:package-dirs, the resolver is called in-process to resolve them.


Work with Flask 1.1.2 and upgrade itsdangerous to 2.0.0 won't parse bytes

Hi,
I work with Flask 1.1.2 and upgrade itsdangerous from 1.1.0 to 2.0.0, find it won't be able to parse bytes.
from flask import jsonify, Flask

def test():
    app = Flask(__name__)
    with app.app_context():
        a = {'1': b'abcd'}
        print(jsonify(a))

test()

with ItsDangerous 1.1.0 this code works but with 2.0.0 it comes out
".../python3.7/site-packages/flask/json/__init__.py", line 370, in jsonify
    dumps(data, indent=indent, separators=separators) + "\n",
  File ".../python3.7/site-packages/flask/json/__init__.py", line 211, in dumps
    rv = _json.dumps(obj, **kwargs)
  File ".../python3.7/json/__init__.py", line 238, in dumps
    **kw).encode(obj)
  File ".../python3.7/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File ".../python3.7/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File ".../python3.7/site-packages/flask/json/__init__.py", line 100, in default
    return _json.JSONEncoder.default(self, o)
  File ".../python3.7/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type bytes is not JSON serializable

Environment:

Python version: 3.7.2/3.7.10 tested
Flask version: 1.1.2
ItsDangerous version: 1.1.0 to 2.0.0

I'm guessing that this stopped working for you because itsdangerous no longer depends on simplejson. Try installing simplejson explicitly. See also https://github.com/pallets/flask/issues/3555 for a similar change in Flask 2.0.0.

In addition to the suggestions above, be sure to use a tool like pip-compile to pin your dependencies and control when you upgrade.


Fix #1497, Use CFE_TIME_SysTime_t instead of indiv. sec/subsec members

Checklist

[x] I reviewed the Contributing Guide.
[x] I signed and emailed the appropriate Contributor License Agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>
Describe the contribution

Fixes #1497

Remaining individual (uint32) members representing seconds/sub-seconds for file creation time were converted to use the unified CFE_TIME_SysTime_t type in 7 locations.

Testing performed
All tests were passing before changes to CFE_FS_Header_t were added (see https://github.com/nasa/cFE/actions/runs/4585928750 and https://github.com/nasa/cFE/actions/runs/4585928751).
The functional test and CodeQL analyses are no longer able to build because elf2cfetbl references the old uint32 members in CFE_FS_Header_t directly, so an additional PR for elf2cfetbl is required at the same time - see https://github.com/nasa/elf2cfetbl/pull/134.
Local tests with full cFS package confirm all tests passing when both changes are made together, and net coverage unaffected.

Note: If this is merged it will require updates to some of the txt files in the Ground System tool to ensure clean and correct interpretation of the data members.
System(s) tested on
Intel(R) Celeron(R) N4100 CPU @ 1.10GHz x86_64
Debian GNU/Linux 11 (bullseye)
Current main branch of cFS.
Expected behavior changes
No change to behavior.
Contributor Info
Avi Weiss @thnkslprpt

Partially implemented in https://github.com/nasa/cFE/pull/2544


Use session when creating user

Use session when checking for pre-existing usernames and email addresses, not just when inserting user.

Codecov Report

Merging #2638 into master will increase coverage by <.01%.
The diff coverage is 30.76%.

@@            Coverage Diff             @@
##           master    #2638      +/-   ##
==========================================
+ Coverage   27.11%   27.12%   +<.01%
==========================================
  Files          86       86
  Lines       17064    17066       +2
==========================================
+ Hits         4627     4629       +2
  Misses      11759    11759
  Partials      678      678

Impacted Files
Coverage Δ

models/user.go
18.24% <30.76%> (+0.17%)
: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 a8717e5...4356b4e. Read the comment docs.

LGTM

LGTM


[1.17] Reimplement World Gen Range Decorators, their respective features, and fix ALL crashing features.

Not completed at the time of opening this PR but the goals of this PR are:
-Re-implement ore, "prototype ore" distributions and other features in relation but not limited to the following classes: BiasedToBottomHeight, ConstantHeight, TrapezoidHeight, UniformHeight, VeryBiasedToBottom, CaveSurface, RangeDecorator, and VerticalDecorator.
-Datapack support for these decorators
-Fix all crashes in relation to the placements listed above.

This commit is ready for review and should complete everything world gen. Bugs most likely exist though and those are to be squashed as they're discovered


error: cannot run C compiled programs.

Hello,
I've been trying to install nanopolish on my mac for a while, but I keep getting all kinds of error's. This one I can't seem to resolve myself. Im running MacOS Mojave
This is the error I keep getting:
checking for gcc... clang
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... configure: error: in /Users/administrator/nanopolish/hdf5-1.8.14': configure: error: cannot run C compiled programs. If you meant to cross compile, use --host'.
See `config.log' for more details
make: *** [Makefile:91: lib/libhdf5.a] Error 1
MacBook-Pro-2:nanopolish administrator$
I also saw on other issues that you might need a config file, so i just added it. It might be a noob problem, but that's because I am one :)
Thanks a million!
config.log

Hi Jared,
Thanks for your reply and my apologies for responding so late. I tried your suggestion in october and it didn't work. Today I tried it again and the makefile was working like a charm. It did get to install nanopolish.
Now when I run 'nanopolish', I get the error:

dyld: Library not loaded: @rpath/libgomp.1.dylib
Referenced from: /Users/administrator/nanopolish/./nanopolish
Reason: image not found
Abort trap: 6

Do you have any ideas?
Thanks again!
Dirk

Hi,
You need to compile nanopolish using gcc, not clang. Can you try to install gcc (from conda or homebrew maybe) then run:
make CC=gcc-<version> CXX=g++-<version>

where is the version number you installed.
Let me know how it goes,
Jared

gcc --version
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-39)
Please suggest exact command syntx


small performance improvement

Codecov Report

Merging #44 (eeaab5f) into main (a98b48c) will not change coverage.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   96.33%   96.33%
=======================================
  Files           7        7
  Lines         819      819
=======================================
  Hits          789      789
  Misses         22       22
  Partials        8        8

Impacted Files
Coverage Δ

filter.go
97.76% <100.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 a98b48c...eeaab5f. Read the comment docs.


[Dashboard] Allow custom proxy URLs for EXTERNAL_X_REDIRECT_URI

A user should be able to swap the redirect URL for a vanity proxy address they have set up
Discussed in https://github.com/supabase/supabase/discussions/3428

Originally posted by ilyabo October  5, 2021
Is it possible to use the actual site URL in the auth callback URL for external OAuth providers? Maybe at least with the Pro plan?
It doesn't look very convincing or trustworthy with the randomly generated hostname:

I tried adding a URL-rewrite on Vercel (from https://…supabase.co/auth/v1/callback to https://mysiteurl.com/auth/v1/callback), but I am getting this error when trying to authenticate with GitHub:
?error=redirect_uri_mismatch&error_description=The+redirect_uri+MUST+match+the+registered+callback+URL+for+this+application.
```</div>

I am having the same issue. Is there any update?

I agree, the current random generated database name is screaming "I've been hacked, don't sign in!!!".
Would it be possible to use the new _middleware.js in NextJs 12 to receive the request then await the response from the real database?
ex:
Google => your site => async request to Supabase => back to your site => back to Google

This looks like it might be a /\subset of the problem described in #143
Shall we close the issue and continue the discussion there instead?

@J0 yeah good call, lets close this one


docs: fix objc diff in upgrade guide

add missing "#" for #import
Goal
fix code in documentation with correct objective-c syntax
Design
N/A
Changeset
upgrade guide for react-native
Testing
N/A

Hey @maxkomarychev, thanks for raising this PR, good catch! Closing this one out as it's a duplicate of https://github.com/bugsnag/bugsnag-js/pull/1245 which has already been merged in :)


Trouble Getting Started

I'm working with the getting started and I'm unable to generate the dart code from the schema.
My  build yaml config:
targets:
  $default:
    builders:
      graphql_codegen:
        options:
          # Remove the $ from the class names
          namingSeparator: ""
          generatedFileHeader: "// GENERATED FILE DO NO MODIFY BY HAND\n"
          outputDirectory: /lib/graphql
          assetsPath: /graphql/**
          clients:
            - graphql_flutter

I have a schema.graphql located within /graphql/.
My pubsec looks like:
environment:
  sdk: '>=3.0.0 <4.0.0'

dependencies:
  flutter:
    sdk: flutter
  graphql: ^5.1.3
  graphql_flutter: ^5.1.2
  flutter_hooks: ^0.20.3

dev_dependencies:
  flutter_lints: ^3.0.1
  build_runner: ^2.4.6
  graphql_codegen: ^0.13.7

The following statement in the readme was uninformative:

Read more about the tool and motivation at the GraphQL Codegen deep-dive and on how you can structure your flutter apps with the tool on Structure your Flutter GraphQL apps.
 In Android Studio this can be done with the GraphQL plugin.

I'm using vscode as my IDE and my flutter env looks like:
Flutter 3.13.8 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 6c4930c4ac (4 weeks ago) • 2023-10-18 10:57:55 -0500
Engine • revision 767d8c75e8
Tools • Dart 3.1.4 • DevTools 2.25.0

Running dart run build_runner build produces:
[INFO] Generating build script completed, took 273ms
[WARNING] Throwing away cached asset graph because the build phases have changed. This most commonly would happen as a result of adding a new dependency or updating your dependencies.
[INFO] Cleaning up outputs from previous builds. completed, took 6ms
[INFO] Generating build script completed, took 52ms
[WARNING] Invalidated precompiled build script due to missing asset graph.
[INFO] Precompiling build script... completed, took 639ms
[INFO] Building new asset graph completed, took 763ms
[INFO] Checking for unexpected pre-existing outputs. completed, took 0ms
[INFO] Running build completed, took 3ms
[INFO] Caching finalized dependency graph completed, took 63ms
[INFO] Succeeded after 73ms with 0 outputs (0 actions)

Is there additional configuration that I am missing?

Thanks for reporting @Nexushunter! A bug here prevents finding GraphQL files in the graphql folder (it works for any sub-folder of graphql). I've fixed this in #319.
Remember to add graphql in your sources options


Inference Time

Can anyone explain to me why the first window in inference takes considerably longer time compared to following windows when input data is larger than window size?  Here's ans example of execution times for my application
Thanks.

Hi @somsol it takes a bit of extra time to do prefetching at the first iteration, but overall it's hiding the I/O latencies at each iteration, the idea is illustrated here: https://www.tensorflow.org/performance/datasets_performance#pipelining


Add details for 1.0 to 2.0 migration

Changes in this pull request
Added migration guide for IGListDiffable changes.
Closes #221

@jessesquires looking at our 2.0.0 changelog it doesn't seem like anything else major requiring explanation (other than what's in the changelog) is necessary. Ya?

Coverage remained the same at 94.116% when pulling 18d132ac2251e9ab4e3a9529463a602318ade014 on migrate into e5afb5b4d0cfc70a2736b02279b6bc239ddf1e5d on master.

Coverage remained the same at 94.116% when pulling 18d132ac2251e9ab4e3a9529463a602318ade014 on migrate into e5afb5b4d0cfc70a2736b02279b6bc239ddf1e5d on master.

Coverage remained the same at 94.116% when pulling 18d132ac2251e9ab4e3a9529463a602318ade014 on migrate into e5afb5b4d0cfc70a2736b02279b6bc239ddf1e5d on master.

Coverage remained the same at 94.116% when pulling 18d132ac2251e9ab4e3a9529463a602318ade014 on migrate into e5afb5b4d0cfc70a2736b02279b6bc239ddf1e5d on master.


Add index names to data

Legacy code depends on index names for some operations, and some ops in the system seem to clobber index names. These can be easily tacked on by the qsm

Fixed by 8963568


next() call on validation error

I've mentioned that validation handler doesn't make next() call on error. As a result, restify doesn't run all following post-request handlers (e.g. loggers attached via server.on('after', ...);).
More restify-friendly solution was proposed in the PR.

This is OK for me. Thank you.


Editing Delegations

User Story
Modifying existing delegations

Epic Title
Delegation

T-shirt size:

User Story
"As an APPROVER in the NeXt app, I want to modify an existing delegation"

Description
Clicking on any of the listed Delegations on the summary page...

takes me to the Delegation setup page.  Current values for this delegation are filled in.
For Active delegations, I can only change the END date

moving the date to today means the delegation will end today.
I can't move end date into the past
moving the date further out extends the duration of the delegation.*

For Pending Start delegations

I can edit both start and stop dates on a delegation*
I can edit the list of roles to add or remove them*

For Pending Approval delegations

can't edit until accepted or rejected?

for Ended/Rejected delegations

I can edit the start and stop dates to resurrect a delegation*
One of the UI mockups shows a "suggested combinations of roles" based on past delegations.  Maybe this is a better way of saving common delegation configurations??
I can change the list of affected roles or the delegate name -- but nothing Send button is deactivated until I have selected viable dates for the delegation.*

*these scenarios will require an acceptance notification to be sent (related story)
Acceptance Criteria/Tests

demonstrate editing of each type listed above

migrated to new tmobile/sawtooth-next-directory repo


Disable Gradle daemon for user

In CI environment, Gradle daemons are discouraged.
See https://docs.gradle.org/current/userguide/gradle_daemon.html#sec:ways_to_disable_gradle_daemon
This resolves https://github.com/travis-ci/travis-ci/issues/8205

Tests:

java: https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/616155#L421-L422
android: https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/616157#L543-L544

@BanzaiMan is this something that we'd like to see baked into image stacks permanently in the future?

@Lyoness Yes, that would be best!


Unable to get v0.0.8 from maven

Thanks for new release v0.0.8 but still can't synced from maven
@scottyab Could you please check if it's been updated? Thank you!

Thanks for flaggin, I having a few issues with the upload.

Hi @scottyab just following up when can v0.0.8 be placed in maven.

It's been year or so since the last update and struggling with the upload to nexus repo. The uploadArchives seems to have disappeared or rather it's not actually doing the upload. I'm not sure what's changed in this regard please bear with I have limited time to fault find.

Confirmed https://search.maven.org/artifact/com.scottyab/rootbeer-lib/0.0.8/aar


[JBTM-1107] Enable XTS recovery in transactions subsystem

https://issues.jboss.org/browse/JBTM-1107

Needs rebase but my fundamental question is just: "Does this mean that to run a server with transactions we need a SOAP stack running"

No. It only bootstraps recovery modules for XTS. Everything else like TM, endpoints etc. are still done in the xts subsystem.


test_stages: speed up the test execution time by aggresive caching

Let's speed up the execution by:

sharing the osbuild store across all tests
checkpointing the build pipelines

If you see any issues with this approach, please raise your concerns. :)

I think there were two concerns about this way back when we first tried:

Does it mean we might not test everything properly as we get cached artifacts - no, there should be dedicated tests to validate that the cache is truly a cache (so using it has no effect on the result)
We ran out of disk space - I guess the CI checks will speak for themselves...


adding config settings for redis.conf.erb

add rdbcompression to default settings & update template
add dbfilename default setting & update template & provider
add appendfilename default setting & update template & provider
update template for slowlog-log-slower-than
update template for slowlog-max-len
add list-max-ziplist-entries to default settings & template for v2.6+
add list-max-ziplist-value to default settings & template for v2.6+
add tcp-backlog to default settings and template for v2.8+
add slave-read-only to default settings and template for v2.6+
add hll-sparse-max-bytes to default settings and template for v2.8+
add rdbchecksum to default settings and template for v2.6+
add repl-disable-tcp-nodelay to default settings and template for v2.6+
add slave-priority to default settings and template

Thank you for putting guards against specific versions for all of these :)

I tried to prevent unexpected consequences as much as I could :-)

I'd like to use tcpbacklog attribute. Is this PR gonna be merged soon? While it's not merged, how can I make a patch? I need to set a tcpbacklog attribute.

PR looks good to me.  Sorry for the delay @scaryguy @drbayer


Cannot build with Cordova 9.0.1

Hi team,
I add this plugin in cordova project but i cannot build it (iOS).
cordova build ios

This is following error.
Cannot find module 'q'

Cannot find module 'q'
resolve with
npm install q

Alternatively, you can fix it by remove q module and use Promise native. Following:
https://github.com/apache/cordova-lib/pull/710


⚠ ClamAV has degraded performance

In f8c071c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 03a21b8.


Allow let! _  as last statement in computation expressions

Consider the following code:
let x() = async { return () }

let y =
    async {
        let! _ = x()
    }  // compile error - Unexpected symbol }

let z =
    async {
        do! x()
    }

If x returns anything else that unit then you can't use do! and you have to do:
let y =
    async {
        let! _ = x()
        ()
    }

This is hard to explain to newcomers - especially given the error mesage.

Isn't this exactly how a regular let works?
You cannot write let t = 2 in a regular function you have to complete the let expression.
Also there is Async.Ignore in order to use do!:
let y = x() |> Async.Ignore
//or
let y = async { do! x() |> Async.Ignore }

We could also consider to allow do! on all values but emit a compiler warning. I guess this would match the regular code flow. This probably would be considered a language change... So yes a better compiler message is probably the first step.

I wasn't even aware you could write a computation expression without a return statement. And in my experience, as long as the return isn't there, the errors are less than useful. And more often than not, it breaks any intellisense, tooltips etc that comes after the CE.
I don't think allowing ending with a let binding is idiomatic, but better errors would be a big improvement.

In the world of task {} with many many side effects it's normal to go without return

Given that a normal let can't be the last line of an expression block, I don't necessarily like allowing let! to "finish" a computational expression.
The standard message is:

The block following this 'let' is unfinished. Every code block is an expression and must have a result. 'let' cannot be the final code element in a block. Consider giving this block an explicit result.

I fully agree that a better error message here, more like the standard one, would be very beneficial for newcomers - it'd be nice if there was some consistency in error messages when inside or outside of a computational expression.

Please convert this to an error message improvement tracking issue?


authorization release

[resource-manager] JS: 'Release for Policy Administrative Services - 2022-05-01-preview' · Issue #4262 · Azure/sdk-release-request (github.com)
Recording: https://github.com/Azure/azure-sdk-assets/tree/js/authorization/arm-authorization_2c91b62eb3

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-authorization


Copyright Headers: SPDX

The copyright headers in our repository do not contain Software Package Data Exchange (SPDX) identifiers, which is required by the Eclipse Foundation Project Handbook.
I think the following tag would be appropriate for the Eclipse Public License v2.0 part:
SPDX-License-Identifier: EPL-2.0 AND EPL-2.0

I do not know however if and how the Eclipse Distribution License v1.0 part should be reflected, as it is not an accepted SPDX license. Should we rather omit it or use the BSD-3-Clause identifier?
Acceptance Criteria

All source code files have an SPDX license identifier
Used identifiers listed in NOTICE.md

SPDX in header sounds new to me, I guess I missed this requirement.
I'm not sure if we should use OR or AND but I think a OR is more appropriated as I understand that user should accept the term of one of the license. :thinking:
About EDL there is no SPDX Id and following this message of Wayne BSD-3-Clause should be used.
So I understand it should be :
SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause

@waynebeaton  could you confirm this ? :pray:

Confirmed.

Adapted the original issue. Do we already have a label to mark easy-issues?

I create it.


Add delay option to macro key (#152)

Popup window instantly when one repeatedly invoke macro is quite annoying. This patch adds an option to disable nowait for @.

I've been looking how to avoid the instant popup for marks or the " key as well and I've stumbled across this, but I'm not sure I understand how this works.
Is this really configurable? Is there a way to configure a plugin's actions table in order to control the value of delay?


Has anyone used this library with OKTA as IDP?

After I click on login I get a 302 Found on the /?sso URL and I get redirected to the /metadata URL and it shows the SAML response. Is this an expected output?
Here is my settings.json:
{
"sp": {
        "entityId": "http://localhost/metadata",
        "assertionConsumerService": {
            "url": "http://localhost/?acs",
            "binding": "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST"
        },
        "singleLogoutService": {
            "url": "http://localhost/?sls",
            "binding": "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        },
        "NameIDFormat": "urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress",
        "x509cert": "",
        "privateKey": ""
    },
    "idp": {
        "entityId": "http://localhost/metadata/",
        "singleSignOnService": {
            "url": "https://dev.oktapreview.com/app/test/sso/saml",
            "binding": "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        },
        "singleLogoutService": {
            "url": "https://dev.oktapreview.com/app/test/sso/saml",
            "binding": "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        },
        "x509cert": ""
    }
}

And are the below settings correct on the OKTA side?

You configured on Okta side the metadata URL as the Sign Sign On URL, that why you end in that URL.
Use instead the http://myapp/?sso URL

@pitbulk - Thanks! That solved the issue.


syncHistoryWithStore clearing localStorage

I'm trying to implement redux-storage and am noticing that right after syncHistoryWithStore is called my localStorage is kinda wiped:
console.log('before syncHistoryWithStore', JSON.parse(localStorage.phrase).phrase.past.length)
// some number like 6 or 10..
const HISTORY = syncHistoryWithStore(browserHistory, STORE)
console.log('after syncHistoryWithStore', JSON.parse(localStorage.phrase).phrase.past.length)
// always 0

Check your reducers. Are you replacing your own reducer with this library's, or are you integrating it via something like combineReducers? That's probably where the error is.

Thank you for the response. After some testing I feel like I've come back to the same bottleneck
This setup works perfectly.. after page load my saved actions restore my previous state:
import thunk from 'redux-thunk'
import { Provider as StoreProvider } from 'react-redux'
import { createStore, compose, applyMiddleware } from 'redux'
import { Router, Route, browserHistory } from 'react-router'
import { syncHistoryWithStore } from 'react-router-redux'
import * as storage from 'redux-storage'
import createEngine from 'redux-storage-engine-localstorage'
import finalReducer from './reducers/reduce.js'

const localStorageEngine = createEngine(`phrase`)
const localStorageMiddleware = storage.createMiddleware(localStorageEngine)

const finalCreateStore = compose(
  applyMiddleware(thunk, localStorageMiddleware),
  window.devToolsExtension ? window.devToolsExtension() : f => f
)(createStore)

const STORE = finalCreateStore(storage.reducer(finalReducer))
const load = storage.createLoader(localStorageEngine)
load(STORE)

import App from 'components/App.js'

window.onload = () => {
  ReactDOM.render(
    <StoreProvider store={STORE}>
      <Router history={browserHistory}>
        <Route path="/" component={App} />
      </Router>
    </StoreProvider>,
    document.getElementById('root')
  )
}

With the simple addition of syncHistoryWithStore the localstorage contents clear up giving nothing to the load function
const HISTORY = syncHistoryWithStore(browserHistory, STORE)
...
<Router history={HISTORY}>


GHA: properly use a concurrency group

Describe the bug
With high frequency pushes onto a PR, gha still goes on with old HEADs CI jobs.
This quickly fills up our gha space and thus freeze new CI jobs.
Expected behaviour
Old HEADs CI jobs should be automatically canceled and we should only run CI against latest HEAD commit.
Additional context
Perhaps https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#concurrency could help us.

This should be fine for us:
concurrency:
  group: ${{ github.ref }}
  cancel-in-progress: true

/milestone 0.11.0


build-script: remove any swift files in the generated file dir before generating new ones

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 23b427f70fe13a628206f0da7f70640d4d718af9


[FEATURE] Server-side WebSockets

As requested per @jvo203, see #6

implemented in 00291117d85144fe62c394afa1ebc634298ec7f4


Step priority

It appears that when using step priority, the macro will start on the 1234 part of the step function instead of 1 12 123...etc
This explains why I see issues many times at the beginning of the fight after macro reset.
Here is a link to a video that I made of testing this: https://youtu.be/5cLbsb7ZUaY

On your macro ensure the Combat Reset button is ticked and/or that the Reset after combat option is activated in GSE’s options.
TimothyLuke

On 6 Dec 2020, at 12:14 am, Bobby21677<EMAIL_ADDRESS>wrote:

It appears that when using step priority, the macro will start on the 1234 part of the step function instead of 1 12 123...etc
This explains why I see issues many times at the beginning of the fight after macro reset.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I will double check, but I believe it is.
In the video, it shows the macro resetting when I leave combat, it appears
to be resetting anyway...unless I’m looking at it wrong.
On Sat, Dec 5, 2020 at 12:41 PM Timothy Minahan<EMAIL_ADDRESS>wrote:

On your macro ensure the Combat Reset button is ticked and/or that the
Reset after combat option is activated in GSE’s options.
TimothyLuke

On 6 Dec 2020, at 12:14 am, Bobby21677<EMAIL_ADDRESS>wrote:

It appears that when using step priority, the macro will start on the
1234 part of the step function instead of 1 12 123...etc
This explains why I see issues many times at the beginning of the fight
after macro reset.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/764#issuecomment-739332900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASAIJCBRHOOO2654J4R5C7TSTJ5E7ANCNFSM4UOR5IGQ
.

--
Bobby McWilliams
620-755-3888

Am I seeing this correctly or is there something that I need to do differently within GSE

Can you please confirm that either the Combat reset is gold or if grey then
the GSE Options to reset after combat are selected?  You were going to
check that and get back to me.
On Thu, 10 Dec 2020 at 08:15, Bobby21677<EMAIL_ADDRESS>wrote:

Am I seeing this correctly or is there something that I need to do
differently within GSE
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/764#issuecomment-742082616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEMCFPG5TI7RA4RK2V7S2LST7VZHANCNFSM4UOR5IGQ
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

Sorry Timothy, the box is checked grey. Also, it is resetting to the first spell when leaving combat in the video.

I cant see your video.  Im on a phone atm and you didn't link it.
As its Grey what do your GSE Options say?
On Thu, 10 Dec 2020 at 11:12, Bobby21677<EMAIL_ADDRESS>wrote:

Sorry Timothy, the box is checked grey. Also, it is resetting to the first
spell when leaving combat in the video.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/764#issuecomment-742156210,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEMCFJGQPQYN26QJMML27TSUAKNVANCNFSM4UOR5IGQ
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

There’s a link in the first post that isn’t working for some reason...I’m new to GitHub.

That one worked for me. And in the options, it shows reset on combat.

Is this behaving now in 2.6.35?

Yes, it is behaving the same way in 2.6.35.  The macro resets to the first spell, but nothing seems to reset the first step in the step function.  In fact, I can't find anything that resets the priority steps other than reloading the addon.  Also, if I press the macro while not in combat (some of the spell are defensive) the macro won't reset.  I noticed that there are keybinds that can be set to manually reset the macro...would it be possible to add an amount of time, where if the macro isn't pressed, it resets?  Like reset=5.  I would be happy to create a video again or share screen and test with you.  I'm concerned that you weren't able to recreate this...makes me feel like I'm missing something.

Unfortunately GSE has no way to tell time within the combat sandbox.  It cant tell that it hasn’t been used for x time.  When I get to login next will just keep digging.
TimothyLuke

On 16 Dec 2020, at 12:15 am, Bobby21677<EMAIL_ADDRESS>wrote:

Yes, it is behaving the same way in 2.6.35. The macro resets to the first spell, but nothing seems to reset the first step in the step function. In fact, I can't find anything that resets the priority steps other than reloading the addon. Also, if I press the macro while not in combat (some of the spell are defensive) the macro won't reset. I noticed that there are keybinds that can be set to manually reset the macro...would it be possible to add an amount of time, where if the macro isn't pressed, it resets? Like reset=5. I would be happy to create a video again or share screen and test with you. I'm concerned that you weren't able to recreate this...makes me feel like I'm missing something.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I've created a test version for you - https://www.curseforge.com/wow/addons/gse-gnome-sequencer-enhanced-advanced-macros/files/3139888 Could you have a look at it?  Its marked as version 2.6.35-3

Priority is still not resetting to the first step with this version.

ok next test:  https://www.curseforge.com/wow/addons/gse-gnome-sequencer-enhanced-advanced-macros/files/3140821

if it works i will release it at 2.6.36 but for now this is 2.6.35-4

It looks like this is working correctly!  Thank you sir!


Update copyright year

Hello @thinkingserious,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


How to load the 8bits model with Huggingface in colab

Hi. Thanks for providing the code for huggingface.
I am trying to use the following code in colab, but the session crashes because it runs out of ram.
I am using colab pro with high-ram setup with 25 gb of ram and T4 gpu. But the session still crashes.
# Load via Huggingface Style
from mplug_owl.modeling_mplug_owl import MplugOwlForConditionalGeneration
from mplug_owl.tokenization_mplug_owl import MplugOwlTokenizer
from mplug_owl.processing_mplug_owl import MplugOwlImageProcessor, MplugOwlProcessor

pretrained_ckpt = 'MAGAer13/mplug-owl-llama-7b'
model = MplugOwlForConditionalGeneration.from_pretrained(
    pretrained_ckpt,
    torch_dtype=torch.bfloat16,
)
image_processor = MplugOwlImageProcessor.from_pretrained(pretrained_ckpt)
tokenizer = MplugOwlTokenizer.from_pretrained(pretrained_ckpt)
processor = MplugOwlProcessor(image_processor, tokenizer)

In the readme it was mentioned that the offline demo can be inferenced with only a single 16GB T4 GPU with 8 bits support.
How can I do this in colab?
Thank you!

You can set the load_in_8bit as True to enable 8-bits support. See
https://github.com/X-PLUG/mPLUG-Owl/blob/main/serve/model_worker.py#L46
Note that the 8bit only works for torch.half not work for torch.bfloat16


Fixed installing multiple plugins in TestBootstrapper.php

1. Why is this change necessary?
Fixes Issue #2676  - in TestBoostrapper.php it shows the error
Error in bootstrap script: Symfony\Component\Console\Exception\RuntimeException:
Not enough arguments (missing: "command").
#0 /var/www/html/project/vendor/symfony/console/Input/Input.php(42): Symfony\Component\Console\Input\Input->validate()
#1 /var/www/html/project/vendor/symfony/console/Input/ArrayInput.php(34): Symfony\Component\Console\Input\Input->__construct(Object(Symfony\Component\Console\Input\InputDefinition))
#2 /var/www/html/project/platform/src/Core/TestBootstrapper.php(373): Symfony\Component\Console\Input\ArrayInput->__construct(Array, Object(Symfony\Component\Console\Input\InputDefinition))
#3 /var/www/html/project/platform/src/Core/TestBootstrapper.php(82): Shopware\Core\TestBootstrapper->installPlugins()
...

When having more than 1 plugin in the TestCase
2. What does this change do, exactly?
The TestBootstrapper#installPlugins method makes use of the plugin:install command. Somehow when running that in a loop, the InstallCommand Definition seems to require the "command" argument. I am using an if check on the $installCommand->getDefinition()->getArguments() to check if the InstallCommand instance requires that argument.
3. Describe each step to reproduce the issue or behaviour.
#2676 describes a way to reproduce this: A custom made TestBoostrap.php in a custom plugin. The custom plugin has another plugin in the composer.json requirements.
My TestBoostrap.php looks like this:
use Shopware\Core\TestBootstrapper;

(new TestBootstrapper())
    ->setProjectDir(env('PROJECT_DIR'))
    ->setLoadEnvFile(true)
    ->setForceInstallPlugins(true)
    ->addActivePlugins('FroshPlatformThumbnailProcessor')
    ->addCallingPlugin(__DIR__ . '/composer.json')
    ->bootstrap()
    ->setClassLoader(require __DIR__ . '/vendor/autoload.php')
    ->getClassLoader();

Running a test on PHPUnit with that TestBoostrap causes the error initially.
4. Please link to the relevant issues (if any).
#2676
5. Checklist

[X] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-23635
Please use this issue to track the state of your pull request.

Hey @florianliebig, thanks again for your contribution, your changes are now merged! :tada: :blue_heart:


Fix domain and email whitelist/blacklist matching

The Lua function string.find() with two arguments only does pattern
matching. The way it's used here means the email and domain returned
by Google is interpreted as pattern. If the email contains characters
which have special meaning in patterns (like . and -) this leads
to unwanted behaviour. Either the email doesn't match even it should
and denies access. Or (worse) it matches even it shouldn't and grants
access.
The fix uses all four arguments, the 4th parameter true turns off
pattern matching so the function does a plain substring search.

Thanks! Sorry about the delay.


Character "#" inside quotes is interpreted as comment

commentjson.loads('{ "aaa": "bb#b" }')

or
commentjson.loads('{ "aaa": "b//bb" }')

causes a crash:
Exception thrown by JSON library (json): Unterminated string starting at

    Traceback (most recent call last):
      File "c:\dev\azure-cli-dev-tools\env\lib\site-packages\commentjson\commentjson.py", line 87, in loads
        return json.loads('\n'.join(lines), **kwargs)
      File "C:\Users\zikalino\AppData\Local\Programs\Python\Python37\lib\json\__init__.py", line 348, in loads
        return _default_decoder.decode(s)
      File "C:\Users\zikalino\AppData\Local\Programs\Python\Python37\lib\json\decoder.py", line 337, in decode
        obj, end = self.raw_decode(s, idx=_w(s, 0).end())
      File "C:\Users\zikalino\AppData\Local\Programs\Python\Python37\lib\json\decoder.py", line 353, in raw_decode
        obj, end = self.scan_once(s, idx)
    json.decoder.JSONDecodeError: Unterminated string starting at: line 1 column 10 (char 9)

and this can happen for example in case of Azure ARM Templates, like here:
https://github.com/opencb/opencga/blob/0d4c942d71894a16aa30b498ecb2c77318449cd8/opencga-app/app/scripts/azure/arm/daemonvm/azuredeploy.jsonc#L219

Yeah there are a lot of bugs like that one in the library right now. It was sort of hacked together for some personal use.
Seeing some interest in this, I am working on fixing these bugs. Watch this space out for updates.

This is fixed. You can try the latest version of commentjson which addresses this and a lot of other unhandled cases like this.


add persistent volume to crowdsec chart

This adds a persistent volume to the crowdsec helm chart so the database can be persistent and will survive a pod update. Currently if you destroy the lapi pod, the list of known bouncers is destroyed and you need to register them again.

Hi @flo-mic,
Thanks for your contributions ! It will help a lot.
For this PR, it seems the user need to specify the storageClass name (specific to user context) in it's crowdsec-values.yaml file.
Isn't better to disabled it by default, and if the user wants to enable, he need to specify the storageClass name, size etc ?

@he2ss the user does not need to specify the storageClass. Currently the storageClass is "", which means that kubernetes will use the default storage provider (if some storage is flagged as default with metadata.annotations.storageclass.kubernetes.io/is-default-class: "true"). With the curreent implementation the user also hast he possibility to choose another storage class if he does not want to use the default one. In this case the value "" needs to be replaced with the storage class to use.
Regarding the size, I used 1Gi for the beginning as from my experience the storage requirements are not that high. Nevertheless, most storage provider support volume expansion and will manage an expansion automatically.

@flo-mic, I just tested your changes, it's working quite well. There is still some issues we need to improve if you want to add them :

Add the persistent volume By default without having to enable the dashboard.
Add persistent volume for /etc/crowdsec/ or at least /etc/crowdsec/online_api_credentials.yaml for the online registration.

If you want we can merge this PR and I'll do directly after those changes to improve the persistency.

@he2ss so I just fixed the dashboard part. Sorry did not see this before as my dashboard was always enabled. Also a second PVC for config files is now added and I changed the version number for the release.

I fixed two additional issues which I just notied during my testing.

on an helm upgrade the storage class is not added to the PVC object if it is "" and the deployment failed. This is a bug in helm 3, see https://github.com/helm/charts/issues/20389#issuecomment-583854511
The mounted volume in etc/crowdsec was empty and the container could not start. Therefore the fix mounts the volume in another location, moves all missing files to the mount (if not present) and creates a link between the folders.


Output result question

The total result is like the superposition of each test result. Is this logic theory wrong? For example, I get my DNS lookup time 2ms for the first time, and the second time will be 4ms, the third time will be 6ms, and the total result will be 40ms(count =20). This situation also appears in other indicators. I don't think this output result is reasonable. Look forward to your reply.

Hello, the result is average.  You can think of multiple times as one time， So it's not necessarily the same thing every time


Broadcast alerts

A feature to alert you when something happened on a board in the same broadcast you are watching, but not the currently selected board.
Things that could happen:

someone moved (ply/position change)
evaluation changed substantially (mistake or blunder)
game ended
sacrifices or other interesting meta moves

You should somehow store when you last looked and check from then.


Wrong super methods called in touchesEnded: and touchesCancelled:

- (void)touchesEnded:(NSSet *)touches withEvent:(UIEvent *)event
{
    [super touchesMoved:touches withEvent:event];
    [self touchesEndedOrCancelled];
}

- (void)touchesCancelled:(NSSet *)touches withEvent:(UIEvent *)event
{
    [super touchesMoved:touches withEvent:event];
    [self touchesEndedOrCancelled];
}

These methods both call touchesMoved: instead of their appropriate super method.
If this is intentional, can we document the reason?

Not intentional.

Fixed in 1.3.0


Remove unused variables from connection manager

I don't see these things used anywhere. A most consequential PR.

Failing test on windows


google map resize is not working

I am trying to resize the map like this google.maps.event.trigger(GoogleMaps.maps.exampleMaps , 'resize') is not working. Can you tell me why it is not working

I am also trying  with this code . It is not working for me also.

If you believe this a bug with this package it would be super helpful if you could produce an example application that reproduces the issue, and as a bonus a working example outside of this package!

@mariyaVj and @dburles  I kind of found the solution to it.  I was doing it wrong I had to access the  resize the instance of the map not exapmleMaps. I did it like this
google,maps.event.trigger(GoogleMaps.maps.exampleMaps.instance , 'resize')
and at the end I would like to say that @dburles I mean no offence but please maintain your repo :)

@mariyaVj  and @dburles  I kind of found the solution to it. I was doing it wrong I had to access the resize the instance of the map not exapmleMaps. I did it like this
google,maps.event.trigger(GoogleMaps.maps.exampleMaps.instance , 'resize')
and at the end I would like to say that @dburles  I mean no offence but please maintain your repo :)

thanks @wal-agardezi


⚠ DesiTorrents has degraded performance

In 1b3a8f2, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 3963 ms

Resolved: DesiTorrents performance has improved in cd63ca4.


Corrupted collection by creating indices programtically

Hi, I found a reproducible case where I can corrupt a collection. When I startup, I programtically check the database and create the collection/indices like so (using arangodoclient C#)
db.CreateCollection (collectionName, collectionType, waitForSync: true);
db.Advanced.EnsureIndex (collectionName, indicesData);

In Debug build, it runs fine, the indices is created properly.

In release build, the EnsureIndex will throw
 Exception ServerSystem setup:Input string '0.5' is not a valid integer. Path 'selectivityEstimate', line 1, position 117.
   at Newtonsoft.Json.JsonTextReader.ParseReadNumber(ReadType readType, Char firstChar, Int32 initialPosition)
   at Newtonsoft.Json.JsonTextReader.ParseNumber(ReadType readType)
   at Newtonsoft.Json.JsonTextReader.ReadNumberValue(ReadType readType)
   at Newtonsoft.Json.JsonTextReader.ReadAsInt32()
   at Newtonsoft.Json.JsonReader.ReadForType(JsonContract contract, Boolean hasConverter)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonSerializer.Deserialize[T](JsonReader reader)
   at ArangoDB.Client.Serialization.DocumentSerializer.Deserialize[T](Stream stream) in D:\XXX\ThirdParty\arangoclient.net\src\ArangoDB.Client\Serialization\DocumentSerializer.cs:line 30
   at ArangoDB.Client.Http.HttpCommand.RequestMergedResult[T](Object data) in D:\XXX\ThirdParty\arangoclient.net\src\ArangoDB.Client\Http\HttpCommand.cs:line 196
   at ArangoDB.Client.Advanced.AdvancedOperation.EnsureIndexAsync(String collection, EnsureIndexData data, Action`1 baseResult) in D:\XXX\ThirdParty\arangoclient.net\src\ArangoDB.Client\Advanced\AdvancedOperation.cs:line 82
   at ArangoDB.Client.Utility.TaskUtils.ResultSynchronizer[T](Task`1 task) in D:\XXX\ThirdParty\arangoclient.net\src\ArangoDB.Client\Utility\SynchronizionUtils.cs:line 33
   at ArangoDB.Client.Advanced.AdvancedOperation.EnsureIndex(String collection, EnsureIndexData data, Action`1 baseResult) in D:\XXX\ThirdParty\arangoclient.net\src\ArangoDB.Client\Advanced\AdvancedOperation.cs:line 61

After I run the release build with above error, the error will persist even after switching back to Debug build. I need to delete the collection.

ArangoDB Version:        3.4.4
Storage Engine:          MMFile
Deployment Mode:       Single Server
Deployment Strategy:   Manual Start |
Infrastructure:               own
Operating System:       Centos 7

@imtrobin : it looks like the collection is created fine, but that the c# connector has a problem reading the serialized collection data. If that's true then working with the collection in ArangoDB (e.g. via the web interface - not using it from c#) doesn't exhibit any problems. Is that correct?

It looks like your json parser expects an int, looks more like a c# driver problem to me.

https://github.com/ra0o0f/arangoclient.net/blob/master/src/ArangoDB.Client/Data/EnsureIndexResult.cs#L35. Maybe you can try to switch the type to a floating point type.

Sorry, I think the above fixes it. The error with the collection is another collection with wrong indices, and it's throwing another error.


How can I disable the bottom controls background fade in?

Hi.
I have disabled all controls in the player screen so I can control it using the remote control of the smart tv (Samsung Tizen).
The problem is that the player's bottom controls background still fade in when we move the mouse over the player's screen.
The mouse will be eventually used in TV by connecting a usb wireless mouse.
I searched for a way to disable this bottom controls background but could not find it anywhere.
Thanks in advance for any clue.

Well, you could override the CSS for that element, to make it have no background:
.shaka-scrim-container {
  background: transparent;
}

If that solution won't work for you, or if you don't like how the controls bar continues to fade in and out invisibly, it'd also be possible for us to make a UI configuration value to keep the UI from fading in.


Network API

Hello,
I saw you guys are using the webadmin from Cuberite even though Cuberite has a network API. Was there a reason to choose the webadmin over that?
The documentation for it is here: http://apidocs.cuberite.org/cNetwork.html

@NiLSPACE Oh this should be way better. We didn't notice that when we started Dockercraft (back in June). We should definitely use the network API instead of webadmin.

I believe it was implemented somewhere in January. It's possible you guys saw an old post on the forum where people asked for a network API.

@NiLSPACE Maybe... anyway this is great news! Let's get rid of this weird webadmin usage! :)

I started to work on some implementation using cNetwork plugin. It's going to clean the code a lot! :)

@gdevillele @dave-tucker Just so you know, I'm still working on this one. Implementation should be ok on Dockercraft side. But I still have issues with the TCP connection, and it seems to be due to cNetwork issues... For example, reading in TCP connection is not blocking, cNetwork plugin answers with empty byte arrays when there's nothing to answer. And at some point I get broken pipe errors when go proxy tries to write data.

You might want to report that at Cuberite. The network API isn't used allot, so we don't have allot of bug reports for it. Currently I think the only places where it is used is here and here

@NiLSPACE  Yes, I'm investigating. I wanna be sure it's not because of one of my own mistakes. But I'll report issues to Cuberite and help fixing them as much as I can! Thanks for the links!

Note that the network API is designed to be non-blocking - Cuberite plugins cannot afford to block for IO, because they hold a mutex to the entire world. You'll need to design the protocol to be async-aware. Usually we do that by caching the received responses until they contain enough data to be parsed as a chunk; then process the data and keep the rest in the incoming buffer, etc.
I'm still wondering, with the cNetwork API, is the go proxy still necessary? Shouldn't the plugin itself be capable of doing all the communication on its own?

@madmaxoft Yes I understand all Cuberite plugins have to be non-blocking.
Also yes, theoretically we could implement everything in the Cuberite Docker plugin and get rid of the go proxy. But we have a bunch of very useful and powerful libraries all written in Go to consume Docker features, it would be really hard to re-implement that in Lua/C++... (compose, machine, networking...etc)

There is also a higher level client you can use: https://api.cuberite.org/cUrlClient.html


[vr] Use registry key to get OpenVR extensions list if available.

The current way of querying Vulkan extensions required for OpenVR is problematic:

The current version can work only if the app didn't initialize VR before creating d3d11 device. Otherwise we will teardown the VR connections as the implemented way to check if VR is already initialized doesn't work, duplicated inits always succeed;
Extra tinkering of VR client connection on and off inside the game process is very fragile, we see that native VR client depends on process state a lot. The native VR objects are per process singletons, and currently there is no known way to make sure that the initialized state is completely cleaned up. There is also seems to be some cached state in the vrserver. This problem is not fully solved by removing the extra OpenVR initialization / tear down as the game can do it itself. E. g., this is done by Until You Fall, and it still doesn't work (hangs the whole VR on attempt to render again after second initialization) on AMD (surprisingly, works fine on Nvidia). I have a standalone native OpenVR / Vulkan sample which reproduces this issue without Proton involved. Still, removing the extra initialization induced by us helps some games (at least Left Hand Path), and hopefully makes the thing more reliable. Besides, it shaves off a some time from the games' startup when Steam VR is running.

The idea is to query the extensions on Proton prefix startup from outside of the game process and store the required information in registry. Currently I used Proton's steam.exe helper for that purpose. The keys are initialized asynchronously (to avoid adding potential delays to game process startup) and it may happen the initialization is not completed yet by the time DXVK tries to create device. There is a registry value holding the OpenVR status, the value of 0 means it is not initialized yet, ~0 means VR is not available and 1 means VR is up and running and keys with extensions are valid. The registry key is volatile, so there is no leftover state from previous runs. I am attaching the Proton patch which sets this registry (as it is not in any release yet), and also a patch for wineopenxr which also going to use this (just for reference).
patches.zip

Just changed some C stuff to the C++ equivalents (casts, NULL->nullptr) etc, fine otherwise.

Just changed some C stuff to the C++ equivalents (casts, NUL->nullptr) etc, fine otherwise.

formatting inconsistency,  goto done; could be replaced with just break; here, sprintf ...
I like this PR in many senses.


It seems that regle is not a Nuxt module when adding Nuxt module

I was following the docs here to install Regle as a Nuxt module and I got the following message:
$ nuxi module add regle

WARN  It seems that regle is not a Nuxt module.                                                                                7:36:18 PM

❯ Do you want to continue installing `regle` anyway?
○ Yes / ● No

Perhaps the command in the docs is wrong?
https://regle.vercel.app/introduction/integrations.html#nuxt
I am also not able to find regle on Nuxt modules page:
https://nuxt.com/modules?orderBy=desc&sortBy=createdAt&q=regle
Screenshot:

Yep that's normal, the PR is still in progress on nuxt-modules part! It will work once it's accepted.
https://github.com/nuxt/modules/pull/1118


Network connection leak

Currently substrate is based on libp2p 0.14.0-alpha.1 which has a few issues:

Network opens a lot of TCP connections. After running for a few hours on kusama the number of connections reported by lsof is over 2000.
Each connection preallocates at least 176 kbytes of memory for buffers in the noise protocol.
Which leads to memory usage in hundreds of megabytes.
Each connection spawns a short-lived thread, which leads to a leak in sysinfo crate.

Each of these should be fixed as soon as possible.  Substrate already has a limit for protocol connections. That limit must be enforced as a global limit for TCP connections, including Kademlia and all other protocols.
All buffers should be preallocated to some small size, like 1k. The rest should be allocated lazily.

https://github.com/paritytech/polkadot/pull/810 switched to async-std's master branch which fixes the spawning of a background thread for TcpStream::connect. In the meantime this version has also been published as async-std v1.5.

Network opens a lot of TCP connections. After running for a few hours on kusama the number of connections reported by lsof is over 2000.

I've been debugging this today. It's going rather slowly, as I have to recompile Polkadot every single time.
I created a version of Polkadot where the Kademlia discovery mechanism is running at a quick rate (10 seconds instead of 60 seconds), but the result of discovery are not transmitted to the PSM.
The result is that we constantly stay between 7 and 9 TCP connections (one per bootnode, plus a few extra connections from time to time for Kademlia).
However, when the results of discovery are transmitted to the PSM but the substrate-specific protocol is prevented from opening new connections, the number of TCP connections explodes.
The first test confirms that Kademlia alone works properly and isn't responsible for this explosion.
The second test shows that the problem comes from connections that are maintained alive by the PSM, rather than new connections being open.

Another test where discovery is normal but the PSM rejects all connections and never opens new ones: connections explode.
This is weird, but is a good clue.

I'm still not sure how we reach 2000 connections, but if we increase the reputation penalty for not being responsive, the number of TCP connections is much closer to the number of peers we're connected with for Substrate.
https://github.com/paritytech/substrate/blob/6e0a4877afbe2309e106b84333bb70200531b604/client/peerset/src/lib.rs#L32

The large number of connections has been fixed by #4889 and #5086 and after #5571 we now have Prometheus metrics that lets us track this, meaning that we should be able to notice if that happens again.


createOrder method has been added

added method for: https://cloudpayments.ru/Docs/Api#createOrder

Thanks!

The package has been updated


Documentation: SwiftUI -> Layout Containers -> Layout transitions: GridLayout

As a PO I'd like to have documentation for the GridLayout

We won't support this one


Frequent failures

This may not be the fault of this package, but I frequently lose a job to an HTTP 400 while running this action.
https://github.com/embroider-build/embroider/pull/1001/checks?check_run_id=3863504298
It's sporadic and it's not super common, but when you have enough jobs the chance of hitting it on any given workflow gets quite high.
Better logging of which HTTP request is failing might help, and maybe retry is appropriate.

https://github.com/volta-cli/action/pull/97 is an initial stab at helping out here, we should now have logging to tell us if the failures are happening in tc.downloadTool or in the actual downloading of node / yarn / etc.
I have a few more ideas to implement also:

add caching for the entire $VOLTA_HOME (cache key would be the contents of package.json's volta property) -- this would basically be like what actions/setup-node does for the global yarn/npm package caches;
properly wrap actions/setup-node so that we can avoid copy/pasta of their code (and lagging behind on the features they create -- this will give us nice yarn/npm caching

#97 is published as<EMAIL_ADDRESS>if anyone wants to give it a whirl...

https://github.com/embroider-build/embroider is now updated to v3, so we should hopefully see improvement from #97.

https://github.com/volta-cli/action/pull/101 should also help a bit here (released in v3.0.2). I was able to get to the bottom of at least one of the major causes of request failure that we were hitting.

I have been receiving intermittent "could not unpack" errors on the lerna repo for some time now:
https://github.com/lerna/lerna/actions/runs/4500110940/jobs/7918811928?pr=3603

Volta error: Could not unpack Node v18.15.0

I recommend volta to everyone I interact with, it's so great for local usage. If we can somehow just resolve this weak link then it will basically be flawless in my eyes!
I'm going to try switching to the setup-node action for now because apparently it will check for volta.node in package.json these days

I have been receiving intermittent "could not unpack" errors on the lerna repo for some time now:
https://github.com/lerna/lerna/actions/runs/4500110940/jobs/7918811928?pr=3603

Volta error: Could not unpack Node v18.15.0

I recommend volta to everyone I interact with, it's so great for local usage. If we can somehow just resolve this weak link then it will basically be flawless in my eyes!
I'm going to try switching to the setup-node action for now because apparently it will check for volta.node in package.json these days

We're getting the same issue since a few days ago.

Volta error: Could not unpack Node v18.15.0

Same here. Any updates on this issue @rwjblue?

We too.

FWIW, the Could not unpack Node issue is more likely to be an issue with volta itself than with this action (since at that point in the runtime, we are calling volta setup directly and the action isn't in the picture...)

The "could not unpack" issue that we continue to see is almost certainly a problem with @actions/tool-cache. It's not clear that this action is really using that cache correctly, as we never see it hit the cache, and overwhelming the network filesystem that backs the cache seems to be the likely cause of the failures we're seeing.

It still reproducible in 5%-10% workflow runs.


Add user documents on readme.io

https://dash.readme.io/project/reactcms/v1.0/dashboard

Needed for Release 1.0


regress-vlg_br_gh199b_iv

Surelog issue created: https://github.com/chipsalliance/Surelog/issues/2341

Issue fixed.


Batch Actions dropdown opened and button disabled on Index page

Have attempted to apply fix as described in Active Admin issues but it still doesn't rectify the issue in this theme.

fixed, conflict with activeadmin/base - removed //=require activeadmin/base completely from active_admin.js file

Custom JS files are not being loaded through arctic_admin/base, although they are if I remove arctic_admin/base and revert to active_admin/base in the active_admin.js file

Which version you use ? activeadmin ? arcticadmin ?

arctic_admin (1.4.3), activeadmin (1.1.0)
All I have in my active_admin.js file is
//= require arctic_admin/base
//= require chartkick

@cle61, have you been able to investigate this yet? I've still not found a way to make it work

sorry, I do not have a lot of time ...
i am open to any merge request if you find the problem

It was a versioning issue - updated from AA 1.1 to 1.2.1 fixed


Layer_by_layer Boosting Tree

<emThe issue related to the performance of the BoostedTree classifier in terms of accuracy.
System information

OS Platform and Distribution: Linux Ubuntu 16.04, Windows 10
TensorFlow installed from (source or binary): pip install TensorFlow
TensorFlow version (use command below): 2.4.1
Python version: 3.6

** Describe the model**
BoostedTree classifier is a model introduced by N Ponomareva, T Colthurst, G Hendry.2017.
It is built over the Xgboost idea and learning is done through building one layer of decision tree regressor over N boosting iteration.
Describe the current behavior
Build one tree for each boosting epoch
Describe the expected behavior
As mentioned in its paper, I want to build one tree for all Gradient Boosting Ensemble, and each boosting iteration would be layers of one tree.
Standalone code to reproduce the issue
You may find an example of the model, here

Hi!@samanemami ,
This does not  seem to be  a bug or feature request, for any further queries you may open this issue in TF discuss forum as there is a larger community there.Thank you!

Thank you @mohantym
But my question is about the library performance, as I could not find any related method in the BoostedTree library in the TensorFlow repository regarding the layer_by_layer model.

Hi @Saduf2019  , Could you please look into this issue?

Hi @Saduf2019
More detail about the method I am looking for in the BoostedTree library
I am looking for the LAYER-BY-LAYER BOOSTING Tree which has explained in SECTION V of the paper.
But when I reviewed the (library)[https://github.com/tensorflow/estimator/blob/master/tensorflow_estimator/python/estimator/canned/boosted_trees.py#L1933-L2102], I could not find any related method, and it works just like other Gradient boosting models.

@Saduf2019
This repository(https://git.kot.tools/nk2/syntaxnet_rus/-/blob/caae66a144f1237eb6b5c19fa00c317ca3bed09c/tensorflow/tensorflow/contrib/boosted_trees/examples/mnist.py#L93) is an old lib from tf.BoosteTree, which remains.
This is the exact code which the paper used, I am looking for the update of this code in your GitHub repository which includes all the paper methods.

Hi @ymodak
May I ask if you found anything?


[FIX] email template - missing apostrophe

FIX
Objective:

apostrophe missing

Already approved, just updated filepath name from 'src' to '../'


Integrating Tesseract OCR into SWT demo

Hey I tried out your OCR demo at https://kostasx.github.io/ccv-js/ and it's pretty cool.
I actually had very similar code that was in the SWT demo itself but I ended up removing it. The reason was because I also wanted to support live webcam (which works today if you change source to "WEBCAM") but tesseract.js consistently crashes the tab if you don't manage the promises you fire off for each frame.
This added some complexity and didn't work well (queuing up promises that you can't cancel and permanently lagging behind the current frame, etc etc). So I removed it to focus on demoing the ccv stuff (even though swt does work nicely with tesseract.js).
For the same reason I probably don't want to merge this either. Thanks for the interest though!

Fair enough.
Amazing library by the way.

Actually I thought about it some more and there's no reason why I can't just disable tesseract for videos. I made a change that's largely based on yours (including using your imgur image!). The demo should now be updated.
Thanks!

👍
That Aha moment when you realize that you've been trying too hard to stick with vanilla JS, and as a result you forget about all these jQuery gems like $.getScript. What was I thinking? :)


What does WARNING:dedupe.training:Ran out of predicates mean?

I am attempting to train a deduper using data that has been manually clustered. I am just using a single field with the 'Name' type.
train_names is a 1500 item long dictionary with the form:
  'Id': '1',
  'name': 'ACME, LLC'},...```

The code I am running for the training is:
```fields = [
        {'field' : 'name', 'type': 'Name'}
        ]
deduper = dedupe.Dedupe(fields)

deduper.markPairs(dedupe.trainingDataDedupe(train_names,'Cluster ID'))

deduper.train()```

After running ```dedupe.train()```  I get the following error:
``` WARNING:dedupe.training:Ran out of predicates ```

What does this warning mean and do I need to do anything to remedy it, or can I carry on despite it?

You can carry on.
Dedupe tries to find blocking rules that will work well with your data. Sometimes it can't find great ones, and you'll get this warning. It means that there are some pairs of true records that you dedupe may never compare.

Thanks for the clarification. Is there any chance that failing to find good blocking rules on the training set would slow things down in finding a threshold or matching on a test dataset?

Yes. If you learn liberal blocking rules you will make many more
comparisons.
On Thu, Oct 1, 2015 at 8:29 AM Dylan Nielson<EMAIL_ADDRESS>wrote:

Thanks for the clarification. Is there any chance that failing to find
good blocking rules on the training set would slow things down in finding a
threshold or matching on a test dataset?
—
Reply to this email directly or view it on GitHub
https://github.com/datamade/dedupe/issues/415#issuecomment-144726186.


[Feature] Replace wbv0 ...url_for with wbv1 [OSF-8696]

Purpose
WBV0 is kill.

Changes
Replace calls to waterbutler_url_for with calls to waterbutler_api_url_for

Side effects
Removed skip logic from test that said "Unskip when figshare addon is implemented" from... 10 months ago.

Ticket
https://openscience.atlassian.net/browse/OSF-8696
QA notes

Make sure you can still upload conference files via email. I only could half test this since I'm not sure how to receive emails locally.
Make sure that files and folders (and addons) are still archived to osfstorage properly when registered

@alexschiller Conflicts


[GSSoC'23] Distance Convertor Project Ready

Title and Issue number

Title - Distance Convertor
Issue Number - 3063
Close #3063

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @Saurabh-kumar09 @pranjay-poddar sir, please review this pull request


Provide Format-Checklist parameter support

Fixes #29
I agree that I'm not sure what the use case it, but I needed to get the tests working to work on something else and this was low hanging fruit to test the tests.

I can't imagine the linting checks ever passed. I can probably fix those as well within this PR if you like.


Psmove service not recognizing Ipisoft PSEYEDRIVER

I've Installed IPisoft pseye driver and when I open PSMoveServiceConfig and go to tracker settings and it says "No trackers found"

[Read all!] To @Expora9 Hey there you need to install usbdevew to look at the name of your Ps3 eye camera. Then you need to install Zadig, witch install the right drivers! Select the ps3 eye camera in zadig, click LibUsb 0.2. (Something like that) then your psmovebrigde or Psmoveservice finally it will find your trackers. If using steamVR go to: C:\Program Files (x86)\Steam\steamapps\common\SteamVR\resources\settings\default
then change the follwing things: activeMultipleDrivers: false (replace it with true)
Next change this to true.” Camera” disable/false (something like that) change that to true. Download zadig and usbdeview for free (you will not get any virus or something else)

I've Installed IPisoft pseye driver and when I open PSMoveServiceConfig and go to tracker settings and it says "No trackers found"

[Read all!] To @Expora9 Hey there you need to install usbdevew to look at the name of your Ps3 eye camera. Then you need to install Zadig, witch install the right drivers! Select the ps3 eye camera in zadig, click LibUsb 0.2. (Something like that) then your psmovebrigde or Psmoveservice finally it will find your trackers. If using steamVR go to: C:\Program Files (x86)\Steam\steamapps\common\SteamVR\resources\settings\default
then change the follwing things: activeMultipleDrivers: false (replace it with true)
Next change this to true.” Camera” disable/false (something like that) change that to true. Download zadig and usbdeview for free (you will not get any virus or something else)

Thanks I will try this method

Good if it don’t work I could guide you.


Fix build for HAVE_LIBPAM=0

This adds a missing #includethat fixes the build for the case where
'HAVE_LIBPAM' is 0.
The following error occurs without this:
[...]
Making all in scheduler...
Compiling auth.c...
auth.c:2017:5: error: unknown type name '_cups_md5_state_t'; did you mean '_cups_message_t'?
    _cups_md5_state_t   state;          /* Primary MD5 state info */
    ^~~~~~~~~~~~~~~~~
    _cups_message_t
../cups/language-private.h:52:3: note: '_cups_message_t' declared here
} _cups_message_t;
  ^
auth.c:2018:5: error: unknown type name '_cups_md5_state_t'; did you mean '_cups_message_t'?
    _cups_md5_state_t   state2;         /* Secondary MD5 state info */
    ^~~~~~~~~~~~~~~~~
    _cups_message_t
../cups/language-private.h:52:3: note: '_cups_message_t' declared here
} _cups_message_t;
  ^
auth.c:2038:5: warning: implicit declaration of function '_cupsMD5Init' is invalid in C99 [-Wimplicit-function-declaration]
    _cupsMD5Init(&state);
    ^
auth.c:2039:5: warning: implicit declaration of function '_cupsMD5Append' is invalid in C99 [-Wimplicit-function-declaration]
    _cupsMD5Append(&state, (unsigned char *)pw, pwlen);
    ^
auth.c:2046:5: warning: implicit declaration of function '_cupsMD5Finish' is invalid in C99 [-Wimplicit-function-declaration]
    _cupsMD5Finish(&state2, digest);
    ^
3 warnings and 2 errors generated.
../Makedefs:260: recipe for target 'auth.o' failed
make[1]: *** [auth.o] Error 1
Makefile:25: recipe for target 'all' failed
make: *** [all] Error 1

That header actually can’t be used anymore since the CUPS MD5 implementation is only used on legacy platforms. PAM will soon be required for authentication, period.

That's fine for me; I just came across that build failure when doing a test build to have a look at something else.
I'm totally satisfied when instead of merging this, PAM is made mandatory and a configure error is caused when the PAM development files are not installed (rather than the build error I encountered).

OK, I opted to just remove the MD5 password code and use crypt() by itself without PAM for now - crypt() is supposed to handle the supported password hashes anyways, and systems have been using SHA and other hashes for a long time as well...

In the 2.2. branch, it looks like there is still a reference to cups_md5_state_t in scheduler/auth.c if HAVE_LIBPAM is 0. See lines 2021-2. Is the backport complete?

No, we didn't remove the code from 2.2.x.
Tracking that fix with Issue #5283...


Unity Editor Freezes (Firebase SDK 6.2.2)

After upgrading Firebase from 6.1.1 to 6.2.2 (almost all firebase packages),
When calling GetValueAsync() In verbose log mode,
the console prints "[conn_0] websocket message received" endlessly.
Even when we stop the game, the console keeps printing the message.
When we reload the scripts / change the scripts / replay the game or anything that causes Unity to 'Reload Assemblies', Unity freezes (probably because the previous connection is still ongoing, and that might cause the freezes.
It also happens with a new project, in different Unity version, so it doesn't relate to our code or Unity editor version.

After turning on 'Verbose' logging, the console keeps printing endless messages of: "websocket message received", even when the game isn't running.
maybe that's part of the issue, then when I press 'Play' again, finally the websocket messages stops (also console clears on play) and it freezes as usual.

Hello. I had the same problem (with different versions tho). You can check out the Unity Editor log to see what possibly goes wrong:
macOS		~/Library/Logs/Unity/Editor.log
Windows 	C:\Users\_YOUR-USERNAME_\AppData\Local\Unity\Editor\Editor.log

In my case I solved the issue by re-importing all the Firebase packages I use with consistent versions.
Do not use some packages in 6.1.1 and others in 6.2.2. Clean the project, run the dependency resolver and if needed run Assets -> Reimport All & Restart the editor.
As a workaround: Some of the APIs are working in the editor for easier testing, but some files needed may be corrupted due to the update. If the reimports doesn't help you, just find which boilerplate code causes problems and comment it during the editor work. In my case I had a RemoteConfig initialization, but the Remote Config Editor dll was missing. As temprorary workaround I commented my remote config initialization during work in the editor. Then uncommented it for native Android/iOS builds. Later found out the actual issue and reimported the package which restored the missing dll.
Hopefully this is helpful ;)

Hi @yakirbu,
Unfortunately I was not able to reproduce this issue.  Could you please alter testapp/database to reproduce the issue, and post the changes you made?

Hello. I had the same problem (with different versions tho). You can check out the Unity Editor log to see what possibly goes wrong:
macOS		~/Library/Logs/Unity/Editor.log
Windows 	C:\Users\_YOUR-USERNAME_\AppData\Local\Unity\Editor\Editor.log

In my case I solved the issue by re-importing all the Firebase packages I use with consistent versions.
Do not use some packages in 6.1.1 and others in 6.2.2. Clean the project, run the dependency resolver and if needed run Assets -> Reimport All & Restart the editor.
As a workaround: Some of the APIs are working in the editor for easier testing, but some files needed may be corrupted due to the update. If the reimports doesn't help you, just find which boilerplate code causes problems and comment it during the editor work. In my case I had a RemoteConfig initialization, but the Remote Config Editor dll was missing. As temprorary workaround I commented my remote config initialization during work in the editor. Then uncommented it for native Android/iOS builds. Later found out the actual issue and reimported the package which restored the missing dll.
Hopefully this is helpful ;)

Unfortunately, I'm not using any packages from 6.1.1 (removed all completely and validated)
Also, reloading all assets didn't help.
I'm not missing any dll, after using debugger, I noticed that my it stops when it reaches the point where I call 'GetValueAsync()' to retrieve data from Realtime-Database.
As I mentioned earlier, when using verbose log mode, it keeps printing "[conn_0] websocket message received" endlessly even after stopping the game.

@yakirbu can you post the code where you call GetValueAsync?

@yakirbu can you post the code where you call GetValueAsync?

public async Task<T> GetData<T>(DatabaseReference dbRef, string child = null, string orderByChild = null, string equalTo = null)
    {
        Query tempRef = dbRef;
        if (!string.IsNullOrEmpty(child))
            tempRef = tempRef.Reference.Child(child);
        if (!string.IsNullOrEmpty(orderByChild))
            tempRef = tempRef.OrderByChild(orderByChild);
        if (!string.IsNullOrEmpty(equalTo))
            tempRef = tempRef.EqualTo(equalTo);
        var tempTask = tempRef.GetValueAsync();
        var snap = await tempTask;
        if (snap != null && snap.ChildrenCount > 0)
        {
            T val = default;
            foreach (var snapChild in snap.Children)
            {
                var json = snapChild.GetRawJsonValue();
                if(json[0] != '{')
                {
                    json = "{\"" + snapChild.Key + "\":" + json + "}";
                }
                val = GetObjFromJSON<T>(json);
                var keyField = val.GetType().GetField("key");
                if (keyField != null)
                    keyField.SetValue(val, snapChild.Key);
            }
            return val;
        }
        Debug.Log("error - no children");
        return default;
    }

Thanks for that sample @yakirbu.
Can you give us some information on your current setup?
Unity version.
Unity Editor platform (mac/win/linux).
Unity target (android/ios/other). I know that this is an editor issue, but there could still could be some configuration changes according to your target platform.
Exceptions are used to indicate errors in data retrieval. Can you let us know if you see any of those?
You may see in our examples/docs that we handle these with task.IsFaulted, but an exception will be raised instead that you should try to capture with a try/catch block if you don't check this field.
If you don't mind, it would help us a ton if you could get us instructions for how to reproduce this issue in the test app.
Another thing that would be super helpful if you could try is trying to take more direct control of the FirebaseDatabase.DefaultInstance. Internal to Firebase, this is currently a weak reference (although it should stay around for the duration of your query). If you have any sort of manager class (say, where you call CheckAndFixDependenciesAsync), it would be really helpful to know if keeping a reference there makes your issue go away.
Similarly, if you're already keeping a reference to FirebaseDatabase.DefaultInstance around, try setting it to null in OnDestroy. Ex (I'm writing this inline, so there may be a typo or two):
public class DatabaseManager: MonoBehaviour {
    FirebaseDatabase _db;
    void Start() {
        _db = FirebaseDatabase.DefaultInstance;
    }

    void OnDestroy() {
        _db = null;
    }
}

This changes behavior a little because in the UnityEditor, all the native plugins are kept around between play cycles (when you stop playing the game, the Database stays in its current state). Clearing all references to the database might clear it out.
Now, this might not work since you have an active query going on (it seem). What you can also do is try putting in a FirebaseDatabase.DefaultInstance.Dispose(); in a manager class (say in an OnApplicationQuit() callback). This should not be necessary, but let us know if it does fix your issue!
I look forward to your response!
--Patrick

Hi,
So I'm glad to say that this issue is now resolved.
I've managed to fix it by doing a thorough clean-up to my project (Including Library folder and unused plugins).
Can't really say what caused the issue, but I'm glad it is no longer happening.


[gardening][NSGeometry] Move operators into types

@swift-ci please test and merge

@swift-ci please test and merge