Initial mode issue in  00_6-layer-increasing-model.ipynb

Dear Liu,
I am getting the error, while running the "inversion" in  00_6-layer-increasing-model.ipynb jupyter file.

NameError                                 Traceback (most recent call last)
 in
2                         model_param = model_parameter,
3                         inv_param = inversion_parameter,
----> 4                         init_model=initial_model,
5                         pvs_obs=pvs_obs,
6                         vsrange_sign="mul",
NameError: name 'initial_model' is not defined
I need your suggestion how to correct the issue.
Thank you

Hello,
Did you execute the code in order? When I run the code in the new environment it works fine. Based on the error information you provided, I think there may be some missing code that is not running, or there may be some problems with the local environment configuration. Can you provide more information to help me find this problem

After update I am getting new error.

Thank you

After update I am getting error in inversion also.

Thank you

what's the version of your pytorch?

amit@amit:~$ pip show torch
Name: torch
Version: 1.6.0+cu101
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /home/amit/.local/lib/python3.6/site-packages
Requires: future, numpy
Required-by: torchvision

I have torch as you mentioned in the requirement but lower version it is.
Version: 1.6.0+cu101

$ conda list
It shows
pytorch                   1.12.1
pytorch-cuda              11.6
torchaudio                0.12.1
torchvision               0.13.1

In ADsurf conda environment, conda list shows as it is given below.
pytorch                  2.1.2               py3.8_cpu_0    pytorch
pytorch-cuda        11.6                h867d48c_1    pytorch
torch                      2.1.2               pypi_0              pypi
torchaudio            0.13.1             pypi_0              pypi
torchvision            0.14.1             pypi_0              pypi
Thank you


make fails with  fatal error: metall/utility/metall_mpi_adaptor.hpp: No such file or directory

I am following the Spack install procedure and after installing Metall with Spack I am still getting this error when building:
$ make
[  4%] Built target run_shm_cc
Scanning dependencies of target CORAL2_benchmark
[  6%] Building CXX object src/CMakeFiles/CORAL2_benchmark.dir/CORAL2_benchmark.cpp.o
In file included from /mnt/c/Users/crm0376/Projects/havoqgt/include/havoqgt/distributed_db.hpp:9,
                 from /mnt/c/Users/crm0376/Projects/havoqgt/src/CORAL2_benchmark.cpp:19:
/mnt/c/Users/crm0376/Projects/havoqgt/include/havoqgt/metall_distributed_db.hpp:9:10: fatal error: metall/utility/metall_mpi_adaptor.hpp: No such file or directory
    9 | #include <metall/utility/metall_mpi_adaptor.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

FYI this is for development in a  WSL Ubuntu 20.04 venv.

Solved with correct flag for metall root


RuntimeError: GpuCorrMM failed to allocate working memory of 576 x 802816

Hello.  Thx for your code. I encountered a problem when runming on my machine.  As the image shows below.

I've tryed to search and fix this problem for long time but failed.  Have you ever got this error, could you please help? Thank you in advance!

Hi.
I'm afraid your GPU features too low memory. What model is it?
Does it fail on all datasets/networks?

Yes, DavideA you're right. I run the voc dataset and the problem lies in my gpu's lack of memory (It's only 2G and can't load in the network parameters).  After change into cpu mode, the error disappeared.
I have to say, your code really helped me a lot for the compreshion of the original paper. You implemented the front-end(VGG-16 with removal of last pooling layer)+dilation7/dilation8(both basic and large channel)/dilation10 respectively in very details. But there are still some questions leaved for you:
one tiny question is, you seems havn't used the input shape as the paper descriped for all the four dataset. Can you explain the reason.
second one: although with the comment, I still can't understand the line74-line75. Why these two layers exist? I found no evidence  descriped in the paper. Some more explaination will be appreciated.

Thank you in advance!

Glad I could help!

Input shapes: the one I used are taken from authors' original prototxt files.

That couple of layers (as the above comments state) pretend to be a deconvolution layer (aka transposed convolution). You can find such deconv layer in the cityscapes prototxt at lines 690-707. I tried to port such layer as a keras deconvolution2d layer but I never managed to get it to work. Looking to the weights, you'll see they're a 16x16 gaussian (i.e. it's just an upsampling). So I implemented it as an upsampling followed by a gaussian deconvolution. It is not the same thing, but it should be very similar.

Bye!

It's really helpful. Thank you very much :)

You're welcome!
Closed


Add the possibility to select Canada as a country

Nice work on this fork so far.  I thought it would be nice to add the ability to change the country. Actually implementing other shops will probably take much more time than this but I will try my best. Let me know what you think.
Cheers

Anyone tested this with a Canada order to see if it works? If so - I'm all for adding love for our Canadian folks 👍

Thanks for the contribution looks great!

Anyone tested this with a Canada order to see if it works? If so - I'm all for adding love for our Canadian folks +1

I actually tried to test it but: the bot will err out both sites due to things unrelated to this PR AND when checking the websites I realized that they don't have an option to select another country at all.. So ¯_(ツ)_/¯ My plan is to try to get some smaller retailers to work in the hopes that the bot protection there is more relaxed


if raise_on_error=False skip following tasks for hosts that failed

With the current behavior a host that failed a task will run the rest of the tasks if raise_on_error=False. This changes the behavior to skip the execution on subsequent tasks.
Requires #56

Looks like the tests are failing because the xyz_connection tasks doesn't return a result object. In general it probably makes sense that a task would return a status/result.
However it would also be nice to be able to write simple tasks which should always end in a success where you don't care about the result object. So either just continue if the result doesn't have the skipped attribute, or enforce the use of the Result object.

Looks like the tests are failing because the xyz_connection tasks doesn't return a result object. In general it probably makes sense that a task would return a status/result.

Yeah, will fix.

However it would also be nice to be able to write simple tasks which should always end in a success where you don't care about the result object

Well, tasks are supposed to do something so they should report something, even if it's an empty result just to indicate everything went fine. Will think about this.


Refactor feature/handlebars

Add in Ben's changes from 4/19 to the MVC version of Streamability

[x] Changed tastedive (removed) to movieDB for suggestions
[x] Removed cors

Pushed these changes to develop on 4/22/22

Closing in favor of v2


Linking object files produced by separate build systems

I need to link object files produced from another language to my Rust program. After a long search, I found this PR that adds this functionality. However, I can not find good examples of how to use it.
I am trying with this build.rs:
fn main() {
    cc::Build::new()
    .object("/path/to/my/o/file/foo.o")
    .compile("foo");
}

Am I missing something?

After a couple of hours of searching, it seems that the solution for this problem is just using this in the build.rs
fn main() {
    println!("cargo:rustc-link-arg=/path/to/my/o/file/foo.o");
}


TQDM output looks odd after stderr stdout change

With the new change to vscode handling stderr and stdout differently TQDM now looks funny. Per Don it is writing to stderr, so the red is expected. However the blue elements might either be writing to stdout or explicitly colored blue. Should investigate.

Seems more likely to be a core VS Code problem.

import sys
sys.stdout.write('stdout')
sys.stderr.write('stderr')
sys.stdout.flush()
sys.stderr.flush()

Here's the expected output from classic Jupyter:

Requested change = Set background color instead of changing the foreground color.
Else it looks weird.

No longer an issue


Report errors if union subtype reduction results in fewer types than were explicitly written

// A function that accepts strings or non-primitive types
function foo(x: Object | string) { ... }
foo('hello'); // ok
foo({ a: 3 }); // ok
foo(1); // ok, meant for this to be an error

Based on examples seen in DefinitelyTyped. Someone misunderstood what Object means, thinking it meant non-primitive types (not an uncommon source of confusion), so they union it with the one primitive type they do want to allow. The compiler performs subtype reduction and ends up with only Object. This is now allowing code the user intended to disallow. At this point if the compiler knows it has reduced an explicitly written union type to a set of types smaller than what was originally written it should just issue an error to alert the user that some part of their type annotation is not having the effect they expect.

We should probably do #1809 first so that this code has somewhere reasonable to migrate to?

The fix here would encapsulate more types of errors than just conflating Objects and primitives so I wouldn't say it's entirely necessary. In an ideal world if this is the majority of cases we see then yeah it might make sense to wait.
On the other hand this would be a breaking change so the longer we wait the more impactful the break becomes.

@danquirk what is the proposal here, if not #1809?

That gives people who want this pattern somewhere to go, but it doesn't get them there. If we implement #1809 tomorrow all the places where anyone wrote Object | string are still valid and still unknown bugs. Making this pattern an error would then alert them to their bug, and we could offer a better alternative.
In addition, any case where one union constituent completely captures the set of values meant to be specified by another union constituent are still relevant. A contrived example:
function foo(x: Base | Derived) { ... }
foo(new Derived2());

What was the developer's intention in saying | Derived if it's completely redundant in a union with Base as one of its constituents?

What was the developer's intention in saying | Derived if it's completely redundant in a union with Base as one of its constituents?

I think that's clear in some contexts, but unclear in others. In the compiler we have some functions like (made up example):
function isExportLike(node: CallExpression|Declaration) { ... }

Then later I made CallExpression extend Declaration. It's not obvious that means isExportLike is now an error.
Similarly, I wanted to write a function:
function isDefineCall(node: Expression|CallExpression) {
  // Pretend we have tagged types for a moment
  if(node.kind === SyntaxKind.CallExpression) {
    // Maybe return true
  }
  return false;
}

This is probably an abuse of union types, but conveys well the intent of the function ("You can pass me anything, but I'm only going to possibly return true for CallExpression") and simplifies its implementation.

Moot now that we don't immediately reduce union types.


SAMZA-2138: Incorrect logging when task to container mapping is written.

Example before this fix:
2019-03-21 08:36:34.810 [main] JobModelManager$ [INFO] Storing ssp: Partition 1 and task: 0 into metadata store

@shanthoosh please take a look.

Thanks @shanthoosh. Please merge when you have some time.

@dnishimura
This build is green in travis. Here's the successful travis build associated with this PR: https://travis-ci.org/apache/samza/builds/509502538?utm_source=github_status&utm_medium=notification . However, it is not updated here in Github.

It happened to me before. Not sure why. Thanks for merging it.


Not working with universal sagetynet fix.

When I install universal safetynet fix the google services constantly crash. How can I use bank apps if I can't use safetynetfix?

Are you able to supply any logs?

@axydavid are you using magisk's deny list? that caused a lot of problems for me
@wacko1805 when I add com.android.gms etc I get a bunch of "Failed to open APK '/system/priv-app/...': I/O error" logs but it all goes away when I lift the deny list? either way, I pass YASNAC with safety net fix and your gapps module

Closed as this relates to old MagiskGApps varients.


Grunt option to do a Dojo build of WAB

We could build into our gruntfile, in addition to the "copier" and "transpiling" tasks, the option to do a build. See https://github.com/gbochenek/web-app-builder-build

I for one would be interested in this as well. In my case the downloaded version of a WAB is 15.5 MB. I also use CMV for highly customized projects and using the gruntfile for CMV I am able to get the size of an entire project to 825 KB. It would be nice to use uglify, cssmin, jshint, and compress all files within a WAB project so we know the files being pulled by clients are as small as possible so they load faster.

Closing - superseded by #99


Update description for all projects

This PR changes update project description script for support different descriptions for some group of projects.
Now it is divided on:
ORML_PROJECTS=('karura acala bifrost interlay kintsugi')
BASE_PROJECTS=('polkadot kusama westend')
ETH_PROJECTS=('moonbeam moonriver astar shiden')
ASSETS_PROJECTS=('statemine parallel parallel-heiko westmint')
REGULAR - all other projects which not include in categories above

Tested logic by commit - https://github.com/nova-wallet/subquery-nova/pull/119/commits/ba3500a72e867d9137b7d496549be586a1145ba5
Acala - is ORML project
Aleph-zero - is REGULAR project
Altair - is REGULAR project
Astar - is ETH project
Basilisk - is REGULAR project
Bifrost - is ORML project
Bit-country - is REGULAR project
Calamari - is REGULAR project
Centrifuge - is REGULAR project
Clover - is REGULAR project
Composable - is REGULAR project
Crab - is REGULAR project
Crab-solochain - is REGULAR project
Edgeware - is REGULAR project
Efinity - is REGULAR project
Encointer - is REGULAR project
Equilibrium - is REGULAR project
Hydra - is REGULAR project
Integritee - is REGULAR project
Integritee-solochain - is REGULAR project
Interlay - is ORML project
Karura - is ORML project
Khala - is REGULAR project
Kico - is REGULAR project
Kilt - is REGULAR project
Kintsugi - is ORML project
Kusama - is BASE project
Litmus - is REGULAR project
Moonbeam - is ETH project
Moonriver - is ETH project
Nodle - is REGULAR project
Nodle-solochain - is REGULAR project
Parallel - is ETH project
Parallel-heiko - is ETH project
Phala - is REGULAR project
Picasso - is REGULAR project
Polkadot - is BASE project
Quartz - is REGULAR project
Robonomics - is REGULAR project
Shadow - is REGULAR project
Shiden - is ETH project
Statemine - is ETH project
Statemint - is REGULAR project
Subsocial - is REGULAR project
Subsocial-solochain - is REGULAR project
Turing - is REGULAR project
Westend - is BASE project
Westmint - is REGULAR project
Zeitgeist - is REGULAR project
Done !

@antonkhvorov Could you review that descriptions:
https://github.com/nova-wallet/subquery-nova/pull/119/commits/92d869064621f66d564255f257cd78abc5a9ecb4
Categories was described and tested in messages above

Was done some logic reworks and tested in on my projects, is examples:

Base+Staking

Base+Assets

Base+Assets+Staking+Staking_analytic

Was added emotions, as so big text isn't responsible without highlighting
This use html highlighting and for queries examples was added code type

Base+multiassets

Base+Staking

Base+Staking+staking_analityc


Unused link definition in AppStrings

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name: n/a
iOS version: n/a
App version: lates source code under development as of 2021-11-20
branch release/2.15.x commit 0deabb62045f6a2f11761b0ce8f317ca65a79568

Describe the bug

The definition AppStrings.Links.healthCertificateFAQ seems not referenced.
git annotate shows that it was added on 06 May.
So, either it was forgotten to add this link to some page, or it is obsolete meanwhile.
Note: the definition resolves to NSLocalizedString("HealthCertificate_Info_register_FAQLink",...)
and that resolves to the localised version of the general FAQ URL, like "https://www.coronawarn.app/de/faq/"
Steps to reproduce the issue

Expected behaviour

Possible Fix
Find a good place to use this link definition, or delete it.

Additional context

@dsarkar I have created a PR #3869 under the assumption that the definition can be deleted.
Please apply the labels. Thanks!

Closing, the fix was merged.


Unifi Controller Initial Commit - Try 2

Let's try this again...

What is the status of this PR?  It doesn't Iook like it has been touched in a while and I would really like to be able to use this feature.  If it has been abandoned by @gottsman I would love to pick it up and try to finish it.

I got frustrated with my lack of programming skills and decided to move on. I didn't want to scrap my work so I left it open. Please run with it as it mostly works.
--
Kevin

On Nov 20, 2015, at 9:02 AM, gpiper14<EMAIL_ADDRESS>wrote:
What is the status of this PR? It doesn't Iook like it has been touched in a while and I would really like to be able to use this feature. If it has been abandoned by @gottsman I would love to pick it up and try to finish it.
—
Reply to this email directly or view it on GitHub.


Update wasmparser: 0.83.0 => 0.90.0

Codecov Report

Merging #415 (293d084) into master (944a88a) will decrease coverage by 0.18%.
The diff coverage is 59.13%.

:exclamation: Current head 293d084 differs from pull request most recent head a5ab873. Consider uploading reports for the commit a5ab873 to get more accurate results

@@            Coverage Diff             @@
##           master     #415      +/-   ##
==========================================
- Coverage   76.97%   76.78%   -0.19%
==========================================
  Files          71       71
  Lines        6826     6836      +10
==========================================
- Hits         5254     5249       -5
- Misses       1572     1587      +15

Impacted Files
Coverage Δ

wasmi_v1/src/engine/config.rs
70.83% <ø> (ø)

wasmi_v1/src/module/compile/mod.rs
38.58% <20.00%> (-0.39%)
:arrow_down:

wasmi_v1/src/module/parser.rs
71.12% <30.95%> (-6.66%)
:arrow_down:

wasmi_v1/src/module/data.rs
57.14% <50.00%> (ø)

wasmi_v1/src/module/element.rs
53.57% <66.66%> (ø)

wasmi_v1/src/module/export.rs
36.53% <66.66%> (+1.99%)
:arrow_up:

wasmi_v1/src/module/import.rs
75.00% <83.33%> (+1.47%)
:arrow_up:

wasmi_v1/src/module/utils.rs
88.09% <92.30%> (ø)

wasmi_v1/src/module/compile/block_type.rs
95.45% <100.00%> (ø)

wasmi_v1/src/module/compile/operator.rs
98.07% <100.00%> (ø)

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


image upload fails

When attempting to upload an image I get an extensive error/debug message.

@typonaut Can you please provide more details?
Please take a look to the demo project https://github.com/inoks/django-classified-demo

I'd go through it step by step, but the demo site is not currently working:
https://django-classified.herokuapp.com/

Application error
An error occurred in the application and your page could not be served. If you are the application owner, check your logs for details. You can do this from the Heroku CLI with the command
heroku logs --tail

Let me know when you have a demo site working again and I'll recreate the problem.
Essentially, I created an account and uploaded an image file, but when I did this it created an error in the application and didn't load the image. You can probably see this is a problem as no other demo ads had images.

I am now able to add images without error when submitting them. However, those uploaded images do not show in the browser.
The HTML gives a URL:
cache/f9/e9/f9e97fee4dce9a7a1c939645b6bf6fb1.jpg
runserver in terminal shows this (404)
Not Found: /django_classified/static/cache/f9/e9/f9e97fee4dce9a7a1c939645b6bf6fb1.jpg [05/Feb/2020 16:38:50] "GET /django_classified/static/cache/f9/e9/f9e97fee4dce9a7a1c939645b6bf6fb1.jpg HTTP/1.1" 404 5050
And I find the files actually here:
~/cache/f9/e9/f9e97fee4dce9a7a1c939645b6bf6fb1.jpg
Is there a setup step I am missing to link the requested URLs to the actual file locations?
Also, not sure it is necessary to with file types from the uploaded .png to a .jpg!?

@typonaut Can you somehow share your project or some part of it, so I can investigate that issue?
It's not clear to me why this recursion error is happening.
Also, I notice you use a pretty old version of the demo app project.

@typonaut Nevermind, I see why it's happening.

This is fixed now.

I can confirm that everything is now functioning as expected.


EC acceleration

Change our current "arith" machine (maybe to generic of a name) take inputs and output via memory. Maybe it also make sense to make it smaller (using a block size of 1)?
Tasks:

[x] Add machine & test to standard library
[x] Integrate end-to-end via syscall in RISC-V machine

This is done.


Treat errno 95 as non-ltfs destination (Fix #236)

Summary of changes
This pull request includes following changes or fixes.

Fix of issue #236

Description
This changes the behavior of ltfs_ordered_copy when xattr is not supported on the destination filesystem. Instead of failing the copy, it disables the -p flag and continues on without attempting to preserve attributes. Previously, users would be unable to copy to a non-ltfs filesystem that does not support xattr. Now, users can still perform the copy, but without the additional metadata added by -p.
Fixes #236
Type of change

New feature (non-breaking change which adds functionality)
This change migth require a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have confirmed my fix is effective or that my feature works

@softloft38p-michael ,
Thank you very much for your PR ! It looks great to me.


Prevent importing test files from extension code

Part of #8975

Codecov Report

Merging #9180 (7721bc1) into main (b604e82) will increase coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff          @@
##            main   #9180   +/-   ##
=====================================
  Coverage     71%     71%
=====================================
  Files        382     382
  Lines      24853   24852    -1
  Branches    4035    4035
=====================================
+ Hits       17707   17802   +95
+ Misses      5528    5444   -84
+ Partials    1618    1606   -12

Impacted Files
Coverage Δ

src/client/common/extensions.ts
88% <ø> (ø)

...ascience/raw-kernel/rawNotebookSupportedService.ts
88% <100%> (-1%)
:arrow_down:

src/client/datascience/dataScienceSurveyBanner.ts
69% <0%> (-7%)
:arrow_down:

...tascience/jupyter/kernels/kernelCommandListener.ts
70% <0%> (-4%)
:arrow_down:

src/client/datascience/jupyter/jupyterWebSocket.ts
53% <0%> (-4%)
:arrow_down:

...atascience/interactive-window/interactiveWindow.ts
66% <0%> (-1%)
:arrow_down:

...t/datascience/notebook/vscodeNotebookController.ts
80% <0%> (+<1%)
:arrow_up:

...lient/datascience/jupyter/kernels/cellExecution.ts
74% <0%> (+<1%)
:arrow_up:

src/client/datascience/telemetry/telemetry.ts
85% <0%> (+<1%)
:arrow_up:

src/client/datascience/commands/commandRegistry.ts
41% <0%> (+<1%)
:arrow_up:

... and 34 more


Use sendRedirectedMessage where possible

This helps slightly with compat, as some mods benefit from player context with packets.

What do you mean by it helps compat? Do other mods inject immptl's sendRedirectedMessage?

I see


Support YAML descriptions of check suites

Following Ag's work, change the compliance-checker code to support reading a YAML file that describes a suite of checks.
Basic idea:

User has some checks that depend on some parameters ('base checks')
Would like to create a check suite where each check calls one of the base checks with different parameters
Create a YAML file containing a list of base checks and parameters to use
compliance-checker reads YAML file and dynamically creates a checker class - each item in the list becomes its own check method that calls the base check with the correct parameters

Implementation ideas/questions:

compliance-checker codebase could contain a class BaseParametrisableCheck. Users would then create subclasses to initialise required parameters, and implement __call__ method to perform the check on a dataset. The same idea Ag has already implemented at https://github.com/cedadev/compliance-check-lib/blob/master/checklib/register/callable_check_base.py#L6)
Could add command line argument --yaml-check=<filename> to add the generated check, and use the normal --test=<name> with the suite name specified in the YAML.
How do we make compliance-checker aware of user's subclasses? Could use same approach as plugins for CC (see documentation here), i.e. users would create a separate python project and add an entry point in setup.py

@agstephens Have I got the basic idea above correct? Is there anything missing?

Fairly rough first attempt at this here: c1a923f569141f8ccb818a2bcb6891c960f8f16a. Currently it uses a hardcoded check method that checks if the dataset filename contains a certain string as a substring (could be anything but this is simple enough to test easily).
So far there are much fewer options available in the YAML config compared to Ag's existing work (e.g. https://github.com/cedadev/compliance-check-maker/blob/master/project/ukcp18/file-info.yml)
Usage is as follows:
my-test.yaml
suite_name: "my_test_suite"

checks:
  - check_id:    "first_check"
    check_level: "HIGH"
    params:      {"string": "bad"}

  - check_id: "second_check"
    params: {"string": "good"}

Then run
cchecker.py --yaml-test my-test.yml --test my_test_suite ~/good_file

Output should be something like the following:
2017-12-14T16:19:46.327647 [INFO] :: ESDOC-PYESSV :: Loading vocabularies from /home/ubuntu/.esdoc/pyessv-archive:
Running Compliance Checker on the dataset from: /home/ubuntu/good_file

--------------------------------------------------------------------------------
                      The dataset scored 1 out of 2 points
                         during the my_test_suite check
--------------------------------------------------------------------------------
                               Scoring Breakdown:

                                 High Priority
--------------------------------------------------------------------------------
    Name                            :Priority: Score
check_first_check                       :3:     0/1

                                Medium Priority
--------------------------------------------------------------------------------
    Name                            :Priority: Score
check_second_check                      :2:     1/1

--------------------------------------------------------------------------------
                  Reasoning for the failed tests given below:

Name                             Priority:     Score:Reasoning
--------------------------------------------------------------------------------
check_first_check                      :3:     0/ 1 : String 'bad' was not found
                                                      in filename
                                                      '/home/ubuntu/good_file'

If this is on the right lines, I suppose the next step would be to specify the base check in the YAML instead of using hardcoded check.

@joesingo: your suggested approach looks good. I suspect that we might get feedback from the IOOS guys later that will require some tweaking - but looks like a sound plan for the prototype.

As of 4d9ceeaa64e1413bd7ce69d7979b54b7649c1e42 I have removed myBaseParametrisableCheck class and moved in CallableCheckBase and subclasses directly from compliance-check-lib.
YAML config can now look something like:
suite_name: "my_test_suite"

checks:
  - check_id:       "fi01"
    check_name:     "compliance_checker.file_checks.FileSizeCheck"
    modifiers:      {'threshold': 2, 'strictness': 'soft'}
    comments:       "This is an advisory check"

  - check_id:       "fi02"
    check_name:     "compliance_checker.file_checks.FileSizeCheck"
    check_level:    "HIGH"
    modifiers:      {'threshold': 4, 'strictness': 'hard'}
    comments:       "This is an strict check"

  - check_id:       "fi03"
    check_name:     "compliance_checker.file_checks.FileNameStructureCheck"
    check_level:    "HIGH"
    modifiers:      {'delimiter': '_', 'extension': '.nc'}

The check_name field can point to any subclass of CallableCheckBase that is importable when cchecker.py is run

Notes from discussion with Ag:

Consider renaming __call__ to something like do_check in CallableCheckBase to make things clearer (and rename, as will no longer be callable)
Move CallableCheckBase and sub-classes back into compliance-check-lib, and make compliance-check-lib a dependency of CC
All sub-classes will live in compliance-check-lib - if someone wants to create their own checks they will change CC-lib code itself
Write lots of documentation, provide simple example of how to use, consider making video (https://screencast-o-matic.com/)


'Is someone responding to bug reports' check failing with AttributeError: 'str' object has no attribute 'get'

Describe the bug
Hi. :wave: First, thanks for making this project — it seems like a cool idea in general and could be a useful tool for doing project/community health summaries (or first passes at least — software metrics are hard). :slightly_smiling_face:
I'm seeing a small bug on check 4 (Is someone responding to bug reports?) when I apply it to some Scikit-HEP projects. As a particular failing example, I'll use pyhf as I maintain that.
To Reproduce
Steps to reproduce the behavior:
$ docker run --rm -ti python:3.9 /bin/bash
root@16ee2e838cc6:/# python -m venv venv && . venv/bin/activate
(venv) root@16ee2e838cc6:/# python -m pip --quiet install --upgrade pip setuptools wheel
(venv) root@16ee2e838cc6:/# python -m pip --quiet install the-well-maintained-test
(venv) root@16ee2e838cc6:/# python -m pip list
Package                  Version
------------------------ ---------
certifi                  2021.10.8
charset-normalizer       2.0.8
click                    8.0.3
colorama                 0.4.4
commonmark               0.9.1
idna                     3.3
pip                      21.3.1
Pygments                 2.10.0
requests                 2.26.0
rich                     10.14.0
setuptools               59.2.0
the-well-maintained-test 0.3.1
urllib3                  1.26.7
wheel                    0.37.0
(venv) root@16ee2e838cc6:/# the-well-maintained-test 'https://github.com/scikit-hep/pyhf'
1. Is it described as 'production ready'?
        The project is set to Development Status Beta
2. Is there sufficient documentation?
        Documentation can be found at https://pyhf.readthedocs.io/
3. Is there a changelog?
        Yes
4. Is someone responding to bug reports?
Traceback (most recent call last):
  File "/venv/bin/the-well-maintained-test", line 8, in <module>
    sys.exit(cli())
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/the_well_maintained_test/cli.py", line 71, in cli
    print(bug_responding(bugs_url, headers))
  File "/venv/lib/python3.9/site-packages/the_well_maintained_test/utils.py", line 67, in bug_responding
    bug_create_date = datetime.strptime(i.get("created_at"), "%Y-%m-%dT%H:%M:%SZ")
AttributeError: 'str' object has no attribute 'get'
(venv) root@16ee2e838cc6:/#

Expected behavior
For this to not fail and to give some output.
Additional context
I see that this is actually happening with the PyPI distributed version of the-well-maintained-test as well
(venv) root@16ee2e838cc6:/# the-well-maintained-test 'https://github.com/ryancheley/the-well-maintained-test'
1. Is it described as 'production ready'?
        The project is set to Development Status Beta
2. Is there sufficient documentation?
        Documentation can be found at https://github.com/ryancheley/the-well-maintained-test/blob/main/README.md
3. Is there a changelog?
        Yes
4. Is someone responding to bug reports?
Traceback (most recent call last):
  File "/venv/bin/the-well-maintained-test", line 8, in <module>
    sys.exit(cli())
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/venv/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/the_well_maintained_test/cli.py", line 71, in cli
    print(bug_responding(bugs_url, headers))
  File "/venv/lib/python3.9/site-packages/the_well_maintained_test/utils.py", line 67, in bug_responding
    bug_create_date = datetime.strptime(i.get("created_at"), "%Y-%m-%dT%H:%M:%SZ")
AttributeError: 'str' object has no attribute 'get'

@matthewfeickert first, thanks for reporting the issue! I ran the code locally and you're correct, it's a rate limiting issue.
If you add a file called auth.json that looks like this do you get the same error?
{
    "github_personal_token": "github_personal_token goes here"
}

Thanks!

@matthewfeickert I've just released version 0.4.0 which does a couple of things to help resolve the issue. It's easier to generate the auth.json file now. Please let me know if the auth.json file fixes the issues for you.
Thanks!
Thanks!

Thanks @ryancheley, indeed, in v0.4.0 things are resolved once I authenticate with my GitHub PAT. :+1:
$ docker run --rm -ti python:3.9 /bin/bash
root@6522daf89297:/# python -m venv venv && . venv/bin/activate
(venv) root@6522daf89297:/# python -m pip --quiet install --upgrade pip setuptools wheel
(venv) root@6522daf89297:/# python -m pip --quiet install 'the-well-maintained-test==0.4.0'
(venv) root@6522daf89297:/# python -m pip list
Package                  Version
------------------------ ---------
certifi                  2021.10.8
charset-normalizer       2.0.8
click                    8.0.3
colorama                 0.4.4
commonmark               0.9.1
idna                     3.3
pip                      21.3.1
Pygments                 2.10.0
requests                 2.26.0
rich                     10.15.0
setuptools               59.4.0
the-well-maintained-test 0.4.0
urllib3                  1.26.7
wheel                    0.37.0
(venv) root@6522daf89297:/# the-well-maintained-test --help
Usage: the-well-maintained-test [OPTIONS] COMMAND [ARGS]...

  Programatically tries to answer the 12 questions from         Adam Johnson's
  blog post https://adamj.eu/tech/2021/11/04/the-well-maintained-test/ URL is
  a url to a github repository you'd like to check, for example:     the-well-
  maintained-test 'https://github.com/ryancheley/the-well-maintained-test'

Options:
  --version  Show the version and exit.
  --help     Show this message and exit.

Commands:
  auth       Save authentication credentials to a JSON file
  questions  List of questions tested
  url        url to a github repository you'd like to check
(venv) root@6522daf89297:/# the-well-maintained-test auth
Create a GitHub personal user token and paste it here:

Personal token:
(venv) root@6522daf89297:/# file auth.json
auth.json: JSON data
(venv) root@6522daf89297:/# the-well-maintained-test url 'https://github.com/scikit-hep/pyhf'
1. Is it described as 'production ready'?
        The project is set to Development Status Beta
2. Is there sufficient documentation?
        Documentation can be found at https://pyhf.readthedocs.io/
3. Is there a changelog?
        Yes
4. Is someone responding to bug reports?
        The maintainer took 0 days to respond to the bug report
        It has been 1322 days since a comment was made on the bug.
5. Are there sufficient tests?
        There are 43 tests:
                - src/pyhf/infer/test_statistics.py
                - tests/benchmarks/test_benchmark.py
                - tests/contrib/test_utils.py
                - tests/contrib/test_viz.py
                - tests/test_backend_consistency.py
                - tests/test_backends.py
                - tests/test_calculator.py
                - tests/test_cli.py
                - tests/test_combined_modifiers.py
                - tests/test_compat.py
                - tests/test_constraints.py
                - tests/test_custom_mods.py
                - tests/test_events.py
                - tests/test_examples.py
                - tests/test_export.py
                - tests/test_import.py
                - tests/test_infer.py
                - tests/test_init.py
                - tests/test_interpolate.py
                - tests/test_jit.py
                - tests/test_mixins.py
                - tests/test_modifiers.py
                - tests/test_notebooks.py
                - tests/test_optim.py
                - tests/test_paramsets.py
                - tests/test_paramviewer.py
                - tests/test_patchset.py
                - tests/test_pdf.py
                - tests/test_probability.py
                - tests/test_public_api.py
                - tests/test_public_api_repr.py
                - tests/test_regression.py
                - tests/test_schema.py
                - tests/test_scripts.py
                - tests/test_simplemodels.py
                - tests/test_tensor.py
                - tests/test_tensorviewer.py
                - tests/test_teststats.py
                - tests/test_toys.py
                - tests/test_utils.py
                - tests/test_validation.py
                - tests/test_workspace.py
                - validation/standard_hypo_test_demo.py

6. Are the tests running with the latest Language version?
        The project supports the following programming languages
                - Python 3
                - Python 3 :: Only
                - Python 3.7
                - Python 3.8
                - Python 3.9
                - Python Implementation :: CPython

7. Are the tests running with the latest Integration version?
        This project has no associated frameworks
8. Is there a Continuous Integration (CI) configuration?
        There are 18 workflows
         - CI/CD
         - Build Binder Image
         - Build Binder Image
         - CI/CD
         - CodeQL
         - HEAD of dependencies
         - dev-tools
         - Docker Images
         - Docs
         - Lint
         - Lint Code Base
         - .github/workflows/list_likelihoods.yml
         - Minimum supported dependencies
         - Merged PR
         - Notebooks
         - publish distributions
         - Current Release
         - Tag Creator

9. Is the CI passing?
        Yes
10. Does it seem relatively well used?
        The project has the following statistics:
        - Watchers: 196
        - Forks: 62
        - Open Issues: 340
        - Subscribers: 11
11. Has there been a commit in the last year?
        Yes. The last commit was on 11-24-2021 which was 4 days ago
12. Has there been a release in the last year?
        Yes. The last commit was on 09-03-2021 which was 86 days ago
(venv) root@6522daf89297:/#


Cache bug

Fixes #51. Please see that ticket for a description of the issue.
This is a semi-critical bug. Once the bugfix is released, all older versions of bitmerchant will be removed from pypi.
If you have been affected by this bug, please contact me directly at<EMAIL_ADDRESS>

Thanks to @shea256 for reporting the caching bug.


Remediate High Protobuf vulnerability

Snyk reports the following High severity vulnerability in HumanCellAtlas/data-consumer-vignettes. Please remediate by the end of Q1 Milestone 1.
Description
com.google.protobuf:protobuf-java
Suggested Remediation
Upgrade com.google.protobuf:protobuf-java to version 3.4.0 or higher.
Details
com.google.protobuf:protobuf-java is a Google's language-neutral, platform-neutral, extensible mechanism for serializing structured data. Affected versions of this package are vulnerable to Integer Overflow by allowing remote authenticated attackers to cause a heap-based buffer overflow in serialisation process.

PR #83 should remediate this vulnerability, please re-open if not


Errors in parsing should throw an exception

I find it a bit counterintuitive that erroneous XYZ files only print something (to stderr I believe) but otherwise simply return an empty list. Instead I would have expected an exception with the error to be raised.

Just to show a case in point: #35 was initially obfuscated by this behaviour and took me a bit to figure out what's the problem.

As you may have seen I'm very belatedly catching up on these issues. I agree with this one as well, but it will require changes in the underlying C library. I've opened at issue at https://github.com/libAtoms/extxyz/issues/14


history orders may be incorrect  after undo or redo

https://github.com/charkour/zundo/blob/f5cbda1912089179ae808f05e136b6e39a0f66c1/packages/zundo/src/temporal.ts#L40
https://github.com/charkour/zundo/blob/f5cbda1912089179ae808f05e136b6e39a0f66c1/packages/zundo/src/temporal.ts#L59
example:
// state eg: current: 5, past states: [0, 1, 2, 3, 4], future states: []
undo(3)   // current: 2, past states: [0, 1],          future states: [5, 3, 4]
redo()    // current: 4, past states: [0, 1, 2],       future states: [5, 3]
redo()    // current: 3, past states: [0, 1, 2, 4],    future states: [5]
redo()    // current: 5, past states: [0, 1, 2, 4, 3], future states: []

fix may be like below.
fs.push(...skippedPastStates.reverse());    // undo
ps.push(...skippedFutureStates.reverse()); // redo

Thank you for reporting @objectisundefined!
This has been fixed in v2.0.0-beta.7.


[Custom Connector][RC-Based]- Custom Connector Re-testing

Is your suggestion a test requirement/change or improvement of the test framework? Please describe.

[x] GitHub
[ ] Office 365
[x] Twitter
[x] Facebook
[x] Google
[x] Microsoft Windows Live - Have some issues that need to be discussed and report
[x] Passive STS
[ ] IWA Kerberose
[ ] Saml
[x] Apple
[ ] OpenID Connect

Optional Fields
Additional Test feature that'll be covered through this
Related Issues:

Reported Issues

https://github.com/wso2/product-is/issues/18986

Update - 08th Feb 2024
Carried out a test round to test external connectors that are available in Connector Store.
In here specifically tested LinkedIn connector which was shares here - LinkedIn Connector


Can i have Shared SaveChangesAsync() or SaveChanges() between multiple DbContext?

I have a DbContext(ChildDbContext) which inherited from Another DbContext(ParentDbContext).
ParentDbContext inherited from DbContext.
I want to hide SaveChanges() and SaveChangesAsync() from clients of ChildDbContext, and then by calling SaveChanges from ParentDbContext() all tracked DbEntities from ChildDbContext then will be save.

@masoud-bahrami This looks more like a C# question than an EF question, and it sounds like inheritance is not the right pattern here; use composition instead.


Fixes #4175

Fixes #4175.

Pull Request Test Coverage Report for Build 1621

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.867%

Totals

Change from base Build 780:
0.0%

Covered Lines:
10960

Relevant Lines:
16896

💛  - Coveralls


Add the HCT color space

Implements the HCT color space.

As a note, this does not port Google's Material Color Utility implementation. This implements the color space as described in https://material.io/blog/science-of-color-design. It combines CAM16 JCh and Lab D65.
I did an experiment with Google's lib, and they snap conversions to HCT to chroma bands and only convert back to about 8 bits worth of precision (hex sRGB). This implements the full color space and is not restricted to sRGB. It generally does well when converting colors within the visible spectrum, but pushing out beyond that, conversion precision will drop. Even CAM16's geometry gets really weird as you push outside the visible gamut, so this isn't much of a surprise.
The last PR should implement a color distancing algorithm while in HCT and a way to gamut map in HCT such that tonal palettes can be produced (similar to Google's). An example of what should be achievable: https://facelessuser.github.io/coloraide/colors/hct/#tonal-palettes.

HCT rendered in the Rec. 2020 gamut

I want to investigate a couple of extreme cases, so putting this in draft.

I think edge cases are understood. Because the only way to reverse transform this space is via approximation, it is simply impossible to perfectly convert every color value that may be thrown at it.
I ran a script for a while and it would randomly create a color within some of the largest spaces: Rec2020, Rec2100 PQ, Rec. 2100 HLG, ACEScg. It would then convert them to HCT and then back and see if the colors were the same up to about 5 digits. After 6 million +, I only had 3 cases, all of which were barely outside of the target:
acescg: [0.5006725107598761, 0.4440632949467216, 0.7342250000017175] != [0.5006725107581568, 0.44406329494514485, 0.7342249999994983]

rec2100-pq: [0.055817905259596734, 0.011004999987581199, 0.9368191851190308] != [0.055817905272786385, 0.011005000156002375, 0.9368191851190375]

acescg: [0.9119609101495745, 0.14183407744178544, 0.8107950000003002] != [0.9119609101461479, 0.14183407744074283, 0.8107949999973072]

I don't think any of these are worth spending time on more iterations or more complex code.
There are of course some more extreme cases. When you push HCT (which is basically CAM16) past the visible spectrum, you can get some harder to calculate colors. The algorithm will have a more trouble.

Sometimes, Newton's method just doesn't converge fast enough.
Sometimes Newton's method will start converging in one direction and then change direction.

I've been able to implement code that can actually steer it in the right direction in both of these cases, but even then, it can't fix all cases, and not always in a reasonable amount of iterations. To explain why this happens, we can look at CAM16 JMh.
Consider the low light color with a chroma way larger than a color at that lightness has any right to have. If we try and round trip the CAM16 color, it ends up with an inverse saturation and a rotated hue. It is out of the visible gamut. This is not unique to our implementation, this behavior has been confirmed in other CAM16 implementations as well.
>>> Color('blue').convert('cam16-jmh').set('j', 2)
color(--cam16-jmh 2 62.442 282.75 / 1)
>>> Color('blue').convert('cam16-jmh').set('j', 2).convert('srgb').convert('cam16-jmh')
color(--cam16-jmh 2 -42.607 102.75 / 1)
>>> Color('blue').convert('cam16-jmh').set('j', 2).convert('xyz-d65')
color(xyz-d65 -0.04159 0.00375 -0.31127 / 1)

This of course affects our ability to approximate colors in HCT to the same accuracy.  While some workarounds can help these converge better, it doesn't change the behavior because this is how CAM16 works with some of these extreme colors.
So, does this affect tonal palettes? Yes, but it can be managed. This explains why Google forces their implementation to keep everything in sRGB, because it is easier to handle. But really, when generating tone maps, I found that this is mainly problematic in low lightness situations. You can generally identify the cases simply by testing to see if the round trip flips the saturation and hue. You can assume in these cases the chroma is massively too large for the lightness and simply cut it and generate and fit a color with a more workable chroma.
We are able to generate tonal palettes that pretty much align with Material's results:

Results can be viewed via the link below. I was experimenting with generating OkLCh tonal palettes vs HCT. Spoiler, you can actually generate some pretty good tonal palettes with OkLCh without all of the complicated overhead of HCT, it just requires toeing the lightness and tightening the gamut mapping: https://gist.githubusercontent.com/facelessuser/0235cb0fecc35c4e06a8195d5e18947b/raw/293c59faa21a550ad23e523f2de7671e3ff5fcb4/exploring-tonal-palettes.md
I think with this analysis, I am confident to say that I feel the conversion algorithm is good enough. As I mentioned there are maybe two things that could additionally be done to improve the accuracy of some already bad cases of colors outside the visible spectrum, but they are still inherently bad just due to how CAM16 handles these extreme colors.
I will move this back into the review state.

Does the Google implementation map to the sRGB gamut in CAM16 JMh or in some other space?

Does the Google implementation map to the sRGB gamut in CAM16 JMh or in some other space?

I'm not quite sure I understand the question, so I'll answer as best I can. The Material library is restricted to the sRGB gamut. That's an implementation detail not a limitation of the space. Their interface allows you to specify 8bit, sRGB hex codes and it converts them to HCT. Conversion back gives you 8 bit, sRGB hex codes. I think their white point is a slightly different, rounded  D65 white point than what CSS uses: 95.047, 100.0, 108.883; we use the CSS white point. Their logic is optimized to return 8 bit sRGB. If you alter an HCT color in their library, it snaps to some sRGB color, if I recall correctly. I assume they clamp to sRGB to give consistent results across all monitors, but this also currently limits them to the sRGB gamut. Specifically, for HCT CAM16 JCh is used, not JMh.
We do not clamp this implementation. It gives full resolution of the space. In order to convert back, we must convert both the LCh part and the CAM16 parts. Since the LCh Y part is easy to convert back to XYZ with no additional context, the bulk of the conversion part is trying to figure out the missing CAM16 J part in order to convert the chroma and hue parts of CAM16 JCh, which is required for the algorithm to accurately convert back to XYZ and then to any other gamut.
The accuracy of the algorithm is limited by time. The Newton algorithm's success is also influenced by the initial guess. There are probably more time-consuming ways to get a better initial guess, but for 99% of the gamuts that fall within the visible spectrum, we are able to hit our accuracy target with a "close enough guess" using our simple polynomial. The CAM16 JMh, JCh, etc. algorithm breaks down a bit with colors outside the visible spectrum, so HCT will also start to distort when the visible spectrum is exceeded.
You can see CAM16 wrapping blue back into itself with ProPhoto which pushes these colors outside the visible spectrum.

HCT does the same and will have less accuracy in these regions which will push Newton's method harder.


Ability to save default ID as friendly name

This has worked perfectly for all functions. The only issue is that after naming the device in HomeKit and upon restart of homebridge, the light defaults to it's default ID (probably during discovery) and hence the light will be removed from my scenes. Please look into allowing friendly names or cached names so that discovery will skip re-setting up the device.
Thanks!

Leave a comment

Issue seems resolved after re-installing and renaming now it doesn't default back to the ID


Fix images reordering for mobile devices

Bug: The reordering functionality is not working on mobile devices. When attempting to move a photo, it unexpectedly stops moving and remains hanging. [only tested on Safari & Chrome latest versions]
Fix: Replaced pointer MouseEvents with touch/move MouseEvents.

@rikschennink can you have a look over my PR please?

I ran a quick test on iOS and it seems to work without issues?
Dropping pointer events feels like a step backward.

@rikschennink Please check the attached demo video showcasing the bug.
bug_filepond_mobile_image_reordering.webm

It doesn't look right, I think you should load the image preview stylesheet.

It doesn't have to do with images preview. I've reproduced the bug multiple times on different environments and mobile devices. Please run another test of the latest version of the module on any mobile OS having allow Reorder: true and try to reorder the images.

bug_filepond_mobile_image_reordering_2.webm

Couldn't reproduce in Android simulator / real Android device / iOS simulator & real iOS device.
Can only reproduce in Chrome dev tools.
✅ Open page in Chrome → ordering works
✅ Open page in Chrome → open dev tools → refresh page → ordering works
💥 Open page in Chrome → open dev tools → don't refresh page → ordering fails
I'm checking to see if there's anything I can see wrong or weird with the PointerEvent when switching to dev tools.

Fixed with touch-action: none CSS style on .filepond--item element.

Thank you! 🙏


How to get third party resources inside controller code

Type of question
how to implement a specific feature
Question
What did you do?
In the operator controller the way to get the list of standard kubernetes objects is like this
listObj := &corev1.PersistentVolumeClaimList{}
	if err := r.client.List(context.TODO(), listObj, client.MatchingLabels{"release": "my-release"}); err != nil {
		return err
	}

There are some custom resources in the cluster those are not managed by this operator, and created using kubectl. So how to get those resource list ? one way to get those following this https://github.com/operator-framework/operator-sdk/blob/master/doc/user-guide.md#adding-3rd-party-resources-to-your-operator but problem is I dont have access to the code of that third party resource to find the type. Inside the cluster only CRD and CR are available. Moreover those thirdparty resources are closed source.
Is there any other way to find those resources inside controller code
What did you expect to see?
Find the list of custom resources without knowing the go type of the resource. using only CRD or CR
Environment

operator-sdk version:

  operator-sdk version: "v0.15.1", commit: "e35ec7b722ba095e6438f63fafb9e7326870b486", go version: "go1.13.6 darwin/amd64"

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.3", GitCommit:"b3cbbae08ec52a7fc73d334838e18d17e8512749", GitTreeState:"clean", BuildDate:"2019-11-14T04:24:29Z", GoVersion:"go1.12.13", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:07:57Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
docker-desktop on mac

Additional context
Goal: The operator will find those thirdparty resources and set the owner reference to itself, so that this operator can watch/manage those resources

HI @ffoysal,
Note that by default you will be able to CRUD just resources/objects which are defined in the K8S API. So, for you can CRUD any other type which can be from other operators as well you need to add the schema to the manager.
Since you have the CRDs applied in the cluster, you will be able to discover its data to do the implementation. All steps are fully described in the blog How to use third-party APIs in Operator SDK projects. Note that this blog will use as an example the OCP API, however, you can do the same steps to use any other third-party.
Also, I truly recommend you check its [Step 2: Use the Discovery API to see if the new API is present] and do the suggested implementation to ensure that the third-party will exists too. There you will see an example over how you can see how to discover and check the API schemas via the go impl as checking the usage of the command kubectl api-resources which can help you with.
In this way, I am closing this since it shows sorted out with the info provided. However, please feel free to ping us here and let us know if you would like to re-open it because still needing help with the same subject.

Thank you very much @camilamacedo86
So far was able to get the list of ApiResoruces
	// Get a config to talk to the apiserver
	cfg, err := config.GetConfig()
	if err != nil {
		//reqLogger.Error(err, "cannot get rest client config")
		fmt.Println(err)
	}
	// Create the discoveryClient
	discoveryClient, err := discovery.NewDiscoveryClientForConfig(cfg)
	if err != nil {
		//reqLogger.Error(err, "Unable to create discovery client")
		fmt.Println(err)
	}
	// Get a list of all API's on the cluster
	apiGroup, apiResourceList, err := discoveryClient.ServerGroupsAndResources()
	if err != nil {
		fmt.Println(err)
	}

	// Looking for group.Name = "apiextensions.k8s.io"
	for i := 0; i < len(apiGroup); i++ {
		fmt.Println(apiGroup[i].Name)
	}
	fmt.Println("resources list")
	for i := 0; i < len(apiResourceList); i++ {
		if apiResourceList[i].GroupVersion == "manager.example.com/v1" {

		}
	}

kubectl api-resources
NAME                              SHORTNAMES   APIGROUP                       NAMESPACED   KIND
machines                           en           manager.example.com            true         Machine

how can I find this using go code so that I can delete the resources inside controller when operator CR get deleted
kubectl get machines
NAME                         AGE
machine-reload   74m

Hi @ffoysal,
Really thank you for your feedback and for let us know that the information provided allow you to achieve your goal.

I know the third-party kind Machine, how can I get the list of machines from inside controller code ?? so that I can delete those third-party instances when operator CR get deleted ??

Hi @ffoysal,
You need to add the schema. Then, after adding the schema to the manager you can CRUD the resources in the same way that you do with the K8S ones.


Removed addition of period from wrap_lons

Updated affected tests using assertArrayAllClose following #3993.
🚀 Pull Request
Description
Please see #3994.

Assuming the tests run for this PR, do let me know if you want me to edit the latest.rst file again to reflect the updated pull request (or remove it entirely from this PR).


Cannot find theme file No such file or directory (os error 2) WSL2

os: WSL2
lsd --version: lsd 0.22.0
echo $TERM: xterm-256color
echo $LS_COLORS:

Background
I created a new config.yaml file and placed it inside of ~/.config/lsd/config.yaml. I changed the theme directory inside the config.yaml file from default to light_theme (without the yaml extension). The contents of ~/.config/lsd/themes/light_theme are the default values found README.md example:
user: 230
group: 187
permission:
  read: dark_green
  write: dark_yellow
  exec: dark_red
  exec-sticky: 5
  no-access: 245
  octal: 6
  acl: dark_cyan
  context: cyan
date:
  hour-old: 40
  day-old: 42
  older: 36
size:
  none: 245
  small: 229
  medium: 216
  large: 172
inode:
  valid: 13
  invalid: 245
links:
  valid: 13
  invalid: 245
tree-edge: 245

Expected behavior
The application should be able to find the light_theme file.
Actual behavior

lsd: Not a valid theme: /home/amerii/.config/lsd/themes/light_theme, No such file or directory (os error 2).
Even though:

Rename themes/light_theme to themes/light_theme.yaml and that should fix it.

@meain Thanks, that seemed to do the trick. I initially thought that both the name of your file in the config and in the path should be without .yaml extension.


Fixed typora version to latest

Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} left no offenses.

No, please don't do this. It breaks our versioning.
Check out cask-repair, it makes Cask updates like this quite easy (details in CONTRIBUTING)

https://github.com/caskroom/homebrew-cask/pull/23349


Future: Static site streams

On the static site allow multiple people to stream for free with a limit of 30 minutes. This will come after we implement multi-streaming functionality. In the meantime we can have a test stream running there for demonstration purposes.

This is for the site I'm working on currently? or something else. If it is then I'll make sure to make a spot for it :eyes:

This is for the site I'm working on currently? or something else. If it is then I'll make sure to make a spot for it :eyes:


HSEARCH-1698 Writing file list asynchronously by default for async confi...

...gured backend

Looking good. Going to merge it.

rebased and applied

Thanks, @hferentschik!
2015-01-22 19:11 GMT+01:00 Hardy Ferentschik<EMAIL_ADDRESS>
Closed #771 https://github.com/hibernate/hibernate-search/pull/771.
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-search/pull/771#event-223193741.


Intention Error in python

Description
There is a python intention error in the example of open file.
Screenshots

You can first complete the issue which is already assigned to you. I will assign you this one later.

You can first complete the issue which is already assigned to you. I will assign you this one later.

Sent PR for that issue. Now you can assign this one.

Sent PR for that issue. Now you can assign this one.

I'll assign you once the PR is merged.


fix: support latest rabbitmq version

Description
We got the following failure in continuous tests
TestPlanICDVersions/3.13 2024-10-24T05:10:42Z retry.go:99: Returning due to fatal error: FatalError{Underlying: error while running command: exit status 1; ╷
│ Error: Invalid value for variable
│
│   on [main.tf](http://main.tf/) line 24, in module "icd_rabbitmq":
│   24:   rabbitmq_version  = var.rabbitmq_version
│     ├────────────────
│     │ var.rabbitmq_version is "3.13"
│
│ Version must be 3.11 or 3.12. If no value passed, the current ICD preferred
│ version is used.
│
│ This was checked by the validation rule at ../../variables.tf:20,3-13.

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Support the latest rabbitmq version 3.13. Remove lowerbound version 3.11.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline


bazel: Permission denied when executing 'bash apollo.sh clean' in docker

I'm trying to build Apollo 1.5 on the nvidia Jetson TX1(Ubuntu 16.04, arm64, kernel patched so docker can be supported), so I downloaded Apollo1.5 source code and followed the instructions in README.md. There was no problem when I executed 'dev_start.sh' and 'dev_into.sh'. But I failed to build the Apollo because it told me the following error when I executed 'bash apollo.sh clean':
apollo.sh: 346: bazel clean: Permission Denied

So I'm just wondering anyone could help me with it. Thanks a lot!

Yep， I do rebuild of docker image based on the dockerfile provided but add sudo package into the image and then I can solve this problem. So thanks for your help. I will have a try on what've you provided. This issue can be closed now.


remove non essential import

Proposed changes:

...

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for that doc improvement :+1:


Upgrade to 0.8.0

https://app.shortcut.com/coinalpha/story/39295/upgrade-aiokafka-version-in-coinalpa-aiokafka


Error pattern needs to direct user to components for message details

The error pattern should point users to the guidance on what error messages should be displayed for each applicable component.
Other errors and typos:

'Enter an number less than or equal to 100.' (in error examples and proto)
Research section refers to 'component' rather than 'pattern'.

 Error pattern needs to direct user to components for message details


Input buffer capacity exceeded when reading large values

When reading results with values larger than the ChunkedInputBuffer's default capacity, a non-descriptive error gets triggered.
It would be nice to have the buffer capacities configurable along with a specific error message.
I'm currently using this as a workaround:
class LargeChunkedInputBuffer(ChunkedInputBuffer):
    def __init__(self, capacity=4194304):
        super().__init__(capacity)

neo4j.bolt.connection.ChunkedInputBuffer = LargeChunkedInputBuffer

I realize it's not the best idea to store large values in Neo4j but as long as it's not prohibited they should be readable.
https://github.com/liutec/neo4j-python-driver/commit/165946f22af4a22c2487eeab74b953d752e6ac05

HI @liutec,
Thanks for bring this issue to us. It should not be needed to enlarge the capacity manually as the buffer should automatically grow for big records. Could you share us what error did you got?
Cheers,
Zhen

Hello,
sorry for the late reply as well as for not including all the steps to reproduce this issue.
I was under the impression that the buffer capacity has a fixed maximum size to avoid re-allocations. If the buffer was meant to grow beyond the capacity then it's clearly a problem only on Python 3 https://bugs.python.org/issue29178
neo4j server  v3.0.5
Python 3.5.2
neo4j-driver >= 1.4.0 (did not test on older versions)
import sys
import neo4j
from neo4j.v1 import GraphDatabase

print(sys.version_info)
print("neo4j-driver", neo4j.__version__)

neo4j_driver = GraphDatabase.driver(
    uri="bolt://xxx.xxx.xxx.xxx:7687",
    auth=("neo4j", "neo4j")
)

cypher = "MERGE (n:TestNode) SET n.data = $data"
params = {
    "data": "x" * 524110  # does not reproduce below 524111
}

with neo4j_driver.session() as tx:
    tx.run(cypher, params)
del params

cypher = "MATCH (n:TestNode) RETURN n.data AS data LIMIT 1"
with neo4j_driver.session() as tx:
    records = tx.run(cypher)
    for record in records:
        print(len(record['data']))

sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)
neo4j-driver 1.4.0
Exception ignored in: 'neo4j.bolt._io.ChunkedInputBuffer.receive'
BufferError: Existing exports of data: object cannot be re-sized
Traceback (most recent call last):
  File "neo4j-bug.py", line 26, in <module>
    for record in records:
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 718, in records
    self._session.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 356, in fetch
    detail_count, _ = self._connection.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 264, in fetch
    self._receive()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 302, in _receive
    raise self.Error("Failed to read from defunct connection {!r}".format(self.server.address))
neo4j.exceptions.ServiceUnavailable: Failed to read from defunct connection Address(host='xxx.xxx.xxx.xxx', port=7687)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "neo4j-bug.py", line 27, in <module>
    print(len(record['data']))
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 263, in __exit__
    self.close()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 296, in close
    self._disconnect(sync=True)
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 278, in _disconnect
    self._connection.sync()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 337, in sync
    detail_delta, summary_delta = self.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 258, in fetch
    raise self.Error("Failed to read from closed connection {!r}".format(self.server.address))
neo4j.exceptions.ServiceUnavailable: Failed to read from closed connection Address(host='xxx.xxx.xxx.xxx', port=7687)

sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)
neo4j-driver 1.5.2
Exception ignored in: 'neo4j.bolt._io.ChunkedInputBuffer.receive'
BufferError: Existing exports of data: object cannot be re-sized
Traceback (most recent call last):
  File "neo4j-bug.py", line 26, in <module>
    for record in records:
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 718, in records
    self._session.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 356, in fetch
    detail_count, _ = self._connection.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 283, in fetch
    return self._fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 300, in _fetch
    self._receive()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 338, in _receive
    raise self.Error("Failed to read from defunct connection {!r}".format(self.server.address))
neo4j.exceptions.ServiceUnavailable: Failed to read from defunct connection Address(host='xxx.xxx.xxx.xxx', port=7687)

The problem is most likely here:
https://github.com/neo4j/neo4j-python-driver/blob/1.5/neo4j/bolt/io.py#L139

This should fix the issue, but for performance reasons I would still prefer to be able to set the buffer capacity:
    def receive(self, socket, n):
        """

        Note: may modify buffer size, should error if frame exists
        """
        new_extent = self._extent + n
        overflow = new_extent - len(self._data)
        if overflow > 0:
            if self._recycle():
                return self.receive(socket, n)
            self._view = None
            new_data = bytearray(new_extent)
            new_data[:self._extent] = self._data
            self._data = new_data
            self._view = memoryview(self._data)
        data_size = socket.recv_into(self._view[self._extent:new_extent])
        new_extent = self._extent + data_size
        self._extent = new_extent
        return data_size

Hi @liutec
Thanks for your rich info.
I got a possible idea to fix the issue you have. Would you be able to try the fix out in your code?
The changes are described as follows:

Adds a close method in MessageFrame class

    def close(self):
         self._view = None

In ChunkedInputBuffer class, change the discard_message method to the following:

    def discard_message(self):
        if self._frame is not None:
            self._origin = self._limit
            self._limit = -1
            self._frame.close() # close the frame to release the reference
            self._frame = None

Please let us know if the suggested change would fix your issue!
Thanks again,
Zhen

Hello,
both changes fix the issue up to 6MB (not hitting TransientError: dbms.memory.heap.max_size).
I never needed more than 4MB to start with, but now, while looking at your proposed fix and while testing both changes, I got a closer look at the io.py file trying to understand the ~6MB limit because both the 2.7 and 3.5 32bit interpreter seem to exhaust all addressable memory.
After adding your changes, the problems became apparent:

recursive ChunkedOutputBuffer.write causes MemoryError
fixed with:

    def write(self, b):
        new_data_start = 0
        new_data_size = len(b)
        while new_data_start < new_data_size:
            chunk_occupied = self._end - self._start
            chunk_remaining = self._max_chunk_size - chunk_occupied
            if chunk_remaining == 0:
                self.chunk()
                chunk_remaining = self._max_chunk_size
            chunk_write_size = min(chunk_remaining, new_data_size - new_data_start)
            new_end = self._end + chunk_write_size
            new_chunk_size = new_end - self._start
            self._data[self._end:new_end] = b[new_data_start:(new_data_start + chunk_write_size)]
            new_data_start += chunk_write_size
            self._end = new_end
            self._data[self._header:(self._header + 2)] = struct_pack(">H", new_chunk_size)

this seems to fix the problem for Python 2.7 as well as it drastically lowers the memory consumption but quite often the GC seems to have a hard time with the 8192 increment in ChunkedInputBuffer.receive which leads to excessive delays and/or hangs.
        data_size = self._end - self._start
        if data_size > new_data_size:
            new_end = self._end + new_data_size
            self._data[self._end:new_end] = bytearray(data_size)

this dirty fix seems to eliminate the hangs but with larger values it may be impossible to have contiguous memory blocks.
Unfortunately, for Python 3.5 the next issue is raised:

recursive MessageFrame.read causes MemoryError

Thank you for your prompt response, and again, sorry for replying so late.

Forgot to add the output for the remaining issue for Python 3:
Traceback (most recent call last):
  File "neo4j-bug.py", line 26, in <module>
    for record in records:
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 718, in records
    self._session.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/v1/api.py", line 356, in fetch
    detail_count, _ = self._connection.fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 283, in fetch
    return self._fetch()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 302, in _fetch
    details, summary_signature, summary_metadata = self._unpack()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/connection.py", line 354, in _unpack
    data = unpacker.unpack_list()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/packstream/unpacker.py", line 137, in unpack_list
    return self._unpack_list(marker)
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/packstream/unpacker.py", line 146, in _unpack_list
    return [self._unpack()]
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/packstream/unpacker.py", line 111, in _unpack
    return decode(self.read(size), "utf-8")
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/packstream/unpacker.py", line 42, in read
    return self.source.read(n)
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/io.py", line 86, in read
    value.extend(self.read(n - (end - start)))
[...]
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/io.py", line 86, in read
    value.extend(self.read(n - (end - start)))
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/io.py", line 84, in read
    self._next_pane()
  File "/opt/anaconda/envs/p352/lib/python3.5/site-packages/neo4j/bolt/io.py", line 47, in _next_pane
    if self._current_pane < len(self._panes):
RecursionError: maximum recursion depth exceeded in comparison

Hello,
I've also changed MessageFrame.read and removed the recursion. With this I no longer have any issues:
https://github.com/neo4j/neo4j-python-driver/compare/1.6...liutec:1.6-buffer-capacity
The performance seems to have improved significantly based on this test:
import sys
import neo4j
import time
from neo4j.v1 import GraphDatabase, TransientError

print(sys.version_info)
print("neo4j-driver", neo4j.__version__)

neo4j_driver = GraphDatabase.driver(
    uri="bolt://localhost:7687",
    auth=("neo4j", "neo4j")
)
for i in range(1, 10):
    try:
        start = time.time()
        data = "x" * (i * 2 ** 20)
        cypher = "RETURN '{}' AS data".format(data)
        with neo4j_driver.session() as tx:
            records = tx.run(cypher)
            for record in records:
                if record['data'] != data:
                    print('ERROR')
        print('%d MB time = %.2f sec' % (i, time.time() - start))
    except TransientError as e:
        print(str(e))
        break

With your proposed changes and no recursion:
sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)
neo4j-driver 1.6.0a1
1 MB time = 0.10 sec
2 MB time = 0.18 sec
3 MB time = 0.32 sec
4 MB time = 0.41 sec
5 MB time = 0.49 sec
6 MB time = 0.66 sec
7 MB time = 0.83 sec
There is not enough memory to perform the current task. Please try increasing 'dbms.memory.heap.max_size' in the neo4j configuration (normally in 'conf/neo4j.conf' or, if you you are using Neo4j Desktop, found through the user interface) or if you are running an embedded installation increase the heap by using '-Xmx' command line flag, and then restart the database.

With just your proposed changes:
sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)
neo4j-driver 1.6.0a1
1 MB time = 0.12 sec
2 MB time = 0.36 sec
3 MB time = 0.67 sec
4 MB time = 1.15 sec
5 MB time = 1.70 sec
6 MB time = 2.42 sec
Traceback (most recent call last):
[...]
self.write(b[chunk_remaining:])
MemoryError

During handling of the above exception, another exception occurred:

SystemError: deallocated bytearray object has exported buffers

Process finished with exit code 0

@liutec
Thanks again for your help for fixing this bug.
The newly released 1.5.3 have the fix included already. Therefore I am closing this issue and consider this as solved. You are most welcome to improve any buffer usage around that code.
Zhen


Bot creation starting with Azure Bot doesn't populate initial publishing profile correctly

Describe the bug
The bot's app ID is present in Azure, but isn't populated when I create a publishing profile.
Version
Version:  1.4.0-nightly.236490.3735dd3
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Create Azure Bot resource
Open it in Composer

Create new Basic Conversation Bot.
Q:What's going on at the tail end, when I see the message "Created Successfully" and the spinner still goes for what seems like another minute?

Create LUIS key (login to Azure, twice)
Create publishing profile (login to Azure, again)

Almost immediately confused.

Q: Why isn't some of this information already pre-populated?
Guess: I created the bot resource, and I still need to create the Azure AD app registration and Web App resource.

Error: At the tail end of all the windows, it asks me to provide the Bot Channels Registration ID or create a new one!

Azure already created an Azure Bot app ID. Why is this missing from my bot's profile?

Expected behavior
Information that Azure created should be pre-populated, at least the first time I create a publish profile.
Screenshots
Waiting screen, where Composer seems to be doing...nothing?

Additional context

@emivers8 Looks like the key challenge here is the lack of clarity that project settings are setting up your local development environment and not the publishing profile and/or target.


Change default star sizes

Default star sizes are still wrong, and include the size 8 type O stars.  I know Chuck liked the default CHV colors and sizes, but I don't have a working XP virtual to drop CHV into just now.  However, I have been messing and find that 4, 3.5,3,2.5,2.5,2,1.5,1.5,1,1 works well for me.  In any event we need to get rid of the size 8 O's, and anything smaller than 1 is too small to click on.

fixed


Add ability to set arguments in shared definitions

Hi all.
Is it possible to add ability to create shared definitions with arguments like this?

Hi all.
Any changes?

I would say that using arguments with shared instances is not supposed to work. The only reason to use a shared instance is when you want the same object to be used by other services, so you should not be able to change the instance with arguments.
In your case, you should be using the factory pattern which returns a new instance according to the arguments you pass in the get method.

@nebulousGirl is indeed correct here.

Hi all.
Not sure I'm quite understand - if I want to have DataMapper which I want to be single instance with single instance of db-connection inside, how do you suggest to code it without shared instances? Create new DataMapper instance on each get call?

You can use  share when you want only want instance, but you can't use arguments when calling get for a shared service.
$container->add('service', 'SomeClass');
//This would work and effectively pass the argument to the class constructor
$container->get('service', ['someArg']);

$container->share('serviceShared', 'SomeClass');
//While this would not pass the argument to the constructor
//because it is a shared service (single instance) and it doesn't get reinstanciated
$container->get('serviceShared', ['someArg']);

@2nick you'd just set the database connection as shared and pass that in to a data mapper at the time you define the objects in the container, you don't need to pass arguments when you retrieve the data mapper.
$container
    ->share('Some\Database\Driver')
    ->withArgument(new RawArgument(['host', 'user', 'pass']))
;

$container
    ->share('Some\Data\Mapper')
    ->withArgument('Some\Database\Driver')
;

$container->get('Some\Data\Mapper');


Typescript

Hi,
do gherkin-testcafe support somewhat running tests written in typescript?
If it is possible do you have some sample configuration how it can be done?
Thanks

Hi @AdaskoTheBeAsT,
unfortunately, it is not possible to use this package with Typescript. This is mainly due to the custom behavior of the cucumber functions (Given, When, Then, ...) that would need custom typings.
It seems to me, that it is not a big problem to add these and to get this package running with Typescript. I think I will add this in the future.

Thanks - it really will be killer feature. I use testcafe with angular 6 and typescript is natural there ;) probably much more people use such combination.

@AdaskoTheBeAsT we are doing this at the moment and it's quite simple really.
The workaround is to just run tsc manually to compile your .ts into .js files and then provide them to testcafe ;)
You will have crappy code completion though due to missing typings, but at least you are writing TypeScript.

@AdaskoTheBeAsT of course
Here you go:
package.json
{
  ...
  "scripts": {
    ...
    "precucumber": "tsc -p tsconfig.e2e.json",
    // The trick here is to instruct gherkin-testcafe to use the transpiled .js files straigh from dist/out-tsc
    "cucumber": "gherkin-testcafe --specs ./e2e/cucumber/**/*.feature --steps dist/out-tsc/e2e/cucumber/**/*.steps.js -b chrome -e",
    "cucumber:headless": "gherkin-testcafe --specs ./e2e/cucumber/**/*.feature --steps dist/out-tsc/e2e/cucumber/**/*.steps.js -b 'chrome:headless --no-sandbox --disable-gpu-sandbox --disable-gpu' -e",
  },
  "private": true,
  "dependencies": {
    ...
  },
  "devDependencies": {
    "@types/node": "10.12.0",
    "gherkin-testcafe": "1.3.0",
    "cucumber": "4.2.1",
    "prettier": "1.14.3",
    "testcafe": "0.22.0",
    "testcafe-browser-provider-browserstack": "1.5.0",
    "testcafe-live": "0.1.3",
    "testcafe-reporter-html": "1.3.0",
    "typescript": "2.9.2"
  }
}

tsconfig.e2e.json (could be that this needs to be slightly altered with different params, but dont remember exacly which ones)
{
  "compileOnSave": false,
  "compilerOptions": {
    "sourceMap": true,
    "declaration": false,
    "moduleResolution": "node",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "noUnusedLocals": true,
    "skipLibCheck": true,
    "target": "es6",
    "lib": ["es2017", "dom"],
    "baseUrl": ".",
    "outDir": "dist/out-tsc/e2e",
    "module": "commonjs",
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "types": [
      "node"
    ]
  },
  "include": [
    "e2e/**/*.ts"
  ],
  "exclude": [
    "e2e/specs/**/*.ts",
    "node_modules",
    "dist",
    "tmp"
  ]
}

e2e/cucumber/features
Add your feature files under this folder
e2e/cucumber/steps/some-page.ts
import { Given, When, Then } from 'cucumber';

import SomePageObject from '../po/some-page';

// Note: in here you are then just 't' as a reference to the testcafe singleton
// Code completion on 't' is currently missing though :(
Given(/The user opens some page/, async t => {
    await SomePageObject.gotoPage();
});

When(/The user does something/, async t => {
    await SomePageObject.doSomething();
});

Then(/The user sees epic stuff/, async t => {
    await SomePageObject.isCorrect();
});

Works like a charm!

@DennisJaamann  Thanks a lot!  : )

Hi all
I see that this project now has a type definition which is great that augments cucumber. I am wondering how folks are using it?
Steps I've taken:

Installed @types/cucmber as the cucumber package no longer bundles types
Set "types": [ "gherkin-testcafe" ] in tsconfig.json to override the compiler finding cucumber's types and use the alternative typings instead.

The only issue I've found is in the typings file:
../../node_modules/gherkin-testcafe/index.d.ts:1:10 - error TS2305: Module '"cucumber"' has no exported member 'DataTable'.

import { DataTable } from "cucumber";

Updating this to import { TableDefinition } from "cucumber"; works

Hey @eddiegroves,
thanks for letting us know. I published v2.3.2 with a fix.
Internally, we do not use typescript for writing e2e tests (yet). Until then, this is more of an unofficial feature.

Hi guys, I don't want create new issue, could u please help me debug my problem. I got typeError when try run tests. I use Ubuntu bionic.
here is my e2e json:
{
  "compileOnSave": false,
  "compilerOptions": {
    "sourceMap": true,
    "declaration": false,
    "moduleResolution": "node",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "noUnusedLocals": true,
    "skipLibCheck": true,
    "target": "es6",
    "lib": ["es2017", "dom"],
    "baseUrl": ".",
    "outDir": "./src/.dist/",
    "module": "commonjs",
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "types": [
      "node",
      "gherkin-testcafe"
    ]
  },
  "include": [
    "src/BDD/**/*.ts"
  ],
  "exclude": [
    "node_modules",
    "dist",
    "tmp"
  ]
}

@maximkoev the error is not related to TypeScript - You've come across issue #36, there are suggested workarounds in that issue until the upstream issue is resolved (a new release of package c21e e.g v1.2.2)


New version: DBnomics v0.3.0

Registering package: DBnomics
Repository: https://github.com/s915/DBnomics.jl
Created by: @s915
Version: v0.3.0
Commit: cfa6a9adeb9ac273c7685620a1fc5fa1c45d3ba3
Reviewed by: @s915
Reference: https://github.com/s915/DBnomics.jl/commit/cfa6a9adeb9ac273c7685620a1fc5fa1c45d3ba3#commitcomment-44094311

Cancel auto-merging. Thanks.


Failed to create SVG react component

seems like babel expects a filename.

I've tested locally by setting the filename:filepath as an option past into babel at it seems to work fine.

Hello @ajith1234 ,I just try to reproduce on a starter project and I was not able to get the error its work just as intended, can you provide a repo with the error above?.
Its stated in the docs that filename is only required if a plugin requires it.

The filename is exposed to plugins. Some plugins may require the presence of the filename.

@babel/preset-react stated that it does not, as In @babel/preset-react the example of usage in node API is:
require("@babel/core").transformSync("code", {
  presets: ["@babel/preset-react"],
});

which is the same as I use (no filename option).
As its seems to work fine on my end we need to reproduce the error first before merge the fix,Thanks for the effort!

hey thanks for the quick response,
I cant share the repo as its a large TS project which im working on and having to migrate from webpack. This was the first time working with snowpack, babel plugins etc, so apologies for the noob questions or attempted solution.
Turns out I had a issue with having babel plugin for snowpack and yours leading to some conflicting issues with filename. I've just removed babel plugin as it dont think its actually even needed for my this project.


Error with relative paths

I am trying to read a mesh that is in a folder above the current one using the following code
import meshio

fname = "../square.msh"
points, cells, point_data, cell_data, field_data = \
     meshio.read(fname)

But I obtain the following error
Traceback (most recent call last):

  File "<ipython-input-5-c5965f4f0bf1>", line 1, in <module>
    runfile('/home/nguarinz/Documents/meshio_test.py', wdir='/home/nguarinz/Documents')

  File "/home/nguarinz/anaconda2/lib/python2.7/site-packages/spyder/utils/site/sitecustomize.py", line 705, in runfile
    execfile(filename, namespace)

  File "/home/nguarinz/anaconda2/lib/python2.7/site-packages/spyder/utils/site/sitecustomize.py", line 94, in execfile
    builtins.execfile(filename, *where)

  File "/home/nguarinz/Documents/meshio_test.py", line 6, in <module>
    meshio.read(fname)

  File "/home/nguarinz/anaconda2/lib/python2.7/site-packages/meshio/helpers.py", line 99, in read
    file_format = _extension_to_filetype[extension]

KeyError: '../square.msh'

Checking lin 99 of helpers.py (98 in the current branch), it seems to be because it takes the extension as all the characters after the first point. I suppose that something like this
extension = filename[filename.rfind("."):]

might work.
I am using meshio 1.11.2 with Python 2.7.13 (Anaconda 64 bits) in Linux Mint 18.2.

I was going to propose the more direct os.path.splitext, but saw that it had been commented out in the corresponding part of helpers.write in d2411cce1f4b6c08a1e3131187b49b20025243d5:
        # _, extension = os.path.splitext(filename)
        extension = '.' + filename.split(os.extsep, 1)[-1]

I think that that would have worked in your case, i.e. os.path.splitext('../square.msh')[-1] == '.msh'.

Ah, I see from the change to test/test_helpers.py in the same commit that it was to allow double-extensions like '.post.gz' for filename 'test.post.gz'.  In which case os.path.basename should fix the present issue.
        extension = os.extsep + os.path.basename(filename).split(os.extsep, 1)[-1]

Fixed by #201.


How to request authorization of the new public spacecraft resource?

[Azure orbital ground station]
I followed the steps mentioned in Tutorial: Downlink data from public satellites: https://learn.microsoft.com/en-us/azure/orbital/downlink-aqua
However, when I arrived the steps of getting authorization of the new public spacecraft resource:
"
4.click Next. If a Solutions page pops up, click Return to support request. click Next to move to the Additional details tab.
5.Under the Additional details tab, enter the following information
“
I found I cannot type anything in the Additional details as shown in the tutorial, and I can't click next.

The only thing showed in the Additional details are two icons for get expert help and ask our community. Is there any other ways to get this authorization?

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

ID: e70189d1-5eab-f552-144c-d4b2fcb5ef23
Version Independent ID: 62ab5f8e-8512-9479-2276-469d3bce7369
Content: Azure Orbital Ground Station - Downlink data from public satellites
Content Source: articles/orbital/downlink-aqua.md
Service: orbital
GitHub Login: @apoorvanori
Microsoft Alias: apoorvanori

@PlusplusJJ
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi, @AjayBathini-MSFT, @apoorvanori ,are there any updates about this issue? I still can't have a try to use the public spacecraft in the Azure orbital ground station.


Terminal Alt+Arrow wrongly mapped to Ctrl+Arrow

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.76.0
92da9481c0904c6adfe372c12da3b7748d74bdcb
x64
OS Version: Arch Linux x86_64 6.2.9-arch1-1

Steps to Reproduce:

Open terminal, start key logging tool such as showkey -a
Press Alt+Arrow key, see it is mapped to Ctrl+Arrow key.
For example: Alt+ RightArrow produces ^[[1;5C, where it should produce ^[[1;3C.

Note: need to have "terminal.integrated.sendKeybindingsToShell": true in config.json to allow shell to receive those keys.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Upgraded to  1.77.1, issue remains the same.

It's actually been this way since at least 2018:
https://github.com/xtermjs/xterm.js/blob/1c706e6fac3fdf1d19eb0c1dd50542043f033058/src/common/input/Keyboard.ts#L114-L151


Update to 2022.07.19

This is a new certificate release.
It consists of a single certificate file.
https://curl.se/docs/caextract.html

LGTM


update icepyx to v0.8.0

@tsnow03 noted the hub image still used v0.7.0, which doesn't get s3 urls for all products.

/condalock

Hmm, there seems to be an unrelated build failure at https://github.com/CryoInTheCloud/hub-image/actions/runs/6632090917/job/18017023795?pr=97#step:5:6493
   Could not solve for environment specs
  The following packages are incompatible
  ├─ jupyter-book 0.13.2  is installable and it requires
  │  └─ myst-nb >=0.13.1,<0.14.0 , which requires
  │     └─ myst-parser ~=0.15.2 , which requires
  │        └─ mdit-py-plugins ~=0.2.8 , which requires
  │           └─ markdown-it-py ~=1.0  with the potential options
  │              ├─ markdown-it-py 1.0.0 would require
  │              │  └─ attrs >=19.3,<21 , which can be installed;
  │              └─ markdown-it-py 1.1.0 would require
  │                 └─ attrs >=19.3,<22 , which can be installed;
  └─ referencing is not installable because it requires
     └─ attrs >=22.2.0 , which conflicts with any installable versions previously reported.

Seems to be an upstream issue, reported at https://github.com/executablebooks/jupyter-book/issues/2044.

/condalock

Thanks for the super fast review, @weiji14!


Bump Angular to 17.3.11 and Angular-CLI to 17.3.8

Bump Angular from 17.3.4 to 17.3.11 and Angular-CLI from 17.3.0 to 17.3.8
Description
This PR is a minor update to Angular 17 to ensure we are on the latest version of our major dependencies.  Includes:

Bump Angular from 17.3.4 to 17.3.11
Bump Angular-CLI from 17.3.0 to 17.3.8

Instructions for Reviewers

Verify all tests pass
Verify everything builds.  I tested it locally & found no issues
Verify UI works as normal.  I tested it locally and found no issues.  (I tested Search, Browse, Login/Logout, external import, complete submission process, and SSR.)

@atarix83 and @artlowel :  Any objections to this minor update of Angular & Angular CLI?  I noticed we are quite outdated on both..and it might be nice to get up-to-date prior to 8.0.

@atarix83 and @artlowel :  Nevermind! It appears this isn't as simple as it seemed.  Unit tests are not running.  So, we'll reschedule for post 8.0

@tdonohue
i'd try to exclude webpack update, for all the rest I'd not expect any issues

@atarix83 : I just pushed up the webpack update (a moment ago) as it seems to be required.   We were hitting this issue, and the problem seemed to be that we had two versions of webpack being brought in.  One by @angular-devkit/build-angular and one that we listed in package.json.
So, I synced those two versions and updated package.json to have the same webpack version as in the (updated) version of @angular-devkit/build-angular.
I tested it locally and found yarn test now succeeds.  I also did a full yarn build:prod; yarn serve:ssr and verified everything seems to work fine & SSR works, etc.
So, I think this is ready for review now by you and @artlowel if one of you finds time.

@atarix83 : I was able to pull your small commit into this PR.  See 472fa1c47348514de4006817d74ccd847d468c13 which is now part of this PR.


Partial rake set update

Rake does not pick up correct file when there are multiple files of the same name from different userids.
rake “set_piece_status_for_partial_piece_csvfile[RG10_640_a.csv,Online]” —trace will need to have an option to set userid.

Resolved, closing