Repository lang stats graph: Poor color contrast

I’m color blind (deutan) and found the CoffeeScript blue nearly impossible to tell apart from the CSS purple with the naked eye:

From repo https://github.com/postcss/autoprefixer
Nothing critical, but I thought it was worth mentioning from an accessibility standpoint.
I was directed to open an issue here by @jasonlong on Twitter: https://twitter.com/jasonlong/status/810887402260692992
Thanks!

The same is the case with Go and Python.
I am not color blind, but even I find it difficult to differentiate between the two.

From the repository https://github.com/taskcluster/json-e

Colour-blind as well (deutan), and just chiming in to say some colours are impossible for me to distinguish... I often find myself clicking the language bar out-of-habit just to bring up the language stats, because I'm so used to seeing CSS/CoffeeScript repos... 😞


Decouple identity policy from endpoint policy

Based against #7616
Decouple the resolving of policy for a particular identity from the endpoint-specific distillation of that policy. The identity level all deals with all rule selectors that apply to a particular identity (of an endpoint), and this performs merging of the policy. The endpoint level then applies that partially-resolved identity policy to a specific set of identity peers in context of generating the endpoint BPF policy map contents.
As part of this, rework the rule selector cache to be based upon identities rather than endpoints. On the plus side, this looks a bit simpler and will share the cache for endpoints that have the same identity. On the downside, it requires walking the policy to remove these identities when the identity is removed. This PR minimizes such work by ignoring local identities and only walking the policy when a global identity is removed, which is marginally worse than what we previously had (walk only when a local endpoint is removed), but in the v1.6 timeline we expect to make further changes to optimize this so I'm not worried about coming up with a more comprehensive fix for worst-case behaviour here.
Since this is part of an active project targeted at v1.6, this should not be merged before v1.5 is branched.
Related: #7515
Conflicts: #7536

This change is

test-me-please

Proposal: Accumulate identity expiry events and hide behind a trigger so that if there is high identity churn, it will limit the amount of iteration over the policy to clean up the cache. Should address the weakness of this PR as-is.

Coverage decreased (-0.01%) to 42.541% when pulling 11853426c34cf5333182d6d8831b7093b8ce195f on joestringer:submit/decouple-policy-calculation into 2935f589aba999bb1de8c7ccf7930b20a9aff5be on cilium:master.

test-me-please

Haven't looked at all the details yet, but would it make sense to rebase this on top of #7539 once merged, or do you think this would make a good intermediate step before utilizing the global cache implemented in #7539?

Haven't looked at all the details yet, but would it make sense to rebase this on top of #7539 once merged, or do you think this would make a good intermediate step before utilizing the global cache implemented in #7539?

This can be done independently, although this adds a new function to IdentityAllocatorOwner that is a subset of the changes that #7539 so it may make sense to at least reuse the change to that interface in #7539, if not rebase against that PR. Could go either way.
I'd prefer to merge this as a step before utilizing global cache just to keep changes incremental. There's at least one other step before utilizing global cache anyway -- sharing the IdentityPolicy between multiple endpoints (which is the next thing on my list).

test-me-please

test-me-please

test-me-please

Green CI run, I can switch over to a map-based implementation for the observers.

test-me-please

Another green build. Will just fix up the hound complaint above.

Feedback from agent team was addressed, so this is good to merge - merging now.


LambdaDsl in Java 8

This is my project dependency
"io.github.jbwheatley" %% "pact4s-scalatest" % "0.10.0-java8"
Somehow I cannot import au.com.dius.pact.consumer.dsl.LambdaDsl.* into my project. Is it because LambdaDsl doesn't support in Java 8?
Assume that it doesn't support, that mean the only way I can write Pact is using PactDslJsonBody. Is there any alternative method less painful than this? upgrading project jdk version is not an option here either.

I just wrote a compatibility matrix in https://github.com/jbwheatley/pact4s/pull/522 that should clarify the situation.
Pact4s Java8 versions are using Pact JVM 4.1.x which doesn't provide LambdaDsl.
JDK8 is soon 10 years old, I would strongly encourage to move to at very least Java 11 or better Java 17. Most popular frameworks and libraries are dropping support for JDK8.


Dialog Refactor

Fixes #197
Fixes #501

@ryceg @Sarenor I've made all updates I could. Left comments on the outstanding issues. I welcome additional testing now that it's marked "ready for review". Otherwise we'll aim to merge soon.


slug-url-first-comment

This is a comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

This is another comment!

Lorem Ipsum Dolores

Wingardium Leviosa!

This one makes a net,
That one stands and wishes
Would you like to bet
Which one gets the fishes ?

This is a new comment


upgrade serde deps

Just updating serde to the latest version.

Essentially same to #86 which has been now merged. Thank you nevertheless!


Fetch pffft from upstream project website instead of bitbucket

Starting from this commit
https://github.com/VCVRack/Rack/commit/2db08f15a00f6792bb3a45db31dd13f94966beed
the upstream project does not expect to use bitbucket anymore. The title
mentions that “BitBucket deleted all Mercurial repos”. Instead, an archive of
the pffft source is hosted on vcvrack.com directly. The unziped sha256 is the
same as before this change.

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/107580
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

We also have the option of pulling the archive from bitbucket still. But, as the source of vcvrack itself uses vcvrack.com, I'm thinking this will be looked at closer by upstream, than bitbucket.

We also have the option of pulling the archive from bitbucket still. But, as the source of vcvrack itself uses vcvrack.com, I'm thinking this will be looked at closer by upstream, than bitbucket.

@ddelabru if you want to have a look as well

@ddelabru if you want to have a look as well

@GrahamcOfBorg build vcv-rack

@GrahamcOfBorg build vcv-rack

Should this be backported to stabled? vcv-rack installs there currently 404ing trying to fetch https://bitbucket.org/jpommier/pffft/get/29e4f76ac53bef048938754f32231d7836401f79.tar.gz


Is there any api interface here?

Just post a form to /login from browser with the following data:

email
password

And your browser should be redirected if success.


update description

Type of change

Documentation update

Description
Update the description of the network components

Simi, can you submit the same PR to branch hcs-dev? Thanks.


Fix security review notice

Fix notice in security report:
Directory ~/.drush/cache/default exists, but is not writable. Please check directory permissions.

@Sanchiz let's change folder permissions instead of running drush via sudo

@podarok Do you mean this this script before this command?

@Sanchiz yup. Just chmod/chown

@podarok let's merge this PR.
There's some points why we should merge it:

There's no way to easy way to get home folder(when you use sudo there should be workaround to get sudoed user home dir)
Other parts of the system are using sudo for drush commands


Creating cluster fails with Windows password validation error - every time

I'm experiencing the same issue as #34547. Creating a new AKS cluster with Windows container support using az aks create. I provide a password that adheres to the password complexity requirements and it fails with the error:
ValidationError: Parameter 'ManagedClusterWindowsProfile.admin_password' must conform to the following pattern: '^(?=.*[a-z])(?=.*[A-Z])(?=.*[!@#$%\\^&\\*\\(\\)])[a-zA-Z\\d!@#$%\\^&\\*\\(\\)]{12,123}$'.
The password has the following characteristics:

24 characters long
lower case letters
upper case letters
numbers
special characters

Versions:

AKS version 1.18.8
azure-cli 2.13.0
aks-preview 0.4.64

Location canadacentral
The documentation says:

The windows-admin-password and windows-admin-username parameters set the admin credentials for any Windows Server containers created on the cluster and must meet Windows Server password requirements.

Perhaps this is a lesser quibble: the regex outputted by az aks create does not match the Windows password requirements from that document, i.e.

Non-alphanumeric characters (special characters): (~!@#$%^&*_-+=`|\(){}[]:;"'<>,.?/) Currency symbols such as the Euro or British Pound are not counted as special characters for this policy setting.
Any Unicode character that is categorized as an alphabetic character but is not uppercase or lowercase. This includes Unicode characters from Asian languages.

User error. The password was surrounded in "quotes".
Quotes are valid characters in a Windows Server password but it appears unsupported in Windows containers.

@jwcw -
Thanks again for bringing this to our attention.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Closes #909 - SHR u claim must include port number, if supplied

See #909
In this PR:

Formatting tweaks for claims
Include port number in u
Adds new unit tests

@iambmelt Looks like all the new tests are failing

@shahzaibj Yeah, I'll need to investigate why these don't work on an emulator, but pass on a real device thx


Enable pytest support for django waffle

It would be very useful if testing utils such as override_switch and override_flag were available for pytest style classes as class decorators. Is this feature somewhere on the road map?

Those already exist. See https://waffle.readthedocs.io/en/stable/testing/automated.html#testing-automated.

right now if you try to write this like this
@override_switch("some_switch", active=True)
class TestFoo:
    ...

it fails with TypeError: Can only decorate subclasses of unittest.TestCase. on the latest 2.6.0 version, sorry for not making it clear right away.

You’re welcome to submit a pull request adding a pytest decorator.


Sampling improvements

Implementation of the suggested improvement in this issue.  Depending on the data and model this speeds up the sampling process.

your PR is on pip :)


Exception thrown when adding event handlers to buttons inside  element

Originally opened as dart-lang/sdk#20931
This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
Example shown here on SO: http://stackoverflow.com/questions/25800082/how-to-know-which-button-the-user-selected-on-paper-dialog
What is the expected output? What do you see instead?
I would expect this to work with no exceptions thrown, but after I click a button to close the dialog, I get:
Uncaught Uncaught Error: Bad state: controller overlay-host is not a Dart polymer-element.
Stack Trace:
Bad state: controller overlay-host is not a Dart polymer-element.
    at dart.wrapException (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:2588:15)
    at PolymerEventBindings_getEventHandler_closure.call$1 (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:24435:17)
    at _RootZone.runUnaryGuarded$2 (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:9865:18)
    at _RootZone_bindUnaryCallback_closure.call$1 (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:9986:26)
    at PolymerEventBindings_prepareEventBinding__closure.call$2 (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:24456:34)
    at dart.Primitives_applyFunction (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:2572:23)
    at _callDartFunction (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:18147:29)
    at paper-button.<anonymous> (http://<IP_ADDRESS>:8081/index.html_bootstrap.dart.js:18136:16)
What version of the product are you using?
Dart 1.6.0, Polymer 0.13.0.+3
On what operating system?
Windows 8 64-bit
What browser (if applicable)?
Chrome
Please provide any additional information below.

 Comment by jakemac53

I was unable to get a repro on this by creating an element with the example template and functions, would it be possible for you to post a full project that contains the repro?

Added NeedsInfo label.

 Comment by sethladd

Added Pkg-Polymer, Area-Pkg, Triaged labels.

 Comment by jakemac53

Set owner to @jakemac53.
Added Started label.

This comment was originally written by<EMAIL_ADDRESS>
The project where this is occurring is currently closed source.  It's really odd, because the event handler method on the buttons is called just fine, but the above exception displays every time.  When I remove the event handlers from the element, clicking the buttons doesn't give the error.  Sorry, wish I could help more.
I've attached the Polymer element in question, along with it's code-behind file.  Perhaps that may be of some use.

Attachment:
alert.zip (1.38 KB)

 Comment by jakemac53

Ah perfect, the actual issue is with the layered="true" attribute on the dialog. This rips out the shadow root from your element and puts it inside a different core-overlay-layer element which it then tries to call the method on (and fails because not only does it not exist but that is not a dart element).
I will do some more investigation and see if this is a bug on the JS side of things as well, or something dart-specific.

Added Started label.

 Comment by jakemac53

Definitely looks to be a dart issue, can't get a repro on the js side alone

 Comment by jakemac53

Removed Priority-Unassigned label.
Added Priority-High label.

This comment was originally written by<EMAIL_ADDRESS>
Removed an unneeded reference in the polymer file.

Attachment:
alert.zip (1.40 KB)

 Comment by jakemac53

No problem, thanks for the bug report!
Fixed in dart-lang/sdk@8c422a0862afcc7149a4dd5340e266d89ea523f0 and released as 0.14.2+1.

Added Fixed label.

 Comment by jakemac53

Fix pending https://codereview.chromium.org/582113002/

This comment was originally written by<EMAIL_ADDRESS>
Thanks for the quick resolution!

This comment was originally written by<EMAIL_ADDRESS>
Confirmed 0.14.2+1 fixes my issue.


refactor(NODE-3273): remove undefined error param from topology open …

Description
NODE-3273 Remove undefined error parameter from topology "open" event
What is changing?
The error parameter is no longer being passed on the topology "open" event
Is there new documentation needed for these changes?
No, this is an internal change
What is the motivation for this change?
The error is always undefined based on the flow of the code, so there is no need to pass it.
Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
[x] Changes are covered by tests
[x] New TODOs have a related JIRA ticket

@gjchong25 Thanks for reviewing! I hadn't had a chance to clean up the PR yet, which is why I put it in Draft. Is there another way to indicate the PR is not yet ready for review?
I've added a commit to address the test wording suggestion.  Do you all typically squash and merge, or do I need to squash these myself?
I'll figure out what's going on with that failing test.

Looks like the tests are all passing after I pushed the commit with your wording suggestion change. It must have been a flakey failure.

I usually assume that a PR isn't ready for review unless you request review from someone on the team (maybe this differs between teammates, though). In terms of squashing and merging, the implementer doesn't merge but instead the primary reviewer, once resolving all the comments, will then request team review and then merge all your changes once the second reviews are in. Here is a more comprehensive guide of the dev process!

@dariakp Thanks for the feedback!  I made the changes you suggested. It felt odd to not have every test in that file use the afterEach(), so I updated them to use it.  I put those changes in a separate commit in case you don't think I should be doing that as part of this PR.
There is some strange test structure in some of the tests that use a restOfTest() function, but I left that alone as it wasn't directly related to this work.

Just saw these failures. Will fix this up this afternoon.

@dariakp Thanks for the detailed feedback!


IBD: Wallet Sync issue

I loaded Stratis Core 1.1.0 on a new Win10 miniPC for the very first time. Here is the spec:
Atom x5-Z8350
4GB ram
64GB HD
IBD started around 10PM (PST). I left it running overnight. Woke up this morning and saw the height was only 120,000 or so with 0 connections. I close the app and relaunch. After about 1 minute, more connections and the height started to increase; this was about 7:30 AM (PST). I came home from work and the sync stopped again; this time I was at 134,000 or so with 2 connections. I close the app and relaunch. Nothing seem to update; I check Task Manager and Stratis.StratisD was not running. I close the app and relaunch again. After about 1 minute, more connections and the height started to increase. I turn off sleep. Now it is at 401154 with 16 connections.
My thoughts on the issue are:

wifi
sleep (default to 10 min)

It seems somehow the process isn't able to recovered from whatever happened. I can give you the full logs if needed. @dev0tion Only error in log :
[2019-03-15 01:50:55.1071 1] ERROR: Stratis.Bitcoin.Builder.FullNodeFeatureExecutor.LogAndAddException An error occurred: 'DBreeze.Exceptions.DBreezeException: Creation of the database folder failed! ---> DBreeze.Exceptions.DBreezeException: Database is not operable, please find out the problem and restart the engine! FSR INIT FAILED: C:\Users\Stratis\AppData\Roaming\StratisNode\stratis\StratisMain\chain_DBreezeSchema ---> System.IO.IOException: The process cannot access the file 'C:\Users\Stratis\AppData\Roaming\StratisNode\stratis\StratisMain\chain_DBreezeSchema' because it is being used by another process.
at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
at DBreeze.Storage.FSR.InitFiles()
--- End of inner exception stack trace ---
at DBreeze.Storage.FSR.InitFiles()
at DBreeze.Storage.StorageLayer..ctor(String fileName, TrieSettings trieSettings, DBreezeConfiguration configuration)
at DBreeze.Scheme.OpenSchema()
at DBreeze.DBreezeEngine.InitDb()
--- End of inner exception stack trace ---
at DBreeze.DBreezeEngine.InitDb()
at Stratis.Bitcoin.Base.ChainRepository..ctor(String folder, ILoggerFactory loggerFactory, DBreezeSerializer dBreezeSerializer) in C:\Users\StratisTest\Desktop\DevOps\Agent2_work\3\s\StratisBitcoinFullNode\src\Stratis.Bitcoin\Base\ChainRepository.cs:line 46
at Stratis.Bitcoin.Base.ChainRepository..ctor(DataFolder dataFolder, ILoggerFactory loggerFactory, DBreezeSerializer dBreezeSerializer) in C:\Users\StratisTest\Desktop\DevOps\Agent2_work\3\s\StratisBitcoinFullNode\src\Stratis.Bitcoin\Base\ChainRepository.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(ConstructorCallSite constructorCallSite, ServiceProviderEngineScope scope)
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitScoped(ScopedCallSite scopedCallSite, ServiceProviderEngineScope scope)
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(ConstructorCallSite constructorCallSite, ServiceProviderEngineScope scope)
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitScoped(ScopedCallSite scopedCallSite, ServiceProviderEngineScope scope)
at Stratis.Bitcoin.Builder.FullNodeServiceProvider.get_Features()+MoveNext() in C:\Users\StratisTest\Desktop\DevOps\Agent2_work\3\s\StratisBitcoinFullNode\src\Stratis.Bitcoin\Builder\FullNodeServiceProvider.cs:line 53
at System.Collections.Generic.EnumerableHelpers.ToArray[T](IEnumerable1 source, Int32& length)    at System.Linq.Buffer1..ctor(IEnumerable1 source)    at System.Linq.OrderedEnumerable1.GetEnumerator()+MoveNext()
at Stratis.Bitcoin.Builder.FullNodeFeatureExecutor.Execute(Action`1 callback, Boolean disposing) in C:\Users\StratisTest\Desktop\DevOps\Agent2_work\3\s\StratisBitcoinFullNode\src\Stratis.Bitcoin\Builder\FullNodeFeatureExecutor.cs:line 117'

Thanks for the report @trunghieud, I'll look into it.
The DBreeze exception is probably caused by the fact that there was still an instance of the Stratis.StratisD running when restarting at some point, so the new instance could not access the data folder. This exception is unrelated to your node stalling in sleepmode.
We'll keep you posted!

Btw, it would be nice improvement to have the logs show in the user’s time zone and not London :)


Bug reports for links in OpenIM docs

Summary

Status
Count

🔍 Total
427

✅ Successful
390

⏳ Timeouts
0

🔀 Redirected
0

👻 Excluded
7

❓ Unknown
0

🚫 Errors
30

Errors per input
Errors in .devcontainer/README.md

[ERR] [file:///home/runner/work/dweb-browser-im-server/.vscode/launch.json](file:///home/runner/work/dweb-browser-im-server/.vscode/launch.json) | Failed: Cannot find file
[ERR] https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/openimsdk/open-im-server | Failed: Network error: dns error: no record found for Query { name: Name("ms-vscode-remote.remote-containers.r45ftdqj0iiudj1ovfs3chqk3e.ex.internal.cloudapp.net."), query_type: AAAA, query_class: IN }

Errors in docs/contrib/environment.md

[ERR] http://your.domain.com/ | Failed: Network error: dns error: no record found for Query { name: Name("your.domain.com.r45ftdqj0iiudj1ovfs3chqk3e.ex.internal.cloudapp.net."), query_type: AAAA, query_class: IN }
[404] https://github.com/openimsdk/open-im-server/blob/main/config/config.yaml | Failed: Network error: Not Found
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-server/release-v3.2 | Failed: Network error: Not Found
[403] https://demo-9999999.oss-cn-chengdu.aliyuncs.com/ | Failed: Network error: Forbidden
[404] https://github.com/openimsdk/open-im-server/blob/main/deployments/templates/openim.yaml | Failed: Network error: Not Found
[403] https://temp-1252357374.cos.ap-chengdu.myqcloud.com/ | Failed: Network error: Forbidden
[403] https://oss-cn-chengdu.aliyuncs.com/ | Failed: Network error: Forbidden
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-chat/release-v1.3 | Failed: Network error: Not Found
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-chat/main | Failed: Network error: Not Found
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-chat/release-v1.2 | Failed: Network error: Not Found
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-server/release-v3.3 | Failed: Network error: Not Found
[404] https://github.com/openimsdk/openim-docker/tree/main/openim-server/main | Failed: Network error: Not Found
[400] http://s3.cn-east-1.qiniucs.com/ | Failed: Network error: Bad Request

Errors in docs/contrib/development.md

[403] https://unix.stackexchange.com/a/79357 | Failed: Network error: Forbidden

Errors in docs/contrib/prometheus-grafana.md

[404] https://grafana.com/docs/grafana/latest/datasources/redis/ | Failed: Network error: Not Found
[404] https://grafana.com/grafana/plugins/grafana-kafka-datasource/ | Failed: Network error: Not Found
[404] https://docs.openim.io/configurations/prometheus-integration | Failed: Network error: Not Found
[404] https://grafana.com/docs/grafana/latest/datasources/zookeeper/ | Failed: Network error: Not Found

Errors in README-zh_CN.md

[404] https://github.com/openimsdk/open-im-server/blob/main/FAQ.md | Failed: Network error: Not Found
[404] https://github.com/openimsdk/open-im-server/blob/main/examples | Failed: Network error: Not Found
[404] https://github.com/openimsdk/open-im-server/blob/main/docs/contrib/interface.md | Failed: Network error: Not Found

Errors in docs/contrib/mac-developer-deployment-guide.md

[404] https://github.com/openimsdk/open-im-server/tree/main/.devcontainer | Failed: Network error: Not Found

Errors in build/README.md

[ERR] [file:///home/runner/work/dweb-browser-im-server/dweb-browser-im-server/build/common.sh](file:///home/runner/work/dweb-browser-im-server/dweb-browser-im-server/build/common.sh) | Failed: Cannot find file
Full Github Actions output

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

This issue was closed because it has been stalled for 7 days with no activity.


feat: allow using CDN for queries with token

Allows the PHP client to also request cached, authenticated data

bump? would love this.


Topscope variables may be facts, but may also be top scope variables

running the check with --fix on a manifest with lots of top scope variables shows the puppet lint check changes every variable into a fact.
solution: provide a list of facts

I'm wondering if we should disable fact manipulation in this plugin. We've https://github.com/mmckinst/puppet-lint-legacy_facts-check which should already fix it and https://github.com/puppetlabs/puppet-lint/issues/41 mentioneds that both plugins conflict. CC: @ekohl @alexjfisher

@SimonHoenscheid Is this filed in the right place?  I don't think this plugin is the one responsible for changing variables into 'facts'.  We have got some conflicting behaviour between plugins though. Maybe this can be fixed by changing the order that plugins are run?  I assume it's alphabetical in some way (but is that filename, or check name)???

@alexjfisher I talked to @bastelfreak before opening this issue, because I was not sure, but the behavior of the lint check is dangerous.

@SimonHoenscheid Maybe we're talking cross purposes.  Could you explain exactly what you mean by changes every variable into a fact.?
I'm wondering if you're actually encountering https://github.com/mmckinst/puppet-lint-top_scope_facts-check/issues/10

@alexjfisher If I memorize it correct yes. I am not sure, but I can try to replicate.


[FIX] Can't install dev and release on the same device

Was conflicting Fileprovider's authorities.

 Error: INSTALL_FAILED_CONFLICTING_PROVIDER: Package couldn't be installed in /data/app/chat.rocket.android.dev-NIX0violFlkc0WZmKcI6CQ==: Can't install because provider name chat.rocket.android.fileprovider (in package chat.rocket.android.dev) is already used by chat.rocket.android

@Adizbek As the documentation states:
"For your own app, specify an authority consisting of the app's android:package value with the string "fileprovider" appended to it."
(https://developer.android.com/training/secure-file-sharing/setup-sharing#DefineProvider)
I know it can sound frustrating to you since the ability to use both build variantes is not possible. But do you know what such a change (declaring only .fileprovider) can lead for especially for other applications for instance?).
I'm going to close this PR, but if you want let us know why we should really include it.
Thank you!

@filipedelimabrito But it wouldn't affect to other apps, since release app's package will be: chat.rocket.android.fileprovider and for beta: chat.rocket.android.dev.fileprovider. They will act as different apps and will not conflict. And it will be handy to have both variants of app while developing and testing.


Change how start date is set for copied notifications

Recommendation: When a notification is created by copying it from another notification, set its start date to the date it was created, not the start date of the notification from which it was copied.

@aarenson, thanks for the feedback. I wouldn't want to do this in general, because someone could create a notification with a start date in the future, and you could end up having a copy that has an earlier start date than the notification from which it was copied. Perhaps the start date should be set to the later of the current date and the start date of the copied notification.
What problem(s) do you see with how things are now?

My newly created copy of a notification looks like it has been running longer than it has actually been running, which looks like an error to me. It hasn't prevented anything from happening, I just find it confusing.
Perhaps

This was fixed (I think in version 1.4.0).


feat: Merge categories/tags and improve filtering

💭 Problem
This was prompted in part by discussion on #443. Tags are effectively unlimited and unregulated right now; however, categories are also quite limited since each entry can only have one. Filtering based on category/tags via the select fields is an okay experience - but it could be better!
🎯 Changes
Heavily inspired by bestofjs.org. Each item will be allowed multiple, pre-defined categories (or tags, the name doesn't matter). This would be a merge of current categories and useful tags to keep. By default, all entries are displayed. A collection of buttons let you choose which categories/tags you want displayed. When you select a category/tag, the list of available categories/tags narrows based on your current selection. This utilises a server load function, and the URL is shareable due to search params.
TODO

[ ] Merge categories and tags
[ ] Add allowed options to zod schemas

Before

After

✅ Checklist

[x] I have given my PR a descriptive title
[x] I have run pnpm run lint locally on my changes

Honestly, I'm not a big fan of this as implemented. We commonly do things like link to https://sveltesociety.dev/packages#svelte-kit-adapters. There's no way to do that now as everything's just mixed together and that filter doesn't exist. It's also going to break tons of links that we have across svelte.dev and kit.svelte.dev. We could clean those up, but we'd need ways to keep the existing functionality for that to be possible

This looks great!

Honestly, I'm not a big fan of this as implemented. We commonly do things like link to sveltesociety.dev/packages#svelte-kit-adapters. There's no way to do that now as everything's just mixed together and that filter doesn't exist. It's also going to break tons of links that we have across svelte.dev and kit.svelte.dev. We could clean those up, but we'd need ways to keep the existing functionality for that to be possible

@benmccann oh wow fair enough, I had no idea it was referenced extensively in the svelte/kit docs. For that above link, with this change it would become https://sveltesociety.dev/packages?tags=svelte-kit-adapters. I guess the worst case scenario for old links is that users get taken to the top of the page rather than the category ID that was linked. I would argue this new solution is more robust than since it only loads the contents of the specific category/tag in the link. Either way, I'm not too fussed if this proposal isn't feasible since it was fun to implement!

There is a feature in Cloudflare to do bulk redirects - maybe that's something we can explore?

Given that we get the same functionality as before - would it be fine to merge this if we updated all the links on the Svelte site? @benmccann

Let me give this another look to verify whether things like the ?tag=svelte-kit-adapters filter will let us reproduce what we were depending on previously. I remember there being other issues we might need to address, but I can't check at the moment because the Netlify deploy is broken. Do we need to close this PR and open a new one to see the Vercel deploy preview or was the Netlify one taken down and the Vercel one not added yet?

I think so.

Testing Cloudflare Preview

When I visit https://sveltesociety-dev-preview-git-fork-lachlan-97a95c-sveltesociety.vercel.app/packages it gives me a 500 error

When I visit sveltesociety-dev-preview-git-fork-lachlan-97a95c-sveltesociety.vercel.app/packages it gives me a 500 error

@benmccann this is fixed by #546 (the site wasn't using adapter-vercel)

For that above link, with this change it would become https://sveltesociety.dev/packages?tag=svelte-kit-adapters

Doesn't seem to work for me. This returns no results: https://sveltesociety-dev-preview-git-fork-lachlan-97a95c-sveltesociety.vercel.app/packages?tag=svelte-kit-adapters

For that above link, with this change it would become https://sveltesociety.dev/packages?tag=svelte-kit-adapters

Doesn't seem to work for me. This returns no results: https://sveltesociety-dev-preview-git-fork-lachlan-97a95c-sveltesociety.vercel.app/packages?tag=svelte-kit-adapters

Yes sorry, I haven't actually merged categories and tags yet, I wanted to get some feedback before I put that effort in! Only existing tags (not categories) can be selected right now. For example: https://sveltesociety-dev-preview-git-fork-lachlan-97a95c-sveltesociety.vercel.app/packages?tag=notifications


Minor cleanup for TSTool 14.0.4 release

I modified the test suite command file comments and need to commit.

I committed the change.  Close this issue.


DEV: Add proper CSS class for sidebar section link content

Adds tests to ensure that CSS class is added when channels are muted
as well.

Thanks for this @tgxworld , much better


When I used php_strip_whitespace to compress my PHP code, this library didn't work as expected.

Serializable Closure Version
1.3.1
PHP Version
8.2.0
Description
When I used php_strip_whitespace to compress my PHP code, this library didn't work as expected. Is a bug?
Steps To Reproduce

I defined a function go to use this Library.

function go(Closure $function, int $timeout = 300)
{
    if (App::$cli) return null;
    return MyCustomClass::start($function, $timeout);
}

In this method start, I used

serialize(new SerializableClosure($function));

and I saved it to a cache file.

When I used the function go:
 public static function start($bool): void
    {
        if ($bool) {
           go(function () {
                   //some functions...
                }, 0);
        } else {
            Log::record("Error");
        }
    }

If I used php_strip_whitespace to compress my code, it means my code will display like this:
 public static function start($bool): void{if($bool){go(function(){}, 0);} else {Log::record("Error");}}

It will cache the code:
if($bool){go(function(){}, 0);} else {Log::record("Error");

But if I don't compress my code, it will cache the code:
function(){}

When only uncompressed,  it meet expectations. Is this a bug?

Heya, thanks for reporting.
We'll need more info and/or code to debug this further. Can you please create a repository with the command below, commit the code that reproduces the issue as one separate commit on the main/master branch and share the repository here? Please make sure that you have the latest version of the Laravel installer in order to run this command. Please also make sure you have both Git & the GitHub CLI tool properly set up.
laravel new bug-report --github="--public"

Please do not amend and create a separate commit with your custom changes. After you've posted the repository, we'll try to reproduce the issue.
Thanks!

Embarrassingly, when I tried to create a demo, I found that it worked normally. Obviously, part of my code affected it.
I will temporarily close the issue, and then reopen the issue after I find out the problem.


[Question] harmful tweakers = winutil

Latest commit https://github.com/farag2/Sophia-Script-for-Windows/commit/e0f2b1f02592e35fc5083b81c7037de78c322bce added winutil to the list of harmful tweakers, my question is why or what exactly is harmful (maybe I can talk to the developer and change it?)

Hi. All because the so-called tweaker provides a feature to break Windows by removing (not disabling) Defender as a folder. Such stuff provides only by trojans. By this act he like says to his audience that this is normal, everyone can break Windows to make it vulnerable to all kinds of viruses (especially to low-skilled users). This is evil. For that winutil was marked as trojan and every Sophia Script user will be notified if they ran such things before because I'm tired of broken Windows after such tweakers which has the only thing to convey to users is that Windows is bad, it has to be "fixed", "optimized" and so on. But the only thing such tweakers provides is to remove poor Defender as TrustedInstaller and allow user to download a god forgotten app from its GUI.

https://github.com/ChrisTitusTech/winutil/blob/a2b12ae4b0bb8f58c9def3865cc8dff64279fbba/winutil.ps1#L2664
Such "scripts" is a cancer and should be blocked on any platform, regardless of being open-source or not.

Normal apps won't provide or even notice about such things. Of course if a developer respects his audience (he doesn't).

While I do agree such feature should not exist, it doesn't mean the entirely of winutil is harmful.

Agree until he will remove Defender installation. Until that, the is marked as a trojan.

I know his position regarding WU. I think that good apps do not provide harmful features regardless what's behind of them. Disabling WU is an evil.
I will stand my ground. :) Younneed to convince him, not me. At least I'm not willing to break users Windows. :cat:

I didn't make a whole code audit of his project, but that was enough for me to add him. So for now, yes, these are 2 only issues.


crictl generating error on run

Describe the bug
crictl is generating the following:
FATA[date] failed to connect: failed to connect: context deadline exceeded
To Reproduce
Steps to reproduce the behavior:
me@k8s1:~ $ k3s crictl version
NAME:
   crictl version - Display runtime version information

USAGE:
   crictl version [arguments...]
FATA[2019-03-06T20:52:59.197614183+01:00] failed to connect: failed to connect: context deadline exceeded
me@k8s1:~ $

Additional context
Fresh install of Raspbian, k3s installed with installer.
me@k8s1:~ $ lsb_release -a
No LSB modules are available.
Distributor ID: Raspbian
Description:    Raspbian GNU/Linux 9.4 (stretch)
Release:        9.4
Codename:       stretch

me@k8s1:~ $ uname -a
Linux k8s1 4.14.79-v7+ #1159 SMP Sun Nov 4 17:50:20 GMT 2018 armv7l GNU/Linux

crictl must be ran as root. We should add a check and error out of not root.

Was expecting to see version without running elevated :)
Yes, a error msg if not run as root would be great.
Closing, cheers!


InvokeVerb Incompatible with String Verb Names

Description
Currently invokeverb can only be used within a do string. Because of this only verbs without spaces can be used with the command, this leaves out most admin smites.
However, > invokeverb self <uid> "Become Bread" will work as it is not encased in quotes.
Reproduction

Try > entities prototyped MobHuman do "invokeverb self $ID "Become Bread""
Console complains that "Become" is not a verb because it is not accepting the nested string. Note it will not accept ' or ` for nested strings.
Try > entities prototyped MobHuman invokeverb self $ID "Explode"
Console complains that invokeverb does not exist.
Try > entities prototyped MobHuman do "invokeverb self $ID Explode"
MobHumans explode.

Additional context
I believe this requires that invokeverb is adapted into using Toolshed's piped inputs, but I am not bright enough to make that happen.

I have been informed that you must escape the quotes.
> entities prototyped MobHuman do "invokeverb self $ID \"Become Bread\"" works.


Todo-show checks non source files

In my project todo results I'm getting a bunch of .ai (adobe illustrator files) and .pdn (paint.net) files under the XXX category, each thousands of characters long.

I think it might do the trick.

fb99c3593c21ecb1583190df10ff0a510c92f28b adds truncation of matches longer then 120 chars.
When I add a non-source file to my project, the scan totally ignores the file and thereby no false matches. So I cannot reproduce what you are seeing. If it still exists, send me the file and I'll investigate a way to solve it.

Just tried it now, looks like it isn't solved.

Not exactly pretty :)
Commit f0d763c1c1f64d0a1aa3e9a0da8a779469ed6c30 solves the wrapping of match texts, so it doesn't overlap with file paths (link text).
But do you have any suggestions to how we can avoid these false hits from binary data completely?

Have you seen this? https://github.com/bevry/istextorbinary

I think that is too much of an overhead for each file searched. And we can't look at comment scope (unless there is a special scope for binary) because that would exclude raw text, e.g. markdown documents with todos.
As I mentioned earlier; I am unable to reproduce this with any binary files I have, so could you attach one of the files you are exteriencing this with, so I can test it locally?

Just sent you a mail with it.

8c1d76c8d138f47a8e248eddd2580c3a407763f7 enforces a space after todos. This prevents false positives from the file you are referencing, thereby solving this issue better then checking for binary content on every match or just truncating long results.

Looks like that library just checks extension first and then some small chunks at the beginning or at the middle of the file, not really that much of a overhead since you're already using a regex with the full content of possibly fairly big files.
A space after the todo seems reasonable, but it depends on how people style their comments.


Link suggestions

Added a link to the Team Data Science Process Overview page as the previous link was calling a 404 Page not found error.

@ntbaccam : Thanks for your contribution! The author, @gopitk, has been notified to review your proposed change.

@cjgronlund , updated the Team Data Science Process link

@ntbaccam Good catch! Thank you for the fix.
#sign-off
Approved by doc manager

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

@GitHubber17 : Thanks for your contribution! The author, @gopitk, has been notified to review your proposed change.


[AzCopyV10][StgExp] Support preservation of tags when copying blobs

Support preservation of blob index tags when transferring blobs from one blob storage to another.
To preserve tags, use the following syntax

azcopy cp "https://[account].blob.core.windows.net/[source_container]/[path/to/directory]?[SAS]" "https://[account].blob.core.windows.net/[destination_container]/[path/to/directory]?[SAS]" --s2s-preserve-blob-tags=true

To read more about blob indexing, refer the public documentation here.
While setting tags on the blobs, there are additional permissions('t' for tags) in SAS without which the service will give authorization error back.

AzBlob has dependencies on Go Version 1.15 . So upgrading from 1.13 -> 1.15
AzFile is also using Go version 1.15


Swich to new gcloud API for GCE MIGs

Fixes #11686
cc @davidopp @mbforbes @roberthbailey

@wojtek-t why did you mark this as do not merge?

@roberthbailey - the new API is not yet released (should be today according to MIG team from Warsaw) - this is why Jenkins tests failed. [I've tested by updating my gcloud to (still) experimental version]

[by new API I mean support for it in gcloud]

Ah. I know that the new raw API is out already; I didn't realize gcloud support hadn't rolled out yet.

Jenkins is now failing with the following error:
You do not currently have this command group installed.  Using it requires the installation of components: [beta]
ERROR: (gcloud) You cannot perform this action because you do not have permission to modify the Google Cloud SDK installation directory [/usr/local/share/google/google-cloud-sdk].

How can we workaround it (and do we want?)?

/cc @jlowdermilk

It looks like the beta component actually is installed for shared gcloud. Maybe someone recently bumped it? @k8s-bot retest this please

Please reassign when this is ready to merge.

@jlowdermilk (sorry for late response). It seems that beta is not installed - i.e. after rebasing today it's still not there. Is there any way to work around it? Is it possible to proceed with that change?

@wojtek-t, sorry I was looking at the wrong project. I don't have access to the pr builder jenkins master to install the beta component, but I've pinged @ixdy, and will update it shortly.

@k8s-bot test this please

@wojtek-t, sorry I was looking at the wrong project. I don't have access to the pr builder jenkins master to install the beta component, but I've pinged @ixdy, and will update it shortly.
@ixdy - did you have chance to look into it?

This is now ready for review.

@roberthbailey - can you please take a look (or delegate)?

LGTM, but I think we should use gcloud compute instance-groups rather than gcloud beta compute instance-groups in all of these commands.

@roberthbailey - PTAL

LGTM.

@wojtek-t Please cherry pick this into the 1.0 release branch (after it's been merged) so that our release branch validation tests are using the current API for MIGs as well (I doubt the preview commands will work for much longer).

@k8s-bot test this [testing build queue, sorry for the noise]

Automatic merge from SubmitQueue


Add option for logfile

With the LorixOne Gateway, I have problem to start the gateway as a deamon and to pipe the stout and sterr to a file. START-STOP-DEAMON has a problem with that. I think I have no other options...
The Gateway run with a yocto linux 2.1.2.

What problem does start-stop-daemon have? Did you for example see this https://stackoverflow.com/questions/8251933/how-can-i-log-the-stdout-of-a-process-started-by-start-stop-daemon?
Also note that it might not be a good idea to persist logs on the gateway. I'm not sure about the LorixOne, but usually these devices do not have a lot of storage and the kind memory might not be suitable for lots of writes.


License

There is no license file. Could you please choose an open source license and add a license file?

I used it in this microproject, the sound mirror, which I created as a toy for a local baby...
https://ipfs.io/ipfs/QmcV8JrsTg9XZ8EZjb2HjYSWKkTVxaNGakZchJ3kyLovZb/zrcadlo.html

You can use the code and do whatever you want with it. I just added an MIT license to this repo. Thanks!


Splitting terminal in directory with ë character fails

Does this issue occur when all extensions are disabled?: Yes/No
Splitting terminal in folder with ë character or any subdirectories errors and doesn't split.
The terminal process failed to launch: Starting directory (cwd) "/Users/liam-nt/Documents/vscode_bug_test/eÃ«e" does not exist.

VS Code Version: 1.74.0 (Universal)
OS Version: macOS Monterey 12.5 (21G72)

Steps to Reproduce:

Create a folder containing 'ë' in name
Open integrated VSCode terminal and split terminal by keyboard shortcut or interface.

related #168872

Regression from https://github.com/microsoft/vscode/pull/165632


Update configure-ssl-app-service-certificate.md

Certificates are issued by GoDaddy, So I think it's public certificates and not private certificates.

@msangapu-msft
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close


Configure Istio proxy PILOT_ADDRESS with DNS_NAME

Describe the feature request
Can we add a feature to Istio, to allow configuring PILOT_ADDRESS with DNS name instead of IP address?
My ingress-gateway is exposed as LoadBalancer on private subnets. External-IP is DNS name of my load balancer.
If I follow the istio.io documentation on how to install Istio on VirtualMachine it doesn't work for people who's on private subnets. I can't map host to host in /etc/hosts file by following the istio.io documentation.
[ ] Docs
[ ] Installation
[ X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context
I couldn't find any work arounds, and if it's already supported please point me to right direction.

@howardjohn if I provide my DNS as PILOT_ADDRESS then I'm getting an error
failed to warm certificate: failed to generate workload certificate: create certificate: rpc error: code = Unavailable desc =

connection error: desc = "transport: authentication handshake failed: x509: certificate is valid for istiod.istio-system.svc, istiod-remote.istio-system.svc, istio-pilot.istio-system.svc, not <DNS_NAME>"

What do I do in that case?

@Arsen-Uulu you need the certificate signed with the DNS name you are using. The best way is to use a gateway to proxy the traffic (see samples/multicluster/expose-istiod-https.yaml)
There is some hack of setting ISTIOD_CUSTOM_HOST env var in istiod to whatever your dns name you are using is but its not likely to stick around forever

@howardjohn so gateway will terminate TLS? Where re-encryption happens again? Little bit confused here.

@howardjohn is it possible to provide  PILOT_ADDRESS with protocol? https://pilot_address:15012
If I just provide pilot_address:15012, istio is attempting to connect over http, and I get error connection reset by peer

@howardjohn If TLS termination is happening on the gateway, is pilot every going to see the certificates that I'm trying to authenticate with?


Wrangler Tail and cloudflared issue fix warning

Original request documented in Jira: https://jira.cfops.it/browse/EW-4988
Wrangler Tail docs should include warning that there should be no cloudflared config file
https://developers.cloudflare.com/workers/cli-wrangler/commands#tail
https://cloudflare.zendesk.com/agent/tickets/2115535

@deadlypants1973 and I worked on this together, lgtm +1+++++1 🚀 🎸


'LongHorizon_Probabilistic.ipynb' - Example Notebook Update

Describe the bug
Loss parameter should be moved from the config variable to the 'AutoNHITS initialization in the 'NeuralForecast' object.
To Reproduce
Steps to reproduce the behavior:

Run 'LongHorizon_Probabilistic.ipynb'
Perform cross validation
See error: 'Please use loss argument instead of config['loss']

Expected behavior
Model(s) are trained via cross validation without a raised issue.
Screenshots
Current notebook issue location:

Recommend update:

Desktop (please complete the following information):

OS: macOS Monterey
Browser: safari
Version: 12.3

Should be implemented with version 1.3. #372


Increase session length

increases session length in production environments from 10 minutes to 3 hours

Also, if this is going to be increased, also increase the RENEW_LOGIN_THRESHOLD in packages/ilmomasiina-frontend/src/api.ts to 30 minutes?


[aggregator] Discard negative rate values

What does this PR do?
Makes rates discard negative values.
Motivation
When rates are used to track counters, and the underlying counters get
reset, we don't want to send a huge negative value that doesn't make
sense.
This change makes the behavior consistent with agent 5.

🙇
About require vs. assert: I use require when the assertion is required for the assertions that follow to not panic, for example to check that a slice is of length >=1 before an assertion on the value at index 0 in the slice.
We're probably not consistent with this everywhere in our test code, so for now I'm just changing this on the test files where I'm making changes...

 Should've just taken a closer look at the changes.


Bump operator-framework/api to v0.3.18

Description of the change:
Bump operator-framework/api to v0.3.18
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

Closes #4104


in windows 7 it's ok,but in linux(including mac), report some errors.

Starting 'document-output'...
[17:26:55] Finished 'document-output' after 59 ms
internal/child_process.js:298
throw errnoException(err, 'spawn');
^
Error: spawn EACCES
at exports._errnoException (util.js:870:11)
at ChildProcess.spawn (internal/child_process.js:298:11)
at exports.spawn (child_process.js:362:9)
at /home/project/node_modules/gulp-jsdoc3/dist/gulp-jsdoc.js:112:121
at /home/project/node_modules/graceful-fs/graceful-fs.js:104:5
at FSReqWrap.oncomplete (fs.js:82:15)

EACCESS is typically a permission issues. Check you have access to all the files and where it is trying to write

in Mac, sudo gulp document-output, the same error occured. i'v checked the directory it's trying to write,  it has w/r  right

EACCES is a permission issue. Do you have access to a root user? If so, can you try not via sudo but with that user? I ask because my next guess is that maybe the permissions are not being transferred correctly with spawn.

i've already used root user before, error also.

Can you comment with the permissions on node_modules/jsdoc/jsdoc.js?
Since the error is coming from spawn itself, I think that jsdoc.js isn't marked executable for some reason.
Try running:
chmod a+x node_modules/jsdoc/jsdoc.js

thank you!  jsdoc.js isn't marked executable.

Good call @AndrewGuenther  thanks for solving this one 👍

@AndrewGuenther @mlucool  thanks for your help


Expand organizations page

Problem description
The organizations page at https://www.pulumi.com/docs/console/accounts/organizations/ could use some more detail on the value of using organizations in the Pulumi Console (collaboration, visibility, RBAC, etc). We should also add the Atlassian (and email?) identity options.

I believe this has been addressed via #1616, which is a general rework of the Console section in the docs.


Upgrade tally

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

Coverage remained the same at 100.0% when pulling 46ac11c16d225ea24832179bf53ff7c3d785dfea on tallyup into b9556711760c45a30bd79c31c8f041d0d9aba997 on master.


Added abstract method test

This is a failing test that illustrates the issue that was found trying out MiMa after this coomit in Akka Build.

Problem, which was illustrated by the failing test case was fixed in #107. This test case now passes, thus merging.


Fixed dotenv in CLI, better success message

Coverage decreased (-0.009%) to 94.979% when pulling 087ec6b364770ebb89802ef60b63fd0e855f1235 on cody-cli-cleanup into 03eb968f200d2d6cb711a460f0c78e3757ca5935 on main.


Emoji picker scroll events can happen while refill_section_head_offsets has unset section_head_offsets

section_head_offsets is temporarily empty while refill_section_head_offsets has yet to run, or while it is in the middle of running.  However, scroll events can attempt to access section_head_offsets[0].section:
https://github.com/zulip/zulip/blob/cd3c4cd991955b73fb8c9f426596dce83d2c9dcc/web/src/emoji_picker.js#L590-L592
This results in an exception.
We should run refill_section_head_offsets synchronously, or just hardcode "Popular" as the value if currently_selected in this case.

Hi! I'm new to zulip and contributions. Can I work on it?

@zulipbot claim

@zulipbot claim

@zulipbot add "good first issue"

@zulipbot claim

Subject: Expression of Interest in Contributing to Issue [Issue Number] - Emoji Picker
Hello,
I hope this message finds you well. My name is Sarthak Pandey, and I am writing to express my interest in contributing to the Emoji Picker module. I have been exploring the project and came across the following issue:
Issue Title: Emoji picker scroll events can happen while refill_section_head_offsets has unset section_head_offsets
Issue Number: 24776
After reviewing the issue, I believe I can make a valuable contribution to resolving it. I have experience in JavaScript, and I am eager to work on this specific problem.
To familiarize myself with the codebase, I have already forked the repository and started looking into the relevant files. I would appreciate any guidance or insights you could provide on how to approach this issue successfully.
I am committed to delivering a high-quality solution and am open to collaboration and feedback throughout the process. If there are specific steps or considerations I should keep in mind, please let me know.
Thank you for your time and consideration. I am looking forward to the opportunity to contribute to [Project Name].
Best regards,
Sarthak Pandey
(Sarthak Paandey)

@zulipbot claim

@zulipbot  claim

@zulipbot claim

@amanagr @alexmv
I want to work on this issue
Can you please assign me this issue?
Thank You : )

Assinged you. You can use @zulipbot claim in future to claim issues

@zulipbot claim

@alexmv @amanagr
Please review my PR
Thank You!!!!

@zulipbot  claim


Conflict with _Controllers

I am  Using  https://pub.dev/packages/solid_bottom_sheet#-readme-tab-
this package and when I tried to open the bottom sheet in onSelected callback of CircularBottomNavigation
it changes the value of the bottom sheet controller too

Closing this cause it is not active for years! If the issue persists please open another one and describe the problem. Thanks!


Change starfile to gemmi

Make sure the star source works with all types of starfile and now just RELION examples

Use ccpem-pipeliner utils


[BUG] The "computeTravelTimeFor" parameter not available in RouteDirectionOptions class

Library name and version
Azure.Map.Routing 1.0.0-beta3
Describe the bug
I need to use the "computeTravelTimeFor" parameter when requesting for route directions, but I am unable to find it in the RouteDirectionOptions class. is it a missing or exposed somewhere else in the library?
Thanks.
Expected behavior
Expect to find the option in the RouteDirectionOptions class just like other URL parameters supported by in the Azure Map Route Directions Rest API
Actual behavior
Not available
Reproduction Steps
Create an instance of RouteDirectionOption class, and try to set the computeTravelTimeFor property.  It is not available.
Environment
No response

Hi @jwangga,
In SDK, it's renamed to TravelTimeType, you can use TravelTimeType.All and TravelTimeType.None to specify computeTravelTimeFor value.
I'll close this issue, but feel free to re-open if you have any other questions. Thank you very much!

@dubiety Thanks for your quick response.  Yes, I found the TravelTimeType option.  However, when I set it to All, the return object Response does not seem to contain:  these values:
"noTrafficTravelTimeInSeconds"
"historicTrafficTravelTimeInSeconds"
"liveTrafficIncidentsTravelTimeInSeconds"
Based on the API, I expect to see these values when TravelTimeType is set to All.  Please advise.

Re-open for further investigation.
@jwangga , I'll get back to you soon!

Hi @jwangga , we've found the issue. We'll fix this issue in our next release around early Jan. Thank you for your patience!

@dubiety Thanks for the update.  Look forward to the next release in Jan. I want to include the integration with Azure Map in our products in the next release scheduled for 1st quarter of 2025.

Hi @jwangga ,
Just confirmed with our API team. There are some discrepancy between the response body and API document. As REST SDK follows the behavior of our API document, you can get the "noTrafficTravelTimeInSeconds",  "historicTrafficTravelTimeInSeconds" and "liveTrafficIncidentsTravelTimeInSeconds" from RouteLegSummary but not from RouteSummary. Please refer to the code samples below:
MapsRoutingClient client = new MapsRoutingClient(credential);

// Create origin and destination routing points
List<GeoPosition> routePoints =
[
    new GeoPosition(13.42936, 52.50931),
    new GeoPosition(13.43872, 52.50274)
];

var options = new RouteDirectionOptions()
{
    TravelTimeType = TravelTimeType.All
};

// Create Route direction query object
RouteDirectionQuery query = new RouteDirectionQuery(routePoints, options);

// Route direction result
Response<RouteDirections> result = client.GetDirections(query);

// Print results and also the additional information
Console.WriteLine($"Total {result.Value.Routes.Count} route results");

Console.WriteLine("Route Let Summary:");
var iRoute = 0;
foreach (var route in result.Value.Routes)
{
    var iLeg = 0;
    foreach (var leg in route.Legs)
    {
        Console.WriteLine($"*** Route {iRoute}, leg {iLeg++} ***");
        Console.WriteLine($"NoTrafficTravelTimeInSeconds: {leg.Summary.NoTrafficTravelTimeInSeconds}");
        Console.WriteLine($"HistoricTrafficTravelTimeInSeconds: {leg.Summary.HistoricTrafficTravelTimeInSeconds}");
        Console.WriteLine($"LiveTrafficIncidentsTravelTimeInSeconds: {leg.Summary.LiveTrafficIncidentsTravelTimeInSeconds}");
        Console.WriteLine($"LengthInMeters: {leg.Summary.LengthInMeters}");
        Console.WriteLine($"TravelTimeInSeconds: {leg.Summary.TravelTimeInSeconds}");
        Console.WriteLine($"ArrivalTime: {leg.Summary.ArrivalTime}");
    }
    iRoute++;
}

And you'll see the output prints something similar to below (the value is subject to change):
Total 1 route results
Route Let Summary:
*** Route 0, leg 0 ***
NoTrafficTravelTimeInSeconds: 131
HistoricTrafficTravelTimeInSeconds: 198
LiveTrafficIncidentsTravelTimeInSeconds: 310
LengthInMeters: 1146
TravelTimeInSeconds: 310

@dubiety Thanks for the update.  It appears that the API documentation on Route Summary does not match the Response returned, which includes the three travel time elements as shown below:
{
"formatVersion": "0.0.12",
"routes": [
{
"summary": {
"lengthInMeters": 104300,
"travelTimeInSeconds": 10448,
"trafficDelayInSeconds": 0,
"trafficLengthInMeters": 0,
"departureTime": "2024-12-03T01:56:15+08:00",
"arrivalTime": "2024-12-03T04:50:22+08:00",
"noTrafficTravelTimeInSeconds": 10448,
"historicTrafficTravelTimeInSeconds": 10448,
"liveTrafficIncidentsTravelTimeInSeconds": 10448
},
... ...
Hence, I expect that the SDK to reflect that.
Here is the URL I used https://atlas.microsoft.com/route/directions/json?subscription-key=&api-version=1.0&query=45.9375%2C123.751%3A45.96875%2C123.791%3A45.90625%2C123.767&language=en-US&computeTravelTimeFor=all&travelMode=truck&traffic=false&routeType=shortest

I understand our document and the APi response is out of sync, but please honor the document and we don't guarantee for those undocumented returned fields will always stay there. Is it possible to look into the RouteLegSummary information instead of the information in RouteSummary since we provide the fields in RouteLegSummary and it also aligns with our API document.

Since no more action item here, I'll close the issue. Let me know if you have any further question. Thank you very much.

@dubiety It seems that the API document needs to be updated.  It just makes sense to have these elements included in RouteSummary. Thanks.

@dubiety   When TravelTimeType is set to All, the RouteLegSummary contains the different types of time for a given leg.  To get, say HistoricTrafficTravelTimeInSeconds, of the route, you would have to loop through all legs to sum them up, while the route summary from the Route Response already includes these values. Therefore, I would strongly believe that the Route summary object in the SDK should expose these values. See the explanation from the API doc, which specifies “all types of traffic information …being included in the summaries in the route response.”  Thanks.

Hi @jwangga , I'll bring this topic back to discuss. Let get back to you soon.


Renaming the init module

As I'm working through the PennyLane tutorial pages I encountered the pennylane.init module. I encountered it for the first time when used like this:
from pennylane.init import strong_ent_layers_uniform

This might be either my inner (tidy) German or my strong feelings about clean & readable Python code but there is something wrong with this. It just doesn't fit into the otherwise very clean and beautiful pennylane package. Therefore, I wanted to ask how you would feel about renaming this module to parameter_initialization so that the import statement looks like:
from pennylane.parameter_initialization import strong_ent_layers_uniform

I'm more than happy to unleash my need for tidiness and implement this change if you are okay with it.

Thanks @markf94! This is something that is on our roadmap to think about and clarify --- this includes moving the initialization functions 'closer' to the actual templates (options include even providing the initialization functions as methods on the templates themselves).

Make sense! So I shouldn't rename this module right now?

Renaming the module here would also require changes made to the tutorials over at XanaduAI/qml, so it might make sense to first discuss and decide if we are going to make more drastic changes to the structure/UI of the initialization functions.
Tagging @mariaschuld to also get her thoughts!

As mentioned in Issue #575, we are planning to overhaul this module soon.
But since this came up a couple of times (for example shortening the function names), we could indeed think of an interim solution? My concern about a rename is that it is feature breaking, and we would then introduce another feature break soon with the larger changes...

Yeah that's a good point actually. We shouldn't be breaking features too often - I guess it all depends on how soon that big refactor is to be expected. Will this take weeks or months? ;)

Months, rather...
I personally would prefer keeping the short name for now (also because it is not likely a severe hindrance for people to use the module), but it is an issue we should keep open to inform the refactor design.
In any case, thanks @markf94, your input is appreciated!

I believe the name was chosen to echo similar naming schemes from ML libraries (e.g., the torch.nn.init module.
Tensorflow has tf.initializers (deprecated), tf.keras.initializers, of just makes things available at top level (tf.constant_initializer).
I think parameter_initialization is probably unnecessarily long, but we can look for a balance here if we think it's unclear to a large set of users


Azure Import and Export pricing

@maku0119 commented on Thu Oct 31 2019
There is "Transaction fee" in "Price details" of Azure import and export, but isn't it wrong for "Transaction fee"? Looks like it costs a special fee.

Standard storage fee and transaction fee will be charged separately.
https://azure.microsoft.com/ja-jp/pricing/details/storage-import-export/

@srvbpigh commented on Thu Oct 31 2019
Hello, @maku0119
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

@CeciAc I think this was added to the wrong repo. This would go to the azure docs repo https://github.com/MicrosoftDocs/azure-docs-pr


syntax error

extra 's:'

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/ide/editorconfig-code-style-settings-reference.md
:white_check_mark:Succeeded

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


fixed(bolt): adsgaefaesfarevba

Changes

Merge activity

Feb 21, 2:54 PM EST: @AngelOnFira started a stack merge that includes this pull request via Graphite.

Current dependencies on/for this PR:

#8  👈
main

This stack of pull requests is managed by Graphite.


Saner division exercises

Name or Id of generator: 3
Issue: example from readme reveals problem with that task: 67/85=  0.788235294117647

it is extremely hard and obnoxious and frustrating for someone learning to divide
division-focused exercise are unlikely to be useful for anyone else

I would propose to switch to integer only results or to something that has no more than N decimal places.
Or have separate "division, integer output task".
This example in readme looks really silly.

Thanks for bringing attention to this issue. A whole number division already existed before this issue was proposed. However, it was strange that the whole number division generator was created after the long decimal division generator. To fix this, I set the division generate to generate whole number solutions and created a new generator called complex_division that generates division problems that can possible have decimal answers which are rounded to the hundredths place after calculation.


Passing CI example for pyglotaran-extras/pull/245

This is just an example of the passing CI when permission changes are applied for
https://github.com/glotaran/pyglotaran-extras/pull/245

The upstream PR is merged so this isn't needed anymore


setDefaultController not working as expected

$routes->setDefaultController() method not working as expected
Default controller not being found with autorouting (without route definition)
Situation:
Routes.php file contains
$routes->setDefaultNamespace('App\Controllers');
$routes->setDefaultController('Pages/showme');
Routes.php does not contain an explicit route definition. The comment in the Routes.php file indicates that this is only required for performance enhancement.
// We get a performance increase by specifying the default
// route since we don't have to scan directories.
//$routes->get('/', 'Pages::showme'); Commented out
A Pages Controller with 'showme' method exists. This is in the Controllers directory and has a namespace of App\Controllers.
Expected behaviour:
Web browser with base url loads the 'showme' method on the Pages controller
Actual behaviour:
404 Controller or its method is not found: App\Controllers\\Pages::showme
If the route is explicitly defined then the route is reached.
It appears that the validateRequest method in Router.php is setting the $this->directory value to '//'. This is leading to auto-routing failing.
CodeIgniter 4 version
v4.0.0-alpha5 develop branch
Affected module(s)
Class Codeigniter\Router
File Router.php
Context

OS: [Ubuntu Bionic]
Web server [Apache <IP_ADDRESS>]
PHP version [PHP 7.2.10-0]

setDefaultController specifies the default controller only, and should not include a method.
Your definition says to look for the Showme controller inside the Pages folder.
The routes->get('/','Pages::showme'), that you commented out, is the correct way to specify a default behaviour for an empty URI.

Thank you for the clarification. Much appreciated. I corrected the definition to specify a controller only as follows:
$routes->setDefaultNamespace('App\Controllers');
$routes->setDefaultController('Pages');
The same behaviour is observed if only specifying a controller (which defaults to the index method)
Controller or its method is not found: App\Controllers\\Pages::index .
Is there an example in the documentation where the default controller is called without explicitly declaring the route?

This could be related to #1400
I am re-opening this, but don't have time to dig into it right now.
I thought we had a bug with double folder separators that was fixed, but I could be wrong.

Thanks for the feedback. This looks to be highly related to the above referenced issue with double folder separators.
Just for clarity, I double checked and the user guide also indicates to use setDefaultController with format 'Controller/method': $routes->setDefaultController('Pages/showme');

Ah - I see it in the tutorial. That is wrong! I will fix it!
With the routing rule "$routes->get('(:any)', 'Pages::showme/$1');", you shouldn't even need to change the default controller.


installing typescript causes file watches with npm start not to work on Linux

i wanted to leave a comment in this thread but it is locked - https://github.com/facebook/create-react-app/issues/659
i hit the same issue where file watches appeared to stop working, and i kept looking at the solutions and none of them seemed to make sense to me. i noticed that whenever i did an npm install typescript, my 'npm start' would stop listening to watches....
spent a lot of time trying to understand why installing typescript in a CRA app would cause this, turns out increasing the # of file watchers fixed the issue as described in the troubleshooting FAQ worked.
please Close - but just wanted to file this ticket in case so no one has to go through the annoyance i went through. my steps to recreate this issue were as simple as create a react app, work on it for awhile and then do an 'npm install typescript', so i feel someone might hit this easily.

@mx2323 can you detail what you did here? That would be great, then I'll close off the ticket.
Thanks for reporting this.

yup - just increased the number of watches seemed to fix it.
https://github.com/webpack/docs/wiki/troubleshooting#not-enough-watchers


ValueError when using custom Generator

Hey there,
I'm having an issue using my own Data Generator in Keras.
I have written the following class:
class DataGenerator(Sequence):
    def __init__(self, filepaths, labels, encoded_labels, max_len=250, batch_size=32, shuffle=True, seed=None):
        self.labels = labels
        self.encoded_labels = encoded_labels
        self.filepaths = filepaths
        self.max_len = max_len
        self.batch_size = batch_size
        self.shuffle = shuffle
        self.seed = seed
        self.total_batches_seen = 0
        self.index_array = None

    def __len__(self):
        return int(np.floor(len(self.filepaths)/ self.batch_size))

    def _set_index_array(self):
        self.index_array = np.arange(len(self.filepaths))
        if self.shuffle:
            self.index_array = np.random.permutation(self.filepaths)

    def __getitem__(self, index):

        if self.seed is not None:
            np.random.seed(self.seed + self.total_batches_seen)

        self.total_batches_seen += 1

        if self.index_array is None:
            self._set_index_array()

        indexes = self.index_array[index*self.batch_size :
        self.batch_size + (index+1)]

        X, y = self.generate(indexes)

        return X, y

    def on_epoch_end(self):
        self._set_index_array()

    def generate(self, filepaths):
        X = []
        y = []

        for fn in filepaths:
            identifier = os.path.splitext(ntpath.basename(fn))[0]
            labels =  self.labels[identifier]
            el = np.array([self.encoded_labels[i][x] for i, x in enumerate(labels.split(";"))])
            el = el[0]
            with open(fn, "rb") as infile:
                seq = pkl.load(infile)
                for index, gs in enumerate(pad_sequences(np.array(seq.generated_sequences), maxlen=self.max_len)):
                    X.append(gs)
                    y.append(el)
        X = np.array(X)
        y = np.array(y)
        return X, y

Which loads in a pre-created pickle object, and then reads its data before feeding back as a standard X, y output.
When I iterate through the generator, and print out the shape of both X and y I see the following:
(1062, 250) (1062,)

However, when I attempt to fit this on the following neural network:
        model = Sequential()
        model.add(Dense(32, input_shape=(250,)))
        model.add(Activation("relu"))
        model.add(Dropout(0.2))
        model.add(Dense(1, activation="softmax"))
        model.compile("sgd", loss="sparse_categorical_crossentropy")

I am greeted to the following error:
(venv) keiron@Pingu:~/Projects/deep-taxonomic-classifier/src$ python test.py
Using TensorFlow backend.
Epoch 1/1
Traceback (most recent call last):
  File "test.py", line 138, in <module>
    clf.fit(datagen)
  File "/home/keiron/Projects/deep-taxonomic-classifier/src/cnntaxpy/deeplearning.py", line 22, in fit
    clf.fit_generator(datagen)
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/engine/training.py", line 1426, in fit_generator
    initial_epoch=initial_epoch)
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/engine/training_generator.py", line 191, in fit_generator
    class_weight=class_weight)
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/engine/training.py", line 1214, in train_on_batch
    class_weight=class_weight)
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/engine/training.py", line 754, in _standardize_user_data
    exception_prefix='input')
  File "/home/keiron/Projects/deep-taxonomic-classifier/venv/local/lib/python2.7/site-packages/keras/engine/training_utils.py", line 136, in standardize_input_data
    str(data_shape))
ValueError: Error when checking input: expected dense_1_input to have shape (250,) but got array with shape (1,)

Is this because the length of X != to the batch_size, due to the prior generated data being loaded in - or have I messed up the Generator object?
Any help would be gratefully appreciated.

you can try this:
X = np.array(X).T
and then shape like this:
(250, 1062 ) (1062,)


Initial audio stutter

@jonnywray - some users are reporting an initial audio stutter and/or desync that can be fixed by quick ff/rewind

I just posted on the plex roku forum that I am having this problem:
http://forums.plexapp.com/index.php/topic/26355-faq-limitations-and-known-bugs/page__gopid__241259#entry241259
I would be happy to debug this issue anyway that I can.  It is mainly the desync that is annoying as I need to rewind everytime I start something on ABC or after every commercial break on Hulu.  I do not have any plex dev experience but would be happy to start dabbling if it is easier for me tweak things myself before you cut an official version.
Here are my specs:
Roku 1
PlexBeta Channel (v1.7.6)
Mac OS 10.5.8
iMac 8,1
2.66 GHz
Plex 9.2.7 (I recently downgraded from 9.3.4 to see if would resolve the issue but still same problem)
I am only running 9.2.7 because 9.5.2 does not support Leopard and have been procrastinating on upgrading to Snow Leopard or Lion.


Added multi surface support

By adding global.surfaces[surface_index] each surface can now have their own streak.

I think the way to deal with no-surface is the same as all-night worlds : use nauvis default.  I mean, how would time be kept in space?

Not sure if you want to keep the announcements, mods that like to add surfaces like to add a lot of surfaces and the console chat gets flooded quite quickly

Is there a chance you can submit such that it doesn't report every line changed because of formatting?  It's really hard to tell what really changed.
Thank you!

I'm still testing things, there's some conflict between the original and this PR, it can't find the surfaces table.  I had no idea the players could stand on many surfaces at once...

Probably because the version wasn't changed in info.json and the save you test with already had this mod loaded. Since there is no configuration change the script.on_configuration_changed doesn't fire
Regarding formatting: Tried to re-commit but my git skills are insufficient and I'm not sure this actually solved the problem, any suggestions?

I think what you've done is excellent and solves multiple things I wasn't sure about (first Factorio mod).
Awesome work!

One more thing I just realized, in the on_tick there's being checked with only if surface always_day then, however I think with this setting the day still "progresses" in the background, perhaps it should be if surface.always_day or surface.freeze_daytime then (or only freeze_daytime)

What's the difference between freeze_daytime and always_day?  it can freeze_daytime at night?

Always_day is only a graphical change to the UI so that it doesn't get dark, but surface.daytime still changes according to surface.ticks_per_day, whereas freeze_daytime doesn't progress the surface.daytime


IOS error on build

Thank you for the plugin. It's a great help.
It works for Android great however at build time I receive the following error message in XCode. I tried what I could but I have no idea where the error comes from. Latest XCode. Anything known?
`Ld /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/CapacitorVoiceRecorder/CapacitorVoiceRecorder.framework/CapacitorVoiceRecorder normal (in target 'CapacitorVoiceRecorder' from project 'Pods')
cd /Users/peterszeman/Documents/chattutor/ios/App/Pods
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -target arm64-apple-ios12.0 -dynamiclib -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS14.5.sdk -L/Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/CapacitorVoiceRecorder -F/Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/CapacitorVoiceRecorder -F/Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/Capacitor -F/Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/CapacitorCordova -filelist /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Intermediates.noindex/Pods.build/Release-iphoneos/CapacitorVoiceRecorder.build/Objects-normal/arm64/CapacitorVoiceRecorder.LinkFileList -install_name @rpath/CapacitorVoiceRecorder.framework/CapacitorVoiceRecorder -Xlinker -rpath -Xlinker /usr/lib/swift -Xlinker -rpath -Xlinker @executable_path/Frameworks -Xlinker -rpath -Xlinker @loader_path/Frameworks -dead_strip -Xlinker -object_path_lto -Xlinker /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Intermediates.noindex/Pods.build/Release-iphoneos/CapacitorVoiceRecorder.build/Objects-normal/arm64/CapacitorVoiceRecorder_lto.o -fembed-bitcode-marker -fobjc-arc -fobjc-link-runtime -L/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphoneos -L/usr/lib/swift -Xlinker -add_ast_path -Xlinker /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Intermediates.noindex/Pods.build/Release-iphoneos/CapacitorVoiceRecorder.build/Objects-normal/arm64/CapacitorVoiceRecorder.swiftmodule -framework Capacitor -framework Foundation -compatibility_version 1 -current_version 1 -Xlinker -dependency_info -Xlinker /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Intermediates.noindex/Pods.build/Release-iphoneos/CapacitorVoiceRecorder.build/Objects-normal/arm64/CapacitorVoiceRecorder_dependency_info.dat -o /Users/peterszeman/Library/Developer/Xcode/DerivedData/App-beknefqbwxnrjvgtzlguknerdxdw/Build/Products/Release-iphoneos/CapacitorVoiceRecorder/CapacitorVoiceRecorder.framework/CapacitorVoiceRecorder
Undefined symbols for architecture arm64:
"OBJC_CLASS$_VoiceRecorderPlugin", referenced from:
_OBJC$CATEGORY_VoiceRecorderPlugin$_CAPPluginCategory in VoiceRecorder-325378aea09476210eff33d5adb008ecf712f34df2b415ce0d8240f209ac93ab.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
asdf
`

I have the same problem too.

This is a clean create-react-app application producing the error: https://github.com/pbackup12345/voice-recorder

Same problem here

I'm on it

I think I found the error itself:
In this file:
/node_modules/capacitor-voice-recorder/ios/Plugin/VoiceRecorder.m
CAP_PLUGIN(VoiceRecorderPlugin, "VoiceRecorder",
should read
CAP_PLUGIN(VoiceRecorder, "VoiceRecorder",
should read

yes i know it comes from the definition, it was because I changed the class name but forgot
to update the definition. i'm fixing another small bug with android and will upload the fixes for android and ios together

uploaded the fix, please let me know if any other error occures


Nested column support

clickhouse-driver PR-s:

nested column
 input_format_null_as_default client_settings setting and array null_value fixes

done. also added formatting for ci pass


rocm-llvm-llvm build failure on x86_64-linux as of a3fe0933

Issue description
Build of rocm-llvm-llvm failed on x86_64-linux as of a3fe093364f627e621deb52e71071e59b14763a4. This is currently breaking python3Packages.wandb.
error: builder for '/nix/store/f19mms8p3cdrj1h63rcalirq1vksjl5l-rocm-llvm-llvm-5.4.4.drv' failed with exit code 1;
       last 10 log lines:
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /nix/store/7wkshj58fcsl1f3zyi67qsxgl1p8nki1-gcc-wrapper-12.2.0/bin/as: BFD (GNU Binutils) 2.40 assertion fail elf.c:3108
       > /build/ccHavjDz.s: Fatal error: lib/Passes/CMakeFiles/LLVMPasses.dir/PassBuilder.cpp.o: No such file or directory
       > [1920/3027] Building CXX object lib/TextAPI/CMakeFiles/LLVMTextAPI.dir/Symbol.cpp.o
       > ninja: build stopped: subcommand failed.
       For full logs, run 'nix log /nix/store/f19mms8p3cdrj1h63rcalirq1vksjl5l-rocm-llvm-llvm-5.4.4.drv'.
error: 1 dependencies of derivation '/nix/store/3kis70f9xjln66pw45myqa8p6x9acinc-python3.10-triton-2.0.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/3v7csz7acrpsip4zxzdg183ncmcnhryj-python3.10-torch-2.0.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/l0bvf3azc47n0fx41yj2akwgjv6n20j2-python3.10-wandb-0.15.0.drv' failed to build

full build log
python3Packages.wandb maintainers cc: @samuela
rocm-llvm-llvm maintainers cc:
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit a3fe093364f627e621deb52e71071e59b14763a4
Run nix-build -A rocm-llvm-llvm

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-1037-azure, Ubuntu, 22.04.2 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.0`
 - nixpkgs: `/nix/store/c29xdsyzw1alhfcrm9zlgww669ws52gb-source`

Contents of ~/.config/nixpkgs/config.nix:
{ allowUnfree = true; cudaSupport = true; cudaCapabilities = ["8.0"]; }

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags: -1830464188520110675<PHONE_NUMBER>309482049

This should be fixed by #232279

Oh wait a second, I just realized that it wasn't running out of space on hydra: https://hydra.nixos.org/job/nixpkgs/trunk/llvmPackages_rocm.llvm.x86_64-linux/all
It looks like it was just failing on whatever device you're running nixpkgs-upkeep on (because it wasn't cached yet and tried to build it from source).
Well... I still think it's useful to mark the ROCm build of llvm as big-parallel anyway 😅

Yeah, sadly this seems to be a sporadic issue with GitHub Actions runners which apparently have quite limited disk space. If this issue persists, I may need to revisit how I'm running this CI...


Search

Elasticsearch needs to be hooked up to the backend to enable search.

PR #15 now solves this problem.


No Features in android-v11.0.0-pre0

Describe the bug
There is no Feature class, and nothing related to that is mentioned in the release notes.
To Reproduce
Steps to reproduce the behavior:

Have an app using MapLibre 10.0.2
Upgrade to android-v11.0.0-pre0 (use .aar file directly)
Do the necessary renames
Notice there is no replacement for Feature and related classes, also Android Studio does not suggest any import with that name

Expected behavior
Either I can continue setting Features in GeoJsonSources, or release notes state what has changed here.
(I guess creating a json string and setting that might work, didn't try)
Platform information (please complete the following information):
irrelevant (app does not compile)

Please import it from
import com.mapbox.geojson.Feature

this class is part of the https://github.com/maplibre/maplibre-java package which is not updated yet.


[Emitted events] day-was-clicked

Hi! Thank you for your work!
Is it possible to add event, after an day in the calendar was clicked?

Hi Maxim.  The option is dayIntervals.displayClickableInterval, and once you've set this to true, you can listen for events with @interval-was-clicked.
Let me know if this is what you pictured, or if you need something else.

Hi Maxim.  The option is dayIntervals.displayClickableInterval, and once you've set this to true, you can listen for events with @interval-was-clicked.
Let me know if this is what you pictured, or if you need something else.

Great!
But what about when month-by-month mode is active (I would like to get a day)? In this case, the event does not occur

Right, that's a good point. In month mode there would be no elegant way of doing that right now. I added this to the road-map, and could try and make it happen this weekend, when we hopefully roll out a couple of other planned features anyway: https://github.com/tomosterlund/qalendar/projects/1#card-86583378
Thank you for your contribution :)

Hello again. This feature is now available in version 1.11.0.
It is also documented here: https://tomosterlund.github.io/qalendar/#emitted-events

Beautiful! Thank you so much!


Move entity types to toofz Leaderboards Core (Data)

Move the following types to toofz Leaderboards Core (Data).

Character
DailyEntry
DailyLeaderboard
Entry
Leaderboard
Mode
Player
Product
Replay
Run

These types currently only serve a purpose as entity types for the data layer. They work entirely independently of the rest of toofz Leaderboards Core. Moving them to the data layer would consolidate related types and make it easier to design and manage them.

Fix by https://github.com/leonard-thieu/toofz-leaderboards-core/commit/db56bd24d52ad92abf3b3e047878df6f52a18b41 and https://github.com/leonard-thieu/toofz-leaderboards-core-data/commit/80b25ec8f616b9d4c407e02f83cb0c5cf324aade.


VS-Code Insiders - Not opening jupyter notebook in Ubuntu 20.04 after latest update of Insider version

After double-clicking on an .ipynb file its opening in a text format.
My VS-Code version details
Version: 1.57.0-insider
Commit: 659175fa39b994fc39381b0af2874c85efeb2a01
Date: 2021-05-13T05:12:59.200Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-53-generic

Have tried uninstalling-and-reinstalling Python and vscode-jupyter extensions, didn't help.
After enabling Developer-Tool getting these details when double-clicking on an .ipynb file.
ERR command 'npm.packageManager' not found: Error: command 'npm.packageManager' not found
    at u._tryExecuteCommand (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1752:3364)
    at file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1752:3245

@rohan-paul can you check the latest Insiders and see if it works?

Yes its working after updating vscode-jupyter extension. Thanks very much.


Rest paramters causing error in node 4.2.6

I am using typescript for my nodejs app, in es6 compilation mode, in some of my classes the generated js code looks like
class mediasController extends lib.baseClasses.apiControllerAsyncBase().apiControllerAsync {     constructor(...args) {         super(...args);         this.xyz = {};     }
it has the es6 rest parameters which nodejs says as "unexpected token...",
please provide me with an alternative or solution of the same .

If you target ES6, TypeScript is going to emit ES6 code (which includes rest args).
If your runtime doesn't support all ES6 features, you can target ES5, or choose not to use the unsupported ES6 features.

i cannot use es5 since i am using async/await and  typescript is automatically generating the rest params how to stop compiler to generate the rest params??

here is the issue tracking more granular transformations: https://github.com/Microsoft/TypeScript/issues/4692

@mhegazy  great ,in which version it is being targeted ??

We are starting with breaking up the lib in the next release, see https://github.com/Microsoft/TypeScript/issues/6974, the emit transformations comes next.

@pranaydutta89 I came across the same problem as yours.
Adding flag --harmony_rest_parameters to launch the app could solve it.
see ECMAScript 2015 (ES6) in Node.js


How To run sample App

Thanks for taking the time for reporting an issue!
Describe what happened
I am not able to run the downloaded sample app and also not able to add the package in new project
Steps to reproduce the issue:
Describe what you expected:
I Just want to run sample app from my android studio
Additional context

Android OS version: 8.0
Device Model:
Datadog SDK version: 1.7.0

Hello @Sagar9821, did @0xnm's advice help you in using the SDK in your project? There's not been activity on this ticket so I'll assume it works, feel free to reopen it if you still have trouble on this specific topic.


Circle shortcut isn't practical

When using one hand for shape selection and another on the mouse for diagraming, its not practical to have the Circle be 'O'. It's much further than the rest of the keys. Why not just have it as 'C' for Circle and call it a day?

I think it's better to group all shapes into single hotkey S followed by W/A/S/D (approach inspired by AoE4)


Errors in latest beta (Godot 3.2.2 beta 4)

The plugin can't load in the latest beta due to the following errors.
SCRIPT ERROR: GDScript::reload: Parse Error: Variable "id" already defined in the scope (at line 101).
   At: res://addons/codeandweb.texturepacker/texturepacker_import_tileset.gd:102.
ERROR: reload: Method failed. Returning: ERR_PARSE_ERROR
   At: modules/gdscript/gdscript.cpp:599.
SCRIPT ERROR: GDScript::reload: Parse Error: Couldn't fully preload the script, possible cyclic reference or compilation error. Use "load()" instead if a cyclic reference is intended.
   At: res://addons/codeandweb.texturepacker/codeandweb.texturepacker_importer.gd:34.
ERROR: reload: Method failed. Returning: ERR_PARSE_ERROR
   At: modules/gdscript/gdscript.cpp:599.

Try removing line 101 in import_tileset.dg
On Mon, Jun 8, 2020, 04:21 Rasmus<EMAIL_ADDRESS>wrote:

The plugin can't load in the latest beta due to the following errors.
SCRIPT ERROR: GDScript::reload: Parse Error: Variable "id" already defined in the scope (at line 101).
At: res://addons/codeandweb.texturepacker/texturepacker_import_tileset.gd:102.
ERROR: reload: Method failed. Returning: ERR_PARSE_ERROR
At: modules/gdscript/gdscript.cpp:599.
SCRIPT ERROR: GDScript::reload: Parse Error: Couldn't fully preload the script, possible cyclic reference or compilation error. Use "load()" instead if a cyclic reference is intended.
At: res://addons/codeandweb.texturepacker/codeandweb.texturepacker_importer.gd:34.
ERROR: reload: Method failed. Returning: ERR_PARSE_ERROR
At: modules/gdscript/gdscript.cpp:599.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CodeAndWeb/texturepacker-godot-plugin/issues/13, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEAT3KVCMYN5QIS55CY2RLRVSGPZANCNFSM4NYC3ESA
.

Thanks for reporting this issue. It's now fixed in the updated version.