(#3) check_powershell resource

Create the new the check_powershell type
Create the new check_powershell provider to use the insync? feature
Use retry mechanism in the check_powershell provider
Update the check_powershell provider with debugging messages
Added spec tests
Fixed rubocop errors

check_powershell {'$PSVersionTable.PSVersion':
}

Results- Success

Results- Failure
check_powershell {'invalid$PSVersionTable.PSVersion':
  retries => 3
}

 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.


[tests-only] Bump WEB_COMMITID to end of 2020-01-05

PR https://github.com/owncloud/web/pull/4589 passes without needing to add the expected-failures that I had to had in the 3rd commit of this PR. Need to investigate why there is a difference when running the web tests from the OCIS repo.

PR https://github.com/owncloud/web/pull/4589 passes without needing to add the expected-failures that I had to had in the 3rd commit of this PR. Need to investigate why there is a difference when running the web tests from the OCIS repo.

I raised issue #1197
See PR #1196 for the next attempt at this.

I raised issue #1197
See PR #1196 for the next attempt at this.


Add 4 test cases for Url.json PathItems

Add 4 test cases for Url.json PathItems
Improve AutoRest Swift to pull query param from Client property and optional Cilent Property
Refactor: Remove adding "" in query param when required query params is empty

/azp where


chore: define some pos dto and enums

@sicpama-huy This is where the pos interfaces are now defined


Invalid child element 'classificationType'

Hello, thanks for the effort you put in this package!
I started playing around, in order to send an invoice, and sent the following to AADE

Example XML
  The following xml was send to AADE
<?xml version="1.0"?>
<InvoicesDoc xmlns="http://www.aade.gr/myDATA/invoice/v1.0">
    <invoice>
        <issuer>
            <vatNumber>111111</vatNumber>
            <country>GR</country>
            <branch>0</branch>
        </issuer>
        <counterpart>
            <vatNumber>222222</vatNumber>
            <country>GR</country>
            <branch>0</branch>
        </counterpart>
        <invoiceHeader>
            <series/>
            <aa/>
            <issueDate>2022-06-09</issueDate>
            <invoiceType>1.1</invoiceType>
            <currency>EUR</currency>
            <dispatchDate>2022-06-09</dispatchDate>
            <dispatchTime>08:57:03</dispatchTime>
        </invoiceHeader>
        <paymentMethods>
            <paymentMethodDetails>
                <type>3</type>
                <amount>13.37</amount>
                <paymentMethodInfo/>
            </paymentMethodDetails>
        </paymentMethods>
        <invoiceDetails>
            <lineNumber>1</lineNumber>
            <quantity>2</quantity>
            <measurementUnit>2</measurementUnit>
            <invoiceDetailType>1</invoiceDetailType>
            <netValue>15</netValue>
            <vatCategory>1</vatCategory>
            <vatAmount>2</vatAmount>
            <incomeClassification>
                <classificationType>E3_561_003</classificationType>
                <classificationCategory>category1_3</classificationCategory>
                <amount>16</amount>
            </incomeClassification>
        </invoiceDetails>
        <invoiceSummary>
            <totalNetValue>0</totalNetValue>
            <totalVatAmount>0</totalVatAmount>
            <totalWithheldAmount>0</totalWithheldAmount>
            <totalFeesAmount>0</totalFeesAmount>
            <totalStampDutyAmount>0</totalStampDutyAmount>
            <totalOtherTaxesAmount>0</totalOtherTaxesAmount>
            <totalDeductionsAmount>0</totalDeductionsAmount>
            <totalGrossValue>0</totalGrossValue>
        </invoiceSummary>
    </invoice>
</InvoicesDoc>

One of the errors I got in the response was that the income classification had the wrong namespace and that the icls namespace was missing as an attribute.
Code:101 - Message:Line:1.Position:871.
The element 'incomeClassification' in namespace 'http://www.aade.gr/myDATA/invoice/v1.0' has invalid child element 'classificationType' in namespace 'http://www.aade.gr/myDATA/invoice/v1.0'.
List of possible elements expected: 'classificationType, classificationCategory' in namespace 'https://www.aade.gr/myDATA/incomeClassificaton/v1.0'.

Now, I've never really used xml before, so not sure if what I understood is correct. I noticed that the incomeClassification namespace was missing as an attribute, and that the names of the fields had the name space icls in front.
From source example
<InvoicesDoc xmlns="http://www.aade.gr/myDATA/invoice/v1.0" xmlns:icls="https://www.aade.gr/myDATA/incomeClassificaton/v1.0">

and
<incomeClassification>
  <icls:classificationType>E3_561_001</icls:classificationType>
  <icls:classificationCategory>category1_2</icls:classificationCategory>
  <icls:amount>1000.00</icls:amount>
</incomeClassification>

After, I added the icls attribute and changed the IncomeClassificationType struct to
type IncomeClassificationType struct {
	ClassificationType     string `xml:"icls:classificationType"`
	ClassificationCategory string `xml:"icls:classificationCategory"`
	Amount                 string `xml:"icls:amount"`
	ID                     *byte  `xml:"icls:id"`
}

I managed to send an invoice.
If this is something acceptable, I could draft up a PR. How does that sound?
Ps: Is there a reason the Amount field in IncomeClassificationType is string instead of float64 like the rest?

@stefpap please do submit a PR 👍🏾 what you described in the ticket makes total sense to me


Remove unnecessary views folder

What is the context of this PR?
Remove the main folder and move the views subdirectory up one level. Rename 'root' routes to 'main'.
How to review
Check if tests continue to pass.

@benjefferies @jonnyshaw89 - I'm just removing and renaming a file chaps, not looking for feedback on anything else.

👍


Will collins notify slack

Squashed version of #246
Adding slack support to collins-notify
@byxorna @maddalab @CaptPhunkosis

Aside from those small points, looks great!

Awesome, lgtm!


Pulling an acorn image by SHA does not work

This issue is easily explained using an example:
$ acorn pull ghcr.io/acorn-io/library/hello-world@sha256:1a6c64d2ccd0bb035f9c8196d3bfe72a7fdbddc4530dfcb3ab2a0ab8afb57eeb
  ✗  ERROR:  repository can only contain the characters `abcdefghijklmnopqrstuvwxyz0123456789_-./`: acorn-io/library/hello-world@sha256

It works by just using a "normal" tag like latest.
Affected Version

Tested with v0.6.0
Tested with build from current HEAD a1e6f2c7

Investigation

Ref: https://github.com/acorn-io/acorn/issues/1014
Tracked it down to the usage of name.NewTag() in various places, which makes this fail
Switching from name.NewTag() to name.ParseReference() makes this work, but also causes the sha256 to be treated as an Acorn image tag, which will be used in name.NewTag() calls in many other places (acorn images, acorn tag, ...), so we'd need a larger refactoring

Alignment

make image pull by SHA work (replacing NewTag() with ParseReference() in some places)
store image with a tag that has no tag (whut? O.o), just registry/repo
but: whenever a user input does not contain a tag or SHA, default to latest tag instead of using some untagged image

this affects e.g. acorn run, acorn tag, acorn push, ...
:exclamation: This has to be handled correctly by the tag normalize function in that it should not automatically default it to the latest tag

Make sure, that pull x@sha256 -> tag y x does not remove the tag x from the image (which could happen due to normalization, setting it to x:latest, which will subsequently be replaced by y)

Checklist

[x] acorn run foo/bar@sha256:xyz works the same as acorn run foo/bar:tag
[x] acorn pull foo/bar@sha256:xyz works and stores image internally with a tag that is only registry/repo and that is not treated as :latest
[x] acorn images lists an image pulled as per above command only when --all is used or if matched by digest
[x] acorn images foo/bar@sha256:xyz works by matching the digest
[ ] above applies to acorn image rm
[x] acorn tag does not remove the reg/repo tag (i.e. does not consider it as latest)
[ ] above applies to acorn push
[ ] general: tag normalization (i.e. correcting tag input from <none> to :latest) should only happen on user input, not internally

Open Discussion
Q: How should we treat user inputs referencing an image by digest, e.g. acorn images foo/bar@sha256:12345...
A1: It should only return the one variant that has no tag.
A2: It should return all variants that match the digest (which is equal to the acorn-internal image ID)
-> This also affects e.g. acorn image rm.
In my opinion it's surely A2 as that's intuitive, but I remember some discussion where we decided for A1 at first.
@ibuildthecloud what's your take on this?
I'd propose:

Input: foo/bar -> Output: variant with tag latest
Input: foo/bar:mytag -> Output: variant with tag mytag
Input: foo/bar --all -> Output: all variants for that repo, with or without tags (i.e. prefix match)
Input: foo/bar@sha256:12345... -> Output: all tagged/untagged variants matching that digest(/image id)

Changes

Change the tagging logic: adding a new tag to an image will replace any existing "reg/repo" tag item in the list, meaning that "real" tags take precedence
Untagged (i.e. images with reg/repo:) should always be shown, only images with no tag items at all (i.e. not even reg/repo assignment) should be hidden (and revealed with the --all flag)

acorn version v0.6.0-132-gdc73d55a+dc73d55a
Not able to pull using SHA:
%acorn pull sangeetha/myfirstacorn@sha256:112d6ad308bc2ec0ef35de51a8d9199933300654b978a5c55863063286c3a279
  ✗  ERROR:  websocket: close 1011 (internal server error): unexpected media type for ImageIndex(): application/vnd.docker.distribution.manifest.v2+json; call Image() instead
%acorn pull docker.io/sangeetha/myfirstacorn@sha256:112d6ad308bc2ec0ef35de51a8d9199933300654b978a5c55863063286c3a279
  ✗  ERROR:  websocket: close 1011 (internal server error): unexpected media type for ImageIndex(): application/vnd.docker.distribution.manifest.v2+json; call Image() instead

docker pull of the same image works:
docker pull sangeetha/myfirstacorn@sha256:112d6ad308bc2ec0ef35de51a8d9199933300654b978a5c55863063286c3a279
docker.io/sangeetha/myfirstacorn@sha256:112d6ad308bc2ec0ef35de51a8d9199933300654b978a5c55863063286c3a279: Pulling from sangeetha/myfirstacorn
18d4b176bd87: Pull complete
Digest: sha256:112d6ad308bc2ec0ef35de51a8d9199933300654b978a5c55863063286c3a279

The problem was that I was attempting to do the pull with manifest digest which is what I get from docker hub UI.
When i attempt to do the pull with  index digest as suggested by @iwilltry42 , I am able to pull successfully.
For getting the  index digest , I had to do  crane digest or get the digest from acron image -o json for the local image that was built and pushed to external registry.
 acorn build -t docker.io/sangeetha/mytestshaimage:v0.0.4 --push .
...
8d71689363867a90e571a5ecc013c4a74006fd8f8afae14b40d0c640818f5774

 acorn pull index.docker.io/sangeetha/mytestshaimage@sha256:8d71689363867a90e571a5ecc013c4a74006fd8f8afae14b40d0c640818f5774
 [55583241/55583241] █████████████████████████████████████████████████ 100% | 0s
sangeethahariharan@Sangeethas-MBP example1 % acorn images
REPOSITORY                                 TAG       IMAGE-ID
index.docker.io/sangeetha/mytestshaimage   <none>    8d7168936386

Acorn image that gets pulled successfully, But it looses it tag.

@iwilltry42 Had 2 followup questions:

1. Should we be able to get the tag when images gets pulled using sha digest ?
2. Wanted to confirm that we do not want to  support pulling images using manifest digest that shows up in Dockerhub UI.


HDR video playback

Hi,
I didnt have iphone 12 before, well still dont have but just realized when i import a hdr video to avplayer, the colors are dull, not converted to standard colors. how can i play the video in the same colors with the Photos app?
videoPlayer?.currentItem?.appliesPerFrameHDRDisplayMetadata = true
is not working.
Checklist

[x] I've read the README
[x] I've searched for existing GitHub issues

"var nsItem: AVPlayerItem = AVPlayerItem(asset: AVAsset(url: StoredSliderValues.incomingVideoURL))"
was causing the problem. It was out of the func, still dont get why..


Update mozilla-firefox extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

This way could work @Laptop765, I'm not sure if there is a better way, but I guess it's hard to do anything better than this. @crisboarna, what do you think?

We could parse profiles.ini as a separate fix but I'm not sure if that's better or not 😅


Include server time with game state returned by the board stream endpoint

I'm trying to implement a custom client for playing games, and it occurred to me that it would be useful if there was a server time included with each object returned by the board stream API.
This would allow updating the client game clock with the correct time regardless of any hiccups / delay in getting the message.
GAME_TIME = REPORTED_GAME_TIME + (NOW - LICHESS_SERVER_TIME)

the correct repository for this is ornicar/lila


quickfix: always set QF list after a test run

otherwise we wouldn't remove obsolete errors after fixing a test.

Thanks for the fix!


Container light occlusion

#515
Requires https://github.com/space-wizards/RobustToolbox/pull/967
Allows containers to be defined as light occluding or not.
Current issue is I have no idea how to implement this for inventory slots with the way they're currently defined and container code is a big shitty mess.

will check when I'm home

This gonna get updated?  It's tricky. From my understanding, the best route is to have a StorageComponent check its Parent if any, and if its parent occludes light, it should too. *circumstances permitting ofc, if closet and door closed etc.)

I basically implemented the exact same system in-engine for sprites now because it was necessary to optimize sprite rendering. Probably good for inspiration/hooking into.
https://github.com/space-wizards/RobustToolbox/blob/99933499b787a7607d217ceaadfb0dfd384ae750/Robust.Client/GameObjects/EntitySystems/ContainerSystem.cs#L67

tried it again and still don't know how to solve this, it's beyond my knowledge


[DnD_5e Charcter Sheet] Add Spells Prepared count to spell table in spellbook

Changes / Comments
This Pull Request adds a new row, "Prepared", to the spell table on the Spellbook tab of the DnD_5e character sheet. Included are the HTML changes to add the row to the table, and javascript changes to calculate and set the attribute whenever a spells preparation status is changed.
The code only listens to changes on spell levels 1-9, since changes to cantrips do not impact the number of spells prepared.
The javascript code starts by checking which field on the spell was changed, and only continues if it was the "prepared" field.
The javascript code also runs when the sheet is first loaded, to ensure the spells_prepared_count attribute exists for all users and does not generate an error.
The fields in the Prepared row of the spell table are auto-calced.
This is informational only, and does not impose any restrictions on how many spells a user can mark as Prepared.
Only classes that prepare their spells (Cleric, Druid, Paladin, Wizard) are updated by the code. However, the field is set for all classes, to match the pattern of the Can Prepare row above.

Sheet name: DnD 5e
Is this a bug fix? No
Does this add functional enhancements (new features or extending existing features)? Yes, adds a row to the spells table to display the count of spells that have been marked as prepared.
Does this add or change functional aesthetics (such as layout or color scheme)? The only layout change is to add a new row to the spells table, which shifts the rest of the page down very slightly.
If changing or removing attributes, what steps have you taken, if any, to preserve player data? No attributes have been changed or removed. To handle having new fields that are autocalc'ed from a new attribute, the javascript code runs when the sheet is first loaded to populate the attribute.
If this is a new sheet, did you follow Building Character Sheets standards? N/A, not a new sheet

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Handling string interpolation in input validation/sanitization sniff

Closes #182

+1. Could this be merged? There is a conflict, but only in the test file.

I resolved the conflict.
/five @shadyvb


ffmpeg 7.1 build fails with" ‘AVFilterLink’ {aka ‘struct AVFilterLink’} has no member named ‘frame_rate’"

Hey!
When building against Arch's upcoming ffmpeg 7.1/x265 4.0 release [1] I'm getting the following output.
==> Starting build()...
The Meson build system
Version: 1.5.2
Source dir: /build/wf-recorder/src/wf-recorder-0.5.0
Build dir: /build/wf-recorder/src/build
Build type: native build
Project name: wf-recorder
Project version: 0.5.0
C compiler for the host machine: cc (gcc 14.2.1 "cc (GCC) 14.2.1 20240910")
C linker for the host machine: cc ld.bfd 2.43.0
C++ compiler for the host machine: c++ (gcc 14.2.1 "c++ (GCC) 14.2.1 20240910")
C++ linker for the host machine: c++ ld.bfd 2.43.0
Host machine cpu family: x86_64
Host machine cpu: x86_64
Program git found: NO
Found pkg-config: YES (/usr/bin/pkg-config) 2.1.1
Run-time dependency wayland-client found: YES 1.23.1
Run-time dependency wayland-protocols found: YES 1.37
Run-time dependency libpulse-simple found: YES 17.0
Did not find CMake 'cmake'
Found CMake: NO
Run-time dependency libpipewire-0.3 found: NO (tried pkgconfig and cmake)
Message: Using default audio backend: auto
Run-time dependency libavutil found: YES 59.39.100
Run-time dependency libavcodec found: YES 61.19.100
Run-time dependency libavformat found: YES 61.7.100
Run-time dependency libavdevice found: YES 61.3.100
Run-time dependency libavfilter found: YES 10.4.100
Run-time dependency libswresample found: YES 5.3.100
Run-time dependency threads found: YES
Run-time dependency gbm found: YES 24.2.4-arch1.1
Run-time dependency libdrm found: YES 2.4.123
Configuring config.h using configuration
Program wayland-scanner found: YES (/usr/bin/wayland-scanner)
Message:
----------------
wf-recorder 0.5.0
----------------
Default audio backend: pulse
  - pulse: true
  - pipewire: false
Build targets in project: 2

wf-recorder 0.5.0

  User defined options
    buildtype: plain
    prefix   : /usr
    pulse    : enabled

Found ninja-1.12.1 at /usr/bin/ninja
WARNING: Running the setup command as `meson [options]` instead of `meson setup [options]` is ambiguous and deprecated.
ninja: Entering directory `build'
[1/19] Generating 'proto/libwl_protos.a.p/xdg-output-unstable-v1-protocol.c'
[2/19] Generating 'proto/libwl_protos.a.p/xdg-output-unstable-v1-client-protocol.h'
[3/19] Generating 'proto/libwl_protos.a.p/linux-dmabuf-unstable-v1-protocol.c'
[4/19] Generating 'proto/libwl_protos.a.p/wlr-screencopy-unstable-v1-protocol.c'
[5/19] Generating 'proto/libwl_protos.a.p/wlr-screencopy-unstable-v1-client-protocol.h'
[6/19] Generating 'wf-recorder.p/xdg-output-unstable-v1-client-protocol.h'
[7/19] Generating 'proto/libwl_protos.a.p/linux-dmabuf-unstable-v1-client-protocol.h'
[8/19] Generating 'wf-recorder.p/linux-dmabuf-unstable-v1-client-protocol.h'
[9/19] Generating 'wf-recorder.p/wlr-screencopy-unstable-v1-client-protocol.h'
[10/19] Compiling C object wf-recorder.p/src_averr.c.o
[11/19] Compiling C object proto/libwl_protos.a.p/meson-generated_wlr-screencopy-unstable-v1-protocol.c.o
[12/19] Compiling C object proto/libwl_protos.a.p/meson-generated_xdg-output-unstable-v1-protocol.c.o
[13/19] Compiling C object proto/libwl_protos.a.p/meson-generated_linux-dmabuf-unstable-v1-protocol.c.o
[14/19] Linking static target proto/libwl_protos.a
[15/19] Compiling C++ object wf-recorder.p/src_audio.cpp.o
[16/19] Compiling C++ object wf-recorder.p/src_frame-writer.cpp.o
FAILED: wf-recorder.p/src_frame-writer.cpp.o
c++ -Iwf-recorder.p -I. -I../wf-recorder-0.5.0 -I/usr/include/libdrm -fdiagnostics-color=always -D_GLIBCXX_ASSERTIONS=1 -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=c++17 '-DWFRECORDER_VERSION="0.5.0"' -Wno-deprecated-declarations -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=3 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wp,-D_GLIBCXX_ASSERTIONS -g -ffile-prefix-map=/build/wf-recorder/src=/usr/src/debug/wf-recorder -flto=auto -D_REENTRANT -pthread -MD -MQ wf-recorder.p/src_frame-writer.cpp.o -MF wf-recorder.p/src_frame-writer.cpp.o.d -o wf-recorder.p/src_frame-writer.cpp.o -c ../wf-recorder-0.5.0/src/frame-writer.cpp
../wf-recorder-0.5.0/src/frame-writer.cpp: In member function ‘void FrameWriter::init_video_filters(const AVCodec*)’:
../wf-recorder-0.5.0/src/frame-writer.cpp:374:53: error: ‘AVFilterLink’ {aka ‘struct AVFilterLink’} has no member named ‘frame_rate’
  374 |     this->videoCodecCtx->framerate = filter_output->frame_rate; // can be 1/0 if unknown
      |                                                     ^~~~~~~~~~
[17/19] Compiling C++ object wf-recorder.p/src_pulse.cpp.o
[18/19] Compiling C++ object wf-recorder.p/src_main.cpp.o
ninja: build stopped: subcommand failed.

Thanks for maintaining this project!
[1] https://archlinux.org/todo/ffmpegx265-rebuild/

Does #279 help?

Why yes, it does! I missed that PR, thanks so much for submitting it :)


Training SimCLR on 1-gpu with VISSL tutorial throws an error on training

When running Training SimCLR on 1-gpu with VISSL tutorial, I tried to kick-off the training using the following command:
!python3 run_distributed_engines.py \
    hydra.verbose=true \
    config=quick_1gpu_resnet50_simclr \
    config.DATA.TRAIN.DATA_SOURCES=[synthetic] \
    config.CHECKPOINT.DIR="./checkpoints" \
    +config.HOOKS.TENSORBOARD_SETUP.USE_TENSORBOARD=true

After outputting various configuration settings, the code has encountered an error while building a model:
INFO 2021-05-04 02:13:10,704 train_task.py: 419: Building model....
INFO 2021-05-04 02:13:10,705 resnext.py:  63: ResNeXT trunk, supports activation checkpointing. Deactivated
INFO 2021-05-04 02:13:10,705 resnext.py:  83: Building model: ResNeXt50-1x64d-w1-BatchNorm2d
INFO 2021-05-04 02:13:11,316 model_helpers.py: 138: Using SyncBN group size: 1
INFO 2021-05-04 02:13:11,316 model_helpers.py: 153: Converting BN layers to PyTorch SyncBN
WARNING 2021-05-04 02:13:11,317 model_helpers.py: 159: Process groups not supported with PyTorch SyncBN currently. Traning will be slow. Please consider installing Apex for SyncBN.
INFO 2021-05-04 02:13:11,322 train_task.py: 378: Initializing model from:
INFO 2021-05-04 02:13:11,323 util.py: 241: Broadcasting checkpoint loaded from
Traceback (most recent call last):
  File "run_distributed_engines.py", line 194, in <module>
    hydra_main(overrides=overrides)
  File "run_distributed_engines.py", line 179, in hydra_main
    hook_generator=default_hook_generator,
  File "run_distributed_engines.py", line 123, in launch_distributed
    hook_generator=hook_generator,
  File "run_distributed_engines.py", line 166, in _distributed_worker
    process_main(cfg, dist_run_id, local_rank=local_rank, node_id=node_id)
  File "run_distributed_engines.py", line 159, in process_main
    hook_generator=hook_generator,
  File "/usr/local/lib/python3.7/dist-packages/vissl/engines/train.py", line 102, in train_main
    trainer.train()
  File "/usr/local/lib/python3.7/dist-packages/vissl/trainer/trainer_main.py", line 155, in train
    self.task.prepare(pin_memory=self.cfg.DATA.PIN_MEMORY)
  File "/usr/local/lib/python3.7/dist-packages/vissl/trainer/train_task.py", line 634, in prepare
    self.base_model = self._build_model()
  File "/usr/local/lib/python3.7/dist-packages/vissl/trainer/train_task.py", line 462, in _build_model
    model = self._restore_model_weights(model)
  File "/usr/local/lib/python3.7/dist-packages/vissl/trainer/train_task.py", line 399, in _restore_model_weights
    append_prefix=append_prefix,
  File "/usr/local/lib/python3.7/dist-packages/vissl/utils/checkpoint.py", line 404, in init_model_from_weights
    state_dict_key_name in state_dict.keys()
AttributeError: 'NoneType' object has no attribute 'keys'

Upon inspection, I noticed that even though I didn't specify a path to pre-trained model weights, the code was still trying to call _restore_model_weights function, thus ultimately causing the NoneType error in init_model_from_weights function.
Here's the output excerpt showing the settings for WEIGHTS_INIT key:
           'WEIGHTS_INIT': {'APPEND_PREFIX': '',
                            'PARAMS_FILE': '',
                            'REMOVE_PREFIX': '',
                            'SKIP_LAYERS': ['num_batches_tracked'],
                            'STATE_DICT_KEY_NAME': 'classy_state_dict'}},

Hi @gurlina , thank you for reaching out. Take a look at the https://github.com/facebookresearch/vissl/issues/248#issuecomment-805357468 on this :) it is a known issue that's fixed in VISSL master but the new packages haven't been released yet.


Image component issues

height & width need to be made optional
set default size for height & width

resolved with https://github.com/warungpintar/renceng-native/pull/43


ruff: pytest style

References and relevant issues
Followup on #6840 (though independent).
Description
Add the PT rule to check use of fixtures and other pytest things. It fixed some formal stuff, but also caught quite a few mistakes.
The two main changes are:

fixtures naming conventions (leading _ is for those that don't return an object)
all the pytest.raises(ValueError) needed an appropriate match= parameter to be less vague

These failures are confusing... @Czaki any ideas? ^^'


Makes subnets configurable.

@rajatchopra PTAL

Addresses #30

Any chance this will be merged within a day or so?

@brenton on the way. A rebased version by the evening.

Awesome, thanks Rajat!

Rebased.

Hand tested, works.

@sdodson, can you build the latest openshift-sdn package today for beta2?


feat: Upgrade apisix-base to openssl3

Build is failing because building apisix expects the shared library with openssl111 [1]

https://github.com/apache/apisix/blob/3b16fce060fbac39ee054784213a6bc81df526fc/Makefile#L64

CI Passing now. Deleting this PR and creating a new PR with squashed commits


Is it ready to use?

Can I use Phug for production or I should stay on pug-php?

Hi, no sorry, we cannot say that. Phug is about half ready. Our goal is to match pugjs specifications then implement extensions dedicated to PHP. For now, we pass 36% of the pugjs unit tests. This means you lack many features in using Phug such as yield or dynamic mixins.

Hi, 100% reached, we will soon publish a candidate release of Phug. Also pug-demo. So you can start to implement it in your project. I will also soon publish a branch 3.0.x for preparing pug-php 3.0 that will use Phug but keep a API close from pug-php 2.0.

Hi, issues in phug-php/renderer will soon be closed to centralize all issues in phug-php/puhg.
So this issue will disappear. If you need something, please go to https://github.com/phug-php/phug/issues
About the renderer, you can use it, just be aware, we run into trouble for recursive mixins (we discover lately because it's not part of the pugjs unit tests). So we need to refactorize a lot our mixin system in phug and it will postpone the release but it should not cause breaking changes. If you do not have recursive mixins, you should be able to use the renderer as it safely.


Endpoint to interrogate RW permissions of a given target-mount (db+user combination).

From @zeeiyerWork on September 23, 2017 22:28
In support of https://github.com/slamdata/slamdata/issues/2011:

We need to add an API that can return to the front-end, whether the target store specified in a mount supports READ or READ/WRITE operations for the user/pwd combination.

cc: @kRITZCREEK
Copied from original issue: slamdata/slamdata-backend#244

From @kRITZCREEK on September 25, 2017 11:18
I think @djspiewak suggested to put this into Quasar, since it needs to be implemented in the connectors (Can't just be a slamdata-backend thing).


added ProtonMail

proof: https://protonmail.com/tor
Note: I'm pretty sure the onion service is only for mail.protonmail.com.

Thanks :+1:


Cant convert by yaml to cbor with bstr as value in map

Steps to reproduce:
CDDL

map = {
"bstr_test" : bstr
}

YAML

bstr_test:
bstr: "12345678aabbccdd"

Commands
python ./zcbor.py -c mytest.cddl convert -i mytest.yaml -o mytest.cborhex -t map
Error
__main__.CddlValidationError: Data did not decode correctly (//TSTR'bstr_test' => BSTR: Wrong type of {'bstr': '12345678aabbccdd'}, expected <class 'bytes'>)

Thanks for the report! Fixed in #173

Thanks for the quick fix!


drop support for openssl 1.0 and 1.1

dependencies on openssl 1.0 have already been eliminated and the port will be removed.
openssl 1.1.1 is also EOL by more than 1 year so I'm working to eliminate dependency on that.
Both versions are hardcoded to USES=ssl support so ravenadm should remove reference to them.

okay, no active port relies on openssl 1.1.1 now.
Lagrange had to be marked broken because it won't build with openssl 3.3.2.  That's the only one.

lagrange has been restored.  openssl 11 had been hardcoded into it.


Fix the problem caused by strange branch names that contain special characters

What this PR dose

Replace label of SCM ref name with field indexer.
Remove branch name validation

Why we need it
Please see:

#104
#397

Which issue this PR fixes
Fix #397
How to test it
Docker images for test:
johnniang/devops-apiserver:dev-v3.2.1-rc.1-fe5d958
johnniang/devops-controller:dev-v3.2.1-rc.1-fe5d958

Create Multi-branch Pipeline, using repo https://gitlab.com/johnniang/jenkinsfile-multi-demo.
Click Branches tab and see all branches.

Click into branch naming 中文测试分支

Please note that KubeSphere console didn't escape the branch name, so we cannot click branch naming feat/featureA util console fixes it:

/kind bug
/cc @kubesphere/sig-devops
/cc @mangoGoForward

also have the escape problem

Select the feature branch. After clicking run, the task does not start running. If you want to check the details and report an error, I think it may still be a problem of escape.

/retitle Fix the problem caused by strange branch names that contain Chinese or special characters

hi @luoyuctl thanks for helping to test it. Please see

/retitle Fix the problem caused by strange branch names that contain special characters

in v1alpha3 version apis，i did not see this api in v1alpha3 version，please add it

and v1alpha3 is missing a lot apis .....

We're using multiple versions at the same time. Some APIs, we don't have any reasons to upgrade them due to no changes. In other cases, we add some new APIs into v1alpha3.
So, IMO, we didn't miss any APIs.

/cherrypick release-3.2

still have this problem ,after my investigation, it may be that devops-apiserver is not updated to cause this error, which is normal when using johnniang/devops-apiserver:dev-v3.2.1-rc.1-fe5d958 images.
I tested using a package packed according to the latest version of release3.2.

and new issue found

and new issue found @LinuxSuRen @JohnNiang

Please help us fire a new issue for this problem, thanks.

and new issue found @LinuxSuRen @JohnNiang

Please help us fire a new issue for this problem, thanks.

https://github.com/kubesphere/ks-devops/issues/400


Error running 0005_regenerate_user_address_hashes migration

Found a bug? Please fill out the sections below.
Issue Summary
I have a legacy project with django-oscar==1.5.2 and I'm trying to update it to 2.0. I'm getting the following error when trying to run the migration 0005_regenerate_user_address_hashes from the Address app:
django.db.utils.IntegrityError: duplicate key value violates unique constraint "address_useraddress_user_id_4e104dbf168846a9_uniq"
DETAIL:  Key (user_id, hash)=(445,<PHONE_NUMBER>) already exists.

This error is only happening with a specific user who has two addresses, one of their addresses already has the new hash that the migration is generating, so I figured out the error is due the Address app has the unique_together = ('user', 'hash') attribute in his Meta class.
So, what is the best way to fix this? Should I fork the Address app and modify that migration?
Technical details

Python version: 3.6.10
Django version: 2.0.13
Oscar version: 2.0

If it's only one address affected then it's probably easier just to manually edit that address to change its hash, or remove a duplicate.
We use Github only for managing bugs and feature requests, and not for support - if you need further help please use Stack Overflow using the django-oscar tag, or joining our slack workspace or mailing list and ask a question there. Thanks.


ISSUE-271 Creating schema throws error in postgres due to wrong type assocaited with schema_version_state#details field

ISSUE-271 Creating schema throws error in postgres due to wrong type associated with schema_version_state#details field
Closes #271

+1 the changes looks good and able to add schemas with postgres db. The travis build is failing but seems unrelated.

travis building is getting failed because log size exceeds travis limit viz 4MB.


Build: Build offsets lib as OBJECT instead of STATIC. Use cmake to ge…

Build offsets lib as OBJECT instead of STATIC. Use cmake to get generated path of lib instead of predefined absolute path. This is needed to eventually be able to have an architecture-out-of-tree build. The path "OFFSETS_O_PATH" is not generated for sources out-of-tree and therefore must be replaced by the cmake generator $<TARGET_OBJECTS:offsets>.
The following lines are needed while we are using a Cmake version < 3.12.3:
+target_include_directories(.....
+target_compile_options(...
+target_compile_definitions(....
This is due to the cmake feature "Linking Object Libraries" not supported until 3.12: https://cmake.org/cmake/help/v3.12/command/target_link_libraries.html#linking-object-libraries
Signed-off-by: Klaus Petersen<EMAIL_ADDRESS>

Codecov Report

Merging #11309 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11309      +/-   ##
==========================================
+ Coverage   51.45%   51.57%   +0.12%
==========================================
  Files         216      216
  Lines       27652    27665      +13
  Branches     6936     6937       +1
==========================================
+ Hits        14227    14269      +42
+ Misses      10672    10647      -25
+ Partials     2753     2749       -4

Impacted Files
Coverage Δ

include/logging/log_core.h
100% <0%> (ø)
:arrow_up:

include/logging/log_msg.h
90.9% <0%> (+1.81%)
:arrow_up:

misc/printk.c
88.55% <0%> (+3.09%)
:arrow_up:

subsys/logging/log_core.c
74.56% <0%> (+3.46%)
:arrow_up:

subsys/logging/log_output.c
72.33% <0%> (+8.73%)
:arrow_up:

Continue to review full report at Codecov.

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


Merging refactored physics code into develop

NOTE: This branch is LOCKED. This is because it was based on v2.1.2 not develop (which is like v2.1.3) so if we "Update branch" we would bring in additional untested non-zero-diff changes.

Scott will work from latest develop.


Allow to load ".set" files, which aggregate a set of coverage files

This patch allows loading the result of the intersection/union/subtraction of a set of coverage files. A (text) ".set" file consists of a line with the desired operations and the list of coverage files to aggregate.

can you please add a unit test for that ?

ping @zxgio

Hi @oddcoder , sorry for the delay; in a couple of weeks, I'll try to refactor my code and add unit tests. Now I'm working overtime for a job-related deadline.

ping @zxgio

I have refactored my code and added some tests. Locally it seems fine, I can't see what's wrong with the "Build phase" (I've never used Google Cloud Build)

The google cloud thing is unrelated, I am in the middle of exams, will finish my exams and check, thanks in advance.

No problem, thanks!


feat: use liveArtwork to determine isOfferableFromInquiry

The type of this PR is: FEAT

This PR resolves PURCHASE-2845
Description

This PR leverages the  liveArtwork field in metaphysics to get the up-to-date isOfferableFromInquiry property. This allows us to more accurately display the 'Make Offer' button and 'Buyer Guarantee' message.
*related to https://github.com/artsy/force/pull/8034
PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[ ] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 05d8987ff2ddd53ef4cb1a046fbff3059ae414b6


Fancy name in discord reborn theme

Do I really need my discord name in the lower right corner? removing it as a stylistic choice would be nice, not to mention that since I use the smallest possible discord window, it's overlapping my emote button in DMs, rendering it unusable until I've written 'enough' to pop it up over the name, even fiddling with zoom and text sizing. E.g. here. I love the rest of the theme, but this one thing bugs me now and then.

As I always say, you get a faster response on the support server, but here's the css to revert it to a more vanilla style:
/*Revert account details text — vanilla */ #app-mount .accountDetails-15i-_e {position:unset !important; min-width:unset !important;bottom:0px !important;height: unset !important;transform: translateY(-25px) !important;} #app-mount .accountDetails-15i-_e .username { margin-left: unset !important; border: none !important; font-family:Whitney,Helvetica Neue,Helvetica,Arial,sans-serif !important; font-size:15px !important; font-weight:700 !important; color:white !important; text-align:unset !important; height:30px!important; line-height:10px!important; text-shadow: 0.5px 0.5px 2px black!important; } .accountDetails-15i-_e .discriminator {display: inline-block !important; font-size: 12px !important;text-shadow: 0.5px 0.5px 2px black!important; position: unset !important;transform: translate(-94px, 14px) !important;margin-bottom: 20px;} .accountDetails-15i-_e + div div{margin-left:unset !important;} .accountDetails-15i-_e .username {background-color: transparent !important;} .channel-members::after {display: none;} [class*="scroller-"].channel-members {height: 100%!important;padding-top: 8px;} .private .channelTextArea-os01xC {margin: 20px 0 32px 0!important} .accountDetails-15i-_e + div div {margin-left: 0px !important;}

Thank you so much, this is perfect!
On Saturday, December 30, 2017 4:40 PM, Matt<EMAIL_ADDRESS>wrote:

As I always say, you get a faster response on the support server, but here's the css to revert it to a more vanilla style:
/*Revert account details text — vanilla / #app-mount .accountDetails-15i-_e {position:unset !important; min-width:unset !important;bottom:0px !important;height: unset !important;transform: translateY(-25px) !important;} #app-mount .accountDetails-15i-_e .username { margin-left: unset !important; border: none !important; font-family:Whitney,Helvetica Neue,Helvetica,Arial,sans-serif !important; font-size:15px !important; font-weight:700 !important; color:white !important; text-align:unset !important; height:30px!important; line-height:10px!important; text-shadow: 0.5px 0.5px 2px black!important; } .accountDetails-15i-_e .discriminator {display: inline-block !important; font-size: 12px !important;text-shadow: 0.5px 0.5px 2px black!important; position: unset !important;transform: translate(-94px, 14px) !important;margin-bottom: 20px;} .accountDetails-15i-_e + div div{margin-left:unset !important;} .accountDetails-15i-_e .username {background-color: transparent !important;} .channel-members::after {display: none;} [class="scroller-"].channel-members {height: 100%!important;padding-top: 8px;} .private .channelTextArea-os01xC {margin: 20px 0 32px 0!important} .accountDetails-15i-_e + div div {margin-left: 0px !important;}—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

reason for closing is the bottom right name got reverted in the main theme


update quick look with new demo notebook

Is your documentation request related to a problem? Please describe
the quick look is a bit outdated and doesn't include all the library's main functionality anymore
Describe the solution you'd like
replace the quick look with an updated version of this demo

based on https://github.com/capitalone/rubicon-ml/issues/208#issuecomment-1142530405, let's also split the updated quick look into three examples for logging, visualizing, and sharing each specifically. we'll add them back to the docs in a way that make it clear that the three of them together are the new "quick look," but now we'll have individual, isolated examples of each part


Table column heading misalignment

Even though the table will still be too large after fixing, the column headings shouldn't be misaligned.

I can try and take a stab at this if needed still?

Thank you! That would be much appreciated. I also don't actually know if the issue still exist, but finding out it doesn't would also be very helpful. :)

Sounds great!  I will take a gander when I have a moment and let you know :)
Chris
On Mon, Feb 3, 2020 at 12:00 PM Martijn van de Rijdt <
<EMAIL_ADDRESS>wrote:

Thank you! That would be much appreciated. I also don't actually know if
the issue still exist, but finding out it doesn't would also be very
helpful. :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/enketo/enketo-core/issues/535?email_source=notifications&email_token=AIDWBCWPAU4Q76ROYXYFSXDRBBEUFA5CNFSM4FNSD63KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUTEOI#issuecomment-581513785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIDWBCXVKOD24YYOQK5Z7JDRBBEUFANCNFSM4FNSD63A
.


Update schema according to specification document

Fixes #68 (unit is a property qualityMetric instead of cvParameter).
TODO in this PR and the specification document @mwalzer: Update figure 1 and the schema graphic to reflect the following changes to the schema (it renders horribly on my Linux):

Remove cvRef from cvParameter.
Move unit from cvParameter to qualityMetric.
Remove ref from controlledVocabulary.

Reorganisation caused a bit of an arrangement tightness. I think this might do, have a look before I upload:

Looks good for me!

@mwalzer Can you update doc/mzQC_schema_graphic.pptx as well?


How to use?

I'm very sorry, if this question is dumb, but I can't get your script to run, because everytime it needs the esm module. I already installed node.js and the esm module, but nothing changes. I have not experience with js-files. Maybe you could give me a short explanation or a few keywords with that I can work trough own research. Would be wonderfull. Thanks for your work!

Ah. It's a bit of an old tool that didn't get updated when we got decent support for ES-modules in node. Using the esm dependency clashes with recent versions of node, so there's that.
I pushed an update that remove that issue; you should be able to run this tool in multiple ways, depending on your needs. The README is also updated.
If you don't particularly want to tinker with the code, I suggest a global installation with npm install -g @keeex/jpegfix. That should work on most OSes.

Wow, that was fast. Now it works. Thanks a lot!


Updating missing licenses from metron-ui.

This is required for the apache release

+1 This should make RAT  a lot happier.

Removed extraneous LICENSE files and COPYING files
Fixed the verbiage for the cisco contribution in the LICENSE file
Added link to the license files for the 3rd party bundled components under metron-ui/lib/public/vendor in the LICENSE file
Added inline licenses in the LICENSE file for the

fontawesome fonts
effective_tld_names.dat data file
bundled portions of kibana (where you saw the app.js without the header file)

+1 works great with EC2 deployment.


Feature Request: Content Server

Hi,
I build a box for my niece and my nethew, so I do not live in the same house.
On Easter I had the problem, I prepared 8 cards but they couldn't use them directly.
My  idea is a content server, as stand alone and/or as integration in nextcloud.
At the phoniebox, will be an configuration point whre you can insert link to your server. And an ID for the box.
When the phoniebox don't have a shortcut for the RFID it will login to content server with ID and check wheter there is a zip file for the RFID. Download the file, compare with hash, extract files and connect with RFID.
There should be audiofiles for smal responses, like. "Your playlist will be downloaded, please be patient", "Your playlist is now ready, have fun", "There is no internet connection, inform your parents". "A playlist is currently downloading, please wait"
During download no other download should be possible. It would be good, if it's possible to hear a local playlist during download. Maybe download can be aborted by special RFID command or by pressing and holding a GPIO button, when the new card is the lastUsedRFID
The content server could be created with a small database offering a rest api to check if the box is allowed to get the content and/or as a plugin for nextcloud.
The content server could also be used for a self created upgrade script.
Download a script and execute. (but this should be discussed, because of security risk)
Maybe in July I will find some time to think about a solution, but maybe someone likes the idea and start immediatly.

Hi @MatthiasCzech
that is a great idea. I will start thinking about this more around the mid of May, when I have more time again to dedicate to Phoniebox. Right now it's a bit tight with other projects.
Especially the idea of cloning configurations and content is interesting.
Thanks and all the best, micz

I was thinking about something similar. I there any news about this topic? Would be great to have such a functionality for sharing and syncing of audiofiles and configuration.

So far nobody has picked up this topic and it's not in any near term planning.
If you are interested in implementing this topic you are more than welcome.
It should be implemented for future3 / V3 and the planned implementation should be discussed here in this issue.
You can also use the chat for quick questions.

Not exactly like described here, but a functionality to sync audio from a server/network share does exist:
https://github.com/MiczFlor/RPi-Jukebox-RFID/wiki/Sychnronising-Phonieboxes-in-a-local-network
Maybe this already fits your case.


Why does "One Time Payment Sample" app from CWS keep taking the user to the chrome sign in page?

I have installed the "One Time Payment Sample" app from CWS to observe the flow of the app. When I click on the app, it opens the app window (see the attached image), which shows "Loading license data..." and it also opens the "Sign in to Chrome" page in a new tab, when I sign in nothing happens, license data does not get loaded. I would like to know why this is happening and what the solution is.

It works flawlessly for me ;(
I got the app window, then the "Sign in to Chrome" window, signed in, and got back the auth token as you can see.

Can you open a tab to chrome://inspect/#apps, click on the Inspect link on the One Time Payment Sample and share what you're seeing there. Here's how it looks for me:

I was using an account which was opened in the United States from India. This time, I used an account which was opened in India and at least got a response (see the attached image), I have attached your requested image also.

I have two questions, the first one is, why I am not getting the same license data as you, and second, why it was not working with the account which was opened from the US.

Could that be because of https://developer.chrome.com/webstore/pricing#seller?
@petele may help us there.

Interesting.


Dummy Issue Title - testing 1, 2, 3

This is a description for the dummy issue created using Postman.

This is a comment added to the dummy issue using Postman.


Fix typo in cv example

Fixes small config typo in the CV example, should be mixed_precision and not mix_precision

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Fixed small bug with SetSlideshowShuffle

It was not using the bool value enable for the setting. I updated it to use the passed in variable for the setting.

Thank you for the contribution!


ipywidgets v8

This work is really great - I am making heavy use of your library in a jupyter GUI.
I am currently using ipywidgets 8.0.0a6, and everything seems to be working. Are there plans to eventually relax the requirements for ipywidgets < 8? Is there a more rigorous way to go about testing this?

Thanks for reporting! I think I added that back when it was less clear what changes 8 would contain. I think I remember seeing somewhere that the default float slider is no longer stepped, which may not play nicely with some parts of this library (e.g. when passing a manually generated slider) but if it works well for you then thats a good sign!

Are there plans to eventually relax the requirements for ipywidgets < 8? Is there a more rigorous way to go about testing this?

Absolutely. And the most rigorous  way to test is to manually go through all the examples and test that they behave as expected (the automated tests in notebooks aren't perfect for capturing what outputs look like), but short of that the automated tests should catch any failures to run.

I am making heavy use of your library in a jupyter GUI.

:heart:  I love to hear that! If it's open source or otherwise shareable I'd love to see it if possible, otherwise totally understand.

Closed by https://github.com/ianhi/mpl-interactions/pull/225
@all-contributors please add @samwelborn  for userTesting

should be working in version 0.20.0 which is now on pypi

Thanks for your quick reply. I will go through the tests soon.
Now that you say that about the float slider - this is actually an issue, you can see in my gif below. Since I just started using your package yesterday, I didn't think too much of it and just dealt with the slider callback (to set another widget's value) in a different, albeit roundabout, way. Sorry for forgetting this before posting - been a long few days trying to solve other small problems....
I am still working on docs/rearranging/cleaning for my GUI, but here is a preview:

I think this example of using your hyperslicer in combination with your interactive histogram could be useful for people. It attempts to mirror this. I didn't show this in the gif, but you can use those xlim/ylim range sliders to change the range of the histogram.
The GUI uses tomopy, a tomography reconstruction package, as a backend.

that's super neat!
Have you seen this example as well? https://mpl-interactions.readthedocs.io/en/stable/examples/range-sliders.html#using-a-rangeslider-for-scalar-arguments-thresholding-an-image
Also, I love using notebooks and find hyperslicer to be really helpful there (we developed it for microscopy data) but I really can't  recommend napari enough. I now mostly use hyperslicer for quickly looking at data in notbeook, but for all stuff where I want really good responsivity I always use napari.
While the gifs on their homepage https://napari.org/ don't really do it justice I think it is absolutely the best in class image viewer for hi dimensional data - gpu backed (fast!) - big dev team workign hard on it - tons of features and plugins - easy enough to make your own plugin. There are two downsides to it:

It doesn't flow as nicely into a notebook as ipympl does
It is definitely easier to get a gui going in ipywidgets then in napari

but that said if you learn a little bit of pyqt it's actually quite easy to slap together a basic gui https://github.com/Hekstra-Lab/napari-labeller#napari-labeller or for a more complex example https://github.com/tlambert03/napari-micromanager#napari-micromanager
Whenever you do put your code up let me know! I'm curious to look at it and happy to look over the matplotlib/mpl-interactions parts to see if there are any optimizations.

Just added you to my private repo, just in case you want to check it out sooner rather than later. I am an experimentalist/just learned python a month or so ago... excuse the hacking in my code. Install instructions probably don't work for you right now, either - still updating them + docs.
Thanks for the recommendation on napari. Looks like a great tool. In my application, we don't really need fast responsiveness - it's only to check out the data quickly/do some basic things in a notebook before running an algorithm on it on the backend. Then, we use dragonfly for reconstruted data analysis/segmentation, which also runs on gpu.
If you don't mind, I planned to snag your docs template. Looks great....

Just added you to my private repo, just in case you want to check it out sooner rather than later. I am an experimentalist/just learned python a month or so ago...

Rad! I'm actually traveling in india at the moment so may a be a little bit before I look but definitely curious to. Also on a first pass it looks good, especially for only hav eused python for a month!

If you don't mind, I planned to snag your docs template. Looks great....

@redeboer gets the credit for overhauling the docs structure - it's from their work on https://github.com/ComPWA/ampform#ampform I htink.
I also have a cookiecutter that generates a package and docs automatically if you're interested: https://github.com/ianhi/pyrepo-cookiecutter

thanks for the heads up, I am using your cookiecutter now.
lots of changes within the past day or so...

https://tomopyui.readthedocs.io/en/latest/
Check the docs out here. Thanks @ianhi and @redeboer !


Supply User-Agent

Rather than the server just seeing python-requests/<version> it might more useful for the client to supply something like simvue-client/<version>.

Implemented in https://github.com/simvue-io/client/pull/81


KeyError at /someurl ('True', 'buy')

Hi, Firstly thanks for your project.
I have ran into a little issue.
When I create a PayPalPaymentsForm using the below code ( which I have copied from the docs I get a KeyError when calling paypal_form.render() this appears to coming from  PayPalPaymentsForm.get_image()
My code
def someview(request):
    paypal_dict = {
        "business": "my@paypaladdress",
        "amount": settings.HOURLY_RATE,
        "currency_code": "GBP",
        "item_name": "Test item",
        "notify_url": "https://www.example.com" + reverse('paypal-ipn'),
        "return_url": "https://www.example.com",
        "cancel_return": "https://www.example.com",
    }
    paypal_form = PayPalPaymentsForm(initial=paypal_dict)

    context = {
        'paypal_form': paypal_form
    }
    return render(request, 'core/template.html', context)

Debugging
I have done some debugging but cannot understand why i'm getting a key error,
def get_image(self):
        return {
            (True, self.SUBSCRIBE): SUBSCRIPTION_SANDBOX_IMAGE,
            (True, self.BUY): SANDBOX_IMAGE,
            (True, self.DONATE): DONATION_SANDBOX_IMAGE,
            (False, self.SUBSCRIBE): SUBSCRIPTION_IMAGE,
            (False, self.BUY): IMAGE,
            (False, self.DONATE): DONATION_IMAGE,
        }[self.test_mode(), self.button_type]

When I run my code and debug, self.test_mode() = 'True' and self.button_type = 'buy' so there should be a hit for the key in the dictionary object.
Workaround
To get round this at the moment, I am runtime patching PayPalPaymentsForm.get_image() like this, but would hope to understand the actual issue.
#views.py
from paypal.standard.conf import IMAGE

def new_get_image(self):
    return IMAGE

PayPalPaymentsForm.get_image = new_get_image

Thanks for any help.

Got the same error when I want to override the url to button image.
def get_custom_image(self):
    return {
        (False, self.BUY): getattr(settings, "PAYPAL_IMAGE", "https://www.paypalobjects.com/webstatic/en_US/i/buttons/buy-logo-large.png")
    }[self.test_mode(), self.button_type]

It sounds like in your settings.py you have:
TEST_MODE = 'True'

instead of:
TEST_MODE = True

Please re-open if this is not the case and you can pinpoint an error in django-paypal.

Yeah, thanks! And I came up with that there were no sense to override get_image() method to have custom image button. Just add PAYPAL_IMAGE to settings.

I was pulling the TEST_MODE from an environmental variable so my True was actually coming through as a string and not a boolean. Will fix now. Thanks much appreciated.


Change createEntity to use generics?

At the moment using createEntity from EntityTypes gives you an Entity back.
Is it feasible / a good idea to generic that so that the return type reflects the type of entity being created?
Otherwise you need to cast, and rely upon the programmers knowledge in order to access methods on the entity specific interfaces, or even set their handslots / armor.
I'm guessing the need for this will be reduced somewhat, once some inventory keys / data are created?

I distinctly remember EntityType generics being discussed at some point. We elected to avoid it since the raw type usage throughout would be necessary in many cases (think about looking up Keys through the registry). I have to find it somewhere at some point to be able to link it to you, unless someone else can find it. I believe I was discussing it with @ST-DDT, but this discussion had to have taken place between a year and two years ago now :/

I searched for that discussion too, but could not find it.
There is one problem with the above proposal, the generic type would be derived and not explicit.

<T extends Entity> T createEntity(EntityType type)

So both of the following would compile, but only one would work.
Zombie zombie = foo.createEntity(EntityType.ZOMBIE);
Horse horse = foo.createEntity(EntityType.ZOMBIE);

The code would then display a ClassCastException on either the return line of the method in (Sponge's code) or in the using line (plugin's code). Which might cause
You can achieve the same "(in)security"/result with just the current one plus an unchecked cast.

Otherwise you need to cast, and rely upon the programmers knowledge in order to access methods on the entity specific interfaces, or even set their handslots / armor.

The main issue is that the EntityType alone does not contain any type information that could be used by the IDE to suggest better interface matches. So the developer still needs to know them. For that to work we would have to change the API of EntityType to include the generic information we need.

public static final EntityType<Zombie> ZOMBIE
<T extends Entity> T createEntity(EntityType<T> type)

This suggestion would work, but I'm not sure about generic CatalogTypes. Is there such a class for all EntityTypes? How about AREA_EFFECT_CLOUD, WEATHER or UNKNOWN?

Don't know if this is the issue you're thinking of but I made a PR a couple years ago https://github.com/SpongePowered/SpongeAPI/pull/496 where it was decided this wouldn't happen.

with the example : <T extends Entity> T createEntity(EntityType type) is there any other difference then the syntax to the current API with these 2 code examples?
'''
Zombie zombie = foo.createEntity(EntityType.ZOMBIE);
Zombie zombie = ((Zombie) foo.createEntity(EntityType.ZOMBIE));
'''

But you're forgetting that it would turn into:
Zombie zombie = foo.createEntity(EntityType.ZOMBIE);

due to it being static final EntityType<Zombie> ZOMBIE = DummyObjectProvider.foo;
I'm less lean on the strict generics than with the cast.

Any approach with generics is muddy. While the cast is annoying, the sub interfaces and the data-like methods in them are for convenience. In most cases, not being of the type of sub-interface isn't an issue.


Failed to fetch data while accessing a non-LTT channel

Hello,
I just enrolled on test flight. The App's really clean, I like it
I have two subscriptions on floatplane, one is LTT, and one is Tyler McVicker (The Passionate Gamer). When clicking on the tab, this appears (with expended details)

I have no idea how I can help you debug this further, but the channel has post that do not have thumbnails it seems. It might be causing some API to respond with some JSON that do not have some key/values you are expecting ?
(see posts with no images below)

Thank you for the pictures! This helps a lot. Yes, I believe that this would be the case (I'm surprised it's possible to not have thumbnails on the videos).
Normally, the same error would appear on the Home tab as well, but the latest McVicker video is 6 weeks ago, it seems, which wouldn't appear in Home unless you scroll down very far.
I'll get working on a fix for this. Much appreciated! I'm glad the extra error information I added recently to that screen came in handy so quickly.

This has been addressed in version 1.2.0. This version will release to Test Flight public beta in 1 or 2 days. Please let me know if this fixes the issue for you once it's available and you're able to test it @Ybalrid!

Thanks! Will try it out when it's our and let you know so you can close

Fiez on the build TestFlight just downloaded! Thanks


rancher-monitoring: support prometheus/alertmanager-instance-namespaces

What?
This PR introduces a new value denyNamespaces for the operator.
If this is specified, it will add the --deny-namespaces=<list> arg on the prometheus-operator.
Why?
This is especially useful if someone tries to use another prometheus-operator on the same cluster.
E.g. I deployed the prometheus-operator helm chart to the namespace prometheus-operator (operator v0.38.1 and v0.39.0 testes) and the alertmanager and prometheus server failed to start, because they were constantly being updated by the operator.
Interestingly, this stopped when I added the --deny-namespaces=prometheus-operator arg to the operator in cattle-prometheus or when I deleted the cattle-prometheus namespace (which will automatically be recreated).
So I guess that the Rancher prometheus-operator and the custom deployed operator are fighting to update the resources, which then results in an update/termination loop.

This could probably be merged with #226

@iwilltry42 Thanks for adding this :) @thxCode @guangbochen can you help review this PR? It looks good to me, the option is already available in upstream chart.

There are surely more useful options in the upstream chart, but without this specific one, extra operator deployments fail and one has to add this to the deployment manually. 👍

In my understanding of the introduction of this parameter: --deny-namespaces, this is not intended to deal with such scenarios. It is a parameter that is mutually exclusive with --namespaces.
For two or more Prometheus Operators which have different purposes in the cluster, this is really difficult to solve through Rancher. Through deny-namespaces, it will only make the entire management more opaque.
@iwilltry42 , I think you should analyze the actual cause of the flipping altermanager and prometheus server in the your scenarios. Is the configuration of the operator causing a label conflict in the generated workload or conflict in executing the command?

Hi @thxCode, you're probably right in a way that the --deny-namespaces parameter is not the correct fix for my use case.
I should probably update the PR to rather introduce the --{prometheus, alertmanager}-instance-namespaces flags instead, which only describe the list of namespaces where Prometheus and Alertmanager instances will be managed (see https://github.com/coreos/prometheus-operator/blob/c7405c847b636e174ce7c2306c57619bca8fcee9/cmd/operator/main.go#L177-L178 & https://github.com/coreos/prometheus-operator/pull/2783).
One reason for the issues could be the updated volumeClaimTemplate stuff (see https://github.com/coreos/prometheus-operator/issues/2950) and the difference in versions used for the prometheus-operator (i.e. version mismatch between Rancher's and my custom prometheus-operator deplyoment).
In any case, I don't think that I should have to adapt my custom prometheus-operator deployment to the built-in operator, which I cannot modify enough to use it properly and which I also cannot delete entirely :thinking:

Should the following be the default here?
prometheusInstanceNamespaces:
  - cattle-prometheus
alertmanagerInstanceNamespaces:
  - cattle-prometheus

I have the feeling, that it's not expected to be managing other instances as well? :thinking:

@thxCode , @guangbochen , @StrongMonkey is there anything left todo here?
What do you think about limiting the operator to managing only instances in cattle-prometheus namespace by default?

Hello @iwilltry42 @StrongMonkey @guangbochen  I ran across this PR as we were tracking for the issue @codayblue mentioned. Would it be possible to get this reviewed?
I'm guessing it would maybe need to be rebased against a newer version? I can assist with this if needed.

Closing due to lack of activity. Old content.


WSL blocks Bitvise SSH Server

WSL appears to bind to port 22 and block Bitvise SSH Server from starting up.
This is the only conflict I've so far incurred from installing WSL, which I will say is quite impressive, as I'm a very heavy user of msysgit and other bash shells on Windows already, and it hasn't messed up my path or anything else. However, I previously had Bitvise configured to load my msysgit bash shell when connecting over ssh (which is my preference as it includes all of my environment variables and can accept scp connections) and I'd like to return to this if possible. Any way I can prevent WSL from taking over port 22 (assuming this is the issue)?

For the moment I'm just running bitvise on port 22222 instead, which appears to work. I only really ssh to this machine from my laptop anyway, so this is an acceptable workaround.

Might I also recommend adding an alias to the WSL bash shell called just "wsl.exe"? Having the mingw binaries earlier on my path than C:\Windows means I always get my msys git bash shell when I call bash, which is good, but when I do want to hop into the wsl environment, I wound up just making my own alias from C:\Windows\System32\bash.exe to wsl.exe.

Incidentally, trying to launch wsl from over an msys bash ssh connection throws Error 80070005.

@axfelix - Thanks for the feedback! bash\WSL by default should not start the ssh service. If you are manually starting the sshd service on WSL and running into conflicts with Bitvise, then yes, that makes sense. As @rodrymbo also notes, WSL is not a container or VM environment, but rather aiming to provide a native experience. To unblock yourself, you should be able to configure the settings of either of these apps to use a different port. And, many thanks for using WSL!

I believe this is the same issue as #777.  @axfelix could you confirm that if you stop the below two services you're able to use Bitvise ssh server on port 22 again?
"Windows SSH Server": "SSH Server Proxy" and "SSH Server Broker"
Full disclosure: those two services don't have anything to do with WSL.  They're just another part of Windows.

WSL no longer requires "Developer Mode" so this issue should now be resolved.


Crash when textView loose Focus and tableview contains No message

Hi jess,
I want to report a bug
I am using the current stable version of the Library 7.2.0.
I have a crash when I focus on the textview in toolbar. When I swip to bottom to hide the keybord the application crash. It happend only when there are No messages in the tableview.

@modiodio please copy/paste crash log to have more information about the issue. FYI the library doesn't use a UITableView it uses a UICollectionView

I have almost no log error.
I only see (lldb) !

I got this error when I updated jsqmessage ! I had not this error before !

Do you have an exception breakpoint enabled? can you check your stack and see if it hangs in one of the library functions?

@GianniCarlo oh yeah it stack on this line it seem to be parse library I am using
0x1002d9828 <+344>: b      0x1002d9834               ; <+356> at PFInternalUtils.m:381
Do you have an idea for this bug ?

mmmmm I haven't use Parse in a while now, but it looks the problem resides with Parse then, you could go ask at their github or in Stackoverflow

ok ok thanks for your answer ! it helps me a lot


chore: Skip slow tests on Windows

Fixes #228
Essentially, Windows takes a very long time to give up with bad resolver addresses, waiting the maximum amount of time for an i/o timeout.
We can just ignore these tests on Windows. It works, it's just really slow.

I don't think there's any information we can use to know if it should fail quicker. I played with the timeout options on the DNS client and all it did is shorten the maximum amount of time before it fails.
Btw the default timeouts are 2s on the DNS client, I'm not sure we need it to be 10s, not sure what the usecase was for making it that long, but at this point I don't want to risk breaking anyone.

I would be surprised if anyone is relying on the timeout being a full 10s... :thinking: I'm not sure who would do that :smile:

Yeah I mean I'm just being careful/paranoid :sweat_smile:
Like really, it's not a problem that it takes long to fail under normal circumstances. The problem is only relevant specifically during testing because we try it so often and multiple times per test run.
If we ever make changes to this part of the code, we can just comment out the skip to run it in CI one time during a PR, then turn it back off before merging. It's a waste of time to test it every time, obviously.


chip-device-ctrl is case sensitive, it shouldn't be

Problem
chip-device-ctrl fails to parse commands unless arbitrary case sensitivity is followed.
chip-device-ctrl > zcl onoff on 99999 1 0
An exception occurred during process ZCL command:
UnknownCluster: onoff
chip-device-ctrl > zcl OnOff On 99999 1 0
CHIP: [ZCL] Successfully encoded 11 bytes
CHIP: [IN] Secure message was encrypted: Msg ID 3
CHIP: [IN] Sending msg from 0x000000000001b669to 0x000000000001869f
CHIP: [IN] Sending secure msg on generic transport
CHIP: [IN] Secure msg send status 0
CHIP: [CTL] SendMessage returned 0
Proposed Solution
Fix argument parsing so it is not case sensitive.

Closing due to lack of activity.


Backend Jobs fixing and refactoring:

Some things you can find in this PR:

In general, reduce the number of requests send to the API
Jobs with multiple circtuits can be used with their respective
results
Instantiating a IBMQJob doesn't imply to submit the information.
There's a separate method: .submit() that will make this
operation explicit to the user.
Fixing some incorrect status handling
Removed active polling in some situations (time.sleep(0.1))
Refactored _wait_for_job() for simplicity
Removed IBMQJob.exception()
Reduced some algorithm complexity

It will probably require a second iteration to polish some things.

Understod, but I don't see the need of mimic python's Futures interface, we have our own use-cases and the only thing in common is that we want async behavior too. By letting Jobs use python's Futures by composition and not "inheritance" (it's not real inheritance, but a simulation) we will have way more flexibility.

Another point that we should review, is the usage of properties instead of methods. My feelings is that is more related to how things are done in a Pythonic way... but as Salva said, properties throwing exceptions are weird, at least for me as well.

@atilag I was wondering what people thought about properties usage as well. Right now it appears inconsistant. In fact I've already diverged from Futures in that regards because they use methods :slightly_smiling_face:


Cannot connect wss:// after change website to https

Hi guys, i have integrated stanza.io in my project.
It worked fine with this config:
service.user = {
stream_management: true,
jid: user.xmpp_username + XMPP_HOST.HOST,
password: user.xmpp_password,
    // config websocket
    wsURL: 'ws://' + XMPP_HOST.BOSH_PORT + '/websocket',
    transports: ['websocket']

};
And in my constant :
.constant('XMPP_HOST', {
HOST<EMAIL_ADDRESS>HOST_RAW: 'playorgo.ejabberd.com',
BOSH_URL: '<IP_ADDRESS>',
BOSH_PORT: '<IP_ADDRESS>:5277'
});
But after i changed my website from http to https, and i changed config wsUrl to :
wsURL: 'wss://' + XMPP_HOST.BOSH_PORT + '/websocket',
But it doesn't work . Anyone can help me with this issue ?
Thanks
Error code :
WebSocket connection to 'wss://<IP_ADDRESS>:5277/websocket' failed: Error in connection establishment: net::ERR_CONNECTION_CLOSED in stanzaio.bundle.js:3749

You need to change your port from 5277 to whatever post is exposing https.

Hi @GregDThomas,
I changed port but it didn't work, i used https service of cloudflare.
Now error message is : "ERR_CONNECTION_REFUSE"

That sounds like a networking problem, not a stanza.io problem.
Check you've got the right port.
Check you've got connectivity to that port, i.e. there are no firewalls in the way.
Can you use curl to access a page server over https on that port?

@dxtri94 are you using a browser to make the connection?


Html medium

Description

added a simple form and wrote the complete css again
Additional Context

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Fix apparmor blocking writes to non-default tmp_dirs

Without this change, we'd get errors like this when running Chef:
Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0, 1, 2], but received '141'
---- Begin output of "bash"  "/tmp/chef-script20160308-2620-vdadfn" ----
STDOUT: Installing MySQL system tables...
STDERR: 2016-03-08 13:54:59 0 [Note] /usr/sbin/mysqld (mysqld 5.6.28-0ubuntu<IP_ADDRESS>) starting as process 7762 ...
---- End output of "bash"  "/tmp/chef-script20160308-2620-vdadfn" ----
Ran "bash"  "/tmp/chef-script20160308-2620-vdadfn" returned 141

Resource Declaration:
---------------------
# In /var/chef/runs/14049d19-d3d6-4ba2-8e78-1e53a4ad4e3b/local-mode-cache/cache/cookbooks/mysql/libraries/provider_mysql_service_base.rb

145:         bash "#{new_resource.name} :create initial records" do
146:           code init_records_script
147:           returns [0, 1, 2] # facepalm
148:           not_if "/usr/bin/test -f #{parsed_data_dir}/mysql/user.frm"
149:           action :run
150:         end
151:       end

Compiled Resource:
------------------
# Declared in /var/chef/runs/14049d19-d3d6-4ba2-8e78-1e53a4ad4e3b/local-mode-cache/cache/cookbooks/mysql/libraries/provider_mysql_service_base.rb:145:in `block in <class:MysqlServiceBase>'

bash("default :create initial records") do
  action [:run]
  retries 0
  retry_delay 2
  default_guard_interpreter :default
  command "default :create initial records"
  backup 5
  returns [0, 1, 2]
  code "        set -e\n        rm -rf /tmp/mysql-default\n        mkdir /tmp/mysql-default\n\n        cat > /tmp/mysql-default/my.sql <<-EOSQL\nUPDATE mysql.user SET password=PASSWORD('root') WHERE user = 'root';\nDELETE FROM mysql.user WHERE USER LIKE '';\nDELETE FROM mysql.user WHERE user = 'root' and host NOT IN ('<IP_ADDRESS>', 'localhost');\nFLUSH PRIVILEGES;\nDELETE FROM mysql.db WHERE db LIKE 'test%';\nDROP DATABASE IF EXISTS test ;\nEOSQL\n\n       mysql_install_db --defaults-file=/etc/mysql-default/my.cnf --datadir=/mnt/took/mysql/ --explicit_defaults_for_timestamp\n       /usr/sbin/mysqld --defaults-file=/etc/mysql-default/my.cnf --init-file=/tmp/mysql-default/my.sql --explicit_defaults_for_timestamp &\n\n       while [ ! -f /run/mysql-default/mysqld.pid ] ; do sleep 1 ; done\n       kill `cat /run/mysql-default/mysqld.pid`\n       while [ -f /run/mysql-default/mysqld.pid ] ; do sleep 1 ; done\n       rm -rf /tmp/mysql-default\n"
  interpreter "bash"
  declared_type :bash
  cookbook_name "took_role"
  not_if "/usr/bin/test -f /mnt/took/mysql//mysql/user.frm"
end

Thanks to my friend and colleague @fabianhoward for doing most of the work figuring this out.
I'm not certain this is the cleanest way of fixing this or whether there are other cases where AppArmor will still cause problems - all I know is that we had AppArmor denials happening on the
mysql_install_db --defaults-file=/etc/mysql-default/my.cnf --datadir=/mnt/took/mysql/ --explicit_defaults_for_timestamp

command until we made this change. I leave the question of whether this change is the right thing to do and whether to merge it in your hands.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@ExplodingCabbage

Please sign the CLA here.

@ExplodingCabbage,
Can you confirm whether the change:
PR #414 of adding the umask resolves the issue as well? This looks like the same area of code, so wondering if that will resolve your problem.
Thanks!

@ExplodingCabbage,
Can you confirm whether PR #414 resolves this? Thanks!

@iennae
Short answer: no, but I don't really understand wtf is going on very well.
Long answer: Hmm. My grasp of AppArmor is admittedly weak, but I don't understand how the two can be related; #414 changes file permissions, which dictate what particular users are allowed to do, but AppArmor, as I understand dictates what particular executables are allowed to do. It's an extra layer of restrictions entirely (I think?) unaffected by file permissions.
It's been a while since I touched this stuff, but I think testing against the current master branch of https://github.com/chef-cookbooks/mysql confirms this? My build reproducibly fails with exactly the same error as I quoted in the OP if I try to build with your current master instead of mine, whereas with my version of the mysql repo my build reproducibly works. I'm slightly confused because I think I recall previously seeing explicit AppArmor denials in my /var/log/kern.log prior to authoring my fix, and I don't see them any more; I'm not sure if I'm misremembering or if #414 has somehow suppressed the error logging I used to see (which would make no sense given my understanding of how AppArmor works) without actually fixing the bug.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.


Issue 156 format pre tn like tn

Closes #156

Due to issue with git, closing this pull request.

DO NOT MERGE


Concurency issue ?

I've troubles when load testing the web part of this service. When I run python test script randomly choosing files from one directory where is mix of the infected (eicar) and not infected files sending to the service and I have only one instance of the script running (sequential delivery) everything is ok. However when I run multiple instances of this test in 10 minutes I got response that the file is infected with note that that is EICAR NOT VIRUS file however file sent is eligible not infected. So we got the inconsistent results.
Test Script (Python):
import requests
import random
import time
from timeit import default_timer as timer
import json
import os
from elasticsearch import Elasticsearch
import uuid
from datetime import datetime

url = "https://<url>/scan"

path = "../testfiles"

items = [f for f in os.listdir(path) if os.path.isfile( os.path.join(path, f) )]

print len(items)

def runVirusScanTest(path):
    start = timer()

    multipart_form_data = {
        'malware': open(path, 'rb')
    }

    try:
        response = requests.post(url, files=multipart_form_data)
        end = timer()

        string = response.text.decode('utf-8')
        print string
        json_obj = json.loads(string)

        data = {
            'timestamp': datetime.now(),
            'time_elapsed': end-start,
            'filepath': path,
            'filesize': os.path.getsize(path),
            'infected': json_obj['avast']['infected'],
            'result': json_obj['avast']['result']
        }
    except Exception:
        data = {
            'timestamp': datetime.now(),
            'time_elapsed': end-start,
            'filepath': path,
            'filesize': os.path.getsize(path),
            'infected': json_obj['avast']['infected'],
            'result': json_obj['avast']['result'],
            'exception': str(Exception)
        }
        pass

    return data

def putToES(json_data):
    es = Elasticsearch(http_auth=('elastic', 'changeme'))
    res = es.index(index="test-results", doc_type='test-result', id=uuid.uuid4(), body=json_data)

def runTests():
    count = 0
    while (count < 1):
        try:
            length = len(items)
            path_nr = random.randint(0,length-1)
            data = runVirusScanTest("../testfiles/" + items[path_nr])
            putToES(data)
            time.sleep(0.2)
        except Exception:
            print Exception
            pass

runTests()

Can you please look at this issue ?

This is a great test.
Have you also tried the other AVs and Avast is the only one with this issue or are you primarily interested only in Avast?

Hi, I've test only the Avast. Well if this issue is Avast related than we are flexible to switch. However I do not have time to test all of them.

I would be curious if this is specific to Avast could you test at least one more AV?

Hi yes this weekend or next week I can test avast with not changed routine and also I will test kaspersky too.

So I was not able to finish the Malice kaspersky AV because I cannot get a trial key for the corporate/business linux file server AV scanner.  I had a github user that thought they could get me a trial key, but hasn't responded yet.

than I will check some other just tell me which one I should try

Maybe Avira?

Here is the master list of my AV plugins: https://github.com/maliceio/malice-av/blob/master/README.md
The readme links to all the individual repos.

Hi I checked the avast - your build and it suffer with the issue, on the other side I checked comodo which does not, so maybe it is not caused by your implementation of the service but by the avast. I'm going to test clamav, avg and escan (in progress) to see if those are not having the issue. What I seen till now is that avast have much better performance than others.

escan does not have the issue, however I would like to test on bigger scale as the scanning time is much slower so I have to put more load on the service, to compensate the service performance.

That is very interesting.  Maybe it is starting the avast service that is non-deterimistic /etc/init.d/avast start. I could add more checks to make sure it is running before scanning, but you said it was mixing up files with responses?  That sounds like Golang wasn't tracking http sessions properly?  Golang is very good at http so I would assume it would be on avast's side.

Another idea for the AV plugins at least when used as a web-service might be to start services/load definitions into memory before starting the HTTP web server?  That way it would hopefully make scanning much faster.

Hi do you have any update on this ?

Hi blacktop any update here ?

Just to remind myself you are only interested in using the malice-av plugins as web-micro-services right?

Also it looks like you were going to test escan on a "bigger scale" did you do that?  Is it important that you use Avast over the other AV plugins?

Hi we need to have webservice which accept file and tells us if it is infected or no, it does not need to be the container or microservice.

So it sounds like you just need to talk to Avast tech support?


[Quesion] about embedding whisper on deivce?

Have you ever thought about deploying whisper on IOS, android? I think whisper is embedded in device which is a breaking-through.
Agreement Local algorithm + Whisper.cpp is a perfect.
How about you?
Best regards,
Tu

no, I have other priorities. Good luck!


ALS004_Alignment-segment-shape-representation

Nice work :)
Should we formulate the rule (and the one of ALB002 I wrote before as well) slightly different to allow for more reuse of the already written code, e.i. 'The value must be X'
https://github.com/buildingSMART/ifc-gherkin-rules/blob/47b6a14abb41da91b97d5ca65dc3c1f1ab243429/features/steps/thens/values.py#L11-L66
An idea would then be (something like) :
Scenario 1 : Agreement on Representation being Axis
Given a file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcAlignmentSegment
And Its attribute Representation
And Its attribute Representations
And Its attribute RepresentationIdentifier
Then Its value must be 'Axis'

Scenario 2 : Agreement on RepresentationType being Segment
Given Repeat steps 1,2,3,4,5 #maybe change this to 1-5?
Then The value must be Segment

The functionality of this 'Repeat steps' can be found in the open PR for GEM005
https://github.com/buildingSMART/ifc-gherkin-rules/blob/0f2a25634cde3ea348faac6aa1be8845b2a82019/features/steps/giventhens/repeat_return.py#L6-L17
Scenario 3 : Agreement on items being IfcCurveSegment
Given repeat steps 1 - 5
Then The value must be 'IfcCurveSegment' #or change to 'items' and add @ to the step impl

This way we get rid of some duplicate code and reuse 'The value must be {constraint}' often. We could even distinquish between 'type' and 'value' in the gherkin implementation by adding 'The {type/value/items} must be {constraint} and perhaps the feature file itself becomes more readable (we could potentially use the scenario description as a description of what is being tested for the platform itself). But maybe I am making it too complicated, I'm not sure .. What do you think?

Hi!
Thank you for a great comment.
I think to setup multiple scenarios with repeatable code we could also use the Background feature, as experimented here: https://github.com/buildingSMART/ifc-gherkin-rules/pull/37/files#diff-f12e75ce63549024543ab0c71a93b3cc880178205d26599482a386aa6041d205R5
I find it a bit more readable than Given repeat steps 1 - 5, but that's debatable of course.
As for using the already present step: 'The value must be {constraint}' (thens/values.py).
I think this is a good idea, but I would have to add a decorator, to have something like:
@then("The value must be equal to: {constraint}")  # HERE CHANGE
@then("The value must {constraint}")
@then("The values must {constraint}")
@then('At least "{num:d}" value must {constraint}')
@then('At least "{num:d}" values must {constraint}')
def step_impl(context, constraint, num=None):

And I would add the logic checking the value being equal to the provided value to this step, as currently {constraint} checks for 'identical', 'unique' mostly.  But I do agree that it would perhaps be cleaner to use this step.
What do you think? @Ghesselink

An idea would then be (something like) :
Scenario 1 : Agreement on Representation being Axis
Given a file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcAlignmentSegment
And Its attribute Representation
And Its attribute Representations
And Its attribute RepresentationIdentifier
Then Its value must be 'Axis'

Scenario 2 : Agreement on RepresentationType being Segment
Given Repeat steps 1,2,3,4,5 #maybe change this to 1-5?
Then The value must be Segment

I think to setup multiple scenarios with repeatable code we could also use the Background feature, as experimented here: https://github.com/buildingSMART/ifc-gherkin-rules/pull/37/files#diff-f12e75ce63549024543ab0c71a93b3cc880178205d26599482a386aa6041d205R5

I agree; the way I understand the proposal, I also think that this reinvents an existing Gherkin feature: the Background keyword.

I would have to add a decorator

This is a code-architecture decision. I don't have any strong opinions on this.

I also think that this reinvents an existing Gherkin feature

think to setup multiple scenarios with repeatable code we could also use the Background feature

Agree with both of you @pjanck @JTurbakiewicz . In this case it seems clearer.
So then it would be something like :
Background : AlignmentSegment Representation

Given a file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcAlignmentSegment
And Its attribute Representation
And Its attribute Representations
And Its attribute RepresentationIdentifier

Scenario : Agreemen on Representation being Axis
Then The value must be 'Axis'

Scenario: Agreement on RepresentationType being Segment
Then: The value must be 'Segment'

Scenario: Agreement on IfcCurveSegment
Then: The value must be 'IfcCurveSegment'

It looks like if someone reads it for the first time they'll immediately know what's being tested, nice :-)
Does this work?

I think this is a good idea, but I would have to add a decorator, to have something like

That's not required I think as it's already covered by the @The value must {constraint}  . In the example of 'The value must be 'Axis', the constraint is 'be Axis' and 'be' part will be stripped from the constraint, leaving only 'Axis' in this case. https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L21-L22 .
Then there seems to be two options of implementation:

If there is nothing 'extra' stated in the constraint (such as identical/unique), as is the case above, implement it as 'equal to'. That also leaves the option open to implement the specific 'equal to' as actually equality between to values if this use case comes up. Then we'll need to check for the combination of a literal 'be' followed by a value.
Require for the gherkin statement to specifically state 'value must be equal to Axis' and implement it in a similar way as identical and unique. https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L29 .
Now I'm writing this down, I think I'm leaning towards the first option. It seems to me that it will more understandable for someone else reading or making own gherkin statements (including future us).

But we could decide to be more specific about this, in which case I think it'll be better just to leave it in the {constraint} variable and treat in a similar way as 'identical' and 'unique https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L29

I also think that this reinvents an existing Gherkin feature

think to setup multiple scenarios with repeatable code we could also use the Background feature

Agree with both of you @pjanck @JTurbakiewicz . In this case it seems clearer.
So then it would be something like :
Background : AlignmentSegment Representation

Given a file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcAlignmentSegment
And Its attribute Representation
And Its attribute Representations
And Its attribute RepresentationIdentifier

Scenario : Agreemen on Representation being Axis
Then The value must be 'Axis'

Scenario: Agreement on RepresentationType being Segment
Then: The value must be 'Segment'

Scenario: Agreement on IfcCurveSegment
Then: The value must be 'IfcCurveSegment'

This would look very clear indeed, nice :-)

I think this is a good idea, but I would have to add a decorator, to have something like

That's not required I think as it's already covered by the @The value must {constraint}  . In the example of 'The value must be 'Axis', the constraint is 'be Axis' and 'be' part will be stripped from the constraint, leaving only 'Axis' in this case. https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L21-L22 . Then there seems to be two options of implementation:

If there is nothing 'extra' stated in the constraint (such as identical/unique), as is the case above, implement it as 'equal to'. That also leaves the option open to implement the specific 'equal to' as actually equality between to values if this use case comes up. Then we'll need to check for the combination of a literal 'be' followed by a value.
Require for the gherkin statement to specifically state 'value must be equal to Axis' and implement it in a similar way as identical and unique. https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L29 .

While I am writing this down, I think I'm leaning towards the first option. It seems to me that it will more understandable for someone else reading or making own gherkin statements (including future us). The implementation in python would then be to change https://github.com/JTurbakiewicz/ifc-gherkin-rules/blob/bb4a7f3c2b3db3a65e20f5b93e5a3e3fa39cd165/features/steps/thens/values.py#L21-L22 With something like :
while constraint.startswith('be'):
   constraint = constraint[3:]
   if re.search(r'\s', constraint): #check if str containts one value, i.e. not containing whitespaces
      if not constraint in context.instances: # untested yet, perhaps needs to be modified to know which instances is causing the error).
          errors.append(some_error_category) #tbd
 while constraint.startswith('in'):
     constraint = constraint[3:]

I'll test and work this out further, but would first like to know if you agree/I make sense :) @JTurbakiewicz

Makes sense. I'll play with it a little bit as well.
The additional constraint of ours is to have all the scenarios, so:

    Then The value of attribute RepresentationIdentifier must be Axis
    And The value of attribute RepresentationType must be Segment
    And The type of attribute Items must be IfcCurveSegment

Be present in the same time. I'll check if background scenario lets us do that.


Private Endpoints Policy

I'm testing with the private endpoints and the policy doesn't create any private endpoints, I created a storage account and waited 24 hours.. Please help.
                     "nodeName": "DNZZones",
                    "assignment": {
                        "name": "Deploy-Private-DNS-Zones",
                        "displayName": "Configure Azure PaaS services to use private DNS zones",
                        "description": "This policy initiative is a group of policies that ensures private endpoints to Azure PaaS services are integrated with Azure Private DNS zones."
                    },
                    "definitionEntry": {
                        "policySetName": "Deploy-Private-DNS-Zones",
                        "displayName": "Deploy Private DNS Zones"
                    },

That policy does not actually create a private endpoint - all it does is ensure that the private endpoint is using the correct private DNS zone. There is a built-in policy available (https://www.azadvertizer.net/azpolicyadvertizer/9f766f00-8d11-464e-80e1-4091d7874074.html) which actually creates the private endpoints - you could create an assignment for that. You will need to provide the subnet ID for private endpoint as a parameter.

That policy does not actually create a private endpoint - all it does is ensure that the private endpoint is using the correct private DNS zone. There is a built-in policy available (https://www.azadvertizer.net/azpolicyadvertizer/9f766f00-8d11-464e-80e1-4091d7874074.html) which actually creates the private endpoints - you could create an assignment for that. You will need to provide the subnet ID for private endpoint as a parameter.

Thanks, is there also policyset which does it for all PaaS services?

There is not - you would have to create one.