GitHub tests workflow fails for macos-11 and macos-12 images

The tests GitHub workflow fails when using the macos-11 or macos-12 runner images. When executing the brew test-bot --only-setup step, the brew doctor command fails with the following error output:
  Please note that these warnings are just used to help the Homebrew maintainers
  with debugging if you file an issue. If everything you use Homebrew for is
  working fine: please don't worry or file an issue; just ignore this. Thanks!

  Warning: "config" scripts exist outside your system or Homebrew directories.
  `./configure` scripts often look for *-config scripts to determine if
  software packages are installed, and which additional flags to use when
  compiling and linking.

  Having additional scripts in your path can confuse software installed via
  Homebrew if the config script overrides a system or Homebrew-provided
  script of the same name. We found the following "config" scripts:
    /Library/Frameworks/Python.framework/Versions/Current/bin/python3-config
    /Library/Frameworks/Python.framework/Versions/Current/bin/python3.11-config

This is demonstrated in the following branches:

task/use-macos-12-in-tests-workflow
task/use-macos-11-in-tests-workflow

OpenSearch 1.3.9 adds support for 64-bit architectures for MacOS. This resolves the errors for the macos-11 and macos-12 runners.
The main branch now uses the macos-12 runner in the {{tests}} GitHub workflow instead of macos-10.15.


Using streaming conformer as transducer encoder

I slightly refactored https://github.com/k2-fsa/icefall/pull/242, and create this new pull request.  I think it is no need to duplicate a new folder, as making conformer to streaming just needs very minor fixes.
I also add the symbol delay penalty to this PR, see https://github.com/k2-fsa/k2/issues/955 for how we penalize and measure symbol delay.
Will post the experiment results here soon.

Cool!
One thing we should perhaps discuss is whether this belongs as a change to pruned_transducer_stateless2, vs.  pruned_transducer_stateless4 which I believe is the latest recipe?  @csukuangfj what are the changes from 2 to 4?

Cool! One thing we should perhaps discuss is whether this belongs as a change to pruned_transducer_stateless2, vs. pruned_transducer_stateless4 which I believe is the latest recipe? @csukuangfj what are the changes from 2 to 4?

pruned_transducer_stateless4 (see https://github.com/k2-fsa/icefall/pull/344) supports saving averaged model and the epoch number counts from 1.

@csukuangfj what are the changes from 2 to 4?

"4" has the changes from zengwei that supports model averaging that you proposed in https://github.com/k2-fsa/icefall/issues/337
Otherwise, 4 is identical to 2. I would suggest to always use 4 as zengwei has showed it is helpful.
From my results of deeper and narrower conformer, it is also helpful.

@csukuangfj BTW, are we recommending the deeper and narrower conformer right now?  How much better was it, again?

@pkufool it may be fairly easy to use git diff to move your changes from 2 to 4.  E.g.
cd pruned_transducer_stateless2
git diff master..HEAD . > ~/diff
cd ../pruned_transducer_stateless4
patch -p4 < ~/diff
(the 4 may need to be changed, e.g. to 3 or 5, if this fails).

@csukuangfj BTW, are we recommending the deeper and narrower conformer right now? How much better was it, again?

I am updating the results and uploading the pre-trained models to huggingface. Please wait for a moment.

Cool! One thing we should perhaps discuss is whether this belongs as a change to pruned_transducer_stateless2, vs. pruned_transducer_stateless4 which I believe is the latest recipe? @csukuangfj what are the changes from 2 to 4?

I think they share the same conformer.py, only a little changes on train.py & decode.py.

@pkufool it may be fairly easy to use git diff to move your changes from 2 to 4. E.g.
cd pruned_transducer_stateless2
git diff master..HEAD . > ~/diff
cd ../pruned_transducer_stateless4
patch -p4 < ~/diff
(the 4 may need to be changed, e.g. to 3 or 5, if this fails).

Thanks, I will try it.

Summarize my experiment results for streaming conformer below:
About the dynamic chunk training
I tried several settings(Note: The chunk sizes and contexts are all in frames after subsampling):

The first row is the results reproduced with https://github.com/k2-fsa/icefall/pull/242 .
The second row  was trained with limited left context, because we can only see limited left context for parallel streaming decoding (Note: the left context is in chunks, and the chunk size is dynamicly sampled from [1, 25]).
The third and forth rows were trained with fixed chunk size, I tried 16 and 8.
The fifth row was trained with fixed left context, I only tried left context = 32.

Comparing with the first and second row,  we can see if we need to decode with limited left context, we have to trained with limited left context. Comparing with the second row and last three rows, we can see the models trained with fixed chunk size and fixed left context do not outperform the model trained with dynamic chunk size.

About using different left-context for different layers
I also tried train the conformer encoder using different left context for different layers. (The models are all 12 layers conformer).

The baseline system was trained with num-left-chunks=4 for all layers and decoded with chunk size =16, left-context=64.
For the second row, the lower 8 layers used num-left-chunks=2, the upper 4 layers used num-left-chunks=4. In decoding time, they also used different left context(32 for lower 8 layers, 64 for upper 4 layers).
The third row is the same as second row, the only differnece is I splited the layers into 6 + 6.

We can see that the WERs are close to each other for all three settings, smaller left context can save more memories when decoding, though. I did not do much experiments on this idea(i.e. how to split layers, how many left context needed by different layers), so the conclusion might not soild.

About streaming decoding and right-context
I also implemented the real paralell streaming decoding for the conformer model, you can see the code in streaming_decode.py in each directory. The table below shows the decoding results of several models using different chunk sizes and contexts.  You can see in the table, adding right context usually helps to decrease the WERs, but for some settings, like decoding with left-context=32 and chunk-size=8 in pruned_transducer_stateless2, pruned_transducer_stateless3, adding right context makes it worse. I still have not figured out the reasons.

Note: You can also try these streaming conformer models in our server framework Sherpa (https://github.com/k2-fsa/sherpa/tree/master/sherpa/bin/streaming_conformer_rnnt).

Merging now. Most of the code added is to support streaming decoding, it won't break any previous recipes. Will fix the issues later in other PRs if there is any.


simavr: 1.5 -> 1.7

Build works fine on x86_64-linux even without "-Wno-error=stringop-truncation". Not sure if it is still needed for other targets.

Motivation for this change
#138460
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg eval


one class  has only one objects??why??

for (int class_idx = 0; class_idx < num_class; class_idx++)
{
float box_cls_score = feat_ptr[5 + class_idx];
float box_prob = box_objectness * box_cls_score;
if (box_prob > prob_threshold)

one class has only one object at most??  why?

It's class loop, but does not mean one class only has one object.

loop?  means?

if i have only one class to detect,then just set category_id = 0, is it ture?


Looking for new owner(s)

Hello all,
It's been over 7 years since I started working on clean-css and it has become one of the most popular tools on npm.
Please come forward if you'd like to actively support clean-css, either by writing new features, fixing bugs, or helping others to jump on board.
Speaking on my involvement I'd l like to stay with you until clean-css 5 is released and in the meanwhile gracefully pass the torch to whoever comes next.
So who's interested?
Jakub

@jakubpawlowicz Maybe we can unite? I'm an active developer https://github.com/cssnano/cssnano.
In history, there are many examples of union libraries/tools/frameworks. I know it is a lot of work, but this would allow us to focus on one instrument and make it the best. We can create an approximate roadmap.

FWIW, when using the OptimizeCssAssetsPlugin with webpack, with my medium-sized project, Clean CSS is much faster than CSS Nano.
Cold builds:
Clean CSS
Time: 99380ms
Time: 97079ms
CSS Nano
Time: 208936ms
Time: 212521ms
More than double (and these builds include Sass, TypeScript, image optimization, etc).
Unfortunately the Clean CSS integration in OptimizeCssAssetsPlugin is incomplete (see https://github.com/NMFR/optimize-css-assets-webpack-plugin/issues/87), so I'm forced to use code like this for now.

@glen-84 i am glad to speedup cssnano :+1: Feel free to investigate, also cssnano should compress better more things

It would be awesome if you guys could unite to make a real contender to be the best out there. In my team we were using CleanCSS for years (with gulp) before moving to webpack and even though css nano is great, it isnt that fast & the compression is not better so it felt like a downgrade.

@heyflo

compression is not better
better :smile:

isnt that fast
yep we are working on this, last version increase speed around 2x-3x

Anyway i am glad to union, but @jakubpawlowicz don't give me answer and don't connect with me

Thanks for the reply !
Sad he's not helping much but it's good that you guys realized some of the cssnano flows and are looking to improve it. Looking forward to it 👍

@jakubpawlowicz take a look at https://www.codeshelter.co/ They are a group of volunteers who offer a helping hand for open source projects. Log in with your Github account and add your project.
Saw them the other day on Hacker News: https://news.ycombinator.com/item?id=19199647

After long deliberation I'll keep maintaining it for foreseeable future. Wish me luck!

After long deliberation I'll keep maintaining it for foreseeable future. Wish me luck!


How to receive one variable from the url, and the rest from the json?

I have a put function using flask-restful, with 'with_args' decorator,
this function receive the id in the url, and the person object via json.
This is the routing:
api.add_resource(PersonResource, '/persons/<person_id>')

This is how I wanted/tried to make it works:
class PersonResource(Resource):`
    @use_args(person_args)
    def put(self, person_id, args):
        result = db.persons.update_one({'_id': ObjectId(person_id)}, { '$set' : args })
        return json_response(result.raw_result)

But it's not working, the error is:
TypeError: put() got multiple values for keyword argument 'person_id'
If I change the order:
class PersonResource(Resource):
    @use_args(person_args)
    def put(self, args, person_id):
        result = db.persons.update_one({'_id': ObjectId(person_id)}, { '$set' : args })
        return json_response(result.raw_result)

I get another error:
'$set' is empty. You must specify a field
This is because 'args' is an empty dict, for some reason.
Is there any workaround for this?

This way is not working neither:
class PersonResource(Resource):
    @use_args(person_args)
    def put(self, args):
        person_id = args['person_id']
        result = db.persons.update_one({'_id': ObjectId(person_id)}, { '$set' : args })
        return json_response(result.raw_result)

The error is:
TypeError: put() got an unexpected keyword argument 'person_id'

I had a mistake in the json object,
that why this code example, didn't work:
class PersonResource(Resource):
    @use_args(person_args)
    def put(self, args, person_id):
        result = db.persons.update_one({'_id': ObjectId(person_id)}, { '$set' : args })
        return json_response(result.raw_result)

It's works fine now.


Update Get-FASTSearchSecurityPublicFilter.md

Format example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Get-FASTSearchSecurityPublicFilter.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix(lttng): change caret-rclcpp violation as warn

Signed-off-by: hsgwa<EMAIL_ADDRESS>caret-rclcpp assertion is checked by whether caret-rclcpp tracepoits are recorded.
However, when all nodes publish with iter-process communication, no caret-rclcpp tracepoints are recorded.

message_construct
rclcpp_intra_publish
dispatch_subwscription_callback
dispatch_intra_subscription_callback

If target application doesn't have any subscription and intra-process publish, the original implementation incorrectly raises an error.
This PR modifies caret-rclcpp check from assertion to warning.
To reproduce: https://drive.google.com/drive/folders/1og8VNwxQdb8Bo-zoSO6_E2IPiTDWx0TE?usp=sharing
from caret_analyze import Lttng
Lttng(''aaa_sample')

InvalidTraceFormatError                   Traceback (most recent call last)
Input In [2], in <cell line: 1>()
----> 1 lttng = Lttng('aaa_sample')

File ~/ros2_caret_ws/build/caret_analyze/caret_analyze/infra/lttng/lttng.py:335, in Lttng.__init__(self, trace_dir_or_events, force_conversion, event_filters, store_events, validate)
    333 self._info = LttngInfo(data)
    334 self._source: RecordsSource = RecordsSource(data, self._info)
--> 335 self._counter = EventCounter(data, validate=validate)
    336 self.events = events
    337 self._begin = begin

File ~/ros2_caret_ws/build/caret_analyze/caret_analyze/infra/lttng/event_counter.py:34, in EventCounter.__init__(self, data, validate)
     32 self._count_df = self._build_count_df(data)
     33 if validate:
---> 34     self._validate()

File ~/ros2_caret_ws/build/caret_analyze/caret_analyze/infra/lttng/event_counter.py:85, in EventCounter._validate(self)
     79     raise InvalidTraceFormatError(
     80         'Failed to found trace point added by LD_PRELOAD. '
     81         'Measurement results will not be correct. '
     82         'The measurement may have been performed without setting LD_PRELOAD.')
     84 if len(set(recorded_trace_points) & trace_points_added_by_fork_rclcpp) == 0:
---> 85     raise InvalidTraceFormatError(
     86         'Failed to found trace point added by forked rclcpp. '
     87         'Measurement results will not be correct. '
     88         'The binary may have been compiled without using fork-rclcpp.')

InvalidTraceFormatError: Failed to found trace point added by forked rclcpp. Measurement results will not be correct. The binary may have been compiled without using fork-rclcpp.

@hsgwa  In this case, Architecture('lttng', 'aaa_sample') might be failed because any communication is not found.
Could you consider the ideal specification  to deal with this case?
---------------------------------------------------------------------------
ItemNotFoundError                         Traceback (most recent call last)
/tmp/ipykernel_8046/2614739864.py in <module>
      2
      3 arch = Architecture('yaml', architecture_path)
----> 4 app = Application(arch, lttng)

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/application.py in __init__(self, architecture, infra)
     74             raise UnsupportedTypeError('')
     75
---> 76         loaded = RuntimeLoaded(architecture, provider)
     77
     78         self._nodes: List[Node] = loaded.nodes

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in __init__(self, architecture, provider)
     59         self._comms = comms_loaded.data
     60
---> 61         paths_loaded = PathsLoaded(
     62             architecture.paths, nodes_loaded, comms_loaded)
     63         self._paths = paths_loaded.data

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in __init__(self, paths_info, nodes_loaded, comms_loaded)
    578         comms_loaded: CommunicationsLoaded,
    579     ) -> None:
--> 580         self._data = [
    581             self._to_runtime(path_info, nodes_loaded, comms_loaded)
    582             for path_info

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in <listcomp>(.0)
    579     ) -> None:
    580         self._data = [
--> 581             self._to_runtime(path_info, nodes_loaded, comms_loaded)
    582             for path_info
    583             in paths_info

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in _to_runtime(path_info, nodes_loaded, comms_loaded)
    594         for elem_info in path_info.child:
    595             child.append(
--> 596                 PathsLoaded._get_loaded(elem_info, nodes_loaded, comms_loaded)
    597             )
    598             if isinstance(elem_info, NodePathStructValue):

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in _get_loaded(path_element, nodes_loaded, comms_loaded)
    618
    619         if isinstance(path_element, CommunicationStructValue):
--> 620             return comms_loaded.find_communication(
    621                 path_element.topic_name,
    622                 path_element.publish_node_name,

~/workspace/caret/build/caret_analyze/caret_analyze/runtime/runtime_loaded.py in find_communication(self, topic_name, publish_node_name, subscribe_node_name)
    694                 comm.topic_name == topic_name
    695
--> 696         return Util.find_one(is_target, self._data)
    697
    698

~/workspace/caret/build/caret_analyze/caret_analyze/common/util.py in find_one(condition, items)
     82         filtered = Util.filter_items(condition, items)
     83         if len(filtered) == 0:
---> 84             raise ItemNotFoundError('Failed find item.')
     85         if len(filtered) >= 2:
     86             raise MultipleItemFoundError('Failed to identify item.')

ItemNotFoundError: Failed find item.

Could you consider behavior CARET should follow?
Architecture('lttng', 'aaa_sample')
ex. generate as possible, stop generating with warning, raises exceptions.

As a basis of CARET, it produces results as possible.
For example, current implementation outputs warnings instead of exceptions

Target application has a node which violates CARET limitations.
Target application has python implemented as well.

CARET produces figures as possible.
And it outputs warnings to give hints when there aren't some nodes to be shown.
However, CARET raises exceptions if measurements is invalid clearly.
This policy makes users to re-measure with correct way.
This PR modifies exceptions patterns to warning patters, because there are some cases which is not invalid measurement clearly.
Following this CARET policy,  CARET should generate architecture too as possible with warning outputs.


Dead link?

Hi there, I came via your blog, but the link back near the top of the README seems dead? Should it be this one?
https://depth-first.com/articles/2021/09/22/beyond-smiles/

You're right. Thanks for the catch.


Check HDD error in background & signal on LCD/WebUI

Ran into a "EXT4-fs error" on my testmachine - I would have not been able to see this if I had not an LCD or had the LCD in a graphics mode.
And easy way to detect that the HDD is down is:
admin@<IP_ADDRESS>:~ ₿ ls /mnt/hdd/ 1>/dev/null
ls: reading directory '/mnt/hdd/': Input/output error

A shutdown is a first option with checking if cables are all in.
This could even be part of the systeminfo (also without logoin?) of the WEBUI so that it is shown with an helping Link.

Hi There,
yesterday afternoon i get an "EXT4-fs error" (see also report #3614) Ver 1.8.0 - SSD is nearly a year old.. after that no Sync progress. Tried to get newest Version with new SD but now stuck in prepare Blockchain sync .. no idea what to do


The transmission will fail after half an hour?

You must fill out the template when opening such an issue or otherwise we do not have enough information to help you. A screenshot is not a sufficient description for a bug.

Closing this issue due to a lack of information. Feel free to leave a comment if you want to continue the discussion :)


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.3 to before v5.4.156",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=bf361231c295d92a28ca283ea713f56e93e55796",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=d6f451f1f60c58d73038c7c3177066f8f084e2a2"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "bf361231c295d92a28ca283ea713f56e93e55796",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "d6f451f1f60c58d73038c7c3177066f8f084e2a2",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "audit: fix possible null-pointer dereference in audit_filter_rules\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.156 by commit d6f451f1f60c58d73038c7c3177066f8f084e2a2, it was introduced in version v5.3 by commit bf361231c295d92a28ca283ea713f56e93e55796. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1002007


Update manifest-schema-dev-preview.md

Manifest Json correction

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
90
26
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 7b0b2ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema-dev-preview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Missing @Subscribe event annotation in green robot example

$subj

My example is for GreenRobot EventBus 2.4.1. As far as I remember that version does not support "Subscribe" annotations and this event bus works without them, so this example is valid. I didn't follow latest changes in this bus, but I see they introduced annotations. PR-s with upgraded version of GreenRobot event bus in example are welcome.

Yeah, sorry, I didn't check what lib version you're using. PR created with update GreenRobot lib version.

Okay, thanks. I'll review it soon. :)

Solved by #114.


[Feat] add cue util to convert jsonschema to cue

Signed-off-by: Nithish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Abhishek-kumar09 ok, sounds good.

Hopefully, we can get @sudo-NithishKarthik into a habit of returning to his PRs for follow up. 🤞 🙂


support gitbook

gitalk so good，can support gitbook in future.

So... you hope how to support?

whether it is possible to implement one plugin for gitbook to easy to use?

So... you hope how to support?

whether it is possible to implement one plugin for gitbook to easy to use?


Notebook Cell Reordering Visual Glitch

Steps to Reproduce:

Create Cells with the following Hierarchy

Header Cell
Code cell 1
Code cell 2
Subheader Cell
Code Cell 3

Grab Subheader Cell and move it to the top of the file
Collapse Header Cell
Grab Header Cell and drop at the top of the file

🐛 => Subheader Cell is missing until you collapse and uncollapse Header Cell

It doesn't reproduce now in latest build.


[BUG] washboard provider view shows "pending" perpetually

Affected project(s)

[ ] documentation
[ ] examples
[ ] wasmCloud host
[ ] wasmCloud CLI (wash)
[X] wasmCloud dashboard UI (washboard)
[X] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
The wasmCloud dashboard always shows that my provider is "Pending", even when the real state is "Running"
Steps to reproduce

Run wash new component to generate a hello-world project
run wash up -d to launch wasmCloud
cd component && wash build
wash app deploy ./wadm.yaml
wash ui
visit localhost:3030
see Pending

Expected behavior
I expect for the provider health to automatically
Environment

OS: [e.g. Linux, MacOS]
Shell [e.g. bash, zsh, powershell]
wasmcloud Version [e.g. 0.81.0]

Screenshots / Logs / Additional context

However, in wadm_state:
➜ nats kv get wadm_state provider_default --raw |jq
{
  "test-arbx_graph": {
    "id": "test-arbx_graph",
    "name": "Custom Provider",
    "issuer": "ABJ6WQ5FBGXECIJBBGA7BG73IKE7ITD2DRAURV5WB5ZUTYTNOOAYGPIG",
    "reference": "file:///Users/brooks/tmp/luk3/footest/fooprovider/build/wasmcloud-provider-custom-template.par.gz",
    "hosts": {
      "NBFN4TRTQ5Q6UANE5R2KAVHCXOIA464MWBCIROGIASV2DRWIBLU3VE5J": "Running"
    }
  }
}

Hello @brooksmtownsend. I would like to work on this.

Thanks @Alex-omosa ! Let me know if you need additional pointers, I haven't poked around the washboard quite yet to see where this may be coming from

Ok  👍 @brooksmtownsend

Hey @Alex-omosa, I was kinda struggling with other things so I went ahead and looked around this. (I was a bit annoyed that it never turns into "Running" too).
Not sure how far you have got with your version, but I made some progress with this
https://github.com/wasmCloud/wasmCloud/compare/main...markkovari:wasmCloud:fix/washboard/provider-status?diff=split&w=
but I am not the most confident that this is the best way to fetch the providers status. 🤔
If you thing I can open a PR and we can work on it together. 🤝
On the other hand I will be brave and ping @lachieh here too. Is this a generic enough fetch for the provider data or do what is the better way to get it?

@markkovari I'm not @lachieh , so I'll let him respond, but I think that your diff looked reasonable and if it fixes the issue we'll take the PR!

@markkovari & @brooksmtownsend Please ignore the PR I just made.

@Alex-omosa Sounds good, are you sure? Just let me know

I'm now very confident about my solution @brooksmtownsend . Maybe @lachieh can review that first.

Ok, I've finally had the time to take a closer look at these changes. First of all, thanks to both @Alex-omosa and @markkovari for doing the digging on this one!
As @markkovari discovered, the reason the status isn't working is because the host info which is passed to the providers table doesn't actually contain the provider status. The data for a single provider row looks like this:
{
    "id": "demo-blobstore",
    "name": "blobstore-fs-provider",
    "annotations": {
        "wasmcloud.dev/appspec": "wasmcloud-demo",
        "wasmcloud.dev/managed-by": "wadm",
        "wasmcloud.dev/scaler": "a4cc53c73b4dd40a97be9649d497248c2c416fea28f3f12bfac88294e580aff6",
        "wasmcloud.dev/spread_name": "default"
    },
    "image_ref": "ghcr.io/wasmcloud/blobstore-fs:0.9.0",
    "hosts": [
        "NB5453O72IYNEUNYO7V32YUN7V4FIOIKCLD3OQXTMT33NUZ52CV4FXMB"
    ]
}

The code for the health is assuming that the "hosts" key is actually an object with the status as the value, like is found in the wadm state bucket for providers. Here's the same provider data directly from the bucket:
{
    "id": "wasmcloud_space-blobstore",
    "name": "blobstore-fs-provider",
    "issuer": "ACOJJN6WUP4ODD75XEBKKTCCUJJCY5ZKQ56XVKYK4BEJWGVAOOQHZMCW",
    "reference": "ghcr.io/wasmcloud/blobstore-fs:0.9.0",
    "hosts": {
      "NB5453O72IYNEUNYO7V32YUN7V4FIOIKCLD3OQXTMT33NUZ52CV4FXMB": "Running"
    }
  }

@markkovari's solution covers this by fetching the extra provider data and merging it with the host data so I think that is the solution to move this one across the line. Mark, if you want to open a PR, we can get the ball rolling on a merge and release.
@Alex-omosa, thanks again for your investigations!

Oh my;
Currently the expanded row of a provider looks like this.

I think the column should say the status of the provider on/at the host and not the name.
Please let me know @lachieh @brooksmtownsend and @Alex-omosa if you think it is a go/no go, I can revert this.

Respresents the following state:
nats kv get wadm_state provider_default --raw | jq

{
  "rust_hello_world-httpserver_2": {
    "id": "rust_hello_world-httpserver_2",
    "name": "http-server-provider",
    "issuer": "ACOJJN6WUP4ODD75XEBKKTCCUJJCY5ZKQ56XVKYK4BEJWGVAOOQHZMCW",
    "reference": "ghcr.io/wasmcloud/http-server:0.23.1",
    "hosts": {
      "NA76VOLBN6PJIZA5YLE5PWVQ4VKNCLAJZZMJYG5F3FX5FMDN2CISAZ5Y": "Pending"
    }
  },
  "rust_hello_world_2-httpserver_2": {
    "id": "rust_hello_world_2-httpserver_2",
    "name": "http-server-provider",
    "issuer": "ACOJJN6WUP4ODD75XEBKKTCCUJJCY5ZKQ56XVKYK4BEJWGVAOOQHZMCW",
    "reference": "ghcr.io/wasmcloud/http-server:0.23.1",
    "hosts": {
      "NA76VOLBN6PJIZA5YLE5PWVQ4VKNCLAJZZMJYG5F3FX5FMDN2CISAZ5Y": "Running"
    }
  }
}

Thanks! I'll update this thread once there is a new washboard release out.


Fix acceptance tests to avoid them to stuck during execution

We currently have a skip against the acceptances tests because they were causing issues during the execution of the tests.
The tests were running successfully but they were getting stuck and avoiding the completion of the run.
Note: might be related to code coverage.

Fixed in https://github.com/ciena-frost/ember-frost-core/pull/337/files#diff-f110515e6af13dcf2214d5f28da950a0R6 thanks to @rox163


Ensure that Nex can be built on non-Linux operating systems

Proposed change
Allows for Nex to be built/released for non-Linux operating systems
Use case
The primary use case for this is for people who want to remotely control/operate Nex nodes from machines that don't meet the node execution requirements but can still act as control clients. This could be either Windows or MacOS.
Ideally if you run nex node ... on non-Linux, you'd get an error message rather than the current behavior, which I believe fails to compile on MacOS.
Contribution
No response

I think this was closed by #78


Increase default connection timeout to 30s

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "issue-13307" git@github.com:lightsofapollo/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

@lightsofapollo Hah, sorry :) You need to sign your commit.

@LK4D4 Magic incantation complete!

LGTM

LGTM if janky's happy


Enhancements to Tagging

This might be multiple enhancements. The Tagging features exist only on "Product" and appear only on the Product View and Edit pages. I propose the following enhancements to "Tagging" as a whole:

Expanded tagging: ability to place tags on endpoints, engagements, tests, maybe even findings
Search: use tags to search in the universal search box. Results should identify what type of data the tag appeared on (separated by UI widget, or by a table field) with a link to view that object page.
Auto-search: clicking on a tag (anywhere they are displayed) should perform a "search: tag=XYZ" function
Inheritance: if a tag is applied to an Engagement, it also shows up on the Product (perhaps in a different color to show inheritance, maybe with a super-script number to show count of inherited instances of that tag)
Metrics: Report on tags in a dashboard view or a "word cloud" kind of thing to show the list of tags and how many times they appear, perhaps broken down into the type of thing they appear on (Tag A appears on Products 7 times, Engagements 3 times and Tests 57 times)

@humor4fun Just pushed a PR to bring out tagging to the various views. Tagging was on finding, engagement, endpoint etc. just not obvious that it was there. Added tags to each view and simple search will categorize where the tag was found.
PR: https://github.com/OWASP/django-DefectDojo/pull/242

I'm going to go ahead and close. Feel free to reopen if necessary.


[release-4.9] Mid-December bump

/override ci/prow/e2e-aws-upgrade
OpenShift APIs remain available with reused connections
OAuth APIs remain available with reused connections


Some Error Makes Game Crash when Ending the Game

Hi, there. Thanks to your greate work that let me enjoy the game. But there is an unknown issue crash the game frequently with reframework.
Here are the log and mini dump files, hopes they can help you to figure out the reason.
re2_framework_log.txt
reframework_crash.zip

seems like some wrong with script.


Rate limiting

I have to admit, the way I implemented rate limiting was cheap. I'm sorry for that. We still have to improve on it.
A few resources that are relevant:
https://github.com/hammerandchisel/discord-api-docs/issues/108
https://github.com/hammerandchisel/discord-api-docs/blob/master/docs/topics/Rate_Limits.md
Please comment with any other resources that could help developing rate limiting for DSharpPlus

Naamloos

Going to work on it tonight

More resources
https://github.com/hammerandchisel/discord-api-docs/commit/c4a4783fbacf69289f3841dedc9658b3016bc5fd

More rate limit info
https://github.com/hammerandchisel/discord-api-docs/blob/master/docs/topics/Rate_Limits.md (duh)
as you can see, every request also gives us some headers telling us how many requests we are able to make for that route, and the time it takes before it resets.

Implemented rate limiting. phew.. what a b*tch..
https://github.com/NaamloosDT/DSharpPlus/commit/588d2cc3bb219cf072e1cde7c8fda67e1b14f1a9
Please check if you guys agree on the way I did it.
If you guys agree, close this issue and read the bottom comment here: https://github.com/hammerandchisel/discord-api-docs/issues/108

No objections? well then I consider it fixed :D


Add data selector to Analytics Pane

Currently there is no way to bind Analytics pane features to measure as it happens in built in visuals. That feature would massively improve functionality of customs visuals (e.g. adding custom lines to visuals that are based on measure calculation made on context of whole visual)

The issue isn't related to the Tools package. Feel free to contact<EMAIL_ADDRESS>with questions related to the development of custom visuals or create a feature request on PowerBI forum


[9.x] Adds handlers for silently discarded and missing attribute violations

This PR adds the ability to register handlers for two of the strict mode violations: silently discarded and missing attributes.
This ability is currently available for only the lazy load violations, this PR basically replicates that behavior for the other 2/3rds of Eloquent's new strict mode.
I think this will make it easier to adopt strict mode, as people can enable strict mode in production, but instead of throwing violations, they can log them.

@aarondfrancis,  maybe this is going to be a good feature but did you see that a lots of changes are showing just due to changes of a single space. I think it was happened due to IDE configuration, so that this type of unnecessary changes are showing. Could you please recheck the changes again please??

Fixed. I kinda thought that would happen automatically, which is why I left it. Thanks @deKuddus.
And thanks to @timacdonald for this incredible tip on how to fix this!


feat: support .md

This PR adds .md as a route file extension and also supports omission of the extension.
This should be provided as a config so that other extensions can do this, but vite does not support it.
closes: #187

Thanks @goisaki !

Wow! It was fast!

You too!


feat: admin users

#47
TODO

[x] Users

[x] API Routes
[x] List
[x] View

[x] Persons

[x] API Routes
[x] List
[x] View

[x] Organisations

[x] API Routes
[x] List
[x] View

[x] TS

[x]  Search

[ ] Pagination

:tada: This PR is included in version 1.33.0 :tada:
The release is available on:

v1.33.0
GitHub release

Your semantic-release bot :package::rocket:


GP-218: Updated Portable Documentation

Updated documented procedures for using Portable Globe Server with the current Open Source implementation.

References to Mac OS X have been removed, since Portable Server hasn't been tested on Mac OS.
The answer document titled "Portable Globe and Server 4.4" has been renamed to "Portable Globe and Server 5.2", and should be removed, since it has been replaced by content in a number of other documents. That task hasn't been performed for this issue ticket.
References to OS installers for Portable have been replaced by references to the build instructions on the online Wiki and running Portable from the built Portable archive.
References to the Google Earth Plug-in have been augmented with notices that the plug-in has been discontinued.
Configuration file, and globe and map directory paths have been updated.
Command-line options for command-line utilities have been updated.
The instructions for enabling broadcasting on a served globe have been updated.
Broadcasting enabled and broadcasting disabled icons (<broadcast_true.gif> and <broadcast_false.gif>) have been copied to the documentation  folder.
CSS classes have been added for displaying these icons inline in document text.

I have signed.

@tst-awankhede: Mac OS is mentioned on the Wiki page for Portable.
I think we may have forgotten to migrate some of the tickets for testing on Mac OS. We can synchonize with #333.

Everything looks good to me.  I'm going to keep it open for now in case @tst-ccamp or @tst-awankhede have anything else to add.


Translate Average to server side

https://github.com/aspnet/EntityFramework/issues/7190

Nothing to do here, implemented inside EF Core.


Changes to Guaranteed Replica Manager CPU should not cause cluster-wide volume restart

Currently, changes to Guaranteed Replica Manager CPU cause all volumes in the cluster to be disrupted, see docs:

After this setting is changed, all replica manager pods using this global setting on all the nodes will be automatically restarted. In other words, DO NOT CHANGE THIS SETTING WITH ATTACHED VOLUMES.
https://longhorn.io/docs/1.2.3/references/settings/#guaranteed-replica-manager-cpu

This request is to have the restart be localized to only a subset of the volumes so running applications can better tolerate the change. For instance, applying the change only to one node at a time would allow the node to be drained and cordoned before which would remove the attached volumes on that host.
An ideal UX:

User makes changes to the Guaranteed Replica Manager CPU setting in the longhorn UI/API
Longhorn records the change but does not apply to a replica manager until that manager no longer has attached pods.
The user can commence a drain/cordon maintenance operation on each node in a rolling fashion, which in turn triggers the replica manager pod to apply the change and restart

I think this is a good idea. Giving the user the ability to manage the change in setting via drain/cordon (Independent from the cordon/drain we only terminate pods once there is no longer a running instance)
The same change would need to be applied on the per node level configuration we have, but both should be able to utilize the same approach:
desired.request = X // spec
current.request = Y // status

if (safeToPerform && desired.request != current.request) {
    // update the resource requests and restart pod
}


How to use lettuce in the right way

Bug Report
When the server is under a lot of pressure, call lettucePool.acquire().get(1, TimeUnit.SECONDS) and the program has the following exception.
Exception
java.util.NoSuchElementException: Pool exhausted
...
Other exception stack information

Java code
// config  RedisClusterClient
ClientResources clientResources = DefaultClientResources.builder()
                                     .ioThreadPoolSize(3 * Runtime.getRuntime().availableProcessors())
                                     .computationThreadPoolSize(4)
                                     .reconnectDelay(Delay.constant(Duration.ofSeconds(1)))
                                     .build();

RedisClusterClient clusterClient = RedisClusterClient.create(clientResources, redisURIS);
ClusterTopologyRefreshOptions topologyRefreshOptions = ClusterTopologyRefreshOptions
                                                         .builder()
                                                         .enableAdaptiveRefreshTrigger(ClusterTopologyRefreshOptions.RefreshTrigger.MOVED_REDIRECT,ClusterTopologyRefreshOptions.RefreshTrigger.PERSISTENT_RECONNECTS)
                                                         .adaptiveRefreshTriggersTimeout(Duration.ofMinutes(3))
                                                         .build();
clusterClient.setOptions(ClusterClientOptions.builder()
                                                         .topologyRefreshOptions(topologyRefreshOptions)
                                                         .autoReconnect(true)
                                                         .pingBeforeActivateConnection(true)
                                                         .build());

// config lettucePool
// maxIdle = 8
// minIdle = 1
// maxActive = 8
// testOnCreate = true
final BoundedPoolConfig boundedPoolConfig = BoundedPoolConfig
                .builder()
                .minIdle(minIdle)
                .maxIdle(maxIdle)
                .maxTotal(maxActive)
                .testOnCreate(testOnCreate)
                .testOnAcquire(true).build();
final BoundedAsyncPool<StatefulRedisClusterConnection<String, String>> lettucePool =
                AsyncConnectionPoolSupport.createBoundedObjectPool(() -> clusterClient.connectAsync(StringCodec.UTF8), boundedPoolConfig);
===============================================================

// way of use
@Autowired
BoundedAsyncPool<StatefulRedisClusterConnection<String, String>> lettucePool;
// way of use
final StatefulRedisClusterConnection<String, String> connection = lettucePool.acquire().get(1, TimeUnit.SECONDS);

Looks like the pool is empty and the connection isn't returned to the pool after using it or the pool is too small.

CPU: 8 core
Memory: 16 G

Why do you use pooling at all?
The pool size depends pretty much on how many connections you use concurrently. This isn't something I can help with.

Why do you use pooling at all?
The pool size depends pretty much on how many connections you use concurrently. This isn't something I can help with.

Isn't using a connection pool like this?
Way of use（Need to call close every time??）
@Autowired
RedisClusterClient clusterClient;
===================================================
// ... code
StatefulRedisClusterConnection<String, String> connection = null;
try {
    connection = clusterClient.connectAsync(StringCodec.UTF8).get();
    String testValue = connection.async().get("testKey").get();
} catch (Exception e) {
    log.error("exception");
} finally {
    if (connection != null)
        connection.close(); // Need to call close ?
}
// ... other code

Config RedisClusterClient
// config  RedisClusterClient
ClientResources clientResources = DefaultClientResources.builder()
                                     .ioThreadPoolSize(3 * Runtime.getRuntime().availableProcessors())
                                     .computationThreadPoolSize(4)
                                     .reconnectDelay(Delay.constant(Duration.ofSeconds(1)))
                                     .build();

RedisClusterClient clusterClient = RedisClusterClient.create(clientResources, redisURIS);
ClusterTopologyRefreshOptions topologyRefreshOptions = ClusterTopologyRefreshOptions
                                                         .builder()
                                                         .enableAdaptiveRefreshTrigger(ClusterTopologyRefreshOptions.RefreshTrigger.MOVED_REDIRECT,ClusterTopologyRefreshOptions.RefreshTrigger.PERSISTENT_RECONNECTS)
                                                         .adaptiveRefreshTriggersTimeout(Duration.ofMinutes(3))
                                                         .build();
clusterClient.setOptions(ClusterClientOptions.builder()
                                                         .topologyRefreshOptions(topologyRefreshOptions)
                                                         .autoReconnect(true)
                                                         .pingBeforeActivateConnection(true)
                                                         .build());

Lettuce connection are designed to be long-lived and they are thread-safe. If you use Redis Cluster with simple GET and SET commands, then you don't need pooling at all. The only thing to do is injecting StatefulRedisClusterConnection into your code and simply using it.


Added profile method to utilize getGavatarProfile method in gravatarlib

@ryanrobertsname If you still want this functionality, can you update your PR? Otherwise this PR will be closed.

thomaswelton/gravatarlib doesn't have a tag of 0.2.x. Do you want to revert the breaking commit?

@sumityadav The issue is that it requires the following PR (https://github.com/thomaswelton/gravatarlib/pull/1) to be merged  to be able to call the getGravatarProfile method on thomaswelton/gravatarlib.
I don't have push access to the other repository, so I can't help here. And I'd rather not do it only in the Laravel specific implementation. @thomaswelton
On a side note, the README will need to be updated to reflect the new feature, with proper documentation with links to https://en.gravatar.com/site/implement/profiles/ or http://portablecontacts.net/draft-spec.html


Add Mountain Bike/Gravel Profile

It would be very useful to have a MTB/Gravel profile like this one: https://raw.githubusercontent.com/zossebart/brouter-mtb/master/mtb-zossebart.brf. Is it possible to include it in the profile list ?
Thanks!

Agree @hugobqd, we need to have https://github.com/abrensch/brouter/pull/179 first, then we can/will be able to add MTB profile(s) in the web interface!

+1
When will it be made available?
Thanks

FYI, several MTB and Gravel profiles are available at the BRouter-Web instance https://bikerouter.de/


MAUI iOS - Bluetooth.ScanForDevicesAsync does not return devices, but RequestDeviceAsync does

I'm trying to communicate with BLE devices in my MAUI app. To provide a better UX I want to filter the scanned BLE devices and show them in my own UI. However, I can't seem to make Bluetooth.ScanForDevicesAsync return anything other than an empty collection.
Here's how I'm using it:
var devices = await Bluetooth.ScanForDevicesAsync(new RequestDeviceOptions() { AcceptAllDevices = true, });

foreach (var device in devices)
{
    AvailableDevices.Add(device.Name);
}

When I use var selectedDevice = await Bluetooth.RequestDeviceAsync(new RequestDeviceOptions() { AcceptAllDevices = true });, the popup that shows up does give devices and I'm also able to communicate with them.
Am I using Bluetooth.ScanForDevicesAsync wrong? Or is this not working properly?
Running on iPadOS 16.5.1.

ScanForDevices (programmatic scan of unpaired devices) is not currently implemented for iOS. You can access paired devices or use the RequestDevice picker.

Good to know, thanks!

I'm going to reopen this to track progress on adding this functionality.

The ScanForDevices method has not been implemented on iOS yet. There is no specific reason why it cannot - it'll be similar to the implementation already used to build the device picker for RequestDeviceAsync but I have not had the time to complete it.

Hi @peterfoot , thanks for the great work! I use your libraries since wince.... do you have a possible date for this request?


Workaround MSVC bug that std::isnan cannot handle integral type

#15213

Ping. Test failures should be unrelated to my change, do I need to rebase?

@hgadig I believe this is ready to pull?

@hgadig I believe this is ready to pull?

@jlebar  Looks like the CL has merge conflicts. Request you to take a look.

Request you to take a look.

Sure, since this appears to be our mistake that this sat for two weeks with no response, I'd like to try merging this PR without asking the contributor to rebase.
What's the best way to do this?

@hgadig friendly ping

@hgadig FYI I will be OOO tomorrow through Tuesday, so if we can't fix this today, please send the internal review to timshen or sanjoy.

@hgadig FYI I will be OOO tomorrow through Tuesday, so if we can't fix this today, please send the internal review to timshen or sanjoy.

Sure. I'm working with Yifei to resolve internal merge conflicts. Thanks !

woohoo, thank you for your patience here.  Very happy this one is done.


feat: add diff confirm modal in create wizard

Signed-off-by: suzhou<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #323 (e2582af) into index-operation (48f5ecb) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head e2582af differs from pull request most recent head accfd02. Consider uploading reports for the commit accfd02 to get more accurate results

@@                 Coverage Diff                 @@
##           index-operation     #323      +/-   ##
===================================================
- Coverage            52.49%   52.46%   -0.03%
===================================================
  Files                  200      201       +1
  Lines                 6414     6427      +13
  Branches              1013     1015       +2
===================================================
+ Hits                  3367     3372       +5
- Misses                2795     2802       +7
- Partials               252      253       +1

Impacted Files
Coverage Δ

...CreateIndex/containers/CreateIndex/CreateIndex.tsx
51.26% <0.00%> (-1.90%)
:arrow_down:

...ashboards-plugin/public/components/Modal/Modal.tsx
62.79% <0.00%> (-1.50%)
:arrow_down:

...in/public/components/MonacoEditor/MonacoEditor.tsx
75.00% <0.00%> (ø)

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


Convert array of JSONModel objects

I have a JSON response which contains array of JSONModel objects inside the array. How can I convert to NSArray by using AFHTTPClient as the http client?
JOSN--
[{id:1, name:"aaa"},
{id:2, name:"bbb"},
{id:3, name:"ccc"]

I have a similar request... if I have {"ids":[1,2,3]} as input, how should I model it?  NSArray ids does not work and it did not seem to want to deserialize the numbers.  thx.

if the response were {"languages":["en","us","ge"]}
How would you do it, I keep getting an error.
Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<LoanModel 0x7c5d3ce0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key description.'
Using
@property (strong, nonatomic) NSMutableArray<NSString*>* languages;

@LilJeff2626 This is not a related issue. Open a new issue with full source code if you need help.


compile on Mac

What version of Rust are you using (rustc --version)?
rustc 1.27.0-nightly (8a37c75a3 2018-05-02)

What operating system and processor architecture are you using?
10.13.2 (17C88)

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.

What did you expect to see?
build success

What did you see instead?
Downloading strsim v0.6.0
Downloading bitflags v0.8.2
Downloading ansi_term v0.9.0
Downloading atty v0.2.2
Downloading unicode-width v0.1.4
Compiling grpcio-sys v0.2.1
Compiling serde_derive v1.0.8
Compiling miow v0.1.5
Compiling mio v0.6.6
error: failed to run custom build command for grpcio-sys v0.2.1
process didn't exit successfully: /Users/code/tidb/tikv/target/debug/build/grpcio-sys-4217a94f67440ebb/build-script-build (exit code: 101)
--- stdout
cargo:rerun-if-changed=grpc_wrap.c
cargo:rerun-if-changed=grpc
cargo:rerun-if-env-changed=GRPCIO_SYS_USE_PKG_CONFIG
running: "cmake" "/Users/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.1/grpc" "-DCMAKE_INSTALL_PREFIX=/Users/code/tidb/tikv/target/debug/build/grpcio-sys-003e4fe1d2027f69/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -stdlib=libc++ -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_BUILD_TYPE=Debug"
-- Renaming
--     /Users/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.1/grpc/third_party/zlib/zconf.h
-- to 'zconf.h.included' because this file is included with zlib
-- but CMake generates it automatically in the build directory.
-- Configuring incomplete, errors occurred!
See also "/Users/code/tidb/tikv/target/debug/build/grpcio-sys-003e4fe1d2027f69/out/build/CMakeFiles/CMakeOutput.log".
See also "/Users/code/tidb/tikv/target/debug/build/grpcio-sys-003e4fe1d2027f69/out/build/CMakeFiles/CMakeError.log".

--- stderr
CMake Warning at CMakeLists.txt:122 (message):
gRPC_INSTALL will be forced to FALSE because gRPC_ZLIB_PROVIDER is "module"
CMake Warning at CMakeLists.txt:184 (message):
gRPC_PROTOBUF_PROVIDER is "module" but PROTOBUF_ROOT_DIR is wrong
CMake Error at third_party/boringssl/CMakeLists.txt:30 (message):
Could not find Go
thread 'main' panicked at '
command did not execute successfully, got: exit code: 1
build script failed, must exit now', /Users/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.28/src/lib.rs:631:5
note: Run with RUST_BACKTRACE=1 for a backtrace.
warning: build failed, waiting for other jobs to finish...
error: build failed

Hi! Please verify that you have go installed as it appears boringssl's cmake wants it.

I can't understand what I should install. boringssl's cmake?

Ah, I'm sorry. Please make sure you have go (as in https://golang.org/) installed.
If you're on Mac and using homebrew this should be sufficient:
brew install go

It's ok.Thanks!

That fixed it? Great! Feel free to open another issue if you have more problems. I'm trying to get an idea of where the kinks are in our READMEs so it's helpful for me too!


withRouter: Change of location doesn't trigger a re-render

When a deep component uses withRouter, because of how React handles context (https://github.com/facebook/react/issues/2517), the component is not updated when the location changed.
The main reason is because the location value is directly provided by the context. When this value change on the top level component, the deep component doesn't re-render if one component of the stack has a shouldComponentUpdate.
One solution may be to provide a constant subscribe function in the context instead of the value it self.
Current solution:
react-router/modules/Route.js#L36
class Route extends React.Component {
  ...

  getChildContext() {
    return {
      router: {
        ...this.context.router,
        route: {
          location: this.props.location || this.context.router.route.location,
          match: this.state.match
        }
      }
    }
  }
}

Proposed solution
Instead of having router in the context, provide a subscribeRouter function. This is a solution similar to what react-sticky is doing.
Here is an example:
class Route extends React.Component {
    ...

    constructor(props) {
        super(props);

        // Array of functions
        this.subscriptions = [];

        // Sorry crappy code ...
        let first = false;
        this.context.subscribeRouter(router => {
            if (first) {
                this.state = {
                    router,
                    match: this.computeMatch(props, router)
                };
            } else {
                this.setState({
                    router,
                    match: this.computeMatch(props, router)
                }, () => {
                    this.triggerSubscribers();
                });
            }
        });
    }

    getChildContext() {
        return {
            subscribeRouter: (fn) => {
                this.subscriptions.push(fn);
                this.triggerSubscriber(fn);
            }
        };
    }

    triggerSubscriber(fn) {
        const router = {
          ...this.state.router,
          route: {
            location: this.props.location || this.state.router.route.location,
            match: this.state.match
          }
        };

        fn(router);
    }

    triggerSubscribers() {
        this.subscriptions.forEach(fn => {
            this.triggerSubscriber(fn);
        })
    }
}

If you agree with the proposed solution, I can try a PR.

React Router intentionally does not use subscribers.
There is a somewhat long article I wrote about why here https://medium.com/@pshrmn/ditching-subscriptions-in-react-router-6496c71ce4ec, but the gist is that subscriptions end up making the code base overly complicated without much benefit.
You end up having to write your own scheduler to ensure that you don't get duplicate re-renders for one location change. Context usage also ends up a bit hacky because you have to manipulate the same object (Object.assign) to ensure that the child components that are subscribed are accessing the current context object.
Instead, React Router relies on updates propagating from the router to all of its children. If you are using update blockers (PureComponent or shouldComponentUpdate), you should refer to this guide to see how to deal with them in a React Router application (you just pass them the location as a prop so that shallow checks detect the change).

Hmm I feel like this is a pretty big gotcha the way things are currently set up! Basically we're told to use withRouter whenever we want to react to the route changing... BUT, in certain cases where you have a nested use of withRouter it just won't work.... that seems very bad, right?
I for one am definitely going to shift to some other solution than withRouter. Too bad though because other than that defect it seemed great.
Also of note, I read the following warning and didn't understand it because there is an important comma missing!

It should read: If you are using withRouter <> to prevent updates from being blocked by shouldComponentUpdate ...etc
Or better yet, get rid of the first part and just start the sentence: To prevent updates from being blocked by shouldComponentUpdate


fixed typos in readme

Codecov Report

Merging #33 (d8a0dd8) into master (6b42bd3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   86.54%   86.54%
=======================================
  Files           7        7
  Lines         587      587
  Branches      107      107
=======================================
  Hits          508      508
  Misses         64       64
  Partials       15       15


Spaces or Tabs

Please can we standardise on one or the other?

Go for tabs (length 4).

Thanks :smiley:


strange TypeError: dest.on is not a function with lazypipe

Hello @OverZealous I'm using lazypipe() as template to DRY my gulp tasks.
Unfortunately, I get an error:
serge@serge:~/projects/haystak$ gulp deploy
[20:41:50] Starting 'deploy'...
Building APP bundle
[20:41:50] 'deploy' errored after 25 ms
[20:41:50] TypeError: dest.on is not a function
    at Duplexer.Readable.pipe (/home/serge/projects/korob/node_modules/readable-stream/lib/_stream_readable.js:485:8)
    at rebundle (/home/serge/projects/korob/Gulpfile.js:69:8)
    at browserifyTask (/home/serge/projects/korob/Gulpfile.js:82:3)
    at Gulp.<anonymous> (/home/serge/projects/korob/Gulpfile.js:229:3)
    at module.exports (/home/serge/projects/korob/node_modules/orchestrator/lib/runTask.js:34:7)
    at Gulp.Orchestrator._runTask (/home/serge/projects/korob/node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/home/serge/projects/korob/node_modules/orchestrator/index.js:214:10)
    at /home/serge/projects/korob/node_modules/orchestrator/index.js:279:18
    at finish (/home/serge/projects/korob/node_modules/orchestrator/lib/runTask.js:21:8)
    at cb (/home/serge/projects/korob/node_modules/orchestrator/lib/runTask.js:29:3)

my gulpfile (I removed unnecessary lines out of scope + plus add references to lines):
var browserify  = require('browserify');
var babelify    = require('babelify');
var pathmodify  = require('pathmodify');
var path        = require('path');
var concat      = require('gulp-concat');
var cssmin      = require('gulp-cssmin');
var glob        = require('glob');
var gulp        = require('gulp');
var gulpif      = require('gulp-if');
var gutil       = require('gulp-util');
var rev         = require('gulp-rev');
var revReplace  = require('gulp-rev-replace');
var buffer      = require('gulp-buffer');
var insertLines = require('gulp-insert-lines');
var livereload  = require('gulp-livereload');
var htmlmin     = require('gulp-htmlmin');
var notify      = require('gulp-notify');
var sass        = require('gulp-ruby-sass');
var shell       = require('gulp-shell');
var slim        = require('gulp-slim');
var source      = require('vinyl-source-stream');
var streamify   = require('gulp-streamify');
var uglify      = require('gulp-uglify');
var watchify    = require('watchify');

var lazypipe = require('lazypipe');
// Add md5 hashsums to assets
function revFiles(dest) {
  return lazypipe()
    .pipe(gulp.dest, dest);
}

var browserifyTask = function (options) {
  // Our app bundler
  var appBundler = browserify({
    entries: [options.src], // Only need initial file, browserify finds the rest
    debug: options.development, // Gives us sourcemapping
    cache: {}, packageCache: {}, fullPaths: options.development // Requirement of watchify
  })
    .plugin(pathmodify, {
      mods: [
        pathmodify.mod.dir('npm', path.join(__dirname, 'node_modules'))
      ]
    })
    .transform(babelify, { presets: ['react', 'es2015'] })

  // The rebundle process
  var rebundle = function () {
    var start = Date.now();
    console.log('Building APP bundle');
    appBundler.bundle()
      .on('error', gutil.log)
      .pipe(source('application.js'))
      .pipe(gulpif(!options.development, streamify(uglify())))
      .pipe(revFiles(options.dest)) // line 69
      .pipe(notify(function () {
        console.log('APP bundle built in ' + (Date.now() - start) + 'ms');
      }))
      .on("finish", function() { refreshView(options) });
  };

  // Fire up Watchify when developing
  if (options.development) {
    appBundler = watchify(appBundler);
    appBundler.on('update', rebundle);
  }

  rebundle(); // line 82
}

gulp.task('deploy', function () {
  browserifyTask({ // line 229
    development: false,
    src: './app/coffee/application.js',
    dest: './public/dist/'
  });
});

I'll be appreciated by your help

You aren't calling the function returned by lazypipe.  That line should be revFiles(options.dest)().  Or call the result of lazypipe from within your function.
That being said, in this case, there's no need for lazypipe.
lazypipe is intended to be used when you need to keep a reference to a multi-step pipe so you can reuse it in multiple setups.  For example, maybe you have a long chain that is used in development builds, production builds, and incremental watches.  In this case you can make a single lazypipe, then add on to it or insert it into another stream.
But in your case, you've already wrapped the pipe in a function, so just skip the extra step:
function revFiles(dest) {
  return gulp.dest(dest);
}

If you have several steps instead, then you might still benefit from lazypipe.

@OverZealous gotcha, thanks! It's working now

I have the same actually.
import replace from "gulp-replace";
import gulpIf from "gulp-if";
// ....
let replacement = lazypipe().pipe(replace, "../ts/", "../dist/");

// ...
    .pipe(gulpIf(replacement ? true : false, replacement))
// or
     .pipe(replacement)

Give me the same error.
[01:26:48] TypeError: dest.on is not a function
    at CompileStream.Readable.pipe (_stream_readable.js:501:8)
    at Object._worker (/home/roman/Documents/projects/base/project/client/src/gulp/package/compile.js:26:18)
    at Gulp.<anonymous> (/home/roman/Documents/projects/base/project/client/src/gulp/package/compile.js:12:31)
    at module.exports (/home/roman/Documents/projects/base/project/client/node_modules/orchestrator/lib/runTask.js:34:7)
    at Gulp.Orchestrator._runTask (/home/roman/Documents/projects/base/project/client/node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/home/roman/Documents/projects/base/project/client/node_modules/orchestrator/index.js:214:10)
    at Gulp.Orchestrator.start (/home/roman/Documents/projects/base/project/client/node_modules/orchestrator/index.js:134:8)
    at /usr/lib/node_modules/gulp/bin/gulp.js:129:20
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

@commute, that's the same mistake, you are calling gulpif wrong.
I'm on my phone, but it's explained how to correctly use gulp-if in the docs here, as well as on that plugin.

Whoops, wrong username. Sorry if I pulled in someone else. 😕

I'm also getting this error, although perhaps I'm also using lazypipe incorrectly?
Note that I'm using gulp-load-plugins to pull data/packages/etc from package.json
var buildJs = $.lazypipe()
	.pipe($.plumber, {errorHandler: plumberErrorHandler})
	.pipe($.newer, { dest: pkg.paths.dist.javascripts, ext: '.min.js' })
	.pipe($.sourcemaps.init)
	.pipe(function() {
		return $.if($.util.env.type === 'production', $.uglify, $.util.noop);
	})
	.pipe($.sourcemaps.write)
	.pipe(gulp.dest, pkg.paths.dist.javascripts)
	.pipe($.rename, { suffix: '.min' })
	.pipe(gulp.dest, pkg.paths.dist.javascripts)
	.pipe($.size, {showFiles: true});

gulp.task('buildJs', function() {

	return gulp.src(pkg.globs.javascripts)
		.pipe(buildJs())
		.pipe($.browserSync.active ? $.browserSync.reload({stream:true}) : $.util.noop());
});

If I remove the gulp-if line/function, it works fine...where am I going wrong on this?

This line is incorrect:
return $.if($.util.env.type === 'production', $.uglify, $.util.noop);

You aren't calling the uglify or noop methods, so no stream is getting built. Please read the docs more carefully.


Gene Erichment Analysis

[ ] Find geneset enrichment analysis (enrichR) to incorporate into our shiny app
[ ] Use the below genes in those tools to start writing code and functions to be used in our shiny app

BPIFA1
SCGB2A1
MUC5B
PRR4
BPIFB1
ZG16B
ANXA1
IRS1
APOA1
LTF
PIGR
PIP
NTC1
NTC2
PTEN
PIK3R1
PIK3CA

Check out https://github.com/nasqar/ClusterProfShinyGSEA

Gene enrichment analysis: http://bioinformatics.sdstate.edu/go/

Gene enrichment analysis: http://bioinformatics.sdstate.edu/go/ https://gencore.bio.nyu.edu/gene-set-enrichment-analysis-in-minutes-with-nasqar/

Awesome! We can take stuff from here: https://github.com/iDEP-SDSU/idep/tree/master/shinyapps/go


Suggestion: Own error messages

Hi! Right now, when I accidentally put a non existent string in the Play() function, I get this error message:
NullReferenceException: Object reference not set to an instance of an object CarterGames.Assets.AudioManager.AudioPlayer.SetClip (CarterGames.Assets.AudioManager.AudioData clip, CarterGames.Assets.AudioManager.AudioClipSettings clipSettings) (at Assets/Carter Games/Audio Manager/Code/Runtime/Audio/Player/AudioPlayer.cs:192)
While this is perfectly fine, it would be cool to see where exactly in my code I used an invalid sound name.

Thanks for reporting this. I'll have a look at this when I get a mo. Likely to be later this week.


Install Elasticsearch db engine in kubernetes without helm

Hi,
I installed Superset in a Kubernetes cluster without helm. I mounted the superset_config.py file in a volume under PYTHONPATH. I connected a postgres db instance successfully. Everything is fine.
Now I would like to connect an Elasticsearch instance. I tried to mount the requirements_local.txt with elasticsearch-dbapi inside, like README states, but it didn't work; when I try to connect with SQLALCHEMY URI and check the logs I see the message Error: could not load database driver superset. I tried to run pip install elasticsearch-dbapi inside the pod but same error appears.
Can u help me?

Given the provided answer, and the lack of follow-up for quite a while, I'll consider this one case-closed. Thanks to both of you! We can certainly revisit/reopen if this needs further discussion, of course.


options: default to core filters

Don't make people write pagespeed RewriteLevel CoreFilters in their config.  We should have filters be on by default, like in apache, and people should have to intentionally turn them off.

This was the wrong way to do this, and it's what's causing https://github.com/pagespeed/ngx_pagespeed/issues/1190
NewRewriteOptions is used in places where we want a clean RewriteOptions that doesn't specify anything, for example, NewRewriteOptionsForQuery.  Explicitly setting CoreFilters is the wrong way to make that the default.


add support for secp160r1

addition to PR on https://github.com/indutny/elliptic/pull/247

Seems like this is blocked on https://github.com/indutny/bn.js/pull/267


Configure ReportStream Production and  Staging to work with new OKTA Instances

Problem Statement
ReportStream is getting its own instance of OKTA for authentication and authorization. We need to set up and configure the production and preview instances and migrate our ReportStream groups and users from the HHS instance to our new instance.
Acceptance Critieria

[ ] Login/authentication/password reset workflows route through new Okta/Okta Preview instances

Other considerations

[ ] #3891 (Previous Issue for Okta Preview set up) - permissions cleanup

Tagged stale. @drew-usds Can you make sure this is refreshed as needed once we have direction on Okta changes?

Closing for now. Old issue. Not sure if it is scope anymore. New issue will arise naturally if it is still needed.


AuReiNAND: adding emuNAND option

For the part of configuring AuReiNand ( https://github.com/Plailect/Guide/wiki/Part-5-(arm9loaderhax-&-AuReiNAND)#configuring-aureinand ), does it is possible to add EmuNAND option (not only updated sysNAND)

No. Read this.


CB-11183 Appium tests: Added image verification

https://issues.apache.org/jira/browse/CB-11183

@riknoll  to help review

It would be nice to refactor the specs a bit, because they seem to be 90% shared code. This looks pretty good! Reading the file and checking the dimensions covers a lot of the bugs for camera.
Here are some other things that can be verified:

Make sure that PNG files and JPEG files are actually encoded/named correctly (check the headers). We've had bugs in the past where we return a JPEG file with a .png file name
Run specs with quality set to 100 in addition to 50. We often treat a quality of 100 as a special case on iOS and Android

It would also be nice to verify the type of URI we get back for the destination type. Unfortunately, that is horribly inconsistent right now on Android for NATIVE_URI and FILE_URI. It is possible to get back a file: URI, a content: URI, or a vanilla path depending on what camera options you pass. Can we try and verify it on iOS at least?
I think that covers most of the camera options that can be tested.

@riknoll I've updated the PR.

De-duplicated specs code
Implemented image type verification
Added a couple of specs with quality 100 (adding it to combined specs would double their number)
Implemented URI type verification for iOS

Awesome! LGTM other than the one comment

Cordova CI Build has completed successfully.
Dashboard - http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera/3/
Results

Windows 8.1 Store - Console Output - Test Results - Device Logs
Windows 10  Store - Console Output - Test Results - Device Logs
Windows 8.1 Phone - Console Output - Test Results - Device Logs
iOS - Console Output - Test Results - Build Artifacts
Android Mac - Console Output - Test Results - Build Artifacts
Android Windows - Console Output - Test Results - Device Logs

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android Mac
Link
Link
Link


Feature/byop

This allows a user to provide their own promise library; this is useful as there are libraries which offer benefits over the ES6 Promise object but are also API-compatible (or could be easily made to be so).  One particular benefit of using the same promise is that some libraries (notably q and bluebird) support long stack traces across multiple callbacks which is useful for debugging; however, when using a different promise in the middle (or particularly the end) of the chain breaks this functionality.
This is referred to in some libraries as Bring Your Own Promise (BYOP) and is very important to those of us who use third party promise libraries.  I have copied the promise unit tests to a byopromises unit test file which performs all of those tests with bluebird set as the Promise type and verifies that it works.

If you want to use the same Promise library everywhere, why not overwrite global.Promise?

@cesarandreu I could be wrong, but it feels like setting a particular promise library for co should be a separate implementation detail from overwriting the entire global ES6 promise (which is what this PR seems to address specifically). Thoughts?

I agree as well; while there may be times when overriding the global makes sense, generally I prefer to avoid overriding globals in order to prevent problems with minor implementation detail differences and modules or libraries which may not have specific unit tests to test alternate promise libraries, etc.  I added unit tests to this PR to make sure it would work with another Promise library, but other libraries which use ES6 promises may not have that.
I submit that overriding a global should not be the required way of doing something that was, in actuality, so simple to implement.  It does not make the code harder to read or to understand, it does not add a significant burden for code maintenance, and it does prevent the need to follow what I consider to be a dangerous practice -- overriding global built-in types.

@joshbeam valid point on the comment format; fixed.

This same basic pattern is used in mongoose, in case that helps you feel any more comfortable with it =]

@taxilian how is this different from https://github.com/tj/co/pull/226?

Interesting; I thought I'd looked through the pull requests and missed that one. I believe they are functionally equivalent; I guess the only real difference is in a few of the details of implementation and that I provided tests to go with this.
Given how long the other pull request has been waiting for review, do I take that to mean that I should give up and use his fork? That would probably work for me.  I don't care which I use, but I need the functionality.

I think the use case is valid and I'm experimenting with co-use right now. Considering that the original PR hasn't received any feedback from the maintainers yet, I'm not sure if it's going to be a welcomed feature or not.

I'm meh on it personally, I think having something like this is cleaner in a sense, but at the same time it would be such a massive pain to go set the promise lib to something new for every single package out there in your app. The polyfill route is reasonable in that sense I suppose. Ideally co doesn't even exist (with async/await etc) so I probably wouldn't bother

Need! :+1:

I like the way mongoose implements it: http://mongoosejs.com/docs/promises.html#plugging-in-your-own-promises-library

Forgot to close. I'm firm on not supporting arbitrary Promise implementations. Libs like bluebird et al should work with promises, not implement their own. If you need a polyfill, then use a polyfill :).


atw_vulkan: Invalid image blit offsets

In samples/apps/atw/atw_vulkan.c, srcOffsets[1].z and dstOffsets[1].z are being set to 0 on vkCmdBlitImage(). However, according to the spec:

If the calling command’s srcImage is of type VK_IMAGE_TYPE_1D or VK_IMAGE_TYPE_2D, then srcOffset[0].z must be 0 and srcOffset[1].z must be 1.
[...]
If the calling command’s dstImage is of type VK_IMAGE_TYPE_1D or VK_IMAGE_TYPE_2D, then dstOffset[0].z must be 0 and dstOffset[1].z must be 1.

Should be fixed in https://github.com/KhronosGroup/Vulkan-Samples/pull/84

Thanks for pointing this out. This should have been resolved a while back,


bio field too short, doesn't support markup

160 characters is very short for a bio (the text box under Edit Profile), and since you can't use any kind of mark-up in that box, even a clickable link to a proper bio is not possible. Could one or both of these restrictions be lifted?

I think you may be mistaking this repo as being for GitHub proper. We're actually a community-run GitHub Javascript client library and have no relationship with GitHub.

Oops, you're right, I'm in the wrong place. Apologies, my bad!

Not a problem! 😄


readme.md update

add usage

Thanks.
I create this repo as an example about how to create a flutter plugin.
I don't want this repo to be used in  productive environment.
If some want use this as a volume plugin, please fork and make your own publish


Provide some more locations for the FAB.

Top left and top right for big FABs, and top left for mini FABs.

cc @HansMuller


The function "search in reponse" is missing

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
The function press ctrl+f to search in response is missing. This function exists in the previous version.
.bru file to reproduce the bug
No response
Screenshots/Live demo link

It's not actually missing, it's just not shown 😕 ; I strongly believe it's related to https://github.com/usebruno/bruno/issues/2580

Duplicate of https://github.com/usebruno/bruno/issues/2580


Change validation to optional (mqtt)

Description:
Missed values from #4396.
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

See my comment here: https://github.com/home-assistant/home-assistant/pull/4381


Some fixes to make codem pipeline work

Description

Fix minor typo
In IO.check_exists add logic branch when item_value.stage not in upstream_stages
In _get_stage, make sure that customized stage have higher priority compare to built-in stages

Discussion
If user define a customized stage with the same with as one of the built-in stage, module property won't get created, and it is causing trouble with the CODEm pipeline PreprocessinStage. I suggest we always save the module path and load stage with module path. Let me know what you guys think

I suggest we always save the module path and load stage with module path. Let me know what you guys think

It seems unnecessary to save the module path if it just points back to the OneMod package, that's just my opinion.
I removed the preprocessing stage since I don't think it makes sense to have a "general" one in the package anyway.


add "back to top" links

Tables of contents are well written and structured the notebooks in a nice manner.
However, the content of the notebook would be more user-friendly if there would be "back to top" link beside the titles inside the notebook.
I've added "back-to-top" links to python notebook (Basics, Object Oriented Programming and Modular Programming)

Writing "Back to top" for each subtopic does not seem very nice. I think it is better to use the relevant icon instead of writing "back to top".

Writing "Back to top" for each subtopic does not seem very nice. I think it is better to use the relevant icon instead of writing "back to top".

Good idea! I will replace text with an icon.

Writing "Back to top" for each subtopic does not seem very nice. I think it is better to use the relevant icon instead of writing "back to top".

hi, again.
I've updated all notebooks and added ICONS for "back to top" links.
This is a screenshot from one of the titles of the advanced folder. The "blue top_arrow", if clicked, goes to the table of contents:

@sfarajnia
good job. I think you also feel it has a nicer and cleaner structure now.

After putting some thought into this, I think we should not do this manually.
You can simply write a code to add the link to a jupyter notebook file cause it's basically a json file.
You should add a cell at the end of it that contains a back to top link.

After putting some thought into this, I think we should not do this manually.

I don't get what your point is. Having a "back to top" link besides each heading would be nice-to-have (actually, I don't use them but someone may find it useful).

You can simply write a code to add the link to a jupyter notebook file cause it's basically a json file. You should add a cell at the end of it that contains a back to top link.

Also, I don't understand what you mean by mentioning ipynb files are just jsons.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 4 committers have signed the CLA.:white_check_mark: hejazizo:x: Ali:x: sfarajnia:x: MahtabRanjbarAli seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fs is not found

Bug report
What is the current behavior?
After running my command:
cross-env webpack --watch --mode development ./run/start.web.js

I get an error:
This dependency was not found:

* fs in ./node_modules/minio/dist/main/minio.js, ./node_modules/mkdirp/index.js

I tried:
node: {
  fs: 'empty'
}

It does remove the error and succesfully launches the client
However it looks like the site is unresponsive and some controls are missing, like there is no javascript applied.
If I remove usage of file which uses minio (from error). Everything works just fine.
Webpack config:
const Dir = global.DIR;
export function loader() {
  return {
    jsx: {
      use: {
        query: {
          presets: [['es2015', { modules: false }], 'stage-0', 'react'],
          plugins: [
            'transform-decorators-legacy',
            'transform-class-properties',
            'transform-runtime',
            'babel-root-import',
            'react-hot-loader/babel',
          ],
        },
      },
    },
    cssModules: {
      use: [
        'style-loader',
        {
          loader: 'css-loader',
          options: {
            localIdentName: '[name]__[local]___[hash:base64:5]',
            modules: true,
            sourceMap: true,
            importLoaders: 1,
          },
        },
        'postcss-loader',
      ],
    },
  };
}
export function config(entry) {
  return {
    devtool: 'cheap-module-eval-source-map',
    entry: {
      app: [
        'babel-polyfill',
        'isomorphic-fetch',
        'whatwg-fetch',
        'react-hot-loader/patch',
        'webpack-hot-middleware/client',
        // ['webpack-hot-middleware/client', webhost].join('?'),
        path.join(Dir.src, entry, 'client'),
      ],
    },
    output: {
      path: '/',
      publicPath: '/',
      filename: 'bundle.js',
    },
    plugins: [
      new FriendlyErrorsWebpackPlugin({
        clearConsole: true,
        compilationSuccessInfo: {
          messages: logServerConfigWebpack(entry),
        },
      }),
      new BrowserSyncPlugin({
        host: getenv('BROWSERSYNC_HOST'),
        port: getenv('BROWSERSYNC_PORT'),
        proxy: webhost(entry),
      }, { reload: false }),
      new webpack.HotModuleReplacementPlugin(),
      new webpack.NoEmitOnErrorsPlugin(),
    ],
  };
}

webpack version:  4.14
Node.js version: 8.91
Operating System: Ubuntu

What are you expecting mkdirp to do in a browser environment. This package probably don't support in browser usage.

This may be a duplicate issue, see https://github.com/webpack-contrib/css-loader/issues/447. You can change your source maps to an *eval* option or remove  sourceMap: true, from the style loader if you wish to generate external source maps

@Steffan-Ennis can you create minimum reproducible test repo?


Switch CI to Ubuntu

crystal spec seems to be broken on Alpine Linux, so switching our CI to Ubuntu.
Related: https://github.com/crystal-lang/crystal/issues/13390.

Closed in favor of #70.


FIX-#3305: Fix read_excel when usecols and index_cols parameters are provided

Signed-off-by: Anatoly Myachev<EMAIL_ADDRESS>
What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #3305
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@dchigarev let's merge


Make DioError catchable by implementing Exception instead of Error

dart.core.Error is intended to be used for program failures and shouldn't be caught. DioError extends Error which is odd because errors like timeouts are intended to be caught. Therefore DioError should implement Exception.
From the dart.core.Error documentation:
 * Error objects thrown in the case of a program failure.
 *
 * An `Error` object represents a program failure that the programmer
 * should have avoided.
 *
 * Examples include calling a function with invalid arguments,
 * or even with the wrong number of arguments,
 * or calling it at a time when it is not allowed.
 *
 * These are not errors that a caller should expect or catch -
 * if they occur, the program is erroneous,
 * and terminating the program may be the safest response.
 *
 * Since errors are not created to be caught,
 * there is no need for subclasses to distinguish the errors.

The favourite retry package follows this rule and doesn't allow catching non-Exception classes like DioError.
// form retry source
     try {
        return await fn();
      } on Exception catch (e) { // doesn't catch DioError

FWIW I just started using Dio (which looks great btw) and immediately ran into an error I couldn't catch (server returned 401 Unauthorized because I was missing a header, but the debugger halts and there's nothing I can do about it).
This PR looks like it would sove this issue easily and painlessly.

Ive been caught into this using version 3.0.7, just today


Copyrights & Licensing of MC Escher

Hello,
With all due respect to your excellent project, I feel you should also respect the excellent work of the artist whose work you reference in your image.

All works by M.C. Escher are copyrighted.
International Copyright laws protect all of the work of M.C. Escher, as well as his spoken and written words. Any reproduction of his work, including downloading, is prohibited without the express written permission of  The M.C. Escher Company, B.V.
Request for Reproduction of Works
If you want to use any of the work of M.C. Escher as illustration in a book, magazine, an advertisement campaign, brochure, or on the Internet, you must submit a request to our Copyright department Attn. Ms. Margareth Verbakel.

https://www.mcescher.com/licensing/
Cheers,
Dan

Thanks, request submitted

Dear Mr. Scholl,

Thank you for your email of the 11th of August.

Unfortunately, we are not able to grant our permission for usage of a modified work by M.C. Escher.

Reason is that no work by M.C. Escher may be altered, changed or manipulated in any way.

We trust that you will understand our point of view.

Sincerely,

Margareth Verbakel

Manager Copyrights

I've removed it.


Reset all column filter

Issue Type (delete the irrelevant ones)

[x] Question

Specs
v2.21.0
What browser?
chrome
Expected Behavior
I want to create button to reset all column filter, so the table data are not filtered.
Actual Behavior
On first time i click my button, it works (the data not filtered).
But when i try to select any filter, then click my button again, it doesn't work (the data still filtered since the column field filter is reset).
Steps to Reproduce the Problem
Please detail your steps here
My Table options :

                             <button data-toggle="tooltip" data-placement="top" title="Reset semua filter data dari table" @click="deFilterAll()"class="btn btn-primary btn-sm">                                Reset All Filter                             </button>

My Method function :
        deFilterAll: function() {           var vm = this;           var table   = vm.$refs.myTable;           table.columns.forEach((column, i) => {             if(column.filterOptions) {               table.$set(table.columnFilters, column.field, undefined);               table.$set(column.filterOptions, 'filterValue', undefined);               table.$emit('filter-changed', table.columnFilters);             }           });           table.$emit('filter-changed', table.columnFilters);         },
jsfiddle
none

If you put a ref attribute on the table, you can invoke the reset function, which is exposed from vue-good-table.
<vue-good-table     :columns="columnHeaders"     :rows="tableRows"     ref="myTable" />
Then when you want to clear all the filters, you can call:
this.$refs.myTable.reset
somewhere in a method in your script section.

closing.


Fixed flake8 E226 warning

Hm, what version of flake8 do you have and why isn't Jenkins complaining? It should be using the latest one (2.5.4) -- I don't see any problems locally either.

I upgraded from flake8 2.5.1 to 2.5.4, and now I do not get any errors on master.


Create vm enhancements

For example:
vm = VirtualMachine(name='test', template='Ubuntu-16.04-64bit', timeout=600)
try:
    vm.find()
    vm.destroy()
except NoObjectFound:
    pass

# Template being cloned as 4 vCPUs, 4096MB RAM, 1 vNIC on 'House Network',
# so it is cloned and updated with the settings below in one operation
vm.create(
    vcdriver_folder='Osiriums',
    cpu_count=6,
    memory_mb=8192,
    nics={
        'Network adapter 1' : {
            'network': 'Personnel Network (10.10.x.x)'
        },
        'Network adapter 2' : {
            'model': 'e1000e',
            'network': 'House Network Test'
        },
    }
)

TODO: Add tests

Closing. It's been open for a while now with insufficient interest to warrant adding the tests.


Cannot snap UE4 secondary window

Microsoft PowerToys version
55.2
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Prior to updating to the latest PowerToys version, secondary UE4 editor windows would snap to FancyZones; now FancyZones no longer works with secondary windows, only with the primary window.
✔ Expected Behavior
FancyZones should work with all UE4 windows.
❌ Actual Behavior
UE4 secondary windows do not snap to FancyZones.
Other Software
UE4 4.26

I'm having the same issue with NinjaTrader 8 windows, after the update.  I normally have to use the Win+[l, r, u, d arrows] to snap them because FancyZones doesn't pick them up when I drag holding shift, but now even that's not working.  Actually, it's only some of the windows that won't snap, anymore.  Some still snap, as before.

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


Recent iOS changes reverted

I believe https://github.com/alexcrichton/cmake-rs/pull/93 ended up causing https://github.com/alexcrichton/cmake-rs/issues/95 by accident so I've reverted that change while this is sorted through.
@simlay  would you be able to look into this?

Sure!

Yeah, but this break my build for iOS :(


disable case tiles when in landscape mode of form entry

Are we sure we want to take this approach to increasing the visible form entry area while in landscape mode? What about making persistent case tiles collapse-able and then auto-collapsing while in landscape mode?

@phillipm Not sure what you mean by "making persistent case tiles collapse-able and then auto-collapsing while in landscape mode" or how it's different from this. Can you explain?

@ctsims Can you weight in here on whether you think just outright hiding case tiles in landscape mode is a bad idea? Phillip was concerned that maybe that's a little aggressive, and suggested instead just making them configurable so users can hide them if they want to. But from looking at an app that uses persistent case tiles in landscape mode, it seemed to me like it probably makes things unusable enough to just never want them in that case.

Case tiles are still very custom, so I'm inclined to not make any drastic
UI changes based on them without thinking hard about it. Anyone using case
tiles already knows how it's affecting their screens/projects, so I kind of
don't want to be making big changes to their experience.
No strong feelings though. Most case tile users are locked in portrait mode.
On Tue, Nov 17, 2015 at 4:43 PM, Aliza Stone<EMAIL_ADDRESS>wrote:

@ctsims https://github.com/ctsims Can you weight in here on whether you
think just outright hiding case tiles in landscape mode is a bad idea?
Phillip was concerned that maybe that's a little aggressive, and suggested
instead just making them configurable so users can hide them if they want
to. But from looking at an app that uses persistent case tiles in landscape
mode, it seemed to me like it probably makes things unusable enough to just
never want them in that case.
—
Reply to this email directly or view it on GitHub
https://github.com/dimagi/commcare-odk/pull/782#issuecomment-157518328.

Ok fair, I think I'm gonna just close this for now then.


Test auto-adding new issues to Triage on our main board

Title says it all. Tries the same thing as #3288

Yes! It works. A bit slow but it works.
Excellent. Muhahah! Excellent!


Possible wrong parsing of HTML

Hi!
When using AngleSharp in one of my projects I noticed the following behavior.
A client is sending "bad" HTML to our system, which looks like this:

Hi!
Lorem ipsum blablabla...

When parsing this with AngleSharp, the
 elements are treated as a start tag which causes our program to process it incorrectly.
I made a small example program to illustrate this behavior:
static void Main(string[] args)
{
  var html = "Line 1
Line 2";
  var parser = new HtmlParser(new HtmlParserOptions(),
    BrowsingContext.New(new AngleSharp.Configuration()));
  var parsedDocument = parser.ParseDocument(html);

  Console.WriteLine("Original:");
  Console.WriteLine(html);
  Console.WriteLine("Parsed:");
  Console.WriteLine(parsedDocument.ToHtml());

  Console.ReadLine();
}

This prints:
Original:
Line 1
Line 2
Parsed:
Line 1
Line 2

I'd expect it to print:
Line 1
Line 2

As a final note, everything works as expected when we receive HTML tags that doesn't contain a backslash character.
Maybe we're doing something wrong - any input is appreciated.

This is all fine.

Please look into the representation of a real browser first.
AngleSharp does HTML5 parsing by the spec, and a tag br\ is not the same as the special tag br. Thus its handled like a generic tag, which is just opened here and auto-closed via the closing paragraph.
Hope that helps!
Remark: Its the fault of the client, not your or AngleSharp's fault. This client would also be interpreted "wrongly" (from the point of view that br was originally meant, otherwise its all fine, of course) in the browser.


Fix MSW in playwright

When running the e2e test the MWS is not setting up and working as expected.

https://github.com/skobak/fluffy-task/pull/15


iPhone 6,2 Non Jailbroken Boot Loop

Fresh iOS 10.2 Install, Jailbreaking with Yalu102 Beta 7, Constant reboot loop on phone. Cannot even get into the Springboard. Restored back to 10.2 with Saved SHSH2 Blobs and jailbroke with Beta 6 instead and working perfectly. Not sure what caused the issues as CrashReporter wasn't installed, but as no tweaks were installed i'm guessing it's code related.

Crashes are always saved, no need for CrashReporter. Closed to clean up issues, but I'm interested in any kernel panic. Also b7 has no significant changes that would break devices that work on b6.

Forgot to reply to this it seems. I upgraded the iPhone 5S to abets 7 again on a fresh restore of 10.2 (so glad I can write BootNonce lol), and same thing happened again. The issues seem to be a Kernel panic 3/4 of the way through the Jailbreak Process. Sometimes it succeeds, other times it's failing with a Kernel panic caused by an overflow I think. I'll reply back in morning with everything it says. Hope I can at least make it a bit more stable for people

Oh god I can't type. Meant to say Beta 7.