bug_fix

The PR fixes a bug related to creating a surficial wetting front. The flag create_surficial_front was true for saturated conditions, which must be false.
Additions

None

Removals

None

Changes

None

Testing

All existing tests passed
Tested on basin 01105933 which triggered the bug, after the fix the basin ran without any issues

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Passes all existing automated tests
[x] Reviewers requested with the Reviewers tool :arrow_right:

Good catch, I agree create_surficial_front should now trigger correctly.


preset-date-range-button template does not work

Describe the bug
Any customizations within the preset-date-range-button template is ignored
To Reproduce
Pass in preset-dates into VueDatePicker, add preset-date-range-button template and change it from the default (in this case add ...) While the left side bar rendered, and the presetDate method works, it does NOT use your customized template
Expected behavior
It would use your customized template for the range button
Screenshots

Desktop & mobile (please complete the following information):

All Browser
Library version 9.0.3

If you copied from the documentation, you need to set preset-date-range-button per date in the provided config.


@searchable Cost

** Which Category is your question related to? **
ElasticSearch
** What AWS Services are you utilizing? **
ElasticSearch, AppSync, Cognito, etc.
I recently got a bill for ~$75 for an elasticsearch instance which I barely use.

I have an app that is in development and I have a User model which has @searchable enabled. I barely use the elasticsearch instance and I'm wondering if anyone else has had a similar experience. Does anyone use @searchable in production? And if so, does it account for the majority of your aws bills? It seems to me that this is a ridiculously high price for a non-production scenario, and I'm wondering how much the cost increases in production. I like the @searchable feature, but it's unusable if it will negate any profit I might make from an app.

Hi - I've notified the AWS support team of this and they are happy to look into it. Can you open a new support case and DM me on Twitter your case number? We can dig into this further then.

@brene Thanks. Check your inbox

Unfortunately @searchable directive is not really serverless as you are basically running a search instance. It is the bulk of my bill as well.

So @searchable will spin up a separate EC2 instance for each dev environment and keep them running? That's going to rack up the bill unnecessarily on a team with multiple devs...

@hisham Yes that seems to be the case. Increasing the # of nodes on the elastic domain will also spin up new servers.

Can the nodes be manually set to 0 when a dev doesn't need their dev environment? (spin down the ec2 instance but keep the ebs block..). Can a dev just manually turn on and off the ec2 instances as they need it?

I am pretty sure you cannot control the instances (ie. they are not exposed on EC2). If you are concerned about the cost for now you can just remove the @searchable directive and push when you don't need it and vise versa.

A solution would be to add some config options to @searchable for example useEndpoint (or maybe useInstance) could use a shared ec2 instance and create separate tables for each environment tableName-{env}.
Another solution would be to have a file in the API folder searchConfig.json where you could specify instance/endpoints for each environment (more flexible). This would allow one large instance for all environments, or two with one for prod and one for all dev environments.

@GeorgeBellTMH's proposal sounds interesting. @renebrandel I would consider re-opening this issue as a feature request to allow more options to configure the @searchable instance to point to one instance that has multiple tables per env. Unless there's a severless version of Elasticsearch coming in the pipeline...

This worked for us


As a user, I want to configure Rafa with my preferred database type, credentials, source schema, and output schema

db etc added; still need schema


Group context extension proposals

This PR is another step in resolving #539.
It implements the required capabilities proposal and adds processing of the required capabilities extension.
It should fix #549 but a follow up is needed with more tests to ensure all cases are covered.
resolves #538, resolves #483, resolves #566

Sorry, rebase conflict. Needs one r+ again.


security concerns

You should not use constant time comparisons (due to possible timing attacks), and is the hashing done using md5 by default?  See https://github.com/goji/httpauth for how it does this.

The random salt generated sometimes contains the character "$". The checking of plain text password against the stored hashed password then fails. If the package allowed an alternate version of the function MD5Crypt([]byte(plainPass), salt, magic) that would return the hashed password, salt and magic separately, the package would work.

You should not use constant time comparisons (due to possible timing attacks)

Don't you mean "you should use constant time comparisons"? Regular comparisons like memcmp are variable time depending on how many leading bytes are the same, which is what timing attacks watch for.

The checking of plain text password against the stored hashed password then fails.

Shouldn't this be filed as a separate issue, since it's a different bug?


Targets built with cargo-fuzz always require POPCNT instruction

libFuzzer uses the POPCNT instruction for data-flow guided coverage and cargo-fuzz builds targets with it turned on. This causes the targets to mysteriously crash when the CPU doesn't support the instruction (like my ancient laptop). References:

cargo-fuzz enabling -sanitizer-coverage-trace-compares
Various libFuzzer functions in FuzzerTracePC.cpp with an attribute to always emit POPCNT

What do you think about adding a flag to turn off any -sanitizer-coverage-trace-* option that cargo-fuzz adds by default?

The only flag that cargo-fuzz enables that uses popcnt issanitizer-coverage-trace-compares. (The -trace-divs and -trace-geps flags are also a problem but cargo-fuzz doesn't use them anymore.) Fuzzing works without trace-compares; it just has potentially less quality.
I'll whip up a pull request.


Refactor current selection into service

Much better!

Great change, now the code is easier to understand.

@owennw Maybe favourites can be refactored in similar way rather than using the events mechanic? Not necessarily now, but when we have more time?


Problem with dataset.

Hello author,because of the network,I can't use your script files to download the dataset.So I download it from the 3DMatch(http://3dmatch.cs.princeton.edu/).However I noticed that the extract_data.py will need .ply file.And I can only get .png file.So is that mean I should get the some .ply files like the raw pointcloud.Then I can use extract_data.py to get some attribute like features,etc.

Hi,
did I get correctly that you cannot download it because of some restrictions on your end? Or is there some problem on our side? If I can somehow assist you in downloading the files let me know.
Otherwise, the point cloud fragments from 3DMatch dataset are fused from 50 consecutive depth frames. The code for fusion is available here. For the test dataset you can use the matlab code (some lines have to be un/commented for specific scenes please check the code). For the training dataset we modified the demo.cu code. Hope this helps you.
Best
Zan

Thanks for your responce,
I think the network problem is on my side.And is that means if I want to use my own date.I can use the link about the tools and demo.cu to preprocess first.

Thanks for your response. If you need any other help with either downloading the data or running the code please open a new issue.


Which benchmark do you use in training elmo?

Sorry to bother again. I find there are two benchmarks in https://github.com/ciprian-chelba/1-billion-word-language-modeling-benchmark, the big one(9.9G) and the small one(1.7G). Would you like to tell me which benchmark do you use in training elmo.

I downloaded the data from http://www.statmt.org/lm-benchmark/
After expanding the tar ball, I used the all the files in the training-monolingual.tokenized.shuffled directory (approximately 3.9 GB decompressed).


System tests and Refactoring

This PR is to merge the following:

Added tests for user story 2.1
Added tests for user story 2.2
Added logging in with organization to logging in test(was removed previously for duplicate/separate test)
Refactored the user login as there were security issues with the returned messages / status
Refactored roles into JSON model (previously hard coded on client side)
Refactored hard coded links on client side to use ones in routes
Refactored Put API call with Patch for User Modification
Refactored mobile login serializer to use standard login one (duplicate)
Refactored Put API test to be removed as no longer used
Added E2E test script for Ubuntu (thanks to @makchamp )
Added make migrations to E2E test script now allowing it to run the tests instead of crashing
Updated Fixtures on server side to match latest data as well as changed passwords to: "password"
Refactored Custom User view to now be more flexible and allow different API calls removing all other user serializers, preventing most future ones from being created and also changing tests and client side to reflect the new change

Codecov Report

Merging #196 (68f7214) into master (e08def9) will increase coverage by 0.92%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master     #196      +/-   ##
==========================================
+ Coverage   92.53%   93.46%   +0.92%
==========================================
  Files          22       22
  Lines         616      734     +118
==========================================
+ Hits          570      686     +116
- Misses         46       48       +2

Impacted Files
Coverage Δ

server/user_account/urls.py
100.00% <ø> (ø)

server/user_account/serializers.py
92.59% <91.30%> (-7.41%)
:arrow_down:

server/user_account/views.py
94.07% <93.45%> (+2.86%)
:arrow_up:

server/user_account/permissions.py
91.66% <100.00%> (+0.55%)
:arrow_up:

server/user_account/tests.py
100.00% <100.00%> (ø)

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 e08def9...68f7214. Read the comment docs.


[dv,chip,reset_info] Fix tests that read reset_info via difs

Signed-off-by: Guillermo Maturana<EMAIL_ADDRESS>

LGTM, barring @drewmacrae's pending comments.

thanks for fixing the program error test!


How can I use SingleTickerProviderStateMixin with BaseViewModel

I want to create animation controller in viewmodel and use vsync property which required SingleTickerProviderStateMixin so I tried
class WorkGalleryViewModel extends BaseViewModel with SingleTickerProviderStateMixin{ }
and it said :
'SingleTickerProviderStateMixin' can't be mixed onto 'BaseViewModel' because 'BaseViewModel' doesn't implement 'State'.
and another question
How can I use initState() and  dispose() in viewmodel
Any suggestions would be appreciated

Did you find the answer?

Did you find the answer?

ViewModels should have no UI code. You should do all UI code in the view.

ViewModels should have no UI code. You should do all UI code in the view.

Can someone please answer this? It's a bit urgent. your help is really appreciated!!
@winsw-dev @FilledStacks

@TechSatya you should have no animation code in your viewmodel. There should be 0 UI related code in your ViewModel.

@FilledStacks can you please give a short snippet of the code on calling SingleTickerProviderStateMixin in View??

@FilledStacks can you please give a short snippet of the code or any reference on calling SingleTickerProviderStateMixin in View??
I didn't find any reference to using any controller in the docs!!

@TechSatya use HookWidget and you should be good to go.
Here is a snippet but you can get faster response on the stacked community slack
import 'package:flutter_hooks/flutter_hooks.dart';

class YourAnimationClass extends HookWidget {
  const YourAnimationClass({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final _tickerProvider = useSingleTickerProvider();
    final _animationController = useAnimationController(
      duration: const Duration(milliseconds: 300),
      vsync: _tickerProvider,
    );

    return ViewModelBuilder<YourAnimationClassViewModel>.reactive(
      viewModelBuilder: () => YourAnimationClassViewModel(),
      builder: (
        BuildContext context,
        BookingInvitePlayersViewModel model,
        Widget? child,
      ) {
        return AnimatedBuilder(
          animation: _animationController,
          builder: (context, child) {
            return Container(
              height: _animationController.value,
              width: _animationController.value,
              child: child,
            );
          },
          child: const Text('Hello World'),
        );
      },
    );
  }
}


2441 revise epithelial cell of uterus defs

Logical and textual definition updated
References added

#gogoeditdiff

The commit has many unwanted changes that revert the labeling of multiple cells. Is your master branch up to date?


Add --bucket_size support for gpt_bigcode #111

What does this PR do?
Currently, bucket_size is only supported for a few models such as llama and falcon. This PR is to add bucket_size support for gpt_bigcode model , this can be useful if the dataset has varying prompt lengths.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[NEW] Why did you choose the zap logger over the built-in echo logger interface?

Purpose
It seems to me that in services accessing a logger from a container is quite inconvenient, so why did you choose to explicitly pass a container instance to every service?
There might be the following option to do that:

Use a native echo Logger
Instead of a container instance you could assign a logger to echo.Context and then accessing the logger from it

@dchebakov
Thank you for your valuable opinion.
I'm using a concept called 'Container' for handling some data and functions which wants to share in the overall application in one place. And, I avoid the package dependencies in this repository are complicate.
Those data is set to a container when this application started, and those instance will always be singleton. As you mentioned, I think it's better to aggregate it in echo.Context because the concept called this container is same role as echo.Context. But, I haven't been able to do it yet because I need to investigate the feasibility.
Regarding logger, I'm using it because a library which I want to try to use was zap.logger. This is one of the implementation example.

Thanks for the quick reply! Do you have an example code repository which shows how one could wrap echo.Context (or even context.Context) so later in some code parts, you can extract the logger with the bound request-id or user fields?

Thank you for your interest in this repository!
Unfortunately, I don't have a repository like that. But, the similar sample source code is provided in the official site.

Extending Context

And I think I would like to include not only zap.logger but also the instances of repository, session, config and env in echo.Context.
Could you please try it using the sample if you would like?
Please let me know if there is any concerns and problems.

Thank you for your interest in this repository! Unfortunately, I don't have a repository like that. But, the similar sample source code is provided in the official site.

Extending Context

And I think I would like to include not only zap.logger but also the instances of repository, session, config and env in echo.Context. Could you please try it using the sample if you would like? Please let me know if there is any concerns and problems.

It feels more complicated to handle this way, and it will be better to use the concept of container

Don't fix it because it will make it more complicated.


Script begins to work normally, then NoMethodError

tl;dr -- I'm running ruby notes_cloud_ripper.rb -f NoteStore.sqlite on Ubuntu 20.04, and the first few steps appear to work, but then I get a (mysterious, to me) NoMethodError.
Traceback screenshot included.
I have no idea where to go from here, and am wondering if this is a bug or just my own error(s).
Please let me know if there is other helpful information I can provide.
Thank you.

Thanks for reporting this! I've updated the AppleCloudKitRecord class to catch this sort of error, but I don't know if any of my test data would have your specific issue in it. Would you please check if 4008323f4c317683139652431313fb514e6e76eb fixed your issue?

Just wanted to check if this has fixed your issue.

Closing this issue as it should be resolved. Please re-open if the fix did not actually resolve it for you.


Added standard Arabic

Please note that Arabic is an RTL language, so you'll need to consider this in the UI.

Will need RTL support as stated, keep open until then.

please pull from staging instead of new-design


Full Screen Internet Explorer support

Hi,
The full screen support doesn't suppprt IE.
I used this code as an example of what to add to provide support
https://github.com/sindresorhus/screenfull.js
It's just a case of adding the entries for the ms??? varients of the request, exit and test for full screen.

Yep, it shouldn't be too difficult. Would you like to open a PR for this?


version suggestion

There should be a dropdown for the API version (like we have for PyMechanical) for the API reference. It is confusing to have it be a namespace

Closed by #84 and creating the 241 & 242 tags individually. The documentation now has "2024.1" and "2024.2" in the dropdown versions


Kpt render taking more time when there is large number of blueprints

We could see a behaviour that the kpt fn render in the blueprint root directory is taking more time when there are large number of blueprints.
As kpt is git aware, is there any way that the render will happen only for the changed or newly created blueprints in the repository?
Or any other steps to improve the rendering stage performance as we are planning to provision large number of projects and resources through our gitops implementation.
Please advise.

Thanks for the report @sarathmohan4
Since this is a kpt issue, I would recommend opening an issue here https://github.com/GoogleContainerTools/kpt

As kpt is git aware, is there any way that the render will happen only for the changed or newly created blueprints in the repository?

AFAIK this is not currently supported within kpt.

any other steps to improve the rendering stage performance

One possibility is if you have a flat structure with independent packages, you could script something where you git diff, find the top level folders that have changed and execute kpt fn render $FOLDER. Scripting parallel kpt execution maybe another possibility if you are running in a machine with more resources.

Yes, this is best addressed at the kpt level (feel free to open a new issue).
I know the team is already looking at improving function performance (tracking issue: https://github.com/GoogleContainerTools/kpt/issues/2469), but caching/fingerprinting is a good suggestion as well.

@bharathkkb Thanks for the reply. We tried with the git diff approach, and end up in a scenario which need separate handling for resource deletions. As only modified packages are rendered, and we cannot update the sync repo by completely removing all files (git rm), but  need to overwrite the rendered packages only in sync repo.
This will not have any implicit knowledge of deleted resources. The deleted resources from the hydration repo will not get delete from the sync repo. Deletion of resources might need to handle separately.


Update README.md

Codecov Report

Merging #5 (637635b) into master (8f8227d) will increase coverage by 2.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   59.50%   61.98%   +2.47%
==========================================
  Files           4        4
  Lines         242      242
==========================================
+ Hits          144      150       +6
+ Misses         82       78       -4
+ Partials       16       14       -2

Impacted Files
Coverage Δ

controllers/loudspeaker_controller.go
80.13% <0.00%> (+4.10%)
:arrow_up:

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 a667e6b...637635b. Read the comment docs.


Fix COLUMN_FORMAT syntax

Index Issue: pingcap/tidb#11486
These are bad (incompatible) SQL cases:
create table t (a int column_format fixed)
create table t (a int column_format default)
create table t (a int column_format dynamic)
alter table t modify a bigint column_format default

Claim this subtask based on the comments below this issue.

let me fix it.


Path to generic plots with GeoInterface

Not sure if this already works, but since this is the most complete package, I'll post it here.
Ideally we want this:
ax = plot(points)
background!(ax, TileProvider())

This requires that GeoInterface sets the crs on the plot object? Then we could derive an extent from the plot object (didn't test it, but I assume that's possible), which is enough (with a possible convert) to load the right tile(s) to plot in the background.

The idea is to be able to put interactive Tyler underneath the current plot, and make it use the current CRS?
I guess we need to attach the crs to the axis somehow. Probably GeoMakie GeoAxis already has that, and we can just build on it instead?
Maybe we need to move to plotting tiles onto a mesh so it can be in any projection, rather than just tiling onto a regular grid. That would also facilitate using a globe, 3d terrain and other fun things. We would also have to increase the mesh detail as we zoom in like google earth does, but I'm not sure how to do that.

Well, I also would want it to work in the general Plots.jl ecosystem. This package is half generic code, half specific to Makie. Maybe this should be moved to GeoInterface for the implementation?

To @evetion's point, it would be super helpful to have a non-interactive version (perhaps using CairoMakie as the backend) to enable plotting geographic data on top of background maps with a fixed (or automatically determined) spatial extent.
I've been using Contextily via PythonCall for a hacky way to do this, but it would be great have native Julia implementation that can easily allow something like this matlab code
Happy to help work on something like this as it would be super helpful for my own work!

Yeah, we need to separate out plotting from the interactivity.
Have a go at a PR.
Im not sure how to structure it because currently so much is done in Map. Maybe you can define a MapSnapshot or something that just makes one map, and see how you can to refactor Map so they can share code.

Possibly define Extent (and crs?) on different plotlike things?


Add AI assistant

Add AI assistant
Feature
Name: AI assistant
Number: 11
Sub-feature
Name: AI assistant back-end service
User Story
As a HR lead, i should be able to get the immediate high-level updates on client and requirement details, so that I can measure the productivity of the HR team.
Task
Name: Create backend service
Description
Create a backend service in FastAPI, which queries the database based on query generated by SQLCoder from natural language query of user. Returns a table as response
Status

[x]  In-progress
[x]  Reviewed
[x]  Completed

Removed orphaned branch that could not be merged to release, and added new branch from release for ai assistant


Update Python3 from 3.10 to 3.11 in CI

Also, refactor CI configs to only specify the Python 3 and Perl versions in one place, so we don't have to multiple lines in the config file.  Additionally, add checks to make sure the Homebrew Python 3 version matches the configured one, so we will have a failed build and be notified this way instead of silently building against an old version of Python in CI, which would still work as GitHub CI has older versions of Python installed.

Need to wait till Homebrew has made the switch to point python3 alias from 3.10 to 3.11 before this can be merged and pass tests.

Why would we need to do that? We can't just build against all permutations, nor would most people want to have to worry about something like this when they download the app and use the software update.
MacVim is built against the common use case (brew install python3). If people want to use off-nominal Python, they can just use pythonthreedll to set it. While using a different version of Python is not guaranteed to work, in reality it works just fine.

I understand you. With Python and MacVim it's always a Russian roulette (you never know what feature break first and how fast), when you use a version different from default.
Let's state some obvious things:

Python ABI is different and it's not backward compatible
there's a lot of native code which is dependent on Python version and platform
there's 3 latest versions of Python Language is officially supported
run all tests for MacVim takes quite a long time for a single run

To minimise time and resources used and maximise result I propose following

build MacVim for all supported Python versions
run all tests for a single version of Python (it's the biggest part of tests),
run only Python tests for all other Python versions (it's only a very small portion of tests)

I understand you. With Python and MacVim it's always a Russian roulette (you never know what feature break first and how fast), when you use a version different from default.

That's why the policy is just to have MacVim build against the latest Python release. Some of the issues we have seen before came from MacVim not staying on top of things and building against old Python versions even though Homebrew has long since pinned to the newer version. As long as we do that, people can just use brew upgrdae python3 and that's all they need to know.

Python ABI is different and it's not backward compatible

Fair enough, but I just don't see this as a huge issue in reality, as long as people keep Python 3 up-to-date. And most of the time when they use an old DLL it's not going to break anyway. The ABI isn't that unstable.

there's a lot of native code which is dependent on Python version and platform

What do you mean native code? The only thing that matters here is Vim plugins. I'm not personally aware of any Vim plugins that require older versions of Python3. I couldn't care less about other Python code the user has for their own code. A lot of people have multiple versions of Python installed on their computer and it works just fine. As long as they have the latest (3.11) installed, MacVim will work fine with their Python Vim plugins.

there's 3 latest versions of Python Language is officially supported

Sure, but that has nothing to do with what MacVim builds against.

To minimise time and resources used and maximise result I propose following
* _build_ MacVim for all supported Python versions

* _run all tests_ for a single version of Python (it's the biggest part of tests),

* _run only Python tests_ for all other Python versions (it's only a very small portion of tests)

I don't think this is a good idea. As I said, I just don't understand the rationale. Why can't users just install Python 3.11? If they care so much about using older versions like 3.10 they can build MacVim themselves, as I currently don't see a good reason why they should be opposed to simply install 3.11. They don't have to make 3.11 the default Python on their system either. It just needs to be installed.
Building multiple binaries is a lot of work. Running tests is the least of our worries. It requires signing/notarization, and it significantly complicates the software update design. It also makes it complicated for the user who has to now understand the intricacies of the different builds to decide which one to download, while 99% of them wouldn't care about this at all.

Merged this for now. Stable ABI is still sitting in Vim's PR queue, unfortunately.


Can be marked as 1.20.2 compatible

Make sure you are not opening a duplicate.

[X] I have searched the issue tracker and did not find an issue describing my bug.

Description of your suggestion.
Tested on Fabric with the newest release 😊
I love this mod.

Tested before 1.0.7, tested now 1.0.8 - all works fine. Modrinth version is still listed as 1.20.1.

Marked it as 1.20.2 compatible on modrinth. Thanks for letting me know.

Thanks 🥳


where could I get that downsampled demo data?

the one referenced in README, e.g. KKI2009_113_1_DTI_s4.bval ?

Hey @yarikoptic - Thanks for reminding me to update the readme! The demo is now run with the following command:
ndmg_demo_dwi

So long as your machine has wget (and the dependencies, such as FSL installed) then that should work. Alternatively, we have a Dockerfile which has everything installed.
Close if it works? Thanks!

(Updated the README.md in 50f1a0a).


POI returns null when a block isn't present in the ethereum_block table

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
If a block isn't cached in the ethereum_block table, POI queries return null because get_proof_of_indexing is unable to determine the block number from the block hash.
This fails without error or warnings.
It is unclear to me why a specific block would be missing from this table. Might be related to #2089 .
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Remove one block from the ethereum_block and query a POI
query { proofOfIndexing(  subgraph: "" blockHash:""  indexer: "" ) }
What is the expected behavior?
graph-node should obtain the information of the missing block and add it to the cache.

@otaviopace do you know if this is still the case?

@madumas can you confirm if this still happens please? I just tried reproducing the error following your instructions and the PoI returned normally.


Upgrading to Electron v1+? Better ES6 Support

The current version of Atom (v1.8) is using Electron v0.36.8 internally (process.versions.electron). Is there a plan to upgrade to Electron v1 +so that many more native ES6 features can be used via NodeJS 6+?

@bigtimebuddy Please see https://github.com/atom/design-decisions/blob/master/electron-update-procedure.md.


Error in initializing SpeakerRecognition.from_hparams

Hi,
I used speaker verification with my trained model successfully in a virtual environment. Now I want to do the same in another project in FastAPI. But an error occurred which I can't understand why it's not working. Here is the stack trace:
  File "api.py", line 31, in <module>
    base_mdl = base.BASE()
  File "/home/ai/src/projects/aigitlab/speech-speaker-api/src/speech_speaker_api/api/core/base.py", line 31, in __init__
    self._verifier = SpeakerRecognition.from_hparams(
  File "/usr/local/lib/python3.8/dist-packages/speechbrain/pretrained/interfaces.py", line 230, in from_hparams
    hparams = load_hyperpyyaml(fin, overrides)
  File "/usr/local/lib/python3.8/dist-packages/hyperpyyaml/core.py", line 182, in load_hyperpyyaml
    hparams = yaml.load(yaml_stream, Loader=yaml.Loader)
  File "/usr/lib/python3/dist-packages/yaml/__init__.py", line 114, in load
    return loader.get_single_data()
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 51, in get_single_data
    return self.construct_document(node)
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 55, in construct_document
    data = self.construct_object(node)
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 107, in construct_object
    for dummy in generator:
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 413, in construct_yaml_map
    value = self.construct_mapping(node)
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 218, in construct_mapping
    return super().construct_mapping(node, deep=deep)
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 143, in construct_mapping
    value = self.construct_object(value_node, deep=deep)
  File "/usr/lib/python3/dist-packages/yaml/constructor.py", line 102, in construct_object
    data = constructor(self, tag_suffix, node)
  File "/usr/local/lib/python3.8/dist-packages/hyperpyyaml/core.py", line 420, in _construct_object
    raise ImportError("There is no such class as %s" % callable_string)
ImportError: There is no such class as custom_model.Xvector

Any help would be appreciated,

Sorry, I forgot to put custom_model.py in the folder the code exists.


CookiesDriver.php

if cookie exists but docker container is recomposed, then
    protected function load()     {         $data = Cookie::get(self::COOKIE_NAME, '{"mocks":{},"fakes":{}}');         $data = json_decode($data, true);     }
$data = null and you get an error.
by adding the following:
       if($data == null) {             $data = [                 'mocks' => [],                 'fakes' => [],             ];         }
you don't try to do a foreach on a null value and it works fine.

the essence of my issue is that there is no verification on a null $data which is a situation that has happened to me.

Hey thanks for reporting.
I can add the null check given that it doesn't hurt, but I'd like to understand better how is this happening. The get method from the Cookie facade already handles non-existent cookies, and returns the second argument to this function if a cookies is not set. Which means you can only get that error if the cookie is set with an invalid value.
Keeping that in mind, the problem is actually the cookie being set with an invalid value at some point. I'd like to understand and fix the root of the cause rather than doing the null check workaround.
You mention docker, but I also use docker and I haven't encountered this problem. I need more details to understand what's going on in you environment.

I will gladly offer some more details towards the issue I am facing.
To reproduce the issue: Using a Linux Docker, Laravel Framework and php artisan. I am calling upon the "docker-compose up" feature, which includes the following commands:
php artisan config:clear
php artisan cache:clear
php artisan view:clear
php artisan route:clear
thus clearing all cookies infos on the server side.
Now creating the container and the Web server of my work the very first time, by logging in the first time, no issues. Cookies are then created. close the website and power off the containers.
Then, by relaunching the containers with the "docker-compose up". Remember, server side cookies have been deleted, but the browser still has "previous cookies". This is when it fails because the cookie exists but has an unrecognized value inside thus the $data variable being null.

I see, in this case I think the problem is related with the browser you're using to execute the tests, isn't it? As far as I understand, each test should be an isolated browser environment, meaning that cookies are not persisted through tests. If you for example have a test checking that a cookie is created, if the code fails to create the cookie but a previous run of the test had created it, the test would incorrectly be successful.
Maybe this has something to do with the browser driver you're using? I would take a look at that, because if cookies are being persisted through test executions, you may have other issues. If you can't fix this for any reason, I guess the correct way to do it would to clear all the cookies in the setUp method of your tests. This is not only relevant to this package, but to the test environment that you're setting up in general.
Does this make sense?

I've created a sandbox project with tests running on github actions, so maybe you can try to reproduce your problem there to be sure it isn't an environment issue.
Check it out: https://github.com/NoelDeMartin/laravel-dusk-mocking-sandbox/

I am closing this issue because I cannot reproduce it. Feel free to reopen it, or open a new one, if you can provide a way to reproduce it.


arg: don't crash when asked to convert a null strv to an array

In that case, return an empty array.
https://phabricator.endlessm.com/T14337

@ptomato You may want to take a look at this and bring it upstream. It looks like it would still be a problem in master.


download_strategy: fix case where filename cannot be parsed

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Follow on from #14829 - fixes a case where the filename could not be parsed correctly resulting in an error.
https://github.com/Homebrew/brew/pull/14829#discussion_r1141582788

Review period will end on 2023-03-21 at 22:13:09 UTC.

Thanks @bevanjkay! Made some tweaks to use blank? here because we want to ensure we handle empty strings nicely too.


IoT Connectivity over LoRa

LoRa is very popular IoT protocol and after we will finished with IoT Connectivity over TCP next right step i think will be to support LoRa to connect remote sensor with Aira telemetry agent.

So, LoraWan provides TCP/IP like functionality: https://www.thethingsnetwork.org/docs/lorawan/architecture.html
I guess it could be used for quick solve this issue.

for future implementation


executing standalone mpi from java and mpi workers with a script

Hi Supun,
I executed mpirun directly from java and used mpiworker.sh script to start workers. You can use the debugging feature now.
I also sent command line parameters from mpimaster to workers as environment variables. So, mpiworker.sh script is much more readable.
If you can test the debugging first, that would be great.
Ahmet

@ahmet-uyar I was trying to understand how the two scripts are connected. I can see they both invoke java. I thought one script is going to use the other in the mpi command

@supunkamburugamuve, both standalone and slurm use a single script.
In standalone, the script is used as the prelaunch script when starting workers.
In Slurm, the script is used in the submitting client to invoke sbatch command to submit the mpi job.

I could find one mpi launcher script. Is there a one in standalone directory?

@supunkamburugamuve There is mpiworker.sh script in standalone directory.

I see you have included the mpi command launch in the Java program. How can we specify custom MPI command arguments?

[I see you have included the mpi command launch in the Java program. How can we specify custom MPI command arguments?]
@supunkamburugamuve, we can either add a configuration parameter for that or have two separate scripts.  One for the mpimaster and the other is for the mpiworker.

I'm fine with both options.

@chathurawidanage wdyt about these options?

+1 for adding a config parameter. Are we going to add that to YAML files or accept from the twister2 submit command itself?

@chathurawidanage   I think YAML file would be better.


Too common PHP errors

Description
According to this issue, we are matching several PHP error messages (rule ID 953100) which are too common in basic language, specifically:
must be less than
must be less than or equal to

At the top of this, we are matching lots of more specific PHP error messages which are already matched by two above:
must be less than 65535 bytes
must be less than argument #2 ($maximum)
must be less than the number of fields for this result set
must be less than the number of the arguments passed to the currently executed function
must be less than the number of fields for this result set
must be less than or equal to 1048576
must be less than or equal to INT_MAX - 4 bytes
must be less than or equal to SODIUM_CRYPTO_KDF_BYTES_MAX
must be less than or equal to argument #2 ($max)
must be less than or equal to the length of argument #2 ($buf)
must be less than or equal to argument #

We should either remove two problematic phrases (i believe they are too common also for PL2) at the top or all of the phrases above - i suggest the first option. I can send a PR.

CRS version (e.g., v3.3.4): 4.0.0_rc2
Paranoia level setting (e.g. PL1) : PL1

Confirmation
[x] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.

Makes total sense. I think commenting the line in the php-errors.data and adding a link to this issue will do.

Done: https://github.com/coreruleset/coreruleset/pull/3391


Creation of new Components for 2d Range slider

In order to create something like this, I will be needing extended support of Range Slider to 2 Dimension. Or it support of canvas with supports better with numl design system.

Released as <nu-slider2d/>.


Is this project dead?

There hasn't been a commit in over a year. I would love it if this project got traction from other contributors if the original author doesn't have any time/effort to continue the project. Unfortunately I don't know Swift (and thus SwiftUI) so I can't help development wise.

Hi @arjpar, well, it's definitely stagnant for the moment. I'd love to contribute more on it, and I would have some time, but the thing is, I feel there needs be more interest in general for it.
Would you like to use SweetIRC? As a user, I mean.

Of course I would love to use it as a user! I think there is a space for a high quality, modern macOS IRCv3 client. Currently Textual (paid) is dead, Colloquy is dead, LimeChat is dead. (These are all the "modern" IRC clients for macOS). The only IRC desktop clients that are actively developed are cross platform (which is fine) including electron apps (which very much has its downsides).
I believe the benefit of using SwiftUI would also be easy porting to iOS/iPadOS.
One thing that could help the adoption of SweetIRC would be adding it to Homebrew Casks. I can definitely do that, but I believe SweetIRC would have to meet certain requirements first.


refresh current location

how can I refresh the location of the user when he's moving. It seems that the plugin just refreshes at onCreate

I'm using like this
I check if the user's gps is enabled and if he gave permission and then I use:

@SuppressLint("MissingPermission")
    public void setCurrentLocation(){
        locationEngine = new LostLocationEngine(MainActivity.this);
        locationEngine.setPriority(LocationEnginePriority.HIGH_ACCURACY);
        locationEngine.addLocationEngineListener(this);
        locationEngine.activate();
        lastLocation = locationEngine.getLastLocation();
        if (lastLocation != null) {
            setCameraPosition(lastLocation);
        } else{
        locationEngine.addLocationEngineListener(this);
        }
        locationPlugin = new LocationLayerPlugin(mapView, mapboxMap, locationEngine);
        locationPlugin.setLocationLayerEnabled(LocationLayerMode.TRACKING);
    }

It's called at onCreate. Sometimes it doesn't give the precise location (the gps is not activated on the status bar) and when the user is driving it doesn't change the location even with this code:
 @Override
    public void onLocationChanged(Location location) {
        if (location != null) {
            setCameraPosition(location);
            locationEngine.removeLocationEngineListener(this);
        }
    }

I stopped using this API for now.


[Message Tracker] Immediate Improvements

The big takeaway is that there is a difference between a "send" action and "batch" action.
Depending on which action type, anything related to the file needs a different label or wording.
Improvement Notes:

Receivers: order by receiver name
Receivers: Incoming Report Id label should be "Outgoing Report Id" if the action type is "send"
Receivers: Incoming Report Id is mislabeled if action type is "send"
Receivers: Incoming File Name label is mislabeled if the action type is "send"

Should be "Outgoing File Name"

Receivers Incoming File URL is mislabeled-ish

there is a difference between "internal" files and "ready" files
difference between action type "batch" and "send"
label the difference "Outgoing File" and "Incoming File" is the internal RS format file

Receivers: Add Time stamp to "Date/Time" submitted information
Sender: Add "Date/Time" field

make sure to have the time stamp there, lol

Acceptance Criteria
this refers to the message details page, i.e. https://reportstream.cdc.gov/message-details/29811087

[ ] In the "Sender" section, add a Date/Time field
[ ] Under the "Receivers" section, order the receivers alphabetically first by receiver name, then receiver service
[ ] Under the "Receivers" section, add the timestamp to the "Date/Time" field
[ ] Under the "Receivers" section, depending on the "action type" of the action log (receiver information):

[ ] label anything that has the "send" action to "Outgoing"
[ ] label anything that has the "batch" action to "Incoming" (shouldn't need to change )
[ ] this includes fields such as "File URL", "File Name", and "Report Id"

Additional Notes:
How to activate the Message Tracker feature:

Log in as an Admin
navigate to /admin/features
type in "message-tracker" to the input text box and then click "Add"
In the "Admin" dropdown, there should now be the "Message Id Search" link

Planning some work right now to look into file lifecycles and how they can fit into these designs. For now, please hold off on labelling based on the file url and whether it includes send, batch, receive, ready, other, or anything else.
@chris-kuryak

Note to self: This is blocked by Design until after more research is finished, per Flo. It is unsure if the ticket as written is telling the engineering team to do the correct task.

On reviewing this ticket again, @chris-kuryak, I think we can close it. All of these changes are design-related and the ticket for redesigning the screens includes almost all of these in the ACs (7698)
My ticket for next sprint (7735) addresses any ACs here that are not addressed in Jillian's ticket, 7698.


fix: race condition in RPC request handler

@julian noticed that he non-determinstically gets an "RPC reference is not valid" error after removing a delay in lean.nvim: https://github.com/Julian/lean.nvim/issues/223#issuecomment-1008082188
It seems plausible that this is related to a race condition in the RPC call handler, which does not atomically update the RPC reference map.  This PR changes all modifications of the RPC reference map to use Ref.modify or Ref.modifyGet, which are atomic (albeit with busy-waiting, we should probably fix this at some point).

cc @Vtec234

I can confirm that with Lean (version 4.0.0, commit 9caf4d6c3978, Release) the issue seems gone (from ~10 times trying it).

I believe this might be our first race condition bug (outside of the runtime)?

Argh! I will accept the dubious honour, and thanks for fixing this.


Add security architecture diagram

Illustrating Corpora's usage of single-cell-infra and the AWS infrastructure provided by the Shared Infrastructure team, namely the set of resources exported by the cloud-env module.

links to the terraform in single-cell-infra that makes all of this happen

For the security architecture (VPC, subnets, etc), these details should be provided by documentation from Shared Infrastructure. Our team's responsibility should begin at importing and using cloud-env (i.e. I don't know and probably shouldn't need to know). I've updated the doc to include a link to the `cloud-env' source code however.


Where do NMDC DataHarmonizer submissions go once the SUBMIT button is clicked?

Thanks. How can I retrieve them?

Oh, hmm, I interpreted this issue as one of proposed implementation. i.e. "where may NMDC DataHarmonizer submissions go?" I do not know where they currently go. @subdavis would know (you could also spelunk in the nmdc-server repo code).

They get saved in postgres in the submission_metadata table as unconstrained json on the metadata_submission field which is postgres jsonb. (like a JSON CLOB type).
That table also has author_orcid, created, and status fields.

OK, then I'm going to want read access on NMDC Postgres, and all of the SPIN middleware or whatever

@turbomam Rather than running a shell within a container in Spin/Rancher - would suggest you might use the API endpoints that Kitware provided.  Specifically: https://data.dev.microbiomedata.org/docs in the metadata_submission section.

You can get the submissions from here: https://data.dev.microbiomedata.org/docs#/metadata_submission/list_submissions_api_metadata_submission_get

Do in need credentials for this?

You need to be authenticated with your Orcid on https://data.microbiomedata.org/ (or dev)
A bit more documentation here: https://github.com/microbiomedata/nmdc-server/wiki/Search-API-Docs

Great. That does work in the browser. I don't see how to authenticate and use the API programmatically yet, but I ahven't read the documentation links you've shared meticulously.

They go immediately into the submission_metadata table in "Kitware's" Postgres table, and yes, the API access seems to meet all of my needs.


Allow opt-in of warning on deprecating static attributes

What?
Static attributes can introduce odd or unexpected behavior, largely because
the value is evaluated once, when the factory definition is loaded. Most
commonly, issues can come up with dates:
factory :post do
  published_on Date.yesterday
end

This may cause problems based on the time of day the suite is running,
especially if there's already a mystery guest introduced (e.g. the test
callsite assumes published_on was always one day ago).
To address this, the first step is letting developers know where their test
suites refer to static attributes. This commit introduces a configuration
option developers can opt into where we trigger a deprecation warning with
the factory name and static attributes.
To opt into this setting:
# spec/support/factory_girl.rb
FactoryGirl.configuration.warn_on_static_attributes = true

If we think this is a good idea, should it be opt out instead of opt-in? And then we remove static attributes in the next major release?

See #1135


Doesn't work if path contains spaces

If tex is located within git, let's say 'https://github.com/user/repo/folder_with_space rest/_of_the_path.tex', then build will fail:
usage: latexrun [-h] [-o FILE] [--latex-cmd CMD] [--latex-args ARGS]
[--bibtex-cmd CMD] [--bibtex-args ARGS] [--max-iterations N]
[-W no-CLASS] [-O DIR] [--color {auto,always,never}]
[--verbose-cmds] [--debug] [--clean-all]
[file]
latexrun: error: unrecognized arguments: /_of_the_path.tex'

Just jumping in here - this appears to be a bash input thing. Did you try:

Wrapping the name of your file in quotes?
"https://github.com/user/repo/folder_with_space rest/_of_the_path.tex" or,

Adding a backslash to escape the space?
https://github.com/user/repo/folder_with_space\ rest/_of_the_path.tex


not displaying search results beyond 10.000

When searching a term that has more then 10.000 hits (e.g. "done"), Hoover will display only up to 10.000 results (no matter if I choose 10 or 1.000 results per page).
When trying to go to the page that would display the messages from 10.001 onwards, an error message is displayed: "Unknown server error while searching". Is this a Hoover search limitation or a Hoover UI ?
The exact same problem I encounter with your UI application @jarib on https://hoover-ui.herokuapp.com/. The message reads "Error: unable to fetch https://hoover-ui.herokuapp.com/search: 500 Internal Server Error"

This is a ES setting, index.max_result_window that is 10 000 by default.
Temporary it can be set to a larger value with this request:
https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-update-settings.html
The best solution is to implement the pagination for large sets:
https://www.elastic.co/guide/en/elasticsearch/reference/5.4/search-request-search-after.html

@ioanvpocol this needs to be done by @salevajo I guess ?

Pagination using search_after would require changes in hoover/ui and hoover/search. Once we're upgraded to an ES version that supports it, it should be a quite simple change to make.

The pagination with search_after is almost done, I will make the PR along with the other changes required for ES6.


Create service links in GUI

From what I have seen right now, the scheme view will indeed display links between services but there doesn't seem to be a way to set up these in the GUI directly.

Hi @Gabology. I think that it doesn't have any sense because all services link between themselves by default. And I don't see any reasons to implement it into GUI.


Disable mocked endpoint after number of calls

I have an application that needs to test that an end point is called twice, with different responses for both calls. Is it possible to have a way to set a mocked endpoint to respond in one way the first time its called and a different way the second time its called?

I suggest taking a look at the Scenarios feature for this.
In future, please can you post questions to the mailing list as per the README.


Download Binary File

Describe the solution you'd like
I have been using HockeyApp to download the latest version of a mobile app (.apk or .ipa). I don't see an option to download the binary from App Center. Is it possible to have a way to download a specific binary from the App Center?

This is already possible. Go to Distribute->Releases and chose a build. The download button is at the top right.
I think this feature is not finished yet, because it still links to hockey app.
I would love to see this option on the main download page like it was on the hockey app download page: Clicking install on mac/pc downloads the ipa, clicking the button on mobile devices installs the app.

@kaischo is correct. We are also working on this related issue #82 in March to enable you to download from the install portal on unsupported devices.

@narayan8291 @kaischo Issue #82 that @patniko mentioned is now closed, we now offer to "Download" a binary on platforms where it is not directly installable. e.g. for .ipa files on macOS.
Closing this for now, please let us know if there still anything you'd like to see improved! 👍🏼


fix: enable CompositeIn for MySQL

close: https://github.com/uptrace/bun/pull/632
MySQL support IN conditions with multiple fields.
See https://dev.mysql.com/doc/refman/5.7/en/comparison-operators.html#operator_in
So, I added feature.CompositeIn to the features.

@vmihailenco
Could you review it ?


identChar definition looks wrong

  identStart { std.asciiLetter | $[<>._=?!*+\-$\u{a1}-\u{10ff}] }
  identChar { identStart (std.digit | ":" | "." | "'" | "#")* }
  simpleIdent { identStart identChar* }

Instead of defining valid characters that can continue an ident, identChar is defined as starting with an identStart, followed by any number of the chars that may occur in a non-start position of an identifier. This seems like it wouldn't match something like s1 (since it requires the second character to also be an identStart).
I assume this is supposed to work something like this instead...
  identChar { identStart | std.digit | ":" | "." | "'" | "#" }

Thank you!


Will I be able to use FTP protocol?

No, unfortunately this module does not currently support FTP.


Add UpdateTypeText in OnAfterGetCurrRecord trigger in Sales Invoice Subform page

Hi,
Could you please add call to UpdateTypeText in OnAfterGetCurrRecord  in page 47 - Sales Invoice Subform?
You did it for all other subforms for Sales and Purchase documents (Quotes, Orders, Invoice and Memos - 7 pages), only in Sales Invoice Subform is missing. We can override Type in page extensions, but not variable TypeAsText which is used as a page control.
Screenshot from Sales Order Subform:

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


MacOSX version v<IP_ADDRESS> runs slow even after bootstrapping

Is there any reason why the MacOSX qt wallet syncs slowly and hangs multiples times, compared to the Win64 qt wallet? It can never keep up with the latest block, unless the bootstrap file happens to be super fresh off the cooker.
I have witnessed this on previous versions of the wallet as well, so it's not a recent issue.
OpenSSL version: 1.0.2g  1 Mar 2016
Does that come packaged with the qt wallet? As my OpenSSL version is higher than that!
:octocat:

OpenSSL comes pre-packaged with the app to ensure compatibility. The next release contains many performance improvements which should address the issue you are describing.
In the interim, you can switch to the current dev branch which is nearing a release candidate or try the following steps:
If your peers are above 100 try lowering the max peers Lindad can use
inspect your debug.log file to see if Linda is processing lost or orphan blocks. if it is, try setting maxorphanblocks to ~300 and obtaining a new peers.dat file from the linda bootstrap to remove misbehaving peers you may have

Okay great, I don't mind trying out the dev branch but how would one package it for macOS?
The peers usually sits around 30.

Anyway you can help direct me to how to build it for macOS @davidstoneham? The macOS version is proving a pain now; I don't know if it is because I am running on 10.14, but I have to wait until the bootstrap is current for the wallet to be in sync. As I understand it, the bootstrap files are updated every 6 hours.

@ckhatton the dev branch isn't completely tested so I can't recommend changing to this branch. If your wallet is having issues keep up once it has synced this sounds more like an issue isolated to your machine. If you could post the last few lines of your debug.log when your computer is having issues so we can see if it's an issue with your peers.

Okay sure, will do!

This is now resolved in the latest release


Add golangci-lint to Makefile and CI config

Overview
This is a new linter that could replace gometalinter that purports to be faster: https://github.com/golangci/golangci-lint
Pull request checklist

[ ] I have read the contributing guide.
[ ] I have agreed to the Contributor License Agreement.
[ ] I have updated the documentation.
[ ] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[ ] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

i have some lint fixes for this, but it turns out i don't see an easy way to change the other branch of a PR, so i'm going to close this and open a new PR from my fork based on this.
confirmed that golangci-lint is way faster and does not make my laptop's fans spin up.


Getting only first 100 bits as output.

Query "Bitmap(frame=' ',id=' ')" returns all the bits in the bitmap. But is there a way to get only first 100 bits as an output ?

Limiting the number of bits in the result is not supported yet, but there's another ticket which is related: https://github.com/pilosa/pilosa/issues/641

we ended up patching pilosa to skip bits altogether. there are two contracts for Bitmap file: json (used by pilosa UI) and protobuf, usually used in integrations. you could potentially play around that, i.e. patching a single contract as a really nasty workaround

Closing since this functionality appears to be covered by #641 - please feel free to re-open or open a new ticket if that isn't satisfactory.


Synthesis failed for Google.Cloud.Tasks.V2

Hello! Autosynth couldn't regenerate Google.Cloud.Tasks.V2. :broken_heart:
Please investigate and fix this issue within 5 business days.  While it remains broken,
this library cannot be updated with changes to the Google.Cloud.Tasks.V2 API, and the library grows
stale.
See https://github.com/googleapis/synthtool/blob/master/autosynth/TroubleShooting.md
for trouble shooting tips.
Here's the output from running synth.py:
2021-03-19 22:46:20,695 autosynth [INFO] > logs will be written to: /tmpfs/src/logs/Google.Cloud.Tasks.V2
2021-03-19 22:46:21,356 autosynth [DEBUG] > Running: git config --global core.excludesfile /home/kbuilder/.autosynth-gitignore
2021-03-19 22:46:21,360 autosynth [DEBUG] > Running: git config user.name yoshi-automation
2021-03-19 22:46:21,363 autosynth [DEBUG] > Running: git config user.email<EMAIL_ADDRESS>2021-03-19 22:46:21,367 autosynth [DEBUG] > Running: git config push.default simple
2021-03-19 22:46:21,370 autosynth [DEBUG] > Running: git branch -f autosynth-Google.Cloud.Tasks.V2
2021-03-19 22:46:21,374 autosynth [DEBUG] > Running: git checkout autosynth-Google.Cloud.Tasks.V2
Switched to branch 'autosynth-Google.Cloud.Tasks.V2'
2021-03-19 22:46:21,600 autosynth [DEBUG] > Running: git rev-parse --show-toplevel
2021-03-19 22:46:21,607 autosynth [DEBUG] > Running: git log -1 --pretty=%H
2021-03-19 22:46:21,612 autosynth [DEBUG] > Running: git remote get-url origin
2021-03-19 22:46:21,644 synthtool [ERROR] > Failed executing git checkout master:

error: Your local changes to the following files would be overwritten by checkout:
	google/cloud/dialogflow/v2/answer_record.proto
	google/cloud/dialogflow/v2/conversation.proto
	google/cloud/dialogflow/v2/conversation_profile.proto
	google/cloud/dialogflow/v2/participant.proto
Please commit your changes or stash them before you switch branches.
Aborting

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 354, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 189, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 312, in _inner_main
    git_source.enumerate_versions(sources, pathlib.Path(temp_dir))
  File "/tmpfs/src/github/synthtool/autosynth/git_source.py", line 202, in enumerate_versions
    source_versions = enumerate_versions_for_source(git_source, temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/git_source.py", line 166, in enumerate_versions_for_source
    local_repo_dir = str(synthtool_git.clone(remote))
  File "/tmpfs/src/github/synthtool/synthtool/sources/git.py", line 85, in clone
    shell.run(["git", "checkout", "master"], cwd=str(dest), check=True)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['git', 'checkout', 'master']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Disable EOS SDK at compile time

Is is possible to disable OpenEOS at compile time with a scripting defines symbol?
The eos_plugin_for_unity does it using EOS_DISABLE and I think it would be nice to have this feature in here as well.
Searching for EOS_DISABLE in the eos_plugin_for_unity also indicates, that it can be done in a not very invasive way.

+1 on this, that would be great

Apologies for the slow response, this ended up being easier to implement than I thought :') I just pushed out a new update with the scripting define EOS_DISABLE added to assembly definition constraints, so you should be able to use that to stop the SDK and OpenEOS library from being compiled in your project. Note that since you're expected to create your own manager to interact with either of these, you will have to check for EOS_DISABLE wherever you import or interact with the package.
Feel free to check out version 2.1.0-1.16.1 and let me know if it works for you! If there's anything that doesn't work as expected feel free to let me know and I can reopen the issue to address problems, thanks!


Change themes based on system theme.

So I know you recently added a feature to automatically switch themes based on geolocation, but an even better feature Dark Reader just added to their extension is to change themes dynamically based on the current system theme. I'm not sure how much work it is, but this would be the last bit needed to make these two extensions work perfectly and seamlessly with MacOS's dark theme.
Honestly, if Firefox would just make the horrible light version (where everything is light but the tab bar) of their default dynamic theme the same as their light theme, none of this would really be necessary for me, but I imagine others would like to use different themes.

I was just about to add an issue requesting the ability to have an offset with respect to the sunrise/sunset time as the OS/system theme itself already switches some time before sunset and after sunrise. I guess if it would just follow the system theme then that is even better!
Currently I sometimes end up with a dark system theme but still very white pages 🙂.

Just wanted to bump this. This is exactly what I want, because this extension doesn't change at the exact same time as the rest of my computer.

This is added in 5bd7f05 and released with 1.2.0, which I just uploaded to addons.mozilla.org.
Please let me know any feedback you have for this. It's not working as well as I'd like on my Windows 10 machine. It's something I want to look into more in the future. Let me know how it works for you!

Didn't work for me on macOS unfortunately, the system changed from light to dark automatically and automaticDark didn't pick up on it and stayed light. This is with the "Change themes based on the system theme" setting enabled

Unfortunately, also not in Debian. Using addon version 1.2.1 on Firefox 76.0.


Not able to Read SMS Broadcast Messages

It is able to read Messages from contacts but not from any sms broadcasters.

Hey. @ayushmankd. Apologies for the delay in replying to this. Any luck?

@briankabiro Sorry, I am currently not working on the product for which this functionality was needed.

@ayushmankd. Going to close this for now as it's inactive.


Opening a file with line number / column number specified from commandline

Prerequisites

[x] Can you reproduce the problem with Debug -> Reload Without Extensions?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?
[x] Did you read the Troubleshooting guide?

Description
This is most likely a Brackets Shell issue too, but correct me if I am wrong but there doesn't seem to be a way to open a file to a specific line / column from command line.

Versions
Latest master, Win / Mac

Related: https://github.com/facebookincubator/create-react-app/pull/2637


vim 8.0.0071 gives error about "drop" parameter for ch_open call

Once I switch to a python file in my mac with vim 8.0.0071 I get the following error
Error detected while processing function kite#bufenter[8]..kite#status#status[4]..kite#utils#logged_in[4]..kite#client#logged_in[3]..<SNR>102_internal_http:
line   13:
E475: Invalid argument: drop
.....
....
..
Among many other errors this is the first one.

In this version the ch_open call does not have the "drop" parameter yet
This shows in
:help channel-open-options
{options} is a dictionary with optional entries:        channel-open-options

"mode" can be:                                          channel-mode
        "json" - Use JSON, see below; most convenient way. Default.
        "js"   - Use JS (JavaScript) encoding, more efficient than JSON.
        "nl"   - Use messages that end in a NL character
        "raw"  - Use raw messages
                                                channel-callback E921
"callback"      A function that is called when a message is received that is
                not handled otherwise.  It gets two arguments: the channel
                and the received message. Example:
        func Handle(channel, msg)
          echo 'Received: ' . a:msg
        endfunc
        let channel = ch_open("localhost:8765", {"callback": "Handle"})

                When "mode" is "json" or "js" the "msg" argument is the body
                of the received message, converted to Vim types.
                When "mode" is "nl" the "msg" argument is one message,
                excluding the NL.
                When "mode" is "raw" the "msg" argument is the whole message
                as a string.

                For all callbacks: Use function() to bind it to arguments
                and/or a Dictionary.  Or use the form "dict.function" to bind
                the Dictionary.

                Callbacks are only called at a "safe" moment, usually when Vim
                is waiting for the user to type a character.  Vim does not use
                multi-threading.

                                                        close_cb
"close_cb"      A function that is called when the channel gets closed, other
                than by calling ch_close().  It should be defined like this:
        func MyCloseHandler(channel)
                Vim will invoke callbacks that handle data before invoking
                close_cb, thus when this function is called no more data will
                be received.
                                                        waittime
"waittime"      The time to wait for the connection to be made in
                milliseconds.  A negative number waits forever.

                The default is zero, don't wait, which is useful if a local
                server is supposed to be running already.  On Unix Vim
                actually uses a 1 msec timeout, that is required on many
                systems.  Use a larger value for a remote server, e.g.  10
                msec at least.
                                                        channel-timeout
"timeout"       The time to wait for a request when blocking, E.g. when using
                ch_evalexpr().  In milliseconds.  The default is 2000 (2
                seconds).
When "mode" is "json" or "js" the "callback" is optional.  When omitted it is
only possible to receive a message after sending one.

Updated vim to 8.0.1650_1
That was the latest available in brew in mac at the time of writing.
There the ch_open function has the drop parameter and this is not an issue anymore.
I guess, the kite sidebar could warn about this. It looks like the installed vim-plugin does not support 8.0.0071 but the vim sidebar installed it anyways.

@ahakanbaba Thanks for reporting this issue!
We do check the plugin version, the currently required minimum version is VIM 8.0.0027. We'll look at the use of the unsupported value drop.

@ahakanbaba This issue has been fixed. It will be in the next release this week.
If there's any other issue with the vim plugin, please let us know.

Similarly to #103 If the fix is part of an open source project, it would be useful to see the change that fixed this.

@ahakanbaba See #103 . The fix for this issue was to drop the the parameter drop as it was not needed in this particular case.


tsconfig with paths and include causes vscode to drop intellisense on paths

From @ca0v on September 21, 2018 16:8

VSCode Version: 1.27.2
OS Version: win10

Steps to Reproduce:
this gist show the steps to reproduce but it comes down to this tsconfig.json file:
{
	"compilerOptions": {
		"baseUrl": "./",
		"paths": {
			"@app/*": ["./app/*"]
		}
	},
	"xinclude": ["app/**/*.ts"]
}

If you remove the "x" in "xinclude" intellisense immediately breaks in vscode:
import { MyApp } from "@app/myapp";

Copied from original issue: Microsoft/vscode#59116

From @ca0v on September 21, 2018 16:14
There is a workaround to this.  Keep your primary tsconfig.json file free if "files", "include" and "exclude" elements and introduce a derived tsconfig file:
{
	"extends": "./tsconfig",
	"compilerOptions": {
		"outFile": "./built/app.max.js"
	},
	"include": ["./app/**/*.ts"]
}

I believe that this is by design since folders in the test directory are not included in the tsconfig.json project that defines the path mapping. Your workaround also seems reasonable.
Someone on the TS team can verify this however

Any new info on this? The workaround isn't working for me using the latest TS and VSC versions.
I have something like this:
/project
  tsconfig.base.json
  /shared
  /apps
    /project-a
      types/
      test.ts
      tsconfig.json

tsconfig.base.json:
{
  "compilerOptions: {
    "baseUrl: "./",
    "paths": {
      "@shared/*": ["shared/*"]
    }
}

project-a/tsconfig.json
{
  "extends": "../../tsconfig.base.json",
  "include": ["./types/**/*"]
}

test.ts
import Roles from '@shared/enums/roles'; // error
import lib from 'bitcore-lib'; // no error; type definitions are in ./types/...

const roles: Roles[] = [];

If I remove the "include" in project-a/tsconfig.json, the errors are switched!

Same problem with latest version of vscode.

For me this issue was related to https://github.com/Microsoft/TypeScript-Sublime-Plugin/issues/605.  Excluding the .spec.ts files caused intellisense to break.  For now I removed the include/exclude and allow typescript to build the .spec.ts files.  Another option would be to restructure and put test files in a sibling directory to the src and provide an additional tsconfig.json.  It would be nice to exclude test files from build and have test file with intellisense.

I just spent a little time reproducing these examples. Everything shown here is expected behavior. I think I can explain what’s happening, why @ca0v’s workaround works, and why @rhyek’s workaround does not work. From this, I think it should be clear how to fix your config. TL;DR: @mjbvz is correct in his earlier comment.
What’s happening
There are two facts you need to be aware of to understand what’s happening here. The first is probably obvious; the second, I would wager, is the root of the confusion here:

For path mapping to work in a given file, that file needs to accounted for by some tsconfig.json that defines the path mapping.
The files accounted for by a tsconfig.json file are determined by its files, include, and exclude settings. If you don’t have any of them, the default is "include": ["**/*"]. If you do provide an include yourself, that overrides the default; it doesn’t add to the default.

So, in the example of the original repro, when you set "include": ["app/**/*.ts"], only files under app/ (and anything those files import) are part of your program. Files under test/ are orphaned (unless imported by files in app/), so when you open them in VS Code, TS Server infers a phantom project with default settings for them. Those default settings, of course, don’t contain your custom path mapping.
Why @ca0v’s workaround works
The workaround shown in this comment implies there is another file called tsconfig.json with default files/include/exclude settings, which means that it represents a project by itself that accounts for all files. This also implies the tsconfig content shown in the comment must be a file named something other than tsconfig.json—let’s call it tsconfig.custom.json for the sake of example. The result is that VS Code will totally ignore that file. But, if you run tsc -b tsconfig.custom.json, it won’t compile files in test/ (unless something in app/ imports them).
Why @rhyek’s workaround doesn’t work
@rhyek’s base tsconfig file is named tsconfig.base.json, which means VS Code won’t see it as a project in its own right; it will only be used where another file actually named tsconfig.json extends it (in this case, /projects/apps/project-a/tsconfig.json). But /projects/apps/project-a/tsconfig.json has "include": ["./types/**/*"], which means it only includes files in ./types, orphaning /projects/apps/project-a/test.ts!
The general solution
Adding include to your tsconfig doesn’t add more files within your project folder; it filters files down from the highly-inclusive default. The irony in all these examples is that the include was never needed, since everything that was being included would have been covered by the default. It’s quite possible these examples were distilled into simpler cases, and your real projects did need to use include in one way or another, but the takeaway here is to make sure your include covers everything that you want the compiler to know about. (My advice is always to go without any files/include/exclude until you need it.)
The specific solution, maybe
Since both of these examples mention test files, it’s worth noting that you might decide that you don’t want your test files included as part of your main program, but you still want to be able to use path mapping in them. For this to work properly from both the command line and from VS Code, the easiest way is to put your tests and sources in separate folders, each folder with its own tsconfig.json (it’s important for TS Server’s sake that it’s named exactly tsconfig.json), each of which extends a common ../tsconfig.base.json that defines the path mapping.
(If you want to mix your tests and sources in the same folder, but have your tests excluded from the main project, you can get this to work with tsc fairly easily, but TS Server / VS Code will have trouble understanding your intent because it looks for files named exactly tsconfig.json, and obviously you can’t have two files by the same name in the same folder. There may be some highly creative workaround to doing this, but I’m not prepared to make any recommendations here. I think @OliverJAsh may have an issue tracking this.)

I just spent a little time reproducing these examples. Everything shown here is expected behavior. I think I can explain what’s happening, why @ca0v’s workaround works, and why @rhyek’s workaround does not work. From this, I think it should be clear how to fix your config. TL;DR: @mjbvz is correct in his earlier comment.
What’s happening
There are two facts you need to be aware of to understand what’s happening here. The first is probably obvious; the second, I would wager, is the root of the confusion:

For path mapping to work in a given file, that file needs to accounted for by some tsconfig.json that defines the path mapping.
The files accounted for by a tsconfig.json file are determined by its files, include, and exclude settings. If you don’t have any of them, the default is "include": ["**/*"]. If you do provide an include yourself, that overrides the default; it doesn’t add to the default.

So, in the example of the original repro, when you set "include": ["app/**/*.ts"], only files under app/ (and anything those files import) are part of your program. Files under test/ are orphaned (unless imported by files in app/), so when you open them in VS Code, TS Server infers a phantom project with default settings for them. Those default settings, of course, don’t contain your custom path mapping.
Why @ca0v’s workaround works
The workaround shown in this comment implies there is another file called tsconfig.json with default files/include/exclude settings, which means that it represents a project by itself that accounts for all files. This also implies the tsconfig content shown in the comment must be a file named something other than tsconfig.json—let’s call it tsconfig.custom.json for the sake of example. The result is that VS Code will totally ignore that file. But, if you run tsc -b tsconfig.custom.json, it won’t compile files in test/ (unless something in app/ imports them).
Why @rhyek’s workaround doesn’t work
@rhyek’s base tsconfig file is named tsconfig.base.json, which means VS Code won’t see it as a project in its own right; it will only be used where another file actually named tsconfig.json extends it (in this case, /projects/apps/project-a/tsconfig.json). But /projects/apps/project-a/tsconfig.json has "include": ["./types/**/*"], which means it only includes files in ./types, orphaning /projects/apps/project-a/test.ts!
The general solution
Adding include to your tsconfig doesn’t add more files within your project folder; it filters files down from the highly-inclusive default. The irony in all these examples is that the include was never needed, since everything that was being included would have been covered by the default. It’s quite possible these examples were distilled into simpler cases, and your real projects did need to use include in one way or another, but the takeaway is to make sure your include covers everything that you want the compiler to know about. (My advice is always to go without any files/include/exclude until you need it.)
The specific solution, maybe
Since both of these examples mention test files, it’s worth noting that you might decide that you don’t want your test files included as part of your main program, but you still want to be able to use path mapping in them. The easiest way for this to work properly from both the command line and from VS Code is to put your tests and sources in separate folders, each folder with its own tsconfig.json (it’s important for TS Server’s sake that it’s named exactly tsconfig.json), each of which extends a common ../tsconfig.base.json that defines the path mapping.
(If you want to mix your tests and sources in the same folder, but have your tests excluded from the main project, you can get this to work with tsc fairly easily, but TS Server / VS Code will have trouble understanding your intent because it looks for files named exactly tsconfig.json, and obviously you can’t have two files by the same name in the same folder. There may be some highly creative workaround to doing this, but I’m not prepared to make any recommendations here. I think @OliverJAsh may have an issue tracking this.)

Point #2 definitely helped me in this case. I thought include basically says to include other files you want, i didn't thought that setting it will totally override the default value.


Histogram:  selected column is not synchronised between in-plot controls and properties panel

1.17.0
Data: any (SPGI)

Add a hisctogram
Switch the column using in-plot controls
Selected column is not synchronised between in-plot controls and properties panel

1.18.0


[Bug]: Slowness in JS value properties eval

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In the JS Objects editor updates to properties like objects or arrays are not getting reflected immediately in the bindings.
Steps To Reproduce
In the JS Objects editor, I have defined a property myVar1 which is an array and later I have updated myVar1 array with a different value set.

I have noticed that the new value does not get ingested in the bindings and continues to refer the older value.

Unless I perform a browser refresh does the value get reflected correctly.
I don't see this issue if the property is a function and any updates made here gets reflected immediately.

Public Sample App
No response
Issue video log
No response
Version
Cloud

This is because of reactive variables not being supported yet

@Nikhil-Nandagopal did not get how this is related to reactive variable; I think the problem says that JS objects sometimes stop evaluating correctly, even does not allow running the function; things start working as soon as you refresh the page.
Can you please give a bit more context I am not getting what you are talking about?

@somangshu it's a bug in the platform where variables defined with a value aren't evaluated till the next render. Since they are only evaluated once at the time of load, this is the result.


Army IDs GenTX

Please make sure your node is up and running. Join #validators Discord for help: https://discord.gg/QeJWCrE.


[Polish] “Back” button on mobile should match what we have in settings

before
after

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


perf: mobile menu perf improvements

After further perf testing it's apparent that this actually isn't an improvement, but rather the opposite. Closing without merging.


As a user. I can modify the details event/task

so that I can update task requirements accordingly


vttest: go side of things

Simple go wrappers for launching py/vttest/run_local_databse.py.
The LaunchVitess test is still not working. We'll have to
troubleshoot it later.

I'll help with launch vitess next week... This looks good otherwise. I can see the environment.go overwrite in google3. :)
(as we discussed, the myslq parameters will have to be more than just the socket, you;ll have to get port and username and stuff).

Got the import working. I'll merge this once the import is approved. Then we can re-import.


new logo update on every navbar

Related Issue
UI related issue
Fixes:  #2302
Description
Updated the old logo with the new one on the navbar.  The task has been completed successfully.

Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Replaced
the old logo with the new one on the navbar and all other pages where the old logo was displayed.
Checklist:

[X] I have made this change from my own.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

CLOSE : #2302

@RitiChandak
review this and merge it!

size of the logo is not same correct it and raise a new or reopen the PR

I did it  Sent from Mail for Windows From: Riti ChandakSent: 04 July 2024 15:35To: anuragverma108/SwapReadsCc: Heli Detroja; AuthorSubject: Re: [anuragverma108/SwapReads] new logo update on every navbar (PR #2338) image.png (view on web)image.png (view on web)size of the logo is not same correct it and raise a new or reopen the PR—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Both the issues have been closed, but I have not received any points. Why? Sent from Mail for Windows From: Riti ChandakSent: 04 July 2024 15:35To: anuragverma108/SwapReadsCc: Heli Detroja; AuthorSubject: Re: [anuragverma108/SwapReads] new logo update on every navbar (PR #2338) image.png (view on web)image.png (view on web)size of the logo is not same correct it and raise a new or reopen the PR—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Add binary feature support

FLANN supports binary feature matching using Hamming distance in a performant fashion using LSH. Unfortunately, the only way to input binary data into the library is to create a GenericArray of bytes, a Vec of bytes, or a &[u8].
This is non-ideal because we would like to store our feature as a vector of bits. The crate bitvec is the most commonly used crate for dynamic bit vectors. It's BitVec type should be used to pass bits into the API more efficiently. There is no equivalent to GenericArray that I could find for bits currently. Any suggestions would be appreciated.

Until const generics is merged, we cannot support a theoretical GenericBitArray. It isn't possible to use type operators from typenum to compute the underlying GenericArray size. This is because there is no Div or Add operator. The PartialDiv operator states:
Division as a partial function. This type operator performs division just as Div, but is only defined when the result is an integer (i.e. there is no remainder).

We could get by without the Add operation by simply implementing an Increment trait for all unsigned integers we care about, but we need Div or Shift to compute the actual number of needed bytes, thus it isn't possible to even assert at compile time that the number of elements and the bit length  are correct unless we limit it to a specific underlying type (like u8) instead of any type like BitVec does and then define a special trait for the number of bytes needed to store it and implement it for all sizes.
Needless to say, these workarounds would be complicated and limiting. Const generics support will solve this issue and allow the development of such a crate, so at least that part of this should be delayed until some form of const generics is available in the compiler. At that point we wouldn't even need unification because we could just produce the array length directly through an expression and would only need to take in the BitArray length.

Upon trying to make unit tests, I have found that hamming distance and LSH in general is not supported in the C bindings, only C++. This will need to be tabled for later.

I found a crate called bit-array which implements the desired functionality for bit slices and have forked it here.


Password with $ failing

I noticed that when using a password with a $ in it I would not be able to authenticate to the endpoint. It would return this error after some seconds:
target "<IP_ADDRESS>:50051" get request failed: "<IP_ADDRESS>:50051" GetRequest failed: rpc error: code = Unauthenticated desc = Authentication failed target "<IP_ADDRESS>:50051" get request failed: "<IP_ADDRESS>:50051" GetRequest failed: rpc error: code = Unauthenticated desc = Authentication failed
When I changed the password to something simpler it started working. Not a huge issue, but something to be aware of. Endpoint was Cisco Nexus fwiw.

How was the password set? In the cli with -p or in the config file ?

I was passing it in on the CLI with -p. Hadn't tried the config file approach.

worth trying to wrap password in single quotes like
-p 'my$pwd'

worth trying to wrap password in single quotes like
-p 'my$pwd'

🤦♂ single quotes works. I tried with double quotes to no avail before opening the issue.


Dynamic multi-channel support in viewer

This PR will add the required changes for multi-channel dynamic configuration.
Related PR: OSD viewer#55
Chaise-wide changes:

Modified the query parameter related functions in utils.js so they can be properly used in the viewer app as well.

Add an option to avoid encode/decoding query parameters (in OSD viewer, query parameters are not URL-encoded.)
Allow multiple query parameters with the same name. The previous code wouldn't properly work with ?url=val1&url=val2 as it was a simple key-value pair of strings. Now the values will be an array of strings in this case.

Viewer app changes:

Add support for reading the Image_Channel table if needed. Also modified the way we're OSD parameters (explained in the next section.)
Pass the OSD viewer parameters by message instead of query parameters.
Refactor the viewer.app.js code a bit and introduce new functions in viewer.utils.js
Add new constants that are used with the new logic.

New logic for OSD parameters:
The following is the new logic for deriving the OSD parameters (what chaise passes to OSD viewer):

There are two different categories of OSD parameters,

channel-related: url, channelName, pseudoColor, isRGB, aliasName
general: waterMark, etc

Different OSD parameters might be derived from different sources based on this priority order:

query parameter on the page

Image.uri value

Image_Channel (Image_Channel is only used for channel-related parameters.)

This means that if a specific parameter is defined on both query parameter as well as Image.uri, we will use the value defined on the query parameter. In this logic, each parameter is treated separately (in this example another parameter might be derived from Image.uri or any other sources).

Channel-related parameters are a bit different as they are treated as all-in or nothing. We first check if channel url is defined (ignoring annotation urls). If that is the case, all the other channel-related will only be derived from this source and will be ignored by others. For instance, if user navigate to a page with ?url=valid_image_url while the Image.uri value is ?url=another_valid_image_url&channelName=DAPI, we will use the url defined in the query parameter and ignore the url and channelNames defined on Image.uri.

@jrchudy I found a typo in my code that was causing limit (and other query parameters) to be ignored which in turn broke recordset test cases.
This PR is ready to be reviewed. As I mentioned in the description the only chaise related changes are changing functions related to query parameters. The rest of the changes are isolated to the viewer app itself.


DriverConstructor

DriverConstructor is not functional and has no real code that means anything.
It also does not have correct formatting.
Server will be creating a few on startup.

Fixed the formatting. Now has no errors.


Text colour issue

Hi,
I completed my writing goal today. The middle of the circle turned green when I did, which is great, but I believe my theme has made it so the text is almost invisible.
Would it be possible to add a text colour option for the words today counter please?
Thanks for the help.

Additional custom colours are available in release 0.8.4

Perfect! Thank you :)

Additional custom colours are available in release 0.8.4

Heyy. Are you here??


Duplicate slashes in permalink

Hey, awesome repo! I just saw a quick bug upon trying the application

post.data.permalink within each post in the posts array already have a dash so in the line
permalink={"https://www.reddit.com/" + post.data.permalink}
When clicking Open in Reddit button

would present a duplicate dash before the r in the url.
Just a really small bug, I hope I can make better contributions in the future!
Thanks.

Thanks so much for catching that! Approved!