Segfault occurs on OS X Yosemite

If I call startScanning() and do not call stopScanning(), what would cause the script to terminate?  My script usually will run indefinitely but does stop unexpectedly at times.  I suspect it could be related to connection reliability to discovered peripherals or lack of connected peripherals but I'm not always seeing disconnect events.
I am:

startScanning() and continue scanning for new / disconnected peripherals
connect to some discovered peripherals (not all)
subscribe to notifications of the target peripherals
read data from the target peripherals on notifications (about 1 / minute)

I'm testing on OS X.

@tompropst thanks, it would be great if we can reproduce this faster (I haven't had time to try out your script for more than a few hours, as I only have one Mac).

@sandeepmistry, I ran a few tests with various scanning / "resting" times. The issue definitely appears faster the more scanning you are doing so I recommend scanning continuously to get to the error as quickly as possible.  It may still take several hours in my experience.
If there is any more debug information that I can collect for you, just let me know.

@tompropst I'll try it out over night ...
Is the following enough to reproduce?
var noble = require('../index');

noble.on('stateChange', function(state) {
  if (state === 'poweredOn') {
    noble.startScanning([], true);
  } else {
    noble.stopScanning();
  }
});

noble.on('discover', function(peripheral) {
  console.log('peripheral discovered (' + peripheral.uuid+ '):');
});

Did you have a chance to look at the memory usage?

I will try running the above script now and monitor memory usage.  I believe it should exhibit the same issue.  I have looked casually at memory usage in the past and it was increasing over time but not an alarming amount.

@tompropst going to start an overnight test now, included segfault_handler and logging date on discover as well.

@tompropst no segfault yet, Activity Monitor reports node is using 11 MB of memory:

Real Memory: 7.8 MB
Virtual Memory: 2.91 GB
Shared Memory: 224 KB
Private Memory: 6.1 MB

I'll run again tonight with more devices in range.

@tompropst still running on my Mac, left it on during the day. I am using noble 0.3.7 from source, which has been upgraded to using node-xpc-connect v0.1.0
var SegfaultHandler = require('segfault-handler');

SegfaultHandler.registerHandler();

var noble = require('../index');

noble.on('stateChange', function(state) {
  if (state === 'poweredOn') {
    noble.startScanning([], true);
  } else {
    noble.stopScanning();
  }
});

noble.on('discover', function(peripheral) {
  console.log(new Date());
  console.log('peripheral discovered (' + peripheral.uuid+ '):');
  console.log('\thello my local name is:');
  console.log('\t\t' + peripheral.advertisement.localName);
  console.log('\tcan I interest you in any of the following advertised services:');
  console.log('\t\t' + JSON.stringify(peripheral.advertisement.serviceUuids));

  var serviceData = peripheral.advertisement.serviceData;
  if (serviceData && serviceData.length) {
    console.log('\there is my service data:');
    for (var i in serviceData) {
      console.log('\t\t' + JSON.stringify(serviceData[i].uuid) + ': ' + JSON.stringify(serviceData[i].data.toString('hex')));
    }
  }
  if (peripheral.advertisement.manufacturerData) {
    console.log('\there is my manufacturer data:');
    console.log('\t\t' + JSON.stringify(peripheral.advertisement.manufacturerData.toString('hex')));
  }
  if (peripheral.advertisement.txPowerLevel !== undefined) {
    console.log('\tmy TX power level is:');
    console.log('\t\t' + peripheral.advertisement.txPowerLevel);
  }

  console.log();
});

@tompropst still no segfault on my test, I haven't stopped the process from the overnight test.
Real Memory: 12.8 MB
Virtual Memory: 2.91 GB
Shared Memory: 224 KB
Private Memory: 12.1 MB
Also, added an Arduino device that advertises every 100ms.

That's odd. The only difference I see is that I am calling startScanning() with no duplicates.  My Mac is also circa 2011.  Maybe it is an issue with that BT driver / HW.
I'm happy to keep working on this and can try to take a more active role since it only occurs on my system.  I'm OK with closing this too if you'd like.  My primary platform is Linux.

Have you done a fresh npm install of noble since upgrading OS X and/or node?

I haven't since a few days before opening this issue.  I started using noble on 11/7.  I haven't upgraded OS X since then but I can't say for sure about node.
I'll update now and run another test.

I updated to noble 0.3.7 last night and the test has been running for 11 hours with no crash. I'll leave it running as long as I can.

Interesting, maybe the conversion to using NAN fixed something (see https://github.com/sandeepmistry/node-xpc-connection/pull/6)

@tompropst closing now, as you appear to be crash free.


New to symforce

Hi, I am new to this topic and currently exploring the use of SymForce library and would like some clarification on the use of it.

In the example on the robot 3D localization, the generate function generates a C++ code which has faster runtime. How do I then proceed to use this C++ code to solve the localization problem?

The generated .h header files all seem to include sym/pose3.h but I can't seem to find the path for pose3.h.

I am still very new to this and would appreciate some help on these. Also, is there anyone that I can contact (email) to gain a deeper understanding on this library and its applications? Thanks and have a great week ahead!

This is related to #209

Thanks for the prompt response. I managed to solve my second issue.
For the first issue, is anyone from skydio or the community able to roughly explain/ go through the steps of solving the localization problem?

What do we do with the generated header files to solve the localization problem in a faster runtime?
Are we not able to solve the localization problem normally without the code generation portion?

I would appreciate any help on this!

What do we do with the generated header files to solve the localization problem in a faster runtime?

You can use them with the C++ optimization API (e.g., here and here).

Are we not able to solve the localization problem normally without the code generation portion?

Yes, that's what main() does in this file.


Godot 4: function rename dialog in Visual Script doesn't focus on the text box by default

Godot version
v4.0.alpha2.official [79077e6c1]
System information
Windows 11, Vulkan Clustered, Intel UHD Graphics 620 v<IP_ADDRESS>0
Issue description
When the user renames a function in Visual Script, the focus will be on the OK button instead of the text box. The users will have to click on the text box one more time before they can actually rename a function. This behavior can be seen in the screen recording below:
https://user-images.githubusercontent.com/1403194/153985053-fa2181c9-d81b-4f34-b0c0-144290b89089.mp4
This issue is a regression from Godot 3.4. In Godot 3.4, the users do not need to click on the text box as the focus will be on the text box by default. They can rename the function immediately, as shown in the screen recording below:
https://user-images.githubusercontent.com/1403194/153985330-0c303a2e-69f8-4a54-bf6d-6f8c2209c4d2.mp4
I expect the focus to be on the text box by default since that is the whole reason I go to this menu in the first place.
Steps to reproduce

Create a function in Visual Script.
Rename the function. The focus won't be on the text box.

Minimal reproduction project
No response

Should be already fixed in master branch via #58026


Retrieving the final URL when following redirects

When following redirects, I would expect that HTTP::Response#uri would return the final URL. However, it returns the original URL:
require "http"

response = HTTP.get("https://httpbin.org/redirect-to", params: { url: "https://httpbin.org/get" })
response.uri.to_s # => "https://httpbin.org/redirect-to?url=https%3A%2F%2Fhttpbin.org%2Fget"

Would it make sense that HTTP::Response#uri returned the final URL?
cc @dkam

Thanks for pointing me to the #uri method - that seems to work in concert with the follow method:
require 'http'

response = HTTP.get("https://httpbin.org/redirect-to", params: { url: "https://httpbin.org/get" })
response.uri.to_s
=> "https://httpbin.org/redirect-to?url=https%3A%2F%2Fhttpbin.org%2Fget"

response = HTTP.follow.get("https://httpbin.org/redirect-to", params: { url: "https://httpbin.org/get" })
response.uri.to_s
=> "https://httpbin.org/get"

Nevermind me, I forgot to add .follow 🙈

😀 Thanks though.  I've updated the wiki redirect page with the uri method.


Fixed surface material overrides are not applied to the new mesh after changing mesh in the editor

Fixed #64163

I made changes directly in master branch instead of create a new branch for this pull request.
So, I close this and make another pull request https://github.com/godotengine/godot/pull/64554 for this.
Sorry for this inconvenient.


Back to Top Button

A static back to top button is required so that user at anytime viewing this website can move to the top of website.

I can do that. Please assign it to me

I will do this unless @Harshit564 is doing it.

@Harshit564 @emerzon1 any of you is very welcome to work on this. :) No need to be assigned to it, you can just go ahead and make the code change and propose a pull request.

Thanks for the suggestion, we'll include a back to top button on the new version of the site.

Working on it!


Add coqPackages ceres and parsec

Motivation for this change
Things done

Built on platform(s)

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137999 at f40b0050 run on aarch64-linux 1

2 packages built successfully:

coqPackages.ceres
coqPackages.parsec


Optimization of enumerated values method

The values method of enum is a deep copy, it is not recommended to use the value method. Specifically see jmh results
Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
As mentioned above,The value method of enum is a deep copy, it is not recommended to use the value method. Specifically see jmh results.
Brief change log
Optimization of enumerated values method
Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
jmh test and junit test

Codecov Report

Merging #2677 into dev will decrease coverage by 0.89%.
The diff coverage is 44.44%.

@@             Coverage Diff              @@
##                dev    #2677      +/-   ##
============================================
- Coverage     37.11%   36.22%   -0.90%
+ Complexity     2537     2478      -59
============================================
  Files           431      431
  Lines         19893    19902       +9
  Branches       2421     2421
============================================
- Hits           7384     7210     -174
- Misses        11844    12050     +206
+ Partials        665      642      -23

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/dolphinscheduler/common/enums/DbType.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...che/dolphinscheduler/common/enums/CommandType.java
80.00% <66.66%> (+7.27%)
3.00 <0.00> (+1.00)

...dolphinscheduler/common/enums/ExecutionStatus.java
61.76% <66.66%> (+6.92%)
3.00 <0.00> (+1.00)

...duler/server/entity/SqoopTaskExecutionContext.java
0.00% <0.00%> (-65.00%)
0.00% <0.00%> (-10.00%)

...ver/master/consumer/TaskPriorityQueueConsumer.java
5.88% <0.00%> (-49.27%)
5.00% <0.00%> (-11.00%)

...e/dolphinscheduler/server/entity/TaskPriority.java
0.00% <0.00%> (-44.45%)
0.00% <0.00%> (-3.00%)

...r/server/entity/ProcedureTaskExecutionContext.java
0.00% <0.00%> (-40.00%)
0.00% <0.00%> (-2.00%)

...heduler/server/entity/SQLTaskExecutionContext.java
0.00% <0.00%> (-36.37%)
0.00% <0.00%> (-4.00%)

...duler/server/entity/DataxTaskExecutionContext.java
0.00% <0.00%> (-35.00%)
0.00% <0.00%> (-7.00%)

...server/master/dispatch/host/CommonHostManager.java
11.11% <0.00%> (-33.34%)
1.00% <0.00%> (-1.00%)

... and 8 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 daea87a...0636251. Read the comment docs.

Codecov Report

Merging #2677 into dev will increase coverage by 0.03%.
The diff coverage is 44.44%.

@@             Coverage Diff              @@
##                dev    #2677      +/-   ##
============================================
+ Coverage     37.08%   37.11%   +0.03%
  Complexity     2550     2550
============================================
  Files           434      434
  Lines         19985    19994       +9
  Branches       2423     2423
============================================
+ Hits           7411     7421      +10
+ Misses        11907    11904       -3
- Partials        667      669       +2

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/dolphinscheduler/common/enums/DbType.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...che/dolphinscheduler/common/enums/CommandType.java
80.00% <66.66%> (+7.27%)
3.00 <0.00> (+1.00)

...dolphinscheduler/common/enums/ExecutionStatus.java
61.76% <66.66%> (+6.92%)
3.00 <0.00> (+1.00)

...duler/server/entity/DataxTaskExecutionContext.java
35.00% <0.00%> (-60.00%)
7.00% <0.00%> (-6.00%)

...ver/master/consumer/TaskPriorityQueueConsumer.java
54.41% <0.00%> (-4.42%)
16.00% <0.00%> (ø%)

...he/dolphinscheduler/common/thread/ThreadUtils.java
68.25% <0.00%> (-3.18%)
13.00% <0.00%> (-1.00%)

...nscheduler/server/entity/TaskExecutionContext.java
87.12% <0.00%> (+5.94%)
60.00% <0.00%> (+3.00%)

.../server/worker/processor/TaskExecuteProcessor.java
37.28% <0.00%> (+27.11%)
3.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

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


[Bug]: dlib doesn't build with FFmpeg 7

What Operating System(s) are you seeing this problem on?
Linux (x86-64)
dlib version
19.24
Python version
No response
Compiler
GCC 14.1.1
Expected Behavior
Build dlib successfully with FFmpeg support
Current Behavior
FFmpeg broke some APIs,
There are some fields that got renamed from channel_layouts to ch_layouts.
Steps to Reproduce
git clone https://github.com/davisking/dlib
cd dlib
cmake -B build -G Ninja
cmake --build build

Anything else?
It would be great if @pfeatherstone could take a look at this, since I don't have multiple FFmpeg installations, I might fix this and break other versions...

Yeah I can have a look later today

I won't be able to test with gcc14 though. Best I can do 13

Thank you! I don't think the compiler poses a problem.

Meta-programming not working. I'm replacing the FFMPEG macro with a custom one. Looks like it's working.

https://github.com/davisking/dlib/pull/2987

I can confirm it works now, thank you so much!

Awesome, thanks guys :D


Payment channel tutorial

We revised the payment channel tutorial.
Ensure to update the perun-example submodule before deployment (otherwise, the imported code blocks will not work).
With Perun 0.9.0 we will need to make a few changes regarding the sender/receiver ID

Earlier pushes to this PR seem to have triggered CI builds (See logs). Could you try force pushing once ? @leandro-ro

Seems like Circle CI is not triggered if the pushing account is not registered with Circle CI?


Hubble Relay TLS flags naming

Multiple TLS related flags for Relay are introduced by https://github.com/cilium/cilium/pull/12900. Some are related to (1) the connection from Relay to Hubble servers, and other are related to (2) the connection from various clients (e.g. Hubble UI) to Relay itself.

Relay itself can connect to Hubble servers with mTLS, TLS, or without TLS. The related flags are:

--tls-client-cert-file
--tls-client-key-file
--tls-hubble-server-ca-files
--disable-client-tls

Relay can be configured to serve with TLS, or without TLS. The related flags are:
--tls-server-cert-file
--tls-server-key-file
--disable-server-tls

Since both (1) and (2) involved client and servers, client/server is not very useful to distinguish between the two. Some other scheme is needed.
cc @Rolinh @gandro
PS: there were other nits related to theses flags raised on https://github.com/cilium/cilium/pull/12906 and https://github.com/cilium/cilium/pull/12900 which could be addressed as well on the way.

The tls-hubble-client-ca-files is missing. Also it would be preferable that hubble would be a prefix for all of these flags.

The tls-hubble-client-ca-files is missing.

I think you're referring to hubble-tls-client-ca-files as mentioned here but this was addressed in the PR itself. Or did I miss something?

Also it would be preferable that hubble would be a prefix for all of these flags.

All Hubble related flags for the cilium-agent (aka daemon) are all prefixed with hubble-. However, Hubble Relay is a different binary/container that does not make use of the cilium configmap and uses its own. Thus, prefixing every Hubble Relay flags with hubble- would clutter I think.

@Rolinh 100% agreed.

I am using the new helm chart 1.9.0-rc1 and coming across a similar issue:
Warning  FailedMount       112s (x14 over 14m)  kubelet            MountVolume.SetUp failed for volume "tls" : [secret "hubble-relay-client-certs" not found, configmap "hubble-ca-cert" not found]

Helm values for hubble are:
    hubble:
      enabled: true
      ui:
        enabled: true
      relay:
        enabled: true
      metrics:
        enabled:
        - dns
        - drop
        - tcp
        - flow
        - port-distribution
        - icmp
        - http
        serviceMonitor:
          enabled: true

Let me know if this warrants a new issue. Thanks!

@onedr0p You need to set .Values.hubble.listenAddress=":4244" as well. Without it, mTLS will not be used anywhere.
If add that value, then the secrets are created as needed for me. We should probably document and/or add a required statement in the Helm charts.

@gandro thanks. I created an issue here, https://github.com/cilium/cilium/issues/13537 I will try that.


Using the file_dir but with another column then primary id

Hi,
I have a table which look like this:
partfiles
id
part_id
file
file_dir
I am trying to save files and I have set the 'dir' => 'file_dir', now when I add a new file it will store the file into a directory that matches the id of the file added. What if I wanted to save the file in another directory based on a the value of item_id. This way it is eassier that many files which belong to the same item are stored into the same directory.
I have tried setting the file_dir value in the add form, but it seems to not use the value that I set, rather it just saves based on the primary id.
Any idea?

Can you paste the code you tried?

This is in the model:
public $actsAs = array(
'Upload.Upload' => array(
'file' => array(
'fields' => array(
'dir' => 'file_dir',
'type' => 'filetype',
'size' => 'filesize',
),
),
),
);
In the view:
Form->input('part_id', array('type' => 'hidden', 'value' =>
$part['Part']['id']));
echo $this->Form->input('user_id',array('type' => 'hidden', 'value' =>
$_SESSION['Auth']['User']['id']));
echo $this->Form->input('file', array('type' => 'file'));
echo $this->Form->input('file_dir', array('type' => 'hidden', 'value' =>
$part['Part']['id']));
?>
The controller:
public function add($id = null) {
if ($this->request->is('post')) {
$this->PartFile->create();
if ($this->PartFile->save($this->request->data)) {
$this->Session->setFlash(('The part file has been saved'));
return $this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(('The part file could not be saved. Please, try
again.'));
}
}elseif(!isset($id)){
$this->Session->setFlash(__('Files needs to belong to a part'));
return $this->redirect(array('action' => 'index'));
}
$part = $this->PartFile->Part->find('first',array('conditions' =>
array('Part.id' => $id)));
$this->set(compact('part'));
}
2016-03-31 14:52 GMT+08:00 Jose Diaz-Gonzalez<EMAIL_ADDRESS>
Can you paste the code you tried?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/josegonzalez/cakephp-upload/issues/380#issuecomment-203787662

Is it an option to set ['pathMethod' => 'flat'] and use nameCallback to prepend the dir based on part_id?
Be aware that this could lead to unexpected behavior when uploading more than one file with the same name and part_id.

@tgoeminne bump

I kind of want if i set the file dir that it is used. Instead of creating
fil dirs based on the id.
On Apr 20, 2016 8:52 PM, "jorisvaesen"<EMAIL_ADDRESS>wrote:

@tgoeminne https://github.com/tgoeminne bump
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/josegonzalez/cakephp-upload/issues/380#issuecomment-212411734

You're losing me here.
Can you give more detail about what functionality you would like to see in this plugin?

You want to force override the directory field? If you want to do that, you can either change the upload path dynamically like so or set handleUploadedFileCallback as mentioned here.
There are no further methods of modifying the path in a 2.x application, and there will be no further active feature development on that branch. Consider upgrading your application if possible to 2.x, where the Upload plugin is a bit more flexible in how you can specify paths.
Note: In my opinion, your users don't care about where the files are stored, so long as they can download them, so trying to make your directory structure look nice is really only for you.


v1.0.8 doesn't include log file changes

Describe the bug
Looks like v1.0.8 was tagged incorrectly. It doesn't include the log file changes.

Sorry about that! The tag has been updated on github.
I will republish v1.0.8 on npm, once the time limit is removed (within 24hrs).

Actually, here is a patched version v1.0.8-1:

git
npm

Please let me know if there are any other issues!

Thanks, don't need to wait for npm since the nix builds pull from github.
Have the PR ready, just having some issues with the pr checks. Refactored the nix package to use mkDerivation + yarnConfigHook/yarnBuildHook instead of the old mkYarnPackage way. I also don't need the fish_wasm_file changes anymore but doesn't hurt just leaving it there.
https://github.com/NixOS/nixpkgs/pull/330996


Dockerfile assumes use of docker or at least privileged container build (not rootless buildah)

Describe the bug
NERSC prefers use of rootless containers focused on buildah/podman pipeline (at present).  Use of the USER canu, and chown prevents build of the container in a rootless environment.  Given I'm unsure of all use-cases this may be expected and needful, however, a solution that also enables rootless build would be appreciated.
canu Version
Current develop branch.
To Reproduce
Steps to reproduce the behavior:
buildah bud -t canu .
Expected behavior
Ideally the container would build and then be usable with podman run -it canu /bin/bash
Screenshots
If applicable, add screenshots to help explain your problem.

Thanks @dmjacobsen.  We'll take a look at this.


ValueError: Dimension has to be a list or tuple

Traceback (most recent call last):
File "", line 1, in
runfile('/Users/sameepshah/Desktop/Data/Practice/skoptHyperParm.py', wdir='/Users/sameepshah/Desktop/Data/Practice')
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/spyder/utils/site/sitecustomize.py", line 705, in runfile
execfile(filename, namespace)
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/spyder/utils/site/sitecustomize.py", line 102, in execfile
exec(compile(f.read(), filename, 'exec'), namespace)
File "/Users/sameepshah/Desktop/Data/Practice/skoptHyperParm.py", line 315, in
x0=default_parameters)
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/skopt/optimizer/gp.py", line 214, in gp_minimize
space = normalize_dimensions(dimensions)
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/skopt/utils.py", line 472, in normalize_dimensions
space = Space(dimensions)
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/skopt/space/space.py", line 570, in init
self.dimensions = [check_dimension(dim) for dim in dimensions]
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/skopt/space/space.py", line 570, in
self.dimensions = [check_dimension(dim) for dim in dimensions]
File "/Users/sameepshah/anaconda3/lib/python3.6/site-packages/skopt/space/space.py", line 70, in check_dimension
raise ValueError("Dimension has to be a list or tuple.")
ValueError: Dimension has to be a list or tuple.
Hi guys I was trying to run your posted hyperparameter tuning model. I am getting this error help much appreciated.
Thanks

Same error.  skopt optimizer really sucks!

You probably are using an invalid form for the search_spaces parameter.
See here: https://scikit-optimize.github.io/#skopt.BayesSearchCV -> Parameters -> search_spaces

Never mind, I figured it out yesterday.

Thanks @dingjinjun , @bertvan !

can you tell us how to figure this error out？ @jim0607

Ahhh.  It's been a long time, but I kinda remember it's some version issue.  Hope it helps! @Sherlock1118


Why does the "pure julia implementation" seem to be stuck/abandoned? though nowhere marked as such

There's just no one focusing on it at this time. Julia's own parser is written in FemtoLisp; this Julia implementation isn't used by Julia itself, nor (at least directly) by any other existing project. It may be revived at some point provided someone has the time and a use case.

Thanks for clarifying, just another question: what are the chances that julia will be fully selfhosted at a future point in time? is it an objective?

There is currently discussion over the merits of that approach. One clear benefit of using Julia would be that we could more easily and accurately record and report syntax errors. The obvious counterpoint is that it makes it much harder to port Julia to new systems and architectures, since then to build Julia you first need a working build of Julia.

For future reference, there has been some discussion on this topic in this thread (from when the discussion forum was still hosted in Google Groups).


Inconsistent underlining in the Blog: Comments exercise

belongs to -> belongs_to

kindly assign this issue to me

Hello @BrooklinJazz
I have solved this issue.


test_version updated

Three tests added to replicate format of other test files.

Everything looks good to me and all the tests are passing on my end.


Is a battery capacity sensor for the SmartShunt possible?

Would be good to get (and possibly update) SmartShunt battery capacity data.

Hi,
Smartshunt only reports the remaining capacity in the batteries. So by creating the template :battery capacity-reporting capacity remaining you get what you need

I am referring to this SmartShunt parameter ...

No I'm sorry but smartshunt does not report this value via ve.direct port


Windows support

Comment:
Windows is supported on the pyg channel but the pyg channel has some binary incompat with conda-forge. Is that possible to add a windows package here?
Upstream relevant ticket: https://github.com/pyg-team/pytorch_geometric/issues/2790

Needs https://github.com/conda-forge/pytorch-cpu-feedstock/pull/63

@DomInvivo the latest pyg release (https://github.com/pyg-team/pytorch_geometric/releases/tag/2.2.0) is likely the last one relying on scatter and sparse and will be replaced by pyg-lib (https://github.com/pyg-team/pyg-lib).
Unfortunately pyg-lib is not compatible with windows yet (https://github.com/pyg-team/pyg-lib/issues/163#issuecomment-1356265963), but I'll build the windows conda package, once it is.


Support multiple annotators on the same image

I work mostly with images and this is something missing in most tools/annotations format.
Basic example where there is annotations propositions from 2 annotators, 2 from auto-labeling strategies (active learning) and 1 that would have been accepted as "ground_truth" by a reviser or someone with the right to "accept" a ground truth annotation  :
{
  "filename": "12345.jpg",
  "key": "12345",
  "size": {
    "width": 4608,
    "height": 2240,
    "depth": 3
  },
  "annotations": {
    "ground_truth": {
      "objects": [
        {
          "name": "cat",
          "bndbox": {
            "left": 600,
            "top": 100,
            "right": 2000,
            "bottom": 1000
          },
          "size": {
            "width": 1400,
            "height": 900,
            "depth": 3
          }
        },
        {
          "name": "dog",
          "bndbox": {
            "left": 2200,
            "top": 500,
            "right": 4100,
            "bottom": 2000
          },
          "size": {
            "width": 1900,
            "height": 1500,
            "depth": 3
          }
        }
      ]
    },
    "Annotator A": {
      "objects": [
        {
          "name": "cat",
          "bndbox": {
            "left": 2300,
            "top": 600,
            "right": 4000,
            "bottom": 1600
          },
          "last_modified":<PHONE_NUMBER>.1324713,
          "size": {
            "width": 1700,
            "height": 1000,
            "depth": 3
          }
        }
      ]
    },
    "Annotator B": {
      "objects": [
        {
          "name": "cat",
          "bndbox": {
            "left": 2300,
            "top": 600,
            "right": 4000,
            "bottom": 1600
          },
          "last_modified":<PHONE_NUMBER>.1324713,
          "size": {
            "width": 1700,
            "height": 1000,
            "depth": 3
          }
        }
      ]
    },
    "Yolo model": {
      "objects": [
        {
          "name": "cat",
          "bndbox": {
            "left": 2300,
            "top": 600,
            "right": 4000,
            "bottom": 1600
          },
          "last_modified":<PHONE_NUMBER>.1324713,
          "size": {
            "width": 1700,
            "height": 1000,
            "depth": 3
          }
        }
      ]
    },
    "SSD model": {
      "objects": [
        {
          "name": "cat",
          "bndbox": {
            "left": 2300,
            "top": 600,
            "right": 4000,
            "bottom": 1600
          },
          "last_modified":<PHONE_NUMBER>.1324713,
          "size": {
            "width": 1700,
            "height": 1000,
            "depth": 3
          }
        }
      ]
    }
  }
}

Thanks for the information about your use case @Ownmarc . Any large dataset will need quality control/user management to make sure data is being completed well with good oversight.


DOC: signal.unit_impulse: add explanation about delta

The unit impulse is known as Dirac delta, (or Dirac delta function), not Kronecker delta.
https://en.wikipedia.org/wiki/Kronecker_delta
https://en.wikipedia.org/wiki/Dirac_delta_function

This is a bit of a preference or purism thing as dirac delta is the continuous kronecker delta so this function generates sequential values thus should be the kronecker delta.
In any case neither are correct here as delta functions are not pointwise defined but engineering usage is well established. We can instead remove the delta mention to avoid all that.
Either way I don't have any strong opinion.

A  Notes section with one sentence is not too helpful, IMO. What about something along the lines of ( essentially citing Wikipedia):
Notes
-----
In digital signal processing literature the unit impulse is often represented by
the Kronecker delta. [1] E.g., a signal :math:`u[n]` which is zero everywhere except
being one at the :math:`k`-th sample can be expressed as

.. math::

    u[n] = \delta[n-k] \equiv \delta[k-n] \equiv \delta_{n,k} \equiv \delta_{k,n}\ .

Furthermore, it is frequently interpreted as the discrete-time version of the
continuous-time Dirac distribution. [2]

References
----------
.. [1] "Kronecker delta", *Wikipedia*,
       https://en.wikipedia.org/wiki/Kronecker_delta#Digital_signal_processing
.. [2] "Dirac delta function" *Wikipedia*,
       https://en.wikipedia.org/wiki/Dirac_delta_function#Relationship_to_the_Kronecker_delta

What do you think, @MarDiehl?

Having a longer explanation is IMHO a nice idea and the current text looks good.

@MarDiehl would you be interested in committing the suggested text?

I pushed the documentation change. The result can be viewed here—any improvements are very welcome!
PS: The test failures seem unrelated.

thanks!


[feature] - Add Database, like SQL and SQLite3

Process a database as a data source

I feel like that's taking it too far. There are infinite possibilities for the design of a database. Often times you have foreign keys, and the id that's used as a reference would mean nothing to the embedding algorithm. Is each row supposed to be treated as a document? Likely, but there will be other scenarios.
In this case I would recommend to use a database client, write the query you want to use and then use the add method with text as the data_type to embed it.


Feature: Bedrock and Bedrock Agents

Support for Bedrock and Agents for Bedrock services.
Includes support for create, delete, list, get, and tagging operations for custom models, model customization jobs, model invocation logging, agents, and knowledge bases.

Thanks for the feedback @bblommers, it should all be addressed now.
All of the testserver tests are failing, and I'm not sure how to fix it.

Hi @JoshLevyMN, I believe the ServerMode tests are failing because of a bug with how Moto handles request matching. I'll fix that first, and then revisit this. All the changes LGTM btw!


update notice file

Current Behaviour

Proposed Changes

Checks

[x] All commits in this Pull Request are signed.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor Licence Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the ockam-network/contributors repository.

Merged https://github.com/ockam-network/ockam/pull/2173


Fixes #860

Width initial value is always identified as 0

Fixed with other changes on the rollup branch, which will merge to master before next release.


Nonce mismatch in refresh-auth

I have setup whole stack with cloud-formation ,
at times i get  this error Nonce mismatch with Try Again
https://github.com/aws-samples/cloudfront-authorization-at-edge/blob/master/src/lambda-edge/refresh-auth/index.ts#L67
What might be leading to this experience ?

That shouldn't happen! I'm very interested to learn why it happens, so we can make the code more robust.
Can you check the nonce value in your "spa-auth-edge-nonce" cookie and compare it to the value in the URL query string? Is there anything special with the values - maybe some special character issue that only occurs every now and then (as nonces are generated automatically, there might only be a small chance to hit the specific character that raises this).

Any updates on this?

Closing for now

@ottokruse  looks like i  might have found the root cause of the issue, still to figure out a fix.
the  sample app is having

HTML Page
m3u8 video stream files in s3 bucket

both in same bucket and served through same cloudfront. intention was to secure all assets request.
As i'm trying to stream the video using videojs,  there is frequent request to cloudfront.
let's say the access token is expired  when the page was idle, video js still try to  request the video files and will get stuck at the auth flow which might result in nonce value to be changed.
Not sure if was able to articulate the  issue well

<!DOCTYPE html>

    <meta charset=utf-8 />
    <meta name="viewport" content="width=device-width, initial-scale=0.86, maximum-scale=3.0, minimum-scale=0.86">
    <title>Home</title>


[EN] Feedback for: /web/fundamentals/engage-and-retain/push-notifications/notifications?hl=en

Getting quite a bit of feedback that is' pretty hard to set up a google developer project via the console for push notifications on the web.
At the moment we recommend the Android or iOS guides which I don't think are particularly helpful.
I think adding a section of setting the console up would be our best bet until the web push protocol is the best approach.

Big +1


Status update

What are the plans for the future of this project?

Hi @Dev1an
Thanks for asking - we are currently working on the 1.5 as we still use Meteor for our projects on embedded hardware.
Not sure when next release is available but like quite fine currently.
Cheers
Tom

@TomFreudenberg Any updates on 1.5?


RBAC - Permissions for dashboards

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/527
Originally created by @valexieva on 2019-03-25 16:37:45

[ ]  Update this issue with an exploratory sprint
[ ]  Opportunity assessment
[ ]  Connect with @tayloramurphy on requirements

--
Problem to solve
(Summarize the problem we are trying to solve in the form of we need to do [A], so that [B] can [C])
Currently a user can only give permissions for reports and designs, but not dashboards
Target audience
(For whom are we doing this? Include a persona)
Admin user
Further details
(Include use cases, benefits, and/or goals)
I should be able to assign permissions to view and change dashboards to users in order to restrict access to confidential insights
Proposal
(How are we going to solve the problem? Try to include the user journey)
Follows the permissioning process for reports and designs
What does success look like, and how can we measure that?
(Define both the success metrics and acceptance criteria. Note that success metrics indicate the desired business outcomes, while acceptance criteria indicate when the solution is working correctly. If there is no way to measure success, link to an issue that will implement a way to measure this)
Regression test
(Ensure the feature doesn't cause any regressions)

[ ] Write adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

Links / references
Please note that this was taken from GitLab, to be changed accordingly
https://gitlab.com/meltano/meltano/issues/370
(cc: @dmor)

View 4 previous comments from the original issue on GitLab


Make definition statement available without wrapper

This is part of a broader feature request, #18060. By breaking it up I hope to make it more manageable.
I would like the definition statement of a view to be available without the wrapping text that DBeaver adds. That would avoid having to remove the wrapping text myself in a common workflow: altering a view by fetching the current SELECT VIEW statement, changing it to an ALTER statement and executing it.
In other words, instead of getting
-- some_view source

CREATE VIEW some_view AS
SELECT some_column
FROM some_table;

from the Definition page of the view object properties, I would like to get just
CREATE VIEW some_view AS
SELECT some_column
FROM some_table

meaning the definition text should be as it is stored in the database information schema, without the comment line and the blank line at the top, and ending the definition with a newline instead of a semicolon.
Perhaps this could be a configurable option.

Hello @dr-1
Your case is described here. https://github.com/dbeaver/dbeaver/issues/16188
You can remove the comment section in some databases, like PostgreSQL. What database do you use?

@LonwoLonwo, thanks for the reference.
I currently work mainly with this database and driver:
Server:
Microsoft SQL Server 12.00.5316
Microsoft SQL Azure (RTM) - 12.0.2000.8
Driver:
Microsoft JDBC Driver 11.2 for SQL Server <IP_ADDRESS>

instead of a semicolon

And what is the issue with a semicolon? Because this is a SQL standard.

Like the -- ... source comment at the top, the addition of the semicolon can also lead to a slight difference between the definition statement that we see and the one that the database has stored (in the case of SQL Server in INFORMATION_SCHEMA.VIEWS.VIEW_DEFINITION). For us this presents a problem because we have the version from INFORMATION_SCHEMA under version control. Altering a view with DBeaver without manually removing the semicolon then becomes an unintended change that shows in Git logs and diffs, making it more difficult for example to get a list of view definitions that changed in a meaningful way (other than having a semicolon added).

Ok. The ''--source" part will be removed here: https://github.com/dbeaver/dbeaver/issues/16188
And we can't remove the semicolon because, without it, SQL execution will not work.
So, I'm closing this ticket as a duplicate.


Support for mapping interfaces

(Apologies for x-posting this from code.google)
Hi, I'm not sure if this is covered in the documentation anywhere but I have not been able to map from an interface to an object.
public class UserAccountMap extends
    PropertyMap<UserModel, UserAccount> {

    @Override
    protected void configure() {
    ...
    }
}

In this example, "UserModel" is an interface with relevant getters/setters and UserAccount is a bean. Modelmapper will allow you to create this with no compile errors, but it seems to ignore the "UserModel" interface and look at the underlying class that is implementing the interface.
Alternative to a fix, is there a way around this so I don't have to implement a map for every object that the interface specifies?

The issue I'm having is that my explicit mappings are not used if they are created with Interfaces as the destination types.
An example is here:
https://github.com/scottresnik/modelmapper-interfaces/tree/master/example.modelmapper

The TypeMap inheritance had been release on ModelMapper v1.0.0, please give it a try!


[Q] Add axis labels to HeatMap

I want to add the labels for x- and y-axis in my HeatMap, like "Predicted" and "Actual".  How can you do it?
Additionaly how can you edit the "Values" name.

Hello @thanos-wandb,
Thanks for the answer. Exactly, that is what I wanted.


Update to download from Edmingle-based Platform

vyoma-download version: 0.3.0
Python version: 3.8.11
Operating System: Ubuntu 18.04.6 LTS

Description
Download doesn't work since the platform has shifted to new domain sanskritfromhome.org which uses Edmingle Platform

TODO

[ ] Module for Edmingle API
[ ] Vyoma Download Module
[ ] Plug-in New Module to CLI
[ ] More Download Related Actions (sectionwise, materialwise)

Version 1.0.0 adds this support.


[IRGen] Delete no-op code

Commit 5c3ccf505065 added a number of blocks of code which were partially removed by 39af79f491b. This partial remove left lines of code doing nothin; we remove them here.
rdar://110005073

@swift-ci test


Kinetics dataloader & model changes for HW implementation

For image pairs, the updated dataloader now folds each image first, then concatenates the two.
This change was necessary to fill the data memory of the accelerator in a more efficient manner, as used in the demo.
~
The model now correctly uses ai8x.add modules instead of simple additions. (Using simple additions without clamping had resulted in ~1% loss in accuracy when evaluating with 8-bit-mode)

Should we change the for loops to images_concat = [torch.cat((x, y, dim=0)) for x, y in zip(images, images[1:])] and images_concat = [np.concatenate((x, y, axis=2)) for x, y in zip(images, images[1:])]

Sure. For loops come more naturally to me than list comprehensions, but we could make that change for a marginal speed improvement.


[tosa] Support legalization of tfl.gather with multiple dynamic dimensions

The existing lowering of tfl.gather targets a sequence of tosa.reshape and tosa.gather operations. Currently, tosa.reshape supports, at most, a single dynamic dimension. This change adds support for multiple dynamic dimensions by targeting the tensor.reshape operator in the presence of multiple dynamic dimensions.

I think I've resolved all the existing issues. Just waiting for @rsuderman to give another pass.

Hi @rsuderman Can you please assist on above comments from @sabauma. Thank you!

Hi @rsuderman Any update on this PR? Please. Thank you!

Hi @rsuderman Any update on this PR? Please. Thank you!

Hi @rsuderman Any update on this PR? Please. Thank you!

Hi @rsuderman Any update on this PR? Please. Thank you!

Hi @rdzhabarov Can you please assist on above comments from @sabauma. Thank you!


Update sqlglot to >=13.0.0

Type of PR

[ ] BUG
[ ] FEAT
[x] MAINT
[ ] DOC

Is your Pull Request linked to an existing Issue or Pull Request?
There were a few issues popping up due to using a slightly dated version of SQLglot. For example - https://github.com/moj-analytical-services/splink/pull/1636#issuecomment-1749317412, shows SQLglot only picking up up one column used within the datediff function.
Additionally, we'll be able absorb any performance improvements and new additions that are now present after a few months of using v11.
See https://github.com/moj-analytical-services/splink/issues/894 for more on the original issue w/ CNF/DNF.
Give a brief description for the solution you have provided
Update SQLglot to >=13.0.0.
I've frozen the version at <v19.0.0 so we can update it incrementally, as and when we see fit.
Thanks to @ADBond for checking over the existing issue around DNF not working as expected.
PR Checklist

[ ] Added documentation for changes
[ ] Added feature to example notebooks or tutorial (if appropriate)
[x] Added tests (if appropriate)
[x] Made changes based off the latest version of Splink
[x] Run the linter

LGTM 👍


Closing file before attempting to delete

This PR concerns (the first half of) #4264.
I experienced a PermissionError when trying to save an ICA object to disk, however, this was not the error causing the write to fail but seemed to be the result of subsequently trying to delete the file (which was being written) before having closed it. Hence, this error was raised before the 'actual' error could be raised (and also prevented manual deletion of the file).
This PR implements a change to close the file before trying to delete it.

LGTM

Codecov Report

Merging #4270 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #4270      +/-   ##
==========================================
- Coverage   86.22%   86.21%   -0.01%
==========================================
  Files         346      346
  Lines       64642    64643       +1
  Branches     9907     9907
==========================================
- Hits        55735    55734       -1
- Misses       6192     6194       +2
  Partials     2715     2715

I think we're good.
thanks a lot @jdue


⚠ Secret Website 3 has degraded performance

In 2903840, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2000 ms

Resolved: Secret Website 3 performance has improved in bc48b92.


Fix for time alignment issue

With my poor Rust skills I should have fixed the issue faced in #310

Should close #310

Closes #310

Thanks for this, had started implementing but you beat me to it :)


Run pipeline CLI

Currently pachctl run pipeline can only send a limited subset of the permissible RunPipeline requests. This is because the RunPipeline request takes as its argument a repeated set of CommitProvenance objects, which are branch commit pairs, but the cli takes a repeated list of Commits, so the Branch field in the CommitProvenance is always empty. This leads to most CLI invocations of run pipeline failing. To fix this we need to introduce a CLI syntax for CommitProvenance objects. I propose we represent them as repo@branch=commit, so running a pipeline would look like:
pachctl run pipeline <pipeline> <repo1>@<branch>=<commit> <repo2>@<branch>=<commit> ...

Where <commit> = <commit or branch> and supports ^ syntax. I figure that was assumed, but wanted to mention explicitly

That is assumed, and already implemented in the API, it should be basically impossible for a Pachyderm command that takes a commit to not also accept branches and ancestry syntax.


Modified the VIEW

In the code snippet for the database object VIEW, it uses the Name field without the alias b
which all other column names have.
Just to be consistent, I propose adding the alias i.e. b.Name as opposed to Name.
Lastly, I have capitalized the from to FROM for readability and consistency as it is a reserved word like JOIN, GROUP BY etc.

@OOberoi good to hear from you, and thank you for this contribution!

Hey Diego,
Trust you’re well.
It’s my pleasure to play a tiny role in this vast ecosystem of EF Core greatnesses!!
Thank you heaps for transforming EF. I see a great deal of migration from brownfield apps to adoption of EF Core in greenfield apps.
Kind Regards,
Obi Oberoi


Updated package.xml to depend on python-requests

This way you won't have to instruct users to install python-requests separately.
See http://docs.ros.org/indigo/api/catkin/html/howto/format1/python_module_dependencies.html
for a more detailed explanation.
PS - Great package! Thank you for taking the initiative to integrate ROS with openHAB :)

Merged.  Thanks for the update!


how to get a canvas from Image or Bitmap

i have not found a Api to to get a canvas from Image or Bitmap

another confusion what is the differences between Bitmap, Image, Picture and Pixmap?

Canvas has ctor that accepts Bitmap.
The difference is:

Bitmap is mutable byte array you can draw into
Image is “GPU texture”, ready to be drawn, but not changed
Picture is “recording of commands”, so it can be reproduced, but has no direct pixel data at all
Pixmap, I think, is something that holds pixels for Bitmap


Sia exits unexpectedly

Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
(6/6) Loading renter...
Finished loading in 0.520014768 seconds
goroutine 61793 [running]:
runtime/debug.Stack(0x4456d7, 0x0, 0xc4203dbc80)
/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Severe(0xc4203dbe58, 0x2, 0x2)
/home/luke/go/src/github.com/NebulousLabs/Sia/build/critical.go:29 +0x92
github.com/NebulousLabs/Sia/persist.(*Logger).Severe(0xc42020d5c0, 0xc4203dbe58, 0x2, 0x2)
/home/luke/go/src/github.com/NebulousLabs/Sia/persist/log.go:71 +0xe0
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc427d8e620, 0xc420390308, 0x0, 0x0, 0x0, 0xeb9360, 0x0, 0x0)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:195 +0x357
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:203 +0x1e5
Severe error: Unable to properly initialize WAL file, crashing to prevent corruption: unable to write WAL metadata: write /home/user/.config/Sia-UI/sia/host/contractmanager/contractmanager.wal_temp: no space left on device
goroutine 61793 [running]:
runtime/debug.Stack(0x4456d7, 0x0, 0xc4203dbaa0)
/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Severe(0xc4203dbc18, 0x2, 0x2)
/home/luke/go/src/github.com/NebulousLabs/Sia/build/critical.go:29 +0x92
github.com/NebulousLabs/Sia/persist.(*Logger).Severe(0xc42020d5c0, 0xc4203dbc18, 0x2, 0x2)
/home/luke/go/src/github.com/NebulousLabs/Sia/persist/log.go:71 +0xe0
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).appendChange(0xc420390308, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlog.go:151 +0x46e
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc427d8e620, 0xc420390308, 0x0, 0x0, 0x0, 0xeb9360, 0x0, 0x0)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:202 +0x423
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:203 +0x1e5
Severe error: Unable to write state change to WAL: write /home/user/.config/Sia-UI/sia/host/contractmanager/contractmanager.wal_temp: no space left on device
panic: unable to append a change to the WAL, crashing to prevent corruption
goroutine 61793 [running]:
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).appendChange(0xc420390308, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlog.go:152 +0x4b8
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc427d8e620, 0xc420390308, 0x0, 0x0, 0x0, 0xeb9360, 0x0, 0x0)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:202 +0x423
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:203 +0x1e5

Severe error: Unable to properly initialize WAL file, crashing to prevent corruption: unable to write WAL metadata: write /home/user/.config/Sia-UI/sia/host/contractmanager/contractmanager.wal_temp: no space left on device

This is the source of the error. Make sure you have enough space on your disk to support the amount of storage you want to host.

Was this compiled from master? Severe errors shouldn't crash the program in the binaries we released

This may have been an older release that included the panic: crashing to prevent corruption


Remove StreamWebSocket from all UWP VPN documentation

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


Windows 10 Microsoft Edge 39.14986.1000.0

Hi, i have in my Cloudflare  "A Better Browser" setup for Windows Browser Highest version "10". Running the site on Windows 10 Microsoft Edge 39.14986.1000.0, A Better Browser recognize Edge as old Browser...
Thank you..

Same issue seen here.

The configuration was probably set up wrong. Edge has two version numbers. One for the app container (the high 30+ version number) and then one for the platform (the 14-16 version number). You should be using the platform version number.


chore: set GITHUB_AUTH_TOKEN for phive

ref: https://github.com/codeigniter4/CodeIgniter4/pull/7016
Run sudo phive --no-progress install --global --trust-gpg-keys B8F640134AB1782E,A98E898BB53EB748 qossmic/deptrac
Phive 0.15.2 - Copyright (C) 2015-2023 by Arne Blankerts, Sebastian Heuer and Contributors
Downloading https://api.github.com/rate_limit
Warning:  Github API Rate Limit exceeded - cannot resolve "qossmic/deptrac"
Downloading https://gitlab.com/api/v4/projects/qossmic%2Fdeptrac/releases/
Fetching repository list
Downloading https://phar.io/data/repositories.xml
Error:    Could not resolve requested PHAR qossmic/deptrac

Error: Process completed with exit code 4.

see https://github.com/datamweb/shield/actions/runs/3862351738/jobs/6583866759

It doesn't seem like the right solution to solve the problem. see:
https://github.com/codeigniter4/shield/actions/runs/3862455954/jobs/6584038136

It is true that ”Github API Rate Limit exceeded” was reported even if GITHUB_AUTH_TOKEN is set.
https://github.com/codeigniter4/shield/actions/runs/3862455954/jobs/6584038136
Run sudo phive --no-progress install --global --trust-gpg-keys B8F640134AB1782E,A98E898BB53EB748 qossmic/deptrac
  sudo phive --no-progress install --global --trust-gpg-keys B8F640134AB1782E,A98E898BB53EB748 qossmic/deptrac
  deptrac analyze --cache-file=build/deptrac.cache
  shell: /usr/bin/bash -e {0}
  env:
    COMPOSER_PROCESS_TIMEOUT: 0
    COMPOSER_NO_INTERACTION: 1
    COMPOSER_NO_AUDIT: 1
    COMPOSER_AUTH: {"github-oauth": {"github.com": "***"}}
    COMPOSER_CACHE_FILES_DIR: /home/runner/.cache/composer/files
    GITHUB_AUTH_TOKEN: ***
Phive 0.15.2 - Copyright (C) 2015-2023 by Arne Blankerts, Sebastian Heuer and Contributors
Downloading https://api.github.com/rate_limit
Warning:  Github API Rate Limit exceeded - cannot resolve "qossmic/deptrac"
Downloading https://gitlab.com/api/v4/projects/qossmic%2Fdeptrac/releases/
Fetching repository list
Downloading https://phar.io/data/repositories.xml
Error:    Could not resolve requested PHAR qossmic/deptrac

Error: Process completed with exit code 4.

Something wrong with this PR.

I can see the same. Maybe line 75 has extra whitespaces instead of an empty blank line.

Looks like it's resolved.
But @Kenjis, I think we shouldn't merge this. My goal with these PR was to fix problem deptract, which I don't think can be fixed. As mentioned in this https://github.com/codeigniter4/CodeIgniter4/pull/7065#issuecomment-1375173599. Unless there is another reason for joining?
Do you agree with Close?

@datamweb Yes, this does not seem to work.
I want to fix the GitHub rate limit issue, but now I don't have a clue.

I close this PR, because now there is no problem for Dependency Tracing.


how to get faster reconnects from deep sleep

Hello,
I currently have a project that uses a ESP32-s3 as a client. It connects with a BLE remote(server) that acts as HID keyboard. I just got everything working and it's catching all the button presses. However, the remote goes into deep sleep after 10 minutes of inactivity. Any button press wakes it up and the ESP32 automatically sees it and reconnects. The issues is that it takes ~2 seconds from starting to completing the reconnection. So any button presses in between those 2 seconds are not caught. This makes it a little awkward to use. Escpeclliy for the non-tech people I am targeting for use. I have a BLE mouse that also goes into deep sleep after a period of inactivity. But it reconnects on my laptop instantly. It seems like sub 1 second. So I know it's possible to reconnect faster. I did a few hours of internet sleuthing and wasn't able to find anything that was helpful. I did come across this post that references the blemouse2xac project. However I didn't find anything there that I'm not already doing. Any ideas on how to speed up the reconnections? Thanks

I was able to get the reconnect down to ~0.75 seconds by specifying the NimBLERemoteCharacteristic to subscribe to. I have it pared down to 3 NimBLERemoteCharacteristic or 3 handles (23, 31, 35). The bulk of the reconnect time is subscribing to the characteristics. One (23) takes ~0.05 seconds to subscribe. But the next two (31, 35) take ~0.35 seconds each. I couldn't get it to go any faster.
One strange thing is that I only need two (23, 31) of the characteristics to catch button presses. But for some reason, if I don't subscribe to the third one (35) I never receive notifications on the other two. If I can get this part figured out then I can cut the time down to ~0.4 seconds.

Is there a sample code you can share? I'm exactly in the same boat except light sleep instead of deep. Also may I ask how did you manage to wake up from deep sleep with so many keys? AFAIK waking up from deep sleep is only possible with RTC pins which are very limited.

@tapir , I just made my code into a library. I can't speak for your device. But the remote I'm using wakes with any button press. I have done 0 modifications to the remote. I've only been working on the ESP32 as a client. I don't touch the server. The remote goes for close to $1 on Aliexpress if you want to check it out.
I'm kinda thinking to achieve near instantaneous reconnects from sleep (like on a firestick remote) there would need to be changes to the BLE remote code. But I don't plan on attempting that because: 1. I'm treating it as a off-the-shelf part and 2. I would have no idea how to do that.


Perf CI parity for FD-2

See CI perf pipeline, main is green: https://github.com/tenstorrent/tt-metal/actions/runs/8827091120

GS: models/demos/resnet/tests/test_perf_resnet.py::test_perf_bare_metal, four different resnet tests, inference too slow

ResNet50-20_ : Inference Time (sec): 0.0578, Expected Inference Time (sec): 0.04
resnet50_ 224x224_batchsize16 : Inference Time (sec): 0.0517, Expected Inference Time (sec): 0.015
resnet50_ 224x224_batchsize20 : Inference Time (sec): 0.0558, Expected Inference Time (sec): 0.0185
resnet50_batch_size16_ 224x224_batchsize16 : Inference Time (sec): 0.047, Expected Inference Time (sec): 0.015
resnet50_batch_size20_ 224x224_batchsize20 : Inference Time (sec): 0.0511, Expected Inference Time (sec): 0.0185

N300: models/demos/mamba/tests/test_mamba_perf.py::test_mamba_e2e_perf[32-10-11.0-0.75], compile time too slow

Compile Time (sec):  13.37, Expected Compile Time (sec): 11.00

N300: models/experimental/functional_stable_diffusion/tests/test_perf_stable_diffusion.py::test_stable_diffusion_perf[1-2-image_size0-3600-1.8], inference too slow

Inference Time (sec):  1.8703, Expected Inference Time (sec):  1.8000

weird that compile time changed, is test_mamba_perf.py compile time threshold flaky on main?

@pgkeller maybe increasing the cmddat_q size could help bridge some of these gaps

weird that compile time changed, is test_mamba_perf.py compile time threshold flaky on main?

Happens consistently here (3/3 runs), haven't see that step fail when monitoring main for the last couple weeks

Same models are less than expected perf when running with increased fetch q (4K entries), cmddat queue (128K -> 256K) and prefetch size (64K -> 128K)
https://github.com/tenstorrent/tt-metal/actions/runs/8838161835
from branch abhullar/fetch-cmddat-sizes
@davorchap @pgkeller


Quick View - Add specific Contact information to the Details tab

Admin requests that the phone number of the Technical POC be added to the Details tab of the QuickView modal.

Contacts types are configurable and a given installation of Ozone may not even have a Technical POC.  Although this request sounds simple, my guess is that the implementation would not be straight-forward.  I am inclined to reject this ticket as Contact details are already available on the Resources tab.

Contact info is already available.


Dependency resolution order difference between pip and pipenv

Took me awhile to track down what I was seeing as well as come up with the smallest test example.
So if you have a requirement of pytest-django<2.10 which in it's setup.py has an install_requires entry of pytest>=2.5.  You also have a requirement of pytest<2.10.
Pip on it's own handles this just fine.  Pipenv ends up with the pytest at the current release version.
[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:51:42] $ ll
.
[-rw-r--r-- christian staff     121 Mar 29 18:29]  Pipfile
[-rw-r--r-- christian staff      31 Mar 29 18:51]  req.txt

0 directories, 2 files

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:51:43] $ cat Pipfile
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
pytest = "<2.10"
pytest-django = "<2.10"

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:51:44] $ cat req.txt
pytest<2.10
pytest-django<2.10

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:51:46] $ pip freeze

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:52:27] $ pip install -r req.txt
Collecting pytest<2.10 (from -r req.txt (line 1))
  Using cached pytest-2.9.2-py2.py3-none-any.whl
Collecting pytest-django<2.10 (from -r req.txt (line 2))
  Using cached pytest_django-2.9.1-py2.py3-none-any.whl
Collecting py>=1.4.29 (from pytest<2.10->-r req.txt (line 1))
  Using cached py-1.4.33-py2.py3-none-any.whl
Installing collected packages: py, pytest, pytest-django
Successfully installed py-1.4.33 pytest-2.9.2 pytest-django-2.9.1

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:52:35] $ pip freeze
py==1.4.33
pytest==2.9.2
pytest-django==2.9.1

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:52:39] $ pip freeze

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:52:55] $ pipenv install
No package provided, installing all dependencies.
Pipfile found at /Users/christian/tmp/test_3/Pipfile. Considering this to be the project home.
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
⠹Locking [packages] dependencies...
⠦Updated Pipfile.lock!
Installing dependencies from Pipfile.lock...
Collecting appdirs==1.4.3
  Using cached appdirs-1.4.3-py2.py3-none-any.whl
Collecting packaging==16.8
  Using cached packaging-16.8-py2.py3-none-any.whl
Collecting py==1.4.33
  Using cached py-1.4.33-py2.py3-none-any.whl
Collecting pyparsing==2.2.0
  Using cached pyparsing-2.2.0-py2.py3-none-any.whl
Collecting pytest==3.0.7
  Using cached pytest-3.0.7-py2.py3-none-any.whl
Collecting pytest-django==2.9.1
  Using cached pytest_django-2.9.1-py2.py3-none-any.whl
Requirement already satisfied: setuptools==34.3.3 in /Users/christian/.virtualenvs/test_3-9pMWYqV9/lib/python3.6/site-packages
Collecting six==1.10.0
  Using cached six-1.10.0-py2.py3-none-any.whl
Installing collected packages: appdirs, six, pyparsing, packaging, py, pytest, pytest-django
Successfully installed appdirs-1.4.3 packaging-16.8 py-1.4.33 pyparsing-2.2.0 pytest-3.0.7 pytest-django-2.9.1 six-1.10.0

⠙

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:53:17] $ pip freeze
appdirs==1.4.3
packaging==16.8
py==1.4.33
pyparsing==2.2.0
pytest==3.0.7
pytest-django==2.9.1
six==1.10.0

[test_3-9pMWYqV9][christian@WorkBookPro:~/tmp/test_3]
[18:53:23] $

this is also apparent now that Django has released 1.11.  I have Django = "<1.11.0" in my pipfile.  Other requirements have as their requirements 'django, Django, django>=1.8, Django>=1.8".  Pipenv will end up installing 1.11 instead of 1.10.7 which is annoying is there are breaking changes.

Hey @c17r, sorry this has slipped through the cracks. I'm working my way through the backlog of tickets that's piled up in the last week.
Could you provide an example Pipfile for me with the Django issues so I can test a few things out. Thanks!

[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
django-environ = "*"
Django = "<1.11.0"

is a good short example, you'll end up with 1.11

Oddly
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
django-extensions = "*"
Django = "<1.11.0"

does what you'd expect

Ok, I'm now convinced we can be doing better here. The issue is when a package isn't specified with an exact version (e.g. ==1.10.8), we choose the first downloaded copy of the package. This works in the second example because Django isn't a (non-test) requirement for django-extensions. It is however a requirement for django-environ which doesn't have a specified version requirement so 1.11.0 is set first and can't be overwritten.
I've opened #305 to hopefully address this better. I've done some initial testing and we seem to be getting the desired results but I'd welcome you, or anyone else with an inclination, to test it out before we merge it.

I tested #305 and all my issues are working now.

@c17r Great! We'll get this slotted for 3.6.0 then.

I found something else somewhat along these lines:
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
Django = "<1.11.0"

[dev-packages]
django-debug-toolbar = "*"

You end up with Django listed in both default (==1.10.7) and develop (==1.11) section of the lock file.  If you do pipenv update --dev, you'll end up with Django 1.11

This piece operates differently in pip that I expected. We download and evaluate dependencies for packages and dev-packages separately, so they have no knowledge of what the other contains.
I think a solution here is to check the contents of packages when generating dev-packages and only write dependencies to dev-packages when they aren't present in packages. This should still get us all of our dependencies, and not overwrite specifics in packages. This is still flawed though because a specific requirement in dev-packages won't hold weight if it's filled by a packages dependency.
pip doesn't really handle dependency conflicts right now, which means we'll have to roll our own solution. We've been avoiding that thus far but I'm not seeing a way to avoid it here. We'd need to compare every package in the Pipfile and choose the "most specific" option for each dependency. It would likely mean raising an exception for conflicting Pipfile requirements (e.g. packages: (pytest==1.0.0, dev-packages: pytest==0.6.0).
@kennethreitz, any thoughts on this? It'll be a semi-significant extension but is functionality pip provides that we've "broken".

This is still flawed though because a specific requirement in dev-packages won't hold weight if it's filled by a packages dependency.

But is that really an issue?  When you install/update, it's either packages or packages AND dev-packages, never just dev-packages.  If a dependency is needed by both, in my opinion, it should be listed in packages.

But is that really an issue? When you install/update, it's either packages or packages AND dev-packages, never just dev-packages. If a dependency is needed by both, in my opinion, it should be listed in packages.

Yeah, I think it definitely could be. If some_package has a dependency flask-oauth specified as a dependency, we'll install flask-oauth at the latest version for packages. Then when we get to dev-packages and the user has specified flask-oauth==0.5.2, the approach listed above will decide that flask-oauth is already covered and ignore the pinned requirement. That's the same problem you're experiencing, just in reverse.

Would it be possible to compare the output of pip freeze against pipfile.lock after a pipenv install, and warn on any packages where the versions don't exactly match?

was this fixed by 56aaa86a ?

i think it's safe to say yes.


Replace Error raised by less than 5 minutes of data by a Warning

PR submission for #158

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (ae9ccc7) 92.59% compared to head (3f2479e) 92.53%.

Files
Patch %
Lines

yasa/staging.py
33.33%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
- Coverage   92.59%   92.53%   -0.06%
==========================================
  Files          23       23
  Lines        3104     3106       +2
==========================================
  Hits         2874     2874
- Misses        230      232       +2

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


CSV import createEntities

Updating CSV import adding an optional field "createEntities" (true by
default) to create missing BusinessServices,TagTypes and Tags in the Hub database.
https://issues.redhat.com/browse/TACKLE-691
https://issues.redhat.com/browse/TACKLE-693

Can be tested (and works for me locally) with:
curl -F "createEntities=false" -F<EMAIL_ADDRESS>localhost:8080/importsummaries/upload

Tested and looks good. Works with https://github.com/konveyor/tackle2-ui/pull/317


Cloud Backoff should be easily configurable for users

As a user, I should be able to both enable and modify cloud provider backoff feature as well as adjust the settings without having to tinker with the generated templates.
Acceptance Criteria

Backoff/rate limiting can be enabled via API model (aka user configuration)
All cloud backoff configuration can be set through the API model

Background
This is a follow-up to https://github.com/Azure/acs-engine/issues/922 which implemented cloud backoff in acs-engine as a PoC but short of a first-class feature. Backoff/rate limiting is needed by the Cloud Console team as a first class configuration vector.
The Cloud Console team in particular called out:

We have full automation pipeline that receives api model, generate template, persist (and deploy in future). It would be really nice if we could simply turn on this feature via api model, instead of manually changing the template parameter files each time.

@yangl900 I'm going to expose the various cloudprovider backoff parameters in the api model in the kubernetesConfig properties.orchestratorProfile object, e.g., nodeStatusUpdateFrequency below:
...
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes",
      "orchestratorVersion": "1.6.6",
      "kubernetesConfig": {
        "nodeStatusUpdateFrequency": "1m"
      }
    },
...

Will that be an acceptable solution?


[KYUUBI #6638][FOLLOWUP] Authz shaded should include jsr311-api

:mag: Description
Issue References 🔗
Fix a ClassNotFound issue.
java.lang.NoClassDefFoundError: org/apache/kyuubi/shade/javax/ws/rs/core/Cookie

Types of changes :bookmark:

[x] Bugfix (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 change)

Test Plan 🧪
Verified manually.

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (2d64255) to head (56e9842).

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6723   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         684     684
  Lines       42282   42282
  Branches     5767    5767
======================================
  Misses      42282   42282

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

Thanks, merged to master (1.10.0).

Cherry-picked to 1.9.3 too, as the original PR goes that.

OK, thanks.


added some art and some blog posts

some minor updates to add 3 art projects to show. finally wrote some more to the blog

its perfect cause i wrote it right


Add support for observing NotificationCenter notifications

@boyvanamstel I noticed I forgot to stop the observation in de observable's deinit. I fixed this in the latest commit. Since the NotificationObservable will be retained by observers when it's observed, and released when the observer is released, you never need to manually remove the observation to avoid an assertion. It might be a good idea to clarify this behavior in the README though.


Fix fieldlist macro geneartion for special struct members X and a

Hello,
while using nanopb I noticed a bug in the python generator. I think my extended special character test case should make clear what the problem is. However, I try to put it in words below.
Naming message (struct) members in a .proto file X or a crashes with the automatic message_FIELDLIST macro generation of the nanopb python generator.  The macro uses X and a as parameters. If the generated *.pb.[ch] files are now compiled, the preprocessor will replace also the member name in the macro which is IMO not intended.
In most cases a .proto snippet
message SimpleMessage {
    required int32 fieldname = 1;
}

is converted to a macro like this
#define X_FIELDLIST(X, a) \
X(a, STATIC,   REQUIRED, INT32,    fieldname,                 1)

However, using the following proto snippet with a as member
message SimpleMessage {
    required int32 a = 1;
}

leads to the following unwanted macro where two as exist:
#define X_FIELDLIST(X, a) \
X(a, STATIC,   REQUIRED, INT32,    a,                 1)

I extended the special character test .proto file to recreate the bug in the first commit. The second commit makes the test pass by checking if the parameters X and a of the macro are a field name. If they are a field name, the generator now appends underscores to X or a until they don't match any field name. The fixed versions produces the following macro from the second .proto snippet:
#define X_FIELDLIST(X, a_) \
X(a_, STATIC,   REQUIRED, INT32,    a,                 1)

I hope this is helpful.
Best regards
Ket3r

Great, thanks a lot!


[AMBARI-22918] Decommission RegionServer fails when kerberos is enabled

What changes were proposed in this pull request?
Modified the unit test (test_hbase_msater.py) to fix the unit test failure.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/615/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/623/
Test FAILed.
Test FAILured.

Test failures were in Ambari Metrics Flume Sink - not related to this PR.


compress/gzip: Reader unable to parse headers with long comments

Using go1.6
RFC 1952 for gzip does not specify a length for the comment and name fields.

If FCOMMENT is set, a zero-terminated file comment is
present.  This comment is not interpreted; it is only
intended for human consumption.  The comment must consist of
ISO 8859-1 (LATIN-1) characters.  Line breaks should be
denoted by a single line feed character (10 decimal).

The current implementation is inconsistent:

gzip.Writer permits the writing of any length comment/name string in gzip.Header.
gzip.Reader fails to read any comment/name string in gzip.Header longer than 511 bytes.

Playground example: https://play.golang.org/p/Zvjf8Q7jXe
Change 512 to 511 in the example, and it works again.
This causes issues reading gzip files produced by GZinga, which produces syntactically valid gzip files, but abuses the comment field to store meta data.
Update: I have no intention of fixing this unless there is someone who needs this functionality. Whatever fix we do will need to be careful that we don't introduce a potential vector for DOS attacks since gzip is a common Transfer-Encoding in HTTP. We don't want an infinitely long comment to cause a server to allocate an infinite amount of memory.

Bumping up milestone since #20083 provides good evidence that this is a real issue.

Pushing to Go1.11 milestone... but I think a more general API for resource limitation might be the better approach. See #20169 for some discussion.

@dsnet This is still an issue. Did it go under?

Running the snippet on Go1.15 clearly shows it's still an issue.
Huh, apparently I had a fix for this that I never submitted: https://golang.org/cl/53637

@dsnet Okay, any chance you can push that forward? I am using gzip.NewWriter() to compress arbitrary text files (up to 1GB in size) while being unable to uncompress those using the same gzip.NewReader().
For anyone else having that issue: a pragmatic solution might be to copy gunzip.go and adjust the maximum buffer size manually. It seems several other gzip modules out there have that exact same issue (because they copied the stdlib code).

We have several options:

We could immediately permit parsing of small headers (I proposed up to 32KiB to match the window size of the DEFLATE algorithm itself). However, this won't help use-cases that abuse the comment field to store machine-parsable information where it can easily exceed 32KiB.
Add a new Reader.MaxFieldSize method so that the user can opt-in to parsing larger headers. Use of this option would still be a bit odd, since you would have to do something like:

var zr gzip.Reader
zr.MaxFieldSize(math.MaxInt32)
if err := zr.Reset(r); err != nil {
    ... // handle err
}
... // make use of zr.Header.Comment

Since this would probably require an API change, I'm adding the NeedDecision label.
\cc @FiloSottile since he's probably interested in any possible DOS-related issues.


fix: keep the Android keyboard visible when pasting

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/41137
PROPOSAL: https://github.com/Expensify/App/issues/41137#issuecomment-2121656214
Tests

Use mobile browser (iOS chrome, Android chrome, ios safari)

Open any chat page, and start typing in the composer some text (abcd)
Move the cursor to the middle of the text (between letter b and c)
Copy the some other text in Clipboard (should be long enough as we need to test the scrolling behavior after paste as well)
Long press and paste in the Composer

Expected:

The value in the composer should be ab<copied text>cd

The cursor should be before the letter c instead of moving to the end of the text

The keyboard doesn't close during the whole process.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/Expensify/App/assets/165644294/522d9496-9442-4e93-aaf8-dad1805dd849

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 1.4.85-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/mountiny in version: 1.4.85-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.4.85-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Parameter referencing in PathItemObject.parameters.

My swagger definition defines a bunch of query parameters in the root.parameters  which I then reference from the pathItem.parameters section. When I try to access the operation parameters using app.op.parameters, it looks like only the 1st parameter referenced in pathItem.parameters is associated with the path's Operation object.
If I understand the swagger definition, pathItems.parameters are applicable to all the operations in the path.
Am I missing something?
Referencing a parameter from operation.parameters works as expected.

It should work when each $ref points to one parameter definition under root.parameters. I just add a test case for it, could you check if this is what you expected?
test-spec
test-case
If this is not what you described, could you help to provide a sample swagger.json/yaml and code?

I meant referencing a parameter in the path.parameter section. Something like this.
"paths":{
      "/a":{
         "get":{
            "parameters":[
               {
                  "name":"p2",
                  "in":"body",
                  "schema":{
                     "$ref":"#/definitions/d1"
                  }
               }
            ],
            "responses":{
               "default":{
                  "$ref":"#/responses/r1"
               }
            }
         },
         "parameters": [
            {
               "$ref":"#/parameters/p1"
            },
            {
               "$ref":"#/parameters/p2"
            }
         ]
      }
   }

The parameters p1 and p2 are being referenced from the path.parameters and not operation.parameters. I can only see p2 and p1_d associated with the operation but (if I understand the swagger spec correctly), p2, p1_d and p2_d are all valid parameters.
Let me know if you want me to send a PR with a test case.

The case you described is new to me, and yes, it's spec compliant. Any PR with test case is welcome, thanks.

PR 106 has the test case.

OK, thanks. I'll work on this part tomorrow evening.

this bug is fixed in v0.8.26, please feel free to reopen it when it's still failed.

Fix verified. Thanks for your quick responses.


Ln address

You are right, the documentation on this is behind the code.
The extension has an settings which can be accessed as admin. See screenshot below:

The hints on the input field should help a bit.
The Lightning Address API Endpoint will attach an LN-address to a particular NIP5. The LN-address functionality is provided by the lnurlp extension.
The above settings just allow the admin to configure the endpoint and the access key (can be on the same LNbits instance or on a different one)


Upgrade queue processor to boto3

Upgrade to boto3 so all major application components are using the same
AWS library.
The name of the fields saved to SQS have been renamed to make things
less ambiguous.
One unexpected (but notable) benefit to upgrading is that SQS messages are
no longer base64 encoded.
Connects #210

Rebased onto develop.

@azavea-bot rebuild

@azavea-bot rebuild

+1 tested.

Fixed lint.

Rebased onto develop.

I'll test again shortly.

One unexpected (but notable) benefit to upgrading is that SQS messages are
no longer base64 encoded.

Nice!


fix(config): add wakeup instructions for Nexia ZSENS930

In addition to wakeup instructions, I also made quoting of the button name consistent.

I've botched this enough that I'm going to redo it.


Fix use of numeric keypad 0 and 9 to move tab to first/last position.

Make numeric keypad work for commands using 0 and 9 in Firefox.
Refactor to build a second Chrome extension to work around Chrome's limitation of 4 action keys per extension.

Reminder:

[ ] Need to update Readme to mention second Chrome version.


1.8.0-dev sporadic hang on extracting layer

Discovered while testing #15040
Sporadically while pulling docker.io/library/golang, after all layers are downloaded and all other layers are extracted, the layer 124e2127157f will just fail to ever extract.  The behavior is exhibited on both, so it is unrelated to tar-split.
vbatts@noyee ~ (master) $ docker version
Client:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.5beta2
 Git commit:   c328e0c
 Built:        Tue Jul 28 14:58:30 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.5beta2
 Git commit:   c328e0c
 Built:        Tue Jul 28 14:58:30 UTC 2015
 OS/Arch:      linux/amd64
vbatts@noyee ~ (master) $ docker info
Containers: 0
Images: 24
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.0.6-300.fc22.x86_64
Operating System: Fedora 22 (Twenty Two)
CPUs: 4
Total Memory: 7.081 GiB
Name: noyee.usersys.redhat.com
ID: EOK5:2MLO:KGVZ:UCYR:N3H3:SBE4:2IPK:VAHK:4U6F:VO4G:FR2K:M5W3
Debug mode (server): true
File Descriptors: 13
Goroutines: 30
System Time: 2015-07-28T11:43:43.52049014-04:00
EventsListeners: 0
Init SHA1: c801c701ea1dc09fdb7a7188952dbd8b0abcba86
Init Path: /home/vbatts/src/docker/docker/bundles/1.8.0-dev/dynbinary/dockerinit
Docker Root Dir: /home/docker
Username: vbatts
Registry: https://index.docker.io/v1/

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

cc @tiborvass @icecrime

just hit this on ubuntu:14.04 layer d2a0ecffe6fa


A bleeding logo for JSPatch

Here is a banner for README.

:+1:

It make me think of menstruation.
CN:他让我想起了大姨妈，好恶心

Thanks lexrus, the design is ingenious, but I think that simplicity style would be more fix in JSPatch , I will redesign a banner base on your idea in future.


Can't remove non-exisiting favorites

Gitea version (or commit ref): ff2464c
Git version: what try.gitea.io is using
Operating system: what try.gitea.io is using
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Description
I have added https://try.gitea.io/gitea/gitea to my favorites, but that repository seems to be deleted / moved. When I'm visiting https://try.gitea.io/jhasse?tab=stars now, clicking on the repo will result in a 404 error and I have no way to remove it from my favorites.

Oh. I've just noticed that I only get a 404 for https://try.gitea.io/gitea/gitea when I'm logged in. So I guess this a different issue?
Sorry for the confusion.

@lunny @bkcsoft Seems that the repo was deleted?

Maybe similar to #1797. I think Units might be the issue.

https://try.gitea.io/gitea/gitea Still exists for me :/

@jhasse since #1797 is closed by #1873, maybe you can confirm that this issue is also resolved.

Yes, it's fixed :) Thanks!


Indentation issue on ExternalIPs

Hello,
When i use the service as ExternalIPs with multiple ip, the indentation is broken
values sample:
---
serviceType: externalIPs
service:
  externalIPs: ['<IP_ADDRESS>', '2345:0425:2CA1:50']

The generated service look like
---
apiVersion: v1
kind: Service
metadata:
  name: coredns-coredns
  labels:
    app.kubernetes.io/managed-by: "Helm"
    app.kubernetes.io/instance: "coredns"
    helm.sh/chart: "coredns-1.19.7"
    k8s-app: coredns
    kubernetes.io/cluster-service: "true"
    kubernetes.io/name: "CoreDNS"
    app.kubernetes.io/name: coredns
spec:
  selector:
    app.kubernetes.io/instance: "coredns"
    k8s-app: coredns
    app.kubernetes.io/name: coredns
  externalIPs:
      - <IP_ADDRESS>
    - 2345:0425:2CA1:50
  ports:
  - {"name":"udp-53","port":53,"protocol":"UDP"}
  - {"name":"tcp-53","port":53,"protocol":"TCP"}
  type: externalIPs

As you can see the external ips have identation issue.
I suppose the issue could be fix by changing the service template line 41 by
{{ toYaml .Values.service.externalIPs | nindent 4 }}

Fixed in https://github.com/coredns/helm/pull/87


PHOENIX-7352 Improve OrderPreservingTracker to support extracting par…

…tial ordering columns for TupleProjectionPlan

This PR is an improvement for  PHOENIX-5148 to further optimize ClientScanPlan and ClientAggregatePlan. It modify OrderPreservingTracker to make OrderPreservingTracker.getOrderPreservingTrackInfos could also extract partial continuous ordering columns which start from the first column even if OrderPreservingTracker.isOrderPreserving is false.

@tkhurana, could you help me have a review, thank you very much.

@comnetwork I will take a look.


hash128 returns unsigned int

I was using mmh3 as part of a project, and was getting invalid values when I tried to rescaled the hashes into the range [0, 1]. Turns out, mmh3.hash128 was returning unsigned integers, not signed integers as the documentation suggests.
I was using Python 3.6.2 and mmh3 2.4.

Hi,
I'm so sorry for responding this late. I located where the cause of bug is, but considering that it was there for many years, I'll change the documentation side and say that hash128 returns unsigned values, so that no one will need to modify his/her code by the next release. In addition, I plan to add a new argument to get a signed value if one really requires signed values. Thank you for reporting an error!