Deprecation warning after upgrading to redis gem version 4.2.0

After upgrading to version 4.2.0 of the redis gem, I see these deprecation warnings:
`Redis#exists(key)` will return an Integer in redis-rb 4.3, if you want to keep the old behavior, use `exists?` instead.
To opt-in to the new behavior now you can set Redis.exists_returns_integer = true.
(rollout-2.4.5/lib/rollout.rb:299:in `exists?’)

Seems like rollout should use exists? for redis 4.2.0 and higher.

Perhaps related, if I run bundle update on the rollout repository itself and run rspec, rollout's specs currently fail:
Failures:

  1) Rollout Check if feature exists it should return true if the feature is exist
     Failure/Error: @storage.exists(key(feature))

     NoMethodError:
       undefined method `>' for true:TrueClass
     # ./lib/rollout.rb:299:in `exists?'
     # ./spec/rollout_spec.rb:669:in `block (3 levels) in <top (required)>'

  2) Rollout Check if feature exists it should return false if the feature is not exist
     Failure/Error: @storage.exists(key(feature))

     NoMethodError:
       undefined method `>' for false:FalseClass
     # ./lib/rollout.rb:299:in `exists?'
     # ./spec/rollout_spec.rb:673:in `block (3 levels) in <top (required)>'

Finished in 0.58978 seconds (files took 0.16322 seconds to load)
83 examples, 2 failures

Failed examples:

rspec ./spec/rollout_spec.rb:667 # Rollout Check if feature exists it should return true if the feature is exist
rspec ./spec/rollout_spec.rb:672 # Rollout Check if feature exists it should return false if the feature is not exist

@mattbrictson thank you for opening the issue.
I would be happy to fix it but that error is actually coming from the redis-rb gem itself from
From: /Users/rene/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/redis-4.2.1/lib/redis.rb:606 Redis#exists?:

    603: def exists?(*keys)
    604:   synchronize do |client|
    605:     client.call([:exists, *keys]) do |value|
 => 606:       value > 0
    607:     end
    608:   end
    609: end

[2] pry(#<Redis>)> value
=> true

value is boolean and they are comparing it with 0
If I'm not mistaken, they first need to actually fix #exists? method before we can update rollout.

I think that is actually due to a bug in fakeredis. If you use the redis 4.2.1 gem directly (i.e. without loading fakeredis), it works as expected:
$ irb
>> require "redis"
true
>> Redis::VERSION
"4.2.1"
>> redis = Redis.new(uri: "redis://<IP_ADDRESS>/1")
>> redis.exists?("foo")
false
>>

If you load fakeredis, that is when the error occurs:
$ irb
>> require "redis"
true
>> require "fakeredis"
true
>> redis = Redis.new(uri: "redis://<IP_ADDRESS>/1")
>> redis.exists?("foo")
NoMethodError (undefined method `>' for false:FalseClass)

@mattbrictson ah, you are right, I completely forgot we use fakeredis and not a real one. Created PR to fix this issue and also to stop using fakeredis

Released as part of 2.4.6


Use Empirical Distribution in TracePosterior

Use EmpiricalDistribution inside TracePosterior. This was mostly to make it easier to generate marginal distribution over sample sites in MCMC, but based on @eb8680 suggestions, I tried to do some refactoring to support Importance and Search (only existing tests, I am not sure if this will work with parallel search).
Changes:

TracePosterior has a .marginal method that returns a marginal over one or more sample sites.
There is a Marginal wrapper separately that calls .marginal under the hood but defers populating traces until we call .sample or .log_prob. This was needed to support Search, e.g. Schelling example. @eb8680 - Let me know your thoughts.

Other utilities that could be built over TracePosterior:

Summary stats for sites in TracePosterior - mean, median, p10, p90.
Other queries like conditioning on certain sample sites.
Predictions: Run the model on the replayed traces and new data, to generate predictions.

Looks great so far!  Let's handle Search in a separate PR and do MCMC and Importance here.

Let's handle Search in a separate PR and do MCMC and Importance here

The only additional change for search is the Marginal class. It might be a bit tricky to separate that as the Search examples (particularly the Schelling examples) will not work with the TracePosterior class otherwise. Will discuss with you in detail.

Summary stats for sites in TracePosterior - mean, median, p10, p90.

We currently have .median() and .quantiles(...) methods in the ADVI classes AutoDiagonalNormal and AutoMultivariateNormal (code). These are useful since they can be computed analytically and allow inspection of the under-the-hood latent. Maybe we can converge on a common interface?

Maybe we can converge on a common interface?

Agreed, this will benefit from a common interface that is reusable (at least consistent) across different inference algorithms, in the spirit of R's summary(model_fit). Will discuss with you more.

Agreed, this will benefit from a common interface that is reusable (at least consistent) across different inference algorithms

As discussed in the design doc, I think representations of marginal distributions are the right intermediate layer for this; they have consistent APIs but may be different for different algorithms - for example, ADVI admits a more specialized representation because the marginals can be computed analytically.  Instead of trying to squeeze everything into TracePosterior, we can just compute a dictionary of marginals with e.g. {name: Marginal(posterior, site=name) for name in site_names} and pass that to a summary function.

@eb8680 - The last commit removes the .marginal in favor of an EmpiricalMarginal class that wraps TracePosterior. This simplifies the code, but this eager version of marginal doesn't work with search. :( We can temporarily disable these examples, and reinstate them later with a lazy marginal implementation.

I'd like to see TracePosterior.__call__ renamed to TracePosterior.run, but otherwise this should be good to merge after fixing merge conflicts introduced by #1044.

Before release let's make sure that EmpiricalMarginal shows up in our sphinx docs.

I'd like to see TracePosterior.call renamed to TracePosterior.run

Done.


C# Enums not looked up in object construction

Looking at the sample below,
GraphServiceClient graphClient = new GraphServiceClient( authProvider );

var value = "NV001";

var name = "x-custom-header-group-id";

var valueVar = "Nevada";

var nameVar = "x-custom-header-group-nameVar";

//create InternetMessageHeader list and populate it
var internetMessageHeaders = new List<InternetMessageHeader>();
internetMessageHeaders.Add(new InternetMessageHeader(nameVar,valueVar));
internetMessageHeaders.Add(new InternetMessageHeader(name,value));

//create instance of EmailAddress
var emailAddress = new EmailAddress
{
	Address =<EMAIL_ADDRESS>};

//create Recipient list and populate it
var toRecipients = new List<Recipient>();
toRecipients.Add(new Recipient(emailAddress));

//create instance of ItemBody
var body = new ItemBody
{
	ContentType = "HTML",
	Content = "The group represents Nevada.",
};

//create instance of Message
var message = new Message
{
	Subject = "9/9/2018: concert",
	Body = body,
	ToRecipients = toRecipients,
	InternetMessageHeaders = internetMessageHeaders,
};

await graphClient.Me
	.SendMail(message,saveToSentItems)
	.Request()
	.PostAsync()

The ContentType member in ItemBody should be an enum and not a string. Snippet generation should look up enum types as well.

Closed with code changes made under PR #27


能否集成关于Conversational Graph Grounded Policy Learning for Open-Domain Conversation Generation提到的 model？

能否集成关于Conversational Graph Grounded Policy Learning for Open-Domain Conversation Generation提到的 model？

Thanks! We will check the paper to decide whether to include the model in the next version.

Thanks! We will check the paper to decide whether to include the model in the next version.


8 bit for first conv  and fc

Hi， did you try to use 8 bit for first conv  and fc for quantized network training? Will the accuracy drop very much?

Yes. I dont remember very well, so it is a bit difficult to give an exact figure. But the accuracy loss is unacceptable.
On Apr 27, 2021, at 11:46, liyue2ppy<EMAIL_ADDRESS>wrote:
Hi， did you try to use 8 bit for first conv and fc for quantized network training? Will the accuracy drop very much?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/zhutmost/lsq-net/issues/11, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE4MHZU67YQV6NLIRPIR2KDTKYXSTANCNFSM43UEUJFA.

I see the initialization of the step parameter for activation in this repo is different from the paper. Why don't use the paper 's method? Is it invalid?

I see the initialization of the step parameter for activation in this repo is different from the paper. Why don't use the paper 's method? Is it invalid?

Sorry to reply so late. I forgot it (T = T)
I am doing more experiments, and the experiment setup is the same as the paper. You can visit https://github.com/zhutmost/neuralzip for more details.


Add custom comments at the top of the rendered configuration

Enhancement summary
Opening issue to discuss the option to include custom comments at the top of the generated config. Example:
!RANCID-CONTENT-TYPE: arista
!
! Config version: x.x.x
!
daemon attr
...

Which component of AVD is impacted
eos_cli_config_gen
Use case example
It is common to use the rendered config from eos_cli_config_gen as tje design configlet in CVP. Sometimes we want to include comments that add information about the configlet such as a generated version or a change ticket, which helps Ops to trace back to the source of the change. This comment section can also be used to force CVP to create a task and reapply a configlet (with no deltas) if we need to do that.
Describe the solution you would like
a new header_comments knob or similar that would allow to add comments at the beginning of the rendered config.
Example:
header_comments:
  - "Config version: XXX"
  - "Change ticket: XXXXX"
  - |
    some extra long text
   in multiple lines

Would render:
!RANCID-CONTENT-TYPE: arista
!
! Config version: XXX
! Change ticket: XXXXX
! some extra long text
! in multiple lines
!
daemon attr
...

Describe alternatives you have considered
Render this information with eos_cli or with a custom template, however it gets rendered at the end, not too intuitive if you have to scroll all the way down to see that info
Additional context
Although comments don't get pushed to the devices, comments in designed configlets for different config section can give some good context about some parts of the config, for those who look at the configlets in CVP.
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Would it be useful to use the EOS command comment so the comments will be part of the running config?

Would it be useful to use the EOS command comment so the comments will be part of the running config?

That would be also useful for most of the configuration sections where this is supported. However comment command cannot be used to add a general comment at the top if I'm not mistaken


staging-next 2023-02-15

workflow docs: https://nixos.org/manual/nixpkgs/unstable/#submitting-changes-commit-policy
important jobs:
https://hydra.nixos.org/job/nixpkgs/staging-next/unstable#tabs-constituents
https://hydra.nixos.org/job/nixos/staging-next-small/tested
jobsets:
https://hydra.nixos.org/jobset/nixpkgs/staging-next
https://hydra.nixos.org/jobset/nixos/staging-next-small
nix-review reports: https://malob.github.io/nix-review-tools-reports/
previous staging-next: https://github.com/NixOS/nixpkgs/pull/214010

Mass breakages (will be edited in future):

[ ] python3Packages.aiohttp fail tests
On *-linux at least.  aarch64 log; I did x86_64 locally with the same three tests reported.

Oh, .aiohttp has a maintainer: @dotlambda ⬆

@NickCao @trofi 11b095e8805aa123a4d77a5e706bebaf86622879 (master) and fa0633e7faac4be522850c55854683db71ef6d8f (staging-next) produce a merge conflict.  I could just resolve it, and we keep both changes, but since they both seem to be different solutions for the same issue, maybe we only need to keep one?

@NickCao @trofi 11b095e (master) and fa0633e (staging-next) produce a merge conflict. I could just resolve it, and we keep both changes, but since they both seem to be different solutions for the same issue, maybe we only need to keep one?

Yeah, I suggest dropping/reverting the master change in staging-next.

Just revert my commit on master, the one in staging-next looks better.

Mass breakages (will be edited in future):
* [ ]  `python3Packages.aiohttp` fail tests
  On `*-linux` at least.  [aarch64 log](https://hydra.nixos.org/log/sgg5yj8nlyy46qghd03ysymnm7473pwl-python3.10-aiohttp-3.8.3.drv); I did x86_64 locally with the same three tests reported.

* [ ]  `libdeflate` on `x86_64-darwin` (only)
  [libdeflate: 1.8 -> 1.17 #216181 (comment)](https://github.com/NixOS/nixpkgs/pull/216181#issuecomment-1435510711)

* [ ]  `ruby_3_0` on `*-darwin` (only)
  [aarch64](https://hydra.nixos.org/build/209780455), same for x86_64 locally.

@dotlambda  The build of python3Packages.aiohttpfails after the dnspython upgrade. I haven't been able to figure out why, though

Alright, reverted https://github.com/NixOS/nixpkgs/commit/11b095e8805aa123a4d77a5e706bebaf86622879 but kept 463ab8deaa17d333be2ff7c61bea4e8e664e71cb.

ruby_3_0 on *-darwin (only)
aarch64, same for x86_64 locally.

cc @marsam

Fixed ruby_3_0 in https://github.com/NixOS/nixpkgs/pull/217024

Working on an ibus fix

Challenge failed :C
If someone else wants to give fixing ibus a try it would be appreciated.

I'm having a look at ibus.

So, I decided to move the checklist with biggest regressions to the first post and edit in-place, because of practical issues with the UI (like visibility).
We still have at least a few hundred regressed builds.  Apart from the checklist, much of them are various individual python failures, presumably due to pythonPackages updates.

python3Packages.uvloop on x86_64-darwin (some test failure)
hydra.nixos.org/build/209939576

Retried once more on hydra and it built just fine. :disappointed:

Eww, that's really lots of retries, but OK for this iteration I guess.

Even worse, the tests segfaulted for me via rosetta on the darwin community builder. I have no energy to dig into this right now.


failed to create: [publicapi]

failed to create publicapi while creating the admin app with following error in cloudformation

branch: main

Thank you for the bug report. A fix is in process.


Draft: Translation & refactor test, product

Yet another part of translation for some part product and test modules.
Also I refactored imports for test/views.py

@StefanFl, @damiencarol, please help me with "Lint Helm chart" failing checks

@shipko You have to rebase your branch to the current dev branch. The helm chart in your branch is too old.

@shipko You have to rebase your branch to the current dev branch. The helm chart in your branch is too old.

I agree, just rebase


Enhance setValue Docs

A couple smaller things in this PR.

remove dead space from format script
visually de-clutter clearError docs
add TS, JS and Types + CodeSandbox for setValue

awesome, thanks @pmaier983


Update to version 1.0.0

Compatible with mutada v1.0.0
Remove event emitters
Fix swallowing of errors when writing to transitions db
Change how attaching detect works to only attach to mutada instance

#approve


Find the accurate relationship between initial contig and final scaffold

Hi,
Thank you for your amazing software! Now I have successfully get the chromosomal level assembly after running run-asm-pipeline-post-review.sh, but I need to know the updated location of the contigs relative to chromosomes to update my annotation(I do it in the contig level), I found the ouput files other than hic.(like assembly) might can help me, but I am not sure the meaning of each column. Do you know which output can help me, and can you briefly explain the meaning of the format? Thank a lot!
All my ouput is like this:

Best wishes,
LIZW

Hi LIZW,
Thanks for the kind words. Here is a slide re assembly format that might be of help. In this case _HiC.assembly (DNA Zoo convention) is the same as .FINAL.assembly in your output:
https://docs.google.com/viewer?a=v&pid=forums&srcid=MTQwMjA4ODQ1NDM4NDQ3NjQ4NzABMDg0NTM3ODMxMzA3NTM1MTg3MzcBY25kcHRhUkVBd0FKATAuMQEBdjI&authuser=0
(Maybe you will find also some comments on this thread useful:
https://docs.google.com/viewer?a=v&pid=forums&srcid=MTQwMjA4ODQ1NDM4NDQ3NjQ4NzABMDg0NTM3ODMxMzA3NTM1MTg3MzcBY25kcHRhUkVBd0FKATAuMQEBdjI&authuser=0
)
Best,
Olga

Dear Olga,
Thanks for your warm reply! I found the files "canu_pilon.final_asm.scaffold_track.txt" and "canu_pilon.final_asm.superscaf_track.txt" more useful for me, it shows the change before and after. I would like to know the meaning of "+" "-" in the scaffold file:

Thank you so much!!
Best,
LIZW

LIZW,
This is calculated based on the assembly file. Please note that this corresponds to rawchrom stage and does not include final gaps between draft sequences. Sign refers to whether the whether the draft needs to be reverse complemented or not.
Best,
Olga

On Dec 13, 2019, at 1:01 AM, LIZW2019<EMAIL_ADDRESS>wrote:

Dear Olga,
Thanks for your warm reply! I found the files "canu_pilon.final_asm.scaffold_track.txt" and "canu_pilon.final_asm.superscaf_track.txt" more useful for me, it shows the change before and after. I would like to know the meaning of "+" "-" in the scaffold file:
Thank you so much!!
Best,
LIZW
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


yordis/chore 1

I welp

Closing this as certain changes has already been addressed. Thank you!

😭


netopeer2-cli mixing rpc-reply and notification, corrupting output

Version: netopeer2-2.1.62
Hi,
I noticed in a automated test a weird behavior, I assume it comes from netopeer2-cli.
In one occasion, netopeer2-cli displayed a notification right at the same time as it received the result from a get rpc
2023-08-09 06:13:54.336 [    INFO] (utils_oran.py:458) 2023-08-09 06:13:54.336262: expect_data: get --filter-xpath /ietf-hardware:*

2023-08-09 06:13:54.493 [    INFO] (utils_oran.py:468) BEFORE:

<data xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">

  <hardware xmlns="urn:ietf:params:xml:ns:yang:ietf-hardware">

    <last-changenotification (2023-08-09T06:13:54.480352233+00:00)

<ptp-state-change xmlns="urn:o-ran:sync:1.0">

  <ptp-state>LOCKED</ptp-state>

</ptp-state-change>

>2023-08-09T06:13:46+00:00</last-change>

    <component>

      <name>AHLOA</name>

      <class xmlns:o-ran-hw="urn:o-ran:hardware:1.0">o-ran-hw:O-RAN-RADIO</class>

      <hardware-rev>hw-rev01</hardware-rev>

      <software-rev>oru-sw-r1</software-rev>

      <serial-num>RY150403577</serial-num>

      <mfg-name>NK</mfg-name>

      <model-name>RADIOGEN1</model-name>

      <state>

        <state-last-changed>2023-08-09T06:13:46+00:00</state-last-changed>

        <admin-state>unlocked</admin-state>

        <oper-state>enabled</oper-state>

        <usage-state>active</usage-state>

        <availability-state xmlns="urn:o-ran:hardware:1.0">NORMAL</availability-state>

      </state>

      <label-content xmlns="urn:o-ran:hardware:1.0">

        <model-name>true</model-name>

        <serial-number>true</serial-number>

      </label-content>

      <product-code xmlns="urn:o-ran:hardware:1.0">474331A</product-code>

      <energy-saving-enabled xmlns="urn:o-ran:hardware:1.0">false</energy-saving-enabled>

      <dying-gasp-support xmlns="urn:o-ran:hardware:1.0">false</dying-gasp-support>

      <o-ran-name xmlns="urn:o-ran:hardware:1.0">AHLOA</o-ran-name>

    </component>

    <component>

      <name>P1</name>

      <class xmlns:ianahw="urn:ietf:params:xml:ns:yang:iana-hardware">ianahw:port</class>

      <hardware-rev>hw-p01</hardware-rev>

      <software-rev>sw-p01</software-rev>

      <serial-num>S01234</serial-num>

      <mfg-name>NK</mfg-name>

      <state>

        <state-last-changed>2023-08-09T06:13:46+00:00</state-last-changed>

        <admin-state>unlocked</admin-state>

        <oper-state>enabled</oper-state>

        <usage-state>active</usage-state>

      </state>

      <o-ran-name xmlns="urn:o-ran:hardware:1.0">P1</o-ran-name>

    </component>

    <component>

      <name>P2</name>

      <class xmlns:ianahw="urn:ietf:params:xml:ns:yang:iana-hardware">ianahw:port</class>

      <hardware-rev>hw-p01</hardware-rev>

      <software-rev>sw-p01</software-rev>

      <serial-num>S01234</serial-num>

      <mfg-name>NK</mfg-name>

      <state>

        <state-last-changed>2023-08-09T06:13:46+00:00</state-last-changed>

        <admin-state>unlocked</admin-state>

        <oper-state>disabled</oper-state>

        <usage-state>active</usage-state>

      </state>

      <o-ran-name xmlns="urn:o-ran:hardware:1.0">P2</o-ran-name>

    </component>

  </hardware>

</data>
---------------------------- Captured log teardown -----------------------------

It's worth noting the automation is done through pexpect on top of the netopeer2-cli.
In the meantime, this test was modified to have 2 instances of netopeer2-cli to separate the "command session" from the "notification session" but I wanted to share this usecase.

Yes, nothing to prevent the data being mixed since the notifications are received and printed in a separate thread. Consider it a CLI limitation.

Ok thanks, that's good to know !
Maybe something that could be put somewhere on the documentation in case next users are wondering about it as well.

The main purpose of the CLI is for simple interactive NETCONF sessions. On the off-chance this happens to you, it should be fine, not so much when using it in a script and expecting exact output/automated processing.

Yes I plan to move to ncclient or something like that in the future, but you know how it is, sometime you have to live with legacy code until the time to refactor comes. 😶


Add FDTDx Extension to GDSFactory

This project looks excellent! 🚀
In GDSFactory, we currently support integrations with:

Tidy3D
Lumerical
Luminescent
MEEP
FDTDZ

I’ve added this project to the Awesome Photonics list.
What do you think about adding an extension for FDTDx as well?
You can find more details on the existing plugin architecture here:
https://gdsfactory.github.io/gplugins/
The best plugin is the one from tidy3d

Thanks for the praise!
Adding an extension for GDSFactory sounds like an excellent idea. I believe we can implement something similar to the gt.Tidy3DComponent for FDTDX as well. I will start reading into that!
I think the best next step would be that we start to implement a prototype and come back to you when inevitably some questions arise.


tests/inspect: reorder blocking operations

This commit reorders blocking operations, so that sleeps are performed
as first/last operations and reading stdin will block before performing
any other task.
It makes multi-step tests easier to write and without races.

Re-triggered one of the jenkins runs, flaking on TestCapsSeveralAppsWithFlags.

LGTM, thanks!


Advanced Paste Feature Request

/### Description of the new feature / enhancement
Hi,
I suggest adding scripting capabilities to manipulate copied text and paste the processed text,
user can add multiple scripts as he wants with its own title ,
when user triggers "Advanced paste" he can choose witch script to apply on the copied text,
the script has can be python or any scripting language and structured as a one main function which accept one argument, and user can add any sub-functions according to processing logic.
Scenario when this would be used?
example:
I'm usually needing to copy a text which include many hyphens and I need to paste the text without them, and this a repeated task
usually I copy the text then paste it to notepad then remove the hyphens and then copy it again

input:    DD-5555-22
output:  DD555522

also I can create script to split the input at a certain delimiter and extract the portion I need.
Supporting information
No response
Many thanks for continues support and upgrades

@amr4alex This would be an interesting feature.
But do note that this is already possible with "Paste with AI" (see demo below). If this is a repeated task, you can also create a "Custom action" (see "Add custom action" on the Advanced Paste settings page) and even assign a system-wide keyboard shortcut for it.


Contribute @nuxt/python

We are able to write Nuxt in Python !

This is awesome! I've created a new repo with Write access to you. python-module. Please use module-template and ping me when pushed initial version :)

Hey @pi0 I've pushed to https://github.com/nuxt-community/python-module as requested.
Happy coding!


snap step returning black or blank images on Windows

I also head a white image for one page. The snap was done on linux.

Fix usable on cutting edge release - https://github.com/kelaberetiv/TagUI/archive/master.zip

@shoeper I see.. Oh this issue is a new bug that appeared for Windows.
Linux and macOS should not be impacted by the code update.

I think there is some alignment issue on Windows depending on the screen resolution. Closing the issue for now until further clues of replication. Potentially related to Windows resolution auto-scaling and perhaps the width of the web-browser scroll bar.

Issue for Windows snap image misalignment found. Solution, change Windows display setting to 100% scaling instead of 125%, 200%, 225% etc.  Running at 100% will capture images of elements correctly.


Clairify port usage in multi-manager-setup.md

Clarified port usages in the assumptions section of multi-manager-setup.md.  This is to address issue #996

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "clarifyports" git@github.com:dreilly1982/swarm.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@dreilly1982 I think there should also be a clarification on the agent's IP addresses. It's confusing for new guys to understand that there's:

A swarm agent running on a different IP than managers
A docker daemon running on the same IP of managers

@aanm Agreed.
@dreilly1982 Sorry for the delay on stumbling upon this, are you up to making this change? Otherwise LGTM and we can merge it as it is and make the clarification in another PR.
/ping @moxiegirl

This one PR is long standing... let's merge it as it is just a clarification. We can discuss on improving the whole Multi-Manager doc later on.

@abronan apologies for letting this linger.

@moxiegirl no worries :smiley_cat:


Multiaddr URI scheme

I'd like to propose the following URI scheme for multiaddrs: multiaddr:<string-encoded-multiaddr>.
This would give us the ability to pass in a multiaddr where a URI is expected (e.g. HTTP redirects). A bit more context here: https://github.com/libp2p/specs/pull/568.
I don't think any multiaddrs in use today are incompatible with the generic URI syntax. Specifically section 3.3 for Paths.

For multiaddr components that use multibase, the generator of the URI string should ensure the multibase used is valid as a path segment.


[PE-D][Tester B] Incorrect  Getting Started command

Labels: severity.Low type.DocumentationBug
original: JunLongling/ped#2

Will be solved in v1.6


Pre Trained Weights

Kindly someone provides the pretrained weights. As they put the link option but still not provides the weights.

Thanks for the request. We have not released the weights yet but it is planned with no timeline unfortunately. I will leave this open and comment once uploaded.

thank you, looking forward to getting the update soon.

I just uploaded the models to the models/ folder via lfs.

thank you so much, I really appreciate that


Laravel 5.3 is dirty

Thanks! Lets just wait on the tests to pass.

Thanks!

@vinkla Perhaps time for a new release?

@torkiljohnsen yeah, sorry about the delay. The tests are still failing.


[Question] Test retries and HTML reporter for .Net?

Hi,
Are there any plans to add the 2 Node.js features test retries and the HTML reporter to .Net? Or do you have a work around to create something similar when you run Playwright in .Net?
Regards,
Johnny

HTML reporter is only available in Playwright Node.js test runner. @mxschmitt would now better about test retries.

Test retries feature request is here: https://github.com/microsoft/playwright-dotnet/issues/2316


Has anyone been able to start a topic using Docker Compose?

I attempted to put
kafka:
    image: spotify/kafka
    ports:
      - "9092:9092"
      - "2181:2181"
    hostname: kafka
    expose:
      - "9092"
      - "2181"
    environment:
      TOPICS: test-topic

But this failed to work, is there something that I'm missing? (I tried with quotes around the topic as well)
Also, if anyone has had any luck using Spring to connect to a topic, I would greatly appreciate some help connecting to the topics.

TOPICS env is for spotify/kafkaproxy image.

kafka:
  image: spotify/kafka
  ports:
  - "2181:2181"
  - "9092:9092"
  hostname: kafka
  container_name: kafka
  environment:
  - "ADVERTISED_HOST=kafka"
  - "ADVERTISED_PORT=9092"
  - "AUTO_CREATE_TOPICS=true"
  command: >
      bash -c
      "(sleep 15s &&
      /opt/kafka_2.11-<IP_ADDRESS>/bin/kafka-topics.sh
      --create
      --zookeeper
      localhost:2181 --replication-factor 1 --partitions 1
      --topic dummy_topic &) && (supervisord -n)"

@terrabl, you can do it like this.

Hi there, I have a question.
How can I create producer and consumer in "command" section like this topic was created?

Hi there, I have a question.
How can I create producer and consumer in "command" section like this topic was created?
Refer below link:
https://gist.github.com/abacaphiliac/f0553548f9c577214d16290c2e751071

AUTO_CREATE_TOPICS

Is this variable really needed? I've managed to get my topic created even without this variable. Could you please explain reason why we need it?

I did a NuGet package update and that solved my issue


Fixed compilation when compiling with no std

When compiling with features = ["core"] (without std), there was an error.
This PR fixes it :)

Thank you for the patch.


Update error message

There are cases where kubectl warnings appear as errors. In this case the component installs correctly, so in general it's better not to assert in the output that the component install failed.
Eventually, we will want to integrate a post install health check into apply, in which case this type of message will be more appropriate.

/cherrypick release-1.4

@ostromart: new pull request created: #606

In response to this:

/cherrypick release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


feat: cached file is saved directly without need of moving the file

I replaced the steps of saving and moving the file with only one step of saving the file directly using file_put_contents().

Hi @marek-mikula, thanks for the contribution! 🙂
However the rename was done on purpose, to avoid concurrent writing to the cache file, so this must stay; the temporary file can still be created in the configured cache directory though.

Just done some changes that better fit how the system was supposed to work. Thanks for the initial work @marek-mikula!
closes #65

@romm Hello, I am really sorry, I actually totally forgot about this. Thank you for your work! I really appreciate it!


Different font

How can I use a different font from the default?

There are at least two ways I can think of that will accomlish this:
1. Quick and dirty
Add this after loading mini.css:
html, * {
  font-family: your-font-here;
}
/*
  Use this to keep the default font styling for code elements
  or replace the font-family as well
*/
code, kbd, pre, samp {
  font-family: monospace, monospace;
}

This basically overrides the default fonts, so you don't have to mess with flavor files etc.
2. The right way
Edit the flavor file you are using (e.g. default flavor file). There is a line with the variable $base-font-family which you can replace to use the font-family of your choice. You can either @import the font file (e.g. as seen on the Sucroa flavor file) or include it through your HTML page. Remember to compile the flavor file with a Sass preprocessor to turn it into a proper CSS file.


.

I also have this problem.
Blender 3.5

I downgraded to blender 4.1. It works

The fix is in the development branch. You can download the fixed version from this link. The development branch version should work with 3.5.

DEV version does not have eye tracking so it is not useful


Document Best Practices of Security for NuGet

https://www.youtube.com/watch?v=UTPct8FKu8I
https://www.youtube.com/watch?v=l2epzyytPGE
Create a cool suite of docs to talk about security from NuGet's perspective.

Completed via https://github.com/NuGet/docs.microsoft.com-nuget/pull/2309


Display error when Markdown file has special characters

Description
When there are special characters in the Markdown file, the display is wrong, after removing the special characters, the display is normal
This repository is on github at https://github.com/firstcontributions/first-contributions
Gitea Version
v1.17.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
2.37.3
Operating System
Windows 10
How are you running Gitea?
./gitea web

Database
MySQL

maybe fixed in rencently patch, can you reproduce in 1.18.0-rc1 or 1.19.0-dev

No such problem in v1.18.0-rc1


Container Image Distribution

It must be great if we distribute LPVS container image from the registry.
Which registry do you prefer? Docker Hub? or Github Container Registry?

@t25kim
It's a good idea. Let's use the same approach as for the LF Home Edge project.
@tdrozdovsky  could you please share your opinion?

When it comes to LF Edge, the Dockerhub is complimentary since its host organization is Open Source Initiative (The Linux Foundation). It would be good for us if we could refer to the same when it comes to the project hosted by Samsung Electronics

@t25kim
We are working on this ticket.
We will prepare the GitHub Actions auto-deploy script.

@tdrozdovsky
Please share the current progress with us.


update docs for v1 api

review pls, if I missed or misinterpreted something
@shinima, @Andarist

addressed comments, pls have a look again


Support for Oracle Wallets

When using Oracle Wallets the user and password is no longer provided to the JDBC driver.
I'm using vesion Quartz-2.2.3 where this is implemented in PoolingConnectionProvider.
I've noticed, in the latest version, it was changed to an interface and implemented in 2 providers.
Anyway, it would be great if the PoolingConnectionProvider would avoid setting user/password and just use the JDBC URL.
As an example of what needs to be done (based on 2.2.3 source code):
public PoolingConnectionProvider(Properties config) throws SchedulerException, SQLException {
        PropertiesParser cfg = new PropertiesParser(config);
        initialize(
            cfg.getStringProperty(DB_DRIVER),
            cfg.getStringProperty(DB_URL), //not defaulting to empty string
            cfg.getStringProperty(DB_USER), //not defaulting to empty string
            cfg.getStringProperty(DB_PASSWORD),
            cfg.getIntProperty(DB_MAX_CONNECTIONS, DEFAULT_DB_MAX_CONNECTIONS),
            cfg.getIntProperty(DB_MAX_CACHED_STATEMENTS_PER_CONNECTION, DEFAULT_DB_MAX_CACHED_STATEMENTS_PER_CONNECTION),
            cfg.getStringProperty(DB_VALIDATION_QUERY),
            cfg.getBooleanProperty(DB_VALIDATE_ON_CHECKOUT, false),
            cfg.getIntProperty(DB_IDLE_VALIDATION_SECONDS, 50),
            cfg.getIntProperty(DB_DISCARD_IDLE_CONNECTIONS_SECONDS, 0));
    }

And in initialize() method something like:
                if (dbUser != null) {
			datasource.setUser(dbUser);
		}
		if (dbPassword != null) {
			datasource.setPassword(dbPassword);
		}

Regards,
Constantin.

Thanks for providing the feedback. We will add this as feature request into the backlog.


twisted 11 compat

buildbot supports twisted 11, and there is a silly import error with autobahn-python:
https://travis-ci.org/buildbot/buildbot/builds/54691826
File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/autobahn-0.10.1-py2.6.egg/autobahn/twisted/util.py", line 30, in
from twisted.internet.address import IPv4Address, IPv6Address, UNIXAddress
exceptions.ImportError: cannot import name IPv6Address

@tardyp should work ..

when do you intend to create a release including this fix?
I think we should wait this release before pushing https://github.com/buildbot/buildbot/pull/1603

@tardyp Alright, released: https://pypi.python.org/pypi/autobahn/0.10.2


Register execution environment information

This PR will allow us to get a range of information on the execution environment of snarkOS. This will be useful for the dev/testnets, as well as bug reports.
Filing it as a draft, as more details might be added; we'll also need a way of extracting this information, either via a runtime argument or (more likely) an API call.

So you get this info and store it in the struct. Then what? Or am I missing something?

@joske you're 100% right, hence the draft - the PR is pending feedback regarding how this info is to be obtainable (HTTP API / runtime argument / both).

Merged as part of a staging proposal.


Build trains reviewAccountRequires improperly set by Apple server side

Related to #4241

@KrauseFx ping in case you didn't see my notes on the original issue.

Looks great, thanks! Pushing a new release now


My JSON is invalid, if I follow the guide

latest version as of today, I have copied the JSON text from here and trying to insert it into my almost empty homebrigde JSON config file (empty as this is will be my only homebrigde plugin.
I get a invalid json file, when I try to save.
It needs to be put into here
{
"bridge": {
"name": "Homebridge XXXX”,
"username": “XXXXXXXXX,
"port": 51464,
"pin": “XXX”-XX-XXXX,
"advertiser": "bonjour-hap"
},
"accessories": [],
"platforms": [
{
"name": "Config",
"port": 8581,
"platform": "config"
}
]
}

latest version as of today, I have copied the JSON text from here and trying to insert it into my almost empty homebrigde JSON config file (empty as this is will be my only homebrigde plugin.
I get a invalid json file, when I try to save.
It needs to be put into here
{
"bridge": {

    "name": "Homebridge XXXX”,

    "username": “XXXXXXXXX,

    "port": 51464,

    "pin": “XXX”-XX-XXXX,

    "advertiser": "bonjour-hap"

},

"accessories": [],

"platforms": [

    {

        "name": "Config",

        "port": 8581,

        "platform": "config"

    }

]

}

there are a couple of issues in the config you posted:
{
"bridge": {
"name": "Homebridge XXXX”,
"username": “XXXXXXXXX`"`,
"port": 51464,
"pin": “XXX~~”~~-XX-XXXX`"`,
"advertiser": "bonjour-hap"
},
"accessories": [],
"platforms": [
{
"name": "Config",
"port": 8581,
"platform": "config"
}
]
}

i’m on my mobile though so i don't know if the above code will render properly (i've seen it not do so previously)

Yeah, check your quotation marks, as indicated by Arthur.
Numbers have no quotation marks and all others have one in front and one in the back.
Also, " is not the same as “.  The one to use is "

I have tried it on Windows and Mac. Basically I copy your JSON, correct it with my info and put it into Homebrigde json config. No matter how I try to paste the example JSON into it, it gives me an error with invalid JSON.

This is the working JSON Config I have tried to edit

Try line 14:
},

Basically I'm saying, the instructions don't work. I'm installing it on a newly installed Homebrigde, with no plugins installed.
The instructions of how to insert the "we connect" JSON section into my existing JSON config, is unclear for a guy like me, who never edits a homebrigde config, since I use home assistant and tinker with my home server.
I would love to get your product up and running, but I have tried to insert the code with my user, VIN, gps locations and so on, but it won't save in my homebrigde config. I have been running VWs friend earlier.
I think the problem is, that it is a bit unclear for me, where the JSON section is supposed to go. Is it at the end of the config (doesn't work. Do I insert it under accessories, but again it doesn't work.
Thanks for the help so far.

I'm following this guide, and set it in like the blue square....that doesn't work
https://github.com/homebridge/homebridge/wiki/Homebridge-Config-JSON-Explained

ahh, i get it. Take the working config from above, put the closing bracket ], from line 9 on a new line and paste the following in between those brackets:
    {
            "name": "We Connect ID",
            "platform": "WeConnectID",
            "weconnect": {
                "username": "We Connect username",
                "password": "We Connect password",
                "vin": "VIN OF YOUR CAR"
            },
            "options": {
                "logLevel": "NONE",
                "apiPort": 1234,
                "chargingAccessory": "Charging",
                "climatisationAccessory": "Climatisation",
                "remainingRangeAccessory": "Remaining km",
                "destinations": [
                    {
                        "name": "Home",
                        "lat": 52.123456,
                        "lon": 5.123456,
                        "address": {
                            "country": "Nederland",
                            "street": "My street 4",
                            "zipCode": "1234 AA",
                            "city": "My town"
                        },
                        "notificationRadius": 300
                    },
                    {
                        "name": "Work",
                        "lat": 52.234567,
                        "lon": 4.234567,
                        "address": {
                            "country": "Nederland",
                            "street": "Cool street 75",
                            "zipCode": "2345 BB",
                            "city": "Your city"
                        },
                        "notificationRadius": 200
                    }
                ],
                "settingSwitches": [
                    {
                        "name": "Reduced AC",
                        "setting": "reducedAC"
                    },
                    {
                        "name": "Plug Unlock",
                        "setting": "autoUnlockPlug"
                    },
                    {
                        "name": "Climatization at unlock",
                        "setting": "climatizationAtUnlock"
                    },
                    {
                        "name": "Window Heating",
                        "setting": "climatisationWindowHeating"
                    },
                    {
                        "name": "Front Left Zone",
                        "setting": "climatisationFrontLeft"
                    },
                    {
                        "name": "Front Right Zone",
                        "setting": "climatisationFrontRight"
                    }
                ],
                "eventMotionSensors": [
                    {
                        "name": "Car unsafe state",
                        "event": "statusNotSafe"
                    }
                ]
            }
        }

so your config as of line 9 looks like:
"accessories": [
----->the part of code pasted above
],
"platforms": [
----->etc.

I'm following this guide, and set it in like the blue square....that doesn't work
https://github.com/homebridge/homebridge/wiki/Homebridge-Config-JSON-Explained

... And if my suggestion above doesnt work, try to paste the code from the previous post under the platforms section (the purple section in your link)..

Thanks for your help so far....your new section works, but now I get a new error from Homebrigde.

try moving the block (with the redline in the previous post) to the platforms section of the config. So adjust "platforms" line 10~16 to this:
"platforms": [
   {
   "name": "Config",
   "port": 8581,
   "platform": "config"
   },
   ---> insert configuration block here. Note the comma on the previous line
]

Now we are trough the JSON config faults, and Homebrigde have accepted the JSON :)
Now I'm just waiting for the devices to pop up. No errors in loading.

It works....thanks for your help :)

Good to hear you got it working. I will improve the installation instructions with the next update.


Added new tests SmtpTransportUnitTests and Pop3StoreUnitTests

Please review and advice

Thanks for taking the time to submit a PR. However there have been many changes to the networking code recently, and our test infrastructure has changed. Your PR also mixes tests code refactors. If possible it would be good to add just the tests in one PR, then the refactor in a separate one.


[WIP] superenv: set effective_sysroot on 10.14 CLT

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

I'm mostly opening this for documentation. It's probably too early in the cycle to start committing workarounds like this.
The current version of the CLT doesn't include headers in the traditional Unixy locations, which breaks some software and which means that we end up not setting some special include paths. I've filed this bug as rdar://40833474, but meanwhile this workaround fixes broken builds like imagemagick.
If the traditional Unix locations aren't coming back, we'll have to look at wider-reaching changes, but I'll want to let more time pass before looking at something like that.
See also https://github.com/Homebrew/homebrew-core/issues/28703 and https://github.com/Homebrew/homebrew-core/pull/28733.

FWIW, the 10.14 release notes make mention of the missing headers and provide a package for installing them in /usr/include, this is an additional step after installing CLT.

The Command Line Tools package installs the macOS system headers inside the macOS SDK.
Software that compiles with the installed tools will search for macOS headers at the standard
include path:
/Applications/Xcodebeta.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include
For legacy software that looks for the macOS headers in the base system under /usr/include, please
install the package file located at:
/Library/Developer/CommandLineTools/Packages/macOS_SDK_headers_for_macOS_10.14.pkg

😭

@andrewpile Yeah, I'm planning on replacing this PR with one that diretly addresses that... thanks!

rdar://40833474
How to open that link $_%?


Fixed confusing lowercase letters

A new developer might think there is a meaning to the words being uppercase when it is not needed
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Internal previews

📄 File
🔗 Preview link

docs/fundamentals/code-analysis/quality-rules/ca2254.md
CA2254: Template should be a static expression

I'm not sure I agree with making the placeholder names lower case. In most examples, the placeholder names start with an uppercase character. I think this is because the placeholder names are sent to the log file along with the values. By making these lower case, the reader might think they need to match the argument name, which isn't true. @IEvangelist - thoughts?

I'm not sure I agree with making the placeholder names lower case. In most examples, the placeholder names start with an uppercase character. I think this is because the placeholder names are sent to the log file along with the values. By making these lower case, the reader might think they need to match the argument name, which isn't true. @IEvangelist - thoughts?

Yes, I agree with @gewarren here. In fact, by convention they should be Pascal cased. Although it's not mentioned here, this is still what's expected, and the way it should remain.


[Bug] Cannot connect external TURN Server via Livekit

Describe the bug
Hi @cloudwebrtc, I have built my own TURN server before so I use it instead of embedded Livekit provide. I tested my TURN Server with Trickle Ice Test and it works. But I run with Livekit Connection Tester, it shows Cannot connect via TURN.
Server

Version: [1.7.0]
Environment: Docker, VM
Deploy follow docs

To Reproduce
Steps to reproduce the behavior:

Setup server follows docs
Test with Livekit Connection Tester
Setup rtc.turn_servers
See error

My Livekit.yaml configuration
port: 7880
rtc:
  tcp_port: 7881
  port_range_start: 50000
  port_range_end: 60000
  use_external_ip: true
  turn_servers:
   - host: redacted
     port: 443
     protocol: tls
     username: redacted
     credential: redacted
redis:
  address: localhost:6379
  username: ""
  password: ""
  db: 0
logging:
  level: debug
keys:
  redacted
webhook:
  api_key: redacted
  urls:
    - redacted

Expected behavior
Can connect to external TURN Server through Livekit.
Screenshots
Tested with Livekit Connection Test on Chrome, Safari. And with Flutter Client.

Folks are using third-party TURN servers like coTURN without issues. Whatever you set is basically passed along to the clients.

When I update Flutter SDK and Livekit to v2.2.0, all problems solved. Thank you!


Rename config to variables

Renames environment variables SPIN_CONFIG_* to SPIN_VARIABLE_* for the EnvVariablesProvider.
Renames runtime config manifest key from config_provider to variable_provider.
Moves crates/config to crates/variables.
All tests/examples updated.

On a less important note, the environment runtime variable provider is now confusingly named EnvVariableProvider. Open to naming suggestions if this irks others as well.

:bike::derelict_house:: VariablesProvider?


feat: added new slides

⚡ Deploying PR Preview c877721f495d04b2969a5890325b6a837ae59756 to surge.sh ... Build logs

🤖 By surge-preview


chore: Update GetSupportedNodeClasses to return []status.Object

Fixes #N/A
Description
This PR adds a change to return []status.object from GetSupportedNodeClasses
How was this change tested?
Tested on local cluster
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 8 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 78.295%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cloudprovider/cloudprovider.go
0
4
0.0%

pkg/fake/cloudprovider.go
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cloudprovider/cloudprovider.go
1
73.28%

pkg/fake/cloudprovider.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
5786

Relevant Lines:
7390

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 78.434%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cloudprovider/cloudprovider.go
0
2
0.0%

pkg/fake/cloudprovider.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cloudprovider/cloudprovider.go
1
73.88%

pkg/fake/cloudprovider.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
5801

Relevant Lines:
7396

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 78.749%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cloudprovider/cloudprovider.go
0
2
0.0%

pkg/fake/cloudprovider.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cloudprovider/cloudprovider.go
1
73.88%

pkg/fake/cloudprovider.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5940

Relevant Lines:
7543

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 78.749%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cloudprovider/cloudprovider.go
0
2
0.0%

pkg/fake/cloudprovider.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cloudprovider/cloudprovider.go
1
73.88%

pkg/fake/cloudprovider.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5940

Relevant Lines:
7543

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 78.842%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cloudprovider/cloudprovider.go
0
2
0.0%

pkg/fake/cloudprovider.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cloudprovider/cloudprovider.go
1
73.88%

pkg/fake/cloudprovider.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5951

Relevant Lines:
7548

💛  - Coveralls


csv.Dialect parameters(delimiter, quotechar, lineterminator) cannot be in bytes format.

I faced this issue while using splunklib on python 3.
version: 1.6.0
platform: Linux(centos)
framework version: Python 3.7.2
Splunk version: 8.0.0 (build: 7f1fa0caa96a)(beta build for testing)
Following is the trace of error.

2019-07-03 14:25:39,156, Level=ERROR, Pid=2240, File=search_command.py, Line=981, TypeError at "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/splunklib/searchcommands/validators.py", line 270 : "delimiter" must be string, not bytes
Traceback:
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/splunklib/searchcommands/search_command.py", line 603, in _process_protocol_v1
self._execute(ifile, None)
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/splunklib/searchcommands/generating_command.py", line 208, in _execute
self._record_writer.write_records(self.generate())
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/splunklib/searchcommands/internals.py", line 521, in write_records
for record in records:
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/ldaptestconnection.py", line 53, in generate
configuration = app.Configuration(self)
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/app/configuration.py", line 54, in init
self._read_configuration()
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/app/configuration.py", line 447, in _read_configuration
self._reset_fields(command.domain, settings)
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/app/configuration.py", line 480, in _reset_fields
host = self._get_value(settings, 'server', require=True, validate=List())
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/app/configuration.py", line 398, in _get_value
value = validate(value)
File "/opt/splunk/etc/apps/SA-ldapsearch/bin/packages/splunklib/searchcommands/validators.py", line 270, in call
value = next(csv.reader([value], self.Dialect))

I looked into it, I think that below code snippet is causing this issue:
class Dialect(csv.Dialect):
    """ Describes the properties of list option values. """
    strict = True
    delimiter = b','
    quotechar = b'"'
    doublequote = True
    lineterminator = b'\n'
    skipinitialspace = True
    quoting = csv.QUOTE_MINIMAL

I changed it to a below code to make it work. Is it okay to make the following change or there is more change required?
class Dialect(csv.Dialect):
    """ Describes the properties of list option values. """
    strict = True
    delimiter = ','
    quotechar = '"'
    doublequote = True
    lineterminator = '\n'
    skipinitialspace = True
    quoting = csv.QUOTE_MINIMAL

@bj8798 thanks for reporting the bug, can you try upgrading to the latest SDK?

@bj8798  what version of SA-ldapsearch are you using?

Hi @xuchen509, we are currently working on the migration of SA-ldapsearch v2.2.1 to python 3.  For that, we have updated the splunklib(used in SA-ldapsearch) with the latest version(v1.6.6) and facing this issue.

fixed in https://github.com/splunk/splunk-sdk-python/pull/259. Thank you!


Fix result UI issue when selection result textview line is 2.

Fix result UI issue when selection result textview line is 2.

Resovled: https://github.com/Park-SM/ParkDateTimePicker/pull/17


Fail to get spiffe-token-provider token for deplayed-start services

🐞 Bug Report
Affected Services [REQUIRED]

 The issue is located in: deplayed-start services
Is this a regression?

 Yes
Description and Minimal Reproduction [REQUIRED]

Generate deplayed-start compose file with command make gen deplayed-start ds-virtual on edgex-compose repo.
Execute docker compose up -d
Found error on services, device-virtual, support-notifications and support-scheduler

🔥 Exception or Error

level=INFO ts=2023-05-24T07:07:07.321975901Z app=device-virtual source=secret.go:79 msg="Reading secret store configuration and authentication token"
level=INFO ts=2023-05-24T07:07:07.322029647Z app=device-virtual source=secret.go:210 msg="runtime token provider enabled"
level=INFO ts=2023-05-24T07:07:07.322039323Z app=device-virtual source=methods.go:138 msg="using Unix Domain Socket at unix:///tmp/edgex/secrets/spiffe/public/api.sock"
level=INFO ts=2023-05-24T07:07:07.391092447Z app=device-virtual source=methods.go:150 msg="workload got X509 source"
level=WARN ts=2023-05-24T07:07:07.41341202Z app=device-virtual source=secret.go:132 msg="Retryable failure while creating SecretClient: failed to get spiffe-token-provider gettoken api call, status code = 404 "

🌍  Your Environment
Deployment Environment: Ubuntu 2204 / x86
EdgeX Version [REQUIRED]: Main
Anything else relevant?

@bnevis-i this issue failed the TAF tests, but it worked fine before yesterday.
could you please help check whether this PR failed anything?
https://github.com/edgexfoundry/edgex-go/pull/4573/files

I just finished regressing the change.  Indeed this is the fix.

Issue was fixed.


Add a time-to-live to the kubernetes-credential manager

The kubernetes-credential-manager (KCM) is now run from the standard sixsq/nuvlaedge:${TAG} (whatever the current TAG is).
When a helm upgrade is called to move to another $TAG i.e. in the case of testing a pull request via a $TAG built from a branch, this causes a problem.
The KCM is in a "Completed" state with a known image (as described above).
The helm update attempts to replace the KCM image with the branch $TAG and fails as this Job is completed and considered immutable by Kubernetes.
This is the correct kubernetes behaviour.
But the upgrade fails and it becomes problematic to test branches.
We need to add a time-to-live (TTL) to the KCM Job so that after TTL the job is completed it is removed.

[ ] does this TTL need to consider the validity time of the certificate request?
[ ] Add the TTL;
[ ] Test.

Looks like we're there... deployed using the deployment branch issue-134-ttl.
The kubernetes-credential-manager is removed after completion (5s configurable).
Subsequent helm upgrade with the issue-81 version of nulvaedge went fine. No errors.
Check the YAML output of the job to confirm that the new image is used.


"Can't initialize GTK backend in function 'cvInitSystem'"

Hi all,
I was following the instructions from here to flash the VCK190 image to an SD card, and then the instructions from here to run the resnet50 example. However, I error out with the following message:
root@xilinx-vck190-20221:~/Vitis-AI/examples/VART/resnet50# ./resnet50 /usr/share/vitis_ai_library/models/resnet50/resnet50.xmodel
WARNING: Logging before InitGoogleLogging() is written to STDERR
I1207 15:21:42.705144   976 main.cc:292] create running for subgraph: subgraph_conv1
XAIEFAL: INFO: Resource group Avail is created.
XAIEFAL: INFO: Resource group Static is created.
XAIEFAL: INFO: Resource group Generic is created.

Image : 001.jpg
top[0] prob = 0.982662  name = brain coral
top[1] prob = 0.008502  name = coral reef
top[2] prob = 0.006621  name = jackfruit, jak, jack
top[3] prob = 0.000543  name = puffer, pufferfish, blowfish, globefish
top[4] prob = 0.000330  name = eel
terminate called after throwing an instance of 'cv::Exception'
  what():  OpenCV(4.5.2) /usr/src/debug/opencv/4.5.2-r0/git/modules/highgui/src/window_gtk.cpp:624: error: (-2:Unspecified error) Can't initialize GTK backend in function 'cvInitSystem'

Aborted

I am using minicom to communicate with my VCK190 platform. Based on this issue from the OpenCV repository, it seems like it's an X11 issue. However, since I'm using minicom, I'm not really sure how to proceed here. Any help is appreciated!

I don't think minicom can support X11 show.
Please use other port such as DP or HDMI.
Or for the aim to run the code, you can comment out the opencv part.

Closing since no activity for more than 2 weeks, please reopen if you still have any questions, thanks

Hi all,
I was following the instructions from here to flash the VCK190 image to an SD card, and then the instructions from here to run the resnet50 example. However, I error out with the following message:
root@xilinx-vck190-20221:~/Vitis-AI/examples/VART/resnet50# ./resnet50 /usr/share/vitis_ai_library/models/resnet50/resnet50.xmodel
WARNING: Logging before InitGoogleLogging() is written to STDERR
I1207 15:21:42.705144   976 main.cc:292] create running for subgraph: subgraph_conv1
XAIEFAL: INFO: Resource group Avail is created.
XAIEFAL: INFO: Resource group Static is created.
XAIEFAL: INFO: Resource group Generic is created.

Image : 001.jpg
top[0] prob = 0.982662  name = brain coral
top[1] prob = 0.008502  name = coral reef
top[2] prob = 0.006621  name = jackfruit, jak, jack
top[3] prob = 0.000543  name = puffer, pufferfish, blowfish, globefish
top[4] prob = 0.000330  name = eel
terminate called after throwing an instance of 'cv::Exception'
  what():  OpenCV(4.5.2) /usr/src/debug/opencv/4.5.2-r0/git/modules/highgui/src/window_gtk.cpp:624: error: (-2:Unspecified error) Can't initialize GTK backend in function 'cvInitSystem'

Aborted

I am using minicom to communicate with my VCK190 platform. Based on this issue from the OpenCV repository, it seems like it's an X11 issue. However, since I'm using minicom, I'm not really sure how to proceed here. Any help is appreciated!

I faced the same issue when working with KV260. You need to use:
export DISPLAY=:0
if you're working directly with the display output of the platform, otherwise use
export DISPLAY=[your_ip_address]:0.0
if you’re working remotely.
I'm astonished at the incompetence of Xilinx support.
Regards!


The evaluation of COCO14 segmentation masks

Hi, thanks for your great work.
Your SIPE achieves considerable performance in COCO14 val set.  I wonder how to obtain the image GT for COCO14 so that I can calculate miou as done in https://github.com/chenqi1126/SIPE/blob/main/eval_cam.py#L52 . As the COCO official only provides the .json file, can you please tell me how to translate the .json to .png?
Thanks.

Hi @mt-cly,
Thanks for your attention. You can use 'pycocotools' to convert annotation to png. An off-the-shelf code: https://github.com/zhaozhengChen/ReCAM/blob/main/mscoco/annToMask.py.

Thanks for your reply.

I meet another question, I directly use your provide deeplab_r38_coco.pth on the modified code of  seamv1, but I can only achieve around 36% mIoU, (+CRF ~37% mIoU), still has a large gap with reported 43.6%, I have no idea about that, have you any suggestions? Or can you provide the evaluation code for COCO14 val set? Thanks.


[TW#20523] removing "phy: phy_version: 383.0, 79a622c, Jan 30 2018, 15:38:06, 0, 0" debug (IDFGH-2043)

Hi,
I'm not able to remove the
phy: phy_version: 383.0, 79a622c, Jan 30 2018, 15:38:06, 0, 0
information debug. It seems in libphy.a library. This debug is always printed even if all the logs are disabled at very startup of app_main() using function
esp_log_level_set("*", ESP_LOG_NONE);
Can you please remove it?
Thanks.

Hi @FayeY,
Is there any update regarding this issue?
Thanks

@davctv You can substitute function in phy_printf(const char* format, ...) with an empty function in components/esp32/lib_printf.c to solve your problem.

@TimXia Thank you. I will do a test and let you know.

@TimXia your suggestion worked. I close the issue. Thanks

This should not be closed i think, and should be a Bug. Reproduced with esp-idf-v3.3

We have a fix under internal reviewing and the issue will close automatically once the fix is available on GitHub, thanks.


Release v3.0.0

Waiting for the Godot 4.0-beta3 release to fix an important C# signal bug. In the meantime, updating docs and writing release notes.

https://godotengine.org/article/dev-snapshot-godot-4-0-beta-3
Let's do this :D


https://huggingface.co/datasets/ccmusic-database/vocal_range/discussions/1

Could HF please print more error messages with code line for our own codes instead of official framework message? I t would be hard for us to debug when some error like this happens

closing as the dataset does not exist anymore.


[all-docker-all-the-time] Use our base Docker image in CI

This uses our base image, but it doesn't use project's image. Maybe it should? Still a WIP.


Adds the clear inventory command.

Adds the clear inventory command, where you can clear your or another player's inventory.
Has the aliases of "/clear (player)", "/clearinventory (player)", "/ci (player)", and "/clearinv (player)".
Adds/Fixes #51

After this pull request, I will be using the name @Meronat


podcast:location has contradictory text

The spec says both of these things:

Channel (optional | single)

This element is required at the channel level.

@daveajones - this should be mantatory, correct?
Good catch, @mattbasta

No we dropped it to be optional all around. I’ll fix that. Thanks for the bug report.


Make the Value indexable

This PR does just what the title says
With this PR, operations like below are made possible:
let value:Value = deserialize(bytes).unwrap();
let inner:Value = value["someItem"]["breads"][3];

Looks pretty good. I like the direction this pr is going forward.
I want some test cases, docs, and more impls like what serde_json has, so could you please be waited until I finish fixing up?

@temeddix
May I have a permission to push to your repository?

@nyuichi Of course :D I invited you as a collaborator, I will boost the permission after the invitation is accepted.

Mmm, seems like I've got into trouble regarding this 🤔  https://docs.github.com/en/codespaces/codespaces-reference/security-in-github-codespaces#working-with-other-peoples-contributions-and-repositories

Hmm, as far as I know repository maintainers can suggest PR changes by writing the code themselves

I managed to push it :)
@umadein Can you review it? If you're quite busy I'll leave it maybe for 3 days or so and merge it without approval then.

I will release 0.8 once this gets merged.


Ensured VCF file input for metasv.py is not None.

metasv.py failed. After some debugging, I could see that both call["vcf_file"] and call["vrn_file"] were None. Therefore this line caused an exception:
cmd += ["--%s_vcf" % call["variantcaller"], call.get("vcf_file", call["vrn_file"])]

Thanks much for the fix. As a heads up, metasv integration hasn't gotten much work recently. Definitely happy to have you revive it and fix any issues if it's useful in your work. We typically use an approach of combining callers and prioritizing in regions of interest with svprioritize (https://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#structural-variant-calling) which avoids the overhead of recalling and recalulcating breakpoints. Hope this helps.

Thanks Brad. I was also read the paragraph about svprioritize when I was making configuration file. However, I did not understand what input file I should use. We have exome data and want to run SV detection as an additional step to variant calling. All of these items do not seem to fit to our case:

    cancer/civic (hg19, GRCh37, hg38) – Known cancer associated genes from CIViC.

    cancer/az300 (hg19, GRCh37, hg38) – 300 cancer associated genes contributed by `AstraZeneca oncology

    cancer/az-cancer-panel (hg19, GRCh37, hg38) – A text file of genes in the AstraZeneca cancer panel. This is only usable for svprioritize which can take a list of gene names instead of a BED file. <https://www.astrazeneca.com/our-focus-areas/oncology.html>`_.

    actionable/ACMG56 – Medically actionable genes from the The American College of Medical Genetics and Genomics

Vang -- if you want SVs in all exons than you're right that svprioritize is not especially useful and the best approach is to look at the individual callers. Manta and CNVkit are good first passes to identify events, and Lumpy and WHAM are useful if you need more specificity. svprioritize would be useful if you have a subset of genes of interest and want to dig into the additional specificity only in those. Most of the current pre-built inputs are cancer based but if none of these fit your needs you can use a BED file of genes of interest. Hope this helps.


Save button too small

The save button is too small on registration

Resolved with ng-style="{'width': '50px'}"


OASIS StyleGAN - S4641799

Task
Create a generative model of one of the OASIS brain, ADNI brain or the OAI AKOA knee data set (see Appendix for links) using either a variant of StyleGAN /StyleGAN2 or Stable Diffusion that has a “reasonably clear image”.
In this project, Stable Diffusion was experimented with, but StyleGAN was the final implementation on the OASIS brain dataset. Stable Diffusion wasn't used due to a lack of familiarity with the model type.
Project Files

dataset.py: Data loader for the OASIS brain dataset, transforms the data for use with the model.
modules.py: The StyleGAN networks as well as the layers and blocks they use.
predict.py: Generates a batch of fake images that can be compared to the original dataset.
train.py: Training process for the StyleGAN networks, will also generate example images and graph the loss function.
utils.py: Functions used by the other scripts and global variables.

Results
By epoch 150 I consider the model a success, however, it was starting to generate repeated and similar images. Training the model any longer would most likely cause the model to become overtrained and collapse the training simulation.

New commits were added to address:

Noise injection image not appearing
Adding noise input examples to the readme
Improving and adding comments to the python files
Updating the Readme content to include justification for why StyleGAN was chosen
Updating the Readme content to include preprocessing done on training images
Updating the readme content to fix minor formatting and style issues

If any more changes are required please let me know, but I believe this addresses all the issues raised in the feedback.

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting minimal -1
No Data leakage
Difficulty: Hard
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe acceptable/good
Model/technical explanation
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good

Cannot merge because of gitignore file and incorrect name of folder, not stable diffusion but Style GAN, please amend. Does not affect grade only the merge.

Feedback marks possible +2 if the requested changes are made (see above).

No feedback attempt and no feedback marks granted.

Sorry, I didn't receive a notification about these changes and as semester had ended was no longer actively checking for comments. I will work on these changes now. @shakes76 and @wangzhaomxy

Addressed issues relating to the pull request's branch

Feedback complete and an issue was raised on edstem: https://edstem.org/au/courses/13207/discussion/1726226

Feedback attempt and feedback marks granted +2.


Improvements in e2e_tests

move all 'source'ing to the beginning. Fixes the issue of 'info_message' being not available since utils.sh is not sourced at the point it's used
change control sequence for terminal colors from \e to \033. This has wider support. \e is breaking on atleast some Macs

/lgtm
/approve


cmd/go/internal/get: stdout does not end in new line

What version of Go are you using (go version)?

[user@localhost ~]$ go version
go version go1.16 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
[user@localhost ~]$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOENV="/home/user/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY="*.sc-corp.net"
GONOSUMDB="*.sc-corp.net"
GOOS="linux"
GOPATH="/home/user/go"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/user/.local/share/umake/go/go-lang"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/user/.local/share/umake/go/go-lang/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/tmp/test/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build698062687=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Create the following go.mod file:
module test

go 1.16

require k8s.io/client-go v12.0.0+incompatible

Create the following main.go file:
package main

import (
        "fmt"

        "k8s.io/client-go/pkg/version"
)

func main() { fmt.Println(version.Get()) }

go mod download<EMAIL_ADDRESS>invalid version: +incompatible suffix not allowed: module contains a go.mod file, so semantic import versioning is required
[user@localhost trash]$ go get<EMAIL_ADDRESS>go get<EMAIL_ADDRESS>invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v12
[user@localhost trash]$ go run .
main.go:6:2: missing go.sum entry for module providing package k8s.io/client-go/pkg/version (imported by test); to add:
        go get test
[user@localhost trash]$ go get test

What did you expect to see?
go: warning<EMAIL_ADDRESS>retracted by module author: pre-module versions are obsolete
go: to switch to the latest unretracted version, run:
        go get k8s.io/client-go@latest
[user@localhost trash]$

What did you see instead?
The go get is ending the printed string without a newline:
go: warning<EMAIL_ADDRESS>retracted by module author: pre-module versions are obsolete
go: to switch to the latest unretracted version, run:
        go get k8s.io/client-go@latest[user@localhost trash]$

Duplicate of #44674

(This should be fixed in 1.16.2: #44676)

yeah, just found the fix as I was in the process of making a cl


Images and links need to be patched by Jupyter

Currently the images to local paths are broken, these need to be properly patched by Jupyter. This is also true for links. We will have to go into the internals as we are taking over the rendering of the markdown cells entirely.
Some pointers from @agoose77:

jupyterlab-markup integrates into JupyterLab's renderermime by exposing a markdown renderer, so JupyterLab takes the generated HTML and post-processes it. If you want to avoid generating HTML fragments and doing that, you can access the same interfaces that  JupyterLab's using

renderers

The URLs are patched here:

url patching

This private function uses these:

private function

Very cool, huge thanks @rowanc1!! I also see that the double-title problem is gone, much appreciated :)
When you fix links, I'd suggest rendering them similarly to the default in pre-existing JLab (blue, no underline), perhaps?
I'm thinking that for the "classic" markup where there's overlap with existing practice, it would help adoption in the community if things continue to work similarly. Here's a comparison (that shows the duplicate titles are now fixed):

Ah, good catch, thanks!! I didn't realize that, I'll remove the old one to leave only one of these extensions active.

Will be merged and fixed up shortly:

This has been released in 0.1.3, links work internally and externally, images were fixed in 0.1.2.
@fperez we also opted to change the behaviour to better deal with jupytext, as of v0.1.3 the default renderer of any markdown cell is MyST. This can be turned off by disabling the extension or uninstalling it. If you have any feedback on that, please let us know!

Awesome, thx so much @rowanc1!!
Was just a minute ago discussing this with @tsnow03 for using it in our CryoCloud hub. Your timing couldn't have been better :)
With the release of JupyterLab 3.6.x I'm also about to do an update of my Stat159 hub for next week, so this is perfect.
Pinging @facusapienza21 too.

Let me know how I can help out here @tsnow03, the extension is getting quite good! Needs more usage to find all the edge cases. :)

@rowanc1 I will! It looks super cool. We are excited to try it out soon.


Update DoD

This work is going into uncharted territory. Gatsby+Tina docs are lacking.
Update the DoD

[x] Ensure relevant Tina Gatsby docs are created/updated for each change
[x] Done Videos
[x] Report the bugs

The workflow will be

Figure something out
Document it
Report any TinaCMS bugs we find
Done

e.g.
Data fetching - do we do this in Gatsby-node.js? or somewhere else?
After this question is answered, create a new docs page explaining how to do data fetching
if we need to do a workaround, we will create a bug in the TinaCMS repo so it gets fixed

Done - This has been updated See the updated Dod https://github.com/SSWConsulting/SSW.Rules/blob/tina/main/README.md


Nodelet imgtransport feature

tested on single camera and stereo cam setup
tested nodes and nodelets
tested with april_tag nodelet in acquisition_nodelet_manager on both x86_64 and arm64(aarch64 jetson tx1)

@mithunvarma does the ReadME need to be updated with these changes?  I will do a quick test with the latest and merge it later today or tomorrow.

@shahvi Updated Readme and added launch files for nodelets

Looks good, couple little suggestions:

Change the acquisition.launch to aquisition_node or aquisition_standalone.launch
Change acquisition_nodelet.launch to acquisition.launch
Change  nodelet_test.launch to nodelet_subscriber_example.launch
Turn default behavior for acquisition.launch to launch a nodelet manager.
Add details of configuration for exposure and target grey levels to ReadME (dynamic reconf params)
exposure setting is duplicated in launch file and dynamic reconfigure, fix behavior to only use one of them.


Add tests for history decision task handler

What changed?
The following tests added in service/history/decision:
TestHandleDecisionStartTimer
TestHandleDecisionCompleteWorkflow
TestHandleDecisionFailWorkflow

Why?
improve code coverage in the package

How did you test it?
unit tests

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018fa6b3-4bdc-4962-b180-08089f1eee69
Details

0 of 0   changed or added relevant lines in 0 files are covered.
117 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.008%) to 69.342%

Files with Coverage Reduction
New Missed Lines
%

service/history/task/transfer_standby_task_executor.go
2
86.21%

service/matching/tasklist/db.go
2
73.23%

service/matching/tasklist/task_list_manager.go
2
76.09%

service/matching/tasklist/task_writer.go
2
82.63%

service/history/task/task.go
3
84.81%

common/log/tag/tags.go
3
50.46%

common/task/fifo_task_scheduler.go
3
84.54%

service/history/task/timer_active_task_executor.go
3
79.33%

common/persistence/nosql/nosql_task_store.go
5
63.97%

service/history/execution/cache.go
6
74.61%

Totals

Change from base Build 018fa5a0-da9b-4f76-9263-3511c2f681f9:
-0.008%

Covered Lines:
102165

Relevant Lines:
147336

💛  - Coveralls


FastImage causing Crash in IOS : Message from debugger: Terminated due to memory issue

Hi Dylan,
I am experiencing memory issues on using your module. Can you please take a look at this issue?

Regards,
Akhil Vasudev

Hi Dylan,
I will appreciate if you can look into this issue.Although the product solves all our requirements and the fetching speed and other features are very good but we are facing random crashes due to memory issues.
I hope that you will provide speedy solution.

Regards,
Akhil Vasudev

I'm closing the open issues related to memory so things are better organised. See: https://github.com/DylanVann/react-native-fast-image/issues/195


Interest in new data sources? (FMI)

Describe the feature
Just feeling the room, if there would be, in general, interest to add new data sources to Pirate Weather?
I'm Finnish myself and the Finnish Meteorogical institute seems to sometimes have more accurate data than Pirate Weather. They have an API. Would be very awesome to use that data via Pirate Weather because it's available to integrate to many more places than the FMI API.
I might take a crack at this if Pirate Weather is open to the idea of new data sources.
Acknowledgements

[X] I have searched this repository and Home Assistant Repository to see if the feature has already been requested.
[X] I have written an informative title.

New source suggestions are always welcome. Not sure what format @alexander0042 needs but there is data available in grib2 format here: https://en.ilmatieteenlaitos.fi/open-data-manual-forecast-models

Cool! Thanks for posting this, and yes, absolutely interested in integrating it! Things are set up so it'll be pretty straightforward to add a new source, so long as it's available in either grib or NetCDF and has hourly outputs! The full source code for Pirate Weather is coming this week (took some time to get everything organized), and I'd love it if you wanted to take a crack at it. Otherwise, I'll add it to the backlog of additional future sources, and it'll make it's way in eventually.


Different parsing of --bh attribute on windows and unix OS

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Versions.
angular-cli: 1.0.0-beta.24
node: 5.11.1
os: darwin x64
@angular/common: 2.4.1
@angular/compiler: 2.4.1
@angular/core: 2.4.1
@angular/forms: 2.4.1
@angular/http: 2.4.1
@angular/platform-browser: 2.4.1
@angular/platform-browser-dynamic: 2.4.1
@angular/router: 3.4.1
@angular/compiler-cli: 2.4.1
Repro steps.
update package.json to have
"build": "ng build --bh '${rootContext}/' --environment=prod --target=production",
line in script section
this will have correct output in index.html on Linux/OSX <base href="${rootContext}/">
but on windows its will be <base href="'${rootContext}/'">
if change package.json to have "build": "ng build --bh \"${rootContext}\/" --environment=prod --target=production", we will got correct otput on Windows but on Linux/OSX it will be <base href="">.
if package.json to have line like this "build": "ng build --bh ${rootContext}/ --environment=prod --target=production",, output will be invalid on both OS.
Note:
${rootContext} - is a var that will be replaced by java backend on server then return index.html to user.
Question
How to make it works on any OS?

This sounds like a bug with how we parse the argument. Will need to investigate.

Windows 8.1, git-bash:
Command: ng build --prod --base-href /foo/
Result: <base href="C:/Program Files/Git/foo/">

@filipesilva, any updates on this?

@s1im05 that happens in git bash because it parses arguments in a certain way - /foo/ really is C:/Program Files/Git/foo/.
@ptsydzik I had a quick look now and as far as I can tell the variables we get inside the CLI are 'correct'. What's happening is that Windows/Linux/OSX interpret quotes differently in npm scripts.
For npm scripts to work correctly with string args in Windows, they really need to be surrounded by \". That seems to not work so well with the ${var} notation in Bash though.
Not sure what we can do on the CLI side, the value we get is already the one interpreted by the terminal.

Maybe implement reading options from config file can help?
e.g.
ng build --build-config build.config.json
and in build.config.json do:
{     "baseHref": "\'${rootContext}\/\'",     "environment": "prod",     "target": "production",     ..... }

Thanks for reporting this issue. This issue is now obsolete due to changes in the recent releases. Please update to the most recent Angular CLI version.
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.

Still getting this issue with CLI v1.1.1
@filipesilva which v. of the CLI should this be fixed in?
Steps to reproduce, git bash Win 10:
$ ng new foo --style scss --routing --prefix foo
$ cd foo/ && ng build --prod --base-href=/something/here/
index.html shows <base href="C:/Program Files/Git/something/here/">

@danwulff what you're experiencing is https://github.com/angular/angular-cli/issues/5606.


Is it possible to change defined ids for every time an svg is rendered?

right now when loading a svg, the first time its loaded the gradients defined in it works fine
but when you load the same svg for the second time, it no longer renders correctly, as ids clash
I was wondering if we could prefix an incremental number to the svg ids used inside it every time its loaded

Hi @jd1378 - thanks for reporting this. I'll try to take a look at it this evening.

hi
I have opened an issue here as well, since I thought it might be an upstream issue:
https://github.com/jpkleemans/vite-svg-loader/issues/98

For now, you can just add this in plugins:
svgo: {
  svgoConfig: {
    multipass: true,
    plugins: [
      {
        name: "preset-default",
        params: {
          overrides: {
            cleanupIds: false,
          },
        },
      },
    ],
  },
},

How is it going to help the problem ? Its gonna have the same result

well with moving vite plugin inside, It may be possible, but without digging deep into svgo it seems to be hard to do
I have lost interest in this, because I simply stopped using svgs with ids, or switched the gradients with similar looking fill colors


App Insights enhancement for Public Preview

@puicchan Can you please provide details on what you're looking for?

I cannot recall what is this for so closing the issue.


Fix setting properties for datasource with ClickHouseProperties class

In one of commits you change getting properties from get to getProperty. And it brakes setting properties with ClickHouseProperties class.
Reason of this:
In getProperty method there is checking that value instanceof String. But in ClickHouseProperties class you have properties(connection_timeout and other) that have time int, boolean etc. So for these values when you call getProperty method you will get null and default value would be set(like in test).
String.valueOf needed only for variables with primitive types, because objects can be null.

Thanks. Included in 0.1.17.


IPv6 cleanup

This contains changes that are already part of my feature/ip6-support branch targeting upstream (https://github.com/Place1/wg-access-server/pull/157), but haven't been merged to ffmuc's fork yet.
Changes

BREAKING CHANGE: The env var WG_VPN_CIDRv6 has been renamed to WG_VPN_CIDRV6 to make it all-caps like env vars should be
If you have the IPv6 CIDR defined via the environment variable not the config file, make sure to adjust it accordingly.

Update documentation to address usage with IPv6: sysctls in README.md, new config keys in 2-configuration.md, expanded 1-docker.md
Closes #26

Allow disabling IPv6 or IPv4 (i.e. single-stack network mode) by setting the CIDR to 0 or the empty string.

The DNS proxy upstream detection got adjusted to handle single-stack deployments

wg-embed now takes a slice of strings for multiple addresses / CIDRs, instead of a single comma-separated string.
See https://github.com/freifunkMUC/wg-embed/pull/2, which needs to be merged before this PR
Tests are expected to fail until then.

Looks good


[Firebase] Analytics

Log that user visited which screen. Implementation can be done using route observer.


Dev server left running when preview app crashes

Hey! Super excited about this project!
I noticed that when the dev server crashes (node errors, etc), the next server is still running on port 3000 and I get the error Port 3000 is already in use. My work around at this point is to just nuke every node process running and then i'm able to restart the server. Is there a more graceful way to handle this?

@mckelveygreg What's the best way to reproduce this?

Oh of course, my bad.
In my work flow, I find the bug by:

start dev server
remove image asset that was copied over
get error: throw new Error(Something went wrong while copying the file to ${constants_1.PACKAGE_EMAILS_PATH}, ${result.cat()});
try to start dev server
get error: Port 3000 is already in use.


CLOUDP-73368: fixing the Operation struct in the atlas go client

Description
We got a deserialization error during the implementation of the suggestedIndex command on mongocli due to the field Predicates inside the structOperation
Type of change:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Required Checklist:

[ ] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation (if appropriate)
[ ] I have run make fmt and formatted my code

Further comments

@themantissa merging this PR since it is a bug fix for the performance advisor api that is not used by terraform


EntityGenerator - table annotation is not for superClass

entity for example:
/**
* Book
*
* @ORM\MappedSuperclass
*/
class Book {
    ....
}

EntityGenerator will make:
/**
* Book
*
* @ORM\Table(name="book")
* @ORM\MappedSuperclass
*/
class Book
{
    ...
}

there should not be @ORM\Table(name="book")

EntityGenerator is going to be removed in Doctrine 3.0, sorry for no response and thanks for reporting.


Feature/job scraper package austin

Description
There needs to be a package that future Underdog Team Members can use to easily scrape LinkedIn for job openings, for multiple purposes, such as gathering current job openings, but perhaps more importantly, use NLP to match mentee's resumes to job openings by similarity. I had already built a notebook that gathered job opening data from LinkedIn for purposes of the Data Science team to build the first NLP model, there will be a need for more data. So I have adapted the code into an importable package for anyone to use in any script or notebook (and for Admin's or those on the Data Science team to have more flexibility in how LinkedIn is scraped).
Loom: https://www.loom.com/share/b426a4d3c39244faaf14ab6687bb67a2
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature

Checklist:

[x] My code follows PEP8 style guide
[x] I have removed unnecessary print statements from my code
[x] I have made corresponding changes to the documentation if necessary
[x] My changes generate no errors
[x] No commented-out code
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Looks and works great. Excellent work! This will help incredibly in the future.

Good job over all but, lets get rid of the vscode config files.

Done!


Investigate upgrading Manta to v. 1.6.0

The release notes for v 1.6 (https://github.com/Illumina/manta/releases) say:

This is a significant engineering update from the v1.5.1 release, in which the SV candidate generation step has been changed to a single multi-threaded process to improve task scheduling. This slightly improves average runtime and reduces runtime variability. With this update, support for job distribution over SGE is dropped.
Added
Add configuration option to turn off the evidence signal filter during candidate generation (DRAGEN-1873)
This enables very high sensitivity during high depth tumor-only calling
Changed
Change the SV candidate discovery and genotyping phase from a multi-process to a multi-thread design for better CPU utilization (MANTA-1521)
As a result, runtime is faster and less variable than before.
Runtime improvements vary by workload and server configuration, for typical WGS workloads on a modern server an improvement of ~5-10% may be expected, but improvements of up to 50% have been observed for cases where work previously was poorly distributed across processes.
SGE support is removed with this change.
Update htslib/samtools to 1.9 (MANTA-1483)

It sounds like we might get performance gains by upgrading (we currently use v1.5.0 in the pipeline).

Addressed in #601


BUGFIX: Adapted delete check for new version (since 6.x)

Newser versions of Elasticsearch do not return found in the DELETE-Action. Meanwhile the result is in result.
Here are two examples:
2.x:
{
    "_shards" : {
        "total" : 10,
        "failed" : 0,
        "successful" : 10
    },
    "found" : true,
    "_index" : "twitter",
    "_type" : "tweet",
    "_id" : "1",
    "_version" : 2
}

6.x:
{
    "_shards" : {
        "total" : 2,
        "failed" : 0,
        "successful" : 2
    },
    "_index" : "twitter",
    "_type" : "tweet",
    "_id" : "1",
    "_version" : 2,
    "_primary_term": 1,
    "_seq_no": 5,
    "result": "deleted"
}

This would also work with 5.x by the way!
https://www.elastic.co/guide/en/elasticsearch/reference/5.6/docs-delete.html


Fixing positional argument error in yaml.load()

While running python gather_results.py in sloop/results/all-joint-sloop it throws the error -

TypeError: load() missing 1 required positional argument: 'Loader'

This change fix that issue

An older version of PyYAML was used. I'll merge this soon.
Just to make sure, are both of these two changes blocking the gather_results.py to run?

An older version of PyYAML was used. I'll merge this soon. Just to make sure, are both of these two changes blocking the gather_results.py to run?

No, only changes in lang_result.py is required to run gather_results.py. But load() will give error in parse_osm_mapping.py without positional argument.

OK. Thanks for the clarification.


[fix] Removing 'frozen' from ExceptionRAPDU dataclass to allow Python…

…3.11 contextlib to work as expected

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

Comparison is base (d6f183f) 75.96% compared to head (ef341bd) 75.96%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #161   +/-   ##
=======================================
  Coverage   75.96%   75.96%
=======================================
  Files          34       34
  Lines        1677     1677
=======================================
  Hits         1274     1274
  Misses        403      403

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


AD: javatpoint.com

URL
https://www.javatpoint.com/data-management-issues-in-mobile-database
FILTERS
Update microshield to include this domain
Browser
Google Chrome
Operating System
Microsoft Windows 10
Ad-Blocker
AdGuard Browser Extension
Userscript Manager
Tampermonkey
USERSCRIPT
Microshield 4.4.10
Comment
No response

@piquark6046

The domain exists in the matched domains list.

The domain already exists in the matched domains list.
@BlazDT

Then Ads are not hidden :(

@BlazDT
Please share a screenshot with DevTool console (verbose enabled).

Version 4.5 seems to have fixed the ad error. But seeing errors in console:

@BlazDT That's an intended behavior.


Text extractor only working on main screen

Microsoft PowerToys version
0.76.2
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
TextExtractor
Steps to reproduce

Move to cursor to second monitor
Type win+shift+T (default shortcut of text extractor).
The UI appear only on main monitor, not the monitor where cursor locate.

✔ Expected Behavior
Whe  user has two or more monitors. TextExtractor'UI follow cursor position. When cursor moves to a new monitor, TextExtractor moves to the new monitor.
❌ Actual Behavior
TextExtractor stay in main monitor, ignoring cursor location.
Other Software
No response

/dup #21676


Failed to generate with oapi-codegen v2.3.0 due to openapi3.CircularReferenceCounter is undefined

Workaround:
To avoid the following error, use github.com/oapi-codegen/oapi-codegen/v2 v2.3.1-0.20240709092903-309b11f85c73 .
# github.com/oapi-codegen/oapi-codegen/v2/pkg/util
../../../../../.go/pkg/mod/github.com/oapi-codegen/oapi-codegen/v2@v2.3.0/pkg/util/loader.go:24:45: undefined: openapi3.CircularReferenceCounter
../../../../../.go/pkg/mod/github.com/oapi-codegen/oapi-codegen/v2@v2.3.0/pkg/util/loader.go:26:12: undefined: openapi3.CircularReferenceCounter
../../../../../.go/pkg/mod/github.com/oapi-codegen/oapi-codegen/v2@v2.3.0/pkg/util/loader.go:33:12: undefined: openapi3.CircularReferenceCounter
app/idp/api/api.go:8: running "go": exit status 1
make: *** [gen] Error 1

Related: https://github.com/oapi-codegen/oapi-codegen/issues/1702
This is fixed as of v2.4.0

I close this issue since this repository is going to be archived.


APP_RENDERING_FAILED not fired even though UI gives out error message

Expected Behaviour
APP_RENDERING_FAILED event is fired if viewer fails to load a file. (corrupted pdf files, renamed non pdf files to .pdf ext)
Actual Behaviour
Viewer gives an error on the UI but the event is not fired at all.
Reproduce Scenario (including but not limited to)
Attach a non-pdf file, or a non-pdf file that you renamed to a pdf.

Hi, issues here should address problems in the samples for this repo. For questions/issues using Embed, please use our forums: https://community.adobe.com/t5/document-services-apis/ct-p/ct-Document-Cloud-SDK?page=1&sort=latest_replies&filter=all&lang=all&tabid=discussions