a little question about paper

It's a grate job, Tanks for what you contribute.
I have a question about the paper. In the paper, you said "All one-layer MLP branches for predicting confidence use a softmax function.". But as far as I know, VCOCO dataset's verbs is multi label, I am wonder how can I use softmax to predict multi label.
Hope you can reply me
Thanks

Mainly because the one-to-one matching strategy and the definition of HOI instance. For example, given an image, their is a person sitting on a horse, and the GT contains 3 HOI categories, e.g. (person, ride, horse), (person, sit_on, horse), (person, straddle, horse).
In training, the model directly outputs 100 HOI instances, the ground truth size is also 100 with 3 positive instances and 97 null padding. That is to say, there are only 3 slots responsible for the positive predictions.
In inference, if given this image, only 3 of the 100 slots are actived, so, after filterring, the model only outputs 3 HOI instances.
To summary, in the set prediction view, we don't think the original GT is one element with 3 labels, but 3 different elements.
The 'Appendix A: Illustration of Inference Process' part may help.
I hope it helped you, thanks.

Thanks, it helps me a lot


Added support for Kotlin DSL / 1.20.4

Warning: Accept this PR after creating the "1.20.4-with-kotlin-dsl" branch!
I understand that this may not be very useful, but for some who want to use KDSL, it mb useful, and you will not have to rewrite from 1.18.2 to 1.20.4

Should the 1.20.4-with-kotlin-dsl branch be based off of 1.18-with-kotlin-dsl or the 1.20.4 branch?

branch is based on 1.20.4

Thanks!

Thanks
Glad to help!


Building native: make implicitHeaders available for compiler in SourceSet scope

Currently, there’s no full support for implicitHeaders in source set for native platforms (actually, implicitHeaders are not available for the compiler and unused).
This pull request makes implicitHeaders available in scope of SourceSet.
Topic at google groups:
https://groups.google.com/forum/#!topic/gradle-dev/qlkuCT0E5IQ

Thank you for the pull request.
Would you mind signing a contributor license agreement? This is required for legal reasons before any code contributions can be accepted into the Gradle codebase.
It may be a while before we actually merge it because of limited capacity of Gradle team to deal with pull requests at the moment.

@IUdalov looking at the mailing list discussion you linked to I don't believe this pull request implementation is inline with the comments made by Adam. I think we should start by continuing that discussion and then potentially moving on to a design spec. Take a look at our contribution guidelines for details.

@IUdalov I'm closing this pull requests as we haven't heard back from you for over a month. In case you want to pick it up again later, feel free to reopen.


[vpj][controller][router][server][test] create dummy zstd dictionary for empty push and suppress redundant dictionary fetch logs

Summary

Dictionary not created:

When there is an empty push, controller creates a dictionary using synthetic data and sends it via SOP. But when send.control.messages.directly is enabled, VPJ driver sends the SOP and not the controller, so no dictionary gets created. Moving this flow from controller to the VPJ driver code such that the dummy dictionary gets created for both the flows.
corrected some misleading logs in VPJ driver about dictionary creation for empty push scenario
Added exceptions in server's processStartOfPush to check for dictionary if ZSTD_WITH_DICT is configured.

Too many logs/retries to fetch dictionary in router

Added redundant log filter to the router logs for fetching dictionary. And a test to satisfy diff coverage.
exponential backoff for router retries (min 100ms, max 5s)
Cancelling the scheduled dictionary fetch retries when removing the version/store: potentially preventing further retires or exceptions.

How was this PR tested?
GH CI
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Thanks @gaojieliu and @nisargthakkar for the review.


Add job to create thumbnails

This should take a list of thumbnail sizes, and be cancellable.

Added in PR #96.


added auth checks on health endpoints

if you are not in a tenant and do not have a system role you will not see the hud, and if you attempt to ping the health endpoints without auth you will be rejected

so when shield runs the routine health ping, if someones not on a tenant/sysrole you want me to dump them to login (maybe with a message)?


Set the maximum length of a text field

I'm building a strict Website layout where I want to limit the maximum character length for the client.

By the example above, If the client puts an entire Shakespeare sentence in the title, it will break all the design. It would be great to limit the character count, but I miss this feature in the Sveltia CMS.
would it be possible to implement this feature using regex? For example, yaml could look like this:
  - name: test
    label: Test
    folder: src/content/test
    create: true
    fields:
      - label: Name
        name: name
        widget: string
        pattern: ['^.{0,70}$', 'Must not exceed 70 characters']  <- using regex

and the UI could show the current character count and maximum count - 65/70 like in the example in Sanity UI.

Going even further could be possible to implement the range:
pattern: ['^.{20,70}$', 'Must not exceed 70 characters']  <- using regex
and to show in the UI:
20//70
20/36/70
but I don't know at what scenario to use the min character limit.

I’m actually thinking about adding the min and max options to the String and Text widgets so that you don’t have to use a custom regex.

that would be even better for DX. 👍
you could give the nice UX addition by changing the color of the character counter to red when the pattern is not met, and to blue when the character count meets the requirements.

Yeah, I’m thinking about it. 😆

Just shipped v0.27.0 with the new options!

this is so clean from DX side:

name: test
label: Test
folder: src/content/test
create: true
fields:

label: Title
name: title
widget: string
minlength: 10
maxlength: 70


On Linux: respect XDG_CONFIG_HOME and don't store config in user's home directory

Describe the feature
The XDG Base Directory standard states that user config files should respect the XDG_CONFIG_HOME environment variable, which defaults $HOME/.config.
Currently, dbt defaults the PROFILE_DIR to ~/.dbt and doesn't attempt to read from XDG_CONFIG_HOME or ~/.config.
mv ~/.dbt ~/.config/dbt
dbt debug
...
profiles.yml file [ERROR not found]

Describe alternatives you've considered
I can locally symlink my ~/.config/dbt directory to ~/.dbt, but it's not my preference to put additional directories in my home directory.  Alternately, I could set a global DBT_PROFILES_DIR variable.
Additional context
This is only relevant to Linux users.  The appdirs pypi package makes the XDG_Base_directory standard simple to implement.
Who will this benefit?
Linux users who have far too many files/folders in their home directories.

Thanks for opening this issue @sethwoodworth. Looping in @beckjake to take a look :)

I'm surprised we don't have an issue about this yet, thanks for writing it up!
I agree, also Windows should write to %APPDATA%\dbt or something like that, and macos should write to ~/Library/Application Support/dbt (I think?). It looks like that library you linked might help on that front.
The problem here is going to be migration. This fix is going to break stuff for a lot of people! We should do it before 1.0. We can fall back to ~/.dbt/profiles.yml if we don't find it in the os-specific directory. I assume we'll still get a lot of complaints this way, but better now than later.

I found this trying to figure out why dbt didn't support xdg properly. It really should.

@dbeatty10 you closed the issue as "not planned" but didn't touch my PR (#11108). Did you mean to reject it? If so, was this like a "Yeah, that'd be nice but we don't have time to do this and make sure it's 100% correct just now" kinda thing? Or was there some reason why y'all didn't like it?

Hey @dradetsky !
Quick explanation: the stale bot closed this as "completed" (done, closed, fixed, resolved) on Jun 23, 2022, but it should have closed it as "not planned" (won't fix, can't repro, duplicate, stale):

(This affects a lot of older issues that were closed by the stale bot, but it is does not affect more recent issues closed by the stale bot.)
So I made a quick fix to reflect that it was not completed, but I probably should have also then immediately re-opened for further triage due to your linked PR. Re-opening now 👍

@dradetsky I removed the good_first_issue label as we further assess if this is something we'd be still interested in adopting or not.
Overview
The folder location where dbt searches for connection profiles is absolutely fundamental to how dbt operates, so any changes in this area of code introduce substantial risk of regressions. So it isn't a change we can take lightly.
Workaround
Did you consider or try out the suggestion in https://github.com/dbt-labs/dbt-core/issues/2515#issuecomment-678279755?
export DBT_PROFILES_DIR="$(XDG_CONFIG_HOME)/dbt"

This looks like a pretty lightweight approach to look in the conventional XDG folder without needing to make any changes to dbt itself.
Unless overridden by the --profiles CLI flag, dbt will look in the folder specified by DBT_PROFILES_DIR for database connection profiles [1].

The folder location where dbt searches for connection profiles is absolutely fundamental to how dbt operates, so any changes in this area of code introduce substantial risk of regressions

Yes, a proper fix for this would probably involve automatic migration of legacy profile dirs, or falling back to the legacy dir if the primary dir doesn't have the profiles, or both. I didn't really want to write all that without talking to someone given that this was likely going to involve talking to DX peoples anyhow.

Did you consider or try out the suggestion

I mean, no, but I saw it and I'm sure it would work if all I wanted was to fix this for me asap. I just did this on the grounds that using xdg dirs is the Correct Thing(tm) and dbt should do the Correct Thing(tm) if possible. I also thought (but didn't know for sure) that the defaults used by appdirs for win/mac would also be an improvement over the status quo.
Using the env var is probably what you should recommend in the short term. I would still recommend that the change in my PR (along with necessary migration changes) should be made. However, that will involve getting some stakeholders to agree on what kinds of migration/fallback/testing is necessary to get P(regression) down to whatever value said stakeholders think is good enough, so it can't be done immediately.
For that reason, I think I'll pull out the change that fixes the circular import issue & induces copypasta and submit that. That will make any 2nd PR smaller & simpler, which is always good.


Add Link to keycloak password edit

The functionality is already in keycloak but should be avaiable from frontend
next to log out menu entry

dupplicate from #146


Add Rndc keys generator

So far, the Designate bind9 container was being held forever, expecting the user to run manually bind9 (as a temporary workaround).
This patch adds an Rndc key generator, as the first piece of the Designate packages to run bind9.

/lgtm


AttributeError: module 'tensorflow' has no attribute 'python_io'

Hi there, updates have depreciated python_io to io.
This error is generated in tfrecord init.py by the following line:
_tf_compression_revmap = {
y: x for x, y in tf.python_io.TFRecordOptions.compression_type_map.items()
if y != ''
}
which needs to be changed from tf.python.io to tf.io to resolve the error.
_tf_compression_revmap = {
y: x for x, y in tf.io.TFRecordOptions.compression_type_map.items()
if y != ''
}

Looks like GOOG is on a path to deprecate access to TFRecord api outside of the OPs...
Maybe the best solution is to replicate protocol buffer definitions and serialization logic (checksumming) and remove TF dependency altogether.
To better understand this, what is your use case?

To use datasets created in TF in other frameworks (PyTorch, etc)
To create datasets in pure python and then use them from TF
Create and consume datasets outside of TF

See also: https://github.com/podgorskiy/DareBlopy


Shortcut Guide Not working after some time

Shortcut Guide will not work after sometime of launching PowerToys. I have to restart PowerToys to make it work again.

@mohdimaduddeen
are you running PowerToys as a user or as an administrator?
Be aware that the Shortcut Guide will not appear if the foreground application is running elevated (aka as administrator).
This may also be a duplicate of https://github.com/microsoft/PowerToys/issues/3090

I have tried both ways, as a user and as a admin. Both have same problem. I have to either restart PowerToys Or restart my PC. It usually happens when it is not used for too long. Sent from Mail for Windows 10 From: Enrico GiordaniSent: 23 June 2020 09:33 PMTo: microsoft/PowerToysCc: mohdimaduddeen; MentionSubject: Re: [microsoft/PowerToys] Shortcut Guide Not working after some time (#4446) @mohdimaduddeenare you running PowerToys as a user or as an administrator?Be aware that the Shortcut Guide will not appear if the foreground application is running elevated (aka as administrator).This may also be a duplicate of #3090—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

@mohdimaduddeen
it seems you have the same issue as described in https://github.com/microsoft/PowerToys/issues/3090
I'll close this one as duplicate.
Thank you.


conform class vcl::vect::Vector to simulated iterators usage

relates to issue #117 .
Let's verify per code review taht simulated iterators are used everywhere in the code of this class.

completed, tested.


Runtime error during training

Hi Team,
I've been using SetFit with a variety of different datasets and base models and this has been a persistent issue. It happens during trainer.train().
The error is RuntimeError: [enforce fail at inline_container.cc:471] . PytorchStreamWriter failed writing file data/158: file write failed, I previously thought it could be a memory issue and have done the following to mitigate:

trying only 1 epoch, batch size 1, max_steps 1, eval_max_steps 1
reducing the dataset to only 100 samples
checked !df -Th and there is plenty of memory

Is this a familiar error?
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
File /databricks/python/lib/python3.10/site-packages/torch/serialization.py:441, in save(obj, f, pickle_module, pickle_protocol, _use_new_zipfile_serialization)
    440 with _open_zipfile_writer(f) as opened_zipfile:
**--> 441     _save(obj, opened_zipfile, pickle_module, pickle_protocol)**
    442     return

File /databricks/python/lib/python3.10/site-packages/torch/serialization.py:668, in _save(obj, zip_file, pickle_module, pickle_protocol)
    667 num_bytes = storage.nbytes()
**--> 668 zip_file.write_record(name, storage.data_ptr(), num_bytes)**

RuntimeError: [enforce fail at inline_container.cc:471] . PytorchStreamWriter failed writing file data/158: file write failed

During handling of the above exception, another exception occurred:

RuntimeError                              Traceback (most recent call last)
File <command-3835036500502814>, line 1
**----> 1 trainer.train()**
      3 metrics = trainer.evaluate()
      5 # Change

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/trainer.py:410, in Trainer.train(self, args, trial, **kwargs)
    405 train_parameters = self.dataset_to_parameters(self.train_dataset)
    406 full_parameters = (
    407     train_parameters + self.dataset_to_parameters(self.eval_dataset) if self.eval_dataset else train_parameters
    408 )
**--> 410 self.train_embeddings(*full_parameters, args=args)**
    411 self.train_classifier(*train_parameters, args=args)

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/trainer.py:462, in Trainer.train_embeddings(self, x_train, y_train, x_eval, y_eval, args)
    459 logger.info(f"  Total optimization steps = {total_train_steps}")
    461 warmup_steps = math.ceil(total_train_steps * args.warmup_proportion)
**--> 462 self._train_sentence_transformer(
    463     self.model.model_body,
    464     train_dataloader=train_dataloader,
    465     eval_dataloader=eval_dataloader,
    466     args=args,
    467     loss_func=loss_func,
    468     warmup_steps=warmup_steps,
    469 )**

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/trainer.py:655, in Trainer._train_sentence_transformer(self, model_body, train_dataloader, eval_dataloader, args, loss_func, warmup_steps)
    652 self.state.epoch = epoch + (step + 1) / steps_per_epoch
    653 self.control = self.callback_handler.on_step_end(args, self.state, self.control)
**--> 655 self.maybe_log_eval_save(model_body, eval_dataloader, args, scheduler_obj, loss_func, loss_value)**
    657 if self.control.should_epoch_stop or self.control.should_training_stop:
    658     break

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/trainer.py:715, in Trainer.maybe_log_eval_save(self, model_body, eval_dataloader, args, scheduler_obj, loss_func, loss_value)
    712     loss_func.train()
    714 if self.control.should_save:
**--> 715     checkpoint_dir = self._checkpoint(self.args.output_dir, args.save_total_limit, self.state.global_step)**
    716     self.control = self.callback_handler.on_save(self.args, self.state, self.control)
    718     if eval_loss is not None and (self.state.best_metric is None or eval_loss < self.state.best_metric):

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/trainer.py:770, in Trainer._checkpoint(self, checkpoint_path, checkpoint_save_total_limit, step)
    767         shutil.rmtree(old_checkpoints[0]["path"])
    769 checkpoint_file_path = str(Path(checkpoint_path) / f"step_{step}")
**--> 770 self.model.save_pretrained(checkpoint_file_path)**
    771 return checkpoint_file_path

File /databricks/python/lib/python3.10/site-packages/huggingface_hub/utils/_deprecation.py:31, in _deprecate_positional_args.<locals>._inner_deprecate_positional_args.<locals>.inner_f(*args, **kwargs)
     29 extra_args = len(args) - len(all_args)
     30 if extra_args <= 0:
**---> 31     return f(*args, **kwargs)**
     32 # extra_args > 0
     33 args_msg = [
     34     f"{name}='{arg}'" if isinstance(arg, str) else f"{name}={arg}"
     35     for name, arg in zip(kwonly_args[:extra_args], args[-extra_args:])
     36 ]

File /databricks/python/lib/python3.10/site-packages/huggingface_hub/hub_mixin.py:64, in ModelHubMixin.save_pretrained(self, save_directory, config, repo_id, push_to_hub, **kwargs)
     61 save_directory.mkdir(parents=True, exist_ok=True)
     63 # saving model weights/files
**---> 64 self._save_pretrained(save_directory)**
     66 # saving config
     67 if isinstance(config, dict):

File /local_disk0/.ephemeral_nfs/envs/pythonEnv-4c4a8968-e6fe-4ab2-9fac-b5ec2d26e6b7/lib/python3.10/site-packages/setfit/modeling.py:695, in SetFitModel._save_pretrained(self, save_directory)
    685     json.dump(
    686         {
    687             attr_name: getattr(self, attr_name)
   (...)
    692         indent=2,
    693     )
    694 # Save the body
****--> 695 self.model_body.save(path=save_directory, create_model_card=False)****
    696 # Save the README
    697 self.create_model_card(path=save_directory, model_name=save_directory)

File /databricks/python/lib/python3.10/site-packages/sentence_transformers/SentenceTransformer.py:375, in SentenceTransformer.save(self, path, model_name, create_model_card, train_datasets)
    372         model_path = os.path.join(path, str(idx)+"_"+type(module).__name__)
    374     os.makedirs(model_path, exist_ok=True)
**--> 375     module.save(model_path)**
    376     modules_config.append({'idx': idx, 'name': name, 'path': os.path.basename(model_path), 'type': type(module).__module__})
    378 with open(os.path.join(path, 'modules.json'), 'w') as fOut:

File /databricks/python/lib/python3.10/site-packages/sentence_transformers/models/Transformer.py:121, in Transformer.save(self, output_path)
    120 def save(self, output_path: str):
**--> 121     self.auto_model.save_pretrained(output_path)**
    122     self.tokenizer.save_pretrained(output_path)
    124     with open(os.path.join(output_path, 'sentence_bert_config.json'), 'w') as fOut:

File /databricks/python/lib/python3.10/site-packages/transformers/modeling_utils.py:1847, in PreTrainedModel.save_pretrained(self, save_directory, is_main_process, state_dict, save_function, push_to_hub, max_shard_size, safe_serialization, variant, **kwargs)
   1845         safe_save_file(shard, os.path.join(save_directory, shard_file), metadata={"format": "pt"})
   1846     else:
**-> 1847         save_function(shard, os.path.join(save_directory, shard_file))**
   1849 if index is None:
   1850     path_to_weights = os.path.join(save_directory, _add_variant(WEIGHTS_NAME, variant))

File /databricks/python/lib/python3.10/site-packages/torch/serialization.py:440, in save(obj, f, pickle_module, pickle_protocol, _use_new_zipfile_serialization)
    437 _check_save_filelike(f)
    439 if _use_new_zipfile_serialization:
**--> 440     with _open_zipfile_writer(f) as opened_zipfile:**
    441         _save(obj, opened_zipfile, pickle_module, pickle_protocol)
    442         return

File /databricks/python/lib/python3.10/site-packages/torch/serialization.py:291, in _open_zipfile_writer_file.__exit__(self, *args)
    290 def __exit__(self, *args) -> None:
**--> 291     self.file_like.write_end_of_file()**

RuntimeError: [enforce fail at inline_container.cc:337] . unexpected pos 201721536 vs 201721424

Hello!
Hmm, I haven't seen this error before. It seems to occur when checkpointing during training. By default, this happens every 500 steps (see docs here).
Other users have experienced similar issues: https://stackoverflow.com/questions/64206070/pytorch-runtimeerror-enforce-fail-at-inline-container-cc209-file-not-fou and they attributed it to either:

A corrupted torch file - perhaps clearing the cache ~/.cache/torch/sentence_transformers/... helps?
Not enough space.

Alternatively, you can set save_strategy="no" to prevent the model from doing any checkpointing during training. That should help, although you might want to save the final model & doing so might still crash if something is corrupted.

Tom Aarsen

Thank you for the tip, changing save_strategy solved the issue - seems to be space related.


Add_test_methods

add more tests

Codecov Report

Merging #11 (85e7854) into master (1321d5c) will increase coverage by 10.69%.
The diff coverage is 97.16%.

@@             Coverage Diff             @@
##           master      #11       +/-   ##
===========================================
+ Coverage   50.40%   61.10%   +10.69%
===========================================
  Files          51       53        +2
  Lines        4291     4412      +121
===========================================
+ Hits         2163     2696      +533
+ Misses       2128     1716      -412

Flag
Coverage Δ

uitests
61.10% <97.16%> (+10.69%)
:arrow_up:

unittests
61.10% <97.16%> (+10.69%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

My_AutoML/_scaling/_scaling.py
77.84% <25.00%> (+7.04%)
:arrow_up:

My_AutoML/_base.py
52.41% <100.00%> (+1.17%)
:arrow_up:

tests/test_balancing/test_balancing.py
100.00% <100.00%> (ø)

tests/test_imputer/test_imputer.py
100.00% <100.00%> (ø)

tests/test_model/test_model.py
100.00% <100.00%> (ø)

tests/test_scaling/test_scaling.py
100.00% <100.00%> (ø)

tests/test_utils/test_utils.py
100.00% <100.00%> (ø)

My_AutoML/_feature_selection/_advance.py
64.28% <0.00%> (-0.30%)
:arrow_down:

My_AutoML/_model/_gam.py
70.27% <0.00%> (ø)

My_AutoML/_utils/_stat.py
34.81% <0.00%> (+0.74%)
:arrow_up:

... and 10 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Push 5cd3c6d75cbce6fe5c82c52c832c7eaafd4b0a2f solves a existed problem that AutoTabular (AutoTabularRegression, AutoTabularClassification) can not run multiple times in one run (only the first fit will be successful while later runs will fail to initialize the hyperparameter space. The problem comes when the default hyperparameter space is modified for evaluation and maintained modified, which causes the hyperparameter space failed to intialize.


Test: Fix GRPC issue on Kubernetes

The namespaces that are waiting for Cilium was default so the
CiliumPolicyAction didn't work correctly and didn't wait until policy
was OK.
Fix #4784
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-nightly

test-nightly

test-me-please

test-nightly

Spoke with @eloycoto this can be merged since the tests that are failing on nightly are not related with GRPC and they will be fixed later on.


:recycle: Refactor mlflow configuration to match mlflow API

Description
Close #77
Development notes
The config entries are renamed for readibility and consistency with the mlflow API.
The mlflow.yml now looks like this:
server:
  mlflow_tracking_uri: mlruns
  stores_environment_variables: {}
  credentials: null

tracking:
  disable_tracking:
    pipelines: []
  experiment:
    name: {{ python_package }}
    restore_if_deleted: True
  run:
    id: null
    name: null
    nested: True
  params:
    dict_params:
      flatten: False
      recursive: True
      sep: "."
    long_params_strategy: fail

ui:
  port: "5000" # the port to use for the ui. Use mlflow default with 5000.
  host: "<IP_ADDRESS>"  # the host to use for the ui. Use mlflow efault of "<IP_ADDRESS>".

Checklist

[x] Read the contributing guidelines
[x] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Update the documentation to reflect the code changes
[ ] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[ ] Add tests to cover your changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Codecov Report

Merging #267 (93dd63b) into master (b71ecf2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #267   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           30        30
  Lines          897       907   +10
=========================================
+ Hits           897       907   +10

Impacted Files
Coverage Δ

kedro_mlflow/framework/cli/cli.py
100.00% <ø> (ø)

kedro_mlflow/config/kedro_mlflow_config.py
100.00% <100.00%> (ø)

kedro_mlflow/extras/extensions/ipython.py
100.00% <100.00%> (ø)

kedro_mlflow/framework/hooks/node_hook.py
100.00% <100.00%> (ø)

kedro_mlflow/framework/hooks/pipeline_hook.py
100.00% <100.00%> (ø)

kedro_mlflow/framework/hooks/utils.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 b71ecf2...93dd63b. Read the comment docs.


No jobs showing in completed

I've just set up bull dashboard and run a few jobs, which I could see in the "active" tab. However, when done, they just disappear, they don't show in completed. I guess to get this to work I need to store completed jobs somewhere, but I can't seem to figure out how this is done. Help would be amazing!

Hi @jasperkennis, your jobs are disappearing probably because of the 'removeOnComplete' flag that you pass when you create the job.
 Thanks, I completely forgot I had added that...


[AMBARI-24451] ambari.ldap.advanced.group_mapping_rules does not work, LDAP sync does not add admin roles for configured group(s)

What changes were proposed in this pull request?
If ambari.ldap.advanced.group_mapping_rules is set, Ambari does not add ambari admin roles to users with the specified group(s).  The AmbariLdapConfiguration is uninitialized in Users.java class, so the adminGroupMappings value is the default "Ambari Administrators" not the configured one(s).
The solution is to use the AmbariLdapConfigurationProvider to get an instance of a AmbariLdapConfiguration when needed. This way the current configuration data is always used.
How was this patch tested?
Manually tested
mvn clean test package  -DthreadCount=2 -pl ambari-server
...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 30:22 min
[INFO] Finished at: 2018-08-10T12:14:35-04:00
[INFO] Final Memory: 103M/1261M
[INFO] ------------------------------------------------------------------------

Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3554/
Test PASSed.


ENH: Add stream processing capability to ShortTimeFFT

Is your feature request related to a problem? Please describe.
This functionality is needed to process incoming signals (e.g. audio, sensor measurements) on the fly.
Describe the solution you'd like.
I'd like to have the option to use an internal buffer in SFT to avoid having to handle samples outside the SFT. stft should add incoming data to buffer and calculate the stft if the buffer contains enough samples.
SFT = ShortTimeFFT(w, hop=10, fs=1/T_x, mfft=200, scale_to='magnitude', buffered=True)
input_device = DataSource()
output = DataSink()

while True:
    x = input_device.read_chunk()
    spectra = SFT.stft(x) # can be empty array up to multiple spectra
    output.write_chunk(spectra)

The API of the buffered mode is just an example, there might be better alternatives.
Describe alternatives you've considered.
The alternative is creating a buffer manually and processing the buffer chunkwise. If window overlapping is required, it is probably easier to use classic fft function instead of ShortTimeFFT.
Additional context (e.g. screenshots, GIFs)
No response

Thanks for the suggestion @F-M-F-B.

I'd like to have the option to use an internal buffer in SFT to avoid having to handle samples outside the SFT.

It seems to me like you could make that comment for many functions in scipy.signal and scipy.fft? It is a feature SciPy doesn't provide anywhere, so it seems odd to me to try and support it in only a single function. Is there a reason that this cannot be done outside of SciPy?

It seems to me like you could make that comment for many functions in scipy.signal and scipy.fft? It is a feature SciPy doesn't provide anywhere, so it seems odd to me to try and support it in only a single function.
In my opinion, the difference between other functions and this one is the statefulness. All functions in fft are "standalone" and do not have an associated class instance. The only other potential candidate would be ZoomFFT but I think there are many more use cases for ShortTimeFFT.
To put it briefly, the statefulness of the ShortTimeFFT makes me wanting that functionality inside SciPy. Implementing this feature for most other functions would add state to them, here the API already requires a state.

Is there a reason that this cannot be done outside of SciPy?
No, it can easiliy be done outside of SciPy. Although I think the use case of "real time" application is not uncommon, so handling this outside SciPy leads to duplicated code/functionality across projects and users.

When writing the ShortTimeFFT, I pondered quite a bit on how to implement a generic incremental processing (or streaming) functionality. I came to the conclusion that custom data management requirements, like streams, are better handled outside the ShortTimeFFT class.  The reason is that for different applications, there exist contradicting objectives, which call for different implementations, e.g.:

Minimize latency by calculating each slice as soon there is enough data versus maximizing data throughput by utilizing the multiprocessing capabilities of the FFT backend.
Minimize RAM usage by implementing a stream architecture versus keeping all data in RAM (e.g., for a recording audio editor)
...

Hand-rolling your own solution boils down to reimplementing this for loop:
        S = np.zeros(S_shape, dtype=complex)
        for p_, x_ in enumerate(self._x_slices(x, k_offset, p0, p1, padding)):
            if detr is not None:
                x_ = detr(x_)
            S[..., :, p_] = self._fft_func(x_ * self.win.conj())

The _x_slices method just provides slices of the input array, as described here.
Unless there is a widely used streaming format/implementation, which would be useful to support, I would vote to close this issue, as "not planned". Does this make sense, @F-M-F-B?

Thank you for laying down your thoughts you had during the development. I agree that there are different requirements for different applications. I'm fine with closing this issue as "not planned".


Use a faster series for calculating the Riemann R function

Use the Gram series
$$R(x) = 1 + \sum_{k=1}^\infty \frac{(\ln x)^k}{\zeta(k + 1) \cdot k \cdot k! }$$
to calculate the Riemann R function. I used a precalculated table for $\zeta(k+1)$. Since this is (almost?) as fast as the series for $\text{li}(x)$, I took the liberty to just use $R(x)$ everywhere.
I also updated the Newton iteration to use $R'(x)$ instead of the approximation $\frac1{\ln x}$. This guarantees quadratic convergence and reduces the number of iterations needed, so it should outweigh the cost of calculating $R'(x)$. The derivative $R'(x)$ can be calculated as
$$R'(x) = \frac1{x \ln x} \sum_{k=1}^\infty \frac{(\ln x)^k}{\zeta(k + 1) \cdot k!}.$$
For the initial guess, I used the Cesaro formula (see https://en.wikipedia.org/wiki/Prime_number_theorem#Approximations_for_the_nth_prime_number) with some modifications that make it work better with smaller values of $x$ (see https://www.desmos.com/calculator/4jm9ulxsvh).

Thanks, I am definitely interested in a faster R(x)!
In fact R(x) is even more important in my other primecount project. I see you have a zetaInv lookup table with a "precision of 128 bits". Does this mean R(x) can be computed fairly precisely (similar to my current implementation) up to 2^128? (I am mainly interested in understanding the upper limit of your implementation)
It will take me some time to study your implementation and throughly test it before I can eventually merge it. Note that I will take care to make the required changes in my other primecount project.

In short, the value of $R(x)$ can be computed as accurately as the floating point type allows (as long as the type has at most 128 mantissa bits) for any value of $x$.
The values of zetaInv have "a precision of 128" bits in the sense that if some platform were to have a long double with a mantissa of 128 bits, the constants have enough decimal digits to specify the exact value of this 128-bit long double that is closest to $1/\zeta(k)$. This may be a bit overkill since long double has usually less than 128 bits of precision.
Let's see how precisely we can calculate $R(x)$ with these values. Consider an approximation
$$R_\text{approx}(x) = 1 + \sum_{k=1}^\infty \frac{(\ln x)^k}{k \cdot k!} \cdot \text{zetaInv}[k]$$
of $R(x)$ using the values of zetaInv rounded to those floating point representations. Note that "zetaInv[k] = 1" if k >= 128. Since $0.5 < 1/\zeta(k+1) \le 1$, the rounding error is bounded by $|\text{zetaInv}[k] - 1/\zeta(k+1)| \le 2^{-129}$.
Then we can calculate a bound on the absolute error:
$$|R_\text{approx}(x) - R(x)| = \left|\sum_{k=1}^\infty \frac{(\ln x)^k}{k \cdot k!} \cdot (\text{zetaInv}[k] - 1/\zeta(k+1))\right|\le \sum_{k=1}^\infty \frac{(\ln x)^k}{k \cdot k!} \cdot |\text{zetaInv}[k] - 1/\zeta(k+1)|,$$
which can be furher bounded by
$$\sum_{k=1}^\infty \frac{(\ln x)^k}{k \cdot k!} \cdot |\text{zetaInv}[k] - 1/\zeta(k+1)| \le \sum_{k=1}^\infty \frac{(\ln x)^k}{k!} \cdot 2^{-129} = 2^{-129} \exp(\ln(x)) = x / 2^{129}.$$
Thus, the absolute error $|R_\text{approx}(x) - R(x)|$ would be less than ${x}/{2^{129}}$ for any value of $x$.
Of course, the above ignores any other floating point rounding that would happen in practice. The series would need to be truncated, but it turns out that summing the first $\max(n, 2 e \ln x )$ terms (in practice probably fewer) is enough to reach a maximum error of $2^{-n}$.
Since $\zeta(k) \approx 1 + 1/2^k$ for large $k$, it turns out that if we want to calculate $1/\zeta(k+1)$ to $n$ bits of precision, we only need to precompute about the first $n$ values. The rest can be safely set to $1$.
In conclusion, if you wanted to calculate $R(x)$ with a maximum absolute error of $x/2^{n}$, you would need:

a floating point type with at least $n$ mantissa bits,
a precomputed table of about the first $n$ values of $1/\zeta(k+1)$ computed to $n$ bits,
to evaluate the sum up to $\max(n, 2 e \ln x )$ terms.

You can see a derivation of the Gram series here: https://math.stackexchange.com/questions/1506980/could-someone-explain-how-the-gram-series-relates-to-riemanns-function

@nipzu I have now merged your code into the master branch https://github.com/kimwalisch/primesieve/pull/145. Your code is really of excellent quality :blush:
I made a few changes on top of your code:

I renamed nthPrimeApprox.cpp to RiemannR.cpp.
I added -R and --R-inverse command-line options to the primesieve application, so that it is easier to play around with those new functions.
I simplified your code e.g. instead of a zetaInv lookup with 1/Zeta[k+1] values I now have a zeta lookup table with Zeta[k] values. In my tests this did not deteriorate the performance and accuracy. I also merged your 2 for loops (k < 128 & k >= 128) into a single loop.
I have added more documentation.


Submit button rendering off screen, no scroll, can't submit.

hey,
first off thanks for providing a page like this where people can practice up on their skills!
so the issue that i'm having is that when i go to a kata, the screen renders larger than my screen, and there's no scroll, and though I can barely see the top of the submit button, it won't let me click on it.
while there might be a way to get there by keyboard shortcut, I could not figure that out either (but hey maybe i'm a n00b and this is the real challenge)
I'm on a MacBook Pro (13-inch, Early 2011) (yeah i know it's a dinosaur but it's still chuggin)
2.3ghz intel i5
8 gb ram
intel hd graphics 3000 512mb
google chrome Version 68.0.3440.106
i tried resizing the window but that didn't seem to help

you can barely see it peeking out at the bottom right there.... that's what i got..

i was able to zoom out the screen, and get it to show, so it works for now.. but perhaps this can be avoided some kinda way...

Closing as duplicate of #1214


[WIP] Fix drone clone on arm - DO NOT MERGE

For the record: this was broken by https://github.com/drone/drone-git/pull/48


Unable to minimize cell in REPL view after expansion.

Testing microsoft/vscode-python#23484
Minimized:

Maximized:

Unable to minimize cell after expansion. Also consider using minimize/maximize icon (shown below) instead of using ... as the UI affordance.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Logo for Cloud Create is needed

Hi Tri,
we need a logo for Cloud Create for a proper visualization on main page: https://docs.otc.t-systems.com/index.html
KR
Tino

Hi Tino,
Please find the logo in the attachment.

Great, thanks. Has been added and will be merged in a minute.


Update package.json

fixes #431

Superceded by https://github.com/keystonejs/keystone/pull/9102, which has an updated prisma version which resolves this problem

Thanks for the contribution though @Jeanclaudeaoun :yellow_heart:


chore(devcontainer): fix docker devcontainer behaviour

Simplify the Dockerfile to only install poetry and other dependencies in the devcontainer
Extend the devcontainer to run the poetry install command after creation of the environment
Update the devcontainer documentation

Fixes #230

With the latest reply about the Docker performance on https://github.com/Deltares/HYDROLIB-core/issues/230#issuecomment-1160039200, this PR is now ok and I will merge it.


Add WaterDrops

Project URL
https://github.com/LeFal/WaterDrops
Description
Simple WaterDrops Animation
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 4 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

309
404
https://github.com/applepride/pidor

1560
404
https://github.com/mancunianetz/iCheckbox

2650
301
http://codewithchris.com/ redirects tohttps://codewithchris.com/

2663
Error
http://jamesonquave.com/blog/tutorials/  Connection timed out - connect(2) for "jamesonquave.com" port 80

  Generated by :no_entry_sign: Danger


[Windows] Writes corrupted image

Hey Arafatk,
when using this library under Windows, it saves the .png file in silence, but the file cannot open.
The problem is that Gnuplot under windows has a problem with the '\' in the filepaths

I could fix this problem with a quick and dirty replacing all '\' with '/'

Just FYI.
PS: Nice library, though.

@Reawel That fix would be awesome thanks a lot


Upgrade to Grafana 9.1.0

Codecov Report

Merging #39 (cef4a2a) into main (224fed5) will increase coverage by 0.82%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
+ Coverage   74.45%   75.27%   +0.82%
==========================================
  Files          41       41
  Lines         458      453       -5
  Branches       58       56       -2
==========================================
  Hits          341      341
+ Misses        104       99       -5
  Partials       13       13

Impacted Files
Coverage Δ

...c/volkovlabs-balenasupervisor-datasource/api/v2.ts
84.88% <ø> (+4.66%)
:arrow_up:

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


Verify if the pkg is corrupted before installing.

Summary
pkg need to verified before the installation.
Branch or installer

Installer: OS X
Version: 0.16.2

Screenshots
Before installation it must check if the pkg is not corrupted.

Corrupted pkg will failed during installation.

We've experienced this today when we've downloaded the Bamboo builds using our mobile internet.
Run pkgutil --check-signature ./KA-Lite.pkg to check a .pkg.
This is the output if the .pkg is NOT corrupted:
$ pkgutil --check-signature ./KA-Lite.pkg
Package "KA-Lite.pkg":
   Status: signed by a certificate trusted by Mac OS X
   Certificate Chain:
    1. Developer ID Installer: Foundation for Learning Equality, Inc. (H83B64B6AV)
       SHA1 fingerprint: D5 8A 4C 0F F0 03 A1 86 D4 8E 63 07 1B 16 B7 71 5D EE 96 FA
       -----------------------------------------------------------------------------
    2. Developer ID Certification Authority
       SHA1 fingerprint: 3B 16 6C 3B 7D C4 B7 51 C9 FE 2A FA B9 13 56 41 E3 88 E1 86
       -----------------------------------------------------------------------------
    3. Apple Root CA
       SHA1 fingerprint: 61 1E 5B 66 2C 59 3A 08 FF 58 D1 4A E2 24 52 D1 98 DF 6C 60

If it emits anything else, like "checksum is invalid" -- means the .pkg downloaded is corrupt.

@mrpau-richard is this still a valid issue that needs to be fixed?

@mrpau-eugene Yeah, this issue still valid to be fix during the installation of .pkg KA Lite mac installer.

I don't understand how this can be implemented at all.  A user will download an installer which will now come in .dmg format.  So I'm not sure how this will become an issue now.
@mrpau-richard Can you explain what you want to achieve on this issue?

As I understand these search results, verification of a .pkg in a .dmg is automatic?
https://encrypted.google.com/search?num=30&hl=en&q=mac+skip+verifying

@cpauya I think this issue can't be reproduced if the .pkg installer is inside the .dmg file. Because if the dmg is corrupted and there's no way we can run the .pkg installer.
I think this issue is already fixed.

Let's go with that, and reopen if someone can reproduce with a broken .dmg
If you wanna test it, you can just corrupt the .dmg
https://unix.stackexchange.com/questions/222359/how-to-corrupt-an-archive-file-in-a-controlled-way


[DEVX-1363] Add support for Sauce report format

Add support for sauce-test-report.json
Example: https://app.saucelabs.com/tests/5fe591ff74c04bc0954536fb38db013d

Testcafe reporters support an output option to write the report to disk (https://testcafe.io/documentation/402656/reference/testcafe-api/runner/reporter). When I use that option, I think it makes sense to get the actual sauce report but instead I get:
 Running tests in:
 - Chrome 95.0.4638.69 / macOS 10.15.7

 tests/fixtures/sauceswag-ok/tests/sauceswag.ok.test.js

  Getting Started Sauce demo
  ✓ SwagLabs username not set
  ✓ SwagLabs locked user login
  ✓ SwagLabs standard user login (screenshots:
  ./sauce-testcafe-runner/screenshots/2021-11-23_11-25-03/test-3/Chrome_95.0.4638.69_macOS_10.15.7/1.png)

  Sauce Labs Report:
  https://app.saucelabs.com/tests/daaf15bd96d247b7b65545bc1f756611

 3 passed (11s)


Fix header paths

Applying solution introduced by @chrisbianca at #310.
This is change is backwards compatible.

Probably closes all of these:
https://github.com/react-native-community/react-native-google-signin/pull/333
https://github.com/react-native-community/react-native-google-signin/pull/310
https://github.com/react-native-community/react-native-google-signin/issues/372
https://github.com/react-native-community/react-native-google-signin/issues/357
https://github.com/react-native-community/react-native-google-signin/issues/338
https://github.com/react-native-community/react-native-google-signin/issues/307

Working here


Add offline maps server to air-gapped.yml

maps:
depends_on:
es01:
condition: service_healthy
image: docker.elastic.co/elastic-maps-service/elastic-maps-server-ubi8:8.14.3
container_name: maps
restart: always
ports:
- 8088:8080
volumes:
- certs:/usr/src/app/server/certs
- certs:/etc/ssl/certs
- ./config/elastic-maps-server.yml:/usr/src/app/server/config/elastic-maps-server.yml
- ./maps/planet.mbtiles:/usr/src/app/data/planet.mbtiles

@jonwelch28323
Hey, I fixed the indentation. If you're not planning on making a pull request, I'll gladly add it to my fork.  Just giving you a heads up! 😄
maps:
  image: docker.elastic.co/elastic-maps-service/elastic-maps-server-ubi8:8.14.3
  container_name: maps
  restart: always
  ports:
    - 8088:8080
  volumes:
    - certs:/usr/src/app/server/certs
    - certs:/etc/ssl/certs
    - ./config/elastic-maps-server.yml:/usr/src/app/server/config/elastic-maps-server.yml
    - ./maps/planet.mbtiles:/usr/src/app/data/planet.mbtiles

Absolutely, please use it. I wasn't planning to do a pull request.

feature merged into main (thank you)


Addition of the correct linking of a community URL.

Replaced the correct URL for a community server

Will merge this into 2.0.0

Not merged after u said u merge it for 2.0.0. 🤔😏


[UI/UX] Handling tables

Tables have always been quite an issue and their rendering can become quite terrible in CSS multicol (cells being cut off, vertical padding not being properly handled, etc.).
There’s no easy way out there, and it’s probably up to authors to find solutions but we can at least try reaching interop with some popular apps that will open the table in a dedicated web view/modal (on top of the current one, with a close button) if the user double-clicks or double-taps one.
A few notes as to why this should be reasonable:

authors’ scripting is already taking this feature into account since you can find it in iBooks for instance;
for large tables, that would allow overflow-x (on the web view itself), something we can’t easily do in the paged and scrolled views (we would have to add an extra wrapper around the table since overflow won’t work for table itself…);
it is expected styling for the table (excepted inline style="") will not apply in the dedicated web view;
there will be huge tables to handle as people can become very creative when it comes to finding reasons why those tables must absolutely stay as they are in print (personally encountered the “moral rights of the author” justification a few years ago).

Please feel free to add your thoughts, obviously.

@danielweck does it concern you?

Let's close this. This is really a ReadiumCSS and navigator concern.

@JayPanoz has proposed elsewhere to have a discussion on EPUB & tables in the W3C EPUB Community Group and I agree this is where discussions about interop should first be held.
Then we'll open an issue in the Navigator repo (for each platform).

Just in case, I’ll reiterate there’s literally nothing ReadiumCSS can do well enough in and of itself here, not even overflow: scroll because I’d have to throw table semantics out with the bathwater by make table a block element. In other words, we would need an extra element (like div) wrapping table to even apply overflow – so altering the DOM, which brings other complications.
Actually, the more I’ve been thinking about this, the more I see JS as the only viable option – I know that if I had to re-tackle tables today in the authoring context, I’d probably go with a custom element because it would be easier to handle than an additional JS plugin modifying <table> on the fly. But boy that’s a lot of work – hence my 2 tweets btw.


Change the indent size in the tree explorer

New entry in the configuration
Added a new entry to handle the intent size of tree explorer
{
"explorer.indentSize": 4
}

Before this, the indent size was hardcoded (2) and the tree explorer could look too narrowed for some users.
The itemRenderer now receives the state so it can get the explorer indent size from the configuration and it simply use it instead of the old hardcoded value

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

Thanks for making the changes, @BenoitPingris !
I saw the CI was failing so I fixed a couple of issues:

Configuration defaults test case was failing when parsing (via esy '@test' run) - trailing comma was causing a problem
Format check was failing - ran esy format
Merged with master

Thanks for the updates & tweaking the configuration value name - looks great!


tests: drivers: counter: Enable test execution on nRF54L20pdk

Add overlay needed to execute counter tests on nRF54L20pdk.

Already done by @e-rk


improvement: extendable sourceMapName

The Source Map Revision 3 Proposal
tells that the naming convention is optional.
The gulp-sourcemaps library is deriving the name for the  .map that is generated. What about making a bit it more extensible to enable the "optional" part of the naming convention?
I mean, one of the benefits of gulp itself is the ability to code and enable coding over fixed configurations ("code over configuration"). This feature goes with that design principle.
(If help is needed, I could create a PR if you like).
It should not have any side-effect to existent functionality since the important is that the actual map file exists and is mapped.
An example of the code would then look like this:
gulp.task('javascript', function() {
gulp.src('src/*/.js')
.pipe(sourcemaps.init({sourceMapName: function(path) {
//whatever logic here. path includes the filename
return path + '.map';
}}))
.pipe(plugin1())
.pipe(plugin2())
.pipe(sourcemaps.write())
.pipe(gulp.dest('dist'));
});

The idea for the improvement came from: https://github.com/floridoo/gulp-sourcemaps/issues/91
It's in principle/ fact not related, as that issue is a bug from another library.

+1 this very useful feature
this my code
    .pipe(sourcemaps.write('../../build', {includeContent: false, sourceRoot: '/',
      sourceMappingURLPrefix: 'http://localhost:8000'}))
    .pipe(gulp.dest(paths.dist + '/js'));

this result
//# sourceMappingURL=http://localhost:8000/../build/app.min-42edcc2b10.js.map
but I want code is
    .pipe(sourcemaps.write('../../build', {includeContent: false, sourceRoot: '/',
      sourceMappingURL: function(mapFileName) {
        return 'http://localhost:8000/' + mapFileName
      }
    }))
    .pipe(gulp.dest(paths.dist + '/js'));

result is
//# sourceMappingURL=http://localhost:8000/app.min-42edcc2b10.js.map

This is now possible through the sourceMappingURL option

Thanks a lot!
+1


ListView with HEADER - ROW layout swipe issue

Hello sir, sorry to disturb you again.
I have a list view like this:
HEADER
ROW
ROW
ROW
HEADER
ROW
ROW
....
When I swipe on the ROW, everything works great. But when I swipe on the HEADER, the app crashes. Any ideas as on how this can be averted?? ROW is an instance of a class whereas HEADER is just plain text. I have different layouts for ROW and HEADER.
Is there any function which gets called when the user swipes?? Or do you have any other idea??
Please help. Many thanks.

Hello sir. Could you tell me if there is a way to disable left-to-right swipes or right-to-left swipes??
Many thanks.


Is the project abandoned?

Sorry to ask, mainly if already asked (I searched).
But the lastest release was two years ago. And there is many open issues and some PR.
I understand that this is open for contributions, but there is a leader?

I think I read somewhere that the author was working on a new version with alot more features, but I got the impression that it may be more than they bargained with and so rather than continuing to maintain the old version they are focusing purely on a new version. For what nitroshare is it just needs some tweaks and polish to its current system, and explorer integration which could be done fairly easily.
I am not a C++ developer but I might have a look at some of the issues and feature requests that I would like. Such as pausing transfers and having transfers run sequentially rather than simultaneously, but time is always the issue. Are there any devs on here that would be willing to contribute towards a new version?

I also with to contribute but I am not good at C++. How about I port it to Python?

There are a lot of new features in the master branch since the latest stable release. These include an expanded API, the ability to send links, the return of shareboxes, and support for native messaging.
Unfortunately, I have become very busy and have had very little free time to continue finishing up the next release and I was halfway through updating the build system when this happened. 😐
I'm hoping that changes in the new year.

Why not share your code as alpha and with hope some people can help you?
If already done, maybe a guideline for contribution. I can't help since I
work only with web technologies.
Em sáb, 14 de dez de 2019 20:13, Nathan Osman<EMAIL_ADDRESS>escreveu:

There are a lot of new features in the master branch since the latest
stable release. These include an expanded API, the ability to send links,
the return of shareboxes, and support for native messaging
https://developer.chrome.com/apps/nativeMessaging.
Unfortunately, I have become very busy and have had very little free time
to continue finishing up the next release and I was halfway through
updating the build system when this happened. 😐
I'm hoping that changes in the new year.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nitroshare/nitroshare-desktop/issues/234?email_source=notifications&email_token=AAIPXVPP2QD2G5DZSY2QD7TQYVSB7A5CNFSM4ISUMIC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG4NKUA#issuecomment-565761360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIPXVMYAPDXMOXJ6B34FVDQYVSB7ANCNFSM4ISUMICQ
.

Why not share your code as alpha and with hope some people can help you?

I have — it's in the master branch. The application will still build from source; it's the packaging scripts that are currently being worked on.

Hi Nathan, it's been over a year with no updates.
It there some way we can support you financially to continue development for those of us that either can't code or don't have the time or inclination to get involved in the development? Perhaps one of these or other options for you to get paid:
https://www.patreon.com/
https://flattr.com/
https://opencollective.com/
I use NitroShare to transfer files from my Android phone to my Mac. It's great as it is the best solution I've found and it's a bonus that it's also open source. However I have a (an already reported) problem when transferring the other way. It sounds like you are totally revamping the program for the next version. Perhaps consider continuing to fix bugs on the current stable version while you work on the this. I would even support a bug bounty to actually get the fix that I need.
Many of us definitely appreciate your efforts and some of us would like to see you rewarded in some way.

Sadly, it isn't so much an issue of cost as it is time. With COVID-19 hitting, I've been extremely busy with work and other things that have come up because of the pandemic.
I have made a number of bug fixes to the QHttpEngine and QMdnsEngine libraries which form an essential part of the application. However, the installers on the website don't include the newest versions of these libraries, so my first step would be to release new installers that include them. I can't make any promises, but if I have time over the holidays, I will look into doing that.
The new version in development (v0.4) is quite complex — the application has been split into several smaller components and there is a new plugin system. It offers a number of new features, such as sending links, shareboxes, a setup wizard, etc. Unfortunately, it is only partially complete. The core features are there but they still need a bit of polishing. I'm still hoping to get to this at some point in the future.


Manual geometry as option for monitor-agnostic bar placement

Is your feature request related to a problem? Please describe.
When using herbstluftwm I can set virtual monitors that are independent from the physical monitors used, so I can have two completely independent virtual monitors in a single ultrawide physical monitor, for example. So using MONITOR=$monitor polybar statusbar & does not solve the problem in this case. I need a way to set the bar position in a generic way that is independent from hardware, which should not be a concern to polybar.
Why does polybar need this feature?
Because this gives the users with big monitors more control of their systems without hacks, while maintaining the common MONITOR approach for basic needs.
Describe the solution you'd like
Something like:
polybar statusbar --width=1920 &
polybar statusbar --width=1920 --x=1920 &
polybar statusbar --width=1920 --y=1080 &
polybar statusbar --width=1920 --x=1920 --y=1080 &

This way I could set four bars in a 4k monitor with 4 virtual 1920x1080 monitors using the same bar configuration, provided that offset-x and offset-y are relative to the given bases.
Or passing the geometries in a more standard way:
polybar statusbar --geometry=1920x1080+0+0 &
polybar statusbar --geometry=1920x1080+1920+0 &
polybar statusbar --geometry=1920x1080+0+1080 &
polybar statusbar --geometry=1920x1080+1920+1080 &

Describe alternatives you've considered
It can be done by using offset-x and offset-y. The problem is that this solution requires having at least one different bar configuration for each monitor with different names, when the bars are essentially the same.
Additional context
Two virtual monitors inside a virtual machine, each with its own instance of a bar (dzen2):

This is achieved with dzen2 by using:
dzen2 -w $panel_width -x $x -y $y

This is a good idea. It's actually something that bothered me quite a bit in the past. On of the only reasons polybar needs a monitor name, is to figure out its position and even worse, if the bar goes outside of the monitor frame, polybar throws an error. When in reality, polybar could position itself anywhere, regardless of monitor.
I'm a bit wary of using commandline arguments for this, as everything in polybar is configured through the config file and I'd very much like to keep it that way.
I would suggest the following things:

A geometry setting that can be set using the standard WxH+X+Y format
Adding x and y fields to set the absolute position of the bar. (the width and height settings already exist)

Since polybar can read environment variables, these could also be set directly from the commandline when running polybar.
There are a few tricky things to consider though:

How do percentages in width. height, and offset-* behave when no monior is given.
How does y work for bottom bars?
Which settings take precendence over others.


Updating CFT spec with renaming and CFTokenTrust tx

First commit changes:

Change the CFTokenIssuanceID construct to be issuer and the transaction sequence. AssetCode is now an optional field.
Change freeze terminology to locking. And replaced lsfCannotFreezeBalances with lsfCanLock, which decides issuer's locking permission both individually and globally.
Removed Account field in CFTokenIssuanceSet, this field is default to be issuer's address. We need another field called CFTokenHolder to specify the optional holder's account.
Renamed Amount field to CFTokenAmount in the CFToken object, because Amount is reserved for STAmount type only
Added 1.7.0 section to describe how reserve requirements for CFTokenPages work.

Second commit changes:

Added CFTokenTrust transaction
New section 1.8.0 describing how allowlisting works for CFTs

Should the 1.1.2. Assumptions section be updated since we are now introducing essentially TrustSet transaction?


$q makes controller instance lose context of this (thisArg) object in its methods

I have just started writing controllers and directives in TypeScript, using controller methods that get compiled as controller function prototype methods.
When I use $q in a method and then rely on this to refer to the controller instance in other (chained) methods, I lose the context of the this object. A live demo of the issue is available in this plunker, but the main issue can be easily described here:
MyDirectiveController.prototype.doSomething = function() {
  // makeSomePromise returns a promise
  // (and is able to access to controller properties via this.someVariable)
  return this.makeSomePromise(1)
    // makeSomeCalculation takes a value and tries to set a property
    // on the controller instance (e.g. `this.anotherVariable`): THIS ERRORS
    .then(this.makeSomeCalculation);
};

The only obvious solution I found (which I don't like a lot) is to use .then(this.makeSomeCalculation.bind(this)) to forcefully bind the thisArg object of the function to the controller instance.
Is this a bug or is there any other way to achieve the same result in a "cleaner" way?
The live demo is running AngularJS 1.4.8 and the behaviour was tested on Chrome.
Related topics:

http://stackoverflow.com/questions/28638600/angularjs-http-interceptor-class-es6-loses-binding-to-this (I am getting window as well, instead of the constructor, when referencing this)
http://stackoverflow.com/questions/30185369/angularjs-promise-this-references-window

This is expected behavior as defined in the Promises A+ spec: https://promisesaplus.com/#point-35

TIL, thanks for the accurate reference @pkozlowski-opensource! :sparkles:

Using the typescript arrow function will result in
this.makeSomePromise(1)
  .then((response) => this.makeSomeCalculation(response));

compiling to
var _this = this;
this.makeSomePromise(1)
  .then(function (response) { return _this.makeSomeCalculation(response); });

this in the makeSomeCalculation function will then reference _this, which I believe is what you want

Arrow fns really eliminate a lot of boilerplate (repeating the parameter would still be unavoidable), I may end up using this form, thanks!


lock auth properly

Required for all non-trivial PRs

[x] Rebased/mergable
[ ] Tests pass
[x] CHANGELOG.md updated
[x] Sign CLA (if not already signed)

Fixes #6760

The c.cacheData.Clone() calls is unnecessary and can be very expensive to run on every request.  That should be removed.
I think you can simplify the locking with the following.
diff --git a/services/meta/client.go b/services/meta/client.go
index 8dad1d6..4f79ef9 100644
--- a/services/meta/client.go
+++ b/services/meta/client.go
@@ -569,19 +569,20 @@ func (c *Client) AdminUserExists() bool {
 }

 func (c *Client) Authenticate(username, password string) (*UserInfo, error) {
-       c.mu.RLock()
-       defer c.mu.RUnlock()
-
-       data := c.cacheData.Clone()
-
        // Find user.
-       userInfo := data.User(username)
+       c.mu.RLock()
+       userInfo := c.cacheData.User(username)
+       c.mu.RUnlock()
        if userInfo == nil {
                return nil, ErrUserNotFound
        }

        // Check the local auth cache first.
-       if au, ok := c.authCache[username]; ok {
+       c.mu.RLock()
+       au, ok := c.authCache[username]
+       c.mu.RUnlock()
+
+       if ok {
                // verify the password using the cached salt and hash
                if bytes.Equal(c.hashWithSalt(au.salt, password), au.hash) {
                        return userInfo, nil
@@ -600,7 +601,9 @@ func (c *Client) Authenticate(username, password string) (*UserInfo, error) {
        if err != nil {
                return nil, err
        }
+       c.mu.Lock()
        c.authCache[username] = authUser{salt: salt, hash: hashed, bhash: userInfo.Hash}
+       c.mu.Unlock()

        return userInfo, nil
 }

@jwilder , this actually worked, with this patch, database was running for 7 hours and then crashed, see #6782 but I don't think it's related to this issue.

👍 Thanks @rubycut!


WebExtensions triage query is wrong

The auto rotation for triage got me scheduled to be triage owner for WebExtensions components this week.  I just got an email with 3 different bugs, two were from our components, but the third one Bug 1749345 clearly wasn't, nor was it moved into/out of one of our components.
It looks like our bugzilla query is wrong, including product: Toolkit and component: General, which it shouldn't.
https://mzl.la/3pDk1Pk

Thank you @zombie for reporting this.
This behaviour is not related to the triage auto rotation. Autonag will send emails about bugs without a severity on Monday and Thuresday. These emails will be sent to the triage owners and their managers. Instead of sending a separate email for each triage owner and CCing the manager, autoang will group bugs based on the common direct manager.
The email you mentioned was sent to you and Dave Townsend since you both share the same manager (Vicky Chin). The email lists bugs from your components and Dave's components as well.
I think adding the name of the triage owner for each bug could remove such confusion.


Reason: Image not found

My Log, run with IP6PLUS IOS 9.3 and Kingfisher at "2.4.2"
dyld: Library not loaded: @rpath/Kingfisher.framework/Kingfisher
Referenced from: /Users/thanhngoxuan/Library/Developer/CoreSimulator/Devices/9FA2F252-0249-4637-BFA6-BE3181C88476/data/Containers/Bundle/Application/DF99B81D-1BA0-4C2D-8C62-C360260B4D92/Alumni.app/Alumni
Reason: image not found
(lldb)
 button.kf_setImageWithURL(url, forState:.Normal)

Please ensure you have embedded Kingfisher.framework in your Embed Framework build phase.

fixed, close issue, thanks


fix(controller): Carry-over labels for re-submitted workflows. Fixes #3622

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[x] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.
[x] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[x] My builds are green. Try syncing with master if they are not.
[x] My organization is added to USERS.md.

Fixes #3622

Available in v2.10.0-rc7.


Remove 'tech preview' tag from RKE2/K3s cluster provisioning buttons

This PR addresses https://github.com/rancher/dashboard/issues/4852 by removing "Tech Preview" from the cluster provisioning page when RKE2/K3s is selected.
It doesn't remove the tag variable completely because Harvester still needs an experimental tag.
This PR is ready for review, but it should not be merged until the cluster provisioning work for v2.6.4 is complete.
Before:

After:

Since RKE2 will become GA in Rancher v2.6.4 so we can remove the experimental label, but for External Harvester support it will still be marked as Experimental, thanks.

@catherineluse Can this be changed from a draft PR now - there is one file conflict - would be good to get this in as its needed for 2.6.4


Question: 1..N relation on EF 2.1: "reached the recursion limit."

I trying to initialize the first migration with dotnet ef migrations add EmailMigration but I have the following error:
The convention invocations have reached the recursion limit. This is likely an issue in EF Core, please report it

Stack trace:
System.InvalidOperationException: The convention invocations have reached the recursion limit. This is likely an issue in EF Core, please report it.
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.ConventionDispatcher.ConventionBatch.Run()
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.ConventionDispatcher.ConventionBatch.Dispose()
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.RelationshipDiscoveryConvention.DiscoverRelationships(InternalEntityTypeBuilder entityTypeBuilder)
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.RelationshipDiscoveryConvention.Apply(InternalEntityTypeBuilder entityTypeBuilder)
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.ConventionDispatcher.ImmediateConventionScope.OnEntityTypeAdded(InternalEntityTypeBuilder entityTypeBuilder)
   at Microsoft.EntityFrameworkCore.Metadata.Conventions.Internal.ConventionDispatcher.OnEntityTypeAdded(InternalEntityTypeBuilder entityTypeBuilder)
   at Microsoft.EntityFrameworkCore.Metadata.Internal.Model.AddEntityType(EntityType entityType)
   at Microsoft.EntityFrameworkCore.Metadata.Internal.Model.AddEntityType(Type type, ConfigurationSource configurationSource)
   at Microsoft.EntityFrameworkCore.Metadata.Internal.InternalModelBuilder.Entity(TypeIdentity& type, ConfigurationSource configurationSource, Boolean throwOnQuery)
   at Microsoft.EntityFrameworkCore.Metadata.Internal.InternalModelBuilder.Entity(Type type, ConfigurationSource configurationSource, Boolean throwOnQuery)
   at Microsoft.EntityFrameworkCore.ModelBuilder.Entity(Type type)
   at Microsoft.EntityFrameworkCore.Infrastructure.ModelCustomizer.FindSets(ModelBuilder modelBuilder, DbContext context)
   at Microsoft.EntityFrameworkCore.Infrastructure.RelationalModelCustomizer.FindSets(ModelBuilder modelBuilder, DbContext context)
   at Microsoft.EntityFrameworkCore.Infrastructure.ModelCustomizer.Customize(ModelBuilder modelBuilder, DbContext context)
   at Microsoft.EntityFrameworkCore.Infrastructure.RelationalModelCustomizer.Customize(ModelBuilder modelBuilder, DbContext context)
   at Microsoft.EntityFrameworkCore.Infrastructure.ModelSource.CreateModel(DbContext context, IConventionSetBuilder conventionSetBuilder, IModelValidator validator)
   at Microsoft.EntityFrameworkCore.Infrastructure.ModelSource.<>c__DisplayClass5_0.<GetModel>b__1()
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
   at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
   at System.Lazy`1.CreateValue()
   at Microsoft.EntityFrameworkCore.Infrastructure.ModelSource.GetModel(DbContext context, IConventionSetBuilder conventionSetBuilder, IModelValidator validator)
   at Microsoft.EntityFrameworkCore.Internal.DbContextServices.CreateModel()
   at Microsoft.EntityFrameworkCore.Internal.DbContextServices.get_Model()
   at Microsoft.EntityFrameworkCore.Infrastructure.EntityFrameworkServicesBuilder.<>c.<TryAddCoreServices>b__7_1(IServiceProvider p)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitFactory(FactoryCallSite factoryCallSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(IServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitScoped(ScopedCallSite scopedCallSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(IServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(ConstructorCallSite constructorCallSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(IServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitScoped(ScopedCallSite scopedCallSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(IServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.DynamicServiceProviderEngine.<>c__DisplayClass1_0.<RealizeService>b__0(ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngine.GetService(Type serviceType, ServiceProviderEngineScope serviceProviderEngineScope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)
   at Microsoft.EntityFrameworkCore.DbContext.get_DbContextDependencies()
   at Microsoft.EntityFrameworkCore.DbContext.get_InternalServiceProvider()
   at Microsoft.EntityFrameworkCore.DbContext.Microsoft.EntityFrameworkCore.Infrastructure.IInfrastructure<System.IServiceProvider>.get_Instance()
   at Microsoft.EntityFrameworkCore.Internal.InternalAccessorExtensions.GetService[TService](IInfrastructure`1 accessor)
   at Microsoft.EntityFrameworkCore.Infrastructure.AccessorExtensions.GetService[TService](IInfrastructure`1 accessor)
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.CreateContext(Func`1 factory)
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.CreateContext(String contextType)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.AddMigration(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigrationImpl(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigration.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)

Steps to reproduce
I create a new template for webapi with dotnet new webapi, and added my models and the bugged context:
public class UserModel
{
    public long Id { get; set; }

    public string Password { get; set; }

    public bool FirstLogin { get; set; }

    public DateTime LastLoginAt { get; set; }

    public DateTime CreatedAt { get; set; }

    public string Roles { get; set; }

    [ForeignKey("Emails")]
    public long EmailModelId { get; set; }
    public ICollection<EmailModel> Emails{ get; set; }
}

public class EmailModel
{
    public long Id { get; set; }

    public string MailAddress { get; set; }

    public bool Main { get; set; }

    public bool ReceiveNews { get; set; } = true;

    public bool Confirmed { get; set; } = false;

    [ForeignKey("Owner")]
    public long UserModelId { get; set; }
    public UserModel Owner { get; set; }
}

public class UserContext : DbContext, IDesignTimeDbContextFactory<UserContext>
{
    public DbSet<EmailModel> Emails { get; set; }
    public DbSet<UserContext> Users { get; set; }

    public UserContext(DbContextOptions options) : base(options)
    {

    }

    public UserContext()
    {

    }

    public UserContext CreateDbContext(string[] args)
    {
        return new UserContext(
            new DbContextOptionsBuilder<UserContext>().UseSqlite(Startup.Configuration["ConnectionString"])
                .Options);
    }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<UserModel>()
            .HasMany(e => e.Mails)
            .WithOne(u => u.Owner)
            .HasForeignKey(e => e.Id);

        modelBuilder.Entity<EmailModel>()
            .HasOne(u => u.Owner);
    }
}

Further technical details
EF Core version: 2.1.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite
Operating system: Windows 10
IDE: JetBrains Rider 2018.2

@JonasUliana It looks like there are multiple issues with the code. For example::

The same navigation properties are used to define multiple different relationships
The FK is mapped to the same property as the PK for a one-to-many relationship.
The ForeignKey attribute suggests that EmailModelId should be used as an FK, but no property of that name exists on the dependent end.

Rather than trying to figure out the intent here, can you describe hat tables and foreign keys you expect from the mapping of these types?

@ajcvickers, I trying to create one to many relationship in User model to Email model with the following logic:

A Users can have many Emails;
One Email can have only one User;

I am trying to do that with annotation and/or fluent API, but no success, the issue is my implementation right?

@JonasUliana Are you intending to use a specific property/column for the foreign key? Or do you not care as long as it works?

@ajcvickers I prefer the best implementation, but with a working behaviour is enough for studding proposes.

It sounds like you are trying to do a simple a user has multiple emails 1- many relationship and the emails must be unique in the system
context:

public class UserModel
{
//….
    [ForeignKey("Emails")]
    public long EmailModelId { get; set; } //I think all you need to do is remove this and the relationship will be found by convention
    public ICollection<EmailModel> Emails{ get; set; }
}

@JonasUliana Thought I posted this yesterday, but I must have not pressed the button or something. Assuming that UserModelId should be the FK and EmailModelId is erroneous, then using entity types like so:
public class UserModel
{
    public long Id { get; set; }
    public string Password { get; set; }
    public bool FirstLogin { get; set; }
    public DateTime LastLoginAt { get; set; }
    public DateTime CreatedAt { get; set; }
    public string Roles { get; set; }
    public ICollection<EmailModel> Emails { get; set; }
}

public class EmailModel
{
    public long Id { get; set; }
    public string MailAddress { get; set; }
    public bool Main { get; set; }
    public bool ReceiveNews { get; set; } = true;
    public bool Confirmed { get; set; } = false;
    public long UserModelId { get; set; }
    public UserModel Owner { get; set; }
}

will result in the relationship you described being created by convention. This means that no additional configuration is needed. However, if you would like to explicitly specify the relationship, then the following code will do that:
modelBuilder
    .Entity<UserModel>()
    .HasMany(e => e.Emails)
    .WithOne(e => e.Owner)
    .HasForeignKey(e => e.UserModelId);


[BBU] Contact content needs to be refreshed

Problem/Concern

link to problem content

Proposed Solution
Setup mail for new consortium using new hostname.

This page should also contain info on how to join the BBU, Technical  Project etc.
If we decide to have a separate Join page that will work well too but we need to update then the link in the MasterDoc Preface section.

Update the fee_schedule.md for link to Join process.


Replace abs() as unsigned with dedicated alternative.

This is an upcoming clippy lint. unsigned_abs() was added in Rust 1.51.0, meeting the current version requirement. This avoids a potential panic in debug mode. cargo test passed and sample bitstreams are identical.

Coverage decreased (-0.07%) to 85.434% when pulling fa4494cd587a92ed89ae6e0a4acbef1f5c5c789a on doctortheemh:master into d8ebb60bea2170feb00fcb1385280c56f583fab7 on xiph:master.


Failed to compile

Hello! I have this error when I try to use ArrowKeyNavigation in my project:

My React Class:
`
import React, { Component } from 'react';
import { ArrowKeyNavigation } from 'boundless-arrow-key-navigation';
class TestPage extends Component {
state = {
items: [ 'lorem', 'ipsum', 'dolor' ],
}
render() {
    return (

Horizontal-only

                <ArrowKeyNavigation className='demo-loose-list' mode={ArrowKeyNavigation.mode.HORIZONTAL}>
                    {this.state.items.map((item) => {item})}
                </ArrowKeyNavigation>

    );
}

}
export default TestPage;
`
Can you help me?
Thank you.

My package.json
{   "name": "react-media-app-vod-accedo",   "version": "0.1.0",   "private": true,   "dependencies": {     "prop-types": "^15.5.10",     "react": "^15.6.1",     "react-dom": "^15.6.1",     "react-redux": "^5.0.6",     "react-router": "^4.2.0",     "react-router-dom": "^4.2.2",     "react-scripts": "1.0.12",     "redux": "^3.7.2",     "redux-saga": "^0.15.6"   },   "scripts": {     "start": "react-scripts start",     "build": "react-scripts build",     "test": "react-scripts test --env=jsdom",     "eject": "react-scripts eject"   } }

@sighrobot I create new test react app.
Steps:

create-react-app react-test-boundless.
Install boundless/ArrowKeyNavigation: npm i --save boundless-arrow-key-navigation
In App.js import ArrowKeyNavigation from 'boundless-arrow-key-navigation'; and add

Horizontal-only
                  <ArrowKeyNavigation className='demo-loose-list' mode={ArrowKeyNavigation.mode.HORIZONTAL}>                      {this.state.items.map((item) => {item})}                  </ArrowKeyNavigation>

But i get the same error.
I uploaded the project to the following repository: https://github.com/abritopach/react-test-boundless
Can you take a further look?
Thank you very much.

@abritopach in the meantime, you can opt out of using the bleeding edge code with this webpack config in your project:
{
    resolve: {
        mainFields: ["browser",  "main"]
    }
}

Oh, it's working :)
I have executed eject in the project: yarn eject
With this command all the build dependencies, configs, and scripts are moved right into project. At this point I can customize webpack.config.dev.js and add the indicated configuration.
Thank you very much @probablyup


Local view using the container doesn't work on Windows

When running on Windows and following the docs to preview content locally, the command (executed in Git Bash) fails:
$ docker run -it -p 3000:3000 -v $(pwd):/github/workspace ghcr.io/bcgov/devhub-techdocs-publish preview
Input parameters: preview
Previewer will be started for content after it is generated.
/entrypoint.sh: line 18: cd: /github/workspace: No such file or directory
cp: target '/github/workspace': No such file or directory
No catalog-info file found in repo. Cannot continue.

Note: the command was run using the following repo, updated with the help of the skeleton builder - https://github.com/bcgov/DITP
The command fails also when running in WSL with the source code checked-out on the Windows filesystem.
Checking-out the code inside the WSL filesystem and running the command yields a success.

We were able to figure out a resolution to this, as follows.
The instructions provided in the How to use the Docker image to preview content locally documentation, were actually written for MAC users (not Windows).
When running the docker run -it -p 3000:3000 -v $(pwd):/github/workspace ghcr.io/bcgov/devhub-techdocs-publish preview command on a Windows system (ie. from a Git Bash command prompt), you experience the following error:
$ docker run -it -p 3000:3000 -v $(pwd):/github/workspace ghcr.io/bcgov/devhub-techdocs-publish preview
Input parameters: preview
Previewer will be started for content after it is generated.
/entrypoint.sh: line 18: cd: /github/workspace: No such file or directory
cp: target '/github/workspace': No such file or directory
No catalog-info file found in repo. Cannot continue.

When you run docker images you will see 2 different images:

spotify/techdocs                        v1.2.4    bf9ca0f99d4e   5 months ago   1.05GB
ghcr.io/bcgov/devhub-techdocs-publish   latest    4742cfc58615   7 months ago   2.9GB

The npx @techdocs/cli serve command seems to pull the spotify image, whereas the other instructions provided say to use the ghcr.io/bcgov/devhub-techdocs-publish image.
Inside the Container
From inside the devhub-techdocs-publish image, running the entrypoint.sh command directly identified that the mounted directory (ie. $(pwd):/github/workspace) had a permissions error.
Our assumption is that this is due to the external directory being from a Windows system, being mounted into the Linux system.

NOTE: From additional testing, it seems like it expects the path to be all lowercase, so if your Windows path has any uppercase, this will also cause errors.

On a Windows System
From a Windows system, the following execution works:

From a PowerShell command prompt, run docker run -it -p 3000:3000 ${PWD}:/github/workspace ghcr.io/bcgov/devhub-techdocs-publish preview
From a WSL command prompt, run docker run -it -p 3000:3000 $(pwd):/github/workspace ghcr.io/bcgov/devhub-techdocs-publish preview

[!NOTE]
This will require the repository to have been cloned within the WSL environment, and not externally in Windows

Summary
From a Windows system (if the repo is not cloned into WSL), use the PowerShell command prompt (which will also use the latest image specified).

@sheaphillips, the index.md should be updated to include instructions for Windows users.
As an example, the README file in the public-cloud-techdocs repo has been updated accordingly.

hi @AErmie . Appreciate your efforts to troubleshoot this for your environment. I can confirm that we had not tested this ourselves in a Windows PowerShell environment. I believe the existing commands will work for Mac, Linux, and WSL as-is. Would you be willing to make a pull request to our README file that includes the modified command for Windows PowerShell?

hi @AErmie . Appreciate your efforts to troubleshoot this for your environment. I can confirm that we had not tested this ourselves in a Windows PowerShell environment. I believe the existing commands will work for Mac, Linux, and WSL as-is. Would you be willing to make a pull request to our README file that includes the modified command for Windows PowerShell?

@sheaphillips , I tried making a pull request, but I do not have the permissions to push to this repo.

@AErmie that's correct - you wouldn't have the ability to push directly to this repo. You'd create a fork of the repo first, and then initiate pull request from the fork to this repo.

@sheaphillips, PR created: https://github.com/bcgov/devhub-techdocs-publish/pull/13


Bug do not publish rf when complete_build_stage is false

What problem does this PR solve?
should_build_hash_table's instance close early and signal this instance
but function make_all_runnable still running and not set this instance's wake_up_by_downstream to true
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Add Typescript typings

Thanks for this handy package. I figured I would share the typings I made in the hopes of saving some other Typescript user a few minutes.

Good call, that was sloppy of me. Thanks again!


Update jnigen link in README.md to actual

Github says that I cannot merge pull request, someone with write access should do this.


setup_functions.sh for Windows OS

Is it possible to create setup_functions.sh for Windows OS?
setup_functions.sh can not run with Windows OS.
It is very helpful for developers if there is setup_functions.bat for Windows OS.
Thank you.

@kyokonishito Thanks for the suggestion
Currently, there are alternatives for running shell scripts on windows through different tools such as: gitbash, Cygwin, win-bash


Problem with computation of government expenditure as a share of national income

There is an error in the computation of government expenditure as a share of national income using the IMF data from Mauro (2015). The original data reports primary government expenditure, interest payment expenditure, and total government expenditure. The latter being the sum of the first two.
In compiling the data, OWID mistakingly reports government expenditure as the sum of total government expenditure and interest payment expenditure. This effectively inflates the reported government expenditure since interest payments are added twice.
As an example,  take Denmark in 1993. The original IMF data, reports 53.55 primary expenditure, 6.66 interest expenditure, and 60.21 total expenditure (the sum of the first two). In contrast, OWID reports total expenditure at 66.88=60.21+6.66 (rounded).

Thank you for reporting, @pedro-s-amaral — we'll look into this (and https://github.com/owid/etl/issues/2756) and report back.

Thanks for this message, @pedro-s-amaral. This was indeed an error in our calculation, so we decided to update to the latest IMF data (up to 2022). You can see the new data in this chart:
https://ourworldindata.org/grapher/historical-gov-spending-gdp


Add dependency for bcrypt

Merging since we broke it during the removal in ManageIQ/manageiq-gems-pending#292


Prometheus client memory usage

Prometheus client is keeping metrics emitted indefinitely in memory.
Over the time, tasklist metrics (but not only), sticky tasklist with uuid name tag will take pretty big amount of RAM, and calling '/metrics' to collect them takes more than 20s. The only way to get rid of those metrics is to restart Cadence instance.
I don't see any way how to control TTL for metrics or disabling part of metrics collection. Or, for example detecting uuid and converting them to a common uuid tag.
Looking on opinions and ideas on how to overcome this situation

Thanks for the feedback Mantas! I will discuss this with the team, have some folks share here too.

Is the desire to have a no-op metrics client so this can just be dropped (hopefully we have a way to set that up easily), or is it more of a "we use Prometheus but it has problems" thing?
Assuming you want to use Prometheus but it's being too costly... any chance a library upgrade would solve this?  It sounds kinda like a thing Prometheus's client should have a way to deal with tbh, though I think Tally has similar memory-bloat issues.  But it may also be a sign we're using it wrong / should change something.

Is that by design or Prometeus is not gathering those metrics and that's why they reside in memory?

@Groxx the idea here is to reduce cardinality, basically. per tasklist metrics are pretty costly, especially when name label is a uuid.  Similar issue filled for prometheus go client: https://github.com/prometheus/client_golang/issues/791
@dkrotx once emitted, metric will be kept forever regardless if you are scraping them or not.

@mantas-sidlauskas got it! That's basically the same for Tally. I would recommend looking towards not having uuid-like values. Otherwise you gonna have this problem with storage + some requests will struggle at runtime.

Just to close the loop. There was a bug in Cadence Java Client which is fixed now: https://github.com/uber/cadence-java-client/issues/856


Return user owned tags by context

Imagine a user has many owned_tags in several contexts. Is it possible to return a users owned tags for only one context?
Here is my current work around:
user.owned_taggings.where(context: 'locations').map {|tagging| tagging.tag.name}

I also created this monkey-patch which can be placed in the acts_as_tagger model (ie the User model):
def owned_tags(context)
    if context.present?
      self.owned_taggings.where(context: context).map {|tagging| tagging.tag.name}
    else
      super
    end
end

Then you can use:
user.owned_tags('context')

Thanks for this example @mrjlynch. The only problem I had was that it was skipping the distinct which is on the owned_tags relation.
So in my code I changed your example to:
def owned_tags_on(context = nil)
  self.owned_tags.includes(:taggings).where(taggings: { context: context })
end

@sandergarretsen, your method returns a relation, mine returns a string (list), and they are named accordingly. It would be smart to create a PR and merge both these methods

I tried @sandergarretsen's method, but for me resulted in an inefficient query that was pretty slow once there were 1M + records in taggings.
Since I also wanted the distinct behaviour, and for it to return a relation, this worked better for me:
  def owned_tags_list(context = nil)
    tags = ActsAsTaggableOn::Tagging
    .distinct
    .where({
      tagger_type: self.class.to_s,
      tagger_id: id,
      context: context,
    })
    .pluck(:tag_id)

    ActsAsTaggableOn::Tag
      .where(id: tags)
  end

Just putting it out there in case it helps anyone else.


Yearn Shares Zapper

The shares zapper converts yearn shares into Principal and Yield tokens.

Coverage decreased (-0.3%) to 91.609% when pulling e16b8d901a4695681c46275a8c63e2b7a2431c5c on zap-lp into 5517dbe0982b85bc7a7207b5119d9a728bf1f830 on main.


Update chroot-environment-linux.md

Learn Build status updates of commit d452015:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/775/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Updates for the Nitrogen Scripts

Adds code that retrieves and prepares the data required by the nitrogen scripts in the LOONE model

LGTM


(Cont #301) Allow edited messages and interactions to be sent without removing existing embeds

Continuing from #301

CC: @jamesbt365


Only works for single tab

I often work with multiple tabs open, and it looks like the filter bar is only present for a single tab. Can we enable this to work for any open tab? Currently using with Xcode 5.1.1. Thanks!

It should be reopened. Steps:
1)run app and type something to filter the console (as usual) but don't fully empty the console
2)open another tab (I do it via "cmd+t") and look at console - the field with regex is empty and the console output appears as without filtering
3)copy the regex from the first tab to a new tab and apply it
Expected result - the same console log for 2 tabs.
Actual result the console log on the second tab is totally empty.
To fix the described problem I need to choose "All output" from the dropdown list to the left of the regex textfiled but it is temp solution only.
Additionally it could be better to save some last regexes because an user must input them each time manually.


how to get raw-body

You might want to look into using https://github.com/stream-utils/raw-body which this library so happens to depend on via co-body.

I dont think so.  I want use this package. this package parse raw-body , and also can add raw-body to ctx