Caching of shader modules and pipeline objects

Right now, each world object gets its own shader modules and pipeline objects (plural because of multiple render passes). However, sometimes they could be re-used. Same holds for e.g. bind groups and bind group layouts.
In cases when there are a lot of objects in a scene, it's likely that most have the same material type, so this form of caching could save significant gpu memory and startup-speed for scenes with many similar objects.
The most tricky part is likely destructing the objects when nothing uses it anymore. -> A solution is a cache that uses weakrefs and also attaching the cached objects on the world object.
Some notes:

A reusable cache as an "internal service" that is used throughout pygfx for all features that require caching, with a global toggle would be a very useful debugging feature. It can also provide statistics such as cache hit/miss ratios and usage stats.
Could make more use of caching in the renderer. There are some todo's indicating possible locations.
We could re-use the command queue instead of recording it on each draw. Could use render bundles (tracking in  #518)

Locations where caching is/can be used:

[x] In the text subpackage, we apply some caching of font objects and obviously glyphs in the atlas, but that's all really specific to text, so it does not seem right to bring that into the same system.
[ ] _mipmapsutil.py #516
[ ] _shadowutil.py.
[ ] _pipeline.py already uses a cache and may do with more.
[ ] _renderer.py
[ ] _environment.py?
[ ] _flusher.py
[ ] May just have to scan over the whole wgpu subpackage to identify more cases.

Some notes:

Caching as an "internal service" that can be toggled off would be a useful debugging feature.

Updated the top post to list locations of interest for caching. Also spun the render bundle idea into a separate issue.

This is exciting! Naive question, does it make sense to store the cache on disk too to reuse it between pygfx runs? I've seen the "Building Vulkan shader cache" thing that pops up on steam the first time you launch a game or after updating gpu drivers.

does it make sense to store the cache on disk too to reuse it between pygfx runs?

That's an interesting idea that I had not thought of yet. I think though, that the initial focus will be on being able to reuse stuff when multiple similar objects are created.

This is exciting! Naive question, does it make sense to store the cache on disk too to reuse it between pygfx runs? I've seen the "Building Vulkan shader cache" thing that pops up on steam the first time you launch a game or after updating gpu drivers.

The idea behind that is mostly to avoid runtime shader compilation and pipeline building. You would need to ensure all cacheable construction occurs on start of the application, so there won't be any stuttering after that.
Storing compiled shaders on disk is an option to speed up the next time the application is launched. I'm not sure if you could do the same for pipelines. Probably not.
I propose to create a separate issue for shader caching.

Storing compiled shaders on disk is an option to speed up the next time the application is launched. I'm not sure if you could do the same for pipelines. Probably not.

No, pipelines are objects on the GPU, we can't serialize those. If we ever want to go down the route of caching to disk, we should do some benchmarks to see if caching shader sources is worth it.


YoutubePlaylists pages can sometimes refer to their contents via /watch?v=...&list= rather than /playlist?list=

Checklist

[X] I'm reporting a broken site support
[X] I've verified that I'm running youtube-dl version 2020.03.24
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/channel/UCPlYCdARxfOHlMUaA4b8a_A/playlists?view=50&flow=list&shelf_id=17666223384013636041']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.8.2 (CPython) - Linux-4.19.114-1-ARCH-armv7l-with-glibc2.4
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2
[debug] Proxy map: {}
[youtube:playlists] UCPlYCdARxfOHlMUaA4b8a_A: Downloading webpage
[download] Downloading playlist: Benny Green - Topic
[youtube:playlists] playlist Benny Green - Topic: Downloading 0 videos
[download] Finished downloading playlist: Benny Green - Topic

Description

The YoutubePlaylists extractor seems to be expecting lists of playlists to refer to their contained playlists via a link to the full playlist (/playlist?list=...) inside each h3.yt-lockup-title element. However, pages such as the exemplar https://www.youtube.com/channel/UCPlYCdARxfOHlMUaA4b8a_A/playlists?view=50&flow=list&shelf_id=17666223384013636041, even when &disable_polymer=true is added to the URL, instead have the h3.yt-lockup-title contain a link to the first video in the playlist with the playlist id included (/watch?v=...&list=...). This causes YoutubePlaylists's regex to not match anything, resulting in youtube-dl downloading nothing from the shelf.
I've attached a --dump-pages output of the same URL, to illustrate:
UCPlYCdARxfOHlMUaA4b8a_A_https_-_www.youtube.com_channel_UCPlYCdARxfOHlMUaA4b8a_A_playlistsview=50_flow=list_shelf_id=17666223384013636041_disable_polymer=true.dump.txt

Strange, I'm having no problems downloading that playlist. But where is yt-lockup-title meant to be? It doesn't appear anywhere in the source of the channel page you linked to when I view it in my browser.

That's really strange. Here's what I see in my browser:

... which seems congruent with the relevant portions of the --dump-pages output that youtube-dl gets me.

Oh yeah I got the yt-lockup-title when I disabled polymer, however it's not in my dump whether I disable polymer or not oddly - and the download works in both cases.


Store date / timestamp in logs

Would be super helpful to me for log-processing for a number of applications. Right now I've got to pull it from the folder structure, and that's just awful.
While I'm at it, the following fields would also be nice, so I don't have to parse them from the filename or from weird places like p1rating:

formatid
battle ID / number

This was resolved by #2732


[1.13] Prune VIPs with no backends in dcos-net

High-level description
Prune VIPs with no backends in order to avoid unbounded growth of state and messages exchanged among dcos-net processes.
Corresponding DC/OS tickets

DCOS_OSS-5356 dcos-net should prune VIP entries with no backends

Checklist for component/package updates:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): diff
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain: there is an integration test in dcos-net repo.
[x] Test Results: CI job
[ ] Code Coverage: none.

@mesosphere-mergebot label Ready For Review


Renames clear-caches to clear-cache

Resolves #353
Do we have a policy when we rename or remove commands? Should we deprecate the old one? Add a warning?

We will handle these differently after 1.0.0, but until then we just note them in CHANGELOG.txt. Once this change is in CHANGELOG.txt, +1


boards/arm/stm32f7: fix nucleo boards break for cmake

Summary
fix nucleo-f722ze and nucleo-f746zg/pysim board break for cmake
Impact
unlock CI
Testing
local build with cmake, previously I tested only make build and errors were not caught by CI

[Experimental Bot, please feedback here]
No, this PR does not fully meet the NuttX requirements.
Here's why and how to improve it:

Missing details in Summary:

Why is the change necessary?  Clearly state the problem the PR addresses.  For example: "CMake builds were failing for nucleo-f722ze and nucleo-f746zg/pysim boards due to [explain the root cause of the break]."
How does the change work?  Briefly explain the technical approach taken to fix the CMake build issue.

Impact Section:

Is new feature added? Is existing feature changed? - Since this is a bug fix, it's not a new feature.  Specify that an existing feature (the build system for specific boards) is being fixed.
Impact on user: This likely has no user impact beyond fixing a broken build. If so, state that explicitly.
Impact on build: Be more specific than "unlock CI." Explain what part of the build process was affected and how the change resolves it.
Other impacts:  Most of the other impacts are probably "NO" but confirm this in your PR.

Testing:

Provide more context:

Specify the exact versions of CMake and Make used for testing.
For "Target(s)", be explicit about the configurations tested on each board (e.g., "nucleo-f722ze:nsh").

No logs included:  You need to include actual build logs (both before and after the change) to demonstrate the issue and the fix.  Trim the logs to show only the relevant error messages and the successful build output after your changes.

Recommendations:

Expand the Summary to clearly explain the problem and your solution.
Be specific in the Impact section, particularly regarding build changes.
Include relevant build logs to demonstrate the problem and the fix.

By addressing these points, your PR will be much more informative and easier for maintainers to review.


Can't reproduce the results of OVMM

I evaluate the rl_agent with GT semantics using this command:
python projects/habitat_ovmm/eval_baselines_agent.py --baseline_config_path projects/habitat_ovmm/configs/agent/rl_agent.yaml
The prertained skill models are downloaded from https://dl.fbaipublicfiles.com/habitat/data/baselines/v1/ovmm_baseline_home_robot_challenge_2023.zip
The result is:
{'episode_count': 1200.0, 'does_want_terminate': 0.18916666666666668, 'num_steps': 1095.5791666666667, 'find_object_phase_success': 0.3258333333333333, 'pick_object_phase_success': 0.28583333333333333, 'find_recep_phase_success': 0.15166666666666667, 'overall_success': 0.0225, 'partial_success': 0.19645833333333335}
Which is lower the reported results in the paper: HomeRobot: Open-Vocabulary Mobile Manipulation.
Can you help figure out the reason?

Yes, the find_object_phase_success in your case is lower than the we reported or expect with RL skill. Can you please send the home-robot commit you used to get these numbers?

commit b643698a372c9b64af8995d7295d28ba8b7dfe99

Thanks for bringing this up, we'll look into it asap

Yes, this is due to a regression that got introduced in our codebase - the RL skill during eval was using a step size different from the one that was used during training. This was fixed in this commit cb6c0255c98aea24bf1d894716cd8a368b4e61d9. You should see better number when using this or the latest main.

Since I run the evaluation process for multiple times, I found that the evaluation time can vary from 1 day to 5 days. What might cause this unstable evaluation time?

Did you keep everything the same when running evaluations?
Our heuristic baseline takes longer than our RL baseline and setting PRINT_IMAGES:1 (like in hssd_demo.yaml config) also increases the evaluation time.

I use the newest version of  home-robot and don't change the config. It took about 10 hours to process 100 episodes which is too slow. I use NVIDIA GeForce RTX 3090.

Can you try making the change noted above (PRINT_IMAGES:0)

https://github.com/facebookresearch/home-robot/blob/main/projects/habitat_ovmm/configs/env/hssd_eval.yaml#L6
I think I'm using this config in which PRINT_IMAGES=0.

The used gpu memory during evaluation is 1902MiB. Is it normal?

Honestly this looks fine to me

Yeah @kellyiss the heuristic baseline with Detic is quite slow! @yvsriram has suggested switching to groundedSAM if this is too slow for you to build on top of

I evaluate the rl agent baseline again with GT semantic, this is the result:
Metrics:
{'episode_count': 1199.0, 'does_want_terminate': 0.3803169307756464, 'num_steps':<PHONE_NUMBER>669724, 'find_object_phase_success': 0.5613010842368641, 'pick_object_phase_success': 0.5162635529608006, 'find_recep_phase_success': 0.41201000834028356, 'overall_success': 0.05254378648874062, 'partial_success': 0.38552960800667224}
All seems normal except for the 'overall_success'. Is there anything wrong with this metric?

Hi,
It is possible that the numbers may be a bit different compared to what we have in the paper because our setup has changed since then. For example, the 6-point increase in FindRec success is due to an issue we fixed related to goal duplication. For the 6-point drop in overall success, I am not aware of any change that could cause this but I will check this once more. We are rerunning all evaluations with the new setup and will update the numbers in the paper this week.

@kellyiss have you solved the problem with overall success? I'm facing the same problem.

Btw, we had fixed this issue in this commit 77746a741639f58975433856cfef60043802ee4c and this comment was addressed.

Closing this issue hoping it has been resolved. Do let us know if you are are still facing any issues.


Disable email delivery during rake db:seed

Because we are using the letter_opener gem, every time an email gets sent, a new browser tab is opened. While testing and resetting the DB, having browser tabs open each time and steal focus from the Terminal gets annoying.
We're not testing or using emails during the seeding of the DB, so we can suppress emails by using Devise's skip_confirmation! method.

Coverage remained the same at 99.4% when pulling af97598c58da86666580b31c01800e99aaacfa5a on disable-email-sending-during-db-seed into 9c4e1426609593a66905280950d5eb2be34c561b on master.


No longer append unit to zero-values in unit plugin

Description
There is no need to add a unit for a zero (0) value (unless it is a duration, which is of no concern for this plugin). This pull-request no longer add a unit for these values.
Versioning

Patch

fela-plugin-unit

Checklist
Quality Assurance

You can also run yarn run check to run all 4 commands at once.

[x] The code was formatted using Prettier (yarn run format)
[x] The code has no linting errors (yarn run lint)
[x] All tests are passing (yarn run test)
[x] There are no flow-type errors (yarn run flow)

Changes
If one of the following checks doesn't make sense due to the type of PR, just check them.

[x] Tests have been added/updated
[x] Documentation has been added/updated
[x] My changes have proper flow-types

Although its a major update isn't it? The output is obviously changed, so tests might fail if ppl are using ^ versions.

It is not breaking anything from a CSS & layout standpoint. If you’d rather bump a major, do so but I don’t think it’s necessary.


sort environment variables.

this will also solve the issue of https://github.com/Microsoft/vsts-agent/issues/837

this only sort the print out variable list in RM, but didn't sort the variable dictionary in JobContext, what's the reason for this change?

@TingluoHuang We want the variables to be printed in sorted order in a release, it's easier for users to spot variables when they are sorted.

makes sense


Custom sapling shapes doesn't work correctly in treepacks

Describe the bug
When using custom sapling shapes in the species.json, only the second array will register for the hitbox.
To Reproduce
Use this for the shape:
  "sapling_shape": [
    [0.375, 0, 0.375, 0.625, 0.1875, 0.625],
    [0.25, 0.1875, 0.25, 0.75, 0.5, 0.75]
  ]

Only the second array gets used.
If the two arrays are swapped, same result.
Screenshots

Mods List
Dynamic Trees v1.3.0-Beta13

Fixed in beta 15


depend-o-pocalipse

Fortune favors the brave.
There seem to be a few dependencies that I can get it to compile without it needing them.

How do you identify these? Some tool/script?

Yup, cargo metadata on the workspace, then cargo edit's cargo rm on each dep and cargo check --all-targets to see if it built afterwards. record good results to a file and then git reset --hard and try a different dep.
It ran all night but it seemed to not do too bad a job.... The down side is I don't have enough color to know if there's any reason we can't remove these dependencies. This is where I need your help to point out anything deeply suspicious - "surely that crate's used by x" etc.
I'm going to have a follow up PR that does the same but for dev-dependencies but I want to land them separately.
(We can brush up the function into something nice and put it into cargo unleashed if it is useful to people - there's already a subcommand in cargo-unleash that does something in this direction but isn't quite as brute force.)

Both good questions. I did build a couple of pallets - is there a
Cargo.toml with all the no_std crates in as a workspace? Or will all frame
& pallets cover it? I was assuming CI tested release mode as well as wasm -
there were certainly some wasm ci errors initially.
Will run a few more checks and will report back here.
On Fri, 30 Jul 2021 at 09:36, Benjamin Kampmann @.***>
wrote:

@.**** approved this pull request.
Neat. Nothing too suspicious coming up glancing over them, there's a good
chance these are indeed not needed (anymore).
But two questions on your process, just to make sure we don't accidentally
dropped one too many:

did you build in release-mode(, too)?
have the frame/pallets been build for wasm/no_std?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/paritytech/substrate/pull/9450#pullrequestreview-718851482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGEJCGG5OOUTRM2O7MWZMTT2JQBPANCNFSM5BDZUL5Q
.

Release build is clean. I'm unclear at the moment how to build just the wasm for the whole workspace.

The pallet/frame crate should be good if a release build of node-runtime is done.

Yup, cargo metadata on the workspace, then cargo edit's cargo rm on each dep and cargo check --all-targets to see if it built afterwards. record good results to a file and then git reset --hard and try a different dep.

I used cargo outdatd in the past to identify unused dependencies. You did try that?

Yup, cargo metadata on the workspace, then cargo edit's cargo rm on each dep and cargo check --all-targets to see if it built afterwards. record good results to a file and then git reset --hard and try a different dep.

Did you try cargo-udeps?

(There's some auto-formatting of the toml files going on here which is nice, though unclear how vscode is managing this?)

Aside from the above no other .rs has been changed - the others are just cargo fmt. (Potentially we should pin the compiler via a rust-toolchain.toml so that we manage when we take these rustfmt changes)


Release 8.0.0

Bringing #1498 forward.
Migration to Kotlin from Java
Migration to swift from ObjectiveC
Fixed Amazon getProducts being called twice
Implemented fix for Promoted product on iOS
Removed Java style plugin
Updated example app to RN 0.64.
Contributors
Co-authored-by: andresesfm<EMAIL_ADDRESS>

I couldn't get my products from the Configuration.storekit file, is this normal ?

@andresesfm It's been a long time. Let's release this to 8.0.0. How do you think?

So far released to 8.0.0-rc.6. @andresesfm Do you think we can release 8.0.0?

I think so @hyochan . So we can minimize merge conflicts between both branches

I think so @hyochan . So we can minimize merge conflicts between both branches

Let's do it 💯


wip: ccloudvm: Add templates for kata containers.

Add templates to tests packages in VM.
Signed-off-by: Jose Carlos Venegas Munoz<EMAIL_ADDRESS>

Hi @markdryan - could you take a look at this?

You could simplify both workloads a fair bit by inheriting from the basic xenial and fedora workloads in the ccloudvm/workloads directory.
For xenial we even have a docker-xenial workload that installs the latest version of docker and correctly configures proxies if defined.
This is how the ciao workload works, e.g.,
https://github.com/intel/ccloudvm/blob/master/workloads/ciao.yaml#L2
Note that it inherits from docker-xenial and so does not need to install docker, configure a user account or setup apt proxies.

@jodh-intel  @markdryan  I like the proposed ideas - I will move this files to ccloudvm to ensure it inherits  from the files there.

@jcvenegas There's no need to move the files to ccloudvm.  I fact I think it's probably better to keep them in the kata-containers project.  You should still be able to inherit from the ccloudvm workloads, even if your workload ins't itself in ccloudvm.

@markdryan  - got it .

@markdryan  @jodh-intel

Added only docker-xenial based workload.
Added a README.

@jcvenegas Have you tested behind a proxy?
I wonder
wget -qO - http://download.opensuse.org/repositories/home:/katacontainers:/release/xUbuntu_$(lsb_release -rs)/Release.key | sudo apt-key add -
will work behind a proxy. Could you check?

@markdryan  I am using redsocks proxy in my host :/

We should make sure that the workload works in a proxied environment without having to use a transparent proxy.

@markdryan by inheriting xenial -> docker-xenial -> kata-docker-xenial.yaml   I want to think that will use the proxy setup from https://github.com/intel/ccloudvm/blob/master/workloads/xenial.yaml#L9-L18

@markdryan  sure - let me disble my redsocks and export proxy manually

that task failed ! :(


Property injection is not working correctly when appending a new implementation for multiple registrations of the same interface and injecting an enumerable of the interface.

Property injection is not working correctly when appending a new implementation for multiple registrations of the same interface and injecting an enumerable of the interface.
It works fine if building the concrete type, but when injecting the enumerable of the interface it gives a null value on the second registered type.
For example you have the following interface and classes:
public interface IFoo
{
    string Test { get; set; }
}

public class Foo1 : IFoo
{
    public string Test { get; set; }
}

public class Foo2 : IFoo
{
    public string Test { get; set; }
}

Then when registering each implementation in the container twice (appending the implementation with a property selector the second time) only Foo1 will have the property set when resolving an IEnumerable of IFoo. Foo2 has null for the Test property and is a different instance to resolving Foo2 directly (which has the property set correctly) despite being registered as a singleton.
const string testFoo1 = "TF1";
const string testFoo2 = "TF2";

var container = new Container();

container.RegisterMany(new[] { typeof(Foo1), typeof(IFoo) }, typeof(Foo1), reuse: Reuse.Singleton);
var propertiesAndFieldsSelector = PropertiesAndFields.Of.Name(nameof(IFoo.Test), _ => testFoo1);
container.RegisterMany(new[] { typeof(Foo1), typeof(IFoo) }, typeof(Foo1), reuse: Reuse.Singleton, made: propertiesAndFieldsSelector, ifAlreadyRegistered: IfAlreadyRegistered.AppendNewImplementation);

container.RegisterMany(new[] { typeof(Foo2), typeof(IFoo) }, typeof(Foo2), reuse: Reuse.Singleton);
propertiesAndFieldsSelector = PropertiesAndFields.Of.Name(nameof(IFoo.Test), _ => testFoo2);
container.RegisterMany(new[] { typeof(Foo2), typeof(IFoo) }, typeof(Foo2), reuse: Reuse.Singleton, made: propertiesAndFieldsSelector, ifAlreadyRegistered: IfAlreadyRegistered.AppendNewImplementation);

var foo1 = container.Resolve<Foo1>();
var foo2 = container.Resolve<Foo2>();
var foos = container.Resolve<IEnumerable<IFoo>>();

Assert.AreEqual(testFoo1, foo1.Test);
Assert.AreEqual(testFoo2, foo2.Test);

foreach (var foo in foos)
{
    switch (foo)
    {
        case Foo1 foo1FromEnumerable:
            Assert.AreEqual(testFoo1, foo.Test);
            Assert.AreEqual(foo1, foo1FromEnumerable);
            break;
        case Foo2 foo2FromEnumerable:
            // This will fail as 'Test' will be null and 'foo2FromEnumerable' is a different instance to 'foo2'
            Assert.AreEqual(testFoo2, foo.Test);
            Assert.AreEqual(foo2, foo2FromEnumerable);
            break;
    }
}

Issue for PR https://github.com/dadhi/DryIoc/pull/534

Yeah, there is some but here.
But I am not sure why you do the registration twice, is it intended?
Anyway, if you comment the first registrations which do not do anything because they are overridden by the next, there will be no bug. I mean to comment those:
container.RegisterMany(new[] { typeof(Foo1), typeof(IFoo) }, typeof(Foo1), reuse: Reuse.Singleton);
container.RegisterMany(new[] { typeof(Foo2), typeof(IFoo) }, typeof(Foo2), reuse: Reuse.Singleton);

@dadhi yep you're right, it will work with just the second registration by itself. Found this bug when implementing DryIoc on a legacy DI abstraction which required me to register it a second time in order to get property injection so thought I'd raise it. I can work around it by using IfAlreadyRegistered.Replace instead too which seems to work.

@jbrookssmokeball thanks for clarifying.


Which Sample? MiAIRR refers to

Is the sample described in the BioSample submission the primary sample collected from a donor (e.g., blood)? Or is it the derived/processed sample from which sequencing template was extracted (e.g., after flow sorting)?

It refers to the primary sample e.g blood and primary sample will link to BioSample ID

This is a bit tricky:
I. In the MiAIRR standard, the primary and the processed sample are separate things, which are described in set 2 / sample and set 3 / process (cell), respectively. These sets have an (implicit) 1:n relationship.
II. In the current NCBI implementation, a single record submitted to BioSample contains all the fields of set 1 / subject, set 1 / diag. & intervent.,  set 2 / sample and set 3 / process (cell), thus linking them in a 1:1 relation.
As MiAIRR is rather clear about this (I) and @ahmadchan's comment, I assume that the uncertainty about what set  2 / sample refers to (primary or processed) originates from the fixed linkage described in (II). The assumption of the NCBI implementation is that the cartesian product of the four sets described in (II) will be used to serialize the 1:n relationships. Therefore, while each BioSample record will have a 1:1 relation, the overall study does not. Third party databases could implement routines to deduplicate the data.

Hi Ahmad,
Does Christians answer your question? Not perfect, but I think we have to live with it for now.
Florian
On Oct 12, 2017, at 6:51 PM, Christian Busse <notifications<EMAIL_ADDRESS>wrote:
This is a bit tricky:
I. In the MiAIRR standard, the primary and the processed sample are separate things, which are described in set 2 / sample and set 3 / process (cell), respectively. These sets have an (implicit) 1:n relationship.
II. In the current NCBI implementation, a single record submitted to BioSample contains all the fields of set 1 / subject, set 1 / diag. & intervent., set 2 / sample and set 3 / process (cell), thus linking them in a 1:1 relation.
As MiAIRR is rather clear about this (I) and @ahmadchanhttps://github.com/ahmadchan's comment, I assume that the uncertainty about what set 2 / sample refers to (primary or processed) originates from the fixed linkage described in (II). The assumption of the NCBI implementation is that the cartesian product of the four sets described in (II) will be used to serialize the 1:n relationships. Therefore, while each BioSample record will have a 1:1 relation, the overall study does not. Third party databases could implement routines to deduplicate the data.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/airr-community/airr-standards/issues/25#issuecomment-336328095, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AefwkMYe3lCde0Z1MWzIO3hneKBEQ2M4ks5srsILgaJpZM4P30ab.

We realized in today's call that there was a mistake in the figure depicting the mapping of MiAIRR elements to NCBI: The elements of set 3 / process (cell) appeared as mapped to SRA instead of BioSample. This was incorrect and has been fixed in commit dbadf8b9dcea444fab1d0a57f4478414fc7c9998.


Add suspenders:development_environment generator

To do:

[ ] Decide which options we want to include from the existing generator
[ ] Decide on the generator name
[ ] Manually test on a real app
[ ] News and readme entries

Notes:

Should the email configuration be part of an suspenders:email_generator?
Rails comes with a setup script. Do we feel strongly enough about our own setup script to override the default one?
Quiet assets is the default development configuration setting already. Do we care about configuring it for every environment? Can we assume that one is using a CDN in production anyway?
What should the generator configuration look like? The customisations are only necessary when using rspec.
suspenders:lint assumes the app uses yarn. Is that on purpose? Should we ship a bin/yarn script that makes sure yarn is installed?

Reference:
https://github.com/thoughtbot/suspenders/blob/55cb5c246fe53aea4cf53436c7c208b40ad13e85/lib/suspenders/generators/app_generator.rb#L80-L92

config.action_view.annotate_rendered_view_with_filenames

Into it.

config.active_record.strict_loading_by_default

Is this a development tool? This seems like something that will change semantics, and I wouldn't want it to be different in different environments.

@mike-burns

Is this a development tool? This seems like something that will change semantics, and I wouldn't want it to be different in different environments.

Yes. We have it enabled on in development and test on a client project. It's a reasonable substitute for bullet. However, it's challenging to work with when queries are made outside the application, like in gems. We've run into this with devise for example. Since we don't have access to the current_user definition, we need to redefine it and call includes.

strict_loading_by_default

Oh I see how it works now. Wow that thing is under-documented.
Yup that makes sense to me.

In Suspenders 2, we had namespaced generators such as suspenders:production:timeout and suspenders:staging:pull_requests. Do we like that pattern? Should this be suspenders:development:environment?

Based on a recent client feature, I think there's value in enabling config.active_model.i18n_customize_full_message

I wonder if we'd want to enable config.eager_load in test?
https://guides.rubyonrails.org/testing.html#testing-eager-loading

I wonder if we'd want to enable config.eager_load in test?
https://guides.rubyonrails.org/testing.html#testing-eager-loading

@stevepolitodesign new versions of Rails generate config/environment/test.rb with config.eager_load = ENV["CI"].present?.
Are you suggesting setting that to true whenever RAILS_ENV=test?

I wonder if we'd want to enable config.eager_load in test?
https://guides.rubyonrails.org/testing.html#testing-eager-loading

@stevepolitodesign new versions of Rails generate config/environment/test.rb with config.eager_load = ENV["CI"].present?.
Are you suggesting setting that to true whenever RAILS_ENV=test?

Good catch. I'll collapse that suggestion 👍

Noting we might want to enable config.active_record.query_log_tags_enabled = true.
https://github.com/rails/rails/pull/51342/files


Fancy zones do not work with Visual studio Solution Explorer

Microsoft PowerToys version
0.53.3
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Open any project in VS. and make solution explorer as float. Then using Shift + Windows key, drag the solution explorer. It is not snapping to any fancy zone.
✔ Expected Behavior
It should snap to fancy zones since it is able to snap to default layout options in Windows.
❌ Actual Behavior
Fancy zones simply do not get activated when dragging Solution explorer
Other Software
Visual Studio 2019

Do you have options Allow child windows snapping and Allow popup windows snapping enabled?
/needinfo


"supplier": "NOASSERTION" getting in generated SBOM

using command
"sbom-tool.exe generate -b  -bc  -pn SBOM.PoC -pv 1.0 -ps PoC -nsb https://www.PoC.com/"
for every pkg i am getting "supplier": "NOASSERTION"
Am I doing anything wrong ?
{
"name": "Swashbuckle.AspNetCore.SwaggerUI",
"SPDXID": "SPDXRef-Package-8D9F0AA9D67828FAEB401376FBF2C5CF2033BCA3ECEA1F5BDC262A7F88A0CC52",
"downloadLocation": "NOASSERTION",
"filesAnalyzed": false,
"licenseConcluded": "NOASSERTION",
"licenseDeclared": "NOASSERTION",
"copyrightText": "NOASSERTION",
"versionInfo": "6.2.3",
"externalRefs": [
{
"referenceCategory": "PACKAGE-MANAGER",
"referenceType": "purl",
"referenceLocator"<EMAIL_ADDRESS>}
],
"supplier": "NOASSERTION"

@sebasgomez238 were you able to figure out what was going on here?

I am having the same issue, I generally have to build project / restore packages at least once before supplier data is retrieved properly.

@sebasgomez238 , please update with your findings

At the moment supplier information is only retrieved for NPM and NuGet. Here is some information about how it works:
Nuget:

Supplier information is retrieved from either the .nuspec or .nupkg files present in the directory.

In the case of dependencies these are detected via the project.assets.json present in every project's '/obj' directory. Example: (
"C:\repos\sbom-tool\src\Microsoft.Sbom.Adapters\obj\project.assets.json").

The project.assets.json contains the list of dependencies but does not contain supplier information as we don't have access to
the individual dependencies' .nuspec/.nupkg files.

This means that supplier information will not be found for transitive dependencies.

NPM:

Node projects contain a @node_modules directory where every package is listed and its corresponding package.json is
present.

Component Detection parses this package.json and pulls out the supplier information.

The @node_modules directory is typically created after running npm install.

The reason why the project needs to be built in order to capture the information is because the build process usually fetches/restores the projects packages which we then use to retrieve supplier information.

Having the build process fetch the project's packages is not ideal, but is working as intended.


GPUImageView orientation question

Hi Brad,
I'm using the following structure to apply filters to a movie file:
GPUImageMovie -> some filters -> GPUImageView
The problem is that the output result orientation isn't correct. for example a movie that was taken in portrait mode will be presented in landscape mode.
So I understand that the GPUImageView doesn't handle the orientation in any way, my question is what the correct way to fix that?
Should I use GPUImageView's setInputRotation:atIndex: method?
Should I use some transform filter in the middle?
Maybe a better solution?
Thanks

Hi mayur43s,
The same issue here. I'm two days trying to solve this problem, the video portrait is forced into landscape. Please someone would have a solution for this?

Think I have the same problem here. Any solution?

I am also getting the same issue. Also, when I saved the portrait video, that was saved in landscape orientation with big size in MB. i.e. my original video's size was 10 MB and the edited video size became 25 MB. I was applying filters using .acv files. You can check my issue ticket from this link https://github.com/BradLarson/GPUImage/issues/2166


fix updated cache function name

Description
I forgot to change cache name after merging different MR
Checklist
Answer each question with "yes", and provide an explanation if the answer isn't what is expected.

Is this PR focused on a single feature without the possibility of being divided into smaller PRs?

Answer: yes

Are all variables set with logical defaults that are backward compatible?

Answer: yes

Can you provide a link to a Neptune experiment verifying the functionality of this PR?

Answer: na

Have you successfully executed the linter and tests?

Answer: yes

Are all functions concise and don't require splitting?

Answer: yes

Is there documentation for arguments and complex logic?

Answer: yes

Are there no temporary solutions in the code? Should a task be added to Trello to refine the code?

Answer: yes

Are all functions placed in appropriate files?

Answer: yes

Is the PR name meaningful and descriptive?

Answer: yes

Is PR description provided, or unnecessary?

Answer: yes

Other questions:

Have you not made changes to the requirements or anything related to the Singularity image? If you have, did you generate and upload a new image to the clusters?

Answer: no

Is the change significant enough to notify all individuals working with this repository?

Answer: no

Is there a need for a second reviewer?

Answer: no

Have you not made changes to the requirements or anything related to the Singularity image? If you have, did you generate and upload a new image to the clusters?

is this image available to anyone else apart from yourself?

Also I would something like (note that I'm not very good at naming things about code quality): is the added code put into the correct level of abstraction - for example, is it not the case that some non-shared low-level code is put into the main loop at a high abstraction level? Is the code extensible?


Update workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (05354d6) to head (d3c184c).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #666   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           89        89
  Lines         2456      2456
  Branches       237       237
=========================================
  Hits          2456      2456

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Can you provide a wrapper for tabler icons similar to your clr-icon component

Summary
Can you provide a wrapper for tabler icons.

Provide nice wrapper for tabler icons.
Seems like a nice compliment to your existing icon set
Should NOT impact existing behaviors

Use case
Larger set of icons to choose from
Examples
See  tabler icons.
Workarounds
Use another icon within your set or write my own wrapper or use tabler svg directly in my app
If you choose not to implement this, I'll completely understand, but you don't get what you don't ask for!

The Clarity Icons API can accolade this.
ClarityIcons.add({"my-custom-icon": "<svg ... >[your SVG code goes here]</svg>"});


ppx-irmin: remove test dependency on Fmt

Currently based on https://github.com/mirage/irmin/pull/952 (unmerged), as we intend to merge that first.

Rebased.


Invalid flag goes out of sync when text field is inside custom field

Description
Note: this all has been done using Java. I have no idea if it is reproducable with pure client-side code, but probably is.
When text field is placed inside a custom field, something overrides invalid flag to be false, despite server-side code setting it to true. The text field has a pattern that "guards" the input, maybe it is related.
Expected outcome
The invalid flag stays in sync with whatever has been set by the server.
Actual outcome
The flag is set to false making the component not display validation error.
Live Demo and steps to reproduce
This issue was spotted in my add-on here: https://github.com/vaadin-miki/super-fields/issues/241
The thread includes code that reproduces the issue. In my server-side code my custom field has this code:
setInvalid:
        super.setInvalid(invalid);
        this.field.setInvalid(invalid);

In certain cases the invalid flag on the custom field is true, while the same flag on the text field is false, when obviously both should be the same.
I have caught the property change in the client-side and here is the relevant log output:
org.vaadin.miki.BinderPage - value changed ComponentValueChangeEvent[source=org.vaadin.miki.superfields.numbers.SuperBigDecimalField@7a4ac994, value = 12345, oldValue = 0]
org.vaadin.miki.BinderPage - validator validating 12345
org.vaadin.miki.superfields.numbers.SuperBigDecimalField - setting invalid to false
org.vaadin.miki.superfields.numbers.SuperBigDecimalField - setting invalid to true
org.vaadin.miki.superfields.numbers.SuperBigDecimalField - client-side property changed to true
org.vaadin.miki.superfields.numbers.SuperBigDecimalField - client-side property changed to false

There is more discussion about a similar behaviour here: https://github.com/vaadin/vaadin-date-picker/issues/751 - while I have not tested it, I suspect it is caused by the same issue (invalid flag being overwritten by client-side code).
Server-side fix
Adding this code to custom field's setInvalid seems to fix the issue:
        this.field.getElement().getNode().runWhenAttached(ui -> ui.beforeClientResponse(this.field, context ->
                this.field.getElement().setProperty("invalid", super.isInvalid())
        ));

However, it simply sets the flag again and thus degrades performance. I think the server-indicated validation should simply not be overridden by anything.
(please note that this will work only because the custom field has only one component, the text field - this may not be a good solution when there are more fields)
Browsers Affected
All. Present in Vaadin 14.4.

Could you please add an example with which the issue can be reproduced, using plain Vaadin components only?

public class GreetingView extends VerticalLayout {
    private static final long serialVersionUID = -2291059367620847679L;

    public GreetingView() {
        setAlignItems(Alignment.CENTER);
        TextField info = new TextField("Is this a valid fraction?");

        Binder<AtomicReference<Fraction>> binder = new Binder<>();

        FractionField fractionField = new FractionField();
        fractionField.addValueChangeListener(event -> {
            Fraction value = event.getValue();
            if (value == null)
                Notification.show("Why is it null? " + event.getSource().isInvalid());
            else
                Notification.show(value.toString() + " " + event.getSource().isInvalid());
        });

        binder.forField(fractionField).asRequired("Please fill out this fraction")
                .withValidator(i -> i.numerator == 3, "numerator has to be 3, because I say so.")
                .bind(AtomicReference::get, AtomicReference::set);
        binder.setBean(new AtomicReference<>(new Fraction(2, 4)));

        binder.addValueChangeListener(event -> info.setValue(fractionField.isInvalid() ? "No" : "Yes"));

        add(fractionField, info);
    }

    public class Fraction {
        private final int numerator;
        private final int denominator;

        public Fraction(int n, int d) {
            numerator = n;
            denominator = d;
        }

        @Override
        public String toString() {
            return numerator + "/" + denominator;
        }
    }

    public class FractionField extends CustomField<Fraction> {
        private static final long serialVersionUID = 1L;
        public TextField numerator;
        public TextField denominator;

        public FractionField() {
            numerator = new TextField("Numerator");
            add(numerator);
            denominator = new TextField("Denominator");
            add(denominator);
        }

        @Override
        protected Fraction generateModelValue() {
            try {
                int n = numerator.getValue().isEmpty() ? 0 : Integer.parseInt(numerator.getValue());
                int d = denominator.getValue().isEmpty() ? 0 : Integer.parseInt(denominator.getValue());
                if (d == 0) {
                    // How to correctly communicate ArithmeticException? And/or what to return in this case?
                    // throw new ArithmeticException();
                    return null;
                }
                return new Fraction(n, d);
            } catch (NumberFormatException e) {
                // How to correctly communicate user input error? And/or what to return in this case?
                // throw e;
                return null;
            }
        }

        @Override
        protected void setPresentationValue(Fraction newPresentationValue) {
            numerator.setValue(Integer.toString(newPresentationValue.numerator));
            denominator.setValue(Integer.toString(newPresentationValue.denominator));
        }

        @Override
        public void setInvalid(boolean invalid) {
            super.setInvalid(invalid);
            denominator.setInvalid(invalid);
            denominator.setErrorMessage(invalid ? "This is invalid" : null);
        }
    }
}

I'm not sure if this is related, but I see a similar problem. Maybe the OP can modify the code to show his problem.
To reproduce the weirdness fill out the fraction field in the following order and leave (blur) the custom field between inputs:
0. 2/4 should not be valid due to validator. I can get over it, but it is not obvious to the user

Enter 2/5 to see that it is invalid
Enter 3/5 to see it is valid
Enter 3/0 to see it is invalid; but only because of asRequired(). It should throw an arithmetic exception, but how to do this?
 One could think of more advanced situations where throwing exceptions is more convenient that relying on returning null and deal with that somehow.
Yes, you can use the IntegerField to mitigate the NumberFormatException, but that is not the point here. I can simulate this behaviour even with IntegerField but the code would be longer then.


Break on delimiter

My apologies, apparently the previous commit I made didn't account for the loop break that the original some callback had when reaching the delimiter, which returns an incorrect queue time if the queue doesn't have enough songs and autoplay is on.
This commit should fix the issue 😅.

No worries 😅👍


how to set the key frame interval?

thank you for providing this implementation of segmentation!
from the rfcn implementation of the original paper, the key frame interval is set under the config. how to set the interval in your implementation?

This is an old project two years before, it's likely just a hyperparameter, which is reported in the original paper. You can refer to it for more details.

This is an old project two years before, it's likely just a hyperparameter, which is reported in the original paper. You can refer to it for more details.

got it, thx :)

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @tvl, @nenadg, @ericmm, @Djarirahmad, @Terrypang89, @hishamank, @federico1828

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @sv-pro, @Chengel-HaltuD, @sesang06, @ActorExpose, @helxsz, @flaviogarrido, @CaptainRock, @beeant, @munj1, @moshthepitt, @mnuestc, @lookfirst, @hopkinch, @Nodd77, @liwenzheng, @Plarkass, @Knowledge-Wisdom-Understanding, @tjkbrown, @bprabhuin, @georgepampalis


Create connection only connects to public, but not random addresses.

At the HCI level, this would be the PeerAddressType flag, which currently is hardcoded as the other detailed parameters.
cmd.LECreateConn{
			LEScanInterval:        0x0004,     // N x 0.625ms
			LEScanWindow:          0x0004,     // N x 0.625ms
			InitiatorFilterPolicy: 0x00,       // white list not used
			PeerAddressType:       0x00,       // public
			PeerAddress:           pd.Address, //
			OwnAddressType:        0x00,       // public
			ConnIntervalMin:       0x0006,     // N x 0.125ms
			ConnIntervalMax:       0x0006,     // N x 0.125ms
			ConnLatency:           0x0000,     //
			SupervisionTimeout:    0x000A,     // N x 10ms
			MinimumCELength:       0x0000,     // N x 0.625ms
			MaximumCELength:       0x0000,     // N x 0.625ms

We need to preserve peer type during the advertisement parsing, so that we know which to be specify when we send the created command.

Fixed by #26


Selecting a new engine from App Options

The program needs to be restarted for the change to take effect.

Duplicate


[WIP] All-in-one Doc Updates

This PR contains updates to our documentation to include details about the AIO, as well as how to programatically deploy it.
This is not done, but uploading for comment. It needs to be converted from MD -> RST, but I didn't want formatting to stop getting the actual content out.
Early comments appreciated.

the direction (and what you have so far) is LGTM.
One thought it to flash out what IS all-in-one installer (takes some reflection) my take is: it's a "production ready reference deployment scaled down to one box for easy evaluation, simple production, and learning how stackstorm components are deployed / wired together)". @jfryman think about it and do your take. if we don't have time for this, np.

@dzimine will certainly add more, but in the dot release. going for MVP here. Great feedback though. Made myself a sticky note. :notebook:

awesome!
make sure it passes travis and +1

@jfryman hope you won't mind some minor structural changes; and you do please override. To me it looks good enough, when travis passes I'll merge and you can hack on top.
Scrutinizer seem to give false negatives.

@dzimine thanks for fixing. Was having some local issues building.


Album "Play" button is confusing

Hi,
first of all thank you very much for your awesome app! Excellent work!
Now to my problem... I find it very confusing that when you open an album and hit the play button in the bottom right corner of the album cover it starts playing a random song from the album instead of the first song in the list. (Shuffle is deactivated)
Also is there a way to see the current titles of the playlist? When I started playing an album and then leave the album page I still can iterate through the album's song using the next and previous buttons, but can't see the list of songs in the playlist?
Thanks!

Hi @CramericaIndustries,
Thanks for your bug report! I'm able to reproduce it easily.
Regarding displaying the current playlist as well as its title, I'm still wondering where it should be in the UI.
Do you have any suggestions?

Do you have any suggestions?

I would put it in the window with the "play","pause","next","prev" controls which slides up from the bottom if you hit the small arrow icon. Looks like so:

The modified view with the playlist could look like so maybe?

Where the list is basically the same (scrollable) view as on the Album page, just with the additional controls and the title of the song which is currently playing.


Filter list of legend items from merit order charts

The legend items needs for the merit order chart need to be filtered. Some carriers may or may not be present in the chart.
--
Don't mind the last green bar.. my mouse was in the way again.

Fixed by here


feature: 构建依赖升级 & title 补充 (closed #197)

Codecov Report

:exclamation: No coverage uploaded for pull request base (v1.0.19-rc@521af9a). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             v1.0.19-rc     #198   +/-   ##
=============================================
  Coverage              ?   53.64%
=============================================
  Files                 ?      182
  Lines                 ?     8955
  Branches              ?        0
=============================================
  Hits                  ?     4804
  Misses                ?     4151
  Partials              ?        0

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


chore: CI

Signed-off-by: George Aristy<EMAIL_ADDRESS>

@troyronda I think you need to set up an azure pipeline for this repo


android 7.0 text relocations

Detected problems with app native libraries
libutility.so:text relocations
how to fix it?

06-13 11:59:20.240 10165-10165/com.mabeijianxi.smallvideo W/linker: /data/app/com.mabeijianxi.smallvideo-1/lib/arm/libutility.so has text relocations. This is wasting memory and prevents security hardening. Please fix.

可尝试使用2.0新版


RDP example

Could you please provide an example to open rdp session and do automation after login.

Hmm... How do you imagine that? Information about UI controls are not provided by RDP. But you may run the script on remote machine using remote PowerShell psexec with cmd key -i (interactive). Of course, need to copy the script to the remote machine before starting it. See issue #401 for more details. Closing as duplication.

One more useful guide: https://support.smartbear.com/testcomplete/docs/testing-with/running/via-rdp/keeping-computer-unlocked.html


Geocodio fixes not in stable package

PR #21 fixed some issues with the Geocodio provider, but it looks like it was only merged into the 1.x branch. Is it possible to get this merged into a tagged release?
Let me know if there are any questions, or if my understanding is incorrect. Thanks!

@willdurand Looks like this is still the case as the 1.x branch is 4 commits ahead of master?

ahm maybe yes. so we should:

tag a version for 1.x
and then merge 1.x into master (which might be a little bit tricky)

@willdurand Tagging 1.x retrospectively is a good idea. Maybe a cherry pick merge for point 2 might work though I'd assume this depends on how much drift there's been since 1.x was last updated.

Lets cherry pick the commit and put it to master.

Fixed in #46


Fix the swagger for the GET request to list all players

Hi Erin,
Just to remember to fix the swagger for the GET request to list all players to include the type that will be returned , so I can see the information (id) in response.
Should be this guy coming back in the list:
PlayerArgument {
_id (string): Unique player id ,
name (string): Player name ,
favoriteColor (string): Favorite color
}
Thanks,

We're all set. Thanks for opening!


wikitext_dataset fails in model forward with "Expected Long, Int; but got torch.cuda.FloatTensor"

Thanks to @felipemello1 for pointing this out.
Repro: tune run lora_finetune_single_device --config llama3/8B_lora_single_device max_seq_len=4096 dataset=torchtune.datasets.wikitext_dataset gradient_accumulation_steps=2 log_every_n_steps=1 log_peak_memory_stats=True max_steps_per_epoch=10 epochs=1
Because the wikitext dataset (https://huggingface.co/datasets/Salesforce/wikitext) splits samples by line in an article, some of the rows will be empty, leading to entire samples with pad_id. Ideally, these rows should just be dropped.
But this leads to our bug - if the first sample is empty, pad_sequence in padded_collate will actually return a FloatTensor instead of keeping the ints (https://github.com/pytorch/torchtune/blob/main/torchtune/utils/collate.py#L46). You can repro this adding pdb.set_trace() above the linked code and continuing until you see a batch where the first sample is empty.
The fix is twofold:

Enforce int tensors with long() in collate
Use the document level version of wikitext: https://huggingface.co/datasets/EleutherAI/wikitext_document_level, which is a cleaner dataset

Can you just modify wikitext_dataset itself to skip empty sequences? My concern with long is that it won't respect the int type used by the other tokens.

addressed in #1258


Build multi-arch charm on amd64 architecture only

Don't build the LXD charm on riscv64 architecture based runners since a stable snapcraft snap isn't yet available on those systems.
Instead build the multi-arch charm for both 20.04 and 22.04 on amd64 runners. The same approach was taken by the Keystone charm and is explained here https://discourse.charmhub.io/t/charmcraft-bases-provider-support/4713.
This also allows us to enable riscv64 support.

@roosterfish I'm afraid the charm-lxd contains binary files (cryptography is one of those IIRC) so we cannot build on one arch and use on all others. It would have been nice though.
I think we either have to:

maintain the status quo (build failure reports on riscv64 until the Charmcraft team cuts a release with https://github.com/canonical/charmcraft/pull/1406 in it)
list all the working architectures now and remember to add riscv64 back in when snap install charmcraft works on that platform

WDYT?

@roosterfish I'm afraid the charm-lxd contains binary files (cryptography is one of those IIRC) so we cannot build on one arch and use on all others. It would have been nice though.
I think we either have to:
1. maintain the status quo (build failure reports on `riscv64` until the Charmcraft team cuts a release with [feat: enable riscv64 support charmcraft#1406](https://github.com/canonical/charmcraft/pull/1406) in it)

2. list all the working architectures now and remember to add `riscv64` back in when `snap install charmcraft` works on that platform

WDYT?

Thanks for the quick check. The topic of having binary files in the charm made me look a bit closer into the code and I found this https://github.com/canonical/charm-lxd/blob/main/.github/workflows/tests.yml#L57. May it be that those extra requirements are only added during the test suite? Maybe I am missing something crucial here but I thought if the final charmcraft.yml doesn't have the charm-binary-python-packages: entry we are good to go.

The test suite is failing since the multi-arch charm has the following name lxd_ubuntu-20.04-s390x-ppc64el-arm64-armhf-amd64-riscv64.charm.

...The topic of having binary files in the charm made me look a bit closer into the code and I found this https://github.com/canonical/charm-lxd/blob/main/.github/workflows/tests.yml#L57. May it be that those extra requirements are only added during the test suite? Maybe I am missing something crucial here but I thought if the final charmcraft.yml doesn't have the charm-binary-python-packages: entry we are good to go.

It's complicated because charms are normally built from source but that is terribly slow and a bit pointless when there are wheels available for the given arch. To make testing quicker, I hacked this quick and dirty solution which works fine for GitHub Action runners that are amd64 and thus have all the wheels readily available. However, this doesn't work on more exotic arches like riscv64 where very few wheels are available and we thus need to build pyca/cryptography to satisfy the pylxd dependency.

@simondeziel Ok thanks for the explanation, then let's stick with 1. as discussed. Will close this now.


language support for Traditional Chinese (zh-tw)

language support for Traditional Chinese (zh-tw)

Could you also translate the additional text? Would be great!

Surely. Will do it!
L.J.
On Fri, Mar 11, 2022 at 5:41 PM Sören Weber @.***>
wrote:

Could you also translate the additional text? Would be great!
—
Reply to this email directly, view it on GitHub
https://github.com/McShelby/hugo-theme-relearn/pull/215#issuecomment-1064941871,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAO3OUFVREJDM2LAG27WKXLU7MILDANCNFSM5QPBWDJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Separate new-syntax contracts with linebreaks

Separate new-syntax contracts with linebreaks.
Implements #331 in a mostly unintrusive way (as far as I can tell).

The sensible way to do this would probably to make this a list, but lists are nasty. Manually adding the line break doesn't really cut it, because when you start breaking, you want to either break the initial contract or to align the break on the initial contract.
Plus, this can be mixed up with a template constraints, attribute and qualifier, etc... This is not something I think we should add special case for, because special cases will call for special cases.
The way to go clearly is to use the list formatting, but I'm not sure how easy or hard this is. I suspect it might be hard.  Also on a side note, I plan on revamping the way list work int he formatter, because in addition to being nasty, they have some pathological performance characteristics in extreme cases (like say, an array literal with thousands of elements in it). the root of the issue is that the formatter is actually completely unaware of lists;, they are implemented via spans, so it has to try all kind of combinations of break that the span is going to subsequently shut down.

This is not something I think we should add special case for

From what I’ve seen while implementing this, we’ll need a more sophisticated approach then.
Without special-casing this to new-syntax contracts one’d end up with very intrusive reformatting.

The sensible way to do this would probably to make this a list, but lists are nasty. Manually adding the line break doesn't really cut it, because when you start breaking, you want to either break the initial contract or to align the break on the initial contract.
Plus, this can be mixed up with a template constraints, attribute and qualifier, etc... This is not something I think we should add special case for, because special cases will call for special cases.
The way to go clearly is to use the list formatting, but I'm not sure how easy or hard this is. I suspect it might be hard. Also on a side note, I plan on revamping the way list work int he formatter, because in addition to being nasty, they have some pathological performance characteristics in extreme cases (like say, an array literal with thousands of elements in it). the root of the issue is that the formatter is actually completely unaware of lists;, they are implemented via spans, so it has to try all kind of combinations of break that the span is going to subsequently shut down.

This would be good dogfood for improving that kind of feature. This PR is quite ugly both in terms of aesthetics and upsetting grug e.g. a counter inside the parser is not great.
For the simple cases of lists I think you might be able to get a very long way with a good heuristic as long as you can prepare the head and tail right e.g. if you can get right rhythm going then I think you could lay out the bulk of the list (e.g. consider the case of lots of an array of struct initializers) geometrically rather than by cost.


Don’t block on write if we’ve been killed

Hi - I've been debugging this issue and fixed it with the changes in this pull request. I'm a bit new to go - so forgive me If I misunderstand, but here's what happened.
Quick context first: I am using hpcloud/tail in my component, called "CsvTail" which tails a file, parses the CSV records in that file, and only stops when it encounters the NUL byte at the beginning of a line. The CSV files are streamed and written by another service, which uses the NUL byte to indicate the end of the file.
The code would sometimes deadlock when calling tail.Stop(). Stop() internally calls tail.Wait(), which waits for the tomb to be declared as dead.
What happens is that another line comes in after my read of the NUL byte, but before my Stop() call, causing the tail library to block on this line.
Tail would never reach the end of tailFileSync() where it checks for tail.Dying(), returns, and sets the tomb as Done().
My fix is to add a select and "cancel" the write if the tomb enters dying state.
Not sure if this is right though let me know if I'm misunderstanding or misusing the library.
Best,
Paul

Sadly, this repo seems dormant for more than 1,5 year. I forked it because I use it in my code. I integrated the PRs waiting merging and refreshed the code. A PR was recently merged that seems to address the same issue: https://github.com/nxadm/tail/pull/7
There were some other fixes that prevent not showing the last line in the buffer while waiting on a newline. If you have the time, please do review the changes.

Ah ok. Thanks for update. I wish the owner of the hpcloud repo would add a note in the readme to point to yours!

FYI I tried switching to your fork @nxadm, but encountered an issue where I would get incomplete lines sometimes so had to switch back. I wish I could give more detail but don't have time to debug further right now.

FYI I tried switching to your fork @nxadm, but encountered an issue where I would get incomplete lines sometimes so had to switch back. I wish I could give more detail but don't have time to debug further right now.

After the merging of the PRs all tests passed. Once you have the time I would love know which kind of logs trigger this error (e.g. what is a incomplete line). It may be related to the fix of the bug where the last line of a log wasn't sent until new messages arrived, e.g. because this last line didn't have a new line). Feel free to contact me once you have the time.

Reviewing the code, I saw the change you proposed is already the nxadm/tail repo, probably by another PR. If you find the time, please test if the problem still occurs, and if it does please reopen an issue at https://github.com/nxadm/tail/issues (if possible with the data file and a small test).

Reviewing the code, I saw the change you proposed is already the nxadm/tail repo, probably by another PR. If you find the time, please test if the problem still occurs, and if it does please reopen an issue at https://github.com/nxadm/tail/issues (if possible with the data file and a small test).


Alacritty doesn't seem to be working with wal

I installed Void Linux today and finally managed to get pywal working (it only has py3.5 so I compiled 3.6.1 with pyenv) and since it doesn't have Termite (my go-to terminal) in it's repos I decided to try Alacritty (it's supposed to be fast).  The printf "%b" "\033]11;#ff0000\007" does nothing so I guess you should add it to the list. I tried to make it work with wal by having a config.yml that contains non-color options and an auto-generated colors.yml that has colors (used termcolors to convert to Alacritty's format), what I did was add this function:
function fix_alacritty() {
	local dir="${HOME}/.config/alacritty"
	rm "${dir}/alacritty.yml"
	cat "${dir}/config.yml" "${dir}/colors.yml" >> "${dir}/alacritty.yml"
}

And call it in the script that gets executed after wal, turns out Alacritty needs everything to be in specific positions for the thing to work (it is possible with some mad regex-foo or yaml parsing, both of which I'm no good at and not going to try doing since the benefits aren't that good, not being able to change colors on the fly is a bummer)
So I guess it's not really an issue with wal but I'm just letting you know

Thanks for doing all of this testing!
We could try and open an issue on Alacritty's bug tracker, I did a quick search and couldn't find anything.

Issue opened on Alacritty's repo: https://github.com/jwilm/alacritty/issues/656

Alacritty is now supported! :D

So, how would I add wal to alacritty? It does work, but I need to re-run wal every-time I open a new alacritty instance.

You should be able to just add wal (and whatever parameters you pass to it) to your shell's rc file, for Bash it's ~/.bashrc.

So what you're telling me is that I should just add the wal command to my shell rc file?
(e.g. wal -i /path/to/wallpaper in .zshrc)

Something like that. Maybe only do it when running in a graphical session, I haven't used wal in a long time so I can't remember what I used to do.

It worked! Thanks.

[MacOS] I've recently installed alacritty, and pywal doesn't seem to be working! Am I missing something?
I've got the command (cat ~/.cache/wal/sequences &) in my .zshrc file
Whenever I try to wal -i /path/to/wallpaper it just does nothing.

I have no way of debugging macOS. Alacrity works just fine on my Linux install.
Are you using the latest version of Alacritty?

Just reinstalled Alacritty to make sure I have the latest version but the same problem still arises. Whenever I use wal -i it reads the wallpaper but the colors never change. Is there a log file or something alike that I can check to know more details about the problem?

What version of alacritty are you using?
On Sun, 11 Mar 2018 at 9:45 am, Francisco Santos<EMAIL_ADDRESS>wrote:

Just reinstalled Alacritty to make sure I have the latest version but the
same problem still arises. Whenever I use wal -i it reads the wallpaper
but the colors never change. Is there a log file or something alike that I
can check to know more details about the problem?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/dylanaraps/pywal/issues/37#issuecomment-372073242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGfAa7TafmYtGebnOH2uhpknoLqOFNJHks5tdFdzgaJpZM4ORbp9
.

I'm using the latest on their repo, can't really give you a number as I don't know any command the shows me the Alacritty version nor have they got any releases or tags on the repo.

This is still a problem with a fairly recent (march 01) version of Alacritty on macOS.
Oddly enough printf "%b" "\033]11;#ff0000\007" does change the background to red, but cat sequences does nothing. Running wal -i <file> changes the colors of iTerm2 windows but not Alacritty.

It might be worth opening a bug on their tracker. Alacritty works fine on Linux.

I just wrote a workaround that may help Mac users who want wal colors in Alacritty: https://github.com/egeesin/alacritty-color-export

@egeesin this is confirmed working for me on MacOS 10.15.2, alacritty 0.3.3

@dylanaraps if you don't mind, I'd like to add this to the Wiki as a section under the terminal customization, something like "Alacritty on MacOS".

Go ahead! :+1:

Done 😃


Create basic block from instructions

This change is

Are we squashing the commits (or rebasing the branch) to avoid the "WIP" commit? 😊

Yes, always squash when completing a PR to encourage small, focused PRs ;)


[BUG] Review all recommend_top_k algos

Description

There are some inconsistencies between the implementations of recommend_top_k. In SAR we are using the users of the test set whereas in SVD we are using the users in the train set. Sometimes they can be the same, but not in general.
Also, as suggested by @yueguoguo, there is a function to optimally remove the seen items, but each algo use its own function

this is a good catch, pretty much all the approaches can work the same if set to score the set of user-item pairs (generated from the crossjoin of unique users and items in the test set, with seen items removed). I think we do that once (using the method @yueguoguo mentioned) in the benchmark script, and then adjust the recommend_k_x methods to call the algo's score method then rank the results.
Actually, can we use the same results from the rating stage (assuming we're generating the same test set described above) and just rank them?

I think we do that once (using the method @yueguoguo mentioned) in the benchmark script, and then adjust the recommend_k_x methods to call the algo's score method then rank the results.

agree

We need to take a look at the fastai function when we are removing the seen items. This can be improved
        total_users, total_items = model.data.train_ds.x.classes.values()
        total_items = total_items[1:]
        total_users = total_users[1:]
        test_users = test[DEFAULT_USER_COL].unique()
        test_users = np.intersect1d(test_users, total_users)
        users_items = cartesian_product(test_users, total_items)
        users_items = pd.DataFrame(users_items, columns=[DEFAULT_USER_COL, DEFAULT_ITEM_COL])
        training_removed = pd.merge(users_items, train.astype(str), on=[DEFAULT_USER_COL, DEFAULT_ITEM_COL], how='left')
        training_removed = training_removed[training_removed[DEFAULT_RATING_COL].isna()][[DEFAULT_USER_COL, DEFAULT_ITEM_COL]]

A better practice (from an ML perspective) would be to remove the training data and the remove_seen flag from the recommend_k() methods and input the set of available items instead. This set would need to be computed outside the method.
A further option is to input users and items instead of a single dataframe. That would be more convenient to the user if they want to do something other than "remove seen" or "include_seen".

If recommend_k operates only on input users then retaining remove seen as an option makes sense. This may only apply to algos which maintain the full set of items internally.
Pregenerating the user-item pairs is fine but should go through a score method not recommend_k.

yeah, maybe a quick benchmark on which is the fastest way to generate all user-item pairs df. then update dataset.pandas_df_utils.user_item_pairs.  we can also leverage common.python_utils.get_top_k_scored_items to speed up getting recommended items.

If recommend_k operates only on input users then retaining remove seen as an option makes sense. This may only apply to algos which maintain the full set of items internally.
Pregenerating the user-item pairs is fine but should go through a score method not recommend_k.

My point was not about implementation, but about the interface we expose to the data scientist. I am proposing that this is very simple i.e. something like
recommend_k(set of users, set of items)
and recommend_k() outputs the top-k items among set_of_items per user.

I like the approach of @anargyri, that way we are making the code more modular. Are you thinking of something like this:
df = remove_seen_items(df_original)
preds = model.recommend_k_items(df)

More like
unseen_items = remove_seen_items(df_original)
users = df_original[usercol].unique()
preds = model.recommend_k_items(users, unseen_items)

The idea is to be able to cover other situations as well. For example, the data scientist may want to recommend only to a subset of users (e.g. the "warmer" ones). Or they may want to exclude certain items (e.g. not show Chinese movies to users from France).

i'm confused, aren't the unseen items only unseen for a specific user? this sounds like excluding sets of items across all users.
I was thinking that you may only want to recommend novel items for a user, in which case you would need to know which item-user pairs were in the training set and remove them from the user-item pairs of the test set

i'm confused, aren't the unseen items only unseen for a specific user? this sounds like excluding sets of items across all users.
I was thinking that you may only want to recommend novel items for a user, in which case you would need to know which item-user pairs were in the training set and remove them from the user-item pairs of the test set

Yes, you are right. I am thinking a bit more generally than the case of seen-unseen items, for example, remove Chinese movies for French users and French movies for Chinese users. I guess Miguel was also thinking of the same thing.  Maybe my syntax is a bit too complicated because the items would be a dictionary of lists. Miguel's syntax with dataframes is probably better.

I think it might be more user friendly to have the "list of the users" as input to function recommend_k_items. In this way, it might be also useful to enhance the computation efficiency, as the user list can be used for subsetting the recommended user-(k)-item pairs. It is also easy to maintain the list of users instead of a dataframe that contains user, item, and probably rating columns.
Currently we have the basic functions like cross join of user-item pairs, with which we can wrap up for performing the "removing seen" operations. It takes dataframes as input however, so we can probably think of implementing something new?

I think we should consider using a data frame, with the same columns as in the other functions, users, items...
It is more consistent with the rest of the code and hopefully it's easier to understand by the user

I think we should consider using a data frame, with the same columns as in the other functions, users, items...
It is more consistent with the rest of the code and hopefully it's easier to understand by the user

Closing old issue, feel free to reopen if needed.


chore(@toss/utils) : Modularize isIOS and isAndroid Function into a Dedicated Module for Enhanced Code Maintainability

Overview

What I Did

By modularizing isIOS and isAndroid, we avoid duplication and centralize the logic for iOS platform checks.
I write test code about isIOS and isAndroid.

Proposal
This change will allow easier updates and improvements to the isIOS function and isAndroid function in the future.
PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

Thank you for your feedback.
i added test code for isIOS and isAndroid.


Redo Checkbox using tutorial

Currently the checkbox used in the BottomBar is styled using the appearance css property, but it is still pretty new and might not behave uniformly in all browsers. We could redo the checkbox using this tutorial https://youtu.be/LMir4WUdz6U

No longer working on this due to relaunch.


fix: Fix tuple- and record struct completions not working with existing braces

bors r+

bors retry


Question: Is it possible to create a new class from an attribute that targets methods?

For simplicity assume my goal is to have an attribute on a method that generates a new class whose name is the methods names followed by "Wrapper".
For example if I have an attribute:
    [AttributeUsage(AttributeTargets.Method)]
    [CodeGenerationAttribute("FunctionGenerator.TestGenerator, FunctionGenerator")]
    [Conditional("CodeGeneration")]
    public class TestGeneratorAttribute : Attribute { }

And a class:
public class TestClass
{
    [TestGenerator()]
    public void FooBar() { }
}

I would want to generate the following class:
public class FooBarWrapper
{
    /*Eventually useful code would go here*/
}

Of course I tried this with my crude understanding of code generation using this code in my generator:
public Task<SyntaxList<MemberDeclarationSyntax>> GenerateAsync(TransformationContext context, IProgress<Diagnostic> progress, CancellationToken cancellationToken)
{
    MethodDeclarationSyntax method = context.ProcessingNode as MethodDeclarationSyntax;

    var wrapper = SyntaxFactory.ClassDeclaration($"{method.Identifier}Wrapper");

    var results = SyntaxFactory.SingletonList<MemberDeclarationSyntax>(wrapper);
    return Task.FromResult(results);
}

Which generates:
/*Using statements and namespace removed for brevity*/
public class TestClass
{
    class FooBarWrapper
    {
    }
}

This is close to what I want but being nested under TestClass causes a build error because I now have 2 definitions of TestClass. I feel like there must be a way to accomplish my goal of creating a completely new (ie. not a class nested in another class).
Is this possible? If so, how?
I really appreciate any help on this 😄

Yes, it's possible. We have a simple codegen endpoint which is apparently what you tried, which puts you in the same (partial) class. But we have a more "I own everything" endpoint that lets you generate the entire CompilationUnit (source file).
The API has changed a lot though, so @amis92 is probably the best one to point you to the right API. I don't see it mentioned in the README.

Hi,
what you want to do is indeed doable as @AArnott guessed.
Suggestion
But first, let me suggest to you the new C# Source Generators feature shipping with .NET 5/C#9: https://devblogs.microsoft.com/dotnet/introducing-c-source-generators/
The development experience currently is actually a bit worse compared to CG.R, but the future is there (we won't be pursuing CodeGeneration.Roslyn development any further).
Answer
Indeed there's no mention in README of the required feature - it's only described a little bit in https://github.com/AArnott/CodeGeneration.Roslyn/wiki/Features#irichcodegenerator

You want to implement the IRichCodeGenerator interface.
Your ICodeGenerator.GenerateAsync can throw, it won't be invoked if IRichCodeGenerator is implemented.
Your IRichCodeGenerator.GenerateRichAsync implementation must return the RichGenerationResult which will be merged with other generator's output (if any).
Your generated result (RichGenerationResult.Members) will no longer be "wrapped" into triggering attribute's parent type+namespace, you will need to do that yourself.


Unable to upload key when using 'Az' module

Step 5 no longer works when using the 'Az' module if no existing application or user permissions exist on the Key Vault.
Context: Uploading a key transfer package (BYOK) to a tenant for use with AIP. AIP has not yet been installed or configured.
Previously, when using BYOK with the old 'AzureRM' module, it was possible to upload a key transfer package to Key Vault without any dependent applications installed and ready to use the key. Now, it would appear that in order to upload the key at least one user account or Service Principal is required to be registered with permissions against the Key Vault before the key transfer package can be uploaded. Can you confirm?
Steps to reproduce:
Create tenant key and key package as per steps in this article.
Create Resource Group - "New-AzResourceGroup -Name 'ContosoGroup' -Location 'northeurope'"
Create Key Vault - "New-AzKeyVault -VaultName 'ContosoKeyVault' -ResourceGroupName 'ContosoGroup' -Location 'northeurope' -SKU 'Premium'"
Output from above Key Vault command shows:
WARNING: The provided information does not map to an AD object id.

Vault Name                       : ContosoKeyVault
Resource Group Name              : ContosoGroup
Location                         : northeurope
Resource ID                      : /subscriptions/XXXXXXXX-27f4-XXXX-abbd-e0bbdcd00240/resourceGroups/ContosoKeyVault/provid
                                   ers/Microsoft.KeyVault/vaults/ContosoKeyVault
Vault URI                        : https://contosokeyvault.vault.azure.net/
Tenant ID                        : XXXXXXXX-94be-XXXX-9ecb-25757e0d402a
SKU                              : Premium
Enabled For Deployment?          : False
Enabled For Template Deployment? : False
Enabled For Disk Encryption?     : False
Soft Delete Enabled?             :
Access Policies                  :
Network Rule Set                 :
                                   Default Action                             : Allow
                                   Bypass                                     : AzureServices
                                   IP Rules                                   :
                                   Virtual Network Rules                      :

Tags                             :

WARNING: Access policy is not set. No user or application have access permission to use this vault. This can happen if
the vault was created by a service principal. Please use Set-AzKeyVaultAccessPolicy to set access policies.

Trying to upload the key package with "Add-AzKeyVaultKey -VaultName 'ContosoKeyVault' -Name 'contosokey' -KeyFilePath 'C:\path to key package
KeyTransferPackage-contosokey.byok' -Destination 'HSM'"
Results in:
Add-AzKeyVaultKey : Operation returned an invalid status code 'Forbidden'
At line:1 char:1
+ Add-AzKeyVaultKey -VaultName 'ContosoKeyVault' -Name 'contosokey' -KeyFilePat ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : CloseError: (:) [Add-AzKeyVaultKey], KeyVaultErrorException
    + FullyQualifiedErrorId : Microsoft.Azure.Commands.KeyVault.AddAzureKeyVaultKey

Trying to add permissions for the AIP SPN thus: "Set-AzKeyVaultAccessPolicy -VaultName 'ContosoKeyVault' -ServicePrincipalName 00000012-0000-0000-c0
00-000000000000 -PermissionsToKeys decrypt,sign,get"
Results in:
Set-AzKeyVaultAccessPolicy : Cannot find the Active Directory object '' in tenant
'XXXXXXXX-94be-XXXX-9ecb-25757e0d402a'. Please make sure that the user or application service principal you are
authorizing is registered in the current subscription's Azure Active directory. The TenantID displayed by the cmdlet
'Get-AzContext' is the current subscription's Azure Active directory.
At line:1 char:1
+ Set-AzKeyVaultAccessPolicy -VaultName 'ContosoKeyVault' -ServicePrincipa ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : CloseError: (:) [Set-AzKeyVaultAccessPolicy], ArgumentException
    + FullyQualifiedErrorId : Microsoft.Azure.Commands.KeyVault.SetAzureKeyVaultAccessPolicy

In our work with customers, there are various times when we will perform a BYOK engagement and upload a BYOK key to a new Key Vault PRIOR to the service that will make use of that key being installed/configured/available. This now appears no longer to be possible.
Can you advise of any workarounds or other methods such that a key transfer package can be uploaded to Key Vault even though the application/service that will make use of that key is not yet installed?

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

ID: dd8f9c6c-9980-9774-6e13-21e4f23a1d8f
Version Independent ID: 9f60034d-d0c0-e56c-c1a9-c215b6a416c3
Content: How to generate and transfer HSM-protected keys for Azure Key Vault - Azure Key Vault
Content Source: articles/key-vault/key-vault-hsm-protected-keys.md
Service: key-vault
GitHub Login: @barclayn
Microsoft Alias: barclayn

Update to the issue:
As the owner of the tenant, I would assume as I created the Key Vault then I would have full permissions. This doesn't appear to be the case. It is necessary to add my tenant owner account to the Key Vault 'access policy' with full permissions. Once this is done, I can then upload the BYOK key package.
This seems counter intuitive. Surely as owner of the tenant and creator of the Key Vault, I also get access to it to upload keys? Not without adding myself to the 'access list' it would seem.

@MPenny-nCipher Thanks for your feedback! We will investigate and update as appropriate.

@MPenny-nCipher Thank you for your message. I consulted with one of the Key Vault team PMs. He said
"This has nothing to do with new or old Az or AzureRm module. The issue here is that the user creating the Key Vault, does not Azure AD graph permissions to query and get the OID for the current user, which results in a key vault created without any default access policy. That is the warning. There has been no change in BYOK toolset."

I am aware that there has been no change to the BYOK toolset. However, prior to the 'Az' module change, it was possible to create Key Vaults and upload BYOK packages without having to add or modify any permissions on a Key Vault.
Now, this may not be as a result of the 'Az' module change (the last time I created a new Key Vault and uploaded a BYOK package was prior to the 'Az' module change and subsequent modifications to this page) but something has changed in the interim.
If this behaviour is now expected then we will need to add the extra Key Vault permissioning steps on engagements.
Thanks for investigating this.

@MPenny-nCipher your welcome. Please let us know if anything else comes up.

#please-close

@MPenny-nCipher We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

I am running into the same issue moving from AzureRm to Az. Very weird that I login as the subscription owner and cannot create a KeyVault without gymnastics.  The documentation for New-AzKeyVault should at the very least do something to address this issue.

...does not have Azure AD graph permissions to query...

Ok. How does one get Azure AD graph permissions?


Schedule Chiado testnet Deneb fork (expected on Jan, 31)

Description
Chiado network will be transitioned to Deneb hardfork on January, 31 2024
https://github.com/gnosischain/configs/pull/26
Target blobs per block is 1, so it's expected that required blobs will take up about 35G of storage
Gnosis and Chiado deneb presets: https://github.com/gnosischain/specs/blob/master/consensus/preset/gnosis/deneb.yaml
Deneb config lines: https://github.com/gnosischain/specs/blob/master/consensus/config/gnosis.yaml#L128-L136
Trusted setup: https://github.com/gnosischain/specs/pull/30

Thanks @zilm13 for opening the issue! To complement your details:
Chiado Dencun fork is scheduled
+DENEB_FORK_VERSION: 0x0400006f
+DENEB_FORK_EPOCH: 516608 # Wed Jan 31 2024 18:15:40 GMT+0000

https://github.com/gnosischain/configs/pull/26

Some config values differ from mainnet
MAX_REQUEST_BLOCKS_DENEB: 128
# MAX_REQUEST_BLOCKS_DENEB * MAX_BLOBS_PER_BLOCK
MAX_REQUEST_BLOB_SIDECARS: 768
# `2**14` (= 16384 epochs, ~15 days)
-MIN_EPOCHS_FOR_BLOB_SIDECARS_REQUESTS: 4096
+MIN_EPOCHS_FOR_BLOB_SIDECARS_REQUESTS: 16384
# `6`
BLOB_SIDECAR_SUBNET_COUNT: 6

No preset values differ from mainnet
FIELD_ELEMENTS_PER_BLOB: 4096
# `uint64(2**12)` (= 4096)
MAX_BLOB_COMMITMENTS_PER_BLOCK: 4096
# `uint64(6)`
MAX_BLOBS_PER_BLOCK: 6
# `floorlog2(get_generalized_index(BeaconBlockBody, 'blob_kzg_commitments')) + 1 + ceillog2(MAX_BLOB_COMMITMENTS_PER_BLOCK)` = 4 + 1 + 12 = 17
KZG_COMMITMENT_INCLUSION_PROOF_DEPTH: 17

https://github.com/gnosischain/specs/blob/master/consensus/preset/gnosis/deneb.yaml


fix: add conditional to database different postgres

Signed-off-by: Ana Luiza Ramalho dos Santos<EMAIL_ADDRESS>Hey, I just made a Pull Request!
When the database is MySQL the ".returning()" on the end of the query didn't work to this type so I add a conditional to fix it.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@freben gonna punt this to you! :pray:

reminder to preferably put a test in place too


Engine API subset for the Merge Interop

Introduces Engine API subset for the Merge Interop

I've started updating our el implementation of it to this spec here: https://github.com/ethereum/go-ethereum/pull/23607


Constructors for types with no arguments

I want to get rid of these lines:
https://github.com/puffnfresh/daggy/blob/master/src/daggy.js#L113-L117
Otherwise you can't add a .map to Maybe.Nothing.prototype.

I think that breaks quite a few things, so will have to check. You can work
around it by making the object aliases later.
const Maybe = extend(maybe, {Nothing, Some})

On Thu, 31 Mar 2016, 06:23 Chet Corcos<EMAIL_ADDRESS>wrote:

I want to get rid of these lines:
https://github.com/puffnfresh/daggy/blob/master/src/daggy.js#L113-L117
Otherwise you can't add a .map to Maybe.Nothing.prototype.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/puffnfresh/daggy/issues/5

I have this same issue. @SimonRichardson is extend defined somewhere?

Not that I'm aware of... Was just an idea, maybe it might be worth while adding, although you should add the prototype to the Maybe type and not each sum, then use the catamorphism to identify it.

Why you need to add value to Nothing? I believe, Nothing is just a value, and if you want to define some method on it you can just define directly
Nothing.map = (f) => Nothing

Think this is closed now by @safareli comment. Re-open if you've got questions.


videoTrack.getCapabilities is not a function in Firefox

Accroding to https://developer.mozilla.org/en-US/docs/Web/API/MediaStreamTrack/getCapabilities#browser_compatibility, videoTrack.getCapabilities is not supported in Firefox.
When I tried to open camera in Firefox, the console log will throw Uncaught (in promise) TypeError: videoTrack.getCapabilities is not a function

Is it able to set a indicator in useZxing to bypass torch init? or any workaround to skip this error?
Thank you.

This should already be fixed in 1.1.3


Sometime Result incorrect  &  scan text not only barcode .How can I fix this.

Sometime Result incorrect  &  scan text not only barcode .How can I fix this.

I also having same issue some time I get wrong barcode text.  @avaneeshkumarmaurya    plz help brother. my ERP app already used in 500+  stores.


Add named argument so scalebar can be used in FXML

Add named argument to scale bar constructor so it can be used from FXML e.g.
<MapView fx:id="mapView" prefHeight="400" prefWidth="400"/>
<Scalebar fx:id="scalebar" mapView="$mapView"/>

without this it isn't possible to use the scale bar in FXML as it has no no-arg constructor.

@Rachael-E please review. Allows use of scalebar in FXML.

@tschie can you do a second review please?


Issue 342 - Deploy with Docker

Resolves #342
This requires first using the build script outside of Docker to create /scripts/jars and /scripts/docker directories. Once that's been done the system can be deployed from inside Docker.
The other Docker images are not currently built as part of the Maven build. They're still built in the pre-deployment phase, that hasn't changed. Other issues will follow up on this.

Still need to update the utility scripts to stop them referencing the scripts directory.


kill aura

title.

ghost client, not adding killaura.

rather not ghost client, i dont want to make this client combat focused, theres way better clients out there that probably have a better killaura than what i can do. i want this client to stay rather render and player focused.


On renew error, there is no way to know subscription id for which this event was emitted

I want to maintain the subscription details of all the subscriptions I have created using my app.
I am not using the cache provided by this SDK.
Instead, I am hooking things with the redux store and I am persisting the redux store details in local storage on every action to hydrate the state on page reloads.
Assuming, I leave the session ideal for a few hours eventually, my auth token expired.
This causes the SDK subscription renewal to fail.
The challenge I am facing is when there is a renewalError event published, the SDK clears the subscription details of the object
https://github.com/ringcentral/ringcentral-js/blob/master/subscriptions/src/subscription/Subscription.ts#L205
I am unable to find a way to clear the subscription from my redux store/self-implemented cache because I do not get to know which subscription Id failed to renew
My code is as follows
import * as RingCentral from '@ringcentral/sdk'
import { Subscriptions } from "@ringcentral/subscriptions";

sdk = new RingCentral.SDK({
    server: serverUrl,
    clientId: appKey,
    clientSecret: appSecret,
    redirectUri: `${window.location.origin}/redirect.html`
});
platform = sdk.platform()
subscriptions = new Subscriptions({ sdk });
const subscription = subscriptions.createSubscription();
subscription.on(subscription.events.renewError, (data) => listener({source: subscription, event: subscription.events.renewError, data}))
await subscription.setEventFilters(eventFilters).register();

Somewhere else in the code
const listener  = () => ({source: subscription, event, data}) => {
if (event === subscription.events.renewError) {
    // Clear redux store
   // No way to know what subscription failed since subscription.subscription() returns empty object as it was removed before emitting this event in the SDK
  }
}

One potential solution is to form a closure and pass the subscription id while attaching the event listener but that feels counter-intuitive.
I think we should return the subscription Id/entire subscriptionData as part of the event data when renewal/subscription fails so that someone implementing their own persistence mechanism can do the cleanup

You can use https://github.com/ringcentral/ringcentral-js/blob/master/subscriptions/src/subscription/Subscription.ts#L120 to get current subscription info

@kirill-konshin
Calling subscription()  does not work since before emitting the renewError message, the sdk is clearing the subscription data. This means calling subscription()  after renewError has occurred returns an empty object
https://github.com/ringcentral/ringcentral-js/blob/master/subscriptions/src/subscription/Subscription.ts#L205
this.reset().emit(this.events.renewError, e);
reset will clear the subscription info leading to returning {} when called .subscription()

For anyone else reading this wanting to look for a potential workaround, you could create a closure to make sure you are maintaining the subscriptionId when renewError is emitted.
Flipping the order of emit (admitting renewError) and clear (clear subscription data) in the SDK could work but that might cause backward compatibility problems.
There are no plans to change this for now considering there is a workaround available


the sample doesn't work

the same dockerfile uses folders and that does NOT work, see here
https://developercommunity.visualstudio.com/t/Bug-in-Azure-devops-pipelineVS-template/1412270?space=21&ftype=problem&preview2=true&entry=myfeedback

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

ID: 3653642e-61ef-717b-5436-9bd0a3c4e6f4
Version Independent ID: 80c1ce51-46fb-0693-525a-7cad37150c7b
Content: Docker images for ASP.NET Core
Content Source: aspnetcore/host-and-deploy/docker/building-net-docker-images.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thank you for contacting us. Due to a lack of activity on this discussion issue we're closing it in an effort to keep our backlog clean. If you believe there is a concern related to the ASP.NET Core which hasn't been addressed, please file a new issue.