TypeError: Cannot read property 'Television' of undefined

Homebridge crashes when I add this plugin:
May 27 22:46:24 pi-hb homebridge[14520]: [5/27/2020, 10:46:24 PM] [Sony TV] Initializing HarmonyTV accessory...
May 27 22:46:24 pi-hb homebridge[14520]: [5/27/2020, 10:46:24 PM] TypeError: Cannot read property 'Television' of undefined

Relevant part of "accessories" from configuration file:
  {
    "accessory": "HarmonyTV",
    "name": "Sony TV",
    "host": "<IP_ADDRESS>",
    "remoteId": "<REDACTED>"
  }

Node.js version:
v12.16.3
Homebridge version:
1.1.0
Plugin version:
5.0.1

Thanks for reporting this @pro-sumer! This is likely a regression introduced by #46, which I merged prematurely. I’ve published a new version of the plugin 5.0.4—would you mind updating to that version and checking whether this issue persists?

Unfortunately it persists with v5.0.4:
May 27 23:23:19 pi-hb homebridge[1065]: [5/27/2020, 11:23:19 PM] [Sony TV] Initializing HarmonyTV accessory...
May 27 23:23:19 pi-hb homebridge[1065]: [5/27/2020, 11:23:19 PM] TypeError: Cannot read property 'Television' of undefined
May 27 23:23:19 pi-hb homebridge[1065]:     at new HarmonyTVAccessory (/usr/lib/node_modules/homebridge-harmony-tv-smockle/dis
May 27 23:23:19 pi-hb homebridge[1065]:     at /usr/lib/node_modules/homebridge/src/server.ts:356:50
May 27 23:23:19 pi-hb homebridge[1065]:     at Array.forEach (<anonymous>)
May 27 23:23:19 pi-hb homebridge[1065]:     at Server._loadAccessories (/usr/lib/node_modules/homebridge/src/server.ts:329:29)
May 27 23:23:19 pi-hb homebridge[1065]:     at Server.start (/usr/lib/node_modules/homebridge/src/server.ts:155:12)
May 27 23:23:19 pi-hb homebridge[1065]:     at cli (/usr/lib/node_modules/homebridge/src/cli.ts:80:10)
May 27 23:23:19 pi-hb homebridge[1065]:     at Object.<anonymous> (/usr/lib/node_modules/homebridge/bin/homebridge:17:22)
May 27 23:23:19 pi-hb homebridge[1065]:     at Module._compile (internal/modules/cjs/loader.js:1133:30)
May 27 23:23:19 pi-hb homebridge[1065]:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1153:10)
May 27 23:23:19 pi-hb homebridge[1065]:     at Module.load (internal/modules/cjs/loader.js:977:32)
May 27 23:23:19 pi-hb homebridge[1065]:     at Function.Module._load (internal/modules/cjs/loader.js:877:14)
May 27 23:23:19 pi-hb homebridge[1065]:     at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:74:12
May 27 23:23:19 pi-hb homebridge[1065]:     at internal/main/run_main_module.js:18:47
May 27 23:23:19 pi-hb homebridge[1065]: [5/27/2020, 11:23:19 PM] Got SIGTERM, shutting down Homebridge...

(after I modified my config file to have a deviceId and commands)

Thanks for the update @pro-sumer. I’ll take a look.
The release notes for homebridge v1.1.0 mentioned:

Plugins Using TypeScript: Homebridge now only exports types that are safe to use in your code and won't result in the homebridge library being a runtime dependency. If you have been using types correctly then you will not be impacted by this change.

I suspect that is the source of the issue.

To expand on my comment above, the yarn.lock file specifies homebridge v1.0.4, so that’s the version the plugin has been tested with, but the yarn.lock file isn’t included in the published package, and the version range in the package.json includes v1.1.0.

I switched to only import type from homebridge in https://github.com/smockle/homebridge-harmony-tv/commit/a02439137689dff392512fadc39ecbf710dd90de, which was published as version 5.0.6. I ran that version and confirmed I could control my TV.
@pro-sumer, would you mind updating to version 5.0.6 and checking whether the issue is resolved?

Thanks! The plug-in does start now and I can partially control my (Sony Bravia) TV, but I'm not sure what to expect (I only got my Harmony Hub this week).
This is what I see:

Switching between Analog/Digital/HDMI1 works, but HDMI4 I can't select, which is a pity, because that's where my Apple TV is connected to.
I was expecting that I would also be able to change channels or volume, but that's not functionality offered by this plug-in?
(I do have commands for all digits, channel down/up, mute, volume down/up in my configuration)

Switching between Analog/Digital/HDMI1 works, but HDMI4 I can't select, which is a pity, because that's where my Apple TV is connected to.

That’s surprising—at first glance, I don’t see a reason the input-related code (harmony-tv-accessory.ts#L41-L44 and harmony-tv-accessory.ts#L202-L259) would omit only HDMI 4.

I was expecting that I would also be able to change channels or volume…

Changing volume is supported by this plugin, but only in the remote app in Control Center (i.e. not the Home app). When the TV is the active device in the remote app, the physical volume buttons on your iPhone/iPad should control the TV’s volume. Here’s a related Apple Support article.
Changing channel is not supported by this plugin. As mentioned in the README, I do not have plans to add additional functionality (recent updates have been in support of Homebridge 1.0 compatibility), but I would be happy to accept pull requests.

…that’s not functionality offered by this plug-in?

This plugin is pretty limited, and not actively developed. I’d recommend trying out homebridge-harmony—it’s more full-featured and is actively maintained.

I only see the accessory in Apple's Home App, not in third-party Apps like Home+ or Eve. Is that expected?

I’m not sure. I did a bit of research: Home+’s FAQ answers a slightly-different question—“Why is my Apple TV and HomePod not listed?”. Because they are also AirPlay 2 devices, perhaps televisions, like Apple TVs and HomePods, only show up in Apple’s first-party app.

Thank you for your support!
Experimenting with homebridge-harmony was already on my todo list, so I'll give that higher priority now.


Some wrong and missing descriptions for operations in Microsoft.EventHub version 2021-11-01

missing descriptions:
https://github.com/Azure/azure-rest-api-specs/blob/8fb0263a6adbb529a9a7bf3e56110f3abdd55c72/specification/eventhub/resource-manager/Microsoft.EventHub/stable/2021-11-01/SchemaRegistry.json#L98-L100
https://github.com/Azure/azure-rest-api-specs/blob/8fb0263a6adbb529a9a7bf3e56110f3abdd55c72/specification/eventhub/resource-manager/Microsoft.EventHub/stable/2021-11-01/SchemaRegistry.json#L150-L152
https://github.com/Azure/azure-rest-api-specs/blob/8fb0263a6adbb529a9a7bf3e56110f3abdd55c72/specification/eventhub/resource-manager/Microsoft.EventHub/stable/2021-11-01/SchemaRegistry.json#L192-L194
wrong descriptions
https://github.com/Azure/azure-rest-api-specs/blob/8fb0263a6adbb529a9a7bf3e56110f3abdd55c72/specification/eventhub/resource-manager/Microsoft.EventHub/stable/2021-11-01/namespaces-preview.json#L442-L444
should be the description of deleting PrivateEndpointConnection here.

Thanks for reporting @yukun-dong , I would investigate with team to add and update right description.


High-dimension Matmul MACs calculation error

Describe the bug
Gives wrong answer on higher-dimensional matmul operations: https://github.com/ThanatosShinji/onnx-tool/blob/main/onnx_tool/node.py#L642-L656
To Reproduce
e.g. a matmul with operands with shape (2, 3)and (4, 3, 5) is a batch matmul, should give a result with shape (4, 2, 5)
The current code will just give the result for MACs as 2 * 3 * 5 which is not correct, where we also need to take into account the batch dim 4.

Thanks for your report!
I've confirmed it's a broadcast bug.

Can provide the shape_infer result of the current code on your model? I suspect the resulting shape is also wrong too.

yeah the shape_infer would just give (2, 5) for the above case, ignoring the batch dim 4

The bug is fixed in the main branch. The code will be released after all model tests.

Do you have ETA for the release?

One new feature is about to release. So there won't be a release only for this bug. You can checkout the main branch code and install it with your pip.

you can install v0.7.1 to test the fix of this bug now.

Thanks, closing the issue.


MQTT Connection.

type : User story
Problem
Methods:
SR04 sensor data:- to publish ultrasound sensor data
ConnectHost:- to connect to local or other host
git tags
User story, Sprint1
notes
Documentation will be added!
Resolves User story #16

Wrong pull request . Duplicate


add orderer plugin

Codecov Report

Merging #37 (8603668) into master (7e99f1d) will decrease coverage by 0.45%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
- Coverage   93.70%   93.24%   -0.46%
==========================================
  Files          96      101       +5
  Lines        2256     2310      +54
  Branches      339      347       +8
==========================================
+ Hits         2114     2154      +40
- Misses        128      142      +14
  Partials       14       14

Impacted Files
Coverage Δ

vedro/plugins/orderer/orderer_plugin.py
50.00% <50.00%> (ø)

.../core/_scenario_orderer/_plain_scenario_orderer.py
85.71% <85.71%> (ø)

vedro/_config.py
94.82% <100.00%> (+0.09%)
:arrow_up:

vedro/core/__init__.py
100.00% <100.00%> (ø)

vedro/core/_scenario_orderer/__init__.py
100.00% <100.00%> (ø)

vedro/plugins/orderer/__init__.py
100.00% <100.00%> (ø)

vedro/plugins/orderer/random_orderer.py
100.00% <100.00%> (ø)

vedro/plugins/orderer/reversed_orderer.py
100.00% <100.00%> (ø)

vedro/plugins/orderer/stable_orderer.py
100.00% <100.00%> (ø)

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


Propagate Buildkite Artifact env vars with mount-buildkite-agent

When mounting the buildkite-agent, can the following env vars also be propagated so uploading artifacts works as expected?
BUILDKITE_ARTIFACT_UPLOAD_DESTINATION
BUILDKITE_S3_DEFAULT_REGION
BUILDKITE_S3_ACL

Not sure what else might be needed for artifact destinations besides S3.

For reference, the complete list of environment variables needed for artifact uploading to your private S3 buckets are documented in the artifact agent command.
It may be interesting to see if implementing #61 would be enough to support this use-case or there will need to be special handling for these variables.
That said, supporting this use case would also require supporting the other two customized destinations for artifacts: Artifactory and GCloud.

Created PR #344 to implement the propagate-environment option that is very likely to take care of this


shiv: 1.0.6 -> 1.0.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for shiv is: Command line utility for building fully self contained Python zipapps
meta.homepage for shiv is: https://github.com/linkedin/shiv
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.7 with grep in /nix/store/7hpym6hylr1cgsl94c8515hy1hk3krqj-python3.12-shiv-1.0.7
found 1.0.7 in filename of file in /nix/store/7hpym6hylr1cgsl94c8515hy1hk3krqj-python3.12-shiv-1.0.7

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.shiv
python312Packages.shiv
shiv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7hpym6hylr1cgsl94c8515hy1hk3krqj-python3.12-shiv-1.0.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shiv https://github.com/r-ryantm/nixpkgs/archive/83ad3c4f03ac7f4f94a5964c403fb05bd8fdbeb1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/83ad3c4f03ac7f4f94a5964c403fb05bd8fdbeb1#shiv

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7hpym6hylr1cgsl94c8515hy1hk3krqj-python3.12-shiv-1.0.7
ls -la /nix/store/7hpym6hylr1cgsl94c8515hy1hk3krqj-python3.12-shiv-1.0.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.shiv
python311Packages.shiv.dist
shiv (python312Packages.shiv)
shiv.dist (python312Packages.shiv.dist)

Maintainer pings
cc @prusnak for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Duplicate of https://github.com/NixOS/nixpkgs/pull/349087


CSS Selectors/Properties

Consolidate CSS selectors and properties

There was no consolidation nessecery.

solved in branch iss7


Fixing issues when using dlls without assembly attributes

As discussed with @anpaz-msft  and @cgranade, I'll merge this to continue with perf.


Should we delete the index in Meilisearch if the collection is deleted in Firebase

Currently, if you delete your collection in firebase, it is still present in Meilisearch.
This results in the user having to create a script to be able to delete in Meilisearch.
It goes against the purpose of this plugin that aims to be a no-code installation of Meilisearch.

What I see in this case is to support pre-existing indexes, which should not be deleted automatically.
What I understand from my POV is that if we cannot add a warning message informing the user, "hey, you will delete your Meilisearch index automatically after you delete your Firestore collection." we should not add this option.

I'm closing this PR until someone has a real use case. In which case I re-open it


Remove Dots from Component Names

It is possible that the server sends component names with dots in them. This is leading to problems because the ReactUI is using the component name as element ids. PrimeReact and I use the querySelector function which will interpret the dots in the id as the start of a className which results in errors.
For now I'm removing the dots on client-side. But as discussed on friday this should happen on server side.

Do you have an example?

Yes I think this happened while I was coding for V2... The "...About" screens components had "About..." in their name which caused a crash.

thank you!

Please verify with latest version in repo.

Doesn't seem to work, I'm on the latest version...

ok, the used Java method works like defined: https://electrictoolbox.com/valid-characters-html-id-attribute/
Sure, I could remove some characters, but what's allowed for react?

I think this sums it up pretty well: https://stackoverflow.com/a/79022

The "...About" frame in full transferType mode works now.
Just to be clear :

Non letters only if name starts with a letter.

You mean that non letters aren't allowed at the start eg. numbers at the start.
Because currently if the name starts with a number, I add an underscore in front of it

Examples:
1234ABC456 -> ABC456
_ABC-_ -> ABC-_
aAbC124 -> aAbC124

It's very restrictive, but it's only the style info.

Ok thats perfect, then I don't have to worry about that on the client anymore!

Yes, there shouldn't be specific handling on client side.


Fix mappings for HmIP-HDM* and HBW-LC-RGBWW-IN6-DR

Fixes #1431

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ec5d62b) 90.42% compared to head (bf96a90) 90.48%.
Report is 6 commits behind head on devel.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #1432      +/-   ##
==========================================
+ Coverage   90.42%   90.48%   +0.06%
==========================================
  Files          49       49
  Lines        6037     6055      +18
==========================================
+ Hits         5459     5479      +20
+ Misses        578      576       -2

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


Gradle Kotlin DSL: Add Support for Named URL Parameter in Maven Repository

In Kotlin, a function call can have named parameters and Strings are only allowed to begin with ".
repositories {
    maven("foo.com")
    maven(url = "bar.com") // equals to maven("bar.com")
    maven('baz.com') // invalid, does not compile
}

This PR adds the optional url =  String to the regex as well as removing ' as String delimiter, which is not allowed in Kotlin.

Thanks, this looks great, just 1 minor request

Thanks, this looks great, just 1 minor request

@jurre added :)

@jurre added :)


Ability attributes on site module

Make the backend module site expandable with attributes

Merged as commit: https://github.com/shopware/shopware/commit/b3d0ff098e53df8e66de8b0574c8ab6832441274
Thank you @Shyim


Change GATT notify/indicate from gattc to gatts

GATT notify/indicate are functions of GATTS, so we should change ble_gattc_notify/ble_gattc_indicate to ble_gatts_notify/ble_gatts_indicate

Hi,
I'm fine with rename but we should try to keep API compatible.
I suggest we add wrappers for old API with note that it is deprecated and new one should be used instead eg:
/**

Deprecated. Should not be used.  Use ble_gatts_notify_custom instead.
*/
static inline int ble_gattc_notify_custom(uint16_t conn_handle, uint16_t att_handle, struct os_mbuf *om)
{
return ble_gatts_notify_custom(conn_handle, att_handle, om);
}

This should be done for all public API (ie from gatt.h that was renamed)

@sjanc issues fixed, thanks

Hi, sorry for late reply, could you please rebase PR so that it can be merged?

@sjanc Having rebased.

Hi,
Looks like in latest rebase you dropped backward compatibility wrappers as discussed before (hence the build failure)

Hi @sjanc , Having updated, but the CI is still failed.
It seem like the error is nothing to do with my change, such as:
nimble/host/test/src/ble_hs_test_util.c:1852:1: error: redefinition of ‘ble_transport_to_ll_acl’


Update pytest command.

Invoke pytest as a python module rather than a standalone executable.

Note that the build fails because there is a failing test:

FAILED tests/test_array.py::TestLimit::test_limit - TypeError: '<=' not suppo...

#830 fixes this test and this actions workflow should start working again post merge.


removing prototype functions, upgrading deps

Removing prototype functions
Upgrades ember-cli and deps

Much of the changes came in the way of ember init after upgrading the deps.  The rest is me fixing deprecation warnings and removing the use of a few Function.prototype.on, Function.prototype.computed

@jasonmit thanks for this PR!


explicit better than implicit (or in this case global)

Are you a current Fivetran customer?
Doug Shawhan, Developer ConstructConnect.com
What change(s) does this PR introduce?
Changes only to README.md. Explicitly moves *_enabled models into hubspot namespace for clarity. I spent entirely too much time staring at my dbt_project.yml wondering why my excluded models were being ignored. @fivetran-joemarkiewicz  (who was supposed to be on VACATION according to his slack palm tree) hit me with the cluebat.
Setting the exclusions as global directly under vars is fine, but gets a bit confusing when the config instructions:
# dbt_project.yml

...
config-version: 2

vars:
  hubspot_source:
    hubspot_database: your_database_name
    hubspot_schema: your_schema_name

reference hubspot_source. It's entirely too easy to miss the position of the models in the example. In my case:
  hubspot_source:
    hubspot_database: data-lake
    hubspot_schema: hubspot
    hubspot_email_event_forward_enabled: false
    hubspot_email_event_print_enabled: false

  hubspot:
    hubspot_email_event_forward_enabled: false
    hubspot_email_event_print_enabled: false

Appears much clearer, and reminds readers unfamiliar with dbt_hubspot* that there are two namespaces affected.
Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[X] No  (please provide explanation how the change is non breaking below.)
Just the docs.

Is this PR in response to a previously created Issue

[ ] Yes, Issue [link issue number here]
[X] No

How did you test the PR changes?

[ ] CircleCi
[X] Other (please provide additional testing details below)

Laid my eyes on it in github.
Select which warehouse(s) were used to test the PR

[X] BigQuery
[ ] Redshift
[ ] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Documentation change in README.md matches local configuration. Runs great.
Provide an emoji that best describes your current mood

:dancer:
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Hi @dougscc!
We've gone ahead and pushed a release to update the README.md to the dbt_hubspot_source package above, so please watch out for it dbt_hubspot_source 0.5.1 ! I've also tagged you as a contributor in the release notes, thank you again so much for bringing this to our attention. Please feel free to let us know if you have any questions. I'll be closing out this ticket now.


 Any error output should be
visible above. Please fix these errors and try again.

Summary
My provisioning is hanging when installing drush.
https://github.com/drush-ops/drush/releases/download/8.x/drush.phar is not a valid url.

I was able to get past this by putting an specific version in the drush_version key in the config.yml
was 8.x
now 8.1.10

Was going to mention that, but just noticed you found it on your own; thanks for posting!

Few things annoy me more than seeing, "Disregard, I fixed it" with no further information.

@frob - Also, for the reasoning behind this change, see: https://github.com/geerlingguy/ansible-role-drush/issues/30
Basically, a lot of sites are now installing Drush as a composer dependency, so don't even need Drush as part of Drupal VM (since it's part of their codebase)... and for those who do want Drush installed globally, it's about 1 minute faster (20s total instead of about 00:01:20s) to install the .phar instead of 'building from source' from GitHub.
Though you can still switch back to the old technique if required using drush_install_from_source.

For D6, you must drush_install_from_source: yes. There isn't a phar file of drush 7.x.

Perhaps adding a comment to the default.config.yml would save a lot of people time and angst:
# Pre 8.x.x versions of drush must install from source because no .phar file
drush_install_from_source: yes
#drush_version: "8.1.10"
drush_version: "6.2.0"
--
ALSO, installing Drush-6.2.0 required me to manually download the following library in order to function:   http://download.pear.php.net/package/Console_Table-1.1.3.tgz
I had to vagrant ssh in order to see the /usr/local/share/drush/lib/ directory.  I also had to prepend sudo to all commands due to need for root permissions:
vagrant@mydomain:/var/www/drupalvm/drupal/web$ sudo mv Console_Table-1.1.3.tar /usr/local/share/drush/lib/.
vagrant@mydomain:/usr/local/share/drush/lib$ sudo tar xvf Console_Table-1.1.3.tar
package.xml
Console_Table-1.1.3/tests/assoziative_arrays.phpt
Console_Table-1.1.3/tests/colors.phpt
Console_Table-1.1.3/tests/filters.phpt
Console_Table-1.1.3/tests/multibyte.phpt
Console_Table-1.1.3/tests/multiline.phpt
Console_Table-1.1.3/tests/no_header.phpt
Console_Table-1.1.3/tests/no_rows.phpt
Console_Table-1.1.3/tests/rules.phpt
Console_Table-1.1.3/Table.php


Adding # generated by chef header

Some templates were missing the header.

I just signed the CLA, I hope this can be merge now :)

No one with write access?


Feature: Rename the "Path:" text to "Location:"

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #11950

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open app ...
Open any folder/file properties

Screenshots (optional)

Should we therefore modify path by location but in different languages

Should we therefore modify path by location but in different languages

Originally, the text given in other languages except English, had Path: in them from the get-go, so it was intuitive to swap them out with Location straight away.

So for example in French I can put "Emplacement"

So for example in French I can put "Emplacement"

If that's the appropriate translation, then I would say yes.

So I replace all the words path with location

So I replace all the words path with location or that "path" and "path:" ?

No, only the part where you see Location:, therefore, only Path: should be replaced. Make sure to not mix up values with names.

And why not the others

And why not the others

I may be misunderstanding, could you elaborate on what the previous question meant? Check the changed files and you will see what I changed, then follow that. That might help.

I meant why not for example "Item Path" and that "Path:"

I meant why not for example "Item Path" and that "Path:"

I only changed what was highlighted and approved in the issue. Otherwise, good point. Also, We are not sure whether that line string of Item Path relates to an actual path (meaning it is named path correctly) or if the same change might be required.

So what do I do I translate all or that "Path: "

So what do I do I translate all or that "Path: "

Best action to take is to wait for the main dev to see this pr, then answer to us and proceed then appropriately.

Ok, so I just translate "Path:" into French and then we'll see the rest

@Lukiluc29 congrats on your first pull request! I think we can go about this one a little differently, instead of modifying the existing strings, we should add a new string called "Location" and update the xaml file to use this string. In general we only make changes to the en-us resw file and let Crowdin take care of the other languages.

@Lukiluc29 congrats on your first pull request! I think we can go about this one a little differently, instead of modifying the existing strings, we should add a new string called "Location" and update the xaml file to use this string. In general we only make changes to the en-us resw file and let Crowdin take care of the other languages.

I see. I will make the according changes per those instructions.

@Lukiluc29 congrats on your first pull request! I think we can go about this one a little differently, instead of modifying the existing strings, we should add a new string called "Location" and update the xaml file to use this string. In general we only make changes to the en-us resw file and let Crowdin take care of the other languages.

I redid the task as you suggested. One really strange thing is that I simply replaced the Location to Path in en-GB but the commit shows some other changes and I didn't write these... I would like to know how did those changes appear there, maybe because of syncing?

Congratulations on your first pull request, LGTM

Thank you so much!


Suggestion: Generic GameState that saves into JSON (or similar)

Currently, the GameLevelLog tracks progress in the config file. This does not follow convention on where that data would typically be stored, as it mixes application configuration settings with game state.
Instead, we could try creating a generic GameState (or even more generic AppState) that stores a dictionary of data and manages the saving/loading behind the scenes. There's a lot of documentation available on how to save and load game state to and from files.
https://docs.godotengine.org/en/stable/tutorials/io/saving_games.html
Eventually, it could have multiple levels, like UserGameState, PCGameState, WorldGameState depending on the level of persistence desired.
GameLevelLog, GameLog, AppLog could then be updated to read from the GameState instead of Config.

I think adding this could really help avoid devs (specifically, me) from getting into a bad practice of saving game state data in the config file instead, just because it is available.

I have a working system from my last game jam that saves state into a Resource object, and then saves with ResourceSaver. It worked great for the jam, and may be better than trying to use JSON. There is a vulnerability that the resource files can hold executable code, but I'm not sure how that is a concern here. ( https://forum.godotengine.org/t/how-to-load-and-save-things-with-godot-a-complete-tutorial-about-serialization/44515#but-i-heard-from-someone-that-resources-are-insecure-13 )
I began porting it over, but stalled when trying to generalize it for the template. I'll try again in the near future.

It's done! This is a first version for a generic system, so I'll dog-food it for a few jams, and see if it needs changes.


cleanup setup.py

This works for me as expected:
$ /tmp/tess-point [master|✚ 1] $ pip2 install -e .
Obtaining file:///private/tmp/tess-point
Requirement already satisfied: numpy in /usr/local/lib/python2.7/site-packages (from tess-point==0.3.1) (1.11.3)
Collecting astropy (from tess-point==0.3.1)
  Using cached https://files.pythonhosted.org/packages/12/5e/33bd9fe91714357b8dd6825cd5f1668e887b72a9d94e39509bb733563c15/astropy-2.0.11-cp27-cp27m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
Requirement already satisfied: scipy in /usr/local/lib/python2.7/site-packages (from tess-point==0.3.1) (1.1.0)
Requirement already satisfied: pytest<3.7,>=2.8 in /usr/local/lib/python2.7/site-packages (from astropy->tess-point==0.3.1) (3.6.4)
Requirement already satisfied: atomicwrites>=1.0 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (1.2.1)
Requirement already satisfied: pluggy<0.8,>=0.5 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (0.7.1)
Requirement already satisfied: py>=1.5.0 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (1.7.0)
Requirement already satisfied: setuptools in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (40.6.3)
Requirement already satisfied: six>=1.10.0 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (1.12.0)
Requirement already satisfied: more-itertools>=4.0.0 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (5.0.0)
Requirement already satisfied: attrs>=17.4.0 in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (18.2.0)
Requirement already satisfied: funcsigs; python_version < "3.0" in /usr/local/lib/python2.7/site-packages (from pytest<3.7,>=2.8->astropy->tess-point==0.3.1) (1.0.2)
Installing collected packages: astropy, tess-point
  Found existing installation: tess-point 0.3.1
    Uninstalling tess-point-0.3.1:
      Successfully uninstalled tess-point-0.3.1
  Running setup.py develop for tess-point
Successfully installed astropy-2.0.11 tess-point

The trick is you need a recent (~year old or so) setuptools, and pip. I have 40.6.3 and 18.0 of them locally,  but bit older ones should work, too (they need to understand the python_requires in astropy that limits the supported python versions. More info here: https://packaging.python.org/guides/distributing-packages-using-setuptools/#python-requires

When I was testing under a conda py2.7 environment.
conda create -n tptest python=2.7
conda activate tptest
When I use the command
python setup.py develop
In order to test out installing tess-point, it leads to the install failing after trying to install astropy-3.1.1.
However, if I use your command
pip install -e .
I concur that it pulls astropy-2.011 and tess-point installs succesfully.
I thought 'python setup.py develop' and 'pip install -e .' were equivalent?  Do you know the reason for the different behavior?

After more googling, I believe I have answered my own question.  These two are not equivalent.  I should have been using your 'pip install -e .' in order to correctly get depencies.  I will merge this in and reversion to v0.3.2.  Thank you for setting me straight Brigitta it was user error on my part.

pip install -e . is short for --editable and can be used when you are planning to use the package you're actively developing, I was just using it to demonstrate it picks up the right version of astropy.


More Cupertino flutter.io website presence

https://docs.flutter.io/flutter/cupertino/cupertino-library.html should either show examples of using Cupertino, or link to some place which does.
I'm interested in having a place to point people when they ask about our support for iOS design aesthetics (whether this URL or another I don't care).

Looks like https://flutter.io/widgets/cupertino/ might be the page I want?

Pointing to https://flutter.io/widgets/cupertino makes sense

Last item done in https://github.com/flutter/website/pull/2585

Still valid since most of the items in the checklist on https://github.com/flutter/flutter/issues/14705#issuecomment-452126077 were not done yet.


[monitoring v2] etcd metrics tab not displaying for RKE2 clusters

etcd metrics tab isn't displaying for RKE2 clusters
this is on v2.5.8-rc6 monitoring v14.5.100-rc7

on v2.5.8-rc7, etcd is showing up for RKE2 metrics


Backend reorder not working correctly

Winter CMS Build
1.1
PHP Version
8.0
Database engine
MySQL/MariaDB
Plugins installed
No response
Issue description
There is an issue where sorting from the backend reorder list view does not work when records all contain the same/duplicate sort order value since the winter.reorder.js file does not recalculate the sort order value and re-uses the existing sort order values from the DB and just switches what record that value is applied to. This is a problem if you add sorting functionality to an existing table with records already created and then the migration assigns a default sort order value of 0 to every record and when you create a new record, it will contain the sort order of 0 which will also cause sorting for that record to stop working if it conflicts with another.

Steps to replicate

Create records in a table without sorting functionality.
Add new sort_order column to table and have the migration assign default sort_order value of 0 to every record.
Implement Reorder Controller functionality
Try to re-order list of records from backend, but notice the new order of the records is not reflected in the DB / after reloading the page.

Workaround

During migration, loop through every record and adjust the sort_order to be the same as the primary key.
When creating new records, override default sort_order so it matches the primary key as well so we don't run into duplicates.

@ericp-mrel the documentation does state there is an expectation that the developer has to assign sort order values for existing records - see the note at the bottom of the Sortable chapter here: https://wintercms.com/docs/database/traits#sortable
Given that they would need to create a migration in the first place (to add the sort_order column to the model's table), that would be the place to put the query to assign sort orders.

Yeah, sorry I see it now that you point it out. I still personally feel it would be a bit of a nicer experience to not have to do a migration like that assigning sort order values in order for it to actually work from the backend UI.
I'll let you guys decide on if that's something you'd be willing to consider changing, otherwise you can close this ticket since as you pointed out in the documentation this isn't really a bug, but could be more of an enhancement.
I'm not sure if this would help (or maybe it's just me), but it might  make the note more obvious and noticeable to move it closer to the example migration snippet line.

@ericp-mrel we'll still consider the PR, I was just making a note that it is documented.
As for the positioning of that notice in the documentation, I think the location is right, it just doesn't stand out enough (like most notes on the current docs). I'm hoping to have these new docs online soon, where it's a bit more apparent ;)


ci: Fix tar download for rts

Description
Fix tar download for rts
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Code review
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

/ok-to-test sha=af940612a1ef9a9a44d5ceef729ee494aa54c96f


Disallow OS text selection in atom buffers.

This admittedly odd-looking change disables OS-level text selection in Atom text buffers. We generally prevent native selection operations anyway (instead managing the selection ourselves), but in some cases (macOS right click, frex), we would still get one (which wouldn't be attached to anything, creating confusion and sowing discord).
Fixes #17799
Fixes atom/spell-check#251

@queerviolet Also appears to fix https://github.com/atom/spell-check/issues/251. The cursor still moves to the word, but that's a separate issue.

Ugh.  Copy/paste from MD preview does actually work. 😔

thanks!! ❤

Vick


Add lint plugin

Description
Add a puppet-lint plugin to check for the presence of a newline character at the end of a file.
Motivation and Context
To ensure style compliance and lessen back and forth with pull requests.

That plugin just ensures \n is last character of manifest, it won't catch extra newlines where file ends with \n\n.

Right, though I saw we were not using it and should.


bug: "The method eth_sendTransaction does not exist is not available"

When I try to call a contract method that requires a user signature, an error is returned to me:
On bsc(id:56) - code: -32000, message: "unknown account", No transaction comes to the wallet.
On mainnet(id:1) - code: -32601, message: "The method eth_sendTransaction does not exist is not available", The transaction does not come to the wallet.
{"jsonrpc":"2.0","id":43,"error":{"code":-32000,"message":"unknown account"}}
I expect that the transaction will come for confirmation in the connected wallet, after confirmation and payment of the fee, the transaction will be successful
async function approveRouter(address) {
const tokenAddress = "0xbb4cdb9cbd36b01bd1cbaebf2de08d9173bc095c"; // WBNB
const approveAddress = "0x10ed43c718714eb63d5aa57b78b54704e256024e"; // PCRouter v2
const tokenContract = new ethers.Contract(
	tokenAddress,
	erc20ABI,
	provider(56).getSigner(address) // address - connected user address
);
const maxApproval = ethers.constants.MaxUint256;

const txRes = await tokenContract.approve(approveAddress, maxApproval, {
	gasLimit: 310000,
});
console.log(txRes);

}
Browser: Google Chrome desktop

Transferred this issue to our rpc repo, where you'll get better help 👍

Okay, thx

Someone has ideas?

I resolve my issue, I just need to get signer from wagmi/core fetchSigner(), not from wc provider


United Meets - Schedule and Manage your meetings on the GO !

Project name
United Meets
Description
United meets is an Application that allows users to create or Schedule an Event/Meeting, Edit and Update an Event/Meeting, Delete an Event/Meeting, and View all the Events/Meetings from the user's Outlook calendar.
This Application allows users to authenticate using their Microsoft identity platform and the application internally connects with the Microsoft Azure Identity Platform to integrate with their registered app in Microsoft Azure and returns an access token to access resources and perform Authorized operations on the outlook calendar events.
I will be Updating my project as much as possible and adding new features to this project.
Repo URL
https://github.com/vaishnavdev/Graph-MeetingApp
Team members
Only me - P Vaishnav

Thank you for your submission @vaishnavdev!

Thank you for your submission @vaishnavdev!

Hi @waldekmastykarz as said in the hackathon details, when I am sending an email to<EMAIL_ADDRESS>with a subject line Hack Together: Microsoft Graph and .NET Sweepstakes winners to know the list of winners, I am getting a Message blocked response from the microsoft's server.
Please view the below image for details

We'll announce the list of winners publicly by Mar 30 and we'll notify the winners via email.


Add Warmup option that will load the site URI before authorization

Since you are adding a new argument, you need to add that argument to ScheduledRenewal.cs, and then make sure the value is saved when it writes to the Registry and retrieved for renewals.
Also, I see your PromptForWarmup method requires a response. Lots of people are having issues trying to automate everything and the prompts are stopping them. So is there any way to get rid of the prompt?

The PromptForWarmup argument adds the prompt during Authorize. If the response is yes it sets the added Warmup property in the Target. Since the Target is stored in the ScheduledRenewal, it does get saved. The Warmup property then gets checked during renewal Authorization.
I agree I'd like this without a user prompt, but I'm not sure how to make an argument that would be site-specific.

All arguments are only valid for the single run of the program.
Renewals should save the arguments in the registry and read them for each site. That's why you need to add your argument to the ScheduledRenewal.cs.
I added a note to the Wiki that specifies that arguments are applied to all sites it is ran for.
So if someone doesn't want it to apply to multiple sites, they just need to run it twice, once with it defined and once without it defined.

OK, that sounds good to me. This commit should make those changes. It did work in my testing.

Also, there were changes merged into program.cs so you might want to pull in the latest changes as well.

That should merge the changes correctly. I haven't had any problems with the bool serialization/deserialization. It seems to read/write it correctly, and it works if the property is not there as well.

If you tested a renewal with both it defined and not defined and the renewals ran, there there should be no issue.
If you can add the new argument to the Command Line Arguments page, and add a section or two to the How To Run page, i'll merge your changes in.

I've added documentation for the argument.

It has been merged in.


Ability to right-click resource box in the Inspector to access its menu

Version: Godot 3.0 (master)

Right now that menu is only accessible by clicking on the little arrow, but I think adding the ability to just right-click the entire box would help a little with productivity. I'm not necessarily saying the arrow should be removed, but having a sort of context menu appear upon being right-clicked would be fairly intuitive to most.

I also mentioned this here #9052. It was assigned as a junior job.

Yikes, can't believe I missed that. Sorry, closing this.


Add republishing rake tasks for non-edition content

This PR:

adds republishing for non-editioned content items
add a new task for backfilling content IDs for non-editioned content items

/cc @boffbowsh

@gpeng and I paired on this; we've pushed this branch to integration and it seems to work for Take Part pages and case studies.

:+1:

Having done further testing, it seems that the 2 loops in the edition republishing Rake task is causing conflicts in the Publishing API so we won't merge this yet. This can probably be fixed by saving each edition only once.

We've taken out the refactoring of the editioned content republishing because it has unresolved problems.

:+1:


Need chick.png

Need to comment the line below because the chick.png is not present in Git
sense.load_image("chick.png")

I have added the chick.png to the images folder, many thanks


How to read content of a row in excel worksheet

Hello,
My requirement is, I want to get the content of a given row in a given excel worksheet when I pass the row number. Currently there is no method available in ZCL_EXCEL_WORKSHEET to do that. All I can do is, first I can call ZCL_EXCEL_WORKSHEET->GET_TABLE( ) to get all the entries in an internal table and then I can get my required record by reading the internal table.
Can we have a method for this requirement if there is nothing already. I can contribute to develop this if you allow.
Thanks,
Mainak

Hi. In fact, you may use the method GET_TABLE to just read one line (here the line number 10). Just my opinion, I don't think it's worth creating a method, but of course other people in the team might think the opposite:
DATA(row_to_read) = 10.
worksheet->get_table( EXPORTING iv_skipped_rows = row_to_read - 1
                                iv_max_row      = row_to_read
                      IMPORTING et_table        = your_table ).
DATA(line_number_10) = your_table[ 1 ].

NB: anybody is permitted to propose a Pull Request, it will be reviewed by anybody of the team to see if it's to be directly merged, needs rework or clarification, and possibly rejected. Of course, as you are doing, people may open an issue first if they feel that the change needs to be discussed first.

Sure, I agree with @sandraros on this on the workaround. I am closing it.


Error while interacting with slider button

While following the instructions in the documentation for slider-setup, the following error emerges.
Uncaught TypeError: Cannot read property 'x' of null     at SVGGElement.<anonymous> (d3-simple-slider:2)     at Dispatch.apply (d3.v5.js:796)     at customEvent (d3.v5.js:1602)     at gesture (d3.v5.js:1967)     at SVGGElement.mousedowned (d3.v5.js:1890)     at SVGGElement.<anonymous> (d3.v5.js:1526)
This error emerges when interacting with the slider to change value.

I just updated to newest version of D3 and it started working for me again!

Hi! I made a mistake when publishing the new prelease v2.0 which meant you would get that rather than the stable 1.8.0. I've since updated the npm tags to ensure this doesn't happen

@janicekchen you're correct that upgrading to d3 v6 would fix it as v.2.0.0-rc.1 is intended to be the version that supports d3 v6.
Apologies for the inconvenience @MrMathur!

I can confirm that it works for me now. Thank you so much for the prompt fix!


feat(rel): introduce rel

Codecov Report

Merging #7 (85cce58) into main (648bfa7) will decrease coverage by 2.60%.
The diff coverage is 89.28%.

@@             Coverage Diff             @@
##              main       #7      +/-   ##
===========================================
- Coverage   100.00%   97.39%   -2.61%
===========================================
  Files           10        9       -1
  Lines          126      115      -11
===========================================
- Hits           126      112      -14
- Misses           0        2       +2
- Partials         0        1       +1

Impacted Files
Coverage Δ

pkg/cert/usecase/issuecert/issue_certificate.go
75.00% <33.33%> (-25.00%)
:arrow_down:

...nal/pkg/adapter/postgres/certificate_repository.go
96.66% <95.65%> (-3.34%)
:arrow_down:

...rnal/pkg/delivery/rest/handler/view_certificate.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Allow connecting to Azure SQL database using AAD auth

If Azure AAD is only way to connect to SQL database, it may not be possible to use mssql-cli. Please add support for Azure AAD auth & add sample command line in documentation
Related:
Azure AAD auth: https://docs.microsoft.com/en-us/azure/sql-database/sql-database-aad-authentication

Unfortunately, mssql-cli does not support currently support AAD auth. This is due to a limitation of SqlClient on .NET Core, which is used by the Microsoft/sqltoolsservice and mssql-cli to connect to SQL Server. There are plans to support this feature in SqlClient, however there aren't any timelines that I'm aware of that I can share.
The issue tracking AAD auth support in SqlClient on .NET Core is below. Please upvote.
https://github.com/dotnet/corefx/issues/8807

@pensivebrian - do you know if SQLClient has rolled out AAD support for .net core since? I am trying to find a link to any upstream issue/worktitem associated to that, do you know which repo it would be in?

Are there any plans to deliver this? It feels like a fairly fundamental feature that is missing.

any updates on this?
I wanted to update my net6.0 function app from the legacy package Microsoft.Azure.Services.AppAuthentication
that is currently doing:
// TODO: Migrate to Azure.Identity if possible, because Microsoft.Azure.Services.AppAuthentication is deprecated
                var tokenProvider = new AzureServiceTokenProvider();
                _log.LogInformation("retrieving sql access token...");
                string accessToken = await tokenProvider.GetAccessTokenAsync("https://database.windows.net/", Environment.GetEnvironmentVariable("Sql:TenantId"));
                var connection = new SqlConnection(_connectionString)
                {
                    AccessToken = accessToken
                };
                connection.Open();

When trying to use System.Data.SqlClient native auth like this:
string ConnectionString = @"Server=demo.database.windows.net; Authentication=Active Directory Default; Database=testdb;";

using (SqlConnection conn = new SqlConnection(ConnectionString)) {
    conn.Open();
}

it fails with error message, that auth keyword is not supported
If I work around this using Azure.Identity methods, it is getting incredibly slow:
var tokenProvider = new DefaultAzureCredential();
                _log.LogInformation("retrieving sql access token...");
                var tokenRequestContext = new Azure.Core.TokenRequestContext(scopes: new string[] { "https://database.windows.net//.default" }, tenantId: Environment.GetEnvironmentVariable("Sql:TenantId"));
                var accessToken = await tokenProvider.GetTokenAsync(tokenRequestContext);
                var connection = new SqlConnection(_connectionString)
                {
                    AccessToken = accessToken.Token
                };
                connection.Open();

Especially when using startup class and injecting the sql client, I would expect a reuse of the token. Instead, it seems, the code is getting a new token with every new call (it takes seconds to finish every single one)
What is the suggested method for accessing Azure SQL in net6.0?


What do I modify to test out my own translation?

I just finished translating but I could only commit instead of testing it locally and then commit, which potentially takes a few more steps for me and also other contributors to perfect the translation.
Thanks

The only thing you have to do is to add an entry for your language in App.xaml right at the top. You can see the format required from the other translations.
After you have done this you should be able to debug and your language should appear in the menu.


Add CSRF protection

Currently no form has CSRF protection.
Tokens are generated using
uniuri.New()
But no validation or session save is done yet

This is now on the main framework. I just need to add the csrfTokens on each controller / template

Commit c1fe771 adds the new filters used to validate and create csrf tokens
You may also need to use
[[ csrfField .csrfToken ]]
On your POST forms to add the token. On GET forms you must add a new URL param called
:token

Commit 3499c29 fixes this


Long table of contents add vertical space to code blocks if they overlap

[ ] This is a question about using the theme.
[ ] This is a feature request.
[x] I believe this to be a bug with the theme.

[x] I have updated all gems with bundle update.
[x] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version: 4.7.0
Jekyll version: 3.6.0
GitHub Pages hosted: yes
Operating system: macOS 10.13.1

Expected behavior

I expect the Table of Contents to not affect the height of the overlapping code blocks.
Steps to reproduce the behavior

When I switched from the toc helper to jekyll-toc (activated in the front matter), I noticed that long ToCs add extra vertical space to code blocks if they overlap.
An online example is here: http://gnss-sdr.org/coding-style/
The website content is here: https://github.com/gnss-sdr/geniuss-place.git
I always can switch back to the toc helper, which did not exhibit this behavior, but I wonder if this possible bug can be fixed in jekyll-toc. I'm quite illiterate in Liquid, so I'm afraid I can't help on this.
Thanks for your time and for your wonderful work on this theme.

I think something else is at play. From the looks of it you don't have all of the latest layouts, includes, and CSS.
If you take a look at the demo site you should see something like this at the top of your HTML source:
<!--
  Minimal Mistakes Jekyll Theme 4.7.0 by Michael Rose
  Copyright 2017 Michael Rose - mademistakes.com \| @mmistakes
  Free for personal and commercial use under the MIT license
  https://github.com/mmistakes/minimal-mistakes/blob/master/LICENSE.txt
-->

and this in your CSS:
/*!
 * Minimal Mistakes Jekyll Theme 4.7.0 by Michael Rose
 * Copyright 2017 Michael Rose - mademistakes.com | @mmistakes
 * Licensed under MIT (https://github.com/mmistakes/minimal-mistakes/blob/master/LICENSE.txt)
*/

When I check yours there's no version in the HTML and your CSS has:
/*!
 * Minimal Mistakes Jekyll Theme 4.5.2 by Michael Rose
 * Copyright 2017 Michael Rose - mademistakes.com | @mmistakes
 * Licensed under MIT (https://github.com/mmistakes/blob/master/LICENSE.txt)
*//

I made some layout adjustments to the right sidebar a few versions ago and doesn't look like your fork incorporates those. You can see in this sample post on the demo site the TOC sidebar doesn't overlap code blocks.
On yours it's also overlapping notice blocks, which tells me it's using the older layout and CSS. This was one of the issues I fixed.
If you haven't made any changes to the theme's core files I'd suggest remove them from your fork, and pulling in the latest _layouts, _includes, _sass, etc. Just to verify if it's your content or theme. I have seen jekyll-toc do some weird things, but that has more to do with extra headings showing up than any sort of overlap with main content.

I see, my mistake then. I will update that. Thanks for looking it out.


Hello and Echo Contracts

Depends on https://github.com/zeta-chain/example-contracts/pull/201

@coderabbitai review


IntelliSense Error When Modifying Static Properties in JavaScript Static Methods (Using this.xxx)

🔎 Search Terms
static member, static property, JavaScript, ES5, this, class
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
No response
💻 Code
function Foo() {}

Foo.bar = "123";

Foo.baz = function() {
  this.bar = "456";
}

🙁 Actual behavior

It shows that 'bar' is a property of 'baz'.
🙂 Expected behavior
In theory, 'bar' should be a property of 'Foo.' This situation occurs when using this.bar instead of Foo.bar. However, using this makes it convenient for class inheritance.
Additional information about the issue
No response

When we see a this assignment, we assume what we're looking at is either an instance method (which this clearly isn't) or a standalone class declaration (i.e. it's a constructor function), i.e. it looks like you are going to write const m = new Foo.baz(). Inference can only guess one way or the other.

Thanks for the explanation! However, I noticed that if a static method is called rather than assigning a value, the inference works fine.

Additionally, it works when using ES6 class syntax, or when compiling as TypeScript instead of JavaScript.

Thanks for the previous explanation. I think the key code might be here:
https://github.com/microsoft/TypeScript/blob/2ac4cb78d6930302eb0a55d07f154a2b0597ae32/src/compiler/checker.ts#L36831
/**
 * Indicates whether a declaration can be treated as a constructor in a JavaScript
 * file.
 */
function isJSConstructor(node: Node | undefined): node is FunctionDeclaration | FunctionExpression {
    if (!node || !isInJSFile(node)) {
        return false;
    }
    const func = isFunctionDeclaration(node) || isFunctionExpression(node) ? node :
        (isVariableDeclaration(node) || isPropertyAssignment(node)) && node.initializer && isFunctionExpression(node.initializer) ? node.initializer :
        undefined;
    if (func) {
        // If the node has a @class or @constructor tag, treat it like a constructor.
        if (getJSDocClassTag(node)) return true;

        // If the node is a property of an object literal.
        if (isPropertyAssignment(walkUpParenthesizedExpressions(func.parent))) return false;

        // If the symbol of the node has members, treat it like a constructor.
        const symbol = getSymbolOfDeclaration(func);
        return !!symbol?.members?.size;
    }
    return false;
}

Ye, that piece is relevant here: return !!symbol?.members?.size;. Together with the binder's code here:
https://github.com/microsoft/TypeScript/blob/2ac4cb78d6930302eb0a55d07f154a2b0597ae32/src/compiler/binder.ts#L2889
The += case exits here:
https://github.com/microsoft/TypeScript/blob/2ac4cb78d6930302eb0a55d07f154a2b0597ae32/src/compiler/utilities.ts#L4096-L4098
That is why you see a different behavior in your latest example


Link Policy to Application

I created a new policy as: New-AzureADPolicy -Definition @('{"TokenLifetimePolicy":{"Version":1,"AccessTokenLifetime":"02:00:00","MaxAgeSessionSingleFactor":"02:00:00"}}') -DisplayName "WebPolicyScenario" -IsOrganizationDefault $false -Type "TokenLifetimePolicy"
I am now looking to link it to one of my SAML web applications used for single-sign-on.
Add-AzureADApplicationPolicy -Id  -RefObjectId  I believe is the command.  For the -Id I went into my Azure portal and got the ObjectId of the SAML web application and the RefObjectId I ran the Get-AzureAdPolicy command to get the ID of the newly created policy.  However, I get an error of"
Add-AzureADApplication Policy: Error occurred while executing the policy
Code: Request_ResourceNotFound
Message: Resource {} does not exist or one of its queried reference-proprty objects are not present.
This is in reference to https://docs.microsoft.com/en-us/azure/active-directory/active-directory-configurable-token-lifetimes#configurable-token-lifetime-properties

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

ID: 02ff3d62-dc0e-ad0c-f5ae-418337cf31e4
Version Independent ID: d50952b5-6b98-c40d-d3a3-f9cbec58dd28
Content: Configurable token lifetimes in Azure Active Directory
Content Source: articles/active-directory/active-directory-configurable-token-lifetimes.md
Service: active-directory
GitHub Login: @hpsin
Microsoft Alias: hirsin

@scott87z I am able to run these commands without any issues.  Please make sure you are passing the correct objectId of the application.  Also, I suggest you please open a new MSDN forum post with the details and community will help address your query. Please refer to below link to MSDN forum to ask new question - https://social.msdn.microsoft.com/Forums/azure/en-US/newthread?category=windowsazureplatform&forum=WindowsAzureAD&prof=required.  Please provide the MSDN link over here so that I can help you getting answered.

@scott87z We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Purchase 1000 HEC Tokens or maximum value $16000

The amount requested to transfer: upto $16000 usdc
To purchase 1000 HEC TOKENS on execution of the vote if passed. ( or the maximum of tokens for $16000 usdc funds to be released from diego wallet as this would be the wallet farming is processed from)
Wallet.
0x644c4A0708aD182dA244DBe00fDc0D19110e6fD3
THREE parts to the proposal.

PURCHASE up to $16000 or 1000 HEC tokens which ever format is the cheapest at time of purchase

PLACE The HEC tokens on there staking pool paying between 16/20% APY

If the HEC tokens reaches a value of $50.00 the amount is withdrawn with yield made and initial capital swapped back to usdc (profits from the token price and yield will be split 90% treasury and 10% buyback and burn.

IF the HEC token dosent reach target sell price within 180 days the token is to stay yielding for a further 180 days. (During this period a new proposal will be made for an exit strategy)

Pros :-
THE HECTOR INSTITUTE has proven itself to be a well established DAO. They operate on a similar principle to agfi buyback/burn. They just announced they have sponsored BVB (Borussia Dortmand FC) champions league football
HEC TREASURY has over 60,000,000 in wallet and over 30,000,000 in farming services. That's now on the bearish market conditions. A few assets in treasury wallet are the following
BTC 123
ETH 811
QNT 9826
AVAX 29662
MATIC 541626
FTM 1410855
NEAR 59566
it gives Agfi the chance to yield up to 20% on part of our treasury in this continued bear market.
Potential 34000 profit for agfi treasury plus the extra yielded tokens if target price met.
Cons:-
HEC Token is not a stable token. Price is subject to votality.
If target price is not met the token will be yielding for 360 days giving apx 200 HEC but the token maybe worth less than entry price.
Overall my thoughts are we have not much farming going on its potential to yield some hec. I always believe the market will turn aventually also I really like the fact  they are proving themselves in the crypto space in more ways than one.
Hec contract 0x5c4fdfc5233f935f20d2adba572f770c2e377ab0
Hec dashboard
app.hector.network
Many thanks

Hi Cryptobuzz1,
Is the staking is for a minimum period of 180 days or can be remove at the convenience?
The idea is great, I'm just thinking about a serious correction on the market after the mid-term on US if they can handle until there.
Regards

There is no time lock the funds in staking pool can be removed at any time.
It is very similar to agfi just cost gas which is less than $0.05
So if a new proposal was made to remove this will bot be an issue.
All gas fees for transfers/staking are in FTM but the agfi wallet holds a
few FTM
On Sat, 27 Aug 2022, 08:08 AGFITRUST, @.***> wrote:

Hi Cryptobuzz1,
Is the staking is for a minimum period of 180 days or can be remove at the
convenience?
The idea is great, I'm just thinking about a serious correction on the
market after the mid-term on US if they can handle until there.
Regards
—
Reply to this email directly, view it on GitHub
https://github.com/aggregatedfinance/agfi-dao/issues/13#issuecomment-1229139308,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZKIA5LP2WLOUTNNOJQJZS3V3G5GZANCNFSM57YIC7BQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for the update.
If this go to vote, I will be FOR

I would like to also make the point. That with this proposal focused on farming the funds should come from Diego's wallet as were all aware he is not working for agfi with no payment operations in place.
Wallet
0x644c4A0708aD182dA244DBe00fDc0D19110e6fD3
Usdc balance 16995. (Which would cover cost of the proposal)

Header amended with the changes

Juat another thought on this proposal if was successful. If tweeted to there official site maybe 1 of there 75k plus supporters may look into our small market cap AGFI.

Juat another thought on this proposal if was successful. If tweeted to there official site maybe 1 of there 75k plus supporters may look into our small market cap AGFI.

Or, they see us as a FaaS which cannot grow our own treasury, and this is the whole point of AGFI.....

After reviewing it, the proposal managers decided to not allow this proposal. Our first mission is to clarifiy if such a proposal could put agfi into peril. We have come to the belief that specific, direct requests for asset purchases place the stability of the Treasury at risk and those decisions should be taken by a professional/delegated trader who has gone through KYC.


Add build rules to find OpenVINO inference engine

Add cmake rules to find and link with Inference Engine when the nGraph IE
backend is enabled.

Why do you need this PR? Why don't you use Inference Engine CMake configs? It should be enough.

Why do you need this PR? Why don't you use Inference Engine CMake configs? It should be enough.

Hi @ilyachur, this is to be able to build the Inference Engine backend outside of the dldt repo.

Why do you need this PR? Why don't you use Inference Engine CMake configs? It should be enough.

Hi @ilyachur, this is to be able to build the Inference Engine backend outside of the dldt repo.

@adk9 From my point of view it is a work around. Tomorrow the Inference Engine will introduce a new library and it will cause CI fails and changes in this file
If you build Inference Engine separately the CMake scripts generates CMake configs and you can use them. The OpenVINO package also contains the CMake configs. I don't see any case when you need this custom CMake config.

I agree with @ilyachur. If there are already available tools for this in IE, we should use them. Maybe we could use find_package mechanism?

Why do you need this PR? Why don't you use Inference Engine CMake configs? It should be enough.

Hi @ilyachur, this is to be able to build the Inference Engine backend outside of the dldt repo.

@adk9 From my point of view it is a work around. Tomorrow the Inference Engine will introduce a new library and it will cause CI fails and changes in this file
If you build Inference Engine separately the CMake scripts generates CMake configs and you can use them. The OpenVINO package also contains the CMake configs. I don't see any case when you need this custom CMake config.

Fair enough. I just use find_package now to locate the CMake configs distributed with the Inference Engine release.

TESTNOW_DLDT_E2E

TESTNOW


changed the defvar c3po-model argumento from "gpt-3.5" to "gpt-4" but still getting errors, any help, please?

Hi, everyone!
As said in the title, I was having the following error when trying with this gorgeous contribution. So tried changing the defvar argument at line 36 in the source code from gpt-3.5-turbo to gpt-4 but not success, I keep getting the same error, any help, with this, please??
error in process filter: Wrong type argument: arrayp, nil

Can you try again? It might be you didn't have access to gpt-4 API last week.
Funny enough, since last week gpt-4 API access is GA and should be available.
https://openai.com/blog/gpt-4-api-general-availability

@touxstone I've published a new version, please give a try if you can change the model.
This version is the one that I've been using internally for many months and I can change easily the model between 3.5 and 4 with c3po-toggle-c3po-model

I have a similar problem. After doing toggle-debug-on-error. The error output is:
Debugger entered--Lisp error: (wrong-type-argument arrayp nil)
  c3po--extract-content-answer((:error (error http 429) ...
  url-http-activate-callback()
  url-http-content-length-after-change-function(382 605 223)
  url-http-wait-for-headers-change-function(1 618 617)
  url-http-generic-filter(#<process api.openai.com> "HTTP/1.1 429 Too Many Requests\15\nDate:...

I hope this helps on detecting the problem :)

@ferzcam if you change the model to gpt-3.5 would it work?
It seems the error is related to rate limiting.
Might want to check the full error message to get the full error details, possible with the cause of the rate limiting.
https://help.openai.com/en/articles/6891829-error-code-429-rate-limit-reached-for-requests
https://help.openai.com/en/articles/5955604-how-can-i-solve-429-too-many-requests-errors

Hi @d1egoaz. I have tried both models (3.5 and 4) but it does not work. I have been using this package for a long time and this error used to happen sometimes only. However, now it seems to happen always. Not sure what is the issue :')

See above, I think you're going over your plan limit of API calls.

@ferzcam does an API call via curl works for you?
curl -s https://api.openai.com/v1/chat/completions \
  -H "Content-Type: application/json" \
  -H "Authorization: Bearer YOUR-API-KEY" \
  -d '{
    "model": "gpt-3.5-turbo",
    "messages": [{"role": "user", "content": "Hello world"}]
  }'

@d1egoaz I cannot do API calls via curl. I noticed what my problem was. I was using a free account and it turns out that API requests had initial free credit that at some point expires. Thanks for taking your time helping on this.

@touxstone I've published a new version, please give a try if you can change the model. This version is the one that I've been using internally for many months and I can change easily the model between 3.5 and 4 with c3po-toggle-c3po-model

Hi, I have just tried it but not success, the output I got is the following to give an idea:
Contacting host: api.openai.com:443
error in process filter: let*: Wrong type argument: arrayp, nil
error in process filter: Wrong type argument: arrayp, nil

@touxstone can you get the current value of the variable? (describe-variable 'c3po-model)
and try running M-x toggle-debug-on-error and then try the variable toggle, the contents of the *Backtrace* might help to verify why it doesn't work for you.


[epic] Remove horizon dependency for sendTransaction and getTransactionStatus endpoints

soroban-rpc currently uses horizon for submitting transactions and obtaining transaction results. Given that soroban-rpc is now running captive core, we should be able to come up with a new architecture where transaction submission in soroban-rpc does not depend on horizon.
This issue will be complete when:

[ ] the design for soroban txsub without horizon is proposed and approved
[ ] the implementation of the design is merged
[ ] remove horizon from soroban rpc requirements, it is not required by rpc runtime. Update existing rpc deployment artifacts to reflect the change, quickstart soroban-dev,  reference to horizon in system-test, i.e. remove the wait loop on horizon readiness since not needed anymore, and sdf helm chart config

We should make sure the design and implementation also addresses these related issues:

[ ] https://github.com/stellar/go/issues/4642
[ ] https://github.com/stellar/soroban-tools/issues/372
[ ] https://github.com/stellar/soroban-tools/issues/339

(This is a comment about getTransactionStatus, not sure if this is the right place - happy to move it)
I suggest that we:

rename getTransactionStatus to getTransaction
remove the pending option from status
only serve transactions that have been ingested

Reasoning: core doesn't provide any information about the task queue and making assumptions about it, like a transaction being pending, is dangerous. For the sake of simplicity I propose that soroban-rpc provides only "known for sure" information, which is information produced by ingestion.


404 at /docker-for-azure/

I found a broken link : https://docs.docker.com/docker-for-azure/
I arrived on this page through: https://github.com/docker/docker.github.io/issues/11899

This appears to be a duplicate of #11899. Closing here, lets track this in the other issue.

This appears to be a duplicate of #11899. Closing here, lets track this in the other issue.


webpack Unexpected token on DropzoneComponent

Hi, I am trying a build using webpack and getting an error.
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected token (68:4)
line 68 char 4 ->     <DropzoneComponent
I have this at the top of the file:
import React from 'react';
import ReactDOM from 'react-dom';
import DropzoneComponent from 'react-dropzone-component';
also tried:
import DropzoneComponent from 'react-dropzone-component/lib/react-dropzone';
neither work.
Your docs state, "Ensure that React and ReactDOM are global variables", how do I do that?

doh! newbie error. fixed by explicitly running webpack with it's config.
i.e.  webpack --config webpack-config.js


Dead Letter Exchange

I ended up with a message in my queue that failed to de-serialize, but since a failure in serialization just nack'd and re-queued, my consumer was stuck in an endless loop.
I decided I'd rather have all nack'd messages end up in a dead letter queue, so I've added support for that. Let me know what you think!

My primary concern with this change is that we'll deadletter for any failure. I'd rather only deadletter things that fail repeatedly. Any thoughts on how we could do that?

Agreed, but let's have this discussion in the related private PR.

Pinging @ryanuber

@danbrakeley in general I agree we should solve the redelivery problem if some bad JSON makes it into a relay queue. I think the dead-lettering should be specifically for the serialization failure case though and not applied broadly to Nack(). There are many cases where Nack() is called due to non-error conditions. Let's dead-letter only the messages we know relay can never handle safely.


Text composables: Android Basics with Compose

URL of codelab
**Step 3 **
**The compose function that returns nothing and bears the @Composable annotation MUST be named using pascal case. Pascal case refers to a naming convention in which the first letter of each word in a compound word is capitalized. The difference between pascal case and camel case is that all words in pascal case are capitalized. In camel case, the first word is not capitalized.
I have given camel case but still worked.  As per documentation composable function MUST be names as pascal.
**
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version: Android Studio Flamingo
_API version of the emulator: _
Additional information
_
_

Hi @srikanth104
Thank you for writing in with your question.
These instructions are referring to the API Guidelines for Jetpack Compose which are the guidelines to follow for best practices.
The "MUST" in this context refers to what the guidelines require in order to adhere to them. It is a coding convention and not a requirement of Jetpack Compose nor Android Studio.
Android Studio does not enforce this as you have seen.
Hope you are enjoying the course!
Regards.


Need For Speed™ Most Wanted [PCSG00106]

App summary

App name: Need For Speed™ Most Wanted
App serial: PCSG00106
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3049
Commit hash: https://github.com/vita3k/vita3k/commit/ea1b3837
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: AMD
GPU: NVIDIA
RAM: 32 GB

Issues
Screenshots
Log
PCSG00106 - [Need For Speed™ Most Wanted].log
Recommended labels
bootable
crash

should used opengl renderer running again. before, it works under previous versions of vulkan renderer

i don't like opengl🙃

opengl


precommit runs twice when using commitizen

I'm using Husky. I have:
"scripts": {
   "precommit": "lint-staged --verbose",
   "commit": "git-cz",
},
"lint-staged": {
    "*.js": [
      "prettier --single-quote --semi=false --write",
      "eslint --fix",
      "git add"
    ]
},
"config": {
    "commitizen": {
      "path": "./node_modules/cz-conventional-changelog"
    }
},

At first I thought perhaps the precommit script isn't needed anymore, so I remove the it and lint-staged doesn't run at all. Then I decided to commit without commitizen and realized that's where the problem is. lint-staged is called once before the commitizen command line form appears, and again after you submit it. Is that the expected behavior? What can be done about it?

If you always commit using npm run commit you could add lint-staged before calling git-cz and remove the precommit hook. I would not advise doing this, though.
Regarding running twice: it should not. Looking at your config I can't see the reason for that. So you should investigate and create a reproducible example on GitHub if the problem is with this package.

I have same issue here
https://github.com/Hotell/typescript-lib-starter

ha! so this isn't definitely issue with this library, rather than husky is the culprit here. I guess you can close this @okonet
https://github.com/typicode/husky/issues/110
thanks!

PR would be awesome! Thanks for investigating.

cheers ! https://github.com/okonet/lint-staged/pull/182

Is there an issue in the husky repository to fix this?

humm so juts to confirm steps to get this working:
"devDependencies": {
   "cz-conventional-changelog": "^2.1.0",
   "husky": "^0.14.3",
},
"commit": "git-cz",
 "husky": {
   "hooks": {
     "pre-commit": "lint-staged"
   }
},

is all you need?
Because when I run git cz with the above config, it goes straight to commitizen and does not run any test.

@wearefridayhutber I get the same. My approach has been to simply run a specific script before husky.
"devDependencies": {
   "cz-conventional-changelog": "^2.1.0",
   "husky": "^0.14.3",
},
"scripts": {
  "commit": "lint-staged && git-cz --no-verify",
},
"husky": {
   "hooks": {
     "pre-commit": "lint-staged"
   }
},

The --no-verify flag prevents husky from being run after commitizen.
Or

"devDependencies": {
   "cz-conventional-changelog": "^2.1.0",
   "husky": "^0.14.3",
},
"scripts": {
  "pre-commit": "lint staged",
  "commit": "yarn run pre-commit && git-cz --no-verify",
},
"husky": {
   "hooks": {
     "pre-commit": "yarn pre-commit"
   }
},

This seems to be happening again with<EMAIL_ADDRESS>@jcarroll2007's fix from the above comment seems to do the trick, though.


Add time localisation

[x] Time of the day (often just "time")
[x] Date

Might be implementable as <time value="01:01:01Z" data-localise-as="time">02:01:01</time>, <time value="2442-01-21" data-localise-as="date">21.01.2442</time> and <time value="2442-01-21T01:01:01Z" data-localise-as="datetime">21.01.2442 02:01:01</time>

I left out datetime for now, as I didn't find anything about official datetime formats in Germany

Timezones are still missing


Wallet is not notified if the transaction cannot be fulfilled

When attempting to spend not matured/recently staked coins the transaction is not accepted to the memory pool and the wallet is not notified of this.
As a result the wallet shows that the transaction has been sent and is pending confirmation wheres in fact the transaction was never shared across the network.

Ah ok this is a bug and a duplicate of #1255

You can say is a duplicate as it is related to the same issue but in general we should have a way to let the wallet know about transactions which cannot be completed for this or other reasons.

This task to me is about fixing how we broadcast transactions, three are some issues.
We should move all broadcasting code to the wallet and remove the states (or move them to the light wallet)
The main wallet should relay on memepool
Light wallet can try to rely on ping backs from peers
The implemented of this please make sure to discuss with me and @bokobza

This will require integration tests as well.
Ideally we'd have a failing integration test reproducing this scenario first, then work on fixing it.

After creating the integration test (not committed yet), when trying to spend CoinStake before maturity the exception is throw here in PosConsensusValidator.CheckMaturity().  This doesn't get caught by the WalletController and is why the UI doesn't report the failure.

As Dan mentioned above we need to see how to broadcast this so that the wallet manager picks up the failed transaction.  Which I am working on.


[Youtube] subscriber counter/ total view time

:clipboard: Description

Service: Youtube

:link: Data
-Youtube API   (I don't know exactly how)

:microphone: Motivation
Maks the information on an internetsite visible to show how "good" the youtube channel is.

Hello @Gamemap ! 👋🏻
I believe we can get the number of subscribers to a channel using the following API: https://developers.google.com/youtube/v3/docs/channels
However, after a quick read through the documentation, I see no way of retrieving a total view time. Could you please have a browse through as well to make sure I haven't missed anything?

Hello @Gamemap ! 👋🏻
I believe we can get the number of subscribers to a channel using the following API: https://developers.google.com/youtube/v3/docs/channels
However, after a quick read through the documentation, I see no way of retrieving a total view time. Could you please have a browse through as well to make sure I haven't missed anything?

Hi,
in the Json list I found this:

"statistics": {
"viewCount": unsigned long,
"subscriberCount": unsigned long,  // this value is rounded to three significant figures
"hiddenSubscriberCount": boolean,
"videoCount": unsigned long
},

The tabel says:

statistics.viewCount |   The number of times the channel has been viewed.

Is this we are searching for?
Could you also add the total videos count?

statistics.videoCount |  The number of public videos uploaded to the channel. Note that the value reflects the count of         the                  >                                    channel's public videos only, even to owners. This behavior is consistent with counts         shown on the >                                    YouTube website.

Thank you,
Gamemap

Hi,
in the Json list I found this:

"statistics": {
"viewCount": unsigned long,
"subscriberCount": unsigned long,  // this value is rounded to three significant figures
"hiddenSubscriberCount": boolean,
"videoCount": unsigned long
},

The tabel says:

statistics.viewCount |   The number of times the channel has been viewed.

Is this we are searching for?
Could you also add the total videos count?

statistics.videoCount |  The number of public videos uploaded to the channel. Note that the value reflects the count of         the                  >                                    channel's public videos only, even to owners. This behavior is consistent with counts         shown on the >                                    YouTube website.

Thank you,
Gamemap

statistics.viewCount |   The number of times the channel has been viewed.

Is this we are searching for?

Well, not really, that gives us no clue of the total view time for a given channel. Videos in a channel will have different lengths and most users would not watch them in full anyway, so you can't link the number of users who pressed play to a duration.

statistics.viewCount |   The number of times the channel has been viewed.

Is this we are searching for?

Well, not really, that gives us no clue of the total view time for a given channel. Videos in a channel will have different lengths and most users would not watch them in full anyway, so you can't link the number of users who pressed play to a duration.

Is this the time, which is displayed on the Youtube Creators site?

Is this the time, which is displayed on the Youtube Creators site?

Is this the time, which is displayed on the Youtube Creators site?

I wouldn't know for sure, I don't think I've ever used that part of YouTube before.

Is this the time, which is displayed on the Youtube Creators site?

I wouldn't know for sure, I don't think I've ever used that part of YouTube before.

You find at https://studio.youtube.com the subscribers and the monetization tab shows the viewtime (if you don't have reacht the goal) (I found only this place with viewtime)

You find at https://studio.youtube.com the subscribers and the monetization tab shows the viewtime (if you don't have reacht the goal) (I found only this place with viewtime)

Okay, so coming back to my initial question, you are confirming that there is no way to retrieve the total view time via the API?

Yes, I think so.

I found this: https://developers.google.com/youtube/reporting/v1/reports/metrics#watch_time_minutes
There you can also find subscribers_gained or other cool statistics.

As far as I can tell, that API is deprecated and there doesn't seem to be an obvious equivalent in the v3 one.

Ok

The total subscriber badge can still probably be done though, feel free to submit a pull request. We've got an nice tutorial on how to add badges and you can also leverage the implementation I wrote for the existing Youtube badges. 😉

Thank you for this tutorial but unfortunately I don't have time to learn this at the moment. If you or someone else have the time to code it would be great but there is no hurry.

This has now been deployed to production, thanks to @tarun7singh for his contribution! 🍾

Thank you very much!


Dial's algorithm for shortest path. (Python)

Hello project maintainers. I would like to add this algorithm to this project. Could you assign this to me?

I want to work on this issue. Please assign to me

I would like to contribute to the above problem in java as I am a participant of GSSOC'21. Kindly assign this to me.


Allow for whitespace after "time":

Previously JSONHandler.TryHandle was searching for "time":", including
the trailing quote. This breaks if there is whitespace after the colon,
as some JSON encoders produce. To fix this, check for the time field
after unmarshalling rather than searching the input string.

@adamwg sorry I just noticed this!

weeee, thanks @adamwg !


Can't load as an unpacked extension.

Even tried loading in the 'Public' folder where the .manifest is but still shows 'Cannot find manifest'

Download newest release, unpack it and import it using chrome.
To build on your own:
In root of project
yarn                           # install packages
yarn build:dev          # Build

And now import dist folder.


feat: Filament Account Management

Closes Linear WEB-126

@AxonC this is probably ready for review when you feel up to it 😂 Majority of the files changed are refactors

@AxonC this is probably ready for review when you feel up to it 😂 Majority of the files changed are refactors

All LGTM - I would like to still have the ability to seed the old local permissions locally. On that point, have we checked the new and the old can co-exist?

@AxonC this is probably ready for review when you feel up to it 😂 Majority of the files changed are refactors

All LGTM - I would like to still have the ability to seed the old local permissions locally. On that point, have we checked the new and the old can co-exist?

I've kept the old permissions in the migrations - these seemed to be duplicated in the seeder and would be in the database upon migration. Once we are done with the old permissions, we should remove them from the migrations (and prod DB manually).
Shouldn't be any conflicts, and even if there were the seeder is setup to only create if the permission doesn't exist, so should be fine!

@CalumTowers please could the following be added to the Forge deployment, after php artisan migrate:
php artisan db:seed --class=RolesAndPermissionsSeeder


Error installing npm

If this is a question about how to use NVM4W, please use stackoverflow instead.
If this is an issue regarding antivirus, make sure you search the existing issues first.
My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[ ] Windows 8.1

[-] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[ ] My Windows installation is non-English.

I'm using NVM4W version:

[-] 1.1.7
[ ] 1.1.6
[ ] 1.1.5
[ ] 1.1.4
[ ] 1.1.3
[ ] 1.1.2
[ ] 1.1.1
[ ] Older
[ ] OTHER (Please Specify)

I have already...

[-] read the README to be aware of npm gotchas & antivirus issues.
[-] reviewed the wiki to make sure my issue hasn't already been resolved.
[-] verified I'm using an account with administrative privileges.
[-] searched the issues (open and closed) to make sure this isn't a duplicate.
[-] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support (Have you tried version 1.1.0+?)
[ ] 32 or 64 bit support (Have you tried version 1.1.3+?)
[ ] Character escaping (Have you tried version 1.1.6+?)
[-] A standard shell environment (terminal/powershell)
[ ] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Expected Behavior
Install node 10.16.3 and npm 6.9.0
Actual Behavior
Downloading node.js version 10.16.3 (64-bit)...
Complete
Downloading npm version 6.9.0... Error while downloading https://github.com/npm/cli/archive/v6.9.0.zip - Get https://github.com/npm/cli/archive/v6.9.0.zip: Proxy Authentication Required
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x20 pc=0x5c6eb0]
goroutine 1 [running]:
/C/Users/corey/Documents/workspace/nvm-windows/src/nvm/web.Download(0x14f7c6f0, 0x2d, 0x15370000, 0x38, 0x150a66e0, 0x5, 0x0)
C:/Users/corey/Documents/workspace/nvm-windows/src/nvm/web/web.go:68 +0x240
/C/Users/corey/Documents/workspace/nvm-windows/src/nvm/web.GetNpm(0x14fce211, 0x24, 0x150a66e0, 0x5, 0x3e)
C:/Users/corey/Documents/workspace/nvm-windows/src/nvm/web/web.go:163 +0x21b
main.install(0x14fbe078, 0x7, 0x660283, 0x2)
C:/Users/corey/Documents/workspace/nvm-windows/src/nvm.go:268 +0x626
main.main()
C:/Users/corey/Documents/workspace/nvm-windows/src/nvm.go:78 +0x8cd
Steps to reproduce the problem:
nvm install 10.16.3
[proxy is configured and i have attempted running the command with --insecure]
[i've attempted with multiple different nvm versions and i'm receiving the same npm error]

The --insecure only applies to accepting TLS/SSL certificates without validating the certificate authority who issued it. It is not related to common proxies.
This issue is explicitly stating Proxy Authentication Required, meaning your proxy requires some sort of user credential. Bottom line is NVM4W can't reach the internet because the proxy is actively blocking it.
Depending on your proxy and how it handles auth, you may be able to to supply a proxy like<EMAIL_ADDRESS>This would produce a basic authentication token, but that's the extent of what NVM4W can currently do. This sets the Authorization HTTP header of the request, which some proxies respect. However; NVM4W does not set a Proxy-Authorization header. It's something that could potentially be added to a future version if demand is high enough.
I'm closing this since the issue is the proxy (not NVM4W) blocking requests, potentially resolvable by passing in credentials.


Security headers are not added to the ingress file when adding a new site

Description
When adding a new site in the admin panel, the following security headers are not set:

X-Content-Type-Options
X-Frame-Options
X-Xss-Protection
Referrer-Policy

Suggested fix
These headers can be added to the ingress file through the configuration snippet:
nginx.ingress.kubernetes.io/configuration-snippet: |
    more_set_headers "X-Content-Type-Options: nosniff";
    more_set_headers "X-Frame-Options: SAMEORIGIN";
    more_set_headers "X-Xss-Protection: 1";
    more_set_headers "Referrer-Policy: same-origin";

Related ticket: https://trello.com/c/0z7gIphE/90-security-headers-toevoegen-aan-ingress-bij-aanmaken-van-een-nieuwe-site

Resolved


Improved doll sync

4.x version: https://github.com/GodotNetworking/network_synchronizer/pull/31

Removed the Interpolator: now the interpolation is handled manually.
Simplified the Interpolation: now the doll doesn't store all the received epoch, it interpolates directly to the most recent one.
Improved the lag compensation: the algorithm that detects the network oscillations is faster and more accurate.
Dropped the concept of batch: it will be re-introduced later by batching all the info to send to a specific client.                                                                                - Simplified the network settings.
Improved logging.

@npeaks Great review :) I've changed it slightly can you please review it again?

Thanks for the review!


Integrate with go test tool?

Is is possible for me to run my features using the go test command?
I would like to have a test file with maybe a single test function which will invoke the godog runner command with a suite which is prepared by it. Then I wouldn't need to depend on the godog command to run my tests. I can just integrate it will all my other tests.

in general it is possible to run it but it needs command line arguments. The argument is a function where you register your contexts. You probably can copy os.Args and modify them for the godog and set them back after running godog. I'll probably add an example later. I'd suggest you to use TestMain to run godog before or after go tests and setup database or whatever is necessary for your suite.

Anyway my boss implmented cucumber over the weekend for our test cases, Since he needed some features that were lacking.
Here's the link check it out it https://github.com/playlyfe/cucumber

what features were required in your case?

Backgrounds and Data tables
https://github.com/cucumber/cucumber/wiki/Scenario-outlines
https://github.com/cucumber/cucumber/wiki/Background

these are all supported.. backgrounds and outlines. gherkin 3 is fully supported. the event stream is also currently being implemented in the pull request.. anyway, guess your boss wanted to simply rewrite it on his own :) cheers

Hi @l3pp4rd, I've been facing a rather sticky problem when taking the approach you suggested to run go-dog from within tests. The issue is that go-dog uses the golang flags system which has a global state. This is conflicting with a logging package glog which also uses the flags system. I think it would be best to provide an alternative run API which will allow the running of go-dog using programmatic parameters instead of command line flags.

yes @atrniv I was thinking about it. will resolve it when I have time for it ;) moving this to #50

@atrniv resolved by #50


Conv2d d_w and im2col unit tests

This PR implements computing the gradients for the convolutional layer. Since I had a some trouble with reshaping/transposing I also ended up implementing a unit test for im2col.

@koenvanderveen want to get the linting errors fixed (make lint) so we can get this merged up?

Hey @koenvanderveen. Awesome PR! The tests were broken for an unrelated thing to your stuff. If you want to merge with master and fix conflicts and make sure the tests will pass I'm sure we can get this merged up.

+1


Idle detection limit seems hardcoded to 15 seconds.

OS version: Linux Kubuntu 24.04
Browser version: Chrome 127.0.6533.119
Extension version: 4.0.4

Relevant integration (if any):
none
🐛 Describe the bug
The idle detection time limit seems to be hardcoded to 15 seconds in this version. I've set it to 180 in the extension settings, restarted the whole system to make sure it picked up, and it still shows 180 in the settings, But warns me if I stop working for 15 seconds.
Expected behaviour
It should act according to the time limit set in the extension options.
Steps to reproduce
Steps to reproduce the behaviour:

Enable idle detection and set it to 100 seconds.
Stand still for 20 seconds.
Move the mouse.
The extension warns you that you have been idle for 5 seconds.

Other details or context
I've tried login out and back in, restarting chrome, restarting the machine and even reinstalling the extension.

Closing as already fixed


[BUG] Swapping between EVM chains that are not available

Describe the bug
When changing swap routes to swaps between EVM chains and tokens, the page does not get updated and able to proceed with the last viable sawp
To Reproduce
Steps to reproduce the behavior:

Go to swap
Click on a available swap
schange to swap between EVM that is not available
See error and able to proceed swapping

Expected behavior
Display errors when a route is not available and block the progress
Screenshots

Version 27

closing , duplicate of 1174, in favor of #1174


Bugfix ionic.d.ts

Ionic typings will not compile due to syntax error. Fixed in this commit.

thanks mate


Added setting for dup content

Added setting for dup content

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Add Screenshot

Added screenshot in ReadMe file

can you add the screenshot in commit as well and link directly to the image.


STORM-2210 remove array shuffle from ShuffleGrouping when counter res…

This is a squashed version of https://github.com/apache/storm/pull/1790 made against master.

+1, because this is from #1790 that was already approved I will not wait the full 24 hours.


Revert "[WorkspaceTests] Disable branch and revision test"

Reverts apple/swift-package-manager#1172

This was fixed by https://github.com/apple/swift/pull/9452

@swift-ci please smoke test


[Wiki Update] Update the list of "Tested Distros" in the wiki

https://github.com/bayasdev/envycontrol/wiki/Frequently-Asked-Questions#tested-distros
I can confirm envycontrol works on these distributions (versions, desktop environments)
- Arch Linux (GNOME, MATE, Plasma, Xfce)
- Debian 11 (GNOME, Plasma, Xfce)
- ElementaryOS 6.x,7.x (Pantheon)
- EndeavourOS (GNOME, Plasma, Xfce)
- Fedora 36,37 (GNOME, Plasma, Xfce)
- Linux Mint 20,21 (Cinnamon, Xfce)
- Manjaro (GNOME, Plasma, MATE, Xfce)
- MX Linux 21 (Plasma, Xfce)
- Regolith 2.x (i3wm)
- Ubuntu 20.04.x,22.04.x (GNOME, Plasma, XFCE, MATE)

Done, and thank you for your contributions to the project ;)
PD: I will be issuing a new release to the AUR containing your EOS fix in the following hours.