fatal: The remote end hung up unexpectedly with multibuildpack

I am starting to move apps to from deis v1 to deis v2 and am stuck with an app that uses heroku multi buildpack.
$ deis config
[...]
BUILDPACK_URL               https://github.com/heroku/heroku-buildpack-multi

In my repo, I have the following .buildpacks file:
https://github.com/heroku/heroku-buildpack-ruby.git
https://github.com/heroku/heroku-buildpack-nodejs.git

The build is successful on deis v1 but fails on deis v2 with a not really helpful error message:
       Resolving node version 6.x.x via semver.io...
       Downloading and installing node 6.3.1...
Connection to deis-builder.deis2.domain.com closed by remote host.
fatal: The remote end hung up unexpectedly
fatal: The remote end hung up unexpectedly

Here's the full log:
$ git push deis                                                                                master
Counting objects: 9155, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (3176/3176), done.
Writing objects: 100% (9155/9155), 56.81 MiB | 1.42 MiB/s, done.
Total 9155 (delta 6512), reused 8080 (delta 5760)
Starting build... but first, coffee!
-----> Fetching custom buildpack
-----> Multipack app detected
       =====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-ruby.git
       =====> Detected Framework: Ruby
-----> Compiling Ruby
-----> Using Ruby version: ruby-2.2.2
-----> Installing dependencies using bundler 1.12.5
       Running: bundle install --without development:test --path vendor/bundle --binstubs vendor/bundle/bin -j4 --deployment
       Fetching gem metadata from https://rubygems.org/
       Fetching version metadata from https://rubygems.org/
       Fetching dependency metadata from https://rubygems.org/
       Installing colorator 0.1
       Installing sass 3.4.21
       Installing ffi 1.9.10 with native extensions
       Installing rb-fsevent 0.9.7
       Installing kramdown 1.10.0
       Installing liquid 3.0.6
       Installing mercenary 0.3.5
       Installing rouge 1.10.1
       Installing safe_yaml 1.0.4
       Using bundler 1.12.5
       Installing jekyll-sass-converter 1.4.0
       Installing rb-inotify 0.9.7
       Installing listen 3.0.6
       Installing jekyll-watch 1.3.1
       Installing jekyll 3.1.2
       Bundle complete! 1 Gemfile dependency, 15 gems now installed.
       Gems in the groups development and test were not installed.
       Bundled gems are installed into ./vendor/bundle.
       Bundle completed (26.25s)
       Cleaning up the bundler cache.

       ###### WARNING:
       No Procfile detected, using the default web server.
       We recommend explicitly declaring how to boot your server process via a Procfile.
       https://devcenter.heroku.com/articles/ruby-default-web-server

       =====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-nodejs.git
       =====> Detected Framework: Node.js

-----> Creating runtime environment

       NPM_CONFIG_LOGLEVEL=error
       NPM_CONFIG_PRODUCTION=false
       NODE_ENV=production
       NODE_MODULES_CACHE=true

-----> Installing binaries
       engines.node (package.json):  6.x.x
       engines.npm (package.json):   3.8.x

       Resolving node version 6.x.x via semver.io...
       Downloading and installing node 6.3.1...
Connection to deis-builder.deis2.domain.com closed by remote host.
fatal: The remote end hung up unexpectedly
fatal: The remote end hung up unexpectedly

I've succesfuly built and deployed regular Node Procfile apps so I assume this is an issue with the multi buildpack.

The actual issue was that the AWS ELB through which git requests transit had a too low time out and would disconnect me if the build took longer than 1minute. This is explained here actually: https://deis.com/docs/workflow/managing-workflow/configuring-load-balancers/#idle-connection-timeouts
Fixed with:
# find elb name with: aws elb describe-load-balancers
ELB_NAME="a5e901f96530c11e6a5e2068755380ef"
aws elb modify-load-balancer-attributes --load-balancer-name "$ELB_NAME" --load-balancer-attributes "{\"ConnectionSettings\":{\"IdleTimeout\":1200}}"

Related: https://github.com/deis/builder/issues/140#issuecomment-178167749


Error: ImportError: DLL load failed while importing fraysum: The specified module could not be found.

Hello, I installed pyraysum with:
conda create -n prs python fortran-compiler obspy -c conda-forge
conda activate prs
conda install jupyter
pip install pyraysum

When I try to run anything, for example:
from pyraysum import Model, Geometry, Control, run

# Build a 1-layer-over-half-space subsurface model
model = Model(
    thickn=[32000, 0],  # m; half-space thickness is irrelevant
    rho=[2800, 3600],  # kg/m^3
    vp=[6400, 8100],  # m/s
    vs=[3600, 4650],  # m/s
)
model.plot()

I get:
ImportError                               Traceback (most recent call last)
Cell In[1], line 1
----> 1 from pyraysum import Model, Geometry, Control, run
      3 # Build a 1-layer-over-half-space subsurface model
      4 model = Model(
      5     thickn=[32000, 0],  # m; half-space thickness is irrelevant
      6     rho=[2800, 3600],  # kg/m^3
      7     vp=[6400, 8100],  # m/s
      8     vs=[3600, 4650],  # m/s
      9 )

File ~\.conda\envs\prs\lib\site-packages\pyraysum\__init__.py:25
      1 # Copyright 2022 Wasja Bloch, Pascal Audet
      2
      3 # This file is part of PyRaysum.
   (...)
     20 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
     21 # SOFTWARE.
     23 __version__ = '1.0.0'
---> 25 from .prs import Model, Geometry, Control, Result, run

File ~\.conda\envs\prs\lib\site-packages\pyraysum\prs.py:32
     30 from numpy.fft import fft, ifft, fftshift
     31 from copy import deepcopy
---> 32 import fraysum
     34 from pyraysum import plot
     35 from pyraysum.frs import read_arrivals, read_traces, _phnames

ImportError: DLL load failed while importing fraysum: The specified module could not be found.

And I cannot find fraysum anywhere.
Thank you for your help!

Hi @tbelinic
I'm sorry to hear that the installation process did not work for you. In fact, in a fresh install, I was unable to install jupyter next to pyraysum, due to conflicts between the fortran-compiler and jupyter. Can you try to execute the above comands in a python console?

There has been a backward incompatibility with recent f2py versions. For the time being, I recommend installing the package with these version constraints:
conda create -n prs python<3.12 numpy<1.23 setuptools<60.0 fortran-compiler obspy -c conda-forge
Please test if this resolves your issue and open a new issue if the problem persists.
Thank you for reporting!


Add checkbox for adding a time offset to lookback time ranges

Add checkbox for adding a time offset to lookback time ranges
Context
Jaeger typically surfaces incomplete traces when a user searches with the default lookback option. To improve the general user experience of first time users, we add a default offset of 5 minutes to the lookback to surface more complete traces.

Jaeger Lookback Offset switch with popover:


[Package Request]: ParaView 5.11.1

How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: https://www.paraview.org/paraview-downloads/download.php?submit=Download&version=v5.11&type=binary&os=Windows&downloadFile=ParaView-5.11.1-Windows-Python3.9-msvc2017-AMD64.msi
Publisher: kitware Inc.
Package Name: ParaView-5.11.1-Windows-Python3.9-msvc2017-AMD64
Description: ParaView is the world’s leading open source post-processing visualization engine. It integrates with your existing tools and workflows, allowing you to build visualizations to analyze data quickly. With its open, flexible, and intuitive user interface, you can analyze extremely large datasets interactively in 3D or programmatically using ParaView’s batch processing.
Package Version: 5.11.1
Installer URL: ?

@hedegaardDK The requested version (v5.11.1) of ParaView is already available as KitWare.ParaView. Above attached PR is a newer version of ParaView.


After combination of setEdge and select by rid index is broken down

OrientDB Version: 3.0.0_RC2
Java Version: 9.0.4
OS: ubuntu docker image under Windows 10
Expected behavior
select by value that is exactly equal with just added should return corresponding vertex
Actual behavior
select by value using eqiuals operation does not find expected vertex. select using 'like' comparision finds it. And comenting out some seemingly irrelevant code lines changes behavior. Details are in comment lines of attached source.
Steps to reproduce
Select2.java.txt

Looks like tx visibility issue

Hi @sreznick,
Thank you for reporting this I could reproduce the problem and fix it.
Regards


934130 and 934131: not required base64 tfunc

Rules 934130 and 934131, created in https://github.com/coreruleset/coreruleset/pull/2411, currently utilize the t:base64Decode tfunc. However, we spot some issues during "make regression tests works for nginx", revealing some problems with attempting to decode the payload from a base64 string.
Additionally:

ModSecurity v2 and v3 uses different libraries for base64 decoding, leading to divergent behaviors.
The use of the base64 encoding approach is inconsistent on CRS, it specifically focused on matching a prototype pollution payload base64 encoded, and we're not following this approach elsewhere. Why only here?
Testing has shown that base64 decoding can fail, even with the "ext" version, resulting in an empty variable that essentially equals to a bypass.

Proposal:
Considering the aforementioned issues, I propose the following actions:

Remove the t:base64Decode Tfunc from Rules 934130 and 934131.
Eliminate all related tests associated with the base64 decoding approach.

Opinions?

@theMiddleBlue Is this related to the anti-evasion work done / false negatives reported in #3311, #3182, #3203?

it's more about the base64 function already present on those rules before the anti-evasion work you mention I guess

But we added Base64 anti-evasion decoding to  934100, 934101, 934130, 934131, and 934160. Is the originally reported false negative no longer considered valid? And we should remove Base64 decoding from all of the rules that were modified?

Can we get examples where the decoding fails with the ext version?

at the moment, on v3, basically every base64 string that contains a + fails
that because urldecode convert + into a whitespace and the base64decodeext function on v3 isn't happy with it.
but again, I think that "the base64 decoding approach" used only here doesn't make any sense

suggested approach: https://github.com/coreruleset/coreruleset/pull/3378

Can we close this on? https://github.com/coreruleset/coreruleset/pull/3378 has been merged 🤔

Yes.


[SPARK-19443][SQL] The function to generate constraints takes too long when the query plan grows continuously

What changes were proposed in this pull request?
This issue is originally reported and discussed at http://apache-spark-developers-list.1001551.n3.nabble.com/SQL-ML-Pipeline-performance-regression-between-1-6-and-2-x-tc20803.html
When run a ML Pipeline with many stages, during the iterative updating to Dataset , it is observed the it takes longer time to finish the fit and transform as the query plan grows continuously.
The example code show as the following in benchmark.
Specially, the time spent on preparing optimized plan in current branch (74294 ms) is much higher than 1.6 (292 ms). Actually, the time is spent mostly on generating query plan's constraints during few optimization rules.
getAliasedConstraints is found to be a function costing most of the running time.
This patch tries to rewrite getAliasedConstraints. After this patch, the time to preparing optimized plan is reduced significantly from 74294 ms to 2573 ms.
Benchmark
Run the following codes locally.
import org.apache.spark.ml.{Pipeline, PipelineStage}
import org.apache.spark.ml.feature.{OneHotEncoder, StringIndexer, VectorAssembler}

val df = (1 to 40).foldLeft(Seq((1, "foo"), (2, "bar"), (3, "baz")).toDF("id", "x0"))((df, i) => df.withColumn(s"x$i", $"x0"))

val indexers = df.columns.tail.map(c => new StringIndexer()
  .setInputCol(c)
  .setOutputCol(s"${c}_indexed")
  .setHandleInvalid("skip"))

val encoders = indexers.map(indexer => new OneHotEncoder()
  .setInputCol(indexer.getOutputCol)
  .setOutputCol(s"${indexer.getOutputCol}_encoded")
  .setDropLast(true))

val stages: Array[PipelineStage] = indexers ++ encoders
val pipeline = new Pipeline().setStages(stages)

val startTime = System.nanoTime
pipeline.fit(df).transform(df).show
val endTime = System.nanoTime

Before this patch: 1786001 ms
After this patch: 48959 ms
How was this patch tested?
Jenkins tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #72303 has started for PR 16785 at commit b4e514a.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72303/
Test FAILed.

retest this please.

Test build #72305 has started for PR 16785 at commit b4e514a.

Test build #72305 has finished for PR 16785 at commit b4e514a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72305/
Test FAILed.

The rewritten logic is not correct. I am working to improve this with other approach.

Test build #72625 has started for PR 16785 at commit 8c98a5c.

I don't find a way to improve getAliasedConstraints significantly by re-writing its logic. The current way to improve its performance is using parallel collection to do the transformation in parallel. It can cut the running time by half (see benchmark in the pr description), but the running time (13.5 secs) is still too long compared with 1.6.
We may consider #16775 which is an another solution to fix this issue by checkpointing datasets for pipelines of long stages, or both of them.

since this change is related to SQL, cc @cloud-fan @hvanhovell

Test build #72625 has finished for PR 16785 at commit 8c98a5c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72625/
Test PASSed.

@viirya this looks like a very big hammer to solve this problem. Can't we try a different approach?
I think we should try to avoid optimizing already optimized code snippets, you might be able to do this using some kind of a fence. It would even be better if we would have a recursive node.

@hvanhovell Yeah, I think so. As in previous comment, I don't find a way now to improve getAliasedConstraints significantly by re-writing its logic without parallel collection.
We may consider #16775 which is an another solution to fix this issue by checkpointing datasets for pipelines of long stages.

can we consider this in a higher-level of view instead of focusing on the method getAliasedConstraints? Maybe there is a way to do constraint propagation faster.

@cloud-fan yeah, i agreed with you and @hvanhovell.
For too slow constraint propagation, except for attacking getAliasedConstraints like this change, maybe we can have other way to improve the process doing constraint propagation.
If we can't, for such long lineages, I think we should use checkpointing to fix it like #16775.

this looks like a very big hammer to solve this problem. Can't we try a different approach?
I think we should try to avoid optimizing already optimized code snippets, you might be able to do this using some kind of a fence. It would even be better if we would have a recursive node.

@cloud-fan @hvanhovell Ok. I've figured out to add a check to reduce the candidates of aliased constraints. It can achieve same speed-up (cut of half running time in benchmark) without parallel collection hammer.
Can you have time to look at it? Thanks.

Test build #73034 has started for PR 16785 at commit 4ba93fe.

Test build #73035 has started for PR 16785 at commit 278c31c.

Test build #73034 has finished for PR 16785 at commit 4ba93fe.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73034/
Test PASSed.

Test build #73035 has finished for PR 16785 at commit 278c31c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73035/
Test PASSed.

cc @sameeragarwal


Multi-VC support - Part 1

Code changes for  below changes

Send hostname from each node.
If config file is not present then Initialize WorkerNode
Modified gcfg config file read struct to make use of new format
CloudProvider calls each provider when node is registered / unregistered

Testing done:

After creating kubernetes cluster on vsphere, manually changed hostname and VMName.
After rebooting workernode, I could see that new hostname node got registered
For new node addition, on master verified that NodeRegistered interfaced got called
On one of the worker node modified  /etc/systemd/system/multi-user.target.wants/kubelet.service and removed cloud-config parameter to kubectl and restarted kubelet. Node was shown in ReadyState once kubelet was started.

Below is the sample output after all above changes :
kubectl get nodes
NAME          STATUS                     AGE       VERSION
master        Ready,SchedulingDisabled   1d        v1.6.0-alpha.0.13043+2211945f3cad85
node2         Ready                      1d        v1.6.0-alpha.0.13043+2211945f3cad85
node3         Ready                      1d        v1.6.0-alpha.0.13043+2211945f3cad85
node4         Ready                      1d        v1.6.0-alpha.0.13043+2211945f3cad85
rohit-node1   Ready                      18m       v1.6.0-alpha.0.13046+389d9886d96fd4

@divyenpatel, @luomiao, @tusharnt, @BaluDontu, @SandeepPissay

LGTM


Moderation

@theminecoder as long as there's no GUI or other bukkit only things all commands can be done via bungee

Yes but what about anti cheat? That would need access to bukkit side bans, no?

And to be fair, why force permission on bungee? I have built networks where all the permissions and command are handled bukkit side and to be honest I find it so much easier to manage

@theminecoder yeah it would probably be best to manage it bukkit side.  Bukkit-side banning utils/commands are fine, and would still be of use

So I think the best way to handle this is commands bukkit side, and database checks on login bungee side? Reopening

@error-dan-overflow Please fix this and split with API.

Ready to merge @erosemberg

@error-dan-overflow Fix issues and i will merge.

@erosemberg My bad, codacy didn't show up at the time of the comment. Will do.

@erosemberg Issues are fixed, but for some reason codacy still thinks some are there...

Yeh I might have to disable the check on that :/
Sent from my iPhone
On 11 May 2016, at 10:39 AM, Dan Beneventano<EMAIL_ADDRESS>wrote:
@erosemberg https://github.com/erosemberg Issues are fixed, but for some
reason codacy still thinks some are there...
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/CraftTogether/CraftCore/pull/7#issuecomment-218332268


Users/catalin andronie/brokers post audio transcription

Closes #210
Request made in Postman:

Request made via test code:

This PR is:

Mixing Brokers with Services
Doesn't follow the proper naming conventions for PRs
Doesn't cover all the tests

Rejected.

This PR is:

Mixing Brokers with Services

Doesn't follow the proper naming conventions for PRs

Doesn't cover all the tests

Rejected.

Yep, it looks like I did not commit the entire code 🤦♂. Don't understand how that heppended. I'll check and create a new PR soon.


Link to User Install wrong

The link points to VSCode's user installer.
One might guess this page was copied from that documentation and not tested.

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

ID: 811ed1f4-1c78-aa1b-e752-307250ca8eb7
Version Independent ID: f36629bf-b2e6-4993-7c77-32fa311ea942
Content: Download and install - Azure Data Studio
Content Source: docs/azure-data-studio/download-azure-data-studio.md
Product: sql
Technology: azure-data-studio
GitHub Login: @markingmyname
Microsoft Alias: maghan

@jimhess -- thank you for your feedback.
@markingmyname -- please look into this issue #4356.

@jimhess
I am finding the User Installer to download Azure Data Studio.
Are you not being prompted to download this file - azuredatastudio-windows-user-setup-1.16.0.exe?

"Get Azure Data Studio for Windows" section has a link which points to VSCode page "User setup for Windows", which in turn has a link to VSCodes user mode installer, not Azure Data Studio's.

@jimhess
Thank you for clarifying.
We are looking into this.
Thank you for bringing this to our attention.

@jimhess
It's by design as it explains why someone would use user setup.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit Azure Data Studio repo(https://github.com/microsoft/azuredatastudio)


Map int column to enum

Problem
I have an old DB, can use int with Enum, how to do it?

Suggested solution

Alternatives

Additional context

What exactly are you trying to do? @map is used to map Prisma Schema names to the actual name in the database. Even if the name was 0, it would still be represented as a String on the database level, so @map("0") instead of @map(0).

There's any solution for this problem?
I'm facing the same issue while trying to map a enum value. Check this out:
schema.prisma
enum Invoice_type {
  subscription @map("0")
  rent         @map("1")
  sell         @map("2")
}

prisma.client
export const Invoice_type: {
  subscription: 'subscription',
  rent: 'rent',
  sell: 'sell'
};


Update codesign tests

Flutter engine post-submit tests run multiple test to verify that we sign expected binaries.   However, one of our tests run_verify_binaries_codesigned_tests.dart may need to be updated as the tests may be redundant.
CC: @christopherfujino for further context

I'm closing this in favor of an umbrella issue: https://github.com/flutter/flutter/issues/153614


TDD Setup and Tear Down Creates a new DB each time tests run

How do you set up and tear down tenants for testing? I can get my tests to work following the documentation. But, each test is creating a new database. Is there a "best practice" way to setup and tear down tests to avoid multiple databases each time? If so can you maybe share a bit of code as to how you are doing this?

But, each test is creating a new database

It's how I do tests for this package.
Is that an issue because of performance or because you're using sqlite and it's bloating your database directory?
If the issue is performance, you could maybe use setUpBeforeClass. If the issue is that it's bloating your database directory, you probably do need a tear down. I'll send some code when I'm on my computer.

I have an idea that might improve testing flow. I’ll submit a pr once I get to work.

I do this:
https://github.com/stancl/tenancy/blob/e98db460ecf8a25a5c5636df082fd7541240a9dc/tests/TestCase.php#L20-L50
and I use sqlite with :memory: as the DB driver for tests.
I don't use any tearDown(), since sqlite database files aren't an issue for me.
If you want to delete those files, you could do something like this in tearDown():
config(['tenancy.delete_database_after_tenant_deletion' => true]);

tenancy()->all()->each(function (Tenant $tenant) {
    $tenant->delete();
});

I tried this in tearDown() at first:
config(['tenancy.delete_database_after_tenant_deletion' => true]);

tenancy()->all()->each(function (Tenant $tenant) {
    $tenant->delete();
});

But Laravel doesn't like it because the method $this->getPdo() in Iluminate/Database/Concerns/ManagesTransactions.php line 122 returns null and throws an error:
Error : Call to a member function beginTransaction() on null
 /*/vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:122

Here's what I ended up with that seems to be working ok:
protected function tearDown(): void
    {
        // Delete all created databases. These are stored in a collection upon tenant creation.
        optional($this->createdDatabaseNames)->each(function ($dbName) {
            DB::statement("DROP DATABASE `$dbName`");
        });

        parent::tearDown();
    }

But Laravel doesn't like it because the method $this->getPdo()

Do you have the queue_database_deletion config set to true by any chance? Some getPdo() errors on deletion are caused by a bug in 2.0.1.

Is the stack trace in tests longer than just this?
 /*/vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:122

Here's the full stack trace:
Error : Call to a member function beginTransaction() on null
 /vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:122
 /vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:106
 /vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:24
 /vendor/stancl/tenancy/src/StorageDrivers/Database/DatabaseStorageDriver.php:136
 /vendor/stancl/tenancy/src/TenantManager.php:88
 /vendor/stancl/tenancy/src/Tenant.php:246
 /tests/TestCase.php:48
 /vendor/laravel/framework/src/Illuminate/Support/Traits/EnumeratesValues.php:176
 /tests/TestCase.php:49


[TEST] Implemented MemoryRepository.countNrJobEntryAttributes() method.

@mattyb149 found this not implemented while adding LoadSaveTest for spark-submit

Probably should have null checks for jobMap and maybe for the various objects returned in entry.getValue().get( code )

@mattyb149 added null checks, thank you

Looks good thanks!


[6.1][Compat] Add GenericArgumentSyntax compatibility initializer

Cherry-pick #2920 into release/6.1

Explanation: https://github.com/swiftlang/swift-syntax/pull/2859 changed the argument parameter from some TypeSyntaxProtocol to GenericArgumentSyntax.Argument. Add back the old initializer to keep the source compatibility.
Scope: Syntax node initialization
Risk: Low. Just adding a "disfavored" initializer overload
Testing: Passes current test suite
Issues: rdar://141562966
Reviewer: TBA

@swift-ci Please test

@swift-ci Please test Windows


Is there a way to enable and disable certain validators within a single validation run?

System Details

FluentValidation version: Latest
Web Framework version (eg ASP.NET Core 2.1, MVC 5, WebApi 2. Delete if not applicable): ASP.NET Core 2.1

Issue Description
I have a large JSON blob that I need to validate, the blob is populated with ~200 questions that have several different rules each that need to be obeyed. However the rules that need to be obeyed by the answer differ according to the properties of each question.
Because of the differing question rules I need to build several different validators, each one being responsible for validating a certain rule
So for example if we have the details for the first three questions being:
Question 1: string, required, maxLength 5
Question 2: int, required, greater than 100
Question 3: int, not-required
This means I'll need 4 validators in total, with one validator enforcing the following rules:
Type (string, int, etc)
Required (not empty)
Max Length
Greater Than
And for question 1 I will need to disable the Greater Than validator, since it's not applicable to the question.
Since the validation rules differ on a question by question basis I need the ability to enable and disable certain validators depending on which are required by the question. I had the idea of creating a List<AbstractValidator<Question>> that would include the various inherited validators that I have created. I could create a list for each instance of a question with only the validators that are required and then pass the question through all of the validators in the list, collating the validation errors at the end of the validation run. However I'm not sure if this will work or if it's the best way of going about this issue. Is there a better way to do this?

Hi, the usual way to do this would be to use conditions, which allow you to specify that a rule should only be invoked in certain situations.

I thought about using when conditions for this, but to do this I'll need to create a new AbstractValidator object for each question that I validate since I also will need to pass in an object that represents the rules that should be used for each question.
Is it a bad idea to re-up a new validator object for each question with regards to performance?

There is a performance hit from rule construction, but it's pretty minor so I wouldn't worry about it.


Dashboard: context menu resize/overlap on smaller viewports

Bug Description
In the new dashboard we have a 'context menu' that opens with the three vertical dots on hover of a story on my stories view. If you are looking at the screen in a smaller size (< 600px) the menu, when open, could have better positioning. Some of the menu cuts off and there's a couple viewport sizes where the left wordpress menu takes priority and overlaps the menu. (see video)
Expected Behaviour
Menu should be visible when open and have space between its left edge and the end of the screen. Ideally, that same gutter as the rest of the dashboard.

Steps to Reproduce
Resize browser window with first story to the left's context menu expanded and see.
Screenshots
Screencast:
https://drive.google.com/file/d/1EvBLoj9QvxA3EZ0VIfrPHf2i8pramFbs/view?usp=sharing
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

I believe we can close this since we are not optimizing dashboard for phone-sized screens.

This bug has been fixed with updates to the layout to prevent this sort of thing, see gif below.


Add kafka support

Add support to track process (consume/fetch) and publish client operations for Kafka in non-Go services.
The PR includes an integration test with a Python example.

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

Project coverage is 67.73%. Comparing base (ccc8f20) to head (b3a6668).
Report is 2 commits behind head on main.

Files
Patch %
Lines

pkg/internal/kafka/kafka.go
20.51%
92 Missing and 1 partial :warning:

pkg/internal/ebpf/common/tcp_detect_transform.go
3.03%
30 Missing and 2 partials :warning:

pkg/internal/export/otel/traces.go
0.00%
9 Missing :warning:

pkg/internal/export/otel/metrics.go
44.44%
4 Missing and 1 partial :warning:

pkg/internal/export/prom/prom.go
75.00%
4 Missing :warning:

pkg/internal/export/debug/debug.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #890      +/-   ##
==========================================
- Coverage   77.57%   67.73%   -9.85%
==========================================
  Files         122      120       -2
  Lines        8594     8756     +162
==========================================
- Hits         6667     5931     -736
- Misses       1482     2300     +818
- Partials      445      525      +80

Flag
Coverage Δ

integration-test
54.96% <23.68%> (-0.75%)
:arrow_down:

k8s-integration-test
60.39% <10.52%> (-1.19%)
:arrow_down:

oats-test
?

unittests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

I fixed up bunch of things:

The payload we want to parse can be in the response from Kafka when the message is pushed, rather than polled.
I think the library we were using in the Python example has issues with Python 3.12 and I also couldn't figure out how to retry connectivity. Kafka takes some time to properly boot and the producer needs to keep trying. I changed to another library and modified the test code to use python-kafka, it's much simpler and explicit.
I updated the semconv to match latest https://opentelemetry.io/docs/specs/semconv/messaging/kafka/ and https://opentelemetry.io/docs/specs/semconv/messaging/messaging-metrics/.

One thing that doesn't work is the metric attributes. I can't see why we aren't adding MessagingSystem and MessagingDestinationName. Something that needs to be debugged. The metrics are split by "publish" and "processing". Processing is the receive part, but according to the spec it's more broad and should include the push of messages too.
Here's an example of my metrics:
messaging_publish_duration_count{instance="468df55efd81-3103683", job="integration-test/python3.11", service_name="python3.11"}

there should be messaging_system and messaging_destination_name attribute there.

One thing that doesn't work is the metric attributes. I can't see why we aren't adding MessagingSystem and MessagingDestinationName. Something that needs to be debugged. The metrics are split by "publish" and "processing". Processing is the receive part, but according to the spec it's more broad and should include the push of messages too.
Here's an example of my metrics:
messaging_publish_duration_count{instance="468df55efd81-3103683", job="integration-test/python3.11", service_name="python3.11"}

there should be messaging_system and messaging_destination_name attribute there.

done


Support for SHT40-B sensors with alternative i2c address

Could you please add support for SHT40-B sensors that have the alternative i2c address 0x45?
Currently it is hardcoded in SensirionI2CSht4x.cpp, line 44:
#define SHT4X_I2C_ADDRESS 0x44
You could just add a uint8_t field to the sensor object and use either the constructor or begin(...) method with the current address as the default. This would even be backward compatible with the current library version.

Unfortunately we don't support to change I2C address on runtime.
You can simply modify the driver and change the I2C address

Maybe I explained the issue with too little details, so for now I will reformulate it as a question.
How could I communicate with a sensor of type SHT40-BD1B-R2 using this library? I can not set the i2c address anywhere and the default is not 0x44 but 0x45

Hej @F-Schmidt99 ,
Not sure @psachs still has this old issue under the radar so I will jump in.
In the file https://github.com/Sensirion/arduino-i2c-sht4x/blob/master/src/SensirionI2CSht4x.cpp you can change the I2C address to 0x45 by setting SHT4X_I2C_ADDRESS
I hope it helps
Cheers

Hello @qfisch,
thank you for the response.
I know how to control the sensor by editing the file. But keeping in mind, that this library is for the Arduino framework, and therefor also used by many beginners, I think, that you should provide either a constructor or a 'begin' method taking the i2c address as uint8_t address argument. Alternatively, you could also pass an enum value like this:
enum SHTSensorType {
A = 0x44,
B = 0x45
};

for better readable code.
See here for guidelines of Arduino: https://docs.arduino.cc/learn/contributions/arduino-creating-library-guide
Kind regards.

Hej @F-Schmidt99 ,
I just saw your PR as well. As explained there, the feature is planned and should be rolled out soon through our typical generation process. I will keep this issue open to make sure to notify you once it is available.
Cheers
Q

Driver is now updated to included this feature


Python 2.7 does not install typing package

When installing PyOTA on a Python 2.7 virtualenv, the typing package does not get installed:
(test) [499][~] python --version
Python 2.7.13
(test) [500][~] pip install --pre pyota
...
Installing collected packages: six, python-dateutil, regex, pytz, filters, requests, pyota
Successfully installed filters-1.1.4 pyota-1.0.0b3 python-dateutil-2.6.0 pytz-2016.10 regex-2016.12.27 requests-2.12.4 six-1.10.0

Fixed.

Wheels do not include setup.py, so all of that fancy version-detection logic is lost.
There is a way to specify environment markers in project dependencies, but it requires a fairly recent version of setuptools.  For compatibility, we will just install typing on all environments, even if not needed.

We might be able to accomplish Python-version-specific dependencies using requirements files; it's worth looking into at some point when more bandwidth frees up (:


More Pico Enviro+ examples

WIFI_CONFIG.py and network_manager.py are going to move into common so I might need to mangle these and add to https://github.com/pimoroni/pimoroni-pico/pull/445


modify git.py to pass ssh_keyfile to ssh client -> ssh connection to github works again

I am not sure what caused it, but when I recently tried to use StaSh's "git push" command, I got the error message:
stash: <class 'paramiko.ssh_exception.AuthenticationException'>: Authentication failed.
My patch fixed the problem for me.

Did this work for you previously?  Did you create a new key recently, or delete the keys in ~/.ssh as well?  What paramiko version are you using?
The git usage of paramiko depended on keys in ~/.ssh.  ssh_keygen creates keys in both locations.  it should not really be necessary to create a custom wrapper, if the keys are in the default paramiko location...

That explains it!
The home directory changed when I upgraded to Pythonista 3. That's why "git" could not find the keys anymore.
However, the "ssh" command did find the keys in the STASH_HOME directory. So I tried to get the "git" command to behave in the same way as the "ssh" command, without having to modify "gittle" and "dulwich".
I did not create new keys, because I would have had to distribute the new public key to a lot of different places. I knew that the old keys still worked, because the "ssh" command still worked.
P.S.: I just checked my Paramiko version: It is version 1.16.0

@jsbain I don't quite follow the conversation here. So is this change necessary and need to be merged? Thanks!

The merge is not necessary. Putting the private key in ~/.ssh fixes the problem without the patch to git.py.


Copy command uses storage API

Codecov Report

Merging #185 into xbrianh-uploads-use-low-level will decrease coverage by 0.23%.
The diff coverage is 100.00%.

@@                        Coverage Diff                        @@
##           xbrianh-uploads-use-low-level     #185      +/-   ##
=================================================================
- Coverage                          87.23%   86.99%   -0.24%
=================================================================
  Files                                 15       15
  Lines                               1034      992      -42
=================================================================
- Hits                                 902      863      -39
+ Misses                               132      129       -3

Impacted Files
Coverage Δ

ssds/__init__.py
84.88% <100.00%> (-2.50%)
:arrow_down:

ssds/storage.py
100.00% <0.00%> (+1.48%)
: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 17e1926...4e91621. Read the comment docs.


Btc withdraw problem!!

@monyxcoin it say I successfully withdraw my btc to ur platform..the problem now is the confirmation of the tx id is 33 confirmation but until now I didn't my btc yet please help me..
Tx id: 787963097c68c13139378e14b143b477b3a5f51f455e953160f302cc104d38b8

@loydz21 i see that withdrawal was successful. BTC was sent from our platform as you can see in block explorer. Are you sure you specified the right address?

Yes this is the address wallet:
32BaTdyKkeB2MNqHhmNyS6fYDc52RSZRoo

@monyxcoin please monyx.

@loydz21 well, you can see the transaction is confirmed and BTC was sent to the address you specified.
So either that's a problem with your wallet or something else, in any case I can't help you.

@loydz21 actually I see that the tx was sent to 19DbuSJrTPfAithKN1csJMRZidXQcePVSe address. So either you specified an incorrect address or idk.

What..you can't do that to you investor..

@loydz21 as I said - you sent the withdrawal to the address you clearly could see in the form.
We are not responsible for users actions.
I see that this address was already used by you previously. So try to find a way to access your old address.

@monyxcoin this is not an incorrect address..this is nuts

@loydz21 look - https://blockchain.info/en/address/19DbuSJrTPfAithKN1csJMRZidXQcePVSe
This is the address you used before to withdraw 0.15 BTC (that even didn't belong to you actually and you got them because of the bug). The current withdrawal also got to this address, so it's your problem you can't access it.


Improve self-publishing experience - self-publishing v0.2

Improve the self-publishing experience. Specifically:

Simpler (less choice)
Faster

Here are some ideas

[ ] don't have to worry where i put assets: can put my assets e.g. images, data etc anywhere and automatically moved to public as appropriate
[ ] don't have to worry about content directory: can put my markdown wherever https://github.com/flowershow/flowershow/discussions/323 💬 we can do this today if you don't install flowershow into the content directory
[ ] faster install (and install the latest): don't have to install flowershow for each project - have one central flowershow installation
[x] Install without assets folder #207
[ ] Install is maybe wrong name - better would be init or similar like that as what we are really doing is "initializing" a flowershow based site.

install could be reserved for actually installing in future we have a pure flowershow install (e.g. that puts flowershow somewhere common)

[ ] #273
[ ] don't require users to put config.js and getters into their content folder (see this discussion #423)

Notes
See install teardown on 2022-12-09: https://flowershow.app/journal/2022-12-09

FIXED. WONTFIX. We didn't do things listed here ... because we didn't something even better to improve self-publishing #509 and #522  i.e.

obsidian plugin
plus repo that is a template ...


RFC: Add return_direct to Tools, Actions, and FunctionCalling Agents

This PR adds a return_direct flag to Tool the style of LangChain that permits a tool to signal that its response should be delivered as the "final answer" to the user.
This prevents a common error condition in which a reasoning agent runs a "terminal" tool, but then instead of returning its output, it either:

Summarizes for the user what it did
Runs other tools unnecessarily

Here's an example of that in action with the Prompt Golf bot:

What happened here is:

The Agent correctly ran the GameTurnTool which generated a new image
The Agent then replied with a summary of what it did rather than replying with the output of the tool

@douglas-reid does this seem like a reasonable way to achieve this? It would result in:

FinishAction being the way the Agent signals completion.
Tool.return_direct being a way a Tool can signal that it always should end reasoning

@eob my first reaction is that this is fine, as it seems to address an immediate need. it feels like "llm smell" in the sense that a summarizing action seems like something that should be ideally smoothed out of workflows at a different level. and i worry that there are likely tools that don't always want to return FinishActions. but both of those concerns seem minimal in the face of our existing reality. i'll take a closer look at the code impl now.

@douglas-reid thanks a ton -- will do some tinkering down these threads.
A few thoughts I had that are in the same vein as yours:

I'm not sure if it should feel weird that a the is_final property of Tool is static as opposed to part of its response.
I like the idea of Action.is_final... that feels cleaner and also aligns it with FinalAction


Fix conditional dependencies for wheel setup

References:

https://github.com/inveniosoftware/troubleshooting/issues/1
http://wheel.readthedocs.io/en/latest/#defining-conditional-dependencies

I'd love for wheel to just magically start doing the right thing per PEP 508. I've been watching this issue on BitBucket for a while.

If setuptools evolves to where it can understand the (non-standard) way for representing conditional dependencies that wheel uses, we can probably avoid having to do this. I'm still waiting to for this to resolve before making any changes to our repo.

Looks like the recently-released setuptools 36.2.7 fixes this. I just confirmed that, using that version of setuptools, I built a wheel that installs correctly on Python 2.7, 3.6, and (to test some of the other conditional dependencies) 3.3.


Question about shutter settings

Hi,
Thanks for your great work.
Since I didn't find any related information in the paper, I'd like to ask authors some questions about the Sony setting.

The Sony camera A7s2 supports silent shutter, or electronic shutter.
Was the setting always on when the dataset is collected?
Or the setting was sometimes on?
Or all images were captured with the mechanical shutter?

Have you tried to estimate the black level or always used the black level embedded in the file (512)

It seems that I did not change the shutter and I am not sure what is the default one.
I used the fixed 512.

I see. Thanks for your reply.


Bug with BatchNormalization(axis=1): ValueError: Shape must be rank 1 but is rank 4 for 'batch_normalization_1/cond/FusedBatchNorm'

Current master version of keras (commit b3cb261b22a73d195a527592b49ca57e8c9ac9f5), TensorFlow 1.8.0
BatchNormalization(axis=1) for 'channels_first' seems to fail.
import os
os.environ['KERAS_BACKEND'] = 'tensorflow'
import keras.backend as K
from keras.layers import Activation, Conv2D, Input
from keras.layers.normalization import BatchNormalization

# declare network model with channels first: ERROR
K.set_image_data_format('channels_first')
input = Input(shape=(3, 1001, 1001), dtype='float32')
x = Conv2D(filters=64, kernel_size=(3, 3), strides=1, padding='same')(input)
x = BatchNormalization(axis=1)(x)
x = Activation('relu')(x)

gives the error
Traceback (most recent call last):
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1567, in _create_c_op
    c_op = c_api.TF_FinishOperation(op_desc)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Shape must be rank 1 but is rank 4 for 'batch_normalization_1/cond/FusedBatchNorm' (op: 'FusedBatchNorm') with input shapes: [?,64,1001,1001], [1,64,1,1], [1,64,1,1], [1,64,1,1], [1,64,1,1].
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "<input>", line 11, in <module>
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/engine/base_layer.py", line 459, in __call__
    output = self.call(inputs, **kwargs)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/layers/normalization.py", line 204, in call
    training=training)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 3069, in in_train_phase
    x = switch(training, x, alt)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 3004, in switch
    else_expression_fn)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 432, in new_func
    return func(*args, **kwargs)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/ops/control_flow_ops.py", line 2072, in cond
    orig_res_f, res_f = context_f.BuildCondBranch(false_fn)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/ops/control_flow_ops.py", line 1913, in BuildCondBranch
    original_result = fn()
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/layers/normalization.py", line 165, in normalize_inference
    epsilon=self.epsilon)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 1894, in batch_normalization
    is_training=False
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/ops/nn_impl.py", line 904, in fused_batch_norm
    name=name)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/ops/gen_nn_ops.py", line 3429, in _fused_batch_norm
    is_training=is_training, name=name)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
    op_def=op_def)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1734, in __init__
    control_input_ops)
  File "/home/rcasero/.conda/envs/cytometer_tensorflow/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1570, in _create_c_op
    raise ValueError(str(e))
ValueError: Shape must be rank 1 but is rank 4 for 'batch_normalization_1/cond/FusedBatchNorm' (op: 'FusedBatchNorm') with input shapes: [?,64,1001,1001], [1,64,1,1], [1,64,1,1], [1,64,1,1], [1,64,1,1].

Meanwhile, BatchNormalization(axis=3) for 'channels_last' works.
import os
os.environ['KERAS_BACKEND'] = 'tensorflow'
import keras.backend as K
from keras.layers import Activation, Conv2D, Input
from keras.layers.normalization import BatchNormalization

# declare network model with channels last: NO ERROR
K.set_image_data_format('channels_last')
input = Input(shape=(1001, 1001, 3), dtype='float32')
x = Conv2D(filters=64, kernel_size=(3, 3), strides=1, padding='same')(input)
x = BatchNormalization(axis=3)(x)
x = Activation('relu')(x)

doesn't give any error.

@rcasero For BatchNormalization layer, if you set axis = 1, the internal implementation would broadcast mean/var/beta/gama tensors to 4 dimension, but tf.nn.fused_batch_norm only accept them as 1 dimension tensor, that leads to the exception. I have sent #10684 to fix it, wish it can help you as well.

Hi, I'm having the same problem. Is there any workaround while still using TF-GPU as backend? Thanks

@damnko , if you check the last part of my comment above, the bug doesn't happen for 'channels_last'.

Thanks for your reply @rcasero , I am currently using 'channels_last' with axis = 1 . I am not sure what should I do to bypass the problem. It works normally on my local pc running on CPU.
Do you think I should try to reshape the input and use 'channels_first'?

@damnko The problem for me only happens with 'channels_first'.  Do you get the problem with both?

This scripts above behave just as you say when running on GPU, but they both work on my laptop running on CPU.
The fact is my input shape is something like [?, 128, 6, 1] where 6 are my channels and 128 is the data along the time domain. My input data are sensor measurings and I am using axis=1 for the normalization.
Do you have any further suggestion? Thanks agan

Yes, the problem, as you say, happens only when running on GPU.
Sorry, @damnko , I haven't worked with temporal data. Why do you have the final  1 in [?, 128, 6, 1]? Can't your data be [?, 128, 6]?

That's because Conv2D expects a 4D tensor with shape (batch, rows, cols, channels). And I cant use Conv1D since it will convolve only along the temporal direction.
I will do some other tests and update the thread eventually. Thanks again for your support

Not sure what you are trying to do, but in any case, I think your data should have shape [?, 128, 1, 6].

I'm still in the process of learning so I might be wrong, but if I reshape it as per your suggestion I will be able to convolve only on [128, 1] which is [rows, cols] which is not what I want. I want to be able to convolve along the temporal direction (for example with a [5,1] kernel) and do cross correlation among different sensors and time (for example with a [5,2] kernel).
As far as I understood, if I have an input shape like [?, 128, 1, 6] this last operation won't be possible since I won't be able to convolve along the channels. Hopefully this makes sense.
Aside of that, the workaround that seems to work for me at the moment is to use an input with shape [?, 1, 128, 6] and then
K.set_image_data_format('channels_first')
...
X = BatchNormalization(axis = 2, name='bn0')(X)

Is this still investigated?
Do I really need to go back to keras 2.1.6 to solve this?

Closing this issue updated version of TensorFlow and Keras fixes it. Feel free to reopen if still running into problems. Thanks!

@ymodak Could you please say which version is fixed? I have the same problem with Keras 2.2.4 and tf 1.9.0-rc0. Thanks!

Try upgrading your TensorFlow version to 1.10 or 1.11

@ymodak still not working with tf 1.12 and keras 2.2.4

@ymodak Did not help. Could you show the commit?
import tensorflow as tf; print(tf.__version__)
# 1.11.0-rc1
import keras; print(keras.__version__)
# 2.2.4

from keras import backend as K
from keras.layers import Input, Conv2D, BatchNormalization
K.set_image_data_format('channels_first')
input = Input(shape=(3, 1001, 1001), dtype='float32')
x = Conv2D(filters=64, kernel_size=(3, 3), strides=1, padding='same')(input)
x = BatchNormalization(axis=1)(x)

ValueError: Shape must be rank 1 but is rank 0 for 'batch_normalization_1/cond/Reshape_4' (op: 'Reshape') with input shapes: [1,64,1,1], [].

@see-- thx, fixed.

@rcasero @see-- @yaarsh @ymodak  @davideboschetto @yanboliang @damnko thank you for your comments. how can i install keras master?

just change your tensorflow_backend.py

users should do that too?

@iperov    I did that,and it works.With the link from @see--  you can know how to change it.

@jacmule I can do that too. I am talk about users :D

how long wait new pip release?

Installing by master version worked for me, thanks !!

better to change network to channels_last

@jacmule How does one just change tensorflow_backend.py?

please fix that

@stiv-yakovenko why you need that?

https://github.com/kbardool/keras-frcnn fails with lastest keras, i have to rollback it

@stiv-yakovenko Which version worked for you ?

This one works correctly:

Name: Keras
Version: 2.1.2
Summary: Deep Learning for Python
Home-page: https://github.com/fchollet/keras
Author: Francois Chollet
Author-email<EMAIL_ADDRESS>License: MIT
Location: c:\users\steve\miniconda3\lib\site-packages
Requires: six, numpy, pyyaml, scipy
Required-by:

@stiv-yakovenko  You cant use GPU ?

problem happens both on cpu and gpu

@kazemSafari just change your tensorflow_backend.py

How can to change the tensorflow_backend.py in Jupyter note book(anaconda) ?
I am getting this error with 2.2.4 (keras) and tensorflow 1.13.1 ,can anyone @iperov  @see--  please help?

please change tensorflow_backend in keras by yourself as suggested by @see--
In keras version - 2.2.4 -> backend --> tesnorflow_backend.py : change "()" to "[ ]" in line no 1908,1910,1914, 1918.

@rcasero @see-- @yaarsh @ymodak @davideboschetto @yanboliang @damnko thank you for your comments. how can i install keras master?

pipenv install -e git+https://github.com/keras-team/keras.git@master#egg=keras

please change tensorflow_backend in keras by yourself as suggested by @see--
In keras version - 2.2.4 -> backend --> tesnorflow_backend.py : change "()" to "[ ]" in line no 1908,1910,1914, 1918.

This worked for me, thank you.

please change tensorflow_backend in keras by yourself as suggested by @see--
In keras version - 2.2.4 -> backend --> tesnorflow_backend.py : change "()" to "[ ]" in line no 1908,1910,1914, 1918.

This worked like magic LOOL hope this gets fixed

please change tensorflow_backend in keras by yourself as suggested by @see--
In keras version - 2.2.4 -> backend --> tesnorflow_backend.py : change "()" to "[ ]" in line no 1908,1910,1914, 1918.

Thank you!! This worked like a charm. Although it's just super sad that you have to manually modify keras' core files to fix this.

You can also run this:
pip install --no-deps --force-reinstall git+https://github.com/keras-team/keras

Workaround: Reduce the keras version. CPU 2.2.0 can be, GPU 2.1.6 can.

It works well with "cuda=9.0, tensorflow=1.11.0, keras=2.2.5"

please change tensorflow_backend in keras by yourself as suggested by @see--
In keras version - 2.2.4 -> backend --> tesnorflow_backend.py : change "()" to "[ ]" in line no 1908,1910,1914, 1918.

why it can be worked?

[ ]

Thank you very much! Excellent :+1:


Simplify the code shown in kubernetes provider in root readme.md

Is your request related to a new offering from AWS?

No
Is your request related to a problem? Please describe.

Not really
Describe the solution you'd like.

The docs in main readme.md should use var.create_eks instead of the obfuscated element(concat(..., [""]), 0) construct.
Describe alternatives you've considered.

provider "kubernetes" {
  host                   = var.create_eks ? data.aws_eks_cluster.cluster[0].endpoint : ""
  cluster_ca_certificate = var.create_eks ? base64decode(data.aws_eks_cluster.cluster[0].certificate_authority.0.data) : ""
  token                  = var.create_eks ? data.aws_eks_cluster_auth.cluster[0].token : ""
}

Additional context

it is simplified now:


✨ feat(CI): CI for Gentest

🗒 Description
This PR introduces an end-to-end CI workflow for gentest.
🔗 Related Issues
closes #860
✅ Checklist

[ ] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[ ] All: Considered updating the online docs in the ./docs/ directory.
[ ] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[ ] Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
[ ] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[ ] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

Hey @danceratopz I have added an environment for Gentest in tox. I have added a new config for ROOT_DIR, since the path was not resolving correctly inside tox.
For this to work, we need to populate env.yaml. The entire yaml file can be sourced from a github secret and consumed in a new workflow.

name: CI Test

on:
  push:
    branches:
      - main
  pull_request:
    branches:
      - main

jobs:
  test:
    runs-on: ubuntu-latest

    steps:
    - name: Checkout code
      uses: actions/checkout@v2

    - name: Set up environment file (env.yaml)
      run: |
        echo "${{ secrets.ENV_YAML }}" >> env.yaml

    - name: Execute end-to-end gentest
      run: |
        uvx --with=tox-uv tox -e gentest


[stable/prometheus] support for toleration on all sub components

Which chart:
stable/prometheus
What happened:
not all components support defining tolerations for tainted nodes.
What you expected to happen:
being able to dispatch all components of the prometheus stack on admin nodes
How to reproduce it (as minimally and precisely as possible):
try to deploy the prometheus stack (all components enabled) on tainted nodes.
Anything else we need to know:
PR: https://github.com/kubernetes/charts/pull/3522

/assign @unguiculus

This was merged. Thanks @unguiculus


How to share player between two views

Idea: Implement full screen with this behavior

nonFullScreenView show video and when click on full screen icon it go to full screen by
show  FullScreenView which is in landscape mode and continue from latest time in video
when user click back or click to exit full screen return to old nonFullScreenView with latest time in video

i think it will be easy somehow if player is shared between two views

@PierfrancescoSoffritti

i had to implement a simple class that held the current state time etc of the video and passed them back and forward. this was the best way i was able to get this working

@RyanVDW would you mind posting a code for your implementation here? I think we would all benefit from it.

Its a lot of code but here is the basic break down:
Create a POJO that holds the current video state:
private final String youtubeId; private boolean isPlaying = false; private boolean isPaused = false; private boolean isStopped = true; private float time = 0; private boolean isFullScreen = false;
Then using the onStateChanged, onYouTubePlayerEnterFullScreen, onYouTubePlayerExitFullScreen and onCurrentSecond methods from the interface, you can store the state of the video in this class. Then when you go Fullscreen (like opening a Activity which uses a layout that just has the YoutubePlayerView class full the layout) passing the POJO through the intent, you can then continue the video there

i did in same view by requesting activity to be in landscape
and then hide other views so Video be alone in screen

Hi, for those who are still interested - I answered a similar issue (how to use the fullscreen) on Stack Overflow. In a nutshell:

create a layout for fullscreen and add it to the parent layout;
in the .getPlayerUiController().setFullScreenButtonClickListener remove the YT player View from the layout (eg. trailerLayout) and add the same YT player View into the fullscreen layout;
when exiting fullscreen, remove the YT player View from the fullscreen layout and add the same YT player View into the trailerLayout.

Consider it as a "fake" fullscreen" - you are rotating the whole layout while remaining in portrait mode :-)

Answering the question of #633 I figured out that the full screen button from the default ui controller matches the parent's dimensions only. Returning back from full screen the player falls back to the 16:9 ratio. The scaling and dimensioning seems to require some work to fix and add proper behavior, since the youtube interface also adds unnecessary padding / borders.


PPOTrainer generate method does not take batched tensors as input

The documentation claims that the first parameter of PPOTrainer.generate is query_tensor (torch.LongTensor) — A tensor of shape (batch_size, seq_len) containing query tokens.
However, it seems to me the function actually expects a List[Tensor].
Given the following code
batch_size = 32
config = PPOConfig(
    model_name='gpt2',
    steps=10,
    learning_rate=1.41e-5,
    batch_size=batch_size,
    remove_unused_columns=False,
    log_with=None,
)
ppo_trainer = PPOTrainer(
    config,
    model,
    model_ref,
    tokenizer,
)

txt_out_len = 8
generation_kwargs = {
    "min_length": txt_out_len,
    "top_k": 0.0,
    "top_p": 1.0,
    "do_sample": True,
    "pad_token_id": tokenizer.eos_token_id,
    "max_new_tokens": txt_out_len,
    "eos_token_id": tokenizer.eos_token_id,
}

the following fails:
# inputs = dataset['input_ids'][:2]
inputs = torch.randint(0, 10, size=(batch_size, 16)).to(device)

print(inputs.shape)
resp = ppo_trainer.generate(
    inputs, **generation_kwargs
)

with error:
RuntimeError: The size of tensor a (32) must match the size of tensor b (2) at non-singleton dimension 0

inputs here is a Tensor of shape [batch_size, seq_len] as requested.
Converting the input to a List of Tensors, as in the following snippet is successful:
resp = ppo_trainer.generate(
    list(inputs), **generation_kwargs
)

However, this causes overhead, as the tensor has to be unpacked in a list, and most importantly, undoes any speed gains from batching.
Upon further investigation, I noticed how the PPOTrainer generate method calls the underlying model's generate function:
resp = ppo_trainer.accelerator.unwrap_model(ppo_trainer.model).generate(
    input_ids=inputs.unsqueeze(0), **generation_kwargs
)

notice the .unsqueeze(0) on the input. This assumes that input is in fact not batched, but just a tensor of shape [,seq_len], which needs to be expanded since the model's generate correctly expects batched inputs.
If we remove the unsqueeze operation, and call the following snippet
resp = ppo_trainer.accelerator.unwrap_model(ppo_trainer.model).generate(
    input_ids=inputs, **generation_kwargs
)

we get the expected output, with the correct input.
Is this a bug in the implementation? Or is this intended for some reason, and the documentation isn't clear about it?

Hi @EdoardoPona
Thanks for raising this up, I think that there is a small typo indeed, it can accept either a list of tensors (in case of batched generation) or a tensor of a single dimension in case of non-batched generation
We had to come up with a custom implementation of batched generation as we need to properly deal with encoder-decoder model case, while padding correctly the input.

Thanks, got it! Does this affect performance compared to hypothetically feeding a single batched tensor to model.generate, assuming padding was already taken care of by the dataloader?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_d6gb4jz0p8/clone/recipe/meta.yaml:24: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_d6gb4jz0p8/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_d6gb4jz0p8/clone/recipe/meta.yaml:52: invalid_url: http://beaker.groovie.org/ : HTTPConnectionPool(host='beaker.groovie.org', port=80): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0xffff74f04700>: Failed to establish a new connection: [Errno -2] Name or service not known'))
ERROR: /tmp/abs_d6gb4jz0p8/clone/recipe/meta.yaml:53: http_url: http://beaker.groovie.org/ is not https
Errors were found


proxylib: Fix unit test flake when counting access log entries

Modify the access log check function to allow for upto 5 seconds to
receive the logs, but also to stop waiting after 50 milliseconds if
the expecgted number of passes and drops have been received. The
additional time allows for successful completion in resource limited
test environments, while the short wait after the expected number of
passes and drops has been received allows for the case where
extraneous logs are being generated.
Fixes: #6135
Fixes: #6447
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.01%) to 43.118% when pulling 4b7831e3cf579d8e03b0935cc88c92f99de2dce4 on pr/proxylib-test-race into 05b4d2b32f1440abdc0e279265db7543ce501147 on master.


In Flutter, when implementing the dark mode feature, the keyboard appearance does not change to dark mode when the system is using light mode. It remains in light mode when the keyboard is open

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
When the system is in dark mode, the keyboard appearance follows the system settings. However, if the system is in light mode and the app is set to dark mode, the keyboard will be displayed in dark mode
Expected results

keyboard show in dark mode
Actual results

Code sample
keyboardApperence  : themeData == darkmode ? Brightness.dark: Brightness.light
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output

actual output

Hi @prituljogiya. I can't seem to reproduce this on the latest versions of flutter. Are you able to upgrade to the latest versions to check if this issue still persists? If it does, kindly provide a minimal complete reproducible code sample that doesn’t include 3rd party plugins or complex production code (a single main.dart file)?
Thank you


Can't use this code in Arduino Nano 33 ble sense rev2, since it has different Accelerometer and gyroscope module

Is there any help @petewarden

+1

@Abishek-Simion Have you found a solution?

@VarusJ  No I haven,t found any solution.
But achieved the same using a different python code, which simply uses accelerometer and gyroscope.

Thanks a bunch!!!


Pull Request from Archive Tool.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

DocsArchiveDemoSrc/TOC.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/docfx.json
:white_check_mark:Succeeded

DocsArchiveDemoSrc/how-do-i-find-the-serial-number-on-my-device-ios.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/how-to-reconnect-a-compromised-ios-device.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/send-errors-to-your-it-admin-ios.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/send-logs-to-microsoft-iOS.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/toc.yml
:white_check_mark:Succeeded

DocsArchiveDemoSrc/troubleshoot-your-device-iOS.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/you-are-asked-to-identify-your-device-when-trying-to-enroll-ios.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/you-get-an-error-while-using-the-company-portal-app-ios.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/you-need-to-update-your-ios-device.md
:white_check_mark:Succeeded

DocsArchiveDemoSrc/your-ios-version-isnt-yet-supported.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Remove default_nettype statements at the end of the RTL

Surfaced by https://github.com/The-OpenROAD-Project/OpenLane/issues/2083. Yosys would use the default nettype wire when compiling the user project. Although this might be an ambiguity in yosys behavior, I don't see the need of adding these lines. Having said that, there presence in the RTL seems to be intentional, I added @marwaneltoukhy and @M0stafaRady as reviewers to asses if this PR is going to have any further impact.

I'm not sure if it a standard this but I recall to notice on compiler was dealing with the `default_nettype as a global macro not just for the current file. So if we removed the line in the end of the file, the next to be compiled will be treated as if the default net type is none. So, I don't recommend it.


fix: replace symlink for fixture by actual copy

We are experiencing some issues with deploying the module and the error message points to the symlink. I therefore suggest to just replace the symlink with an actual copy.

 I took a quick look into the test cases and the Suse distro seems to refer to the RedHat directory already there, so the symlink seems to be unused...


[refactor] refactor how livequery features can be retrieved via id

https://github.com/FZJ-INM1-BDA/siibra-python/pull/366 pathces this issue but it can be improved.
This is important especially on the siibra-api side.

currently live queries features are wrapped in ProxyFeature class. Not an eloquent solution, but it works. we should consider how this can be better represented in the future.


Migrate to R 4.0

Last month, we officially created a blog post announcing that the Carpentries lessons would be migrating to R 4.0: https://carpentries.org/blog/2020/08/r-4-migration/
This tool should migrate as well, and afact, there are changes to how this container works on the rocker side, so that will need to be taken into account, but I think it should be as simple as changing the version number.

I've ended up doing this to work on the r-gapminder-lesson, if you'd like a PR?

If the modified container worked for you, then yes, a PR would be lovely :+1:


add docs for lua filter and change log

followup of https://github.com/envoyproxy/envoy/pull/36920
Commit Message: add docs for lua filter and change log
Additional Description:
Risk Level: none
Testing: no
Docs Changes: yes
Release Notes: yes
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/retest

@fl0Lec docs failure is real
 Warning: /tmp/tmp7b4f_pq0/generated/rst/version_history/v1.33/v1.33.0.rst:144: WARNING: undefined label: 'config_http_filters_lua_stream_info_downstream_direct_local_address'

/docs
should render the docs for you to validate

/wait
For addressing comment above re linking

thanks for updating the other headings @fl0Lec
looks like one was missed tho


starting profiler failed

profiler -i wlxa09f1038f9e2 --noAP
2021-10-29 17:24:23,490 [WARNING] setup_config: can not find config at /etc/wlanpi-profiler/config.ini
2021-10-29 17:24:23,800 [WARNING] interface: wlxa09f1038f9e2 operating state is down when we expect up
Starting profiler in listen only mode:
 - Listening for association frames with wlxa09f1038f9e2 on channel 36
 - Results are saved locally and printed to screen

Instructions:
 - Associate your Wi-Fi client to any AP on channel 36
 - We should passively detect the association request

2021-10-29 17:24:23,802 [WARNING] start: beacon process not started because user requested listen only mode
Process sniffer:
Traceback (most recent call last):
File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/root/.local/pipx/venvs/profiler/lib/python3.8/site-packages/profiler/fakeap.py", line 341, in init
sniff(
File "/root/.local/pipx/venvs/profiler/lib/python3.8/site-packages/scapy/sendrecv.py", line 1263, in sniff
sniffer._run(*args, **kwargs)
File "/root/.local/pipx/venvs/profiler/lib/python3.8/site-packages/scapy/sendrecv.py", line 1210, in _run
session.on_packet_received(p)
File "/root/.local/pipx/venvs/profiler/lib/python3.8/site-packages/scapy/sessions.py", line 108, in on_packet_received
result = self.prn(pkt)
File "/root/.local/pipx/venvs/profiler/lib/python3.8/site-packages/profiler/fakeap.py", line 354, in received_frame
ssid = packet[Dot11Elt].info.decode()
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc6 in position 1: invalid continuation byte

@nimbo78  my presumption was that there is an issue decoding some non utf-8 characters detected from neighbor APs in your environment. So, I took 0xc6 from your bug report, and put it on the air with hostapd. I then tried to reproduce and I am unable to.
Can you provide additional information about the character encoding found in your neighboring SSIDs? For example, if we decode 0xc6 with latin-1 we get Æ. When we try to decode with utf-8, it fails.
>>> b"\xc6".decode()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc6 in position 0: unexpected end of data
>>> b"\xc6".decode("latin-1")
'Æ'

@joshschmelzle I can try to collect a dump on Monday, I will not be in the office where these networks exists, earlier than Monday

I may have just been able to reproduce, I made a silly mistake earlier when trying to reproduce. I will work on a fix that hopefully you can test on Monday.

In my testing, this issue may be fixed in 88ef9b6c5fc272c2fe00cedf8d021fd7512ba290 which I authored a few hours ago.
@nimbo78 when you are able to, can you try upgrading profiler on your WLAN Pi with the instructions at https://github.com/WLAN-Pi/wlanpi-profiler/blob/main/UPGRADING_WITH_PIPX.md and report back if it resolved the issue?

@joshschmelzle after upgrade seems now it works well.
also found this:
$ sudo profiler  --oui_update
2021-11-01 08:05:32,932 [INFO] update_manuf: manuf file is located at /opt/wlanpi/pipx/venvs/profiler/lib/python3.7/site-packages/manuf/manuf
2021-11-01 08:05:32,933 [INFO] update_manuf: manuf file last modified at: Sun Oct 31 15:32:31 2021
2021-11-01 08:05:32,934 [INFO] update_manuf: running 'sudo manuf --update'
Traceback (most recent call last):
  File "/opt/wlanpi/pipx/bin/profiler", line 8, in <module>
    sys.exit(main())
  File "/opt/wlanpi/pipx/venvs/profiler/lib/python3.7/site-packages/profiler/__main__.py", line 27, in main
    manager.start(args)
  File "/opt/wlanpi/pipx/venvs/profiler/lib/python3.7/site-packages/profiler/manager.py", line 78, in start
    sys.exit(0) if helpers.update_manuf() else sys.exit(-1)
  File "/opt/wlanpi/pipx/venvs/profiler/lib/python3.7/site-packages/profiler/helpers.py", line 441, in update_manuf
    cp = run_cli_cmd(["sudo", f"{sys.prefix}/bin/manuf", "--update"])
  File "/opt/wlanpi/pipx/venvs/profiler/lib/python3.7/site-packages/profiler/helpers.py", line 52, in run_cli_cmd
    capture_output=True,
  File "/usr/lib/python3.7/subprocess.py", line 487, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['sudo', '/opt/wlanpi/pipx/venvs/profiler/bin/manuf', '--update']' returned non-zero exit status 1.

@nimbo thank you for testing and confirming the changes. I've opened a new issue for the --oui_update topic and am closing this one.


Fixing failing Travis build

Hi @naokazuterada
I have given some attention to the failing build. Everything looks very promising, the last part needed is the coverage report, where I observe the following error.
==> Collecting reports

    Generating coverage xml reports for Python

    Error running `['/home/travis/virtualenv/python3.7.1/bin/python', '-m', 'coverage', 'xml', '-i']`: Command '['/home/travis/virtualenv/python3.7.1/bin/python', '-m', 'coverage', 'xml', '-i']' returned non-zero exit status 1.

    - Ignored: [Errno 2] No such file or directory: '/home/travis/build/naokazuterada/MarkdownTOC/coverage.xml'

Error: No coverage report found

It seems as if covdecov cannot locate the generated coverage data, I suspect a path issue, since the data is displayed in the CI log.
Could you give my proposed PR a review ? - I will dig into the remaining issue later unless you can spot it and fix it before me. I just have to squeeze in some work today :-/

Oh appveyor fails... need to look at that too - guess it is a one problem at a time:-)

@jonasbn Oh, the CI was failing... I'm sorry! I'll pick up the appveyor issue here as well.

@jonasbn I ran the build and now it looks working well. thank you!


How to access technical indicators

Summary (include Python version)
The API provides the ability to query technical indicators https://iexcloud.io/docs/api/#technical-indicators
This functionality appears to be missing from iexfinance. Any plans for adding a method to query this in future? if not, then any pointers on what is the starting point in code where I can build one ?
Date/time of issue
Expected behavior
Actual behavior

iexfinance should output to pandas format using
output_format="pandas"
then you should be able to use this https://github.com/twopirllc/pandas-ta

I would like to use this package to access technical indicators as well. I am going to open a PR to add that functionality.


IE11 Issue - Error when playing clip [object Event]

Current Behavior
I have a React-Table with one of the columns having a react-player component in it.
The component doesn't play when clicking play in IE 11. The component updates to the "playing" state visually but no sound is played back and progress doesn't change either.
On page load the error below is thrown for every react-player component on the page
Error when playing clip [object Event]
   "Error when playing clip"
   {
      [functions]: ,
      __proto__: {
         [functions]: ,
         __proto__: { },
         AT_TARGET: 2,
         bubbles: <Permission denied>,
         BUBBLING_PHASE: 3,
         cancelable: <Permission denied>,
         cancelBubble: <Permission denied>,
         CAPTURING_PHASE: 1,
         constructor: { },
         currentTarget: <Permission denied>,
         defaultPrevented: <Permission denied>,
         eventPhase: <Permission denied>,
         isTrusted: <Permission denied>,
         srcElement: <Permission denied>,
         target: <Permission denied>,
         timeStamp: <Permission denied>,
         type: <Permission denied>
      },
      AT_TARGET: 2,
      bubbles: false,
      BUBBLING_PHASE: 3,
      cancelable: false,
      cancelBubble: false,
      CAPTURING_PHASE: 1,
      constructor: {
         [functions]: ,
         __proto__: { },
         AT_TARGET: 2,
         BUBBLING_PHASE: 3,
         CAPTURING_PHASE: 1,
         prototype: { }
      },
      currentTarget: { },
      defaultPrevented: false,
      eventPhase: 2,
      isTrusted: true,
      srcElement: { },
      target: { },
      timeStamp:<PHONE_NUMBER>679,
      type: "error"
   }

Expected Behavior
Expected clip to play when play is clicked
Steps to Reproduce

Using IE 11 with babel polyfill
Using react-table with column that has voice clip component in it
Clip is a mp3 file

Environment

URL attempting to play: any, even tried the default provided on the react-player site
Browser: IE 11
Operating system: Windows 10
-React-player is imported with
import ReactPlayer from 'react-player';

Are you sure the mp3 you are trying to play is in the correct encoding etc? It looks like this is a problem with IE11, not with ReactPlayer, judging by the various search results for this problem.


The container's scroll-bar auto-scrolling to bottom when it has focus.

I have used select2 and need the scroll bar as below pic.

But found the scroll bar will auto-scrolling to bottom when I click it or open options. Debugging the source found the root cause is when container close/open/focus, the "search" will be focused.

My question is why the "search" should be focused automatic?
Select2 : 4.0.3
Jquery : 2.2.3
browser: Chrome Version 79.0.3945.130
Cheers.

Select2 : 4.0.3

I'm 99% sure we fixed this in a version that was released in the last 5 years.


Which files need to be hosted on cdnjs?

Hi, I'd like to host your library, isomorphic-fetch on cdnjs — It is one of the most famous free and public web front-end CDN services which is used by ~1,143,000 websites worldwide.
There are three files which seem relevant to me in your repository:
fetch-bower.js
fetch-npm-browserify.js
fetch-npm-node.js

Which files out of the ones listed above will work in a browser and let developers use your library directly? Also, please let me know if there are any other files that should be hosted.

I think none of them will work on their own in the browser…  It needs to be compiled via browserify / babel etc to work in the browser.

(Also thank you for your kind message and interest)

@matthew-andrews Thanks for letting me know! 😄

Hi @matthew-andrews, is it possible to provide it in the repository? Since one of your users is asking CDNJS to host it :)

I don't really like to commit built files into repositories…  Could you pull the file from a web service that npm install's and webpack's the module instead?  e.g.
<EMAIL_ADDRESS>

or this one:-
https://wzrd.in/bundle/isomorphic-fetch

hmmmm ... we don't support that yet.


required map UI issue

In a form with a PointField widget (it probably does the same for any map widget) the UI in Chrome is less than ideal.  The issue is that all the data is encoded and placed in a hidden textarea that's marked as required (the html5 attribute).  If a person leaves the field empty and hits "Submit" then you get an error message in the console saying "An invalid form control with name='loc' is not focusable." (where 'loc' is the name of the textarea) and the form simply doesn't submit.  This should probably be changed so required is left off of the hidden textarea regardless and the missing data is caught afterwards in Django validation.
This seems to be easily fixed by changing:  {% include "floppyforms/textarea.html" %}
to:  {% include "floppyforms/textarea.html" with required=False %}
in the openlayers.html file.

False as a template variable was added in 1.5, though, so maybe 0 instead.

For the moment you can disable HTML5 validation on the form with <form ... novalidate>.
Or you can use the current master, I just pushed a commit including the required=0 as you suggested. :)
Thanks for the report!


docs/getting-started-guides/ubuntu.md refers to wrong k8s version - 1.1.4 instead of 1.1.8

The script actually defaults to k8s version 1.1.8, but the docs says it is 1.1.4

Thank you for the PR, Mike!


[BUG] Rockets getting stuck at y=1000 when launching

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry

1.12.2-1.7.0-221

Why is this important?
Knowing the version of Advanced Rocketry helps us narrow down an offending piece of code faster because we can easily determine what code is added and when.]

Have you verified this is an issue in the latest unstable build

[x] Y

Version of LibVulpes

1.12.2-0.4.2-75

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] N/A

If Y, what is the MINIMUM set of mods required.

Description of the problem

When I take my rocket to a space station, it hangs up in space, the countdown to descent does not start, and it does not descend. Shortly after it says "press space to take off" to go back to the overworld. Sometimes it will descend if I go back to the overworld, sometimes it does the same thing when I go back. I am not able to land on the space station unless I restart my game.

Why is this important?
If we cannot reproduce the problem, we probably cannot fix it at all.]

What environment are you running in?  Are you using other mods (I'm not sure how N/A applies here), particularly sponge or similar?  What version of forge?

Running Forge <IP_ADDRESS>54 using a multiMC instance.
MultiMC has a console where you can see what's getting logged in real time. I looked at that when the rocket was launching, no exceptions logged relating to other mods, in fact it didn't log exceptions at all. The one thing I saw once was a message relating to IC2's worldgen. I take IC2 out, same thing happens. When I restart the game the rocket is landed where I left off from, so it appears to be a disconnect between what's happening to the rocket server side and client side.

It's a JourneyMap issue.
[11:23:12] [Client thread/WARN] [journeymap]: core (Initialized) Bad configField entry during updateFrom(): optionsManagerViewed=null
Removing JourneyMap solved it. (This was the latest version of Journeymap)

Was going to say that Journeymap was an issue, but the same thing happened after a few times after I took Journeymap out. If I look at the altitude meter, the altitude goes down but the player remains stuck in the original position.

Does this occur with no mods other than AR and libvulpes installed?

I set up a test for that and didn't see it, so I looked back at the main pack where I first saw the issue. My main pack is a tech heavy one and runs at 60% of my 8GB memory allocation. The test instance barely cracked 20% of that. Could lag be a factor here? It doesn't happen all of the time, but I am seeing a "server overloaded" message in the console when it does.

Does logging out of the server then logging back in fix it?  If so, then it could be a factor (some kind of client desync due to something being time sensative?).

I think I found a mechanism.  So there's a segment of code after going between worlds that checks if the number of ticks the rocket entity is alive is exactly 20 (to give the world some time to load), if the server is "falling behind", it can skip processing ticks, so I'm wondering if the entity is never exactly 20 ticks old because some are being skipped.  Therefore the initial setup that is supposed to occur after moving the player and such never gets run.

Ruled out as a possible cause.

AdvancedRocketry-1.12.2-1.7.0-universal.jar.zip
Could you try this build and let me know if it works for you?

This was happening for me on single player so logging out and in to the server just meant restarting the game. Once I reduced the size of my modpack, the issue never happened again.

Would you be able to at least test it in the same conditions it was happening before just to check?  I have not had luck replicating the issue locally so I've got no idea if it's really fixed or not

I added more mods to my pack to get it to the same level of memory usage as before and did not experience the same issue with the jar you linked to. I think it's safe to call it fixed at this point.


Automatic instrumentation support for java like Pinpoint and SkyWalking

Requirement - what kind of business use case are you trying to solve?
Automatic instrumentation support for java like Pinpoint and SkyWalking.

Problem - what in Jaeger blocks you from solving the requirement?
Client lib for java etc.

Proposal - what do you suggest to solve the problem or improve the existing situation?
Using some tech like javaagents like Pinpoint and Skywalking.

Any open questions to address

It's not in the scope of Jaeger project. We believe instrumentation should be vendor-neutral, not tied to a specific tracing backend.
See https://github.com/opentracing-contrib/java-specialagent

It's not in the scope of Jaeger project. We believe instrumentation should be vendor-neutral, not tied to a specific tracing backend.
See https://github.com/opentracing-contrib/java-specialagent

Thanks a lot for your clarification and suggestion.


Initial implementation of new register description

Initial implementation of a more generic way to describe registers.
The goal is to have a struct which works for different platforms, at least RISCV and ARM.
The code of the ARM cores itself is not yet fully adapted right now.

LGTM, thanks!


import binding not linked in define function

Bug report

Keywords:

AMD
define
import

What is the current behavior?
import binding in define function will cause referenceError.
If the current behavior is a bug, please provide the steps to reproduce.

Playground: https://stackblitz.com/edit/node-y2sktu?file=dist/main.js&view=editor
Input:
// lib.js
export default 42;

// index.js
import foo from './lib.js';
const bar = require('./lib.js');

define(function (require) {
  console.log(foo); // not work
  console.log(bar); // ok
});

noop(function () {
  console.log(foo); // ok
  console.log(bar); // ok
});

function noop() {}

output:
// omitted

/* harmony import */ var _lib_js__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./lib.js */ "./src/lib.js");

const bar = __webpack_require__(/*! ./lib.js */ "./src/lib.js");

define(function (require) {
  console.log(foo); // not work
  console.log(bar); // ok
});

noop(function () {
  console.log(_lib_js__WEBPACK_IMPORTED_MODULE_0__["default"]); // ok
  console.log(bar); // ok
});

function noop() {}

// omitted

What is the expected behavior?
I am not sure if it's a bug.
But I expect that foo should be transpiled into _lib_js__WEBPACK_IMPORTED_MODULE_0__["default"].

Other relevant information:
webpack version:  5.38.1
Node.js version: 16.20.0
Operating System: Darwin arm64
Additional tools:
related issues:

#14307
swc-project/swc/issues/7334

yeah, AMD again, not sure why developers still use it in 2023, feel free to send a fix

I am willing to fix it.

bump

Despite several attempts, I did not get the expected result.
If someone is ready to work on it, go ahead.

I found that for define callExpression. Its args will skip parsing in harmony module that causes import binding can't be linked.
https://github.com/webpack/webpack/blob/964c0315df0ee86a2b4edfdf621afa19db140d4f/lib/dependencies/HarmonyDetectionParserPlugin.js#L104

I found that for define callExpression. Its args such as (function (require) {console.log(foo); // not work}) will skip parsing in harmony module that causes import binding can't be linked.
https://github.com/webpack/webpack/blob/964c0315df0ee86a2b4edfdf621afa19db140d4f/lib/dependencies/HarmonyDetectionParserPlugin.js#L104
webpack@4 also have this logic.
https://github.com/webpack/webpack/blob/d4603c6d7ef4acfbe956ef49d8476c7368526989/lib/dependencies/HarmonyDetectionParserPlugin.js#L77
The difference is webpack4 will process VarDeclaration defined in top-level. It may be caused by innerGraph.

// lib.js
export default 42;

// index.js
import foo from './lib.js';

let d = function (require) {
  console.log(foo);
}

define(d);

and  optimization.innerGraph: false.
It works.

After look into this issue in depth.
I found the issue is caused by innerGraph.
foo dep.usedExport will be set to false which makes 2 module can't be connected. So we can temporarily workaround it by setting optimization.innerGraph: false
Meanwhile, there is another problem. We need to declare the arguments of define function call at top-level. Otherwise, it will skip parsing of callExpression.arguments by HarmonyDetectionParserPlugin. https://github.com/webpack/webpack/blob/964c0315df0ee86a2b4edfdf621afa19db140d4f/lib/dependencies/HarmonyDetectionParserPlugin.js#L104
Following can work.
// lib.js
export default 42;

// index.js
import foo from './lib.js';
const bar = require('./lib.js');

const arg = function (require) {
  console.log(foo); // not work
  console.log(bar); // ok
}

define(arg);

noop(function () {
  console.log(foo); // ok
  console.log(bar); // ok
});

function noop() {}

// webpack.config.js
module.exports = {
	// ...
	optimization:{
		// ...
		innerGraph: false
	}
}


Support for progress bar based on arbitrary position (e.g. file position)

I'm trying to use pyprint to add a progress bar for processing a large textfile.
My understanding is that that pypring needs the total number of loop iterations before-hand, in order to calculate the progress bars.
However, if you just have a large textfile, there is no way of telling how many lines there are without actually reading through the whole file.
It would be nice if there was some way of setting a progress bar, such that you could give it a total file-size, then update it with the current progress through the file - e.g.
import pyprind
import os
import time
n = os.path.getsize('queries.txt')

bar = pyprind.ProgBar(n)
with open('queries.txt', 'r') as f:
    for line in f:
        time.sleep(0.5)
        # do some computation
        # Use f.tell() for current position - pass to bar, somehow
        bar.update()

Hi, Victor,
thanks for the request, that sounds useful.
Would this solve your problem ?
num_lines = int(subprocess.check_output(['wc', '-l', 'path to file']).decode('utf8').split()[0])

I think getting the number of lines via the Unix/Linux 'wc -l'  tool is probably the most efficient way imho. Maybe sth like this could be added as additional "utils" to pyprind; it just needs to be tested on Windows

Is there any way of using the number of bytes instead of lines?
The reason is that for lines, you need to read sequentially through the entire file (even if using wc). In my case this is several GB each time/file.
Whereas for bytes, you can get the total size instantly, and file.tell() will also tell you your position.

I think that's a great idea! I have sth similar implemented here for reading a specific file format.
Right now, I have a book to finish, but in August, I will have more time for tinkering and could try to implement sth. like that. Or if you'd want to do it, you are very very welcome to submit a pull request!
One question is how to implement it Interface-wise. I don't want to make too many changes to the current usage since it may be annoying if it breaks people's code. So I am thinking to either pass a file buffer to to the iterations argument in ProgBar/ProgPercent or alternatively have a new class ProgReadfile with an argument kind to toggle between "percent" or "bar" indicator.
Maybe @DevMoore94 would be interested to implement sth. like that?

Plus a separate ProgReadfile class would have the advantage of having less "if-else" checks, which are potentially slowing done the code if the number of iterations is large.