Minor tweaks to be more Pythonic

This is a series of commits refactoring some code idioms to be more Pythonic in style as discussed on https://github.com/openjournals/joss-reviews/issues/766. None are intended to have a functional effect on the program's behaviour.

Thank you.


/eth/tx/receipt doesn't have simplified response

status, cumulativeGasUsed, gasUsed, effectiveGasPrice, blockNumber and transactionIndex would be more suitable as decimal

{
  "type": "0x2",
  "root": "0x",
  "status": "0x1",
  "cumulativeGasUsed": "0x57f2",
  "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000040000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000002000000000000000000200000000000000000000000000000020000000000000000000000000000000000000000000000040000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
  "logs": [
    {
      "address": "0x5fbdb2315678afecb367f032d93f642f64180aa3",
      "topics": [
        "0x24abdb5865df5079dcc5ac590ff6f01d5c16edbc5fab4e195d9febd1114503da"
      ],
      "data": "0x000000000000000000000000000000000000000000000000000000000000002a",
      "blockNumber": "0x2",
      "transactionHash": "0x60573721ca1f0c4dd80f11dac04eb6403c6b72bc27ceb9ce93dddb5e333bfc62",
      "transactionIndex": "0x0",
      "blockHash": "0xa4e1b9ae07724a35190410d7de7a85d084200c47661b6a0a2fb2d463711958da",
      "logIndex": "0x0",
      "removed": false
    }
  ],
  "transactionHash": "0x60573721ca1f0c4dd80f11dac04eb6403c6b72bc27ceb9ce93dddb5e333bfc62",
  "contractAddress": "0x0000000000000000000000000000000000000000",
  "gasUsed": "0x57f2",
  "effectiveGasPrice": "0x6948dacc",
  "blockHash": "0xa4e1b9ae07724a35190410d7de7a85d084200c47661b6a0a2fb2d463711958da",
  "blockNumber": "0x2",
  "transactionIndex": "0x0"
}

An example for most of the keys which could be in transaction receipt


update confirmation email with feedback.

BB & BH are to get me feedback.

feedback received aug-31.

subject and part oc body update implemented in commit https://github.com/birkin/easyrequest_project/commit/9ab3465c15689a863ef0b4b51d79afa23696b12a.

adds pickup-location in commit https://github.com/birkin/easyrequest_project/commit/91926db65eb27c333c14b36092759d1f1be86f38.


Update docker example for new images

The new docker images don't have the simple-ping.canton anymore and the simple-topology.conf has been moved.

I don't feel I am in a position to review this, as I'm lacking a lot of context. My understanding of the company position is still that we should not provide Docker images for open-source versions of our components, as Docker images are an enterprise feature.
Ignoring that, I'm not aware of where the canton open-source Docker image is defined and how/why it has changed.
Tangentially related, I would point out that the commands a couple lines above of this change should be changed to
docker login digitalasset-docker.jfrog.io
docker pull digitalasset-docker.jfrog.io/canton-enterprise

but I'm unsure what else needs to change accordingly as I'm wholly unfamiliar with the documentation of the Canton Docker images.


Make anchoring presets an enum instead of being assigned based on ints.

Godot version
4.2 Dev 5
System information
Fedora Linux 38 (Workstation Edition)
Issue description
Having the anchor preset be an ordered  int list  is incredibly obnoxious when trying to set up any UI stuff from code . To make matters worse the corresponding ints don't seem to be in the documentation. And don't even seem to align with the order of the anchor presets in the editor

I appreciate that changing this could be considered a breaking change most likely but it makes the development experience considerably unpleasant. The enum for this seems to exist so i'm inclined to think this was an oversight.

Steps to reproduce
just make a control node in a script and try and give it a preset
Minimal reproduction project
N / A

After a discussion about this on 🚀: Control.AnchorsPreset and Control.LayoutMode aren't supposed to be used. That is why they are undocumented, even tho exposed in the API.
Since we can easily distinguish such "internal use only" methods/properties, I suggested using [EditorBrowsable(EditorBrowsableState.Never)] and [Obsolete("This member exists for internal use only and should not be used.", false)] on them, to have the situation explicitly stated out in IDE tools. But it doesn't seem it'd be a welcomed change.

I also prefer enum a bit more than magic num!

Please take note of the comment by @paulloz. These are undocumented properties which are a part of the editor's internal logic. You are not supposed to use them in scripting, despite code assistance tools suggesting them.
Please prefer using exposed and documented API, such as Control::set_anchors_preset, Control::set_anchors_and_offsets_preset, or Control::set_offsets_preset to set valid presets, or Control::set_anchor, Control::set_anchor_and_offset, and Control::set_offset to set custom values. They are also exposed as properties: anchors and offsets, and go well together with the grow direction settings.
Control.AnchorsPreset and Control.LayoutMode are not safe to use, which is why they are not present in the engine documentation. They will likely disappear one day from the API surface. Alongside about 100 other internal properties wrongfully exposed to scripting (both, GDScript and C#).

With https://github.com/godotengine/godot/pull/87381, internal properties will no longer appear in GDScript and won't show confusing tooltips in the editor. For .NET I think the issue persists for now.


My problem with xbox app

After a while of attempting, il finally found why my ScpVBus was outdated. I simply manually updated it. And now! The program is working pretty well! But in the xbox stream, with the cursor stuck at the middle of the screen. No key are working on the app. Is it my virtual controller or something else ?
Thanks for replying soon!

Does the application turn green when you are inside the xbox app?

Yes it is, and when I'm doing alt-tab, it's turns blue. The program works perfectly fine. But I think it's the virtual controller the problem. What do you think about that?

@FreeXKill blue means standby, red is off and green is on. Does the app turn green when you go inside the app, it needs to be the Original Xbox App not the beta

I have to repair my pc, I'll by able to try in 3 or 4 days. I got a screw in my laptop's fan. Don't even imagine how it's frustrating xD I'll reply in few days :)


 should deal with switching elements.

Right now  does a good job if changing from content to no content & vice versa. It should also handle when the content changes key, and provide some transition between the do.
I'm thinking the old content fades out, new content fades in, meanwhile the height transitions.
This would be useful when changing compressor.

Hey @jakearchibald , I was looking at <Expander> and was wondering if it's worth moving away from animating height, instead opting for a keyframes animation (generated on-the-fly) that transitions transform: scale (like is explained here).
I'd be happy to pick this up if it's something you deemed worth it!


Feature request: Query database systems

Hello,
we are using the OCI Grafana Plugin to visualize metrics for databases and pluggable databases in Grafana. We would like to template our dashboards with variables for databases and database systems so that we can then filter using those variables as dimensions. However, I do not see how to query database systems and databases when creating template variables in Templating Docs.
When creating dashboard query variable using OCI Grafana Plugin, I would expect the Query for database systems variable db_system to be something like:
database_systems($region,$tenancy)

For database variable database:
databases($region,$tenancy)

Then I could reference these variable in the in the raw query like:
CpuUtilization[1m]{resourceType="PDB", instanceName="$database", hostName="$database_system"}.mean()

As a workaround, I was thinking about using resourceId somehow to be more reusable, but I don't see how to query it using Templating Docs. Referencing database systems and databases directly seems more elegant.

Hi. I think instanceName and hostName are dimensions for the database metrics, and not namespaces. You propose a template variable setting which can be done for a namespace (like oci_compute, oci_lbaas, etc) but cannot be done for a dimension. You may want to try to set a dimension template variable where you can choose a specific instanceName for your database metric.

Hello, thank you for quick response.
Yes, instanceName and hostName are dimensions of the database metrics. I need to filter according to these dimensions, where the value is a given by a query variable from the plugin. The value of the query variable depends on currently selected compartment.
If I understood your answer correctly, this is not possible right now. Correct?

Hi. It should be possible, but you need to use a template var for the dimension accordingly. Please refer to the Dimension Template variable in the Using document: https://github.com/oracle/oci-grafana-metrics/blob/master/docs/using.md

Hello @mamorett, thank you for your advice. I tried template variable for dimension and it is what I needed. However, I have one question. If I choose multi-value selection option for my dimension variable and select at least 2 options, the grafana plot breaks. I inspected the query and it looks like this:
CpuUtilization[1m]{{deploymentType=\"vm\",resourceName_dbnode=\"my_db_node\"}}.avg()"

Is this a bug or do I need to create new dimension variable for each dimension I want to filter?


FRI session state changes and disposes in SmartPad

Hi,
Sorry to bother you guys multiple times, but we do have another problem regarding to running the LBRServer.java.
When we run the LBRServer application on KUKA SmartPad, the FRI session state will change to "MONITORING_READY" after some time (around 60s). And soon after this the FRI connection disposes as you can see below in the screenshot.

We are checking the FRI manual from KUKA but have not figured out why it is happening. Please let us know if you have any idea on how to increase the timeout or prevent the FRI session from getting disconnected.
Thanks for your help!

Thanks for your reply! We are still trying to figure out why it is happenning. The cable and connection seem fine.
What we found so far is that, the LBRServer application can keep running if we reboot the KUKA robot (and its controller of course), which is similar to your experience.
Btw may I ask why do you use com.kuka.roboticsAPI.motionModel.BasicMotions.positionHold instead of com.kuka.roboticsAPI.motionModel.positionHold at the line below? Are they the same?
https://github.com/KCL-BMEIS/lbr_fri_ros2_stack/blob/d6d5b4eaa547eba4756add527a37a77c6208fd7c/server/fri_ros2/LBRServer.java#L3

Sorry for replying so late! No there is not specific reason. Will double check tomorrow. Have you used the other one before ? The Java side of the code could definitely use an improvement !

re lost connection. The error here is possibly this line
https://github.com/KCL-BMEIS/lbr_fri_ros2_stack/blob/d6d5b4eaa547eba4756add527a37a77c6208fd7c/server/fri_ros2/LBRServer.java#L157
When an error is thrown here, the error is caught and printed, but not handled. Therefore, dispose is never called. This causes subsequent connections to drop. This needs to be fixed

Sorry for replying so late! No there is not specific reason. Will double check tomorrow. Have you used the other one before ? The Java side of the code could definitely use an improvement !

No worries. Thanks for your hard work!
We have tried both methods a bit but did not notice much difference. I feel like com.kuka.roboticsAPI.motionModel.positionHold is an older version of com.kuka.roboticsAPI.motionModel.BasicMotions.positionHold. Also, it (positionHold) may not be cancelled as well when the FRI connection get disposed as we specify timeout as "-1".

this issue should be resolved now, feel free to re-open in case in occurs again. Please make sure to update LBRServer.java on your controller

Many thanks! Now the FRI session can be properly disposed.


PayConex: Add gateway

https://www.bluefin.com/

Got the encoding issue to work in Ruby 2.0 locally.  Let's see if it's fixed in Travis.


Feature/launch with fix

Only show discourse comments on public page
Will always show launch information if tool/workflow has at least one valid version
If default workflow is invalid, will select the first valid workflow to show launch information

Codecov Report

Merging #117 into develop will decrease coverage by 1.41%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #117      +/-   ##
==========================================
- Coverage    65.01%   63.6%   -1.42%
==========================================
  Files           94      94
  Lines         2524    2624     +100
  Branches       319     345      +26
==========================================
+ Hits          1641    1669      +28
- Misses         712     786      +74
+ Partials       171     169       -2

Impacted Files
Coverage Δ

src/app/workflow/launch/launch.component.ts
61.11% <100%> (ø)
:arrow_up:

src/app/workflow/dag/dag.component.ts
35.11% <0%> (-0.06%)
:arrow_down:

src/app/search/query-builder.service.ts
14.28% <0%> (+1.05%)
:arrow_up:

...rc/app/workflow/paramfiles/paramfiles.component.ts
74.19% <0%> (+12.9%)
: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 d875a84...32d5e28. Read the comment docs.


Possible Issue with CUDA Device and CPU Support in Inference Script

Hello,
I believe there may be an issue with line 45 of the inference script. Specifically, the script is forcing the CUDA device, which may prevent CPU support when passing the argument '--device cpu'. Would it be possible for you to investigate this further?
Thank you.

Already implemented to the code


run as non root user

Please ignore travis-ci.ORG checks. I migrated this repository to travis-ci.COM. Future PRs should not show the .ORG tests


replace missing angular-storage 0.5.0 with a0-angular-storage 0.0.15

May need to do this in other samples' bower.json, too, as it appears angular-storage no longer exists in bower's index. Replacing with a0-angular-storage works.

Thanks for your PR :)
I've elected to take the dep out since the samples aren't using angular-storage.


Problem with color of info icon

Info icon is not visible because it's always white(skip_this_image_button).
Android 6.0.1
Samsung note 4.

This issue is beginner friendly

I am able to see the blue info icon on Android 8.
Is this issue specific to Android 6?
The drawable_tint_color is set to blue in the activity layout, so it should ideally be showing in blue color. However, if the issue still persists, we can change the color of the drawable icon itself?
I am a beginner. Please correct me if I am missing out something.

Oh, thanks for reporting this @madhurijain97 . Can you please make sure tried with both dark and light theme (from settings).

Yes, I tried with both.
The blue info icon shows up in the dark theme too.

I can't reproduce the issue either,

@Szymon20000 can you share the about page screenshot of the device you reproduce this issue?

This seems to be an android 6 specific issue, everything fine here (Huawei WAS-LX1A Android 8 used)

I am a beginner and if this issue is still open, I would like to work on it.

If this issue is still open I would like to work on it.

This is open but someone else already works on this. So please check for a new issue. Looking for beginner friendly label and asking if anyone else working on it is a good practice for newcomers :) Additionally you can check out our documentation before starting.

@thetimeheist76 Did you manage to reproduce the issue? If yes feel free to start working on it, thanks a lot :-)

@thetimeheist76 Sorry I had not seen Neslihan's comment! And the issue should have been tagged as "assigned".

@nicolas-raoul  yes I was able to reproduce the issue. I first tried with Android 7 and it did not raise the issue. Later I tried with Android 5 and was able to reproduce. I'll now start working on it. Thanks a lot.

Was able to resolve the issue. This is how it now looks

Was able to resolve the issue. This is how it looks now. Wasn't able to get rid off

the blue background around the icon though.

Was able to fix the issue. This is how it looks now.

Oh okay, seems like a second PR came about this issue. @PavelAplevich do you still working on this issue?

Is this issue still open?

The issue has been fixed. Thanks all! :-)


Warning new NativeEventEmitter() was called with a non-null argument without the required removeListeners method.

Warning new NativeEventEmitter() was called with a non-null argument without the required removeListeners method.
	React.useEffect(() => {
		RNNotificationCall.addEventListener('answer', () => {
			console.log('press answer');
		});
		RNNotificationCall.addEventListener('endCall', () => {
			console.log('press endCall');
		});

		return () => {
			RNNotificationCall.removeEventListener('answer');
			RNNotificationCall.removeEventListener('endCall');
		};
	}, []);
	const display = () => {
		RNNotificationCall.displayNotification(
			'22221a97-8eb4-4ac2-b2cf-0a3c0b9100ad',
			null,
			30000,
			{
				channelId: 'com.abc.incomingcall',
				channelName: 'Incoming video call',
				notificationIcon: 'ic_launcher', //mipmap
				notificationTitle: 'Aleksandr yurkovsky',
				notificationBody: 'Incoming video call',
				answerText: 'Answer',
				declineText: 'Decline',
				notificationColor: 'colorAccent', //path color in android
			},
		);
	};

React version<EMAIL_ADDRESS>

@godlikeAlex Thanks, can you check the latest version? https://github.com/linhvovan29546/react-native-full-screen-notification-incoming-call/releases/tag/0.1.3

thank you very much! Works! I'm very grateful to you <3


Add Search Utility

Generalizes isin search to support new types of search queries.
Use case:
I would like to be able to keep all yahoo finance code contained in the yfinance lib. Right now to perform a ticker search, I have to perform the following in my application:
    response = requests.get(
        "https://query1.finance.yahoo.com/v1/finance/search",
        params={"q": q, "newsCount": 0, "quotesCount": 5, "enableFuzzyQuery": False},
        headers={
            "User-Agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36"
        },
    )

Related to https://github.com/ranaroussi/yfinance/issues/837

Give an example use code, so people can try out.

Sorry, by example use code do you mean providing a snippet here or adding to the README?
In either case, here's a quick snippet.
import json
from yfinance.utils import search

user_input = input("Enter a ticker query: ")
search_result = search(user_input, newsCount=0)            # Just search for tickers
search_result_quotes = search_result.get("quotes", [])

print(json.dumps(search_result_quotes, indent=2))           # Pretty print

Input
Enter a ticker query: apple

Output
[
  {
    "exchange": "NMS",
    "shortname": "Apple Inc.",
    "quoteType": "EQUITY",
    "symbol": "AAPL",
    ...
  },
  ...
]

I'm thinking how the data can be presented better. What is search_result['quotes']? It can be shoved into a pandas dataframe btw - pd.DataFrame(search_result['quotes']).

Just so I understand the use case correctly - how is this different to Ticker.news? Should Ticker.news be changed to use your code.

This use case targets searching for tickers - the underlying API does also fetch related news articles, but the primary motivation behind this is some kind of ticker search functionality. I should also note that I haven't found anything in the library currently for this use case, but I could be missing something. Hope that clears this up!

https://github.com/ranaroussi/yfinance/blob/930b305327e2e3769b5d62115b3ab25bc58f28de/yfinance/base.py#L502

@ValueRaider, the get_news function currently only returns the news list and excludes the quotes list in the /search response. Having a search utility function would be a valuable addition and filtering the data to show just the quotes would help eliminate redundancy with get_news.

would help eliminate redundancy with get_news

Agree. I wasn't rejecting the idea.


refactor(concurrency): add execution flags struct

This change is

crates/blockifier/src/blockifier/transaction_executor.rs line 92 at r1 (raw file):

        );
        // Executing a single transaction cannot be done in a concurrent mode.
        let excution_flags =

Suggestion:
let execution_flags =


Frends.FTP.UploadFiles task needs better operations log

The operations log needs improvements. For example if timeout exception is thrown when connecting to server, no operations log is created at all. Task only throws exception: Timed out trying to read data from the socket stream!.

https://github.com/FrendsPlatform/Frends.FTP/pull/27


task/93/more-coverage

Codecov Report

Merging #97 (eddcf00) into main (491c8e3) will increase coverage by 1.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
+ Coverage   79.85%   81.35%   +1.50%
==========================================
  Files          56       56
  Lines        1638     1636       -2
  Branches      235      235
==========================================
+ Hits         1308     1331      +23
+ Misses        330      305      -25

Impacted Files
Coverage Δ

client/src/components/ChoiceButton.vue
87.35% <ø> (+8.70%)
:arrow_up:

client/src/store/cards.store.ts
74.35% <0.00%> (+3.84%)
:arrow_up:

client/src/store/user.store.ts
63.82% <0.00%> (+10.63%)
:arrow_up:

client/src/components/CardList.vue
94.73% <0.00%> (+15.78%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Replicating and refactoring style-swin-gan

Zhanxiang (Sean) Sun @seansunn made great implementation of style-swin-gan here https://github.com/budai4medtech/midl2023/tree/main/notebooks/style-swin-gan which will be great to add in the lib to improve reproducibility and potentially refactor few bits!

Some issues to solve:

[ ] CUDA memory > https://github.com/budai4medtech/xfetus/blob/32-style-swin-gan/examples/style-swin-gan/styleswin_tt.ipynb

OutOfMemoryError: CUDA out of memory. Tried to allocate 192.00 MiB (GPU 0; 7.79 GiB total capacity; 6.42 GiB already allocated; 8.50 MiB free; 6.58 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

[ ] checkpoint >https://github.com/budai4medtech/xfetus/blob/32-style-swin-gan/examples/style-swin-gan/styleswin_tc.ipynb

FileNotFoundError: [Errno 2] No such file or directory: 'data/saved_tt_256/d_500.pt'


GEM-139 Added styling to article category links in footer menu

@saeedmb8 @KaitCrawford
This is the solution Indrani and I decided on instead of using a dropdown.

:+1:


suggestion: adding optional import alias

It would be great to have a optional import alias. Imo import alias like a default thing that we have to include to avoid ../../ hell at long term. Default may be @/*, but the user should be able to choose according to their taste at the init stage. Here is example at following:
import Counter from "@/islands/Counter"

We can include it as an option at init. For instance, What import alias would you like configured? › @/*.

I'm not sure if adding more steps to the init script is a good idea. The more decisions we confront first time users with, the more likely we are causing friction in the onboarding process. Not everyone uses these either. Seeing that this is a oneliner that can be copy & pasted directly into deno.json I'm not sure what we'd gain by that.

Closing, because whether folks have an aliased import or not is personal preference. So far there doesn't seem to be a clear winner of either approach. It's easy enough to add if you want it I think.


Export getindex!

Would it be possible to export getindex! defined here? It's very useful.

I suppose we could, but it feels like a function that we maybe should have imported from base, had it existed there. It doesn't (yet), though...
@timholy, what do you say to this?

I've often wanted this from Base. But we can export it even if Base doesn't. The biggest argument against it is that you can do this with copy!(dest, sub(src, indexes...)), except that SubArrays don't take fractional indexes.


reverse_delete_rule fails in ListField(ReferenceField)

for this test case
import datetime
from mongoengine import *
connect('teste')

class Person(Document):
    name = StringField()

class Car(Document):
    owners = ListField(ReferenceField(Person,
                       reverse_delete_rule=NULLIFY))

Person.drop_collection()
Car.drop_collection()

p1 = Person(name="wilson")
p1.save()

p2 = Person(name="maria")
p2.save()

c = Car(owners=[p1, p2])
c.save()

p1.delete()

c = Car.objects.first()
assert c.owners == [p2]

Sorry for the delay, I will review and get a fix in 0.6 ..

Ross, thanks for the reply! Hoping to see this patched soon. :)

Done and in dev - we dont support MapFields / DictFields and will throw an InvalidDocument Error


Performance: Stop using pkg_resources/declare_namespace

Loading pkg_resources and calling declare_namespace caused startup performance to suffer. This change adopts the solution from the azure sdk, where pkg_resources are only used for dev/editable installs and are stripped out when building wheels.
In order to be able to reuse the azure_bdist_wheel.py wheel extension implementation from the azure sdk, we also introduced an additional namespace package (the name of the namespace package is used to determine which init.py files to exclude when building the wheel)
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[N/A] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #2951 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2951      +/-   ##
==========================================
+ Coverage   63.33%   63.39%   +0.06%
==========================================
  Files         466      463       -3
  Lines       26539    26579      +40
  Branches     4069     4081      +12
==========================================
+ Hits        16809    16851      +42
+ Misses       8617     8612       -5
- Partials     1113     1116       +3

Impacted Files
Coverage Δ

src/azure-cli-core/azure/cli/core/__init__.py
100% <ø> (ø)
:arrow_up:

...li-redis/azure/cli/command_modules/redis/custom.py
32.43% <0%> (-7.57%)
:arrow_down:

...c/azure-cli-testsdk/azure/cli/testsdk/preparers.py
68.93% <0%> (-0.16%)
:arrow_down:

...rc/azure-cli-testsdk/azure/cli/testsdk/__init__.py
100% <0%> (ø)
:arrow_up:

...-redis/azure/cli/command_modules/redis/commands.py
100% <0%> (ø)
:arrow_up:

...ault/azure/cli/command_modules/keyvault/_params.py
86.15% <0%> (+0.21%)
:arrow_up:

...vault/azure/cli/command_modules/keyvault/custom.py
67.73% <0%> (+0.35%)
:arrow_up:

...-cli-role/azure/cli/command_modules/role/custom.py
33.1% <0%> (+1.14%)
:arrow_up:

...li-role/azure/cli/command_modules/role/commands.py
86.66% <0%> (+2.22%)
:arrow_up:

src/azure-cli-testsdk/azure/cli/testsdk/base.py
68.42% <0%> (+2.51%)
:arrow_up:

... and 3 more

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 86c5414...4f215ff. Read the comment docs.

@johanste LGTM but didn't clone the branch. If you want I can clone it tomorrow and build to check everything is fine

@lmazuel, we would very much appreciate you cloning & building. We do intend to merge today.


There is no response when clicking "Click to view resource" in "Azure: Activity Log" window

OS: Win10
Build Version: Azure Function v1.9.1-alpha.0 + Azure Resources v0.5.7-alpha.0
Repro Steps:

Right click Function App node to create a Function App.
Click "Click to view resource" in "Azure: Activity Log" window.

Check whether the new created Function App is focused or not.

Expect:
The new created Function App is focused.
Actual:
There is no response when clicking "Click to view resource".

Need to implement reveal so this works.
Upstream: https://github.com/microsoft/vscode-azureresourcegroups/issues/479


Uncaught TypeError: undefined is not a function in 1.0 API Preview Mode

This issue does not happen when Atom isn’t launched using the preview mode. It is also not shell dependent, as I’ve tried all possible variations (Atom 1.0 API Preview Mode, bash/zsh/fish).
Step Reproduction

Run atom --one in a directory.
Run atom-script on the file.
Get the “Uncaught TypeError...” alert.

Atom Version: 0.198.0 :warning: in 1.0 API Preview Mode :warning:
System: Mac OS X 10.10.3
Thrown From: script package, v2.21.0
Stack Trace
Uncaught TypeError: undefined is not a function
At /Users/David/.atom/packages/script/lib/script-view.coffee:236

TypeError: undefined is not a function
  at ScriptView.module.exports.ScriptView.run (/Users/David/.atom/packages/script/lib/script-view.coffee:236:10)
  at ScriptView.module.exports.ScriptView.start (/Users/David/.atom/packages/script/lib/script-view.coffee:120:6)
  at ScriptView.module.exports.ScriptView.defaultRun (/Users/David/.atom/packages/script/lib/script-view.coffee:85:6)
  at atom-workspace.subscriptions.add.atom.commands.add.script:run (/Users/David/.atom/packages/script/lib/script-view.coffee:37:25)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:238:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:519:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:354:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:178:20)

Commands
     -0:00.6.0 script:run (atom-text-editor.editor.vim-mode.command-mode.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "linter-ruby",
      "symbols-view",
      "wrap-guide",
      "space-tab"
    ],
    "followSymlinks": true
  }
}

Installed Packages
# User
atom-ctags, v3.2.0
atom-pair, v1.1.5
autocomplete-plus, v2.13.1
language-fish-shell, v1.0.2
language-haml, v0.20.0
language-scheme, v1.4.0
language-slim, v0.4.0
language-swift, v0.4.0
linter, v0.12.2
linter-flow, v0.1.4
linter-scss-lint, v0.0.14
rspec, v0.3.0
script, v2.21.0
tabs-to-spaces, v0.10.0
vim-mode, v0.46.0

# Dev
No dev packages

Oh boy, can definitely reproduce. Thanks for the heads up.

@davidbarksy Thanks even more for the reproduction steps. I forgot what those looked like for a while.

Yeah... we're not using the right messaging API, which effects Atom achievements too.
atom.emit 'achievement:unlock', msg: 'Homestar Runner'

Deleted the offending code in #436.

@rgbkrk @erran happy to help! :+1:


Create berklee.txt

Please add Berklee College of Music, Boston. MA
Regards,

@tavikano Please rename  lib/domains/berklee.txt to  lib/domains/edu/berklee.txt if you wish to add  berklee.edu domain. Thank you.

@tavikano You might miss my previous comment. Please react.

@tavikano the file lib/domains/edu/berkelee.txt already exists. It means that your school is already in this repository. I am closing your request as a duplicate. Shall you have any concerns, please reopen this request and comment. Thank you.


make it work with XCode 11 Beta 6

Example App works again with XCode 11 Beta 6
Just add
    public func send(subscription: Subscription) {
    }

That is required now https://developer.apple.com/documentation/combine/subject/3362858-send

Thanks Kevin! Can you confirm this is needed for the GM?

I don't understand what you means by GM 😄

So GM is for Golden master
i haven't tested yet on it. I will do it ASAP and comment and/or update this PR if needed !

Interesting trivia... The first three letters of the greek alphabet... α β ɣ (alpha, beta, gamma) are the three major releases of a software product. Over the years, "gamma" has transformed "gold master candidate", but for some reason "beta" and "alpha" haven't changed. :-)

Love it :)) I’m a fan of trivia

Fixed it here with some other minor things. Thank you!


[sailfish-secrets] Call setFuture() in daemon after signal connection. Contributes to JB#36797

There is a risk of race condition when connecting the finished signal after launching the thread with setFuture(). This is documented in the Qt doc.
I agree that the risk is quite neglectible here since the thread may have some work to do and may not return instantaneously, but it's safer by calling the setFuture() after the connection.

Ok, corrected.


Feature - Clean the GitHub Page and put some content there

Problem Related
We need some space on GitHub to render all the info we want to show to the users. This space must respect the user's interface, so it must only be activated through a button.
Describe the solution you'd like
When clicking at the button described on #25, clean all the content below tabs and put our badges there. When clicking at another tab, the content must be restored.
Additional context
To have a better understanding of how and why it is needed, see Vision Document
Acceptance Criteria

[x] Clean page when clicking button
[x] Put badges on cleaned space
[x] Restore content when clicking at another tab
[ ] Test coverage at 80%

To solve this issue we need the button to put the click action to delete the elements of the page. Issue # 37 proposes to create the button in a new place of the screen, as there is no button yet what should I do, since the method to clean the page is ready

The button is there. It was created with #33.
Just create the script to clean the screen. When #37 is being done, just transfer the created method to the new button.

We were able to create a method that removes the elements from the screen and inserts the badges in place. The only problem we found was that we need to open the page in a new tab, since chrome does not allow injecting JS code into screen elements.

I was able to create the click event to remove elements from the screen like this:
var hubbutton = document.getElementById('hubcare-button');
hubbutton.addEventListener("click", function() {
    cleanPageContent()
}, false);


initial commit for ultimate renderer dimensions solution

According to #3641, we really have a problem. Not even core developers know what does "width" or "height" mean.
Proposition:
screen.width and screen.height are used for the stage
view.width and view.height are size of canvas, thus they are measured in real pixels
If this one will be approved, I will move forward to
ultimate resize logic: resizeView, resizeScreen, resizeCss with autoUpdate tying them to each other. resizeBack for overriding by canvas&webgl. onResize event/signal/whatever
possible accessability fixes, I just saw a number of uses of "renderer.width" there which aren't that clear
jsdocs for all "width"s and "height"s, explaining which are actually screen coords and which are physical pixels. Most of them will be screen'ed, of course.
possible deprecation of "renderer.width", "renderer.height", just to be clear. I won't press for that, but may be team decides so
EXAMPLES FOR ALL THE POSSIBLE CASES of autoUpdate, setting width, height and resolution logic.

Build is here: http://pixijs.download/dev-screen-width/pixi.js

I would suggest we deprecate SystemRenderer.width and SystemRenderer.height getters. These can easily be added to src/deprecation.js and provide pass-thru for this.view.width and this.view.height. Per our earlier discussions, this would help to further clarify the view size vs the screen size.

I don't know what to do about "renderer.width/height", deprecation is kinda a big move. I was thinking something about SETTINGS, may be we can add a switch there, whether or not its VIEW or SCREEN.
Also, people are using it too often, if we deprecate it only one message should be shown.

@bigtimebuddy you can commit deprecation into that branch, I dont mind

I thought about this some more, and I think you're right @ivanpopelyshev, it's a radical change. Let's leave it as-is for now until someone else feels strongly. Maybe we can revisit in future PRs.

@bigtimebuddy yeah, its for v5.


add nvim-nvic and configure it for languages supported by lspconfig

nvim-navic

Potentially available languages:

C/C#/C++
Rust
Go (needs lsp setup)
Js/Ts (needs lsp setup)

nvim-navic is a dependency for barbecue.nvim, likely will be functional once I manage to implement barbecue

being tracked by #95


comment in a read-only Pad?

How do you think? Could it be possible to comment in a read-only Pad?
They are stored to database, too
key:
comments:padID

value:
"commentId":{ "author": data.author, "name": data.name, "text": data.text, "changeTo": data.changeTo, "changeFrom": data.changeFrom, "timestamp": new Date().getTime()}

(commentManager.js, line 62, right?)
But in
key:
pad:padId

there won't be the comment (commentId), so after reload the pad the comment won't be shown.
Is that the consciously behavior? Does anyone know a way / fix to save comments on read-only pads?

You can already comment in a read only pad afaik.

Or is it you can see them but not reply?

After press on "Comment" the input is visible like in the editable view - but after reload no more. In the database I also can't find the given input. Deletion comments in read-only mode (saved in editable mode) will also not working persistently.
It's a uncommon behavior on my system, right?

I swear I added support for this..  Did you checkout latest version?

I added via "npm install ep_comments_page" version 0.0.32
after reinstall via "git clone" it's unfortunately the same behavior

Mheh, will take a look tonight or tomorrow.

https://github.com/JohnMcLear/ep_comments/pull/39

I'm not gonna get time to look at this now until Late August, apologies!

Hey, how would you fix this? Maybe one of us can try.

Just allow comments through as an allowed message type and add the comments container to the read only view.

If you say that, it sounds so easy...
I have no idea how I can define comments as allowed.

@gulaschskanone I'm available as a gun for hire if you want to get this done sooner than I get spare time?

Current (ep: 1.8.3, ep_comments: 0.1.0) behavior is like this: Comments can be added on the pad as well as on the read-only pad, but the contents of the comments cannot be seen in the other version. The corresponding regions are highlighted in both though.
Would be nice if one could also see the contents of the comments in both, pad and read-only pad, regardless of the origin of the comment.

@thmo it's probably not a huge job to fix, assuming this is for a commercial project did you want to setup a bounty so we can get more eyes on it else it's gonna just fall naturally on our roadmap.

Just found a real bug: You can make and later accept a suggestion in a read-only pad, resulting the original pad text to be changed.
(No, it's not for a commercial project.)

Ah okies, well when we're doing our next big push on ep_comments we will try address these bugs.  Fair point RE the security issue, that should be getting rejected!

Is there any activity on this? Or can someone point out where the issue could be? I might be able to come up with a fix.

Take a stab!

I did some more testing and my PR did seem to resolve the issue completely. I have to take another look.
Right now live syncing does not happen, only after refresh. And the readonly side cannot make new comments but only comment an existing one.

Hello, I can make comments in readonly pad but they disappear after the refresh. Is this an expected behaviour? Thanks!


--follow-exec causes segfault on GitHub Actions

I'm trying to set up & configure tarpaulin under GitHub Actions with Codecov.  At first, the only problem was that my binary crate wasn't marked as covered despite being tested by an integration test; then, I found the --follow-exec option.  Unfortunately, if I pass this option to tarpaulin (using v0.20.1 on Rust stable, beta, & nightly), it segfaults when running the binary crate tests.
Here is my GitHub Actions workflow in which I run tarpaulin:
name: Test

on:
  push:
  pull_request:
  schedule:
    - cron: '0 0 1 * *'

jobs:
  test:
    runs-on: ubuntu-latest
    strategy:
      fail-fast: false
      matrix:
        toolchain:
          - stable
          - beta
          - nightly
    steps:
      - name: Check out repository
        uses: actions/checkout@v3

      - name: Configure Git identity
        # Needed for a couple tests
        run: |
          git config --global user.name 'GitHub Actions'
          git config --global user.email<EMAIL_ADDRESS>
      - name: Install Rust
        run: |
          rustup update ${{ matrix.toolchain }}
          rustup default ${{ matrix.toolchain }}
          cargo install cargo-tarpaulin

      - name: Test with coverage
        run: cargo tarpaulin --out Xml --exclude-files "tests/*" --follow-exec

      - name: Upload coverage to Codecov
        uses: codecov/codecov-action@v3
        with:
          fail_ci_if_error: false

and here is a link to the segfaulting runs: https://github.com/jwodder/ghrepo-rust/actions/runs/2632179108.

Could you try tarpaulin 0.19.1 and report if it works with that? --follow-exec changed a lot between 0.19.x and 0.20.x which removed a bunch of deadlocks and other segfault issues some people had but I'm wondering if it's caused a regression on some previously working projects :eyes:

@xd009642 Version 0.19.1 works.


OmniSharp giving explicitly disabled warnings

Issue Description
I'm working on a legacy project, which was written for an old .NET Framework version and thus doesn't support newer language features such as simple using statements and target-typed new() expressions.
Already tried not overriding the MSBuild being selected for this project, but it doesn't change the warnings still getting emitted.
Still, mandatory snippet from the startup log:
[warn]: OmniSharp.CompositionHostBuilder         It looks like you have overridden the version of MSBuild with a version lower than 16.3 which is the minimum supported by the configured .NET Core Sdk.  Try updating your MSBuild to version 16.3 or higher to enable better .NET Core Sdk support. [info]: OmniSharp.MSBuild.Discovery.MSBuildLocator         Registered MSBuild instance: Proper MSBuild 15.9.21 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\MSBuild\15.0\Bin"
Steps to Reproduce
Open old .NET 3.5 Framework project, turn the relevant warnings off and do something like
using (OleDbCommand updateCmd = new OleDbCommand(query, conn))
{
    int res = updateCmd.ExecuteNonQuery();
    if (res < 0)
    {
        LogDbErr(update, somedata, LogLevel.Error);
    }
}

That produces both of the info flags at once.
Just doing OleDbCommand updateCmd = new OleDbCommand(query, conn) on a line of its own makes it present the new expression can be simplified warning just as well.
In the .editorconfig I put in the root of the folder I set the project formatting options and they all work, but there's two entries being ignored:
csharp_prefer_simple_using_statement = false
csharp_style_implicit_object_creation_when_type_is_apparent = false

Expected Behavior
No warnings about disabled new language features because checking for them is disabled.
Actual Behavior
Get a lot of noise in my warnings with the report state of the two warnings is set to false.
Also tried false:none and false:silent to no effect
Environment information
VSCode version: 1.65.2
C# Extension: 1.24.1
Dotnet Information
.NET SDK (reflecting any global.json):
 Version:   6.0.201
 Commit:    ef40e6aa06
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19044
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.201\
Host (useful for support):
Version: 6.0.3
Commit:  c24d9a9c91
.NET SDKs installed:
6.0.201 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

auto-close-tag
formulahendry
0.5.14

cpptools
ms-vscode
1.8.4

csharp
ms-dotnettools
1.24.1

EditorConfig
EditorConfig
0.16.4

msbuild-project-tools
tintoy
0.4.3

remote-containers
ms-vscode-remote
0.224.3

remotehub
GitHub
0.26.0

vscode-docker
ms-azuretools
1.20.0

vscodeintellicode
VisualStudioExptTeam
1.2.17

xml
DotJoshJohnson
2.5.1

Omitted a couple extensions like Rust, Python and Perl stuff, since they error and exit on this office machine that doesn't have my dev env installed

@333fred Could you have a look on this one, please?

Update:
Mangling the .csproj file and adding the <Langversion> and explicitly setting it to 7.3 seems to be resolving the issue. This indicates that either OmniSharp or Roslyn isn't processing the <TargetFrameworkVersion> the same way as when running the compiler.
Seems like the original project is also old enough that <LangVersion> was either not included by default, or did not yet exist. I have no idea when that tag is from though.

Closing due to lack of activity. If you are still encountering problems, please open a new issue at https://github.com/omnisharp/omnisharp-roslyn/issues

So let me get this straight, I post a workaround and a best guess where the issue is coming from, there are  no referencing commits or any indication there's a change, and it gets closed? The issue still exists and the lack of activity is and, as far as I can tell, nothing relevant has changed

@RivenSkaye Sorry, since you had posted the workaround, I had assumed you were unblocked. This scenario is very niche and we have not had any other reports about it. Being a small team (even inside a big company) mean that our resources are limited and we always have to prioritize which bugs we can fix.
I believe your workaround is actually the solution here. Without specifying the language version, the Roslyn that we ship will default to a higher version than 7.3. When building with the older MSBuild everything works out because the version of Roslyn included with it likely matches the 7.3 language more closely.
There is no direct mapping between TargetFrameworkVersion and LangVersion. In many cases you can use newer language features when targeting older frameworks so long as they do not require an updated runtime.

Thanks for the explanation, that actually makes sense. The closure reason just struck me as very odd due to the combination of no references and the last activity being mine. Perhaps this makes a good documentation point either here or upstream in the MS tooling docs?

PS: do those lint configs behave properly on newer projects? I've never had a use for them before, so I couldn't say

They do although there is one stumbling block when configuring them. Each rules severity can be configured by the rule's name, csharp_prefer_simple_using_statement = true:warning for instance, and by its id, dotnet_diagnostic.IDE0063.severity = error in this case. The severity configured by id takes precedence which can catch users by surprise.


Is there a callback to detect buffering?

Hi,
Similar to react-native-video, is there a callback when stream is buffering?

Yes, you can tryonPlaybackStalled and onPlaybackResume .

Thanks @NikolaBorislavovHristov !


hello-world: inserts sample solution at top of test runner

The information at the top of the file feels kludgy to me, but I am not sure how to spruce it up.

fixes #99

Let's bring it in.  As people see it, and make comments, on it, we will notice.  The comment count on Hello exercise will be higher than "normal", with some yet to be seen level of "normal".  We also link, hopefully clearly, to the gitter room, and to issues here.  And so it is unlikely that it will not be brought to attention if it is unclear or confusing.


Easy way to center the button?

Hi! I love how simple your library is. My issue is that I'm not using the button for a menu, but just as an appearing and disappearing floating button. I would like to have it bottom-centered, but because of the set ENUM, I can't figure out how to center it. Would this just need to have a new case for bottom center or is there something I can do without modifying the code?

That would be great!


fix: move error.css to public folder

Fixes #2599
Describe the changes you have made in this PR -
moved error.css to public folder
Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Great, Thanks


SCO references using objects vs IDs cause different deterministic ID generation

As shown in the following example:
import stix2.v21

pd = stix2.v21.Directory(path="/path/to")
f_obj = stix2.v21.File(name="foo", parent_directory_ref=pd)
f_ref = stix2.v21.File(name="foo", parent_directory_ref=pd.id)

print(f_obj.serialize(pretty=True))
print(f_ref.serialize(pretty=True))

outputs:
{
    "type": "file",
    "id": "file--bc403dd7-af2c-588b-a24e-d0f5150202e1",
    "name": "foo",
    "parent_directory_ref": "directory--b2d85bf6-7b9e-5de3-af38-0f856e7009ef",
    "spec_version": "2.1"
}
{
    "type": "file",
    "id": "file--f7de0951-c047-5f7d-bcc1-4075a44821f1",
    "name": "foo",
    "parent_directory_ref": "directory--b2d85bf6-7b9e-5de3-af38-0f856e7009ef",
    "spec_version": "2.1"
}

Note that the supposedly deterministic IDs for the two serialised File SCOs are different. This is caused by the handling of instances of _STIXBase in the kwargs passed to _STIXBase._generate_id(). When (in this example), the parent directory SCO is passed to _generate_id(), it makes a deep copy  and adds that to the streamlined_obj_vals list which is canonicalised on a later line (which is also problematic, see #395). The deep copy of the STIX object being referenced is obviously not equivalent to the serialised reference itself, resulting in the variance in the generated ID for each object.
This can could presumably be naively fixed by making _generate_id() aware of ReferencePropertys (which I'd suggest be done since it's an easy win), but that would still leave open a potential issue where a ReferenceProperty is nested within some other ID contributing property. As a concrete example, since File SCOs include their extensions as an ID contributing property, a ReferenceProperty nested within an extension (e.g. a custom extension constructed similarly to my example code in #395) would be similarly mishandled.

Here's an example of the nested identifiers using archive-ext:
import stix2.v21

ad = stix2.v21.Directory(path="archived/path")
f_obj = stix2.v21.File(name="foo", extensions={
    "archive-ext": {
        "contains_refs": [ad, ],
    }
})
f_ref = stix2.v21.File(name="foo", extensions={
    "archive-ext": {
        "contains_refs": [ad.id, ],
    }
})

print(f_obj.serialize(pretty=True))
print(f_ref.serialize(pretty=True))

Outputs:
{
    "type": "file",
    "id": "file--7e26ad71-b261-5289-a8e3-3d7126321233",
    "name": "foo",
    "extensions": {
        "archive-ext": {
            "contains_refs": [
                "{\n    \"type\": \"directory\",\n    \"id\": \"directory--c1af83bc-164d-546d-8a73-207f7ad87324\",\n    \"path\": \"archived/path\",\n    \"spec_version\": \"2.1\"\n}"
            ]
        }
    },
    "spec_version": "2.1"
}
{
    "type": "file",
    "id": "file--916b26ba-65fd-5d95-95d7-866d24c9f98e",
    "name": "foo",
    "extensions": {
        "archive-ext": {
            "contains_refs": [
                "directory--c1af83bc-164d-546d-8a73-207f7ad87324"
            ]
        }
    },
    "spec_version": "2.1"
}

Interestingly, this also exposes another handling bug where the referenced Directory isn't rendered by its ID! So the deterministic IDs for the Files are different, and the serialised File which referenced the Directory object was incorrectly serialised.

Thanks for all these fixes, @clenk and @chisholm . Much appreciated!


Documentation: alert.rules. fix labels bug

Hi~
When I used the default alert rule files to do the alert job, I found some small bugs. Here is the fix and tested on my environment.

@heyitsanthony will do

I happend to see prometheus-operator with wrong configuration too . https://github.com/coreos/prometheus-operator/blob/master/contrib/kube-prometheus/manifests/prometheus/prometheus-k8s-rules.yaml

I will fire a pr to fix this same bug in prometheus-operator

CI failure is irrelevant. Merging now. We will fix CI soon.


Missing state parameter in OAuth 2 authentication

A 3rd party security researcher noted that Falcon does not use a "state" parameter when performing OAuth 2 authentication. This makes it vulnerable to login CSRF, which allows an attacker to force a user to log in as the attacker by having them visit a link (which could be done without their knowledge via an <iframe>, etc).
Sample link: http://localhost:9494/oauth2/callback#access_token=tCnHmxkeZxU2FVN6mMM83W1ZRUpEpr&token_type=Bearer&state=&expires_in=31622400&scope=read+write+create (clicking this will add the vic.tim Plotly Cloud account to your Falcon).
Since the account used (and grid updated) by Falcon is controlled by the streambed frontend, there's no actual security vulnerability here but this is a missing best practice, and if we change the way information flows between streambed and Falcon it would be easy to introduce a vulnerability.

@tarzzz, would you be able to take on this issue?


Improve unsafe hygiene

Should contain all of, and only the desired changes discussed in https://github.com/faern/oneshot/pull/17. If there are any additional changes desired, I'll do them on this branch and deal with merge/rebase conflicts after this is merged.

Awesome! I think everything is addressed now and ready for merge. I'll have another look at the code, and hopefully merge a bit later today.

@Cassy343 Do you want to clean up/rewrite the git history? Otherwise I'll merge this after I've had some food :)

You can go ahead and squash and merge, I always worry I'll mess something up when I edit git history so I'd rather not haha.

Thanks for the contribution! Please come with feedback on the changelog entries I added for this: https://github.com/faern/oneshot/commit/3b27d35104845d9c76fabd6be1859c56377b57f3.
Looking forward to the remaining fixes and improvements :)

The changelog entries look good to me!

Please have a look at this comment: https://github.com/faern/oneshot/pull/18/files#r922725443. Am I missing something here, or is the test not testing what it's supposed to test?


Dynamic Lifecycle Injection

Start Date: 03-05-2019
Target Major Version: 2.x & 3.x
Reference Issues: N/A
Implementation PR: N/A

Summary
Introduce APIs for dynamically injecting component lifecycle hooks.
Basic example
import { onMounted, onUnmounted } from 'vue'

export default {
  beforeCreate() {
    onMounted(() => {
      console.log('mounted')
    })

    onUnmounted(() => {
      console.log('unmounted')
    })
  }
}

Motivation
In advanced use cases, we sometimes need to dynamically hook into a component's lifecycle events after the component instance has been created. In Vue 2.x there is an undocumented API via custom events:
export default {
  created() {
    this.$on('hook:mounted', () => {
      console.log('mounted!')
    })
  }
}

This API has some drawbacks because it relies on the event emitter API with string event names and a reference of the target component instance:

Event emitter APIs with string event names are prone to typos and is hard to notice when a typo is made because it fails silently.

If we were to extract complex logic into external functions, the target instance has to be passed to it via an argument. This can get cumbersome when there are additional arguments, and when trying to further split the function into smaller functions. When called inside a component's data() or lifecycle hooks, the target instance can already be inferred by the framework, so ideally the instance reference should be made optional.

This proposal addresses both problems.
Detailed design
For each existing lifecycle hook (except beforeCreate), there will be an equivalent onXXX API:
import { onMounted, onUpdated, onDestroyed } from 'vue'

export default {
  created() {
    onMounted(() => {
      console.log('mounted')
    })

    onUpdated(() => {
      console.log('updated')
    })

    onDestroyed(() => {
      console.log('destroyed')
    })
  }
}

When called inside a component's data() or lifecycle hooks, the current instance is automatically inferred. The instance is also passed into the callback as the argument:
onMounted(instance => {
  console.log(instance.$options.name)
})

Explicit Target Instance
When used outside lifecycle hooks, the target instance can be explicitly passed in via the second argument:
onMounted(() => { /* ... */ }, targetInstance)

If the target instance cannot be inferred and no explicit target instance is passed, an error will be thrown.
Injection Removal
onXXX calls return a removal function that removes the injected hook:
// an updated hook that fires only once
const remove = onUpdated(() => {
  remove()
})

Appendix: More Examples
Pre-requisite: please read the Advanced Reactivity API RFC first.
When combined with the ability to create and observe state via standalone APIs, it's possible to encapsulate arbitrarily complex logic in an external function, (with capabilities similar to React hooks):
Data Fetching
This is the equivalent of what we can currently achieve via scoped slots with libs like vue-promised. This is just an example showing that this new set of API is capable of achieving similar results.
import { value, computed, watch } from 'vue'

function useFetch(endpointRef) {
  const res = value({
    status: 'pending',
    data: null,
    error: null
  })

  // watch can directly take a computed ref
  watch(endpointRef, endpoint => {
    let aborted = false
    fetch(endpoint)
      .then(res => res.json())
      .then(data => {
        if (aborted) {
          return
        }
        res.value = {
          status: 'success',
          data,
          error: null
        }
      }).catch(error => {
        if (aborted) {
          return
        }
        res.value = {
          status: 'error',
          data: null,
          error
        }
      })
    return () => {
      aborted = true
    }
  })

  return res
}

// usage
const App = {
  props: ['id'],
  data() {
    return {
      postData: useFetch(computed(() => `/api/posts/${this.id}`))
    }
  },
  template: `

        Loading...

        {{ postData.data }}

        {{ postData.error }}

  `
}

Use the Platform
Hypothetical examples of exposing state to the component from external sources. This is more explicit than using mixins regarding where the state comes from.
import { value, onMounted, onDestroyed } from 'vue'

function useMousePosition() {
  const x = value(0)
  const y = value(0)

  const onMouseMove = e => {
    x.value = e.pageX
    y.value = e.pageY
  }

  onMounted(() => {
    window.addEventListener('mousemove', onMouseMove)
  })

  onDestroyed(() => {
    window.removeEventListener('mousemove', onMouseMove)
  })

  return { x, y }
}

export default {
  data() {
    const { x, y } = useMousePosition()
    return {
      x,
      y,
      // ... other data
    }
  }
}

in vue2
<Child @hook:mounted="doSomething"></Child>

parent components can listen to its children's’ lifecycle hooks. especially for third-party components.
What should we do in VUE3?


Iris chunking policy for netcdf files - new fix applied

While writing CMOR-ization scripts for the FLUXCOM data set I encountered a problem  - The program hung each time I tried to access the data (for writes and saves). After consulting with @valeriupredoi  and @mattiarighi  I posted this issue on the SciTools issue page. It looks like the problem was with chunk size in netcdf files. I am reporting here two issues (threads) from there which clarify the fix applied for the problem. It appears as though the fix was applied to v2.3.0 but the esmvaltool distribution links to iris v2.2.1?
I just wanted to bring this to the core developers' attention and also ask what I should do for this fix to propagate through to my environment for esmvaltool on JASMIN.
https://github.com/SciTools/iris/pull/3361
https://github.com/SciTools/iris/issues/3362

Note this comment in issue SciTools/iris#3362.
The fix for this issue is PR SciTools/iris#3361 which will be included in iris release 2.3.0, scheduled for release on 30 August, see https://github.com/SciTools/iris/milestones

awesome, cheers a bunch @bjlittle - we are breathlessly looking forward to that 2.3.0 :grin:

@rswamina Does the iris 2.3 release solve this issue?

We'll be cutting iris 2.4.0 soonish (the last to support both python2+3)
It would be good to know whether your issues are resolved...

cheers for the heads up @bjlittle - looking forward to it :beer: @rswamina it'd be good to test with the current 2.3.0 :beer:

Will test and check with the pre-chunked data.

@rswamina any news here?


Backport CAMEL-20410

TODO:

[x] include #13269

I'm marking to draft because it's probably good to wait for the website to finish building one more time (including the last set of changes) before merging this.

The website built without problems. Marking this one as good to review.


How to change the object encoding dimension for rendering

Hi, could you please share how to extend the object identity encoding rendering by enlarging the encoding dimension. I have tried directly changing the macros NUM_OBJECTS in submodules/diff-feature-gaussian-rasterization/cuda_rasterizer/config.h from 16 to 64. However, such error occurs during setting up this changed rasterization package.

So could you please share what else should I to enlarge the object encoding dimension for rendering, thx!

Hi, this problem comes from too much shared data for rasterization. You can reduce the block size to avoid this problem. Note that you also need to change the corresponding dim in this line.
In cuda rasterization, every channel is operated in a for-loop. So increasing the dimension may reduce speed for rendering a lot. We use 16 to balance performance and speed. You can choose a suitable dimension to experiment with.

Thank you for your reply. I'll have a try on it.


App behaviour when offline

When the app has no internet access, the behaviour should be rather clean, or login should be disabled
see @OFFLINE markers

@rg911 @bassemmagdy wallet has now offline mode. Could we close the issue?


Tips & Tricks refresh

Completely refreshed the Tips & Tricks section.
Merged Troubleshooting content into Tips & Tricks, incorporating the single backtrace instructions and added redirects.
Created a new button theme component for this section and for implementing cleaner button links in the documentation moving forward.

I have two different styles for the buttons under Telemetry & Explorers: a primary style and a secondary variant. Currently, I'm using the primary style for in-house tools and the secondary (white) style for third-party explorers. Do you like this approach, or do you prefer using all primary buttons or all secondary (white) buttons?

@vexr I was just about to resolve the conflicts, perfect timing :)


Use fragment shader instead of compute shader in mobile

On the mobile renderer it turns out that compute shaders are much more limited then on desktop. Some of the techniques we currently implemented as compute shaders therefor do not work.
This PR ensures that on the mobile renderer we're using fragment shaders to perform these effects while on the clustered renderer we are using compute.
The effects implemented in this PR are:

DOF
Auto exposure
Glow

Right now I've add a new function guassian_glow_fragment and changed EffectsRD so it only initialises the blur shader if the mobile renderer is enabled and disables the effected compute shaders. This prevents compiling shaders we're not actually using.
Alternatively however it would be possible to create a base implementation and subclass it for the clustered and mobile renderer.
Potentially overkill.

Glow is basically using the logic from Godot 3 in the mobile renderer:

Depth Of Field is currently implemented as a variation on the logic in Godot 3 but with near and far passes combined so instead of 4, we only do 2 passes

Reduz suggested trying to port the Godot 4 compute version to a fragment shader which I'll try as well but i have a feeling this one will actually be faster.

After a lot of testing the Godot 3 DOF shader ported to Godot 4 turned out to have a lot better performance then attempting to convert the newer compute version to a fragment shader so we decided to stick with the older approach. I may put more effort into seeing if I can get the newer version to work better in a separate PR as it does give a nicer result.
I don't have a good test scene but auto exposure is working now as well.

@reduz did all the renames, agree its a better name

Thanks!


Request to add Zonal ARM template link to the document

It will be really good and handy. If we can add the entire ARM template link or template in the document "instead of the sample part of the ARM"

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

ID: d05795f9-5027-e37d-7862-29c72f5c4edb
Version Independent ID: 7ccc48be-64dd-10ed-a0bf-e0c90db04bed
Content: Availability Zone support for App Service Environments - Azure App Service Environment
Content Source: articles/app-service/environment/zone-redundancy.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@abdulrm Thanks for the feedback! We are currently investigating and will update you shortly.

@abdulrm Thanks! Your feedback has been shared with the content owner for further review.

Zone redundant parameter has been added to the article: https://docs.microsoft.com/en-us/azure/app-service/environment/create-from-template
#please-close


updated logo sizing

Formatted informedk12 and operation code logo

✅ By submitting this PR, I have verified the following

[ ] Checked to see if a similar PR has already been opened 🤔
[ ] Reviewed the contributing guidelines 🔍
[ ] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥

@fvcproductions some of the logo images that I found on Google are pretty large for Operation Code. Would you recommend switching out the image URL to another one or using styling?

@YourMobileGeek Switching out the images would be easiest. If the image still doesn't look good - let us know and we can upload one to Imgur for you to use 😄

@fvcproductions yea, some of the newest images I found don't look good either being a bit blurry or large. If you're able to upload it to Imgur that would be great!
Here's the links for Informedk12 and Operation Code:

InformedK12 : https://www.informedk12.com/hubfs/Logo%20%5BJune-17%5D/mark-rgb-isolated.png?t=1534880822719
Operation Code: https://operationcode.org/static/media/large-stacked-logo.2e44.pnghttps://operationcode.org/static/media/large-stacked-logo.2e44.png

@YourMobileGeek Can we add these two images locally instead of uploading them through Imgur for optimization purposes through ImgBot? 😄

@YourMobileGeek Both images are now ready to go! 😄

@fvcproductions sweet, thanks for sending these over!

@fvcproductions would that require me creating another pull request specifically to a deployment branch or just doing a git fetch upstream ?

@fvcproductions I went ahead and updated the listed changes mentioned earlier that was needed. I also did a npm run build, not sure if this helps with the deploy preview

@YourMobileGeek npm run build isn't needed for local development but instead does what the Netlify check above does and build the prod (not dev) version of the website.

@YourMobileGeek Your PR is so almost done! One thing I noticed is that the Operation Code and InformedK12 images linked earlier aren't showing up. Otherwise, this should be good to go!

@fvcproductions I'll look into those updates now, definitely want to make sure everything is accounted for before being approved. When viewing the deploy preview links it's currently showing the updated operation code & informedk12 logo images.
https://deploy-preview-184--techqueria.netlify.com/sponsors/
https://deploy-preview-184--techqueria.netlify.com/resources/open-source/

@fvcproductions Sounds good & likewise! Definitely is a bit of a learning curve for me but I appreciate allowing the time and patience for me to get it done!


SplitPageLayout API

Summary
This PR includes documentation for the proposed API of the SplitPageLayout component.  I'm looking for feedback on the API before I begin implementation.
Here's a preview of the SplitPageLayout documentation: https://primer-48c11ddef3-13348165.drafts.github.io/SplitPageLayout
Here's a diagram of the proposed API:

(FigJam file)
Note about responsive props
SplitPageLayout adopts the responsive prop API that @vdepizzol proposed in this ADR.
Any prop marked "responsive" will optionally accept an object mapping viewport ranges to prop values. Example:
// Use "line" divider for all viewport ranges
<SplitPageLayout.Header divider="line">

// Use "line" divider on regular viewports and "filled" for narrow viewports
<SplitPageLayout.Header dividier={{regular: 'line', narrow: 'filled'}}>

Part of https://github.com/github/primer/issues/793

I'm going to go ahead and merge this draft to keep SplitPageLayout moving forward. Feel free to leave any additional feedback about the SplitPageLayout or PageLayout API in the #primer-react channel in Slack.


Building for Magic Leap 2 (x86_64)

Plugin Version or Commit ID
v0.12.0
Unity Version
2022.2.19
Your Host OS
Windows 11
Target Platform
Android
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
Build Tool: 30.0.3
NDK: 21.4.7075529
[iOS Only] XCode Version
No response
Command Sequences
I am building the plugins with a Linux Docker, using the commands and outputs below. After building the plugin, I then open the root folder as a Unity project as instructed and export it, and import it into my project. Everything works fine in the editor, but the errors below are created when I deploy my program.

Log
mediapipe@38c1c9bf10fd:~$ python build.py build --android fat --opencv cmake --android_ndk_api_level 29 -vv
INFO (build.py): Building protobuf sources...
DEBUG (build.py): Running bazel build -c opt --action_env ANDROID_NDK_API_LEVEL="29" --verbose_failures //mediapipe_api:mediapipe_proto_srcs
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_google_benchmark' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'flatbuffers' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_googlesource_code_re2' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'zlib' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'rules_python' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'xctestrunner' because it already exists.
DEBUG: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_github_glog_glog' because it already exists.
INFO: Build options --action_env, --copt, --cpu, and 5 more have changed, discarding analysis cache.
INFO: Analyzed target //mediapipe_api:mediapipe_proto_srcs (99 packages loaded, 1050 targets configured).
INFO: Found 1 target...
Target //mediapipe_api:mediapipe_proto_srcs up-to-date:
bazel-bin/mediapipe_api/mediapipe_proto_srcs.zip
INFO: Elapsed time: 1.917s, Critical Path: 0.41s
INFO: 147 processes: 1 internal, 146 linux-sandbox.
INFO: Build completed successfully, 147 total actions
DEBUG (build.py): Unarchiving 'bazel-bin/mediapipe_api/mediapipe_proto_srcs.zip' to 'build/Runtime/Scripts/Protobuf'...
INFO (build.py): Built protobuf sources
INFO (build.py): Downloading dlls...
DEBUG (build.py): Running nuget install -o .nuget -Source https://api.nuget.org/v3/index.json
Feeds used:
/home/mediapipe/.nuget/packages/
https://api.nuget.org/v3/index.json
All packages listed in /home/mediapipe/packages.config are already installed.
DEBUG (build.py): Copying '.nuget/System.Memory.4.5.3/lib/netstandard2.0/System.Memory.dll' to 'build/Runtime/Plugins/Protobuf/System.Memory.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Memory.dll'...
DEBUG (build.py): Copying '.nuget/System.Buffers.4.4.0/lib/netstandard2.0/System.Buffers.dll' to 'build/Runtime/Plugins/Protobuf/System.Buffers.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Buffers.dll'...
DEBUG (build.py): Copying '.nuget/Google.Protobuf.3.20.0/lib/netstandard2.0/Google.Protobuf.dll' to 'build/Runtime/Plugins/Protobuf/Google.Protobuf.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/Google.Protobuf.dll'...
DEBUG (build.py): Copying '.nuget/System.Runtime.CompilerServices.Unsafe.4.5.2/lib/netstandard2.0/System.Runtime.CompilerServices.Unsafe.dll' to 'build/Runtime/Plugins/Protobuf/System.Runtime.CompilerServices.Unsafe.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Runtime.CompilerServices.Unsafe.dll'...
DEBUG (build.py): Copying '.nuget/packages/system.memory/4.5.3/lib/netstandard2.0/System.Memory.dll' to 'build/Runtime/Plugins/Protobuf/System.Memory.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Memory.dll'...
DEBUG (build.py): Copying '.nuget/packages/google.protobuf/3.20.0/lib/netstandard2.0/Google.Protobuf.dll' to 'build/Runtime/Plugins/Protobuf/Google.Protobuf.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/Google.Protobuf.dll'...
DEBUG (build.py): Copying '.nuget/packages/system.runtime.compilerservices.unsafe/4.5.2/lib/netstandard2.0/System.Runtime.CompilerServices.Unsafe.dll' to 'build/Runtime/Plugins/Protobuf/System.Runtime.CompilerServices.Unsafe.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Runtime.CompilerServices.Unsafe.dll'...
DEBUG (build.py): Copying '.nuget/packages/system.buffers/4.4.0/lib/netstandard2.0/System.Buffers.dll' to 'build/Runtime/Plugins/Protobuf/System.Buffers.dll'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Protobuf/System.Buffers.dll'...
INFO (build.py): Downloaded protobuf dlls
INFO (build.py): Building resource files
DEBUG (build.py): Running bazel build -c opt --action_env ANDROID_NDK_API_LEVEL="29" --verbose_failures //mediapipe_api:mediapipe_assets
INFO: Analyzed target //mediapipe_api:mediapipe_assets (28 packages loaded, 97 targets configured).
INFO: Found 1 target...
Target //mediapipe_api:mediapipe_assets up-to-date:
bazel-bin/mediapipe_api/mediapipe_assets.zip
INFO: Elapsed time: 3.525s, Critical Path: 3.24s
INFO: 21 processes: 1 internal, 20 linux-sandbox.
INFO: Build completed successfully, 21 total actions
DEBUG (build.py): Unarchiving 'bazel-bin/mediapipe_api/mediapipe_assets.zip' to 'build/PackageResources/MediaPipe'...
DEBUG (build.py): Unarchiving 'bazel-bin/mediapipe_api/mediapipe_assets.zip' to 'Assets/StreamingAssets'...
INFO (build.py): Built resource files
INFO (build.py): Building native libraries for Android...
DEBUG (build.py): Running bazel build -c opt --action_env ANDROID_NDK_API_LEVEL="29" --verbose_failures --config=android_fat --java_runtime_version=remotejdk_11 //mediapipe_api/java/com/github/homuler/mediapipe:mediapipe_android
INFO: Build options --copt, --cpu, --crosstool_top, and 4 more have changed, discarding analysis cache.
WARNING: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/com_google_mediapipe/mediapipe/framework/tool/BUILD:197:24: in cc_library rule @com_google_mediapipe//mediapipe/framework/tool:field_data_cc_proto: target '@com_google_mediapipe//mediapipe/framework/tool:field_data_cc_proto' depends on deprecated target '@com_google_protobuf//:cc_wkt_protos': Only for backward compatibility. Do not use.
WARNING: /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/external/com_google_mediapipe/mediapipe/framework/BUILD:56:24: in cc_library rule @com_google_mediapipe//mediapipe/framework:calculator_cc_proto: target '@com_google_mediapipe//mediapipe/framework:calculator_cc_proto' depends on deprecated target '@com_google_protobuf//:cc_wkt_protos': Only for backward compatibility. Do not use.
INFO: Analyzed target //mediapipe_api/java/com/github/homuler/mediapipe:mediapipe_android (157 packages loaded, 20500 targets configured).
INFO: Found 1 target...
INFO: From Executing genrule @org_tensorflow//tensorflow/lite/acceleration/configuration:configuration_schema:
When you use --proto, that you should check for conformity yourself, using the existing --conform
INFO: From Executing genrule @org_tensorflow//tensorflow/lite/acceleration/configuration:configuration_schema:
When you use --proto, that you should check for conformity yourself, using the existing --conform
INFO: From Executing genrule //mediapipe_api/java/com/github/homuler/mediapipe:mediapipe_android:
Archive:  /home/mediapipe/.cache/bazel/_bazel_mediapipe/6c11c387a04fa8f17c87a5c6db6ee555/sandbox/linux-sandbox/15927/execroot/mediapipe_api/bazel-out/arm64-v8a-opt/bin/mediapipe_api/java/com/github/homuler/mediapipe/mediapipe_android_dummy_app_unsigned.apk
creating: lib/arm64-v8a/
inflating: lib/arm64-v8a/libopencv_java3.so
inflating: lib/arm64-v8a/libmediapipe_jni.so
creating: lib/armeabi-v7a/
inflating: lib/armeabi-v7a/libopencv_java3.so
inflating: lib/armeabi-v7a/libmediapipe_jni.so
adding: jni/arm64-v8a/libmediapipe_jni.so (deflated 74%)
adding: jni/arm64-v8a/libopencv_java3.so (deflated 59%)
adding: jni/armeabi-v7a/libmediapipe_jni.so (deflated 73%)
adding: jni/armeabi-v7a/libopencv_java3.so (deflated 50%)
Target //mediapipe_api/java/com/github/homuler/mediapipe:mediapipe_android up-to-date:
bazel-bin/mediapipe_api/java/com/github/homuler/mediapipe/mediapipe_android.aar
INFO: Elapsed time: 1219.983s, Critical Path: 339.88s
INFO: 4944 processes: 11 internal, 4930 linux-sandbox, 3 worker.
INFO: Build completed successfully, 4944 total actions
DEBUG (build.py): Copying 'bazel-bin/mediapipe_api/java/com/github/homuler/mediapipe/mediapipe_android.aar' to 'build/Runtime/Plugins/Android/mediapipe_android.aar'...
DEBUG (build.py): Changing the mode of 'build/Runtime/Plugins/Android/mediapipe_android.aar'...
INFO (build.py): Built native libraries for Android
INFO (build.py): Installing built resources...
DEBUG (build.py): Copying 'build' to 'Packages/com.github.homuler.mediapipe' recursively...
DEBUG (build.py): Running cp -r build/* Packages/com.github.homuler.mediapipe
INFO (build.py): Installed
mediapipe@38c1c9bf10fd:~$
Additional Context
I am building the plugin for the Magic Leap 2 (x86_64) using the "fat" option on the build command, as shown in the log. When it is built and imported, the device cannot find the plugin files "mediapipe_ini" specifically (see error screenshot). I posted on the Magic Leap Developer Forum and was told a certain file structure is not being created. Can I build this plugin specifically for x86_64? Does the response from ML2's devs make any sense to you?

***Edit. One additional comment. After talking some more to the ML2 devs, I changed the .aar file build from the "python build.py build --android fat --opencv cmake --android_ndk_api_level 26 -vv" command to explore the file system. There are no x86 or x86_64 directories created (see the inset below).  Is this an error? It should be possible as in here

I think it's possible to build for x86_64, but it's currently not supported, so you can't build it as-is.
What if you change the settings in .bazelrc?
https://github.com/homuler/MediaPipeUnityPlugin/blob/01cdd567ac5939b81114e27e8c13a92bde8275f4/.bazelrc#L63

I have done this, as well as edited line 62 to say "x86_64", and the output .aar now contains the x86_64 folder, but the program is still throwing a DLLNotFoundException on Mediapipe_ini as shown above.

Will you try loading mediapipe_jni by yourself?
cf. https://github.com/homuler/MediaPipeUnityPlugin/wiki/Installation-Guide#dllnotfoundexception

public void LoadLibrary()
{
    using (var system = new AndroidJavaClass("java.lang.System"))
    {
        system.CallStatic("loadLibrary", "mediapipe_jni");
    }
}

I was able to add the code, it compiles fine. Also, I checked the apk, the .so is being imported. However, I get the message below.
2023-09-14 14:03:17.192;10106;9536;9557;Info;Unity;;
2023-09-14 14:03:17.228;10106;9536;9557;Error;Unity;AndroidJavaException: java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "__isOSVersionAtLeast" referenced by "/data/app/com.UnityTechnologies.com.unity.template.urpblank-Jocvmcf7NYEcTdEm2E20hA==/lib/x86_64/libmediapipe_jni.so"...;
2023-09-14 14:03:17.228;10106;9536;9557;Error;Unity;java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "__isOSVersionAtLeast" referenced by "/data/app/com.UnityTechnologies.com.unity.template.urpblank-Jocvmcf7NYEcTdEm2E20hA==/lib/x86_64/libmediapipe_jni.so"...;

Try specifying --android_ndk_api_level as 25 or lower.
python build.py build --android fat --android_ndk_api_level=21 -vv

Hi guys! I made a fork to build for x86_64 a while back. Just added this to .bazelrc to buil 0.10.3:
build:android_fat --fat_apk_cpu=armeabi-v7a,arm64-v8a,x86_64
And that worked fine. Don't know if it still works, though. I'll try updating the fork.

OK, using @karmacod3r 's fork, I have made progress. The libraries load!. However, now the errors below are being thrown whe I declare a new CalculatorGraph like so "_faceGraph = new CalculatorGraph(_faceDetectConfigAsset.text);"
I can tell the libraries are loading because the resource manager is working correctly immediately before this issue. Again, the code works perfectly on the Unity Editor.

It's likely there's an issue in the config, so please check the logs (e.g. adb logcat Unity:V native:V tflite:V CRASH:E AndroidRuntime:E "*:S").
Note that using different code for building the library and building the app can sometimes lead to issues.
(Although you may haven't tested it, I think specifying the --android_ndk_api_level as 25 or below should resolve the original problem without the need for the fork)

@homuler you were correct. I am only getting the errors above when I use a CPU-based config file. If I switch to a GPU-based file, the errors cease. However, the graph isn't producing any detections, and throws no errors, with a GPU file. I checked the input image produces facial detection in the Unity Editor, so its not a lack of actual faces.

Does the sample app work on your device? If it does, I think this issue is due to your implementation.
In that case, please open another issue if you need help.
If there are no errors, the most likely scenario is that the input images are not being passed correctly (e.g. CPU and GPU are not synchronized).

You are correct. It was an implementation issue. I reworked your samples to add a class to use the Magic Leap MLCamera (standard WebCamera method did not work) and that solved it.  Thank you!


Vulnerability roundup 97: brave-1.17.73: 1 advisory [5.5]

search, files

[ ] CVE-2020-8276 CVSSv3=5.5 (nixos-unstable)

Scanned versions: nixos-unstable: 83cbad92d73.
Cc @jefflabonte
Cc @nasirhm
Cc @rht
Cc @uskudnik

I will keep an eye on the next release! I will make a PR once it is available.
I already have a PR opened to backport on nix 20.09. I will use it to release the latest package at the same time.

fixed in https://github.com/NixOS/nixpkgs/pull/106718

fixed in https://github.com/NixOS/nixpkgs/pull/106718


is_tx_error() fails due to code now being 0 instead of None

terra_sdk version: 2.0.3
Python version: 3.10
Operating System: Windows 10

Description
Doing a broadcast now returns result.code = 0 instead of result.code = None so now is_tx_error(result) fails
What I Did
sendtx = self.wallet.create_and_sign_tx(CreateTxOptions(
    msgs=[execute],
    fee_denoms=["uusd"],
))
result = self.terra.tx.broadcast(sendtx)

error = is_tx_error(result)
if error:
    raise Exception(
        f"Transaction failed, TX_ID: {result.txhash}\nError: {result.raw_log}"
    )

this always triggers because is_tx_error always returns true now.
2.0.3
result: ......., code=0, codespace='', info=None, data=None, timestamp=None)
1.0.2
result: ........, code=None, codespace=None)

I fixed this locally just by changing is_tx_error to check for 0 instead of None but i dont think thats the solution you guys want, i wasnt sure where the 0 was coming from instead of None and figured it would be faster for you guys to check and decide what you guys want.

thank you for reporting! i think just 0 is fine. new version will be released after adding more fixs and features.


How to add errors per minute graph along with throughput and response times in skywalking UI

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
How to add errors per minute graph along with throughput and response times in skywalking UI

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

@TinyAllen I think you may need to add a developer document to guide, if you have time.

OAL Metrics and UI have supported to set by users.


fix: Set coordinator http port to 80

We don't expose the coordinator directly anymore, but proxy through our nginx on port 80.

bors r+


Select-Search: Dropdown stuck after the first click

Problem,
I was just trying to add v-model to v-select but after I've added v-model='lokasi_select' to the code it can't be clicked! Tried to reproduce in JSFiddle, however everything goes fine there..
Reproduction Link
https://jsfiddle.net/irfandyjip89/tuL9w1g8/16/
Steps to reproduce
I just copy pasted the needed code for it. The only difference is that my lokasi_list is actually connected to database using PHP. And it's already encoded to json.
### Expected behaviour
Select with search like in the documentation

User type
User click the selected label
Input shown in the input table

### Actual behaviour

User type
User click the selected input
Dropdown stuck, won't close.
However it can be seen that the value change when I clicked on the label. It only change once. Tried clicking other label, didn't change.

What it looks like in my project:

I know the variables are different, but I assure you that's the only thing that's different. So after the first click of 'Mall Ciputra Jakarta' it stucks. But when I tried using Google Dev tools to check wheter the values are in, they have been filled.
Photo after first click, and dropdown element deleted using Google Dev Tools:

However the second time I clicked none have changed. I can still interact with my other elements.
Error shown in Google Dev Tools:

Also, side note, I discovered if you use Bootstrap 4 you got a little 'dropdown logo' after the vue-select's dropdown button. Can be checked on my FiddleJS, however that's another thing. For now, I'm just wondering what should I do in order to get that search-select dropdown in documentation.
Thanks.

🙃 No problem :)


✨ Syncer transfos and coordination controller helpers

Summary
This PR implements:

a synchronous resource transformation framework that can be used in the virtual workspaces
the use of this framework to maintain syncer-specific views of a single KCP resource
builds on top of this to define a layer that very easily allows defining syncer transformations
defines coordination controller helpers

This PR is the second part of PROOF PR https://github.com/kcp-dev/kcp/pull/1239, which implements the first stories of the EPIC Multi-SyncTarget syncing and coordination controllers.
Thus this PR should be reviewed with the wider context and use-case in mind (especially the deployment Splitter POC in the PROOF PR).
Reviewing this PR commit-by-commit will make it easier.
Related issue(s)
https://github.com/kcp-dev/kcp/issues/1992

/retest

/lgtm

/approve


[Mint NFT] Mint NFT Page is Blank

Describe the bug
As a user, I see the blank page when access the /createNFT link.
To Reproduce
Steps to reproduce the behavior:

Go to NFTs tab
Click on Mint NFT
See blank page

Expected behavior

I'm going to remove this feature on wallet, moving it to marketplace


RequireJS example doesn't work

It happens because this piece of code in injected into common.js and reaches the client.
// Create a simple path alias to allow browserify to resolve
// the runtime on a supported path.
module.exports = require('./dist/cjs/handlebars.runtime');

Error:
Uncaught ReferenceError: module is not defined(anonymous function) @ common.js:6658

@iMoses yes I get the same error. I can't figure out a solution right away, but I'll look into it.
Thanks for bringing it up!

Closing this as it has been fixed.

Awesome. Thanks :)


add the office theme

Added the Office theme

Thank you for the contribution @swatiaggrawal !! 😃

Glad you liked the new theme 😁😁


echo steps

:x: 6 Tests Failed:

Tests completed
Failed
Passed
Skipped

20
6
14
0

View the top 3 failed tests by shortest run time

python/pytest/random_test.py::::test_10_percent

Stack Traces | 0s run time

def test_10_percent():
        random_number = secrets.randbelow(100)
>       assert random_number <= 10
E       assert 82 <= 10

python/pytest/random_test.py:72: AssertionError

python/pytest/random_test.py::::test_30_percent

Stack Traces | 0s run time

def test_30_percent():
        random_number = secrets.randbelow(100)
>       assert random_number <= 30
E       assert 54 <= 30

python/pytest/random_test.py:62: AssertionError

python/pytest/random_test.py::::test_coin_toss

Stack Traces | 0s run time

def test_coin_toss():
        random_number = secrets.randbelow(100)
>       assert random_number <= 50
E       assert 65 <= 50

python/pytest/random_test.py:27: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
:loudspeaker:  Thoughts on this report? Let us know!


Incomplete output in JupyterLab and Notebooks

When using pn.pane.Markdown the height of output is very small unless specify a fixed height value, please see below:

Also, when using pn.Column the output is also incomplete and there is no way to specify the height for the Column:

I'm using JupyterLab 0.35.3 and Panel 0.4.0 from conda-forge. Similar behavior in the notebook mode.
Thanks!

I would not recommend using panel 0.4.0 at this point. The new version 0.5.0 is out on conda now and will be on pypi next week we hope.

I have tried update panel but just realised I should use pyviz channel instead of conde-forge.
Thanks!

Yes, we'll hopefully have everything up on conda-forge and even defaults in the next couple of weeks.


Remove dns requirement, add hook/plugin support

build hooks
remove domain as a core concept, replace with namespace
bucket used for CF templates prepended with 'stacker-'
asg blueprint optionally sets up DNS if BaseDomain provided
route53 hook - requires BaseDomain parameter

Cool. Lgtm :+1:


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Tue Mar 29 2016 19:07:01 GMT-0400 (US Eastern Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\bapat\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\bapat\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


System.Runtime.Loader errors out after adding in a new csharp script

Godot version
v3.4.beta6.mono.official [3e2bb415a]
System information
Windows 10, Nvidia GTX 750 Ti
Issue description
Before you ask, yes this builds and runs. Though as soon as I add in a new csharp script and click back into Godot, Godot crashes with this error:
ERROR: Mono: FATAL ERROR '* Assertion at /root/mono-<IP_ADDRESS>/mono/metadata/class.c:2834, condition `is_ok (error)' not met, function:mono_class_get, Could not load file or assembly 'System.Runtime.Loader, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies.
 (in domain , error)', ABORTING! Logfile: 'C:/Users/caleb/AppData/Roaming/Godot/mono/mono_logs/2021-10-18_04.11.58_42468.log'.

Note that I'm not using version <IP_ADDRESS>, I'm using version 4.3.1
csproj
<Project Sdk="Godot.NET.Sdk/3.3.0">
  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
  </PropertyGroup>
  <ItemGroup>
  </ItemGroup>
  <ItemGroup>
    <PackageReference Include="System.Runtime" Version="4.3.1" />
  </ItemGroup>
  <ItemGroup>
    <Compile Remove="Addon\**" />
    <Content Include="Addon\**" />
  </ItemGroup>
</Project>

Steps to reproduce

Get one of your csharp files to use System.Runtime.Loader.
Build project in Godot.
Run project in Godot.
See that there is no errors and it runs fine.
Add new csharp file to Godot.
Godot crashes.
Project will keep crashing every time you open it until you remove .mono/temp and restore your project.

Minimal reproduction project
No response

The Godot C# project is a library so netcoreapp3.1 is not a valid framework. You have to use netstandard.