Keyboard Navigation (Accessibility)

Confirm whether all content is accessible through a keyboard interface and whether it requires specific timings for keystrokes? I.e. is it possible to navigate around and complete the task without ever using the mouse?
Whilst it appears possible to navigate around the site by keyboard only – it can be very difficult to see where you have navigated too. Sometimes there is a dotted outline but when there is a button with a black outline this becomes invisible and some of the icons have almost a white outline which likewise is very difficult to see.

Consider using a clear outline (yellow block is best) consistently to indicate to users where they have tabbed to make it easier for users to tell where they are.

Yellow has problems with contrast and some colour blindness types if you are concerned with accessibility light blue within the WCAG contrast guidelines is pretty much the standard

Any pointers on how to control the color of that outline?

light blue, it looks like some of the menus can be navigated by keyboard and have a light blue icon but a lot don't.

Changed focus now to LightBlue


Moving Models to a separated project help loosely-coupled.

When we want to an implement  on IClientStore ,IScopeStore  we need to reference to IdentityServer4 project with have many Hosting environment and MVC related packages. So having  separated project that contains interfaces and domain models help to have a more loosely-coupled  solution.

I don't see this happening, as we want the number of IdentityServer assemblies to be as few as possible.

Maybe this is my sensitivity that wouldn't like to add an assembly that brings MVC requirement (assemblies ) to a project (Store or CQRS  implementation) that doesn't needs MVC assemblies.

Hmm, I'm happy to see this changing

@leastprivilege to discuss

this adds complexity - and adds little value IMHO


#1511 #1507

#1511 Encoding list in the import dialog may appear empty
#1507 VS crash: ObjectDisposedException for MarshalingWindowFrame

:+1:


Fix incorrect error message for arguments with choices

If user provided an invalid value for an argument with a list of choices, then the error message will not indicate this. Rather it will indicate that cli command group is invalid. This is a quick fix and I am not sure that it doesn't break Knack. From my tests, this fix is better than removing overridden _check_value method.

Fixes #236

@fralik, Thanks a lot for the contribution and this is a nice finding!
The corresponding fix in Azure CLI is done by https://github.com/Azure/azure-cli/pull/6636. The change is later backported to knack by https://github.com/microsoft/knack/pull/146, but the logic for invalid argument value handling got lost during the backporting.
As the logic of _check_value in Azur CLI has become rather complex due to addition of new features (https://github.com/Azure/azure-cli/pull/16254, https://github.com/Azure/azure-cli/pull/16257), we need more time investigating and writing tests.

I have created a new PR https://github.com/microsoft/knack/pull/244 based on your commits so that you can still get credit and become a contributor. Please kindly see if the new PR satisfies your needs. Thanks.


Batch Size Setting

I am reaching out for clarification regarding the batch size setting in your paper. In the paper, you mention a batch size of 10 for training, while the provided code sets it to 2. Could you please advise on which setting would be more appropriate to achieve the results discussed in your work? Thank you very much for your time and help!

please look our code carefully !!! We never use batch size 2.


How to register routes at runtime?

I'd like to not use static modules, but rather add routes programmatically at runtime (prior to calling NancyHost.Start()). Searching didn't return much, any pointers? Thanks!

I have the same problem. Consider this module:
    public class MyModule : NancyModule
    {
        private static bool _pluginInstalled = false;

        public IndexModule()
        {
            Get["/install"] = _ =>
            {
                _pluginInstalled = true;
                return "Plugin installed";
            };

            if (_pluginInstalled)
                Get["/plugin"] = _ => "
Plugin working
";
        }
    }

Expected behavior would be (without any knowledge of the inner workings of Nancy) that when I open the /plugin page after opening /install I get a response other than the actual 404.
Seems like Nancy is registering routes only when NancyHost.Start() is called.

@BalassaMarton That can't possibly work because at the time the constructor is called, _pluginInstalled is always false. You might want to try the following:
Get["/plugin"] = _ => {
    if(_pluginInstalled) {
        return "
Plugin working
";
    } else {
        return 404;
    }
}

But that's not really related to what I meant. I don't even have a module, I'd rather do this (pseudo-code):
Nancy.Routes.Get['/foo'] = (_) => "foo";

Nope, _pluginInstalled is a static field, so after executing the /install route, it will be true next time the constructor is called, but the route is not recognised.

To make this work you would have to do a bit of re-plumbing of Nancy. The reason is that, because of performance reasons, all routes are registered once - during the startup of the application. This build the route cache and also helps propulate the routing trie (the thing that figures out, with lightning speed, which route to invoke). So if you added routes at runtime you would have to rejigg them so they are refreshed somehow (no capability in them for that at this time).
Optionally you could look at implementing your own INancyModule and returning your own route collection (we don't really care how it's populated), or look into implementing a new IRouteCache that not only looks at INancyModule instances, to get an additional IEnumerable<Route> (https://github.com/NancyFx/Nancy/blob/master/src/Nancy/Routing/Route.cs) from another place that's not INancyModule (in effect, creating a form of aggregate route cache builder)

@thecodejunkie Just to be clear, even if I would implement my own INancyModule, there is still no way to add routes at runtime, because they would be registered at startup, right?
And when you say at startup, do you literally mean application startup or when NancyHost.Start() gets called?

Well, if you want to register a route - at any given time during the lifetime of your application, then no it would not work to register a route because of the route cache / trie that I mentioned.
Startup = the first time a request comes in and the bootstrapper runs for the first time
What is is that you are trying to achieve?

This seems like something that should be handled by a Before Hook. You can flag the modules you wan't  disabled with an extension method like this.DisableUntilInstallation() that basically returns 404 while a static IsInstalled is set to false.

@thecodejunkie Oops, somehow I didn't get a notification for your reply.
I'm working on a little framework that abstracts resources away from the underlying transport mechanism (HTTP in the case of Nancy, but could also be WebSockets/XMPP/whatever). Resources are just model classes that you can perform CRUD methods on.
The part that interacts with Nancy receives the models and should then register the routes for each model (like GET /foo, GET /foo/{id} etc). This is where I'm having trouble; the models are passed to my framework during the setup phase so I can't use static NancyModules.
If that isn't clear enough, here is how this would look in code:
void Program()
{
    var fx = new MyFramework();
    fx.RegisterResource(MyModel);
    fx.RegisterProvider(new NancyProvider());
    fx.Listen();
}

class MyModel : Model
{
    public string Name = "MyModel";
}

class NancyProvider()
{
    void RegisterResource(Model model)
    {
         string route = String.Format("/{0}", model.Name);
         nancyHost.Routes.Get[route] = (_)  => { return GetAll() }
         nancyHost.Routes.Post[route] = (_)  => { return Insert(_.model) }
    }
}

Hi @jgillich did you ever get this to work for you?
I am currently helping investigate a similar approach by which it would allow for a controlled set of model properties to define routes for the purpose of query filtering.
Plan would be to wire into a LINQ syntax to retrieve the required data set.
I could imagine this would wreak havoc on any form of API documentation (eg swagger) though.

@wildshane I don't remember and don't have access to the code anymore, sorry!


reorganizing and adding docs

Hey @coroa this PR does a few things

it updates the config.yaml and workflow.py to streamline data locations, specifically requiring paths for history, scenarios (I see now it has proxies and gridding, maybe we want to further reduce this..)
removes config.yaml from git tracking
moves regionmapping information into the config.yaml and start to think about how to support running the workflows for multiple models/scenarios

I did a poor man's version of your path expansion to support configured paths in the region mapping, feel free to rip that apart.
Here's the version of my config I have on disk.
# Setting `base_path` redirects everything else from there
out_path: "../results"
data_path: "../data/rescue"

history_path: "$data_path/historical"
scenario_path: "$data_path/scenarios"
gridding_path: "$data_path/gridding_process_files"
proxy_path: "$data_path/gridding_process_files/proxy_rasters"
postprocess_path: "$data_path/postprocessing"

encoding:
  zlib: true
  complevel: 2

region_mappings:
  REMIND-MAgPIE 3.2.0-4.7.0:
    path: "$data_path/regionmapping_remindmagpie_h12.csv"
    country_column: "CountryCode"
    region_column: "RegionCode"
    sep: ";"

country_combinations:
  sdn_ssd: ["ssd", "sdn"]
  isr_pse: ["isr", "pse"]
  srb_ksv: ["srb", "srb (kosovo)"]

variable_template: "CEDS+|9+ Sectors|Emissions|{gas}|{sector}"
base_year: 2020

luc_sectors:
  - "Agricultural Waste Burning"
  - "Grassland Burning"
  - "Forest Burning"
  - "Peat Burning"
  - "Agriculture"
  - "Aggregate - Agriculture and LUC"

Everything now lives in a single directory (data_path), which I can host on sharepoint (it's just copied over from our previous folder): ...\RESCUE - Documents\WP 1\data_2024_03_14
A next step would be to set up the gridding proxy and masks to write into these locations.
Let me know what you think!

@coroa 44d0911 implements file skipping. I noticed by the way that we not longer use aneris.grid()?

I noticed by the way that we not longer use aneris.grid()?

Yes. See also: https://github.com/iiasa/aneris/pull/64#issuecomment-1959884424

Got it, thanks!

This is good to go now, noting existing conflicts..

I rebased this on the latest changes on main and then added to fixup commits to stuff i'd have done differently
(for the path resolution for regionmappings and how to check for a single model)


Fix incorrect grammar in README

Issue #, if available:
Description of changes:
Fix incorrect grammar in README
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/ok-to-test


Fixed styles info prompt about statuses in the mobile version the tex…

Changed styles in info prompt about statuses in the mobile version the text about statuses is cut off #1513

well done!!


Snap 646

In SnappyStore partition by column translates internally to partition by expression. This ensures column data is in de serialized form and can be passed to Snappy hashing algorithm.
Snappy hash algorithm is same as that of Spark.

@suranjan @kneeraj @soubhik-c @hbhanawat  Please review


docs(readme): const where possible

The purpose of this PR is to update the readme to use const where possible.

:tada: This PR is included in version 0.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Crashes on 1.20.1

crash-2023-09-30_22.38.28-server.txt
Fabric doesn't detect any incompatibility, after loading a world, it just crashes instantly

Same here.
Running fabric API 0.89.3
cucumber refabricated 2.0.1
Mystical refabricated 3.0.2
ForgeConfigAPI 8.0.0
doesn't detect any issues, game launches
game crashes as soon as i click on single player.


WIP Add information from the Readme file

Changes description
Add requirements, related and how to install information in index file

Checklist
Please check if your PR fulfills the following specifications:

[ ] Tests for the changes have been added
[x] Docs have been added/updated

References

Closes #40

Index


Bug 70

Cherry-picking the commits that solve issue #70 from https://github.com/ossrs/librtmp/commit/c89b57271fa4e8548930574c462f3fb98be9276f and https://github.com/ossrs/librtmp/commit/3cd3440f0eb018f32a1f0ada32fef077bf7821b4. @ojw28

Thank you for your contribution @vavadhani . It's merged


set PFun free of PLift. Rebuilt entirely with Scott Plut

Turns out generating Hash takes longer than compiling it.

I ran some more test and turns out this implementation is quite a bit slower than Lifty one, naturally. Knowing that, I don't want to fully deprecate the Lifty PFun.
I'm currently struggling to fine a good way to combine them together...

I tried putting PFunLift as a second constructor in PFun, and it doesn't work because it causes very complex mess up in Constraints.
Second best solution is making a separate type PFunLift, but this would make fromPFun less terse because it will require user to specify which PFun they want to use...


Reorganize fud2 directory

Removes the fud2/fake directory and splats things into fud2/src. @sampsyo hopefully I'm not clobbering an intentional design choice here? If I did, please feel free to revert this.

Okay, these changes have been published as the fud crate. @sampsyo I'm going to wait for your approval to merge this. I think one important next step after this is documenting the idea of how state, setup, op, and rules interact. While doing some of the reorg, it took me a bit to understand what's going on which each.

Ah, this was indeed an intentional choice… I originally had things conflated in one place, but I separated things into two crates in https://github.com/sampsyo/fake/commit/67f1d8d30438de5ca7734272537be813571b91d0. The reason is perhaps misguided, but I wanted the fake core to be reusable: to be the generic compiler driver library that could hypothetically be used to build other fud-like tools for other ecosystems. Then, the fud2 crate brings all the Calyx-specific "meat", i.e., all the actual states & ops. Part of the point is that main.rs is meant to be extremely focused on Calyx-y build rules and totally separated from the handling of the build.
Not 100% sure if this makes sense… happy to elaborate today if it helps!

I think one important next step after this is documenting the idea of how state, setup, op, and rules interact.

Yeah, this "how to add functionality" documentation is at the top of my list.

Yeah, I'm less sure about this change the more I think about it actually. It makes a lot of sense to keep the core separate from calyx-specific configuration to allow reuse and building other kinds of drivers/build tools. What do you think of a fud-core library and the fud library? fake is unfortunately already taken on crates.io.

Yeah, absolutely, "fud-core" is good. The "fake" name was overly cute anyway!

Okay, added the fud-core package and published them on crates! I think there is still some work to move things around. For example, should the cli.rs stuff be in fud-core or fud? Thinking about tools mentioned in the original discussion might provide some answers!


COMP: Add workaround flag for win22-20240603.1.1 update

Re:
https://github.com/actions/runner-images/issues/10004

Better, more tests pass, but a few still fail:
https://github.com/InsightSoftwareConsortium/ITKParabolicMorphology/actions/runs/9423335679/job/25961499357?pr=44


Update validation for multiple CSV files.

The "ValidateCsv" command has been updated to handle multiple CSV files from given paths, providing the ability to validate multiple files at once. Relevant error messages and logs have been updated to reflect this change. Additionally, helper methods have been added to the Utils class for improved path handling.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

93 of 96 (96.88%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 95.382%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Commands/ValidateCsv.php
54
55
98.18%

src/Utils.php
21
23
91.3%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
537

Relevant Lines:
563

💛  - Coveralls


test: fix ddb import e2e tests

Codecov Report

Merging #8711 (32378e3) into extOverrides3 (79ed68f) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           extOverrides3    #8711   +/-   ##
==============================================
  Coverage          56.31%   56.31%
==============================================
  Files                835      835
  Lines              47226    47226
  Branches            9461     9461
==============================================
  Hits               26597    26597
  Misses             19811    19811
  Partials             818      818

Continue to review full report at Codecov.

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


MPS Support

Thank you for open sourcing the code.
I was trying to get this to work on Mac MPS machines, and got it to work but not sure if I am doing it correctly since I didn't dig deep into the code. I'll just share the parts I updated from the original gradio demo on huggingface:

check the device for mps: https://huggingface.co/spaces/cocktailpeanut/InstantID/blob/main/app.py#L27-L34
used torch_dtype of float32 for MPS (otherwise it keeps giving me black images) and use enable attention slicing: https://huggingface.co/spaces/cocktailpeanut/InstantID/blob/main/app.py#L53-L76

It works, but it uses a lot of memory, so I was wondering if there's a better way. Also even though it's working, I might be completely missing some parts or making a mistake. Would appreciate feedback. It's very fast on CUDA but on MPS takes much longer, so hopefully there's a way to improve the speed as well. Thank you!

I am on a Mac studio M2 Ultra 128 GB, Sonoma 14.1.2
I created a conda venv and installed pytorch suit from https://developer.apple.com/metal/pytorch/
Then I ran the command to install your requirements.txt file
But when I run the app.py file I get a fatal error which I could not resole.
Intel MKL WARNING: Support of Intel(R) Streaming SIMD Extensions 4.2 (Intel(R) SSE4.2) enabled only processors has been deprecated. Intel oneAPI Math Kernel Library 2025.0 will require Intel(R) Advanced Vector Extensions (Intel(R) AVX) instructions. Intel MKL WARNING: Support of Intel(R) Streaming SIMD Extensions 4.2 (Intel(R) SSE4.2) enabled only processors has been deprecated. Intel oneAPI Math Kernel Library 2025.0 will require Intel(R) Advanced Vector Extensions (Intel(R) AVX) instructions. /Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/transformers/utils/generic.py:441: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.   _torch_pytree._register_pytree_node( /Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/transformers/utils/generic.py:309: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.   _torch_pytree._register_pytree_node( /Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/diffusers/utils/outputs.py:63: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.   torch.utils._pytree._register_pytree_node( Traceback (most recent call last):   File "/Users/aggokce2/Desktop/Altay/InstantID/app.py", line 16, in <module>     import insightface   File "/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/__init__.py", line 18, in <module>     from . import app   File "/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/app/__init__.py", line 2, in <module>     from .mask_renderer import *   File "/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/app/mask_renderer.py", line 8, in <module>     from ..thirdparty import face3d   File "/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/__init__.py", line 3, in <module>     from . import mesh   File "/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/mesh/__init__.py", line 9, in <module>     from .cython import mesh_core_cython ImportError: dlopen(/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/mesh/cython/mesh_core_cython.cpython-310-darwin.so, 0x0002): tried: '/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/mesh/cython/mesh_core_cython.cpython-310-darwin.so' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/mesh/cython/mesh_core_cython.cpython-310-darwin.so' (no such file), '/Users/aggokce2/miniconda3/envs/instantid/lib/python3.10/site-packages/insightface/thirdparty/face3d/mesh/cython/mesh_core_cython.cpython-310-darwin.so' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64'))
I tried to install and compile insightface from source, but did not work.

try

git clone https://github.com/deepinsight/insightface.git
cd python-package
pip install -v .

Thank you a lot for the quick response.
Unfortunately I receive the same error as above, perhaps I need to set some flags specific to MacOS when compiling.


Approve changes when failsafe is hit

When we hit a failsafe on the number of hosts to modify, we usually have to manually inspect the changes ZAC wants to make* in order to approve the changes. Afterwards, we have to  temporarily change the config to allow for a greater number of hosts than usual to be modified, restart the application, let it run, then change the config back and restart it again.
This is cumbersome and not super ideal. We should have some way to signal to the application that we approve the changes. After the application has made the changes, the approval (however we give it) should be voided, so that we have to re-approve the next time the failsafe is hit.
How to approve
There are many ways to implement one-time approval. Here are some suggestions:
Intrinsic Solutions
The following changes don't require changes to the architecture or associated services of the application. They are purely source code-driven changes.

File-based

The simplest solution
The file is deleted after the approval has been parsed.

Signals

Send a signal to the application (SIGUSR1, SIGUSR2, SIGHUP, etc.)
Non-trivial because we need to either find the PID of the specific ZabbixHostUpdater process or send the signal to the main process, after which the main process has to communicate the approval to the host updater process.

Environment variable

Very easy to implement the approval, but harder to implement automatic retraction of re-approval.

Can't rely on modifying environment variables on runtime.
Would need some sort of datetime in the envvar value

Format: YYYYMMDD-HHMM
Where ZAC_FAILSAFE_APPROVE=20240110-1450 would be valid from 14:50 to 14:55 on 2024-01-10

Extrinsic (add-on) solutions
These changes require larger changes to the codebase itself via new dependencies or via new services attached to it.
Could be good for similar issues in the future, but overkill for this issue alone.

REST API (FastAPI, Flask, etc.)

Implement endpoint for approving the current failsafe limit
Could also implement other introspection endpoints.

Get list of hosts waiting to be approved.

Filter by add, modify, remove.

And other resources...

Requires new dependencies.
Requires some way for the main process to communicate the approval with the host updater process.

Message Queue (RabbitMQ, Kafka, etc.)

Main process can consume messages from queue.

* Inspecting the changes it wants to make is not easy, and often requires either print debugging or attaching a debugger. Lack of introspection is a huge issue.

Basic introspection by dumping list of hosts to add/remove as a JSON file is added in #74.


black 20.8b1

Hi @singingwolfboy, the black release we're on is the last stable release, as far as I'm concerned. If you look at the repo (https://github.com/psf/black/issues), there are quite a few issues regarding inconsistent formatting.
The moment black is stable I'd love to upgrade, but we're not there yet.


Fixes AMD name in web/the pdf_viewer.js.

Addressing https://github.com/mozilla/pdf.js/issues/7287#issuecomment-219457631 issue.

Trivial


Add option for ignoring missing keys

It'll be useful for dry-struct where new struct can be created based on existing ones. Now we can check only new keys and skip type application for previously existed.

refs #292


python310Packages.bitarray: 2.4.0 -> 2.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.bitarray is: "Efficient arrays of booleans"
meta.homepage for python310Packages.bitarray is: "https://github.com/ilanschnell/bitarray"
meta.changelog for python310Packages.bitarray is: "https://github.com/ilanschnell/bitarray/blob/master/CHANGE_LOG"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.4.1 with grep in /nix/store/av8mxq21q4s3x89v5j1g3rlgg73d5k0z-python3.10-bitarray-2.4.1

found 2.4.1 in filename of file in /nix/store/av8mxq21q4s3x89v5j1g3rlgg73d5k0z-python3.10-bitarray-2.4.1

Rebuild report (if merged into master) (click to expand)

21 total rebuild path(s)

11 package rebuild(s)

11 x86_64-linux rebuild(s)
10 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
glasgow
home-assistant-component-tests.apple_tv
python310Packages.bitarray
python310Packages.glasgow
python310Packages.pyatv
python310Packages.scancode-toolkit
python39Packages.bitarray
python39Packages.glasgow
python39Packages.pyatv
python39Packages.scancode-toolkit
sacad

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/av8mxq21q4s3x89v5j1g3rlgg73d5k0z-python3.10-bitarray-2.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.bitarray https://github.com/r-ryantm/nixpkgs/archive/873f52e18989b4aaf5a0e49c7a56e63342e0766d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/av8mxq21q4s3x89v5j1g3rlgg73d5k0z-python3.10-bitarray-2.4.1
ls -la /nix/store/av8mxq21q4s3x89v5j1g3rlgg73d5k0z-python3.10-bitarray-2.4.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

python310Packages.scancode-toolkit

9 packages built:

glasgow (python39Packages.glasgow)
home-assistant-component-tests.apple_tv
python310Packages.bitarray
python310Packages.glasgow
python310Packages.pyatv
python39Packages.bitarray
python39Packages.pyatv
python39Packages.scancode-toolkit
sacad

Maintainer pings
cc @bhipple for testing.

Luckily, this is the version on nixpkgs


consult your parents!

docs state "You can find any .editorconfig files in parent directories" but then instructs dir/s which only scans deeper in folder hierarchy. Alas nothing that searches higher [would be nice to implement a dir /-s to scan upwards!]
even suggesting dir %USERPROFILE%\.editorconfig /s is likely to produce excess hits outside the intended root=false hierarchical scope.
are there any PowerShell wizards who can write a commandlet to explore upwards?

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

ID: 72f45eb4-3596-866a-2c16-ee225737d956
Version Independent ID: cea055e7-fe9b-95c7-6d25-086f182c01d2
Content: EditorConfig settings - Visual Studio (Windows)
Content Source: docs/ide/create-portable-custom-editor-options.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @mikadumont
Microsoft Alias: midumont

@DickBaker -- Dick, thank you for your fun feedback.  Please consider these general resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

For a PowerShell script to search upwards, see this post:

Search directory for a file, iterating up parent directories if not found

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Add function error

URL of codelab
In which task and step of the codelab can this issue be found?
Practice Problems: Kotlin Basics
task 7 : Implement basic math operations
Describe the problem
add function is not working in kotlin playground
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

@irinaalexandran
Thank you for reaching out to us and providing the screenshot. The error occurred as add() function was not defined as requested. Please refer to the screenshot below:


[Model] HRED

Refer to Building end-to-end dialogue systems using generative hierarchical neural network models

Dataloader

Ubuntu

Metric

Perplexity BLUE

Done


Using curl to retrieve a HTTPS url that requires a custom DNS server fails on macOS

Describe the bug
Let's say you have a website at https://testing.local, and you need to set your DNS server to w.x.y.z in the Preferences Pane to access it. curl https://testing.local will fail with curl: (35) OpenSSL/3.0.14: error:16000069:STORE routines::unregistered scheme.
Steps To Reproduce
Steps to reproduce the behavior:

Run nix-shell with

# shell.nix
let
  pkgs = import  <nixpkgs> {};
in
  pkgs.mkShell {
    packages = [ pkgs.git pkgs.curl ];
    # ...
  }

Run a server on a.b.c.d (it doesn't matter what it is)
Set up a DNS server such that foo.bar resolves to a.b.c.d
Add this DNS server to your DNS settings in the Preferences Pane
Run curl https://testing.local
See error

Expected behavior
There should be no error when running the command.
Screenshots
N/A
Additional context
Interestingly, curl https://a.b.c.d works as expected.
This seems to be a certificate issue: bypassing SSL certificate verification with -k fixes the issue. I am using a custom SSL certificate, but I'm not sure how to point NIX to it.
After some digging, I found that this has been brought up before: https://github.com/NixOS/nixpkgs/issues/283793
Notify maintainers

@lovek323
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 20.4.0, macOS 11.3.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/Users/system/.nix-defexpr/channels/nixpkgs`

Add a :+1: reaction to issues you find important.

I got around this issue by exporting the keychains in Keychain to a single PEM-encoded PEM file, and exporting the path to NIX_SSL_CERT_FILE and GIT_SSL_CAINFO (since git uses libcurl, at least by default):
rm ${keychainPath}; export PATH="${builtins.getEnv "PATH" }:$PATH"; security list-keychains | xargs -I{} sh -c 'security find-certificate -a -p "{}" >> ${keychainPath}; cat ${pkgs.cacert}/etc/ssl/certs/ca-bundle.crt >> ${keychainPath}'

Adjust as needed.

I also experienced this with a x86 system, adding the -k flag solves it but, this has never been an issue in other distributions with curl for me. Is there something that needs to be looked at here? Or are we missing something critical in our NixOS systems for Curl?
I think this should be reopened.


Add Value to Donations

Much like we list the value of distributions we also need to list the value of donations to make it easier for book keeping at diaper banks to list the in-kind donation amount.
We should be able to copy most of this directly from how we do it on distributions and we should label this on the index page In Kind Value.

Hi! I want to understand it better... should just add value_per_itemizable in the donations list and maybe in the show too?
Donations index:

That seems like a good solution to me! :) We just need something for the diaper banks to quickly let folks who give them donations the in kind value.

Thanks! I'll work on this.


no messages are disabled by default

this should resolve #4

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 40.607%

Totals

Change from base Build 10:
0.1%

Covered Lines:
348

Relevant Lines:
857

💛  - Coveralls


Use correct end point format

it's because of unreleased changes.
fixed by release 0.16.0: now endpoint should start with http:// or https://


fix: demoloyment-scripts/aks-run-helm needs 'rbacRolesNeeded' parameter option

fixes #776
Description
I added this param because aks-run-helm can't install charts propery because of insufficent roles assignments.
And test template were also failed becaose of same reason.
I also fixed the test case bacause

typo of helm option
update the chart version

fixes #776
Updating an existing module

[x] This is a bug fix:

[x] Someone has opened a bug report issue, and I have included "Closes #776" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.
[x] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[x] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

[x] I have updated the examples in README with the latest module version number.

@microsoft-github-policy-service agree

I'm sorry to boher you, but I cannot fix the deployment test failure.
Could someone tell me how to fix them?

Dear @apc-kamezaki , Azure Verified Modules (AVM) will become the single Microsoft standard for Bicep modules. Transitioning to AVM, requests for new features or bug fixes will need to be submitted against the appropriate AVM module via AVM Module Issues.
For bugs or new features with existing modules in the /modules directory of this repository, we encourage you to first try the AVM equivalent module if available and if the module is lacking any features, you can file an AVM Module Issue. If the module is not available yet, you can file a new AVM Module Proposal.
See this informational notice for more details on the upcoming changes.
Thank you for your understanding!


We might try to get this in upstream, would make it easier to keep it attributed to you in the commit log that way


build: simplify tsconfig.json, update babel-dual-package.

Codecov Report
Patch and project coverage have no change.

Comparison is base (efd600f) 100.00% compared to head (ddfcdc4) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          261       261
  Branches        92        92
=========================================
  Hits           261       261

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


vendorin libnetwork @f9fa6e0

fixes docker #28836
fixes docker #22004
fixes docker #20569
fixes docker #27539
fixes docker #23910
fixes docker #14627
fixes docker #29100
fixes #29726
Signed-off-by: Santhosh Manohar<EMAIL_ADDRESS>

also fixes #29726

@sanimej CI is failing for various tests; looks related. Also; looking at comments in the the linked issues, should this be included in 1.13?
/cc @vieux @tiborvass

@thaJeztah this PR is targeted only for master. The required patch for https://github.com/docker/docker/issues/28836 is already in 1.13 branch via #29858

@thaJeztah This is only for master. Couple of more PRs have been merged in libnetwork. I will update the PR to pick up the latest libnetwork and also fix the IT issues.

@aboch the CI is green. can you PTAL the docker IT changes ?

Looks good to me

Fixes https://github.com/docker/docker/issues/541 as well; adding impact/changelog


Move resolveUrl to Polymer.Element, get is from ctor. Fixes #3992

Reference Issue
Fixes #3992
Details
The reason resolveUrl didn't work even on class extend LegacyElement is because in the class-based syntax the static get is() getter is only on the constructor, and never gets put on the prototype/instance.
For this change, I just made resolveUrl look at this.constructor.is instead of this.is.  However, we might want to re-consider if we really want to lose is from the prototype/instance or not (was this an intentional change?).

Placing is on the class rather than the prototype was intentional but it's arguable. The idea is that this is class related metadata (is, properties, template) so it makes sense to be there and not unnecessarily polluting the prototype.

Ok, so we'll leave it as-is.  Just wanted to ensure it was intentional. The legacy Polymer() API retains is on the prototype, so we're covered there.

Rebasing against 2.0-preview, which should fix test issues.


Port to futures 0.3

While this requires nightly, the API is now stabilized and will become part of Rust 1.36. IMHO we should merge this now so that it can all get wider testing.

This is currently marked as WIP because it uses https://github.com/gtk-rs/glib/pull/476 and https://github.com/gtk-rs/gir/pull/753 via Cargo patching

This should also be ready now

Waiting for CI as well. :)

@GuillaumeGomez green

Thanks!


Fix lint

This began as a simple fix, but as I unpacked this, there were a few things needed.
This PR:

switches from gometalinter to the current golangci-lint project
switch to go modules files so the linter could test all the compliance
(deprecating glide)
the go.mod is now nicely leveraged, as the newer blackfriday is incompatible...
selinux gets in the way of using the pandoc container image, so ignore that
bump up the gitvalidation marker, as some whitespace issues slipped in
and fix those couple of whitespace issues
drop go1.10.x for testing

Fixes #735
Fixes #785
Obsoletes #756

omg, tools are not working on old golang, and then newest golang is not liking the mismatch of github.com/urfave/cli <=> github.com/codegangsta/cli ...

OHMAN
ALL GREEN

I realize this is a [trash] heap of changes, but @opencontainers/image-spec-maintainers  PTAL so we can keep the CI clear

outdated


Remove "please"

Issue Type: Feature Request
When software seems to earnestly desire that I accomplish some goal, by asking me with "please" before the action it recommends, it always seems strange to me.
Just now I was searching for something that shows up in a lot of data files, and it said "The result set only contains a subset of all matches. Please be more specific in your search to narrow down the results."
But I think it would be more reasonable to say "The result set only contains a subset of all matches. To narrow down results, be more specific in your search."
I get that the word "please" is an attempt to be polite, but it's software, nobody thinks it's rude when it tells you plainly that if you do this one thing, another thing will happen.
VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:36:32.643Z)
OS version: Darwin x64 19.6.0

Thanks for the issue, but I feel like you're overthinking it, it's just a sort of meaninglessly polite expression. I count about 150 localized messages in vscode that contain "please", all along the same lines, I think it's fine.

I've thought this for decades, ever since I read a blog about it somewhere a long time ago. Can't find it now, but this StackOverflow post says that Microsoft has a guideline on avoiding it. Anyway just an idea, thanks for reading it.

Thanks for the link, that's an interesting perspective. I changed a few messages and will mention it to the team.


fix(samples-test): compare time in seconds in stale resource cleanup

[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

@AVaksman did something go wrong here? No changes in the PR!

@AVaksman @stephenplusplus Sorry, this is probably my fault. I (thought I) was running the issue that was addressed by this PR for one of my PRs, and therefore included this fix in my PR as well to avoid build failures.

The tests are running successfully now. Changes in this PR where applied in one of the later commits. I am closing this PR


test pr 12

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0


Update Portfolio

Fixed the checkstyle issues and updated my portfolio page.

Merged commit to fix checkstyle issues.


[backport] Do not create a.out on running python setup.py develop

This PR is backport of #287

LGTM


Error: Draco encoding failed.

我在进行gltf文件压缩Draco时一直再报错误

@YIyiyi-3  how it is closed? any solution?


dec2hex

Closes #146

[x] Running yarn lint does not trigger any linter errors
[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!
[x] You have written tests to accompany your skeleton method

@BlakeGuilloud Done. Issue #623.


Flags for run do not override any values when a manifest is passed with -f

There is an issue where custom values specified via command line flags do not override content that is in a generated manifest (the file you are passing with -f)
Originally posted by @zubron in https://github.com/vmware-tanzu/sonobuoy/issues/984#issuecomment-561608412
This may be the intended behaviour but it is confusing for users. If they specify both, we should either prevent them from doing that, or give a warning that the command line flags will have no effect.

Yea; this is intended behavior because we actually can’t go from the file to a GenConfig object to later edit.
If they specify a file then we need to just error if they specify other flags/options.
Sent from my iPhone

On Dec 4, 2019, at 5:47 AM, Bridget McErlean<EMAIL_ADDRESS>wrote:

There is an issue where custom values specified via command line flags do not override content that is in a generated manifest (the file you are passing with -f)
Originally posted by @zubron in #984 (comment)
This may be the intended behaviour but it is confusing for users. If they specify both, we should either prevent them from doing that, or give a warning that the command line flags will have no effect.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Added support of follow HTTP redirects

Note: This is very much so needed as httpx >=0.20.0 has disabled following redirects by default.

Probably will be a breaking change whenever we get around to #202, but this is great for now. Thanks!


Add age penalty

How much does cumulative attention correlate with age?

Good question. If we can map attention to age, based on the average age of a story that has accumulated a given amount of attention, then apply the same age penalty that HN applies for that amount of time.
My guess is that this could result in a similar story "velocity" to what exists on HN -- that is, a similar distribution of the amount of time a story spends at each rank. I think velocity may be an important part of the dynamic of HN.


Reformat the code so clippy is happy with me

oh I fixed this in the newer one too lol, you never merged it lmao


Fix for  use download_and_extract in SBU dataset (#7046)

Hi @pmeier please kindly review my PR for #7046

I signed the CLA right after my PR submission. I was notified that it takes time for changes to take effect.


fix(build): install openssl on release

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
fix #16938
QQ: how to trigger release on this PR?
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

I think right now it cannot be triggered under PR
but tag it and push it to the repo will trigger it on buildkite:
https://github.com/risingwavelabs/risingwave/blob/main/ci/workflows/main-cron.yml#L932-L951


PocketSVG for using custom images as masks

I'm using DPMeterView in one of my current projects and used PocketSVG (https://github.com/arielelkin/PocketSVG) to convert SVG images to CGPathRefs. I've had pretty good results with it, just a heads up.

I'm building an app using DPMeterView as well, but I opted for UIBezierPath-SVG to convert SVG to CGPaths.
Fantastic library :)

I close this issue, because it wasn't even an issue actually :)
Cheers


Iterating on X-Ray support

This addresses some PR comments in #427 that we decided to hold up for a followup PR.

This looks good, can you rebase or otherwise resolve?

I'll do that!

superseded by #468


DiffDock v1.1 cannot read Amber residue names for the histidine amino acid residue

Amber can give different names to histidine amino acid by examining which protons are present : HID, HIE, or HIP instead of HIS.
This raises an issue when featurizing the protein in Diffdock because those residues are matched to the one letter name X instead of H.
https://github.com/gcorso/DiffDock/blob/d3791a885e504ea7d7c3587951e259e338e4808b/datasets/constants.py#L3
It can be easily fixed by modifying all HID, HIE and HIP by HIS.
Is it a good way to fix it ? If it is the case, may be it could be done automatically in the inference code.
Otherwise, is there a way to read the PDB file that takes into account those variants of amino acids ?
PS-1 : When running DiffDock v1 on the same protein, everything is running fine. That's why I suspect the match of those modified histidines to X coming from the new package Prody.
PS-2 : I had this issue specifically with histidine, but may be it also happens with others amino acids ?

Thank you, you solved my problem!!! A thousand thanks!


[Refactor] Remove duplicate function

TabletColumn::get_string_by_field_type is the same as field_type_to_string. So in this CL, I replace field_type_to_string with TabletColumn::get_string_by_field_type. Because TabletColumn::get_string_by_field_type is used by metadata serialization.  So we can't change its implementation.
I rename TabletColumn::get_string_by_field_type to logical_type_to_string in this CL too.

run starrocks_admit_test


feat(openai): support azure openai

New
Support Azure OpenAI
Change
Fix
#542

cc @EstrellaXD @Rewrite0


Update README

Motivation
Currently it is quite hard to know how to setup the project. The README should include:

How to setup the capacitor plugin for iOS
How to setup the capacitor plugin for Android
How to update the PACE Cloud SDK for iOS
How to update the PACE Cloud SDK for Android
How to update the plugin (adjusting of interface files etc.)
How to release a new version of the plugin via npm

Maybe add documentation on how to run the plugin with the example app in https://github.com/pace/cloud-sdk-ionic-example-app as well.

@trmi1012 Will start updating the read me. You can use my branch as soon as I'm done. I will put the documentation on how to setup the example app in the example app repo


[!] No podspec found for flutter_native_contact_picker in `.symlinks/plugins/flutter_native_contact_picker/ios

This is an issue faced in ios

@Zorawar920  Please used latest version, issues resolved.
Thank you.


extra-mcmc -- move some burden to SS3

Some of the parameters are not reported in the posteriors.sso file, so lots of "extra" work is done for the assessment that takes quite a bit of time. Some of this extra stuff can be moved to the SS framework to save post-processing time. @iantaylor-NOAA it was suggested to contact you first to see if you had made an previous progress on this and what can actually be moved to SS?

[x] see what tasks can be moved
[x] develop SS3
[x] test changes to SS3

[x] @aaronmberger-nwfsc will test burn in with adnuts
[x] @kellijohnson-NOAA will test general use
[x] @cgrandin will test use of executable with workflow

[x] develop hake-assessment workflow to use new executable

I agree and good to revisit later.  I wonder if couldn't just have a list with parameter or derived quantity names that then populates that we then populates either keyposteriors or nusianceposteriors or similar.  Good idea.  This will be particularly beneficial when moving to all (or most) MCMC runs when using adnuts.

You know it might be as simple as adding these parameters to the extra reporting section of the control file but I am not sure

This is big on the wish list. The code for running and maintaining the extra mcmc is large and awkward. It would be nice to do away with it.

@Cole-Monnahan-NOAA is also interested in this workflow as it relates to applied use of adnuts, so we should keep him in the loop also

@aaronmberger-nwfsc Thanks for looping me in here. Let's have a more formal discussion later. One quick note. As I understand it, you take a row from the .psv file and run it in "MLE" mode to get a Report.sso file. You're accounting for the bias correction difference between MLE/MCMC modes right? That would lead to the wrong predictions.

I had a conversation with Teresa A'mar (@amart) about this years ago and we discussed two alternative approaches to building in the calculation of additional quantities from MCMC into SS.
1. Adding all the needed output to files that got written automatically in the -mceval phase
The problem here is that it's easy to think of additional things that you might want to get from the mcmc output. Right now, I think the list includes

Matrix of numbers-at-age for all years
Matrix of selectivity at age for all years
Expected fit to index observations for all years within the range of the index observations
Total biomass for all years
Summary (age 2+) biomass for all years

Which adds up to a few thousand quantities. I don't think it's good to include these in the extra reporting section of the control file (I think index fits and numbers-at-age for an individual year are already there) because those quantities become sdreport variables and slow down calculations after inverting the Hessian in MLE mode. Nevertheless, it seems feasible to add an option for the list above to be reported in posteriors_vectors.sso or a new output file created during -mceval.
2. Adding a switch to automatically generate the report file for each mcmc sample
This probably would not speed things up but would save the extra steps in the hake workflow of mapping the posteriors.sso rows into ss.par and then iteratively re-running the model and renaming the output. This was the path we preferred at the time and for a while the starter file listed these options:
0 # MCMC output detail (0=default; 1=obj func components; 2=expanded; 3=make output subdir for each MCMC vector)

But only options 0 and 1 ever got implemented, so the list has since been reduced to reflect that.

@Cole-Monnahan-NOAA regarding bias adjustment, it looks like that is indeed sorted out here: https://github.com/pacific-hake/hake-assessment/blob/master/R/extra-mcmc.R#L57-L69

Another option would be command line switch to include Q in the output during the mceval phase. This should be fairly straightforward to implement in the TPL (no dealing with control file parsing, etc.)

@iantaylor-NOAA - I am not sure if the bias adjustment is actually fully dealt with because when you look at the resulting likelihood values in the MLE context from each run they have non-zero values for the difference in likelihoods between Recruitment and NoBias_corr_Recruitment(info_only).

This topic is currently being talked about on the SS developers vlab website under #75754 (referencing this for personal trace-ability in the future, but the forum is private).

@kellijohnson-NOAA, regarding bias adjustment, I just compared this likelihood equation from Methot and Taylor (2011)

to the code in the SS TPL file that calculates NoBias_corr_Recruitment(info_only):
      noBias_recr_like=recr_like-sd_offset_rec*log(sigmaR)+(recdev_end-recdev_first+1.)*log(sigmaR);

It looks to me like that NoBias_corr_Recruitment removes the ln(sigmaR) term from the likelihood entirely, whereas in MCMC that term is included and the b_y values are all equal to 1.0.
I should note that I've never actually used the NoBias_corr_Recruitment(info_only) value, but I believe that it is for Jim Thorson's Laplace approximation calculations as discussed in https://github.com/r4ss/r4ss/issues/44.

@cgrandin calculated 35 minutes for 2000 samples, now we are trying to use system('xcopy ...') to move files to see if that is faster than file.copy()

xcopy was slower so now I am trying to split the posteriors up based on number of cores and then use the start command in batch files to run the MLEs in parallel processes. Within R in Windows you cannot run outside processes in parallel so this is kind of a hack but if it works it should speed things up quite a lot.

within ss3sim we run SS simultaneously using multiple cores
On Wed, Mar 4, 2020 at 10:58 AM Chris Grandin<EMAIL_ADDRESS>wrote:

xcopy was slower so now I am trying to split the posteriors up based on
number of cores and then use the start command in batch files to run
the MLEs in parallel processes. Within R in Windows you cannot run outside
processes in parallel so this is kind of a hack but if it works it should
speed things up quite a lot.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pacific-hake/hake-assessment/issues/656?email_source=notifications&email_token=AA7LCFEYKTM2LKHA24DXH2DRF2QEPA5CNFSM4KRRE6D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENZTJPI#issuecomment-594752701,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7LCFEYSNMEPYJPROLKN3LRF2QEPANCNFSM4KRRE6DQ
.

--
Kelli Faye Johnson, PhD
Research Fish Biologist
Northwest Fisheries Science Center
National Marine Fisheries Service
2725 Montlake Boulevard East
Seattle, Washington 98112
(206) 860-3490
<EMAIL_ADDRESS>

Yea you can do it from inside a program (i.e. parallelism is compiled into the executable) you just can't run already-compiled executables in parallel in Windows. There has no fork() command in the Windows OS. It's seen as a huge downfall of Windows. We should all be using Linux/MAC

I don't think this is strictly true. adnuts accomplishes this with snowfall by opening new R terminals, copying the ss.exe into temporary folders and running them in parallel (here  and here for the curious). There's nothing special about it. Rstan does the same thing.
It should be easy to recreate this on the processing end I think... just chunk up the .psv and split it off into separate SS calls, read in files in each process, then merge them. It's that last merging that could be complicated depending on the output file structure. But trivial for flat files.

On Sean's advice I used the furrr::future_map() function with future::plan("multisession") in Windows to get this to work.
The extra-mcmc runs now take only 5 minutes for 2,000 as opposed to 35 (with 7 cores).
I'll also parallelize the forecasting, catch-levels, and retrospective routines. I need to clean up the code before pushing but I wanted you all to know I think we can keep this stuff within the hake assessment project for now.
Following up on my earlier statements about ```fork()``: This is true, you cannot spawn child processes in Windows. This method using the future package spawns new R sessions on the machine, which are not children of the original R session so it is allowed. It makes it more difficult to exchange data between them but it works well for something like this where you are running an executable which generates output files.

Update for 2022: a new version of Stock Synthesis will be available that stores all of the Report.sso files from MCMC sampling in a directory. This will be available for testing in December 2022.

Check with Rick to see if the derived quantity of q can be reported to Report.sso.

@kellijohnson-NOAA
The ss_output.par files are not used, so no need for those. The CompReport.sso files contain Composition Database which we need for the mcmc -at-age tables. If that could appear in the Report file then we wouldn't need the CompReport files.

The executable to test is located on the drive here. Please test as soon as possible so I can get feedback to Rick.

I have compiled and run this SS at this commit:
https://github.com/nmfs-stock-synthesis/stock-synthesis/commit/c050381406d6d588d1cb5fa4ae09bc9d37cabe79
and all the posteriors.sso, derived_posteriors.sso, etc  are now populated and all "extra-mcmc" files are there.
The poeteriors.sso etc filess are still in the sso folder though when they should be outside it in its parent directory for reproducibility. I have no doubt that will break a lot of people's code.
I still have to build the doc with the new output before saying it's fine but this is a step forward.

@cgrandin you can also opt to put all of the Report_ files in the main folder by not creating a sso folder. The thought of the sso folder is that people can delete every output file with one easy delete command. So, I think it might be easier to expect all output files to be in the original folder for us rather than creating sso?

OK, that's fine.. I was just thinking anyone who uses SS (if this is merged into main) may not expect that to happen and break their reading-in code. We can totally work with it though for hake. I did make a comment on the SS GitHub issue about it. Hope that was ok to do.

The MCMC results from last years assessment model that used SS version  (#V<IP_ADDRESS>;_2020_10_19) compared to the same model using SS version (#V3.30.xx.yy;_safe;_compile_date:_Dec 23 2021) produce very similar (near exact) estimates and derived quantities.
The only large difference was in the total likelihood.  The contribution from Age_comps is considerably different (see inserted snippet), where the latest SS version (not released yet) was considerably greater (right side of insert).  Is this related to a recent change in the Dirichlet multinomial likelihood?

working good enough for 2022, potentially refine in 2023 with smaller .rds files and better file management via a reduced report.sso file using options in the starter.ss file.


Create contact.html

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #41

This would be great if you provide any of this   📝

[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Navbar design
[ ] Footer
[ ] Single page website
[ ] Animation design
[ ] parallax effect design
[ ] Others

Checklist ✅
]

[ ] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[ ] I'm Hacktoberfest contributor

Any tutorials or documentation have you added
(made by you only not from google)

[ ] Tutorial
[ ] Documentation

Screenshots / Gif (Compulsory) 📸

Mention the issue no and add ss @nandini7637

Mention the issue no and add ss @nandini7637

Yeah I have mentioned the issue no. 41
and sorry mistakingly I added the video in issue and not in PR,here it is
https://drive.google.com/drive/folders/1jd4D1-KbB3BKestouttTtROihWUQ7rOw?usp=sharing


Look at locking the rubocop version

I'm not really sure why the bundle exec is  being ignored, so I want to first verify that it's using 0.50.x

Found it, the default ruby .gitignore ignores the Lockfile. So I've applied semver to rubocop.
# for a library or gem, you might want to ignore these files since the code is
# intended to run in multiple environments; otherwise, check them in:
# Gemfile.lock


Apply the Cedar schema terminology

Description of changes
Issue #, if available
#842
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[x] A breaking change requiring a major version bump to cedar-policy (e.g., changes to the signature of an existing API).

I confirm that this PR (choose one, and delete the other options):

[x] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).

I confirm that cedar-spec (choose one, and delete the other options):

[x] Requires updates, and I have made / will make these updates myself. (Please include in your description a timeline or link to the relevant PR in cedar-spec, and how you have tested that your updates are correct.)

All APIs and comments should be updated. The remaining work is to rename tests, which I will do once a preliminary review is done.

I like the proposed renaming.
Will this extend to renaming the policy format?

Good idea. Will do.

Overall, the changes look good. The new names are more consistent.
But I'd prefer more symmetry between the public (aka defined in api.rs) PolicySet and Schema interfaces. For PolicySet we provide:

a FromStr impl (which parses the Cedar syntax)
from_json_str
from_json_value
from_json_file
to_json

I think we should provide the same sort of interface for Schema and SchemaFragment. So:

the FromStr impl should parse the Cedar syntax, and we should have a separate from_json_str to parse the JSON syntax
from_file should be renamed from_json_file
from_file_cedar should be renamed to from_file
maybe  rename as_cedar to to_cedar

How about using the file suffix (cedarschema vs json) to name these APIs? For instance,

Schema::from_str_cedar to Schema::from_cedarschema_str
Schema::from_file_cedar to Schema::from_cedarschema_file
SchemaFragment::as_cedar to SchemaFragment::to_cedarschema


Renamed file to match playground documentation

Renamed file to match playground documentation
Closes #541
Signed-off-by: Rashed Kamal<EMAIL_ADDRESS>

@cari-lynn makes sense. I will update the PR. Thanks!

@cari-lynn updated PR, using schema. Thanks!

Thank you!


User_full   Error as "CRITICAL:root:twint.get:Multi:'name'"

Hi,
it shows the error "CRITICAL:root:twint.get:Multi:'name'" after I add "c.User_full = True".
It run well before I add this.
My codes are as follows.
c = twint.Config()
c.Search = "Trump"
c.Lang = "en"
c.User_full = True
c.Location = True
c.Since = "2019-06-19"
c.Until = "2019-06-20"
c.Limit = 3000

c.User_full is used to get all the info about an user (bio, user ID, full name, etc), it should be used with functions like .Followers and .Following
If you want to get all the infos about an user, you can do as showed here https://github.com/twintproject/twint/wiki/Module#fetch-informations-from-a-users-profile


[Question] Kibana with enhanced-table running on Kubernetes

I'm trying to install Kibana with a plugin via the initContainers functionality and it doesn't seem to create the pod with the plugin in it.
The pod gets created and Kibana works perfectly, but the plugin is not installed using the yaml below.
apiVersion: kibana.k8s.elastic.co/v1
kind: Kibana
metadata:
  name: quickstart
spec:
  version: 7.11.2
  count: 1
  elasticsearchRef:
    name: quickstart
  podTemplate:
    spec:
      initContainers:
      - name: install-plugins
        command:
        - sh
        - -c
        - |
          bin/kibana-plugin install https://github.com/fbaligand/kibana-enhanced-table/releases/download/v1.11.2/enhanced-table-1.11.2_7.11.2.zip

Well, without logs, it’s complicated to say what is the problem.
I don’t think that the problem is tied to enhanced table plugin itself, but more with the way you install in kubernetes.
However, I recommend to use a custom image than init-containers.
More info here:
https://www.elastic.co/guide/en/cloud-on-k8s/1.8/k8s-custom-images.html

dockerfile for reference
FROM docker.elastic.co/kibana/kibana:7.11.2
RUN /usr/share/kibana/bin/kibana-plugin install https://github.com/fbaligand/kibana-enhanced-table/releases/download/v1.11.2/enhanced-table-1.11.2_7.11.2.zip
RUN /usr/share/kibana/bin/kibana --optimize


A11y fixes on Impact Report (DL and colour contrast)

Summary of changes
After updating International statistics for accessibility, I'm updating Impact Report to fix the same accessibility issues (colour contrast, definition lists). The same stat style is being used on the Lang template, so I fixed it here (on a page that's already live) in preparation to fix it on Lang:

Frontend

Ensure definition lists only contain a single div (containing dt and dd elements) instead of two nested divs

The issue: Once I removed all the extra column wrapper divs, the bordered statistics no longer had a gutter between them (a.k.a the extra column wrapper div was necessary to contain the entire bordered card within the boundary of a column).
Solution: Added a Statistic.Grid component that uses CSS Grid instead of Bootstrap row-cols-* classes. This allows us to use gap-4 which gives us our gutter. I pass in the number of columns as a parameter.

Update colour contrast for links on statistics

Backend
None
Test Plan

Ensure you have Accessibility Insights for Web installed
Prepare your build and open the updated Impact Report in Chrome:

Either compile the a11ystats branch locally (using api.liveugconthub.uoguelph.dev as the Drupal source) and navigate to /grce/impact-report
OR use the following GatsbyCloud build at https://build-67baa8dd-fdef-48e0-b054-3f3b89520a23.gtsb.io/grce/impact-report/

Run Impact Report through the Accessibility Insights Fast Pass on Chrome.

Expected Result: There should no longer be errors resulting from colour contrast issues or definition lists.
Also the statistics on the International page should be:

visually unaffected (a.k.a. they are designed to be NOT full width) by the changes
the definition elements should still pass the Fast Pass on Chrome

Ran the tests, and didn't get any errors.


Add one-to-many relationship (and reverse)

would be nice if hasMany() and belongsto() were available in the Model class, like here in docs: https://laravel.com/docs/8.x/eloquent-relationships#one-to-many

@b166er Those methods are still available. We don't change any of the methods.
Here's an example: https://gist.github.com/ryangjchandler/51bb825695f84656731ead0546e6940e

thanks, my fault! my migrations script has created database throw docker-instance but with wrong permissions (cache-directory 755 and sqlite-cache-file with 644), i got this exception: "General error: 8 attempt to write a readonly database (SQL: drop table" after googling i found i hint with file permission, after adding more permissions for all files (database+cache) now works. thank you!


Advanced BIOS?

I read about a method here and here to achieve bios unlock that should work with other models of Lenovo notebooks. I'm yet to try it out on my Yoga Slim 7 14ARE05 but it is here for anyone who is brave enough to try it:
"1. Turn off OneKeyBattery mode in BIOS setup (this won't work with it enabled), save settings, power off
2. Type, with the laptop powered off: F1 1 Q A Z F2 2 W S X F3 3 E D C F4 4 R F V F5 5 T G B F6 6 Y H N. If you notice the pattern, it's just going down the column from F1 to the letter in the last keyboard row, from F1 to F6.
3. Boot the laptop with F2 pressed to get into BIOS as usual, you'll get into the BIOS but with a few options more unlocked."

Nah, it doesn't seem to work :(

since you pointed out that this is not working, I'll close that issue for now.
regards crito


investigate tree-sitter rst handling of trailing letters after backticks.

I believe that tree sitter rst parse a few `object`s differently than docutils, I think that raises warning in building some of the docs.

a few `object`s gives a WARNING: Inline interpreted text or phrase reference start-string without end-string. warning with Sphinx, and renders as-is (i.e., the backquotes are ignored).
The SciPy examples are both wrong. You can see in the online docs that they don't render correctly https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.bootstrap.html https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.geninvgauss.html. Unfortunately, it seems these do not actually produce any warnings from Sphinx/docutils. If tree-sitter-rst actually does give warnings here that's a nice feature.
The RST spec is very specific about what is allowed here:

Inline markup end-strings must end a text block or be immediately followed by

whitespace,
one of the ASCII characters - . , : ; ! ? \ / ' " ) ] } >
or a similar non-ASCII punctuation character.

Everything else must be escaped with a backslash
Of course, I have no idea if tree-sitter-rst actually follows all these rules, but the examples given here don't show any issues with it. I would suggest closing this PR and submitting upstream fixes to libraries like scipy to fix their RST formatting.

Thanks for looking at the SciPy docs, it's indeed improperly render.
Let's keep this open in case we found other case, but let's try to send patches upstream to fix the SciPy docs, so we don't have errors.

I think there might be some case where tree-sitter-rst does
791:        `statistic`. Memory usage is O(`batch`*``sample.size[axis]``). Default
            ^ open quote                   ^ closing quote

I have to recheck.

https://github.com/scipy/scipy/pull/19227


Multiple column filter

i'm using enterprise version is it possible to have multiple column filter e.g checkbox selection and search by typing

Please use the Members Forum for enterprise requests - you can find the forum here: https://ag-grid.com/forum/forumdisplay.php?fid=5
thanks