Create support ticket

Depends on #1382, closes #1201

Coverage decreased (-0.02%) to 82.971% when pulling f720a71874971bf530dd73faee34044ef6b38e8b on eatonphil:create-support-ticket into bb968f749f48027376e9680989199b3b083ab572 on linode:master.

Should this be [Do Not Merge] too?
"We appreciate your ticket and enjoy helping our customers"*

Coverage increased (+0.2%) to 83.284% when pulling 51b08e732c6a5ec9098f1f02770fbbe0c643d04b on eatonphil:create-support-ticket into 37254f2737b3c99828d4f9c7d6233aca08955e7a on linode:master.

Coverage increased (+0.2%) to 83.328% when pulling e5d58ae9a712c13479408ecc3e86d40218edde9e on eatonphil:create-support-ticket into 37254f2737b3c99828d4f9c7d6233aca08955e7a on linode:master.

Coverage increased (+0.2%) to 83.328% when pulling e3d9aad579f6fe2256f14307b4d8228a43043108 on eatonphil:create-support-ticket into 37254f2737b3c99828d4f9c7d6233aca08955e7a on linode:master.

resource links need target="_blank"

@mparke

resource links need target="_blank"

Done

Coverage increased (+0.2%) to 83.276% when pulling 6f760ae58673df2adf8b6a06e25ab7ff27258140 on eatonphil:create-support-ticket into 37254f2737b3c99828d4f9c7d6233aca08955e7a on linode:master.

thanks!


rtp: support text payloads

RFC 4103 specifies how to transport UTF-8 encoded text in the payloads of RTP packets.
If we write some code to handle text payloads, that gives us an easy way to synthesise "fake" streams and specify them in Go source code. That should be a lot easier to reproduce & verify than, for example, video streams.

Done - see rtp/session_test.go


A way to apply SfM from frames belonging to Multi-camera with shared FoV?

Hi,
I am trying to understand whether it is possible to run SfM for a scene involving moving targets, but static multi-view cameras. (Initial pose of the camera wouldn't change with time). I don't plan to use it for real-time but as a post-processing step. How can I pass multiple-images from the same camera, which are temporally aligned? (in other words, a video stream from multiple cameras).
Furthermore, once the a) Focal length of the camera, b) Relative pose of the cameras are obtained, I would like to use this information to sequentially compute feature, map them and obtain depth maps. I am only interested in the depth maps and hence would prefer to not save the features, matches, etc. locally. Is there a stand-alone script that does it?
Thank you in advance for your time and response.
Regards,

Hi @Sentient07 ,
OpenSfM doesn't support such setups for moving targets.
If you already have image poses and cameras intrinsics can just construct a reconstruction.json file and start the process form the undistot_images step.
Yann


implement sqrt function

Implment the u256_sqrt in utils/helper.cairo.
math detail is about Babylonian method: https://en.wikipedia.org/wiki/Methods_of_computing_square_roots#Babylonian_method
solidity version can be found at uniswapv2: https://github.com/Uniswap/v2-core/blob/master/contracts/libraries/Math.sol

It's great. Can you rewrite it in cairo1 and add tests fot it? @mehrdadmms


Small Tickets

#280 Default Ordering - Now displays most recently approved first
#333 Clarifying Date Field Labels - Overriding verbose_name attributes to match requested nomenclature
#293 State Name Abbreviations - Appended API method to also retrieve organization.state and concatenate it with the name
#92 Icon - Finally fixed formatting of "Request a Change" button icon

Coverage increased (+0.02%) to 88.889% when pulling 4a3386023f0e9dd760a1866c3f000e8788e906ae on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Just for the sake of keeping our tests up-to-date, could you add state fields to the api tests?

Coverage increased (+0.02%) to 88.889% when pulling a5574605a24484fc30db58e45a5d7fec284d491a on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Should now be up to date.

Oops, looks like my cat stepped on my keyboard and deleted part of a function name and broke things!

Coverage increased (+0.03%) to 88.896% when pulling 4b5d58b22794be27ee394bcc580369adbbc30e8e on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling 25fdcbf6f66062d8f62b9c97820b295fba977e2d on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling 55b44972cc39f49873aa203b36264e2c6a90424d on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling e7304e3beedd492b67b1af71c5eee5284c3fea84 on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling 98a7c76f548f679f5f990268292c5bac6b8b3f81 on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling 78cb4fbc19596857bb810c790acb8326bbf6b423 on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling fa0b9e6fe7cffcbaf82e5a184d39b0eca544527f on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

Coverage increased (+0.03%) to 88.896% when pulling a296a34c53c0e71efbe2f542c9e58970300857cb on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.

#259 - the position of this button is pretty weird. It just hangs below the search box. I think there's gotta be a better look for this. Was it meant to sit to the right to the "Go" button?

That's just exactly where it was requested in the ticket.

Coverage increased (+0.03%) to 88.896% when pulling f7f1f39675ea4b08dae3695ef1e823c67c353aef on scott-dev into 651e238858ddb9c45e0919bd93d1382311964e87 on master.


sox FAIL formats: no handler for file extension `mp4'

I have .mp4 file and a .wav file.
I am passing .mp4 as good file and .wav as bad file. I am getting correct offset value.
But, out.wav is not getting generated and I am getting this error.
sox FAIL formats: no handler for file extension 'mp4'
So, what are the supported formats for sync audio/video?

At least WAV, FLAC and Ogg Vorbis work for me.
Boring details:

SoX is used to read good file and to write output. So everything that SoX can read should work for good file, everything that SoX can write — should work for output file (see sox --help-format all)
both good and bad files are also read by ffmpeg. So everything that ffmpeg can read is supported (see ffmpeg -formats and ffmpeg -codecs)


Encryption add - fails on Github.io host

Hi, added the following to the mkdocs.yml after I installed the plugin.
plugins:
    - search: {}
    - encryptcontent:
        global_password: 'password!'
        remember_password: True
        default_expire_dalay: 12

Global encryption works well on dev environment via mkdocs serve.
But once pushed to github with gh-pages, the passwod authentication never works.

@CoinK0in sure no problem thanks for getting back to me..

mkdocs 1.2.3
mkdocs-encryptcontent-plugin 2.0.0
mkdocs material additonal plugin
mkdocs.yml

site_name: site name
nav:
 - Home: index.md
 - About: about.md

theme:
  name: material
  logo: assets/logo.png
  favicon: assets/favicon.png
  palette:
    scheme: slate
    primary: blue

plugins:
    - search: {}
    - encryptcontent:
        global_password: 'Password12345!'
        remember_password: True
        default_expire_dalay: 12
        password_button: True
        password_button_text: 'login'

push to gh pages via a mkdocs gh-deploy
404 error on /assets/javascripts/decrypt-contents.js
but that js is there it is not missing
gh repo is also private

Humm . . .
Using the exact same versions, same configuration.
Publishing the project via mkdocs module for gh-page with cli  :
mkdocs gh-deploy --config-file ../mkdocs_poc/mkdocs.yml --remote-branch master

I do not reproduce the issue.
If the script is present, the generation of the JS template is correctly done in the build process.
And you should have in the "md-content" class div of the material theme the following line:
<script defer="" src="/assets/javascripts/decrypt-contents.js" type="text/javascript"></script>
Do you use a CNAME to rename your site ?
I am thinking about "CSP policy" or "HTTPS headers" which, by default, can block the loading of resources.
Have you trie to set "site_url" values on your Mkdocs config file ?
Honestly, I don't have any other ideas here :/
And i can't check the configuration in case of a "private repo".
(Although I think there is no difference with the public version)

Thanks will run through checks and report back

Ok much reviewing I have some breadcrumbs that I have been following...  On review of the github page the filename decrypt-contents.js is being requested from /assets/javascripts/decrypt-contents.js.
Rather than /site_dir/assets/javascripts/decrypt-contents.js

Thank you for this additional information.
I was able to reproduce the problem and I was able to understand that when the site uses a substructure folder, my current configuration does not allow the loading of the decryption script, cause de path is wrong.
I made some tests and modifications, which I published in version 2.0.1.
The script loading path is now based on the site_url configuration value, which should be set according to your site subtree.
As described here
I think this time we're good, I'll let you validate that.
Don't hesitate if you have any feedback, your welcome :)

@CoinK0in yes this now works in 2.0.1 - thank you very much for reviewing and sorting.

Hello, sorry for reopening this old issue, but I am having the same problem James had before with the 2.2.1 version. My config looks like this:
site_name: site
repo_url: repo_url

theme:
  name: material
  locale: en
  highlightjs: true
  hljs_language:
    - python
    - R
    - bash

nav:
  - Home: index.md
  - About:
    - 'Contribute to this doc': 'contribute.md'

markdown_extensions:
  - pymdownx.arithmatex:
      generic: true
  - toc:
      permalink: "#"
      separator: "_"

plugins:
  - search: {}
  - encryptcontent:
      title_prefix: '🔒'

extra_javascript:
  - javascripts/mathjax.js
  - https://polyfill.io/v3/polyfill.min.js?features=es6
  -<EMAIL_ADDRESS>
where my index.md has the password tag. When I run locally everything works fine, but the Github build the asset decrypt-contents.js is missing in the index. There is something I am missing here? I am seeing the following in the debugger:
<script type="text/javascript" src="/assets/javascripts/decrypt-contents.js" defer></script>

Hi, thanks for checking the closed issues first.
James' problem was related to the use of a subdirectory, as the root of the site (example.com/foo/). Before the changes in version 2.0.1, my plugin was not able to adapt to the custom root path. But that shouldn't be the case now.
For the decryption script path to be correct, you must specify this additional path in the site_url directive (described here). You can also use the site_dir directive (described here) so that the build is generated and hosted correctly via "gh-deploy".
Configuring your website with a modified root path should look like this:
site_name: PoC_encryptcontent_plugins
site_url: http://coink0in.github.io/subdir/               # Path is parse and add to the default JS path
site_dir: subdir/                                         # Used to build the site in the "subdir" directory
theme:
  name: material
[...]

With this configuration, you should observe a change in the path to the decryption JS script. The path must match your root url.
<script defer="" src="/subdir/assets/javascripts/decrypt-contents.js" type="text/javascript"></script>

Tell me if your problem is different (You can open a new issue).


SLF4J runtime error for swagger-diff v1.2.1

Hi. With new v1.2.1 I am getting this error/warning in runtime:

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

This is my Java environment:

java 10.0.2 2018-07-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.2+13)
Java HotSpot(TM) 64-Bit Server VM 18.3 (build 10.0.2+13, mixed mode)

Are you using the command line tool? I re-uploaded the package and fix the problem： https://github.com/Sayi/swagger-diff/releases/tag/v1.2.1

Great. Problem is fixed with the new upload.


Add functions and test helpers for account-linked emails

This PR also does some refactoring of the existing tests, probably easiest to review commit by commit.
See also https://github.com/alphagov/account-api/pull/111

Trello card

Looks great, will re-review once the other one is merged.


Add PerformancePathologySpec for finding performance bugs

Add DeduplicationSpec as first example of this kind of test
This will fail until https://github.com/ucb-bar/firrtl/pull/375 is merged and issue pointed out in https://github.com/ucb-bar/firrtl/pull/375#issuecomment-262149561 is resolved.
The basic idea is that certain things should scale linearly. This PR provides a simple mechanism for writing tests that check some possible performance pathology by putting an upper bound on how long it should take. It's a very rough way to do this, and is certainly not a replacement for real performance regression testing. However, it could serve as a useful sanity check.

Glad to see de-dupe and performance tested!


Chance of no augmentation being applied

It seems there's always a chance of no augmentation applied. Is it possible to make sure that at least 1 random augmentation is applied?

iaa.SomeOf((1, 3), [iaa.Dropout(...), iaa.Affine(...), iaa.AdditiveGaussianNoise()])

would always apply 1 to 3 of the 3 augmenters (i.e. at least one).
iaa.OneOf([iaa.Dropout(...), iaa.Affine(...), iaa.AdditiveGaussianNoise()])

always applies exactly 1 of the 3 augmenters.
iaa.Sequential([iaa.Dropout(...), iaa.Affine(...), iaa.AdditiveGaussianNoise()])

always apply 3 of the 3 augmenters.
If you use Sometimes or augmenters with probabilities (e.g. Invert), there is always a small chance that none of them will be applied. This can currently only be avoided if you add some other augmenters of which at least one is always applied (using above methods).

I've taken your advice, but I've detected that no augmentation can still happen with:
iaa.SomeOf((1, None) ...)

Everything else works, except after checking the output augmentations, I found no augmentation was applied to one specific image. This image is nothing special. Testing in isolation allowed me to augment the image normally. It's just as I used it a batch of many images, a single image kept not being augmented.
I created an "augmentation schema" which I apply:
{
  "augs": [{
      "type": "Fliplr",
      "args": [1.0]
    },
    {
      "type": "Flipud",
      "args": [1.0]
    },
    {
      "type": "CropAndPad",
      "kwargs": {
        "percent": [-0.25, 0.25],
        "pad_mode": "constant"
      }
    },
    {
      "type": "Multiply",
      "args": [
        [0.9, 1.1]
      ],
      "kwargs": {
        "per_channel": 0.5,
        "name": "Multiply"
      }
    },
    {
      "type": "GaussianBlur",
      "kwargs": {
        "sigma": [0, 0.1],
        "name": "GaussianBlur"
      }
    },
    {
      "type": "AdditiveGaussianNoise",
      "kwargs": {
        "scale": [0, 5.1],
        "per_channel": true,
        "name": "AdditiveGaussianNoise"
      }
    },
    {
      "type": "Affine",
      "kwargs": {
        "scale": {
          "x": [0.8, 1.2],
          "y": [0.8, 1.2]
        },
        "translate_percent": {
          "x": [-0.2, 0.2],
          "y": [-0.2, 0.2]
        },
        "rotate": [-50, 50],
        "shear": [-8, 8],
        "order": 3,
        "mode": "constant"
      }
    }
  ],
  "mask_filter": ["Multiply", "GaussianBlur", "AdditiveGaussianNoise"]
}

def parse_aug_config(config_file):
    aug_config = json.load(config_file, cls=AugmentationDecoder)
    augs = []
    for aug in aug_config['augs']:
        aug_constructor = getattr(iaa, aug['type'])
        aug_args = aug.get('args', [])
        aug_kwargs = aug.get('kwargs', {})
        augs.append(aug_constructor(*aug_args, **aug_kwargs))
    augs_mask_filter = aug_config['mask_filter']
    aug_pipeline_template = iaa.SomeOf((1, None), augs, random_order=True)

    def masks_activator(images, augmenter, parents, default):
        if augmenter.name in augs_mask_filter:
            return False
        else:
            return default

    masks_hook = ia.HooksImages(activator=masks_activator)
    return (aug_pipeline_template, masks_hook)

def augment_image_mask(image, mask, aug_pipeline_template, masks_hook):
    aug_pipeline = aug_pipeline_template.to_deterministic()
    if mask is not None:
        (image_aug, *_) = aug_pipeline.augment_images(np.array([image]))
        (mask_aug, *_) = aug_pipeline.augment_images(
            np.array([mask]), hooks=masks_hook)
    else:
        (image_aug, *_) = aug_pipeline.augment_images(np.array([image]))
        mask_aug = None
    return (image_aug, mask_aug)

I'm running imgaug 0.2.6.
Are you sure there's no way that iaa.SomeOf especially with the augmentations that I'm using that 0 augmentations may be applied?

Your json code contains a lot of lists for parameter ranges, e.g. "sigma": [0, 0.1]. These are parsed by json.load as lists and provided to the augmenter constructors without changes. The augmenters then interpret these parameters as lists of allowed values and pick randomly from these lists. So for the above sigma example you would get either 0 or 0.1 sampled, but not anything in between. To get a uniform distribution, you have to provide either (0, 0.1) (i.e. a tuple) or imgaug.parameters.Uniform(0, 0.1) as a parameter.
There is of course also a chance that a uniform distribution randomly sampled 0 or 0.1.
btw
(image_aug, *_) = aug_pipeline.augment_images(np.array([image]))

can probably be replaced by
image_aug = aug_pipeline.augment_image(np.array([image]))

Oh... well JSON only has lists, I wonder what's a good way to integrate the difference between lists and tuples in our augmentation schema.

I think the best way for now is always interpret the lists as tuples, as that was what I originally was doing before I extracted the code into a JSON schema.


uninitialized constant OS_VERSION

~/.linuxbrew/Library/Homebrew/global.rb:30:in `<top (required)>': uninitialized constant OS_VERSION (NameError)
        from /usr/share/rubygems/rubygems/core_ext/kernel_require.rb:55:in `require'
        from /usr/share/rubygems/rubygems/core_ext/kernel_require.rb:55:in `require'
        from ~/.linuxbrew/Library/brew.rb:15:in `<main>'

This happened after a brew update. I have tried removing ~/.linuxbrew and ~/.gem and reinstalling and any brew command I run after gives this result.

Same here

Dup of #648

If brew update doesn't work you'll need to run:
git fetch && git reset --hard origin/master

Inside your Linuxbrew install to get brew functional again. Hopefully not many people updated over the last few hours. Sorry about this folks.

@mikemcquaid Sorry, Mike. Slip of the finger.


Copy should be named CopyOperation

xenon.files.Copy is in fact not a copy (noun) of something, but a copy (verb) operation. I think we should change its name to CopyOperation to reflect this.

xenon.engine.files.CopyImplementation then also needs to be xenon.engine.files.CopyOperationImplementation
xenon.files.CopyStatus then also needs to be xenon.files.CopyOperationStatus
xenon.files.CopyOption then also needs to be xenon.files.CopyOperationOption

After @jmaassen 's draft for a design document (here), perhaps Copy should be CopyOperationHandle?

In API change branch for 2.0 we decided to go for CopyHandle instead.


docs: add user-guide

Move some common docs into ROOT and use xref
Add initial user guide connecting from Drogue Cloud to Drogue Dopplegänger.

I'm working on the command example for the user guide as well, I just wanted to get this in before I deviate too much from main.


Feature: replace web3connect with onboardjs

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

Some things without looking at the code:

I went to the PR review link and immediately it showed me "Connect a wallet" popup - It shouldn't

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

@francovenica could you please check if issues 3 and 5 are applicable to other environments? (or maybe you are already found similar bugs)

@mikheevm I was not able to reproduce 1) 2) and 4)

and 5) are probably not specific to this PR as I have seen them before.

Would be good if we could try to fix these issues today (if they are reproducible), but merge nevertheless as users are waiting for the hardware wallet integration and we should aim to release next week if possible.

looks like 1) happened because I had tested the PR previously and it saved my last used provider

so 2 and 4 are fixed by this PR i think https://github.com/gnosis/safe-react/pull/625

@francovenica could you please check if issues 3 and 5 are applicable to other environments? (or maybe you already found similar bugs)

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr456--safereact.review.gnosisdev.com/app


I am reverting this to avoid these tests resulting in more problems than solutions in the future.
See the related discussion.

@danielbate I don't have the full context, and your points make sense.
Could you please summarize them in a new issue?
It'll be great to address this pattern more broadly.

@Torres-ssf We can close this for now and tackle the matter via https://github.com/FuelLabs/fuels-ts/issues/2373.

@Torres-ssf We can close this for now and tackle the matter via #2373.

Ok @arboleya, thanks


Menubar Flyout item accelerators are aligned differently when opened for the second time

Describe the bug
Within a MenuBar the MenuFlyoutItem control's accelerator keys are aligned to the right when you first open the flyout, but all consequent opens will align the accelerator keys to the left.
Steps to reproduce the bug
<MenuBar>
        <MenuBarItem Title="File">
            <MenuBarItem.Items>
                <MenuFlyoutItem Text="AAA">
                    <MenuFlyoutItem.KeyboardAccelerators>
                        <KeyboardAccelerator Key="A" Modifiers="Control" />
                    </MenuFlyoutItem.KeyboardAccelerators>
                </MenuFlyoutItem>
                <MenuFlyoutItem Text="BBB">
                    <MenuFlyoutItem.KeyboardAccelerators>
                        <KeyboardAccelerator Key="A" />
                    </MenuFlyoutItem.KeyboardAccelerators>
                </MenuFlyoutItem>
            </MenuBarItem.Items>
        </MenuBarItem>
    </MenuBar>

With this example code:

Open the File menu (Accelerators are aligned to the right)
Close the File menu
Open it again (Accelerator are aligned to the left)

Expected behavior
alignment should be consistent when opening the flyout
Screenshots
First open:

All consequent opens:

NuGet package version
WinUI 3 - Windows App SDK 1.3.3: 1.3.230724000
Windows version
Windows 10 (21H2): Build 19044
Additional context
No response

Interestingly, it was a bug that caused it to show correctly the first time otherwise the actual bug would have caused them to always be left aligned.
The menu aligns the accelerator values by explicitly setting the size of all the TextBlock that contains the accelerator key text to the size of the largest one, and right aligning the TextBlock.  The first calculation for these widths occurs too early so the TextBlocks are just the size of their content and when they are right aligned, the appear justified to the right.  However, the second (and subsequent) times that the dialog is shown it has computed the correct size of the largest Text blocks and so all the textblocks are the same size and aligned on both the right and left.  BUT, the text in the text block was still being left aligned.  Consequently, the accelerator strings appeared left aligned on the second (and subsequent) invocations.
The fix is in and scheduled for 1.5


From private repo

@v-alje : Thanks for your contribution! The author, @mimig1, has been notified to review your proposed change.


ROX-11283 re-sync tests with recorded events (final state of deployments)

Description
This PR aims to provided testing for sensor's re-sync behaviour using pre-recorded k8s events.
The events were recorded using the local-sensor tool.

Create the trace.jsonl file containing the k8s events.

go run tools/local-sensor/main.go -record -record-out=trace.jsonl -resync=0s

Create the central-out.bin file containing sensor's output events.

go run tools/local-sensor/main.go -replay -replay-in=trace.jsonl -resync=10s -delay=0s -format=raw -central-out=central-out.bin

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Manually testing with:
LOGLEVEL=panic go test github.com/stackrox/rox/sensor/tests/replay -count=1

/test all

/test style-checks

LGTM. The only thing I'm not too happy about is how k8s/fake.go (this also applies to other debugger/* packages) is becoming a file with utility functions. Do you think it makes sense to extract all the record/replay functionalities into its own package?

Yeah, I think so too. Maybe we can create a utils package at sensor/utils. Or maybe sensor/utils/replay to separate it from possible future utilities. What do you think?


Disconnects do not return ConnectionError for lazy connections

The documentation for lazy connections says that

If the connection with redis is lost, all commands will raise the ConnectionError exception, to indicate that there's no active connection. However, if the reconnect argument was set to True during the initialization, it will continuosly try to reconnect, in background.

My understanding is that even if reconnect is set to True, I should expect a ConnectionError. This would be useful in the case when it is important to reasonably quickly return an HTTP request with a failure if there are no available redis connections. However, instead the deferred never fires and txredisapi keeps trying to reconnect. For example, examples/twistedweb_server.py will not return an error to a GET request if there is no active redis connection.

I think docs are misleading about this point. reconnect=True is certainly coded to not raise any errors and is trying to make reconnects invisible to the end user. ConnectionError are only raised for commands that was sent before connection failure but not received responses until disconnected.
I think if you need response from Redis or failure within guaranteed period of time, you can't rely on ConnectionError anyway. Network connection might stall, server might be turned off, wire might be broken and you won't get any response (including failure) until TCP will drop connection by timeout. It seems to me that the only reliable way to guarantee response time is to do timing out on the client side. Deferred.addTimeout() from Twisted 16.5 can probably help.

Thanks! Makes sense.


Add TypeScript support (tslint)

We nice work! Keep it up. Would tslint be a good fit for the linters?

Hi @mCzolko, thanks!
Since TSLint is deprecated I'd say no. You should switch to ESLint which also supports Typescript, see this test for an example. Is there a specific reason why you need TSLint?

Oh no, I didn't know that it will be deprecated, sorry, I should switch in my project away from TSLint. Thanks!

@ocean90 It is "deprecated". I use quotes because it is still used out of the box by Angular which is certainly not an edge case. There are definitely more Angular users than golint users.


⚠ GitLab Server has degraded performance

In 4d0ebad, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 13394 ms

Resolved: GitLab Server performance has improved in 3094c4b.


Added better types for typescript

Added possibility to use:
import VueScrollTo from "vue-scrollto";

VueScrollTo.scrollTo(element.$el, {offset: -125, duration: 1});

Hey @ChristianStornowski thanks for the PR!
I'm not a TS user (yet), but I wonder if these type definitions will account for the returned cancel function from this.$scrollTo and VueScrollTo.scrollTo?

Sorry. I missed this cancel function. :-/
I added this function

:tada: This PR is included in version 2.17.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: adding new metric: up down sum observer

Which problem is this PR solving?

Fixes #1115

Short description of the changes

It adds new metric UpDownSumObserver

What is the plan to export this new Metric type to Prometheus exporter?

What is the plan to export this new Metric type to Prometheus exporter?

This is currently already exported as point has value which is number and this number will be exported

@open-telemetry/javascript-approvers ^^

Just left a few nits, but as I leaving them realized that we seem to use the word metric  to represent the instruments, so they can be ignored.


Allow allocation of the sinlge IP in a host prefix

Codecov Report

Merging #109 (b050220) into master (af2969e) will increase coverage by 0.22%.
The diff coverage is 100.00%.

:exclamation: Current head b050220 differs from pull request most recent head 4d6cd2a. Consider uploading reports for the commit 4d6cd2a to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #109      +/-   ##
==========================================
+ Coverage   70.31%   70.54%   +0.22%
==========================================
  Files          10       10
  Lines        1132     1134       +2
==========================================
+ Hits          796      800       +4
+ Misses        213      212       -1
+ Partials      123      122       -1

Impacted Files
Coverage Δ

prefix.go
79.44% <100.00%> (+0.10%)
:arrow_up:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remaining suggestions from the second preprint version

@tamunro suggested several additional revisions after https://github.com/greenelab/scihub-manuscript/pull/19 was merged. I'm copying them here so we don't forget to address them in the next round of revisions.
From https://github.com/greenelab/scihub-manuscript/pull/19#commitcomment-25037077:

"are without subscription access to articles published by Oxford University Press since May 2017" → "cancelled all Oxford University Press subscriptions in May 2017"

Also see the following discussion on how subscriptions work: access is perpetual for years that have been subscribed to:

Yes, perpetual access contracts have been common practice for 20 years now (e.g. https://doi.org/10.1300/J123v34n01_25 and https://doi.org/10.5860/lrts.50n2.91). You may want to clarify this in a sentence, but I don't think it deserves much of your time or of the paper.

From https://github.com/greenelab/scihub-manuscript/pull/19#commitcomment-25037158:

"access outages" → "the cancellation of all subscriptions by multiple institutions, an extremely drastic step given that Elsevier is the world's largest scientific publisher".

From https://github.com/greenelab/scihub-manuscript/pull/19#commitcomment-25103300

It might be worth spelling out the implications of this with widely-known examples, i.e. a centralized service with a single point of failure is more vulnerable to censorship than a decentralized one (e.g. Napster vs Bittorrent). Although sharing tens of millions of rarely-read papers would involve different challenges than sharing popular movies, some people like a challenge.

@tamunro feel free to propose these changes yourself by making a pull request!
Info on how the formatting works is here. You can probably make the changes entirely through the GitHub interface (and won't have to mess with git on your computer).

Regarding cancellations, I just came across SPARC's Big Deal Cancellation Tracking (heard through this post). Worthy of more investigation and potentially a reference.


When I debug the DeepMimicCore project, somthing wrong happen

I can get the DeepMimicCore.py file and run the DeepMimic project successfully. However, I can debug the DeepMimic project while can not debug DeepMimicCore project.  Anyone know the solution? Thank a lot in advance.

DeepMimicCore is a python module, it's not a application, so you won't be able to run it like a usual app. But if you want to debug the c++ code by building the "Debug" configuration. In visual studio, you can select the "debug" option under the configuration manager. This will build a c++ executable that you can attach a debugger.

DeepMimicCore is a python module, it's not a application, so you won't be able to run it like a usual app. But if you want to debug the c++ code by building the "Debug" configuration. In visual studio, you can select the "debug" option under the configuration manager. This will build a c++ executable that you can attach a debugger.

@xbpeng Thanks for your patience sincerely and it is very clear. But after I finish the "Debug" configuration and start debugging, I get some other errors. Should I pass some arguments?

After I have filled out the command arguments, it also dose not work.

The command line arguments are not the same as the contents in the arg files. The arguments are the same as the ones used to run the python scripts. Try:
--arg_file args/run_humanoid3d_spinkick_args.txt

The command line arguments are not the same as the contents in the arg files. The arguments are the same as the ones used to run the python scripts. Try:
--arg_file args/run_humanoid3d_spinkick_args.txt

@xbpeng  I have tried it. It is not still work.

I attempt to click the retry button and it shows that DeepMimicCore.exe has stopped working. What is something wrong in my software configuration or environment?

there's an error in the console, what does it say?

It works. The solution is to copy the two directories into DeepMimicCore directory.

But the motion isn't done fluently, like dropped frame.  And it can not finish the spinkick.

The c++ code only runs the physics simulation, it doesn't run the policy. So the character will just fall over. That is expected behavior. All of the learning code is on the python side.

Fine, I get it. Thanks for your patient replies again.


File upload from android device to node server

I am trying to upload a file from an Android device using ionic 3 framework to node server. It giving the below error. Same code is working fine from iphone.
0|npm      | Dev : Error error { Error: stream ended unexpectedly
0|npm      |     at Form.<anonymous> (/home/ec2-user/application/apiserver-v2/node_modules/connect-multiparty/node_modules/multiparty/index.js:764:24)
0|npm      |     at Form.emit (events.js:187:15)
0|npm      |     at Form.EventEmitter.emit (domain.js:442:20)
0|npm      |     at finishMaybe (_stream_writable.js:654:14)
0|npm      |     at endWritable (_stream_writable.js:662:3)
0|npm      |     at Form.Writable.end (_stream_writable.js:602:5)
0|npm      |     at IncomingMessage.onend (_stream_readable.js:634:10)
0|npm      |     at Object.onceWrapper (events.js:275:13)
0|npm      |     at IncomingMessage.emit (events.js:187:15)
0|npm      |     at IncomingMessage.EventEmitter.emit (domain.js:442:20)
0|npm      |     at endReadableNT (_stream_readable.js:1091:14)
0|npm      |     at process._tickCallback (internal/process/next_tick.js:174:19) status: 400, statusCode: 400 }

Hi @dougwilson, I am using the Ionic framework file transfer plugin to upload the image file. Using the same code on iPhone is working fine, but from the Android phone is not working. Could you please let me know, what information, you need from me. I will try to provide you asap.

Hi @yogeshbansal I'm just looking for you to either (a) make a pull request with a fix or (b) provide a reproduction case I can look into. Thanks!

Closing due to inactivity.


Remove Privacy links from the footer

Remove the following from the footer until we get the content.

Privacy
Terms
Cookies
Just comment them. So that we can add them quickly once the content is available.

Can I work on this


ci: adding configuration for lighthouse ci in staging environment #273

Codecov Report

Merging #276 (3749d61) into main (491cf1f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #276   +/-   ##
=======================================
  Coverage   76.68%   76.68%
=======================================
  Files          31       31
  Lines         386      386
  Branches       53       53
=======================================
  Hits          296      296
  Misses         87       87
  Partials        3        3

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 491cf1f...3749d61. Read the comment docs.


Add role attribute to div

Summary
Add a role attribute for accessibility purposes to terra-section-header
Additional Details
@JakeLaCombe originally suggested adding this as a part of this PR.
Adding this now.
As he mentions in https://github.com/cerner/terra-core/issues/1371, I'm required to keep the eslint-disable statement wrapping the outermost div of terra-section-header.
Thanks for contributing to Terra.
@cerner/terra

@dylan-sftwr Looks like terra-toggle-section-header snapshots need to be updated as well.

@bjankord done bb548eec


fix: make arrow_op handle optional tagged values

Internal JIRA: https://wandb.atlassian.net/browse/WB-14787
Fixes an issue where our arrow_op abstraction did not properly handle optional tagged values. It did not perform any tag stripping in that case and instead passed in an AWL with tags present, breaking a key assumption of arrow_op resolvers.
This PR updates arrow_ops to correctly perform tag stripping/rewrapping for optional tagged input types.
Note: this only fixes 1 of 2 bugs in the above ticket, there is another that will be fixed in a future PR.

merging as test-local-integration did pass, but its not showing up here for some reason


Q: WebAPI

Following your guides on securing an WebAPI using this lib. My questiong is, if the API needs some simple claims like role, is there anyway those can be extracted from the token so that you don't have to look those claims up using get user info?

You can configure the Claims on the Scope configuration in IdSvr to include additional claims about the user in the access token.


NIMBLE: nimble_port.h uses undefined  CONFIG_NIMBLE_PINNED_TO_CORE symbol (IDFGH-1266)

This file is still using CONFIG_NIMBLE_PINNED_TO_CORE but it looks like it should now be CONFIG_BT_NIMBLE_PINNED_TO_CORE

@ecc1 Can you please update the NimBLE submodule and check. I can see the updated change here: https://github.com/espressif/esp-nimble/blob/681a639c1cc24ea6b8bdb36c9a571dae8462614d/porting/nimble/include/nimble/nimble_port.h#L25

This was my error due to an out-of-date git submodule. Closing.


[Bug][Android]: offlineManager.subscribe logs error Unable to set pack observer [TypeError: undefined is not a function]

Mapbox Implementation
Mapbox
Mapbox Version
10.13.0
Platform
Android
@rnmapbox/maps version
10.0.6
Standalone component to reproduce

Observed behavior and steps to reproduce
no native android implementation in<EMAIL_ADDRESS>for method setPackObserver for call
await MapboxOfflineManager.setPackObserver(packName);
from<EMAIL_ADDRESS>from subscribe method
STR:

start loading offline map in android app
close the app
open app and call
MapboxGL.offlineManager.subscribe(packName, ...);

Expected behavior
no error logged
Notes / preliminary analysis
No response
Additional links and references
No response

I'm experiencing the same issue on 10.0.5

@ShepelE pls include a component, we can use to reproduce the issue, thanks!


Certain apps don't allow Google logins, others do using methods in other issues.

Homework

[X] I took the time to write a good, descriptive issue title
[X] I read nativefier --help and API.md.
[X] I checked CATALOG.md for community suggestions & workarounds.
[X] I searched existing issues, open & closed. Yes, my question is new.
[X] I'm running the latest version.

Your question
So far I've encountered 2 different sites that don't allow me to login with Google even with the methods mentioned in other issues. I can either get it to open a blank Google Login page, or a page to accounts.youtube.com with the error "That’s an error. The server cannot process the request because it is malformed. It should not be retried. That’s all we know."
Steps to reproduce
nativefier --honest https://typed.do/
nativefier --honest https://chroniclehq.com/

Debug info
No response
Nativefier version
50.0.1
Node.js version
16.15.0
npm version
9.1.1
OS
MacOS Ventura 13.1

I would appreciate any help with this as these are 2 crucial apps for my everyday needs.


Fix set_params with sequence

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Proposed Changes
Teach set_params to work with list and tuples.
Closing issues

Codecov Report

:exclamation: No coverage uploaded for pull request base (automl-2.0@447a032). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@              Coverage Diff              @@
##             automl-2.0    #1201   +/-   ##
=============================================
  Coverage              ?   55.14%
=============================================
  Files                 ?      177
  Lines                 ?    10467
  Branches              ?        0
=============================================
  Hits                  ?     5772
  Misses                ?     4695
  Partials              ?        0

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


tensorflow.math.abs crashes Python on Windows

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Trivial example given.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 Enterprise
TensorFlow installed from (source or binary): using pip
TensorFlow version (use command below): v1.12.1-48227-g77eafffd690 2.5.0-dev20201230 tf-nightly==2.5.0.dev20201230
Python version: 3.8.5
CUDA/cuDNN version: CUDA 11.1, cuDNN 8.0.5
GPU model and memory: RTX 3080 10 GB

EDIT:
I've also managed to reproduce the issue using the same software, but on a GTX 970, as well as on a GTX 1060.
Describe the current behavior
Any usage of tensorflow.math.abs crashes Python. Functions which depend on tensorflow.math.abs on the backend also crash, such as keras.losses.Huber and keras.losses.MeanAbsoluteError.
Describe the expected behavior
It should just compute absolute values without crashing.
Standalone code to reproduce the issue
This trivial example should yield the error:
import tensorflow
x = tensorflow.convert_to_tensor([0], dtype='float32')
y = tensorflow.math.abs(x)

Other info / logs
2020-12-30 12:47:20.566855: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library nvcuda.dll
2020-12-30 12:47:20.597895: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1760] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce RTX 3080 computeCapability: 8.6
coreClock: 1.74GHz coreCount: 68 deviceMemorySize: 10.00GiB deviceMemoryBandwidth: 707.88GiB/s
2020-12-30 12:47:20.598347: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll
2020-12-30 12:47:20.626950: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cublas64_11.dll
2020-12-30 12:47:20.627203: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cublasLt64_11.dll
2020-12-30 12:47:20.631687: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cufft64_10.dll
2020-12-30 12:47:20.635184: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library curand64_10.dll
2020-12-30 12:47:20.639243: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cusolver64_10.dll
2020-12-30 12:47:20.642267: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cusparse64_11.dll
2020-12-30 12:47:20.643017: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudnn64_8.dll
2020-12-30 12:47:20.643278: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1902] Adding visible gpu devices: 0
2020-12-30 12:47:20.644054: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-12-30 12:47:20.644953: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1760] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce RTX 3080 computeCapability: 8.6
coreClock: 1.74GHz coreCount: 68 deviceMemorySize: 10.00GiB deviceMemoryBandwidth: 707.88GiB/s
2020-12-30 12:47:20.645412: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll
2020-12-30 12:47:20.645637: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cublas64_11.dll
2020-12-30 12:47:20.645846: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cublasLt64_11.dll
2020-12-30 12:47:20.646045: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cufft64_10.dll
2020-12-30 12:47:20.646244: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library curand64_10.dll
2020-12-30 12:47:20.646443: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cusolver64_10.dll
2020-12-30 12:47:20.646648: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cusparse64_11.dll
2020-12-30 12:47:20.646859: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudnn64_8.dll
2020-12-30 12:47:20.647092: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1902] Adding visible gpu devices: 0
2020-12-30 12:47:21.201575: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1300] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-12-30 12:47:21.201806: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1306]      0
2020-12-30 12:47:21.201945: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1319] 0:   N
2020-12-30 12:47:21.202263: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1446] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 6703 MB memory) -> physical GPU (device: 0, name: GeForce RTX 3080, pci bus id: 0000:01:00.0, compute capability: 8.6)
Process finished with exit code -1073741571 (0xC00000FD)

@instigatorofawe
I have tried in colab with TF nightly version(2.5.0-dev20201230) and I am not seeing any issue. Please, find the gist here.But colab uses cuda 10.1 . Might be problem related to Cuda 11.1.Thanks!

@instigatorofawe
I have tried in colab with TF nightly version(2.5.0-dev20201230) and I am not seeing any issue. Please, find the gist here.But colab uses cuda 10.1 . Might be problem related to Cuda 11.1.Thanks!

I don't think it's a problem with CUDA 11.1 so much as it's a problem with CUDA on Windows. I'm able to run without issue on the same hardware in Ubuntu 20.04 using the same versions of CUDA and TensorFlow. I am downgrading and trying again, though.

Thank you for reporting this. I have verified that this is now fixed when building from source. The next nightly build should also contain the fix.


Using RoomView with Honey Select

I've just been playing around a bit in the Character Editor of Honey Select and activated Vive's RoomView mode (doubletap on system button) to untangle the cord.
While being crude the AR of RoomView made me actually feel like the model was standing in my own room, which was a pretty powerful feeling. Please do try yourself.
That being said, while in RoomView the application pauses and becomes transparent.
Would it be possible to integrate an RoomView option into the Chara-Editor of HoneySelect?
Maybe making the blue background transparent and superimposing the camera image onto it?
I believe you can get access to the camera feed quite easily so in my naive understanding it shouldn't be too difficult.
Alternatively there is the option to just change SteamVR to not pause/make the game transparent while in RoomView and change the chara editor background to transparent.
Anyways, I clearly don't have the knowledge how to get this done, but I do strongly believe it would be a very awesome feature. Please do try it out.
Wünsche Dir einen Guten Rutsch!

Now that's a cool idea. Too bad the image isn't stereoscopic, but I'll give it a shot.
Guten Rutsch!


Upvote button isn't sufficiently clear as to whether you've successfully upvoted an article

https://github.com/FreeCodeCamp/freecodecamp/pull/214
I can have something else trigger instead. Just let me know what you would like or if this feature works for you.


Can't get WSL2 socket working

Hi,
I've followed the WSL2 steps but it doesn't seem working:
$ source $HOME/omni-socat/ubuntu-bash.setup.sh
$ ssh-add -l
error fetching identities: communication with agent failed

However, it seems the $SSH_AUTH_SOCK has changed:
$ echo $SSH_AUTH_SOCK
/home/maicol07/.ssh/agent.sock

I'm using the 1Password proxy mode.
Can you help me?
Thanks

I've also tried with MSYS2 and Git Bash and, although they use a different system to connect to the agent, I get:
$ ssh-add -l
Error connecting to agent: Permission denied

@masahide any hint?

Is the setting "Enable Named pipe agent" enabled?

If I enable it, then 1Password proxy mode stops

Please check if socat is activated in WSL2.
~$ ps axuw |grep socat
masahide   317  0.0  0.0   6968  3444 ?        Ss   Apr17   0:00 socat UNIX-LISTEN:/home/masahide/.ssh/agent.sock,fork EXEC:/home/masahide/omni-socat/omni-socat.exe,nofork

To be sure, please check if the named pipe is running by using the following powershell command.
PS C:\Users\xxxx> Get-ChildItem \\.\pipe\|findstr open
------        1601/01/01      9:00              1 openssh-ssh-agent

It worked for a bit but now doesn't work anymore.
Outputs:
~$ ps axuw |grep socat
maicol07   581  0.0  0.0  10280  4652 ?        Ss   11:22   0:00 socat UNIX-LISTEN:/home/maicol07/.ssh/agent.sock,fork EXEC:/home/maicol07/omni-socat/omni-socat.exe,nofork
maicol07  2955  0.0  0.0   6480  2168 pts/0    S+   11:46   0:00 grep --color=auto socat

PS C:\Users\Maicol> Get-ChildItem \\.\pipe\|findstr open
------        01/01/1601     01:00              1 openssh-ssh-agent

Ok, I think I've found the issue. I currently use fish as default shell in WSL. When wsl boots it starts its own ssh-agent. When I enter bash works (and that because it correctly runs my .bashrc). When I exit the bash and return to fish, the old variables are back (since bash was a subshell).
Instead running fish from bash works since it inherit all the variables.
I would like to add a fish version of the bash script, however I'm a bit blocked with the main command substitution...

I'm sorry.
I only prepared the script for bash, so that's why.

I'm not sure I understand the syntax of fish,
The information on this page may be helpful. -> https://sean.mcgrath.nz/post/fish-shell-setup/
    setsid nohup socat UNIX-LISTEN:$SSH_AUTH_SOCK,fork EXEC:"$HOME/omni-socat/omni-socat.exe" >/dev/null 2>&1 & disown

Thank you! That works. I've also cleaned up a bit the script. Sending a PR now


Fix shutdown crash

Clearing breakpoints requires that there is a current context. If
there's not then just silently fail.
Cleaned up the sample and updated it to clear the current context
before disposing the debugger library.
Fixes #48

/cc @digitalinfinity @sethbrenith @jackhorton @mike-kaufman @akrantz

I verified that it fixes the crash.


[c2][decoder] Set default output to 8-bit pixel format

cases: testDefaultOutputColorFormat

Onevpl will change format according to bitstream,
but Google hopes the default of output is 8-bit
when the key "KEY_COLOR_FORMAT" is not set.
So we set back to 8-bit after decodeDecodeHeader
when m_pixelFormat->value wasn't changed by framework.

Tracked-On: OAM-119832

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web  for more details. For Binaries: /cactus-absp-or-local/celadon-merge/660


change manifest loading de-duping logic to be based on identity instead of manifest name

motivation: existing logic de-duped manifests avaialbe for graph resolution based on the mainfest name. this is causing issues when a repository is renamed and both the old and new URL are in the graph (as in the case of SR-11338), which means the graph never get resolves as one of the equired manifests is not found
changes:

change the logic to de-dup based on the identity instead of name. this means we are deduping by URLs instead of manifest names which is more consistent with the way the rest of the code base works and more helpful in most case/
move the lgoic to identify incorrect overrides to the resove function instead of being based on the fact a manifest is missing (which is a side effect)
added some code comments to explain what the code does
a bit of formatting/gardening in related code

Note that SwiftPM will still have a problems with situations like SR-11338 since the current identity is based on last segmentof the URL which is not unique enough. Further, renamed repositories still create issues when some of the graph pulls the old name and the rest pulls the new name. This is something we need to discvuss in the context of the package registry identity discussion. The good news is that with this change the errors would be more to the point and more actionable.
rdar://54556059
https://bugs.swift.org/browse/SR-11338

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


bumping js-slang again

seems there was a regression in the previous js-slang version. fixed now.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3164

Relevant Lines:
9780

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3164

Relevant Lines:
9780

💛  - Coveralls


[BEAM-9372][BEAM-9980] Configure Python versions for Spark Postcommit suite and Portable runner precommit suite.

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Seed Job

Run Portable_Python PreCommit

Run Python Spark ValidatesRunner

Run Portable_Python PreCommit

retest this please

R: @TheNeuralBit


50% probability at each node

Hi Steven,
Thank you for this great program. I'm using it for the first time now and I'm facing the following issue. I have a phylogeny with about 180 spp. I have just one trait that can have two states (files attached). The trait is not ordered (either social or non-social). When I run the function, I obtain a tree that shows a 50% probability at each node. I might be missing something but not sure what.
Could you help me? Thank you in advance!
The NWK tree comes in TXT format so I could upload it :)
file_trait.txt
plot.settings.txt
tree_PB_good.nwk.txt
Pol Sorigué

Hi Pol,
Thanks for pointing this out. I left your tree and trait data files as you sent them, but I adjusted your plot settings file to make the output pdf a little more pleasing. I got a result similar to what you described when I re-ran it using 1,000 samples. So, I increased to 10,000 samples, re-ran again, and then looked at the auto-discard to see what's happening. I have included that for you to see (gens_v_likelihood.pdf).
It seems that the complexity of your tree and tip data should require at least 5,000 samples. Only samples to the right of the red vertical line AND ALSO above the red horizontal line are summarized in your result. I included the commands that I pasted into R so you can try to recreate. From what I can tell, requesting 5k or 10k samples will give you a very similar outcome. So if you are in a rush, 5k samples is probably fine.
Please let me know if you encounter any other issues.
All my best,
Steven
MBASR_Pol.Sorigue.zip

Thank you so much Steven!
Now the ASR looks much nicer :)
Missatge de stevenheritage @.***> del dia dc., 28 d’ag.
2024 a les 22:14:

Hi Pol,
Thanks for pointing this out. I left your tree and trait data files as you
sent them, but I adjusted your plot settings file to make the output pdf a
little more pleasing. I got a result similar to what you described when I
re-ran it using 1,000 samples. So, I increased to 10,000 samples, re-ran
again, and then looked at the auto-discard to see what's happening. I have
included that for you to see (gens_v_likelihood.pdf).
It seems that the complexity of your tree and tip data should require at
least 5,000 samples. Only samples to the right of the red vertical line AND
ALSO above the red horizontal line are summarized in your result. I
included the commands that I pasted into R so you can try to recreate. From
what I can tell, requesting 5k or 10k samples will give you a very similar
outcome. So if you are in a rush, 5k samples is probably fine.
Please let me know if you encounter any other issues.
All my best,
Steven
MBASR_Pol.Sorigue.zip
https://github.com/user-attachments/files/16788975/MBASR_Pol.Sorigue.zip
—
Reply to this email directly, view it on GitHub
https://github.com/stevenheritage/MBASR/issues/11#issuecomment-2316263287,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZZAVFWWPUUIOXZGBX3UKUDZTY4TTAVCNFSM6AAAAABNI6SO4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGI3DGMRYG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Upcoming changes for version "2"?

There are a few things I would like to change in a version "2".
It's not really big fundamental changes, but they are "breaking" changes nonetheless requiring a major version bump.

[x] Make register and authenticate functions accept a single object containing all properties directly instead of "(name/creds, challenge, options)"
[x] The authenticator.synced property will be moved to credential.synced. I find it belongs there rather than in the authenticator, it's more intuitive/logic.
The userHandle property during registration will be removed. Instead you will be able to set user as either a string or as {id: ..., name: ..., displayName: ...} like in the original protocol.
The userHandle from the authentication result will be renamed userId
Get rid of expected.origin and rely only on the newer expected.domain
The debug flag will be renamed verbose
make a triple build (modules, commonjs, browser script)

...I also wonder if I should simply rename the "webauthn v2.0" into "passkeys v1.0" instead.
In case you have something else that you'd like to see changed/improved, now is the time to speak up!

Update:

no change to the debug flag after all. It's not only verbosity, it's also parsed data in results
moving authenticator.synced to credential.synced is non-trivial because it depends on a flag in the authenticator and not available before parsing. Yet the credential object is used browser-side too.

Currently, the implementation favors using the device itself as authenticator (platform) because the UI was smoother. I'm going to revert that though because security keys are now the only safe hardware bound option, which has simply higher security guarantees.

Preview of v2 is here: https://webauthn-ciy.pages.dev/

TODO:

add transports to allowCredentials
adapt response to toJSON

What about users using dynamicImport to use the module via such as NestJS? Will that version be depreaced?

Thank you. Basically NestJS is CommonJS based but with the help of dynamicImport and the current backend structure of the webauthn module of yours it's possible to be used and my question was related to this. I guess even tho you're not familiar with NestJS in any way maybe you still can tell a little information about if this code https://github.com/passwordless-id/webauthn/issues/35#issuecomment-1900264705 still may work or not. It's important to me.
If you still cannot, that's okay and thank you for your time

PS:
DynamicImport is part of NodeJS and it's not NestJS based just for clarification. :)

The packaging will be extremely similar, so you should have no issues.

make a triple build (modules, commonjs, browser script)

Modules = ES Modules, leaving the consumers' build pipeline to decide how they integrate the sources and enable static analysis and tree shaking?

Yes

closed by #59 🎉


SwiftTheme WARNING: Not found string key path

SwiftTheme WARNING: Not found string key path: Global.backgroundColor
I keep receiving this warning when I use setTheme in the viewDidLoad method. However, it is actually working fine and recognizes the Global.backgroundColor and sets it to the correct color depending on the current theme; there isn't any actual problem. I don't know how to stop this warning.

In theory, print this sentence at the same time that return nil, I can not think of other circumstances, this is a strange phenomenon. 😢

Same problem for me.


Column tooltip config no longer working

I believe we might have had a regression re. Column.tooltip

Thanks for catching, I had left a change in to set the tooltipField on the ag-Grid column def while experimenting with the tooltips, but that takes precedence over the tooltip config, so was causing ag-Grid to ignore the tooltipValueGetter we were setting

Great thanks for the quick fix - and happy new year! 🎉


[tls] Create static factory for FileWatcherCertificateProvider that validates PEM formatting of credentials.

Currently there is no check to ensure that credentials consist of valid PEM blocks before a TLS handshake commences. This PR creates a static factory for FileWatcherCertificateProvider (and marks the constructor as deprecated) which performs this validation check. The analogous work for StaticDataCertificateProvider will be done in a follow-up PR.

@gtcooke94 Any further concerns?


Remove Huawei Mobile Services

Hi,
I want to use this wallet, but I don't want "Huawei Mobile Services" to know about it.
is there a solution to build this wallet without "any Tracker" ?
Thanks.

I'm sorry, this may be more than we can handle. :(


nixos/tests: fix random failures of privacy test

Motivation for this change
I was rebasing another PR and realized this test will randomly fail on the first try (when the kernel hasn't configured a temporary address yet).
Things done

[x] Tested via nixos/tests/networking.nix
[x] Fits CONTRIBUTING.md.

cc @joachifm

Thank you.


Clone default settings before using it in generator

There are some parts in the renderer where the options object is being mutated by some functions (e.g _line function in renderer). If you are using custom parameters, because default parameters and the custom parameters are merged, the resulting object is a cloned object that we can work on. However, if custom parameters are not passed to these functions, the original value of the default parameters will be used. As a result, any custom object that doesn't specify a value will be overridden by the changing default value.
By cloning default options we can always preserve the original default values, which will stop bleeding of parameters between rendered elements.

Not needed anymore. roughnessGain has been removed from options and is not mutated


DBZ-5374: docker-maven-plugin needs to be upgraded for Mac Apple M1

Running 'mvn clean verify' for vitess connector (which includes launch docker for integration tests) would fail when run on an Mac M1 laptop:
[*ERROR*] Failed to execute goal io.fabric8:docker-maven-plugin:0.31.0:build (start) on project debezium-connector-vitess: Execution start of goal io.fabric8:docker-maven-plugin:0.31.0:build failed: An API incompatibility was encountered while executing io.fabric8:docker-maven-plugin:0.31.0:build: java.lang.UnsatisfiedLinkError: could not load FFI provider jnr.ffi.provider.jffi.Provider

The problem is specific to Apple M1 laptop, and is discussed further in: https://github.com/fabric8io/docker-maven-plugin/issues/1257
This is already fixed in docker-maven-plugin 0.39.1

@jpechane Thanks for merging the PR.  I also want to apply this PR to 1.9 branch.  Should I submit another PR or will the build system has a way to do that integration?

Never mind, I saw the PR already integrated into 1.9 branch.


Create EmptyAuthenticator

if the server doesn't provide username/pwd auth; we need a default no-authentication for that purpose. In case the server requests for password and client never provided one; the client should return an Error back

Maybe we should just use Option because AuthenticationNone which returns None feels over complicated

@ernestas-poskus also could be an options, but requires more effort.


fix: custom icon scheme

Types

[x] 🐛 Bug Fixes

Background or solution
getIcon basePath maybe has custom scheme
Changelog
fix: custom icon scheme

/publish


Data Factory problem

CopyPipeline-FactTripDataBlobToSQL failed with

Database operation failed. Error message from database execution : The given value of type String from the data source cannot be converted to type nvarchar of the specified target column..

input: FactTripData-Blob - /tripdata/tables/factTripDataoutput/000000_0
sample

19334855-337d-4695-a8ca-ac614133c51d|Twitter:2793763672|Input string was not in a correct format.|2016-04-22 14:39:17|-125.61639962299718|470|463|false|21|49.243319886275735|16.5530183922208

second position! (should be date time) output table columnt is nvarchar(20)
output: FactTripDataTemp-SQL
factTripDataoutput: factTripData.hql
tripDataWIP: FROM_UNIXTIME(minUnixTimestamp) as tripStartTime FROM tripDataWIPView
tripDataWIPView: MIN(unixtimestamp) as minUnixTimestamp FROM tripDataInt
tripDataInt: tripDataInt.hql
UNIX_TIMESTAMP(CONCAT(SPLIT(RecordedTimeStamp,'T')[0],' ',SUBSTR(SPLIT(RecordedTimeStamp,'T')[1],0,8)),'yyyy-MM-dd hh:mm:ss') as unixtimestamp FROM tripdata
tripdata is external table mapped to storage/rawdata/tripdata
sample of recorededtimestamp: 2016-04-22T13:31:37.1735276Z
And it probably fails
is it some way how to test this part of script?

I probably found the problem
testing on HDInsight cluster with access to my storage
second position is not time but VIN and as input is "Input string was not in a correct format." = 41chars
vin in factTripDataTemp is nvarchar(20)
i will update find where to fix it

fix hive query
fix storing to VIN from myDrive app

PR created with fixes


Ability to pass structured data types by value as kernel parameters

This PR is directly related to issue #4828.
Currently, we can only pass by value the following numpy items as kernel parameters: np.int8, np.int16, np.int32, np.int64, np.uint8, np.uint16, np.uint32, np.uint64, np.float16, np.float32, np.float64, np.complex64 and np.comples128 and other numpy builtin datatypes if their itemsize is either 1, 2, 4, 8 or 16 bytes.
This patch aims to enable pass by value for any structured data type built with numpy.dtype as kernel parameter.
It enhances current kernel launch capabilities of cupy without changing its interface. It is the responsibility of the user to define a numpy dtype that matches exactly device structure layout.
It provides the ability to:

pass cuda builtin vector types (char2, double3, int4) as kernel parameters.
pass "static arrays" of parameters such has struct double32 { double[32] val }; and other packed structures.
define and use any cuda composite type as kernel parameter (including structure of structures) such as

  struct custom_user_struct {
   int4 a;
   char b;
   double c[2];
   short1 d;
   unsigned long long int e[3];
 };

Implementation status:

[x] Patch cupy/cuda/function.pyx to enable pass by value to any dtype
[x] Add one test for each use case in  tests/cupy_tests/core_tests/test_function.py: builtin vector type, static array and custom struct
[x] Document this under User-Defined Kernels section
[x] Throw an error when numpy array argument is not of size 1 (this would be error-prone)
[x] Add some examples of complex structs

close #3359, close #4828

This is neat! thanks for the contribution :)

Everything has been implemented, including tests and documentation.
The cupy kernel lauch interface should not be affected at all by these changes.
This PR closes issue #4828.

Thanks for your nice contribution, @keckj! I've skimmed over your PR and am happy to help review, but before I find time to do that, could you apply this patch please? Your PR do not work without it on ROCm due to a bug in hipRTC...
diff --git a/tests/cupy_tests/core_tests/test_function.py b/tests/cupy_tests/core_tests/test_function.py
index 5f8d71004..f2970e5f2 100644
--- a/tests/cupy_tests/core_tests/test_function.py
+++ b/tests/cupy_tests/core_tests/test_function.py
@@ -4,12 +4,16 @@ import pytest
 import numpy

 import cupy
+from cupy.core import core
 from cupy.cuda import compiler
+from cupy.cuda import runtime
 from cupy import testing

 def _compile_func(kernel_name, code):
-    mod = compiler.compile_with_cache(code)
+    # workaround for hipRTC
+    extra_source = core._get_header_source() if runtime.is_hip else None
+    mod = compiler.compile_with_cache(code, extra_source=extra_source)
     return mod.get_function(kernel_name)

Also, you can add this phrase "close #3359, close #4828" to your PR description so that these two issues get automatically closed upon merging of this PR 🙂

Thanks for this quick feedback, the patch has been applied and the description has been changed to incorporate issues. I just began to use the CuPy module instead of pyCUDA, and I am not yet aware of cupy best practices such as complex numbers handling. Could you check that my additions to the documentation make sense ?

I fixed the documentation and added back the check for scalar arguments.
I also added three examples in cupy/examples/custom_struct.

Thanks for this in depth feedback ! I added a test for the examples and applied all of your suggestions. Hopefully I will give you less work for the next pull request :)
I think this PR opens the way for further improvements:

Simplify custom user type creation and type registration (as done in PyOpenCL for example).
Official support for cupy arrays with user defined types (at least for RawKernels).

Currently, in my examples, I think there is no guarantee that the arrays buffer allocated by the default cupy memory pools meet device side pointer alignment requirements. It seems that memory allocation routines from the CUDA driver or runtime API is always aligned to at least 256 bytes. This will need to be checked for further developments.

Jenkins, test this please

Jenkins CI test (for commit e574919647565d046b40f31de765e111ffee9268, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 374d34a0130ccec3e731269ab3bae750ad1e6ecc, target branch master) succeeded!

Windows build failed:
https://storage.googleapis.com/chainer-artifacts-pfn-public-ci/cupy-ci/70247/cupy_test_log.txt

____________________ TestCustomStruct.test_complex_struct _____________________

self = <example_tests.test_custom_struct.TestCustomStruct testMethod=test_complex_struct>

    def test_complex_struct(self):
        output = example_test.run_example('custom_struct/complex_struct.py')
>       assert re.match(
            r"Overall structure itemsize: \d+ bytes\n"
            r"Structure members itemsize: \[(\s*\d+){5}]\n"
            r"Structure members offsets: \[(\s*\d+){5}]\n"
            r"Complex structure value:\n"
            r"\s+\[.*\]\n"
            r"Kernel output matches expected value.",
            output.decode('utf-8'),
        )
E       AssertionError: assert None
E        +  where None = <function match at 0x000001E346772378>('Overall structure itemsize: \\d+ bytes\\nStructure members itemsize: \\[(\\s*\\d+){5}]\\nStructure members offsets: \\[(\\s*\\d+){5}]\\nComplex structure value:\\n\\s+\\[.*\\]\\nKernel output matches expected value.', 'Overall structure itemsize: 80 bytes\r\nStructure members itemsize: [16  1 16  2 24]\r\nStructure members offsets: [ ... structure value:\r\n  [((0, 1, 2, 3), (4,), (5., 6.), (7,), (8, 9, 10))]\r\nKernel output matches expected value.\r\n')
E        +    where <function match at 0x000001E346772378> = re.match
E        +    and   'Overall structure itemsize: 80 bytes\r\nStructure members itemsize: [16  1 16  2 24]\r\nStructure members offsets: [ ... structure value:\r\n  [((0, 1, 2, 3), (4,), (5., 6.), (7,), (8, 9, 10))]\r\nKernel output matches expected value.\r\n' = <built-in method decode of bytes object at 0x000001E420499D80>('utf-8')
E        +      where <built-in method decode of bytes object at 0x000001E420499D80> = b'Overall structure itemsize: 80 bytes\r\nStructure members itemsize: [16  1 16  2 24]\r\nStructure members offsets: [... structure value:\r\n  [((0, 1, 2, 3), (4,), (5., 6.), (7,), (8, 9, 10))]\r\nKernel output matches expected value.\r\n'.decode

tests\example_tests\test_custom_struct.py:26: AssertionError
_____________________ TestCustomStruct.test_packed_matrix _____________________

self = <example_tests.test_custom_struct.TestCustomStruct testMethod=test_packed_matrix>

    def test_packed_matrix(self):
        output = example_test.run_example('custom_struct/packed_matrix.py')
>       assert re.match(
            r"Kernel output matches expected value for type 'float'.\n"
            r"Kernel output matches expected value for type 'double'.",
            output.decode('utf-8'),
        )
E       AssertionError: assert None
E        +  where None = <function match at 0x000001E346772378>("Kernel output matches expected value for type 'float'.\\nKernel output matches expected value for type 'double'.", "Kernel output matches expected value for type 'float'.\r\nKernel output matches expected value for type 'double'.\r\n")
E        +    where <function match at 0x000001E346772378> = re.match
E        +    and   "Kernel output matches expected value for type 'float'.\r\nKernel output matches expected value for type 'double'.\r\n" = <built-in method decode of bytes object at 0x000001E442A4CB60>('utf-8')
E        +      where <built-in method decode of bytes object at 0x000001E442A4CB60> = b"Kernel output matches expected value for type 'float'.\r\nKernel output matches expected value for type 'double'.\r\n".decode

tests\example_tests\test_custom_struct.py:18: AssertionError

Forgot to reply:

Currently, in my examples, I think there is no guarantee that the arrays buffer allocated by the default cupy memory pools meet device side pointer alignment requirements. It seems that memory allocation routines from the CUDA driver or runtime API is always aligned to at least 256 bytes. This will need to be checked for further developments.

I am not sure what's been referred to here. If you're asking about CuPy's device mempools, it always returns memory in chunks of size 512 bytes:
https://github.com/cupy/cupy/blob/6badbe0e61d4b604cf3eac39e423f9d7ed22248d/cupy/cuda/memory.pyx#L809-L811
But I thought it's irrelevant for the purpose of constructing custom structs for passing by value? Maybe I missed something?

But I thought it's irrelevant for the purpose of constructing custom structs for passing by value? Maybe I missed something?

For passing struct by value I agree this is irrelevant.
However, In some examples I cast __global__ char* pointers to __global__ custom_struct* pointers. Those custom_struct pointers should be aligned at most on the biggest contained scalar type (I imagine this is 8B in Cuda). I just expect undefined behavior on user structs that are artificially aligned on more than 256B (or 512B when using CuPy memory pools). I think such extreme alignment requirements can only be achieved with alignas and __align__ directives.
The following example works on my machine test_alignment.py, but the compiler may optimize the copy assuming 4096B alignment for T2 and T3.

Jenkins, test this please

Jenkins CI test (for commit 993774fcf2f11fa48875a0b095c0d725b69e09ab, target branch master) succeeded!

I still don't think the 512B chunk size matters here. The compiler can freely make its own padding and alignment decisions, but in any case we will be retrieving the correct offsets, wouldn't we? As long as the memory allocated from either raw malloc or CuPy's mempool is enough to accommodate a custom struct (so that 512*N >= sizeof(custom_struct)) we should be ok.

Jenkins, test this please

Jenkins CI test (for commit 993774fcf2f11fa48875a0b095c0d725b69e09ab, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 8d9b5425b6f6cd53cd3804d7f7d5db461769a181, target branch master) succeeded!

@emcastillo This pull-request is marked as st:test-and-merge, but there were no activities for the last 3 days. Could you check?


Expose method for changing video background color

When you set video style to fit (setStyle(BaseVideoRenderer.STYLE_VIDEO_SCALE, BaseVideoRenderer.STYLE_VIDEO_FIT)) on PublisherKit or SubscriberKit, the video will be fit into the view and so it's possible to have black borders on some sides.
I would like to be able to change that color. IMO it makes sense to expose it via PublisherKit/SubscriberKit or BaseVideoRenderer. It seems like just storing it in renderer and using it in GLES20.glClearColor(); method should do the trick.

Thanks, @victorkifer, for this suggestion.
There's no plans for this API but that may change in the future.
CC'ing @Daniel085, since this is product request.


Fix Github action test failure

The following error https://github.com/medilies/xssless/actions/runs/10284119857/job/28459470742#step:7:28 does not occur on my machine.

Will not check exit codes internally and in tests.


kroach/OASIS-2743-release201-master

This objective-c-sdk P.R. will add to master branch the 2.0.x changes
that Optimizely released in objective-c-sdk 2.0.1 .

Coverage remained the same at 97.403% when pulling 1991c52f77e74ddfc05710ea11a230c316625435 on kroach/OASIS-2743-release201-master into 2778e55448a22639f6135149d1c268ac65d0316f on master.


ImportError: There is no such class as templates.speech_recognition.LM.custom_model.CustomModel

Hi,this error is came from "Training a Speech Recognizer"'s part2:

cd ../LM
python train.py RNNLM.yaml
I don't know why this error happen,because "templates.speech_recognition.LM.custom_model.CustomModel"  really exist in my local program,and nothing changed after i download this great program.

Thank you!

Hi, did you install with pip install or via Git ?

Hi,yes.Yestoday i uesd python version 3.7,so I try python3.8 today,and install with pip and git.This error also happened.

Hi,this mistake is path problem,my interpreter find "site-packages" directly,and  don't fine.After I put the floder of "templates" to "site-packages",interpreter can find,maybe i need to set a path for loacl program,Thank you for your help.


Compiler assertion fails if local package cannot be found

Description
#import "@our-local-ns/el-package:0.1.0"

$ ~/typst/typst compile --package-path=packages foo.typ
thread 'main' panicked at crates/typst-kit/src/package.rs:154:9:
assertion `left == right` failed
  left: "our-local-ns"
 right: "preview"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
$ file packages/our-local-ns/el-package/0.1.0
packages/our-local-ns/el-package/0.1.0: cannot open `packages/our-local-ns/el-package/0.1.0' (No such file or directory)

Reproduction URL
No response
Operating system
macOS
Typst version

[X] I am using the latest version of Typst

Regression from #4540 - seems like the preview namespace check below was removed:  https://github.com/typst/typst/blob/810491c9d31b614a435020f888fbd380e8e039a1/crates/typst-cli/src/package.rs#L60-L66
vs now:
https://github.com/typst/typst/blob/672f6e5f97c2bdcd3e94754c9486869bf7b8de56/crates/typst-kit/src/package.rs#L87-L91

Don't know how I maged that, easy fix, but I won't be home for a few hours,so you could do it too.

The way Git presents the changes in a refactoring PR like #4540 unfortunately makes it super hard to review so that both this and #4710 slipped by me.


Gutter not showing as in README

Describe the bug
I cannot able to see full gutters as shown in the README instead shon me as side colors. DO I have to enable some settings?
To Reproduce
Steps to reproduce the behaviour:
#app.py
def fibonacci(n):
    if n <= 2:
        return 1
    return fibonacci(n - 1) + fibonacci(n - 2)

def factorial(n):
    if n <= 1:
        return 1
    return n * factorial(n - 1)

#test_app.py
from app import fibonacci
class TestFibonacci:
    def test_fin(self):
        assert fibonacci(1) == 1

    def test_fin_10(self):
        assert fibonacci(10) == 55

    def test_fin_30(self):
        assert fibonacci(30) == 832040

run pytest --cov=app --cov-report xml
Expected behaviour
As shown in README
Screenshots

Desktop (please complete the following information):

OS: windows
Extension Version 2.9.1
VSCode Version 1.65.2

Additional context
Added coverage.xml contents
<?xml version="1.0" ?> <coverage branch-rate="0" branches-covered="0" branches-valid="0" complexity="0" line-rate="0.625" lines-covered="5" lines-valid="8" timestamp="1647330129847" version="6.2"> 	<!-- Generated by coverage.py: https://coverage.readthedocs.io --> 	<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd --> 	<sources> 		<source>C:\Users\Owner\OneDrive\PC Desktop\temp\pytest coverage</source> 	</sources> 	<packages> 		<package branch-rate="0" complexity="0" line-rate="0.625" name="."> 			<classes> 				<class branch-rate="0" complexity="0" filename="app.py" line-rate="0.625" name="app.py"> 					<methods/> 					<lines> 						<line hits="1" number="1"/> 						<line hits="1" number="2"/> 						<line hits="1" number="3"/> 						<line hits="1" number="4"/> 						<line hits="1" number="10"/> 						<line hits="0" number="11"/> 						<line hits="0" number="12"/> 						<line hits="0" number="13"/> 					</lines> 				</class> 			</classes> 		</package> 	</packages> </coverage>

got the answer. Goto settings> enable "show line coverage"


Topup and Dilute integration tests

As part of new endpoints of the new Bee 1.2 version that has topup and dilute postage batch, there were some challenges to make functional tests. As priority was to roll out compatible Bee-js we have decided to manually test its functionality and postpone the automated testing.

The problem is that calling these functions creates transactions on the Bee side, which then Bee awaits for enough confirmations. Currently, there is no way how to detect if this "confirmation period" is over or not.


Support for resource override

Fixes #97
Proposed changes

Add a resources element to dev.swap.deployment to allow for resource override. The cnd.yml would look something like this:

swap:
  deployment:
    name: vote
    container: vote
    command: ["python", "app.py"]
    resources:
      limits:
            cpu: "250m"
            memory: "128Mi"
       requests:
            cpu: "25m"
            memory: "32Mi"
mount:
  source: vote
  target: /app

This is an exploratory PR. If approved, I'll add tests and docs.

Should we have a way to unset resource limits/requests?
Or maybe unset by default and set resources if they are specified in cnd.yml?

I think we should just unset them for now. I think it makes more sense from a dev experience perspective.


NSG Flow Analytics can now be enabled using ARM/Bicep templates

The FAQ for NSG traffic analytics states that these cannot be created using ARM templates.  This is does not appear to be true, and these can be created using ARM or Bicep templates.  See the following:
https://learn.microsoft.com/en-us/azure/templates/microsoft.network/networkwatchers/flowlogs?pivots=deployment-language-bicep
https://github.com/Azure/azure-quickstart-templates/blob/master/quickstarts/microsoft.network/nsg-flow-logs-with-traffic-analytics/README.md

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

ID: aa32b540-55b2-1691-6f38-e8687565cecc
Version Independent ID: 8ad04d57-6370-c8dd-1049-1826a823bd12
Content: Traffic analytics FAQ - Azure Network Watcher
Content Source: articles/network-watcher/traffic-analytics-faq.yml
Service: network-watcher
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

@bowlerma
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@bowlerma
Thank you for your feedback!
I've just updated the Q&A in the Traffic analytics FAQ. It'll be reflected in the doc in the next 24 hours.
I'll close this GitHub issue for now, but feel free to comment here as necessary.
#please-close


Bump @mirai73/bedrock-fm library

adds library source code to facilitate debugging

Would like to see if this might solve #264


Build: Add grunt-cli to devDependencies

Removes need for custom code in .travis.yml, and makes it so that
npm install && npm test works out of the box without requiring
grunt-cli pre-installed.
This works because npm always makes exported bin commands available via the local node_modules/.bin directory in the environment PATH as used by of npm commands.

:+1: LGTM, I like the simplification

@Krinkle, we are avoiding the merge commits on the master branch, regardless how tempting the GH merge button is.
+1 to the changes

@leobalter, for my own information, what's the proper procedure for merging then? For GitHub projects I'm used to just hitting the button.

the button is great and easy, but it adds the merge commit.
The usual procedure is doing a local checkout on the PR branch, rebasing it with the master branch from this remote and merging it to master using the fast-forward mode (without the merge commit).
Using tools like hub helps the process, mine is like this:
$ hub checkout https://github.com/jquery/qunit/pull/932
$ git fetch upstream
$ git rebase -i upstream/master
$ git checkout master
$ git merge --ff-only <branch_name>
$ git push upstream master

At least it's possible to include a Closes gh-### on the rebased commit, to automatically close the respective PR.

@leobalter Thanks for the heads up. It's actually a pet-peeve of mine as well and I don't always avoid the button in my own projects but mistakenly thought it was preferred here to use the merge button. I'll avoid it going forwards :)


Add support for Mapkey caching

What this PR does / why we need it:

Protobag reference counts StringMap separately.
2. IL uses StringMap interface to fetch subkeys in addition to map[string]string
3. mixc prints out referenced attributes in the output.

Special notes for your reviewer:
This PR is resubmitted with fixes.
Previously it failed post submit because it had dropped support for plain map[string]string in favor of an interface. However, attribute producing adapters like Kubernetes, do inject map[string]string and therefore they must be supported.
Now IL supports map[string]string and the StringMap interface.
Release note:
Mixer protocol supports sub key level caching for stringmap attributes. This enables efficient caching based on request headers.

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
7a56cc457d452c65c031a23c344a114427b63124
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

Could you increase test coverage please? The attribute package was at 100%, it would be nice if it stayed there.


Pods are stuck at Pending status

What happened:
After the deployment using Helm is completed, Pods are stuck at pending status forever
What you expected to happen:
Pods are expected to running status, not pending
How to reproduce it (as minimally and precisely as possible):
Deploy using Helm
Anything else?:
I have found a solution for the problem by adding the node-role.kubernetes.io/presslabs-sys= to each node manually in Kubernetes using the following
kubectl label <NodeName> node-role.kubernetes.io/presslabs-sys=

Related to #86

Fixed by https://github.com/presslabs/stack/commit/9cdc97c7e794978b096b5037f092cd2f3ad834a5


Add static buyAsset() integration

Pending:

Make it dynamic (index according to the token value)
Add the sellAsset() integration


System properties are only read if configuration file exists in class path

Actual behaviour
Configuration via system properties is ignored as long as there is no "simplejavamail.properties" file in the class path.
Expected behaviour
Configuration via system properties is possible, even without a properties file.
Why it happens
This is because the ConfigLoader loadProperties that gets called in the static init block does NOT call readProperties. readProperties is where the system properties are used if set.
The loadProperties variant for InputStream does call readProperties, but is itself only called if the loadProperties for String finds a file on the class path.
Ergo to enable system properties you need to add a file to the class path.
How to reproduce

Use a basic builder, with no explicit optional settings, letting defaults happen
set a system property (for example simplejavamail.defaults.verifyserveridentity) to any valid, non-default value (e.g. false)
Check the setting in the Config of the Mailer the Builder built. In this example, OperationConfig isVerifyingIdentity should be false, but is true
add an empty "simplejavamail.properties" file to the classpath
Check the setting again. This time, it will have the correct value

I made a simple repro. Just run the test.sh (or read it)
simple-mail-repro.zip

Fix released in v8.12.0.


vcpkg unable to use BITS when running as a Windows Service

As you'd expect, vcpkg runs as part of our CI infrastructure. We run Jenkins as a Windows Service on Windows 10, and the user under which this service runs, is a domain user that has admin rights on that machine (ie it's not LocalSystem or Network Service).
We frequently see the following error message emitted by vcpkg:
Start-BitsTransfer : The operation being requested was not performed because the user has not logged on to the
network. The specified service does not exist. (Exception from HRESULT: 0x800704DD)
At C:\Jenkins\workspace\Maps-Windows-Analyze\windows_deps\vcpkg\scripts\fetchDependency.ps1:73 char:13
+             Start-BitsTransfer -Source $url -Destination $downloadPat ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Start-BitsTransfer], COMException
+ FullyQualifiedErrorId : System.Runtime.InteropServices.COMException,Microsoft.BackgroundIntelligentTransfer.Management.NewBitsTransferCommand

This happens when vcpkg is trying to use BITS to download a package.
If I hunt around on the internet, using the top line as the search query, I come across various suggestions, but I'd appreciate some clarity from the vcpkg authors. Obviously a build tool such as vcpkg must be runnable by automated systems. Is there something wrong in our setup? If there's a simple scriptable thing that we can run on the CI slaves to fix this issue, then that's OK (I'd love to know what that magic incantation is). But if it requires human button pressing, then I think this is a real problem.

We definitely intend for vcpkg to run in automated systems; in fact, this is the reason why we avoided requiring admin (with one exception: the first time you run vcpkg integrate for VS2015, but that should not affect automated scenarios).
It appears BITS could not be reached in your case for some reason. Is this consistent?
Either way, we use BITS for transferring files because it is faster, but we also have a fallback download mechanism.
ce9927f7327bc71ade246108a7d984deda6293fd aims to fix said fallback in case the BITS invocation throws.
(no need to rebuild vcpkg, just do a git pull).
Please let me know if this resolves the issue.

Something relevant:
This download is most likely trying to get CMake. We recently bumped the required version of CMake to 3.7.2 due to Visual Studio 2017 support. It is possible that the download was not triggered for your systems before because they already had CMake <3.7.2.

Thanks for the swift turnaround. I literally opened the ticket before I went to sleep, and woke up with the fix! I've updated to latest, and haven't seen any errors yet, but will report back again in a few hours.

Looks good! A bunch of CI machines have been building without any problems today.