When Spring Cloud Kubernetes Is On The Classpath AP4k Generates Invalid YAML

With Spring Cloud Kubernetes on the classpath the YAML that is generated is invalid.

There are multiple RoleBindings
The roleRef is missing the kind and apiGroup properties

Here is an example https://gist.github.com/ryanjbaxter/f0411d70a19faad409715796b28dd656#file-kubernetes-yml

In addition I have noticed 2 additional problems when using AP4K and SCK is

When deploying with the generated YAML the kubernetes profile is not enabled when the app runs in k8s.  It should be activated by default by SCK when deployed to k8s
The role created for the app does not appear to have the necessary permissions to access configMaps so SCK cannot use configmaps for properties sources

It appears the issue with the configmap might be do to the view role not existing in standard Minikube deployment, it does exist as a ClusterRole.
➜  bestseller-toys git:(ap4k) ✗ kubectl get ClusterRole
NAME                                                                   AGE
admin                                                                  90d
cluster-admin                                                          90d
edit                                                                   90d
system:aggregate-to-admin                                              90d
system:aggregate-to-edit                                               90d
system:aggregate-to-view                                               90d
system:auth-delegator                                                  90d
system:aws-cloud-provider                                              90d
system:basic-user                                                      90d
system:certificates.k8s.io:certificatesigningrequests:nodeclient       90d
system:certificates.k8s.io:certificatesigningrequests:selfnodeclient   90d
system:controller:attachdetach-controller                              90d
system:controller:certificate-controller                               90d
system:controller:clusterrole-aggregation-controller                   90d
system:controller:cronjob-controller                                   90d
system:controller:daemon-set-controller                                90d
system:controller:deployment-controller                                90d
system:controller:disruption-controller                                90d
system:controller:endpoint-controller                                  90d
system:controller:expand-controller                                    90d
system:controller:generic-garbage-collector                            90d
system:controller:horizontal-pod-autoscaler                            90d
system:controller:job-controller                                       90d
system:controller:namespace-controller                                 90d
system:controller:node-controller                                      90d
system:controller:persistent-volume-binder                             90d
system:controller:pod-garbage-collector                                90d
system:controller:pv-protection-controller                             90d
system:controller:pvc-protection-controller                            90d
system:controller:replicaset-controller                                90d
system:controller:replication-controller                               90d
system:controller:resourcequota-controller                             90d
system:controller:route-controller                                     90d
system:controller:service-account-controller                           90d
system:controller:service-controller                                   90d
system:controller:statefulset-controller                               90d
system:controller:ttl-controller                                       90d
system:coredns                                                         90d
system:csi-external-attacher                                           90d
system:csi-external-provisioner                                        90d
system:discovery                                                       90d
system:heapster                                                        90d
system:kube-aggregator                                                 90d
system:kube-controller-manager                                         90d
system:kube-dns                                                        90d
system:kube-scheduler                                                  90d
system:kubelet-api-admin                                               90d
system:node                                                            90d
system:node-bootstrapper                                               90d
system:node-problem-detector                                           90d
system:node-proxier                                                    90d
system:persistent-volume-provisioner                                   90d
system:public-info-viewer                                              63d
system:volume-scheduler                                                90d
view                                                                   90d

➜  bestseller-toys git:(ap4k) ✗ kubectl get Role --all-namespaces
NAMESPACE     NAME                                             AGE
kube-public   kubeadm:bootstrap-signer-clusterinfo             90d
kube-public   system:controller:bootstrap-signer               90d
kube-system   extension-apiserver-authentication-reader        90d
kube-system   kube-proxy                                       90d
kube-system   kubeadm:kubelet-config-1.13                      90d
kube-system   kubeadm:nodes-kubeadm-config                     90d
kube-system   system::leader-locking-kube-controller-manager   90d
kube-system   system::leader-locking-kube-scheduler            90d
kube-system   system:controller:bootstrap-signer               90d
kube-system   system:controller:cloud-provider                 90d
kube-system   system:controller:token-cleaner                  90d

Some of the role issues were addressed here https://github.com/ap4k/ap4k/commit/8aac9069fe9964a34259a83ddc2e5780a8fe0339


Gcp cleanups

trying to be better about cleaning up after the integration test framework is unexpectedly and uncleanly terminated.

!ci-build-me

!ci-build-me


Reverse function for array

Describe the problem you are trying to solve

I would like to print out a string backwards. I have split the string into an array of characters, but there is no function to reverse the array.
Describe the solution you'd like

I would like a reverse function to be added for an array. Maybe a for loop could be used in this.
Notes

If this is an implementation of an RFC provide a URL
to the RFC this enhancement implements.
If this is a major enhancement or contribution an RFC may be required. It is ok to submit an enhancement
first and our core team will assist with major contributions. In general, major contributions should be
discussed with the community before submission.

great catch, this is really missing!


.env

What's purpose of this issue?
It's about the .env support?

I’m guessing so. It would sure be great to have more details


Code in comments break parser

// { is enough to break the parser of tst

Same issue - I spent a day trying to find out why my tst was throwing lots of errors, it turned out because I had don't in a single line comment. Changed it to do not and it all works again.

Unfortunately, an apostrophe in valid code also breaks the parser.  E.g. var foo = "don't do it."; doesn't work.
For the sake of reporting a bug well, when everyone says "break the parser" above, what we mean is that if you do any of the above within the curly braces above the template, then every single line of the template code following the curly braces will report a variety of confusing error messages.


Some rules does not check behavior when the mock is instantiate in field variable (instead of local).

In this scenario, the MockBehavior is not tested for some rules.
public class TestClass
{
   private Mock<IService> mock;

   public TestClass()
   {
        this.mock = new Mock<IService>(MockBehavior.Strict);
   }

   [Fact]
   public void Test()
   {
        this.mock.Setup(m => m.GetData());       // No error is reported here.
   }
}

Impacted rules:

PosInfoMoq1000
PosInfoMoq2000
PosInfoMoq2003

To complex to fix/improve.


🌐 Add Korean translation for /docs/ko/docs/tutorial/schema-extra-example.md

Add Korean translation for /docs/ko/docs/tutorial/schema-extra-example.md

📝 Docs preview for commit ebfb6425f1e094efa98f77ff8b32debbc72ce3da at: https://c71f2ae7.fastapitiangolo.pages.dev

Thanks to @riroan for your suggestions, I changed it.

📝 Docs preview for commit<PHONE_NUMBER>0f4a66b113a86102516c622bae6b2f at: https://80b9dfde.fastapitiangolo.pages.dev

Great! Thanks @KaniKim 🚀✨
And thanks for the review @riroan 🤓


Allow for quick-creating Colors from other Colors

(sorry if the title isn't clear, but I wasn't sure how to name it)
The issue is:
Godot 3.1 (or 3.0?) introduced color constants. You can now use Color.red for red etc. However, I sometimes need a translucent red. So I still need to use Color(1, 0, 0, 0.5). This is minor inconvenience of course, but since the Color.red was introduced to avoid Color(1, 0, 0) then why not go further?
There are two "fixes" I can think of:
Color.red.with_alpha(0.5) - it's actually longer, but I'd prefer that for whatever reason. Of course shorter version is possible (and preferred?), like Color.red.w_a(0). (or Color.red.translucent(0.5))
Or Color.red.a = 0.5 could magically return red with 0.5 alpha`
This might of course apply to other Color properties to quickly create colors from other colors, but with minor changes.
Well, this >_>

You can just do
var col = Color.red
col.a = 0.5

or
const HALF_ALPHA = Color(1, 1, 1, 0.5)

var col = Color.red * HALF_ALPHA

Um, the first one doesn't fix anything. I want it to be a one-liner, because I often need this as a method argument. Doing it in two lines is counter-productive.
As for the second one, I didn't know we can multiply colors like that. Looks like it could be enough, but still I need to define the constant each time and it's only for one alpha value.

@KoBeWi you don't have to define the constant:
var col = Color.red * Color(1,1,1, 0.5)

What about ColorN? The first argument is the same constant name in String.
var col = ColorN("red", alpha)

Although a Color constructor that clones a Color and assign new alpha might be handy too.

I added this to my PR for self-constructors and set methods: https://github.com/godotengine/godot/pull/21348
Now you can do this in GDScript:
var myRed = Color.red
var transparentRed = Color(myRed, 0.5)
var copyOfRed = Color(myRed)


Macro limit?

There seems to be a 20 macro limit.
Do you know what could be the cause of this or any idea how to fix it?
If I map more than 20 keys to macros, the rest just stop working.

(This has already been discovered before by a reddit user: https://www.reddit.com/r/MechanicalKeyboards/comments/9hq70z/vortex_core_users_i_present_you_with_a_better/ebi8fuw)

It's a firmware limitations. I am not exactly sure whether it is intentional or not. There is nothing I can do.

Thanks for the explanation (And all your work as well, you made my keyboard MUCH more useful!), we will have to live with this limitation then.


[Fix] Crash in discount screen with low api level

[Fix] Crash in discount screen with low api level

Se agrega el style de DiscountActivity en el archivo styles.xml de api level menor a 21 para evitar el crash.
Se borran algunos styles que ya no se usan.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@d5808cd). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #871   +/-   ##
==========================================
  Coverage           ?   31.62%
==========================================
  Files              ?      326
  Lines              ?    17383
  Branches           ?     1555
==========================================
  Hits               ?     5497
  Misses             ?    11227
  Partials           ?      659

Continue to review full report at Codecov.

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


NETOBSERV-1429 Netflow tab crash

Description
Fix Network traffic tab crash
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[X] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

@jotak that should fix the tab crash you had during last demo :smile:

/ok-to-test

/label qe-approved


Issue opened to join PatternHouse

Hello!
My name is Vaidehi Sinha. I came across this project through the SWOC 2.0 website and would love to contribute to it, considering that its very creative and interactive.
Github Email<EMAIL_ADDRESS>GitHub Username : vaidehisinha1
Thank you!

Hi there, invite has been sent, do change your membership public so that badge is visible on your profile

Thank you! I have accepted the invite and made my membership public.


Forced to reimport compute shader .glsl file after every change for the engine to pick up on the new shader code

Tested versions
v4.2.stable.official [46dc27791]
System information
Godot v4.2.stable - Ubuntu 22.04.3 LTS 22.04 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 970 (nvidia; 535.154.05) - Intel(R) Core(TM) i9-10900K CPU @ 3.70GHz (20 Threads)
Issue description
When working on compute shaders the engine will only notice saved changes after reimporting the .glsl file, either through right-clicking the file and clicking reload or reloading the entire project.
Steps to reproduce

create a project with basic compute shader
make a valid change to the shader code in the .glsl file
save
try running the project again to see the project is still using the old version of the shader

Minimal reproduction project (MRP)
compute_shader_reload_bug.zip

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/72281.


Clean up remote benchmarking documentation

Right now the benchit user guide mentions a --ip argument, however it is not implemented in the code. The code does implement a backend argument for the benchit API (aka benchmark_model() api), which is similar. We are also not sure that we want to release remote benchmarking in the first release.
Proposed solution:

Remove the benchit user guide --ip argument for now. Add it back when we have fully robust remote benchmarking.
Mention in the user guide that remote benchmarking is enabled in the code, but the feature is not under test yet and it requires SFT.
Add a --backend argument that allows selection between local and cloud. Local is the default. Related to #6

Thoughts @danielholanda?

remote sgtm


Adding full width share sheet

Adding option to set a full width share sheet with option to customize
divider
@aaustin @Sarkar @EvangelosG

Compiles / works. I agree w/ #android chat that some header text would look great.

looks great 👍


Remove useless mpv files in apk

Steps to reproduce

For example, the files under the is.xyz.filepicker package should all be useless
Expected behavior
These useless files should not be included.
Actual behavior
A lot of useless mpv files are included in the apk
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
None
Device
None
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

The classes in the is.xyz.* packages are not useless, but legitimate dependencies of Aniyomi. The is.xyz.mpv provides the implementation for video playback using libmpv and my educated guess is that the filepicker package is a dependency pulled in by mpv.

The classes in the is.xyz.* packages are not useless, but legitimate dependencies of Aniyomi. The is.xyz.mpv provides the implementation for video playback using libmpv and my educated guess is that the filepicker package is a dependency pulled in by mpv.

The file picker in the files is the one mpv-android uses, not mpv itself. It's not required to run the player so i see his point in asking for them to be removed.
Sure removing them will give us back 8kb but i think that's quite insignificant compared to the stability that their existence provides when merging from upstream mpv-android in my opinion.

The developer only needs to remove useless files when packaging the mpv-android, and if done properly, this will not reduce the stability of the app.
Details of how to do this can be found here: https://github.com/jarnedemeulemeester/libmpv-android

Also I'm a developer, I use aniyomi's mpv library, but gradle doesn't support removing files from 3rd party libraries, so I can't remove these useless files in my apk, I hope the developer can remove them when upstream packaged with mpv-android, thank you very much!

I've made a similar lib to aniyomi-mpv-lib
But with minimal amount of classes and resources (you'll probably notice ~2mb decrease in apk size if you use it)
https://github.com/abdallahmehiz/mpv-android


Support for Multiple Simultaneous Video Streams with external camera

Description:
We are building an app that requires streaming video from two cameras simultaneously: one built-in camera and one USB external camera. We need support in the Agora Flutter SDK to stream both cameras together. App will be running on Android, Linux and Windows platforms.
Current Capability:
We can detect both cameras using the following code:
_videoDeviceManager = _engine.getVideoDeviceManager();
_videoDeviceInfos = await _videoDeviceManager.enumerateVideoDevices();

for (var device in _videoDeviceInfos) {
  log('device: ${device.deviceId}-${device.deviceName}');
}

Feature Request:
Provide the ability to add multiple camera video streams, including both built-in and external cameras, to the same session.
Proposed Solution:
Methods to create and manage multiple local video tracks for different cameras.
API to publish and render multiple video streams simultaneously.
I am open to use existing solution if available.

I think this example may meet your requirements.
https://github.com/AgoraIO-Extensions/Agora-Flutter-SDK/blob/main/example/lib/examples/advanced/send_multi_camera_stream/send_multi_camera_stream.dart


grpc init

Signed-off-by: Yao Zengzeng<EMAIL_ADDRESS>

Boilerplate header is wrong, you should check "Copyright 2016 The Kubernetes Authors." in your newly added file and make sure they are the same as others.

As the CI error said: you should run hack/verify-flags-underscore.py -e > hack/verify-flags/exceptions.txt to update the list.

@YaoZengzeng pkg/kubelet/hyper/api/ should be renamed to pkg/kubelet/hyper/types/, api.proto should be renamed to types.proto and other scripts/reference of this package.
The grpc pkg name should be kept same with hyperd's one, or else hyperd will complain the PublicAPI is not defined.

lgtm


Error in running it

When I am trying to use this package in Mac OS, Its showing this error:
Traceback (most recent call last):
                                    File "/Users/vasudevgupta/miniconda3/envs/insights/bin/present", line 8, in <module>
                                                                                sys.exit(cli())
                                                                                File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/click/core.py", line 829, in __call__
                                                                                return self.main(*args, **kwargs)
                                                                                File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/click/core.py", line 782, in main
                                                        rv = self.invoke(ctx)
                                                                               File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
                    return ctx.invoke(self.callback, **ctx.params)
                                                                    File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/click/core.py", line 610, in invoke
        return callback(*args, **kwargs)
                                          File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/present/cli.py", line 20, in cli
                                                                                with Slideshow(slides) as show:
            File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/present/slideshow.py", line 90, in __enter__
                                                                                self.screen = Screen.open()
                                                                                File "/Users/vasudevgupta/miniconda3/envs/insights/lib/python3.7/site-packages/asciimatics/screen.py", line 1294, in open
                                                                                curses.cbreak()
                                                                                _curses.error: cbreak() returned ERR

What can be the possible reason??
Thanks

Try running it with TERM='xterm-256color' present slides.md? I would recommend trying it on another terminal emulator.

I tried it. same error occurred.

Which terminal emulators did you try on? I don't have a macOS machine to test, but maybe I can install those terminal emulators and see if it works on them.

@VasudevGupta7 I also added a contributor's guide. Can you read through the filing issues section and provide some more info that's listed there?

Closing due to inactivity, please reopen if you still face the issue.

I got the same error. I've tried both the build-in terminal and iTerm.
>>> import platform; print('Platform', platform.platform())
Platform Darwin-19.6.0-x86_64-i386-64bit
>>> import sys; print('Python', sys.version)
Python 3.7.7 (default, May  6 2020, 04:59:01)
[Clang 4.0.1 (tags/RELEASE_401/final)]
>>> import present; print('Present', present.__version__)
Present 0.5.1

I'm not able to reproduce this as I don't have a MacOS machine. Could you try on another terminal emulator like guake and raise a new issue if you still face the bug?


App Redirect in Provider Hosted Apps is taking long time to load.

Hi @VesaJuvonen ,
Hope you are doing good!
I have been working with provider hosted apps for a while, i have found an issue while loading the app.
The actual app logic in the IFrame loads very fast , but the app redirect which says "Working On it " takes time(Approximately 8-10 seconds) to load.
Could you please give us a solution or any work around for this issue?
Many Thanks

Hey @sidjustsid - any chance you could get a correlation ID for a slow request?  That would help us sort out what might be going on.

Closing as no response to the request for needed information.


Add retext-lexrank to list of plugins

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
A retext plugin for generating unsupervised text summarization using the Lexrank algorithm.

Thanks!


Polymer 2.0

Hi,
May I know if there is any timeline and intention to port this element to Polymer 2.0?
Regards,
John

No objections from me to migrating to the hybrid mode for this element.


Literature data with missing values

Is there a suggested approach for dealing with literature data with missing values? For example a reaction may not have a reported stirring speed or specific analytical method specified. Should the value be left blank or a string like NA?

Hi Nicholas
Thanks for raising that point. So far, it works best if you just leave the fields blank.
Let me know if you need more info around SURF, happy to help.

Thanks! Also, with the atmosphere column, is there any recommendations for how that should be formatted when it's composed of different gasses? Eg, varying ratios of CO and H2 in a hydroformylation. It seems like these should be included as reactants.

This is a good point. So far there's no good way to encode mixtures of atmosphere gases. One could think of adding an atmosphere role and a pressure field, e.g. in atm.

actually I thought about it and what if you would encode the H2 and carbon monoxide as a reagent each? we could add a new column for pressure then

I think "reactant_1_pressure" would be the best solution 👍🏼


feat: migrate to spago@next

can I be added to this repo as maintainer?
@paluh

@Kamirus do You have a right to add me as a member to this repo? I would deploy new version, tnx

@Kamirus do You have a right to add me as a member to this repo? I would deploy new version, tnx

Unfortunately I cannot do that

@rightfold @paluh can we move this repo to https://github.com/rowtype-yoga community?


demo's error is  bash: symbol lookup error: /usr/local/vgpu/libvgpu.so: undefined symbol: cuMemAllocAsync

hi, I need help, when I use demo :
apiVersion: v1
kind: Pod
metadata:
  name: gpu-pod
spec:
  containers:
    - name: ubuntu-container
      image: ubuntu:18.04
      command: ["bash", "-c", "sleep 86400"]
      resources:
        limits:
          nvidia.com/gpu: 3 # 请求2个vGPUs
          nvidia.com/gpumem: 3000 # 每个vGPU申请3000m显存 （可选，整数类型）
          nvidia.com/gpucores: 20 # 每个vGPU的算力为30%实际显卡的算力 （可选，整数类型）

it's error for:
 bash: symbol lookup error: /usr/local/vgpu/libvgpu.so: undefined symbol: cuMemAllocAsync

which CUDA Version and driver version are you using

hi，CUDA version is 11.0, driver version is 450.80.02

please update to the latest version using "helm repo update" and try again

please update to the latest version using "helm repo update" and try again

thank you,I'll try it.


Inline snaphots do not update when ran in vs code or terminal with swift tests when using new swift-testing

Describe the bug
See reproducer project https://github.com/doozMen/swift-snapshot-xcodebuild
To Reproduce
The reproducer project has description in readme.md. But in short

make an inline snaphot test using swift testing
set recording to .all or parameter record: true
run swift test

note: nothing happens

run `xcodebuild -scheme <#package#> test -destination "platform=macOS" -quiet

note: test is updated

Expected behavior
Both inline and file tests should update, but they do not
Environment

swift-snapshot-testing version 1.17.6
Xcode 16.1
Swift 6.0.2
OS: macOS 15

Hi @doozMen, the snapshot recording mechanism depends on XCTestObservationCenter of which Swift Testing has no version right now. They are aware of the issue, but without this there isn't much we can do. Recording for inline snapshots will just need to be done through Xcode for the time being.
Since this isn't an issue with the library I am going to convert it to a discussion. Please feel free to continue the conversation over there!


Release Google.Cloud.PolicyTroubleshooter.V1 version 2.4.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.PolicyTroubleshooter.V1-2.4.0


Avoid changing buffer on error

Great plugin, super useful.
One issue I had is that if for some reason astyle errors out, it will replace my buffer with whatever error message it produced. To reproduce, if you add --foobar or some other bad option to format args, the buffer will be replaced by:

Invalid command line options:
foobar
For help on options type 'astyle -h'
Artistic Style has terminated

Would it be possible to restore the original buffer if the formatter returned 1?

This should be fixed in the new update.


Added end to end test for mem ballast

This commit adds an end to end test to verify performance gain acheived
with memory ballast
It mainly checks the CPU usage and does not test against memory usage.
I'll be submitting another PR that will specifically test memory
behaviour and ensure the ballast does not actually consume memory.
TestNoBackend10kSPS test was very reliably consuming >20 CPU on my
machine so I increased the accepted values to 30 for this test.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5b9be2f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #80   +/-   ##
=========================================
  Coverage          ?   71.33%
=========================================
  Files             ?       91
  Lines             ?     6025
  Branches          ?        0
=========================================
  Hits              ?     4298
  Misses            ?     1503
  Partials          ?      224

Continue to review full report at Codecov.

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

@owais  Build status is not reported from Travis. You may want to restart.

Looks like a bug. The build has passed: https://travis-ci.org/open-telemetry/opentelemetry-service/builds/552958842?utm_source=github_status&utm_medium=notification
I don't have the option to restart builds for this repo and it's PRs.

It will restart automatically after you merged the other PR and rebase this one.

Failed because of https://github.com/open-telemetry/opentelemetry-service/issues/43

Yea, I've been seeing quite a few random failures. Will rebase after we merge #85

@owais did you make changes or just rebased? If you need to rebase it is better to use "Update branch" button in Github. It reruns CI, but does not invalidate reviews (unlike rebasing and pushing the branch manually, which requires re-approval). So you don't need to ask for reapproval in that case.

@tigrannajaryan I rebased. Update branch features ends up creating a merge commit as it merges master into the branch. Is that acceptable?

Update branch features ends up creating a merge commit as it merges master into the branch. Is that acceptable?

@owais I don't think this is true.

CI hit https://github.com/open-telemetry/opentelemetry-service/issues/59

@owais I don't think this is true.

I see. I meant it creates a  merge commit in the branch when updating from master but that wouldn't matter if the repo has only "squash and merge" available.
Update again. Thanks.


Add Grant to the Authentication section

OAuth middleware for Express and Koa - the simplest way to deal with OAuth in the NodeJS world. You can take a look at the code and the documentation https://github.com/simov/grant and the showcase https://grant-oauth.herokuapp.com/
Currently supported providers 84 with Hapi support underway.

Thanks :+1:


datalad catalog create requires an option --catalog_dir

The naming is off wrt to datalad conventions (should be --catalog-dir). Conceptually this should probably be an argument, not an option.
Maybe it could/should also have a sensible default (maybe CWD)?
Update: so the option name is OK, just the error message mislead me:
❯ datalad catalog create
catalog_create(impossible): [Datalad catalog create requires a path to operate on. Forgot -c, --catalog_dir?]

Update2: It would be useful to clarify the semantics of catalog-dir in the help output. It says "Directory where the catalog is located or will be created". Too many things are a catalog, I think. --metadata can also be considered to point to a catalog (of JSON records). Maybe spell it out "rendered" or "HTML" catalog?

Closing this since the command was revamped with the v1 release previously.


Document scalar arrays

We support scalar arrays with Postgres. That means the user can write the following:
model User {
  id @id
  names String[]
}

However, this is only support by some databases such as Postgres and not MySQL/SQLite. It would be great if we could add "scalar arrays" in a separate section under "Reference"->"Prisma Schema" of how it works, and also noting for what databases this works.

Documented here: https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-schema/models#lists


✨ Support showing hover information for identities

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Multiple terminfo locations

I'm on a Ubuntu 16.04 box. Automatic terminfo detection in the color.py module didn't work for me.
There are several terminfo directories in my root.
find / -type d -name terminfo -print

returns
/usr/share/terminfo
/etc/terminfo
/lib/terminfo

/etc/terminfo contains nothing but a README saying

This directory is for system-local terminfo descriptions. By default,
ncurses will search ${HOME}/.terminfo first, then /etc/terminfo (this
directory), then /lib/terminfo, and last not least /usr/share/terminfo.

Indeed, I found the relevant entry /lib/terminfo/x/xterm-256color. The directory /usr/share/terminfo is empty.
I can submit a PR fixing this by iterating over the possible terminfo directories in get_terminfo_file(), looking for an existing file.

For my own future reference, I did look up how ncurses does it. Turns out they have no standard place for the database either.

Released as 0.1.5!


Added link to a popular SQL injection tool

sqlmap is a very handy tool to use when testing for SQL injection vulnerabilities.

Needs to merge master into this branch before it can be accepted.

Upstream changes have been merged

20/100


replace overlay schedule with arithmetic checks

This PR replaces the overlay schedule mapping with a couple arithmetic checks. Note that this PR does break the ledger state serialization (the overlay schedule is now gone).
Checking if a relative slot s is an overlay slot amounts to checking:
ceiling (s*d) < ceiling ((s+1)*d)

ie the next multiple of d (the decentralization parameter) raises the ceiling.
If a relative slot s is indeed an overlay slot, then it is an active slot if 1/f (where f is the active slot coefficient) divides ceiling (s * d). And we can find the core node for a given active overlay slot by checking it's remainder mod the number of core nodes (after dividing by 1/f, which is 20 on mainnet).

We no longer store the overlay schedule in the ledger state, the mapping is replaced by performing the checks above in the appropriate changes. The formal spec will need to be updated to match.
A few of the STS environments changed as a result. The NEWEPOCH and TICK environments are now empty/unit. The OVERLAY and PRTCL environment now takes the decentralization parameter d in place of the overlayschedule.
Removing the overlay schedule and the above changes caused several of the API methods to change.

Lastly, I added more validation when updating the d parameter. We now require that d be in hundredths.
closes #1825

FYI, a draft PR propagating it to consensus: https://github.com/input-output-hk/ouroboros-network/pull/2601/

At a pinch, we could avoid doing this and instead ensure we have run this version on the existing chain and it validates (we obviously will need to do this anyway). But I think it would be better if we had a quickcheck test here.

yea, I was thinking that the only "proof" that really matters is that we can sync a node an mainnet with this chain. but a quickcheck test with the old code seems like a good idea!

At a pinch, we could avoid doing this and instead ensure we have run this version on the existing chain and it validates (we obviously will need to do this anyway). But I think it would be better if we had a quickcheck test here.

yea, I was thinking that the only "proof" that really matters is that we can sync a node an mainnet with this chain. but a quickcheck test with the old code seems like a good idea!

I'm syncing a node with mainnet from scratch (with this change included of course) as we speak :slightly_smiling_face:

At a pinch, we could avoid doing this and instead ensure we have run this version on the existing chain and it validates (we obviously will need to do this anyway). But I think it would be better if we had a quickcheck test here.

yea, I was thinking that the only "proof" that really matters is that we can sync a node an mainnet with this chain. but a quickcheck test with the old code seems like a good idea!

I'm syncing a node with mainnet from scratch (with this change included of course) as we speak slightly_smiling_face

I synced the entire mainnet without issue.

Looks like an interesting (unrelated?) error on hydra

Looks like an interesting (unrelated?) error on hydra

I'm getting it consistently failing in the same way locally as well. Both before and after the code review changes. It seems it passing last time may have actually been lucky/unlucky.
It's weird though, I don't think anything having to do with the deposit mechanism was changed...

thanks to @uroboros we figured out why the property tests were failing. the assertions were a red herring, masking the real error. the block generator was creating the wrong starting epoch slot due to my change. with the assertions off, you could see that the error was the "wrong overlay slot". it was an easy fix from there.


Consolidate Contact Us/report a bug pages between Release Planner and Scheduler apps

It will be good to have both apps feel the same, so expose information similarly.
See different approaches:

As for the Release Planner app reporting a bug on Github, we should tell them the repo and to add the Engagement Experience label.
FYI @ladonnaq @josefree

Agree. Let me bring this to engineer for triage

team channel: https://teams.microsoft.com/l/channel/19%3A59dbfadafb5e41c4890e2cd3d74cc7ba%40thread.skype/Engineering%2520System%2520%25F0%259F%259B%25A0%25EF%25B8%258F?groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121&tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47
GitHub Issue: https://github.com/Azure/azure-sdk-tools/issues/new

For GitHub Issue, I wonder if we would create an Github issue template, so that the feedback can directly flow into Engagement Service project with some context infor.  Some questions in the template maybe

Category: Bug report, Feature requirement
Which app: General, Release planner, API spec readiness, SDK release
Description
Screenshot

This issue is realted to https://github.com/Azure/azure-sdk-tools/issues/4642

For teams channel the conclusion was to use the Onboarding channel. The Scheduler Tool has already been updated to reflect that. I am unsure about the status of the Release Planner App.
Love the idea about the Github issue template :)

Hi @maririos , The screenshot you posted in Scheduler app is not contact us page. It's bug reporting page. I looked at the contact us page in Schedular app.

So, Did you want to us follow the page to update contact us or add a new page for bug reporting?

Updated to 'Report a bug' page in Release planner app, API readiness app and SDK release app.

thanks @honghr . @ccbarragan is adding this screen into her UI design so we can have the best experience for our users


Fix F.det with CuPy to be consistent with NumPy

Fixes a recent incompatibility between CuPy/NumPy in F.det. C.f. https://github.com/chainer/chainer/pull/8434#issuecomment-553723111

Related to cupy/cupy#2437 and #8434.

Jenkins, test this please.

Jenkins CI test (for commit a1b799a88d6b12b2f8a1b2d3c49a25fca28275a4, target branch master) failed with status FAILURE.

Shouldn't we avoid synchronization in non-DEBUG mode?

Please refer to this comment (https://github.com/chainer/chainer/pull/613#discussion_r53724185) and a subsequent fix in this PR (#989).

Can I close this PR after #8482 is merged?

@asi1024 Yes, I believe so.


[Feature]: More Documents on Motivations and What is Remix

What is the new or updated feature that you are suggesting?
Full-fill more documents about what is Remix and inside.
Why should this feature be included?
Better understanding for new people

The documentation have evolved and there a new dozen of samples.


Question around seemingly optional <Suspense /> usage

Hello RevereCRE, congratulations on the 2.0.0 release!
I've been adopting relay-nextjs where I work and it's pretty unbelievable how much of an improvement the 2.x release is in terms of boilerplate reduction.
In my testing, I've found that in the component there appears to be an optional suspense boundary. And I'm wondering if either I'm missing something, or if this can optionally be entirely removed.
I've been patching relay-nextjs and removing said Suspense boundary and it can create the illusion that the layout is "shared" across navigations..
The default behavior with the Suspense boundary:
https://github.com/RevereCRE/relay-nextjs/assets/2148168/58c15fdd-b61d-4828-a35c-41d763aea336
With a relay-nextjs patch, removing the Suspense boundary:
https://github.com/RevereCRE/relay-nextjs/assets/2148168/082aebc0-0037-4ced-9a00-5e851f3b66a0

And finally, the patch itself
relay-nextjs+2.0.1.patch

Hey @hanford, thanks for the kind words!
This behavior is actually something we were looking for at Revere. Rather than waiting on the data for the page to complete navigation we wanted an early transition. It's actually one of the main reasons this repo entirely avoids getServerSideProps!  We have it set up in our app such that our placeholders render a similar UI to the real rendered page (basically <Wrapper> <LoadingLogo /> </WrapperForRelay>) which avoids the very jarring Loading... and makes the transitions much smoother.
I totally understand this isn't desirable for everyone using this library and we were hoping with the release of the startTransition API this would be configurable by users, however we haven't really investigated too deeply. I'm a bit concerned that removing that Suspense boundary may impact other use-cases like changing GraphQL parameters and will also look into that.
In the meantime I definitely recommend using a placeholder option that looks a lot more like the actual application UI. It's actually possible to get it really close to what would actually be rendered with user data too by using a useLazyLoadQuery to read from the store UI. Here's an example of how we do it:
function PlaceholderWithWrapper() {
  const { viewer } = useLazyLoadQuery<placeholder_WrapperQuery>(
    graphql`
      query placeholder_WrapperQuery {
        viewer {
          name
        }
      }
    `,
    {},
    { fetchPolicy: 'store-or-network' }
  );

  return (
    <Wrapper viewerName={viewer.name}>
      <LoadingLogo />
    </Wrapper>
  );
}

export function Placeholder() {
  return (
    <Suspense fallback={<LoadingLogo />}>
      <PlaceholderWithWrapper />
    </Suspense>
  );
}

Hope this helps!


A error reported when using preview

the following error will be reported when opening the recent files if preview is used to generate the header.

reproduce step fillissue template

Actually the error happens when opening any files directly in the dashboard, such as using mru hotkeys or commands like :enew, :e filepath. But, Opening files using recent projects and nvimtree won't cause this error.


PCRE2 support still requires PCRE1

libmodsecurity 2.9.7 compiles fine using --with-pcre2 only when both pcre1 and pcre2 dev packages are installed, which is wrong...
Moreover, the resulting binary links against both libraries... which is further wrong.
ldd mod_security2.so |grep pcre
ldd: warning: you do not have execution permission for `./mod_security2.so'
libpcre.so.1 => not found
libpcre2-8.so.0 => /usr/lib64/libpcre2-8.so.0 (0x0000ffffb9194000)
...
If only pcre2 dev package is installed, it fails at ./configure --with-pcre2 saying:
checking for libpcre config script... no
configure: *** pcre library not found.
configure: error: pcre library is required
In other words, --with-pcre2 still requires pcre1 dev.
This question is the same as the previous one,however, this issue is not resolved in v2.9.7.

Hello @zhaoshikui ,
Just checking my config.log from when the relevant testing was done and I can see:
configure:16348: checking for libpcre config script
configure:16399: result: no
configure:16412: *** pcre library not found.
configure:16445: checking for libpcre2 config script
configure:16475: result: /usr/bin/pcre2-config
configure:16507: using pcre2 v10.39

... and a successful completion of the configure step.
Moreover, the ModSecurity v2 message 'pcre library is required' was removed here: https://github.com/SpiderLabs/ModSecurity/commit/8fc0b519b7a6c023259753a21f33bf3649a25b14
Could you please double-check what you've done? If you still see the same result, a more complete description of your build environment and steps to reproduce might be helpful.

Compiling master on Rocky 9. pcre is still linked:
> ldd /usr/local/lib64/httpd/modules/mod_security2.so
        [...]
        libpcre.so.1 => /usr/lib64/libpcre.so.1 (0x00007f73ca1b4000)
        libpcre2-8.so.0 => /usr/lib64/libpcre2-8.so.0 (0x00007f73ca118000)
        [...]

Compiling master on Rocky 9. pcre is still linked:

you mean the current mod_security2 master?
With 2.9.7 on Debian 12 I get:
$ ldd /usr/lib/apache2/modules/mod_security2.so | grep pcre
	libpcre2-8.so.0 => /lib/x86_64-linux-gnu/libpcre2-8.so.0 (0x00007ff7be17b000)

only pcre2 dependency, after commit 05396552.

Yes, latest v2/master.
Here are my configure flages: --enable-pcre-study --enable-pcre-jit --with-pcre2

We may be able to compile normally, but we cannot use it。
When I use this library, prompt me：
libpcre.so.1: cannot open shared object file: No such file or directory.

Thanks, all, for the various comments and observations.
I cannot reproduce what has been described above.
I mostly build in Ubuntu, and if pcre1 is not installed and --with-pcre2 is specified, then the build runs to completion and the resulting .so file functions normally. If pcre1 is installed and --with-pcre2 is specified then configure will search for the relevant files and include references to same in the Makefile ... but pcre1 is still not present when checking via ldd.
Nevertheless, to whatever extent the current situation is causing difficulty or annoyance for anyone, I have created PR #2975 that will omit the check for pcre1 entirely if pcre2 is specified. Feel free to apply the PR if you find it helps you.
I'm not sure that this will be merged immediately since this would break builds of the optional alp2 (I'm moderately sure that it is obsolete but ...)

Thank you all.


chore: upgrade sdk to v0.45.0x-osmo-v7.9

Closes: #1356
What is the purpose of the change
Upgrades SDK to v0.45.0x-osmo-v7.9 with the fix for https://github.com/osmosis-labs/osmosis/issues/1356
The relevant SDK PR changes query handling to avoid the data races experience by integrators.
Brief change log
updated the SDK version
Testing and Verifying
This change is a trivial rework / code cleanup without any test coverage.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not applicable

Codecov Report

Merging #1390 (1608003) into v7.x (f82a6fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v7.x    #1390   +/-   ##
=======================================
  Coverage   20.63%   20.63%
=======================================
  Files         199      199
  Lines       25803    25803
=======================================
  Hits         5325     5325
  Misses      19509    19509
  Partials      969      969

Continue to review full report at Codecov.

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


Mark decoder and parser not as unstable anymore

This PR marks the defmt-decoder and defmt-parser as not unstable anymore. We did uphold semver for years (except one accidental breakage), so making that explicit just makes sense.
Users now do not need to pass unstable anymore in order to use this crate. To be precise they are not allowed to pass it anymore, therefore this is a breaking change, as it requires users to update their Cargo.toml. It will be released together with two other breaking changes. For probe-rs we gonna change it in https://github.com/probe-rs/probe-rs/pull/2809.

Would it be better to leave the feature flag, but as a no-op, and mark it as deprecated in the documentation? Then it isn't a breaking change.
We can take it out later, but let's give people a nice notice period so they can upgrade at their leisure.

@jonathanpallant Added it back


Build: Fix Demo/RX200_RX231-RSK_GCC_e2studio_IAR RTOS demo project IAR build error

Description
This PR fixes the following link error of Demo/RX200_RX231-RSK_GCC_e2studio_IAR RTOS demo IAR project by removing no longer necessary files at least using EWRX V4.14.1. Additionally, code for new evaluation board TB-RX231 is added to test this PR.
Link error messages:
Error[Ta001]: Mismatching linker configuration file version. Compare your file with the ready-made linker configuration files (.icf) in the rx\config directory!
Error[Lp011]: section placement failed
            unable to complete "place at" directives with a total estimated minimum size of 0xfc bytes in <[0xffff'ff80-0xffff'fffb], [0xffff'fffc-0xffff'ffff]> (total space 0x80).
            Needed:
            "A1"
            [0xffff'ff80-0xffff'fffb]: 0xf8 min, align 0x1 (size: 0x7c)
            "A2"
            [0xffff'fffc-0xffff'ffff]:  0x4 min, align 0x1 (size:  0x4)
Error[Lp015]: section placement failure: overcommitted content in [0xffff'ff80-0xffff'fffb]

Cause of the error: Both compiler's default exceptvect.o and this project's exceptvect_rxv2.o are linked.
Fix of the error: The following files and folder are no longer necessary so that these files are removed.
src/IAR_Support/defaults.s
src/IAR_Support/exceptvect_rxv2.s
src/IAR_Support/option_rom_rxv2.s
src/IAR_Support/lnkr5f52318.icf
src/IAR_Support/

TB-RX231:
Target Board for RX family
https://www.renesas.com/products/software-tools/boards-and-kits/eval-kits/rx-family-target-board.html
Test Steps
Both full_demo and blinky_demo are tested to be run expectedly. TB-RX231 is used by defining 'TB_RX231' on EWRX as following screen copy. SREC file is burn into MCU on TB-RX231 using RFP(Renesas Flash Programmer) software via on-board E2 Lite emulator hardware. But I don't have any RSK-RX231 so that programs aren't tested using any RSK-RX231.

Related Issue
None.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

In this Demo folder, source files are shared between IAR project and GCC project. So, I test also GCC project using e2 studio v<IP_ADDRESS> and KPIT GNURX v15.01-SP1. (According to .info file, the project was originally built with the same versions of the tools. To be exact, I'm not sure which version of KPIT GNURX v15.01 or KPIT GNURX v15.01-SP1 was used to build the project but I think that it isn't so important.)
Both full_demo and blinky_demo are tested to be run expectedly. TB-RX231 is used by defining 'TB_RX231' on e2 studio as following screen copy. MOT file is burn into MCU on TB-RX231 using RFP(Renesas Flash Programmer) software via on-board E2 Lite emulator hardware. But I don't have any RSK-RX231 so that programs aren't tested using any RSK-RX231.

/bot run checks

Hi @ VanNamDinh, can Renesas help review this PR to make sure the changed are expected.

@lundinc2 test

/bot run checks

/bot run checks

Dear @NoMaY-jp , @mingyue86010
Thanks for your PR and review for this issue. It's ok from us.

/bot run checks

/bot run checks

/bot run checks

/bot run checks


ArgumentException when Animation.Duration has a Binding in XAML

Setting Animation.Duration with a Binding like so:
<Animation IterationCount="Infinite" Duration="{Binding Duration}">
    <!-- ... -->
</Animation>

Causes the following exception on startup:
System.ArgumentException: 'Value of type Avalonia.Data.Binding cannot be assigned to member with type System.TimeSpan
Parameter name: value'
   at Portable.Xaml.MemberExpressionExtensions.<>c__DisplayClass8_0.<BuildCallExpression>b__0(Object instance, Object value)
   at Avalonia.Markup.Xaml.PortableXaml.AvaloniaXamlObjectWriter.DelayedValuesHelper.ApplyAll()
   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.LoadFromReader(XamlReader reader, AvaloniaXamlContext context, IAmbientProvider parentAmbientProvider)
   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.Load(Stream stream, Assembly localAssembly, Object rootInstance, Uri uri)
   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.Load(Type type, Object rootInstance)
   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.Load(Object obj)
   at Sage.Views.MainWindow.InitializeComponent()
   at Sage.Views.MainWindow..ctor()

Setting Duration to a hardcoded value, or creating the binding in code works as expected:
public class MainWindowViewModel : ViewModelBase
{
    public void Test()
    {
        var anim = new Animation();
        anim.Bind(Animation.DurationProperty, new Binding { Source = this, Path = nameof(Duration) });
        Console.WriteLine("Duration: " + anim.Duration.Seconds);
    }

    public TimeSpan Duration => TimeSpan.FromSeconds(3);
}

The above example prints Duration: 3.

Those properties (Animation.cs) are Direct properties, perhaps they could be changed to Styled ones? But iirc bindings should work with Direct Properties too...

Yes, bindings should work with direct properties.

This has been fixed at some point. Closing the issue.


The request interface and parameters of the model gpt-3.5 have changed

new interface address：https://platform.openai.com/docs/api-reference/chat/create
Updated the request interface, as well as the request parameters, there may be a lot of things that need to be changed
I'd like to know if I'm already following up.

thank you, i see https://github.com/sashabaranov/go-gpt3/pull/90

It looks like there is no way to get the token consumed each time in the case of using stream.


beginning damage depth

We need to handle beginning damage depth here:
https://github.com/HydrologicEngineeringCenter/HEC-FDA/blob/62df772cf4c0c4d666fe6abf49b9196eec5bdb46/fda-model/structures/DeterministicStructure.cs#L41

I am going to set beginning damage depth to 0 feet by default. this way we don't check for beginning damage depth, we'll just always use it and it won't matter unless beginning damage depth has been identified by the user to be something different.

So I should forget about this?

No, sir. We still need to handle beginning damage depth at the line indicated above.
We had previously discussed checking to see if beginning damage depth is not null, and if not, truncating the stage-damage function. Now that I think this through, the default value should be -999. You can truncate the depth-percent damage function at the beginning damage depth for all structures - independent of whether the structure is actually using the parameter. This way, if the structures beginning damage depth is the default value of -999, it will be impossible to truncate the depth-percent damage function. Truncation should only occur for values of about -8 and above.

Part of this issue should be to reset the default value back to -999

I'm tracking, thank you for clearing that up


Keyboard Shortcut to go back and forth between reading area and editor.

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to reach my mouse to go back and forth between reading area and editor, just because I'm using a small screen, I can't read the whole thing and need to scroll down and then go back to editor using mouse. This causes frustration which often prevents me from coding, for long hours.
Describe the solution you'd like
A keyboard shortcut to go back and forth would definitely help.
Describe alternatives you've considered
I have tried to scale down my browser window, but it creates readability issues, which not only makes it hard to read but it's difficult for the eyes too.
Screenshot

Hi, that's possible, though we haven't documented it yet, sorry!  The plan has been to make all this clear via a legend, but that PR, https://github.com/freeCodeCamp/freeCodeCamp/pull/38203, is still in the works.  A package I needed has been updated today, so that should be doable now.
Anyway, as it stands, you can use Esc to leave the editor and e to go back to it.  That's going to change (because Esc has multiple other uses), but when it does the new hotkey will appear in the legend.
I'll leave this open for now, since it highlights the issue, but that PR will close it.

I tried using Esc and e, it somehow does the job but not complete. Anyway, I hope it gets added soon and goodluck for the work.

Thanks!
What goes wrong with Esc and e? It might be something we can fix.

No, the e works. It does take you to the editor, only if you are in reading pane. However, the Esc just takes you out from there to nowhere. If it could take you back to reading pane or maybe a combination of it, that would be great.

something like r if its not used? In escape mode obviously.

Nope, didn't help.

Nope, didn't help.

Oh - I meant we would need to add this as a shortcut combination. It does not exist yet. 😅

Oh - I meant we would need to add this as a shortcut combination. It does not exist yet. 😅
My bad, I misunderstood. Anyway, that'd be convenient.


1

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "C:\Users\Dicer\AppData\Local\Programs\Python\Python36\lib\site-packages\qtconsole\base_frontend_mixin.py", line 138, in _dispatch
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
File "C:\Users\Dicer\AppData\Local\Programs\Python\Python36\lib\site-packages\qtconsole\base_frontend_mixin.py", line 138, in _dispatch
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
File "C:\Users\Dicer\AppData\Local\Programs\Python\Python36\lib\site-packages\qtconsole\base_frontend_mixin.py", line 138, in _dispatch
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.7
Qt Versions: 5.10.1, PyQt5 5.10.1 on Windows

Dependencies
pyflakes >=0.6.0 :  2.5.0 (OK)
pycodestyle >=2.3:  2.9.1 (OK)
pygments >=2.0   :  2.13.0 (OK)
pandas >=0.13.1  :  None (NOK)
numpy >=1.7      :  None (NOK)
sphinx >=0.6.6   :  5.2.3 (OK)
rope >=0.9.4     :  1.1.1 (OK)
jedi >=0.9.0     :  0.17.2 (OK)
nbconvert >=4.0  :  6.0.7 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  None (NOK)
qtconsole >=4.2.0:  5.2.2 (OK)
IPython >=4.0    :  7.16.3 (OK)
pylint >=0.25    :  2.13.9 (OK)

Hey @nikoioi, thanks for reporting. This error happened because you're using a very old version of Spyder.
To fix it you need to update it by opening a system terminal (i.e. cmd.exe) and running there
pip install -U spyder

That will give you a more recent Spyder version.


Add curl examples to the devdocs

Is your feature request related to a problem? Please describe.
As a consumer of the oCIS dev docs i want to know more about api calls
Describe the solution you'd like
Add more curl examples to the dev docs
Describe alternatives you've considered
none
Additional context
This will be important for beta testers
List of interesting APIs

[ ] Drives
[x] Users (https://github.com/owncloud/ocis/pull/3149)
[x] Groups (https://github.com/owncloud/ocis/pull/3149)
[x] WebDAV Spaces Endpoints
[ ] Share OCS (add people to spaces)
[ ] Permissions and Roles API

@phil-davis @pmaier1 @dragotin FYI

[ ] Share OCS (add people to spaces)

do we still need to do it?

[ ]  Share OCS (add people to spaces)

do we still need to do it?

Not needed anymore.


Creating a "New" Hub Virtual Network using AZFW Manager ignores the A…

Despite selecting two or more AZs at creation time, post deployment, the properties of AZFW inside of the AzureFirewallSubnet of the new Hub virtual network points to an absence of AZs.  Furthermore, the deployment summary inside of the Azure Portal doesn't specify that AZs were selected.  This needs to be fixed urgently, since customers will be of the opinion that they're protected against DC failures in region when they're not.

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

@vhorne - Could you review this pull request? It looks like the update in the article is a meta comment and not a literal edit; do not merge as-is. Thanks.
#hold-off

Investigating with product team...

This issue has been fixed in the portal, so no need for a doc change.
#please-close


Git assignment #2:Branching and resolution

From here:
https://gist.github.com/cengique/41ab5ce6cc881262e5e50e4b66144235

Good


Prevent double termination in test pullable source

This is necessary because of nested function calls, i.e. nested pulls.

Thanks!


fix(artifacts:zk): serialize byteecode without prefix

With #35 an accidental change was introduced which serialized bytecode with the 0x prefix where before it was stripped.
This PR reintroduces the stripping behavior by ensuring the serialization happens thru alloy_primitives::hex::encode first

Is it worth it to add a test that deserialization is what we expect?

Is it worth it to add a test that deserialization is what we expect?

Added such test, tried to keep it simple


Disable ThemeCheck on vite-tag liquid snippets

We run all liquid through the Shopify Theme Check linter as part of deploy CI/CD
This prevents the generated snippets from raising any lint errors/warnings.
Before:

After:

Thanks for the great project

I would love your thoughts on not turning off these checks so we can catch the issue and then build production assets for deployment (:

@builtbylane - thanks for your contribution. We discussed and think the production build should be the final asset that you run the Shopify Theme Check against since when running in the dev/local context, the tags are specific to the development process and are not meant to be run against any validations. We will close this PR at this time. Thanks again!


Expansion teleporter radius does not work

Title. The teleporter radius never expands past vanilla.

Do you have ZetTweaks installed? Expansion is currently bugged when ZetTweaks' tele radius changes are enabled, but I plan to fix this sometime.

That must be it. I definitely did have ZetTweaks installed.

Fixed it.


feat(aot):export default routes array to work with AoT compiler

The AoT compiler needs the variables used in the @NgModule declaration to be exported as well.
This doesn't fail when the array is empty (come compiler optimization magic maybe?), but as soon as the user enters their first route into the array, their first experience with AoT becomes an error. So, this improves the default to be compatible with AoT compiler from the beginning.

I signed it!

Can you please create an e2e test to verify?

Hmm. Can you help me with this one? I'm not sure what the e2e test would do, given:

The CLI does not add routes to the route array by itself. Adding routes is disabled at the moment, and I imagine might be unneeded even (generating modules with --routing is good enough for CLI tooling)
The error only happens when the user manually adds a route object to the routes array. The empty array does not cause an AoT compiler error

I had a thought that maybe I can test that routes is being exported, by maybe require()ing the file and checking for the routes property.
But then I looked at the e2e tests (in tests/e2e/tests/generate), and it seemed that all tests only ensure files are created, no checks for the shapes of the files.
Am I missing something, or would this be good to go?
Thanks a lot.

Heya @Meligy, could you give me some more context on the issue that the PR aims to resolve? That sounds like a AoT compiler limitation, but one that I was not aware of. Is there an issue on angular/angular maybe?
Regarding tests, our test infrastructure just restores everything after each test, so you should be fine manually creating some files and replacing the import statement. An example of something of this sort can be found in https://github.com/angular/angular-cli/blob/master/tests/e2e/tests/build/styles/scss.ts

When you don't export the routes, and you add an actual module route (lazy or not), trying to compile with --aot fails with something
Error encountered resolving symbol values statically. Reference to a local (non-exported) symbol 'routes'. Consider exporting the symbol (position 5:7 in the original .ts file), resolving symbol AppRoutingModule in /Users/Meligy/Code/github/Meligy/routing-angular-cli/src/app/app-routing.module.ts

There`re several related issues in the @angular/angular repository. I can't seem to find the most accurate issue unfortunately.
I have a simple repro at @Meligy/routing-angular-cli.
FYI, this only solves build issues (output when you run the command). AoT routing (components, bundling other modules, and lazy loaded modules) is broken in runtime still (browser console), which I'm working on improving my repro to show all cases, and then will see if I can add to existing issue or create new one for.

The problem this PR fixes seems to be gone in Angular 2.1.0, so, I'm closing this PR.
The problem explained in #2735 though is still there.
The difference if not clear is:

The problem this PR was fixing was happening in the terminal console when running ng build --aot or ng serve --aot
Keyword here is "terminal"

The problem #2735 explains is a problem that still happens in the browser when running ng serve --aot
Keyword here is "browser console"


NameError: name 'CriterionType' is not defined

This is the error I get:
NameError                                 Traceback (most recent call last)
/tmp/ipykernel_18106/1529090364.py in <module>
     31
     32 upstream = 'wav2vec2_hug_base_960'
---> 33 runner = Runner(args, config)

~/Desktop/ASR/s3prl/s3prl/downstream/runner.py in __init__(self, args, config)
     50         self.upstream = self._get_upstream()
     51         self.featurizer = self._get_featurizer()
---> 52         self.downstream = self._get_downstream()
     53         self.all_entries = [self.upstream, self.featurizer, self.downstream]
     54

~/Desktop/ASR/s3prl/s3prl/downstream/runner.py in _get_downstream(self)
    131             upstream_rate = self.featurizer.model.downsample_rate,
    132             **self.config,
--> 133             **vars(self.args)
    134         ).to(self.args.device)
    135

~/Desktop/ASR/s3prl/s3prl/downstream/

I reviewed this issue but I did not help me https://github.com/flashlight/flashlight/issues/416#issue-783305697

when I run this import flashlight  --> does not raises any error
when I run this from flashlight.lib.text.decoder import CriterionType   --> it raises this: ModuleNotFoundError: No module named 'flashlight.lib.text'
I can successfully run examples in the flashlight without any error. so this command python flashlight/bindings/python/example/criterion_example.py does NOT raise any error
And I have kenlm in this address /usr/local/share/kenlm

@jacobkahn I tried to address the questions @tlikhomanenko had asked in that issue in order to make it easier.
I really really appreciate it if you could give me some hints where should I check.

I really appreciate it if you can have a quick look at this issue, I've included all the information and steps I took and was mentioned in other issue pages to fix it but no luck :(. @jacobkahn @tlikhomanenko
Thanks again for your time

I don't know if it can help but this are the warnings I get before raising error:
/home/ai-labs/.conda/envs/asr/lib/python3.7/site-packages/torchaudio/_internal/module_utils.py:87: UserWarning: Failed to import soundfile. 'soundfile' backend is not available.
  warnings.warn("Failed to import soundfile. 'soundfile' backend is not available.")
################################################################################
### WARNING, path does not exist: KALDI_ROOT=/mnt/matylda5/iveselyk/Tools/kaldi-trunk
###          (please add 'export KALDI_ROOT=<your_path>' in your $HOME/.profile)
###          (or run as: KALDI_ROOT=<your_path> python <your_script>.py)
################################################################################

[hubconf] can not import .upstream.pase.hubconf: No module named 'pase'... Pass.
[Override] - config.downstream_expert.datarc.decoder_args.decoder_type = kenlm
[Override] - config.downstream_expert.datarc.decoder_args.kenlm_model = ../../backup/4-gram.arpa.gz
[Override] - config.downstream_expert.datarc.decoder_args.lexicon = ../../backup/librispeech_lexicon.lst
/home/ai-labs-sgoudarzvand/.conda/envs/asr/lib/python3.7/site-packages/transformers/configuration_utils.py:365: UserWarning: Passing `gradient_checkpointing` to a config initialization is deprecated and will be removed in v5 Transformers. Using `model.gradient_checkpointing_enable()` instead, or if you are using the `Trainer` API, pass `gradient_checkpointing=True` in your `TrainingArguments`.
  "Passing `gradient_checkpointing` to a config initialization is deprecated and will be removed in v5 "
Some weights of the model checkpoint at facebook/wav2vec2-base were not used when initializing Wav2Vec2Model: ['quantizer.weight_proj.bias', 'project_q.bias', 'project_q.weight', 'project_hid.bias', 'quantizer.weight_proj.weight', 'quantizer.codevectors', 'project_hid.weight']
- This IS expected if you are initializing Wav2Vec2Model from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
- This IS NOT expected if you are initializing Wav2Vec2Model from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
[Featurizer for <class 's3prl.upstream.wav2vec2_hug.expert.UpstreamExpert'>] - The input upstream is only for initialization and not saved in this nn.Module
[Featurizer for <class 's3prl.upstream.wav2vec2_hug.expert.UpstreamExpert'>] - Take a list of 13 features and weighted sum them.
[Runner] - Loading Featurizer weights from the previous experiment
/home/ai-labs/Desktop/ASR/s3prl/s3prl/downstream/asr/w2l_decoder.py:42: UserWarning: flashlight python bindings are required to use this functionality. Please install from https://github.com/facebookresearch/flashlight/tree/master/bindings/python
  "flashlight python bindings are required to use this functionality. Please install from [https://github.com/facebookresearch/flashlight/tree/master/bindings/python"](https://github.com/facebookresearch/flashlight/tree/master/bindings/python%22)

Any help on this please @jacobkahn @tlikhomanenko

@benam2 opened https://github.com/s3prl/s3prl/issues/407 -- I'll play around with this for a minute to see if there's an easy fix, but the bindings haven't changed.

I really appreciate it. I have already installed flashlight text as you suggested but did not fix this error. Does that mean for some reason it has not been installed correctly? (didn't raise any error though)
This is the last part of the log in terminal
copying build/lib.linux-x86_64-3.7/flashlight/lib/text/flashlight_lib_text_dictionary.cpython-37m-x86_64-linux-gnu.so -> build/bdist.linux-x86_64/egg/flashlight/lib/text
byte-compiling build/bdist.linux-x86_64/egg/flashlight/lib/__init__.py to __init__.cpython-37.pyc
byte-compiling build/bdist.linux-x86_64/egg/flashlight/lib/text/dictionary.py to dictionary.cpython-37.pyc
byte-compiling build/bdist.linux-x86_64/egg/flashlight/lib/text/decoder.py to decoder.cpython-37.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying flashlight_text.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying flashlight_text.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying flashlight_text.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying flashlight_text.egg-info/not-zip-safe -> build/bdist.linux-x86_64/egg/EGG-INFO
copying flashlight_text.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
writing build/bdist.linux-x86_64/egg/EGG-INFO/native_libs.txt
creating dist
creating 'dist/flashlight_text-0.1-py3.7-linux-x86_64.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing flashlight_text-0.1-py3.7-linux-x86_64.egg
removing '/home/ai-labs-sgoudarzvand/.conda/envs/asr/lib/python3.7/site-packages/flashlight_text-0.1-py3.7-linux-x86_64.egg' (and everything under it)
creating /home/ai-labs-sgoudarzvand/.conda/envs/asr/lib/python3.7/site-packages/flashlight_text-0.1-py3.7-linux-x86_64.egg
Extracting flashlight_text-0.1-py3.7-linux-x86_64.egg to /home/ai-labs-sgoudarzvand/.conda/envs/asr/lib/python3.7/site-packages
flashlight-text 0.1 is already the active version in easy-install.pth

Installed /home/ai-labs-sgoudarzvand/.conda/envs/asr/lib/python3.7/site-packages/flashlight_text-0.1-py3.7-linux-x86_64.egg
Processing dependencies for flashlight-text==0.1
Finished processing dependencies for flashlight-text==0.1

@jacobkahn

I did not uninstall the flashlight, could it be the reason? @jacobkahn

@benam2 — did you install Flashlight Text -- https://github.com/flashlight/text?
from flashlight.lib.text.decoder import CriterionType

works for me on a fresh install of Flashlight Text

@jacobkahn Yes I did as you had suggested in another issue. I followed this:
git clone https://github.com/flashlight/text && cd text
cd bindings/python
python3 setup.py install

and this is what I see in condo env: flashlight-text           0.1                      pypi_0    pypi

@benam2
And
from flashlight.lib.text.decoder import CriterionType

still fails? Can you check that you're using the correct Python interpreter in all cases?

Im trying to run https://github.com/s3prl/s3prl/blob/main/s3prl/downstream/asr/w2l_decoder.py script and it raises error while reaches the line to import from flashlight.lib.text.decoder import CriterionType.
Yea I have activated the env for the asr and then try to get that script running. I will create a new env again and try from scratch. Thanks for taking the time. I will share If I was successful.

@jacobkahn Just before starting everything from scratch, could you please comment on this error log that anything else looks suspicious to you. I really appreciate it if you can have a quick look:
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
Input In [3], in <cell line: 33>()
     30 hack_isinstance()
     32 upstream = 'wav2vec2_hug_base_960'
---> 33 runner = Runner(args, config)

File ~/Desktop/ASR/s3prl/s3prl/downstream/runner.py:52, in Runner.__init__(self, args, config)
     50 self.upstream = self._get_upstream()
     51 self.featurizer = self._get_featurizer()
---> 52 self.downstream = self._get_downstream()
     53 self.all_entries = [self.upstream, self.featurizer, self.downstream]

File ~/Desktop/ASR/s3prl/s3prl/downstream/runner.py:129, in Runner._get_downstream(self)
    127 def _get_downstream(self):
    128     Downstream = getattr(downstream, self.args.downstream)
--> 129     model = Downstream(
    130         upstream_dim = self.featurizer.model.output_dim,
    131         upstream_rate = self.featurizer.model.downsample_rate,
    132         **self.config,
    133         **vars(self.args)
    134     ).to(self.args.device)
    136     return self._init_model(
    137         model = model,
    138         name = 'Downstream',
    139         trainable = True,
    140         interfaces = ['get_dataloader', 'log_records']
    141     )

File ~/Desktop/ASR/s3prl/s3prl/downstream/asr/expert.py:99, in DownstreamExpert.__init__(self, upstream_dim, upstream_rate, downstream_expert, expdir, **kwargs)
     94 self.objective = nn.CTCLoss(
     95     blank=self.blank,
     96     zero_infinity=self.datarc['zero_infinity']
     97 )
     98 decoder_args = self.datarc.get('decoder_args')
---> 99 self.decoder = get_decoder(decoder_args, self.dictionary)
    100 self.register_buffer('best_score', torch.ones(1) * 100)

File ~/Desktop/ASR/s3prl/s3prl/downstream/asr/expert.py:36, in get_decoder(decoder_args_dict, dictionary)
     34     if isinstance(decoder_args.unk_weight, str):
     35         decoder_args.unk_weight = eval(decoder_args.unk_weight)
---> 36     return W2lKenLMDecoder(decoder_args, dictionary)
     38 return None

File ~/Desktop/ASR/s3prl/s3prl/downstream/asr/w2l_decoder.py:126, in W2lKenLMDecoder.__init__(self, args, tgt_dict)
    125 def __init__(self, args, tgt_dict):
--> 126     super().__init__(args, tgt_dict)
    128     self.unit_lm = getattr(args, "unit_lm", False)
    130     if args.lexicon:

File ~/Desktop/ASR/s3prl/s3prl/downstream/asr/w2l_decoder.py:55, in W2lDecoder.__init__(self, args, tgt_dict)
     52 self.nbest = args.nbest
     54 # criterion-specific init
---> 55 self.criterion_type = CriterionType.CTC
     56 self.blank = (
     57     tgt_dict.index("<ctc_blank>")
     58     if "<ctc_blank>" in tgt_dict.indices
     59     else tgt_dict.bos()
     60 )
     61 if "<sep>" in tgt_dict.indices:

NameError: name 'CriterionType' is not defined

@jacobkahn

@benam2 — nothing interesting happening here besides that Text isn't properly installed -- your goal should be to get
from flashlight.lib.text.decoder import CriterionType

working

By chance I just tried to just import the library and not run the code and it does not raise any error.
It seems that it has been installed correctly since from flashlight.lib.text.decoder import CriterionType does not raise any error :|
Do you still think that the error log I shared before looks ok? @jacobkahn

@benam2 if the import succeeds, it means Flashlight Text is installed correctly and that there's an issue with s3prl. You should open an issue in their repo.

Im really sorry to get back to you again. I did test a lot of things. The problem is not with s3prl script I think.
I created a new env, and then checked the code.
In the script where it imports the library it had used try except that's why it did not refer to the exact error line correctly. What I did, I just remove the try/except. it means I changed this lines:
try:
    from flashlight.lib.text.dictionary import create_word_dict, load_words
    from flashlight.lib.sequence.criterion import CpuViterbiPath, get_data_ptr_as_bytes
    from flashlight.lib.text.decoder import (
        CriterionType,
        LexiconDecoderOptions,
        KenLM,
        LM,
        LMState,
        SmearingMode,
        Trie,
        LexiconDecoder,
    )
except:
    warnings.warn(
        "flashlight python bindings are required to use this functionality. Please install from https://github.com/facebookresearch/flashlight/tree/master/bindings/python"
    )

to this line:
from flashlight.lib.text.dictionary import create_word_dict, load_words
from flashlight.lib.sequence.criterion import CpuViterbiPath, get_data_ptr_as_bytes
from flashlight.lib.text.decoder import CriterionType, LexiconDecoderOptions,KenLM, LM, LMState,SmearingMode,Trie,LexiconDecoder

Now the error is :
  File "run_downstream.py", line 206, in <module>
    main()
  File "run_downstream.py", line 201, in main
    runner = Runner(args, config)
  File "/home/ai-labs/Desktop/ASR/s3prl/s3prl/downstream/runner.py", line 52, in __init__
    self.downstream = self._get_downstream()
  File "/home/ai-lab/Desktop/ASR/s3prl/s3prl/downstream/runner.py", line 129, in _get_downstream
    model = Downstream(
  File "/home/ai-labs/Desktop/ASR/s3prl/s3prl/downstream/asr/expert.py", line 99, in __init__
    self.decoder = get_decoder(decoder_args, self.dictionary)
  File "/home/ai-labs/Desktop/ASR/s3prl/s3prl/downstream/asr/expert.py", line 32, in get_decoder
    from .w2l_decoder import W2lKenLMDecoder
  File "/home/ai-labs/Desktop/ASR/s3prl/s3prl/downstream/asr/w2l_decoder.py", line 29, in <module>
    from flashlight.lib.sequence.criterion import CpuViterbiPath, get_data_ptr_as_bytes
ModuleNotFoundError: No module named 'flashlight.lib.sequence'

And when I just do this:
from flashlight.lib.sequence.criterion import CpuViterbiPath
it raises:
ModuleNotFoundError: No module named 'flashlight.lib.sequence'
So it means even from the beginning it had problem with this with flashlight.lib.sequence (I can import `flashlight.lib without error).
Could you please comment on this that what could be incorrect?

@benam2 you still to install sequence bindings which are still in this repo.
You were having trouble with FFTW before -- you can disable installing it with this diff.
End to end flow that works for me:
git clone https://github.com/flashlight/flashlight.git
cd flashlight
wget https://gist.githubusercontent.com/jacobkahn/f345a94974838c89df6b07f77824fca5/raw/65c3ca05961a92a08fbe1e2629a9290774baf6fd/diff.out
git apply diff.out
USE_CUDA=0 USE_MKL=0 pip install -e bindings/python
python -c "import flashlight.lib.sequence" # succeeds

I'm in the process of moving flashlight.lib.sequence bindings to another repo this week, but this should unblock you for the time being. Sorry for the build challenges here.

@benam2 The package you're trying to use doesn't use ASG, which is the only thing in the sequence library that is eligible to be build with CUDA. Building with it won't affect performance since that codepath isn't being called but you can enable it if you feel like it :)

@jacobkahn
So this is the message for the flashlight Successfully installed flashlight-1.0.0
And this is the error after importing ModuleNotFoundError: No module named 'flashlight.lib.sequence'

@benam2 — well — turns out this is a bug in some envs!
Fix is in https://github.com/flashlight/flashlight/pull/1032. I'll let you know when those both land and we can retry.

@jacobkahn the error related to flashlight.lib.sequence fortunately got fixed but I think there has been a change in flashlight/text that now it raises error. Should I wait for a new commit or is there anything I need to do in my end?
(If that helps, I just tried this approach on an older env that I had installed the flashlight-text successfully and I could import successfully yesterday. But now when I activated that env and just followed the first part of the instruction to get flashlight.lib.sequence fixed, import flashlight.lib.text also raises error.
What Im trying to say is that there is a change that affected "import flashlight.lib.text".)

@benam2 — you need to pip uninstall flashlight and pip uninstall flashlight-text before installing them both again, from source, with up to date branches.

I had uninstalled flashlight-text but same thing happened. I will try to uninstall both of them and if it did not work create a new env again. I hope that fix it. I will update here. Thanks again.

Uninstalling did not help. I go to create a new env and do everything again. Just one quick thing that git apply diff.out raises error but it installs successfully. Is that ok? @jacobkahn

@jacobkahn I created a new env, and followed your instruction:

python -c "import flashlight.lib.sequence".  Runs successfully
python -c "import flashlight.lib.text". Raises Error

and like I said git apply diff.out raises error but I think that is fine since the original error is fixed.
I use python= 3.8
Did you do it in conda env?

@benam2:
Updated diff for debugging purposes: https://gist.githubusercontent.com/jacobkahn/4b26e5146b70c5cfc18c429a8bc9cc52/raw/be75c21d27e1ebe4f7e407a1b32316a7fe0637ed/diff.out
Can you nuke and reclone your Flashlight Text repo, pip uninstall flashlight-text, and install again? I've confirmed that everything works with multiple envs.

@jacobkahn Still raises error. Im gonna share what exactly Im doing so that if you confirm the steps I will spend some time to try different things.
These are my step:
1. pip uninstall flashlight
2. pip uninstall flashlight-text
3. sudo rm -r flashlight (so both flashlight and text are removed)
4. git clone https://github.com/flashlight/flashlight.git && cd flashlight
5. wget https://gist.githubusercontent.com/jacobkahn/4b26e5146b70c5cfc18c429a8bc9cc52/raw/be75c21d27e1ebe4f7e407a1b32316a7fe0637ed/diff.out
6. git apply diff.out (without error)
7. USE_CUDA=0 USE_MKL=0 pip install -e bindings/python
8. python -c "import flashlight.lib.sequence" # succeeds
9. Inside flashlight folder ---> git clone https://github.com/flashlight/text && cd text
10. cd bindings/python
11. python3 setup.py install
12. python -c "import flashlight.lib.text" # ERROR

I have 1 question as well.

I remember initially I was trying to install the flashlight, MKL .... so they are somewhere in my system. do you think that can affect what Im experiencing now? if so I may need to remove my account. (But the flashlight.lib.text was running ok yesterday that's why I think that could not be the case).

@benam2 — this is now an error with the package you're using. The error's likely that the file you're trying to load doesn't exist, but I'd open an issue with s3prl and they'll help you debug. Glad everything is working on the Flashlight side.

Actually the file exist and that was the first thing I checked.
The code in the s3prl is very straightforward in the sense that it raises error in this line self.lexicon = load_words(args.lexicon) and this load_words is one functionality in flashlight-text not something that s3prl has implemented. Could you please share your reasoning why this is not a flashlight-text error?

Alright, @jacobkahn  I will check various ways to see where could be wrong. thanks again. and I greatly appreciate your quick reply.


GitHubCommits should return plain old CSV text

Rather than a JSON serialized list.

Also, should include commit hash.


Clarification Needed on Disconnect

hi
I am exploring the option of disconnecting and connecting the IOT based on the usage.
Consider a scenario, where the device is not used for  many days,  i don't want the device to connect to IOT Core every time the device boots up.  Shall i forcibly Disconnect MQTT Client  which is  TLS/OpenSSL based.?
-> In this case, will the client get Reconnect call back or not which has been subscribed during the connect ?
-->is there any issue if i do multiple time connect and disconnect based on the usage of IOT Core?

First of all it is highly recommended that you use the v2 sdk
How do you plan on forcibly disconnecting? The sdk will keep trying to reconnect with exponential back-off until it hits the SetMaxReconnectBackoffTimeout of 128 seconds. This can be changed, and the sdk will keep trying to connect at this interval. There shouldn't be any problems with doing this multiple times.
If you haven't already take a look at the Dev Guide.


Bearer middleware fails with 404 when using Azure AD B2C

We are using Azure AD B2C for our authentication and we have been unable to get it to work, we get a 404 error, which seems like it is due to the issuer actually not being a valid url. From what I understand that is expected behavior for a B2C issuer url. Is this a limitation of the middleware component or a bug that it seems to rely on calling the issuer url?
Expected Behavior
Should be able to start the service successfully, verify a token, and authorize the user on a request.
Actual Behavior
The service fails to start because this error shows up in the dapr container logs:
failed to open http channel to app: error creating HTTP middleware middleware.http.bearer/: 404 Not Found: The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

Steps to Reproduce the Problem
Bearer component:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: bearer
spec:
  type: middleware.http.bearer
  version: v1
  metadata:
    - name: clientId
      value: {client id}
    - name: issuerURL
      value: https://{tenant name}.b2clogin.com/{tenant name}.onmicrosoft.com/v2.0/.well-known/openid-configuration?p={policy name}

I also tried this with the issuer url that is provided in the openid configuration: https://{tenant name}.b2clogin.com/{tenant id}/v2.0/ which if you call the issuer endpoint in a browser you get this in the 404 response: The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.
Using the above component and including in the middleware configuration:
apiVersion: dapr.io/v1alpha1
kind: Configuration
metadata:
  name: http-pipeline
spec:
  mtls:
    enabled: true
  appHttpPipeline:
    handlers:
    - name: bearer
      type: middleware.http.bearer

This is all that is needed with any service to run into the same issue.
Deployment yaml annotations:
template:
    metadata:
      labels:
        app: myapp
        kind: api
      annotations:
        dapr.io/enabled: "true"
        dapr.io/app-id: "myapp-api"
        dapr.io/id: "myapp-api"
        dapr.io/app-port: "6003"
        dapr.io/app-protocol: "http"
        dapr.io/app-ssl: "false"
        dapr.io/enable-api-logging: "true"
        dapr.io/api-token-secret: "dapr-api-token"
        dapr.io/app-token-secret: "app-api-token"
        dapr.io/config: "http-pipeline"

I have not had issues with other functionalities and components, including other authorization methods and OPA policies, but I cannot get this component to work with Azure B2C.
The specs:

Windows 10 22H2
docker desktop 4.17.0 (Engine 20.10.23) using WSL2
kubernetes 1.25.4
dapr version 1.10.2

Release Note

RELEASE NOTE:

@tophallen when you say that you use https://{tenant name}.b2clogin.com/{tenant name}.onmicrosoft.com/v2.0/.well-known/openid-configuration?p={policy name}... Do you mean this literally, or do you replace {tenant name} with your tenant name?

I used {x} as a placeholder in the GitHub issue, since using that would not work as it is not a valid tenant name.

I will take a look


update readme

some readability changes
some extra info about the --template-dir flag

Thank you very much for sharing this repository.

@allen-munsch
Thank you for creating great PR :smile:


Blur when zooming。And The Z of translate3d is not used

Software version (please complete the following information):

Browser [e.g. chrome, safari]
Vue Version [e.g. 2.5.7]
vue-grid-layout Version: [e.g. 2.3.3]

Describe the bug
A clear and concise description of what the bug is.
Please use the CodeSandbox Template to demonstrate your bug. It is much easier for us to help you if you do.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

var translate = "translate3d(" + left + "px," + top + "px, 0)";
translate3d blur when zooming.
how about --> var translate = "translate(" + left + "px," + top + "px)";

When zooming how? Please describe the problem properly


concurrency pattern added

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I have added the reference. Is there anything wrong ?

@kabircse check this -
https://github.com/freeCodeCamp/freeCodeCamp/blob/master/docs/how-to-work-on-guide-articles.md#when-do-we-close-pull-requests


Support for Deno 2.0 in Angular

Which @angular/* package(s) are relevant/related to the feature request?
core
Description
With the release of Deno 2.0, compatibility with Node.js and npm has been significantly enhanced, allowing frameworks like Angular to operate within the Deno environment. Integrating Angular with Deno 2.0 would provide developers with a modern, secure, and efficient runtime for building Angular applications.
Proposed solution

Update the Angular CLI to support Deno 2.0 for project scaffolding, building, and serving applications.
Ensure that @angular/core and related packages are compatible with Deno's module system and runtime APIs.
Provide documentation and examples demonstrating how to develop, build, and deploy Angular applications using Deno 2.0.

Implementing these changes would expand Angular's versatility and offer developers more choices in their development environments.
Alternatives considered
Before proposing official support for Deno 2.0 in Angular, I explored the following alternatives:

Manual Configuration: Attempted to manually configure Angular projects to run on Deno by adjusting build scripts and dependencies. However, this approach proved complex and error-prone, leading to maintenance challenges.
Community Plugins: Searched for community-developed plugins or tools that facilitate Angular's integration with Deno. While some experimental solutions exist, they lack comprehensive support and are not actively maintained.
Alternative Frameworks: Considered using other frameworks that natively support Deno. However, this would require significant refactoring of existing Angular applications and retraining of development teams.

These alternatives highlighted the need for official support to ensure a seamless and reliable integration of Angular with Deno 2.0.

Since Deno 2.0 is intended to be fully compatible with both Node.js and npm, Angular should work without any special instructions.  Angular can currently be used with a variety of Node package managers such as npm, pnpm, yarn, or any other by skipping the install on project creation.  Just as with these package managers, it should also be able to be used with Deno or Bun assuming that these runtimes provide compatibility as documented.  If differences arise where Angular works within Node.js but not Deno or Bun then an issue should ideally be created with the respective runtime so that the runtimes can improve their compatibility with Node.js.

I'll also point out that Angular SSR is already compatible with Deno and other non-Node JS runtimes. We're working to make that experience even better with each release. The CLI doesn't explicitly support Deno, but like @clydin mentioned, any issues there are compatibility issues with those runners moreso than bugs in Angular CLI.


Add ability to parameterize reclaim policy for ES data nodes

For demo purposes it would be nice to be able to switch the reclaim policy to Delete currently its hardcoding to Retain:
https://github.com/upmc-enterprises/elasticsearch-operator/blob/6b47b2edb60c6682eef4f9488bc9d8bdfb8be66d/pkg/processor/processor.go#L212

How do you utilize this fix?

There's a new volume-reclaim-policy on the storage option in the CRD. You can add that and set the policy to Retain or Delete.


About Views

Hi. Thanks for this tool, really! This is exactly what I was looking for.
i was try it. But this tool not generated views.
Is there a problem or is this tool not like the picture?

Hello @salihonder05
I hope you're doing well and enjoying CRUD Generator.
Generating views was not a part of my plan for this tool, But any PR would be appreciated.


add separate config

I guess that is faster than you expected 😆  @Flova

We could still add an launch argument for the camera device.


Prevent infinite resize when vertical scrollbar appears

A workaround to the infinte resize problem.
Pens to demonstrate:
master
master - 3 columns
THIS PR:(https://github.com/chartjs/Chart.js/pull/6011/commits/840d11a8d32260849f9ff3ad0b4f775eda99e9c2)
single
3 rows
3 columns
collapse rows
collapse columns
You get the effect (in master) by resizing window so that scrollbar just appears.
Fixes: #2127

I'm still getting a jumpy behavior (not infinite) just before the scrollbar become visible or hidden. I think it's better than master but still unexpected and will certainly be reported as a bug.
Anyway, I'm a bit worried about this PR because we are considering that a container width change during resize is due to scrollbar visibility change. But what about animations (via JS or CSS) or similar? can you setup a pen that resize the container with a CSS animation (expand / collapse for example)?
Also, can we test this behavior with multiple charts on the same page (same row or not)?

I still see the infinite scroll even just before my Y axis scrollbar appears.  Any tips on how to avoid this?  Chart.js 2.9.4