feat(badger-api): Generate query wrapper for fetching harvests

Update Badger DAO subgraph URL
Re-generate schemas
Add query to fetch harvests via Badger DAO

lgtm, we can merge this one if you want since its just the query up to you.

Cool, I'll merge it now and we can adjust the query in the future if needed


Add Imprint

This PR adds a optional imprint at the bottom (below copyright) of the email.

Codecov Report

Merging #73 into master will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   97.81%   97.94%   +0.13%
==========================================
  Files           3        3
  Lines        1053     1073      +20
==========================================
+ Hits         1030     1051      +21
+ Misses         16       15       -1
  Partials        7        7

Impacted Files
Coverage Δ

hermes.go
65.00% <ø> (-3.19%)
:arrow_down:

default.go
100.00% <100.00%> (ø)

flat.go
99.80% <100.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

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


Divs created by react_component modify my UI

Hello,
I would like to know if we can apply css or replace div by a React.Fragment / span on div created by react_component ?
Thanks

More details are needed for your question. How does this matter for React on Rails?


[TESTS] Integration test for filters RPC endpoints

Currently, we are missing some integration tests for the filtering endpoints (filter.rs, logs.rs) that tests properly with block generation, we should add those, an idea here is to leverage hive, also it would be nice to standardize integration tests that use multiple parts of the project (eg: an RPC endpoint that waits for storage changes or something like that).

to fix your trouble check this solution click
maybe this will solve your problem.


Update test action based on latest cookiecutter and tweak tests

Tried to sync the test section of the GitHub action based off the napari/cookiecutter.
Dropped python 3.7, added python 3.10 and ubuntu.
Let's see how it goes!

Codecov Report

Merging #35 (5086798) into main (cb8424d) will decrease coverage by 0.75%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   33.18%   32.43%   -0.76%
==========================================
  Files           3        2       -1
  Lines         229      222       -7
==========================================
- Hits           76       72       -4
+ Misses        153      150       -3

Impacted Files
Coverage Δ

cellpose_napari/__init__.py

cellpose_napari/_dock_widget.py
21.69% <0.00%> (+0.11%)
:arrow_up:

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

For ubuntu, seems like I've hit the same thing as: https://github.com/MouseLand/cellpose-napari/pull/15
🤷♂

It's bothering me that this cellpose plugin
https://github.com/guiwitz/napari-serialcellpose/actions/runs/2858166269
has ubuntu passing and macOS stalling out, while here it's reversed.
😡

sorry for leaving you hanging here @psobolewskiPhD ... it's late here but I'll try to have a look tomorrow

No worries! Just venting—late here too 🤣
It's probably not important, can drop the ubuntu tests as before—the actual tests do pass.

Progress. Actual tests, including segmentation running, just windows 3.10 fails, due to numpy error?

all ✅
🎉
@tlambert03 I can't say I fully understand but the ubuntu failure is due to some library importing order of shared libraries between napari and cellpose.
See https://github.com/pytorch/pytorch/issues/75912
Importing torch at the top of the tests fixes it.
Then I had to drop Windows python 3.10, because it's not actually supported by pytorch yet:
https://pytorch.org/get-started/locally/#windows-python

amazing work @psobolewskiPhD ... way to be persistent! :)
only two small questions then LGTM

Assuming this is merged at some point, is there a way to re-run the tests in the other PRs?
Or do they need something? I think I've seen something like "pushed main" in various places—is that how that's done?

So just the diameter test passes... just like just the segmentation...
Maybe it's related to the fixture?

ah, so either in isolation works, but not together? ... if so, must be a side effect.  can you reproduce locally?

ah, so either in isolation works, but not together? ... if so, must be a side effect. can you reproduce locally?

Not sure I follow, locally everything passes no matter the order or combo of tests.

Not sure I follow, locally everything passes no matter the order or combo of tests.

sorry, when you said

So just the diameter test passes... just like just the segmentation...

I assumed it meant that if you commented out either one, it worked, but when they both ran it failed?
In any case, if you want to go back to around 4ad001c and simply skip the compute_diameter tests on ubutntu  with something like @pytest.mark.skipif(sys.platform.startswith('linux'), reason="linux sometimes makes me sad"), that would be fine with me! :)

Assuming this is merged at some point, is there a way to re-run the tests in the other PRs?

a quick-and-dirty way to force rerun tests anywhere, even if you don't have rights, is to just close the PR and reopen it immediately.  Alternatively, you can merge in main locally, then repush.  or, in some repos, they will have enabled a setting that shows a little button on the PR to let you automerge in the main branch

I assumed it meant that if you commented out either one, it worked, but when they both ran it failed?

Yes, this is exactly the case on ubuntu. Either test alone is fine. So it's something related to torch I guess.
Here's the live log of the currently stalled test:

The first test passes, the 2nd stalls out.

Either test alone is fine. So it's something related to torch I guess. Here's the live log of the currently stalled test:

it could be, or it could be some OS-dependent quirk of anything else going on here.  I'd say just skip it for now.  between the test you improved, and running it on all the other platforms, it's already a strict improvement. :)


Update concept-deployments.md

@jl-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@jl-ms: I'm sorry - only the author of this article, @duongau, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Cannot Install Powertoys 0.20

ℹ Computer information

Windows build number: [run "winver"]
PowerToys version:
PowerToy module:

📝 Provide detailed reproduction steps (if any)

…download powertoys 0.20 from github
…press install
…uninstalls previous version and halts ... see screenshot

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots:

Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Running into a similar issue, except I don't get the pop up. I can see the PowerToysSetup process in Task Manager but nothing happens. It also prevents me from reinstalling version 0.20 until io kill that preocess.

I cannot install it either. It hangs at 99%.

My notification disappears at around 60% and then nothing.. windows event log shows nothing.

Killing dotnet.exe in taskmanager when installing completes the installation.

@icyhoty2k
do you get the same error installing 0.19.2? https://github.com/microsoft/PowerToys/releases/tag/v0.19.2


[RFC] Allow DiffPairs and Pins to be inverted individually

Currently, DiffPairs and Pins can be either inverted completely or not at all.
This is a problem in corner cases where the hardware inverts some parts of a logical Bundle of ios (for example to ease routing in case of differential pairs; see #386).
I hereby propose to extend the Platform DSL in the following way:

Add a invert= kwarg to the constructor of DiffPairs
Allow Tuples of booleans of the width of the pin list to be passed to the invert= kwarg of both DiffPairs and Pins. Each bool in the tuple then controls the inversion of one pin / differential pair.

Right, this is the obvious solution. The main problem I have is that when you have a lot of diffpairs, it can be hard to correlate the booleans in the invert= tuple with the actual diffpairs.

As an alternative I was wondering if something like DiffPairs("A1 !A2", "B1 !B2") would work. The ! sigils would then have to match in both the P and the N pin lists.

I like putting it in the pin list string(s), it's compact but I think also quite clear. Maybe "~A1" would be a better match to the nmigen negation syntax?

Sure, ~ is fine by me.

I also really like that idea :)

Sounds great. I'll bring this up on the next meeting but I think you can start implementing it as it seems unlikely anyone's going to be opposed.

sounds good :)

I think we should support similar syntax for the use with connectors (which can be routed through e.g. inverted level shifters or have crossed differential pairs). So I think it would be nice to have syntax elements in connectors that can say "invert this signal if it is a diff pair" or "invert this signal always".

I would like to avoid introducing that functionality (and the associated syntactic burden) until someone actually needs it.

/me needs the latter (inverted differential pairs in connectors)

Okay. I still think it should be done separately because while this one is fairly straightforward (it's just a way to write a tuple that's less error prone than usual), what you request with connectors requires completely rethinking the way we do connectors.

Okay I see that.
So I will implement this RFC as-is and go with out of tree hacks for now.
If I really need this feature to become upstream I will write another RFC :)

I think we can do it upstream; I said earlier that I'd like to avoid introducing it because it sounded theoretical (which is usually bad because features done in abstract tend to not survive collision with reality). If you actually hit that problem I think we can certainly add it. Let's discuss the connector semantics on the next meeting?

Okay :+1:

We discussed inversion of connector signals at the last meeting. Because inversion of single ended signals is rarely needed and can also complicate things even further (you have to keep track of two layers of inversion rather than one), we decided that we would implement inversion in connectors but only for diff pairs. Connecting a single-ended signal to an inverted connector pin would be a hard error. If it turns out to be worth the complexity in the future we can always accept it.


JSON support

Add support to save translations into JSON

To save as json file, just install https://github.com/php-gettext/Json


fix(device_info_plus): fix type casting for WASM

Description
Fix some types casting on web implementation for WASM compilation
Related Issues

FIX #2966

Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I titled the PR using Conventional Commits.
[x] I did not modify the CHANGELOG.md nor the plugin version in pubspec.yaml files.
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.

FIX https://github.com/fluttercommunity/plus_plugins/issues/2966

The mentioned issue is about package_info_plus, while your changes are for device_info_plus.
Could you explain why you did the change in this PR? device_info_plus supports a range of web versions https://github.com/fluttercommunity/plus_plugins/blob/main/packages/device_info_plus/device_info_plus/pubspec.yaml#L40 and works well with both web 0.5.1 and 1.0.0 at the moment.

I'm sorry,I got confused, I did notice the PR was for package_info_plus:
Still that the package device_info_plus, the one I'm fixing in this PR, has a similar problem when used in web compiled in WASM.

The type of deviceMemory came out as a double on WASM
The languages field it's a JSArray<JSString> so it needs to pass through toDart

Still that the package device_info_plus, the one I'm fixing in this PR, has a similar problem when used in web compiled in WASM.

I would ask you to open the issue first and fill in all the required data, because from just this statement it is not clear in which environment, which versions you used etc. As I mentioned we check WASM compilation as well and everything worked fine so far. So if you found an issue, it would be better to report is properly first, so we could have a constructive discussion about this PR.

The changes in this PR are the same as https://github.com/fluttercommunity/plus_plugins/pull/3254
Let's follow up on that other PR, I will close this one.
In all cases, thanks for the contribution @GiacomoPignoni !


Why it is showing undefined  error  with var c ?  plz  help

Challenge Understanding Uninitialized Variables has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.112 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Initialize these three variables
var a = 5;
var b= 10;
var c = " I am a";

// Do not change code below this line

a = a + 1;
b = b + 5;
c = c + " String!";

@krishgarg Thank you for reporting this issue. Your code is incorrect. Hint: check your spacing.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on Free Code Camp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


test: add unit tests to JobMixin and SquashAndMerge

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

78 of 78 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 86.421%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
7809

Relevant Lines:
9036

💛  - Coveralls


upgrade faker (fixes #82)

@lfolco Why did you close this? Looks like it works with Magento 2.4, I'll happily drop the Magento 2.3 compatibility for the next release

That's why I closed it :)


maya 0.5.1 with StoragePool CR named as default does not obey the path setting

Refer:

http://openebs.readthedocs.io/en/latest/Policies/storage_policy.html#storage-pool-policy
http://openebs.readthedocs.io/en/latest/Policies/storage_policy.html#storage-pool-policy

apiVersion: openebs.io/v1alpha1
kind: StoragePool
metadata:
  name: default
  type: hostdir
spec:
  path: "/var/openebs"
  nodes: [ "all" ]

When above's .spec.path's value is change to anything other than /var/openebs, it does not work.
What I expect is maya api service should provision the volumes that makes use of the path set in the StoragePool CR named with default.

Adding sp, sc & pvc snippets (-o yaml o/ps) from one of the users (Rafael) :
StoragePool :
apiVersion: v1
items:
- apiVersion: openebs.io/v1alpha1
  kind: StoragePool
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"openebs.io/v1alpha1","kind":"StoragePool","metadata":{"annotations":{},"name":"default","namespace":"","type":"hostdir"},"spec":{"nodes":["all"],"path":"/mnt/ebs"}}
    clusterName: ""
    creationTimestamp: 2018-01-31T10:45:49Z
    name: default
    namespace: ""
    resourceVersion: "211415"
    selfLink: /apis/openebs.io/v1alpha1/default
    type: hostdir
    uid: e68aaf6f-0673-11e8-a61d-0aae8da7e752
  spec:
    nodes:
    - all
    path: /mnt/ebs
kind: List
metadata: {}
resourceVersion: ""
selfLink: ""

StorageClass :
apiVersion: v1
items:
- apiVersion: storage.k8s.io/v1
  kind: StorageClass
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"storage.k8s.io/v1","kind":"StorageClass","metadata":{"annotations":{},"name":"openebs-standalone","namespace":""},"parameters":{"openebs.io/capacity":"4G","openebs.io/jiva-replica-count":"2","openebs.io/storage-pool":"default","openebs.io/volume-monitor":"true"},"provisioner":"openebs.io/provisioner-iscsi"}
    creationTimestamp: 2018-01-31T10:53:13Z
    name: openebs-standalone
    namespace: ""
    resourceVersion: "211954"
    selfLink: /apis/storage.k8s.io/v1/storageclasses/openebs-standalone
    uid: ef0a2474-0674-11e8-a61d-0aae8da7e752
  parameters:
    openebs.io/capacity: 4G
    openebs.io/jiva-replica-count: "2"
    openebs.io/storage-pool: default
    openebs.io/volume-monitor: "true"
  provisioner: openebs.io/provisioner-iscsi
  reclaimPolicy: Delete
kind: List
metadata: {}
resourceVersion: ""
selfLink: ""

*PVC: *
ind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: demo-vol1-claim
spec:
#  storageClassName: openebs-percona
  storageClassName: openebs-standalone
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 2G

PVC folder created on /var/openebs :
root@k8s01:/var/openebs# ll
total 12
drwxr-xr-x  3 root root 4096 ene 31 12:05 ./
drwxr-xr-x 15 root root 4096 ene 31 12:05 ../
drwxr-xr-x  2 root root 4096 ene 31 12:06 pvc-ac53af5f-0676-11e8-a61d-0aae8da7e752/

*kubectl get pods: *
root@k8s01:/var/openebs# kubectl get pods
NAME                                                            READY     STATUS    RESTARTS   AGE
maya-apiserver-798f5d6887-p4hgl                                 1/1       Running   0          1h
mysql-master-f4668fccb-spck7                                    1/1       Running   0          1h
openebs-provisioner-74f87ccd6-wc9gg                             1/1       Running   0          1h
pvc-ac53af5f-0676-11e8-a61d-0aae8da7e752-ctrl-6456f776c-flpmj   2/2       Running   0          1h
pvc-ac53af5f-0676-11e8-a61d-0aae8da7e752-rep-6f4d78d6bc-2clbx   1/1       Running   0          1h
pvc-ac53af5f-0676-11e8-a61d-0aae8da7e752-rep-6f4d78d6bc-md89m   1/1       Running   0          1h


Program stuck at 'pad_across_processes'

System Info
Copy-and-paste the text below in your GitHub issue

- `Accelerate` version: 0.14.0
- Platform: Linux-4.15.0-194-generic-x86_64-with-debian-buster-sid
- Python version: 3.7.3
- Numpy version: 1.21.6
- PyTorch version (GPU?): 1.9.1+cu111 (True)
- `Accelerate` default config:
        - compute_environment: LOCAL_MACHINE
        - distributed_type: DEEPSPEED
        - mixed_precision: no
        - use_cpu: False
        - num_processes: 4
        - machine_rank: 0
        - num_machines: 1
        - gpu_ids: None
        - main_process_ip: None
        - main_process_port: None
        - rdzv_backend: static
        - same_network: True
        - main_training_function: main
        - deepspeed_config: {'gradient_accumulation_steps': 1, 'offload_optimizer_device': 'cpu', 'offload_param_device': 'cpu', 'zero3_init_flag': True, 'zero3_save_16bit_model': False, 'zero_stage': 3}
        - fsdp_config: {}
        - megatron_lm_config: {}
        - downcast_bf16: no
        - tpu_name: None
        - tpu_zone: None
        - command_file: None
        - commands: None

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction

The code at here

Can replace the datasets with own small datasets for debugging.

3.Model just stuck at line674 and finally said time limit error.
Expected behavior
Can run successfully but program stuck at pad_across_devices

Hello @ZeyiLiao, when using DeepSpeed Zero Stage-3, the generate() call in eval loop hangs because it is missing synced_gpus=True argument. Please refer  issue https://github.com/huggingface/transformers/issues/20130 which details what's happening and how to resolve it.

@pacman100 Gotcha! Reaaly helps!!
And would you mind checking why the program can not exit correctly with --report_to wandb --with_tracking?
And I just simply modify the code like this:
    if args.with_tracking:
        if accelerator.is_main_process:
            experiment_config = vars(args)
            # TensorBoard cannot log Enums, need the raw value
            experiment_config["lr_scheduler_type"] = experiment_config["lr_scheduler_type"].value
            tmp = {"entity":"zeyiliao","name":str(args.output_dir).replace("./","")}
            accelerator.init_trackers("t5-finetune", experiment_config,init_kwargs={"wandb":tmp})

error looks like
Configuration saved in ./t5-large-79a06195-d1b9-4b5a-8b7f-b2c0181fced8/config.json
Model weights saved in ./t5-large-79a06195-d1b9-4b5a-8b7f-b2c0181fced8/pytorch_model.bin
tokenizer config file saved in ./t5-large-79a06195-d1b9-4b5a-8b7f-b2c0181fced8/tokenizer_config.json
Special tokens file saved in ./t5-large-79a06195-d1b9-4b5a-8b7f-b2c0181fced8/special_tokens_map.json
Copy vocab file to ./t5-large-79a06195-d1b9-4b5a-8b7f-b2c0181fced8/spiece.model
100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 11/11 [01:21<00:00,  7.44s/it]
Exception in thread SockSrvRdThr:
Traceback (most recent call last):
  File "/home/zeyi/miniconda3/envs/longtail/lib/python3.7/threading.py", line 917, in _bootstrap_inner
    self.run()
  File "/home/zeyi/miniconda3/envs/longtail/lib/python3.7/site-packages/wandb/sdk/service/server_sock.py", line 112, in run
    shandler(sreq)
  File "/home/zeyi/miniconda3/envs/longtail/lib/python3.7/site-packages/wandb/sdk/service/server_sock.py", line 173, in server_record_publish
    iface = self._mux.get_stream(stream_id).interface
  File "/home/zeyi/miniconda3/envs/longtail/lib/python3.7/site-packages/wandb/sdk/service/streams.py", line 199, in get_stream
    stream = self._streams[stream_id]
KeyError: 'cs1ulxr6'

All the checkpoints can save correctly and wandb can also log correctly, just the code can not exit correcly.

Hello @ZeyiLiao, this is different issue, anyways, I don't know the root cause for this but to resolve it just remove the following lines
- if args.with_tracking:
-        accelerator.end_training()

If this is resolved, can you please go ahead and close the issue?

Sure!


Is there some key metric for ardb, can be monitor?

for example, the time of db merge on disk , etc

that depends on the underlying key/value storage engine. if u using rocksdb as the storage engine ,there would be some log info produced by rocksdb which related with some background  operations done by  rocksdb itself.


Typos on homepage

Describe the bug
Some typo correction suggestions:
Current: "We do accept extended talks but prefer presentors to have done a lightning talk before. If you're unsure please email us to discuss your talk."
Suggestion: "We do accept extended talks, but prefer presentors to have done a lightning talk before. If you're unsure, please email us to discuss your talk.
Comma between "talks" & "but".
Comma between "unsure" and "please".
Current: "presentors"
Suggestion: "presenters"
Screenshots
If applicable, add screenshots to help explain your problem.

Working on it

Thank you for your contribution to San Diego JS.

Thanks for helping out, we have incorporated your changes into another pull request.


Let Viewer to take imageryProvider or terrainProvider with BaseLayerPicker

We've had a check in Viewer forever to see if the base layer picker was being used when a terrainProvider or imageryProvider was specified and throw if so.  This lead to a ton of code where we work around that error by creating the viewer and then specifying viewer.terrainProvider anyway:
var viewer = new Cesium.Viewer('cesiumContainer');
viewer.terrainProvider = new Cesium.CesiumTerrainProvider({
    url: 'https://assets.agi.com/stk-terrain/v1/tilesets/world/tiles'
})

Basically, this check was pointless and just made the code more verbose.  The check also prevented devs from specifying a terrainProvider but still using BaseLayerPicker for imagery (and vice versa). Now we just let the user do what they want, which was probably the right call from the beginning:
var viewer = new Cesium.Viewer('cesiumContainer', {
    terrainProvider: new Cesium.CesiumTerrainProvider({
        url: 'https://assets.agi.com/stk-terrain/v1/tilesets/world/tiles'
    })
});

This is a really small change in Viewer.js and then a bunch of cleanup in Sandcastle examples to match.  This will also allow us to clean up some website examples which are using CesiumWidget instead of Viewer for the above reason.

@mramato, thanks for the pull request! Maintainers, we have a signed CLA from @mramato, so you can review this at any time.
:warning: I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

@hpinkos can you review this please?

This didn't work for me.  The terrain doesn't load in for any of the examples you changed.  Maybe it's being overwritten by the EllipsoidTerrainProvider from the base layer picker?

I think I made a last minute change that broke it, looking at it now.

@hpinkos this is ready
The longer I looked at this the more annoyed I got at Viewer, it's doing way too much and is overdue for a clean-up.  For now I just took the easy route and delay assigning the imageryProvider or terrainProvider until after the BaseLayerPicker is initialized (if at all).

Okay, this works now.  I find it a little weird that the base layer picker still shows bing maps and ellipsoid terrain selected when you pass in an option though.  Can we have selectedImagery and selectedTerrain be undefined for this case?

ready

Thanks @mramato!


Nested files in VS 2015 show false TypeScript compilation errors in IDE

We just upgraded to from VS 2013 Ultimate with TypeScript 1.6 to VS 2015 Enterprise with TypeScript 1.6 and now we are seeing an issue with nested files in our solution where the IDE is reporting errors despite the project building without errors.
We use nested files (using Mads Kristensen's File Nesting VS extension v2.2.31) to nest our test specs under the related .ts file to test. For example:
Test.ts
L Test.spec.ts

When Test.spec.ts is nested under Test.ts and you open it in the VS IDE, errors are shown as follows:

Un-nesting the file resolves the problem and the TypeScript tooling is able to resolve the describe(...) function and errors are no longer displayed in the IDE.
A repro project can be found here:
https://github.com/rwhepburn/TsNestingIssue
Thanks,
Richard

I have no ideas how that extension performs the nesting, but if it is restructuring the way items are represented in the project somehow, then it likely breaks how TypeScript infers the project contents.
@madskristensen Any insight you can provide here?

@paulvanbrenk

Nothing in the project file immediately stands out as problematic, I would expect this to just work. I don't think this a problem with the 'nesting extension'.

Same problem here, it was working fine in VS2013.

Found the issue, we assumed TypeScript files would never have children... have a fix which I'll check in today, which may make it into 1.8.


Promote good possible references in the UI

When we display possible references to add, we should maybe display first the ones from more trustfully websites.
Asked here: https://www.wikidata.org/wiki/Wikidata_talk:Primary_sources_tool#Blacklisting_versus_whitelisting.3F

This has been archived on the 2015 talk page.

I have started a whitelist. Who should fill it?

The editors. You can promote it in the next weekly summary for example: https://www.wikidata.org/wiki/Wikidata:Status_updates/Next

Closing for the same reasons and rationale as https://github.com/google/primarysources/issues/106. Happy to merge Pull Requests if someone works on a visual makeover.

Re-opening and tagging as 'open task'.

In order to ease us (the Wikidata dev team) to track the issues of the primary sources tool, we have created a Phabricator ticket for this issue: https://phabricator.wikimedia.org/T148178
Moving forward, please drop your comments onto that Phabricator ticket.


'model.getLinesToParent is not a function' error in HTML mode on a specific file

There is a specific file that fails to render in HTML mode.  To repro, just visit the following file:
http://shalynn.pencilcode.net/edit/shalynntaboutme.html
<!DOCTYPE html>

<title>About Shalynn</title>
<link rel="stylesheet" href="shalynntstylesheet.css">

About Shalynn

Home

Thing You Might Want To Know About Me

I am fifteen years old.

I love country, hip hop, and Rap.

My favorite color is pink and purple.

I have two tattoos and a nose piercing.

My favorite animal is a cat and a dog.

When loading in HTML mode, we get the following exception:
editor.js:this.model.getLinesToParent is not a function:formatted:84642
Uncaught TypeError: this.model.getLinesToParent is not a function
114.c.View.a.computeDimensions @ editor.js:formatted:84642
114.c.View.a.computeDimensions @ editor.js:formatted:84666
114.c.View.a.computeDimensions @ editor.js:formatted:84666
114.c.View.a.computeDimensions @ editor.js:formatted:84666
114.c.View.a.computeDimensions @ editor.js:formatted:84666
114.c.View.a.computeDimensions @ editor.js:formatted:84666
114.c.View.b.layout @ editor.js:formatted:85024
100.r.redrawMain @ editor.js:formatted:75586
100.r.setTopNubbyStyle @ editor.js:formatted:75510
100.r.resizeNubby @ editor.js:formatted:75514
100.c.Editor.a.resizeBlockMode @ editor.js:formatted:75
453(anonymous function) @ editor.js:formatted:75344

This happens because of trailing spaces on the lines with  and ; these are not included in any block, and Droplet doesn't know how to render text directy inside Indents. In this cases, it might be good to include the spaces in the  or  block.

I've done what I felt was necessary. Let me know if I need to change anything.

The fix has been pushed!


Past twofold in .ipynb

Issue Type: Bug
File extension .ipynb
When I copy and paste code in a .ipynb, I get the past twofold.
Example:
Copy: "one"
Paste result: "oneone"
Is it a bug or a setting option.
Thanks
Extension version: 2020.6.91350
VS Code version: Code 1.47.0 (d5e9aa0227e057a60c82568bf31c04730dc15dcd, 2020-07-09T08:02:06.629Z)
OS version: Windows_NT x64 10.0.14393

Dupe of https://github.com/microsoft/vscode-python/issues/12808


Use avatar for IRC user

Hello,
When user is only on IRC and not at all on discord they don't have avatar.
Maybe we should use "http://avatars.adorable.io/" for this case ?

All users on the IRC side that aren't in Discord are automatically assigned the webhook avatar.

It can be customized

Yes, that's what I'm telling you.

And the ability to use other service with configuration (https://robohash.org)

Last line of getDiscordAvatar :
return "https://robohash.org/" + nick + "?set=set4"
instead of :
return null;

If anyone wants to give this a go: slack-irc—which this is based on—does exactly this, through a configurable avatarURL parameter. It defaults to Adorable Avatars, but RoboHash would probably be a good choice too 👍


Daemonset checker detects pods that don't clean up after the daemonset is deleted

Kuberhealthy routinely makes a daemonset, then, once all the pods in the daemonset are in the 'Ready' state, Kuberhealthy deletes the daemonset and waits for all the pods to clean up.
It should be impossible for the daemonset to be deleted, but its pods to stay behind.  However, this is exactly what occurs.  The daemonset finishes removal but several daemonset checker pods are remaining.
Steps To Reproduce

Run kuberhealthy for a long time
Watch daemonset checker alerts

Expected behavior
The daemonset checker pods are always cleaned up and a daemonset is never deleted before all of its pods are deleted.
Screenshots

We should probably do the following:

make sure that the api call to delete pods is not missing an option which would cause the daemonset checker to leave pods around
ensure that the shutdownGracePeriod on the daemonset test pods is set extremely low
file a bug with Kubernetes for a race condition in daemonset deletion

I took a look:

There is no applicable option that would cause pods to be left (though you can intentionally tell it to orphan pods which we obviously dont want)
shutdownGracePeriod is 1s (https://github.com/Comcast/kuberhealthy/blob/e6fab00968adefde1a58dd608b6d8acc905a49a2/pkg/checks/daemonSet/daemonSet.go#L46)

We think there is a race condition around creating a daemon set and immediately deleting it

I believe this line may be the cause of the race condition: https://github.com/Comcast/kuberhealthy/blob/e6fab00968adefde1a58dd608b6d8acc905a49a2/pkg/checks/daemonSet/daemonSet.go#L594
There is no spec.minReadySeconds defined in the DS spec, and once a DS is identified as healthy we immediately try to delete the pods that may have have only just been created.  Additionally, this may be a real issue with certain hosts having trouble deleting pods in time.


[HERMES-3842] Add npm build action

Summary
Adds a git action that runs the build_npm.ts script on PRs and commits to main.
The Npm Build step this adds should fail currently, this is expected. A separate PR will fix the issues causing the action failure.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

We can close this PR has #107 supersedes this, right @srajiang ?

@filmaj Yup!


using __builtin_memcpy_inline from LLVM in few places.

Differences with __builtin_memcpy are it does not generate
extra calls and it can work only with compile time sizes.

Interesting addition to LLVM, thanks!


Upgrade addon-framework to 0.9.3

@xuezhaojun What commit needs to be included here? We need the WithManagedClusterClient() function, so technically the version we've tagged here diverged from v0.9.3. Would it be sufficient to tag the latest commit from main to pick up the commit?

Hi, @dhaiducek  No worries, we are working on cherry-pick WithManageClusterClient into 0.9 branch in this PR: https://github.com/open-cluster-management-io/addon-framework/pull/272.


files.upload fail

when i use files.upload
It throw a error
TypeError: <open file 'test.png', mode 'rb' at 0x7fda02706780> is not JSON serializable
So I change the code
--- a/slackclient/_slackrequest.py
+++ b/slackclient/_slackrequest.py
@@ -21,12 +21,14 @@ class SlackRequest(object):
         '''
         post_data = post_data or {}

+        files = {'file': post_data.pop('file')} if 'file' in post_data else None
         for k, v in six.iteritems(post_data):
             if not isinstance(v, six.string_types):
                 post_data[k] = json.dumps(v)

         url = 'https://{0}/api/{1}'.format(domain, request)
         post_data['token'] = token
-        files = {'file': post_data.pop('file')} if 'file' in post_data else None

         return requests.post(url, data=post_data, files=files)

And it works!!

I have a similar issue but i was not able to fix it by moving the single line suggested above.
My error is also slightly different:
TypeError: <_io.BufferedReader name='test.png'> is not JSON serializable
Any ideas?

I actually just got it to work. I had to change more than i expected in the SlackRequest class:
`         post_data = post_data or {}

   files = {'file': post_data.pop('files').pop('file')} if 'files' in post_data else None
   for k, v in six.iteritems(post_data):
       if not isinstance(v, six.string_types):
           post_data[k] = json.dumps(v)

   url = 'https://{0}/api/{1}'.format(domain, request)
   post_data['token'] = token

   files = {'file': post_data.pop('file')} if 'file' in post_data else None

   return requests.post(url, data=post_data, files=files)`

+        files = post_data.pop('files', None)
 +
          for k, v in six.iteritems(post_data):
              if not isinstance(v, six.string_types):
                  post_data[k] = json.dumps(v)

          url = 'https://{0}/api/{1}'.format(domain, request)
          post_data['token'] = token
 -        files = {'file': post_data.pop('file')} if 'file' in post_data else None

Can't we just pop out files?

The files argument needs to be a dictionary, with a file key and the open file as the value. @ziemlak's change fixes things for me (and is very much appreciated.)

Right, but it only works if you already send it with that format.

That's the format which the requests packages uses, and which appears to match the requirements listed in the files.upload documentation. Are you proposing that it map two keys (files and file) to the single call, or to defer building the dictionary?

It doesn't look like this (files = {'file': post_data.pop('files').pop('file')} if 'files' in post_data else None) was merged to master
https://github.com/slackhq/python-slackclient/blob/master/slackclient/_slackrequest.py


Add yearly comparison

Codecov Report
Patch coverage: 83.78% and project coverage change: -1.21 :warning:

Comparison is base (994661d) 98.55% compared to head (b0b30b9) 97.35%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
- Coverage   98.55%   97.35%   -1.21%
==========================================
  Files           8        9       +1
  Lines         416      453      +37
==========================================
+ Hits          410      441      +31
- Misses          6       12       +6

Impacted Files
Coverage Δ

src/helpers.jl
93.18% <70.00%> (-6.82%)
:arrow_down:

src/plots_history_distribution.jl
100.00% <100.00%> (ø)

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

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


luci-app-tor: Manage onion services

There was a PR #711 but it was focused on the Proxy configuration which is not fully yet supported by UCI.
The PR adds an ability to configure Tor Hidden Services as a good option to bypass NAT and have a stable access to a router.

Later once the main Tor package gains a support of a Proxy/Bridge configuration we can add this to the same luci app as a different view.
If you accept the PR please also check this small improvement https://github.com/openwrt/packages/pull/21642
@ja-pa you created the tor-hs package so please check and leave your feedback

There is a Name fields that is later used by the tor-hs package when a keys folder created for a service. Ideally instead it should be a uci section name used.
Anyway we can't update existing configuration where sections doesn't have a name.
Still I think that the section name must be set clearly.
So the PR has named hidden-service sections and when adding a new the Name field will be populated with a name of section. Users are free to change it if they need.
Existing unnamed sections will be shown as cfg1234 but their Name field remain intact and even can be edited.
Luci doesn't provide an option to edit section names but that's not critical.

That's a good idea. Still I feel that it would be better to keep as is for
two reasons:

We may have a translated Wiki page
This is more clear and less error-prone.

On Thu, Aug 3, 2023 at 12:57 PM Stan Grishin @.***>
wrote:

You may want to make things a little easier and more error-prone for
localizers, by changing things like:
_('Tor Onion (Hidden) Services are proxy tunnels to your local website, SSH and other services')
to
_('%sTor Onion (Hidden) Services%s are proxy tunnels to your local website, SSH and other services').format('', '');
—
Reply to this email directly, view it on GitHub
https://github.com/openwrt/luci/pull/6476#issuecomment-1663687936, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADFODVIQ6XPOX7GOXOUK5DXTNY2HANCNFSM6AAAAAA2UEDYKQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sergey Ponomarev https://linkedin.com/in/stokito,
stokito.com

Damn, it's already exists a PR with the app for the tor-hs #4805
But it stalled. I'll check what left there and will merge with mine work.
I made a search by tor in issues but didn't checked existing PRs :(

Ok, I checked and looks like my PR would be better:

The Hidden service status tab is not needed. Instead in my version just a link with onion domain is shown.
Common settings tab is not needed. No one will ever change anything there.
Hidden service configuration has inline form while in my version it's modal.

That's a good idea. Still I feel that it would be better to keep as is for two reasons: * We may have a translated Wiki page * This is more clear and less error-prone.

There is a reason that links should be formatted like this: safety/security is one of them. It also unburdens the language translators.

@stangri I changed the link to:
m = new form.Map('tor-hs', _('Tor Onion Services'),
	_(
		'Tor Onion (Hidden) Services are proxy tunnels to your local website, SSH and other services.
' +
		'For further information check the documentation'
	).format('https://openwrt.org/docs/guide-user/services/tor/hs')
);

@stangri I made as you you suggested and spitted.

I simplified acl file and made an explicit page for hidden services because in future we will have other pages to configure Tor client options line excluded counties

OK, so is the tor-hs and tor service difference here important? Should the luci app be (re)named appropriately? Or can this also manage tor?

In future we can add a new tab for the Tor routing configuration: SOCKS port, exclude countries etc. Maybe even we can allow to configure the Tor as VPN.
That's all can be added later but for now the tor package UCI doesn't have that much  options.

ok, fixed in the 612738d4

Great, would you like to add the other fields?
e.g.
PKG_LICENSE:=?
PKG_MAINTAINER:=Sergey Ponomarev<EMAIL_ADDRESS>
LUCI_DESCRIPTION:=

PKG_PROVIDES:=

I added the package version, release and maintainer. Everything else like License will be inherited from Luci.

OK, and now for consistency: can you order those additions so they're like other Makefiles? e.g.

PKG_*

LUCI_*

As far I know that should be enough: it has own version and maintainer but everything else same as for the luci.
Here is for example transmission app that has only Title and Depends:
https://github.com/openwrt/luci/blob/master/applications/luci-app-transmission/Makefile

Thank you

Thanks for your patience 😄


[AutoPR] dns/resource-manager

Created to accumulate context: dns/resource-manager

(message created by the CI based on PR content)

Codecov Report

Merging #3000 into master will decrease coverage by <.01%.
The diff coverage is 83.78%.

@@            Coverage Diff             @@
##           master    #3000      +/-   ##
==========================================
- Coverage   56.35%   56.35%   -0.01%
==========================================
  Files        8124     8124
  Lines      166280   166282       +2
==========================================
+ Hits        93713    93714       +1
- Misses      72567    72568       +1

Impacted Files
Coverage Δ

...v2018_03_01_preview/operations/zones_operations.py
79.48% <82.35%> (-0.52%)
:arrow_down:

...03_01_preview/operations/record_sets_operations.py
78.85% <85%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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

@AutorestCI rebase

Rebase done and pushed to the branch

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/dns/resource-manager/readme.md

Working on generating https://github.com/azure/azure-rest-api-specs/blob/master/specification/dns/resource-manager/readme.md for you!!!

Nothing to rebuild, this PR is up to date


Shapiro

Edits to Kent's Shapiro-Wilk and Kruskal-Wallis.
Functions + test file ready to go. Script is a bit sketchier. Looks like about half of the voxels for subject 1 are approx. normal, alpha=0.05.
Kruskal-Wallis doesn't seem to be working as expected if you compare data with random normal sample using data's mean and standard deviation. Should talk about this later.

Gonna merge for now because I need to edit the report. Should probably go over the shapiro stuff later.


Make Year dynamic in copyright notice.

See also: https://github.com/Stravaig-Projects/Stravaig.Extensions.Logging.Diagnostics/issues/29

This was fixed in PR https://github.com/Stravaig-Projects/Stravaig.Extensions.Configuration.Diagnostics/pull/146


Code inset feature

This PR introduces a new feature to allow an HTML document to be inset inside a code editor at a given line of code. This feature is exposed to extensions to support a variety of use cases. To illustrate the feature, the PR includes an extension that looks for specially formatted comments in JS/TS files, e.g.
// INLINE some-url
and embeds the contents of that URL directly below the comment line. Another use case is to support a user experience like computations notebooks (e.g. Jupyter) in which script results are shown inlined inside the script code.
This PR is an MVP to illustrate the feature.

@jrieken  I like this tweak to the API, particularly since it means that extensions won't have to implement both a CodeInsetProvider and TextDocumentContentProvider to put original HTML in the editor. This suggestion nicely combines these into one class.
I'm trying to update the PR to implement this change, but it's a struggle. The issue is this. The webview needs to be created by MainThreadWebviews.$createWebview in order for an extension to make calls on it. However, the webview also needs to be available to the code inset editor contribution, namely CodeInsetController. So far, I'm not seeing a good way to wire these two components together. Any suggestions would be very welcome. Maybe MainThreadWebviews needs to be a service so that the code inset editor contribution can ask for it in its constructor.

I've updated the PR to reflect the requested API change that @jrieken requested. The new API is definitely nicer as can be seen in extensions/inline-doc/extension.ts. I found a way to wire up the pieces that I think is reasonable, but of course all suggestions are welcome.
I think there's an interesting speed/space trade-off with this feature. Webviews both take up memory and are noticeably slow to load. To optimize for space, one could allocate webviews only for the currently visible viewport, but the loading delay would be noticeable to the user. I think it's up to the core Code team to decide an appropriate policy for this trade-off.

@rdeline We have finished our February planning and we wanna go forward with this. The goal is to get this merged quickly and that we take over for the final bits. No promise on when we expose this a real API, for now this will be proposed and we will experiment with the overall performance first. As next steps I would propose

[ ] merge with master one more time
[ ] move the sample extension so that it doesn't become part of vs code

Can we get a screenshot please?

@rdeline I have squashed and rebased your commits into https://github.com/Microsoft/vscode/pull/66418/commits/98aec46197005bf122028b53ade944b24a9ede22. I have also pushed a few commit which makes code insets a workbench feature only - in short I move the declarations of interfaces and settings. Also, I have pushed https://github.com/Microsoft/vscode/pull/66418/commits/2517974c91b806a1cb33987e7a833aca2f1d0724 which undoes some seemingly unrelated changes.

Screen shot of the sample extension to inline arbitrary URLs inside the code:

This will be merged now, but that doesn't mean the feature is done. I think we now have a solid proposal on which we can work. Huge thanks to @rdeline for starting this 🎉
Things that aren't tackled/discussed yet

Lifecycle - each web view is a process, a single editor cannot have 10s or 100s. We should consider (a) restarting, e.g. only visible insets are running, (b) adding a non-interactive inset, no code execution which would allow for an iframe, (c) limit the number of insets
API - the current proposal is very much like code lens. The flow is that whenever a text inside the editor changes, we ask for the places at which insets will occur and then we ask to fill those insets. We should study the use-cases but a "push" API might be better. E.g. have something like createTextEditorInset(editor, position, options) which an extension calls whenever it wants to add an inset (when a command is invoked or when text changes). The extension will then manage insets as long as the editor exist - this approach is similar to editor decorations.

Could we use this to display type inference result or parameter names in function calls a la intellij (see: https://github.com/Microsoft/vscode/issues/16221#issuecomment-264527520)?

Could we use this to display type inference result or parameter names in function calls a la intellij

Nope, insets are always in-between lines, like code lens or peek. Adding stuff in-between character is mush harder, unless you take the decoration approach, e.g. what css color decorations do.

@jrieken Thanks so much for taking this PR seriously. I think the idea of allowing iframe-based code insets is excellent -- I wish I had thought of it in time for the initial PR. For the use case I care about (data science workflows, including scripting sessions, notebooks, etc.), the vast majority of info to be displayed (script results) are just text or images, so it makes perfect sense to optimize for that case. I also think that language extensions probably would have sufficient ability to reflect over results to choose between interactive and non-interactive, so a flag or something makes sense.
In terms of push versus pull API, I freely confess that I based the initial PR on code lenses because it was the closest existing feature I could find. As a newbie, I needed a pattern to follow. I think push might be better.

Wow -- this is awesome... I also agree that a push api might be better, since it will provide more explicit control over when an indent is shown -- which could also in theory naturally reduce the amount of them shown.

I would also love to see the ability to "anchor" insets -- more like margins. To anchor a webview to the top/bottom/left/right of the editor.

@eamodio You mean floating atop the text?

Not on top -- sort of like a custom gutter, but one can be positioned/anchored to any of the sides of the editor.

Will it be possible to create floating webviews with this?

@rdeline Do you have any examples on how to get this api working?

fyi - while tackling some debt I have also jumped on this api proposal and I have changed it to a push model. surely some new regressions have been added, but the API really looks nice: https://github.com/microsoft/vscode/blob/6f1da34c2efc68c8145061e072e053625107306e/src/vs/vscode.proposed.d.ts#L138-L152

sample, implements command that adds an inset:

    vscode.commands.registerCommand('extension.sayHello', async (args, brgs, crgs) => {
        if (!vscode.window.activeTextEditor) {
            return;
        }

        const inset = vscode.window.createWebviewTextEditorInset(
            vscode.window.activeTextEditor,
            vscode.window.activeTextEditor.selection.with({ end: vscode.window.activeTextEditor.selection.end.translate(8) }),
            // { enableScripts: true, enableCommandUris: true }
        );
        inset.onDidDispose(() => {
            console.log('WEBVIEW disposed...');
        });
        inset.webview.html = `<head><meta></head><body><img src="https://imgs.xkcd.com/comics/plutonium.png"/><body>`;
    });

@jrieken Is there a way to make the webview html interactive? (So that the user can click on a button in the code inset, for example)

@jrieken Great work. The most astonishing point is that this is so fast and lightweight.
I have a question. Insets disappear when we change tabs. Is this an intended behavior?

I am looking forward to this feature coming to the stable branch.

I love the simplified the API, but I'm wondering about this code in the implementation:
		const webview = this._webviewService.createWebview({
			enableFindWidget: false,
			allowSvgs: false,
			extension: undefined
		}, {
				allowScripts: options.enableScripts
			});

As far as I can tell, the fact that extension is null means that the vscode-resource protocol will not be defined properly for the webview, since extension location will be undefined. This means that extensions can't load local resources into inset webviews, which is a real limitation. I think the typical use case is that an extensions will ship with some HTML to be displayed. This is what I'm trying to do in the Python extension, for example.

Insets disappear when we change tabs. Is this an intended behavior

Yes - for the API it means the editor goes aways and comes back (as new editor) and therefore you need to re-add them. This is inline with how decorations work.

As far as I can tell, the fact that extension is null means that the vscode-resource protocol will not be defined properly for the webview,

Ops, that was an oversight and not in the intent. I will push a fix for that

@jrieken Another piece of feedback on the new API, based on my PR for the Python extension to inline Jupyter results. First, in the new API, the inset range encodes two unrelated pieces of information: the line number where the inset should appear, and the height (in lines) of the inset. The fact that range.end.line - range.start.line encodes the height is unexpected. I think it would be clearer and more useful to provide two separate parameters: createWebviewTextEditorInset(editor, line, height, options).
Second, I think the height needs to be specified in either lines or pixels. Consider the Python case below (still a work in progress). A textual result is easiest to express in lines, but an image result is easiest to express in pixels. Indeed, I cannot find an extension API method to get the line height to do the conversion myself. If such an API call exists, then I suppose it's okay to make the extension writer do the conversion.

the fact that range.end.line - range.start.line encodes the height is unexpected.

Took a short cut there... The neat thing with hight in lines (vs pixels) is that is honours different line height settings without further ado. Not yet sure how we could expose that to extension author tho...

latest proposal
export function createWebviewTextEditorInset(editor: TextEditor, line: number, height: number, options?: WebviewOptions): WebviewEditorInset;

how access createWebviewTextEditorInset inside vscode .
i update my vscode package but doesn't appear this method?
thanks

Hello, maybe I don't fully understand how this works, but is there a piece of API documentation relevant to this functionality, or is it not fully completed?

@dddom check out this branch by @rdeline https://github.com/rdeline/vscode-python/tree/inset-results

Wait, I don't get it. I'm really confused.
Can I now use the //INLINE <img_url> command?
(In my code editor it doesn't seem to work)
I rather don't build a Python VSCode extension that is 20+ commits ahead and 300+ commits behind when I'm coding in JavaScript.

@dddom: The API is still in Proposed status, which means it can't be used in extensions yet, but that's usually a matter of time. You can see the implementation of window.createWebviewTextEditorInset here and what it provides here.
@melvinroest No, the "//INLINE" comment is an example of an extension which could be built to use Code Insets, rather than a description of this feature. So far, there are no user-visible changes to VSCode, and no published extensions can use Code Insets until it leaves Proposed status.

hi guys complete noob here, is this in production now?

FYI: issue #85682 opened.


Added clipBehavior to Overlay, Flow, AnimatedSize and AndroidView

Description
Added clipBehavior to Overlay, Flow, AnimatedSize and AndroidView.
/cc @liyuqian
Related Issues
Closes #63246
Tests
Added tests to check if the mentioned widgets can set and update their clipBehaviors.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

@liyuqian Did the same to AndroidView and AnimatedSize because some tests were failing


I'm running Enterprise Plus, why does it says "isn't enabled on your system"

I'm running Enterprise Plus, the most expensive available tier. Yet, the pull request annotation says:

Can someone explain how to make this work without this error message?

@EvertonSA,
Could it be that your subscription doesn't include the Advanced Security package? I recommend approaching your JFrog account contact or JFrog Support for assistance with this. Please let us know if you need further assistance and we'd be happy to assist.

that could be the case, will check internally. thanks!


Globally exposed webpack variables not found when upgrading to React.net 4.x and migrating from expose-loader

I am in the process of updating our webpack, babel, and react.net packages.  We are using .netMVC.   When using react.net 3.x and webpack 3 everything has been working fine.  However,  since trying to upgrade (react.net 4.1 and webpack 4.29, babel  7.2.2, and babel-loader 8.05) I've run into a problem with React.net finding my components to server side render.  If I turn off server side rendering, then there is no problem.
When I have  ReactSiteConfiguration.Configuration.SetLoadReact(false)  I get this error:

[ReactNotInitialisedException: React has not been loaded correctly: missing (React, ReactDOM, ReactDOMServer).Please expose your version of React as global variables named 'React', 'ReactDOM', and 'ReactDOMServer', or enable the 'LoadReact'configuration option to use the built-in version of React.]

However, I have globally assigned the React variables in the file that I am loading in my ReactConfig.
import React from 'react';
import ReactDOM from 'react-dom';
import ReactDOMServer from 'react-dom/server';
import RootComponent from 'components/demo/';
global.React = React;
global.ReactDOM = ReactDOM;
global.ReactDOMServer = ReactDOMServer;
global.DesktopAndMobileDemo = RootComponent;

My RootComponent:
import React, {Component } from 'react';

export default class Demo extends Component {
    render() {
        return (

Hello

        );
    }
}

The @HTML.React :
@Html.React("DesktopAndMobileDemo", new { })
Some of the bundled javascript that is loaded in the React.config, which looks like it is being assigned to global.
"use strict";
__webpack_require__.r(__webpack_exports__);
/* WEBPACK VAR INJECTION */(function(global) {/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! react */ "../node_modules/react/index.js");
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_0__);
/* harmony import */ var react_dom__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! react-dom */ "../node_modules/react-dom/index.js");
/* harmony import */ var react_dom__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(react_dom__WEBPACK_IMPORTED_MODULE_1__);
/* harmony import */ var react_dom_server__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! react-dom/server */ "../node_modules/react-dom/server.browser.js");
/* harmony import */ var react_dom_server__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__webpack_require__.n(react_dom_server__WEBPACK_IMPORTED_MODULE_2__);
/* harmony import */ var components_demo___WEBPACK_IMPORTED_MODULE_3__ = __webpack_require__(/*! components/demo/ */ "../src/components/demo/index.js");

global.React = react__WEBPACK_IMPORTED_MODULE_0___default.a;
global.ReactDOM = react_dom__WEBPACK_IMPORTED_MODULE_1___default.a;
global.ReactDOMServer = react_dom_server__WEBPACK_IMPORTED_MODULE_2___default.a;global.DesktopAndMobileDemo = components_demo___WEBPACK_IMPORTED_MODULE_3__["default"];

If I have Reactjs.net load the react, I still get this error:

[ReactInvalidComponentException: Could not find a component named 'DesktopAndMobileDemo'. Did you forget to add it to App_Start\ReactConfig.cs?]
React.ReactComponent.EnsureComponentExists() +203

How does react.net know to look on the 'global' object for the components or react itself?  Before upgrading, I was using the expose-loader so the components would be available to reactjs.net.  Is there any difference in how to expose components globally for .net and .netcore? I couldn't find an example for .net that was equivalent with the .netcore version that exposes components using the 'global' variable.  Oh, btw, I'm also using V8 as my javascript engine.  Does the engine affect how we can expose components?
Thanks for any feedback.

I figured it out.   I needed to add the runtime and vendor scripts to the ReactSiteConfiguration.Configuration.  I initially overlooked that in the example project. 🙁
I didn't need to include any of my shared files in my webpack 3 setup in the ReactSiteConfiguration.Configuration....  I think it may have been due to how I was using the expose-loader when building our old server bundles.
Anyway, great job with the project :)


'YoutubeAPI' object has no attribute 'Video'

trying to scrape multiple video.. but getting this 'YoutubeAPI' object has no attribute 'Video' inside a loop

update! change the doc to video not Video! its working now

Should be
YoutubeAPI().video(video_id)
On Mon, 28 Feb 2022, 12:19 pm Dom, @.***> wrote:

trying to scrape multiple video.. but getting this 'YoutubeAPI' object has
no attribute 'Video' inside a loop
—
Reply to this email directly, view it on GitHub
https://github.com/melvinchia3636/YouTube-Scraping-API/issues/17, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APMTCUB2U6YBFW6V6AV2J7LU5LZUZANCNFSM5PP7Z5JQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Cool, I'll update it ASAP.
On Mon, 28 Feb 2022, 12:22 pm Dom, @.***> wrote:

update! change the doc to video not Video! its working now
—
Reply to this email directly, view it on GitHub
https://github.com/melvinchia3636/YouTube-Scraping-API/issues/17#issuecomment-1053872867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APMTCUFKF53623E4YVQGICLU5L2AZANCNFSM5PP7Z5JQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Documentation changed.


[Bug] Creating an additional account displays existing account address if miner is active

🐛 Bug
When miner is active if user attempts to create an additional account the mnemonic generation screen displays the existing account address (that which is mining), rather than the new address derived from the mnemonic (this works as expected when mining is off). Clicking the generate different keys button results in the mnemonic changing however the account address still remains the same in this scenario.
Note, the new account is properly created upon clicking create and displays correctly in accounts list regardless.

Mining is sunsetted.


IOS - No OpKernel was registered to support Op ‘SpaceToBatchND' with these attrs

I am trying to use tf.nn.atrous_conv2d to solve a segmentation problem. It works on my mac but when I saved this graph using convert_variables_to_constants and tf.train.write_graph then read it in using C++ API trying to get it running on my iPad, I got this error:
Could not create TensorFlow Graph: Invalid argument: No OpKernel was registered to support Op 'SpaceToBatchND' with these attrs.  Registered devices: [CPU], Registered kernels:
  <no registered kernels>

	 [[Node: CCC/SpaceToBatchND = SpaceToBatchND[T=DT_FLOAT, Tblock_shape=DT_INT32, Tpaddings=DT_INT32](BBB, CCC/SpaceToBatchND/block_shape, CCC/SpaceToBatchND/paddings)]]
F /workplace/testTensorFlow/testTensorFlowXcode/Classes/test_tensorflow.mm:162] test_tensorflow.mm : Couldn't load model: Invalid argument: No OpKernel was registered to support Op 'SpaceToBatchND' with these attrs.  Registered devices: [CPU], Registered kernels:
  <no registered kernels>

	 [[Node: CCC/SpaceToBatchND = SpaceToBatchND[T=DT_FLOAT, Tblock_shape=DT_INT32, Tpaddings=DT_INT32](BBB, CCC/SpaceToBatchND/block_shape, CCC/SpaceToBatchND/paddings)]]

What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
https://github.com/tensorflow/tensorflow/issues/1269
https://github.com/tensorflow/tensorflow/issues/2680
https://github.com/tensorflow/tensorflow/issues/2929
If possible, provide a minimal reproducible example
In Python:
import tensorflow as tf
from tensorflow.python.framework.graph_util import convert_variables_to_constants

flt = tf.Variable(tf.random_normal([3, 3, 3, 3], stddev=0.35), name='AAA')
inputs = tf.placeholder(shape=(1, 224, 224, 3), dtype=tf.float32, name='BBB')
conv = tf.nn.atrous_conv2d(value=inputs, filters=flt, rate=2, padding='SAME', name='CCC')

sess = tf.Session(config=config)
init = tf.initialize_all_variables()
sess.run(init)

with sess.as_default():
    minimal_graph = convert_variables_to_constants(sess, sess.graph_def, ["CCC"])
    tf.train.write_graph(minimal_graph, '.', 'rand_init_min_cpu.pb', as_text=False)

I can load and run graphs not using atrous_conv2d correctly, so I think my C++ code is ok.
What other attempted solutions have you tried?
Following the related issues above, I tried to add SpaceToBatchND kernel to
tensorflow/contrib/makefile/tf_op_files.txt
I found SpaceToBatchND in array_ops.cc (which has already been included in tf_op_files.txt)
and maybe in these two files:
tensorflow/core/kernels/spacetobatch_op.cc
tensorflow/core/kernels/batchtospace_op.cc
So I added them to tf_op_files.txt, but then when I ./build_all_ios.sh, I got these errors (short version, I can post them all out if you need):
Undefined symbols for architecture armv7:
"tensorflow::functor::SpaceToBatchFunctor<Eigen::ThreadPoolDevice, float, 4, $
rue>::operator()(Eigen::ThreadPoolDevice const&, Eigen::TensorMap<Eigen::Tensor$
float, 6, 1, int>, 16, Eigen::MakePointer>, long long const*, long long const*,
Eigen::TensorMap<Eigen::Tensor<float const, 6, 1, int>, 16, Eigen::MakePointer>$
", referenced from:

...

void tensorflow::(anonymous namespace)::SpaceToBatchOpCompute<Eigen::Threa
dPoolDevice, int>(tensorflow::OpKernelContext*, tensorflow::Tensor const&, tenso
rflow::Tensor const&, tensorflow::Tensor const&) in libtensorflow-core-armv7.a(s
pacetobatch_op.o)
ld: symbol(s) not found for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [workingspace/tensorflow/tensorflow/contrib/makefile/gen/b
in/ios_ARMV7/benchmark] Error 1
+ '[' 2 -ne 0 ']'
+ echo 'armv7 compilation failed.'
armv7 compilation failed.

Environment info
Operating System:
mac: OSX 10.11.6
iPad: IOS 10.1.1
Installed version of CUDA and cuDNN:
None
Source:
commit d6b25985ac219a6e58d186a2beb74d5e8d9e4533
bazel version:
.................................................
Build label: 0.4.0-homebrew
Build target: bazel-out/local-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Wed Nov 2 19:15:37 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

So, any luck on solving this?
We are currently trying to use a new model without atrous_conv2d but the result is not as good. Maybe we haven't fine tune the new model yet but using atrous_conv2d would be a big time saver.
Any quick fixes or ideas would be appreciated, thanks.

@PhilipZeratul: I worked around this by adding the following to tf_op_files.txt:
tensorflow/core/kernels/spacetobatch_op.cc
tensorflow/core/kernels/spacetobatch_functor.cc

I've also replaced __ANDROID_TYPES_SLIM__ with __ANDROID_TYPES_FULL__ everywhere in the Makefile, but admittedly I am not sure whether this is needed.
For a couple of other flavours of the same error I also had to add
tensorflow/core/kernels/cwise_op_mod.cc
tensorflow/core/kernels/batchtospace_op.cc

This is along the lines of the suggestions in issue 4863
I hope this can help you (or someone else) through the fun times of compiling for iOS.

@schackv: Thank you for your response.
It has been a while since we met this problem so we kind of manually implemented  atrous-conv using normal conv and achieved our goal with some efficiency sacrifice.
We are currently working on another project so I can not test this now, so I'm going to close this issue but still thank you very much.


cat9000v-s1 bootstrap config not loaded

Hi team. I know it's a beta image but cat9k nodes don't load the bootstrap config if the autoinstall process is not manually canceled and an enable secret pw is provided (altough there is a complex enable secret in the bootstrap config). See https://community.cisco.com/t5/tools/cml-bootstrap-cat9000v/m-p/4898367

This is a known issue with the C9Kv.  If you leave it, it will load the bootstrap config, but it will require some console interactivity.  This is what I wrote up for internal users:

The day 0 config does apply, but it takes a while. If you see the autoinstall menu, wait until you
see the message about trying DHCP on Gi0/0 before you hit any key. Also note that if you do
not have an enable secret configured, you will be prompted to configure one after answering,
“no” to the initial config dialog.


Add MIT License to pom.xml

rewrite-clj is distributed under the MIT license, but this is not reflected in the pom.xml. Therefore tools like clojure -T:deps list will not recognise the license.
This PR adds a <licenses> tag to pom.xml.

Thanks!


GPII-3899: Initial implementation of global validation component.

See GPII-3899 for details.

The tests failed because stringify takes longer on the windows VM.  I can relax or remove that check when I'm working on the hashing.

The last build error was the same timing error in response to a later commit.  The upcoming build should include the timing change.

@amb26, I've addressed the first round of comments.

The last build failed because grunt wasn't available for linting, I assume a transient provisioning failure.  Ok to test.

Second failure, I dug into it, looks like the VM had trouble connecting to chocolatey to download the required packages.
==> windows10: [NuGet] An error occurred while loading packages from 'https://chocolatey.org/api/v2/': Unable to connect to the remote server
==> windows10: chromedriver not installed. The package was not found with the source(s) listed.
==> windows10:  Source(s): 'https://chocolatey.org/api/v2/'

The process aborted on that install, and missed the step to install grunt as well, which is why the linting checks failed.

I see the same problem in recent windows builds in CI:
python2 not installed. The package was not found with the source(s) listed.
 Source(s): 'https://chocolatey.org/api/v2/'

Ok to test again.

Seems like we're past the networking issues that killed the previous two builds.  @amb26, this should be ready for further review.

Thanks, @amb26.  I cut the release and updated the forward-facing version in master.  I also updated the companion pull against universal (will comment there shortly).


Confirm traffic monitoring and metric aggregation in reverse-proxy

Verify the monitoring of traffic through the reverse-proxy, including aspects such as request rate, latency, and data throughput. Ensure that metrics are aggregated by subdomain and HTTP status and are accessible via Prometheus

All metrics for Prometheus plugin are enabled
https://docs.konghq.com/hub/kong-inc/prometheus/


QNAP.QNAPJBODManager version <IP_ADDRESS>4

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Retested requires Marvell Drivers to be installed


Web Hooks of Visual Studio Online is doesn't work with Kudu deployment trigger in Azure Websites

Web Hooks of Visual Studio Online has fields below:
Content:
{
"subscriptionId": "####",
"notificationId": 28,
"id": "#####",
"eventType": "git.push",
"publisherId": "tfs",
"message": null,
"detailedMessage": null,
"resource": null,
"createdDate": "2015-10-17T16:49:57.5382357Z",
"resourceVersion": null
}
https://www.visualstudio.com/get-started/webhooks-and-vso-vs
And is no way to change it manually. But Kudu in /deploy triger await the url field in additional. So if I requesting the deploy from VSO to Azure Websites I receive "Internal Server Error": "Value cannot be null. Parameter name: uriString".

Not sure I completely understand the question, but please see this page. Bottom line is that if you are deploying using VSO, you are not using Kudu, so nothing related to Kudu will apply to your scenario.

Yes, if you deploy the Visual Studio project. Like a C#, VB.NET or other .NET projects. But if you store static or PHP sites in VSO repository you can not use VSO build. For this case VSO have Web Hooks, which can request deploy trigers like one in Kudo. So the Kudo's triger on Azure Websites is not work properly with VSO Web Hooks. Request of the /deploy api returning Internal Serever Error (Error 500). You can check out error here: https://YOURACCOUNT.visualstudio.com/DefaultCollection/YOURPROJECT/_admin/_servicehooks
Trace files:

I have the trace file from Azure Website, but can't attach it here. So I put here main part:
title="Error occurred" date="2015-10-17T12:52:22.827" type="error" text="Value cannot be null.
Parameter name: uriString" stackTrace="   at System.Uri..ctor(String uriString)
at Kudu.Services.ServiceHookHandlers.VSOHandler.GetDeploymentInfo(HttpRequestBase request, JObject payload, String targetBranch)

Oh I see, you're trying to use the new VSOHandler. @suwatch @shrimpy is this even supposed to work today? I'm guessing it's meant for a payload that's different. Here, base on the error, it seems that RepoUrl is not set.

I checked RepoUrl var in Azure Resource Explorer and it is set:
{
"id": "/subscriptions/###/resourceGroups/###/providers/Microsoft.Web/sites/###/sourcecontrols",
"name": "####",
"type": "Microsoft.Web/sites/sourcecontrols",
"kind": null,
"location": "West Europe",
"tags": {
"hidden-related:/subscriptions/###/resourcegroups/###/providers/Microsoft.Web/serverfarms/###": "empty"
},
"plan": null,
"properties": {
"repoUrl"<EMAIL_ADDRESS>"branch": "master",
"isManualIntegration": true,
"deploymentRollbackEnabled": false,
"isMercurial": false
}
}

In fact VSO is send the url field, but not in root of JSON document. It send the url of repository in two levels lower under resource -> repository section of JSON document as an two levels of a nested array:
Example:

{
  "subscriptionId": "###",
  "notificationId": 30,
  "id": "###",
  "eventType": "git.push",
  "publisherId": "tfs",
  "message": {
    "text": "Jamal Hartnett pushed updates to branch master of repository Fabrikam-Fiber-Git."
  },
  "detailedMessage": {
    "text": "Jamal Hartnett pushed 1 commit to branch master of repository Fabrikam-Fiber-Git.\n - Fixed bug in web.config file 33b55f7c"
  },
  "resource": {
    "commits": [
      {
        "commitId": "33b55f7cb7e7e245323987634f960cf4a6e6bc74",
        "author": {
          "name": "Jamal Hartnett",
          "email"<EMAIL_ADDRESS>          "date": "2015-02-25T19:01:00Z"
        },
        "committer": {
          "name": "Jamal Hartnett",
          "email"<EMAIL_ADDRESS>          "date": "2015-02-25T19:01:00Z"
        },
        "comment": "Fixed bug in web.config file",
        "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git/commit/33b55f7cb7e7e245323987634f960cf4a6e6bc74"
      }
    ],
    "refUpdates": [
      {
        "name": "refs/heads/master",
        "oldObjectId": "aad331d8d3b131fa9ae03cf5e53965b51942618a",
        "newObjectId": "33b55f7cb7e7e245323987634f960cf4a6e6bc74"
      }
    ],
    "repository": {
      "id": "278d5cd2-584d-4b63-824a-2ba458937249",
      "name": "Fabrikam-Fiber-Git",
      "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249",
      "project": {
        "id": "6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c",
        "name": "Fabrikam-Fiber-Git",
        "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/projects/6ce954b1-ce1f-45d1-b94d-e6bf2464ba2c",
        "state": "wellFormed"
      },
      "defaultBranch": "refs/heads/master",
      "remoteUrl": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_git/Fabrikam-Fiber-Git"
    },
    "pushedBy": {
      "id"<EMAIL_ADDRESS>      "displayName": "Jamal Hartnett",
      "uniqueName": "Windows Live<EMAIL_ADDRESS>    },
    "pushId": 14,
    "date": "2014-05-02T19:17:13.3309587Z",
    "url": "https://fabrikam-fiber-inc.visualstudio.com/DefaultCollection/_apis/git/repositories/278d5cd2-584d-4b63-824a-2ba458937249/pushes/14"
  },
  "createdDate": "2015-10-18T12:23:14.965571Z"
}

Note that one key aspect of making this work is authentication. i.e. Kudu needs to be able to access the VSO repo in order to pull from it. Today, this is not supported, but it is work in progress, and hopefully will be available in the next couple of months.

We haven`t official support VSO yet. At the time we implement VSOHandler, webhook payload from VSO is static other than a short-ive session token, which allow Kudu to access user repo. And I have not tried since then.
Things seems have been changed a lot, from the payload @denisyfrolov posted, i don`t see any session token.
As @davidebbo mention there is work in progress, we will adjust our code to consume any new changes in the short coming future.

@davidebbo @shrimpy is no any problems with authentication and access to the VSO repo from the Kudu and Azure Websites, URL of repo contains all required data as login and password. The Kudu works fine If I manually click the Sync button in the Continuous deployment section on the Azure Portal. Any new commits from VSO successfully deployed.
I got it, the work in progress. I really hope it is will work fine in short time, thank you! I just want to you understand the issue, is no problem with authentication and session token, The Kudu works fine If I manually click the Sync button in the Continuous deployment section on the Azure Portal. Any new commits from VSO successfully deployed! The issue is WebHook Deploy api only!

@denisyfrolov
When you click on the "Sync" button, Azure Web App actually trigger a WebHook notification, see Test hook section.
I suspect the WebHook payload from "Test WebHook" and when you do a "Push" might be different. Is it possible for you to capture both payload? so that we can compare?

Yes I know, that Sync button is actually trigger to the same WebHook. That's why I say that Kudu works fine with repo in VSO, the issue happens only with request to WebHook from VSO.
Of course I can, the full PUSH payload below:

stdClass Object
(
    [subscriptionId] => ########-3d81-####-be18-####36c4afdc
    [notificationId] => 55
    [id] => ########-8e1f-####-b67e-####e7e9b170
    [eventType] => git.push
    [publisherId] => tfs
    [message] => stdClass Object
        (
            [text] => Denis Frolov pushed updates to branch master of PROJECT
(https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/#version=GBmaster)
            [html] => Denis Frolov pushed updates to branch master of PROJECT
            [markdown] => Denis Frolov pushed updates to branch [master](https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/#version=GBmaster) of [PROJECT](https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/)
        )

    [detailedMessage] => stdClass Object
        (
            [text] => Denis Frolov pushed 1 commit to branch master of PROJECT
 - Test of Continuous Deployment 6b594fbc (https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/commit/6b594fbc892118b1c3d458249fa1c3110380618f)
            [html] => Denis Frolov pushed 1 commit to branch master of PROJECT

Test of Continuous Deployment 6b594fbc

            [markdown] => Denis Frolov pushed 1 commit to branch [master](https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/#version=GBmaster) of [PROJECT](https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/)
* Test of Continuous Deployment [6b594fbc](https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT/commit/6b594fbc892118b1c3d458249fa1c3110380618f)
        )

    [resource] => stdClass Object
        (
            [commits] => Array
                (
                    [0] => stdClass Object
                        (
                            [commitId] => 6b594fbc892118b1c3d458249fa1c3110380618f
                            [author] => stdClass Object
                                (
                                    [name] => Denis Frolov
                                    [email] =><EMAIL_ADDRESS>                                    [date] => 2015-10-19T20:51:40Z
                                )

                            [committer] => stdClass Object
                                (
                                    [name] => Denis Frolov
                                    [email] =><EMAIL_ADDRESS>                                    [date] => 2015-10-19T20:51:40Z
                                )

                            [comment] => Test of Continuous Deployment
                            [url] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b/commits/6b594fbc892118b1c3d458249fa1c3110380618f
                        )

                )

            [refUpdates] => Array
                (
                    [0] => stdClass Object
                        (
                            [name] => refs/heads/master
                            [oldObjectId] => 76cd2432fbfa88bd5ec6efa2e9181b8d115be0a8
                            [newObjectId] => 6b594fbc892118b1c3d458249fa1c3110380618f
                        )

                )

            [repository] => stdClass Object
                (
                    [id] => ########-7ba3-####-8c51-####9019219b
                    [name] => PROJECT
                    [url] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b
                    [project] => stdClass Object
                        (
                            [id] => ########-4e3f-####-b1c3-####cebd5fd4
                            [name] => PROJECT
                            [url] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/projects/########-4e3f-####-b1c3-####cebd5fd4
                            [state] => wellFormed
                        )

                    [defaultBranch] => refs/heads/master
                    [remoteUrl] => https://denisyfrolov.visualstudio.com/DefaultCollection/_git/PROJECT
                )

            [pushedBy] => stdClass Object
                (
                    [id] => ########-9ff0-####-b86a-####0312861c
                    [displayName] => Denis Frolov
                    [uniqueName] =><EMAIL_ADDRESS>                    [url] => https://denisyfrolov.vssps.visualstudio.com/_apis/Identities/########-9ff0-####-b86a-####0312861c
                    [imageUrl] => https://denisyfrolov.visualstudio.com/DefaultCollection/_api/_common/identityImage?id=########-9ff0-####-b86a-####0312861c
                )

            [pushId] => 20
            [date] => 2015-10-19T20:51:50.9236287Z
            [url] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b/pushes/20
            [_links] => stdClass Object
                (
                    [self] => stdClass Object
                        (
                            [href] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b/pushes/20
                        )

                    [repository] => stdClass Object
                        (
                            [href] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b
                        )

                    [commits] => stdClass Object
                        (
                            [href] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b/pushes/20/commits
                        )

                    [pusher] => stdClass Object
                        (
                            [href] => https://denisyfrolov.vssps.visualstudio.com/_apis/Identities/########-9ff0-####-b86a-####0312861c
                        )

                    [refs] => stdClass Object
                        (
                            [href] => https://denisyfrolov.visualstudio.com/DefaultCollection/_apis/git/repositories/########-7ba3-####-8c51-####9019219b/refs
                        )

                )

        )

    [resourceVersion] => 1.0
    [resourceContainers] => stdClass Object
        (
            [collection] => stdClass Object
                (
                    [id] => ########-6f49-####-9b7f-####e9a93aa7
                )

            [account] => stdClass Object
                (
                    [id] => ########-ae32-####-b21b-####f8843b94
                )

            [project] => stdClass Object
                (
                    [id] => ########-4e3f-####-b1c3-####cebd5fd4
                )

        )

    [createdDate] => 2015-10-19T20:51:56.1492376Z
)

@denisyfrolov
alright, since you do most of the configuration manually, and you aware that we haven't officially support VSO, let`s see if we can enable you with some work-around.
About the null-ref you are experiencing, it is from this line, where Kudu store user`s repository url during setup (as i mention earlier, TestWebhook payload is static, only useful data is the short-live session token). Since there is no setup for you (you did it manually, there is no repo url). So you have to manually create it.
Please go to your SCM site, go to debug console and navigate to
D:\home\site\deployments

see if there is a "settings.xml" there?
if yes, you can see the file:
open it and see if you have "RepoUrl" there, if not, add it
if no, no such file:
let`s create one and paste below content
<?xml version="1.0" encoding="utf-8"?>

<settings>
  <deployment>
    <add key="branch" value="[your branch here]" />
    <add key="RepoUrl" value="[your repo url here with basic auth]" />
  </deployment>
</settings>

this should get you moving, let me know if you hit other issue along the way

Yea, this solution do moving process, thanks. I added the RepoUrl key to my settings.xml and now I get another Internal Server Error at the same function but in another line:

<step title="Error occurred" date="2015-10-20T11:18:19.899" type="error" text="Object reference not set to an instance of an object." stackTrace="   at Kudu.Services.ServiceHookHandlers.VSOHandler.GetDeploymentInfo(HttpRequestBase request, JObject payload, String targetBranch)
   at Kudu.Services.ServiceHookHandlers.VSOHandler.TryParseDeploymentInfo(HttpRequestBase request, JObject payload, String targetBranch, DeploymentInfo& deploymentInfo)
   at Kudu.Services.FetchHandler.GetRepositoryInfo(HttpRequestBase request, JObject payload, String targetBranch, DeploymentInfo& info)
   at Kudu.Services.FetchHandler.<ProcessRequestAsync>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
</pre>

I think you may run into seesion token is null from VSO hook payload.  Note that one would need to add VSO hook in a specific way (intended for Azure the moment) in order to have sessionToken flow with the payload.  Unfortunately, there is no UI to achieve that.
You can look at the VSO APis section of the wiki.  There is a sample how one would set it up manually.   If you don't see your VSO accounts, you may need to go to preview portal to link the VSO account with your Azure subscription.

I think the point is that he doesn't want to use the session token since the creds are embedded in the URL. Basically, we'd want the VSO handler to gracefully degrade to work the same as the external repo case (i.e. what happens when you click the sync button). It may be as simple as skipping the line you point to is there is no session token, and instead just using the url as is.

@denisyfrolov
i have a fix for your latest error. PR https://github.com/projectkudu/kudu/pull/1756
if you can share your site name or tell us your site name in-directly, I can patch your site to unblock you right away.

@shrimpy thanks a lot, lets try do that on test the site onetile.azurewebsites.net

@davidebbo @suwatch thanks for the tip, I didn't know about ARMClient, it is great to have ability to manualy setup webhooks on VSO and I will try to use it for this case. But of course the best way is work with VSO as simple as any other external hosted repos. Thanks for the help again!

@suwatch I try use ARMClient for VSO, but any request from ARMClient to VSO like ARMClient.exe get "https://app.vssps.visualstudio.com/_apis/profile/profiles/me?api-version=1.0" return the authentication page. In some time all requests to SCM like: ARMClient.exe get "/subscriptions?api-version=2014-04-01" is successful and return JSON.
ARMClient is authenticated in Azure through ARMClient.exe login and my VSO account is linked to my Azure account. Is this bug or I doing soething wrong? Maybe better is forwarding this question to ARMClient Issues?

@denisyfrolov just patched your site. can you give it a try?

@shrimpy just tried, it is work fine finally! However the Continuous deployment settings is missing and first try is return 403, but I re setup the connection to VSO as external repo and everything works fine now. Thanks! Is this patch will applying to all new sites? Or old sites will patched too in some time?

It will apply to all sites once we redeploy the next  build of Kudu. Once we do that, you'll want to delete your D:\home\SiteExtensions\Kudu folder to move away from the private build.

@davidebbo OK, it is great. Thank to all you guys again.

@denisyfrolov I moved the discussion for issue VSO ARMClient to https://github.com/projectkudu/ARMClient/issues/23.  Let discuss it there.

Change has been published to all sites. Closing this issue now.


New app: Test Patterns

Show a colorful test card on your Tidbyt.
Author: harrisonpage

I have read the CLA Document and I hereby sign the CLA


Separate out seed configuration

Can we move out the seed config into a config file? Mainly I want to do this because I'm going to generate the config file in Docker.
Let me know if there are any issues. I'm new to Elixir and the project.

Here's the Docker stuff I made:
https://github.com/slipo/neo-scan-docker

Great Work! Sometimes node calls timeout and travis fails... It's nothing to do with the code though. Let me know if you have any doubts!


Missing License

What do you think about adding License information to this project? I think that this could help to share this project with more people, it would look more professional. I was thinking about some simple permissive license like MIT.

Done :)


Deneb follow up

📝 Summary

Some logging and test improvements
⛱ Motivation and Context

Follow up PR addressing comments on https://github.com/flashbots/mev-boost/pull/553.
📚 References

✅ I have run these commands

[x] make lint
[x] make test-race
[x] go mod tidy

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (4ff4d89) 67.74% compared to head (8d83655) 68.07%.

Files
Patch %
Lines

server/service.go
0.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #608      +/-   ##
===========================================
+ Coverage    67.74%   68.07%   +0.32%
===========================================
  Files            7        7
  Lines         1203     1206       +3
===========================================
+ Hits           815      821       +6
  Misses         331      331
+ Partials        57       54       -3

Flag
Coverage Δ

unittests
68.07% <0.00%> (+0.32%)
:arrow_up:

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

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


Handle when bookmark and bookmark folder name in same path are the same

At least in linux, a file and a folder cannot have the same name in a given directory


[WIP] [POC] Add provisioned? lifecycle status info to retireable? check

We need to factor in the lifecycle status when we ask a service if it's retireable. Contrary to previous suggestions, it makes no sense to use the supports_feature mixin, because that's a class check and this is specific to the instance. It may as well just be part of this check, as well.
It's basically a proof of concept, I just wanted to start the conversation and get feedback and buy-in. I'm not taking into account places this is overridden yet cause I wanted opinions first. Also there are no specs, cause I wanted feedback first.
skeleton work for https://bugzilla.redhat.com/show_bug.cgi?id=1595511
@miq-bot assign @tinaafitz

@miq-bot add_reviewer @lfu
@miq-bot add_reviewer @bdunne
@miq-bot add_reviewer @gmcculloug

@Fryguy, thoughts? Since you are likely to be vociferous about it...

Anyone other than Lucy want to look at this, please?

@miq-bot remove_label wip
everyone else, please re-review?

@tinaafitz Can you take another look?

Have the UI tests been run with this change?  I suspect they will have tests around this

@bdunne gimme just a minute, please

Of the 11 failures I get, they're all
 8) Mixins::EmsCommon::Angular get_task_args openstack cloud returns connect options for openstack cloud AMQP tab
     Failure/Error: expect(@ems_cloud_controller.send(:get_task_args, 'ManageIQ::Providers::Openstack::CloudManager')).to eq(expected_connect_options)

       expected: ["v2:{k8Sm5ygvDAvvY5zkvev1ag==}", {"amqp_api_port"=>"5462", "amqp_hostname"=>"host_amqp", "amqp_security_protocol"=>"non_ssl", "amqp_userid"=>"xyz"}]
            got: ["v2:{6yyYgLW3nPt9T1wkYsOzJQ==}", {"amqp_api_port"=>"5462", "amqp_hostname"=>"host_amqp", "amqp_security_protocol"=>"non_ssl", "amqp_userid"=>"xyz"}]

       (compared using ==)

       Diff:
       @@ -1,4 +1,4 @@
       -["v2:{k8Sm5ygvDAvvY5zkvev1ag==}",
       +["v2:{6yyYgLW3nPt9T1wkYsOzJQ==}",

and not a one looks related to any of this code change.

I suspect they will have tests around this
HAHAHA

@miq-bot add_label enhancement
I guess you know it's buggy when it has its own label, looking at you, Services.


Only render sigil in chat if new @p message

You can hover on the message to reveal the time instead, as in the image below.

Thank you for writing this! I’m going on vacation for a week so I’ll give this a more thorough review when I get back. It looks structurally great! I’ll run it, potentially tweak the styles, and merge once I’m back.