Add Chrome content script target

Do you want to request a feature or report a bug?
feature

What is the current behavior?
When I lazy load a chunk from a Chrome content script, it inserts a script tag into the page loaded in the active tab, which then causes the code to execute in the context of the webpage instead of that of the Chrome extension. It then throws an error as the webpackJsonp exists in the isolated execution environment and is not visible from the context of the webpage.
If the current behavior is a bug, please provide the steps to reproduce.
N/A

What is the expected behavior?
There should be a Chrome content script target available which uses the appropriate Chrome API calls to lazy load chunks in the isolated execution environment.
If this is a feature request, what is motivation or use case for changing the behavior?
There does not seem to be a way to lazily load chunks from a Chrome content script.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

How to do this chunks xhr+eval can anybody show us code

Closing due to inactivity. Thanks!


Not able to start the project in development mode

Error: --schema path does not exist: C:\Users\Mikk\este\packages\relay\'..\api\schema.graphql'

Feel free to send PR. It works for me.


python311Packages.types-requests: <IP_ADDRESS>40106 -> <IP_ADDRESS>40125

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 283827 run on x86_64-linux 1

2 packages failed to build:

python312Packages.tabcmd
python312Packages.tabcmd.dist

22 packages built:

cwltool
cwltool.dist
gphotos-sync
gphotos-sync.dist
python311Packages.djangorestframework-stubs
python311Packages.djangorestframework-stubs.dist
python311Packages.jsonargparse
python311Packages.jsonargparse.dist
python311Packages.pyarr
python311Packages.pyarr.dist
python311Packages.tabcmd
python311Packages.tabcmd.dist
python311Packages.types-requests
python311Packages.types-requests.dist
python311Packages.unstructured-api-tools
python311Packages.unstructured-api-tools.dist
python312Packages.jsonargparse
python312Packages.jsonargparse.dist
python312Packages.pyarr
python312Packages.pyarr.dist
python312Packages.types-requests
python312Packages.types-requests.dist

Result of nixpkgs-review pr 283827 run on x86_64-linux 1

24 packages built:

cwltool
cwltool.dist
gphotos-sync
gphotos-sync.dist
python311Packages.djangorestframework-stubs
python311Packages.djangorestframework-stubs.dist
python311Packages.jsonargparse
python311Packages.jsonargparse.dist
python311Packages.pyarr
python311Packages.pyarr.dist
python311Packages.tabcmd
python311Packages.tabcmd.dist
python311Packages.types-requests
python311Packages.types-requests.dist
python311Packages.unstructured-api-tools
python311Packages.unstructured-api-tools.dist
python312Packages.jsonargparse
python312Packages.jsonargparse.dist
python312Packages.pyarr
python312Packages.pyarr.dist
python312Packages.tabcmd
python312Packages.tabcmd.dist
python312Packages.types-requests
python312Packages.types-requests.dist


FileNotFoundError while building binary from the Aider codebase

Issue
Hi @paul-gauthier & everyone,
I'm encountering an issue while trying to build a binary from the Aider codebase. During the build process, I encountered a FileNotFoundError. The specific error message is as follows:
File "cli.py", line 659, in <module>
  File "cli.py", line 654, in __init__
  File "cli.py", line 638, in start
  File "cli.py", line 627, in handle_tool
  File "langchain_core/tools/base.py", line 397, in invoke
  File "langchain_core/tools/base.py", line 585, in run
  File "langchain_core/tools/base.py", line 554, in run
  File "langchain_core/tools/structured.py", line 69, in _run
  File "cli.py", line 194, in implement_task
  File "poc_framework/agent_base.py", line 21, in execute_job
  File "code_implementation_sequence/agent_code_change.py", line 57, in define_job
  File "aider/coders/base_coder.py", line 701, in run
  File "aider/coders/base_coder.py", line 752, in run_one
  File "aider/coders/base_coder.py", line 979, in send_message
  File "aider/coders/base_coder.py", line 925, in format_messages
  File "aider/coders/base_coder.py", line 610, in get_files_messages
  File "aider/coders/base_coder.py", line 582, in get_repo_map
  File "aider/repomap.py", line 105, in get_repo_map
  File "aider/repomap.py", line 448, in get_ranked_tags_map
  File "aider/repomap.py", line 76, in token_count
  File "aider/models.py", line 529, in token_count
  File "aider/models.py", line 515, in tokenizer
  File "aider/llm.py", line 20, in __getattr__
  File "aider/llm.py", line 27, in _load_litellm
  File "importlib/__init__.py", line 90, in import_module
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 378, in exec_module
  File "litellm/__init__.py", line 789, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 378, in exec_module
  File "litellm/cost_calculator.py", line 24, in <module>
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "PyInstaller/loader/pyimod02_importers.py", line 378, in exec_module
  File "litellm/utils.py", line 113, in <module>
  File "importlib/resources/_legacy.py", line 25, in wrapper
  File "importlib/resources/_legacy.py", line 62, in open_text
  File "pathlib.py", line 1013, in open
FileNotFoundError: [Errno 2] No such file or directory: '/var/folders/j5/klw27nvj1q5dj27j0qcp1dc40000gq/T/_MEIyE29jr/litellm/llms/tokenizers/anthropic_tokenizer.json'

It seems the build process is looking for a file (anthropic_tokenizer.json) that doesn't exist in the specified directory. Has anyone encountered this issue before, or can someone guide me toward a solution?
The build is done using pyinstaller,
Any help would be greatly appreciated
Version and model info
No response

Thanks for trying aider and filing this issue.
I'm not sure I understand what you mean by "trying to build a binary from the Aider codebase". It is intended to be installed via pip or pipx.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Loading images inside sass

This may not be specific to sass-loader, or maybe it is, but I'm trying to load images into sass files.  Any suggestions on a good way to do this? I've tried file-loader and url-loader but didn't seem to do anything.  Is there a way within sass loader that's recommended for using images?
Thank you

The sass-loader does not resolve url() statements in your styles. You should chain the css-loader as described in the README.
Also take a look at the section about url() in the README.

@jhnns What about a situation where you arent using the css-loader I.E. Angular 2 where you need to run.
`
{test: /.(woff|eot|ttf|svg)$/, loader: 'file-loader?name=assets/[name].[ext]?[hash]'},
  {
    test: /\.scss$/,
    loaders: ['raw-loader', 'postcss-loader', 'resolve-url-loader', 'sass-loader?sourceMap']
  },

`

I don't know. Why aren't you allowed to use the css-loader with Angular 2?

@jhnns I have the problem that @austbot mention; I can't use css-loader because angular styles receive an array of strings so it generates an error when try to render if you use it, and if you don't use it the sass loader don't handle the url images and assets of component styles.

You could try to use the extract-loader, it evaluates a CommonJS module to generate a string. Chain the loaders like this:
  {
    test: /\.scss$/,
    loaders: ['extract-loader', 'css-loader', 'sass-loader']
  },

@jhnns extract-loader fail's on line 1 of the first scss, I just have an import on that line ´@import "../../../sass/variables";´
I use the loaders exacts as you put it.

That sounds like the sass-loader is not applied. The css-loader should not see any Sass imports.
Also, the error would be helpful.

To use the css-loader with Angular2's @Component-Decorator styles use this loader-chain:
exports-loader?module.exports.toString()!css-loader!sass-loader
this will allow you to do:
@Component({
  selector:   'bar-foo',
  template:   require('./foo.template.html'),
  styles:     [ require('./foo.template.scss') ]
})
export class FooComponent {
    ...
}

or this load-erchain for global styles, like ionic-angular or fonts-awesome:
style-loader!css-loader?sourceMap!resolve-url?sourceMap!sass-loader?sourceMap
(resolve-url is required if you don't want to set font-path's and such manually)

Thx for sharing.


lfilter: clamp in [-1, 1] to be optional

🚀 Feature

Make clamp [-1, 1] behavior to be optional by adding an optional argument clamp=False or True to torchaudio.functional.lfilter.
Motivation

I want to apply filters for non-waveform signals, which have values outside [-1, 1].
Pitch

It seems that the input and output are assumed to be waveform ([-1, 1]). However, we can apply filters on arbitrary signals, which may not be in the range of [-1, 1]. Removing or making the clipping behavior optional should give more flexibility to users.
I think the clipping behavior should be disabled in general, but we'd want to make it optional and enabled by default for backward compatibility.
Additional context

https://docs.scipy.org/doc/scipy/reference/generated/scipy.signal.lfilter.html
scipy.signal.lfilter doesn't apply clipping.
A different topic though, function arguments (name, order, etc) are not same as the scipy's one and I was confused at first. We may want to improve compatibility. Any comments are appreciated.

Some of the transform may assume the input is in the range [-1,1] even without clamp, so removing clamp doesn't mean that the transform does what one would want :)
I agree with the idea nonetheless. This also gives the freedom to the user to decide how to keep the waveform within the range.
Either we add a toggle to disable clamp or we deprecate it altogether. Even in the latter case, we should give the user at least one release to update their code. And to do so we would need a toggle anyway. Let's add an option clamp that defaults to True. Would you like to open a pull request?
We can decide later to add a deprecation warning that the option will be remove and that the future behavior will be clamp=False without an option.

I think this assumption of signals being within range [-1, 1] is something that is a bit larger than this particular ask. If various operations make this assumptions they should be clearly flagged as such as we might want to consider some kind of dtype dependent standardization around this.
In my experience [-1, 1] is used for conversions of audio signals into float32, whereas for integers the full range is used at their width often corresponds to the bit depths of the original signal (e.g. int16 for CD audio). We, however, mostly operate on floats for many reasons.

I am not quite familiar with other transforms which may or may not assume signals being in the range [-1, 1]. Let me first fix this for the particular lfilter case. I will open a PR for adding a toggle.
For your information, my real use-case is that I wanted to apply digital filters on a fundamental frequency sequence of speech. In speech synthesis, there are also other cases where we want to apply filters on signals not being in the range [-1, 1]. .e.g., mel-cepstrum, band aperiodicity features.


error: empty enum is invalid (IDFGH-13845)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.3.1 & 5.4-dev ( master branch )
Operating System used.
Linux
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
What is the expected behavior?
Compile without errors
What is the actual behavior?
Error thrown v5.3.1/esp-idf/components/wifi_provisioning/include/wifi_provisioning/manager.h:214:1: error: empty enum is invalid
Steps to reproduce.
Disable all protocomm configuration in menuconfig
Build or installation Logs.
In file included from /home/xxxxxx/esp/v5.3.1/esp-idf/components/wifi_provisioning/include/wifi_provisioning/scheme_softap.h:20,
                 from /home/xxxxxx/esp/v5.3.1/esp-idf/components/wifi_provisioning/src/scheme_softap.c:16:
/home/xxxxxx/esp/v5.3.1/esp-idf/components/wifi_provisioning/include/wifi_provisioning/manager.h:214:1: error: empty enum is invalid
  214 | } wifi_prov_security_t;
      | ^

More Information.
bugged code
manager.h define empty enum when all protocomm security versions are unchecked in Component config → Protocomm

Hello @RDobrinov could you please let me know what is your use case? If you want no security you can enable CONFIG_ESP_PROTOCOMM_SUPPORT_SECURITY_VERSION_0 which is equivalent to no security.

Hello @RDobrinov, closing this issue to due no response, please feel free to reopen the issue if you have any queries or issues further.


'MPMoviePlayerController' is deprecated

OnboardingViewController.h:42:23: 'MPMoviePlayerController' is deprecated: first deprecated in iOS 9.0 - Use AVPlayerViewController in AVKit.

+1

+1

Working on a fix right now. I'm assuming this is an error in XCode 8? I haven't installed it on my personal laptop yet, so I'm not seeing deprecation warnings on my end.

Pushing v2.2.1 to cocoapods right now - let me know if you have any issues.

v2.3.0 should have this fixed.


No longer pin development dependencies

Just use the latest available (and compatible) versions for local development and CI.
Without a proper upgrading process in place, which is overkill for a small project like this, we should just deal with fallout whenever it happens instead of slowly getting stuck in the past… and missing issues occurring with newer tools as they become available.
For example, an older pytest version (6.x) was used, while 7.x is out for a while already. Same for other tools like twine. Latest version as of 2022-09-16:
$ pip freeze | grep --fixed-strings --file=requirements-dev.txt
build==0.8.0
flake8==5.0.4
mypy==0.971
mypy-extensions==0.4.3
pytest==7.1.3
pytest-cov==3.0.0
pytest-mypy-plugins==1.10.0
twine==4.0.1

prior discussion: https://github.com/rustedpy/result/pull/101#discussion_r973178733


fix(web) - nudging fixed, pencil icon

https://github.com/systeminit/si/assets/8495209/0a01d18b-a444-489e-ae89-bf1a8fc7bf16

bors merge


[ENH, MRG] Project sEEG contacts to inflated brain

Adds the ability to project sEEG contacts to an inflated brain originally made in MATLAB by Dora Hermes et al. in discussion with @agramfort.

Here's the artifact https://output.circle-artifacts.com/output/job/d56a4a1d-fc9e-492e-9504-264ae1221e1a/artifacts/0/dev/auto_tutorials/clinical/20_seeg.html

A bit crude and not readily implemented as an MNE-Python function but this looks pretty good.
import os.path as op
import numpy as np
import mne
import imageio

misc_path = mne.datasets.misc.data_path()
sample_path = mne.datasets.sample.data_path()
subjects_dir = misc_path / 'seeg'
subject = 'sample_seeg'

raw = mne.io.read_raw(misc_path / 'seeg' / 'sample_seeg_ieeg.fif')
trans = mne.coreg.estimate_head_mri_t('sample_seeg', subjects_dir)

view_kwargs = dict(azimuth=120, elevation=100, distance=600,
                   focalpoint=(0, 0, -15))

proj_info = mne.preprocessing.ieeg.project_sensors_onto_inflated(
    raw.info, trans, subject, subjects_dir=subjects_dir)

surf_data = dict(lh=dict(), rh=dict())
x_dir = np.array([1., 0., 0.])
for hemi in ('lh', 'rh'):
    for surf in ('pial', 'inflated', 'curv'):
        for img in ('', '.T1', '.T2', ''):
            surf_fname = op.join(subjects_dir, subject, 'surf',
                                 f'{hemi}.{surf}')
            if op.isfile(surf_fname):
                break
        if surf == 'curv':
            surf_data[hemi]['curv'] = np.array(mne.surface.read_curvature(
                surf_fname, binary=False) > 0, np.int64)
        else:
            coords, faces = mne.surface.read_surface(surf_fname)
            if surf == 'inflated':
                x_ = coords @ x_dir
                coords -= np.max(x_) * x_dir if hemi == 'lh' else \
                    np.min(x_) * x_dir
            surface = dict(rr=coords, tris=faces)
            mne.surface.complete_surface_info(
                surface, copy=False, verbose=False, do_neighbor_tri=False)
            surf_data[hemi][surf] = surface['rr'], surface['tris'], surface['nn']

for hemi in ('lh', 'rh'):
    surf_data[hemi]['vectors'] = \
        surf_data[hemi]['inflated'][0] - surf_data[hemi]['pial'][0]

montage_start = raw.get_montage()
montage_start.apply_trans(trans)
ch_pos_start = montage_start.get_positions()['ch_pos']
montage_end = proj_info.get_montage()
montage_end.apply_trans(trans)
ch_pos_end = montage_end.get_positions()['ch_pos']

ch_pos_vectors = dict()
for ch in raw.ch_names:
    ch_pos_vectors[ch] = ch_pos_end[ch] - ch_pos_start[ch]

n_steps = 101
images = list()
for t in np.linspace(0, 1, n_steps):
    brain = mne.viz.Brain(subject, subjects_dir=subjects_dir,
                      cortex='low_contrast', alpha=0, background='white')
    # brain.add_sensors(raw.info, trans=trans)
    brain.show_view(**view_kwargs)

    geo_kwargs = brain._cortex_colormap('low_contrast')

    for hemi in ('lh', 'rh'):
        coords, faces, nn = surf_data[hemi]['pial']
        coords = coords.copy()
        coords += surf_data[hemi]['vectors'] * t
        mesh = mne.viz._3d_overlay._LayeredMesh(
            renderer=brain._renderer,
            vertices=coords,
            triangles=faces,
            normals=nn,
        )
        mesh.map()  # send to GPU
        mesh.add_overlay(
            scalars=surf_data[hemi]['curv'],
            colormap=geo_kwargs["colormap"],
            rng=[geo_kwargs["vmin"], geo_kwargs["vmax"]],
            opacity=0.5 + t / 2,
            name='curv',
        )
        # surf = brain._renderer.mesh(*coords.T, faces, 'gray',
        #                             opacity=0.5)

    for ch in raw.ch_names:
        loc = ch_pos_start[ch].copy()
        loc += ch_pos_vectors[ch] * t
        brain._renderer.sphere(loc * 1000, 'yellow', 3)

    images.append(brain.screenshot())
    brain.close()

brain = mne.viz.Brain(subject, subjects_dir=subjects_dir, surf='inflated',
                      cortex='low_contrast', alpha=1, background='white')
brain.add_sensors(proj_info, trans=trans)
brain.show_view(**view_kwargs)

for i in range(10):
    images.append(brain.screenshot())

brain.close()

imageio.mimwrite('inflated.mp4', images, fps=24)

https://user-images.githubusercontent.com/13473576/215608675-668bd689-fa80-43e3-8eeb-1748c2a68668.mp4

Okay this version runs much quicker, putting it in a gist and adding it to the tutorial @larsoner suggested
import os.path as op
import numpy as np
import mne
import imageio

misc_path = mne.datasets.misc.data_path()
sample_path = mne.datasets.sample.data_path()
subjects_dir = misc_path / 'seeg'
subject = 'sample_seeg'

raw = mne.io.read_raw(misc_path / 'seeg' / 'sample_seeg_ieeg.fif')
trans = mne.coreg.estimate_head_mri_t('sample_seeg', subjects_dir)

view_kwargs = dict(azimuth=120, elevation=100, distance=600,
                   focalpoint=(0, 0, -15))

proj_info = mne.preprocessing.ieeg.project_sensors_onto_inflated(
    raw.info, trans, subject, subjects_dir=subjects_dir)

surf_data = dict(lh=dict(), rh=dict())
x_dir = np.array([1., 0., 0.])
for hemi in ('lh', 'rh'):
    for surf in ('pial', 'inflated', 'curv'):
        for img in ('', '.T1', '.T2', ''):
            surf_fname = op.join(subjects_dir, subject, 'surf',
                                 f'{hemi}.{surf}')
            if op.isfile(surf_fname):
                break
        if surf == 'curv':
            surf_data[hemi]['curv'] = np.array(mne.surface.read_curvature(
                surf_fname, binary=False) > 0, np.int64)
        else:
            coords, faces = mne.surface.read_surface(surf_fname)
            if surf == 'inflated':
                x_ = coords @ x_dir
                coords -= np.max(x_) * x_dir if hemi == 'lh' else \
                    np.min(x_) * x_dir
            surface = dict(rr=coords, tris=faces)
            mne.surface.complete_surface_info(
                surface, copy=False, verbose=False, do_neighbor_tri=False)
            surf_data[hemi][surf] = surface['rr'], surface['tris'], surface['nn']

for hemi in ('lh', 'rh'):
    surf_data[hemi]['vectors'] = \
        surf_data[hemi]['inflated'][0] - surf_data[hemi]['pial'][0]
    surf_data[hemi]['normal_vectors'] = \
        surf_data[hemi]['inflated'][2] - surf_data[hemi]['pial'][2]

montage_start = raw.get_montage()
montage_start.apply_trans(trans)
ch_pos_start = montage_start.get_positions()['ch_pos']
montage_end = proj_info.get_montage()
montage_end.apply_trans(trans)
ch_pos_end = montage_end.get_positions()['ch_pos']

ch_pos_vectors = dict()
for ch in raw.ch_names:
    ch_pos_vectors[ch] = ch_pos_end[ch] - ch_pos_start[ch]

brain = mne.viz.Brain(subject, subjects_dir=subjects_dir,
                      cortex='low_contrast', alpha=0.5, background='white')
# brain.add_sensors(raw.info, trans=trans)
brain.show_view(**view_kwargs)

sensor_meshes = dict()
for ch in raw.ch_names:
    loc = ch_pos_start[ch].copy()
    sensor_meshes[ch] = brain._renderer.sphere(loc * 1000, 'yellow', 3)[1]

images = [brain.screenshot()] * 5

n_steps = 101
for t in np.linspace(0, 1, n_steps):

    for hemi in ('lh', 'rh'):
        coords, faces, nn = surf_data[hemi]['pial']
        coords = coords.copy()
        coords += surf_data[hemi]['vectors'] * t
        nn = nn.copy()
        nn += surf_data[hemi]['normal_vectors'] * t
        brain._renderer.plotter.update_coordinates(
            coords, brain._layered_meshes[hemi]._polydata, render=False)
        brain._layered_meshes[hemi]._polydata.point_data.active_normals = nn
        brain._layered_meshes[hemi].update_overlay('curv', opacity=0.5 + t / 2)

    for ch, sensor_mesh in sensor_meshes.items():
        loc = sensor_mesh.points.copy()
        loc += ch_pos_vectors[ch] * 1000 / n_steps
        brain._renderer.plotter.update_coordinates(
            loc, sensor_mesh, render=False)

    brain._renderer._update()
    images.append(brain.screenshot())

for i in range(5):
    images.append(images[-1])

brain.close()
imageio.mimwrite('inflated.mp4', images, fps=24)

Haha, I'll meet with Dora first (we didn't get to meet today) but I think this is ready to go other than confirming that our results match.

Sorry to be late to the party, but why not just have this automatically be done within brain.add_sensors? I don't see the utility of this function outside of visualization, and it seems dangerous because people can end up with an info that is very wrong for any other purpose (source localization, etc.).

I think similar to the ECoG function, you might want to assign labels based on the projected sensor locations instead of the original sensor locations and this would facilitate that

To me this is different because in this case you're at least projecting to a realistic location in 3D space where the electrode could have been. It's much less wrong than the inflated space.
I think it makes more sense as a purely visualization option. The API also seems cleaner to me that way, you just need a max_dist arg in add_sensors. It can even be 'auto' meaning one sensible thing for ECoG, and one different sensible thing for EEG.

Sounds good, you want it in mne.viz.plot_alignment too right?

I don't think plot_alignment should ever use inflated as a surface, it won't make sense or be helpful (we might want to disallow it at some point actually!). So I wouldn't bother adding it there

I made a separate PR with your suggestion so you can see the code. I think it's very unwieldy, this version works and it's very modular and simple, I really prefer it. I think it would be obvious to someone when they plot the result from stc_near_sensors if they've accidentally used the wrong info, we're not talking about a small shift here. I agree in principle with user-proofing everything as much as possible but I think, in this case, it's not worth the tradeoff of adding even more complexity to the already unwieldy add_sensors + _3d._plot_sensors code that's already a bit of a mess if we're being honest.

Okay @larsoner, I thought about it and I think the complexity was just from trying to model it off of eeg in plot_sensors which wasn't necessary. I think the auto-magic version where it just projects to inflated if your mne.viz.Brain is in inflated is the best of both worlds solution.
Dora has an external deadline of submitting this tomorrow so if this could be merged today, that would be fantastic, sorry for the rush. I'll put the artifact up as soon as it finishes, I think it looks really good!

Check it out, I think that looks pretty cool! https://output.circle-artifacts.com/output/job/4bc08cba-d0ee-43ae-951a-2fb0be1c7934/artifacts/0/dev/auto_tutorials/clinical/20_seeg.html

@larsoner, do you have a minute to review this? Sorry to bother, it would just be great to get it in to submit

Not too bad for ECoG too, I could add that in a follow-up PR

The white electrodes are not a great color is the only issue.

Thanks for the review, I wasn't sure about whether to make the function private, that's fine with me


funding: ignore maxWaitNumBlocksFundingConf for zero conf channels

This is so that if the funding transaction hasn't been confirmed in 2016 blocks, the channel isn't marked as closed by the responder.

LGTM 🌮
I think we can delete this check now that it cannot happen

Deleted the check, good catch


Hijacking the animation system isn't ideal

Using the animation system for playback isn't great, it'd be better to either:

Override the animation player UI and classes to better support Midi (possibly adding a Midi editor as well??)
Use threads to play back the midi in the background, invisible to the user/developer. We'd still trigger signals like the current implementation.

https://github.com/nlaha/godot-midi-4.0/tree/no-animation-refactor
Work on this topic will take place in the above branch. I tried building a custom resource loader/saver for MIDI files, but I couldn't get it to work. Regardless, I think it would work best if instead, I created a simple custom import plugin as it is currently implemented on the main branch. This time though, instead of storing MIDI data in an animation resource, we simply store it in a custom MidiResource.

The last few commits over on no-animation-refactor add a new MidiResource type and the initial UI for a MIDI resource viewer.

In the future I might add editing functionality to this

Closed in https://github.com/nlaha/godot-midi/pull/9


Microphone Waveform Works on Desktop, but not in Mobile browsers nor Hybrid App

Hi, I have this issue where the wave animation is displaying on desktop browsers, but not in mobile browsers nor in the native app. Everything is working fine on the desktop browser.
My current version is wavesurfer.js version 2.1.1
Thank you.

@stoltenheim mobile browsers? please be more specific (like the ticket guidelines told you to).


Sidewalk easel for Finale

We have an aluminum one that I can throw in the car and it's one that if something happens to it damage wise it won't be a big loss.  What are you wanting to put on the easel?

Sponsor board.
On Tue, Nov 7, 2017 at 7:18 AM, TaxLadyTracy<EMAIL_ADDRESS>wrote:

We have an aluminum one that I can throw in the car and it's one that if
something happens to it damage wise it won't be a big loss. What are you
wanting to put on the easel?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/930#issuecomment-342465596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6IxgiHqE0d34kWuzmTne8_DseXF-kKks5s0EqKgaJpZM4QUDlk
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose

I will bring it.


Dogebeatz not working

I reconfigured the JS file to use an internal API key and prefix, Downloaded ffmpeg and it does nothing. No error too. Everything works but audio.

I don't know what is wrong with it, it should be working.


[ISSUE #1545] Replace plugin config loading method with common utils

Fixes #1545
Motivation

add charset param to common loading method.
add unit test of common loading method.
replace plugin config loading method with common loading method

Modifications
#1545
Documentation
None

please split several small pull request for these files changed too much, thanks. @horoc

please split several small pull request for these files changed too much, thanks. @horoc

@xwm1992  Ok, I will submit several PRs of different plugins


Publish docker image

publish geodan/terraintiler

docker image is published by Github Actions after push to main. Repository: https://hub.docker.com/repository/docker/geodan/terraintiler


Ray distribution loss (sigma_loss) is not used?

Hi, thanks for sharing the great work.
I run with the default config file and it works well. However, it use the MSE loss for depth supervision and the ray distribution loss is not used by default. I try to enable it by setting sigma_loss = True in the config file but it raise an error as below:
File "run_nerf.py", line 993, in train
    sigma_loss = extras_col['sigma_loss'].mean()
KeyError: 'sigma_loss'

Could you explain why not use the sigma_loss by default and give some instruction to enable it or do i miss something?
Thanks for your help.

Hi, I also found that sigma_loss can not be used in the code of current version. It seems that depth needs to be passed into render(...) function. Have you succeeded to use sigma_loss?

I have the same question

Same question here. Did anyone tested the performance difference of depth_loss and sigma_loss?

Same question here. Can anyone share how to use sigma_loss?

I think you can find here

@LongruiDong the file you link computes the loss, but this line seems to make it impossible for that to ever run since ray_batch always has 11 columns:
https://github.com/dunbar12138/DSNeRF/blob/f31225b1a1150c9d8e34a74a0b0c8b30dc19ff0f/run_nerf.py#L462

I have the same question

I have the same question


Kernels 2021-07-19

Things done

[x] Fits CONTRIBUTING.md.

Diff LGTM, building 5.13.3

JFYI: hardened patches just released


Feature | adding python 3.12 in pyproject.toml

What?

Support python 3.12

Why?

Support latest python 3.12 version

Before release
Review the checklist here

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2018

Relevant Lines:
2812

💛  - Coveralls


Carousel: Allow disabling keyboard arrows to change slide

components: Carousel
reactstrap version #8.9.0
import method umd/csj/es
react version #17.0.1
bootstrap version #4.5.3

What is happening?
Given a carousel on the page, when I use the left/right keyboard keys the carousel moves to another slide, which I'd like to avoid.
What should be happening?
A disableListeningKeyboard property could be added in order to skip the addEventListener call.
Steps to reproduce issue

Have a Carousel
Press left or right keyboard keys

OMG I missed it, thank you very much ! 🙇


Primary bitstream support for the REST api - Rework

References
Add references/links to any related issues or PRs. These may include:

Rework of https://github.com/DSpace/DSpace/pull/8798
Fixes https://github.com/DSpace/DSpace/issues/3349
UI PR: https://github.com/DSpace/dspace-angular/pull/2213

Description
Rest changes according to the following contract: https://github.com/DSpace/RestContract/pull/222
Instructions for Reviewers
Take a look at the changes in the contract & try them on the branch.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

Hi @tdonohue , I've added the extra auth checks to some existing ITs.


For purging FastCGI cache, add 'cache_purge' module to nginx modules

Motivation for this change
It is common to use something like this for WordPress or Craft CMS sites.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I'm running with this module in production using nginxMainline.


Express vhosts falls back to xhr

For some reason websockets are failing when using express vhosts with socket io. I setup a quick repository of my code that shows how its falling back to xhr.
https://github.com/jamescharlesworth/vhost

That sucks, but it sounds more like an issue of connect instead of socket.io as regular HTTP/HTTPS servers do not suffer from this

Thanks. I'll try posting to connect if I can't debug it myself.


Making adjustment for dpy2

Just minor adjustments made from main branch to fit dpy2

wew, LGTM, thanks for fixing the ctx.channel bugs as well. Helps a lot

pretty sure owo fixed that, I just poked the code with stick


Update nomenclature from base -> steth and bond -> hysteth

Waiting on protocol team to update name of tokens in artifacts (?)

completed by https://github.com/delvtech/elf-simulations/pull/1006


Create Blog “this-is-an-example”

Automatically generated by Netlify CMS

👷 Deploy Preview for famous-kulfi-ae21db processing.

Name
Link

🔨 Latest commit
776cd6b1c054cec515c84f30b1ffd11c7ae9fd5c

🔍 Latest deploy log
https://app.netlify.com/sites/famous-kulfi-ae21db/deploys/657d9b578c5c270008b8ce8a


Google play Requires Prominent App Disclosure for using the useVisitorData hook

Our app uses version 3.1.0 and calls useVisitorData when our app initializes, in order to prevent fraud in our app. Recently we got the following correspondence from Google Play, stating that our app would be removed from the store unless we provide a prominent app disclosure inside our app:
"Your app is uploading users' Installed application information to https://api.fpjs.io/ without a prominent disclosure.
As per Google Play’s [User Data] link
In cases where your app’s access, collection, use, or sharing of personal and sensitive user data may not be within the reasonable expectation of the user of the product or feature in question, you must provide an in-app disclosure of your data access, collection, use, and sharing and seek affirmative user consent."
We found no instructions for compliance with Google Play in order to user this paid plugin service. The documentation should include how to resolve compliance, in order to be able to use the plugin on an app released via Google Play.

Hello @wardjk87,
Thank you for the issue. We are aware of this problem and already working on the solution. We are going to stop collecting the inventory of installed apps from user devices.
We are planning to release this fix to the Android agent first and then we will update the agent version in the React Native SDK. We are planning to do it at the beginning of October.

@ilfa Thank you for your prompt communication and feedback. Since we do not want our app to be delisted, is there an alternative solution other than to remove the SDK? For example, downgrading? We only have until Sept 30 to comply or appeal.

@wardjk87 let me ask my colleagues about the exact ETA for the Android Agent release. (As I remember, you can even update it as a peer dependency without waiting for a new version of the RN SDK.)

Hello @wardjk87!
I appreciate your patience. We couldn't guarantee a new release before September 30th, but I discussed available options with my team.

You could add a prominent disclosure that the list of installed apps is collected and shared with a 3rd party vendor.
If you use the library only for identification, you could downgrade to React Native SDK v2.2.0 and lower (to have Android SDK version lower than v2.3.0).
If you use smart signals let us know which, and we could check if they were supported in this version.
Here you can check the reasons for the major release v3.0.0, looks like it shouldn't be a problem to downgrade.

@ilfa We have investigated v2.2.0, and it looks like we don't need to refactor our front end code. On the backend, we pass the visitor ID, and we use the following enforcement options to determine fraud: suspect score, vpn, ip geolocation, and bot detection. We are going to test out our integration to see if we find any issues, and then we will submit a new version to Google Play.

@wardjk87 VPN detection is significantly worse in this version, other signals are presented with the same quality.
I'll let you know as soon as we release a new version with the fix.

@ilfa We successfully released a version of our app with v2.2.0 of fingerprint, and so far Google has indicated we are compliant. Thanks for guidance. We will update the fingerprint version once a compatible version is available that does not require an in app disclosure.


when squishing the following css Squishit threw an exception (because of regex metacharacters in the url)

.header {
background-image: url(c:\documents and settings\usr8\local settings\temp\1\d1b73b93-5ff0-11de-9339-0017317c60aa);
}

Looks simple enough - good catch!
Thanks for writing a test too - nice to be able to just merge from here and kick off a build :)

Thanks to you and your great work guys :)
Hope to see the change soon on NuGet, to be able to return to "stock
version" :)
2011/12/9 Alex Ullrich <
<EMAIL_ADDRESS>

Looks simple enough - good catch!
Thanks for writing a test too - nice to be able to just merge from here
and kick off a build :)

Reply to this email directly or view it on GitHub:
https://github.com/jetheredge/SquishIt/pull/119#issuecomment-3078507


feat: add read signed url of the vector chunks

Add a function to fetch artifacts of the data directory by data dir fqn.
Add function to get read signed URL of a data dir and local file path
Add read_signed_url along with the file chunk during ingestion to the vector database. This signed URL can be used in the QA response to provide a source file reference to the user on UI.
Refactor MultiModalParser

Move default multi-modal parser prompt to constants
Move LLM instance creation to __init__
Remove unnecessary passing of instance objects like prompt, and llm between functions and use instance vars instead.

Upgrade langchain-openai from 0.1.20 to 0.1.25 to fix issues introduced in https://github.com/truefoundry/cognita/pull/371
Remove unused libraries: openai and orjson
Closes https://github.com/truefoundry/cognita/issues/363

Need clarity on two things:

Need to update chunk metadata with document metadata, if any for all the parsers

How does caching work for parsers with same extension but different config, as the caching key is the file extension.

Not sure about this, need an example.
I missed the part around config. I'll add an md5 hash of the config and extension.

Not sure about this, need an example.
@mnvsk97 - if you would check get chunks function of any parser, it takes file_path and metadata lets call this as doc_metadata as input argument. So when an individual document chunk generates it's own metadata, we should also add the doc_metadata to the chunk metadata.

Another thing that is missing is,
QueryController's required_metadata list should be updated to send pre-signed urls in the response.
https://github.com/mnvsk97/cognita/blob/16c20fb006e9065cc435439d77a44bd58e3981a6/backend/modules/query_controllers/base.py#L24

Not sure about this, need an example.

@mnvsk97 - if you would check get_chunks function of any parser, it takes file_path and metadata (lets call this as doc_metadata for sake of explanation) as input argument. So when an individual document chunk generates it's own metadata, we should also add the doc_metadata to the chunk metadata.
Another thing that is missing is, QueryController's required_metadata list should be updated to send pre-signed urls in the response. https://github.com/mnvsk97/cognita/blob/16c20fb006e9065cc435439d77a44bd58e3981a6/backend/modules/query_controllers/base.py#L24

Resolved all the above mentioned points in the latest commits


how to get the quantitative test results on KITTI?

Do you have the test script on KITTI?

See the code linked in #29
The same BadPixelMetric was used to get the numbers on KITTI


Run mesh conformance tests as part of PR process

Recently we let a pretty egregious istio bug slip into istio-latest which breaks all of knative when sidecars are injected.
We should run a small subset of mesh tests as part of PRs to avoid this. Conformance tests should be enough.
/area test-and-release

We have https://github.com/knative-sandbox/net-istio/issues/136 opened, can I assign it to you @arturenault ?

Sure, I think these are slightly different though. Looks like we need to add a mesh test environment in the first place, and then have it run as part of the PR process. Up to you if this needs two different items though

Close as duplicate; turns out we already support mesh tests but aren't running them.

Close as duplicate; turns out we already support mesh tests but aren't running them.


Allowing passing a null pointer to getrandom() when length is 0

The Linux kernel will handle a null pointer passed to 'getrandom'
without error, as long as the length is also 0. The getrandom crate
relies on this behavior: https://github.com/rust-random/getrandom/blob/ab44edf3c7af721a00e22648b6c811ccb559ba81/src/linux_android.rs#L26
Since it works fine on the actual kernel (and should continue to, due to
the kernel's backwards-compatibility guarantees), Miri should support it
as well.

We already handle NULL pointers just fine when the length is 0. The not_undef just checks that the pointer is initialized, and we should definitely still do that.
I think you ran into the same getrandom bug that I also just ran into. See https://github.com/rust-random/getrandom/pull/73.

FWIW, even with this PR, getrandom still fails here. (You can do cargo update && cargo miri test in test-cargo-miri/ to reproduce.)

@RalfJung: Wow - I opened a duplicate PR within a few minutes of yours: https://github.com/rust-random/getrandom/pull/74
However, applying either of our PRs is not sufficient to make that call to getrandom work. I think the blame actually lies with gen_random:
https://github.com/rust-lang/miri/blob/ed30152b52f650be4e8e2eee8616f7fa41f89569/src/helpers.rs#L86
We unconditionally try to use that pointer. I think we should add a flag to skip doing anything when the length is 0. We can enable that behavior for getrandom, and keep the current behavior for all other usages of gen_random.

I opened a duplicate PR within a few minutes of yours: rust-random/getrandom#74

Talk about race conditions. ;)

However, applying either of our PRs is not sufficient to make that call to getrandom work. I think the blame actually lies with gen_random:

Ah, good catch. check_ptr_access complains on NULL. So dangling pointers (and unaligned pointers) work fine here with size 0, but NULL pointers do not.
I find it likely that all users of gen_random are fine with NULL pointers, so I think gen_random itself should be changed to bail early if the size is 0.

:umbrella: The latest upstream changes (presumably #883) made this pull request unmergeable. Please resolve the merge conflicts.

@RalfJung: I've made the change you requested

@RalfJung Updated

Thanks!
@bors r+

:pushpin: Commit 4d3398fc6262a0159af7311524ade980637121ae has been approved by RalfJung

:hourglass: Testing commit 4d3398fc6262a0159af7311524ade980637121ae with merge bc82f8329829d7efbade879839b5846bfc877a4f...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing bc82f8329829d7efbade879839b5846bfc877a4f to master...


Limit CriticalAddonsOnly taint to a single compute

The CriticalAddonsOnly taint and toleration is meant to be used by K8s
addons that can be scheduled on a compute node but are critical for the
proper functionality of the cluster. KubeVirt is one example for such
addon.
In the existing test, we used to taint every node that runs a kubevirt
component and our components can be scheduled on the master nodes we
might end up tainting masters. The problem with doing it is that some
core components might not tolerate this taint and we might end up with a
broken cluster which might not recover properly.
This patch limits the taint in the test to be set only on one compute
node. It should be enough for us to ensure that our components can
actually tolerate while not risking any core components.
Signed-off-by: Daniel Belenky<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest

/retest


Avoid error if some gateway config is not defined

In downloadReport template to avoid :
Key "reports_sftp_username" for array with keys "client_id, client_secret, merchant_id, sylius_merchant_id, partner_attribution_id, use_authorize" does not exist.

Q
A

Bug fix?
yes

New feature?
no

Hello @Jibbarth @Zales0123 !
Have you got some time to check this change ?


I can't install zvm on Fedora 39 aarch64 (Asahi Linux)

I'm using Fedora 39 aarch64, but I can't install zvm.
$ curl https://raw.githubusercontent.com/tristanisham/zvm/master/install.sh | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3808  100  3808    0     0   6459      0 --:--:-- --:--:-- --:--:--  6454
Installing zvm-linux-aarch64.tar in /home/kebo/zvm
wget is installed. Using wget...
tar: This does not look like a tar archive
tar: Exiting with failure status due to previous errors

Run the following commands to put ZVM on your path via /home/kebo/.profile

echo "# ZVM" >> $HOME/.profile
echo 'export ZVM_INSTALL="$HOME/.zvm/self"' >> $HOME/.profile
echo 'export PATH="$PATH:$HOME/.zvm/bin"' >> $HOME/.profile
echo 'export PATH="$PATH:$ZVM_INSTALL/"' >> $HOME/.profile
Run 'source ~/.profile' to start using ZVM in this shell!

It seems that install.sh tried to download zvm-linux-aarch64.tar, not zvm-linux-arm64.tar.

Thanks for reporting! I’ll take a look at this. In the meantime, the instructions in the README should be sufficient to set up a new install.

I'm using Fedora Asahi Remix, but AFAIK, this is the issue that occurs on all aarch64 Linux, regardless of the ditro. For instance, it also occurs on Ubuntu aarch64.
I found a cause. zvm seems to retrieve a host architecture here. However, in Linux, there is aarch64 but no arm64 in uname -m.
https://stackoverflow.com/a/45125525/1587435
So I think you should rename zvm-linux-arm64.tar to zvm-linux-aarch64.tar on a release note.


Update README.md

Proposed changes

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.

Types of changes

What types of changes does your code introduce to morfeo?
Put an x in the boxes that apply

[ ] 🐛 Bugfix (non-breaking change which fixes an issue)
[ ] ✨ New feature (non-breaking change which adds functionality or enhancements)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] ✅ Tests (added tests for an existing feature)
[ ] 📚 Documentation Update (if none of the other choices apply)
[ ] 🙌 Other (please, write a clear and concise description of the proposal in the section above)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] 👀 I have read the CONTRIBUTING doc
[ ] ✅ Lint and unit tests pass locally with my changes
[ ] 🧪 I have added tests that prove my fix is effective or that my feature works
[ ] 📚 I have added the necessary documentation (if appropriate)
[ ] 🔀 Any dependent changes have been merged and published in downstream modules

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@089c567). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main        #1   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?       104
  Lines           ?       927
  Branches        ?       160
========================================
  Hits            ?       927
  Misses          ?         0
  Partials        ?         0


[Bug Report] Dropdown does not react as expected

Element UI version
1.4.2
OS/Browsers version
Win7/Chrome v61
Vue version
2.3.3
Reproduction Link
https://codepen.io/stevobm/pen/PJPLzo
Steps to reproduce
First, click the right button of the "Menu" button to show the dropdown menu.
Second, click the left button of the "Menu" button.
What is Expected?
The dropdown menu hides  automatically, like bellow:
Bootstrap Dropdown: https://codepen.io/stevobm/pen/veNQBP
Ant Design DropDown: https://codepen.io/stevobm/pen/dVYgEg
What is actually happening?
The dropdown menu doesn't hide automatically.

Fixed in #7089


Issue: Custom Logo does not refresh when saving / resending the configuration profile

I am using Jamf Pro with the JSON Custom Schema from the SupportApp repository.
So far, all configuration changes are working instantly, except the Custom Logo.  I changed from one custom logo to another, and the support app did not refresh that logo immediately.  I had to quit the Support App to refresh it.
My Mac is currently in Dark Mode.  The custom logo (old and new, light and dark) were both in a shared folder (/Library/Application Support/ACMEInc/SupportApp/...).
This Mac is running Ventura 13.4.1.

@bradtchapman do you see any improvements in v2.5?


WIP: Export Prometheus metrics from Streaming module

At the moment, Lua code running in the context of HTTP module cannot share
any state with Lua code running in the context of Streaming module
(even through shared memory).
That's why we're forced to have 2 separate endpoints with Prometheus metrics:
one for HTTP module and another for Streaming module.
However, we don't want our users to start depending on it -
from their perspective there should be a single Prometheus endpoint.
To achieve this, Prometheus endpoint of HTTP module will be making
a sub-request to the Prometheus endpoint of Streaming module under the hood.

Please re-open when this is picked up again. Thanks!


VSCode creates a gpu process after it is started with the --disable-gpu option if there is an opening editor.

VSCode Version: 1.40, 1.41, insiders
OS Version: Ubuntu 16.04

Steps to Reproduce:

Start VSCode with the option --disable-gpu. Run command top -c in terminal, there is no gpu-process of VSCode.
Open a file in VSCode, a process code --type=gpu-process ... can be seen in the output of top -c

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

Related: https://github.com/electron/electron/issues/14273

Thanks for the report, this is as designed, you can find more information in the issue @connor4312  linked above.


Feature-gated SimulateValidatorSet

#42 / #61 follow-up. The idea is to remove SimulateValidatorSet from the release build.
Publishing as draft as this is currently broken: cargo build --features integration --target wasm32-unknown-unknown fails with weird compilation issues.
This seems to be caused by the change in the feature set, produced when setting features explicitly. Couldn't find a way around it. Perhaps cargo build with features is broken for the wasm32-unknown-unknown target? By the way, how do you keep the default features, when explicitly setting a specific set of features? (something like adding a feature to the default feature set).
My intuition tells me there should be a way to do this (conditionally removing SimulateValidatorSet) easily. Open to suggestions / ideas.

Now that I think of it, we can merge this as it currently is (it will effectively disable the dangerous entry point in production), and work on fixing the integration tests later.

This also closes #75

This also closes #75, I think?

Yes, the part about the simulate validators entry point. Is was in Draft, perhaps that's why you missed it.
Marked it as ready for review, as I found a workaround for the wasm compilation.

I'm late, but should QueryMsg::SimulateActiveValidators also be feature-gated?

I don't think so. It's read-only, and useful for debugging. In fact, that's how we detected why testnet-3 was broken.


Enable wrapped/embedded errors for better info

Signed-off-by: Dan Lavine<EMAIL_ADDRESS>Right now the convertErrors() method in etcdstoreadapter masks errors - for example, a failure to connect shows up as ErrorTimeout, which is not strictly accurate in terms of diagnosing underlying failures.
Our change does have the result that users can no longer do simple equivalence checks to determine if an error has a specific type, however, but we don't see any way around that if we still want to pass the underlying errors through while still preserving the existing storeadapter-specific error text.

Hey @shashwathi, @wfernandes, @jfmyers9 do you have any thoughts on this PR?  I might actually propose that the Runtime OG team take ownership of this repo and be able to commit directly.  Any thoughts on that idea?

@Amit-PivotalLabs That seems like a good landing place for this if it is still being used by HM9000 and other Runtime OG components.
On another note, I think it would be nice if etcd-release could provide some sort of "blessed" test runner to spin up ETCD in test. Are there any integration tests in the etcd-release that make use of some sort of test runner? If not I don't see an issue with our continued use of this test runner, but maybe it could be extracted into a different location someday.

@ScarletTanager @DanLavine Why didn't some of these conform to the net.Error? Especially on the connect failure.

@jfmyers9 etcd-release doesn't use a test runner, since it's just concerned with BOSH-ifying etcd.  The etcd-release tests are only concerned with spinning up BOSH deployments of etcd and making assertions, not about spinning up an etcd process itself locally.
@fraenkel would your team be happy to take over ownership of this repo?

If we take it over, it will die when the dea-hm-workspace goes.
My other option was just to clone the code into our workspace and let
the current repo die.  Either way it accomplishes what we all need.
Again, anything the runtime-og team owns falls under the same rules,
changes are only made because they are deemed important either for
security or necessary fixes. There will be no active maintenance or
major changes.

Michael

On 10/14/16 1:17 AM, Amit Gupta wrote:

@jfmyers9 https://github.com/jfmyers9 etcd-release doesn't use a
test runner, since it's just concerned with BOSH-ifying etcd. The
etcd-release tests are only concerned with spinning up BOSH
deployments of etcd and making assertions, not about spinning up an
etcd process itself locally.
@fraenkel https://github.com/fraenkel would your team be happy to
take over ownership of this repo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/storeadapter/pull/12#issuecomment-253578254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4uYQLQMQhP278QRysk_2T5DXq95UxMks5qzmeWgaJpZM4KVBtZ.

It actually sounds like everyone is moving away from etcd (yay!).  I'll switch it over to Runtime OG ownership (just sending ask ticket to change collaborator settings on the repo, and update gitbot to send issues and PRs to the Runtime OG tracker -- I expect there will be 0 of these).  When dea-hm-workspace goes, if someone else still wants this repo, we can figure out who needs to take ownership of it at that time.
I'll close this PR, y'all will be able to commit directly shortly.


Example App throwing erros on iOS

I wanted to test the given example. I simply downloaded it and ran the example. However XCode just throws erros in my face:

Swift Compiler Error (Xcode): Value of type 'Any' has no member 'hasItemConformingToTypeIdentifier'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:58:34
Swift Compiler Error (Xcode): Cannot convert value of type 'Any' to expected argument type 'NSItemProvider'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:59:67
Swift Compiler Error (Xcode): Value of type 'Any' has no member 'hasItemConformingToTypeIdentifier'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:60:41
Swift Compiler Error (Xcode): Cannot convert value of type 'Any' to expected argument type 'NSItemProvider'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:61:65
Swift Compiler Error (Xcode): Value of type 'Any' has no member 'hasItemConformingToTypeIdentifier'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:62:41
Swift Compiler Error (Xcode): Cannot convert value of type 'Any' to expected argument type 'NSItemProvider'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:63:66
Swift Compiler Error (Xcode): Value of type 'Any' has no member 'hasItemConformingToTypeIdentifier'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:64:41
Swift Compiler Error (Xcode): Cannot convert value of type 'Any' to expected argument type 'NSItemProvider'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:65:64
Swift Compiler Error (Xcode): Value of type 'Any' has no member 'hasItemConformingToTypeIdentifier'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:66:41
Swift Compiler Error (Xcode): Cannot convert value of type 'Any' to expected argument type 'NSItemProvider'
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:67:67
Swift Compiler Error (Xcode): Extraneous argument label 'preferredTimescale:' in call
/Users/chris/Programmieren/App-Development/receive_sharing_intent-master/example/ios/Sharing%20Extension/ShareViewController.swift:307:80
Could not build the application for the simulator.
Error launching application on iPhone SE (2nd generation).

What am I missing here? Do I have to configure something else? I couldn't find anything in the readme  on this.

lets be honest. The package is not maintained anymore and we are all hoping for a messia which will create a new package for this. After I have completed my latest 2 projects, I'll try to be the messia and implement a package which will actually work.

lets be honest. The package is not maintained anymore and we are all hoping for a messia which will create a new package for this. After I have completed my latest 2 projects, I'll try to be the messia and implement a package which will actually work.

I decided to take on the task as this wasn't maintained (last published 14 months ago...) and I wanted to provide share to suggestions for deep linking the share into a specific conversation. Hence my open source project Share Handler, https://pub.dev/packages/share_handler. Give it a look. I'll do my best to be active on it.

update swift to version 5


Why the exponential function for the time cost?

Hi, I was wondering why you chose the exponential function instead of the Gaussian for the time cost. For the rest of the costs (distance, color appearance, bbox) you used the Gaussian function.
For example, the distance cost is expressed as:
$$C_d(u,v) = -exp \left( \frac{-d^2}{\sigma_d^2} \right)$$
While the time cost is expressed as:
$$C_t(u,v) = -exp \left( -(\Delta t-1)\sigma_t \right)$$
Why is that? For a unit change in the value of distance (measured in pixels), the Gaussian distance cost changes faster than the exponential cost for a unit change in time (measured in number of frames). It seems that, in the total cost, you wanted to give more relevance to a unit change of distance instead of time. Is it correct, and if yes, why?

Hi, If I recall correctly, I chose the exponential to privilege even more the detections closer in time. This because, in general, the exponential has a steeper descent than the gaussian. If you have three detections where the furthest in time has an appearance and/or distance closer than the closest in time, a steeper descent in the time cost might allow still to pick the closest detections in time.
This was working better in my projects but I cannot say that it generalizes well to different setups.

@lcit thank you for the reply! Indeed, I agree with you that it's a good idea to give a higher relevance to closer detections in time. I did some simple calculations and plotting to investigate the phenomenon better. It seems that the sigma has an effect on whether the exponential or the Gaussian is changing faster. Indeed, the Gaussian has the distance squared in the exponent, and it should thus change faster than the exponential for a unit change of distance and time. However, based on the sigma, there is a range in which the exponential changes faster than the Gaussian. I thus made some plots of the exponent value vs distance/time.
For $\sigma_d=1, \sigma_t=1$, the exponent of the Gaussian is always changing faster, and so are the associated costs. Thus, detections farther in time but at a closer distance will be preferred (the opposite we would like).

For $\sigma_d=3, \sigma_t=1$, there is a small range (until about 7 of distance/time) in which the time exponent changes faster.

For $\sigma_d=10, \sigma_t=1$, the range is even bigger, and in this range, detections closer in distance will be preferred than detections closer in time.

Do you agree with my analysis, or do you see any errors?

There seems to be a problem with your plots. The two functions should be bounded in the range [0,1]. And in general, since the time is dictated by the frame count, the time range to consider is something in the range [0, 3]. You probably would never match detections that are 20 frames apart. Also keep in mind that the first detection has time $t=1$ this is why we have the -1 inside the exponential function.
These are the plots I would expect (the not negated ones). The default parameters in the code are $\sigma_{d}=2$ and $\sigma_t=1$

@lcit my plots were just showing the exponents, so $-d^2 / \sigma_d^2$ for the distance, and $-(\Delta t-1)\sigma_t$ for the time. I just wanted to show that, depending on sigma, the two exponents change with different rates in different ranges. The total powers will behave similarly since we have the same base ($e$) for both exponents. Indeed, we can recognize the same behavior in your plots. If we take the two sigmas = 1, we see that the gaussian is decreasing faster from a time/distance > 1. This means that we will prefer detections closer in distance even if they are farther in time compared to others. The opposite happens with sigmas = 2. Now, the exponential is decreasing faster, and we have the desired behavior, that is, detections are preferred if close in time than in distance.
I am also missing something from your graph. The minimum distance is 0 (when the two detections overlap), but the minimum delta time shouldn't be 1, that is, with detections in immediate successive frames? So, if $-(\Delta t = 1$, then $-(\Delta t-1)\sigma_t = 0$ and the exponential function is 1. So, why your exponential function is 1 at time 0 and not at time 1?

I see. I have shifted the exponential curves to the left to have the first at t=0.

@lcit thank you for the exchange ;)


0.5.0 doesn't include LatexSyntax

I've installed the latest version of compute-engine (0.5.0) and am trying to call LatexSyntax.getDictionary; however, it seems that the exported dist/compute-engine.min.js doesn't actually include LatexSyntax's code. The types seem to indicate that a line like this should work:
import { ComputeEngine, LatexSyntax } from '@cortex-js/compute-engine';

However, this fails when running in the browser. You can also reproduce this by running
$ node
> const ce = require('./dist/compute-engine.min.js')
undefined
> ce
{ ComputeEngine: [class (anonymous)], version: '0.5.0' }
>

Is there another way to import LatexSyntax, now that it was dropped from math-json.min.js?

Ah, yes...
LatexSyntax is not visible because there's no point (I think) in creating a LatexSyntax instance without a ComputeEngine (the two used to be more separated, but correct parsing actually requires access to a ComputeEngine instance).
I did forget about the static LatexSyntax.getDictionary(), though... One option would be to re-export it via ComputeEngine, something like ComputeEngine.getLatexDictionary(). Would that work for you?

That would work great! That's the only use we have for LatexSyntax.

If you get a chance, can you cut a new release with this feature?

mathlive 0.71.0 should have the new static method.

I don't see anything related to getDictionary or getLatexDictionary in the release notes or diff for 0.71.0 -- is this the same as mf.getOptions('macros')?

Oh... sorry, I got confused. It's the Compute Engine that thas the getLatexDictionary() static method, but the version of the Compute Engine that has this is not yet integrated into MathLive.

Yeah, and it doesn't look like there's a new release of Compute Engine with that static method. We actually import Compute Engine directly as well, so if you have a chance to cut a new version with the static method we'll be good to go.

As far as I can tell, this still isn't working, right? I'm trying getLatexDictionary() in this Codepen,  but it isn't defined. Maybe I'm doing it wrong? I just want the proper way of viewing the engine's Latex dictionary.
I'm trying:
import { ComputeEngine } from 'https://unpkg.com/@cortex-js/compute-engine?module';

let engine = new ComputeEngine();
console.log(engine.parse("2 + 3").json)  // ["Add",2,3]
console.log(engine.getLatexDictionary()) // not a f(x)

I can get the engine to parse things, but I can't get getLatexDictionary() to work.
Thanks.

Sorry to clutter the comment section.
I think the issue may be that I'm calling the function on an instance of a compute engine rather than using ComputeEngine.getLatexDictionary(). If that's the case, I'll open an issue for a follow-up question at some point.

@MatousAc yes, getLatexDictionary() is a static method, so you invoke it by using ComputeEngine.getLatexDictionary().

Alright. Makes sense. I am used to this structure in other languages like Java or C++, but I haven't quite explored JS thoroughly enough to take all of this into account properly yet. Thanks.
Before I open a suggested feature issue, am I correct in assuming that this documentation shows the only method for extending the latex dictionary? That is, you must create a new instance of ComputeEngine each time you want to extend the latex dictionary?

That's correct.


Running on MacOS

Just wanted to let you know that I got this working on MacOS, however I had to make a few changes.

Installed click library manually using pip - not sure if this can be added to the requirements.txt (python is not my strength)
Received the following error: NotImplementedError: Low latency not supported on this platform - worked around by commenting out the ser_instance.set_low_latency_mode(True)  # only for FTDI? block.

Thanks for the great project!

Thanks...
The click library is used by the CLI tool, client.py, which is not part of ramses_rf (but which uses it), so is in requirements-dev.txt rather than requirements.txt.
I think I will change the whole code block like so:
    try:  # FTDI on Posix/Linux would be a common environment for this library...
        ser_instance.set_low_latency_mode(True)
    except (AttributeError, NotImplementedError, ValueError):
        # AttributeError: not supported on this OS (i.e. not Posix)
        # NotImplemented: not supported on this Platform (e.g. Darwin)
        # ValueError:     Failed to update ASYNC_LOW_LATENCY (i.e. not FTDI)
        pass

... is there any chance you could test that & report back?

I believe this has been addressed - please re-open if it is not fixed in the next release (>0.11.x).


How to specify a column for distinct_on?

I'm trying to do a query with distinct_on and can't figure out how to get typescript happy:
const languatesQuery = query((q) => [
  q.chatbot_conversations_synth(
    {
      distinct_on: ["detected_language"],
    },
    (c) => [c.detected_language]
  ),
])

This gives me the dreaded "Type instantiation is excessively deep and possibly infinite." error. Is this the right way to specify the columns for distinct_on?

Never mind! Figured out I need to import chatbot_requests_synth_select_column and then use chatbot_requests_synth_select_column.detected_language in the query.


Refactor " : nil" used in ternary operators

Run git grep " : nil" will show over a hundred occurrences.
Refactor to use one of the following techniques:

From object ? object.method_name : nil to object.try(:method_name)
another approach that is more Ruby-esque

I'm on it.

https://github.com/ManageIQ/manageiq/pull/11966

#11973 Kindly review my PR ,it will definitely optimize performance
I am done with all re-factoring changes

This is still valid for anyone that wants to continue it.

@chrisarcand is this still an open issue ?


Not working in modal

The library didn't work if putting it inside modal.

Hi, @phattanun can you show me an example of it? What's happening.

Hello, I am having the same problem, the drag n drop system works fine on normal pages
but once inside modals :

the draggable is offset from the mouse handle
the @drop doesn't trigger when the draggable is dropped

Note that the code for the drag n drop system is the exact same
Video Reference


Custom Dataset

How do I train this on a custom dataset?

You just need to create your dataset (preferably) using TfRecords. Inside the dataset/ folder there is a notebook that might help you creating your new dataset using TfRecords.


SSV2_AutomationUpdateGA failed

Hi everybody,
Since a few days, we receive the following error from the StartStop-Deployment. We already tried to redploy the solution with no success.
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"Failed","message":"Package deployment failed\r\nARM-MSDeploy Deploy Failed: 'Microsoft.Web.Deployment.DeploymentDetailedUnauthorizedAccessException: Unable to perform the operation ("Delete File")  for the specified directory ("C:\home\site\wwwroot\bin\Microsoft.Azure.Management.AppService.Fluent.dll"). This can occur if the server administrator has not authorized this operation for the user credentials you are using.  Learn more at: http://go.microsoft.com/fwlink/?LinkId=221672#ERROR_INSUFFICIENT_ACCESS_TO_SITE_FOLDER. ---> Microsoft.Web.Deployment.DeploymentException: The error code was 0x80070005. ---> System.UnauthorizedAccessException: Access to the path 'C:\home\site\wwwroot\bin\Microsoft.Azure.Management.AppService.Fluent.dll' is denied.\r\n   at Microsoft.Web.Deployment.NativeMethods.RaiseIOExceptionFromErrorCode(Win32ErrorCode errorCode, String maybeFullPath)\r\n   at Microsoft.Web.Deployment.FileEx.Delete(String path)\r\n   at Microsoft.Web.Deployment.FilePathProviderBase.Delete(Boolean whatIf)\r\n   --- End of inner exception stack trace ---\r\n   --- End of inner exception stack trace ---\r\n   at Microsoft.Web.Deployment.FilePathProviderBase.HandleKnownRetryableExceptions(DeploymentBaseContext baseContext, Int32[] errorsToIgnore, Exception e, String path, String operation)\r\n   at Microsoft.Web.Deployment.FilePathProviderBase.Delete(Boolean whatIf)\r\n   at Microsoft.Web.Deployment.DeploymentObject.DeleteOperation(DeploymentSyncContext syncContext)\r\n   at Microsoft.Web.Deployment.DeploymentObject.Delete(DeploymentSyncContext syncContext)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.HandleDelete(DeploymentObject destObject, DeploymentObject sourceParentObject)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncDirPathChildren(DeploymentObject destRoot, DeploymentObject sourceRoot)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildren(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncDirPathChildren(DeploymentObject destRoot, DeploymentObject sourceRoot)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildren(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildrenNoOrder(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildren(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildrenOrder(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildren(DeploymentObject dest, DeploymentObject source)\r\n   at Microsoft.Web.Deployment.DeploymentSyncContext.ProcessSync(DeploymentObject destinationObject, DeploymentObject sourceObject)\r\n   at Microsoft.Web.Deployment.DeploymentObject.SyncToInternal(DeploymentObject destObject, DeploymentSyncOptions syncOptions, PayloadTable payloadTable, ContentRootTable contentRootTable, Nullable`1 syncPassId, String syncSessionId)\r\n   at Microsoft.Web.Deployment.DeploymentObject.SyncTo(DeploymentProviderOptions providerOptions, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)\r\n   at Microsoft.Web.Deployment.DeploymentObject.SyncTo(String provider, String path, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)\r\n   at Microsoft.Web.Deployment.DeploymentObject.SyncTo(DeploymentWellKnownProvider provider, String path, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)\r\n   at Microsoft.Web.Deployment.WebApi.AppGalleryPackage.Deploy(String deploymentSite, String siteSlotId, Boolean doNotDelete) in C:\__w\1\s\src\hosting\wdeploy\Microsoft.Web.Deployment.WebApi\AppGalleryPackage.cs:line 343\r\n   at Microsoft.Web.Deployment.WebApi.DeploymentController.<DownloadAndDeployPackage>d__25.MoveNext() in C:\__w\1\s\src\hosting\wdeploy\Microsoft.Web.Deployment.WebApi\Controllers\DeploymentController.cs:line 492'"}]}
Thanks in advance.

Duplicate of #116


More detailed exception messages when retrieving columns

refs https://github.com/k1LoW/runn/issues/124
I could not reproduce the error in the test code using sqlite, so I would like to detail the error and check the contents.

fm-
https://github.com/k1LoW/runn/runs/8103770196?check_suite_focus=true#step:7:55
--- FAIL: TestLoad (0.03s)
    operator_test.go:344: invalid desc: <nil>
{
  "a": {
    "bool": "[func]",
    "compare": "[func]",
    "filename": "include_a.yml",
    "int": "[func]",
    "steps": {
      "a": null,
      "e": {
        "exit_code": 0,
        "stderr": "",
        "stdout": "hello a\n"
      },
      "t": null
    },
    "string": "[func]",
    "upcase": "[func]",
    "urlencode": "[func]",
    "vars": {
      "filename": "include_a.yml"
    }
  },
  "b": {
    "bool": "[func]",
    "compare": "[func]",
    "int": "[func]",
    "steps": {
      "t": null,
      "t_func": null,
      "t_main": null
    },
    "string": "[func]",
    "upcase": "[func]",
    "urlencode": "[func]",
    "vars": {
      "filename": "include_a.yml",
      "filename_main": "include_main.yml"
    }
  }
}
{
  "filename": "include_main.yml"
}
FAIL

https://github.com/k1LoW/runn/runs/8103860582?check_suite_focus=true#step:7:98
TestLoad and TestShard tests have a resource conflict-like appearance.

Execution Result
invalid column: evaluated created_at, but got DATETIME(2022-08-31 09:14:58): strconv.Atoi: parsing "2022-08-31 09:14:58": invalid syntax

DATETIME
I guess we need to make it compatible with the date/time format.
A separate PR will be prepared.


[Feature] Make easyclick/easyclick_all work on all monitors during

When using  you can use EasyClick to select the window you want, however it only seems to show the hints on the main monitor. It would be nice if it were to show the hints on all monitors during . I tried using <easyclick_all>, but it didn't seem to work during task view. If that's not possible having an option to bind a custom key instead of using enter for selecting a window during <switch_window> would be nice.

I have added a macro capability to <switch_window> since it seems to close when <easyclick_all> is executed on <taskview>.
You can use the hint-selection by configuring a mapping like map <c-w>s <switch_window><easyclick><click_left> on the following build.
win-vind_build_138_206_64bit_portable.zip
If there are any bugs, please report it.
Thanks.

It's working pretty well. I used it throughout yesterday and it made switching windows very comfortable, thanks!
The only thing I noticed is that it will sometimes show two letters over a window and pressing either of them focuses it. I think it's always putting a hint in the center, so it's probably showing that center hint next to the actual window hint. Not really a big issue in my opinion.
I would attach a screenshot, but I can't figure out how to take a screenshot with the hints during the window switching.

@Salted-Bread
Sorry for my late reply.
I will release this build after finishing some integration tests.
Thanks!


JRI from an AsyncAPI perspective

@handrews do correct me if I am wrong in any of the cases 😆

Should always produce a valid JSON document (resolving referenced resource should always be compatible with JSON)

Defined by https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#general-syntax

Should define the behavior for referencing non-JSON resources in a JSON document (this can be as simple as resolving the content into a string value)
Should define the behavior for referencing JSON data

Same as JRef, this is decided per media type, and explicitly allow the context specification to determine the behavior, any of the above use-cases seem to be covered.

Should define the behavior for referencing JSON data that also have reference behavior, and how they interconnect / or not (say two standards both use $ref the new standard should define a clear separation between the two and how referencing tools should interpret it)

https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#extending-jri-extending
https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#standalone-jri-implementations
https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#avoiding-behavioral-overlap
Everything is context aware, and is dynamic based on media type, i.e. resolving references in a JSON Schema context follows different rules for $ref then in the context specification of JRI.

Should define the behavior of nested schemas within the same file (so there is no difference between what the spec allows and what tooling enables)

https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#fragments-and-secondary-resources defines the behavior for fragments and secondary resources.
https://github.com/json-schema-org/referencing/blob/main/draft-handrews-jri.md#discovering-caching-and-serving-resources-serving defines the behavior for caching and bundling.
Context specifications take priority, which clears up the confusion.

Thanks, @jonaslagoni - let's start with:

Should always produce a valid JSON document (resolving referenced resource should always be compatible with JSON)

I assume "produce" here means removing all references and replacing them with their targets in some way.  This is not possible with cyclic references.  Are you just trying to remove all of the non-JSON Schema references because they are guaranteed to not be cyclic?
What exactly is driving the need to produce a valid JSON document (which can then be parsed and loaded into memory like any other JSON document) vs being able to load the original document(s) into a usable in-memory representation mixing JSON and non-JSON values in a way that is clear while traversing the data-structure?
If you resolve non-JSON reference targets to a string that is embedded into a JSON file, then when you parse that JSON file you will need to know that the string in that spot requires further parsing and interpretation.  So with either approach, you need to know when and how to handle that bit of in-memory data.
This feels a bit to me like a tension between "$ref"'s late-binding design (which is necessary for cyclic reference support among other things) and some other requirement I don't quite understand.

I assume "produce" here means removing all references and replacing them with their targets in some way

It was to emphasize that the referencing standard takes it's root in JSON (cause that's what you write the AsyncAPI file in), so naturally the referencing standard (as JSON Reference) should work within that restriction, even though you might reference something that is not JSON.

Are you just trying to remove all of the non-JSON Schema references because they are guaranteed to not be cyclic?

Nothing is guaranteed, while something like protobuf definitely does not support circular types, XSD does.

What exactly is driving the need to produce a valid JSON document (which can then be parsed and loaded into memory like any other JSON document) vs being able to load the original document(s) into a usable in-memory representation mixing JSON and non-JSON values in a way that is clear while traversing the data-structure?

It comes from an implementation point of view, and specifically these two cases come to mind:

Without a JSON structure, how will you apply fragments? Cause if it's not clear what is arrays and what is objects fragments might be mismatching what is intended (if fragments are allowed of course)
We need a way to interact with the referenced resource, to be able to use it in tooling. Here the easiest IMO was just to keep it JSON 🙂


Allow for passthrough of extra args to pg_dump

I started off looking for a way to restrict the anonymized database dump to a particular schema, which pg_dump already supports via the -n flag. While we could conceivably define an equivalent pg-anonymizer flag and pass it through, the pg_dump tool already supports a rich variety of CLI flags that help control what objects get dumped and we wouldn't want to have to define separate flags for each one that somebody might want to use.
A more flexible approach, as implemented here, could be to switch from a single, named positional arg (a database connection string) to instead use variable-length arguments which we can then pass through to pg_dump (after sanitizing to ensure that we aren't breaking pg-anonymizer by altering the dump format or writing to a file rather than stdout).
By setting the strict = false flag, we instruct he OCLIF arg parser to gather up any unrecognized flags as positional arguments which are then exposed in argv. You can also force the issue by using the special -- argument to separate flags from arguments a la GNU getopt. So the following commands are equivalent and both result in passing -n myschema mydb to pg_dump:
$ pg-anonymizer -n myschema -l first_name:faker.name.firstName mydb
$ pg-anonymizer -l first_name:faker.name.firstName -- -n myschema mydb

I believe this change to be fully backwards compatible with the existing CLI-parsing behavior, but in the absence of a test suite I'm not sure what the best way of demonstrating that might be.

This is a potential fix for https://github.com/rap2hpoutre/pg-anonymizer/issues/23

Hi! Just wanted to see if there are any questions I can answer or anything else I can do to help get this PR reviewed.

Hi @alexhall! Thank you for your contribution and sorry for the delay. Your addition seems legit and I guess it's a good move.

I believe this change to be fully backwards compatible with the existing CLI-parsing behavior, but in the absence of a test suite I'm not sure what the best way of demonstrating that might be.

Oops sorry about that!
The only thing I want to be sure is: will this command (the default command) still work? (TBH I don't remember how to test a CLI without publishing it so I prefer asking)
npx pg-anonymizer postgres://user:secret@localhost:1234/mydb -o dump.sql

Hi @rap2hpoutre, thanks for getting back to me!

The only thing I want to be sure is: will this command (the default command) still work? (TBH I don't remember how to test a CLI without publishing it so I prefer asking)
npx pg-anonymizer postgres://user:secret@localhost:1234/mydb -o dump.sql

Yes, I can confirm that this still works. Here's a simple test run to demonstrate:
~/src/pg-anonymizer (add-pgdump-args) > createdb testdb
~/src/pg-anonymizer (add-pgdump-args) > psql testdb -c "create table people(name text, email text); insert into people (name, email) values ('John Doe', 'jdoe@example.com');"
INSERT 0 1
~/src/pg-anonymizer (add-pgdump-args) > bin/run postgres://alex:**********@localhost/testdb -o dump.sql
Launching pg_dump
Command pg_dump started, running anonymization.
Output file: dump.sql
Anonymizing table public.people
Columns to anonymize: name, email
~/src/pg-anonymizer (add-pgdump-args) > cat dump.sql
--
-- PostgreSQL database dump
--

-- Dumped from database version 12.10 (Ubuntu 12.10-1.pgdg20.04+1)
-- Dumped by pg_dump version 12.10 (Ubuntu 12.10-1.pgdg20.04+1)

SET statement_timeout = 0;
SET lock_timeout = 0;
SET idle_in_transaction_session_timeout = 0;
SET client_encoding = 'UTF8';
SET standard_conforming_strings = on;
SELECT pg_catalog.set_config('search_path', '', false);
SET check_function_bodies = false;
SET xmloption = content;
SET client_min_messages = warning;
SET row_security = off;

SET default_tablespace = '';

SET default_table_access_method = heap;

--
-- Name: people; Type: TABLE; Schema: public; Owner: alex
--

CREATE TABLE public.people (
    name text,
    email text
);

ALTER TABLE public.people OWNER TO alex;

--
-- Data for Name: people; Type: TABLE DATA; Schema: public; Owner: alex
--

COPY public.people (name, email) FROM stdin;
Samuel Bogisich<EMAIL_ADDRESS>\.

--
-- PostgreSQL database dump complete
--

This would be a huge win for our use!

Thank you for your contribution (and sorry for the late answer!)


After client.shutdown node healthcheck still runs [JIRA: CLIENTS-683]

If connection to riak failed, i want to shutdown the client and expect that it will stop all riak processes, but it is still running and firing healthcheck errors every 30 seconds.

Hello,
Can you provide the code you're using to shut down the client? Thank you.

Hi, Thanks for your reply.
I'm doing something like:
this.client = new Riak.Client(nodes);
var self = this;
this.client.ping(function(res){
if (res.code == 'ECONNREFUSED') {
                winston.info("shutting down riak...");
                self.client.shutdown(function (state) {
                    if (state === Riak.Cluster.State.SHUTDOWN) {
                        console.log("done.");
                    }
                });

            }
        })

Shutting down the client will be improved in the next release.
In the meantime, please see how the integration tests shut down the client and wait for the correct state:
https://github.com/basho/riak-nodejs-client/blob/v1.8.1/test/integration/kv/storevalue.js#L220
Basically, subscribe to the stateChange event and only continue when you see the Riak.Cluster.State.SHUTDOWN state.
Yes, I know this is clunky, which is why it will be fixed in 1.9

Will be fixed in 1.9
[posted via JIRA by Luke Bakken]


Subtitle For Video Doesn't Work

Hello, I don't know why but I added sub for my video on Google Drive, if i watch it from original link, its ok, but with this one, i can not see sub, can you please tell me how can i add sub for this?

Please visit the jwplayer site for more information how to add subtitles https://www.jwplayer.com/


Parsing json-object with dot in their name?

Hi, I've been trying but not succeeding to parse json data as follows:
{
  "version" : "4.0.0",
  "gauges" : { },
  "counters" : {
    "TransferManager.downloads.queue" : {
      "count" : 2
    },
    "TransferManager.forwards.queue" : {
      "count" : 0
    },
    "TransferManager.transfers.active" : {
      "count" : 5
    }
  },
  "histograms" : { },
  "meters" :
    {
    "TransferManager.downloads.failed" : {
      "count" : 1012,
      "m15_rate" : 0.0031013012486067124,
      "m1_rate" : 0.020422990475295447,
      "m5_rate" : 0.008097230208595894,
      "mean_rate" : 0.0013440113841602863,
      "units" : "events/second"
    },
    "TransferManager.downloads.succeeded" : {
      "count" : 20407,
      "m15_rate" : 0.02925335031554721,
      "m1_rate" : 0.019210550446002225,
      "m5_rate" : 0.025774833786456255,
      "mean_rate" : 0.027102016120216895,
      "units" : "events/second"
    },
    "TransferManager.forwards.failed" : {

Didn't paste the whole json but the idea is that I would like to parse that data from the endpoint without modifying it. I got this working by changing all dots to _
My config before changing the JSON:
---
modules:
  default:
    metrics:
    - name: execution_status
      type: object
      path: '$.meters["TransferManager.downloads.failed"]'
      labels:
        count: '.count'
      values:
        status: 1`

This didn't work. It says that Transfer Manager cannot be found.
This works after changing the data:
---
modules:
  default:
    metrics:

    - name: Failed_Downloads
      type: object
      help: Metrics for Failed Downloads
      path: "{.meters.TransferManager_downloads_failed}"
      labels:
        environment: xxxxxx
        metrics: transfer statistics
        system: xxxxxxx
        owner: xxxx
        count: '{.count}'
      values:
        status: '{.count}'

I would love to get things parsed without first fetching the JSON and then modifying and after that serving modified data from localhost to the scraper.
Probably it's just me that doesn't understand but wanted to ask you guys.

Cheers

I guess this might be helpful for you
https://github.com/prometheus-community/json_exporter/issues/126#issuecomment-1058337924

this is an open bug in client-go/jsonpath()
https://github.com/kubernetes/client-go/issues/1018


Somehow document which packages and tools are part of the installers

I doubt most users will be aware of the fact that we bundle FSLeyes, the GitHub CLI, and other super helpful tools if we don't tell them we do.
We removed this list from the MNE docs out of fear it would be outdated very quickly.
Since all installed "extra" conda packages are listed in construct.yaml, it should be relatively easy to generate a nicely-formatted list of bundled packages every time we build the installers. I think it could be as simple as a markdown table (left column: package name; right column: short description of what it does)
This list could live in this repository, and the official MNE docs would just link to it.
WDYT, @agramfort @larsoner?

no strong feeling here. I never user fsleyes and I still don't use the GH
CLI :)

Message ID: @.***>

But even so, there's many more packages and tools that we bundle :) like dcm2nixx
In the future, we could even include git is we wanted to …

Cross-ref https://github.com/mne-tools/mne-python/issues/11831 which is related


issue/3777-Project venv path gets deleted without prompt (#5436)

Pull request opened by the merge tool on behalf of #5436

Merged in 6d8e29b5bf5d95a7d05ec831a81e797b44cb347b


Add experimental article on authenticator flows

Quick question: did you run the diagrams past Alex Cohen?

I didn't, I first wanted to see if this sort of article makes sense in the DocC content.
I'll check those figma templates out, and will pass the resulting diagrams past Alex. Thanks for the tip!

This is a great set of information for developers using the SDK. The first thing that occurred to me when I read it is that it should be in the developer.okta.com documentation: Guides > Sign users in > Mobile authentication > Authentication using OAuth2 (or some appropriate title).
If you're OK with that, I'd like to use this content as the basis for a new article.

After a conversation with @mauricesharp-okta it looks like this is ready to merge.


take Paths and Files

would be useful for static typing and debugging purposes to have some of the methods take Path and File
for example I'm currently doing this
    private Path dir;
    loader.setPrefix(dir.toString());

          var template = engine.getTemplate(skel.getSourcePath().toString());

this isn't particularly useful
com.mitchellbosecke.pebble.error.LoaderException: Could not find template "TestApplication.java.pebble" (?:?)

but if I change my code to
          var template = engine.getTemplate(skel.getSourcePath().toAbsolutePath().toString());

suddenly my error message contains the full path it was looking for, and I can see exactly why it couldn't find it. pebble could use Path.resolve to do this internally. It would make my code simpler and easier to debug. My code could become
    private Path dir;
    loader.setPrefix(dir);

          var template = engine.getTemplate(skel.getSourcePath());

additionally you could use Files.list() which returns a Stream<Path> easier to iterate over templates in a single directory.
lastly, IMHO, strings are the weakest static type, because people (have to) put lots of naughty things in them, nothing really guarantees that the string that you pass is what you mean to these methods is what you mean.
note: I mention File because it seem an obvious extension of this, but I personally don't care about it as much.

It makes sense if you are using a FileLoader. But what if you have a custom loader which loads templates from database ? Or if you have a ServletLoader ?
It only applies for FileLoader I think

fair, perhaps, though see my follow up to #556, I'm not certain about the servlet loader, but due to the fact that you in theory could use a FileLoader in conjunction with another loader, if you did this.
even if this isn't done, could we improve the error message when using file loader to give the absolute path? at first I didn't realize my CWD was at the root of my repo (running via IDE), because my application/gradle isn't.

also, could StringLoader be added as an example in the docs? I did look for it.

Yes. You can access a map entry as a field, this is documented at https://pebbletemplates.io/wiki/guide/basic-usage/#variables. The documentation doesn't say it but I just checked the code and List values can be accessed like arrays. (com.mitchellbosecke.pebble.attributes.ListResolver)

If foo is a map, foo.get("bar")

suggests that you have to do foo.get("bar") and foo["bar"] won't work in that case.

I've created a patch for the documentation at #562.

cool.

Behind the scenes foo.bar will attempt the following techniques to to access the bar attribute of the foo variable: