[BUG] Rancher chart migration script failling

Describe the bug
I get the following error when I run the migration script:
Looking up Rancher Project App 'longhorn-system' ...
error: the server doesn't have a resource type "apps"
To Reproduce
Steps to reproduce the behavior:
./longhorn_rancher_chart_migration.sh -u PATH -d PATH -t migrate
Expected behavior
No error
Log or Support bundle
If applicable, add the Longhorn managers' log or support bundle when the issue happens.
You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment
Rancher 2.6.6

Longhorn version: 1.2.4
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Rancher Catalog App
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of management node in the cluster: 3
Number of worker node in the cluster: 9

Node config

OS type and version: Ubuntu 20.04 LTS
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Bare metal
Number of Longhorn volumes in the cluster: 10

Additional context
Add any other context about the problem here.

I confused the upstream kubeconfig file. In my case, the right upstream kubeconfig was from the local cluster.
The upstream concept is not clear in the documentation (step 2 of https://longhorn.io/kb/how-to-migrate-longhorn-chart-installed-in-old-rancher-ui-to-the-chart-in-new-rancher-ui/) and I suggest it is clarified.

I confused the upstream kubeconfig file. In my case, the right upstream kubeconfig was from the local cluster.
The upstream concept is not clear in the documentation (step 2 of longhorn.io/kb/how-to-migrate-longhorn-chart-installed-in-old-rancher-ui-to-the-chart-in-new-rancher-ui) and I suggest it is clarified.

In the KB, the upstream means your rancher cluster, and the downstream is for the cluster Longhorn is running on. Any part you suggested to clarify?
Download the kubeconfig file for the upstream rancher cluster. Download the kubeconfig file for the downstream cluster where Longhorn is running on.

BTW, did the migration work after using the right kubeconfig?
cc @PhanLe1010

The first part would need clarification: Download the kubeconfig file for the upstream rancher cluster.
From my point of view, I installed Longhorn using the catalog in the same cluster as Longhorn is running. This is where I can see it in the Legacy/Project section. I did not expect it was referring to the local cluster. I don`t know if the Rancher is always named local but it could be added.
Yup the migration looks fine. Nothing broke yet. Hope the update to 1.3 will good smooth.

The first part would need clarification: Download the kubeconfig file for the upstream rancher cluster.
From my point of view, I installed Longhorn using the catalog in the same cluster as Longhorn is running. This is where I can see it in the Legacy/Project section. I did not expect it was referring to the local cluster. I don`t know if the Rancher cluster is always named local but it could be added.

It seems your point is more about the cluster name on Rancher UI confusing you when the cluster is the same for upstream & downstream in your case.
@PhanLe1010 What do you think @courcelm suggestion?

Yup the migration looks fine. Nothing broke yet. Hope the update to 1.3 will good smooth.

It's good to know it works well 👍

I don`t know if the Rancher cluster is always named local but it could be added.

It is not always local

@PhanLe1010 What do you think @courcelm suggestion?

I modified the doc to mention that it is the kubeconfig file of the cluster on which the rancher instance is running on. Thank you


Flutter v2.2

I have been in issues trying to use this package with Flutter 2.2.
The runtime.diff is for dark sdk, not flutter. README does not clarify which repo.. I assume I need to build dark sdk first then flutter engine? Any chance we don’t need it anymore in Flutter 2.2?
Right now, if I add this package to pubspec and not importing it in any code, I get Can't load Kernel binary: Invalid kernel binary format version and other errors when I start android build.

This is what I am seeing when trying to build for Android with Flutter 2.2.
E/flutter (10789): [ERROR:flutter/shell/common/shell.cc(242)] Dart Error: Can't load Kernel binary: Invalid kernel binary format version.
E/flutter (10789): [ERROR:flutter/runtime/dart_isolate.cc(169)] Could not prepare isolate.
E/flutter (10789): [ERROR:flutter/runtime/runtime_controller.cc(401)] Could not create root isolate.
E/flutter (10789): [ERROR:flutter/shell/common/shell.cc(571)] Could not launch engine with configuration.

I can confirm it can run up to Flutter v2.0.6. Probably the binary used is not compiled for v2.2 yet.

Hi, thanks for reporting.
Indeed the Preview version of the SDK supports specific engine versions and has other limitations listed here: https://github.com/realm/realm-dart#limitations.
Our next SDK version will remove this limitation

Any timeframe for the new sdk version?
I'm currently looking for a persistence layer for my app and really would like to use Realm :)

Any timeframe for the new sdk version?
I'm currently looking for a persistence layer for my app and really would like to use Realm :)

Right now, I have to use flutter v2.0.6 for the project and hope they will release a new version before publishing my project 🤣

use
flutter clean
then
flutter pub get
try running the app from terminal
flutter run
This method worked for me 👍


Multiple Tenants/Orgs Fails to Upload Data to InfluxDB2

Doesn't pull data when there are multiple organizations (tenants) within Veeam M365 on Veeam M365 v7
Error: failed to write data: 400 Bad Request: unable to parse 'veeam_office365_jobs,veeamjobname=SharePoint,\ OneDrive,\ and\ Teams,bottleneck=Source totalDuration=2324,status=1,processingRate=2770350,readRate=2446599,writeRate=3329979,transferredData=6432304892,processedObjects=393<PHONE_NUMBER>': missing tag value

The issue is resolved.  The Job named can NOT have ","   Removed the comma and the script works.

I think the issue here is that the spaces are being escaped properly but commas are not.
nameJob=$(echo "$veeamJobsUrl" | jq --raw-output ".[$arrayJobs].name" | awk '{gsub(/ /,"\\ ");print}')

Adjusting the script so awk will escape commas seems to do the trick:
nameJob=$(echo "$veeamJobsUrl" | jq --raw-output ".[$arrayJobs].name" | awk '{gsub(/ /,"\\ "); gsub(/,/, "\\,"); print}')


Fix handling of nested nulls in min_by and max_by Presto Aggreates

In Presto, min_by and max_by functions with complex types for the "compare" parameter should be checked and thrown in case of nested nulls in "compare" data. For example, the following query would throw an exception, "ARRAY comparison not supported for arrays with null elements",
presto> SELECT min_by(col0, col1) FROM (VALUES (5, array [1, 2]), (4, array [null, 3])) AS tbl(col0, col1);
Query 20230916_120046_00011_28jc9 failed: ARRAY comparison not supported for arrays with null elements

However in Spark, the above query would return 4,
spark-sql> SELECT min_by(col0, col1) FROM (VALUES (5, array (1, 2)), (4, array (null, 3))) AS tbl(col0, col1);
4

This PR uses the methodSingleValueAccumulator::compareadded in https://github.com/facebookincubator/velox/pull/6570 to make the "compare of min_by/max_by aggregate functions able to detect nested nulls that

use compare flag  StopAtNull in presto
use compare flag NoStop in Spark.

Part of https://github.com/facebookincubator/velox/issues/6314

@duanmeng Thank you for the fix. There are CI failures. Would you take a look?

@duanmeng Thank you for the fix. There are CI failures. Would you take a look?

@mbasmanova  OK got it. Lots of failures related to velox/exec/tests/MultiFragmentTest.cpp in https://app.circleci.com/pipelines/github/facebookincubator/velox/33676/workflows/2e2dda94-e5a1-476b-88c6-45bb633b9321/jobs/218607. I'll rebase and rerun the test locally after resolving your comments.


Language files seem to indicate Spanish (Español / es) rather than English (en)

Paths like lang/es/index.md have usually referred to Español in my experience. Were these files intended to be placeholders for Spanish?

@jameswomack yeah, I left this file as a placeholder indicating how we could eventually internationalize -- this is the file structure used by semver.org.

@bcoe got it


Fix ArrayDataProvider bug with pagination

Fixes #7074. Don't slice models in ArrayDataProvider when pageSize is set to 0

Fixed. Thanks!


Failed to build microservices tutoriel

Hi,
Build fail when executing ~/microservice $ mvn -pl rest-app package, the following error message is displayed

[ERROR] Failed to execute goal on project rest-app: Could not resolve dependencies for project org.osgi.enroute.examples.microservice:rest-app:jar:0.0.2-SNAPSHOT: The following artifacts could not be resolved: org.osgi.enroute.examples.microservice:dao-impl:jar:0.0.2-SNAPSHOT, org.osgi.enroute.examples.microservice:rest-service:jar:0.0.2-SNAPSHOT, org.osgi.enroute:osgi-api:pom:7.0.1-SNAPSHOT, org.osgi.enroute:impl-index:pom:7.0.1-SNAPSHOT, org.osgi.enroute:debug-bundles:pom:7.0.1-SNAPSHOT: Could not find artifact org.osgi.enroute.examples.microservice:dao-impl:jar:0.0.2-SNAPSHOT in OSGi (https://oss.sonatype.org/content/groups/osgi)

Any suggestions ! I'm a beginner with this framework .. I will be very grateful for your help, thanks

@timothyjward may you help me

Closing as I believe this is now working.


NOTICE: upcoming default branch name change

The master branch of this repository will soon be renamed from master to main, as part of a coordinated change across the USGS-R and USGS-VIZLAB organizations.  This is part of a broader effort across the git community to use more inclusive language. For instance, git, GitHub, and GitLab have all changed or are in the process of changing their default branch name.
We will make this change early in the week of February 6, 2022. The purpose of this issue is to give notification of the change and provide information on how to make it go smoothly.
When this change is made, forks and clones of the repository will need to be updated as well. Here are instructions for making the necessary changes.

[ ] On GitHub, change the default branch name from master to main (<your repository> -> Settings -> Branches).
[ ] Change the default branch on any forks you have, as well as local clones. (See details below)
[ ] If you have collaborators on this repository, let them know that they will need to change their forked/local repos. Point them to this issue to facilitate the process!
[ ] Search within your repository for "master" so that you can change references (e.g. URLs pointing to specific commit points of files) to point to "main" instead.
[ ] When you are done, feel free to close this issue!

Changing default branches on forks and local clones
First, update your local git configuration so that new repositories created locally will have the correct default branch: git config --global init.defaultBranch main.
Now, for any forks, do the following:

Go to <your fork> -> Settings -> Branches and edit the default branch from master to main.
Update the settings for your local clone of this fork to match this change.

git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

This repository has now been converted, and its default branch is now main instead of master. Please see above for instructions on how to adjust any forks and/or clones you have.
@ceenell, @srikar-eranky, @Si-Campbell, @abybordi, @michellehoda, @gregorydillon, @shroomeater, @persep, @lindsayplatt, @hc10024, @lukereding, @penelopeaton, @bestwpw, @wdwatkins, @McGunagle, @RiggsOwen, @joshnr13, @HydroLogic, @xiaoli-chen, @naturalsoda, @lisashiozaki, @jmolayem, @chrisriggs, @xuexianwu, @jread-usgs, @wokkaflokka, @nvirk, @kalbury, @andersoner, please note that you have forks of this repository!


Adjust permissions for Troubleshooters

Rationale
Troubleshooters should be able to view metrics information and shouldn't be able to invoke expensive tests on data sources. We also should check for read permissions in folders with external schemas before linking to them.

This seems OK but it would be good to capture this in our definition of the Troubleshooter role so we can aim to be consistent going forward. Until now, I think troubleshooters could effectively do all read-only actions. Now they can do almost all read-only actions.
https://www.labkey.org/Documentation/wiki-page.view?name=permissionLevels

Yeah, bit of a judgement call here. They can still read the details of all data sources, they just can't invoke the potentially time-consuming test actually that's really intended for development & testing of dialects. (I debated making the test button devMode-only.) I added "(not) invoke resource-intensive processes" to the doc, which is consistent with how I've been tagging actions. We don't let Troubleshooters invoke system maintenance tasks or clear the query profiler, which aren't technically "changing settings."


VirtualBox Guest additions check sum not matching

I haven't used packer before. Was refered to this as a solution to building a VM development machine.
I cloned the repo, installed packer and ran the commad specified on the README.md
I got the following error.
 virtualbox-iso: Downloading or copying: http://cdimage.ubuntu.com/ubuntu/releases/bionic/release/ubuntu-18.04-server-amd64.iso     virtualbox-iso: Error downloading: checksums didn't match expected: 73ae6579ef7c51d944a0be5c4c48f748bfd689df ==> virtualbox-iso: ISO download failed. Build 'virtualbox-iso' errored: ISO download failed.
I presume you are using a version you already downloaded. Could you let me know what version so I can match it please.
is it possible to update this so its negated for others in the future?

I created pull request #2 which I believe addresses this by updating to 18.04.1 and updating the checksum.

Thanks! I'm sorry I didn't notice immediately.


Hard failure on bad data file

The following gist demonstrates a case where node-osmium fails when simply reading an xml file: https://gist.github.com/rclark/b957f10a435a8c71827e
Error message looks like this:
node: ../../libosmium/include/osmium/osm/node_ref_list.hpp:92: const osmium::NodeRef &osmium::NodeRefList::front() const: Assertion `!empty()' failed.

My initial suspicion was that the data file is bad (which it is) -- there are a number of ways included in the file without any references to nodes:
    <way changeset="32919043" id="362662527" timestamp="2015-07-27T21:35:46Z" uid="188704" user="Karta" version="1"/>
    <way changeset="32918129" id="362658561" timestamp="2015-07-27T20:45:32Z" uid="188704" user="Karta" version="1"/>
    <way changeset="32918129" id="362658337" timestamp="2015-07-27T20:44:39Z" uid="188704" user="Karta" version="1"/>
    <way changeset="32918356" id="362660420" timestamp="2015-07-27T21:07:29Z" uid="188704" user="Karta" version="1"/>
    <way changeset="32918356" id="362660287" timestamp="2015-07-27T21:03:57Z" uid="188704" user="Karta" version="1"/>
    <way changeset="32918356" id="362660719" timestamp="2015-07-27T21:12:34Z" uid="188704" user="Karta" version="1"/>
    <way changeset="34673665" id="362660218" timestamp="2015-10-16T11:42:30Z" uid="463879" user="tumsi" version="3"/>
    <way changeset="32918356" id="362660523" timestamp="2015-07-27T21:11:10Z" uid="188704" user="Karta" version="2"/>

However, I couldn't reproduce the problem by just isolating those ways.
My biggest concern here is not that node-osmium fails on this crummy data file, but that it fails in such a way that the error cannot be caught and handled within node.js.
cc @joto

oops @joto I had forgotten some prior conversation we had about this. Do you think that https://github.com/osmcode/libosmium/commit/657579bb27018d2af15266a04d13c90190aeee71 applies to this case? If so, can we consider a patch bump on node-osmium to pre-bundle with a new libosmium?

With osmium<EMAIL_ADDRESS>and the testcase (https://gist.github.com/rclark/b957f10a435a8c71827e) on OS X I don't see an assertion/abort but rather a JS error:
$ node a-simple-test.js

/Users/dane/projects/b957f10a435a8c71827e/node_modules/osmium/lib/osmium.js:84
            this._reader = new osmium.FlexReader(file, location_handler);
                           ^
TypeError: XML parsing error at line 5707, column 4: unclosed token
    at new exports.Reader (/Users/dane/projects/b957f10a435a8c71827e/node_modules/osmium/lib/osmium.js:84:28)
    at Object.<anonymous> (/Users/dane/projects/b957f10a435a8c71827e/a-simple-test.js:5:32)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:935:3
~/projects/b957f10a435a8c71827e[master]$ npm ls osmium
/Users/dane/projects/b957f10a435a8c71827e
└──<EMAIL_ADDRESS>

@springmeyer like I said yesterday, the gist truncated the data file so that led to your seeing an XML parsing error in JS.
The good news is, I just came across an XML doc that hits the assertion and crashes in<EMAIL_ADDRESS>but does not crash in<EMAIL_ADDRESS>Thanks for the build!


Possibility to customize the label of the radial track

I would like to display "$10" instead of "10.0" on the label of the radial track
Currently, labels on the radial track are for example "10.0", "20.0", "30.0", and I would like them to be for example "$10", "$20", etc
It seems there is currently no way to change how these labels are displayed.
Add a RadialTrack constructor argument to convert double track number to string label
It would be great if the RadialTrack constructor accept a function String Function(double) mapTrackLabel as argument.
Then, when drawing the label, instead of calling exactValue.toString(), mapTrackLabel(exactValue) may be called.
The default value for mapTrackLabel may be (double value) => value.toString() to keep current behavior.
Thanks for your great flutter gauge indicator !!

I have done a PR to try to implement that here : https://github.com/GeekyAnts/GaugesFlutter/pull/246

Hello everyone, any news about this? This is feature is exactly what I'm looking for.

Hi, is this feature available in geekyants_flutter_gauges: 1.0.1?

Hi @LudoPL
I have merged your PR in the development branch and it can be used as a workaround.
Usage in pubspec.yaml
geekyants_flutter_gauges:
   git:
      url: 'https://github.com/GeekyAnts/GaugesFlutter.git'
      ref:development

cc : @PushBackNeil @Huascar321 @Afroz-Shaikh

We would like to use this feature as decimal points take up too much space on our dashboard (0 - 8000 RPM).  Can we get this released so we don't have to use a development branch in production?

We would like to use this feature as decimal points take up too much space on our dashboard (0 - 8000 RPM).  Can we get this released so we don't have to use a development branch in production?


:lady_beetle: Prevent slash from appearing in coverage random ID

Changes

:lady_beetle: Prevent slash from appearing in coverage random ID

/kind bug
Relations
Fixes https://cloud-native.slack.com/archives/C04LY4M2G49/p1725366161644159
Follows up on https://github.com/knative/actions/pull/223

/cc @dsimansk


Multiple currencies v2

Added the ability to donate in multiple different currencies. This is a follow-on to the closed PR. Specifically by:

Adding functionality to convert between any two currencies, together with appropriate unit testing
Adjusting front-end to select a major currency to donate in (sats, USD, EUR, AUD, JPY, GBP)
To be able to set currencies via configuration

Also as part of the PR, I have removed unused dependencies in the files I have used / updated formatting to PEP8.
As per comment here, I haven't yet been able to test this. Also, I'm relatively new to Javascript - presumably the user will need to install npm for this to work? I know this isn't perfect but thought better to get this in a PR rather than sitting on it any longer, happy to action any comments / suggested changes swiftly

I'm relatively new to Javascript - presumably the user will need to install npm for this to work?

No, that is needed only for server side JS (nodejs), not client side (browser). But new JavaScript code could and should be avoided here (although that could be done in a follow-up PR), see #47.

Should remove base_currency and add supported_currencies to config.py too.
[2022-06-27 16:28:55 +0300] [ERROR] Exception on / [GET]
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/user/.local/lib/python3.9/site-packages/flask_restplus/api.py", line 584, in error_router
    return original_handler(e)
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/user/.local/lib/python3.9/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/user/git/SatSale/satsale.py", line 54, in index
    params["currency"] = config.supported_currencies
AttributeError: module 'config' has no attribute 'supported_currencies'

All done - thanks for your patience on this, do let me know if anything else is required

Now there are no errors, but dropdown is empty. Currently busy with some other stuff, but I could just debug and fix remaining stuff myself, otherwise it's going slowly, but I need this stuff for myself soon. :)

Hope that does the trick, should hopefully be swift to change if not.

Ok, finally got time to test. Now there is different error:
[2022-07-14 00:40:00 +0300] [ERROR] Exception on /api/createpayment [GET]
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/user/.local/lib/python3.9/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/user/.local/lib/python3.9/site-packages/flask_restplus/api.py", line 325, in wrapper
    resp = resource(*args, **kwargs)
  File "/home/user/.local/lib/python3.9/site-packages/flask/views.py", line 89, in view
    return self.dispatch_request(*args, **kwargs)
  File "/home/user/.local/lib/python3.9/site-packages/flask_restplus/resource.py", line 44, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/home/user/git/SatSale/satsale.py", line 146, in get
    btc_value = PriceFeed().get_price(base_amount, currency)
  File "/home/user/git/SatSale/payments/price_feed.py", line 63, in get_price
    currency_from_price = self.get_btc_price(currency_from)
  File "/home/user/git/SatSale/payments/price_feed.py", line 55, in get_btc_price
    btc_price = self.api_results[currency.lower()][value_attribute]
AttributeError: 'NoneType' object has no attribute 'lower'

@eddiepease I will just create new PR based on your code (with you as a co-author, of course), that way will be faster, have time for that now.


Reinstate manifest sandboxing

Manifest loading has been sandboxed on macOS for a while, but the change in #2518 broke it for 5.3.
https://bugs.swift.org/browse/SR-13346
rdar://problem/66586184

@swift-ci please smoke test

Looks like the 5.3 profile is actually too restrictive.

We need to allow process launches, so that we can even launch the compiled manifest. We also need to allow reads, because otherwise we can't load the PD dylibs.

@swift-ci please smoke test

14:36:51 2020-08-05 14:36:28.668 foo-manifest[77032:57189500] Failed to obtain sandbox extension for path=/Users/buildnode/Library/Caches/foo-manifest. Errno:1
14:36:51 Fatal error: 'try!' expression unexpectedly raised an error: Error Domain=NSCocoaErrorDomain Code=260 "The file couldn’t be opened because it doesn’t exist." UserInfo={NSURL=http://swift.org, NSUnderlyingError=0x7f82ef804180 {Error Domain=NSURLErrorDomain Code=-1003 "A server with the specified hostname could not be found." UserInfo={NSUnderlyingError=0x7f82de40c8c0 {Error Domain=kCFErrorDomainCFNetwork Code=-1003 "A server with the specified hostname could not be found." UserInfo={NSErrorFailingURLStringKey=http://swift.org/, NSErrorFailingURLKey=http://swift.org/, _kCFStreamErrorCodeKey=-72000, _kCFStreamErrorDomainKey=10, NSLocalizedDescription=A server with the specified hostname could not be found.}}, NSErrorFailingURLStringKey=http://swift.org, NSErrorFailingURLKey=http://swift.org, _kCFStreamErrorDomainKey=10, _kCFStreamErrorCodeKey=-72000, NSLocalizedDescription=A server with the specified hostname could not be found.}}}: file /var/folders/_8/79jmzf2142z2xydc_01btlx00000gn/T/TemporaryFile.tFKpoE.swift, line 3
14:36:51 Test Case '-[PackageLoadingTests.PackageDescription5_2LoadingTests testManifestLoadingIsSandboxed]' failed (2.533 seconds).

Hm, looks like we were able to leave the sandbox on CI 🤔

Failure happens on both macOS CI hosts, so looks like it is consistent. Locally, the test works as expected, the network call is denied.

maybe use a different action that does not require network access?

I was intentionally using a network request since it seems important to test that this is still blocked.

@swift-ci please smoke test

Looks like using an IP address instead of a domain works.


Check assets:clobber is defined before enhancing

Related to a1d4c6b5e86f0b0da7c9e2a8de506c1497051dc2 and #58.
Now that sprockets is no longer a dependency, my upgrade to Rails 7 got stuck because assets:clobber was no longer a defined task in my local Rails app.

Duplicate of https://github.com/rails/cssbundling-rails/pull/71


Chat view: fix timezone issue

For messages that come back from the API (and presumably the websocket) it ends up displaying "in 2 Stunden" for new messages. This is because the existing foodsharing site returns dates without timezone offset information in local German timezone, the frontend interprets them as UTC (by appending +0000).
The +0000 was added during development as it was displaying the messages as 2 hours ago... Should also resolve timezone for the docker setup so it is consistent with the live foodsharing.de service.

I turned it off for now in https://github.com/foodsharing-dev/foodsharing-light/commit/9c1a0cf8db9a9c385c530a290bc18b86568291b7


docs: adds testing manual

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 6 committers have signed the CLA.:x: evenyag:x: v0y4g3r:x: zyy17:x: killme2008:x: waynexia:x: shuiyisongYou have signed the CLA already but the status is still pending? Let us recheck it.

I'm very sorry, I made a mistake with the upstream branch.

No worries @killme2008, thanks for reaching out. Feel free to reopen or create another PR.


Generic header handler

What kind of change does this PR introduce?
Added a builder method to add a custom header to support things like header API key auth like Kong in SupaBase.
What is the current behavior?
The current auth() method only supports bearer JWT formatted authorization headers.
What is the new behavior?
Additional custom headers can be added one at a time as needed in the builder for the request.
Additional context
This is the compromise solution to https://github.com/supabase/postgrest-rs/pull/25

Thanks again for the PR :)


Add basic integration tests

This PR adds the ability for integration tests to be run so that as we improve the project generator, we can guarantee the expected behavior without having to run it manually.
At the moment, this PR is pending some improvements to how paths are treated in the program (will come in another PR) so that we can distinguish  the base path from the output path.

@liamfraser Do you remember why we had to set up a self-hosted runner? Is it worth re-using that here?

@liamfraser Do you remember why we had to set up a self-hosted runner? Is it worth re-using that here?

Yes, we could do. It is only currently used for the sdk and examples. Mainly because we had a spare machine and didn't want to pay for cloud time.


Length of MFA session

In v1, sessions that connect using MFA ended after an hour. Is this still a limitation in v2?

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

ID: de2836a0-0af3-c147-2d4a-78220ac4e0f1
Version Independent ID: 2cde9735-6281-bb9f-bf07-b30b2f6c7f37
Content: Exchange Online PowerShell V2
Content Source: exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

Hi @Dottn, thank you for your feedback.
Sorry but I don't have this information.
The Exchange Online PowerShell V2 module is currently in Preview. If you have any feedback, concerns, or are facing any issues with the EXO V2 module, contact us at exocmdletpreview[at]service[dot]microsoft[dot]com (email address intentionally obscured to help prevent spam).

@dariomws306  Thank you very much for the contribution and sharing this explanation. @Dottn
Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


WIP: Support using symlinks to find correct driver

Addresses oracle/oci-volume-provisioner#131
The volume provisioner has been patched to now not use names to dispatch custom logic. A storage class can now use it's provisioner property to determine what driver should be used. At the moment only block volume storage is supported provisioner: oracle.com/oci-bvs with aim that file storage will soon be supported also oracle/oci-volume-provisioner#90. Backwards compatible so that existing storage classes using provisioner: oracle.com/oci will default to using the block volume storage however this should be marked as deprecated and removed in a future release.

Note: Only needed if FSS Dynamic Shared feature is implemented. There’s only one driver currently so it’s not needed. If we need to use the driver as part of the phase 2 FSS work we’ll need the above or another flex driver binary


Zero downtime Deployments and HPA with services using Istio

Describe the feature request
When rolling out new pods in Kubernetes, it's a common solution to add a preStop lifecycle hook to wait before cleaning up the pod so that the application can handle the sigterm. In Istio, this doesn't work because the cause is different -- it's based on local caching.
Details for these are written up very well by @sdaschner here:
https://blog.sebastian-daschner.com/entries/zero-downtime-updates-kubernetes
https://blog.sebastian-daschner.com/entries/zero-downtime-updates-istio
The solution for Istio, then, is to use blue/green deployments -- which also works for native Kubernetes. This solves deployments, but then we're stuck without using HPA. I'd like to see something more native to Istio that solves this problem for both Deployments as well as HPA.
Describe alternatives you've considered
As mentioned above, blue/green was considered but it only solves the issue for actual deployments and not HPA. Using Istio retries was also considered but it feels more like a band-aid.

Not stale.

@jaygorrell we introduced a new feature to do a graceful termination of Envoy. So when a sigterm is received, envoy will stay alive for 5s (by default) and stop accepting new connections before shutting down.
This seems to solve the problem you are having, and from my testing it seems to pretty reliably allow zero-downtime deployments. I have tested with deploying a server repeatedly while sending requests and seeing no 503s.
Is this sufficient for this or are there other changes required as well?

@howardjohn What version is that in?
It's typical to add a preStop hook with a sleep on k8s deployments so the service changes can propagate (removing the pod from the pool) before the pod is actually told to shut down. Then the pod receives the usual termination handler. With that change, it would replace this preStop hook, right?
Aside from that, I'm not sure if this really solves this issue, at least conceptually - I haven't tested the new changes. It sounds like there would still be the eventual consistency problem that's sorta' the focus of this issue that requires 5xx retries.
This sounds like it would be more of a fix for allowing a pod to continue with outbound requests after the sigterm is received. I'll check it out though.

It should be in 1.1

Please try out the new feature and open an issue if that isn't working. Thanks!


check failed: !PyErr_Occurred() when constructing two uint64 tensors

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.17.0
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When running the following code, tensorflow will directly raise program abort with the error message: ./tensorflow/python/eager/pywrap_tensor_conversion.h:58] Check failed: !PyErr_Occurred()
import os
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '3'
os.environ['CUDA_VISIBLE_DEVICES'] = ''
import warnings
warnings.filterwarnings("ignore")
import tensorflow as tf

lower1 = -1
try:
    lower1 = tf.constant(lower1, dtype='uint64')
except:
    ...
lower2 = -2
lower2 = tf.constant(lower2, dtype='uint64')

It seems the problem occurs when TensorFlow tries to construct two uint64 tensors. Although it is invalid to convert negative int to unsigned, an exception is more proper as program abort will directly kill the process.
Indeed, only constructing one uint64 tensor will properly raises an OverFlow exception.
This issue only occurs when repeatedly constructing two uint64 tensors.
Another weird thing is that, if I change the value of lower2 to either -1 or -3 instead of -2, this issue does not occur.
Standalone code to reproduce the issue
import os
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '3'
os.environ['CUDA_VISIBLE_DEVICES'] = ''
import warnings
warnings.filterwarnings("ignore")
import tensorflow as tf

lower1 = -1
try:
    lower1 = tf.constant(lower1, dtype='uint64')
except:
    ...
lower2 = -2
lower2 = tf.constant(lower2, dtype='uint64')

### Relevant log output

```shell
F ./tensorflow/python/eager/pywrap_tensor_conversion.h:58] Check failed: !PyErr_Occurred()
Aborted (core dumped)

@maybeLee,
Thank you for reporting the issue. I was also able to reproduce the issue on tensorflow v2.17 and tf-nightly. Kindly find the gist of it here.
Please allow to deepdive into the issue. Thank you!


google map is now loaded async by default and always require api key. Which breaks existing examples.

Notice from google:
Updates to Google Maps APIs Standard Plan

We no longer support keyless access (any request that doesn't include an API key). Future product updates are only available for requests made with an API key. API keys allow us to contact developers when required and help us identify misbehaving implementations.

Much troublesome to setup new demos >_< ..
And besides requiring an API key, the google map JS libraries are now loaded async by default...
For old demo to work on new sites.. need to add both API key=xxx and aysnc=false
Like this...
<script src="https://maps.google.com/maps/api/js?key=<YOUR API KEY>&async=false"></script>

Thanks for such a nice ngDirective  👍

Thanks, but I don't manage to have the autocomplete example/demo working with the key and the async=false
Did you manage to have it working too?


[new feature]: add CLI command to show pending chain transactions and payments

Basically do what /pending does on the telegram bot but output for the CLI

Is there a way to see in flight forwarding requests and/or requests which failed and the reason? Especially via Telegram? i.e. get a notification when something is pending?
I seem to remember hearing of this in a youtube video years ago but can't see anything like that in the documentation.

Notifications would be annoying but you can see things in flight with /pending in telegram


CSV import with non utf-8 files

Importing a well-formatted csv file with students (for example) in a non utf-8 format (with iso-8851 in my case) blocks MarkUs.
Students are correctly imported (updated or created). But, in the sorting page, nothing appears.
You can try it using a downloaded csv file from MarkUs and converted it to iso-8851 encoding with iconv.

Alex Krassikov will be working on this issue.
Looked into it. From the looks of things, regular characters are parsed correctly, but special characters like e with an accent disappear.
Apparently FasterCSV (what we use for parsing) does not support ISO-8859-1, but in ruby 1.9 the native CSV parser might. Currently looking into using iconv for conversion in our version of ruby.

Good :)
Don't hesitate to put Ruby1.8 selectors in your code if it is only Ruby 1.8 compliant.
Have a look at my branch here : https://github.com/benjaminvialle/Markus/tree/ruby19
I am working on Ruby 1.9 compatibility. You will find some "Ruby selectors" in this code. For example :

if RUBY_VERSION > "1.9"
require "csv"
unless defined? FCSV

 class Object

   FCSV = CSV

   alias_method :FCSV, :CSV

 end

end
unless defined? FasterCSV

 class Object

   FasterCSV = CSV

   alias_method :FasterCSV, :CSV

 end

end
else
require "fastercsv"
end

Thank you to work on this issue

Review for this is here: http://review.markusproject.org/r/1135/ It only requires some minor finishing touches. @Nifter has already completed most of the work.

This depends on code from #426.


Optimize all card images

Fixes #62

Dup

Sorry :P

Hey @MaherFa could you take a look at this please? I think it looks good but as you did this work, maybe you have a comment or something. Thanks!

@martincarrera I haven't looked at all the images, but I assume it's removing extra pixels for better cropping.
I vote merge as long as they have the same quality and aspect ratio

Great, thanks!


Added pickpocket stun timer to timers plugin.

A simpler implementation of the feature request brought up in #12720.

Timer displays how long the player is stunned for (5 seconds for most pickpocket targets).

5 seconds isn't a multiple of 0.6. Do stuns work outside the tick system? 😮


[Bug Report]: Disabling react.refresh and devServer.hot is not work for stopping injecting HMR runtime code.

System Info
No response
Details
[Bug Report]: Disabling react.refresh and devServer.hot is not work for stopping injecting HMR runtime code.
Reproduce link
No response
Reproduce Steps
No response

closed it because it isn't exits


Support KMS ARN

Adds support for KMS ARN that encrypts secrets at REST

thanks!

Nice, thank you @DaveBlooman! 🙌


[stable/sonarqube] can not upgrade

Describe the bug
I can't upgrade sonarqube.
# helm ls -n sonarqube-prod
NAME                	NAMESPACE     	REVISION	UPDATED                                	STATUS  CHART          	APP VERSION
sonarqube-prod-cloud	sonarqube-prod	1       	2019-11-14 06:49:15.507976927 +0000 UTC	deployedsonarqube-3.2.5	7.9.1
# helm upgrade sonarqube-prod-cloud -n sonarqube-prod -f sonarqube-prod-cloud-values.yaml stable/sonarqube --dry-run
Error: UPGRADE FAILED: unable to build kubernetes objects from current release manifest: unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1"

Version of Helm and Kubernetes:
k8s : 1.16.4
helm 3.0.3
Which chart:
stable/sonarqube 3.4.0
What you expected to happen:
I think postgresql has problem.
# k get deploy -n sonarqube-prod
NAME                              READY   UP-TO-DATE   AVAILABLE   AGE
sonarqube-prod-cloud-postgresql   1/1     1            1           81d
sonarqube-prod-cloud-sonarqube    1/1     1            1           81d

Recently postgresql is changed to statefulset.

I cannot upgrade sonarqube as well, but I use an external instance of postgres.

The chart is broken. It now defines the following environment variables regardless of whatever you set your Postgresql values to:
+           env:
+             - name: SONARQUBE_JDBC_USERNAME
+               value: "sonarUser"
+             - name: SONARQUBE_JDBC_PASSWORD
+               valueFrom:
+                 secretKeyRef:
+                   name: sonarqube-sonarqube
+                   key: postgresql-password
+             - name: SONARQUBE_JDBC_URL
+               value: "jdbc:postgresql://%!s(<nil>):5432/sonarDB"

Check your values file.
In my setup I have external Postgres container with PVC that is attached with sonarqube pod. When I upgraded the sonar to use new image , I met with this error.
If we would use StatefulSet , it might be handled or is this the issue with the update strategy ?
I am only running single pod of sonarqube.
Events:
  Type     Reason              Age                   From                                       Message
  ----     ------              ----                  ----                                       -------
  Warning  FailedAttachVolume  62m                   attachdetach-controller                    Multi-Attach error for volume "pvc-02341115-174c-11ea-847b-6aba0db3a668" Volume is already used by pod(s) sonarqube-sonarqube-7b4bdfd759-phc98
  Warning  FailedMount         3m7s (x297 over 11h)  kubelet, aks-basepool-20035195-vmss000075  Unable to mount volumes for pod "sonarqube-sonarqube-57d4b84cbd-f8pww_dev-qa-tools(d359663c-1b9c-11ea-847b-6aba0db3a668)": timeout expired waiting for volumes to attach or mount for pod "XXX-namespace"/"sonarqube-sonarqube-57d4b84cbd-f8pww". list of unmounted volumes=[sonarqube]. list of unattached volumes=[config install-plugins copy-plugins sonarqube tmp-dir default-token-ztvcd]


Text Align not working on iOS. iphone 6 ios 9.2

Short description of the problem:
I have a modal where i have a couple of text inputs, one of type "tel", the other of type "date", i have added the 'style="text-align: right"' attribute to these text inputs, and it does not seem to work on the phone in ionic-view, thus i am suspecting that it won't work on the regular phone as well.
Based on an earlier thread looks like this issue is on android as well.
here is the issue link: [https://github.com/driftyco/ionic/issues/1929]
What behavior are you expecting?
the text should align right on the phone
Steps to reproduce:
I have forked a similar codepen and made changes to what I have in my app, where a modal shows up. I am using the form in a modal as well, thus this issue is pretty similar in my case as well.
http://codepen.io/soul_/pen/WrBpLG
Once you run the code pen, click on "Form" button and type any content in it, on the browser it will align fine. It align's fine in the emulator as well, the issue is on the real device though.
//code is there in the link above in codepen.
/*For me in the preview, the code pen did not seem to take me to code pen, i had to manually copy the codepen link and paste it into the browser.*/

Ionic Version: 1.7.14
Browser & Operating System: iOS / Android
Run ionic info from terminal/cmd prompt:
Your system information:

Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Version: 1.0.0
Ionic CLI Version: 1.7.14
Ionic App Lib Version: 0.7.0
ios-deploy version: 1.8.3
ios-sim version: 5.0.4
OS: Mac OS X El Capitan
Node Version: v4.2.4
Xcode version: Xcode 7.2.1 Build version 7C1002

Please see below for images for the issue on the phone:
input type "tel"

input type "date"

I have same issue here, have tried using display on css. But, not fixed.
Have I missing something?

any traction on this?

maybe will be fix on next update @gnasit :)

Sorry the delay :smile: ! So in order to debug this, we're going to need an example to test. Please provide a codepen that we can load on to our devices and test. Thanks!

http://codepen.io/soul_/pen/WrBpLG. I have explained the instructions as well in the issue above.

may i please know how often do issues get looked upon?

This issue was moved to driftyco/ionic-v1#196


Fix multihorizon kerchunk dimensions

When there are multiple time dimensions in a dataset, the kerchunk mapping will conflate all of the time readings and mash them together, when instead there should be different time dimensions for each time horizon.
This is handled in cfgrib by the filter rules and not letting multiple time horizons, but I do not want that solution

I think the way to do this is to use attributes to remap the times the same way that the dataset reading process occurs.


A better format for the Hipchat messages

@feangulo  Can you please take a look at the Travis output.
Thanks!

@mattyjones sorry, please disregard, I created the PR is error... it was supposed to be to my own forked master branch.


fix(feather-light,flatwhite,one-light): 'nil value is invalid' warnings

Friendly ping @hlissner

Sorry for getting to this so late. Thanks for the help!


Use new version 0.51 of Ktfmt

⚠ Is your feature request related to a problem? Please describe

The IntelliJ Ktfmt plugin are now using different versions of Ktfmt.
It would be great, if this plugin could use also the latest version
💡 Describe the solution you'd like

There are some breaking changes in the latest version.
The dependency must be upgraded and the changes resolved
🤚 Do you want to develop this feature yourself?

[x] Yes
[ ] No

Please do @simonhauck I see that the PR from renovate failed.
I do have some time tomorrow to review your other PR and can I also look into doing the bump if you don't have capacity


Allow dumping of fuzzer final state on fuzzer runs

We have a lot of expects tests of the form 'perform a fuzzer action on a sample program, and check the final program and variable output'.  These could be a lot nicer if they are instead rewritten to regression tests over initial (minimal viable) programs and traces, similar to the trace replay regressions we have at the moment.  This would need us to be able to get at the final state - which might be a nice feature to have in general, for debugging.
This might need me to bypass the filter boilerplate a bit, I'm unsure.

Done; see -state-output.  (One day, it might be nice to let this be stdout.)


unbound creates an AAAA record for the link-local address

I'm running OPNSense 18.7.4 and have DHCPv4 and unbound enabled.
When doing a quad-A lookup against the router's hostname from another machine on the LAN network, it correctly returns the router's LAN IP, but also it returns the link-local IP, which I believe is incorrect:
# full hostname redacted
$ dig AAAA firewall.example.net +short
fe80::1:1
2601:410:0:4c37:20d:b9ff:fe4d:4fe4

I see the entries being set in /var/unbound/host_entries.conf:
local-data: "firewall.example.net AAAA fe80::1:1"
local-data: "firewall AAAA fe80::1:1"

I'm not an IPv6 expert but my understanding is that link-local addresses (fe80::/10) are effectively useless for anything except NDP and other lower-level stuff because the applications would need to properly handle a zone index and practically nothing except ping does.
So far I have not actually observed any problems arising from this so maybe the applications or libraries already know not to bother with a link local address by default.

Hi @cu,
I think unchecking "DHCP IPv6 Link-local" does what you want, but looking at it the "DHCP" is misleading and was likely added incorrectly.
In general, link-local is actually used for routing, gateway access and DNS with Unbound being particularly clever about link-local because binding supports scopes so unbound can listen on fe80::1:1%em0 correctly unlike e.g. SSH we discussed earlier. That being said, the code for this is still in the development version and the behaviour of SSH in 18.7.4 is what Unbound will do eventually in 19.1, but that has no bad impact in deployments.
If the option does work I'll change the label accordingly.
Thanks,
Franco

Ah yes, bitten by another configuration option I didn't bother to go looking for. :P Unchecking the "DHCP IPv6 Link Local" setting does remove that AAAA record. Although I'm not totally clear on how an AAAA record with a link-local address is useful, that's something I'll put on my todo list soon.
Feel free to close this bug if you want and thanks for taking the time to respond.

Always happy to discuss! Removed the "DHCP" from the label, will land in 18.7.5. :)
Cheers,
Franco


Migrate to Yeoman?

Been considering migrating this generator to Yeoman for quite a while. Some limitations of using Gulp as a scaffolding tool are showing up and these may become more difficult to address over time.
My main concerns are:

Testing: current test suites are brittle, but not on purpose. Some cases, like inspecting the contents of some generated files, are not covered yet. Writing some of these tests require reading the Gulp streaming internals directly. I can write the necessary fixtures for that (I'll have to research how to accomplish this, though), but I'm pondering if this is worth all the trouble.

gulp-install: incompatible changes made to this task plugin since 0.3.0 are incompatible with the current project setup. Version 0.4.0 conflicted Bower and tests. Now, Bower is gone from the template project, so upgrading to 0.4.0 is not a problem, but tests run much slower with that version. 0.5.0 completely breaks the test suites, and I couldn't get it work at all. Again, I may replace this plugin with a more suitable one, perhaps gulp-spawn but, again, I'm considering if I should be doing this.

Suffice to say that much of these issues are addressed in Yeoman already. Only thing that holds me back is there are many Yeoman generators related to Phaser published already.
Should I decide to move forward and begin developing a new project based on this one's project template, further development and support for slush-phaser-plus will cease and its npm package will be deprecated in favor of the new solution instead.
I may be adding more details to this issue later.

So many options for Yeoman, yet none seem to be keeping maintained. I think yours is the most recent and active scaffolding project for Phaser which caught my attention - but just seen this issue so wondering if to wait.
I have no prior experience with Yeoman to input on your questions sadly.

I'm closing this issue now. As of today (2015-11-29), this project is fully ported to the Yeoman infrastructure. v0.1.0 of generator-phaser-plus is its first release, now on npm. Time to focus on the documentation of the sample project now.


Fix /html error in travis

Indentation error for style guidelines. For future note, everything must be indented at least one more than the html wrapper. The code outside of the  was what was failing the Travis test.


Wrong doc -> Unknown property 'spring.cloud.config.server.health.enabled'

Unknown property 'spring.cloud.config.server.health.enabled'
...bot documented in http://cloud.spring.io/spring-cloud-config/spring-cloud-config.html

It is certainly a valid property. Are you getting an error?

I think its only a missing entry in /META-INF/spring-configuration-metadata.json ?

Are you seeing an error? That property isn't in an @ConfigurationProperties object, so it won't automatically end up in /META-INF/spring-configuration-metadata.json.

it only "unknown to the STS IDE in yml editor.


is it possible to check the memory usage in TA while the Training running?

Hi, mofan
I'm using the darknetz with classifier.c which is quite a basic thing, but I would like the check the total memory usage of TA and memory usage by each layer while training.
I tried several ways to figure out that as below

checked the VMSIZE / VMRSS / MAX (By each epoch)
checked the total size of region 0 to region 8 when the error occurs in TA (sum of all mapping sizes)
checked the TA_RAM, TEE_RAM_RX, TEE_RAM_RW

but still, I'm wondering which one is presenting the exact total memory size of TA. (each epoch)
And I have no idea how can I check the TA memory usage of each layer while training.
Thanks!

Thanks for the quality response.
I will keep studying based on what u told me.
Thanks a lot again.
Cheers


Add Decimal256Builder and Decimal256Array

Which issue does this PR close?

Closes #1999.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Marked as draft now and I will add some tests and dedup some codes.

If it is ready, please at me. @viirya

Codecov Report

Merging #2000 (8b2e387) into master (2309157) will decrease coverage by 0.02%.
The diff coverage is 72.64%.

:exclamation: Current head 8b2e387 differs from pull request most recent head 1dd13ea. Consider uploading reports for the commit 1dd13ea to get more accurate results

@@            Coverage Diff             @@
##           master    #2000      +/-   ##
==========================================
- Coverage   83.58%   83.56%   -0.03%
==========================================
  Files         222      222
  Lines       57529    57621      +92
==========================================
+ Hits        48088    48149      +61
- Misses       9441     9472      +31

Impacted Files
Coverage Δ

arrow/src/array/equal_json.rs
87.89% <0.00%> (-1.81%)
:arrow_down:

arrow/src/array/iterator.rs
96.11% <ø> (ø)

arrow/src/array/mod.rs
100.00% <ø> (ø)

arrow/src/array/ord.rs
69.17% <ø> (ø)

arrow/src/compute/kernels/cast.rs
95.79% <ø> (ø)

arrow/src/compute/kernels/sort.rs
95.67% <ø> (ø)

arrow/src/compute/kernels/take.rs
95.27% <ø> (ø)

arrow/src/csv/reader.rs
89.89% <ø> (ø)

arrow/src/util/decimal.rs
91.50% <ø> (ø)

arrow/src/util/display.rs
73.17% <ø> (ø)

... and 5 more

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 2309157...1dd13ea. Read the comment docs.

@tustvold  is this one good to go (for arrow 18?)

I see now that @liukun4515  asked to be pinged on this PR for review, which @viirya did but then I merged it prior to @liukun4515 having a chance to review.
@liukun4515  I will make the 18.0.0 release candidate but if you find issues with the API or some other reason we should not release 18.0.0 with this change in it, please let me know and we can make changes prior to release.

I see now that @liukun4515 asked to be pinged on this PR for review, which @viirya did but then I merged it prior to @liukun4515 having a chance to review.
@liukun4515 I will make the 18.0.0 release candidate but if you find issues with the API or some other reason we should not release 18.0.0 with this change in it, please let me know and we can make changes prior to release.

I see now that @liukun4515 asked to be pinged on this PR for review, which @viirya did but then I merged it prior to @liukun4515 having a chance to review.
@liukun4515 I will make the 18.0.0 release candidate but if you find issues with the API or some other reason we should not release 18.0.0 with this change in it, please let me know and we can make changes prior to release.

Sorry for the later reply, I am busy with other task.
This pr looks great for me.
And from https://github.com/apache/arrow-rs/pull/2000#discussion_r916386516, it's better to refactor the builder and generate the array data from fixed list builder to fixed binary builder.


A question of the code

Is there a problem with the judgment condition here(sender\apsi\senderdb.cpp:row 1202) ?

Thanks for pointing this out! I'll push an update right away (v0.8.0).


Pass auth instead of just token to common fetch in order to update token on each fetch

JIRA: HAC-874

/lgtm


wip: nano-optimize final method checks

Final method checks are (probably) still too slow.  We've taken care of -- I think -- all the "don't do unnecessary work" bits, so the only path left is to make the actual checking faster.

b7dfd83 was the first stab at this: we're creating the lookup key for @final_methods twice: once in the hook, and once when we're running the method for the first time, plus we're uselessly grabbing the instance method just for the purposes of constructing that key.  Let's not do that by creating the key once for everything to use.
821197e removes some confusion: T::Private::Final has its own bits for determining whether a particular module was itself declared as final, and indeed the hooks already consult that.  So we should be safe in removing the bit that inserts the module into the table @modules_with_final maintained by T::Private::Methods.  Said table is now easier to describe: "the set of modules that have at least one method as final".
90eb529 is motivated by @final_methods not really being in the form we want for fast checks: we have to construct a string key out of a module and method_name, requiring string formatting and consing during final method checks.  (And we're repeatedly formatting the owner_id of the relevant ancestor for each possibly-final name, which is just wasted work.)  Let's not do that: instead, now that @modules_with_final has been clarified, we can turn it into a mapping from Module to the set of final method names defined on that module.  That turns our "is this method final on this module?" check into set membership, with no consing.  We are maintaining more information, which is not great for memory, but...
96dbe63 notices that, given the above change, we're not using @final_methods anymore, so we can delete all the code associated with that, which should make the overall memory changes...roughly neutral?
cda482e recognizes that we can do slightly less work if we didn't have to do final method checks; since not doing final method checks is the common case, this is a very small speedup.

All of this ~should make things faster, but the only way to be sure is testing.
Motivation

Speed.
Test plan

See included automated tests.

@froydnj unassigning because title has wip

This branch has improved and is somewhat easier to explain since #4089 landed separately.  The main motivation we stated above still applies:

... @final_methods not really being in the form we want for fast checks: we have to construct a string key out of a module and method_name, requiring string formatting and consing during final method checks. (And we're repeatedly formatting the owner_id of the relevant ancestor for each possibly-final name, which is just wasted work.) Let's not do that

but we've reworked things slightly because -- at least for now -- we really do need to track "modules with final methods" and "modules with final methods somewhere in their inheritance chain" simultaneously.  But we can at least track both of those pieces in a way such that we can do efficient checks for final method violations.

I went ahead and inlined references to @was_ever_final_names too for good measure.


Feature request on display function params

Hi I was wondering if its possible to display the parameters of a function, similar to VS code's builtin like so:

instead of having to highlight the function itself like:

Because as soon as I start typing it goes away, and I might not remember what was next
Thanks!

dup #18

PR welcome


Test creation of user resource as part of the E2E tests

Add the user_resource creation process as tests in the e2e tests.
duplicate of #831

@anatbal if this is duplicate of #831, why was it reopened? Thanks.

Confirmed with Anat. Duplicate of #831


Add svg compression

The webpack build should optimize svgs with svgo before passing them to svgr

This is already included in svgr


Pong Tutorial Doesn't Export for Win or HTML5

I would have tested for other platforms, but I can't get those to export at all at the moment. This is my first day really tinkering with the engine.
The tutorial and finished pong game run fine from within the engine, but exporting them to windows 32 or 64 bit exe fails. The window pops up, you see the godot logo for a second and then it fails. I get the same outcome on windows and wine. When executing it with wine from linux, here is the output:
EXEC PATHPEXEC PATHPERROR: set_custom_property_info:  Condition ' !props.has(p_prop) ' is true.
At:  core/globals.cpp:1320
ERROR: set_stylebox:  Condition ' p_style.is_null() ' is true.
At:  scene/resources/theme.cpp:267
editor: 0
ERROR: load:  Condition ' local_path=="" ' is true. returned: RES()
At:  core/io/resource_loader.cpp:156
ERROR: poll:  Method/Function Failed, returning: error
At:  core/io/resource_format_binary.cpp:636
ERROR: load:  Condition ' err!=OK ' is true. returned: RES()
At:  core/io/resource_loader.cpp:135
ERROR: load:  Method/Function Failed, returning: RES()
At:  core/io/resource_loader.cpp:203
ERROR: instance:  Method/Function Failed, returning: __null
At:  core/io/object_format_binary.cpp:1417
ERROR: load:  Condition ' !loader ' is true. returned: __null
At:  scene/io/scene_format_object.cpp:389
ERROR: load:  Condition ' !node ' is true. returned: __null
At:  scene/io/scene_loader.cpp:135
ERROR: start:  Condition ' !scene ' is true. returned: false
At:  main/main.cpp:1155
ERROR: free:  Attempt to free invalid ID
At:  servers/spatial_sound/spatial_sound_server_sw.cpp:598
ERROR: free:  Attempt to free invalid ID
At:  servers/spatial_sound_2d/spatial_sound_2d_server_sw.cpp:612
WARNING: cleanup:  ObjectDB Instances still exist!
At:  core/object.cpp:1656
Not sure if this is fat finger error on my end or not. Figured I'd submit a bug report anyway. I'm running version 1.0.3917-beta1. If I can provide anymore output, let me know.

I tried exporting a project which uses an AutoLoaded Singleton to Windows and Html5 and I always get the same error messages:

Godot version: Godot_v2.1_beta_20160712_win64
Windows: 8.1

@jahd2602 this is not related to this (very old) issue. Please use the QA site to post such questions. Also, why you're using the 2.1 beta version if the stable is already released? Make sure to update the export templates too.


Running into timeout issues when downloading Serverless cli

Orb Version
1.0.1
Describe the bug
The following curl command to install the Serverless cli in the setup command occasionally results in a timeout:
curl -o- -L https://slss.io/install | bash

To Reproduce
Use the circleci/serverless-framework orb in a build, and run the setup command.
Expected behavior
The Serverless cli should be installed after running the setup command.
Additional context
This is an intermittent issue, but I noticed it happened around 10%~15% of the time.
Adding a retry to the curl command on timeout might address the issue.

For me this timeout is 80%+ of the time and means th9s orb doesn't work.
The issue seems to be that CircleCI is appearing to be in china to the serverless install script. The timed out log entry is:
Connection reset by peer in connection to sls-standalone-1300963013.cos.ap-shanghai.myqcloud.com
From the orb source:
curl -o- -L https://slss.io/install | bash
This bash script seems to use myqcloud.com only if it is detected that the caller is in China. And that URL is definitely trying to download from China.
Because the only part of the script that I need is the actual serverless install I have resolved this issue by adding the following to replace -serverless/setup
curl -o- -L https://slss.io/install | SLS_GEO_LOCATION=us bash
echo "export PATH=$HOME/.serverless/bin:$PATH" >> "$BASH_ENV"
source "$BASH_ENV"

This forces it to use the US source and voila... no timeout.
If CircleCI only builds in the US, then adding this to the orb would probably fix the issue.

was happy to have found this workaround a few days ago, but it seems to be timing out for SLS_GEO_LOCATION=us as well

Sorry I should have updated. It turned out to be two things: the first is that IFF it gets the script, the script was downloading from China. But also the script itself is being hosted in china (wtf serverless?)
`curl -I -L  https://slss.io/install
HTTP/1.1 301 Moved Permanently
Server: nginx
Date: Wed, 29 Sep 2021 19:47:23 GMT
Content-Type: text/html
Content-Length: 178
Connection: keep-alive
Location: https://tinycc.com/tiny/custom_domain_redirect/slss.io/install
HTTP/1.1 303 See Other
Server: nginx
Date: Wed, 29 Sep 2021 20:21:17 GMT
Content-Type: text/html; charset=utf-8
Connection: keep-alive
X-Powered-By: PHP/7.0.33
X-Frame-Options: sameorigin
X-XSS-Protection: 1; mode=block
X-Robots-Tag: nofollow, noindex
Location: https://sls-standalone-1300963013.cos.ap-shanghai.myqcloud.com/install.sh
Strict-Transport-Security: max-age=31536000;
Content-Security-Policy-Report-Only: default-src https:; script-src https: 'unsafe-eval' 'unsafe-inline'; style-src https: 'unsafe-inline'; img-src https: data:; font-src https: data:; report-uri /csp-report
HTTP/1.1 200 OK
Content-Type: application/x-sh
Content-Length: 3897
Connection: keep-alive
Date: Wed, 29 Sep 2021 20:20:45 GMT
ETag: "d97c0ea9754ead9a0e98f3066793f590"
Last-Modified: Mon, 08 Feb 2021 10:39:46 GMT
Server: tencent-cos
x-cos-hash-crc64ecma:<PHONE_NUMBER>863612494
x-cos-request-id: NjE1NGNhOWRfZDU0MTIyMDlfNWMyNl83OTQ5MTA=`
So we can see it is being pulled from  https://sls-standalone-1300963013.cos.ap-shanghai.myqcloud.com/install.sh which can time out because it is in China.
Serverless mirrors its files in the US too so replacing the URL with https://sls-standalone-sv-1300963013.cos.na-siliconvalley.myqcloud.com/install.sh means china doesn't get included.

@nshgraph thank you for providing this info. It seems like maybe something we might be looking to serverless to correct but I wanted to know if you ended up finding a proper solution that worked for you long time. I am in a maintenance sprint for this orb, but this is very difficult for me to reproduce. I will add SLS_GEO_LOCATION=us as mentioned above but as you also pointed out, this may not be enough.
We can pull from that second URL but I have concerns this will be difficult to keep up to date and based on the age of this issue, likely already is.

Hello,
Thank you all for reporting this issue. We have just released version 2.0.0 which should resolve this problem. Please try the latest version of the orb and let us know if you experience any issues.
https://github.com/CircleCI-Public/serverless-framework-orb/releases/tag/v2.0.0
https://circleci.com/developer/orbs/orb/circleci/serverless-framework


Fix Console polyfill to allow Mocha.js reporters to work correctly

The current Console polyfill only implements a limited subset of console functionality. Currently, the built in Mocha.js reporters (which print test results to the console) will not work. We are using a custom reporter as a workaround, but eventually we should fix the polyfill.

Relevant code: https://github.com/BabylonJS/BabylonNative/blob/master/Apps/UnitTests/Scripts/tests.js#L1-L16


Use the same image of Redis in all docker compose files

Seems like redis image is breaking things, let's wait with this one


[web] change fonts on webpage

I liked the old font as well. Though using modern fonts it not bad either.

I liked the old font as well. Though using modern fonts it not bad either.


⚠ Mobitopup has degraded performance

In 810781e, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3767 ms

Resolved: Mobitopup performance has improved in b143310.


FastSurfer Segmentation Modules: disable conformation of input image to isometric spaces

Question/Support Request
I was attempting to perform a whole brain segmentation using a 3d MRI scan that is already registered to 1mm MNI space. However, every time I run fastsurfer, it conforms my image to isometric spaces (e.g. 1mm), and for calculation reasons I don't want that to happen (I want the image to stay in 1mm MNI space). I checked documentations and there doesn't seem to be a flag to turn off this space conformation, so what options do I have
Environment

FastSurfer Version: 2.2.0+a000faa
Installation type: google colab (using the tutorial provided).
GPU: none

Execution
$FASTSURFER_HOME/run_fastsurfer.sh --t1 $img
--sd "{SETUP_DIR}fastsurfer_seg"
--sid result
--seg_only --py python3
--parallel --threads 8
--allow_root

Hi, no, it can only work on conform, isotropic images. You can choose a different voxel size, but it needs to be isotropic. Also it is not recommended to map to MNI (or other spaces), to avoid yet another interpolation step.

If you need segmentations to be in the MNI space because some other analysis is performed there and so you want labels to be defined there, there is currently only one possible procedure that is not well-tested (this would also be similar to what you would/could do with FreeSurfer):

make sure your image is at 1mm isotropic (in MNI-space)
modify the image to fix the RAS matrix (the matrix needs to be along main directions, ((1 0 0), (0 1 0), (0 0 1)) or ((1 0 0), (0 0 -1), (0 -1 0)) .... or the like.... NOTE: you are intentionally breaking the consistency of RAS-coordinates here.
run FastSurfer on the image you created --- the output segmentations will be in the same isotropic image space, ...
copy the segmentation files after the completed run (e.g. mri/aseg.mgz and revert your RAS-matrix to what it was before --- this fixes the consistency issue noted in 2. (also note, aseg.mgz is always stored in LIA, you may need to convert it to RAS or whatever you had in step 2 first.)
Repeat the step 4 for any other segmentation files

I believe this should get you the outputs you wanted... you still should quality-check everything.

As an additional note: the conformation is needed for 2 reasons:

The network needs isotropic images and the intensity rescaling to 0-255
The FreeSurfer-based surface pipeline needs images to be stored as LIA (RAS matrix only +-1 or 0).

Thank you very much for the explanation. Here's a quick follow-up question:
Suppose I set a t1w image that is in 1mm MNI space as the input for fastsurfer, would the conformation to 1mm isotropic space change voxel sizes (meaning that each voxel would be streched or compressed) or would the voxels remain the same size (1 milimeter squared) as those from the original 1mm MNI space? I am mainly using fastsurfer for volume calculations and I am wondering if there needs a conversion.

Hi,
quick answer: it is padding, but voxels size will not be changed again if it is already 1mm isotropic, BUT it will rotate your image.
Just to explain and document this better. FastSurfer follows the FreeSurfer convention (with its advantages and disadvantages).
What the conform step in FreeSurfer (and FastSurfer) does:

it re-orients the image so that the axes are aligned with the RAS coordinate system (LIA orientation specifically). This is usually a rotation.
at the same time it re-slices inputs to 1mm isotropic voxels. FastSurfer is a little different here, as it keeps a higher resolution by default if it finds it in the input. FreeSurfer needs a specific hires flag for that and always uses 1mm by default.
it adjusts image dimension to 256x256x256 (usually by padding) for 1mm and larger cubic images for higher resolutions, to make sure that no part of head is cropped during rotation and re-slicing.
image intensities are robustly scaled to 0 .. 255, also aiming to normalise WM intensity to around 110
images are converted to unsigned character data type

This happens to any input image and cannot be switched off.  The only way to prevent it, is to overwrite your image header and pretend that it is already in LIA orientation. Then fix the header again of all outputs. This is a very ugly workaround and can lead to all kinds of issues (e.g. derived surfaces etc are also affected).
We have considered moving away from the FreeSurfer way to allow more flexibility here, but it could have many unintended consequences and will likely produce incompatibilities between FastSurfer and FreeSurfer. It would also require retraining and redesign of all our other deep learning segmentation modules. Lot's of work.
Another workaround could be to not store the hard labels but instead write out the soft-labels (predictions) and map those to the input or talairach space and then convert to hard labels there (this would not help the surface pipeline, but for the segmentation). It would not be optimal but still much better than mapping hard labels with e.g. nearest neighbour interpolation. I am not sure if we have a flag for writing soft labels (which would require a lot disk space), but it could probably be added relatively easily, e.g. as a 4D image stack. We could also have a command line tool that reduces this to a 3D hard segmentation again after mapping. It could even be done in a single step, inside fastsurfer, if requested by the user as we know the original position of your input. We are thinking about these options.


fix(ipfs): Deleting /blocks dir if it prevents app from starting

:tada: This PR is included in version 1.4.39 :tada:
Your semantic-release bot :package::rocket:


Characters not echoing after after using autocomplete

Hello, I am the maintainer for the element package on NixOS!
I noticed that after using autocomplete, characters do not echo on the command line after the program exits. The only way known to me to fix this is by executing reset. This happens when using regular bash and bash --posix.
I've included an asciinema demonstrating the problem. Unfortunately I'm not familiar with Go, so I can't provide a PR, but I'd love to see a fix for this program!

Good morning and thank you for looking into element!
I confirm that with bash I am seeing the same problem (which does not manifest if using zsh or fish); in particular it seems that such problem occurs only when exiting after the prompt selection: if instead the program exits because of a wrong command or an incorrect query, such inconsistency does not reproduce.
I would be inclined to think that the responsibility lies in the library I am using to show the prompt, namely go-prompt and the way it passes the exit codes to the shell (I have re-created a minimal prompt example independent of element where I see the same). If this is the case (I am not entirely sure yet, will do more testing) I believe we would have to open an issue in that project instead.
Just for reproducibility sake: could you report the specific of the system you are using (terminal, shell, operating system)?

I have reproduced the issue by copying and pasting the documentation example: see below
https://user-images.githubusercontent.com/15387611/124736401-42a0ac00-df17-11eb-81e2-a0bca43da430.mov
so the problem is indeed the way go-prompt interacts with bash (independent of the rest of the code in element).

Just for reproducibility sake: could you report the specific of the system you are using (terminal, shell, operating system)?

Terminal: Konsole
Shell: bash 4.4.23
OS: NixOS 21.11.20210704.20887e4

I could also reproduce the bug by running the dependency's provided examples.
I looked into the project and it appears this issue has already been reported in https://github.com/c-bata/go-prompt/issues/228, https://github.com/c-bata/go-prompt/issues/233 and a fix is provided by https://github.com/c-bata/go-prompt/pull/239. It may be nice to integrate as a patch.

So it seems that the PR is waiting to be merged. I would say we wait for the merge so that the dependency can be just updated and it should work fine afterwards.

Yes -- for those that want an immediate fix: if you do git clone --depth=1 https://github.com/Xiami2012/go-prompt -b fix-termios-bad-save, cd into go-prompt, then go build, then do go get -u -v github.com/gennaro-tedesco/element again, it should work.

The PR has been merged, I am closing this as the issue should be fixed.


option to import new members

An option to import new member from json, CSV, Excel, or other types can be very helpful

Hi Yuval, it's coming very soon! (likely this week)

Good news as we actually just deployed the feature! We're looking for a couple of beta users.
Is that a feature that you would like to try?

Good news as we actually just deployed the feature! We're looking for a couple of beta users.
Is that a feature that you would like to try?

sure, I would love to

Great, closing the issue in favor of Discord!

In case anyone is following up, now available: https://docs.orbit.love/docs#add-members-to-your-workspace-via-csv


Security

[ ] Security disclaimers in the documentation
[ ] GPG-sign binaries (or use sigstore) when publishing to PyPI
[ ] Add some guidelines for users for verifiability of downloaded wheels

Additional: look into security policies for Hugo, Chocolatey, Homebrew, MacPorts, and finally, pip


Search box no longer visible by default for pages where sidebar is automatically scrolled down

PR https://github.com/pydata/pydata-sphinx-theme/pull/502 made the whole sidebar scrollable, and not only the toctree part of it. That fixed some annoying usability bugs when having multiple elements in the sidebar (see screenshots in https://github.com/pydata/pydata-sphinx-theme/issues/500).
But, that also introduced the minor usability issue that if you go to a page for which the sidebar is automatically scrolled down (eg got to https://pydata-sphinx-theme.readthedocs.io/en/latest/demo/subpages/subsubpages/subsubpage19.html), the search box is now never visible out of the box.
(you have to scroll up in the sidebar (not scroll the page itself, that doesn't influence that) to get to the search box, and scrolling in the sidebar is not something you typically do except for going to another page)
I am not directly thinking of a good solution, but recording the slight concern.
(or at least not a generic solution, without special casing the "search-field" template if it is the first element of html_sidebars, or something like that)

this is the issue that finally convinces me that the search box should default to being in the top navbar instead of the left sidebar (which has been suggested lately in another issue, but I can't immediately find it to cross-ref).


added logo as a home navigation

@Manishkr1007 This code is working fine in my machine ................

this is code is working fine you can merge directly I don't know why vercel bot giving error you merge @Manishkr1007


Multigpu

multi-gpu commandline option added.

multi-gpu commandline option added.

Sorry, my mistake :(
Please ignore this pull request.


rolling_update: fix rbdmirror group name

The rbdmirror group name was using the wrong variable definition.
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>

jenkins test centos-non_container-update

jenkins test centos-container-update


added support for NODE_OPTIONS

Fixes #52
Tested locally using:
git clean -dxf
npm i
npm run test-ava
npm run test-uvu
npm run lint

All of the above were repeated for each of the following versions of node:

[x] Node v14.120.0
[x] Node v15.14.0
[x] Node v16.16.0 lts
[x] Node v17.9.1
[ ] Node v18.6.0 latest (see #53)

Also tested the option locally in various scenarios, including by running:
NODE_OPTIONS="--loader=esmock" npx ava ./spec/ava/*.spec.js

@Swivelgames thank you for this valuable contribution. I feel guilty for not adding this many months ago and am relieved to get this PR.
I started a different branch a few months ago to make the import file paths smaller so runtime errors will not have the very long file paths. I've been kind of energy-zapped.
I'll start the process to publish this.

@iambumblehead No worries at all! I know exactly how you feel. I got a burst of energy to work on some of my own projects this evening, and then I ran into the Node 18 error and noticed the open issue.
I'm glad to help! I was surprised to see your response so quickly :smile:


Tests: bluetooth: tester: accomodate L2CAP connect commant to BTP change

BTP change extended L2CAP_CONNECT command by ECFC flag, which determines
which connection procedure is used (non-enhanced or ecred). Now, only
this flag determines the procedure used, not number of requested
channels.
This was affecting L2CAP/ECFC/BV-25-C
Signed-off-by: Krzysztof Kopyściński<EMAIL_ADDRESS>

Rebased onto main


Fixed #294 -- Using user.is_authenticated() as a method is deprecated in Django >= 1.10.

Coverage remained the same at 100.0% when pulling fae30373f6a5a48456b8bfd70a0b82a3a80fed3a on felixxm:issue-294 into aaa4516d4b455f79fa0fe9f6626b34d1e67e2abe on pinax:master.

Django master build failure is not related with this PR.

I fixed tests on Django master in this PR.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 321585be1c5d5bbec30a3a20cc71f6f890836bf5 on felixxm:issue-294 into d12c36748b8aa85029ec249ea7eb8993fb2db8b4 on pinax:master.

Coverage remained the same at 100.0% when pulling 9e9e0b8e54da68301a6412c439ba39c0c2133955 on felixxm:issue-294 into acc400e746c3ad2da7eef1d58b3bf9f929164954 on pinax:master.

Coverage remained the same at 100.0% when pulling c8317b98317e3fda91c3338cf83bd1b418a6ad19 on felixxm:issue-294 into 661538816dacba9287d104a58b4cdbce8e460deb on pinax:master.


Install Config update

Description
Expand install config info, and combine with EE and OSS.

https://jira.mesosphere.com/browse/DOCS-1934

New page: https://github.com/dcos/dcos-docs/blob/1f7836faf17cfa727a36b34642d8ca7b662e10c1/1.9/installing/custom/configuration/index.md

Updated page:
https://github.com/dcos/dcos-docs/blob/1f7836faf17cfa727a36b34642d8ca7b662e10c1/1.9/installing/custom/configuration/configuration-parameters.md

New page created from existing content:
https://github.com/dcos/dcos-docs/blob/1f7836faf17cfa727a36b34642d8ca7b662e10c1/1.9/installing/custom/configuration/examples.md

Urgency

[ ] Blocker
[ ] High
[x] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

I did a quick scan and this looks like a nice improvement. 👍

This is good. Looks much more organized now. A good change :D

@sascala implemented your feedback. thanks!


Update dependency de.gdata:vaas to v3.1.1

This PR contains the following updates:

Package
Type
Update
Change

de.gdata:vaas
dependencies
patch
3.1.0 -> 3.1.1

Configuration
:date: Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
:warning: Warning: custom changes will be lost.


Cannot run

The data directory doesnt exist anymore and the link to download the model which is there in the earlier commits is also taken out and doesnt work anymore

The download_model.sh downloads both the model and the example data and it works fine for me!


[CLI] Clipboard module

This module should deal with accessing the system clipboard and check if it's able to work in all platforms

Investigating why it's not working properly

Might have to shell out

Clipboard feature reimplemented using external clipboard binaries and OSC52 escape seq


Update default config for java runtime

Update webdefault.xml to match the currently shipped version but with development functionality removed. Previously the build was not replacing this configuration when rebuilding the JARs.

Daily build no. 6178


Hey please send me a way to say thank you!

I looked in the readme for some way to pay you for doing this. Im very thankful i spent all day yesterday trying to figure this exact issue out and you have already solved it!  Thank you so much for making this!

No need lol . Just a like on pub.dev and a star on this repo will be fine 😁


solving error -302

After inserting a new record to source table the below error was happened:
(-302 THE VALUE OF INPUT VARIABLE OR PARAMETER NUMBER position-number IS INVALID OR TOO LARGE FOR THE TARGET COLUMN OR THE TARGET VALUE)

By editing LSN class (increment method --> 16 byte to 10 byte) the issue was resolved.

By editing LSN class (increment method --> 16 byte to 10 byte) the issue was resolved.


feat: interact with user settings/profile

What changes this PR introduce?
It allows interacting with users settings/profile pages (doing APIs requests).

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added paranthesis

Added parenthesis around the format-trend output. Without it, only +/day is returned, not the value.

@jwmoss : Thanks for your contribution! The author(s) have been notified to review your proposed change. @cosmosdarwin

Ha! Better late than never. Thanks for merging!


feat(axum): provide state to server fn context

Note that this is a minimal implementation and will not allow the
user to expect_state if they have external calls to rendering their
app (i.e. using render_app_to_* directly).
Additional work should be taken up to outline these changes through documentation.
Unfortunately, my personal work takes up most of my time right now.

oof did I screw something up or has CI just been fussy recently. I can take a deeper look if its needed

It's not you, it's clippy testing rkyv and feeling sad.
This looks great -- thanks!


Fix overlay removal bug

Does any other open PR do the same thing?

Nope
What issue is this PR fixing?
When <Overlay/> component is unmounted before the image had a chance to load (either a large image, or slow internet connection), after image finishes loading overlay is still rendered, and cannot be removed afterwards.
How did you test this PR?

There was a bug on Android which didn't cancel image rendering after removing from map. My change erases map reference, thus avoiding this bug.
Tested on several Android devices, does not seem to affect iOS.

LGTM
@alvelig 🐽

Could #2027 relate to it?

@alvelig I don't really think so. This has a very specific and narrow use-case, and the issue you've linked does not ever use any Overlays, so it should not be related.


About the Symbol table ,string table address

I do not understand why the  linkedit_base = (uintptr_t)slide + linkedit_segment->vmaddr - linkedit_segment->fileoff;
not
linkedit_base = (uintptr_t)slide + linkedit_segment->vmaddr;

I'm not quite sure either, the names seem like what you said makes more sense. But it works this way — is this causing an issue for you?

@grp

I'm not sure the linkedit_base is the pic describe or not

@grp  Do you have any idea to get the symbol from __DATA _data section . the symbol read from local Mach-O file is more than runtime Mach-O file .And I found the symbol in __DATA _data section is gone in runtime.

I'm not sure; is that related to fishhook?

@grp  Nope ,I just want to ask you something and figure out what the meaning of linkedit_base


Re-implement StaticTransformBroadcaster to persist transforms

This PR re-implements tf2_ros.StaticTransformBroadcaster to persist transforms, like its C++ equivalent already does.

Rebased to fix failing test in https://github.com/bdaiinstitute/ros_utilities/pull/36/commits/4b21977520d0aad8acc3af52b144da686a396051

Three approvals and green CI, going in !


Because the use of faiss, the windows is not supported

Unable to use in windows,  have a solution?

While this is true, we do have the file auxiliaries_nofaiss.py which doesn't use faiss and should work under Windows! Note however that it hasn't been extensively tested! Other than that, feel free to take a look at the https://github.com/bitsun/faiss-windows project, who are reimplementing faiss for Windows!

Hi Auther, may in the losses.py, theres an import of faiss, may i know how to skip that? i've changed the auxiliaries.py import into auxiliaries_nofaiss.py

Hi Auther, may in the losses.py, theres an import of faiss, may i know how to skip that? i've changed the auxiliaries.py import into auxiliaries_nofaiss.py


Add install inscructions for vim plugged

I tried to install the plugin with Vim plugged:
Plug 'RaafatTurki/hex.nvim'

require 'hex'.setup()

:PlugInstall gave me a successfull installation, but I cant use the commands like :HexDump
It would be good to have instructions for vim plugged.
I assume at least that it did not work because of my install method.

closing due to inactivity


How to test on custom images?

Hello, How to test on custom images or videos? Thanks

Please refer to valid.py. It provides the pre/post-processing details which can be used to test on custom images/videos.

The details seem to be coming from cfg files. Seem like I need both images and annots to do inference. Can you point me to the code for inference on images without any labelled annots available? Thanks


Revert "Don't use Linuxbrew/homebrew-core (yet, at least)."

This reverts commit 70a9a674112e55ff7f7fe9e8ca8f357f38e988f1.

😆

Codecov Report

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

@@            Coverage Diff             @@
##           master     #660      +/-   ##
==========================================
- Coverage   68.66%   68.65%   -0.02%
==========================================
  Files         382      382
  Lines       20617    20619       +2
==========================================
- Hits        14157    14155       -2
- Misses       6460     6464       +4

Impacted Files
Coverage Δ

utils/analytics.rb
82.05% <0%> (-7.7%)
:arrow_down:

tap.rb
93.78% <0%> (-0.27%)
:arrow_down:

Continue to review full report at Codecov.

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


Enhancement - Scrap data when status code 429, instead of exiting scraping - Linkedin

Hi @cullenwatson !
I was wondering, if we put a simple check on status code 429. If status code is 429, it should keep retrying/attempting the website until it receives status code 200 because if status code 429 appears we can refresh browser and resume. It will most help with proxy, which provide auto rotating. This is suggestion is specifically for Linkedin.
Something like this for without auto-rotating proxies:
if page.status_code == 429:
        print("Error fetching page, Error: " + str(page.status_code))
        while True:
            page = requests.get(url, headers=headers)
            if page.status_code == 200:
                break
            else:
                print("Retrying website!")

If proxy provider is providing auto-rotating, we can pass another new params as "auto-rotating_proxy = True" and use/execute code something like this:
if page.status_code != 200:
        print("Error fetching page, Error: " + str(page.status_code))
        while True:
            page = requests.get(url, headers=headers)
            if page.status_code == 200:
                break
            else:
                print("Retrying website!")

This will change the proxy automatically on each re-attempt, whenever there is an error. I hope this helps the community and users.

I think the write way to handle this here is to hand the user the session, so they can handle the responses however they want using requests hooks? Agree?

@ZacharyHampton Yes that will work out well but a newbie might find it bit difficult to handle. What we can do is provide both functionalities. We pass parameter something like (session_response), if true it will automatically handle similar to the solution I have proposed and if it is false, it lets user handle the session. How does that sounds?

we are using retry config on linkedin. 3 attempts at 5s,15s,35s delay before exiting.


Feature/bocm 63 api endpoint for case officer to confirm the outcome of lpa questionnaire

Describe your changes

Issue ticket number and link
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[√ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[√ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[√ ] I have performed a self-review of my own code
[√ ] I have double checked this work does not include any hardcoded secrets or passwords
[√ ] I have made corresponding changes to the documentation
[√ ] I have provided details on how I have tested my code
[√ ] I have referenced the ticket number above
[√ ] I have provided a description of the ticket
[√ ] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed issue where display would get highlighted

Ahh, a nice small PR.
Closes #30

o i see this nice change in the other cl too. Might want to submit this first, then merge the others.
And nice! I noticed this too


Improvements to startup of notebook

For #7242
To be reviewed after https://github.com/microsoft/vscode-python/pull/8319

Codecov Report

Merging #8332 into master will decrease coverage by 1.1%.
The diff coverage is 54.25%.

@@            Coverage Diff             @@
##           master    #8332      +/-   ##
==========================================
- Coverage   59.33%   58.23%   -1.11%
==========================================
  Files         505      507       +2
  Lines       23147    22824     -323
  Branches     3745     3738       -7
==========================================
- Hits        13734    13291     -443
- Misses       8520     8663     +143
+ Partials      893      870      -23

Impacted Files
Coverage Δ

src/client/datascience/jupyter/notebookStarter.ts
51% <51%> (ø)

src/client/datascience/jupyter/kernelService.ts
54.67% <54.67%> (ø)

src/client/datascience/jupyter/jupyterExecution.ts
46.2% <87.5%> (-6.16%)
:arrow_down:

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-75%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
0% <0%> (-66.67%)
:arrow_down:

...rc/client/datascience/jupyter/jupyterConnection.ts
12.5% <0%> (-57.7%)
:arrow_down:

...science/jupyter/jupyterKernelPromiseFailedError.ts
0% <0%> (-50%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-50%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
50% <0%> (-30%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

... and 46 more

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 9e7bce4...6dc177c. Read the comment docs.


LinkPager set class for li item's

Hi,
I'm using BEM methodology and in it I need set class for each elements.
Can you add a properity $internalPagesCssClass = null; in LinkPager?
I can do PR for it, if you agree.

$pageCssClass could be added with null as default value and used in renderPageButton() method:
$options = ['class' => $class === '' ? $this->pageCssClass: $class];

@samdark it works if $class === '' changed to $class === null or $buttons[] = $this->renderPageButton($i + 1, $i, null, false, $i == $currentPage); to $buttons[] = $this->renderPageButton($i + 1, $i, '', false, $i == $currentPage);

We can check $class as in https://github.com/yiisoft/yii2/blob/master/framework/helpers/BaseHtml.php#L1772 changed $class === '' to empty($class)? Maybe it more correct.

Yes.


runs jshint during Travis build.

Purpose
Add jshint to Travis CI builds. closes #2621
Changes
add invoke jshint to tasks.py and changes  .travis.yml to run it on build.
Side Effects
new task 'invoke jshint' runs jshint on every js file in project.

We'll want to make sure we can have passing builds before this is merged. The remaining jshint errors need to be addressed (either fixed or explicitly ignored).

Everything should now conform to jshint.

@Johnetordoff Can you get this up to date with develop? I believe there is a conflict because test_projectorganizer.js has been removed on develop (the tests are stale and we no longer need them).

Pass finished. Just minor changes and this is good to merge. :key:


term API doc: added referenced ACTIF sample file

Terminology API documentation is referencing a sample ACTIF term file in the "create term entries" section
this ACTIF file is now added (to the doc/ subfolder)

@madeline-brennan I applied all your final style suggestions, but I asked you for a re-review, because I think you still need to approve the PR so that it can be merged.