The latest release doesn't work

Seems to be a trick of ghr we are using - the "latest" release on GitHub is still of the date Feb 18, which confuses the buildbot of archlinuxcn. As a result, archlinuxcn's package is not up to date.
(No wonder why I cannot type out names of MusicRay'n artists - many of those are created on March)

It seems there is no error log? and the build looks success.
https://github.com/outloudvi/mw2fcitx/runs/6888542675

昨天的没了，看build为Error: Process completed with exit code 1.

昨天的没了，看build为Error: Process completed with exit code 1.

Thanks for reporting! However, they are different issues. Moved to #9.

Closing thanks for archlinuxcn/repo@4e63947ab00de8f1b977dc7933f0934c4feefaa4.


Tests optimizations

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   91.68%   91.68%
=======================================
  Files          41       41
  Lines         601      601
=======================================
  Hits          551      551
  Misses         50       50

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 114567f...d29ee1d. Read the comment docs.


Load from SW fetches index with different version hash which breaks the app

First load:
everything works well, index file name is for example: index-6c4fa119.js
Second load from SW:
App doesn't load.
Index file name has different hash and error is thrown:
index-bc0ce142.js:1 Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
Node version: 18.17.1
Vite version: 4.4.9
Vite plugin pwa version: 0.17.0
App uses React version: 17.0.2
vite config:
import { sentryVitePlugin } from "@sentry/vite-plugin";
import svgr from 'vite-plugin-svgr';
import { VitePWA } from 'vite-plugin-pwa';

export default {
  root: './',
  // base: './',
  server: {
    port: 3000,
    host: '<IP_ADDRESS>',
  },
  build: {
    outDir: 'build',
    assetsDir: '',
    sourcemap: 'hidden',
  },
  plugins: [svgr(), sentryVitePlugin({
    org: "testapp",
    project: "test"
  }), VitePWA({
    registerType: 'autoUpdate',
    // buildBase: './build',
  })],
  // optimizeDeps: { include: ['firebase/app', 'firebase/firestore', 'firebase/auth'] }
};

Review your html pages cache headers, should have public, max-age: 0, must-revalidate + Date + Age, on hard reload still can fails, any asset inside assets folder should have public, max-age=31536000, immutable with Date + Age + optional Etag


Fixed  tag in seed/challenges/01-front-end-development-certific…

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of FreeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

…ation/basic-javascript.json

closes #8713

:sparkles: :sparkles: LGTM :+1: :sparkles: :sparkles:
:tada: Thanks for contributing @InsomniacSabbir and congratulations on your first PR submission to FreeCodeCamp! :tada:


Added back --args to xcrun simctl launch, was wrongly removed in #2540

Description:
Fix an issue introduced in #2540, where --args was dropped from xcrun simctl launch.
Missing: some sort of testing for this, to make sure args are being passed correctly.

@rotemmiz @LeoNatan @alon-ha what use case could we apply as a test in order to cover future regressions?

This was removed on purpose I think. @noomorph ?
It was being sent to AppleSimUtils which is incorrect.

🤦  Leo, but you told it is not needed?

@LeoNatan it was sent to xcrun simctl launch, not to AppleSimUtils, or am I getting you wrong?

It’s not needed for AppleSimUtils.

@LeoNatan, so it looks like a reliance coming from the app engine itself?

Yes, it’s buggy app behavior. This should be reverted and app fixed.

@rotemmiz @noomorph


conda recepies

nf-core/circrna modules that require package builds for the conda repository:

[ ] CIRIquant

PR: https://github.com/bioconda/bioconda-recipes/pull/38029
Not working because argparse>=1.2.1 and scipy==1.2.2 are not found using defaults/conda-forge/bioconda.
argparse>=1.2.1 is available from this channel: pdrops::argparse==1.2.2 but I cannot figure out how to tell my build to incorporate this channel. I have added it to my local conda config, to no avail...
scipy==1.2.2 does not exist on conda.
I have tried both pypi skeletons and generic noarch python skeletons here, so I am not sure how to proceed.

[x] DCC

PR: https://github.com/bioconda/bioconda-recipes/pull/37786

 posterity..
PR: https://github.com/bioconda/bioconda-recipes/pull/37786
The author of DCC made a pypi package called circtools available at https://pypi.org/project/circtools/. This is convenient for me as it bundles DCC and CircTest together, both of which are used in the workflow.
grayskull pypi circtools
conda build recipes/circtools

Successfully builds, pushed to anaconda to test
conda config --set anaconda_upload yes
anaconda upload \
    /Users/bdigby/opt/anaconda3/conda-bld/osx-64/circtools-1.2.1-py39_0.tar.bz2

TODO: This package works as expected, but I need it to be hosted by bioconda and not b.digby
https://anaconda.org/b.digby/circtools
bioconda-utils build --docker --mulled-test --packages recipes/circtools fails, see below:

 error log
(bioconda) bdigby@sr-loaner-c02ytamllvcf bioconda-recipes % bioconda-utils build --docker --mulled-test --packages circtools
14:40:40 BIOCONDA INFO Considering total of 1 recipes (circtools).
14:40:40 BIOCONDA INFO Processing 1 recipes (circtools).
14:40:41 BIOCONDA INFO Generating DAG
Loading Recipes: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:00<00:00, 69.64it/s]
14:40:41 BIOCONDA INFO 1 recipes to build and test:
circtools
14:40:41 BIOCONDA INFO Determining expected packages for recipes/circtools
Setting build platform. This is only useful when pretending to be on another platform, such as for rendering necessary dependencies on a non-native platform. I trust that you know what you're doing.
14:40:41 CONDA_BUILD WARNING Setting build platform. This is only useful when pretending to be on another platform, such as for rendering necessary dependencies on a non-native platform. I trust that you know what you're doing.
Updating build index: /Users/bdigby/opt/anaconda3/envs/bioconda/conda-bld

No numpy version specified in conda_build_config.yaml.  Falling back to default numpy value of 1.16
14:40:41 CONDA_BUILD WARNING No numpy version specified in conda_build_config.yaml.  Falling back to default numpy value of 1.16
Adding in variants from internal_defaults
14:40:41 CONDA_BUILD INFO Adding in variants from internal_defaults
Adding in variants from /Users/bdigby/opt/anaconda3/envs/bioconda/conda_build_config.yaml
14:40:41 CONDA_BUILD INFO Adding in variants from /Users/bdigby/opt/anaconda3/envs/bioconda/conda_build_config.yaml
Adding in variants from /Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/bioconda_utils-conda_build_config.yaml
14:40:41 CONDA_BUILD INFO Adding in variants from /Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/bioconda_utils-conda_build_config.yaml
defaults/noarch: 4.77MB [00:00, 30.2MB/s]                                                                                | 0/9 [00:00<?, ?files/s]
defaults/linux: 33.5MB [00:00, 39.3MB/s]                                                                               | 1/9 [00:00<00:03,  2.61files/s]
bioconda/linux: 29.0MB [00:01, 16.4MB/s]▍                                                                              | 2/9 [00:01<00:07,  1.07s/files]
defaults/osx: 30.8MB [00:03, 10.0MB/s]█████████████▋                                                                   | 3/9 [00:03<00:06,  1.07s/files]
conda-forge/noarch: 69.2MB [00:04, 16.6MB/s]██████████████████▉                                                        | 4/9 [00:04<00:06,  1.20s/files]
bioconda/osx: 23.4MB [00:04, 5.89MB/s]MB/s]]
bioconda/noarch: 26.1MB [00:00, 86.9MB/s]█████████████████████████████████                                             | 5/9 [00:08<00:08,  2.09s/files]
conda-forge/linux: 215MB [00:10, 20.8MB/s]██████████████████████████████████████████████████████▌                      | 7/9 [00:09<00:02,  1.30s/files]
conda-forge/osx: 187MB [00:20, 9.59MB/s]███████████████████████████████████████████████████████████████████▊           | 8/9 [00:19<00:03,  3.83s/files]
Downloading: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████| 9/9 [00:27<00:00,  3.09s/files]
Updating build index: /Users/bdigby/opt/anaconda3/envs/bioconda/conda-bld

Adding in variants from argument_variants
14:41:20 CONDA_BUILD INFO Adding in variants from argument_variants
Attempting to finalize metadata for circtools
14:41:20 CONDA_BUILD INFO Attempting to finalize metadata for circtools
bioconda/noarch                                      3.8MB @   4.0MB/s  1.1s
conda-forge/noarch                                  10.1MB @   4.4MB/s  2.6s
bioconda/osx-64                                      3.7MB @   1.3MB/s  2.9s
conda-forge/osx-64                                  24.7MB @   4.9MB/s  6.1s
Reloading output folder: /Users/bdigby/opt/anaconda3/envs/bioconda/conda-bld
Users/bdigby/opt/anaconda3/envs/bioconda/conda-b..  ??.?MB @  ??.?MB/s 0 failed  0.0s
Users/bdigby/opt/anaconda3/envs/bioconda/conda-b.. 127.0 B @ 296.0kB/s  0.0s
Transaction

  Prefix: /Users/bdigby/opt/anaconda3/envs/bioconda

  Nothing to do

Reloading output folder: /Users/bdigby/opt/anaconda3/envs/bioconda/conda-bld
Users/bdigby/opt/anaconda3/envs/bioconda/conda-b..  ??.?MB @  ??.?MB/s 0 failed  0.0s
Users/bdigby/opt/anaconda3/envs/bioconda/conda-b.. 127.0 B @   1.2MB/s  0.0s
Transaction

  Prefix: /Users/bdigby/opt/anaconda3/envs/bioconda

  Updating specs:

   - pip
   - python=3.9[build=*_cpython]

  Package              Version  Build               Channel                                                                          Size
───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
  Install:
───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

  + bzip2                1.0.8  h0d85af4_4          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     159kB
  + ca-certificates  2022.9.24  h033912b_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     154kB
  + libffi               3.4.2  h0d85af4_5          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64      51kB
  + libsqlite           3.39.4  ha978bb4_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     891kB
  + libzlib             1.2.13  hfd90126_4          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64      66kB
  + ncurses                6.3  h96cf925_1          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     937kB
  + openssl              3.0.7  hfd90126_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64       3MB
  + pip                   22.3  pyhd8ed1ab_0        conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/noarch       2MB
  + python              3.9.13  hf8d34f4_0_cpython  conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64      13MB
  + readline             8.1.2  h3899abd_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     272kB
  + setuptools          65.5.0  pyhd8ed1ab_0        conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/noarch     787kB
  + sqlite              3.39.4  h9ae0607_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     897kB
  + tk                  8.6.12  h5dbffcc_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64       4MB
  + tzdata               2022f  h191b570_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/noarch     121kB
  + wheel               0.37.1  pyhd8ed1ab_0        conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/noarch      32kB
  + xz                   5.2.6  h775f41a_0          conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64     238kB

  Summary:

  Install: 16 packages

  Total download: 26MB

───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

Traceback (most recent call last):
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/bin/bioconda-utils", line 10, in <module>
    sys.exit(main())
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/cli.py", line 971, in main
    bioconductor_skeleton, clean_cran_skeleton, autobump, bot
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/argh/dispatching.py", line 328, in dispatch_commands
    dispatch(parser, *args, **kwargs)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/argh/dispatching.py", line 174, in dispatch
    for line in lines:
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/argh/dispatching.py", line 277, in _execute_command
    for line in result:
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/argh/dispatching.py", line 260, in _call
    result = function(*positional, **keywords)
  File "<boltons.funcutils.FunctionBuilder-5>", line 2, in build
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/cli.py", line 130, in wrapper
    func(*args, **kwargs)
  File "<boltons.funcutils.FunctionBuilder-4>", line 2, in build
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/cli.py", line 59, in wrapper
    func(*args, **kwargs)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/cli.py", line 476, in build
    keep_old_work=keep_old_work)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/build.py", line 320, in build_recipes
    pkg_paths = utils.get_package_paths(recipe, check_channels, force=force)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/utils.py", line 1126, in get_package_paths
    platform, metas = _load_platform_metas(recipe, finalize=True)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/utils.py", line 1035, in _load_platform_metas
    return platform, load_all_meta(recipe, config=config, finalize=finalize)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/utils.py", line 447, in load_all_meta
    for non_finalized_meta in metas
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/bioconda_utils/utils.py", line 453, in <listcomp>
    bypass_env_check=False,
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/api.py", line 52, in render
    bypass_env_check=bypass_env_check):
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/metadata.py", line 2122, in get_output_metadata_set
    bypass_env_check=bypass_env_check)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/metadata.py", line 782, in finalize_outputs_pass
    permit_unsatisfiable_variants=permit_unsatisfiable_variants)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/render.py", line 548, in finalize_metadata
    exclude_pattern)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/render.py", line 409, in add_upstream_pins
    permit_unsatisfiable_variants, exclude_pattern)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/render.py", line 375, in _read_upstream_pin_files
    permit_unsatisfiable_variants=permit_unsatisfiable_variants)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/conda_build/render.py", line 148, in get_env_dependencies
    channel_urls=tuple(m.config.channel_urls))
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/boa/cli/mambabuild.py", line 124, in mamba_get_install_actions
    solution = solver.solve_for_action(_specs, prefix)
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/boa/core/solver.py", line 242, in solve_for_action
    self.index + self.local_index,
  File "/Users/bdigby/opt/anaconda3/envs/bioconda/lib/python3.7/site-packages/boa/core/solver.py", line 79, in to_action
    entry = lookup_dict[get_url_from_channel(c)]
KeyError: 'https://conda.anaconda.org.-a24869bc-cf14-4239-84c7-1b52b5a11e95/conda-forge/osx-64'

[x] mapsplice

Mapsplice is on bioconda: https://anaconda.org/bioconda/mapsplice. How do I get this into a biocontainer?

[x] find_circ

PR: https://github.com/bioconda/bioconda-recipes/pull/37923

[x] circrna_finder

PR: https://github.com/bioconda/bioconda-recipes/pull/37922

[x] targetscan

PR: https://github.com/bioconda/bioconda-recipes/pull/37960
I cannot find a license file for targetscan, and it is denoted as being Copyright (c) The Whitehead Institute of Biomedical Research.
Is it appropriate to add this to bioconda? If not, how can I proceed to ensure I am following nf-core best practices?

[x] DEA

Mulled container for differential exp scripts. PR: https://github.com/BioContainers/multi-package-containers/pull/2382/

Hi @apeltzer, I was away at a conference last week but have got around to creating PRs to add tools to bioconda. Have you permission to review and merge the above PRs?
Two areas of concern are detailed above, any guidance here would be greatly appreciated

I have

Hi @BarryDigby  - just checked (sorry, super busy time right now at work): It looks quite good at the moment to me, e.g. targetscan biocontainer is here: https://quay.io/repository/biocontainers/targetscan

After x-checking, it seems not to have been uploaded - we should maybe ask in their Support Channel. I also opened an issue on bioconda to ask if something was odd there.
See here: https://github.com/bioconda/bioconda-recipes/issues/38099

@apeltzer Really appreciate you taking the time to check Alex. The conversion to DSL2 is going pretty smoothly, it's just the containers that are holding up the show (and linting, probably 🤖 ). Trying my best to have it ready for review next week

@apeltzer Hey Alex, the workflow has been successfully converted to DSL2, the only catch being I am still waiting on those bioconda containers, and for a reason I cannot pin down, singularity fails to pull this image  from quay despite it working perfectly fine for Docker. Irritating to say the least 😁
Given the deadline for the response to authors for BMC is December 5th... what's the gameplan here?

Just asked on the bioconda chat to unhide these & also asked whats going on with Singularity.

Singularity there in << 24hours, the quay.io should already be there. So "just" in time :-D


[MM-27354] Progressive image loading

Summary
This PR first loads very tiny preview of an image in the post. Once the post comes into the viewport, the high resolution image is loaded
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15644
Device Information
This PR was tested on: Poco F1, Android 9

Hello @bhargav50,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Reminder to test this with and without servers that have the mini-preview available. Also test other images like md inline images, open graph and link previews, not sure if those also have a mini_preview (base64 images)
One thing I noted is that this PR is not making use of the inView to render the previous version of the thumbnail and then load the full image, according to the description looks out of scope but it would be nice to have.

@bhargav50 some test seem tl be failing

Building app in separate branch.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich

The files .circleci/config.yml, scripts/build.sh, scripts/build.sh, scripts/clean.sh, scripts/clean.sh, scripts/postinstall.sh, scripts/postinstall.sh, scripts/pre-commit.sh, scripts/pre-commit.sh are in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

There is something messed up while rebasing this branch. I am closing this PR and will reopen again with a new branch.

@bhargav50 no worries, let us know if you need any help when resubmitting your changes


Fatal exception 0x80000003

Debug build based on aa31afc. Launched application, hit F10 couple times then F11 and got exception:

It looks like allocation size is huge.

File df_gfx.c line:6677 looks suspicious. I've made a shot and replaced it with:
pted_eval.offset   = value_eval.offset;
MemoryCopyArray(pted_eval.imm_u128, value_eval.imm_u128);

and issue disappeared but its pretty blind shot though.

This should be fixed by 509089d95d34ef0d562ca04039847b8da4371872 - let me know if you run into any more issues with it.


ImageMask Problem

Hi @kaspermarstal !
I had a problem when I wanted to add ImageMask in SimpleElastix.
For example, I want to use 'RandomSparseMask' as 'ImageSampler', So I set the FixedMask by using SetFixedMask. Meanwhile, I print the GetNumberOfFixedMasks and get 1.
elastixImageFilter.SetParameter("ImageSampler", "RandomSparseMask" )
elastixImageFilter.SetFixedMask(movingMask)
print elastixImageFilter.GetNumberOfFixedMasks()

But, it still said 'fMask unspecified, so no fixed mask used, mMask unspecified, so no moving mask used', when I run the program.
And I get a RandomSamplerSparseMask ERROR: do not call this function when no mask  is supplied.
So what is the problem?

Hi @kaspermarstal !
I have solved the problem.
That is because I modified the source code named "SimpleElastix/Code/Elastix/src/sitkElastixImageFilterImpl.cxx", due to my mask is not a sitkUInt8.
Now I use  movingMask = sitk.Cast(movingMask, sitk.sitkUInt8) to transform the Mask, and I recover the source code. So it works.


[boot_rom] Add bootstrap pre flash programming debug messages

When bootstrap pin is asserted - required peripherals (SPI and Flash)
are initialised, and execution enters a busy loop waiting for SPI data
to program flash.
It is useful to have a debug message describing what is happening to
prevent the system looking unresponsive.
For example:
When I had both prog and uart cables connected to my laptop and FPGA running. After running fusesoc when it flashed Boot ROM, it asserts the bootstrap pin and awaits the SPI input. The bootstrap pin is only deasserted when you power off and back on the board. It just makes it the system look hang up.

It would be nice to eventually change all the uart calls to the LOG functions, I can look into it next week (there are some "TODOs" that mention it in the bootstrap.c).


fix(@angular/cli): update schematics to use RouterModule when --routing flag is present

update schematics to use RouterModule instead of RouterTestingModule when generating a new project with the --routing flag
Fixes #27691
PR Checklist
Please check to confirm your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Running ng new --inline-style false --inline-template false --package-manager bun --routing --skip-git --standalone false --strict true --ssr false --style scss my-test-app generates a new project with the RouterTestingModule which is deprecated
Issue Number: #27691
What is the new behavior?
Running ng new --inline-style false --inline-template false --package-manager bun --routing --skip-git --standalone false --strict true --ssr false --style scss my-test-app generates a new projects with the RouterModule

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
NA

Thank you for your contribution. Could you please update the commit message scope to @schematics/angular?

@alan-agius4 My bad, I've updated it now.

The changes were merged into the following branches: main, 18.0.x


v2.1.1

Hi,
could you please release version 2.1.1 for commit 5e8d009dba37b18651122251d30f82ad0ddbc298
Thanks

@martenb Done

@foxycode don't see it in packagist https://packagist.org/packages/carrooi/nette-menu

@martenb I don't have control over Carrooi packages. Watch discussion here https://github.com/contributte/nette-menu/issues/41 and after it will be resolved, new package under Contributte will be created.

@foxycode ok, thanks update.


RSDK-7531 - Add GetProperties to vision service

The file worth viewing here (to see the changes) is the vision.proto . Adding this as scoped here (https://docs.google.com/document/d/1P0eFy1Y5zBT91oFxz32isku3l5HJUHfdzSKYuS-MFMI/edit?usp=sharing)
Everything else is autogenerated.

I see that in some places, the protoc version gets downgraded to a previous version. Can you make sure your local setup is up to date ('make setup' on your api repo) and that you are basing your commit on top of the latest commit of the api repo, too?

Added docstrings and also did the make setup (I'm on the freshest api commit). That protoc version is still downgrading though, so I feel like it's supposed to.

Okay great catch on the versioning thing. It's fixed now, I had to nuke my old /opt folder before the make setup.


feat: Support dynamic responseFormat

OpenAI recently introduced Structured outputs in their API. This is done by passing the desired JSON schema in the "response_format" parameter as described here: https://openai.com/index/introducing-structured-outputs-in-the-api/
Since the JSON schema is an object i.e. Map<String, dynamic> and not a String, we should remove the type requirement for "response_format" and use dynamic instead, similar to "logitBias"

@anasfik please merge this

@anasfik can you go ahead and merge this?

@anasfik

@anasfik please merge, very important.

This library has not been maintained for almost a year. Use openai_dart instead which is aligned with the latest OpenAI API.


extended rcf with thresholding

Description
This is the initial pr for adding a new anomaly detection model that extends the current rcf with thresholding, named Extended Random Cut Forest (ERCF). For the sake of time and unblocking further development, the initial unit tests only include those for the primary features/api to be used in this plugin. Implementation changes, testing, and documentation will ensue in following pr for this plugin or the rcf package.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #163 (6739ac3) into main (6843b29) will increase coverage by 0.09%.
The diff coverage is 69.90%.

@@             Coverage Diff              @@
##               main     #163      +/-   ##
============================================
+ Coverage     70.30%   70.39%   +0.09%
- Complexity     2970     3066      +96
============================================
  Files           268      276       +8
  Lines         14051    14483     +432
  Branches       1409     1472      +63
============================================
+ Hits           9878    10195     +317
- Misses         3554     3641      +87
- Partials        619      647      +28

Flag
Coverage Δ

plugin
70.39% <69.90%> (+0.09%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...mazon/randomcutforest/threshold/state/Version.java
0.00% <0.00%> (ø)

...on/randomcutforest/threshold/BasicThresholder.java
56.60% <56.60%> (ø)

...andomcutforest/threshold/CorrectorThresholder.java
61.03% <61.03%> (ø)

.../randomcutforest/ERCF/ExtendedRandomCutForest.java
68.29% <68.29%> (ø)

...om/amazon/randomcutforest/threshold/Deviation.java
80.00% <80.00%> (ø)

...va/com/amazon/randomcutforest/ERCF/ERCFMapper.java
100.00% <100.00%> (ø)

...st/threshold/state/CorrectorThresholderMapper.java
100.00% <100.00%> (ø)

...ndomcutforest/threshold/state/DeviationMapper.java
100.00% <100.00%> (ø)

...port/SearchAnomalyDetectorInfoTransportAction.java
55.55% <0.00%> (-8.89%)
:arrow_down:

... and 12 more

As discussed offline, we will use the code from RCF directly.  The dependency doesn't have to be from maven.  It can be a binary file dependency.


Wenk should close on mouse over

Hover over an element with a wenk tooltip
Move the mouse from the element onto the wenk tooltip
The wenk tooltip stays visible

Expected: Tooltip closes/gets hidden.

Ah, this worked with:
 [data-wenk]:after {
	pointer-events: none;
 }


'manta get configuration' does not work on 1.22.3

(ansible2.9) 💻 [mcaubet@castaneda:~]# manta get configuration
2024-02-13 12:52:20 | INFO  | /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/mesa-0.27.3/src/common/authentication.rs:47 — Reading CSM authentication token from configuration file
2024-02-13 12:52:20 | INFO  | /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/mesa-0.27.3/src/common/authentication.rs:140 — Validate Shasta token against https://api.cmn.alps.cscs.ch/apis/cfs/healthz
2024-02-13 12:52:20 | INFO  | /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/mesa-0.27.3/src/common/authentication.rs:151 — Shasta token is valid
thread 'main' panicked at src/common/cfs_configuration_utils.rs:76:43:
called `Option::unwrap()` on a `None` value
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

(ansible2.9) 💻 [mcaubet@castaneda:~]# manta --version
manta 1.22.3

Hi Marc,
this is a bug because manta is trying to print a commit id which does not exists, this is weird because normally CFS will assign a commit id to every CFS layer automatically, otherwise, the cluster creation process would fail. This is something done by CFS, manta just submits the job to create the CFS session.
I have fixed this so manta will show a "Not defined" text whenever the commit is is missing.
thank you

solved in 1.22.4


Fix instructions for PIT

Don't forget the coveralls-gradle-plugin installation.

Coverage remained the same at 95.077% when pulling c707b5543cf0e2c2d9aa6469d60d8052c84cb569 on thejohnfreeman:patch-1 into bce17918d0574af06e9eff0747a416ced8c50e55 on kt3k:master.

Good catch. Thanks!


Show GitHub issue search error only in verbose mode

Fixes confusion, e.g. https://github.com/fastlane/fastlane/issues/7274

💯


Notion Repackaged Still Lunching as Normal Notion Even after trying the fix

What is happening?
Notion Repackaged is Still Lunching as Normal Notion even after deleting .notionenhancer
What should be happening?
Screenshots (if applicable):
Here's the console, as advised

Operating system/browser name & version:
Windows 11
notion-enhancer and/or notion-repackaged version:
notion enhancer v0.11.0, notion repackaged 2.0.18-1
notion-enhancer configuration (enabled/disabled mods and their settings):
none

please scroll up and show the top of the console and also provide a screenshot of the notion app

please scroll up and show the top of the console and also provide a screenshot of the notion app

ohhh nvm i thought the settings didnt move, it did work I'm so sorry ;-;

ohhh nvm i thought the settings didnt move, it did work I'm so sorry ;-;


Sprockets::Rails::Helper::DependencyError in RailsAdmin::Main#dashboard

Created a Rails 4.1 app and ended up with this issue. Error seems to be a legitimate gem issue.
Asset depends on 'rails_admin/bootstrap/glyphicons-halflings.png' to generate properly but has not declared the dependency
Please add: `//= depend_on_asset "rails_admin/bootstrap/glyphicons-halflings.png"` to '/Users/abhishek/myapp/vendor/bundle/gems/rails_admin-0.6.2/app/assets/stylesheets/rails_admin/rails_admin.scss.erb'
  (in /Users/abhishek/myapp/vendor/bundle/gems/rails_admin-0.6.2/app/assets/stylesheets/rails_admin/rails_admin.scss.erb)

Fixed by #1976.


Support getting a page preamble

Looks like the markdown generated starts out with a ## [NAME] for everything thing documented, but there doesn't seem to be a way to get any preamble on to the page, especially a # [TITLE] since that likely will be used by some system.
The easiest might be to just support a title on the rule to insert this vs. and then let something like #159 end up providing any content after the title before the listing of symbols.

Would using the file-level docstring as a preamble be sufficient?

The file-level docstring would provide an overview, but likely wouldn't map the the # title directive well.


Improve README

More informatic: Easier to get started with widgets
More up to date with current status
More engaging visual example

Notes:

Image path needs to be fixed once merged
See the rendered outcome: https://github.com/juhamust/mozaik/tree/readme
My motivation for the change is get more contributors for the project/widgets

Thabks a lot for your contribution, I hope it will help people contributing!


[Runtime] Implement Full watchEffect Functionality in Vue Vapor

The current implementation of effect within the vue-vapor project does not fully conform to the standard watchEffect behavior, particularly lacking in comprehensive flush timing options.
We aim to align vue-vapor's watchEffect with the standard version by introducing detailed flush options (pre, post, sync). A fully realized flush mechanism will facilitate the implementation of the beforeUpdate directive lifecycle hook.
In addition, we plan to introduce a new onEffectCleanup API analogous to onScopeDispose, which will aid in implementing dynamic arguments for v-on. first, extend watchEffect in the runtime-vapor package. If necessary later, extract it to the reactivity package.

强


Load FB automatically for selected sites

FB3 needs a "Open for all sites" Option
or
Have it remember sites that FB was activated for and open FB when they are accessed.

Agreed, we need this.
Honza

As of #310:
Previous versions of Firebug had their activation based on a per URL basis following the same origin policy instead of opening per tab.
It should be considered to change this functionality back to the old behavior.
Sebastian


AudioSystem on Web

This is a companion issue to #174.

Why the bug label?

Fixed @mewmew

Fixed @mewmew

Thanks. Could you remove the bug label from #174 as well?

@mewmew I removed and re-added because I'm not sure if it isn't a bug. Windows is a "supported platform" at the moment and audio not working on it is a bug in my opinion.

@mewmew I removed and re-added because I'm not sure if it isn't a bug. Windows is a "supported platform" at the moment and audio not working on it is a bug in my opinion.

Sounds reasonable.

Before implementing this, take a look at https://github.com/hajimehoshi/ebiten/blob/master/audio/internal/driver/driver_js.go to see how it solves audio playing for the JS backend.

I already have a more higher level way of doing it @mewmew Thanks for the pointer though, might be useful.

Closed in favor of: https://github.com/EngoEngine/engo/issues/272


chore(ci): use pr backports from upstream projen

By using the upstream version, we fix an issue with the Backport workflow causing all PRs to look like they've failed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

💚 All backports created successfully

Status
Branch
Result

✅
maintenance/v5.3

Questions ?
Please refer to the Backport tool documentation and see the Github Action logs for details


Question: Pairing shadcn/ui with a Carousel component...

This is more of a question...
I'm seeking to pair shadcn/ui with a Carousel (or slider depending on your nomenclature) component. Does anyone have recommendations for this? Requirements would be:

fully typed
react hooks
composable
highly accessible
unstyled (or capable of that)

I used embla carousel recently on a project with shadcn/ui.
Meets quite a lot of your requirements and they have a react package too. The only thing I don't think is covered is accessibility. This brief discussion covers it but since then it looks like they're providing a new project that helps generate a carousel that includes accessibility.
If it's useful, here's how I implemented embla carousel in my project alongside shadcn/ui. It isn't using TypeScript but hopefully gives a rough idea of what can be done.

Thank you @cerimorse! That's very helpful. I might give it a try and see if it works. For context, I'll be using it in a component using an aging and bloated swiper.js based implementation.


ACTION REQUIRED: Changes to pulling Chainguard Images

Hey there Chainguard here.
We noticed that you are using Chainguard Images, thank you! We wanted to make you aware of an upcoming change that will impact your project.
Starting August 16, 2023 public users will no longer be able to pull images from our registry (cgr.dev/chainguard) by tags other than latest or latest-dev. Please see the announcement for more information.
You are currently using the following.
In https://github.com/nais/nais-verification/blob/391fe71515df7fca8b57f040ca04265ae36140e3/.github/workflows/main.yml:

cgr.dev/chainguard/python:3.10

Our goal is to prevent your project from experiencing any disruptions. Please see the migration guide for options.
If there's more we can do to help please reply to this issue or email us at<EMAIL_ADDRESS>Thank you!

Python imgs har ikke tag med nøyaktig versjon, det skaper problemer med venvs og denslags. Avventer disse til vi har snakkert med Chainguard, evt så får vi bare gå tilbake til de offisielle Python imagene


Documentation relies on shortcut that is not always available

For me, right now, the Azure AD Domain Services' Overview tab does not list Required configuration steps. There's no apparent way to turn the panel back on, nor other means to find the DNS service addresses. This document relies on the panel being present, and cannot help me proceed.

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

ID: db354a18-aec3-a624-eb0b-de969d753909
Version Independent ID: 1bbcf33d-9918-574e-e246-0be512cf3bfc
Content: Azure Active Directory Domain Services: Update DNS settings for the Azure virtual network
Content Source: articles/active-directory-domain-services/active-directory-ds-getting-started-dns.md
Service: active-directory-ds
GitHub Login: @mahesh-unnikrishnan
Microsoft Alias: maheshu

This experience has changed in recent builds. We are working on a doc update to cover the changed experience.

The tile within 'Required Configuration steps' for configuring DNS automatically disappears if the DNS server settings are configured correctly for the virtual network.
You can see the DNS server IP addresses from the 'Properties' tab within the UI.
#please-close

@garthk We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Discussion: are there data sets in the pre-federal NepalMap that can be migrated to the federal NepalMap

This issue is meant to serve as a public discussion about if and how we can migrate data from the pre-federal NepalMap to the federal NepalMap.
Much of the data we have is data for the pre-federal government boundaries. Are there data sets that can be migrated? What would be necessary to do in order to migrate data accurately? If you have ideas or thoughts, please contribute them in a comment in this issue.

@ravinepal, @amitness, @nikeshbalami, @crackjack,
(I do not have the github usernames for Asmita and Roshan, so I cannot mention them here.)
This is intended to help with our discussion about the possibility of migrating data sets from pre-federal to the federal instance of NepalMap.
For any given level (national, provincial, district, local), there must be data for every unit at that level. In other words, if we are presenting district-level data, we must have data for all 77 districts. If we are presenting local data, we must have data for every locality. The Wazimap framework cannot handle presenting data for only some of the units at a given level.
If we are trying to migrate data from the pre-federal data sets to the federal instance, we have to have data for all of the federal units at the lowest level we are trying to migrate for a given data set. In addition, either the boundaries for each geographic area need to be the same or we need new data. I do not see a way for us to take district-level data for the old 75 districts and move it to the new federal structure that has 77 districts. We cannot present data for just the 73 districts that have not changed because the Wazimap framework does not work with data for only some areas at a given level.
Does anyone know if any of the organizations who published the data we use in the pre-federal instance have published new data that is divided along the new district boundaries?

Closing. Did not generate any discussion.


JS Debugging

When trying to debug the js code, upon expanding the tool tip for "prototype" of an object, the application is terminating as in the attached image. There are forty to fifty properties defined on the object's prototype which in turn trigger the getter callback and gather the info from c# code. However, each of the properties is accessible when accessed individually. Is this expected? If so, it would have been convinient if prototype did not show up as the application terminates if prototype is expanded accidentally.

Thanks for the feedback :). The application is not supposed to hang while expanding the tooltip. Is this an recurring issue?

Surprisingly, I am unable to reproduce the issue now. I will see if I can identify what code change has fixed it!!!!!!!!

Thanks :+1:.


10.3.9 - Release notes

10.3.9 news

Aggregated fields in list group headers (Aggregated grid) - Sum etc on archives with Column options (FT: AggregatedGrid)

Is there any created doc's on this?

Status from Team C
I have nothing to report from Team C – we're just working behind a feature toggle with the new UI at the moment.
From the AI team, there is an update for the pilot customers: a new, improved Sparkle button in the Froala editor – hidden behind Feature Toggle HugoAiGpt: US 60452, 60453, and 61953. Nothing to report that is generally available.S 60452, 60453 og 61953.
Nothing to report that is General Available.
Sales / Admin
Sales: Field/column calculations in GRID will be included for all in all archives (no pilot program or feature toggle).
Marketing: Flow features:

Flow setting: When enrolled, remove the contact from other flows
Copy flow
Step type:  Internal action - Notify by SMS
Change mailing name from within step configuration

Service:
New Service
•	Support for Before/After Save Custom object Triggers (62873)
•	Support for using Screen choosers from CS in combination with Custom objects (62865)
Service
•	Better support for formatting CRMScript code in Monaco (62423)
Mobile CRM
•	No new version, a new is coming with bugfix in September  (11.0.2)
Community

A new version is coming very soon
https://community.superoffice.com/en/product-releases/release-notes/release-details/?release=Community_2024-R06
Not mentioned yet, but there will be a way to clean up subscriptions for users who no longer have access to the Community or a Forum they previously subscribed to.

API's
SuperOffice.WebApi nuget package has adopted the APACHE 2.0 license, which is more permissive. This allows partners to bundle the package binaries without restrictions.
Developer portal
Appstore
Nothing new
Office integration
Nothing new
Help/
A new guide for SuperOffice for Outlook
Maybe the calculation possibilities on section tabs and lists needs to be added?
Userflow

Product NPS testing - Filip and Kirsti have created an NPS for Product in Userflow, will be "sent" to a random selection of 11.000 users.
Internal info from Marianne about price adjustments

API:

SuperOffice.WebApi nuget package has adopted the APACHE 2.0 license, which is more permissive. This allows partners to bundle the package binaries without restrictions.

Webhook update: new design moves webhook calls to a separate thread for each receiver, so that slow crmscript webhooks go to a separate queue.

New online status change events for tenant api subdomain changes. New event type: Move. See Status change webhooks for more details.


Run tests against a read-only "dashboard" database connection

Related to #16. I want to encourage using a separate "dashboard" database alias which is configured something like this:
DATABASES = {
    "default": {
        "ENGINE": "django.db.backends.postgresql_psycopg2",
        "NAME": "mydb",
    },
    "dashboard": {
        "ENGINE": "django.db.backends.postgresql_psycopg2",
        "NAME": "mydb",
        "OPTIONS": {"options": "-c default_transaction_read_only=on -c statement_timeout=100"},
    },
}

I want to write the tests against this - but I'm running into some trouble because the test framework isn't designed to handle read-only database connections like this. I'm seeing errors like this:

Got an error creating the test database: cannot execute CREATE DATABASE in a read-only transaction

I tried doing this in my pytest_use_postgresql.py plugin module:
import os

import pytest
from dj_database_url import parse
from django.conf import settings
from testing.postgresql import Postgresql

_POSTGRESQL = Postgresql()

@pytest.hookimpl(tryfirst=True)
def pytest_load_initial_conftests(early_config, parser, args):
    os.environ["DJANGO_SETTINGS_MODULE"] = early_config.getini("DJANGO_SETTINGS_MODULE")
    settings.DATABASES["default"] = parse(_POSTGRESQL.url())
    settings.DATABASES["dashboard"] = parse(_POSTGRESQL.url())
    settings.DATABASES["dashboard"]["OPTIONS"] = {
        "options": "-c default_transaction_read_only=on -c statement_timeout=100"
    }

def pytest_unconfigure(config):
    _POSTGRESQL.stop()

That's what triggered the above errors.

https://pytest-django.readthedocs.io/en/latest/database.html#django-db-setup describes the pytest-django django_db_setup() fixture and says:

The default implementation creates the test database by applying migrations and removes databases after the test run.
You can override this fixture in your own conftest.py to customize how test databases are constructed.

So maybe I need to over-ride that fixture for my project?
Or I could try django_db_modify_db_settings() which says:

This fixture allows modifying django.conf.settings.DATABASES just before the databases are configured.

The implementation of the pytest-django fixtures is in https://github.com/pytest-dev/pytest-django/blob/edd33115e9ed205d241a6833839dcea5186d7d03/pytest_django/fixtures.py#L84

I moved that logic to conftest.py and it mostly seems to work:
import pytest

@pytest.fixture(scope="session")
def django_db_modify_db_settings():
    from django.conf import settings

    settings.DATABASES["dashboard"]["OPTIONS"] = {
        "options": "-c default_transaction_read_only=on -c statement_timeout=100"
    }

Just one catch: at the end of the tests I get these warnings:
test_project/test_dashboard.py::test_anonymous_users_denied
  /Users/simon/.local/share/virtualenvs/django-sql-dashboard-ZqMBSvf9/lib/python3.9/site-packages/django/db/backends/postgresql/base.py:304: RuntimeWarning: Normally Django will use a connection to the 'postgres' database to avoid running initialization queries against the production database when it's not needed (for example, when running tests). Django was unable to create a connection to the 'postgres' database and will use the first PostgreSQL database instead.
    warnings.warn(

test_project/test_dashboard.py::test_anonymous_users_denied
  /Users/simon/Dropbox/Development/django-sql-dashboard:0: PytestWarning: Error when trying to teardown test databases: RuntimeError("generator didn't stop after throw()")

Oh this is nasty: tests on my laptop are now flaky. Running pytest some of the time gets me a clean test run, and other times it dies with nasty Got an error creating the test database: cannot execute CREATE DATABASE in a read-only transaction errors inside the django/test/utils.py:170: in setup_databases function.

CI is failing too: https://github.com/simonw/django-sql-dashboard/runs/2107262024 - lots of messages like this one:
==================================== ERRORS ====================================
__________________ ERROR at setup of test_superusers_allowed ___________________

self = <django.db.backends.utils.CursorWrapper object at 0x7f93bb35f7b8>
sql = 'CREATE DATABASE "test_test" ', params = None
ignored_wrapper_args = (False, {'connection': <django.db.backends.postgresql.base.DatabaseWrapper object at 0x7f93bb35f518>, 'cursor': <django.db.backends.utils.CursorWrapper object at 0x7f93bb35f7b8>})

    def _execute(self, sql, params, *ignored_wrapper_args):
        self.db.validate_no_broken_transaction()
        with self.db.wrap_database_errors:
            if params is None:
                # params default might be backend specific.
>               return self.cursor.execute(sql)
E               psycopg2.errors.ReadOnlySqlTransaction: cannot execute CREATE DATABASE in a read-only transaction

https://github.com/django/django/blob/3.1.7/django/test/utils.py#L155-L196 is that setup_databases() function.

Alternative idea: have the test framework create the default database as usual, then use the settings fixture to add the extra dashboard database later on:
pytest.fixture(autouse=True)
def use_dummy_cache_backend(settings):
    settings.CACHES = {
        "default": {
            "BACKEND": "django.core.cache.backends.dummy.DummyCache",
        }
    }

I tried that recipe but I got this error:

ScopeMismatch: You tried to access the 'function' scoped fixture 'django_db_modify_db_settings' with a 'session' scoped request object, involved factories

So I'm going to use an explicit fixture instead.


Add gas optimizations

Wrote up a few optimizations on the main contracts, open to feedback especially for the unchecked blocks if you think that over/underflow checks are needed in those functions.
1b32797: _exists(uint256 id) returns id < currentIndex, and we call it with currentIndex, which always results in it returning false.
ae1cdcf: Explanations in the code comments.
5fca737: Initializing a variable with a 0 is more expensive than not doing so, even if both results are the same. Source:

uint256 value; is cheaper than uint256 value = 0;

c23e2e0: All values are integers which cannot be negative, saves a few opcodes.
ee2abe2: Previous changes applied to ERC721AOwnersExplicit.
81daf25: Loading nextOwnerToExplicitlySet into memory saves an SLOAD opcode later on line 23.

Thanks for that! I missed that last one. Also added an unchecked block to ownershipOf and added a condition to prevent underflow.

Love this


adds phased training to CLI

ilab model train --e2e added to support phased training that accomplishes the phased training of the LAB paper.
Phase1 training followed by MMLU checkpoint evaluation, followed by Phase2 training, with the final checkpoint selected via MT-Bench.
Closes #1729, with further to-do list included on that issue that needs to be broken down into subsequent issues.

NOTE PR open early for functional review, need to add tests and lint.

Can we not use the term e2e for the flag? This term is already being used in a lot of different contexts in this project - I think adding yet another will make things more confusing for folks.
Something like --multiphase or --phased I think would be more appropriate.

I think it may also make more sense to have an additional flag in basic-workflow-tests.sh to run this rather than introduce a new script - otherwise I'm not sure how we incorporate it with our current runners.

Can we not use the term e2e for the flag? This term is already being used in a lot of different contexts in this project - I think adding yet another will make things more confusing for folks.
Something like --multiphase or --phased I think would be more appropriate.

Sure that's fine, good thought. I'll make that change

I think it may also make more sense to have an additional flag in basic-workflow-tests.sh to run this rather than introduce a new script - otherwise I'm not sure how we incorporate it with our current runners.

Dropped that script, didn't make sense in PR

@RobotSail tysm for the thorough review! I'll address everything asap

--e2e replaced by --phased

@Maxusmusti
A couple of things need to be user-configurable:

save_samples should be based on a proportion of the total number of samples that weren't dropped because of their length.
effective_batch_size should be 128 by default for phase1

Nice work! A few comments:

Is it normal that the config.yaml does not have the newly added flags? Or did I miss that somewhere?
What's the testing strategy for this?
When we use paths we should use  pathlib.Path instead of str.
A note in the CHANGELOG would be great, this seems like a massive addition to the project

Thanks!

Thanks so much @leseb!

You're right, I need to add this. Configuring this is kinda tricky because we're trying to not have infinite configuration fields in config.yaml, but each phase of training here could be configured differently, so I'm kinda workshopping that right now.
Currently thinking this through. Any recommendations would be very appreciated!
Switching this over is kinda annoying because we're using Path non-uniformly across the different libraries. Usually str paths are accepted, so I'm following that internal convention, but that might be an exciting refactor opportunity for a new contributor. What do you think?
Added the Changelog note!

Writing tests blocked by #1723, waiting for that to be in to solidify full-training testing pattern

I didn't have a chance to review anything below _run_phased_training in the call stack, so this is just a start. I also haven't yet looked at the CHANGELOG.md changes (but wonder why the formatting change was needed.)
I may follow up with more review comments later for the code that I haven't yet looked at. I hope an earlier partial review is at least slightly more helpful than none at all.

I don't remember changing the changelog formatting explicitly. It might have ended up in the PR from a previous convention. I'll try to overwrite that change.

@booxter Thanks a lot for all your feedback, I'll make those changes in a few.

@JamesKunstle one thing I notice is that you git merge updates from main. Which is ok for development, but I think we forbid PRs to include merge commits, so this will have to be cleaned up before merging. Just a heads-up. (You may want to use git pull --rebase instead to keep a straight line commit history.)

@JamesKunstle one thing I notice is that you git merge updates from main. Which is ok for development, but I think we forbid PRs to include merge commits, so this will have to be cleaned up before merging. Just a heads-up. (You may want to use git pull --rebase instead to keep a straight line commit history.)

Good note- that was from a github sync-fork. I'll drop that.

Removed myself from the review to help move it forward for merging. I opened https://github.com/instructlab/instructlab/issues/1960 to track future work on the concern I had with this PR.


Change spice button active colour

Is your feature request related to a problem? Please describe.
The spice button active bugs me when I hover over it.  Reason, because it remains green in color at all times.
Describe the solution you'd like
I would like it to highlight either with the album color or maybe black in color when I hover or press it.
Describe alternatives you've considered
Tried to change the user.css file but did not make any difference when change is made.
Additional context
Change can only be made when I click on the buttons under devTools, other than that I can't change the source code (User.css)

Thank you very much for the fix for issue #178


3.1 jdbc driver broke float handling for nan/inf

When upgrading from the jdbc driver from 2.x to 3.x I noticed this following error occurring in tests (for the trinodb project)
Caused by: ru.yandex.clickhouse.except.ClickHouseUnknownException: ClickHouse exception, message: Error parsing 'inf' as java.lang.Float; For input string: "inf"
	at ru.yandex.clickhouse.response.parser.ClickHouseValueParser$ClickHouseValueParserFunctionWrapper.parse(ClickHouseValueParser.java:243)
	at ru.yandex.clickhouse.response.ClickHouseResultSet.getObject(ClickHouseResultSet.java:749)

The root cause of the problem is ResultSet.getFloat and ResultSet.getObject(Float.class) use different parsing functions (a custom double parser for getFloat and Float::valueOf for getObject): https://github.com/ClickHouse/clickhouse-jdbc/blob/v0.3.1-patch/clickhouse-jdbc/src/main/java/ru/yandex/clickhouse/response/parser/ClickHouseValueParser.java#L43-L45
Clickhouse sends [+|-]?inf over the wire (not [+|-]?Infinity) so if you use ResultSet.getObject(Float.class) the Float::valueOf function errors out.
The fix is just creating a custom float parser (much like the double parser) such that both float and double parsing perform the same logic.

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

Thank you @grantatspothero for the contribution. We're working on 0.3.2 which is a complete rewrite and we'll move to a RowBinary-based new JDBC driver, which supports nan/inf(examples).
Anyway, I changed target branch from master to develop, could you resolve the conflict there so that I can merge?

@zhicwu Just a bump

@zhicwu Just a bump

Sorry for the late response. Will merge this pull request after I'm done with another.


GatewayRetryRequestStatsTest transient test failures

The GatewayRetryRequestStatsTest is looking at the ClientSideRequestStatisticsTraceDatum to figure out reties. The issue is that the method GetClientSideRequestStatsFromTrace is always returning the first ClientSideRequestStatisticsTraceDatum it finds, but the diagnostics contains multiple instances. It sometimes grabs the wrong instance which causes the test to fail even though it should have passed.
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/b2614ce7329ce81b36566a062384c580acf34b50/Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.EmulatorTests/GatewayClientSideRequestStatsTests.cs#L54

Fixed in https://github.com/Azure/azure-cosmos-dotnet-v3/pull/2716/


release/0.0.1-dev.4

Target Goal

[ ] unit test setup
[ ] handle cache time
[ ] invalidate key

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bdf503c). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   42.32%
=========================================
  Files             ?        7
  Lines             ?      189
  Branches          ?        0
=========================================
  Hits              ?       80
  Misses            ?      109
  Partials          ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Algorithm for stochastic convergence

I want to study the limit of Gini coefficient over time assuming random transactions. The goal is to approximate the inverse function to support systems designed for a particular Gini index, given some trading volume (amounts and number of transactions) vs number of players. Random transactions can later be used to compare the decision bias of more complex system. The lower limit makes it rather hard to derive the limit mathematically, so I think about running simulations and measure it.
The problem is to get a source of approximate perfect randomness, and an algorithm that converges efficiently but with high numerical accuracy. These two things are interdependent.
Ideally I want a guarantee that doesn't require me to store the sum or use a floating average, since that might accumulate rounding errors. If the random source is biased, I want a method to detect it.
One approach is to treat is the measurement as a spring, which gradually gets stiffer and less sensitive to measurements over time. When it stops changing the last decimal one can say it has "converged to a limit". The rate where it gets stiffer should be low enough to pick the next digit. If the random source is biased, two runs on the same input (except the random source) should yield different limits.
The problem is: I don't know what change in stiffness is sufficient for picking the next digit. If I knew that I could detect bias in the random source at an earlier moment of time.
It might be that a such algorithm is susceptible to rounding errors, but I am not sure.
Another approach is to repeat the spring algorithm in two layers, since the limit of many results also should converge to a limit. Alternatively use average in the second step.

Neither thread_rng or OsRng could reproduce the limit accurately. I have been thinking some about why, and believe that the stiffness should slow down exponentially. This is equivalent to increasing samples exponentially to get higher accuracy.
For now, I think it is best to settle for a goal of 1/1000 accuracy.


Revert "Avoid '0 new commits' messages to be send to webhook"

Reverts go-gitea/gitea#11082
As per comments by @CirnoT this breaks drone and likely other things as it directly moves is away from GitHub behaviour.
The 0 commits information should be handled elsewhere.
Reopens #10498

Relevant https://github.com/drone/drone/pull/2966#issuecomment-627683953

@CirnoT - I see you've had 10 PRs merged - would you like to apply to be a maintainer?

I can confirm this fixes it for drone on my instance - thanks!


Fix single-key segmentation bug

Description
When segmenting on a single column, segmentation was made, and tagged, based on the first character of said column.
Adding a check on the length of segments, in order to get the proper df subgroup and tags
General Checklist

[ ] Tests added for this feature/bug
if it was a bug, test must cover it.
[ ] Conform by the style guides, by using formatter
[ ] Documentation updated
[ ] (optional) Please add a label to your PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 82.641%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
4944

Relevant Lines:
5819

💛  - Coveralls


zbus: Fix parameter order of net buf pool fixed define

Fixed order of pool-size and data-size parameters  in use of NET_BUF_POOL_FIXED_DEFINE()
NET_BUF_POOL_FIXED_DEFINE (declared here) specifies count and then data size. The current usage of this macro specifies data size and then count.

(sorry I misclicked and of course didn't mean to mark as ready for review!)


Need a check on c$http$user_agent

https://github.com/corelight/cve-2021-44228/blob/develop/scripts/CVE_2021_44228_java_GET.zeek#L69
This line needs to check for the existence of user_agent as it's populating reporter.log on a cluster I'm running this on.

Closed by #48


Fix ButesOp::Mov and Swp Display implementations

Codecov Report

Merging #76 (d6021a0) into master (4514db9) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@          Coverage Diff           @@
##           master     #76   +/-   ##
======================================
  Coverage    41.1%   41.1%
======================================
  Files          19      19
  Lines        4542    4542
======================================
  Hits         1865    1865
  Misses       2677    2677

Impacted Files
Coverage Δ

src/isa/instr.rs
10.5% <ø> (ø)

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


dockerprune: add a setting to keep the most recent N builds of a tag. Fixes https://github.com/windmilleng/tilt/issues/3211

Hello @maiamcc,
Please review the following commits I made in branch nicks/ch6284/prune:
ff8774c73af994c3f591f346ab75e203a94b4db1 (2020-04-16 19:19:35 -0400)
dockerprune: add a setting to keep the most recent N builds of a tag. Fixes https://github.com/windmilleng/tilt/issues/3211
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

BTW did you update the docs for this?

nope, it's not out yet!
On Mon, Apr 20, 2020 at 12:12 PM Maia McCormick<EMAIL_ADDRESS>wrote:

BTW did you update the docs for this?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/windmilleng/tilt/pull/3218#issuecomment-616655151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEA4JX3QGMSTZ6UQIKHPTRNRYAFANCNFSM4MKJIZVQ
.


[Feature] TransactionInterface for Pipeline

I noticed that Pipeline<T> does not implement TransactionsInterface which means I cannot use MULTI on it. Is there any specific reason why it's not implemented? AFAIK Commands in a pipeline are not executed together as a transaction.
And also commands in a transaction are not executed in a pipeline(see below).

But I want to have commands executed together in a transaction AND in a pipeline together so I can cut on round trips and have them executed in order.
Describe the solution you'd like
I would like to see TransactionsInterface implemented for Pipeline<T>.

Yeah, I intentionally disallowed that in order to properly handle cluster redirections in the middle of transactions. The docs state that clients should DISCARD a transaction in response to a MOVED/ASK error, then replay the entire transaction on the new node, and the only way I could think to do that safely in a transaction was to wait for a QUEUED or MOVED/ASK response from each command.

After looking into this a bit more it seems Redis has supported this since 2.6.5. In 9.2.0 there will be a new pipeline function on the Transaction struct that can do this.

Added in https://github.com/aembke/fred.rs/releases/tag/v9.2.0, but let me know if you have any issues with it.


Add PointPen support to TT Glyph objects

See https://github.com/unified-font-object/ufo-spec/issues/74
Using Glyph.drawPoints() will make sure the point indices are not changed from the original. This is vital when the font has TrueType instructions which are based on point indices.

I was thinking about tests ... but that would require a PointPen, which as far as I can see is not part of fontTools. Would it be OK to copy it from ufoLib just into the test module, or is it even possible to add dependencies just for tests?

i was actually on working today on moving ufoLib inside fontTools.ufoLib. I would also move the pointPen module alongside it.
Anyway, for the test you don't need to subclass the AbstractPointPen, just make a test pen object that has the same beginPath, addPoint, endPath interface

it is possible to add dependencies just for the test, yes, but this is not the case I think


[#8481] Remove deprecated alluxio filesystem commands

Alias commands removed: lsr, rmr, umount
FileInfoCommand removed
LoadMetadataCommand removed

Fixes #8481

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2204/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2205/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2209/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2210/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2212/
Test PASSed.

@ZacBlanco is this draft ready to review ?

No, I still need to add functionality for non-deprecated aliases. I need another 1-2 days

@apc999 Just added in last touches on this. Feel free to review the rest

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2264/
Test PASSed.

I agree this first solution felt more like a stopgap measure as I was writing it. Another idea I had been thinking of was to instead create some kind of inner class CommandAlias


Face ID permission no return .authorized

Should line 55 return .authorized? 🤔  Because I encounter the status stay  .notDetermined and the controller would not dismiss automatically (which has set controller.dismissCondition = .allPermissionsDeterminated)
https://github.com/ivanvorobei/SPPermissions/blob/6618b3a84b8063c82652bee67ea47044b7bd46b0/Sources/SPPermissionsFaceID/SPFaceIDPermission.swift#L55

Here some problem. Line 55 not sure about status, its may be . notDetermined  or allowed. Maybe you have idea how detect it correctly?

Hi, @ivanvorobei I think .allowed is better than .notDetermined, because device is allowed to use biometric authentication  already, rather than notDetermined. To do that we can solve the controller not dismiss issue. 🤔  Thanks 🙏

@dan12411 when user first see Face ID, he see allowed state. Maybe it confused him.

@ivanvorobei The condition is isReady == true, means the user is passed the bio auth, so .allowed should not be confused, am I right?
  let isReady = context.canEvaluatePolicy(.deviceOwnerAuthenticationWithBiometrics, error: &error)

  case nil where isReady:
    return .notDetermined

doc: https://developer.apple.com/documentation/localauthentication/lacontext/1514149-canevaluatepolicy

isReady  property indicate if user has FaceID. It not same as allowed or not allowed for permission.

If you change it as you suggest, then even at first startup the user will see that permission has already been granted, although this is not the case.
There is really no way to check the current status of the FaceID permission. It works like this - if this is the first authorization attempt, permission will be shown. If it is the second and subsequent attempts, permission will not be requested. We don't know which attempt it is and there is no way to get that information.

I can add maybe flag or something else for have specific process for FaceID permission.

Got it, If the changed you suggest can dismiss the controller automatically will be great,
Thanks for your kindly reply : )

My idea this - for FaceID always show .notDetermined UI, but close controller with request even user not allow permission.


How to use this?

I'm pretty lost on how to actually use this. Any help?

GO TO: https://github.com/gibbed/SteamAchievementManager/releases/tag/7.0.11
And download SteamAchievementManager-7.0.11.zip it has 48kb
and run it

[1] Download zip.
[2] Run SAM.Game.exe.
[3] Choose game.
[4] Select achievements to unlock.
[5] Top right click 'commit changes'.
[6] Sit back and have a coffee for following simple instructions :)

Okay thanks, guys, I figured it out


correcting git auth by adding callback-path

When trying to configure github authentication, i figured out that the callbackpath setting is missing.
Regards

Hey @messaddek - can you please make this change against branch called release/2.0.0 - it would be perfect!
Thanks.

Hey @skoruba I will perform this today.
Thank you

Hey @messaddek - can you please make this change against branch called release/2.0.0 - it would be perfect!
Thanks.

it has been already done

I can see some conflicts here in PR after switching to release/2.0.0, can you check it please? 😊
Thanks!

Done, could you please check the release/2.0.0

Pr closed, a new one has been created for release/2.0.0


separate new with begin on unit of work struct

To inject TUnitOfWork Dependency into Handler, it's crucial to pull away context from a unit of work.
This is a part of an attempt to set the composition root, where the executable draws an object graph.

Turns out, separation has nothing to do with drawing object graph.


help asap please, module wont work

i downloaded it via pip install. it is not being accessed by thonny (python 3.6.3 with a 3rd party gui), i also tried all of my other versions of python.
i am working on a raspberry pi 3b, the os is raspbian stretch
please tell me how to use it on there
btw, the whole thonny thing dose not matter it is just a gui that uses combines the console and the interpreter

seriously need help

found it, downloading for python 3 i have to use
pip3
not plain old
pip


SharedAdam bias correction wrong

The timestep parameters of the SharedAdam optimizer are not shared. This should lead to bias overcorrection, leading to incorrect unbiased estimates. Does the current implementation work?

Yes, that's true. Thanks! I will fix it in the next couple of days.

Fixed in
https://github.com/ikostrikov/pytorch-a3c/commit/5d9b07d80740e26f78cb283f74b5b802906a9d83


convert pytorch's tensor in Python API

regarding #1087, this PR implements the ability to convert torch tensor from/to candle tensor as follows:

candle to torch: extract candle tensor value and use PyO3 interop to create torch tensor.
torch to candle: extract numpy value from the input torch tensor using PyO3 interop and create candle tensor with it.

Merged, thanks again for the PR!


Fix #305

In case user has overridden JsonConvert.DefaultSettings naming strategy (e.g. camelCase, etc.), it needs to be reset here to produce correct window.JSConfig serialized settings.

Hey thx for this - great catch! Just wondering if, at this point, we should avoid JsonConvert altogether and use an isolated instance of JsonSerializer:
private string GetJSConfigJson()
{
    var serializer = new JsonSerializer { NullValueHandling = NullValueHandling.Ignore };
    using (var writer = new StringWriter())
    {
        serializer.Serialize(writer, JSConfig);
        return writer.ToString();
    }
}

Yep, that's probably the safer approach that can avoid any other side effects from the static converter.

Closing in favor of 8377c006378a7db46186a425f082b798d63c35e9


Missing package dependencies

I have formerly used version 4.0.6. Now after updating to 4.1.1 I have a compilation error at code like
var subscriberTcpListener = observerTcpListner.Subscribe
In the docs there are 3 parameters, 4.0.6 had only one there. After researching, where the 3 parameter signature comes from, it is an extension from Syste,.Reactive.Core. But these packages are not referenced as dependencies.

What platform are you seeing this issue with?
The only changes I did between 4.0.6 and 4.1.1 was to add support for .NET Standard 2.0. I didn't change the any signatures between those two release.

Regarding System.Reactive. Looking at the NuGet System.Reactive 3.1.1 is a dependency for all platforms.

How do I repo this?

I hope I figured out the problem.
In the dependencies of Socketlite, there are only .Net Standard 1.5 and 2.0.
I use it with 1.4, for that no dependencies are defined, thus they are not inlcude. If you have a look into version 4.0.6, there are dependencies for the Reactive package also defined for .Net standard 1.0.
Can Socketlite 4.1.1 no more been used with .Net standard 1.4?

I didn't change anything from 4.0.6 to 4.1.1 except add .NET Standard 2.0.
Version 4.0.6 supported:

NET 4.5.1
Android
iOS
UWP
.NET Standard 1.5

I don't recall why I made a dependency on .NET Standard 1.5.
Why do you need .NET Standard 1.4? I may go back and look at that option, but would be good to understand why it is necessary now that we have .NET Standard 2.0.

I used 4.0.6 with .Net Standard 1.4 and it worked well. I think you can easily go back, even if nothing has been changed. How I understand the .Net standared version is always the minimal version needed. Thus if you enable it for 1.4 it is also possible to use with 2.0.
The need for using 1.4 for me comes from our customer. I use SocketLite in a tool which the customer uses on their software which only supports 1.4. So I cannot go to 1.6. But how you statet if nothing has been changed, then I can stay on 4.0.6 or include System.Reactive.Core manually.

Thank you. I'm on the road traveling this week. Will look at it in the weekend.
It is still a mystery why 4.0.6 works for you and 4.1.1 does not.
Is there any change that you could create a simple project to demonstrates this issue, so that I can reproduce and find a way around this for a future release?

I created a new version 4.1.2-beta1 which includes support from .NET Standard 1.3.
Could you try and see if this fixes the issue?

Thanks, I can only try this on Monday, will report you asap.
Gruß, Wernfried
Von: 1iveowl
Gesendet: Donnerstag, 19. Oktober 2017 00:08
An: 1iveowl/SocketLite
Cc: Schwenkner; Author
Betreff: Re: [1iveowl/SocketLite] Missing package dependencies (#15)
I created a new version 4.1.2-beta1 which includes support from .NET Standard 1.3.
Could you try and see if this fixes the issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


sig width in action when execute_meter is used

I have an action:
action save_and_meter(meter_id) {
modify_field( meter_metadata.meter_id, meter_id);
execute_meter( color_meter, meter_id, meter_metadata.meter_color);
}
for which the metadata field meter_id is 10 bits.
The presence of the execute_meter statement seems to be over-riding the correct determination of the signature width:

h.p4_actions['save_and_meter'].signature_widths
[32]

Without the execute_meter statement the correct value (10) will be reported.

The behavioral model assumes that all meter indexes are 32-bit wide, so I am not surprised by this behavior. This should not be a major problem given that the modify_field primitive action will correctly truncate the 32-bit input to copy it to your 10-bit field. It just means that your action data for this table will be wider than what you want. Ideally however, the compiler would use the instance_count attribute of the meter to determine a more accurate width. For example, if instance_count = 1024, a 10-bit index is good enough. Thanks for reporting this issue, I will try to make this better in the future.


How to insert static obstacle in Rviz?

Expected Behavior
Insert static obstacle in Rviz
Actual Behavior
I have used OpFlag to create static obstacles in Autoware 1.7.0.
However, since I upgraded Autoware to 1.10.0, it looked like it didn't work. The Rviz didn't show any obstacles created.
How can I insert obstacles?
Thanks.
Specifications

Ubuntu version: 4.13.16
ROS version: kinetic
Autoware branch: 1.10.0

Hello Developers,
We study "Lane change of Autoware with Obstacle" demo video from youtube, end user can simulate obstacle on map via RVIZ. According to demo video, this feature is done by dp_planner.
But dp_planner is removed from runtime_manager on Autoware 1.11.0.  Is there any other way to simulate obstacle in Rviz in Autoware 1.11.0?
Many thanks for your help.


CSS Intrinsic Sizing sample

R: @addyosmani @gauntface @beaufortfrancois et al.
CC: @cbiesinger
A sample showing off CSS Intrinsic Sizing keywords used on the width of a <div>.
@cbiesinger, it would be great to have you confirm that this demo sufficiently shows off the new functionality before we merge.
There's a deployed version at https://jeffy.info/samples/css-intrinsic-sizing/ that you can try out live.

LGTM from me.


CSV.generate is not working with Rails 7

I currently updated my app to rails 7.0.2 and since unable to generate the CSV using this gem. I am getting the following error,
> CSV.generate(headers: true)
ArgumentError: Cannot parse nil as CSV
from /usr/local/bundle/gems/csv-3.2.3/lib/csv.rb:1893:in `initialize'

I have tested this and this is working fine with the previous version of rails.
Ruby version
ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux]
Rails version
Rails <IP_ADDRESS>

Can you provide an script to reproduce this issue? it will be easier  to pip-point the issue.
Probably this is not a CSV problem, specially if it's noted only when you upgrade Rails - so I'd suggest to review with and without Rails in your bundle.

Hey @esparta
> CSV.generate(headers: true)
ArgumentError: Cannot parse nil as CSV
from /usr/local/bundle/gems/csv-3.2.3/lib/csv.rb:1893:in `initialize'

I used this code. I used this with my Rails 6 and it is working with all the same gems. This is not working only after I updated my rails and ruby version. Thanks.

Sorry, if was not clear, but with script I mean a complete example which the maintainers (and supporters as myself) can just execute and be able to reproduce without having your bundle. A line with the error it's not enough.
Here's something I did, and it's clear ruby 3.1.2 with bundled CSV gem has not the problem executing this minimal script:
require 'csv'
csv_str = CSV.generate(headers: true) do |csv|
  csv << ['id', 'date', 'amount']
  csv << 123, '2022/05/09', 12.40]
end

Thanks a lot @esparta Did you test this with rails 7.0.2?

Thanks a lot @esparta Did you test this with rails 7.0.2?

No, I didn't and that's what I'm gently asking you: create a reproducible script:
https://guides.rubyonrails.org/contributing_to_ruby_on_rails.html#create-an-executable-test-case

Hey @esparta Thanks a lot for your suggestion. Honestly, I did not have an idea about test scripts like this. Here is my script.
# frozen_string_literal: true

require "bundler/inline"

gemfile(true) do
  ruby '3.1.2'
  source "https://rubygems.org"

  git_source(:github) { |repo| "https://github.com/#{repo}.git" }

  # Activate the gem you are reporting the issue against.
  gem 'rails', "~> 7.0.2", "= <IP_ADDRESS>"
  gem "csv"
end

require "active_support"
require "active_support/core_ext/object/blank"
require "minitest/autorun"

class BugTest < Minitest::Test
  def test_csv
    attributes = %i[id name version]

    all = [{ id: 1, name: 'Ruby', value: '3.1.2' },
      { id: 2, name: 'Rails', value: '7.0.2' },
      { id: 3, name: 'Bootstrap', value: '4' }
    ]

    CSV.generate(headers: true) do |csv|
      csv << attributes

      all.each do |data|
        csv << [
          data[:id],
          data[:name],
          data[:value]
        ]
      end
    end
  end
end

Hey @esparta Thanks a lot for your suggestion. Honestly, I did not have any idea about these scripts. Here is my script,
# frozen_string_literal: true

require "bundler/inline"

gemfile(true) do
  ruby '3.1.2'
  source "https://rubygems.org"

  git_source(:github) { |repo| "https://github.com/#{repo}.git" }

  # Activate the gem you are reporting the issue against.
  gem 'rails', "~> 7.0.2", "= <IP_ADDRESS>"
  gem "csv"
end

require "active_support"
require "active_support/core_ext/object/blank"
require "minitest/autorun"
require "csv"

class BugTest < Minitest::Test
    def test_csv
        attributes = %i[id name version]

        all = [
            { id: 1, name: 'Ruby', value: '3.1.2' },
            { id: 2, name: 'Rails', value: '7.0.2' },
            { id: 3, name: 'Bootstrap', value: '4' }
        ]

        CSV.generate(headers: true) do |csv|
            csv << attributes

            all.each do |data|
                csv << [
                data[:id],
                data[:name],
                data[:value]
                ]
            end
        end
    end
end

Hey @esparta  Thanks a lot for giving your time. I created a new app with specified ruby and rails version and it did work. I highly doubt if any of the gem I installed is causing this. I am still investigating this and dropping the gem file as well here in case any one wants to have a look.
# frozen_string_literal: true

source 'https://rubygems.org'
git_source(:github) { |repo| "https://github.com/#{repo}.git" }

ruby '3.1.2'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', "~> 7.0.2", "= <IP_ADDRESS>"
# Use Puma as the app server
gem 'puma'
# Use SCSS for stylesheets
gem 'sass-rails'
# Use Uglifier as compressor for JavaScript assets
gem 'uglifier', '>= 1.3.0'
# See https://github.com/rails/execjs#readme for more supported runtimes
gem 'mini_racer', platforms: :ruby

# Use CoffeeScript for .coffee assets and views
gem 'coffee-rails', '~> 4.2'
# Turbolinks makes navigating your web application faster. Read more: https://github.com/turbolinks/turbolinks
gem 'turbolinks', '~> 5'
# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.5'
# Use Redis adapter to run Action Cable in production
gem 'redis', '~> 4.0'

# Reduces boot times through caching; required in config/boot.rb
gem 'bootsnap', '>= 1.1.0' # , require: false

group :development, :test do
  # Call 'byebug' anywhere in the code to stop execution and get a debugger console
  gem 'byebug', platforms: %i[mri mingw x64_mingw]
  gem 'rb-readline'
end

gem 'pg'

group :development do
  # Access an interactive console on exception pages or by calling 'console' anywhere in the code.
  gem 'listen'
  # Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
  gem 'bullet'
  gem 'spring', '~> 3.0.0'
end

gem 'capybara-webkit', git: 'https://github.com/thoughtbot/capybara-webkit.git'

group :test do
  gem 'factory_bot_rails'
  gem 'headless', git: 'https://github.com/leonid-shevtsov/headless.git'
  gem 'rails-controller-testing'
  gem 'rspec-rails'
  gem 'stripe-ruby-mock', git: 'https://github.com/stripe-ruby-mock/stripe-ruby-mock.git', require: 'stripe_mock'
  gem 'webmock'
  gem 'webrick', git: 'https://github.com/ruby/webrick.git'
  # Adds support for Capybara system testing and selenium driver
  gem 'capybara'
  gem 'selenium-webdriver'
  # Easy installation and use of chromedriver to run system tests with Chrome
  gem 'webdrivers'
end

# Windows does not include zoneinfo files, so bundle the tzinfo-data gem
gem 'tzinfo-data', platforms: %i[mingw mswin x64_mingw jruby]

# Used for server-side processing of datatables.
gem 'ajax-datatables-rails', '~> 1.2.0'

# Ability to debug code in real time.
gem 'pry'

# Used for mass imports.
gem 'activerecord-import', git: 'https://github.com/zdennis/activerecord-import'

gem 'sidekiq', github: 'mperham/sidekiq'
# Sidekiq used for backgrounding tasks.
group :production do
  source 'https://gems.contribsys.com/' do
    gem 'sidekiq-pro'
  end
end
gem 'sidekiq-scheduler'

# CSV parsing.
gem 'csv'

# user authentication
gem 'devise'

# for document validation
gem 'activestorage-validator'

# Generates a visual diagram to represent model relationships.
gem 'railroady'

# pagination
gem 'will_paginate' # remove

# Convert number to words.
gem 'humanize'

# Slack-ruby-client
gem 'async-websocket', '~> 0.8.0'
gem 'eventmachine'
gem 'slack-ruby-client'
gem 'websocket-driver'

# Message implementation
gem 'twilio-ruby'

# For sidekiq web
gem 'rack'

# For detecting mime types
gem 'mime'

# to call the api from rails application
gem 'rest-client'

gem 'activerecord-session_store'

# To get the next available day after a provided day
gem 'chronic'

# To get current season for password attacks
gem 'ruby-seasons'

# active storage validation
gem 'active_storage_validations'

# necessary for AWS client
gem 'aws-sdk-ec2'
gem 'aws-sdk-s3'
gem 'aws-sdk-ssm'
gem 'aws-sdk-wafv2'

gem 'terminal-table'
gem 'time_difference'

# For PDF generation
gem 'grover'

# Dumping database
gem 'seed_dump'

# Amazon SQS
gem 'aws-sdk-sqs'

gem 'combine_pdf' # to combine the pdfs for report

# Establish Audit Trail
gem 'audited'#, '~> 4.9'

gem 'stripe'

# Ability to duplicate records
gem 'amoeba'

# Calculating percentage changes
gem 'percentage'

# XML/HTML Parser
gem 'nokogiri'

# Atomic Red Team
gem 'atomic-red-team', git: 'https://www.github.com/redcanaryco/atomic-red-team.git', branch: :master

# Better Rails C Colors
gem 'awesome_print'

gem 'jwt'

# Mailchimp
gem 'MailchimpMarketing'

gem 'cocoon'

# Handle expanding and compacting IP addresses and ranges.
gem 'ipaddr'

gem 'rotp'
gem 'rqrcode'

# Sentry.io
gem 'sentry-rails'
gem 'sentry-ruby'
gem 'sentry-sidekiq'

# rubocop
gem 'rubocop'
gem 'rubocop-rails'
gem 'rubocop-rspec'

# Brakeman static analysis tool
gem 'brakeman'

# Security checks
gem 'bundler-audit'

gem 'zeitwerk'

# Rail 7 upgrade
gem "sprockets-rails"
gem 'cssbundling-rails'
gem 'jsbundling-rails'
gem 'stimulus-rails'

Hey Team, I could not find which gem caused this but I used the following and it did the trick.
CSV.generate("", headers: true)
Thanks a lot all

Could you also show the output of https://github.com/ruby/csv/issues/245#issuecomment-1121949544 ?
(I couldn't reproduce this...)

Could you also full backtrace of
> CSV.generate(headers: true)
ArgumentError: Cannot parse nil as CSV
from /usr/local/bundle/gems/csv-3.2.3/lib/csv.rb:1893:in `initialize'

Hey @kou I only have these details. The exception is only showing this.

Could you also show this line "/root/myapp/app/models/root_cause.rb:20:in `to_csv'" ?

CSV.generate(headers: true) do |csv|
This line is at line @kou

Could you add p str debug print before "/usr/local/bundle/gems/csv-3.2.3/lib/csv.rb:1411"?

Could you also add p str.method(:nil?) debug print before "/usr/local/bundle/gems/csv-3.2.3/lib/csv.rb:1411"?

Hey @kou This is a gem file and I don't have any local file with this name so I won't be able to make any changes you have mentioned.

You can use local csv by using gem "csv", path: "/local/path/to/csv" in your Gemfile:
$ git clone https:///github.com/ruby/csv.git /tmp/csv
$ editor Gemfile
(Replace 'gem "csv"' with 'gem "csv", path: "/tmp/csv".)
$ bundle instlal
$ ...

@kou This is what I got.
1404         str = StringIO.new(str)
1405         str.seek(0, IO::SEEK_END)
1406         str.set_encoding(encoding) if encoding
1407       else
1408         str = +""
1409         str.force_encoding(encoding) if encoding
1410       end
1411       puts "str.nil? ----->   #{str.nil?}"
1412       puts "str.method(:nil?) -------> #{str.method(:nil?)}"
1413
1414       csv = new(str, **options) # wrap
1415       yield csv         # yield for appending
1416       csv.string        # return final String
1417     end

This is what I got @kou
str.nil? ----->   true
str.method(:nil?) -------> #<Method: String#nil?() /root/myapp/config/initializers/string.rb:25>

Hey @kou Thanks a lot for suggesting this. I found the issue. .nil? was overwritten in my string.rb file for some reason which caused this. I removed it and now it is working. Thanks a lot all. closing this out.


rename get-starknet-wallet to get-starknet

changed github repo links
renamed get-starknet-wallet -> get-starknet

great, thanks! Lets add automatic releases first, pr incoming

sure!


Add executable package template

Take into account an option for an executable package.

Add or not a main.py file

Add or not a main.py file

Done with commit 17d5329180f920ed1326644e41b58a2c55752051

Done with commit 17d5329180f920ed1326644e41b58a2c55752051


Remove flag parsing operator requirement for docker-machine ssh, and allow SSH command binary args to be passed

This removes the requirement for the -- operator when using SSH, e.g.:
$ docker-machine ssh default df -h

will work again.
It also allows the use of additional SSH command arguments, such as -A for SSH agent forwarding:
$ docker-machine ssh default -A

The syntax is a little weird since the SSH args come after the machine name but it's better to have the feature than nothing I think.  People really missed this about boot2docker.
cc @ehazlett @SvenDowideit @hairyhenderson @sthulb  WDYT
NOTE:  If there are existing scripts with ssh --, it's possible they may break using this method, so maybe we can account for that / leave out that arg if it exists?
TODO:

[x] Docs
[ ] Integration tests

Signed-off-by: Nathan LeClaire<EMAIL_ADDRESS>

Looking good! :+1:

Ya the syntax is a bit odd but I think overall it's good.  I would rather have the ability to pass the options than not.  :+1:

cc @moxiegirl for docs review

ping @theJetzah for any nits

@moxiegirl, such a comprehensive re-write of the documentation in the ssh section seems pretty out of scope for what this PR is changing, how about we go ahead and merge as-is and you can follow up later with a PR if to iterate on it?

@nathanleclaire  That is what a carry is --- I'll make the changes in another PR
If you want me to do that, please answer the technical questions in the first part of my comments, those do apply to your specific paragraph addition and are still outstanding.

Erm, want me to review this, or wait for the carry for nitpicking? :smile:

My understanding was that "carrying" a PR included finishing up work in an existing PR, but not introducing new work.  That's a new PR with a different scope.

This clause is ambigous: the default arguments for the command generated by Docker Machine what do you mean by "the command generated by Docker machine` are you referring to the "ssh" command or the command passed to the shell or something else?

The ssh command generated by Docker Machine for execution by the shell.

Also unclear, does this limitation only apply to the Go native ssh?

Which limitation?  The inability to pass SSH command line arguments to it?  If so, then yes.

Also, this paragraph re Gp native doesn't really belong in here with the forward reference. In general, this should be an edge case that a user is using the Go native driver. If we say to them, don't set the ssh implementation explicitly and they don't, the user shouldn't even be aware which implementation they are using. So, they won't be aware enough to use the feature you point out.

If it's not explicitly called out, I find that users just assume every "default" feature is available with every option.  They should absolutely be able to figure out that they are using "native ssh" or "external ssh" if needed.  Docker Machine will default to native SSH if the ssh binary is not found, so it's a default behavior in some cases as well.

@nathanleclaire Thanks LGTM on your initial paragraph answers. I'll incorporate that perspective into the command rewrite PR.  So, I'm cool with merging and I'll immediately open a replacement if you are more comfortable with doing it that way.  This PR is ok to merge as is. LGTM
@thaJeztah Let's nit on the replacement rather than this as mine will be a rewrite of the command entire.

@moxiegirl gotcha, will save my time for that one :dancer:

Thanks @ehazlett I will be ready to merge this once I add / update integration tests.

Sounds good.  I know a lot of people will love this :)

Boom, tests updated and added a new one for this feature.
I'm ready to merge this whenever you are comfortable @ehazlett .

LGTM

:+1:

@nathanleclaire This happens when i try to set up a tunnel, it justs gets stuck and I can't even Ctrl-c out of it.
kevinsimper$ docker-machine-dev --version
docker-machine-dev version 0.5.0-dev (ebd45f5)
kevinsimper$ docker-machine-dev ssh kubernetes -L 8080:localhost:8080

Running a ps aux gives this
kevinsimper      9498   0.0  0.0 573388680   6288 s002  S+    9:51PM   0:00.02 docker-machine-dev ssh kubernetes -L 8080:localhost:8080

VSZ is unusually high!


Add resource timing spans with events to DocumentLoad plugin

I would suggest that we use the Resource Timing API to get spans for all of the resource timing events that occur before the end of the load event in the browser as part of the DocumentLoad plugin. Each span should include any network events (DNS resolution, SSL handshake, request start, etc.) as TimedEvents on the span when present.
We did this in OpenCensus Web, see this trace screenshot and this code link
@obecny what would you think about adding this feature?

please assign this to me