"Get masked password for user" task takes more than 2 minutes per users after some iteration

SUMMARY
I have about 60 'amq_broker_users' users. The playbook takes more than 2 hours to run.
The issue is that the https://github.com/ansible-middleware/amq/blob/main/roles/activemq/tasks/mask_password.yml is masking the password for every user to check if it is still valid.
The 12 first takes about 3 seconds each to be checked.
From the next one it takes between 2 and 3 minutes to process each one.
ISSUE TYPE

Bug Report

ANSIBLE VERSION

ansible [core 2.11.12]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/myuser/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/myuser/.local/lib/python3.6/site-packages/ansible
  ansible collection location = /home/myuser/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/myuser/.local/bin/ansible
  python version = 3.6.8 (default, Sep  9 2024, 06:12:39) [GCC 8.5.0 20210514 (Red Hat 8.5.0-22)]
  jinja version = 3.0.3
  libyaml = True

COLLECTION VERSION

Collection                   Version
---------------------------- -------
ansible.posix                1.5.4
ansible.utils                5.1.0
middleware_automation.amq    2.2.5
middleware_automation.common 1.1.4

STEPS TO REPRODUCE

- name: Install AMQ
  hosts: all
  vars:
    amq_broker_version: 7.12.1
    amq_broker_mask_password: true
    amq_broker_users:
      - user: test-user1
        password: ENC(1024:0E58900B956BBEED271D71CA393BF7C893853EE94491F391D5F0CB45C18E1FB5:3587268B3DF2287A486301ED900C599AFD549955A4A9D08A743B5E9D90F2DFCF425B654B012DD1474DEBCBB97D9242990C7D35E7EF74C4BC8607A526BCD43D03)
        roles: [ pub-user ]
#ADD 59 more similar users here
    amq_broker_roles:
      - match: test.v1.#
        name: pub_user
        permissions: [ createAddress, createDurableQueue, createNonDurableQueue, deleteAddress, deleteDurableQueue, deleteNonDurableQueue, manage, send ]
  tasks:
#AMQ installation with Red Hat amq_broker playbook
  - name: Install AMQ
    ansible.builtin.import_role:
      name: redhat.amq_broker.amq_broker
    no_log: False

EXPECTED RESULTS

That running the activemq role takes a reasonable amount of time to run (<5 minutes).
ACTUAL RESULTS
The playbook runs for more than 2 hours.

wow; I hope it's a glitch somewhere or keeping idempotence with masked password will be trouble

I am trying to reproduce and fix in the PR above (added 60 users with password to encode and 60 with already encoded pass); can you please provide the amount of memory of the instance running artemis?

I am trying to reproduce and fix in the PR above (added 60 users with password to encode and 60 with already encoded pass); can you please provide the amount of memory of the instance running artemis?

It is running on a VM with 4Gb of memory:
$ free -h
total        used        free      shared  buff/cache   available
Mem:          3.6Gi       1.8Gi       318Mi        63Mi       1.5Gi       1.5Gi
Swap:         4.0Gi       1.2Gi       2.8Gi

This was exactly like that while running the collection. I mean 1.8Gi free.
The Artemis process has  -Xms512M -Xmx2G.
The masked password process has the same memory configuration:  -Xms512M -Xmx2G

Committed a change in the PR which improves the running time a bit in case of already encrypted passwords

thank you. Skipping the password masking when it is already encrypted is what I was looking for

Yeah that was a inattention on my side.. if it fixes for you I can merge this change only, or did you see slowness in the encryption side of the tasks too?

hi @guidograzioli. Do you know when the release 2.2.6 release will be available on the downstream amq_broker collection?

Hello; upload is happening today, it should be available in 24-48 hours


README.md - fix Youtube link

The YouTube link was broken, fix per info in
https://github.com/AVSystem/Anjay-freertos-client/issues/1

@Mierunski - please review.

Thanks!


make the Benefits page readable: add Converters TOC

To upgrade the Benefits page from an unfriendly scroll journey --
This PR adds internal links in the Converters section and a mini-TOC for them

probably worth revisiting this PR to see if what's main looks "good enough"
https://bids-website.readthedocs.io/en/latest/tools/converters/

I think we've lost something with the subpages here.  -- the ability to search the page for (e.g.) "EEG" since a lot of tools are under one category but could be under multiple.
is it easy to add a "search all converters" box/feature or is a table of contents left-side still with a long scroll in the main panel worth trying?

Closing as this is now obsolete on the new website


How to mimic previous bottombar positioning

the older version of bottombar was working great for me, but I decided to bite the bullet and make sure I was using the latest version. My problem is that I want to mimic the positioning of the previous version of bottom bar. How can I set up my layout to exactly replicate what the original version was doing - is it possible?
Right now I have a relative layout with the bottombar stuck to the bottom of parent, and then my framelayout positioned above bottombar - this is not working for me. When I open up keyboard in one of my fragments, my framelayout is pushed upwards, and content at the top of the framelayout becomes not visible.
Is there anyway to 100% get the same positioning as the previous bottom bar?

I've worked around this by using a listener for keyboard hide/show coupled with android:windowSoftInputMode="adjustResize". Ideally I'm just missing something and there is an easier way to do this, but this will hold up for now.


Icons feature

Hello,
thanks for new version. I have idea for new feature - can you add possibility to edit the font that you are using for toolbar icons? Because I need to change some of them according to users needs and I will want to do it in straightforward way, not to make workaround solutions.
Thanks,
Adrian

Hi
If you are developing in nodejs environment, you can change it as below.
As we modify the icon object, it will be applied to all editors.
import icons from '../../src/assets/defaultIcons';
icons.bold = '...'

I'll add the option to change the default icon in the next version.
Thank you.

The 2.28.0 version has been updated.
If there is an issue, please register as a new issue.
Thank you.


Sign In Method

Summary

Changed google-services.json (API Issues)
SignIn Method
Added User Model
Code Refactor


How to access kbatch pod logs?

I would like to access the logs of my (failed) jobs, but I think the docs are slightly outdated and/or I'm doing things wrong.
Attempt 1:
In the docs it's mentioned that you can get the logs by doing kbatch job logs CONTAINER_ID
"""
With kbatch job logs you can get the logs for a job. Make sure to pass the container id.
'''"
I don't think the current version has a logs argument for the job command. The command I see for job are delete, list, show and submit; see help message below:
kbatch job --help

Usage: kbatch job [OPTIONS] COMMAND [ARGS]...

  Manage kbatch jobs.

Options:
  --help  Show this message and exit.

Commands:
  delete  Delete a job, cancelling running pods.
  list    List all the jobs.
  show    Show the details for a job.
  submit  Submit a job to run on Kubernetes.

Attempt 2
The pod command has a logs argumen; see help output below:
kbatch pod logs --help

Usage: kbatch pod logs [OPTIONS] POD_NAME

  Get the logs for a kbatch pod.

Options:
  --kbatch-url TEXT       URL to the kbatch server.
  --token TEXT            File to execute.
  --stream / --no-stream  Whether to stream the logs
  --read-timeout INTEGER  Timeout for reading data
  --pretty / --no-pretty
  --help                  Show this message and exit.

Now I can list my job names with kbatch job list -o table and then use the output to pick one of the job names "add-coastline-distances-job-q6pbj".
However, when I want to get the logs like this I get a http request error:
kbatch pod logs add-coastline-distances-job-q6pbj

[18:02:21] INFO     HTTP Request: GET https://pccompute.westeurope.cloudapp.azure.com/compute/services/kbatch/jobs/logs/add-coastline-distances-job-q6pbj/ "HTTP/1.1 500 Internal Server Error"      _client.py:1026
Traceback (most recent call last):
  File "/Users/calkoen/mambaforge/envs/jl-full/bin/kbatch", line 8, in <module>
    sys.exit(cli())
             ^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/kbatch/cli.py", line 348, in logs
    result = _core.logs(pod_name, kbatch_url, token, read_timeout=read_timeout)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/kbatch/_core.py", line 196, in logs
    result = next(gen)
             ^^^^^^^^^
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/kbatch/_core.py", line 233, in _logs
    r.raise_for_status()
  File "/Users/calkoen/mambaforge/envs/jl-full/lib/python3.11/site-packages/httpx/_models.py", line 761, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Server error '500 Internal Server Error' for url 'https://pccompute.westeurope.cloudapp.azure.com/compute/services/kbatch/jobs/logs/add-coastline-distances-job-q6pbj/'
For more information check: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/500

Trying to bring this project back up. I think #79 would have helped you find your problem by showing you the actual error, but kbatch pod logs must take a pod name (i.e. found with kbatch pod list) and add-coastline-distances-job-q6pbj is the name of the job (job pods will have two random bits on the end, e.g. job-q6pbj-abc123, while job names have just one). #81 adds kbatch job logs JOB_NAME so you can get the logs by job name without having to look up the pod for the job yourself.


Use alpine and pause images from MSFT mirrors in tests

Occasionally we've started seeing issues with docker rate limiting image pulls when running cri-containerd tests locally. Switch to MSFT mirrors.
Signed-off-by: Maksim An<EMAIL_ADDRESS>

Excellent. Thanks for the quick turnaround on this.

woot! two approvals! merge coming?


gatsby set-up & deployment

i got up to here: https://github.com/gatsbyjs/gatsby#deploying-to-github-pages-and-other-hosts-where-your-sites-links-need-prefixes

to deploy i ran:
gatsby build --prefix-links
git subtree push --prefix public origin gh-pages


Fix/get nonce block

Closes #153

Codecov Report
Base: 62.67% // Head: 62.65% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (78c93d4) compared to base (7e05522).
Patch coverage: 60.00% of modified lines in pull request are covered.

:exclamation: Current head 78c93d4 differs from pull request most recent head 3be349f. Consider uploading reports for the commit 3be349f to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
- Coverage   62.67%   62.65%   -0.03%
==========================================
  Files          48       48
  Lines        1219     1221       +2
  Branches       75       75
==========================================
+ Hits          764      765       +1
- Misses        431      432       +1
  Partials       24       24

Impacted Files
Coverage Δ

...kotlin/com/swmansion/starknet/provider/Provider.kt
20.00% <0.00%> (-5.00%)
:arrow_down:

...swmansion/starknet/provider/rpc/JsonRpcProvider.kt
78.90% <ø> (ø)

...nsion/starknet/provider/gateway/GatewayProvider.kt
83.50% <50.00%> (-0.42%)
:arrow_down:

...n/kotlin/com/swmansion/starknet/account/Account.kt
100.00% <100.00%> (ø)

.../com/swmansion/starknet/account/StandardAccount.kt
100.00% <100.00%> (ø)

...m/swmansion/starknet/service/http/OkHttpService.kt
51.85% <0.00%> (+1.85%)
:arrow_up:

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

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


Pending Balance

Hi I am new mining, I think I did it correctly, I dowloaded a miner console and I added my conflux wallet but when I look my stats It said I have 0,1 pending balance. What that's means?, I have to wait or there is a minimun balance?. Because I see that I have 0 cfx on my wallet.
Thank you for your time  :)

Hi, I have the same problem. I have chosen the beepool mining pool and on beepool.com I can see that I have already mined some coins, when I search for my wallet address. But in my confluc wallet I have as well 0 cfx.
Would be great if you would post your solution here when you solve it.
thx

@LoKoNaoX @Habi82 Hi, I think you can contact the Miner company to ask whether they have already sent the CFX to your Conflux address first

Thanks for the feedback
In the meantime I have received the coins. I think I just had to wait a bit.

@LoKoNaoX @LoKoNaoX Ok, I will close this issue.
Contact us when you encounter problems in using ConfluxPortal.


linking with mingw32-w64: undefined to references

Hi, I see this (a bit unexpected and arcane) issue when linking with QUaServer (latest) on mingw32-w64 platform:
/usr/bin/x86_64-w64-mingw32-ld: QUaServer/libQUaServer_static.a(quaoptionsetvariable.cpp.obj):quaoptionsetvariable.cpp:(.rdata$.refptr._ZN16container_traitsI5QListI16QUaLocalizedTextEE7arrTypeE[.refptr._ZN16container_traitsI5QListI16QUaLocalizedTextEE7arrTypeE]+0x0): undefined reference to `container_traits<QList<QUaLocalizedText> >::arrType'
/usr/bin/x86_64-w64-mingw32-ld: QUaServer/libQUaServer_static.a(quaoptionsetvariable.cpp.obj):quaoptionsetvariable.cpp:(.rdata$.refptr._ZN16container_traitsI5QListIbEE7arrTypeE[.refptr._ZN16container_traitsI5QListIbEE7arrTypeE]+0x0): undefined reference to `container_traits<QList<bool> >::arrType'

I cannot produce a MWE (the code is quite big), but maybe it rings the bell for you immediately as to what the cause could be? If not, leave it, I will figure it out at some point, hopefully.

I only use GCC on Linux, and indeed it is much picky with templates.
What Qt version and g++ -v do you have?
It probably has a problem with this valueInternal overload.
If you are not using the OptionSet type (I think last time I couldn't even make it work), you can comment it out in quaserver.pri.
If you find a more elegant solution, let me know or hit me with a PR.

Maybe try to add const QList and const QVector specializations in quabasevariable.h?

Oh wait, yes, I did have the same in Linux, so maybe just add a preprocessor condition for GCC on Windows here

I use gcc 9.3 on Linux, the mingw32-w64 cross-compiler is more recent:
x86_64-w64-mingw32-gcc (GCC) 10-win32 20210408

This is not really a compiler issue (the code compiles, and gcc is quite conforming to the standard) but a linker issue. As I read the message (and I have probably less understanding of template-fu than you), it is arrType which is not known; that is a static const inside a templated class. I am not sure how are those allocated — normally static members need explicit instantiation so that they have a defined storage, right?. Perhaps constexpr could be used instead of static const for arrType?

You can check if constexpr works.
The container_traits struct is just there to try and deduce if the user passes a QList or QVector and the inner type. VS2013 and forward has no sweat on these types of expressions.
The ideal solution would be to use template-template parameters, but that seems supported only starting C++17 (and I need C++11 compat for my projects).
PS: I also don't like the template syntax at all. But I love more a simple user-facing API than I hate the template syntax.

You can check if constexpr works.

I just tried and it works :) static constexpr. Does it break your compatibility requirements? If not, I will do a PR.


Fix old version code example in README.md

Replace old, not working spz.inventory.data_tree.amda with spz.inventories.data_tree.amda

@jgieseler thank you! I totally missed this one.


[BUG] (Performance) SkSwapChaninPanel makes unnecessary per-frame heap allocations

SkSwapChainPanel::OnRenderFrame(Rect) makes 2 heap allocations. It would be preferable to not have any allocations in the hot path.

SkAutoCanvasRestore(...), The RAII semantics are neat, but maybe this could be changed to one of the new disposable stack-scoped structs?
Can SKPaintGLSurfaceEventArgs be reused rather than allocated each frame?

https://github.com/mono/SkiaSharp/blob/354d85d129e898071e002a3157e7fcba1d7830a7/source/SkiaSharp.Views/SkiaSharp.Views.UWP/SKSwapChainPanel.cs#L85

We can most definitely have a look at the disposable structs - or actually not use it at all. All it does is a Restore
We can't reuse the args as this is a breaking change. For some reason, the developer may store a reference to the object and changing it may be unexpected. You shouldn't, but someone might...

Fair enough.
Is there any option get a no-garbage path? (Maybe it already exists somewhere else?)
Certainly not the most elegant, but maybe add a second PaintSuface type event to SKSwapChainPanel with a new signature. At the same time make it possible to get a reference to the SKCanvas without using SKPaintGLSurfaceEventArgs.Surface.Canvas{get;} (which, rather surprisingly, makes a managed heap allocation).
I'd be happy to submit a PR if it fits the project.

I see the two allocations per frame... We can totally replace the SKAutoCanvasRestore with a canvas.RestoreToCount(0); after the draw. We just need to do this to "reset" the canvas each frame.
We could potentially add a property to the view to actually opt-into a zero allocation loop. If you are really looking for this perf, then set the prop and the args will be re-used....
I am thinking that SKPaintGLSurfaceEventArgs.Surface.Canvas allocation is now fixed on the later previews. I now cache everything since they stay valid. If you are not able to upgrade, then I think you might be fine to just cache the value of that property and make sure to only use it in the draw frame.

Awesome. Thanks!


Support for Ecobee4

Is Anybody Supporting this? Can We Add Ecobee4? Thanx

Hello @fratinize !
Nope. Nobody actively supports it. The repos is my hobby open source project. I don't plan to work on adding support for Ecobee 4.
I'll be happy to merge your PR, though. From what I know you shouldn't need to change many things. Ecobee 4 only added support for voice commands and the API is identical.

Hello @fratinize !
Nope. Nobody actively supports it. The repos is my hobby open source project. I don't plan to work on adding support for Ecobee 4.
I'll be happy to merge your PR, though. From what I know you shouldn't need to change many things. Ecobee 4 only added support for voice commands and the API is identical.

Ty for the quick response
I did read through the enhancement issue from virtualboy
Found the line 231 replaced the athenaSmart with the apolloSmart
But now hb won’t start.
Forgive my noobness first time digging this deep
in my logs I see several lines mentioned? Am I to understand I need to check those lines and correct them?
I am not really sure how to go about a PR and how it’s different then changing existing files?🤔☹
Thanx for any direction
Sent from my iPhone

On Jan 17, 2021, at 10:01 PM, Vojta Molda<EMAIL_ADDRESS>wrote:

Hello @fratinize !
Nope. Nobody actively supports it. The repos is my hobby open source project. I don't plan to work on adding support for Ecobee 4.
I'll be happy to merge your PR, though. From what I know you shouldn't need to change many things. Ecobee 4 only added support for voice commands and the API is identical.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Ty for the quick response
I did read through the enhancement issue from virtualboy
Found the line 231 replaced the athenaSmart with the apolloSmart
But now hb won’t start.
Forgive my noobness first time digging this deep
in my logs I see several lines mentioned? Am I to understand I need to check those lines and correct them?
I am not really sure how to go about a PR and how it’s different then changing existing files?🤔☹
Thanx for any direction
Sent from my iPhone

On Jan 17, 2021, at 10:01 PM, Vojta Molda<EMAIL_ADDRESS>wrote:

Hello @fratinize !
Nope. Nobody actively supports it. The repos is my hobby open source project. I don't plan to work on adding support for Ecobee 4.
I'll be happy to merge your PR, though. From what I know you shouldn't need to change many things. Ecobee 4 only added support for voice commands and the API is identical.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

No worries about noobness :) Everyone was a noob before. Without realizing it I actually meant that the changes would be easy for a developer. I'm not sure what software development experience you have and what's your background. There may be some learning involved in order to get the support for Ecobee 4 implemented.
I'm a quite curious about why do you need this plugin. It was developed in early days after the release of Ecobee 3. It initially didn't expose the wireless temperature/occupancy sensors to HomeKit so this plugin fixed that. About a year later they released a firmware update and now the individual sensors are accessible in the Home app by default for everyone.
Anyway, if you still want to hack around, I'd start with some general node.js tutorials. It's a simple way to learn how to debug an app and see what it is doing. YouTube works pretty well and you can findy plenty of tutorial videos. PRs are a way to propose changes to repositories on GitHub and again I'd start by watching a video and trying it out.

No worries about noobness :) Everyone was a noob before. Without realizing it I actually meant that the changes would be easy for a developer. I'm not sure what software development experience you have and what's your background. There may be some learning involved in order to get the support for Ecobee 4 implemented.
I'm a quite curious about why do you need this plugin. It was developed in early days after the release of Ecobee 3. It initially didn't expose the wireless temperature/occupancy sensors to HomeKit so this plugin fixed that. About a year later they released a firmware update and now the individual sensors are accessible in the Home app by default for everyone.
Anyway, if you still want to hack around, I'd start with some general node.js tutorials. It's a simple way to learn how to debug an app and see what it is doing. YouTube works pretty well and you can findy plenty of tutorial videos. PRs are a way to propose changes to repositories on GitHub and again I'd start by watching a video and trying it out.

Honestly Not Sure why I went down this road. I had Just added 2 New Sensors to my Ecobee and Did not see them Right away in Home Kit. Didn’t Really Think the Whole Thing out When I raised the Flag. Basically failed to do proper research :(
I was abe to Add the 4 using the addition of the apolloSmart to the Platform.js That was Educational. :)
Sorry about that
Thank You for the Tips Though.
Regards
Frank

On Jan 18, 2021, at 9:27 PM, Vojta Molda<EMAIL_ADDRESS>wrote:
No worries about noobness :) Everyone was a noob before. Without realizing it I actually meant that the changes would be easy for a developer. I'm not sure what software development experience you have and what's your background. There may be some learning involved in order to get the support for Ecobee 4 implemented.
I'm a quite curious about why do you need this plugin. It was developed in early days after the release of Ecobee 3. It initially didn't expose the wireless temperature/occupancy sensors to HomeKit so this plugin fixed that. About a year later they released a firmware update and now the individual sensors are accessible in the Home app by default for everyone.
Anyway, if you still want to hack around, I'd start with some general node.js tutorials. It's a simple way to learn how to debug an app and see what it is doing. YouTube works pretty well and you can findy plenty of tutorial videos. PRs are a way to propose changes to repositories on GitHub and again I'd start by watching a video and trying it out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/vojtamolda/homebridge-ecobee3-sensors/issues/26#issuecomment-762560413, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHCD4E5CRUKPCNQ2WKQL3RTS2TU2TANCNFSM4WGSHPXA.

Honestly Not Sure why I went down this road. I had Just added 2 New Sensors to my Ecobee and Did not see them Right away in Home Kit. Didn’t Really Think the Whole Thing out When I raised the Flag. Basically failed to do proper research :(
I was abe to Add the 4 using the addition of the apolloSmart to the Platform.js That was Educational. :)
Sorry about that
Thank You for the Tips Though.
Regards
Frank

On Jan 18, 2021, at 9:27 PM, Vojta Molda<EMAIL_ADDRESS>wrote:
No worries about noobness :) Everyone was a noob before. Without realizing it I actually meant that the changes would be easy for a developer. I'm not sure what software development experience you have and what's your background. There may be some learning involved in order to get the support for Ecobee 4 implemented.
I'm a quite curious about why do you need this plugin. It was developed in early days after the release of Ecobee 3. It initially didn't expose the wireless temperature/occupancy sensors to HomeKit so this plugin fixed that. About a year later they released a firmware update and now the individual sensors are accessible in the Home app by default for everyone.
Anyway, if you still want to hack around, I'd start with some general node.js tutorials. It's a simple way to learn how to debug an app and see what it is doing. YouTube works pretty well and you can findy plenty of tutorial videos. PRs are a way to propose changes to repositories on GitHub and again I'd start by watching a video and trying it out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/vojtamolda/homebridge-ecobee3-sensors/issues/26#issuecomment-762560413, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHCD4E5CRUKPCNQ2WKQL3RTS2TU2TANCNFSM4WGSHPXA.

No worries. I'm glad it all worked out!

No worries. I'm glad it all worked out!


Introducing InternalTektonResultType as a ResultType

Changes

This PR introduces a new TektonResultType namely 'InternalTektonResultType'.
Currently TaskRunResultType, PipelineResourceResultType and UnknownResultType exist as a ResultType for a PipelineResourceResult.
All results that conform to one of these ResultTypes are normally exposed as either as a task or resource result.
In light of #3087 the need for a ResultType that is not exposed arises.
In #3087, a Step can emit a result indicating a Step timeout has occurred.
This is a result that should not be exposed hence the need for a new ResultType named InternalTektonResultType.
This PR ensures results of this type are filtered out.
Introducing an InternalTektonResultType ensures a future proof solution to internal results that should not be exposed.
Aside from the example in #3087, a present candidate is the result written out by a Step containing a "StartedAt" key.
Up until now this result is filtered out by a specific function.
Marking it as an InternalTektonResultType now allows for this result to automatically be filtered out.
Up until this PR Sidecars can write results.
Since this is not explicitly documented it's unclear whether this is intended behavior.
In this PR it's ensured only Steps can write results.
Additionally this commit brings about refactoring (and sometimes renaming) of functions related to converting Pod statuses to TaskRun statuses from pkg/reconciler/taskrun/taskrun.go to pkg/pod/status/status.go.
This is accompanied with moving unit test cases from taskrun_test.go to status_test.go.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

/kind feature
/kind clean-up

/ok-to-test

While migrating unit tests from pkg/reconciler/taskrun/taskrun_test.go to pkg/pod/status/status.go I encountered one test case I don't fully understand.
I believe this test case aims to check whether a taskrun status is set to false if the corresponding task fails.
Nevertheless, this test case does not actually execute a function and the function it presumably would call is not responsible for setting the taskrun status. The described podStatus is also not set to fail.
In this PR I changed the test such that if a task (pod) fails it's checked whether the taskrun status is set accordingly.
Another thing I noticed in the same test is that the description has a termination message with a result that does not follow the struct format of a PipelineResourceResult. For example, the key name does not exist as PipelineResourceResult field.
On master and also in this PR, such "results" get dropped from the taskrun termination message.
Once the termination message is unmarshalled into PipelineResourceResults, JSON objects that don't adhere to the PipelineResourceResult structure are dropped without error by the unmarshal function.
I am wondering, is this behavior intentional? On master, no test case catches this behavior.
This test now records the behavior of dropping incorrect results from the taskrun termination message.
Additionally, ParseMessage now logs incorrect results.
Alternatively,  we could decide that incorrect results are kept "as is" for the taskrun termination message.
Note that it's normally impossible for incorrect results to show up in the taskrun termination message if the Tekton code only writes PipelineResourceResults to its termination message.
Lastly, ResourceRef is now a pointer. This ensures that an empty resourceRef field is not marshalled  back into the termination message (zero values like a nil pointer are not marshalled) as is the case on master currently.

I left some small nits but no blockers from my perspective!
/approve

/remove-label cleanup
/remove-label feature

/kind feature


add renovate user test

Renovate is a large TypeScript project with over 12k GitHub stars. Not enough to make it into the top 100 TypeScript repositories on GitHub, although not far off! We frequently see issues with TypeScript updates. Most recently we hit microsoft/TypeScript#50479 with the upgrade to TypeScript 4.8. Here's a search for TypeScript upgrade PRs sorted by the number of comments.
Given the size and complexity of the Renovate codebase, as well as the frequency of issues we experience when upgrading TypeScript, I believe it is appropriate to include it in the test suite.
This PR is a 2nd attempt of #97. This time I managed to test locally and confirm that the test is working.

I believe the runner is now smarter about classifying failures, so if this breaks it shouldn't cause confusion in PRs. This looks good to me, so I'll get it in and we'll see what happens.


support for legacy***

We should ask users to should we remove the package or not. and we should add option to skip the prompt


Document what --memory-swap=0 means

The 1.13.0 docs do not describe the effect of using --memory-swap=0. The default is -1 which means unlimited. Does 0 mean no swap will be allowed, or do I have to set --memory and --memory-swap to the same value to accomplish that?
https://docs.docker.com/engine/admin/resource_constraints/#/memory-swap-details

Can you report this in the documentation issue tracker, because that's where this document is maintained; you can use the "request docs changes" link on that page
I'll close the issue here, but feel free to continue the conversation


Make Voucher Icon more "clear"

It should at least have a "tooltip" or become  a Button, making it clear to the user the intent of the button.

Hi @fmoura and @guidanoli I changed it to use a button with a different icon and tooltip.
I used that blue color (same as in the submit bounty button) because it enables natural understanding of what is actionable/clickable (blue color) and what is not (other colors).
Is it good enough?

Regarding the tooltip, I would prefer either:

List vouchers (with lowercase "v")
Vouchers

Regarding the tooltip, I would prefer either:

List vouchers (with lowercase "v")
Vouchers

Thanks for your feedback,Gui!
I will adjust the tooltip. 😉

Regarding the tooltip, I would prefer either:

List vouchers (with lowercase "v")
Vouchers

Done!


Can you provide the setting that reaches to 85.x% accuracy?

@dengdan
I tried to reproduce the result that you've reached, which is 85%. However, I failed to make that result. It goes as much as 83% in my experiments. Can you provide the setting(e.g hyper parameters that you've used for pretraining and finetuning) using both Synth Text and ICDAR datasets?
Thank you.

@GodOfSmallThings @kulkarnivishal
please help me to evaluate my result
how can you calculate your result?

how can you calculate your result?

@GodOfSmallThings @kulkarnivishal @mn7142
How did you test the results of the model?你们是阿如何检验结果的
Could you share the script?可以分享一下脚本吗
I don't konw how to do!

@XiaoYigwr
Go to http://rrc.cvc.uab.es/?ch=4&com=mymethods&task=1 after login.
There is standalone evaluation script and it is easy to use.

@XiaoYigwr Go to http://rrc.cvc.uab.es/?ch=4&com=mymethods&task=1 after login. There is standalone evaluation script and it is easy to use.

standalone evaluation script is not available.  do you have other source?


Roll Flutter Engine from ecd3c4857899 to 0cbdf464c084 (1 revision)

https://github.com/flutter/engine/compare/ecd3c4857899...0cbdf464c084
2022-11-07<EMAIL_ADDRESS>Add Matrix::LookAt (flutter/engine#37361)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[Fixit] Superfund import scripts

This PR updates the import scripts used for the superfund import. The following are the notable enhancements:

DCID change for the zinc element from inChiKey/* to ZincElement
Fixes to the command line args and help messages
Minor improvements on the documentation

@beets , Thanks for the review. I have already submitted the changes to the mcf in a separate CL


Cannot read properties of undefined (reading 'dragStartDisposable')

TypeError: Cannot read properties of undefined (reading 'dragStartDisposable')
at b.R in src/vs/base/browser/ui/list/listView.ts:578:9
at b.splice in src/vs/base/browser/ui/list/listView.ts:562:16
at <anonymous> in src/vs/base/browser/ui/list/splice.ts:17:35
at Array.forEach (<anonymous>)
at t.splice in src/vs/base/browser/ui/list/splice.ts:17:20
at <anonymous> in src/vs/base/browser/ui/list/listWidget.ts:1550:57
at fn in src/vs/base/common/event.ts:1576:13
at X.splice in src/vs/base/browser/ui/list/listWidget.ts:1550:22
at X.splice in src/vs/base/browser/ui/tree/abstractTree.ts:1901:9
at g.s in src/vs/base/browser/ui/tree/indexTreeModel.ts:306:14
at g.splice in src/vs/base/browser/ui/tree/indexTreeModel.ts:157:9
at D.l in src/vs/base/browser/ui/tree/objectTreeModel.ts:122:14
at D.setChildren in src/vs/base/browser/ui/tree/objectTreeModel.ts:71:8
at w.setChildren in src/vs/base/browser/ui/tree/objectTree.ts:56:14
at re.I in src/vs/base/browser/ui/tree/asyncDataTree.ts:1024:13
at re.B in src/vs/base/browser/ui/tree/asyncDataTree.ts:789:8
at async re.z in src/vs/base/browser/ui/tree/asyncDataTree.ts:549:3
at async re.setInput in src/vs/base/browser/ui/tree/asyncDataTree.ts:522:3
at async W.a in src/vs/workbench/contrib/debug/browser/variablesView.ts:91:5

Go to Errors Site

Duplicate


"Lehliu" is listed in the drop-down menu

Description:
"Lehliu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Lehliu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Lehliu, CĂLĂRAȘI (Lehliu)".


Unused Twitter Link Remove from Settings Page

[x] Removed Dead Twitter Link from Settings Page

Attempts to close #750

Nice. Thanks @0xSaksham  :pray:
Can you also remove the settings.jam_twitter from the language files?

Sure man.

Nice. Thanks @0xSaksham 🙏
Can you also remove the settings.jam_twitter from the language files?

Ready to Merge 👍

Nice. Thanks @0xSaksham 🙏
Can you also remove the settings.jam_twitter from the language files?

Ready to Merge 👍

Should it also be removed from the other language files?

I didn't know that. I'll do a full codebase search and remove it from everywhere.

Removed the Translation "jam_twitter" from files.

@theborakompanioni Should I Squash and Merge this into main branch?


Fix typo

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Nice catch! thank you for the PR @kasperaamodt. Would you like me to add the label hacktoberfest-accepted

@kevinzunigacuellar no problem. Yeah, sure! :)


BGP Route APIs for Virtual Network Gateways

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

Hi @lamchester, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (chelam). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Hey @lamchester  , thanks for the PR!
Can you ensure that you've completed the checklist at the top (and check the boxes)
Once that's done we can assign a reviewer.
thanks!

@lamchester - Any updates on the review comments?

@anudeepsharma - The PR LGTM. Can you please complete your review and merge the PR.

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-ruby/pull/670

 https://github.com/Azure/azure-sdk-for-python/pull/981


AOT error after updating to latest seed

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior

can't build the AOT version of the app
Please tell us about your environment:

Angular Seed Version: 2.1.X

Node: node --version = 7.0.0

Output from console:

[04:16:17] Starting 'build.js.prod.exp'...
[04:16:41] Finished 'build.js.prod.exp' after 25 s
[04:16:41] Starting 'build.bundles'...
[04:16:42] Finished 'build.bundles' after 1.09 s
[04:16:42] Starting 'build.bundles.app.exp'...
[04:16:56] 'build.bundles.app.exp' errored after 14 s
[04:16:56] Error on fetch for<EMAIL_ADDRESS>at<EMAIL_ADDRESS>        Loading dist\tmp\app\webapp/common/common-dialog/commondialog.component.ngfactory.js
        Loading dist\tmp\app\app.module.ngfactory.js
        Loading dist\tmp\app\main-prod.js
        Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>[04:16:56] 'build.prod.exp' errored after 2.37 min
[04:16:56] Error in plugin 'run-sequence(build.bundles.app.exp)'
Message:
    Error on fetch for<EMAIL_ADDRESS>at<EMAIL_ADDRESS>        Loading dist\tmp\app\webapp/common/common-dialog/commondialog.component.ngfactory.js
        Loading dist\tmp\app\app.module.ngfactory.js
        Loading dist\tmp\app\main-prod.js
        ENOENT: no such file or directory, open<EMAIL_ADDRESS>Details:
    originalErr: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>Stack:
Error on fetch for<EMAIL_ADDRESS>at<EMAIL_ADDRESS>        Loading dist\tmp\app\webapp/common/common-dialog/commondialog.component.ngfactory.js
        Loading dist\tmp\app\app.module.ngfactory.js
        Loading dist\tmp\app\main-prod.js
        Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

Looks like the error seen here.  Did you modify any of the task scripts or build your app from a forked version of this repo?

Hi @RoxKilly
I have cloned the repo and then compiled:
npm run build.dev --> OK
npm run build.prod --> OK
npm run build.prod.exp --> Error:
[17:05:24] Starting 'compile.ahead.prod'... Error: undefined, resolving symbol ApplicationModule in<EMAIL_ADDRESS>   at simplifyInContext (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:532:15)     at StaticReflector.simplify (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:536:20)     at StaticReflector.annotations (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:93:28)     at NgModuleResolver.resolve (/Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:14050:46)     at CompileMetadataResolver.getNgModuleMetadata (/Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:14296:49)     at /Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:13284:43     at Array.forEach (native)     at _extractModulesAndPipesOrDirectives (/Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:13283:28)     at analyzeNgModules (/Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:13039:16)     at OfflineCompiler.compileModules (/Users/Quiroz/Development/Virtualab/iSlides/fquiroz-white-2.0/node_modules/@angular/compiler/bundles/compiler.umd.js:13115:20) Compilation failed

@felipequiroz this is not the same error from the original post. Please don't conflate multiple issues on the same thread.

Did yo follow the steps here before?

Tried to reproduce the issue with the latest master but I wasn't able to. Judging from the error message you're missing SYSTEM_BUILDER_CONFIG property. Take a look at the updated config here.


Create VideoWriter for writing video file

This PR addresses Issue https://github.com/Geek-1001/Clue/issues/11.
In addition to the above, this PR also migrates CLUVideoModule to Swift.

Hello, @andrea-prearo !
I'm back and I'm ready to take on the Clue 🙂
This week will be super busy but on the next week, I'm going to updates some issues, add new features (based on feature requests) and classify some refactoring tasks.
I'm merging this PR.
And thank you for keep helping me with Swift migration, I really appreciate it!

Hi @Geek-1001,
I am glad you are back!
You are welcome. And thanks for merging this.
I have some follow up changes that will submit for review shortly.


Add support for Tenderly through automation

[ ] bug report
[x] feature request

Current Behavior
We have to push deployed contracts to Tenderly manually for monitoring. This means if it's forgotten and we lose the build artifacts we don't have contract level monitoring for that environment.
Suggested Fix
This is intended to be achieved through a module/migration within Truffle.

Introduce a flag that disables Tenderly (assume that a Tenderly deployment is desired by default). Suggestion: export TENDERLY=false
Add support for obtaining the Tenderly authentication token. Suggestion: export TENDERLY_TOKEN=foo
Add support for creating the Tenderly project: export TENDERLY_PROJECT=bar
Add support for creating the Tenderly project:

tenderly init --create-project --project $TENDERLY_PROJECT

After the contracts have been deployed issue:

tenderly login --authentication-method=token --token=$TENDERLY_TOKEN

Push the contracts to Tenderly:

tenderly push

This would fit with the work in https://github.com/omisego/devops/issues/242

@jbunce It's not clear to be how the tenderly integration steps fit in the current repository and how this is then executed during deployment. Is it that we define the steps in package.json and then you call them sequentially?
1.) install tenderly
2.) set env variables
3.) create the project
4.) truffle migrate
5.) tenderly login
6.) tenderly push

Hey @thec00n. Without actually going through the process myself while implementing this I'm not sure. If you want to take a stab at this let me know and I'll see if I can help.

From a conversation @thec00n and I had in person:

We've agreed to facilitate this for development environments
Manual process for production environments
Implementation in JS with exec/subprocess
plasma-contracts change required so we don't pass authority key material into Truffle

Feel free to add if I've missed anything :)

Closing this issue as it addressed with #493. Splitting up dev/prod deployments is tracked here https://github.com/omisego/plasma-contracts/issues/499


Azure Optimization Engine migration to FinOps Toolkit

🛠 Description
The Azure Optimization Engine (AOE) is migrating into the FinOps Toolkit (FTK). It is a like-for-like migration, as no new features have been added nor there was any significant code change other than the inclusion of the enableDefaultTelemetry parameter in the Bicep template. The AOE documentation was refactored to fit FTK's structure and can be reviewed here.
📋 Checklist
🔬 How did you test this change?

[ ] 🤏 Lint tests
[ ] 🤞 PS -WhatIf / az validate
[X] 👍 Manually deployed + verified
[ ] 💪 Unit tests
[ ] 🙌 Integration tests

🙋♀ Do any of the following that apply?

[ ] 🚨 This is a breaking change.
[ ] 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

[X] ✅ Yes (required for dev PRs)
[ ] ➡ Will cover in a future PR (feature branch PRs only)
[ ] ❎ Not needed (small/internal change)

📖 Did you update documentation?

[X] ✅ Public docs in docs (required for dev)
[X] ✅ Internal dev docs in src (required for dev)
[ ] ➡ Will cover in a future PR (feature branch PRs only)
[ ] ❎ Not needed (small/internal change)

Thank you for the thorough review, @flanakin! Still lot of work to do, but let's look ahead on how AOE can integrate with FinOps Hubs.


Can't type b/m/k for charges with plugin enabled

I don't have any keybinds setup in the config, and cannot type letters to represent thousand/million/billion.

Hi,
Yeah, that actually makes sense, it's an oversight. The current
implementation only handles numbers, I'll try to fix it this weekend.
Keybinds don't affect this, it's caused by overriding the internal event
listener. It does this regardless if keybinds are set or not.
On 24/05/2024 23:23, umer-rs wrote:

I don't have any keybinds setup in the config, and cannot type letters
to represent thousand/million/billion.
—
Reply to this email directly, view it on GitHub
https://github.com/vikke1234/charge-calc/issues/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC6XQ5BQPGUOQLTG6HWJ2HLZD6OSTAVCNFSM6AAAAABIIFSHDCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYTMMRQGMZDENA.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--

Viktor

Hi,
It should be fixed by 79fe09e422263f0edb94b77c70d5abd3cc97c4ae, I'll try
to PR it tomorrow or early next week.
On 24.5.2024 23.23, umer-rs wrote:

I don't have any keybinds setup in the config, and cannot type letters
to represent thousand/million/billion.
—
Reply to this email directly, view it on GitHub
https://github.com/vikke1234/charge-calc/issues/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC6XQ5BQPGUOQLTG6HWJ2HLZD6OSTAVCNFSM6AAAAABIIFSHDCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYTMMRQGMZDENA.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--

Viktor

Fixed now, closing.


Fixed MacOS issue

Applied Header guards.
Option to set shared/static library builds.
Set different names for the library when built in Debug mode. This allows the installation of Release and Debug libraries in the same install prefix.

Still need some tests on Windows 11.

Hey, I saw your fork and request and I wanted to reopen this since it's something I thought might be a useful change and fix
I will look at it again later too
Thanks for the request

Good news: vsgSDL works on macOS and Windows now. Actually only the static library build can be used on Windows.

Modern C++ support use of #pragma once on all platforms/compilers so you can avoid the #ifdef/;define/endif blocks around headers. This is what I've done for the VSG and other projects and it keeps the headers clean and functional.


Add CodeQL security scanning

Hi, I'm a PM on the GitHub security team. This repository is eligible to try the new GitHub Advanced Security code scanning beta.
Code scanning runs a static analysis tool called CodeQL which scans your code at build time to find any potential security issues. We've tuned the set of queries to be only the most severe, most precise issues. We'll show alerts in the security tab, and we'll show alerts for any net new vulnerabilities on pull requests as well. We've tried to make this super developer friendly, but we'd love your feedback as we work through the beta.
If you're interested in trying it out, you can merge this pull request to set up the Actions workflow. You can also get this set up yourself in any additional repositories in this organization by following these instructions

That sounds great - thank you! Does the JavaScript scanner work with TypeScript code bases?

@darthtrevino Yep! TypeScript is fully supported.


[BUG] Robocopy of OneDrive folder during OneDrive removal fails if username contains a space

Describe the bug
In "Advanced Tweaks", when choosing "Remove OneDrive", the script performs a RoboCopy of the OneDrive folder content to the root user profile directory. If the username is in two or more words, like "Firstname Lastname", the Robocopy fails, with an "ERROR : Invalid Parameter #3".
The full message from Robocopy is as follows (Anonymized for Privacy)

ROBOCOPY     ::     Robust File Copy for Windows
Started : Friday, 19 April 2024 14.31.28
Source - C:\Users\FirstName
Dest - C:\WINDOWS\system32\LastName\OneDrive\
Files :

Options : /DCOPY:DA /COPY:DAT /R:1000000 /W:30

ERROR : Invalid Parameter #3 : "C:\Users\FirstName"
   Simple Usage :: ROBOCOPY source destination /MIR

         source :: Source Directory (drive:\path or \\server\share\path).
    destination :: Destination Dir  (drive:\path or \\server\share\path).
           /MIR :: Mirror a complete directory tree.

For more usage information run ROBOCOPY /?

To Reproduce
Steps to reproduce the behavior:

Create a Windows User with a space character in the username, ex. "Chris Titus"
Log in as the newly created user.
Install or ensure OneDrive is installed for the current user
Open the WinUtil program
Select "Tweaks"
Select "Remove OneDrive" from "Advanced Tweaks" section
Click "Run Tweaks"
Examine Script Window, and look for where the copy prior to removal of OneDrive takes place

Expected behavior
What was expected was that the content of the OneDrive Folder was copied to a directory called OneDrive in the Users directory.
Screenshots

Additional context
Add any other context about the problem here.

This was fixed in a Recent PR, if you want/have the time to test out the Test Branch of WinUtil, then please go ahead 😄. If not, then it's completely understandable.


Uncaught TypeError: Cannot read property 'nonce' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 1.5.4
System: Mac OS X 10.11.3
Thrown From: atom-telegram package, v0.1.1
Stack Trace
Uncaught TypeError: Cannot read property 'nonce' of undefined
At /Users/zork/Projects/Atom/atom-telegram/node_modules/telegram.link/node_modules/telegram-mt-node/lib/auth/request-pq.js:36

TypeError: Cannot read property 'nonce' of undefined
    at mtproto.service.req_pq.callback (/Users/zork/Projects/Atom/atom-telegram/node_modules/telegram.link/node_modules/telegram-mt-node/lib/auth/request-pq.js:36:38)
    at /Users/zork/Projects/Atom/atom-telegram/node_modules/telegram.link/node_modules/telegram-mt-node/lib/net/rpc-channel.js:85:21
    at ClientRequest.onError (/Users/zork/Projects/Atom/atom-telegram/node_modules/telegram.link/node_modules/telegram-mt-node/lib/net/http-connection.js:122:13)
    at ClientRequest.g (events.js:260:16)
    at emitOne (events.js:77:13)
    at ClientRequest.emit (events.js:169:7)
    at Socket.socketOnEnd (_http_client.js:288:9)
    at emitNone (events.js:72:20)
    at Socket.emit (events.js:166:7)
    at endReadableNT (_stream_readable.js:893:12)
    at doNTCallback2 (node.js:441:9)
    at process._tickCallback (node.js:355:17)

Commands
     -2:00.9.0 project-plus:toggle-project-finder (atom-text-editor.editor.vim-mode.normal-mode.is-focused)
     -1:59.7.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:59.7.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:59 command-palette:toggle (atom-text-editor.editor.vim-mode.normal-mode.is-focused)
     -1:56.2.0 editor:delete-to-beginning-of-word (atom-text-editor.editor.mini.is-focused)
     -1:54.4.0 core:backspace (atom-text-editor.editor.mini.is-focused)
  3x -1:50.4.0 editor:delete-to-beginning-of-word (atom-text-editor.editor.mini.is-focused)
     -1:45.4.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:45.4.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:45.2.0 settings-view:open (atom-text-editor.editor.vim-mode.normal-mode.is-focused)
     -0:14.3.0 command-palette:toggle (div.package-detail.panels-item)
  2x -0:12.2.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -0:09.6.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:09.6.0 atom-telegram:start (div.package-detail.panels-item)

Config
{
  "core": {
    "disabledPackages": [
      "about",
      "archive-view",
      "autosave",
      "welcome",
      "terminal-plus",
      "expose",
      "linter-rubocop"
    ],
    "packagesWithKeymapsDisabled": [
      "tool-bar",
      "circle-ci",
      "ruby-syntax-replacer",
      "emmet",
      "merge-conflicts",
      "dash"
    ],
    "projectHome": "/Users/zork/Projects"
  }
}

Installed Packages
# User
atom-telegram, v0.1.1
docblockr, v0.7.3
editorconfig, v1.2.4
emmet, v2.4.2
esc2, v0.0.0
file-icons, v1.6.18
imdone-atom, v1.3.27
language-docker, v1.1.6
last-cursor-position, v0.9.0
linter, v1.11.3
linter-docker, v0.1.2
linter-erb, v1.0.2
minimap, v4.19.0
project-plus, v0.7.0
react, v0.14.1
ruby-syntax-replacer, v0.5.1
symbol-gen, v1.0.0
synced-sidebar, v0.4.3
tool-bar, v0.3.0
vim-mode, v0.64.0
vim-mode-visual-block, v0.2.15
vim-surround, v0.8.1

# Dev
No dev packages

Looks like this error came from within the telegram API. Since when does it occur? Does it happen everytime you start atom-telegram?

Yes that was weird issue, I just reinstall package and it's gone, I am closing it


Add Skeleton component

Figma
https://www.figma.com/file/YMfJkmUMV8xiY7fgeW6QAr/[Orbit]-Native-Components?node-id=5797%3A3217
Orbit
https://orbit.kiwi/components/progress-indicators/skeleton/

Hmm.. how will this be applied in practice? Since iOS 14, SwiftUI does this using the .redacted modifier... can't we just use that? I guess that doesn't animate by itself...

Same as with other existing components, that have some native basic version in SwiftUI. This is not hiding the actual underlaying component as .redacted does. This component will probably be a building block for any custom loading Skeleton shape and layout.

But... why? We have platform native version of this that we don't need special code for...


[#32] Asynchronous pull implemented

Asynchronous pulling of images using goroutines and waitGroups. Resolves #32

Closed with reference to #34. Purpose fulfilled in #34.

Closed with reference to #34. Purpose fulfilled in #34.


[doc] Add instructions for tests for experimental features

Description
Add instructions for tests for experimental features. The lint failed due to previous commits and I tried to fix along with the PR. (Thanks @ashwin-pc for creating an issue for lint https://github.com/opensearch-project/opensearch-dashboards-functional-test/issues/307)
Issues Resolved
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ohltyler @kavilla could you take a look?

The CI on main is running against 3.0.0 which is not stable yet. (e.g https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3199042693/jobs/5224337698)


Angelfire Apostle spellbook merge

It's been impossible to merge Angel with Angelfire Apostle since release, first reported to Owlcat nearly a month ago. Not sure fixing vanilla bugs is exactly the priority here though :)

Actually this is VERY low priority because it turns out Toy Box can fix failed spellbook merges reliably :)


Refactor Spike (created) and (expires) Implementation

We might need to look at the way the extractors are set up for (created) and (expires).
There are two issues that I see.

The IEFT spec  defines the created (and by extension expires) fields as "number of seconds since
the Epoch". i.e. parseInt(Date.now()/1000) // =><PHONE_NUMBER>
The HTTP Date Header is defined as a "forbidden header" and is formatted as a UTC date string e.g. Date: Wed, 21 Oct 2015 07:28:00 GMT

extractors.js defines the default extractors so:
  '(created)': (req) => parseInt(req.headers.Date),
  '(expires)': (req) => parseInt(req.headers.Date) + EXPIRATION_OFFSET,

While the current implementation will work, I'm not sure it conforms with the IEFT spec ... It also requires the user to add a Date header to the proto-request object before having it signed. Would it be more ergonomic to determine and set the created value within the extractor function?
I see two options:
Option 1:

Users need to pass in the reference for the (created) field with a Date header in the request passed to signRequest()

we need to handle converting from a UTC date string => milliseconds since Epoch => seconds since Epoch.
I'm also not sure without more research what "forbidden header" means in this context ... maybe we shouldn't be using this header at all?

Option 2:

Change the "extractor" for (created) to actually set the created value using parseInt(Date.now()/1000) and not require a Date header value to be passed in with the request.
This makes testing slightly more difficult since Date.now() needs to be mocked for repeatable testing.
This is the approach I took with the current release

I would prefer option 2 but we should have a discussion about the ideal solution.


Servers

Fixes #59

@Tratcher Please review.

@Tratcher Can you explain why the IApplicationBuilder is used to configure the authentication mechanism (for WebListener), rather than there being a feature interface for that purpose?

IApplicationBuilder is for setting setting up of your application. Feature interfaces are for per-request data and operations.

:watch:

Thanks for the feedback - hopefully almost there now, @Tratcher

Minor comments, then :shipit:


Ability to set the view controller tint

TextView is using default window blue tint...

Fair point.
I think it should have class methods for setting that as well:
+ (UIColor *)tintColor;
+ (void)setTintColor:(UIColor *)tintColor;

Then, on - (void)setupStyles method, could do:
self.view.tintColor = [ESFeedbackViewController tintColor];


Support multiple sampling params in LLM API

Hi,
sglang support parallelism link.
Like the example in the link, can I call the API with different sampling parameters in parallel?
for example
if I have a batched data ;
data=["hi","hello","i'm your assistant"]

I want to set temperature as 1.0 for data[0] and 0.7 for data[1] and 0.0 for data[2] then call them simultaneously.

TLDR: you will need to make calls to AsyncLLMEngine via multithreading
For reference, here's the sglang implementation of parallelism, which is essentially making multiple async calls to the endpoint.

As @ywang96 mentioned, you can use LLMEngine to achieve this right now, see example https://github.com/vllm-project/vllm/blob/main/examples/llm_engine_example.py#L34
We current do not support this in LLM offline inference wrapper. But it's a good issue to work on!

@ywang96 @simon-mo Thank you! I hope this feature to be integrated soon!

For this feature request, concretely, it would mean changing the generate function in LLM class https://github.com/vllm-project/vllm/blob/03d37f24413b13a4e42ee115f89f647c441d1fcd/vllm/entrypoints/llm.py#L124-L125 to support sampling_params: Optional[Union[SamplingParams, List[SamplingParams]]
When the sampling_params is None, we should use the default. When it is a single value, it should be applied to every prompt. When it is a list, the list must have same length as the prompts and it is paired one by one with the prompt.

I'll be working on this task!


Add ROM and ISO dumping features

For the future, it might be interesting to try and add accurate dumping of ISOs and ROMs via rom kit somehow. While ROMs are in the air since copiers/usb dumpers usually take care of the process most of the time, there should be an open source community supported tool that can be easily used to create accurate dumps of CD/DVD based media. Certain media will require different methods of dumping though, so this might be difficult to do.

As far as discs go, there are two open source programs we could possibly fork and change to suit our needs:
cdrdao
cdrecord

https://github.com/saramibreak/DiscImageCreator
http://forum.redump.org/topic/10483/discimagecreator/
This looks interesting. Maybe we could fork it?


Production problem

When do the deploy into server, the map doesn't show the points, it's only in production using react.
I don't know why in localhost working but en host dont

Seeing this same issue. Just to add some information: the console produces lots of errors about refusing to load resources from Apple’s MapKit CDN because of CORS.
XMLHttpRequest cannot load https://cdn.apple-mapkit.com/ti/csr/1.x.x/res/manifest-0.fb due to access control checks. https://cdn.apple-mapkit.com/ti/csr/1.x.x/res/manifest-0.fb
Looking at the response from a request to https://cdn.apple-mapkit.com/ti/csr/1.x.x/res/manifest-0.fb, it seems as though Apple sets the Access-Control-Allow-Origin header to *, * (two wildcard values are not allowed).
This is slightly outside my expertise, and I'm not sure if these access control issues are even related to annotations not being visible.

Following up on this: turns out these problems were caused entirely by deploying to Netlify. Can’t say exactly why, but this issue doesn’t happen on other production hosts. I’m betting that this is also @itexicoCarlos’s problem.

Depending on your production setup I hadn't realised until recently that child.props.name may get stripped out in the production build. I'll look into an alternative solution to stop this in the future.


Build benchmarks with upstream pytorch

WIP
Reference: #1911

@Wheest @ESI-SYD @chengjunlu could you please take a look at this DRAFT changes. I've converted it to work with upstream pytorch and made it work, so it compiles and produce number. But I don't have expertise in the product and how legit those changes are and if they negatively affects results. Could you review conceptually if this may take place. If so, I'll prepare it for the code review and make it production ready.

Basically looks good to me.
Need to confirm that the Torch profiler (Kineto) can capture the Triton kernel performance. Here is the steps to build the Torch with Kineto XPU supporting.

Install the Intel PTI package: l_intel-pti-dev_p_<IP_ADDRESS>
Source the PTI env: source ${HOME}/intel/oneapi/pti/latest/env/vars.sh
Build the stock Pytorch.

I've build pytorch with PTI and was managed to get numbers out of gemm benchmark, but I don't have enough domain knowledge to confirm that they makes sense.

I've build pytorch with PTI and was managed to get numbers out of gemm benchmark, but I don't have enough domain knowledge to confirm that they makes sense.

We can compare the benchmark reports to the nightly performance reports.

@ZzEeKkAa , The Kineto has issue in tracing the Triton kernel performance. The triton.testing.do_bench is used in the benchmarks for Pytorch 2.5.
We can use the benchmarks do_bench after the Kineto issue has been fixed.

Can we update to setup upstream pytorch? Then we can run it in Actions.
https://github.com/intel/intel-xpu-backend-for-triton/blob/386ae76aebefa156d858f8cf6f58247764067340/.github/workflows/triton-benchmarks.yml#L52-L56

Upstream PT(9f1945a3bc1defa56c01cb1fbdfdd3a938e8c025)
vs pt+ipex
: Has performance lost
preprocessed-data-upstream-ipex.csv
performance-plot-upstream-xetla-tflops-upstream-ipex-xetla-tflops-ipex.pdf
performance-plot-upstream-triton-tflops-upstream-ipex-triton-tflops-ipex.pdf

Upstream PT(9f1945a3bc1defa56c01cb1fbdfdd3a938e8c025) vs pt+ipex : Has performance lost
preprocessed-data-upstream-ipex.csv performance-plot-upstream-xetla-tflops-upstream-ipex-xetla-tflops-ipex.pdf performance-plot-upstream-triton-tflops-upstream-ipex-triton-tflops-ipex.pdf

Not sure how to read the result, looks like it failed to run?
If there is performance lost, do you know why? Is the relative performance still the same between XeTLA and Triton?

I updated this PR, skipped upstream torch sdpa failure to get successful run to unblock Fa benchmark.
Take this one as reference (with IPEX)
From reports generated by script and focus on cases whose relative difference value far from zero.
rm -rf performance* preprocess* ipex/ upstream/ && python compare_runs.py -d "ipex:10623940969" -n "upstream:10629513475" --triton-benchmark

Difference summary:

Ratio of Triton/ XeTLA keep same except for attention.

Attention XeTLA cases degraded (reproduced, needs fix)

Both Triton and XeTLA softmax cases degraded, so ratio not changed.

(required for #1933)

Blocked by https://github.com/intel/intel-xpu-backend-for-triton/issues/2042, converting to draft.

I updated this PR, skipped upstream torch sdpa failure to get successful run to unblock Fa benchmark.
Take this one as reference (with IPEX)
From reports generated by script and focus on cases whose relative difference value far from zero.
rm -rf performance* preprocess* ipex/ upstream/ && python compare_runs.py -d "ipex:10623940969" -n "upstream:10629513475" --triton-benchmark

Difference summary:

Ratio of Triton/ XeTLA keep same except for attention.
Attention XeTLA cases degraded (reproduced, needs fix)
Both Triton and XeTLA softmax cases degraded, so ratio not changed.

I still see RECORD_FUNCTION("xetla softmax", {input}); (and similar) function calls. Can this affect the performance?


remove fmt.sh

Keep it consistent with "make format"

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mandarjog
Assign the PR to them by writing /assign @mandarjog in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

bin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Given the makefile targets format/format.gofmt/format.goimports, I agree, this second approach is inconsistent and should be removed.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

bin/OWNERS [sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

fmt.sh ensures consistency by removing blank lines. The in turn ensures imports always come out in 3 sections.   (stdlib, 3rd party and istio.io)
If there are blank lines, combination of goimports and gofmt does not do anything.
The make target should call bin/fmt.sh
@xiaolanz please revert the PR and fix it in the makefile. If there was an issue associated please link it here.


Released notes update to reflect new feature launch

The Partner Center dashboard now includes Reviews, for extension developers. Devs can respond to user reviews. This feature is moved from Roadmap to Released notes

Docs Build status updates of commit a3a4012:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add singular and plural logic to tweets.

Basically right now our tweets say the following when the flag is:
Our predictive model is reporting that the following reaches are unsafe as of 07:01:30 PM, 10/13/2020: 2, 3, 4. https://crwa-flagging.herokuapp.com/

We want it to instead say, when there is one reach:
The CRWA is reporting that reach 2 is unsafe for recreational activities as of 07:01:30 PM, 10/13/2020. https://crwa-flagging.herokuapp.com/

When there are 2 or more reaches:
The CRWA is reporting that reaches 2 and 3 are unsafe for recreational activities as of 07:01:30 PM, 10/13/2020. https://crwa-flagging.herokuapp.com/
The CRWA is reporting that reaches 3, 4, and 5 are unsafe for recreational activities as of 07:01:30 PM, 10/13/2020. https://crwa-flagging.herokuapp.com/

The reaches should be the actual numbers of the reaches that are unsafe.
You will need to modify the message logic in twitter.py. Use the flask shell (see the docs) along with the admin panel to test the code with the compose_tweet() function.

Done! #121


Add support for gitter.im

So we can get rid of the inconsistent button at the end of button lists:

I attempted to make a simple one I suppose this could work for it, the icon... is what irks me:

https://img.shields.io/badge/%E2%8A%AA%20GITTER%20-JOIN%20CHAT%20%E2%86%92-brightgreen.svg?style=flat

I agree wholeheartedly. We in fact have a very similar badge for talk.ai (a Shanghai-based equivalent).
Having more information would probably be nice. Talk.ai provides an API with the topic of the rooms and a room-specific color, for instance.
Regardless, this is probably, for now, the closest thing we can do that maintains the visual consistency:
https://img.shields.io/badge/gitter-join%20chat%20%E2%86%92-brightgreen.svg

Lowercase is more readable, which is why it is used on all badges. With uppercase, you cannot distinguish letters by how high they go up or by how low they go down. That's in essence the result of a NASA finding I read at some point, which lead them to change their important warnings to be lowercase.
I am not sure how to add the logo. I could store the image server-side, I presume. I need this ability for this anyway: https://github.com/badges/shields/issues/337. In that case, you can help me by making it in SVG.

Copy of @bardiharborow's issue:
Current Gitter badge looks like this:
[![Join the chat at https://gitter.im/sahat/hackathon-starter](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/sahat/hackathon-starter?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)

Discussions are underway here about switching to shields.io.
This is how a you can make a shields.io badge for Gitter currently:
[![Join the chat at https://gitter.im/sahat/hackathon-starter](https://img.shields.io/badge/GITTER-join%20chat-green.svg)](https://gitter.im/sahat/hackathon-starter?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)

I thought it would be much better to have a badge which looks like this:
[![Join the chat at https://gitter.im/sahat/hackathon-starter](https://img.shields.io/badge/GITTER-4%20users%20online-green.svg)](https://gitter.im/sahat/hackathon-starter?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)

I will try to put together some API examples when I get a chance.

@espadrine whoops. Sorry about the duplicate issue.
In terms of getting the icon to work, how about this:
<svg xmlns="http://www.w3.org/2000/svg" width="142" height="20"><linearGradient id="b" x2="0" y2="100%"><stop offset="0" stop-color="#bbb" stop-opacity=".1"/><stop offset="1" stop-opacity=".1"/></linearGradient><mask id="a"><rect width="142" height="20" rx="3" fill="#fff"/></mask><g mask="url(#a)"><path fill="#555" d="M0 0h52v20H0z"/><path fill="#4c1" d="M52 0h90v20H52z"/><path fill="url(#b)" d="M0 0h142v20H0z"/></g><g xmlns="http://www.w3.org/2000/svg" fill="#fff" text-anchor="middle" font-family="DejaVu Sans,Verdana,Geneva,sans-serif" font-size="11"><g xmlns="http://www.w3.org/2000/svg"><rect x="5" y="5" fill="#fff" width="1" height="5"/><rect x="7" y="6" fill="#fff" width="1" height="7"/><rect x="9" y="6" fill="#fff" width="1" height="7"/><rect x="11" y="6" fill="#fff" width="1" height="4"/></g><text x="31" y="15" fill="#010101" fill-opacity=".3">gitter</text><text x="31" y="14">gitter</text><text x="96" y="15" fill="#010101" fill-opacity=".3">4 users online</text><text x="96" y="14">4 users online</text></g></svg>

I got this to work by requesting a badge with iiiigitter (to make a space for the logo), removing the is, inserting the logo (which is already SVG in the official badge) and adjusting the text to be centered again (probably isn't pixel perfect). I don't, however have any idea how to do that programmatically as I'm not familiar with the codebase.

@bardiharborow SVG logos are the idea, except they'd be in base64 in an image tag. The tricky bit is to allow arbitrary logos. I think I'll add a logo URL parameter that accepts an SVG logo in base64, but I'm not sure how fast we'll hit the size limit and how problematic that'd be. For third-party users, it won't be an issue, since they'll be able to application/x-www-form-urlencoded it.
I'll try to make something tonight.

Hi! Mauro from Gitter here. I just create a PR with basic support for gitter badges. At the moment doesn't display usercount or anything else but at least will give people the option to personalise the style :)
wdyt? the PR is here #440

@malditogeek, can you implement an api that would get @bardiharborow's badge to work (user count)? That would be something I'd be extremely interested in.
@espadrine, if this goes through, would you be willing to move forward with it?

Sure!

@malditogeek Should I open a bug in https://github.com/gitterHQ/realtime-client/issues to request an enhancement to the API?
Currently, the only way to fetch the number of users in a room is through
curl https://api.gitter.im/v1/rooms/555f74e315522ed4b3e0ce42/users -H 'Authorization: Bearer XXXX' -H 'Accept: application/json'

However, having 555f74e315522ed4b3e0ce42 as the input is way less clear than Microsoft/TypeScript (I don't even remember how I found the hex id, but it probably involved reading the JS downloaded from the page). Is there a way to map (what the doc calls) a url to a room id? Could we have /room-from-url/:url/users? Or even better, /room-from-url/:url/usercount?

@espadrine, greetings.
I was just wondering what the status of this is, considering that you have recently closed #440 with some commits pushed to master, yet there is still no Gitter badge on http://shields.io/ as I write this.
Thanks in advance for your feedback !

@dregad It's been live a while, although there was no example for it on the main page. Fixed now!
https://img.shields.io/gitter/room/nwjs/nw.js.svg

Awesome, many thanks !
I don't care all that much about the room count, that's just a nice to have for me. On the other hand, it really bothered me that Gitter's standard button does not have the same look&feel than the others I use.

For the record, Gitter's "official" badge is now standardized

Official: https://badges.gitter.im/Join Chat.svg
![Gitter](https://badges.gitter.im/Join Chat.svg)
Shields.io: https://img.shields.io/gitter/room/badges/shields.svg

With regards to the user count feature, here's the feedback from Gitter team:

Does this mean you're not going to do the user count thing ? Or is that tracked somewhere else ?

Not right now. At the moment you have to have an auth token to make any api request.

I also don't care much for the room count. However, flat-squared renders with no icon :(

 <-- icon missing
![](https://img.shields.io/gitter/room/TechnologyAdvice/Stardust.svg)
![](https://img.shields.io/gitter/room/TechnologyAdvice/Stardust.svg?style=flat)
![](https://img.shields.io/gitter/room/TechnologyAdvice/Stardust.svg?style=flat-square)

@levithomason Isn't there one? I see one. What browser and OS are you using?

If you have more information, feel free to open a new issue.

I just got pinged from this issue, but not sure why the notification came just now.  At any rate, the issues seems to have been resolved.  At the time of writing, the icon was missing.


Kubernetes

Hello,
Are there any plan support kubernetes/openshift like with reportportal?

Not yet. Planned for Q3-Q4 this year

We are planning to do this now and would also make it runable in openshift. Are you open to pull requests?
We would do it as a helm chart.

Hi
When do you plan to create an official release? Do you already have some date in mind?
I have tried to install your app from master branch into our env but didnt work properly. I was unable to start grafana since it tried to fetch plugins from public repo. Nevertheless when i tried to add project in UI it didnt work out.
The other branch sqlite_version i tried on my private desktop and installation worked however seems to be quite behind master. Also seems that it uses installer which i find as a nice idea but would be harder to move to k8s in case that fetching from public repos is coded in.
Therefore i would like to write for you helm chart for k8s installation but i would like to know if i should try with master or wait for stable release.

Hi @tomassatka official release is planned on mid July, beginning of July is installer availability and extensive testing start

Hi @jbadeau
We are open for pulls and contributions, will be glad to help.


Implementation requirements for verbs_in_action?

My question is regarding the implementation of this paper "Verbs in Action: Improving verb understanding in video-language models" (verbs_in_action).
Im having trouble figuring out the dependencies to run this code. Can this scenic code (verbs_in_action) run with jax for CUDA 11.3?
Also, wondering where I can access the requirements for this implementation?

It seems that this project has many bugs. For example, the following dependencies are missed: dmvr, clip and official.
You can install them using the following code:
pip install git+https://github.com/deepmind/dmvr.git conda install --yes -c pytorch pytorch=1.7.1 torchvision cudatoolkit=11.0 pip install ftfy regex tqdm pip install git+https://github.com/openai/CLIP.git pip install -U -q "tf-models-official"
Still, some other parts of the code need to be improved.

Hi Masoud,
Yeah I noticed. I installed some of these. My biggest issue is the system requirements. I have CUDA 11.3, so if I need to install the corresponding jaxlib for my CUDA, I wonder if this code works for CUDA 11.3? Were you able to run this code eventually?

Hi Reza,
I run the code on Colab. Although after installing dependencies the problem of requirements has been solved, it still has many problems that need to be fixed. (The code is not clean enough and needs so much improvement!) I finally gave up. I hope developers clean up the code and provide some comments.

Hi Masoud,
1-Can you elaborate what you mean by "it has many problems that need to be fixed"? what specific problems?
2- What version of jaxlib and Cuda did you use?

Hi Reza,
I suggest you first run the code on Colab. When you use Colab you have no problem with versions of jaxlib and you could be sure that the code is clean. When you are sure about the code itself, then you can run it locally. I did not run the code on a local machine, so I cannot tell you about versions difference.


[Feature] InheritedAttribute -  Extend Mix functionality with custom attributes.

@leoafarias Hello, excellent work here! This lib is amazing!
Currently not is possible to use Mix in components not yet supported by the lib, such as TextField and DropdownButton.
I worked on an implementation that makes possible to create new attributes and consume them from the context so that it is possible to work with the Mix in any component.
Related issues:
https://github.com/leoafarias/mix/issues/93, https://github.com/leoafarias/mix/issues/91 and https://github.com/leoafarias/mix/issues/46;
Design
Create the extended inherited attribute
class InputDecorationThemeAttribute extends InputDecorationTheme
    implements InheritedAttribute<InputDecorationThemeAttribute> {
  const InputDecorationThemeAttribute({
    Color? iconColor,
    Color? fillColor,
    InputBorder? border,
  }) : super(
          iconColor: iconColor,
          fillColor: fillColor,
          border: border,
        );

  @override
  InputDecorationThemeAttribute merge(InputDecorationThemeAttribute other) {
    return InputDecorationThemeAttribute(
      iconColor: other.iconColor ?? iconColor,
      fillColor: other.fillColor ?? fillColor,
      border: other.border ?? border,
    );
  }

  static InputDecorationThemeAttribute inputDecoration(
      {Color? iconColor,
      Color? fillColor,
      InputBorder? border,
      Color? hoverColor,
      InputBorder? enabledBorder}) {
    return InputDecorationThemeAttribute(
      iconColor: iconColor ?? iconColor,
      fillColor: fillColor ?? fillColor,
      border: border ?? border,
    );
  }

  @override
  Object get type => InputDecorationThemeAttribute;
}

Define a short utils
const inputDecoration = InputDecorationThemeAttribute.inputDecoration;

Create a Mix
final inputDecorationMix = Mix(
        inputDecoration(
          border: const UnderlineInputBorder(
            borderSide: BorderSide(color: Colors.grey),
          ),
        ),
        myVariant(
          inputDecoration(
            enabledBorder: const UnderlineInputBorder(
              borderSide: BorderSide(color: Colors.yellow),
            ),
          ),
        ),
      )

Consume attributes from mixContext
   ...

 @override
  Widget build(BuildContext context) {
    return MixContextBuilder(
      mix: inputDecorationMix,
      builder: (context, mixContext) {
        final inputDecorationTheme =
            mixContext.fromType<InputDecorationThemeAttribute>();

        return TextFormField(
          decoration: const InputDecoration(
            filled: true,
            labelText: 'Label Text',
            suffixIcon: Icon(
              Icons.check_circle,
            ),
          ).applyDefaults(inputDecorationTheme),
        );
      },
    );
  }

Reference
The implementation is based from: https://api.flutter.dev/flutter/material/ThemeData/extensions.html

@pbissonho looks good, that is a great contribution. Might make some small adjustments as this feature matures, but I think its good to be merged!

@all-contributors please add @pbissonho for code, ideas

@all-contributors please add @pbissonho for code, ideas

@all-contributors please add @pbissonho for code, ideas

@all-contributors please add @pbissonho for code, ideas


Change serializers interface to loads and dumps

Custom serializer class expects serialize/deserialize methods. Seems reasonable to move them to expect dumps/loads since pickle, json, marshmallow and others use that.

Fixed by #12


Fix error at DeformConvPack module

Fixed error TypeError: super(type, obj): obj must be an instance or subtype of type

Thanks for the fix!

Hi @lyuwenyu！
First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


List of ideas for Vanguard

Here's a list i gathered from perplexity in no specific order:

Magic class
Animation support
Sound support
Auto aim mechanic
Hazard class (traps)
Raycast support
Charge mechanic (imo that's just "firerate" reversed but ok)
Cooldown mechanic (same thing as charge)
Durability
Ammo types

Since this is a general purpose 2D weapon system, i wanna discuss it with you @SP4R0W

to clarify:

Magic class is a weapon that shoots some cool looking effect and has mana mechanic (number that depletes as you're shooting, goes from 100 to 0, naturally regenerates at a speed) + area of effect.
Example of Charge is preparing a bow.
Example of "Cooldown" is minigun mechanics, if you continuously shoot, it reaches a point where it overheats and you have to wait before using it. so you have to shoot every now and then and wait for it to cooldown.


OpenVPN Client Specific Override is not bound to a server

The client override should be bound to a server.
User A can connect to Server 1 and Server 2, he could need specific settings that is different between server 1 and server 2, say give him a fixed IP in 2 different networks. This is currently not possible.
Ideally there should be a checkbox list for each server. So a profile can be enabled on multiple servers too.
I guess the easiest implementations is to store them somewhere and symlink them in a per server directory.

@sjorge I've just created a fix for this feature request, if you have time to test, you can download the next files over a 16.1 release.
curl -o /usr/local/etc/inc/openvpn.inc https://raw.githubusercontent.com/opnsense/core/master/src/etc/inc/openvpn.inc
curl -o /usr/local/www/vpn_openvpn_csc.php https://raw.githubusercontent.com/opnsense/core/master/src/www/vpn_openvpn_csc.php
curl -o /usr/local/www/vpn_openvpn_server.php https://raw.githubusercontent.com/opnsense/core/master/src/www/vpn_openvpn_server.php

I'm not near my setup at the moment so testing is a bit hard.
If I don't forget I'll try to poke it this weekend

~ sjorge
On 2016-02-11 18:30, Ad Schellevis wrote:

@sjorge [1] I've just created a fix for this feature request, if you have time to test, you can download the next files over a 16.1 release.
curl -o /usr/local/etc/inc/openvpn.inc https://raw.githubusercontent.com/opnsense/core/master/src/etc/inc/openvpn.inc
curl -o /usr/local/www/vpn_openvpn_csc.php https://raw.githubusercontent.com/opnsense/core/master/src/www/vpn_openvpn_csc.php
curl -o /usr/local/www/vpn_openvpn_server.php https://raw.githubusercontent.com/opnsense/core/master/src/www/vpn_openvpn_server.php
--
Reply to this email directly or view it on GitHub [2].

Links:
[1] https://github.com/sjorge
[2] https://github.com/opnsense/core/issues/296#issuecomment-182967742

Thanks in advance!

@sjorge any news here? :)

Looks good at first glance, due to losing my 2nd external IP i can't give it a very thorough shake down though.

@sjorge Thanks for testing!


Lock forum threads

This PR adds ability to lock forum thread.
resolve #697

Unlocked thread [Admin/Moderator]

Locked thread (By clicking on the lock)

Admin/Moderator can unlock the thread by clicking on the lock again
User can't comment on locked thread

@faissaloux can you rebase with main? That should fix the StyleCI issues.

@faissaloux heya, I'm so sorry I didn't get to this yet. Could you maybe rebase this PR with master and make sure it's adapted for Tailwind v3? I'll try to work on it in the next couple of weeks. Thanks!

@driesvints done 🚀

Thanks for this @faissaloux! Sorry it took me so long to get to this.


HHH-16498 Hibernate DDL validation fails on enum database column type

Backport of https://github.com/hibernate/hibernate-orm/pull/6527 for https://hibernate.atlassian.net/browse/HHH-16498

Guess that in case of 6.2 initial fix will be the solution. I will check that.

Fixed in PR 6541


Add live coding (and simulator) app reference to readme

Demo: https://sirxemic.github.io/xmastree-app
Was not sure where to share this. Since I've seen people share their own utilities here, figured I could do the same.
Added a reference to an app I made where you can write JS code and see the effect directly in the browser. It also supports exporting the resulting loop to a CSV file, and importing and playing back CSV files.

Thanks for sharing this and great work on the simulator! I love how quick it runs!


Patch 1

Starting content for README and CONTRIBUTING.
CONTRIBUTING borrowed heavily from https://github.com/github/docs/blob/main/CONTRIBUTING.md

This is looking good! I vote to merge it, and we can keep refining afterwards as needed.

I didn't see opposition to merging it (specially after @abkfenris's changes), so I'll make the executive decision to do it right now


daxpy! in README.md

I just saw that it basically used DAXPY while it was called SAXPY

Oh, I switched from saxpy to daxpy at some point but must have forgotten to rename things. Thanks!


Adjustment in regexp

I had to change the regexp to have it working for me
regex = (
    r'.*'
    r'\*\* ((?P<error>Error)|(?P<warning>Warning)): '
    r'.*(?P<file>\w+\.vhd)'
    r'\((?P<line>\d+)\): '
    r'(near "(?P<near>.+)": )?'
    r'\((?Pvcom-\d+)\)?'
    r'(?P<message>.*)'
)

regex = (
    r'.*'
    r'\*\* ((?P<error>Error)|(?P<warning>Warning)): '
    r'.*(?P<file>\w+\.vhd)'
    r'\((?P<line>\d+)\): '
    r'(near "(?P<near>.+)": )?'
    r'(\((?Pvcom-\d+)\))?'
    r'(?P<message>.*)'
)

Notice the extra parenthesis around the  group line

@vengin do you want to test this out and add it to your PR?

From my tests I believe I have fixed this problem. Closing.


Hit missing _timestamp

Hit<T> should have a Timestamp property in order to support https://github.com/elastic/elasticsearch-net/issues/985

_timestamp is returned as a double representing milliseconds since epoch. Should we parse attempt to parse to DateTime? ?


Fix Windows Release Build

As mentioned in #3 this is changing the windows build to use rtaudio.libinstead of rtaudiod.lib in release mode. Tested on current Windows 11.

Looks good, thanks for the fix!


Setup frontend

Hi!
Framework looks nice, taking a look at it as an alternative to Sails etc.
Some more detailed docs would be nice:
. Quickstart quide for front and backends
. Architecture overview
. Deployment guide for Heroku or similar

Hi Paul,
I hope you are doing great. Thank you so much for looking into this small(at the moment) repo. Currently, I have setup CRUD for User Roles, Roles, Role Permissions, User Permissions.  Regarding roles, I need to add to middleware on each GraphQL request.
I am working hard to add many features daily. Creating a saas project is not an easy thing. So I created this repo to set up backend and frontend React/Vue.
Other Answers.

I will add documentation to it and make it much simpler.
I will add this as well.
I will add this to next month.

I will add more features to this repo and you can star this repo.
Once again, thank you for taking interest in this repo. I hope this repo will be a popular repo soon.
Best,
Ilyas Karim

Hey Ilyas,
Sounds great. I'll have a poke through it in the meantime and see if I can get a handle on it.

Creating a saas project is not an easy thing. So I created this repo to set up backend and frontend React/Vue.

I think you should just focus on Vue as the front end for now to focus your time, leaving it open for react. It looks like that's where it's at anyway as there are no files under react anyway?
I like the fact there's a couple of database implementations. It's a good start I think!

Hi Paul,
Yes, I am starting with Vue as the front end. Yes, there are no files right now but I cloned https://github.com/react-boilerplate/react-boilerplate.
Regarding the database, different people like different databases and I will be adding more in the future but now focusing on making current MySql + Mongo features perfect.
Overall thanks for checking this tiny repo. If you have any suggestions you can write them as comments in this issue https://github.com/react-boilerplate/react-boilerplate.
See some fresh photos of Hunza Valley: https://www.instagram.com/hunzaavalley/.
Cheers,
Ilyas Karim

I am closing this as I have created a board on Trello for this, here it is https://trello.com/b/mJB0xR2u/nodeboilerplate. This issue board is to capture bugs.
Trello Board: https://trello.com/b/mJB0xR2u/nodeboilerplate.
Bug Report: https://github.com/ilyaskarim/NodeSaaSBoilerplate/issues/new.

aok, ty