halign not working in nested Layout

Affected hardware version
Bangle 2
Your firmware version
2v12
The bug
var clockRunning = false;

var Layout = require("Layout");
var layout = new Layout( {
  type:"v", fillx:1, filly:1, c: [
    {type:"txt", font:"27%", label:"15:00", id:"time" },
    {type: "h", fillx:1, bgCol: '#ff0', c: [
      {type:"txt", font:"14%", label:"Q1", id:"quarter", halign: -1, col: '#fff', bgCol: '#f00' }
    ]}
  ]}, {btns: [
  {label:"Start/Stop", cb: l=>clockRunning = !clockRunning}
], lazy: true});

g.clear();
layout.render();

The txt-elemnt is render centered, although the parameter halign is set to '-1':

Expeted behaviour:
The parent 'h'-element is expanded to full width (yellow color), so the child element should be aligned the right way.

This is actually expected behaviour - I forget where it was now but there was big discussion about this.
The docs in the Layout module say: https://github.com/espruino/BangleApps/blob/master/modules/Layout.js#L48

A halign field to set horizontal alignment WITHIN a v container. -1=left, 1=right, 0=center
A valign field to set vertical alignment WITHIN a h container. -1=top, 1=bottom, 0=center

In an h container you might have more than one element in it, so I'm not sure what's expected to happen if you have one item with right align and one with left since they would then conflict...
I'm open to changes for the layout library but it has to be as quick as possible and lightweight, and that means not being able to handle every eventuality.


Complete The Circle

Experiment yourself with making different circles at different positions and different gradients by completing the 2D-Canvas Functions.

is this issue still open ?

Yes


Regenerate headless schema on app redeploy or site change

As of now, headless schema is generated once (on app startup) so any changes done to types in the app won't be reflected after the app is redeployed. The same issue happens whenever user adds application(s) to or removes application(s) from the site - the schema will not be updated with the new changes.
To solve this we should regenerate schema on app redeploy and whenever site config is updated.

This is strange, app redeploy == stop and start? How does this not trigger rebuild of the schema?

Yes, you are right. If app is redeployed a schema will be updated.
In this task we should focus on update/re-create a schema when any app was added or removed from the site and to generate types based on apps from site configs, not only current app.

How will we handle a case when one of the apps (with new schema changes) assigned to the site was redeployed? If we only track when apps are added/removed then this will not solve this case.

Ah, we are talking about an app which added to site, not about an app which based on headless-starter.
Then a description of the issue is correct. Sorry for confusing.

Ok, maybe the title should be updated then.
However, currently we don't load CTY's from any other apps at all - using the headless starter. So I don't see how this task is relevant at all?

Oh.. maybe this actually is the task about generally being able to pickup changes from other apps? PLS consider the task titles more carefully in the future.

We might separate this into two different tasks if that's the problem, but I think the two problems are very related. In both cases you regenerate schema on event.

Sure, as many tasks as you like. My point is essentially that this task title does not make any sense?

Feel free to update :)

will be done on lib-guillotine side


[PE-D][Tester A] Postal code not existent

postal code 119999 is not existent in Singapore, but it is valid for the application.

Labels: severity.Medium type.FeatureFlaw
original: CedricPei/ped#11

We only check for postal regions i.e. first 2 digits and do not keep a map of every single postal code in Singapore as that is infeasible


cv2.stereoCalibrate behaves differently to c++ implementation

When I call cv2.stereoCalibrate(...) with 14 element distortion coefficient vectors, the returned distortion coefficient vectors have only 5 elements.
Actual behaviour (Python 3.8.10)
When the following gets called
ret, mtxL, distL_out, mtxR, distR_out, R, T, E, F = cv2.stereoCalibrate(obj_pts, img_ptsL, img_ptsR,
                                                                        mtxL, distL_in,
                                                                        mtxR, distR_in,
                                                                        (w, l), criteria_stereo, flags)

Input left distortion parameters are:
distL_in =
[[-9.28027807e+00  5.37383609e+01 -3.06749966e-04  3.71928959e-03
  -2.39827139e+02 -9.28424990e+00  5.31178190e+01 -2.35537833e+02
   0.00000000e+00  0.00000000e+00  0.00000000e+00  0.00000000e+00
   7.26131243e-03 -4.69211193e-03]]

Output left distortion parameters are:
distL_out =
[[-9.28027807e+00  5.37383609e+01 -3.06749966e-04  3.71928959e-03
  -2.39827139e+02]]

The flags have been set as follows:
flags = 0
flags |= cv2.CALIB_RATIONAL_MODEL
flags |= cv2.CALIB_TILTED_MODEL
flags |= cv2.CALIB_FIX_INTRINSIC
flags |= cv2.CALIB_USE_INTRINSIC_GUESS
flags |= cv2.CALIB_FIX_FOCAL_LENGTH
flags |= cv2.CALIB_FIX_PRINCIPAL_POINT
flags |= cv2.CALIB_FIX_ASPECT_RATIO
flags |= cv2.CALIB_FIX_TANGENT_DIST
flags |= cv2.CALIB_FIX_K1
flags |= cv2.CALIB_FIX_K2
flags |= cv2.CALIB_FIX_K3
flags |= cv2.CALIB_FIX_K4
flags |= cv2.CALIB_FIX_K5
flags |= cv2.CALIB_FIX_K6
flags |= cv2.CALIB_FIX_S1_S2_S3_S4
flags |= cv2.CALIB_FIX_TAUX_TAUY

Expected behaviour (C++)
When I run the same function in c++ with 14 element vectors for the distortion coefficients, I get the correct result - the vectors remain 14 elements long.
I noticed in the c++ version that if I only enable the cv::CALIB_FIX_INTRINSIC flag, then the output distortion coefficient vectors do get reduced to 5 elements each, just like in the Python version.
The rotation R and translation T matrices produced by the python and C++ implementation are very different
Python translation vector (with 5 element output distortion vectors):
T = [-13.38267045 1.59477086 22.51344999]
C++ translation vector (with 14 element output distortion vectors):
T = [-5.067262324969635 0.01664551456133418 0.04499006665577673]
Also for your reference here is the C++ translation vector when only cv::CALIB_FIX_INTRINSIC flag is enabled (hence with 5 element output distortion vectors):
T = [-12.44688911164896 2.005676550526267 22.86746701014153]

operating system: Windows 10
architecture: amd64
opencv-python version: 4.5.2

Please ensure that arrays are passed to C++ code with the same layout. Use:

cv.utils.dumpInputArray() or cv.utils.dumpInputArrayOfArrays()

See also:

https://github.com/opencv/opencv/issues/18413

Hi alalek,
Thank you for your response.
As far as I can tell there are no problems with the way arrays are passed into the C++ code. The C++ code behaves as expected - 14 element distortion vectors are passed in and stereoCalibrate outputs the vector unchanged (as expected with the flags that I have enabled).
The problem is with the python implementation - I pass in a 14 element distortion array and a 5 element distortion array is returned.
Also, the array shapes does not seem to be the issue because:

Both the C++ Python implementation run and produce and output without any assertion errors.
The produce almost the same rotation and translation matrices when only the cv::CALIB_FIX_INTRINSIC flag is enabled.
I tried reshaping the 'obj_pts' and 'img_pts' arrays as suggested in opencv#18413, but this did not affect the python output in any way.

It looks like the python stereoCalibrate function is only using the first 5 elements in the distortion coefficient arrays regardless of the flags passed into the function, where as the C++ version is correctly using all 14 distortion coefficients.

Thanks alalek. Yes, I am aware of this. I use the lazy language "Python implementation" to discriminate between the C++ and python tests that I had performed.
I'm not familiar with the process used to generate the wrappers, but is there a way to look at the wrapper code for the stereoCalibrate function?

@AndrewMartchenko Could you provide full code to reproduce the issue?

@asmorkalov Attached is a simple example showing different results in python and C++. Both the Python and C++ implementation load all stereoCalibrate parameters from the attached XML file. Thank you!
stereo_calibrate_example.cpp.txt
stereo_calibrate_example.py.txt
stereo_calibration_arguments.xml.txt

@AndrewMartchenko
stereoCalibrate signature:

stereoCalibrate(objectPoints, imagePoints1, imagePoints2, cameraMatrix1, distCoeffs1, cameraMatrix2, distCoeffs2,
                imageSize[, R[, T[, E[, F[, flags[, criteria]]]]]])

stereo_calibrate_example.py.txt:

...
cv2.stereoCalibrate(object_points, camera_image_points, projector_image_points, new_camera_matrix,
                    camera_distortion, new_proj_matrix, proj_distortion, (2448, 2048),
                    criteria_stereo, flags)
...

Parameter flags is after criteria, and as flag use first element of criteria. Swap this parameters or consider using "names" (flags=flags, criteria=criteria_stereo)
This will fix your problem.

@AleksandrPanov Thank you, that fixed the problem. I can't believe I missed this.


add a field to capcode creation /edition

Add a field on the capcode alias creation/edit page to place a comment.
Examples :
"validated on 01 janv 21" or
" also linked with capcode xxx" or
" owner is john Doe".
A checkbox or another sign,  on the main page,  to inform/remember  that a comment does exist on this capcode/alias.
Thanks !

The comment field is added easily enough, The display on the main page is probably a bigger discussion. something for the future

The comment field is added easily enough, The display on the main page is probably a bigger discussion. something for the future

on the alias template, replace
{{alias.alias || ''}}

with
{{alias.alias || ''}}

obviously 'comment' column has to exist in the capcodes table. renders this on hover:

You could add the same way in main page, or as part of popover template, but would have to reference an extra column in messages. that could get messy. if all you wanted was a checkbox you could have a boolean "hascomment" column i guess

Hello,
More than one year later, I see this answer ...
Thanks,
I will test

With "main page", do you refer to the alias list or the message list?

With "main page", do you refer to the alias list or the message list?

Im going to be honest. I dont even remember writing this and would probably do it differently now 😂


Feat/VISE-CPC-111 Add Visit Type

JIRA: link to jira ticket
https://champlainsaintlambert.atlassian.net/jira/software/c/projects/CPC/boards/12/backlog?view=detail&selectedIssue=CPC-111&issueLimit=100
It is a work a progress I think so guys from the visits and bills teams, listen up, we need to find a way to send the visit type from the visits services to the bills services. Thing is the only way to do that would be by creating a get or a post mapping somewhere but that's where your review will be helpful to find out what to do exactly. Please be precise.
As for what works, the design for the label have been modified and the select boxes work.

Brandon, the design as been changed to what it was originally, as for the functionality of the post method Jean-Christophe I am currently developing this part. Let's hope I finish this feature soon for the bill team

I think @bthorng wanted the visit type select to be under the practitioner information because it looks a bit out of place when it's next to the practitioner select

I think @bthorng wanted the visit type select to be under the practitioner information because it looks a bit out of place when it's next to the practitioner select

Wait, that's actually a good point though. I think the form would look better that way.


TypeError: object of type <class 'numpy.float64'> cannot be safely interpreted as an integer.

when i run  ### python3 ./pytorch/train.py evaluate --config_path=./configs/car.lite.config --model_dir=/media/buaa/MyPassport/Deeplidar/second.pytorch/second/configs/car_lite --measure_time=True --batch_size=1
I got a problem as follow,
/home/buaa/.local/lib/python3.6/site-packages/numba/cuda/envvars.py:16: NumbaDeprecationWarning:
Environment variables with the 'NUMBAPRO' prefix are deprecated, found use of NUMBAPRO_NVVM=/usr/local/cuda/nvvm/lib64/libnvvm.so.
For more information visit http://numba.pydata.org/numba-doc/latest/reference/deprecation.html#deprecation-of-numbapro-environment-variables
warnings.warn(errors.NumbaDeprecationWarning(msg))
/home/buaa/.local/lib/python3.6/site-packages/numba/cuda/envvars.py:16: NumbaDeprecationWarning:
Environment variables with the 'NUMBAPRO' prefix are deprecated, found use of NUMBAPRO_LIBDEVICE=/usr/local/cuda/nvvm/libdevice.
For more information visit http://numba.pydata.org/numba-doc/latest/reference/deprecation.html#deprecation-of-numbapro-environment-variables
warnings.warn(errors.NumbaDeprecationWarning(msg))
/home/buaa/.local/lib/python3.6/site-packages/numba/cuda/envvars.py:16: NumbaDeprecationWarning:
Environment variables with the 'NUMBAPRO' prefix are deprecated, found use of NUMBAPRO_CUDA_DRIVER=/usr/lib/aarch64-linux-gnu/libcuda.so.
For more information visit http://numba.pydata.org/numba-doc/latest/reference/deprecation.html#deprecation-of-numbapro-environment-variables
warnings.warn(errors.NumbaDeprecationWarning(msg))
[  41 1280 1056]
Restoring parameters from /media/buaa/MyPassport/Deeplidar/second.pytorch/second/configs/car_lite/voxelnet-15500.tckpt
feature_map_size [1, 160, 132]
remain number of infos: 3769
Generate output labels...
[100.0%][===================>][8.35it/s][07:36>00:00]
generate label finished(8.19/s). start eval:
avg example to torch time: 3.211 ms
avg prep time: 5.921 ms
avg voxel_feature_extractor time = 1.500 ms
avg middle forward time = 49.402 ms
avg rpn forward time = 42.073 ms
avg predict time = 18.303 ms
/home/buaa/.local/lib/python3.6/site-packages/numba/compiler.py:588: NumbaPerformanceWarning:
The keyword argument 'parallel=True' was specified but no transformation for parallel execution was possible.
To find out why, try turning on parallel diagnostics, see http://numba.pydata.org/numba-doc/latest/user/parallel.html#diagnostics for help.
File "utils/eval.py", line 129:
@numba.jit(nopython=True, parallel=True)
def box3d_overlap_kernel(boxes,
^
self.func_ir.loc))
Traceback (most recent call last):
File "/home/buaa/.local/lib/python3.6/site-packages/numpy/core/function_base.py", line 117, in linspace
num = operator.index(num)
TypeError: 'numpy.float64' object cannot be interpreted as an integer
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./pytorch/train.py", line 665, in
fire.Fire()
File "/home/buaa/.local/lib/python3.6/site-packages/fire/core.py", line 138, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/home/buaa/.local/lib/python3.6/site-packages/fire/core.py", line 471, in _Fire
target=component.name)
File "/home/buaa/.local/lib/python3.6/site-packages/fire/core.py", line 675, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "./pytorch/train.py", line 543, in evaluate
str(result_path_step))
File "/media/buaa/MyPassport/Deeplidar/second.pytorch/second/data/kitti_dataset.py", line 149, in evaluation
z_center=z_center)
File "/media/buaa/MyPassport/Deeplidar/second.pytorch/second/utils/eval.py", line 884, in get_coco_eval_result
z_center=z_center)
File "/media/buaa/MyPassport/Deeplidar/second.pytorch/second/utils/eval.py", line 704, in do_coco_style_eval
min_overlaps[:, i, j] = np.linspace(*overlap_ranges[:, i, j])
File "<array_function internals>", line 6, in linspace
File "/home/buaa/.local/lib/python3.6/site-packages/numpy/core/function_base.py", line 121, in linspace
.format(type(num)))
TypeError: object of type <class 'numpy.float64'> cannot be safely interpreted as an integer.
what can i do to solved it?
system info：
Xavier ubuntu18.04
pytorch 1.3.0
cmake 3.13.2
pillow 6.2.2
numba 0.44.1
numpy 1.18.2
llvmlite 0.29.0

Hi,
numpy 1.18 will have this problem, earlier versions work, see e.g.
https://github.com/xingyizhou/CenterNet/issues/547
To solve it, you can either downgrade your numpy, or modify utils/eval.py, line 704:
for i in range(overlap_ranges.shape[1]):
        for j in range(overlap_ranges.shape[2]):
            a, b, c = overlap_ranges[:, i, j] #extracting the three numbers
            min_overlaps[:, i, j] = np.linspace(a, b, int(c)) #casting to integer
            #min_overlaps[:, i, j] = np.linspace(*overlap_ranges[:, i, j])

I left the original line in there for reference. The trick is that the third param of linspace has to be an int.

i  also meet the same problem, thanks!

I had the same issue and the change in eval.py suggested by @paland3 solved it. Thanks


DRILL-7046: Support for loading and parsing new RM config file

@HanumathRao  - Can you please help review this PR ?

Quick question. Why do we need a separate RM config file? Why can't this just be part of the existing Drill config file? By having a separate file, it just makes yet another detail for users to understand and distribute. I also wonder if this would suggest to future developers that each new feature should have its own config file?
As a counter-example, the earlier "large/small queue" RM system hosted it's config settings within the existing Drill config file?
Perhaps an intro comment for this commit could explain the reasoning?

@paul-rogers - For question related to new configuration file.

Usually number of configurations for a new feature in Drill ranges from 5-6 at max. For RM the configuration is extensive to provide lots of flexible options for an admin. As contrast to earlier queue now admin can configure as many queues needed and hierarchy of ResourcePool can be both deep and wide. Each ResourcePool have options to configure selectors, memory resource percentage, queue, etc. Given the configuration is so huge I think it calls for a separation into it's own file.
Also when RM is enabled then only these configurations will be needed. If RM related configuration is present in drill-override.conf with feature disabled then still Drill will load all of it in memory which won't happen with this separate file in place.

This PR only add's support for the new RM configuration parsing and creating in-memory structures. There are still lots of components which needs to be developed like planner components, scheduler, etc. All the changes will come in phases. I have already added functional spec in the JIRA which has detailed explanation about the configuration. But I have also tried adding package-info.java with necessary information. Also I have moved all the files to a new config package under resourcemgr package to represent this PR contains only config changes. Please let me know if it looks sufficient to you. As far as detailed design is concerned you can find the document explaining it in DRILL-7026 which is parent JIRA of DRILL-7046.

@paul-rogers - I have rebased the PR on latest master to resolve conflicts. Can you please review the last changes ?

@sohami  Thanks for making the changes. Changes look good from my end. LGTM +1.


Are delayed_jobs hooks designed to work with active_job ?

It looks like it requires redefinition of a lot of the methods that are defined in active_jobs, causing collisions. For example, #enqueue
Further, it seems like there is no easy way to access the delayed_jobs data base columns from within an active_job, for example, in a job:
class AutoResolverJob < ApplicationJob
  queue_as :low_priority

  def perform(id)
    # do stuff
  end

end

How would I add the #enqueue hook  without redefining the ActiveJob#enqueue method AND
How would I access a custom DelayedJob db column to set it to a given value, for example id.
What I am trying to do is to  store a reference to the delayed business entity in the job as per
https://www.sitepoint.com/delayed-jobs-best-practices/    ("Columns for your delayed entity")

ActiveJob uses a wrapper job that obscures the actual jobs from delayed job. At this point, there is no way to trigger what you want through ActiveJob, but that is an interesting idea for an improvement to ActiveJob's integration with DJ.


Add explicit handling of null relationships

we're not adding the tests to this repo?

The tests have forked, so adding the tests here would be a bunch of work for relatively little value, so I've decided not to bring them across in this case.


Add the ability to delete your account

Account deletion happens in the background (since a user may have many scenarios, access tokens, etc) and needs to also tell ETModel to delete its data about the user.
Deleted users are flagged with a deleted_at column, which prevents them from being able to sign in through the web interface. The user will be completely removed by the DeleteUserJob background job.

Thank you once again for the translations, @mabijkerk. Very much appreciated!


swift run danger-swift edit doesn't include plugins; swift run danger-swift (ci|local) works

I have a danger subdirectory in my project.
This is it's structure:
.
├── DangerDependencies
│   └── Fake.swift
├── Dangerfile.swift
├── Package.resolved
└── Package.swift

This is Dangerfile.swift:
import Foundation
import Danger
import DangerSwiftKantoku

let danger = Danger()
let kantokuConfig = XCResultParsingConfiguration(
    codeCoverageRequirement: .none,
    reportingFileType: .modifiedAndCreatedFiles
)
danger.kantoku.parseXCResultFile(at: "somePath", configuration: kantokuConfig)

and this is Package.swift:
// swift-tools-version:5.8
import PackageDescription

let package = Package(
    name: "DangerDepsProject",
    platforms: [.macOS(.v12)],
    products: [
        .library(name: "DangerDeps", type: .dynamic, targets: ["DangerDependencies"])
    ],
    dependencies: [
        .package(url: "https://github.com/danger/swift.git", from: "3.17.1"),
        .package(url: "https://www.github.com/yumemi-inc/danger-swift-kantoku.git", from: "0.1.1")
    ],
    targets: [
        .target(
            name: "DangerDependencies",
            dependencies: [
                .product(name: "Danger", package: "swift"),
                .product(name: "DangerSwiftKantoku", package: "danger-swift-kantoku")
            ],
            path: "DangerDependencies"
        )
    ]
)

This works and it successfully uses plugins:
cd danger
swift build
swift run danger-swift local --failOnErrors --cwd ../ --base origin/develop

But I want to be able to edit the project, so I call this:
cd danger
swift build
swift run danger-swift edit

And this does not work, it generates the project where Package.swift does not contain plugins, so it won't build:
Generated Package.swift:
// swift-tools-version:5.8
// danger-dependency-generator-version:3

import PackageDescription

let package = Package(
    name: "Dangerfile",
    platforms: [.macOS(.v13)],
    products: [.executable(name: "DangerDependencies", targets: ["Dangerfile"])],
    dependencies: [
        .package(url: "https://github.com/danger/swift.git", exact: "3.17.1")
    ],
    targets: [.executableTarget(name: "Dangerfile", dependencies: [
        .product(name: "Danger", package: "swift")
    ])],
    swiftLanguageVersions: [.version("5")]
)

Swift Danger Version: 3.17.1
Example project: danger.zip

Thank you for the really detailed issue, let me give it a look and see if I can help :)


[Feature request] New API method for getting some assets stats

Platforms
Android, iOS
Description
I'd like to have a new method of getting assets statistics:
getStatAsync(assetPath, assetType: [RequestType.image, RequestType.video]) - getting stat by assetPath, and assetType filter in api call
Example result:
[
	["2023.12.01 00:00:00Z", 65], // Month in year (just first date of month), and count of assets
	["2023.11.01 00:00:00Z", 100],
        [...]
]

Why
I need this for building lazy gridView with some kind of indexed draggable scrollbar (for example A-Z scrollbar panel in contacts list), when knowing current offset I calculate asset index and show a hint about day or month where in time current scroll position located in gallery.
For now I'm calculating this data by loading and processing all the assets by page, but for large galleries this is becoming very lagging and memory consuming.
Similar request https://github.com/fluttercandies/flutter_photo_manager/issues/594 - but I need aggregation by months, or may be days.
Thanks in advance.

I managed to "improve" performance slightly by sampling the album instead of fetching all AssetEntities directly. However, this approach is still far from optimal in terms of efficiency.
I've submitted a related feature request (#1244) that proposes a more efficient solution for grouping assets, which might be of interest for this use case. Feel free to check it out and share your thoughts!


Better Routing Required

In a big application I see myself having to write routes for every single end-point.. Is there a better way or something I'm missing?
Something like the following should be available:
'get /account/:action': {
    controller: 'AccountController',
    action: ':action'
  }

Is there a way to simplify routes as such so that I don't have to write a route for every end point?

http://sailsjs.org/#/documentation/reference/blueprint-api

Bingo, @Dallas62.  Thanks!


Wasm unreachable EmitInfo

Create a new type Unreachable for EmitInfo.
Unreachable is valid for typecheck.
If 1 statement is unreachable in a block all the following statement has to be unreachable (not really important at first, shouldn't happen in real code)
Also, this will allow to correctly invalidate the following
function () : i32 {
  if (cond) {
    return 0;
  } else {
    call something // Right now, the IfElse block will yield void and we will assume unreachable when checking for the function return value
  }
}

There may be an overlap between this issue and an outstanding issue of type checking expressions (c.f. closed PR #735) such as xor (i32.const 1) (return (f32.const 0.0)) and if-else expressions with non-yielding control flow and yielding expressions. We have a pending fix for that which I'll rebase on the current WebAssembly.


Multiple runs for big tables

For large tables (~30GB), is it possible to run pgcompacttable till say 10%, do a manual VACUUM and REINDEX, and then run pgcompacttable again which will just do the remaining 90% of the work?

@Melkij
Hello! Could you please answer this question?
What happens if I stop the script? Does it go all over again and processes every single page or does it somehow knows last processed page and starts from this page?
In other words, how can I run script multiple times? In my production environment I can't just leave it running because it would run for 5 days.

Hello, if you stop pgcompacttable and run it again - it will start from the beginning: run vacuum, check bloat and then will try to move data from tail pages if there is enough bloat left.
During vacuum phase it's possible that it will remove tail pages that was cleaned up in previous run so it won't have to process them again. Also pgcompacttable runs vacuum periodically after certain amount of processed pages.
If other transactions prevent vacuum from acquiring lock and remove pages - pgcompacttable will process these pages again, but it will be faster because they should already be empty.
So in some cases - yes, it's ok to run/stop it multiple times to process big table, eventually it should finish. If table is not so big but pgcompacttable runs slowly - perhaps there are many bloated indexes which better be compacted before data movement with --initial-reindex option. Also there is --delay-ratio option which could speed up process (if there is enough resources).
pgcompacttable won't clean up for itself if stopped and will leave it's function in database (shouldn't affect anything), which may be dropped using this query:
select 'drop function ' || proname || '(text, text, integer, integer, integer);' from pg_proc where proname ~'pgcompact_clean_pages';

(it will return query/queries to run for removing pgcompacttable functions).

@alexius2
Thanks for the detailed answer!

If table is not so big but pgcompacttable runs slowly - perhaps there are many bloated indexes which better be compacted before data movement with --initial-reindex option.

Based on your experience, is adding --initial-reindex option also good not just for small, but big tables?
Our production tables is 190GB and it's biggest index is about 70GB (total indexes size is 483GB)
I'm interested in how this option will affect performance. Now I know that pgcompacttable doesn't really affect our database performance. I'm just a little scared to turn --initial-reindex option...
Thank you in advance!


Cookie Prefixes sample

R: @mikewest @addyosmani @ebidel @beaufortfrancois @gauntface etc.
Here's my sample for Cookie Prefixes, shipping in M49.
It would be nice if @mikewest could give a :+1: to the explanation I include, which is mostly cribbed from his spec draft. But I might be misrepresenting something...
There's a live version at https://jeffy.info/samples/cookie-prefixes/

One nit, otherwise LGTM.

Thanks for the review! I'm merging now that the feedback is addressed.


Ember upgrade 3.16

connect https://github.com/ember-learn/cli-guides/issues/173
npm install -g ember-cli-update
ember-cli-update --to 3.16

I can replicate the errors locally with ember s --environment production, if that's helpful to know.

@jenweber yes, I tried that already as it was failing only in the pipeline. The actual issue is when I do that, the error is on
pre-render /release/writing-addons/deploying failed with exception: FetchError: request to http://localhost/:7784undefined//localhost:7784/content/versions.json failed, reason: connect ECONNREFUSED <IP_ADDRESS>:80
It has undefined for some reason. That's why it is failing.

Ok, I’ll take a look at this tonight.

@jenweber @vinodhrajamuralidharan We (@mansona  and me) are also currently investigating if there's an issue with prember in the Netlify build step - will keep you posted if we can resolve the issue!

@vinodhrajamuralidharan Netlify deploys are unblocked after an upgrade of the prember dependency! Can you git pull the changes and do another rebase of your feature branch onto the latest master? Then this should be ready to go already! :sparkles:

@jayjayjpg seems it works now. Just it shows conflicts on package and package-lock.json and I don't have access permission to resolve that.

this looks like it still has conflict markers somewhere 🤔 maybe in the package-lock?

@mansona yes it is in package-lock.json


check entries in vocab box for correctness

already checked dictionary entries. need to compare with vocab box entries

checked all and made updates as needed


randomised training data

Hi,
I went through finetune/lora.py and saw that random batch is being fetched and used in training. Would it better if I use a dataloader which tracks samples already used in training? Plus, doesn't validation and saving checkpoint on nth tokens trained be better choice? Can we have more control on this - save by tokens trained, iteration steps?

I agree that using a DataLoader for the finetuning scripts would be a good idea.


Add: Camelcase Matching

Codecov Report

Merging #556 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #556   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         174    175    +1
  Lines        1738   1747    +9
=====================================
+ Hits         1738   1747    +9

Impacted Files
Coverage Δ

problems/camelcase-matching/camelcase_matching.go
100% <100%> (ø)

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 07b721b...f3b5f7f. Read the comment docs.


removeTrack: throw when called on a closed connection

fixes an upstream test failure in adapter

Codecov Report

Merging #66 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
+ Coverage   97.27%   97.27%   +<.01%
==========================================
  Files           1        1
  Lines         806      808       +2
==========================================
+ Hits          784      786       +2
  Misses         22       22

Impacted Files
Coverage Δ

rtcpeerconnection.js
97.27% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


docs: Update adding_state_migrations.md

Make the migration docs a bit more concise.

Closing this as apparently it wasn't a great idea! :)


Crop image on ImageContainer and data

Good day,
Thanks for developing this great tool. I want to use tangram to deconvolve each spot, but I have problems cropping using the same coordinates on the ImageContainer object and the data with the ST data. I do not understand how to subset the ST data to match the image. When I use crop_coor within the scanpy workflow to display the plot (for instance sc.pl.spatial(adata, img_key="hires", color="clusters",  crop_coord=[15000, 25000, 20000, 10000], alpha=0, size=1.3)) and try to use those on the ImageContainer, it does not seem to work. How can I subset the data and crop the image to match the same coordinates?
Thanks in advance!

hi @ccruizm ,
thanks for your interest in squidpy! in #324 we are working toward a method that makes it convenient for subsetting anndata according to the imgcontainer crop (give us another 2 weeks to this one in master and well documented with example/tutorial).
what's the type of cropping you are trying to achieve? with more details I could provide you witha working example for the time being.

Implemented via  #324 (on the dev branch) as e.g.:
adata = ...
img = sq.im.ImageContainer(....)
adata_subset = img.crop_corner(0, 0, (1000, 1000)).subset(adata)

Thank you! I used the implemented approach mentioned by @michalk8, and it works nicely!
The segmentation usually gives some false positives, and we have tried to use other tools to overcome this. I would like to know how to input info (and which format) generated from other tools into the data object to use these segmentation features instead. I saw in #324 that there seems to be a better segmentation method, similar to the tool we are currently using to do that, as seen in the image below. Is there a tutorial to use that approach?

Attached the output file from the tool we are currently using.
Thanks for your help!
spot_info.csv

hi @ccruizm ,
we are gonna release squidpy 1.2 sometime next week. Meanwhile, if you install it with pip install -e . from the dev branch, you can already try out the new tutorials.
We have added tutorials to work with Stardist and Cellpose:

https://squidpy.readthedocs.io/en/latest/external_tutorials/tutorial_cellpose_segmentation.html
https://squidpy.readthedocs.io/en/latest/external_tutorials/tutorial_stardist.html

let us know if that helps, they should also be general enough to help you plug in your own pretrained/fine tuned favourite model.

This looks great!!! Thank you very much! will give it a try

from #339 comment I understand this is solved. I'm gonna close this but feel free to reopen if needed.
you can also try the new segmentation tutorials from here: https://squidpy.readthedocs.io/en/latest/tutorials.html#external-tutorials

I know this issue has been closed, but I am having trouble with the same problem still. The .subset() function consistently gives me an AnnData object with n_obs = 0, and I can't figure out why.
adata = sc.read_visium(...) img = sq.im.ImageContainer(adata.uns['spatial'][libraryid]['images']['hires']) img_crop = img.crop_corner(x=xx, y=yy, size=s) adata_subset = img_crop.subset(adata) print(adata) print(adata_subset)
gives
AnnData object with n_obs × n_vars = 1434 × 19465     obs: 'in_tissue', 'array_row', 'array_col'     var: 'gene_ids', 'feature_types', 'genome'     uns: 'spatial'     obsm: 'spatial' View of AnnData object with n_obs × n_vars = 0 × 19465     obs: 'in_tissue', 'array_row', 'array_col'     var: 'gene_ids', 'feature_types', 'genome'     uns: 'spatial'     obsm: 'spatial'
Any advice? I've also tried the crop_coord input to scanpy.pl.spatial, which just shows a blank square.

Hey @nsauerwald ! Have you solved that problem? I got constant blank as well. Would you please give me some help? Thank you very much!

hi @RYY0722 @nsauerwald I believe the problem is because the crop you are selecting does not contain any observations. This is revealed by the fact that plotting also gives an empty crop. Are you sure you selected the coordinates for the bounding box correctly?

Hi @giovp thank you so much for your quick reply!
But I don't think that the region that I am cropping is empty.
Firstly,
img = sq.im.ImageContainer(adata.uns["spatial"]['c']["images"]["hires"], layer='image', library_id='c')

by calling   upper_img.show(layer='image'), I can see that it contains what I want.
By calling
upper_img = img.crop_corner(y=0.0, x=0.0, size=img.shape)
upper_img.subset(adata)
, where I am subsetting the whole image.
it returns
View of AnnData object with n_obs × n_vars = 0 × 36601

However,
img.subset(adata) works,
and it gives me AnnData object with n_obs × n_vars = 827 × 36601

@giovp Sorry that I still cannot solve this wired problem. Would you please help me look into that? Thanks a lot!

@giovp @nsauerwald Hi! I think that it is caused by that I didn't pass the scale factor when I create the image container.
It should be
scale = adata.uns['spatial']['c']["scalefactors"]['tissue_hires_scalef']
img = sq.im.ImageContainer(adata.uns["spatial"]['c']["images"]["hires"], layer='image', library_id='c', scale=scale)


Update: VirtualMagnifyingGlass.VirtualMagnifyingGlass version 3.6

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


kk_add_file_extension_iis_control

Add the ability to control file extension inclusion and exclusions.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

@kkennedy1 hey! two parts

can you sign the CLA
i've gone through the code but due to merge i'm unsure exactly what the changes are

Hello!
I've signed the ICLA.  Would you like to see any documentation from me?  All of the changes are made are named file_extensions.rb.  I've added a library, provider and resource, also a set of spec tests. Let me know if there's anything else I should do and thanks for your help.
Kevin

@smurawski Can you confirm the signed ICLA? I went to the contributors page but it isn't showing up.

Ugh, I keep getting "The Github account you are trying to link is already linked to another Supermarket user".  I must have crossed streams somewhere.  I'll get it fixed.

You'll need to make sure the email address tied to the commit is also associated to the github account you've tied to your supermarket account.  The email associated to the supermarket account doesn't matter.. It's the github account, email addresses associated there, and the email on the commits.

currently this shows no changes to the branch at all, closing. @kkennedy1 please re-open if needed.


Move mask to img2img

Hi there! Thank you for creating this extension, first of all.
Question: Is there a way to obtain the mask from the Inpaint Anything tab or can you only apply it right there using the inpainter there? I would really like having access to the mask.

Like a 'download mask' or 'move mask to img2img' would be nice. Is there a way?

Never mind, solved the issue myself.

That button already exists, with literally all the features I need.
Thanks!


eth1 won't get an IP address with libvirt provider

Bug Report
What happened:
I ran: vagrant up --provision
It resulted in this error:
msg": "eth1 does not have any IPv4 address on mon0"   (for each mon nodes).
How to reproduce it (minimal and precise):
Run "vagrant up" with libvirt as provider
Share your group_vars files, inventory
Environment:

OS (e.g. from /etc/os-release):
NAME="Pop!_OS"
VERSION="19.04"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Pop!_OS 19.04"
VERSION_ID="19.04"
HOME_URL="https://system76.com/pop"
SUPPORT_URL="http://support.system76.com"
BUG_REPORT_URL="https://github.com/pop-os/pop/issues"
PRIVACY_POLICY_URL="https://system76.com/privacy"
VERSION_CODENAME=disco
UBUNTU_CODENAME=disco

Kernel (e.g. uname -a):
Linux pop-os 5.0.0-15-generic #16pop0-Ubuntu SMP Wed May 15 14:41:25 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Ansible version (e.g. ansible-playbook --version):
ansible-playbook 2.6.16.post0 (stable-2.6 7a1ccef131) last updated 2019/05/16 15:48:09 (GMT -400)
config file = /home/admin/projects/ceph-ansible/ansible.cfg
configured module search path = [u'/home/admin/projects/ceph-ansible/library']
ansible python module location = /home/admin/projects/ansible-2.6/lib/ansible
executable location = /home/admin/projects/ansible-2.6/bin/ansible-playbook
python version = 2.7.16 (default, Apr  6 2019, 01:42:57) [GCC 8.3.0]

ceph-ansible version (e.g. git head or tag or stable branch):
stable-3.2

vagrant_variables.yml.txt
error.log

I just noticed that the VM's interfaces file has:
auto eth1
iface eth1 inet manual
It should be:
iface eth1 inet dhcp

Similar to https://github.com/ceph/ceph-ansible/issues/4036 this comes from the vagrant box used.

@deejay2 share output of ip a from one monitor please. thanks!

@guits sure!
Here it is:
vagrant@ubuntubox:~$ ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
inet <IP_ADDRESS>/8 scope host lo
valid_lft forever preferred_lft forever
inet6 ::1/128 scope host
valid_lft forever preferred_lft forever
2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
link/ether 52:54:00:aa:64:69 brd ff:ff:ff:ff:ff:ff
inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth0
valid_lft forever preferred_lft forever
inet6 fe80::5054:ff:feaa:6469/64 scope link
valid_lft forever preferred_lft forever
3: eth1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
link/ether 52:54:00:5b:54:9d brd ff:ff:ff:ff:ff:ff
inet6 fe80::5054:ff:fe5b:549d/64 scope link
valid_lft forever preferred_lft forever

I changed for vagrant_box: guits/ubuntu-bionic64
I got the very same error after:
vagrant destroy -f
vagrant up
TASK [ceph-validate : fail if eth1 does not exist on mon0] *********************
task path: /home/deejay2/projects/ceph-ansible/roles/ceph-validate/tasks/check_eth_mon.yml:2
Tuesday 04 June 2019  12:04:52 -0400 (0:00:00.508)       0:00:48.322 **********
META: noop
fatal: [mon0]: FAILED! => {
"changed": false,
"msg": "eth1 does not exist on mon0"
}
META: noop
META: noop
META: noop
META: noop
META: noop
fatal: [mon1]: FAILED! => {
"changed": false,
"msg": "eth1 does not exist on mon1"
}
fatal: [mon2]: FAILED! => {
"changed": false,
"msg": "eth1 does not exist on mon2"
}

Ahhh got it! It was ens6 for  guits/ubuntu-bionic64


Display Inactive Materials Setting

This PR adds the option to display all materials rather than only active ones. This functions as a "depot view" which allows the user to review their stock of materials. In this state, inactive items will not be marked with a requirement, nor will they be marked as completed or craftable. They retain their natural sorting order regardless of whether "Sort completed to bottom" is selected as well.

Seems very reasonable, thanks for taking this on!


Engine doesn't build on Rodete

With an in-place-upgraded Goobuntu-to-Rodete machine, I was unable to get an android engine to compile because the compilation environment is either missing some standard C++ headers, or because  the compile line is missing an appropriate include path.   -Iclang_x86/gen -I../../third_party/zlib -fno-strict-aliasing -fstack-protector --param=ssp-buffer-size=4 -m32 -momit-leaf-frame-pointer -mstack-alignment=16 -mstackrealign -fPIC -pipe -pthread -fcolor-diagnostics -Wall -Wextra -Wendif-labels -Werror -Wno-missing-field-initializers -Wno-unused-parameter -fvisibility=hidden -Wheader-hygiene -Wstring-conversion -Wthread-safety -O2 -fno-ident -fdata-sections -ffunction-sections -g0 -Werror -Wall -Wextra -Wno-unused-parameter -Wnon-virtual-dtor -Wvla -Wno-conversion-null -Woverloaded-virtual -g3 -ggdb3 -fno-rtti -fno-exceptions -O3 -fvisibility-inlines-hidden -std=c++11 -fno-rtti -fno-exceptions -c ../../third_party/dart/runtime/bin/namespace_android.cc -o clang_x86/obj/third_party/dart/runtime/bin/gen_snapshot_dart_io/namespace_android.o
In file included from ../../third_party/dart/runtime/bin/namespace_android.cc:5:
In file included from ../../third_party/dart/runtime/platform/globals.h:79:
../../third_party/dart/runtime/platform/math.h:12:10: fatal error: 'cmath' file not found
#include <cmath>
         ^~~~~~~
1 error generated.

Another header that is not found is "<limits>".
Fixing the android and regular build deps scripts to understand Rodete (as I mentioned in #13460 ), still doesn't fix the problem.  The host_debug_unopt target builds correctly.
I suspect that this is a mismatch in the libcc for Rodete.

For posterity, I couldn't build the engine just now on a clean Rodete installation; to fix it, I had to do the following:
$ sudo apt-get install pkg-config
$ sudo apt-get install libfreetype6-dev
$ sudo apt-get install glx-alternative-mesa mesa-common-dev


Components names and repo, projects structure

I'm working on providing some default templates/layout and a custom slides, to let user create their presentations faster using templates.
Once this split project done, user will be able to write presentation like the following:
<deckdeckgo-deck>
  <deckdeckgo-slide-title>

DeckDeckGo

Hello World  🚀

  </deckdeckgo-slide-title>
</deckdeckgo-deck>

This will be split in a separate project built with StencilJS and published in npm. Following this I'm not sure about how I should structure all the DeckDeckGo components.
Idea 1

deckdeckgo: the project where I write my presentation + webpack for the bundle
deckdeckgo-deck: the templates and custom slider, published in npm

aka
git clone https://github.com/fluster/deckdeckgo my-presentation
cd my-presentation
npm install
-- optionaly npm i deckdeckgo-deck, the templates
-- I write my presentation
npm run build

Idea 2

deckdeckgo: the templates and custom slider, published in npm
deckdeckgo-starter: the project where I write my presentation + webpack for the bundle

aka
git clone https://github.com/fluster/deckdeckgo-starter my-presentation
cd my-presentation
npm install
-- optionaly npm i deckdeckgo, the templates
-- I write my presentation
npm run build

Idea 3
In case I would add a Cli (https://github.com/fluster/deckdeckgo/issues/10)

deckdeckgo: the templates and custom slider, published in npm
deckdeckgo-starter: the project where I write my presentation + webpack for the bundle
deckdeckgo-cli: the cli which allows me to create deckdeckgo-starters with the meta I entered during the process of creation

npm install -g deckdeckgo-cli
deckdeckgo go my-presentation
cd my-presentation
-- I write my presentation
npm run build

The current project has been moved to its new repo deckdeckgo-starter https://github.com/fluster/deckdeckgo-starter
I've created a brand repo for the deckdeckgo core https://github.com/fluster/deckdeckgo


fix: #1195: Hide the redundant scroll caused by long addresses

Closes #1195

Can you add before and after screenshots to the PR?
This would be very helpful for reviewers when it comes to UI changes

@VanishMax just flagging if there are any PRs you want the entire web team to review, you can select web-reviewers


Problem with ngFor in AngularJS

Hi,
Thanks for the plugin. During my testing images were missed when I had tried to load multiple images in one page using ngFor. Will you please check?
<ScrollView orientation="vertical" >
        <StackLayout>
            <StackLayout class="cardStack" *ngFor="let post of articles" >
                <NSImage *ngIf="post.img.image_fulltext" style="height: 120; width: 120"  src="{{baseURL}}/{{post.img.image_fulltext}}"></NSImage>
            </StackLayout>
        </StackLayout>
    </ScrollView>

Thanks in advance :)

Hey @jibon57 I haven't tried that but I've noticed an issue if height and width is not explicitly defined. I would try and put height and width as their own tags, not wrapped in style and see if that fixes it.

Hey @bnussey I'm experiencing the same issue. Declaring the width & height as you've proposed works. However, in my implementation I would prefer not having to set width & height. I think images should be displayed even if width & height aren't set, do you agree?

Hey @mfrro just try setting it as 100% for width and height.. I am also looking into a solution myself but haven't found anything yet.

Setting a percentage value only works if the parent element has a set width & height. Maybe this will bring you closer to a solution.
This works:
<StackLayout width="200" height="200">
   <NSImage stretch="aspectFit" [src]="pictureUrl" width="100%" height="100%"></NSImage>
</StackLayout>

This doesn't work:
<StackLayout>
   <NSImage stretch="aspectFit" [src]="pictureUrl" width="100%" height="100%"></NSImage>
</StackLayout>

Thanks @mfrro .. always open to a PR if you wanna work on this as well


Error: more than one library with package name 'com.burnweb.rnpermissions'

New to react-native, trying to get example working. I am getting this error while trying to run example on android.

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugResources'.
Error: more than one library with package name 'com.burnweb.rnpermissions'

Here is my package.json
{
  "name": "ReactBLEScanner",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "react": "16.0.0-alpha.6",
    "react-addons-pure-render-mixin": "15.6.0",
    "react-native": "0.44.1",
    "redux": "3.6.0",
    "react-redux": "5.0.5",
    "redux-logger": "3.0.6",
    "react-native-router-flux": "3.39.2",
    "immutable": "3.8.1",
    "redux-immutable": "4.0.0",
    "react-immutable-proptypes": "2.1.0",
    "react-native-button": "1.6.0",
    "react-native-android-permissions": "1.0.0",
    "buffer": "4.7.1",
    "react-native-ble-plx": "0.6.1"
  }
}

Are there any other examples on github or similar which would give me some
working examples to look at?
Thanks for any help if you can provide!

Example app is not up to date unfortunately and I don't have time to fix this issue. I recommend to build new app and start by reading introduction in documentation.

Okay. No problem. Thanks for the reply.


Binary Indexed Trees [C++]

This issue is about Binary Indexed Trees.

[x] Issue Title is in form Algo/DS Name [Language].
[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[x] This Algo/DS is worth adding to this repository.
[x] Assign this issue to me (I want to work on this).
[x] I ensure that I am not already assigned to 2 or more issues.
[x] I will send my PR only after I'm assigned to this issue by any maintainer.
[x] I understand that I will be unassigned from this issue if I fail to send a PR for this within a week.

can i work on it?

@sirjan13: Since you are already working on #481, is it okay if @riktimmondal takes this?


podman play kube does not accept pod descriptions that have neither command nor args

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

Podman should accept a pod description like this:
apiVersion: v1
kind: Pod
metadata:
  name: metrics
spec:
  containers:
  - name: mosquitto
    image: eclipse-mosquitto
    ports:
      - hostPort: 1883
        containerPort: 1883
        protocol: tcp

But this will fail with:
Error: container create failed: args must not be empty

That means there is no way to declare a container that uses the
default entrypoint and command declared in an image.
Steps to reproduce the issue:

Use the pod description referenced above.

Describe the results you received:
(see description)
Describe the results you expected:
I expected the container to start using the ENTRYPOINT defined in
the image.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
podman version 1.2.0

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.1
  podman version: 1.2.0
host:
  BuildahVersion: 1.7.2
  Conmon:
    package: Unknown
    path: /usr/bin/conmon
    version: 'conmon version , commit: 8fba206232c249a8fc4e2fac1469fb2fddbf5cf7'
  Distribution:
    distribution: arch
    version: unknown
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    package: Unknown
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc7
      commit: 69ae5da6afdcaaf38285a10b36f362e41cb298d6
      spec: 1.0.1-dev
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 2
  hostname: data
  kernel: 5.0.5-arch1-1-ARCH
  os: linux
  rootless: false
  uptime: 23h 20m 45.74s (Approximately 0.96 days)
insecure registries:
  registries: []
registries:
  registries:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 2
  GraphDriverName: overlay
  GraphOptions:
  - overlay.mountopt=nodev
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "true"
  ImageStore:
    number: 14
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

I thought we fixed this one recently (@haircommander did some work on configuring containers from play kube from images), but you're running 1.2.0, and I was pretty sure that work landed there, so maybe I'm mistaken.

@mheon I think you're mixing my volume work and the work I did with the pod pause image :) I don't think we have encountered this with play kube yet!

PR here: #2856

PR is merged, closing :)


Issue/#30

@rultor merge

@rultor merge

@kerelape OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@kerelape Done! FYI, the full log is here (took me 2min)


Use absolute positioning of dfn panels and highlight ref links

This PR is for some UI fixes and affordances regarding dfn panels and references links.

dfn panel is always raised to the root level, changed to absolute positioning next to the dfn when clicking on a dfn.  This avoids any truncation when the dfn is contained in a scrolling container.  It also scrolls with the page.
Clicking on links in dfn panel don't scroll the page if the destination of the link is already in the view.
Destinations are always highlighted for 3 seconds.
When the dfn panel is "activated" and moved to the bottom left corner, only the horizontal animation works for some reason. But this is perhaps enough.
I experimented with smooth scrolling, but it is still rather janky and not very helpful.

Tabbing now works as follows:

When a panel is shown, the tabIndex the dfn is set to 100 (no difference with 1)  and all anchor and button tags in the panel are set to increments above that.
Tab and shift-tab from the dfn to the panel (and within the panel) and back work.
Before and after the dfn and its panel, everything has undefined tabIndex, so tabbing skips to the window level or "Jump to Table of Contents" .

I suspect the only way to avoid this is to coordinate all tabIndexes globally.
Leaving the dfn tabIndex at 0 and incrementing the panel elements to 1 and above does not work.
1 is the same as 100.

Pinning the panel preserves all this.
Closing a panel sets the dfn tabIndex back to undefined.

I'm surprised to learn that many pages try to hide the focus ring around links and buttons.  This is a useful user affordance that is important to preserve in some cases.
In pinned dfn panels, to make click and keyboard behavior consistent, independent of whether the link destination needs to be scrolled to, the focus should remain on the link in the panel. So the focus should also remain visible either way.

Excellent! I've just done a few edits:

I selected some better-looking colors (I did this before you committed your latest color patch; if you feel you liked the colors you chose better, feel free to change them)
I moved the positioning code to a separate function so I could invoke it in the resize event rather than just pinning it.
Fixed the positioning when the panel would overflow; it now just gets nudged to the left if it would overflow to the right (and avoids nudging too far into the unscrollable zone).

And then just a touch of refactoring while I was doing the above; in particular, rather than constantly passing around the panel and the dfn, I just store the dfn on the panel when it's generated so I can grab it when needed. (I needed this since the resize event wouldn't have both elements in hand.)

When the dfn panel is "activated" and moved to the bottom left corner, only the horizontal animation works for some reason. But this is perhaps enough.

Oh yeah, this happens because the abspos code sets the element's 'top' (leaving 'bottom' as auto) while the fixpos code sets the 'bottom' (leaving 'top' as auto). So neither 'top' nor 'bottom' is transitionable. We could fix this by swapping to setting 'bottom' in the activation function before we switch to fixed, but I think the animation as it currently exists is enough to draw the eye.

Yup, fixed in trunk now.


Only include the bindings.node file in prebuilds

Connects-To: #20
Change-Type: patch
---- Autogenerated Waffleboard Connection: Connects to #20

Why is this necessary?

Why is this necessary?

@jhermsmeier it's described in the issue: #20


Any options for inline attachments MailgunAdopter

In past, we were using mailgun wrapper https://github.com/evercam/mailgun/commit/8c73ada08137ce875b487949167d60fecb1715dc and added inline attachment options by ourselves.
Now we are planning to switch to Bamboo, and I have done formal emails method conversion from mailgun to bamboo mailgun adopter but it's not sending inline attachments, Is there any option for that in bamboo?
For emails like

Instead of showing image as mentioned in template.
<img src="cid:snapshot.jpg" alt='Last Snapshot' style="width: 100%; display:block; margin:0 auto;">

its just showing it alt text. and in the bottom it attached the snapshot. I am sorry if that is not the right place to ask this. I have tried in Elixir Slack channel but no one is interested to answer.

Hi @ijunaid8989 thanks so much for opening the issue and for the patience! I'm just starting to maintain Bamboo and trying to look through the open issues.
This seems related to https://github.com/thoughtbot/bamboo/issues/502. It seems like it might be good to allow people to set the content_disposition and then map that to the adapters as needed.
If this is still something of interest, let me know. I'll try to get to it, or if you're up for it, I'd be happy to review a PR.


Support set layout version for TimeRangePicker

Please support to set layout version 2 for TimeRangePicker like TimePickerDialog. Thanks

Hi, please support for this feature. Thank you.


With tofu

I've submitted a pull request that adds trace information to the TOFU-MAaPO pipeline.
I wanted to discuss the possibility of training models without specific metrics but with common ones like tpeak_vmem, read_bytes, write_bytes, vol_ctxt, and inv_ctx. If this approach is feasible, we may not need to make significant changes to the initial scripts of nextflow pipelines. Instead, we could run the nextflow pipeline directly from the GitHub repository with a specific configuration indicating that we require this trace information.

@bentsherman @eikematthias


feat(web): add modal info window

Add a modal window with info
Show the window on 'info' click
Add proxy status indicator & button

Deployed the updated version at https://sergey-melnychuk.github.io/beerus/


Some blob comparison stuff

Compare blobs in vm, might work incorrectly when subcomparison returns Value::Nil.
Add blob comparison test
Remove comparison with nil from lua preamble, might be wrong of me to do this in this commit since it was a comment in #615.

Also, maybe rename comparison to equality. I associate comparisons with < and >.


Added support for new fused Winograd algorithms in cudnn 5.1

Can one of the admins verify this patch?

LGTM.
@tensorflow-jenkins, test this please.

@benbarsdell, conv_ops_test is failing on jenkins with this CL. Could you confirm that it runs well for you locally with Cuda 7.5 + Cudnn 5? Thanks.
The log is here:
https://ci.tensorflow.org/job/tensorflow-pull-requests-gpu/2119/consoleFull
If you cannot see it, this is the relevant portion:
FAIL: testInceptionBackFilter_4 (main.Conv2DTest)
Traceback (most recent call last):
File "/var/lib/jenkins/workspace/tensorflow-pull-requests-gpu/bazel-ci_build-cache/.cache/bazel/_bazel_jenkins/eab0d61a99b6696edb3d2aff87b585e8/execroot/workspace/bazel-out/local_linux-opt/bin/tensorflow/python/kernel_tests/conv_ops_test.runfiles/org_tensorflow/tensorflow/python/kernel_tests/conv_ops_test.py", line 1272, in Test
strides, padding)
File "/var/lib/jenkins/workspace/tensorflow-pull-requests-gpu/bazel-ci_build-cache/.cache/bazel/_bazel_jenkins/eab0d61a99b6696edb3d2aff87b585e8/execroot/workspace/bazel-out/local_linux-opt/bin/tensorflow/python/kernel_tests/conv_ops_test.runfiles/org_tensorflow/tensorflow/python/kernel_tests/conv_ops_test.py", line 569, in _CompareBackFilter
self.assertAllClose(values[0], values[i], rtol=1e-4, atol=1e-4)
File "/var/lib/jenkins/workspace/tensorflow-pull-requests-gpu/bazel-ci_build-cache/.cache/bazel/_bazel_jenkins/eab0d61a99b6696edb3d2aff87b585e8/execroot/workspace/bazel-out/local_linux-opt/bin/tensorflow/python/kernel_tests/conv_ops_test.runfiles/org_tensorflow/tensorflow/python/framework/test_util.py", line 449, in assertAllClose
np.testing.assert_allclose(a, b, rtol=rtol, atol=atol)
File "/usr/local/lib/python2.7/dist-packages/numpy/testing/utils.py", line 1391, in assert_allclose
verbose=verbose, header=header)
File "/usr/local/lib/python2.7/dist-packages/numpy/testing/utils.py", line 733, in assert_array_compare
raise AssertionError(msg)
AssertionError:
Not equal to tolerance rtol=0.0001, atol=0.0001
(mismatch 5.26315789474%)
x: array([[[[ 46.046501,  45.859997,  56.073101, ...,  45.907337,  51.214821,
44.84082 ],
[ 47.817616,  42.558422,  52.609974, ...,  45.89362 ,  46.778015,...
y: array([[[[ 33.582195,  36.529572,  56.073128, ...,  45.90733 ,  51.214813,
44.840816],
[ 35.047573,  32.791656,  52.609985, ...,  45.893631,  46.778004,...

Sorry for the delay, I'm in the process of looking into this failure.

SG, if you're interested in merging this at some point, please make sure you follow the instructions to sign the CLA.

@benbarsdell: if you have signed the CLA under your nvidia address, maybe git amend your commits to that?  It's currently set to your gmail account.

(clearing this out of our open PRs -- ping me or this thread when you want me to reopen after the CLA issues are resolved)


refact by Java 8 feature

Coverage decreased (-0.06%) to 76.372% when pulling e148bd1c3960f50d35e1167fc1bd73fb8c2b047a on diguage:java8 into 6a156830c73f139ff9928e737329e8f2793bfae2 on brettwooldridge:dev.

-1 may introduce performance regression and I see no need to introduce a stream here.

@diguage @johnou We do use streams in a couple of the ConcurrentBag methods, but this particular one is called in some of the hotter code paths, so I'd rather not touch it.
@diguage If you can run the benchmarks, and collect a before/after comparison showing no impact, I will consider re-opening this request.


not able to jump while in crouch position

I am not able to jump while in crouch position, only after letting go of crouch can the player jump

This is expected behavior. In some source games you cannot jump while crouched, but I may be mistaken.
If you do want to jump for your project, go to Run.gd and change the line
if Input.is_action_pressed("jump") && (not stats.crouched) && stats.shouldJump:
to
if Input.is_action_pressed("jump") && stats.shouldJump:


Bug 721729 (pageupdate)

As previously noted, a single commit with 3 repeated merge commits that can be ignored.

r+'ed in bugzilla.


Add Contributor Covenant

Closes #1142
CC @evanp

Please don't use the words closes, it has negative connotations and triggers me. Your not being inclusive.

Merging since @evanp hasn't said anything and there's no reason to delay this.


Usage and format questions

It seems we're migrating all repos to use this project. I have a few questions which I think would be nice to have answers to while we migrate (ideally I would have liked to have answers to them before we migrate):
Taken from https://github.com/huggingface/huggingface_hub/issues/759
By reading the "how to write documentation", it's clear how to write docstrings, but it's not clear how to write user guides and examples:

I'm thinking something along the lines of having something resembling sphinx-gallery examples, and then referencing those examples from user guides.
How does one reference other objects, classes, and methods from the current library and external libraries that are rendered with a hyperlink to the corresponding API doc?
how to we reference other sections in the user guide from the API doc and the user guide itself?
what is the workflow to add directives such as versionadded, versionchanged, etc?
I see this in the docs:

It helps to keep the old links working when renaming section header and/or moving sections from one document to another.

In rst you reference a tag, so renaming headers doesn't actually do anything to break existing links. How does mdx handle this?
I'm not familiar with mdx, so I'm not sure how to approach these issues.
cc @LysandreJik

I'm thinking something along the lines of having something resembling sphinx-gallery examples, and then referencing those examples from user guides.

Do you mean you'd like to have a tool that from a .py file with docstring, generates a HTML render of that file and puts it in a given folder? Right now the tool takes everything and

How does one reference other objects, classes, and methods from the current library and external libraries that are rendered with a hyperlink to the corresponding API doc?

If I'm not misunderstanding, this is covered in the guide regarding how to write the documentation:
"If you want to create a link to some internal class or function, you need to provide its path. For instance: [file_utils.ModelOutput]. This will be converted into a link with file_utils.ModelOutput in the description. To get rid of the path and only keep the name of the object you are linking to in the description, add a ~: [~file_utils.ModelOutput] will generate a link with ModelOutput in the description."
The same holds true for cross-library references: [`~datasets.Dataset`] will do a link to the Dataset object documented in the datasets library.

how to we reference other sections in the user guide from the API doc and the user guide itself?

As far as I know, other sections/parts of the documentation can be referenced by their local paths: [text](/other_page) and [test](/other_page#section)

what is the workflow to add directives such as versionadded, versionchanged, etc?

For the beginning, I think we would put them manually as a <Note> so that it's displayed prominently on the methods. @mishig25, what do you think? See the following directive in Sphinx, it would be very useful for huggingface_hub as downstream libraries would be aware of what became available when.

In rst you reference a tag, so renaming headers doesn't actually do anything to break existing links. How does mdx handle this?

I think @mishig25 has actually been involved heavily in the links/headers, so may be able to provide a better answer than I can.

Do you mean you'd like to have a tool that from a .py file with docstring, generates a HTML render of that file and puts it in a given folder? Right now the tool takes everything

I mean something like this file which is then rendered like this page which also generates a python notebook from the .py file and you can also run the notebook on binder.

We already have the ability to auto-generate notebooks from all the doc files, it sounds like what you want to do is similar, we just do it from a .mdx instead of a .py.
See for instance the quicktour of Transformers with the open in colab/AWS studio at the top.

A few other comments:

The same holds true for cross-library references: [~datasets.Dataset] will do a link to the Dataset object documented in the datasets library.

@LysandreJik note that this feature has not been developed yet, but it's on the roadmap :-)
We can definitely add support for the versionadded, versionchanged directives. We didn't use them in Transformers and Datasets, which is why they're not supported yet.

In rst you reference a tag, so renaming headers doesn't actually do anything to break existing links. How does mdx handle this?

You can define an html anchor and reference it. See for instance the Training tutorial in transformers. GitHub does not let me link to specific lines but you have a trainer tag at line 64, and you can then link to it via training#trainer from any page in the doc.

Awesome, this clarifies things quite a bit for me.

We already have the ability to auto-generate notebooks from all the doc files, it sounds like what you want to do is similar, we just do it from a .mdx instead of a .py.
See for instance the quicktour of Transformers with the open in colab/AWS studio at the top.

This is pretty cool, and from the user's perspective, exactly what I was hoping to see. However, what I think is missing is from the development side: working on a code heavy documentation page where there is a lot of code, it's much easier to work on a .py file where the contributor/developer can test/run/debug the file and the notebook cells in the same file. For instance, consider this part from the quick tour file:
For a larger dataset where the inputs are big (like in speech or vision), you will
want to pass along a generator instead of a list that loads all the inputs in memory.
See the [pipeline documentation](./main_classes/pipelines) for more information.

### Use another model and tokenizer in the pipeline

The [`pipeline`] can accommodate any model from the [Model Hub](https://huggingface.co/models),
 making it easy to adapt the [`pipeline`] for other use-cases. For example, if you'd like
a model capable of handling French text, use the tags on the Model Hub to filter for
an appropriate model. The top filtered result returns a multilingual
[BERT model](https://huggingface.co/nlptown/bert-base-multilingual-uncased-sentiment)
fine-tuned for sentiment analysis. Great, let's use this model!

```py
>>> model_name = "nlptown/bert-base-multilingual-uncased-sentiment"
```

Use the [`AutoModelForSequenceClassification`] and ['AutoTokenizer'] to load
the pretrained model and it's associated tokenizer (more on an `AutoClass` below):

```py
>>> from transformers import AutoTokenizer, AutoModelForSequenceClassification

>>> model = AutoModelForSequenceClassification.from_pretrained(model_name)
>>> tokenizer = AutoTokenizer.from_pretrained(model_name)
>>> # ===PT-TF-SPLIT===
>>> from transformers import AutoTokenizer, TFAutoModelForSequenceClassification

>>> model = TFAutoModelForSequenceClassification.from_pretrained(model_name)
>>> tokenizer = AutoTokenizer.from_pretrained(model_name)
```

The same doc could potentially look like:
# %%
# For a larger dataset where the inputs are big (like in speech or vision), you
# will want to pass along a generator instead of a list that loads all the
# inputs in memory. See the [pipeline documentation](./main_classes/pipelines)
# for more information.

# %%
# Use another model and tokenizer in the pipeline
# -----------------------------------------------
#
# The [`pipeline`] can accommodate any model from the [Model
# Hub](https://huggingface.co/models), making it easy to adapt the [`pipeline`]
# for other use-cases. For example, if you'd like a model capable of handling
# French text, use the tags on the Model Hub to filter for an appropriate
# model. The top filtered result returns a multilingual [BERT
# model](https://huggingface.co/nlptown/bert-base-multilingual-uncased-sentiment)
# fine-tuned for sentiment analysis. Great, let's use this model!

model_name = "nlptown/bert-base-multilingual-uncased-sentiment"

# %%
# Use the [`AutoModelForSequenceClassification`] and ['AutoTokenizer'] to load
# the pretrained model and it's associated tokenizer (more on an `AutoClass`
# below):

from transformers import AutoTokenizer, AutoModelForSequenceClassification

model = AutoModelForSequenceClassification.from_pretrained(model_name)
tokenizer = AutoTokenizer.from_pretrained(model_name)
# ===PT-TF-SPLIT===
from transformers import AutoTokenizer, TFAutoModelForSequenceClassification

model = TFAutoModelForSequenceClassification.from_pretrained(model_name)
tokenizer = AutoTokenizer.from_pretrained(model_name)

I don't think this is an urgent request/feature. But sphinx-gallery which handles these cases has been quite helpful in the projects I've worked in, and it'd be nice to have something similar for the doc-build infra we're building.

Good point! We can definitely add support for this and it shouldn't be difficult to parse such a py file to generate a MDX and the corresponding notebooks.
The main difference with your suggestion is that the text would be in full Markdown, so not
# Use another model and tokenizer in the pipeline
# -----------------------------------------------

but
# ## Use another model and tokenizer in the pipeline


Docker - Fatal error when starting container

When I start the container, it doesn't show any error related about the parsing itself, but in the end I have the following :
[SETTING UP STORAGE goaccess] {0} @ {0/s} GoAccess - version 1.4.5 - Feb  3 2021 00:47:46 Config file: /goaccess/goaccess.conf Fatal error has occurred Error occurred at: src/parser.c - set_initial_persisted_data - 3255 Unable to fread the specified log file 'goaccess'
My access.log file is in the following format :
my.host.net <IP_ADDRESS> - - [08/Feb/2021:14:36:27 +0000] "GET / HTTP/2.0" 302 0 "-" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:85.0) Gecko/20100101 Firefox/85.0"
Here's my docker command:
docker run -d --name goaccess --restart=always \   -p 7890:7890 \   -v /home/rinku/docker/goaccess_docker:/goaccess \   -v /home/rinku/docker/nginx_proxy_conf/log/access.log:/goaccess/access.log \   allinurl/goaccess \   goaccess --no-global-config --config-file=/goaccess/goaccess.conf
And my goaccess.conf file :
log-format %v %h %^[%d:%t %^] "%r" %s %b "%R" "%u" %T %^ date-format %d/%b/%Y time-format %T log-file /goaccess/access.log output /goaccess/index.html real-time-html true

Sounds like it may not be a valid path. Can I ask you to try generating the report as:
cat access.log | sudo docker run -p 7890:7890 --rm -i -e LANG=$LANG allinurl/goaccess -a -o html --log-format='%v %h %^[%d:%t %^] "%r" %s %b "%R" "%u" %T %^' --date-format=%d/%b/%Y --time-format=%T --real-time-html - > report.html

It works perfectly that way. Thank you

Glad to hear that. Closing this. Feel free to reopen it if needed.

Hi there, I've just started facing the same error today, using a docker compose setup (in a swarm), which I've been successfully using in production for at least one year.
The only thing I tried doing today was update to the latest Goaccess image (redeploying my stack), and since then it's gone belly up.
Here's my docker compose file (minus some Traefik config):
version: "3.7"

services:
  #  An nginx server is required, to serve the HTML report page generated by Goaccess
  nginx:
    image: nginx:1.18.0-alpine
    networks:
      - proxy
    volumes:
      # Pick up the HTML report generated by Goaccess
      - "${DIRECTORY}/html:/usr/share/nginx/html"
      - "/var/run/docker.sock:/tmp/docker.sock:ro"
  goaccess:
    # digest: 6dd53b48c8962ffd430b2b2a261b805bab0307f2f8b4a727db11cbe2881bb1f0
    image: allinurl/goaccess:latest
    networks:
      - proxy
    environment:
      - TZ=Europe/Athens
    volumes:
      - "${DIRECTORY}:/goaccess"
      # This is where the HTML report that gets picked up by nginx goes
      - "${DIRECTORY}/html/index.html:/goaccess/index.html"
      # Traefik access log to be processed
      - "${ACCESS_LOG}:/goaccess/access.log"
    command: ["goaccess", "--no-global-config", "--config-file=/goaccess/goaccess.conf",
              "--ws-url=wss://ws.${HOST_NAME}:443"]

networks:
  proxy:
    external: true

And here are the pertinent bits of my Goaccess conf file (everything else is as per the default config "template"):
log-file /goaccess/access.log
output /goaccess/index.html
real-time-html true
daemonize false
#log-format COMMON
log-format %h %^[%d:%t %^] "%r" %s %b "%R" "%u" %^ "%v" %^ %T
time-format %T
date-format %d/%b/%Y

Re-iterating the error, for the sake of completeness:
[SETTING UP STORAGE goaccess] {0} @ {0/s}
GoAccess - version 1.4.5 - Feb 11 2021 04:10:18
Config file: /goaccess/goaccess.conf
Fatal error has occurred
Error occurred at: src/parser.c - set_initial_persisted_data - 3255
Unable to fread the specified log file 'goaccess'

Any help would be appreciated, cheers!

@el95149 please take a look at my previous post on how to run it. The docker file has changed in the last few releases.

After a bit of tinkering, and based on your updated command above, this eventually did the trick:
.....
    volumes:
      - "${DIRECTORY}:/goaccess"
      # This is where the HTML report that gets picked up by nginx goes
      - "${DIRECTORY}/html/index.html:/index.html"
      # Traefik access log to be processed
      - "${ACCESS_LOG}:/access.log"
    command: ["access.log", "-a", "-o html", "--no-global-config", "--config-file=/goaccess/goaccess.conf",
              "--ws-url=wss://ws.${HOST_NAME}:443", "-"]
....

combined with this:
log-file /access.log
output /index.html
....

It's working like a charm now, thanks a bunch!

@el95149 Glad to hear it works now. Feel free to report any other issues you may find.

I am running goaccess with ansible
this how i mount volumes
volumes:
- "/srv/docker/nginx/www:/srv/html"
- "{{ goaccess.log_file }}:{{ goaccess.log_file }}:ro"
- "{{ fact_goaccess_logs_dir }}:/srv/logs"
command: "{{ goaccess.log_file }} --log-format=COMBINED --output=/srv/html/report.html --real-time-html"
```
goacccess.log_file= /srv/docker/gitlab/logs/nginx/gitlab_access.log
```
but i got error Unable to fread the specified log file

@holiday25 It seems like there might be an issue with how you're specifying the log file path in your Ansible config. Ensure that this path actually exists on your system and that the file gitlab_access.log is present within that directory.  Also, ensure that the container has the necessary permissions to read the log file.


fix issues with examples (#135)

(fixes issue #135)
Adding instructions to the readme to make sure virtual-dom is installed
Fixing the path of the index.js file in the into example
Updating purescript-affjax dependency in the ajax example, to make it work in IE

:+1: Thanks!


sasa

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

Closing this issue as incomplete.


Is there a way to update the format of VisibleFlags?

Is there a way to update the format of VisibleFlags?
By default, help shows --out file, -o file Usage
I want to change it to -o, --out file Usage.

@fcying Currently there is no way


keyboard handlers patch

Fixes #1405.
Keyboard handlers patch:

event firing added
possibility of preventing default keyboard behaviour
possibility of preventing vertical and/or horizontal pan with keyboard

Example of usage:
viewer.addHandler('canvas-key', function (e) {
   e.preventDefault = true; // disable default keyboard controls
   e.preventVerticalPan = true; // disable vertical panning with arrows and W or S keys
   e.preventHorizontalPan = true; // disable horizontal panning with arrows and A or D keys
});

Built and tested with grunt:
>> 223 tests completed with 0 failed, 0 skipped, and 0 todo.
>> 2051 assertions (in 45948ms), passed: 2051, failed: 0

Now that I think of it, I think we only need one copy of the doc comment, even though it's raised in two places. I'll delete one after I merge this.
Thank you for making this patch; it came together nicely!


WIP: coupling of AC irrigation to LIS irrigation module and bug fixes

This PR mainly introduces the improvements in the coupling of AquaCrop irrigation to LIS.
The main scientific improvement is the following:

Implementation of dynamic irrigation season, similar to the "Irrigation scheduling based on dynamic vegetation" option in Noah-MPv4.0.1.
In the lis.config file, the same options as for Noah-MPv4.0.1 are expected. Following the LISv7.4 user manual, the documentation has been updated to keep track of the options in the lis.config file. For simplicity, the same input as for Noah-MP is expected, meaning that the Irrigated Fraction map and crop type map should be added to the LDT file (even though this is currently not used for irrigation in AquaCrop). LDT Noah-MPv4.0.1 input netcdf files can simply be used. A "dummy" irrigation file needs to be selected in the lis.config file. This file has the following structure:

Default irrigation file, variables are overwritten in LIS
   7.0   : AquaCrop Version (August 2022)
   1     : Sprinkler irrigation
 100     : Percentage of soil surface wetted by irrigation
   2     : Generate irrigation schedule
   3     : Time criterion = allowable fraction of RAW
   1     : Depth criterion = back to FC

  From day    Depleted RAW (%)   Back to FC (+/- mm)       ECw (dS/m)
=========================================================================
       1           9999                   0                   0.0

The file is read by AquaCrop, but variables are overwritten in LIS. Note that this setup can readily be extended to other types of irrigation (drip, basin).
The reading of the irrigation file generated problems for large simulations in LIS due to the opening of the irrigation file for each pixel, each year. This caused the run to crash raising the following error "too many open files in system". The issue was solved by closing the dummy irrigation file after the initialization (in ac70_main). This is probably fine for my applications (as I use a dummy irrigation file with one line), but using real irrigation files with several lines will likely not be possible for now. We should come up with a more elegant solution in the future.
The main technical improvement is to couple irrigation from AquaCrop to the irrigation module in LIS [WIP].

Regarding the input, everything works fine.
The irrigation output is now stored in an IRRIGATION folder, consistent with the other land surface models. However**, there is a shift in the output**, e.g. the IRRIGATION file LIS_HIST_201005310000.d01.nc corresponds to the SURFACEMODEL LIS_HIST_201005300000.d01.nc. This is due to the suite of procedures in lis_run_retrospective (the surface model is run after the irrigation module is called). @mbechtold would you have any idea on how to tackle this (writing the output for the previous day)?

Other edits:

Introduce a flexible simulation period to allow simulations in the Southern Hemisphere (or simulation of winter crops). The start date defined in the lis.config file is now taken as the start data of the AquaCrop simulation period. The simulation period lasts one year, as before.
Addition of output variables for AquaCrop
Fixed bug originating from Sara's dynamic irrigation season implementation that is only intended for Noah-MP3.6 and raised errors if the option was not specified in the lis.config file.

Note: I found a bug in the implementation of the flexible simulation period (start is shifted), I'll fix it!


PLAT-43853: Number input field allows any character in Safari

Issue Resolved / Feature Added
Enyo Input : Input allows any character in Safari
Resolution
Safari not handled the same. regExp validation is done in keypress event
Links
PLAT-43853
Comments
Enyo-DCO-1.1-Signed-off-by: Anish<EMAIL_ADDRESS>

LGTM!

Why don't you just use the 'pattern' attribute?

Can we discriminate when the pattern is evaluated, though?

So we would want to apply the numeric pattern only for number input type, and if Safari, and only if no pattern is specified by the developer.

Numeric pattern may not be the ideal solution for this issue. pattern mainly for form validations and pattern will allow characters to input and validate on submit. this scenario not suited in this case. so added extra check for safari is done.
@aarontam , if you think this is not be fixed then i will close this issues

closing out as the jira associated with this is closed.


Error: certificate signature failure

we are getting above error when trying to connect to an openldap server. our complete code is as follows and modeled after 1:
var ldap = require("ldapjs");
var fs = require("fs");
const assert = require("assert");
var client = ldap.createClient({
        url: "ldap://jnj-ldap-server:389/dc=jnj,dc=com",
        tlsOptions: opts
});
var opts = {
  ca: [fs.readFileSync("jnj-ca-chain.pem")],
  // Necessary only if the server requires client certificate authentication.
  key: fs.readFileSync('jnj-web-app-tls.key'),
  cert: fs.readFileSync('jnj-web-app-tls.pem'),
  host: "jnj-ldap-server",
  ecdhCurve: 'auto', // see https://github.com/nodejs/node/issues/16196 and https://github.com/joyent/node-ldapjs/issues/464
};
var controls = client.controls;
var dn = "cn=admin,dc=jnj,dc=com";
var password = "superman";
client.starttls(opts, controls, function(err, res) {
  assert.ifError(err);
  client.bind(dn, password, function (err) {
        assert.ifError(err);
    });
    log.info("StartTTLS connection established.")
  // Client communication now TLS protected
});

output:
root@6fc2ae248018:/usr/src/app# node app.js
assert.js:816
assert.ifError = function ifError(err) { if (err) throw err; };
                                                  ^

Error: certificate signature failure
    at TLSSocket.<anonymous> (_tls_wrap.js:1116:38)
    at emitNone (events.js:106:13)
    at TLSSocket.emit (events.js:208:7)
    at TLSSocket._finishInit (_tls_wrap.js:643:8)
    at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:473:38)

on server logs we see this:
5cb79962 conn=1008 fd=12 ACCEPT from IP=x.x.x.x:59212 (IP=<IP_ADDRESS>:389)
5cb79962 conn=1008 op=0 EXT oid=<IP_ADDRESS>.4.1.1466.20037
5cb79962 conn=1008 op=0 STARTTLS
5cb79962 conn=1008 op=0 RESULT oid= err=0 text=
TLS: can't accept: (unknown error code).
5cb79962 conn=1008 fd=12 closed (TLS negotiation failure)

our server is started as explained in https://github.com/osixia/docker-openldap#use-your-own-certificate
How can we fix this? we tried changing url to use ldaps and 636 but that gives another error
Further diagnostics:
root@6fc2ae248018:/usr/src/app# openssl s_client -state -nbio -connect jnj-ldap-server:389 -CAfile jnj-ca-chain.pem
CONNECTED(00000003)
Turned on non blocking io
SSL_connect:before SSL initialization
SSL_connect:SSLv3/TLS write client hello
SSL_connect:error in SSLv3/TLS write client hello
write R BLOCK
SSL_connect:error in SSLv3/TLS write client hello
read:errno=0
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 176 bytes
Verification: OK
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID:
    Session-ID-ctx:
    Master-Key:
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)
    Extended master secret: no
---

siddjain@jnj:~/temp$ openssl verify -CAfile jnj-ca-chain.pem jnj-ldap-server-tls.pem
jnj-ldap-server-tls.pem: OK

if clientauth is disabled on the server by setting LDAP_TLS_VERIFY_CLIENT=never, we still get same output on client:
root@6fc2ae248018:/usr/src/app# openssl s_client -state -nbio -connect jnj-ldap-server:389 -CAfile jnj-ca-chain.pem
CONNECTED(00000003)
Turned on non blocking io
SSL_connect:before SSL initialization
SSL_connect:SSLv3/TLS write client hello
SSL_connect:error in SSLv3/TLS write client hello
write R BLOCK
SSL_connect:error in SSLv3/TLS write client hello
read:errno=0
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 176 bytes
Verification: OK
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID:
    Session-ID-ctx:
    Master-Key:
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)
    Extended master secret: no
---

root@6fc2ae248018:/usr/src/app# node app.js
assert.js:816
assert.ifError = function ifError(err) { if (err) throw err; };
                                                  ^
Error: certificate signature failure
    at TLSSocket.<anonymous> (_tls_wrap.js:1116:38)
    at emitNone (events.js:106:13)
    at TLSSocket.emit (events.js:208:7)
    at TLSSocket._finishInit (_tls_wrap.js:643:8)
    at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:473:38)
root@6fc2ae248018:/usr/src/app#

but on server we see this:
5cb7bbba conn=1001 fd=12 ACCEPT from IP=x.x.x.x:40946 (IP=<IP_ADDRESS>:389)
5cb7bbba conn=1001 op=0 EXT oid=<IP_ADDRESS>.4.1.1466.20037
5cb7bbba conn=1001 op=0 STARTTLS
5cb7bbba conn=1001 op=0 RESULT oid= err=0 text=
5cb7bbba conn=1001 fd=12 TLS established tls_ssf=128 ssf=128
5cb7bbba conn=1001 fd=12 closed (connection lost)

It looks like you are using a TLS v1.2 certificate on your server (as you should). Unfortunately, this is not going to work until ldapjs supports Node >= 10. Please track issue #524. Once a new release is out that supports Node 10, please try again and open a new issue if it is still a problem.

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


how to get the dataset?

hi,
how did you collect the dataset in CARLA,when i run the ./client_example.py --autopilot --images-to-disk i can just get the rgb , depth image and print some measurements , but i can not get all the 28 measurements in targets,how did u get it?

Hi did you got any idea on this...As iam also facing the same problem in getting 28 control signals...


fix import of vector data from URL

see Ticket#4598227

[x] increase timeout
[x] always use URL as ID of the GeoResource

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 99.487%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
10061

Relevant Lines:
10063

💛  - Coveralls


MudMenu: Wrong offset for submenu's inside MudAppBar

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudMenu
What happened?
Starting with v7.9.0, the offset of submenu's is wrong:
7.8.0:

7.9.0, the submenu appears underneath the appbar

Expected behavior
Same positioning as in v.7.8.0
Reproduction link
https://try.mudblazor.com/snippet/QYGevYbSqLzorRDG
Reproduction steps
The example already uses an appbar.
Remove the <MudAppBar> tags to see the submenu appear at the correct position.
Relevant log output
No response
Version (bug)
7.12.1
Version (working)
7.8.0
What browsers are you seeing the problem on?
Edge
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I would assume this is regression of https://github.com/MudBlazor/MudBlazor/pull/9933
cc @versile2
Don't have any other PR that manipulates popover offset.

It's possible if the submenu has overflow behavior set to flip and starts in a negative area (top or left - value). I will check this out tomorrow.

the submenu does have overflow behavior to flip, but never starts in a negative area (top + offsetY or left + offsetX). I don't mind looking at and possibly suggesting a solution but I'm backed up right now might be two weeks.

I even removed my code just to be sure

The submenu offset in your picture is correct.
It is just cut-off because of a lower z-index than the appbar (which has its own workaround).

Based on that very long thread, the javascript that handles positioning has an appbaroffset that it applies, might be something there that can be fixed instead of changing any global value.

Any progress with that? Otherwise might need to revert https://github.com/MudBlazor/MudBlazor/pull/9933

Reverting won't fix this problem. It's caused by something else altogether different as noted above.

I feel like you are confusing something, or I might be the one who’s confused.
@Yomodo just confirmed that when you removed #9933—“I even removed my code just to be sure” (c) @versile2—the offset is back to normal, just the z-index is not correct since you didn’t use the workaround from here: https://github.com/MudBlazor/MudBlazor/issues/6909.

@Yomodo posted an issue that the offset of the submenu is now incorrect in the appbar.
@versile2 shared a screenshot showing that the submenu has the correct offset again after commenting out the code from #9933, but it is just cut off because of the z-index.
@Yomodo confirmed that the offset is correct in your screenshot, but you are missing the z-index workaround.

Which leads me to believe the issue with #9933 is real and it changed the offset of submenu?
Am I missing something?

I obviously misunderstood Yomodo's message. I think reverting is the best move. Thanks for setting it straight.

I will wait for @Yomodo confirmation that this is a still problem and that my flow of events was correct.

But I wonder if it's related to this?
https://github.com/MudBlazor/MudBlazor/blob/c3969731f49855bec873c2502c845906fd072960/src/MudBlazor/TScripts/mudPopover.js#L180
If it's removed, will be the offset be correct for the appbar with your changes...?

Honestly I think my fix was such a rare use case it's not worth preserving if it causes a more common use case to be messed up. Which it will because of that and the line you posted. I would like to take another shot at it, but to preserve both and fix the appbar/menu zindex problem.

@ScarletKuro spot on, you got it right.
Tried v7.9.0 again and I see that I need to clarify a bit more.
Recap: v7.8.0 works as expected, MudAppBar inside MudMenu:

Starting from v7.9.0, the mainmenu's offset is wrong.
The menu now appears underneath the MudAppBar
v7.9.0+:

Only when I was creating the TryMudBlazor, did I notice that the submenu's offset is wrong:
(which is the issue I reported)
However, I now see that the mainmenu's offset remains correct in TryMudBlazor.
With MudAppBar (correct mainmenu offset, wrong submenu offset):

Without MudAppBar (both offsets are correct):

I'm not sure if this was always the case for TryMudblazor.

Honestly I think my fix was such a rare use case it's not worth preserving if it causes a more common use case to be messed up. Which it will because of that and the line you posted. I would like to take another shot at it, but to preserve both and fix the appbar/menu zindex problem.

Your fix, did fix cutting off a MudAutoComplete popover, that is on the left edge side of the screen.

It's the same issue as the zindex. Popover is used in a lot of places. Since I'm in JS I should be able to fix both the zindex issue and my issue using specific targetting. I think.

@ScarletKuro Did my PR get reverted? I don't see my changes in the latest fork.

No, it wasn't reverted.
https://github.com/MudBlazor/MudBlazor/blob/dev/src/MudBlazor/TScripts/mudPopover.js#L259
You need to rebase your fork to latest change from upstream

Just an update. I know what and where the problem is and am still working on it. It's not really straight forward and I didn't do the initial design so having to walk through it very slowly to ensure I don't miss anything.

Just an update. I know what and where the problem is and am still working on it. It's not really straight forward and I didn't do the initial design so having to walk through it very slowly to ensure I don't miss anything.

Awesome. Your effort is very much appreciated!


Fix master

This PR will fix broken imports  and will make sure that our master is build-able and test-able. Tested for protostar version 0.2.5/3.0/3.1/3.2(newest one). WILL NOT work for protostar v0.2.6.

Also, for import from OZ do: "from src.openzeppelin..... import"


Update integration tests to use mock-server container

See https://github.com/OpenLineage/OpenLineage/issues/90
To integration test the various OL integrations, we should replace all of the existing references to the Marquez container with a mock-server, such as was done in the Spark integration. This avoids a dependency on the Marquez implementation in the OpenLineage integration tests, so a bug in the implementation won't cause failures in the integration tests.

Removed dependency in https://github.com/OpenLineage/OpenLineage/pull/129


README suggests using git:// URL, but this is unauthenticated

As the manpage for `git fetch´ suggests:

The native transport (i.e. git:// URL) does no authentication and       should be used with caution on unsecured networks.

Please always use https://.

Good catch, it's been fixed thanks.