ffpb not working for some extensions

Hi. ffpb results in this traceback when I try to convert from .m4a to .mp3.
$ ffpb -i test.m4a test.mp3
Exception in thread STDOUT/ERR thread for pid 8681:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/threading.py", line 917, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.7/threading.py", line 865, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 1540, in wrap
    fn(*args, **kwargs)
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 2484, in output_thread
    done = f.read()
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 2943, in read
    self.write_chunk(chunk)
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 2918, in write_chunk
    self.should_quit = self.process_chunk(chunk)
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 2839, in process
    return handler(chunk)
  File "/home/ritiek/.local/lib/python3.7/site-packages/sh.py", line 1624, in fn
    return handler(chunk, *args)
  File "/home/ritiek/.local/lib/python3.7/site-packages/ffpb.py", line 89, in __call__
    self.progress(line)
  File "/home/ritiek/.local/lib/python3.7/site-packages/ffpb.py", line 129, in progress
    total=self.duration * self.fps,
TypeError: unsupported operand type(s) for *: 'int' and 'NoneType'

and no progress bar is shown although it does invoke ffmpeg and the output file test.mp3 works as expected.
I also tried to convert from .mp4 to .mkv and the progress bar indeed gets displayed but for some reason it fails for .m4a to .mp3.

I noticed that ffmpeg displays this line for video conversion:
frame=  147 fps= 18 q=-1.0 Lsize=     366kB time=00:00:06.06 bitrate= 494.7kbits/s speed=0.725x

and this for audio conversion:
size=    1083kB time=00:01:09.25 bitrate= 128.1kbits/s speed=25.1x

It skips displaying some fields that have no meaning in audio. Maybe we could skip these too or something, if ffpb fails to capture the fps value?

Thanks for the report, I'll be patching that.

Fixed in v0.1.2.


missing go.sum entry after ran go mod tidy

You can use go bug to have a cool, automatically filled out bug template, or
fill out the template below.

EOF

Expected behavior
go generate ./... should run without error
Version
go 1.16

github.com/google/wire v0.5.0

Additional context
Following the error message and run the suggest go get commands fixes the problem. But as soon as I run go mod tidy, go generate ./... reports error

Possibly related to https://github.com/golang/go/issues/44129
As a hack, you can run GOFLAGS=-mod=mod go generate ./...

Possibly related to golang/go#44129
As a hack, you can run GOFLAGS=-mod=mod go generate ./...

I can confirm the workaround working

FWIW, I think the "correct" work-around is to follow the advice here and create a tools.go file to track your development-only dependencies. This should resolve the go.sum problem and doesn't require setting GOFLAGS that introduce potentially unintuitive behavior.

I have the same problem. I do have a tools.go file yet the go.sum entry is still missing after go mod tidy
// +build tools

// Place any runtime dependencies as imports in this file.
// Go modules will be forced to download and install them.
package tools

import (
	_ "github.com/google/wire"
)

// +build tools

// Place any runtime dependencies as imports in this file.
// Go modules will be forced to download and install them.
package tools

import (
	_ "github.com/google/wire"
)

(Go 1.16.7)

I believe this is because you need to be importing "github.com/google/wire/cmd/wire". This is the actual tool you use, not the library.

you need to be importing "github.com/google/wire/cmd/wire".

Good catch!
This was the culprit, now it works.
Thanks a lot 🎉

Yes, I'm using the // +build tools workaround as well
There is some guidelines from here about tracking tools: https://github.com/golang/go/wiki/Modules#how-can-i-track-tool-dependencies-for-a-module
And in my case importing github.com/google/wire/cmd/wire is enough

https://github.com/ca-love/server/blob/master/Makefile#L76
も
Go1.18を見越して、
go install
に変えておいたほうがいいのかな?

add import:
`
package tools
import (
_ "github.com/google/subcommands"
)
`
then :
go mod tidy.

you can try go clean --modcache, then go generate` again


Remove Google App Engine Standard Facet

close #199

@elharo @briandealwis

Looks good. Please patch in https://github.com/GoogleCloudPlatform/gcloud-eclipse-tools/pull/220 and test against that.

LGTM. Please merge with master and verify it all still works before checking in (though I don't see any reason it shouldn't).

Do we really want to remove this facet?  Might it be useful to act as a placeholder to distinguish between general Servlet webapps and those written for AppEngine?

YAGNI. If we need it we'll write it or restore it. Until then, it's better if we don't clutter the projects with it.


Tensorboard writers are not cleared between hydra configurations

🐛 Bug
Tensorboard writers are not cleared between hydra configurations
To Reproduce
This problem was spotted while running training of Wav2Vec-U with default parameters.:
TASK_DATA=/path/to/features/precompute_unfiltered_pca512_cls128_mean_pooled
TEXT_DATA=/path/to/data/phones  # path to fairseq-preprocessed GAN data (phones dir)
KENLM_PATH=/path/to/data/phones/kenlm.phn.o4.bin  # KenLM 4-gram phoneme language model (LM data = GAN data here)

PYTHONPATH=$FAIRSEQ_ROOT PREFIX=$PREFIX fairseq-hydra-train \
    -m --config-dir config/gan \
    --config-name w2vu \
    task.data=${TASK_DATA} \
    task.text_data=${TEXT_DATA} \
    task.kenlm_path=${KENLM_PATH} \
    common.user_dir=${FAIRSEQ_ROOT}/examples/wav2vec/unsupervised \
    model.code_penalty=2,4 model.gradient_penalty=1.5,2.0 \
    model.smoothness_weight=0.5,0.75,1.0 'common.seed=range(0,5)'

When hydra switches to a different configuration , a _tensorboard_writers from fairseq.logging.progress_bar.py are not cleared which results in a SummaryWriter trying to write to a file named as in previous configuration but in a different directory. Writers from previous configurations are still used because keys in _tensorboard_writers are valid, train which is the same across configs.
Traceback (most recent call last):
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/threading.py", line 917, in _bootstrap_inner
    self.run()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/summary/writer/event_file_writer.py", line 233, in run
    self._record_writer.write(data)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/summary/writer/record_writer.py", line 40, in write
    self._writer.write(header + header_crc + data + footer_crc)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/io/gfile.py", line 519, in write
    self.fs.append(self.filename, file_content, self.binary_mode)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/io/gfile.py", line 150, in append
    self._write(filename, file_content, "ab" if binary_mode else "a")
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/io/gfile.py", line 154, in _write
    with io.open(filename, mode, encoding=encoding) as f:
FileNotFoundError: [Errno 2] No such file or directory: b'tb/valid/events.out.tfevents.1629890258.somemachine.3252802.0'

^CTraceback (most recent call last):
  File "somepath/someuser/miniconda3/envs/w2vu/bin/fairseq-hydra-train", line 33, in <module>
    sys.exit(load_entry_point('fairseq', 'console_scripts', 'fairseq-hydra-train')())
  File "somepath/someuser/fairseq/fairseq_cli/hydra_train.py", line 84, in cli_main
    hydra_main()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/main.py", line 37, in decorated_main
    strict=strict,
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/utils.py", line 355, in _run_hydra
    lambda: hydra.multirun(
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/utils.py", line 198, in run_and_report
    return func()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/utils.py", line 358, in <lambda>
    overrides=args.overrides,
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/hydra.py", line 136, in multirun
    return sweeper.sweep(arguments=task_overrides)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/core_plugins/basic_sweeper.py", line 154, in sweep
    results = self.launcher.launch(batch, initial_job_idx=initial_job_idx)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/_internal/core_plugins/basic_launcher.py", line 80, in launch
    job_subdir_key="hydra.sweep.subdir",
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/hydra/core/utils.py", line 129, in run_job
    ret.return_value = task_function(task_cfg)
  File "somepath/someuser/fairseq/fairseq_cli/hydra_train.py", line 28, in hydra_main
    _hydra_main(cfg)
  File "somepath/someuser/fairseq/fairseq_cli/hydra_train.py", line 53, in _hydra_main
    distributed_utils.call_main(cfg, pre_main, **kwargs)
  File "somepath/someuser/fairseq/fairseq/distributed/utils.py", line 369, in call_main
    main(cfg, **kwargs)
  File "somepath/someuser/fairseq/fairseq_cli/train.py", line 173, in main
    valid_losses, should_stop = train(cfg, trainer, task, epoch_itr)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/contextlib.py", line 74, in inner
    return func(*args, **kwds)
  File "somepath/someuser/fairseq/fairseq_cli/train.py", line 302, in train
    cfg, trainer, task, epoch_itr, valid_subsets, end_of_epoch
  File "somepath/someuser/fairseq/fairseq_cli/train.py", line 388, in validate_and_save
    valid_losses = validate(cfg, trainer, task, epoch_itr, valid_subsets)
  File "somepath/someuser/fairseq/fairseq_cli/train.py", line 466, in validate
    progress.print(stats, tag=subset, step=trainer.get_num_updates())
  File "somepath/someuser/fairseq/fairseq/logging/progress_bar.py", line 374, in print
    self._log_to_tensorboard(stats, tag, step)
  File "somepath/someuser/fairseq/fairseq/logging/progress_bar.py", line 392, in _log_to_tensorboard
    writer.add_scalar(key, stats[key], step)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/torch/utils/tensorboard/writer.py", line 349, in add_scalar
    self._get_file_writer().add_summary(summary, global_step, walltime)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/torch/utils/tensorboard/writer.py", line 96, in add_summary
    self.add_event(event, global_step, walltime)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/torch/utils/tensorboard/writer.py", line 81, in add_event
    self.event_writer.add_event(event)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/summary/writer/event_file_writer.py", line 113, in add_event
    self._async_writer.write(event.SerializeToString())
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/summary/writer/event_file_writer.py", line 166, in write
    self._byte_queue.put(bytestring)
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/queue.py", line 139, in put
    self.not_full.wait()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/threading.py", line 296, in wait
    waiter.acquire()
KeyboardInterrupt
^CError in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/site-packages/tensorboard/summary/writer/event_file_writer.py", line 176, in flush
    self._byte_queue.join()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/queue.py", line 89, in join
    self.all_tasks_done.wait()
  File "somepath/someuser/miniconda3/envs/w2vu/lib/python3.7/threading.py", line 296, in wait
    waiter.acquire()
KeyboardInterrupt

I will attach a pull request as my (simple) proposition to resolve this.
Expected behavior
Continue training with different configuration without thrown exception.
Environment

fairseq Version (e.g., 1.0 or master): master
PyTorch Version (e.g., 1.0) 1.9.0+cu111
OS (e.g., Linux): linux
How you installed fairseq (pip, source): source
Python version: 3.7

Hello, I ran into the same problem as you during gan training. I'm sorry I didn't understand your solution, can you send me a repaired fairseq.logging.progress_bar.py? My email is<EMAIL_ADDRESS>Thank you very much for your help. If possible, can I contact you by email?

It's best if we communicate here, someone else might benefit too :))
There's my PR linked to this issue where both fairseq/logging/progress_bar.py and fairseq_cli/train.py  are properly changed.
To describe the problem a little bit more: hydra sweeper is running multiple configs in series but it is all within one process which means global variables are shared. If you look at how objects are written to _tensorboard_writers you will see that the keys correspond to data splits valid, train. The problem is that when sweeper is running the next configuration these writers are not cleared and so they are being re-used since these keys are used in every training procedure. So writers are re-used but location changes and because writers are opened they are tryingt o write to a file with the name corresponding to the old configuration but  in the new place, hence the error. If you look at the tfevent file in the previous configuration you willl notice that the name it is trying to write in new configuration is the same., which in general should not take place.

Ok, using your PR has successfully solved my problem. Now I can train for a long time without being interrupted by loading new configurations. I am currently conducting a wav2vec-U experiment on the timit data set, using the same parameter settings as in the paper, but during the training process, the loss shows an upward trend and it is difficult to converge, resulting in the final WER as high as 80%. Have you encountered any problems in this regard, and would you like to give me some suggestions?


Cards not working in MS Teams

I am using Hero Cards, Receipt Cards, and Adaptive cards in my bot. I am testing it out in ms teams right now and have the following issues:

The buttons for my Hero Card are not working properly (pressing them enacts a Luis Intent). Here is the code for said hero card:
       [LuisIntent("Help")]      public async Task showBotInformation(IDialogContext context, LuisResult result)      {          var helpMsg = context.MakeMessage();          var helpHeroCard = new HeroCard          {              Title = "Hi, I'm Bridget.",              Text = "I'm here to provide easy, information. ",              Buttons = new List<CardAction> {                  new CardAction(ActionTypes.PostBack , "Common Qs" , value: "show common questions"),                  new CardAction(ActionTypes.PostBack , "Settings" , value: "settings")              }          };          helpMsg.Attachments.Add(helpHeroCard.ToAttachment());          await context.PostAsync(helpMsg);      }

Receipt cards aren't showing up at all? Are receipt cards not supported?

@meganlcooper can you please share the code, we can review and debug the issue?
From: meganlcooper<EMAIL_ADDRESS>Sent: Thursday, June 8, 2017 9:29 AM
To: Microsoft/BotBuilder<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Microsoft/BotBuilder] Cards not working in MS Teams (#2906)
I am using Hero Cards, Receipt Cards, and Adaptive cards in my bot. I am testing it out in ms teams right now and have the following issues:

The buttons for my Hero Card are not working properly (pressing them enacts a Luis Intent). Here is the code for said hero card:
[LuisIntent("Help")] public async Task showBotInformation(IDialogContext context, LuisResult result) { var helpMsg = context.MakeMessage(); var helpHeroCard = new HeroCard { Title = "Hi, I'm Bridget.", Text = "I'm here to provide easy, information. ", Buttons = new List { new CardAction(ActionTypes.PostBack , "Common Qs" , value: "show common questions"), new CardAction(ActionTypes.PostBack , "Settings" , value: "settings") } }; helpMsg.Attachments.Add(helpHeroCard.ToAttachment()); await context.PostAsync(helpMsg); }
Receipt cards aren't showing up at all? Are receipt cards not supported?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FBotBuilder%2Fissues%2F2906&data=02|01|v-muarum%40microsoft.com|95e06ed597224ac9c19908d4ae8b89c0|72f988bf86f141af91ab2d7cd011db47|1|0|636325361716344036&sdata=BvVXb6xioZZKQvaG6V2P%2ByRzwmONU3qFzeeNLZ0YhEg%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAbu1HIjYlqxPEMeE1WkliMLVBnzGLRhwks5sCCFngaJpZM4N0UQl&data=02|01|v-muarum%40microsoft.com|95e06ed597224ac9c19908d4ae8b89c0|72f988bf86f141af91ab2d7cd011db47|1|0|636325361716344036&sdata=JBXDBlIlXsn1bSFXDUzPLhazqH3i4rpZXbNFa7Re6Yo%3D&reserved=0.

Adaptive Card support in MS Teams is still officially listed as "Coming Soon"

Receipt card + Adaptive Cards are not currently supported in MS Teams.

Yes sorry, here is the code for my Hero Card:
[LuisIntent("Help")]         public async Task showBotInformation(IDialogContext context, LuisResult result)         {             var helpMsg = context.MakeMessage();             var helpHeroCard = new HeroCard             {                 Title = "Hi, I'm Bridget.",                 Text = "I'm here to provide easy, information. ",                 Buttons = new List<CardAction> {                     new CardAction(ActionTypes.PostBack , "Common Qs" , value: "show common questions"),                     new CardAction(ActionTypes.PostBack , "Settings" , value: "settings")                 }             };             helpMsg.Attachments.Add(helpHeroCard.ToAttachment());             await context.PostAsync(helpMsg);         }
The buttons trigger LUIS intents when the postback happens, but for some reason in teams all I see is the postback and the LUIS intent isn't triggered? I am using ngrok to test in teams.

@meganlcooper PostBack is actually not supported in Teams.  We have imBack and invoke, which are the separate pieces of PostBack (imBack for the text response, invoke for the silent JSON post.)


pkcs11-helper: git rev -> 1.11, openvpn w/ pkcs11-helper support

Motivation for this change
New version, and yubi keys with openvpn.
Things done
Tested building with and without the feature.

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

[X] NixOS
[ ] OS X
[ ] 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.

Sorry for these careless errors, I'll fix and push in ~30 minutes


add hot reload modules for templates

for example
import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import * as serviceWorker from './serviceWorker';

const rootEl = document.getElementById('root')

ReactDOM.render(
  <App />,
  rootEl
)

if (module.hot) {
  module.hot.accept('./App', () => {
    const NextApp = require('./App').default
    ReactDOM.render(
      <NextApp />,
      rootEl
    )
  })
}

// If you want your app to work offline and load faster, you can change
// unregister() to register() below. Note this comes with some pitfalls.
// Learn more about service workers: https://bit.ly/CRA-PWA
serviceWorker.unregister();

A duplicate of #4732, but I agree there should be some kind of statement on HMR within the readme, even if that's to state why they suggest against it.

This has been superseded by react-refresh which was merged in a couple months ago but is still marked as experimental.


upgrade to selenium-java 4.0.0-alpha-2

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.
Checklist

[ ] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

What's the status of this? Any help required? I would like to use Selenide with Selenium 4.0.0-alpha3. Let me know if there's a way to contribute to the upgrade efforts.

@martinschneider We are not working on this pull request. Since Selenium 4.0 is not released yet, I don't see any reason why we should upgrade to it. If I understood correctly, it will not be released soon.
By the way, why do you want to use Selenium 4.0.0-alpha3?

@asolntsev Fair enough if the plan is to wait for an official release. My point is, that once Selenium 4 is out, people will start upgrading and expect other libraries (like Selenide) to "just work".
While I agree that it doesn't make sense, to release a version of Selenide which uses an alpha version of Selenium, I think it would make a lot of sense to keep this PR active and track the implications and amount of work supporting Selenium 4 will bring.
After all, that's what alpha releases are there for, to give tool developers like us a heads-up of what's coming.

@martinschneider Thank you for the remainder!
I am absolutely agree, alpha releases are there for us to try them as early as possible.
I have merged master to this PR, fixed failing tests. Will keep it up-to-date until Selenium 4 is released.

@martinschneider Hey, but the way, if you want to use version from this branch you can easily build it and get for using via jitpack:
https://jitpack.io/#selenide/selenide/upgrade-to-selenium4-SNAPSHOT

Overwritten by https://github.com/selenide/selenide/pull/1099


Fix unlock vault on controller key for zfs

We should unlock zfs vault in different way on controller key received

@giggsoff   can please write a brief description on the problem found and how this fix helps.

@giggsoff can please write a brief description on the problem found and how this fix helps.

The logic described here #1423. I noticed that we do not handle zfs decryption.

@giggsoff thank you for the link. LGTM.

This might be sufficient, but it is very hard to tell given the naming and structure in vaultmgr. Hard to tell which functions are generic (such as setupDefaultVault) and I would think that unlockVault should also be generic.
Thus I think it makes sense to refactor/rename things so that there aren't checks for the type in seemingly random places. The cleanest way to do that would be to define an interface with the functions which can then be implemented for ext4 and zfs (and it would make sense to put the ext4/fscrypt specific functions in a new vaultmgrext4 file). If an interface is too painful one could have the generic functions have a switch on the type and call genericext4() or genericzfs() functions.
But we can go ahead with this fix/PR and do that refactoring as a separate PR.

Make sense. Let's schedule it for the next PR


The True Black theme need improve contrast,

The True Black theme need improve contrast, the text is not clear. Maybe need change the text color to 0xFFFFFF

@xiamr thanks for the feedback i will make sure to address this in the next release


six.moves htmlentitydefs error

Error during import:
from htmlentitydefs import codepoint2name, name2codepoint
ModuleNotFoundError: No module named 'htmlentitydefs'

Code is not trying to use 'htmlentitydefs':
from six.moves.html_entities import codepoint2name, name2codepoint
In Python 3.8, html.entities.entitydefs, html.entities.name2codepoint, html.entities.codepoint2name are all under html.entities  not htmlentitydefs

I don't understand your problem. Six lets you import codepoint2name etc. in Python2 and Python3 with the six.moves statement you posted. Works as designed.

Yes, sorry, pip has an old version of the code which does not use six and hence the error.
I've emailed the maintainer to update pip to the current version.


Adding position id and segmentation models

Adding id_position and segmentation models from GL:alexdesiqueira/mothra_models here.

Auto-merging.


project: support multiple cnames on the app

Instead of requiring a single cname, we now look for multiple cnames,
and if we can find the cname that matches the environment, that's ok.
This makes migration to tranor easier.

 Last update df2acda...6256c5b

:+1:


[Bug]: Required option '-i' is missing when running spec:regen

Package name
schematics
Package version
10.1.1
Reproduction steps
I created a SDK and failed relative path of my .yaml.
So in order to regen the spec, I copy paste my yaml in root sdk directory and renamed it as "swagger-spec.yaml".
Then I ran
npm run spec:regen --workspace=@irs/sdk
Current result
The output is :
UPDATE swagger-spec.yaml (1988 bytes)
[error] Required option '-i' is missing
Error: OpenApiGeneratorCli failed to run OpenApiGenerator with command 'openapi-generator-cli generate<EMAIL_ADDRESS>--generator-key irs-sdk'
npm ERR! Lifecycle script `generate` failed with error:
npm ERR! Error: command failed
npm ERR!   in workspace<EMAIL_ADDRESS>npm ERR!   at location: C:\embedded_insurance\irs-frontend\libs\irs-sdk
npm ERR! Lifecycle script `spec:regen` failed with error:
npm ERR! Error: command failed
npm ERR!   in workspace<EMAIL_ADDRESS>npm ERR!   at location: C:\embedded_insurance\irs-frontend\libs\irs-sdk

Expected result
No Error
Additional comments
I ran with the new swagger-spec.yaml :
npm run generate --workspace=@irs/sdk
And it works but after, I tried to regen and had the same error as above.

Hi @ADR-Clizy ,
Thanks for your feedback.
We have tested the usecase using the 11.1 version of @ama-sdk/create, with a spec file coming from a relative path and with a spec file coming from an npm module. We could not reproduce the issue with spec:regen anymore. npm run spec:regen works in both cases.
In case we missed something with the reproduction, don;t hesitate to come back to us.


Sound not playing

something to do with the src attribute i think

sound only plays if it's in a live server but doesn't in html


Lenis Incompatibility with ScrollArea in shadcn

Describe the Bug
I am building a React app that uses shadcn. While lenis worked fine in most areas, an issue occurred in my Dashboard Layout, where I am using a ScrollArea component to display content. lenis caused the scroll area to malfunction, preventing it from scrolling as expected. The shadcn ScrollArea component is derived from @radix-ui/react-scroll-area. (Note: I can still scroll manually using the scrollbar in the scroll area, but cannot scroll using the mouse wheel.)
To Reproduce

Create a React app with shadcn.
Use the ScrollArea component from shadcn.
Apply lenis on top of it.

Here’s how my layout is set up
<>

      {/* Sidebar */}
      <StudentNav />

      {/* Main content area */}

        {/* Fixed Header */}
        <StudentHeader />

        {/* Scrollable area for the Outlet */}
        <ScrollArea className="flex-1 overflow-auto" style={{ overflowY: 'auto' }}>

            <Outlet />

        </ScrollArea>

</>

I have the same problem

Please share a minimal reproduction link, you can use codesandbox.

I'm closing this issue until your share a minimal reproduction link.


bad import of gaussian_latitudes

https://github.com/NOAA-MDL/grib2io/blob/6acd48cb71bcfa65c7ebd29fffd541ffbcb92773/grib2io/_grib2io.py#L965C35-L965C35
The line should be:
from .utils.gauss_grid import gaussian_latitudes

Thank you for report this.  This fix will be available in the next couple of days in v2.1.2 release.


Where is the path to the“ text_encoder” defined

When running domain_shift.py, I get the following error:
Traceback (most recent call last):
File "\cross_modal_adaptation-main\domain_shift.py", line 425, in
main(args)
File "\cross_modal_adaptation-main\domain_shift.py", line 215, in main
head, num_classes, in_features = make_classifier_head(
TypeError: make_classifier_head() missing 1 required positional argument: 'text_encoder'
I want you to tell me the exact location of the text_encoder path so that I can modify the path to avoid the error

Thank you for spotting this bug. I updated the domain_shift.py file that hopefully fixes this issue (sorry I didn't have time to test).
If you want to fix it yourself, you just need to move up the line of code that loads the text_encoder, which was right below.
image_encoder = torch.load(
                        image_encoder_path).partial_model.train().cuda()
text_encoder = torch.load(
                        text_encoder_path).partial_model.train().cuda()
head, num_classes, in_features = make_classifier_head(
                        args.classifier_head,
                        args.clip_encoder,
                        args.classifier_init,
                        text_dataset,
                        text_encoder
                    )
                    ```

Thank you for your reply. After your tips, I realized that this error might be caused by the missing keyword, change the code to the following, it can run successfully
#zero_shot_weights = get_zero_shot_weights(text_dataset, num_classes, in_features)
zero_shot_weights = get_zero_shot_weights(text_dataset, num_classes, in_features,text_encoder)

Owner
Thank you for your reply. After your tips, I realized that this error might be caused by the missing keyword, change the code to the following, it can run successfully
#zero_shot_weights = get_zero_shot_weights(text_dataset, num_classes, in_features)
zero_shot_weights = get_zero_shot_weights(text_dataset, num_classes, in_features,text_encoder)


Add more functionality to cvdr connect and mark version 0.2.0 for unstable release

let's move ConnectDevice, and gatherDisconnectedDevices to a different file keeping cli.go about commands layout.

Done


Fix vercel deployment mismatch

Description
My 99th try of trying to fix this.
context #419 and all referenced PR inside it.
Obviously there is a catch, had to add next as devDependency of monorepo because of https://github.com/orgs/vercel/discussions/6047 but now atleast it uploads whole monorepo (mainly yarn.lock) which fixes all the issues
Work flow doesn't change at all you just need to choose all defualt options and add vercel project :
https://github.com/scaffold-eth/scaffold-eth-2/assets/80153681/32a76d20-c6e2-49fc-9072-bdac9f7a947b

My 99th try of trying to fix this.

hahaha Thanks for this @technophile-04 great stuff.,

Obviously there is a catch, had to add next as devDependency of monorepo

I can live with this haha. This is a great solution.
All good for me. Let's see if @Pabl0cks can test on Windows for final confirmation

Thanks all merging this 🙌


Node version 20

I also saw the Node 16 warning in Issue #54
I've bumped to Node 20 and also updated package.json.

[ ] But to get this PR's build working, I think the push-prebuilt-action PR needs merging first.

[ ] I'm also trying to remember how to test this... it's been a while. Is it enough that the various Github Actions run successfully?

@jpribyl any chance you can help this get merged in? Thanks! 🙇

@GrantBirki yeah, although a number of tests are failing-- we might need to have a look at breaking changes from node 16--20 and see what else (if anything) needs updating!

@jpribyl any updates here?

Need some help?


Adding tests/impl tags for Utils

This PR adds test/impl taags for Utils

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
[x] The documentation is still up-to-date in the doc folder.
[x] No requirements were altered.
[x] The dependencies are still up-to-date in pyproject.toml.

Ignore the code coverage diff.
 Someone has a PR out that increases code coverage from main, and coveralls can't tell the difference between main and a PR branch.


Show why list permission should never be given if 'GET' isn't

Currently it's possible to give the LIST permission, without granting GET. Due to this some Kubernetes administrators may be under the impression that those items cannot be obtained by users or service accounts with only the LIST permission.
This is incorrect, and I'd like to add this example to show why this shouldn't be done.
I'm happy to move this elsewhere in the documentation!

/sig security

@RichardoC 👋🏻 Please feel free to /reopen when you're ready to continue.
/close


How to Enter Seed after password is forgotten ?

Thank you for filing an issue!

Version
Version 0.4.3 (0.4.3)
Description
No clear indication of how to enter seed password when you forget password. Where is recovery "tab" ?
Also note issue 468 was closed without fix.
That is very useful , here we have one attempt to enter a pwd we cannot see and hope we have typed it as expected. Most systems now enforce password confirmation to check both fields.
Log

Importing seeds it feature flagged for now to prevent people from actually entering seeds for accounts that have value. Voyager is not properly security audited yet.
You can clone the repo and run Voyager as a developer to preview all features.
Issue #468 was closed because the implementation was completed in #532.
There was no release since this implementation, but we aim to release early next week. So stay tuned!
Thank you for your feedback.
May I ask for which project you use Voyager for?

Hi ,
No I am an ICO investor. Just looking to play with Voyager and the Cosmos SDK . I have a technical background in databases , so just interested.
Thanks


Fix deprecated usage

https://docs.python.org/3.3/library/urllib.request.html#urllib.request.Request.get_host
https://docs.python.org/3.3/library/urllib.request.html#urllib.request.Request.get_type

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

Awesome!


Add bot-initiated messaging ability

The approach is to use an idle handler, which gets called whenever something else hasn't already happened, or once per second. This should help reduce rate limiting.
Functional testing has been added.
Updated README.md with how to use the new functionality.
Resolves #92.
Dev Notes
To allow the idle handler to send messages, I give it the instance of the SlackClient as a parameter.
To make SlackClient easier to use, I made send_message, rtm_send_message, and upload_file all take a "channelish" string, instead of a channel. This turns a string into a channel as follows:

Given a channel id, return that same channel id.
Given a channel name, return the channel id.
Given a user id, return the direct message channel with that user, opening a new one if necessary.
Given a user name, do the same as for a user id.

This way a user can just client.send_message("some_user", "Here the message") instead of having to call any other helper functions.

Oh boy, didn't see #89 and #54 already. Hmm well I'll leave this open as it's another approach. It's basically like #54 except it provides easy access to the client right there, and a nicer interface for sending messages (using channel names & usernames instead of ids).
Also it opens direct message channels if they aren't open yet. I think this may be necessary for messaging a user who has never messaged you, for example.

@lins05 due to @prathyush not updating his PR, do you think we could focus on this one and fix what is needed to merge it? I think this would be really useful for slackbot.

sad this isn't merged.. this is a killer PR

Looks awesome! I'm excited to see this merged!

Any chance of this being merged?

Is there any plan for this PR to be merged? I've seen that there are requests for changes. @csaftoiu any thoughts? In a fork of this repo I managed to build on top of the functionality added by @csaftoiu and added the possibility of running a function at certain times.


Bower package name as lowercase

A problem arised when I had already installed Materialize ~0.97.6 in my repo, then angular-materialize has dependency on materialize ~0.97.2 (all lowercase). They end up being installed side by side as different packages, resulting in this dir tree:
bower_components
├── materialize
└── Materialize

... which cordova did not like (which is how I spotted this). However, attempting to install with bower install Materialize yields Package Materialize not found.
Would it be good to name the package with all lowercase letters in bower.json? This may be a deeper issue with bower as well.
Reproduce
bower install angular-materialize
bower install materialize

bower_components
├── angular
├── angular-materialize
├── jquery
├── materialize
└── Materialize

Would this issue be resolved if materialize was always lowercase? If so, then this may be a duplicate of
https://github.com/Dogfalo/materialize/issues/2389
and should be fixed by this PR
https://github.com/Dogfalo/materialize/pull/3087

Another issue referencing this is #3195 which shows the warning displayed by bower
I've also included a PR to fix both the naming and the other warnings in #3212

Fixed by #3212


LANG-1034: Recursive and reflective EqualsBuilder

patch by yathos UG

Coverage decreased (-0.02%) to 93.438% when pulling b6eee76036d4e44f89f029aa3d45b6842598e69b on PascalSchumacher:recursiveAndReflectiveEqualsBuilder into eb2b89efbe15ab0b70fd94f0ecd0aa03866fb4d2 on apache:master.

I haven't locked at this in-depth yet, but seems like a useful enhancement.
I would remove the support for including transient fields, because that seems like a bad practice.

Coverage decreased (-0.04%) to 93.442% when pulling f4cf194decebba1b6aad156b8a0d146195708676 on PascalSchumacher:recursiveAndReflectiveEqualsBuilder into 89afbb0c3eaa2d074dc1ef46d09675f24da0e120 on apache:master.

Coverage decreased (-0.007%) to 93.555% when pulling 2d6ce6ad6a1575c13da903e233bf21a0421f5352 on PascalSchumacher:recursiveAndReflectiveEqualsBuilder into 05647d46e9ac2bf674b320e8467616aa72954f3e on apache:master.

Coverage decreased (-0.02%) to 93.542% when pulling b8c733670847731c4108aecc6ae4b433e042fbcb on PascalSchumacher:recursiveAndReflectiveEqualsBuilder into 05647d46e9ac2bf674b320e8467616aa72954f3e on apache:master.

Coverage decreased (-0.02%) to 93.542% when pulling 015fb1125ba10c2894fb858a21eac3ca4c625b9b on PascalSchumacher:recursiveAndReflectiveEqualsBuilder into 05647d46e9ac2bf674b320e8467616aa72954f3e on apache:master.

I start to use your feature and I discover a bug.
String are now compared with internal data (hash and char[]). But sometimes hash value is not initialized, this is probably a cache for the hashcode. So the comparison fails even if the char[] content is the same.

@douddle I created https://issues.apache.org/jira/browse/LANG-1356 to track the bypass issue.


major: upgrade to fastify v5

I am waiting for v5 before merging it.
Closes #66

@Eomm I'm not sure if this is the proper place for this, but Fastify v5 just shipped. Right now this is breaking the build, and I doubt I'm the only one.
Thanks in advance!

Too fast @darinrogers , too fast 😄


Select zod version

It would be useful to add a version picker to the Zod Playground, allowing users to seamlessly switch between different versions of the Zod library. This would enable users to test and learn about the various features and changes introduced in each version, making the playground an even more effective learning and testing tool.

This is interesting but without a server, or an online service that allows lazy loading of NPM modules, it's not feasible IMHO due to the app bundle size that would become huge.
Do you have any idea here?

WIP https://github.com/marilari88/zod-playground/pull/38


clusterName not honoring global override

What happened:
Deployed the splunk-connect-for-kubernetes chart configured with a custom global.kubernetes.clusterName and events showed up with the default cluster_name value
What you expected to happen:
Events should use my overridden clusterName
How to reproduce it (as minimally and precisely as possible):
global:
  splunk:
    hec:
      protocol: https
      host: foo.bar
      token: very_secret

  kubernetes:
    clusterName: my_custom_name

Anything else we need to know?:
Fix is to set kubernetes.clusterName to null in each dependency chart as then the template logic will correctly apply, e.g.:
    <filter kube.**>
      @type jq_transformer
      jq '.record.cluster_name = "{{ or .Values.kubernetes.clusterName .Values.global.kubernetes.clusterName }}" | .record'
    </filter>

This is trivial but I can submit a PR if you want.


Fix a bug in sample code.

The original code sample was wrong and would throw run-time error. That's because keeper, as an instance property, is not in the prototype of the instance's constructor, so I changed keeper to be a getter of Animal/Bee.

Fixes #

The findKeeper seems like a convoluted way to show generics. i would change the function instead to take an instance instead of the class. i.e. findKeeper(new Lion())

Sorry for the late response. A new commit has been made.

hmm.. looks like something went wrong there.. github is not happy :(


Unify the for syntax

For short single commands, prefer a one-liner for with the zsh syntax:
for x (foo bar) print ${x}

Otherwise just place ; do on the same line as the POSIX for ... in:
for x in foo bar; do
  print ${x}
done

Also, I've already updated the Wiki accordingly.
@Eriner, do you agree with this?


Allow to add spinner without bar

Hey,
I've noticed you can't actually add the spinner when bar is not included, so I provided a hotfix.

I confirm your fix work.
Until this is merged, you can load a dummy loading bar instead:
    cfpLoadingBarProvider.includeBar = true;
    cfpLoadingBarProvider.loadingBarTemplate = "

";

The spinner starts to work.


Cache not requesed (not found) between workflows

Hi!
I have a shared (called) workflow, which I am reusing between few others. As an input param it receive a sha for the commit code. This sha is used as a key.
As result, when I run sequently those workflows with same sha

for the first one cache obviously is empty, so caching happened
for the second workflow cache should be present, but it missed

Here is my called workflow
name: Prepare code

on:
  workflow_call:
    inputs:
      commit_sha:
        required: true
        type: string

jobs:
  main:
    runs-on: ubuntu-latest

    steps:

      - name: 📦 Activate cache for last commited code
        id: cached-commit
        uses: actions/cache@v3
        with:
          path: ./
          key: ${{ inputs.commit_sha }}

      - name: ⏬ Checkout repo last commit
        if: steps.cached-commit.outputs.cache-hit != 'true'
        uses: actions/checkout@v2
        with:
          ref: ${{ inputs.commit_sha }}

And here is how I call it from 2 other workflows:
jobs:

  Prepare_code:
    uses: ./.github/workflows/prepare_code.yaml
    with:
      commit_sha: ${{ github.event.pull_request.head.sha }}

jobs:

  Prepare_code:
    uses: ./.github/workflows/prepare_code.yaml
    with:
      commit_sha: ${{ github.sha }}

Am I doing something wrong?
Thanks 🙏🏼

Hi 👋🏽
Caches are scoped to the branch on which they are created (more details). Are both the runs you mentioned on the same branch?
Would you be comfortable replicating the issue on a public repo and share the workflows here so that we can take a look at the logs?

Hi  Aparna,
thanks for answering.
Yes, will provide a separate repo today/tomorrow and ping you here 😉

Okay, looks like I found the problem:
one trigger happens when pull_request receive a new commit (PR from branch A to dev)
the other one happens when push done to branch (push commit to branch A)
But probably pull_request event is not on the same scope, even the branch that will be merged is the same.
Is it right?

That sounds correct. You can list the caches in your repository (using this api or this gh-cli extension) to see if there are two caches with same key belonging to different scopes to confirm this.
Here, the order in which caches are created matters. The caches created in the scope of the main branch will be available to the pull_request scope. But if the cache is created first on the pull request scope, it will not be available to the main branch. Therefore, a new cache will be created.

Thanks.
I used CLI extension and indeed, I have 2 different cache...
Is there any way to reuse it?

Unfortunately, no. Caches created on the default branch can be reused in other branches. But no other reusability is possible.

Got it 😉
Thank you for your help!
Have a nice day!


Fix stubbed rollup API

As this API is not documented, quite complex, and experimental, we have agreed to treat the request and response body as UserDefinedValue for now. I'm awaiting some feedback from the Elasticsearch engineers to understand if this API is valid still.

@delvedor I did do before submitting. Have run again and it's not modified any files?

Try to merge main in your working branch.


Add Invoke-History support for invoking a collection of commands

PR Summary
Resolves #12488.
Adds support for invoking a collection of command IDs while using Invoke-History.
PR Context
When there are a series of commands in history, it is often useful to execute several of them at once in series. Currently, the -Id parameter on Invoke-History is a string. This PR changes it to a string[] and adds a loop to process a collection of IDs instead of a single ID. This also adds some consistency with Get-History whose -Id parameter does support collections of IDs.
Previously:

After this PR:

There are a variety of other ways to implement this kind of change, but I thought it was best to perform the same checking and validation per ID so that if some IDs were valid but others weren't, the valid IDs up until the invalid ID still run.
Example:

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[x] Issue filed: MicrosoftDocs/PowerShell-Docs/issues/6250

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

Remove the review-needed label as the PR is WIP.

Is there any recommendation on what kind of doc update is required for this PR? Should there be a new help example, something in docs.microsoft.com, etc? Or is it all mostly auto-generated?

I think some of it will be autogenerated but at the least an issue will need to be filed in the docs repo to ensure that the parameter is added with a description etc., alongside any necessary examples.

I think some of it will be autogenerated but at the least an issue will need to be filed in the docs repo to ensure that the parameter is added with a description etc., alongside any necessary examples.

Does that issue with the docs repo need to be resolved before this can merge? I'm just trying to figure out what I have to do in order to finish this PR.

Nope, as long as there's a tracking issue for any required docs the code is generally OK to be merged in.

I messed it up trying to satisfy CodeFactor. What's the right way to organize the code it's complaining about? (L1093 - L1115)

@PoshChan please retry macos

@vexx32, did not find matching build context: PowerShell-CI-macOS; allowed contexts: PowerShell-CI-SSH

/azp run

@iSazonov and @vexx32 can you please continue with the review when you get time? Thanks!

Negotiating this merge conflict is going to take some doing.

@daxian-dbw The PR is ready for your review and merge.

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 10 days of this comment.

Heh, I'm still here! It's just been tough to prioritize this with so much on my plate at the moment. Sorry for the delay.

I messed up my local git history, and now my fork's history. I'll re-open this later with a clean history. Apologies.


Check HRV parameters

Check the calculated parameters shown in the MeasuredValueFragment and calculated in the presenter.
There are some with no unit (maybe no error) and sd1sd2 is duplicated but with different values (probably an error)

Many parameters don't have a Unit.
Duplication of SD1SD2 is now fixed in HRV-Lib and hopefully soon in hrv-band...


Add support for CUDA occupancy calculation functions.

I think this looks good in general - there are just a couple of other things to note as well:

As you provide the option to call the occupancy calculator functions with flags, could you add the values of CU_OCCUPANCY_DEFAULT and CU_OCCUPANCY_DISABLE_CACHING_OVERRIDE to numba.cuda.cudadrv.enums please? These values will be available somewhere in the CUDA toolkit headers (which are not distributed in the conda cudatoolkit package, but will be found in an install of NVidia's CUDA toolkit).
Just a nit, but there are some overly-long lines, some of which don't fit inside the diff viewer. Generally in Numba, lines are kept to 80 characters - could you wrap the long lines to make it consistent please?

BTW, I merged this before Graham made his comments (oops! I pulled the trigger a little too quickly), so his suggestions will need to be addressed in a new PR.


Hiding Poster Image

I am facing a problem in this plugin Whenever I change the resolution the poster image is displayed for some time. Is it possible to hide poster Image whenever I change resolution?
I tried to hide poster image in player initialization code in resolution click handler method.
// Change the source and make sure we don't start the video over
player.src( player.availableRes[this.resolution] ).one( 'loadedmetadata', function() {

	player.currentTime( current_time );

	player.posterImage.hide();

	if ( ! is_paused ) { player.play(); }
});

Any progress on this issue?

No new progress at this time. I'm open to suggestions on how to improve the UX during a resolution change.

@dominic-p I was facing this problem and the only solution I came to was to put a canvas on top of the video. So before source changing I draw current frame to this canvas, then change source, wait for loading and then hide this canvas. So it looks like on youtube. Seems like youtube is a good example of UX.

That's an interesting solution. Thanks for sharing it. Maybe you could post a jsFiddle or something like that so we could see it in action? I'm curious to see how you implemented the CSS for the canvas element.

@dominic-p currently I have stopped working on this feature, but I could try to make a fiddle this weekend. The css was very simple - absolute positioning and z-index: 1 to place it on top of the video. But there is a problem with fullscreen mode, because we need to calculate where to place canvas in this case. Otherwise it will be stretched to whole viewport.
Right now I can post a diff:
diff --git a/static/source/libs/videojs/plugins/video-quality-selector.js b/static/source/libs/videojs/plugins/video-quality-selector.js
index c64d235..1f3a058 100644
--- a/static/source/libs/videojs/plugins/video-quality-selector.js
+++ b/static/source/libs/videojs/plugins/video-quality-selector.js
@@ -198,6 +198,8 @@
 			available_res = { length : 0 },
 			current_res,
 			resolutionSelector,
+			canvas = document.createElement("canvas"),
+			ctx = canvas.getContext("2d"),

 			// Split default resolutions if set and valid, otherwise default to an empty array
 			default_resolutions = ( settings.default_res && typeof settings.default_res == 'string' ) ? settings.default_res.split( ',' ) : [];
@@ -325,20 +327,45 @@
 				|| ! player.availableRes
 				|| ! player.availableRes[target_resolution] ) { return; }

+			player.pause();
+
 			// Make sure the loadedmetadata event will fire
 			if ( 'none' == video_el.preload ) { video_el.preload = 'metadata'; }
+
+			// Hide bigPlayButton
+			player.bigPlayButton.hide();
+
+			canvas.width = player.width();
+			canvas.height = player.height();
+
+			ctx.width = canvas.width;
+			ctx.height = canvas.height;
+
+			ctx.drawImage(video_el, 0, 0, canvas.width, canvas.height);
+			canvas.style.zIndex = 1;
+			canvas.style.position = "absolute";
+			player.el().insertBefore(canvas, player.el().querySelector(".vjs-poster"));
+
+			// player.el().querySelector(".vjs-poster").style.backgroundImage = "url(" + canvas.toDataURL("image/jpeg") + ")";
+

 			// Change the source and make sure we don't start the video over
-			player.src( player.availableRes[target_resolution] ).one( 'loadedmetadata', function() {
+			player.src( player.availableRes[target_resolution] )
+			.one( 'loadedmetadata', function() {

 				player.currentTime( current_time );

+				if ( ! is_paused ) { player.play(); }
 				// If the video was paused, don't show the poster image again
 				player.addClass( 'vjs-has-started' );

-				if ( ! is_paused ) { player.play(); }
+				// Update the classes to reflect the currently selected resolution
+				player.trigger( 'changeRes' );
 			});

+			player.one( 'loadeddata', function() {
+				canvas.remove();
+			});
 			// Save the newly selected resolution in our player options property
 			player.currentRes = target_resolution;

@@ -361,8 +388,7 @@
 				}
 			}

-			// Update the classes to reflect the currently selected resolution
-			player.trigger( 'changeRes' );
+
 		};

 		/*******************************************************************


Fix list passed from toml settings

Resolves #923
When toml is used for config, and especially with the seed-isort-config pre-commit function, the setting gets set to a list of strings, rather than a comma-separated string. This checks that the value passed is already a list and if so strips each item to be safe, and returns that.
This does not update the type-hints in the signature, as the function isn't really supposed to expect a string, but it's just a safeguard in case.

Codecov Report

Merging #926 into develop will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #926      +/-   ##
===========================================
- Coverage     89.1%   89.03%   -0.08%
===========================================
  Files            8        8
  Lines         2415     2417       +2
  Branches       431      433       +2
===========================================
  Hits          2152     2152
- Misses         177      178       +1
- Partials        86       87       +1

Looks great, thank you!


MODOAIPMH-302: Response encoding does not use UTF-8 represenation of Unicode

Jira: https://issues.folio.org/browse/MODOAIPMH-302
PURPOSE
Fix encoding to display the Cyrillic symbols in the correct manner.

I don't see any problems with the code changes here, but it's not clear exactly what the problem was, or how it was solved.  I'm deducing that this has something to do with the version of mar4j we were using, and that by upgrading that dependency and providing those two additional arguments to the MarcXmlWriter.writeSingleRecord(...) call solves the problem.  Since there aren't any details in the JIRA, can you please provide some details in the PR description?

I don't see any problems with the code changes here, but it's not clear exactly what the root cause was, or how it was solved. I'm deducing that this has something to do with the version of mar4j we were using, and that by upgrading that dependency and providing those two additional arguments to the MarcXmlWriter.writeSingleRecord(...) call solves the problem. Since there aren't any details in the JIRA, can you please provide some details in the PR description?

Thank you for adding context/explanation to the description @TotoroDevEbsco


vdirsyncer metasync fails copying displayname to local store

Describe the bug
Running vdirsyncer metasync fails with the following error (once for each of the calendars being synced):
Copying displayname to xandikos_calendar_local/Travel
error: Unknown error occurred for xandikos_calendar/Travel: [Errno 2] No such file or directory: '~/.local/share/vdirsyncer/calendars/Travel/tmphakdn261'
error: Use `-vdebug` to see the full traceback.

To Reproduce
Steps to reproduce the behavior:

Setup vdirsyncer and point it at a CalDAV server that allows metadata sync.
Run vdirsycer metasync

Expected behavior
vdirsyncer metasync should not fail
Additional context
Output when using vdirsyncer -vdebug metasync
Metasyncing xandikos_calendar/Travel
debug: Key: displayname
debug: A:
debug: B: Travel
debug: S:
Copying displayname to xandikos_calendar_local/Travel
error: Unknown error occurred for xandikos_calendar/Travel: [Errno 2] No such file or directory: '~/.local/share/vdirsyncer/calendars/Travel/tmpxf8x6l1p'
error: Use `-vdebug` to see the full traceback.
debug:   File "/nix/store/y302fxqgprk0380j1jlwzcavzbzqs49p-python3.8-vdirsyncer-unstable-2018-08-05/lib/python3.8/site-packages/vdirsyncer/cli/tasks.py", line 134, in metasync_collection
debug:     metasync(
debug:   File "/nix/store/y302fxqgprk0380j1jlwzcavzbzqs49p-python3.8-vdirsyncer-unstable-2018-08-05/lib/python3.8/site-packages/vdirsyncer/metasync.py", line 58, in metasync
debug:     _b_to_a()
debug:   File "/nix/store/y302fxqgprk0380j1jlwzcavzbzqs49p-python3.8-vdirsyncer-unstable-2018-08-05/lib/python3.8/site-packages/vdirsyncer/metasync.py", line 25, in _b_to_a
debug:     storage_a.set_meta(key, b)
debug:   File "/nix/store/y302fxqgprk0380j1jlwzcavzbzqs49p-python3.8-vdirsyncer-unstable-2018-08-05/lib/python3.8/site-packages/vdirsyncer/storage/filesystem.py", line 69, in set_meta
debug:     with atomic_write(fpath, mode='wb', overwrite=True) as f:
debug:   File "/nix/store/z65l1jqvxa58zzwwa3bvglb6asj4y8cv-python3-3.8.5/lib/python3.8/contextlib.py", line 113, in __enter__
debug:     return next(self.gen)
debug:   File "/nix/store/8yd0h8z3x6dpq47lxj0wqm41acxyn29h-python3.8-atomicwrites-1.4.0/lib/python3.8/site-packages/atomicwrites/__init__.py", line 166, in _open
debug:     with get_fileobject(**self._open_kwargs) as f:
debug:   File "/nix/store/8yd0h8z3x6dpq47lxj0wqm41acxyn29h-python3.8-atomicwrites-1.4.0/lib/python3.8/site-packages/atomicwrites/__init__.py", line 183, in get_fileobject
debug:     descriptor, name = tempfile.mkstemp(suffix=suffix, prefix=prefix,
debug:   File "/nix/store/z65l1jqvxa58zzwwa3bvglb6asj4y8cv-python3-3.8.5/lib/python3.8/tempfile.py", line 332, in mkstemp
debug:     return _mkstemp_inner(dir, prefix, suffix, flags, output_type)
debug:   File "/nix/store/z65l1jqvxa58zzwwa3bvglb6asj4y8cv-python3-3.8.5/lib/python3.8/tempfile.py", line 250, in _mkstemp_inner
debug:     fd = _os.open(file, flags, 0o600)

Notify maintainers

@loewenheim
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.64, NixOS, 21.03pre244416.daaa0e33505 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.7`
 - channels(root): `"nixos-21.03pre244416.daaa0e33505"`
 - channels(sumner): `"nixos-21.03pre244416.daaa0e33505, nixpkgs"`
 - nixpkgs: `/home/sumner/.nix-defexpr/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
  - vdirsyncer
# a list of nixos modules affected by the problem
module:

Just from looking at the logs: Are you sure this is a NixOS specific? Looks to me like a problem that upstream could provide a solution for.

The reason I believe this is an NixOS bug is that I'm using vdirsyncer metasync successfully on an Arch Linux machine, but it does not work in NixOS. It seems like the vdirsyncer packages for NixOS are in a weird state right now, and I can't really tell if it's actually up-to-date. Maybe we should just unify both
https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/vdirsyncer/stable.nix
and
https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/vdirsyncer/default.nix
into a single vdirsyncer/default.nix and update it to the latest version?

Closing this because it seems to be working, now. Not sure what's changed, but I haven't had issues with this in a while now.


Add missing native reflection config

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #40461
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

It seems only the function mock cases need this reflection configuration. Can't we add this config only when function mocking exists?

It seems only the function mock cases need this reflection configuration. Can't we add this config only when function mocking exists?

we may need to verify that... It takes time... can we go with this for the release as this part was there and worked earlier?? we can check and improve it later...

It seems only the function mock cases need this reflection configuration. Can't we add this config only when function mocking exists?

we may need to verify that... It takes time... can we go with this for the release as this part was there and worked earlier?? we can check and improve it later...

Please create an issue to track this.


Allow drop of JoinHandle in web

By making the send a panic on fail
https://github.com/fzyzcjy/flutter_rust_bridge/blob/acf0d3248b2cd1edc2514d17a03e982f0fa6df78/frb_rust/src/rust_async/web.rs#L44
it prevents a scenario where you aren't concerned with the output or the join handle, and is inconsistent with the non-web behavior.
In my case I'm spawning the local to process some messages
#[flutter_rust_bridge::frb]
pub async fn start_processing()->Ctx{
  let (tx, mut rx) = tokio::sync::mpsc::unbounded_channel::<String>();
  flutter_rust_bridge::spawn(async move {
      loop {
          let Some(msg) = rx.recv().await else {
              log::info!("Exiting");
              return;
          };
          log::info!("process msg");
          process(msg).await;
      }
  });
  Ctx{tx}
}

Everything works as expected.
I eventually drop the returned Ctx and am presented with a panic when the spawn returns and attempts to send the output on the channel.
Since I believe the only failure for sending the output would be when the user drops the join handle it would seem reasonable to ignore that error and not panic.
I'd be happy to submit a PR if that is acceptable.

If we align the web behavior to be consistent with non-web behavior, then I guess that change would be quite welcome!


Questions about how to enlarge the base vision tower input resolution

Currently, there is an option to using two image grid to double the input, but this introduce sort of heavy compute.
I want just make the input resolution slightly large, say from 336 -> 448, while keep the Convnext input resolution same (although I think currenctly it should larger if base vision tower are larger).
Could that be possible? Can u give me some advisor how to adopt it?

Hi, of course, it is possible. The main question is how to split the input image. A simple solution is to divide 448x448 to 4x336x336 images with overlap. However, the computational cost is actually the same as that of 672x672.
In this case, I guess you can try to use 4x224x224 to represent 448, which brings low cost but slightly larger resolution.

Hi, how about directly make input 448, and then interpolate for clipvit, that's would add more visual tokens, is that possible and will enhance performance?


[Translatable] Allow to save an empty string translation

Fixes Atlantic18/DoctrineExtensions#788

well, this is an ugly if statement already..


Review blanket processing rules in current process

Deliverable:

Review all the blanket processing rules that MVPS currently applies.  These would be on the first two tabs of the "NND Requirements Implementation Matrix" spreadsheet attached here.

https://app.zenhub.com/files/510836864/de2a7ee7-2697-4327-abc5-21ecf3062181/download

Compare these business rules to the validation rules/errors/warnings thrown by Structural validator and MMG validator.  Identify the gaps/deltas where we don't have coverage and think/plan on how to integrate that in.

*We should also note that the error/warning messages outlined in this spreadsheet could be used to update the error/warning messages for the structural validator and MMG validator.  <Meaning: re-use the verbiage>  This work should be put on a new issue.

Per our meeting with Erik Knudsen on 12-16-22, we have determined the following:

A separate Blanket Processor service is not needed.
There are some blanket processing rules that MVPS has implemented that we will not support.
There are some blanket processing rules that MVPS has implemented that we have not yet implemented, but will, most of which will be incorporated into the MMG Validator.
The attached spreadsheet details the blanket processing rules and indicates which of the above categories applies to each rule.
NND Requirements Implementation Matrix - 2022-12-16 DEX.xlsx


Update paragraphs 3 and 4 in 'Callbacks and Eager' Options section

fixes #2113

Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[x] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Duplicate of #2114


[8.x] Fix issue with polymorphic morphMaps with literal 0

Background
Ran into project where
Relation::morphMap([
 0 => Model::class
 1 => Model2::class
]);

During a legacy upgrade I noticed failures after completing the 5.8.x upgrade. At first I tracked it to this - https://github.com/laravel/framework/issues/27175, which then resulted in changing my morphTo parameters to null, null, null, 'id' so ownerKey would be set properly.
It still failed. It appeared that getting a 0 zero value back from entity_type (entity morph) was passing the empty check in the wrong way no longer resulting in getting the morphed instance back.
The comment above the code change even says "if value is null", so probably just a mistake.
Test
Never wrote a test for the DB area before. Hope it works out. Open for feedback.

@iBotPeaches @taylorotwell Thank you for this fix. Could you tell if this change will also be added to 6.x version?
Or, what can I do to get this also fixed in 6.x?

@dmytro-laba I've sent this to 6.x here: https://github.com/laravel/framework/pull/35487. Thanks

@driesvints Thanks a lot!


Revise TdcTestResultWatcher

Why?
On Fri, 6 Jul 2018, 19:59 Stefan Birkner<EMAIL_ADDRESS>wrote:

Merged #2 https://github.com/saucelabs/rdc-appium-junit4/pull/2.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/saucelabs/rdc-appium-junit4/pull/2#event-1720801853,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ9R2vIfalL1gkwCRRhnWVMrNFaWhEvPks5uD6VsgaJpZM4VFCSK
.

I saw that the button was green and thought that you already approved. I
can revert if you like.
2018-07-06 20:29 GMT+02:00 Abdulla Hawara<EMAIL_ADDRESS>
Why?
On Fri, 6 Jul 2018, 19:59 Stefan Birkner<EMAIL_ADDRESS>wrote:

Merged #2 https://github.com/saucelabs/rdc-appium-junit4/pull/2.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/saucelabs/rdc-appium-junit4/pull/2#event-1720801853
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AQ9R2vIfalL1gkwCRRhnWVMrNFaWhEvPks5uD6VsgaJpZM4VFCSK>
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/saucelabs/rdc-appium-junit4/pull/2#issuecomment-403113024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAratVjOvqHp2ZyjujqYeJ-pbPaiIOQuks5uD6yZgaJpZM4VFCSK
.


MAID-2330: deletes file entries when deleting service

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Code-for-the-Netherlands vs Code-for-NL

Code-for-the-Netherlands (as chapter id) has been replaced by Code-For-NL by @jgroenen. And now both show up in http://api.codeforamerica.org/api/organizations.geojson
Please advice if the two should be merged first or if one of them needs to be deleted from the database and if so, which one.

Hi @milovanderlinden, my course of action to fix this will be to delete "Code-for-the-Netherlands", as I assumed @jgroenen was from there and was somewhat authoritative on its name.
Sorry if this is causing any bugs on your end -- if I may ask, what is your usage of the CfAPI? As its current maintainer, knowing how people are using it will help me to not break your use-cases going forward.

Hi @tdooner I can work with your fix. please let me know when it is implemented, so I can alter my code to match.
I am working on the codeforall website. Work in progress here: http://www.codefor.nl/codeforall.org/ it uses the cfapi to pull in organization and projects information for the code for all chapters and projects. It then mixes the information with standard pages and posts. Currently I am pulling the json files with a small batch script because of the ssl certificate issue, but in a later stage I can always revert this to have a live connection.

Cool, thanks. I deleted the extra record yesterday. Now I'm seeing only one record form the Netherlands. Let me know if you need anything else!


Tutorial 7 single head GAT layer produces same output for nodes with identical neighbours

Tutorial: 7 (pytorch)
Describe the bug
The GAT layer is introduced as superior to plain graph convolutions, because it does not treat notes with identical neighbours identically by default, but when I set num_heads=1 in the provided short example, that is not the case.
To me, this does not make sense, but I also currently do not understand why it is happening.
To Reproduce (if any steps necessary)
Run the GAT example with num_heads=1:
layer = GATLayer(2, 2, num_heads=1)
layer.projection.weight.data = torch.Tensor([[1., 0.], [0., 1.]])
layer.projection.bias.data = torch.Tensor([0., 0.])
layer.a.data = torch.Tensor([[-0.2, 0.3], [0.1, -0.1]])

with torch.no_grad():
    out_feats = layer(node_feats, adj_matrix, print_attn_probs=True)

print("Adjacency matrix", adj_matrix)
print("Input features", node_feats)
print("Output features", out_feats)

Expected behavior
Different output features for the last two nodes.
Runtime environment (please complete the following information):

Local computer
Run on GPU

Hi, when you set the number of heads to 1, you need to adjust the weight, bias and a-vector accordingly. At the moment, the code above cannot run since there is a mismatch of shapes. A version that would run is the following:
layer = GATLayer(2, 2, num_heads=1)
layer.projection.weight.data = torch.Tensor([[1., 0.]])
layer.projection.bias.data = torch.Tensor([0.])
layer.a.data = torch.Tensor([[-0.2, 0.3]])

with torch.no_grad():
    out_feats = layer(node_feats, adj_matrix, print_attn_probs=True)

print("Adjacency matrix", adj_matrix)
print("Input features", node_feats)
print("Output features", out_feats)

This gives the output:
Attention probs
 tensor([[[[0.3543, 0.6457, 0.0000, 0.0000],
          [0.1096, 0.1450, 0.2642, 0.4813],
          [0.0000, 0.1858, 0.2885, 0.5257],
          [0.0000, 0.2391, 0.2696, 0.4913]]]])
Adjacency matrix tensor([[[1., 1., 0., 0.],
         [1., 1., 1., 1.],
         [0., 1., 1., 1.],
         [0., 1., 1., 1.]]])
Input features tensor([[[0., 1.],
         [2., 3.],
         [4., 5.],
         [6., 7.]]])
Output features tensor([[[1.2913],
         [4.2344],
         [4.6798],
         [4.5043]]])

The output features for all nodes are different.

Sorry, I failed to copy paste my actual code.
I assumed the projection weight matrix shape would still be (2,2) and the shape of a would be (4,):
layer = GATLayer(2, 2, num_heads=1)
layer.projection.weight.data = torch.eye(2)
layer.projection.bias.data = torch.zeros(2)
layer.a.data = torch.tensor([[-0.2, 0.3, 0.1, -0.1]])

with torch.no_grad():
    out_feats = layer(node_feats, adj_matrix, print_attn_probs=True)

print("Adjacency matrix", adj_matrix)
print("Input features", node_feats)
print("Output features", out_feats)

Which results in
Attention probs
 tensor([[[[0.5000, 0.5000, 0.0000, 0.0000],
          [0.2500, 0.2500, 0.2500, 0.2500],
          [0.0000, 0.3333, 0.3333, 0.3333],
          [0.0000, 0.3333, 0.3333, 0.3333]]]])
Adjacency matrix tensor([[[1., 1., 0., 0.],
         [1., 1., 1., 1.],
         [0., 1., 1., 1.],
         [0., 1., 1., 1.]]])
Input features tensor([[[0., 1.],
         [2., 3.],
         [4., 5.],
         [6., 7.]]])
Output features tensor([[[1.0000, 2.0000],
         [3.0000, 4.0000],
         [4.0000, 5.0000],
         [4.0000, 5.0000]]])

I wasn't aware that a nn.Linear(2,2) can even have a weight matrix with shape (1,2).
While I still dont understand this, its definitely not a bug, so this issue can be closed.
Thanks for the quick answer!

One remaining problem I see is that the default initialization of projection and a would not have the shapes you described. So without initializing manually, the following happens:
gat = GAT(2, 2)
with torch.no_grad():
    out_features = gat(node_features, adjacency_matrix)

print(f"Node features:\n{node_features}")
print(f"Adjacency matrix:\n{adjacency_matrix}")
print(f"GAT Output\n{out_features}")

→
Node features:
tensor([[[0., 1.],
         [2., 3.],
         [4., 5.],
         [6., 7.]]])
Adjacency matrix:
tensor([[[1., 1., 0., 0.],
         [1., 1., 1., 1.],
         [0., 1., 1., 1.],
         [0., 1., 1., 1.]]])
GAT Output
tensor([[[ 0.5140, -1.2451],
         [ 2.1499, -1.8266],
         [ 2.7505, -2.0401],
         [ 2.7503, -2.0401]]])

So you run into the same problem as I did in the beginning.

Hi, sorry I mixed up the shapes there indeed, should have been with c_out=1. But I see now the problem you run into. This is only because of the specific value combination of w, b, a and the node features in this case: the 0.1, -0.1 in a leads to all nodes having the same impact, hence becoming a uniform distribution for all neighbors. If you change it to 0.1, -0.2, you will see again all values change. In a classical training, it is very unlikely to hit a combination of w, b, a and the node features like we have initialized them here and wouldn't be a problem anyway, since a single gradient step will take you away from it.

Very interesting that torch does not mind changing the shape of the weight matrix...
I've tried random node_feats and GATLayer parameters and fairly often the resulting features for the last two nodes are still the "same" for the first four decimal places, but not always:
for x in range(10):
    node_feats = torch.rand((1,4,2))
    layer = GATLayer(2, 2, num_heads=1)

    with torch.no_grad():
        out_feats = layer(node_feats, adj_matrix, print_attn_probs=True)

    print(f"Output features\n{out_feats[:, -2:]}")

Output features
tensor([[[-0.4876,  0.1265],
         [-0.4876,  0.1265]]])
Output features
tensor([[[-0.8985, -0.4233],
         [-0.8380, -0.3486]]])
Output features
tensor([[[-0.4323, -0.1635],
         [-0.4323, -0.1635]]])
Output features
tensor([[[ 0.1025, -0.3395],
         [ 0.1025, -0.3395]]])
Output features
tensor([[[ 0.8478, -0.7233],
         [ 0.8478, -0.7233]]])
Output features
tensor([[[0.6933, 0.9880],
         [0.6939, 0.9911]]])
Output features
tensor([[[1.0354, 1.2086],
         [1.0354, 1.2086]]])
Output features
tensor([[[-0.6915,  0.1529],
         [-0.7278,  0.1536]]])
Output features
tensor([[[-0.5950, -0.1346],
         [-0.6858, -0.2273]]])
Output features
tensor([[[-0.6591, -0.7194],
         [-0.6591, -0.7194]]])

It's probably just very likely to primarily "attend" to the same neighbours for nodes 3 and 4 and softmax just causes the results to be similar enough to be indistinguishable in 4 decimals.
Thank you for looking into this!

no bugs here


Add merge-master Workflow

Codecov Report

Merging #10 (a110db3) into master (287fd82) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   89.92%   89.92%
=======================================
  Files           6        6
  Lines         645      645
=======================================
  Hits          580      580
  Misses         65       65

Continue to review full report at Codecov.

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


Ics

Brings in the ICS feature

Sets route using IOCTL calls
Edits resolv.conf
Stores user config inside .local/share/beagle-config/beagle.conf

Sorry, I didn't wanted to merge to master. Please close this PR.


blocking is bad

The AIDL calls should probably be oneway as well right? But I don't think we need that now, if others agree, I'll file a bug

The AIDL calls should probably be oneway as well right? But I don't think we need that now, if others agree, I'll file a bug

ack


PixiJS 4.5.0 mouse events wont trigger in Google Chrome using Wacom tablet

My Wacom Bamboo & Wacom Intuos Pro (up to date drivers) won't trigger mouseover / mouseout events in Chrome (v57) in the following example:
https://pixijs.github.io/examples/#/demos/interactivity.js
This interaction is working in Firefox and Safari.
The weird part is, when I use a mouse or touchpad on laptop the mouseover / mouseout the events do trigger.
Is there a way to fix this?
PixiJS 4.3.0 is working.

Related issue: https://github.com/pixijs/pixi.js/issues/3695#issuecomment-288448959

Going to close this as a duplicate since it seems answered in a different thread.

The issue isn't resolved in: https://pixijs.github.io/examples/#/demos/interactivity.js

@andrewstart Can you take a look at this?

Sorry to inform that the issue is still present. Let me know if you need more information about the issue.

Still present using the latest dev?

yes, the issue is still present in the latest dev.
a. when I roll over a button with a wacom pen, the rollover state isn't triggered.
b. when I do the same with the touchpad, it works.
c. after using the touchpad, the rollover also works with the wacom pen. (which is pretty weird)

Would you mind collecting some data for me? Using a pointerdown event, I'm curious what the pointerType and pointerId properties are on the original event, in each of those 3 situations.

@Qubica Sorry to bug you again, but would it be possible to have another look at this please using latest?

This reproduces for me with the current build.  Wacom tablets aren't really mainstream though I guess, is this uninteresting to address I guess?

What browser? Could you put up a demonstration somewhere of the simplest code that doesn't work?


MEITNER Q4 Milestone report

The report ready for review.
Needs @mitchellwhalen review as well.

For review of documents this size, please just merge, then create an issue (with the built pdf) for each review that is desired.


Popup Menu doesn't work properly in Adapter

OS: Naugate and Oreo
I am working on a project where I need to use TextView in Listview with verious String types like URL, Phone Number, Email etc. I have implement your lib and it works in activities but in the listview, I am facing some problem.
Assume, I have a dynamic listview with String content. At first, the popup menu doesn't show from the linkify. If I scroll up/down, it works again. Need to add that in Lolipop, it works like a charm.
Is there any solution regarding this.
Here is the sample of my XML and Java Code -
 <TextView
        android:id="@+id/body"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:autoLink="all"
        android:focusable="true"
        android:linksClickable="false"
        android:longClickable="true"
        android:textColorHighlight="@color/gray"
        android:textColorLink="@color/white"
        android:textSize="@dimen/font_size_small"
        tools:text="@tools:sample/full_names" />

--
in list adaper -
public View getView(int position, View view, ViewGroup parent) {
...
BetterLinkMovementMethod.linkify(Linkify.ALL, mViewGroup).setOnLinkClickListener(urlClickListener);
}
private BetterLinkMovementMethod.OnLinkClickListener urlClickListener =
new BetterLinkMovementMethod.OnLinkClickListener() {
@Override
public boolean onClick(TextView textView, String url) {
if (isPhoneNumber(url)) {
PhoneLinkPopupMenu phonePopupMenu =
new PhoneLinkPopupMenu(activity, textView, url.replace("tel:", ""));
phonePopupMenu.show();
}
return true;
}
};

I'm sorry, but the Popup menu is not a part of the library. It's something I quickly wrote for showcasing BetterLinkMovementMethod in a sample project. :)

Thanks for your response. :)


Hope to get a copy of the data stored in BaiduYun

Download the data from origin url is too slow, can you share a download url by BaiduYun?

URL: https://pan.baidu.com/s/13Bf-BLWMiMvN5MRQKViwJw  Password: gwrs

执行unzip train_k400.zip报错
Archive:  train_k400.zip
End-of-central-directory signature not found.  Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive.  In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.

Can you try to use tar zxvf train_k400.zip? Since this dataset has been uploaded for a long time, I'm not sure whether they are a .zip file or a .tgz file.

It works, thx and happy new year :)


Composable startup configuration with extra bindings

Proposal
Overview
To provide more flexibility around how WireMock is configured at startup.
Refactor the hierarchy under Options, creating a composite class with hierarchical delegation and a set of new bindings for properties files, environment variables and system properties.
This should support construction of cases like this:
new WireMockServer(CompositeConfiguration
                .add(new CommandLineOptions())
                .add(new EnvironmentConfig())
                .add(wireMockConfig().dynamicPort())
                .build()
        );

Where the CLI options will take the highest precedence and the programmatically constructed config the lowest.
It should also be possible to provide a default configuration via a template method in WireMockServerRunner so that different (or end-user custom) builds of WireMock can provide different defaults.
Implementation
Ideally we would create an abstract base class for all of the config binding impls, which would define all of the keys in a similar manner as CommandLineOptions currently does, but sufficiently abstractly that it could be used for env vars, CLI, system props etc.
The main challenge here might be figuring out way to describe, parse and validate these in a way that's not specific to CLI, although it might be possible to wrap the current joptsimple based solution.
References
No response

@dirkbolte as we discussed last week


Changed bufloaded back to buflisted

The last PR (#2522) caused some issues for me where __CtrlSF__ and [No Name] tabs started appearing in the tabline, and the author @KSR-Yasuda suggested that this change could be the culprit.
I also use the Startify plugin with session saving, and found that when restoring a session, the tabs would no longer appear until I had visited a buffer again, whereas beforehand they would load up a soon as I opened Vim. So I commented out my entire vimrc, and could reproduce this issue by opening an existing session with vim -S mysession. Without this fix, only a single buffer from the session is shown in the tabline.
I'm using regular Vim on the command line, version 8.2 on MacOS 12.3.1.

thanks!


PYIC-5875: Test harness for API tests

Proposed changes
What changed
Test harness for API tests
Why did it change
This adds the basics of a new cucumber testing project for running API tests through core. It includes one feature that runs a happy path journey through an app journey.
It includes a GHA workflow to run the tests pre-merge.
Lots of details in the commit messages for anyone interested.
Issue tracking

PYIC-5875

One additional thought - can we add this to dependabot.yml?


Feature/environment update

          1 Message

:book:
What a good PR :white_flower:

Develop PR Check Result

Item
Result

Head Branch check
:o:

Base Branch check
:o:

Merge Commits check
:o:

Modification Volumn check
:o:

  Generated by :no_entry_sign: Danger


enter event fixed

PR for #23

@rultor merge pls

@rultor merge pls

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge pls

@amihaiemil Done! FYI, the full log is here (took me 3min)


[Bug] MSIX - Uninstall from Intune gives error 0x87D103E8, but package gets removed

I don't know if this is an Intune MDM or PowerShell 7 MSIX issue.
I've tried deploying 7rc3 MSIX with Intune, it seems to work well. It did not work previously, so I don't know whether you did something with the MSIX package in latest release or if something changed in Intune.
However, when I tried uninstalling it on a client today, the package got removed from the computer, but Intune reports error:

Unknown error (0x87D103E8)

I will try on myself and see if I can get some logs or other information.

Next time Intune syncs it says application is uninstalled successfully.
The only traces I found was in Event Viewer \ Applications and Services Logs \ Microsoft \ Windows \ AppxDeployment-Server
Heres related events, oldest event first:

[Information] Started deployment Remove operation on a package with main parameter Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe and Options 0 and 0. See http://go.microsoft.com/fwlink/?LinkId=235160 for help diagnosing app deployment issues.
[Verbose] Creating Resiliency File C:\ProgramData\Microsoft\Windows\AppRepository\8eb05c6e-8f85-4c9a-80db-8f4514459b74_S-1-12-1-391392237-1099207582-1637945766-2443643826_1.rslc for Remove Operation on Package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe.
[Information] Deployment Remove operation on package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe has been de-queued and is running for user SID S-1-12-1-391392237-1099207582-<hidden_by_me>
[Information] Finished resolving action lists. DeploymentRequest action lists: removePackageList: Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe.
[Verbose] About to service package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe. Setting the package state to disabled returned with 0x0.
[Verbose] 0x0: AllowUninstall successful for package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe.
[Error] AppExecutionAlias directory missing, error code is 0x8007010B.
[Verbose] Finished servicing package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe. Setting the package state to enabled returned with 0x80070490.
[Information] Deployment Remove operation on Package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe: PostDeploymentOperationCheck cost: 0 ms
[Information] Deployment Remove operation on Package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe: PostDeploymentOperationCheck cost: 0 ms
[Information] Deployment Remove operation on Package Microsoft.PowerShellPreview_<IP_ADDRESS>_x64__8wekyb3d8bbwe: State repository CommitTransaction cost: 78 ms

/cc @SteveL-MSFT for information.


[SUPPORT] Hudi only support one table one HoodieJavaWriteClient

Describe the problem you faced
The tableName needs to be specified when HoodieJavaWriteClient is created, so that each table needs to create a corresponding HoodieJavaWriteClient. When there are a lot of tables in the database that need to be updated, a lot of HoodieJavaWriteClient needs to be created
Steps to reproduce the behavior:
  HoodieWriteConfig cfg = HoodieWriteConfig.newBuilder().withPath(tablePath)         .withSchema(PERSON_SCHEMA).withParallelism(2, 2)         .withDeleteParallelism(2).forTable(tableName)             .withIndexConfig(indexConfig)         .withArchivalConfig(HoodieArchivalConfig.newBuilder().archiveCommitsWith(20, 30).build()).build();
 HoodieJavaWriteClient<HoodieAvroPayload> client =         new HoodieJavaWriteClient<>(new HoodieJavaEngineContext(hadoopConf), cfg);
Expected behavior
HoodieJavaWriteClient supports operating all tables in the database instead of only one table
Environment Description

Hudi version : 0.12.0

@xxWSHxx IMO, a write client per table makes sense as tables could have different configs for different purposes. Could add a little bit more detail about your use case, why would you want a single write client per database? If that's the case how would you specify different configs for the tables in the database?

Similar to the jdbc connection, a database can use only one connection. The main purpose is that if each table has a client, many clients need to be created when many tables need to be updated, which will consume memory resources

We need to decouple configs from the client and use fluent builder APIs to build the client.
@xxWSHxx I would suggest creating a JIRA and put some ideas, and let's discuss there.

The existing design is all meant for interacting with tables. To handle interactions at database level, you could come up with some model say HoodieJavaWriteClientPool. But this is a broader design problem, which is more suitable to be raised in dev email discussion. Feel free to start a [DISCUSS]  email thread with<EMAIL_ADDRESS>


Receive: fix stats

If we account stats for remote write and local writes we will count them twice since the remote write will be counted locally again by the remote receiver instance.

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Only count local writes for stats.
Verification
Not verified.

This pr made the thanos_receive_write_timeseries_* and thanos_receive_write_samples_* disappear. See @roth-wine reported issue, I think we should revert this one. The bug of counting twice or in our setup 3 times is because replicationFactor = 2 or 3 since the map is grouped by endpointReplica.
I think we should rethink if this metric should represent incoming samples persist stats or absolute counts which accounts for any replicationFactor for quorum. Any ideas? @MichaHoffmann @GiedriusS


Got ANALYSIS ISSUE on pub.dev due to Flutter/Dart version constraints

Actual error shown in "Pass static analysis" section:
The current Dart SDK version is 3.3.0-279.3.beta.
Because pdfrx requires SDK version >=3.3.0 <4.0.0, version solving failed.

If I fix the Dart SDK version to 3.3.0-279.3.beta, the next is  Flutter SDK version to 3.19.0-0.4.pre. And if fix both, it results in another error :(

https://github.com/dart-lang/pub-dev/issues/7484

package:web 0.5.0 depends on sdk: ^3.3.0-76.0.dev:
https://github.com/dart-lang/web/blob/a54a1f0447979f9a3ea220199eca849ffb214e91/pubspec.yaml#L7

version: 1.0.5+test1
homepage: https://github.com/espresso3389/pdfrx

environment:
  sdk: '>=3.3.0-76.0.dev <4.0.0'
  flutter: '>=3.19.0-0.4.pre'

results in
The current Dart SDK version is 3.3.0-279.3.beta.
Because http >=1.2.1 requires SDK version ^3.3.0 and http >=1.1.2 <1.2.1 depends on web >=0.3.0 <0.5.0, http >=1.1.2 requires web >=0.3.0 <0.5.0.

e9e58dc6bbb61b538d6bd26d50904b547b49dbd6 (1.0.6) downgrades web package to 0.4.2 as a temporary workaround.

pub.dev now works correctly!
https://github.com/dart-lang/pub-dev/issues/7484#issuecomment-1953984527


Move reconnect logic to main thread

Description
Has been reported some cases where Tower Agent after a long time running it gets stalled without reconnecting or sending the heartbeat anymore. It has been impossible to reproduce this scenario.
Trying to mitigate this problem this PR moves all the re-connection logic to the main thread.

Looking at the code I think the System.exit should only be used when there's a not recoverable error e.g. lack of the access token or when the expected work directory cannot be created.
Instead it looks to me there's a system exit even for transient errors e.g. timeout.
Even better, there should be classified in recoverable and not-recoverable with a corresponding exception. Recoverable error should be managed in the infinite while loop, instead the non-recoverable should be managed in the main method producing a system exit eg.
    public static void main(String[] args) {
      try {
        PicocliRunner.run(Agent.class, args);
      }
      catch( Throwable e ) {
         log.error("Some message ${e.message}", e)
         System.exit(1)
      }
    }


Create 0000-01-02-GregCocks-NOAA.md - 2nd slide

create 2nd slide

merge the changes to the main branch, and delete branch


ReadTheDocs: Switch to the Alabaster theme

Refs #1477

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1840   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files           1       1
  Lines        4038    4038
  Branches      714     714
======================================
  Hits         4038    4038

Continue to review full report at Codecov.

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

This is of course slightly subjective, but the "clean" Alabaster theme seems to demonstrates a better similarity to the various documentation styles of other popular JavaScript projects.

👍


Restore order_js config

Currently javascripts are separated into [name].min.js and standard javascripts. By default pre-minified javascripts are added to the top of a build, given that they are usually libraries. To avoid double compression issues, they were removed from the regular js builds, and added back after. That negates the order_js config set by users.
This update should restore the order and Octopress Ink should be updated to handle compressing files independently and caching as necessary.

I think I have something else for you to consider as well:
If I add the Foundation framework to a site using Bower, I end up with 2 JS files I need to include:
<script src="bower_components/jquery/dist/jquery.min.js"></script>
<script src="bower_components/foundation/js/foundation.min.js"></script>

The issue is that, in that same directory, there is also the uncompressed JS file as well. Now, I could either

move the JS file that I want to use (minified or not) to /js but have to copy again when I update Foundation down the road
symbolically link the file to /js, which seems like the best current option
specifically list the file under the asset-pipeline key (which would require some further coding)

I'm fine the symbolically linking the file, but I do think it's worth considering whether or not it'd be worthwhile to simply have to call out the JS files you want to put through the pipeline, in their particular order and including their directory location, like so:
  combine_js: true
  compress_js: false
  order_js:
    - bower_components/jquery/dist/jquery.min.js
    - bower_components/foundation/js/foundation.min.js
    - js/app.js
    - js/wufoo.js
    - js/sharing.js

I agree with @chauncey-garrett here, although the defaults are probably nice there tends to be some weird loose ends here and there that you might not want to include, I'd love to be able to choose what files are put through the minification process like I'd currently do it with Gulp. Right now I'm not sure if it's totally worth using this compared to Gulp but I'm taking a look :) Thanks for the work @imathis

Thinking about this some more, maybe I drop the compression of the javascripts altogether and just add them to the build as is. This way you could use whatever tool you want to build the javascripts and this would just inject it into the blog and write the <script> tags for you.

I fixed it! All javascripts are compressed independently (and cached) and order is set as you'd expect.

Looks great!


RFE: German Localization

as discussed in Forum

I starteda patch WiP here, will be completed next days

German_Translation_004.txt
Hi, can someone please integrate attached text file as KanboardSupport/Locale/de_DE/translations.php
Guthubdrives me crazy ...

German_Translation_004.txt
Hi, can someone please integrate attached text file as KanboardSupport/Locale/de_DE/translations.php
Guthubdrives me crazy ...

sure I will do it shortly


norm_num support for rings and fields.

norm_num needs to support

<, >, ≠
Ring operations: Neg, Sub, %
Field operations: /, ⁻¹
Reals and rationals

These are done.