WatcherRPC's status.get is timing out

while looking into https://github.com/omgnetwork/elixir-omg/issues/1659 I found that
status.get is timing out when the request contains empty body with not empty content-length header. For example
curl -w<EMAIL_ADDRESS>\
 -X POST \
 --data "{}" -H "Accept: */*" \
 -H "Accept-Encoding: gzip"  -H "Content-Length: 51876961" \
 -H "Content-Type: application/json" \
 "http://localhost:7534/status.get"

{"data":{"code":"server:internal_server_error","description":"Server has failed to render the error.","object":"error"},"service_name":"watcher_info","success":false,"version":"1.0.3+522fb32"}

 time_namelookup:  0,004s
       time_connect:  0,005s
    time_appconnect:  0,000s
   time_pretransfer:  0,005s
      time_redirect:  0,000s
 time_starttransfer:  15,008s
                    ----------
         time_total:  15,008s

I think it's related to  https://github.com/elixir-plug/plug/issues/968

Doesn't seem to affect overall system perf so will treat this as low priority


Temporary Directory for Vivado Synthesis Files Can be Specified Thru a Flag

(Minor, Not Super-Urgent PR)
Sometimes when running resource estimates on havarti, fud would write some of the temporary (very large) Vivado synthesis files into a /tmp folder somewhere in /home, which would temporarily fill up havarti. Using this PR, we can tell fud where to store the temporary file (in this case somewhere in /scratch) as to not fill up space.
Quick question: should this change to fud be documented somewhere? If so, I'll do it on this PR.

LGTM apart from one comment!


How to mark multiple defects in one drawing

A picture
There are several defects at the same time
Whether the label names are all the same
All defects have one name
Or separate
If separated
How to divide data sets into folders

@samet-akcay I've stumbled upon the same question in my research. Is here any support for images containing more than one kind of defect?


fix(components): adjusting whitespace causing style issue

Description
Removing white space from covalent code component that was causing extra space.
Before

After

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

v8.10.1
GitHub release

Your semantic-release bot :package::rocket:


lein ancient does not use the mirror configured in ~/.lein/profiles.clj

I am using an organisation's internal mirror, configured in profiles.clj (it works much as described here: http://maven.apache.org/guides/mini/guide-mirror-settings.html)  However, ancient.clj is still trying to download artifacts from clojars.

@ekimber Thanks for reporting - I can verify the issue.
I seem to have expected that Leiningen automatically merges :repositories and :mirrors which is not the case, as far as I can tell. I'll have a look and fix it.

Can you try the latest SNAPSHOT?
[lein-ancient "0.6.4-SNAPSHOT"]

Great!  That snapshot build fixes the issue I was seeing.

Glad to hear. Released as
[lein-ancient "0.6.4"]


FetchStatus->Queued is decremented, but never incremented

We're currently getting the following output after running locally:
Accumulator Results:

FetchStatus->HTTP_NOT_FOUND (java.lang.Long): 1047
FetchStatus->UNFETCHED (java.lang.Long): 10038
FetchStatus->ABORTED_FETCH_SETTINGS (java.lang.Long): 10
FetchStatus->FETCHING (java.lang.Long): 2099
FetchStatus->QUEUED (java.lang.Long): -1991
FetchStatus->FETCHED (java.lang.Long): 826

This is because the decrement of FetchStatus->QUEUED has no matching increment. All of the manipulations of the FetchStatus counters should be reviewed.

Hi @Schmed - I think you fixed this already

This was indeed resolved by pull request #150

This was indeed resolved by pull request #150


[ko] Update outdated content in dev-1.22-ko.2 (Task M8)

content/en/docs/concepts/configuration/overview.md

refs: #30000

감사합니다~~!
/lgtm
/approve


Allowing (optional) dynamic max in progress bar

If max's value was changed after the progress bar was created, the value of max in the progress bar was not updated.
This update will allow the user to dynamically change the max value and update the progress bar correctly.
Max is treated as an optional variable and is being initialized using the default value (100) if not provided.

@raananw - Can you add some tests for this?

Sure, I'll extend the progressbar tests.

The test added is the only thing that didn't work. Hope it's enough.

Just fixed the adjust to adjusts.

Already did it, landed as 7ccff028ff5ed19af63d9cf1f5e078223924f3a0, thanks :) .


chore: bdd-sbhg-1587005760 to 0.0.1

chore: Promote bdd-sbhg-1587005760 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


[backend-integration] processing assets not directly used by Vite

Clear and concise description of the problem
If static assets are referenced in templates on backend (.twig, .blade, .latte, etc.), Vite will not process them and they are not included in manifest file.
A way how to get around that is described here https://github.com/vitejs/vite/issues/2375#issuecomment-908579361
Suggested solution
rollupOptions.input should accept any kind of file type, including png, jpg, svg, etc and treat them as assets. This is not currently possible and are generated as empty chunks.
Generated an empty chunk: "fragment.svg"
Generated an empty chunk: "icon.png"

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

What about introducing option like build.assetsInput as string[] with glob paths?
And each glob could then include assets viaimport.meta.glob('/path/to/assets/**') perhaps? This way they could be added to build as described in https://github.com/vitejs/vite/issues/2375#issuecomment-908579361

Oh, I had a similar problem when I pack a mpa.
I use <img<EMAIL_ADDRESS>alt="" /> directly in a html and I also put the picture into rollupOptions.input, but it became an empty js after build, and there's a info like this:
Generated an empty chunk: "images\picture".


Review Ticket Week 3

Crossover Grade: 2.7

AppLab Activity Plan/Reflection
AppLab Quiz
Scrum Board to begin planning Creative Performance Task
New Tag created for CPT

Goals:

Learn to add score to AppLab quiz
Learn how to work if/else on AppLab
Add to ideas "Guess the Lyric" for Creative Performance Task

This week was challenging, due to the change in tools that we used while coding. Although AppLab helped visualize and understand repeats in code, it was challenging to operate functions like if/else. I believe we were able to successfully make a quiz (with good aesthetics and functionality), however, were unable to figure out how to provide a score at the end.

2.7/3.0
While we did what we were supposed to, we definitely encountered some challenges this week. We tried to go above and beyond with making another app, adding a scoring feature, and using loop functions, however none of those worked. Thus, 2.7 and not 2.7+


Add support for checkpointing on steps

If training_args.save_strategy=="steps", checkpointing every training_args.save_steps will occur

lgtm


Code example and explanation not really helpful

Shouldn't this sample and/or the description reference <typeparamref name="Type"/>?
I would have expected something more like:
XML comment on 'param' has cref attribute 'attribute' that refers to a type parameter
This error is generated by an XML comment that refers to a type parameter using <see />. Use <typeparamref /> if you truly want to refer to a typeparam.
// CS1723.cs
// compile with: /t:library /doc:filename.XML
///<summary>A generic list class.</summary>
///<see cref="T" />   // CS1723
// To resolve use <typeparamref name="T" />
public class List<T>
{
}

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

ID: df1a6c53-0693-76d3-dafe-bd0c5c00fd5d
Version Independent ID: fe77c7b1-6154-2b08-8a3a-c071ec1c817d
Content: Compiler Warning (level 1) CS1723
Content Source: docs/csharp/misc/cs1723.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @eberleitguy
I've added it to our backlog to address.

I will work on this.


fixed links under Documentation in README.md

[x] You have run tests on this submission, either by using Travis Continuous Integration testing testing or running nosetests on your changes?
[x] This pull request is directed to the pysal/master branch. This is important, as any PRs submitted against any other branches will be delayed.
[ ] This pull introduces new functionality covered by
docstrings and
unittests?
[ ] You have assigned a
reviewer and added relevant labels
[x] The justification for this PR is:
Fixed links under 'Documentation' section of README.md

Thanks for the PR!


Add support for custom preprocessor definitions

By default, kernel tuner inserts the tunable parameters as preprocessor parameters into the source code. However thanks to the cupy backed, these parameters can now also be inserted using template arguments, meaning the preprocessor definitions are no longer necessary. It would nice if there was a way to disable them since they can cause conflicts with external libraries (for instance, CUB uses the variable "block_size" in a few places).
This PR adds a new keyword argument defines to tune_kernel (and run_kernel) that allows customizing the behavior of kernel tuner around these preprocessor definitions.
Examples:

if defines=None, the behavior is identical to how it is today
if defines=dict(), no preprocessor definitions are inserted, not even the default ones
if defines=dict(floating_type="float"), insert #define floating_type float
if defines=dict(block_size=lambda p: p["block_size"]), define only block_size
if defines=dict(twice_block_size=lambda p: 2 * p["block_size"]), insert twice_block_size with the value of 2 * block_size.

Unresolved issues:

[ ] Should this keyword be called defines or definitions?
[ ] Do we need more unit-tests?
[ ] Is this behavior intuitive? An alternative would be to have a flag disable_default_defines=True to disable the default definitions, but this would add yet another keyword argument :-(

Hi Stijn!
Thanks for this! I can definitely see this being useful in certain cases. defines seems like a fine name for this. Also passing an empty dict to disable default behavior is similar to how we disable other defaults in Kernel Tuner, for example with grid divisors.
I don't fully understand yet how the parameters are inserted through template arguments in the absence of defines. If you look at this example:
https://github.com/benvanwerkhoven/kernel_tuner/blob/master/examples/cuda-c%2B%2B/vector_add_blocksize.py
We insert the parameters as template arguments but these currently only have meaningful values because of the preprocessor defines.
I've added an example to this branch just now, which currently isn't working:
https://github.com/stijnh/kernel_tuner/blob/custom-defines/examples/cuda-c%2B%2B/vector_add_defines.py
I believe the question now is how to enable the insertion of tunable parameter values into template arguments. I don't know if you already had something in mind for this, but my first guess would be to just allow someone to pass
kernel_name = "vector_add<block_size_x>" and then use a util function like replace_param_occurances to replace the tunable parameters with their value at this configuration. This would be a very easy solution in my opinion. Do you agree or did you have something else in mind?

Indeed, I had not considered that disabling the preprocessor defines would make it impossible to pass the arguments as template parameter. I have fixed this in prepare_kernel_string so that it now also replace the parameters in the kernel name.
I have also modified replace_param_occurrences such that it first tokenizes the input instead of simply performing a string replace.
I feel this branch is ready for merging!

Thanks a lot Stijn!


Client Allocation Request for: asd

Client Allocation Questions
Core Information

Name: asd
Website / Social Media: asd, (asd, asd)
Region: North America
DataCap Requested: 576 GiB
Addresses to be Notarized: asd
Notary Requested: Deon Erda

This user’s identity has has been verified through filplus.storage

This user’s identity has been verified through filplus.storage


request with cookie

i wanna send some requests to a website but that website require to login first how can i define my  cookie in requests that i wanna send?

For context, I'm making the assumption that you'll need to use a Cookie header to send your cookie.
When you create a channel - and when you submit requests through your opened channel to end points, you can attach a list of metadata to your request using a metadata parameter.
I use JWTs in a lot of the requests I make through the library - so hopefully a rough psuedo-code example of that will help you out.
request = {'field1':'value1',...}
auth_metadata = [{'authorization',f'bearer {generate_jwt()}]

client.request(
   "SAMPLE_SERVICE",
   "SAMPLE_METHOD",
   request,
   metadata=auth_metadata,
   timeout=10
)

this worked for me metadata=(("authorization", f"bearer {token}"),)


Some attempts to fix flakey test - can't replicate locally :(

You can actually ssh into the node running the tests in circleci and reproduce the failure


User Issue: Start in private cluster without public internet access

Describe the bug
A clear and concise description of what the bug is.
kubectl logs -f chaos-controller-5f897979d-fxv9z -n chaos-engineering

{"level":"fatal","ts":1710340783015.9324,"caller":"chaos-controller/main.go:154","message":"error initializing CloudProviderManager","error":"could not get the new ip ranges from provider AWS: Get "https://ip-ranges.amazonaws.com/ip-ranges.json\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)\ncould not get the new ip ranges from provider GCP: Get "https://www.gstatic.com/ipranges/goog.json\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)\ncould not get the new ip ranges from provider Datadog: Get "https://ip-ranges.datadoghq.com/\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)\n","stacktrace":"main.main\n\t/go/src/github.com/BkfwVXeB/0/DataDog/chaos-controller/main.go:154\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"}

We have EKS cluster without Internet access, what would be the correct way to start? I have proxy to Internet, and is there a config to setup it?

The simplest approach is to disable the feature that reaches out to the internet: https://github.com/DataDog/chaos-controller/blob/main/chart/values.yaml#L47
As for solving the networking so that the controller can proxy to the internet, unfortunately that's something I have no experience with at all, and is outside the scope of what we're willing to support.


Support for jinja

Jinja in python and nunjucks in node is quite popular , any recommendations on how to have highlighting for these ?

I'll try to implement them tomorrow if I have time.

In the meantime, you can still use the Twig component. It should do a pretty decent job highlighting Django's templates, jinja and nunjucks.

Yes twig does a good job :-)

And this leads me to the conclusion that those templating languages should share a common dependency. I'll think about how to do this properly.

Might want to take Handlebars and Mustache into account. They use very similar syntaxes, too.

Closing as twig works great


[Event-Request] Codeunit 99000856 "Planning Transparency" OnLogSurplus

Hello,
please add the following event to codeunit 99000856 "Planning Transparency", function "LogSurplus":

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Support Typescript for lifecycle events

What is in PR

Allow to use Typescript files in lifecycle events: --before and --after
Fix #690

Example:

tap --ts --before=test/setup.ts --after=test/teardown.ts

Hi @isaacs, please review this PR


[code-completion] Suggest module names, and handle module name bases for member completion in type position

Previous we didn't suggest modules for top-level completions, and for module type member completions we would just return if the parsed type loc didn't type-check as a type.

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 7350917c221635e9a04ab33acc433c103b352510

Build failed
Swift Test OS X Platform
Git Sha - 7350917c221635e9a04ab33acc433c103b352510

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 2fefd5bfc16ad6740c098a68c18e392eaced4697

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - b871454b8f5e83aab5d4c7ca7342bbf2818f8616

Build failed
Swift Test OS X Platform
Git Sha - b871454b8f5e83aab5d4c7ca7342bbf2818f8616


Make all remaining fragments reactive

Changes:

Following up from #1269
Make all DB interactions in fragments reactive
Fix navigation drawer lag

Please review @the-dagger @mananwason @mariobehling


[20.09] thunderbird, thunderbird-bin: 78.4.3 -> 78.5.0 [High security fixes]

Motivation for this change
#104378 for 20.09.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 104480 1

2 packages built:

thunderbird
thunderbird-bin


DWEET support for when CO2 sensor is not available

When CO2 sensor is not available, AQ uses the convention of setting sensorData.internalCO2 to 10000. DWEET will attempt to push CO2 values even when there is no sensor reading it, so it will push 10000 upstream. Is that the intended behavior?

Good catch!   I see from looking at readSensor() that it sets internal CO2, temperature and humidity to 10000 if no data was read from the sensor.  I hadn't spotted that before.  Knowing that indicates an error condition I will add a test so post_dweet() doesn't get called if that error (no data) exists.  Thanks.

I added the required if() test in the network update branch I'm working on, so will follow up here when that pull request goes in.  (I added that same logic to apply to uploading data to InfluxDB.)

I modified the logic due to another bug for DWEET, MQTT, and Influxdb, and verified in MQTT and Influxdb that values publish when CO2 = 10000 (no CO2 sensor) and when we have an actual value. I'm going to close this bug. @disquisitioner you can reopen if there is an issue.


[openapi-alps] Fix flaky tests

  1) UtilsTest
       TestDoOnTargetBranch:
     Error: Command failed: git init
fatal: unable to get current working directory: No such file or directory

      at ChildProcess.exithandler (node:child_process:402:12)
      at ChildProcess.emit (node:events:513:28)
      at maybeClose (node:internal/child_process:1100:16)
      at Socket.<anonymous> (node:internal/child_process:458:11)
      at Socket.emit (node:events:513:28)
      at Pipe.<anonymous> (node:net:301:12)

  2) UtilsTest
       TestGetPRChangedDir:
     Error: Command failed: git init
sh: 0: getcwd() failed: No such file or directory
fatal: unable to get current working directory: No such file or directory

      at ChildProcess.exithandler (node:child_process:402:12)
      at ChildProcess.emit (node:events:513:28)
      at maybeClose (node:internal/child_process:1100:16)
      at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

https://dev.azure.com/devdiv/DevDiv/_build/results?buildId=7514349&view=logs&j=5d4285e7-3158-52a0-43de-ea678c83745f&s=45a4dd8c-d89a-566b-d89f-6724086326ec&t=f21af9c8-1342-5aaa-8506-f8baf1069ba1&l=734

I suspect this is a race condition where await pfs.mkdir(remote, { recursive: true }) (where pfs is @ts-common/fs) returns before the directory is fully created in the filesystem.  Attempting to fix by using await fs.promises.mkdir() instead.
https://dev.azure.com/devdiv/DevDiv/_git/openapi-alps/pullrequest/460226

Previous PR did not fix issue.  We could add a loop to keep retrying until directory.exists() returns true.

Fixed in https://dev.azure.com/devdiv/DevDiv/_git/openapi-alps/pullrequest/464135


About hunt.js file manual edit

Question
What would you like to ask?
I couldn't find way to make resources/hunt.js file with util/get_hunt_data.py.
So, I manually edited it.
I mean, I don't know how to make that python program to work.
C:\>python get_hunt_data.py
Traceback (most recent call last):
  File "get_hunt_data.py", line 85, in <module>
    get_from_coinach()
  File "get_hunt_data.py", line 63, in get_from_coinach
    reader = coinach.CoinachReader(verbose=True)
  File "C:\coinach.py", line 46, in __init__
    if not os.path.isfile(os.path.join(self.coinach_path, _COINACH_EXE)):
  File "C:\Program Files\Python38\lib\ntpath.py", line 78, in join
    path = os.fspath(path)
TypeError: expected str, bytes or os.PathLike object, not NoneType

I couldn't fix it. However, I tried half-manual editing it.
I extracted bnpcnames.exd with ffxiv_explorer, to English and Korean csv files. and made my own program with C# to make Korean texts into Unicode Encoded strings, hunt.js was written in Unicode Encoded ones.(maybe it doesn't matter though) I manually Ctrl+F and Ctrl+C, Ctrl+V to hunt.js with matching names.
and found it working well with Korean FFXIV.
But... hunt.js says
// Auto-generated from get_hunt_data.py
// DO NOT EDIT THIS FILE DIRECTLY

so... it would be okay to use this "self edited file" my own.
but I don't know if it's okay to upload here that hunt.js file. Without editing get_hunt_data.py program.
I think hunt.js file will be overwritten after re-executing that get_hunt_data.py, but hunt data won't be updated until next expansion, expected from previous patch experiences.

I added ko support in the python script.  Let me know if you have any issues or comments on what got automatically added there.

I think my problem is not related to code. I get similar error with latest gen_hunt_data.py with python v3.8. Is there any requirements to execute that python code?
Traceback (most recent call last):
  File "gen_hunt_data.py", line 118, in <module>
    get_from_coinach()
  File "gen_hunt_data.py", line 99, in get_from_coinach
    reader = coinach.CoinachReader()
  File "C:\coinach.py", line 46, in __init__
    if not os.path.isfile(os.path.join(self.coinach_path, _COINACH_EXE)):
  File "C:\Program Files\Python38\lib\ntpath.py", line 78, in join
    path = os.fspath(path)
TypeError: expected str, bytes or os.PathLike object, not NoneType

I've cloned repository, and tried python gen_hunt_data.py on git-bash and cmd.
Oh, I need SaintCoinach to run this.
I didn't realize that. leaving comment here to notice other contributors who might wondering about this.
But, another error occured..
처리되지 않은 예외: System.IO.DirectoryNotFoundException: 디스크에 없는 경로에 액세스하려고 했습니다.
translates: unhandled error: System.IO.DirectoryNotFoundException: tried to access not existing directory
   위치(means location): SaintCoinach.IO.PackCollection..ctor(DirectoryInfo dataDirectory) 파일(means file) C:\projects\saintcoinach\SaintCoinach\IO\PackCollection.cs:줄(means line) 30
   위치: SaintCoinach.ARealmReversed.Update(Boolean detectDataChanges, IProgress`1 progress) 파일 C:\projects\saintcoinach\SaintCoinach\ARealmReversed.cs:줄 361
   위치: SaintCoinach.Cmd.Program.Main(String[] args) 파일 C:\projects\saintcoinach\SaintCoinach.Cmd\Program.cs:줄 50
Traceback (most recent call last):
  File "gen_hunt_data.py", line 118, in <module>
    get_from_coinach()
  File "gen_hunt_data.py", line 101, in get_from_coinach
    update_coinach(monsters, reader)
  File "gen_hunt_data.py", line 77, in update_coinach
    exd = reader.exd('NotoriousMonster', lang=locale)
  File "C:\coinach.py", line 52, in exd
    return self._coinach_cmd('exd', table, lang)
  File "C:\coinach.py", line 78, in _coinach_cmd
    raw_output = subprocess.check_output(
  File "C:\Program Files\Python38\lib\subprocess.py", line 411, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "C:\Program Files\Python38\lib\subprocess.py", line 512, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'C:\SaintCoinach\SaintCoinach.Cmd.exe "C:\Program Files (x86)\SquareEnix\FINAL FANTASY XIV - A Realm Reborn" "lang de" "exd NotoriousMonster"' returned non-zero exit status<PHONE_NUMBER>.

I have Global server client installed(C:\Program Files (x86)\SquareEnix\FINAL FANTASY XIV - A Realm Reborn), but why is this happening? Maybe this problem has something relative with #913.


Add two-token web session management demo app.

Add an example Google App Engine app, which leverages service worker to support two-token based web session management.

Hello @kongguibin—I'm not sure that this repo is the best home for this kind of sample, given its dependencies on Java App Engine. It's intended purpose is to house samples closely tied to new functionality added to each version of Chrome (tracked at https://www.chromestatus.com/features) that can be demonstrated as part of a static hosted page served from the gh-pages branch.
Would you be open to going through the normal Google open source releasing process and creating a new repo for this?

I changed my primary email address to my google.com one. Hope the CLA is not an issue any more.

@owencm Owen, can you explain why we select this repository?

I think we should make a new repo, it can be done easily. This is about a
process/method so I think it can standalone.
On Mon, 11 Apr 2016, 18:11 Guibin Kong<EMAIL_ADDRESS>wrote:

@owencm https://github.com/owencm Owen, can you explain why we select
this repository?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/samples/pull/319#issuecomment-208453205

I've started a thread with all of you and Taylor to discuss if this is the right place for this to live. Lets discuss there.

Closing, as this has moved to https://github.com/GoogleChrome/two-token-sw


Destroy does not remove MutationObserver

If you attach and destroy onScreen regularly in a page, it will slowly build up a list of listeners for DOM mutations.

Yup, that's true. Thanks for catching it.


Draft facilitation guide for pilot retro with CA

Background and context
We want create a facilitation guide for our retro the pilot STLT (CA) after the flu pilot has met the success criteria
Open Questions

What tool do we want to use for this? Mural, EasyRetro... (needs to be accessible by CA)
What estimated date should we target? Should this come before or after the sender retro? Based pilot progress and eng capacity
Who should be invited? CDC stakeholders?
What topics are most important to focus on?

Acceptance Criteria

[ ]  Tool selected and set up for session
[ ] Facilitation guide for session created
[ ]  Meeting date selected
[ ]  Calendar invite sent

@jsutantio thoughts?


Add new helper method /meta/aksCheckCredentials

Add a new POST endpoint /meta/aksCheckCredentials that accepts AKS
credentials, minus tenant ID, as a POST request, and checks whether they
are valid. This allows the UI to validate the user's input when a cloud
credential or cluster is being created before the cloud credential
exists.
https://github.com/rancher/rancher/issues/33861

Probably doesn't matter since anyone can just use the SDK to bypass us

Yeah when I was working on this I found the errors from Azure through the SDK were very verbose, they tell you exactly whether the subscription is missing or the format is invalid or the client ID doesn't exist or the secret for it is unauthorized or whatever.


Use of ESD CAN-USB/2 on Windows with YARP

Hi everyone, I have a ESD CAN-USB/2 (the blue one), I am trying to get it to compile with Windows, as if I remember correctly, such device should be working fine in Windows.
In the (I supposed outdated) documentation in http://wiki.icub.org/wiki/Esd_on_windows, it is written that the ecan YARP device present in icub-main supports only the ESD CAN-USB-MINI. Assuming that information was wrong, I downloaded the CAN-USB/2 SDK from https://esd.eu/en/products/can-usb2 and I ensure that CMake is finding the SDK.
However, when compiling the ecan device in icub-main on Visual Studio 2013 I get the following compilation errors:
Build FAILED.

"C:\Code\Production\icub-main\build\ALL_BUILD.vcxproj" (default target) (1) ->
"C:\Code\Production\icub-main\build\src\libraries\icubmod\esdCan\esdCan.vcxproj" (default target) (20) ->
(ClCompile target) ->
  C:\Code\Production\icub-main\src\libraries\icubmod\esdCan\EsdCan.cpp(59): error C2664: 'NTCAN_RESULT canGetBaudrate(NTCAN_HANDLE,uint32_t *)' : cannot convert argument 2 from 'DWORD *' to 'uint32_t *' [C:\Code\Production\icub-main\build\src\libraries\icubmod\esdCan\esdCan.vcxproj]
  C:\Code\Production\icub-main\src\libraries\icubmod\esdCan\EsdCan.cpp(100): error C2664: 'NTCAN_RESULT canRead(NTCAN_HANDLE,CMSG *,int32_t *,OVERLAPPED *)' : cannot convert argument 3 from 'long *' to 'int32_t *' [C:\Code\Production\icub-main\build\src\libraries\icubmod\esdCan\esdCan.vcxproj]
  C:\Code\Production\icub-main\src\libraries\icubmod\esdCan\EsdCan.cpp(102): error C2664: 'NTCAN_RESULT canTake(NTCAN_HANDLE,CMSG *,int32_t *)' : cannot convert argument 3 from 'long *' to 'int32_t *' [C:\Code\Production\icub-main\build\src\libraries\icubmod\esdCan\esdCan.vcxproj]
  C:\Code\Production\icub-main\src\libraries\icubmod\esdCan\EsdCan.cpp(129): error C2664: 'NTCAN_RESULT canWrite(NTCAN_HANDLE,CMSG *,int32_t *,OVERLAPPED *)' : cannot convert argument 3 from 'long *' to 'int32_t *' [C:\Code\Production\icub-main\build\src\libraries\icubmod\esdCan\esdCan.vcxproj]

    0 Warning(s)
    4 Error(s)

I received various feedback and apparently this is indeed a bug in the master branch of icub-main the ecan device driver, moving the issue to icub-main to discuss about it.
In a nutshell, moving to the devel branch (or cherry-picking this commit: https://github.com/robotology/icub-main/commit/72261273f4e60eb65230a34ec8f383cfb9c04440 ) should solve the issue.,

Issue moved to robotology/icub-main #434 via ZenHub


enable the use of a custom ObjectMapper

fix: https://github.com/spring-projects/spring-pulsar/issues/723 using builder of JsonPulsarHeaderMapper.
AS-IS
JacksonUtils.enhancedObjectMapper() was always used.
JsonPulsarHeaderMapper.builder().build()
TO-BE
Inject ObjectMapper to builder in the constructor of AbstractPulsarMessageToSpringMessageAdapter
JsonPulsarHeaderMapper.builder().objectMapper(objectMapper).build()
By adding ObjectMapper to the constructor of AbstractPulsarMessageToSpringMessageAdapter, we enable the use of a custom ObjectMapper.

Thanks for the contribution @jitokim . The proposed changes would allow a user-provided object mapper only for Pulsar message headers JSON de/serialization.  To allow a user-provided object mapper when de/serializing the message payload is a bit more complicated. I have a PR in progress to do this. What I will do is combine that PR w/ part of this PR. I will try to get something for review in the next few days. I will ping you when I do.
Thank you.

I agree that allowing a user-provided object mapper for the message payload is complicated
Thank you for the review 😃

Hi @jitokim
I have used your initial code proposal as a starting point for https://github.com/spring-projects/spring-pulsar/pull/762.

Thank you for the reminder to implement this for the JSON headers.
Thank you for the great starting point.

You can see I tweaked it quite a bit, namely:

now looking up of the ObjectMapper bean later than afterSingletonInstantiated as that is a bit too early to get access to the mapper
lookup the mapper by special name pulsarHeaderObjectMapper
added tests for @PulsarReader @PulsarListener, @ReactivePulsarListener
added docs

I had intentions of keeping your original commit in my PR but it diverged enough that I did not go that route.
I would however like to give you the credit you deserve via a co-author on the new PR. If you agree can you reply here w/ the following info:

Email address you want to use as the Co-Author (see
https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors#required-co-author-information)

Your first name + last name you want to use in the Javadoc @author tag for the classes that you originally modified (annotation post processors and listener endpoints).

Closing in favor of https://github.com/spring-projects/spring-pulsar/pull/762

Hi, @onobc
Thank you so much for considering me and for the thoughtful acknowledgment.
For the information, I would like to use the following:
First name: Jihoon
Last name: Kim
Email address<EMAIL_ADDRESS>I really appreciate the great open-source project you're creating.


Dependencies section in package.json for @types/fs-extra is incorrectly generated

TypeScript Version:
nightly (1.9.0-dev.20160217)
Repro
If you run:
npm install --save-dev @typings/fs-extra

you get:
npm ERR! addLocal Could not install /Users/igor/.npm/@types/fs-extra/0.0.16-alpha/node_modules/@types/fs-extra/equire("stream"@*
npm ERR! Darwin 15.5.0
npm ERR! argv "/Users/igor/.nvm/versions/node/v5.4.1/bin/node" "/Users/igor/.nvm/versions/node/v5.4.1/bin/npm" "install" "--save-dev" "@types/fs-extra"
npm ERR! node v5.4.1
npm ERR! npm  v3.9.2
npm ERR! path /Users/igor/.npm/@types/fs-extra/0.0.16-alpha/node_modules/@types/fs-extra/equire("stream"@*
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall open

npm ERR! enoent ENOENT: no such file or directory, open '/Users/igor/.npm/@types/fs-extra/0.0.16-alpha/node_modules/@types/fs-extra/equire("stream"@*'
npm ERR! enoent ENOENT: no such file or directory, open '/Users/igor/.npm/@types/fs-extra/0.0.16-alpha/node_modules/@types/fs-extra/equire("stream"@*'

This is because the package.json of @types/fs-extra is incorrectly generated:
{"name":"@types/fs-extra","version":"0.0.16-alpha","description":"Type definitions for fs-extra from https://www.github.com/DefinitelyTyped/DefinitelyTyped","main":"","scripts":{},"author":{"name":"midknight41","email":"https://github.com/midknight41"},"license":"MIT","typings":"index.d.ts","dependencies":{"equire(\"stream\"":"*","@types/node":"*"},

Notice: "dependencies":{"equire(\"stream\"":"*","@types/node":"*"},
This should most likely be just "dependencies":{"@types/node":"*"},.
// @RyanCavanaugh

Fixed and republishing everything as we speak (currently in the Rs). Thanks for trying this stuff out!

yay! thanks. I came across another broken package. I'll file a new issue for that.


removed rendermode default option

removed render mode default option

@doneill please review

@PuneetEsri Looks good, merging now.


enable grafana anonymous access

My question is, how could I integrate the generated grafana config in grafana-deployment.yaml?
What did you do?
I did a customization for enabling grafana anonymous access like this:
{
   _config+:: {
     namespace: 'monitoring',
     grafana+:: {
         config: { // http://docs.grafana.org/installation/configuration/
           sections: {
             "auth.anonymous": {enabled: true},
           },
         },
       },
   },

 };

A grafana-config.yaml is generated correctly.
Did you expect to see some different?
I had expected that this secret is used in grafana deployment descriptor, grafana-deployment.yaml, but it seems not.
Environment

Prometheus Operator version:
prometheus-operator:v0.29.0

Kubernetes version information:
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"windows/amd64"} Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.10-eks-5ac0f1", GitCommit:"5ac0f1d9ab2c254ea2b0ce3534fd72932094c6e1", GitTreeState:"clean", BuildDate:"2019-08-20T22:39:46Z", GoVersion:"go1.11.13", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
I use AWS EKS 1.13.8

I appreciate for any help.
Thanks
Dingjun

I did similar and did get find that the config was referenced under volumes in my grafana-deloyment.yaml:
  - name: grafana-config
     secret:
       secretName: grafana-config

Yes what @blandflakes said is what should happen.


Would it be possible to apply Dark Reader on top of pdf.js?

I want to apply dark-theme on top of pdf.js (dark background)
Would it be possible to apply https://addons.mozilla.org/en-US/firefox/addon/darkreader/ on the of pdf.js?

Duplicate of #2071


How to works with ES6 (ES2015)

Coverage remained the same at 100.0% when pulling 97c301edf1d73aefe12cd246fc0414d793e63a28 on yuricamara:master into 1eb00a85f465c70f8ecfe19bdee638bd359c54d7 on terinjokes:master.


About the code of prompt search

Hi, thanks for your excellent work.
Painter provides three ways of prompt tuning in paper, but I only found the code of random prompt, which means setting the single name of an image in training set.

Can you point out where is the code of searched prompt or learned prompt?
I would be appreciate.

I have the same question


How do you strip html tags when pasting?

I've tried using the cleanpaste and allowtagsfrompaste plug-ins and none will clean a pasted drop from a drag.

Because drop is not paste and so we maybe need to check the new event. Maybe you could help on this :)

I did try to incorporate a drop event into the main script and was able to get as far as getting the value but couldn't get the value into place into the editor. Briefly looked at making an add-on but didn't know how to integrate. Any helpful tips and I can certainly pursue further.
Regards,
Tom Hare
On Feb 17, 2018, at 7:54 PM, Alexandre Demode <notifications<EMAIL_ADDRESS>wrote:
Because drop is not paste and so we maybe need to check the new event. Maybe you could help on this :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAlex-D%2FTrumbowyg%2Fissues%2F716%23issuecomment-366488585&data=02|01||9ba8fd2b6f4242d21da708d5767ae1e1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636545192497774973&sdata=hl%2F1jzl4NuoFCBOM7Aok480kknuuycwM2ieScx7ZwP8%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMLxFCljoSQCmMyBqhfRtehK-OJLO0FMks5tV5DPgaJpZM4SJf43&data=02|01||9ba8fd2b6f4242d21da708d5767ae1e1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636545192497774973&sdata=F%2B%2B9YCnOu0ccGAhnFB7DFew2T6jfGbPlKmVdHoI6BWk%3D&reserved=0.


Translate Chapter 5 and modify the source file format

Description

Related Issue: CC https://github.com/rust-embedded/rust-raspberrypi-OS-tutorials/issues/166
Pre-commit steps

[ ] Tested on QEMU and real HW Rasperry Pi.

Not needed if it is just a README change or similar.

[x] Ran ./contributor_setup.sh followed by ./devtool ready_for_publish

You'll need Ruby with Bundler and NPM installed locally.
If no Rust-related files were changed, ./devtool ready_for_publish_no_rust can be used instead (faster).
This step is optional, but much appreciated if done.

CC @colachg @readlnh  Please review, thanks.

@posborne Please review, thanks.

@eldruin Hi friend, I noticed that three friends who maintain Chinese translations have not been active on Github for a long time, and the PR for this translation may not receive feedback. Therefore, I suggest finding a partner to proofread translation files. I want to complete this work earlier.

Hi @Jzow, I have asked in our chat room but nobody responded.
Do you feel confident enough with your translation so that we could merge it as-is (after rebasing to master so that CI works)?
Users can always submit improvements/fixes later on.

I have asked in our chat room but nobody responded.

Thanks for your prompt reply.

Do you feel confident enough with your translation so that we could merge it as-is (after rebasing to master so that CI works)?

Firstly, I am happy to do translation work, making it easier for some friends to read. At the same time, I can also understand embedded knowledge from it so that I can try to contribute code from rust-embedded repositories in the future. I believe I should be competent for this translation job. If there are any translation issues, I will proofread and update them at any time.
If there are new colleagues to submit improvements/fixes in the future, that would be great.

Ok, thanks. Once you rebase this to master so that CI works, I will merge it.

Don't need to change to the master branch, just reopen it and have it perform CI validation once.
Each chapter translation should correspond to a branch, and after merging, I will delete the corresponding branch of my fork.


Normalize format of config file

This will let us script changes to it with less noise in the diffs.

Thanks @kytrinyx.  I just merged them in order.  I think everything should be OK.


README: add a community Docker MCP server

Description
Adds a Docker MCP server to the Community section of the README
Server Details
n/a
Motivation and Context

It can be tedious to spin up containers, even with a Docker Compose file.
An LLM is a powerful natural language interface for running containers, needing only a few words to orchestrate potentially many containers.
How Has This Been Tested?

All resources, tools, and prompts have been tested with both the Claude desktop app and the MCP Inspector.
Breaking Changes

n/a
Types of changes

[ ] 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 change)
[x] Documentation update

Checklist

[x] I have read the MCP Protocol Documentation
[x] My changes follows MCP security best practices
[x] I have updated the server's README accordingly
[x] I have tested this with an LLM client
[ ] My code follows the repository's style guidelines
[ ] New and existing tests pass locally
[x] I have added appropriate error handling
[x] I have documented all environment variables and configuration options

Additional context

n/a

Great, thanks!


Fix WMT dataset loading issue and docs update (Re-opened)

This PR is a fix for #4354
Changes are made for wmt14, wmt15, wmt16, wmt17, wmt18, wmt19 and wmt_t2t. And READMEs are updated for the corresponding datasets.
Let me know, if any additional changes are required.
Thanks

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


MGDOBR-430: fix Slack processor creation

MGDOBR-430
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Great. Pretty much identical to my PR.

yes let's close this @kostola

Closed in preference to https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/456

Closed in preference to https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/456


capture broadcast work first time only

when i broadcast capture first time it work fine. when i broadcast it again it not perform capture.   why is that?

Having the same issue here also... Capture works only on first broadcast, then it refuses to capture after that. Any help will be appreciated. Thank you

try to broadcast ngWebcam_off and ngWebcam_on events in on-capture-complete callback function, may be it helps you.

ok... I'll try that. Thanks a lot


Generic TCP proxying support?

Already tracked by several issues (#10, #608, #1409, #1376), but all of them have been closed / locked and I didn't see any answer from the maintainers.
Are there any plans to support generic TCP proxying in Traefik? I believe it would be super useful to get insights and statistics on connections to services such as MySQL, Kafka, RabbitMQ, etc.

Duplicate of #10

That's indeed a duplicate of #10. I wanted to write there, but the conversation is locked and restricted to project maintainers.
The goal of this issue was to discuss whether a generic TCP proxying feature was planned for the future - which other issues do not have information about.
Thanks.

Yes it's planned for the future.


Commit Localization MaxLengthAttribute and compareAttribute

I made a fix for aspnet-5 localization for ErrorMEssage in the MaxLengthAttribute and compareAttribute.
See issue https://github.com/aspnet/Mvc/issues/3527 and all other related issues

Hi @Anderman, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@dougbu Is this the right way todo this?. I can make a fix for other ErrorMessage localization

@Anderman this looks like a solid improvement over what we have today. Main question is about the higher-level design i.e. params versus adding the following or some other approach.
protected virtual LocalizedString GetLocalizedErrorMessage(string displayName)
{
    return _stringLocalizer[Attribute.ErrorMessage, displayName];
}

@kirthik what fits best with how you've done localization elsewhere?
Separately, should fix DataAnnotationsModelValidators at the same time as the DataAnnotationsClientModelValidator<TAttribute>s.

@kirthik the [EmailAddress] attribute set Attribute.ErrorMessage. The code use this to chose between    the _stringLocalizer[] and the default Attribute.FormatErrormessage()
A workaround could be something like this
CustomErrorMessageSet = attribute.ErrorMessage != (new DataTypeAttribute(attribute.DataType)).ErrorMessage;

Any suggestions?

I found the problem. It is in System.ComponentModel.DataAnnotations/DataAnnotations/EmailAddressAttribute.cs
This line was set 28 days ago. But it should not be set at all
DefaultErrorMessage = DataAnnotationsResources.EmailAddressAttribute_Invalid;

@Anderman : You implemented one way of solving this issue. @dougbu suggested another way. I am working on a prototype to address this issue. I will let you know once we have a finalized design.

Oke fine, I created just a issue about the emailadressAttribute


Add meat crate to cargo

About the PR
Screenshots

Changelog

:cl:

add: You can now buy a crate of meat for a barbecue in the cargo department.

Can you not just order animals from cargo? I feel like having to butcher live animals is more interesting than just buying meat in a crate.

i dont see why this needs to exist when you can just buy animals yeah

Just buy a live animal, more interesting.

@moonheart08

Just buy a live animal, more interesting.

Although, I agree that livestock crates are more interesting in gameplay and department collaboration terms, I pretty sure that separate meat crate can coexist as alternative, simpler but expensive solution.
Perhaps, price can be increased up to 4k points, simultaneously with chicken (currently 2.5k\4 chickens), duck (3.5k\6 ducks), and cow (3.2k) crate's price decrease.
However, if your decision is final - so be it.


chore: Upgrade bf-lu to latest dev version

In https://github.com/microsoft/BotFramework-Composer/pull/7399, the bf-lu package was upgraded to dev-20210602 version.
There are two more places using bf-lu. In this PR, I also upgraded the package version here.
#minor

Coverage increased (+0.01%) to 50.339% when pulling 661037245f34c4698b744974d42aeaa9f9d9d395 on shuwan/upgradeBfLu into 868a0adbefb210efb73c63eb3573b5958530b759 on main.


code comments for Scheduler and ComponentLink

Description
Code comments to help clarify the ComponentLink interface, and help others understand how the Scheduler currently operates.

thanks @jstarry for the review help! enjoying getting stuck more into yew :)

I think this is probably ready for merge, might like to do a squash though beforehand to take out some of those extra commits.

I usually squash and merge 😉


Move to STL Containers

network_interface currently uses mostly C-style arrays. This should be updated to use STL containers like vector or deque.

@rgunjala93 - Can you please test the branch maint/return_vectors with the following repos/branches:

ibeo_core : maint/updates_to_network_interface
ibeo_lux : maint/updates_to_network_interface

We need validation that this new method will work and it will allow us to make the required updates to ibeo_lux and ibeo_scala.

Closed by #25.


FR: Add hotkey to toggle timer display

There's currently no way to toggle the display of the timer without removing the mod and restarting the game. Requesting a configurable hotkey to turn the display on and off. Otherwise a great mod!

Has been added with v0.2.0 for Minecraft 1.21/1.21.1


Added layout for People page

Fixed lookups in new branch - see https://github.com/AndersenLab/CAENDR/pull/177


[BAT]add endpoint case

Coverage decreased (-0.03%) to 54.431% when pulling 8c469f76f6ee664bcee57f674ce0672a1d27b413 on sigsbee:endpoint into fa495d2763fa601913e65da351013d72e7d2b955 on vmware:drone.


Regression test with next incorrect symlink handling

charybdefs 11 and 10

I suspect it fixed indirectly with #37.


[augmentation] fix type hints for Output of Transform.forward

[x] fix Output type annotation, now reflecting that it can return y as a
tuple of torch.Tensor
[x] define new Operation type hint, to annotate the Transform.operation
attribute, to better check operation return types
[x] define Batch, Output and Operation as type aliases

seems that TypeAlias was not supported before Python 3.10, removed it
fixed the trailing whitespaces as well (flake8 should now pass, although we should switch to ruff - yes, saw the other PR)

@bruAristimunha any idea why the Windows test are failing? seems it's failing why before running tests, during a pip install that fails

Okay. Thank you @tgy for fixing the CI for the window. All good for me =)

Thank you for fix type @tgy


A11y_AzureDataStudioSQLMigration_AzureSQLMigration_Colorcontrast:The link color is similar with the text in the screen and when hovering over the link using mouse pointer there is no visible underline provided for the link.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#SH_AZUREDATASTUDIOSQLMigration_June2021;#SQL Azure Data Studio;#Win10;#FTP;#DesktopApp;#WCAG1.4.1;#AINotInScope;#Useofcolor
Environment Details:
Application Name: SQL Azure Data Studio
Windows Version: Win10
Repro Steps:

Launch the application-Azure Data Studio-Insiders and install extension Azure SQL Migration.
Connect to any sql server.
Tab till Azure SQL migration Tab and press enter.
Tab till learn more link and check if there is a visible indication of underline present on the link.

Actual:
The link color is similar with the text in the screen and when hovering over the link using mouse pointer there is no visible underline provided for the link.
This issue is observed with links throughout the application.
Expected:
When hovering over the links there should be a visible underline provided for the links.
User Impact:
Users facing difficulty in distinguishing colors will not be able to identify the links.

@alanrenmsft, is this something we want to add?

Yes, all links should have the highlight + underline, so if this isn't happening for modelview links or something we should fix that.

none of the links in ADS and VSCode have underline when hovered, it is by design, @Shivamd1 which MAS rule requires this?

just noticed the WCAG 1.4.1 rule, it says we should use a color to indicate a link, which we already did, the color of the link is different from the normal text.

please reopen if this is a new requirement to have underline for links. @Shivamd1

That's not true - for example the link button in Notebook toolbar :

and then also links in Notebooks themselves :

This is a pretty standard thing, but let's see what the a11y folks say about that being a requirement.
The links would need to have some sort of focus indicator though so (such as the box you added) so if that isn't happening then we should be fixing that for sure.

I have verified the issue in the latest build 1.42.0 issue is fixed hence closing the bug

GithubTags:#Closed;#Regressed:02-20-23;


Issue installing librealsense module

Required Info

Camera Model
D435

Operating System & Version
Win (10)

Issue Description
Hello team,
I am completely new to the overall idea of working with camera and realsense in specific. And I am facing issue at the very 1st step of installing the librealsense.
I tried installing the librealsense following the steps mentioned in the node-librealsense page.
I have set all the necessary prerequisites in terms of cmake, visual studio, python2.7. All the paths are set correctly. And to the best of knowledge I have installed the correct 64bit version for the needed softwares.
However, I am getting this below error when I perform npm install.
I have tried uninstalling everything and reinstalling it again multiple times to get this thing work but still the same error reoccurs.
ERROR:

CMake
Error at CMakeLists.txt:4 (project):
Generator
NMake Makefiles

does not support platform specification, but platform
x64

was specified.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
'Failed to configure librealsense'
E:\2_Unhali_sem\SP_Kray\NewLasttry\SP-RCARAP\node_modules\node-librealsense\scripts\npm_dist\build-librealsense.js:67
throw new Error('Failed to build librealsense, build-dist.bat exited with code ${code}');
^
Error: Failed to build librealsense, build-dist.bat exited with code ${code}
at ChildProcess.child.on (E:\2_Unhali_sem\SP_Kray\NewLasttry\SP-RCARAP\node_modules\node-librealsense\scripts\npm_dist\build-librealsense.js:67:13)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at maybeClose (internal/child_process.js:925:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>preinstall: node scripts/npm_dist/build-librealsense.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Lenovo\AppData\Roaming\npm-cache_logs\2018-06-06T11_03_21_144Z-debug.log

In the error logs below message was displayed.

142 verbose argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
143 verbose node v8.11.1
144 verbose npm  v5.6.0
145 error code ELIFECYCLE
146 error errno 1
147 error<EMAIL_ADDRESS>preinstall: node scripts/npm_dist/build-librealsense.js
147 error Exit status 1
148 error Failed at the<EMAIL_ADDRESS>preinstall script.
148 error This is probably not a problem with npm. There is likely additional logging output above.
149 verbose exit [ 1, true ]

Ach

It may not be an issue as such but could find anyone facing this issue on other forums and groups. Any help would be appreciated by my team even if it may turn out to be a trivial issue. :)

Hi, @AkhilJPatil That seems to be an env issue. Did you use the Developer Command Prompt for VS xxx? That prompt should have setup the dev env for you. I used Developer Command Prompt for VS 2017.

[Realsense Customer Engineering Team Comment]
@AkhilJPatil
is the issue still there?


fix(userspace): add explicit constructors and initializations

Signed-off-by: Jason Dellaluce<EMAIL_ADDRESS>What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
This cleans up the codebase by adding explicit constructors and initialization, and makes the copy/move semantics explicit  where missing. This should solve some tedious bugs and non-predictable behaviors (and make cppcheck happier).
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

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

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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

LGTM label has been added.  Git tree hash: ecaca46288e027c630fcdf0c1d1a2af470f5c5dd

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

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


switch to poetry-core

See https://github.com/python-poetry/poetry-core#poetry-core.

Hey !
Can you rebase your PR (so it triggers the pipeline) ?

Working on siblings repos:

hacf-fr/meteofrance-api#260
hacf-fr/synologydsm-api#84

Merging.


Allow to configure font settings for preview

This PR changes preview type to kUITypeHTML and adds keys to user defaults: fontFamily, fontSize, lineHeight.
It solves issue #31
Configuration examples:
$ defaults write com.whomwah.quicklookstephen fontFamily "InputMono-Regular"
$ defaults write com.whomwah.quicklookstephen fontSize "13px"
$ defaults write com.whomwah.quicklookstephen lineHeight "21px"

Thanks @estum. Will merge this soon.

Any chance this will be merged soon? Cheers!


Missing functions on mpl.py

python 2.7.12 64 bits, Qt 4.8.7, PyQt4 Spyder 2.3.8 windows 10
version_version = 'b1abf5e2f1b444cfd5251627710510372e27c33a'
version_full = 'b1abf5e2f1b444cfd5251627710510372e27c33a'
I tried to run http://www.fatiando.org/cookbook/gravmag_mag_prism.html and got error messages because the following functions are missing on mpl.py:
mpl.figure()
mpl.title()
mpl.axis()
mpl.colorbar()
mpl.xlabel()
mpl.ylabel()
mpl.show()

@ffigura try this:
from fatiando import mesher, gridder, utils
from fatiando.gravmag import prism
from fatiando.vis import mpl, myv
import matplotlib.pyplot as plt

[rest of the code]

plt.figure()
plt.title("Total-field anomaly (nT)")
plt.axis('scaled')
mpl.contourf(yp, xp, tf, shape, 15)
plt.colorbar()
plt.xlabel('East y (km)')
plt.ylabel('North x (km)')
mpl.m2km()
plt.show()

Is myavi working for you? My problem is with the myavi original package

Hey @victortxa, thanks for the reply. I did this workaround when I faced the issue, I reported because something need to be changed: the example in the cookbook or add the function in the library mpl.py. From my point of view these functions should be added to the lirbrary in order to keep everything on mpl. style.
Myavi worked fine in this verison

I think it's because this module will be removed in the version 0.6. But, it is true it can be confusing.

@ffigura Do you get errors or warnings?
These functions are the same ones from matplotlib.pyplot so you should use that instead. We're removing support for vis.mpl in the near future. The cookbook is also very outdated and will also be removed in favor of the new Gallery (here is a preview http://www.fatiando.org/dev/gallery/).

@leouieda as I said the error messages are because the example at the cookbook calls the functions on mpl.py that I listed but this functions are not on that library. The workaround I did was the same as Victor suggested, call directly from matplotlib.

@ffigura I just noticed that you're using the latest development version of Fatiando. Those functions were removed and the whole module will be reworked before v0.6. If you're using the development version, then also use the development version of the docs: http://www.fatiando.org/dev/
Otherwise you'll get these kinds of errors because we changed something in master.

Great, thanks for the clarification @leouieda! I was worried that something would be missing in a new version, this happens often.


Enable glog printing to stderr for KNE based testing

Pull Request Functional Test Report for #1758 / 52764e933a8aad3c1a83047f7e389d4c28dd2fc1
No tests identified for validation.
Help

2023-06-14 10:52:07.525 PDT
Step #3 - "cisco_xrd": + go test -alsologtostderr -v ./feature/system/tests/... -timeout 0 -kne-topo /tmp/kne/cisco/xrd/dutate.textproto -vendor_creds CISCO/cisco/cisco123
2023-06-14 10:52:07.661 PDT
Step #3 - "cisco_xrd": no Go files in /tmp/featureprofiles

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.722%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1320

Relevant Lines:
2766

💛  - Coveralls


GH-38630: [MATLAB] arrow.array.BooleanArray's toMATLAB method does not take slice offsets into account

Rationale for this change
While working on #38415, I noticed that the toMATLAB method of arrow.array.BooleanArray does not take the slice offset into account. This will cause the toMATLAB method to return the wrong value.
What changes are included in this PR?

Updated arrow::matlab::bit::unpack function to accept a start_offset as input.
Updated clients (BooleanArray::toMATLAB and Array::getValid) to supply the array offset as the start_offset.

Are these changes tested?
The existing tests cover these changes. Additionally, the changes for #38415 will include tests that verify the toMATLAB method returns the correct MATLAB array when the underlying Arrow array has been sliced.

Are there any user-facing changes?
No.

Closes: #38630

The dev failure seems unrelated to my changes.

Looks like #38639 will fix the Dev/Lint CI failure.

Looks like the Dev GitHub Action failure was fixed by #38639.

+1


FALCON-1611: Falcon Admin Commands

@prongs Can you please rebase this pull request? Sorry for the churn, will take it up on priority once you rebase.

Closing in favor of https://github.com/apache/falcon/pull/249


Add setter/getter for Pose's each element - Issue 35

Solved #35
Add methods to Pose3.hh to get X,Y,Z,Roll,Pitch and Yaw directly, as suggested in the issue description :

pose.Pos.X() -> pose.X()
pose.Pos.Y() -> pose.Y()
pose.Pos.Z() -> pose.Z()
pose.Rot.Roll() -> pose.Roll()
pose.Rot.Pitch() -> pose.Pitch()
pose.Rot.Yaw() -> pose.Yaw()

@chapulina Here we go! first PR!! \o/

are you able to merge this branch with ign-math6? it is currently out of date with the base branch

[DO NOT MERGE] @chapulina @scpeters
This code is not finished. The approach used to implement the mutable X(), Y() and Z() can not be used in Roll(), Pitch() and Yaw(). When this approach is used I got these errors:

That is expected because the Quaternion.Roll() , Pitch() and Yaw() are const : https://github.com/ignitionrobotics/ign-math/blob/ca49e983bc3939b6626e78f036ff5ea9bf10d270/include/ignition/math/Quaternion.hh#L423
How should I solve this point? I thought in some ways but, all of them I need to edit the library Quaternion.hh and, also, there is pattern and architecture questions, so I decided to ask

humm nice catch
In fact, this reminds me that we've been deprecating the mutable accessors in favor of Set* functions, see the Migration guide and issue #101 .
So I think the solution is to substitute the mutable functions with Set versions.

Humm, nice, so, maybe, stop working with this issue and help with this functionality of the Quaternio.hh ? After that continue to working here?

Oh I don't think you need to address the entire issue. I'd just change the new functions you're adding. Remove all the mutable functions and add:
Pose3d::SetX
Pose3d::SetY
Pose3d::SetZ
Pose3d::SetRoll
Pose3d::SetPitch
Pose3d::SetYaw

Ok, good, should I remove Pos() and Rot() mutable, as well, and change for SetPos() and SetRot(), or is it better to do this in another PR?

should I remove Pos() and Rot() mutable, as well, and change for SetPos() and SetRot()

Good call, yes, please!

Ok, good, should I remove Pos() and Rot() mutable, as well, and change for SetPos() and SetRot(), or is it better to do this in another PR?

those API's are already released in ign-math6, so I wouldn't remove them in this PR

those API's are already released in ign-math6, so I wouldn't remove them in this PR

Ah sorry, thank you Steve. Let's only remove the ones being added by this PR.

This code is not finished. The approach used to implement the mutable X(), Y() and Z() can not be used in Roll(), Pitch() and Yaw().

Quaternion doesn't allow modification of Euler angles independently; it only allows them to all be set at once. I would suggest simply adding Get functions for roll, pitch, yaw in this PR and address setting them in a separate pull request

This code is not finished. The approach used to implement the mutable X(), Y() and Z() can not be used in Roll(), Pitch() and Yaw().

Quaternion doesn't allow modification of Euler angles independently; it only allows them to all be set at once. I would suggest simply adding Get functions for roll, pitch, yaw in this PR and address setting them in a separate pull request

Ok, It was I thought, and also using the Euler method would not be a good idea now since it is deprecated, so I'll do your suggestion!
Thanks @scpeters and @chapulina

@scpeters @chapulina sorry for the delay. Is this the way that you suggested? About the failure of the CI test, I couldn't understand the error, what should I do to fix it?

it looks like the build machines had an error; I'll start the jobs again

@osrf-jenkins run tests please

Thank you @scpeters!


IoReader enhancements

These patches are by-catch of my work on #79, but should can stand on their own; in summary:

Add more test cases for indefinite length strings (to catch what I think would be the most likely to go wrong with the following changes)
Simplify .read_into() by using a provided method of the Read trait that does mostly the same (what's left is EOF error wrapping)
In .read(), use Read::take() (with Read::by_ref() as in its example to keep the original reader in place) to directly put the read data into the buffer vector without zeroing out a fresh area before

I haven't run any benchmarks on them, but I expect the read_into change to result in the same machine under optimization, and the read change to negligibly remove a cheap memory clearing -- but the former reduces code complexity, and the latter uses the vec's length cursor (which is available anyway) to more clearly express which portion of the scratch buffer is usable.

Looks good to me! But I found the code complicated to understand. Would mind to add some comments to the reader?

I've pushed the requested additional comments along with a typo as fixups, which I'd squash once the PR is good to go otherwise (or earlier if you prefer that workflow).
There's an additional commit on top now that does away with the "to_read be only up to 16k" loop, because I've read up on the read_to_end function that's now being used, and that already does something similar internally. The requested size truncated to 16k is still requested upfront to utilize the knowledge we have of the expected size, but if it's actually more, read_to_end will do about as well as the loop had.

Good work. I think this is good to go.
Do you have any more comments @sfackler?

LGTM other than the possible need for a check for n == 0.

Squashed into sensible commits (ie. all fixups resolved).
The n==0 was a good point, it's now fixed, commented and tested for. (Though the peek-and-then-becomes-0 branch is something I wouldn't know how to trigger w/o testing deep into the internals.)

Rebasing the other branch, I noted that the promised fix for n == 0 got lost somehow during the squashing or merging; I've picked it from the reflogs and have it as the first commit in the next series upcoming for #81 (where I'm ironing out a few readability issues).


Return ThinkResult.NoJob;  on map start with modlist

// Token: 0x060028B1 RID: 10417 RVA: 0x000EFFE4 File Offset: 0x000EE1E4
private ThinkResult DetermineNextJob(out ThinkTreeDef thinkTree)
{
ThinkResult result = this.DetermineNextConstantThinkTreeJob();
if (result.Job != null)
{
thinkTree = this.pawn.thinker.ConstantThinkTree;
return result;
}
ThinkResult result2 = ThinkResult.NoJob;
try
{
result2 = this.pawn.thinker.MainThinkNodeRoot.TryIssueJobPackage(this.pawn, default(JobIssueParams));
}
catch (Exception exception)
{
JobUtility.TryStartErrorRecoverJob(this.pawn, this.pawn.ToStringSafe() + " threw exception while determining job (main)", exception, null);
thinkTree = null;
return ThinkResult.NoJob;
}
finally
{
}
thinkTree = this.pawn.thinker.MainThinkTree;
return result2;
}
Links to photos
https://ibb.co/p0wNQt9
https://ibb.co/hW6VD9Q

https://ibb.co/4SgS3J4

Closing bug. Many bugs have been fixed since this was reported. Please reopen in new bug report using the new templates if this issue persists.


Multiple Mac machines lost external connection from iPhone device.

Multiple Mac machines lost external connection from iPhone device.

All external connections have been restored by rebooting each machine.


fix: refactor RPC server

Looks like everything is cleaner now, server/rpc_server.go changes are so big that they are hard to understand on the first sight.

rpc_server.go is one massive file with massive functions. Trying to break it up to make it more digestable.

todo: don't retry on business logic error


Fix SR-7829: Bad error message when a tag doesn't exist

Fixes SR-7829.
Hi, this is my first contribution here, so let me know if there's anything I should do differently!
Given a Package.swift which depends on a tag which does not exist, like so:
.package(url: "https://github.com/apple/swift-package-manager.git", .exact("123.0.0"))

SwiftPM would generate a diagnostic like this:
error: dependency graph is unresolvable; found these conflicting requirements:

Dependencies:
    https://github.com/apple/swift-package-manager.git @ 123.0.0

After this patch it generates this error:
error: couldn't find tags for the following specified versions:
    swift-package-manager[https://github.com/apple/swift-package-manager.git] @ 123.0.0

Some questions:

Is the place I do the diagnosis here a reasonable place to do so?
Does it make sense to make this its own DependencyResolverError? The current solution prevents an unsatisfiable error from being thrown in the case that there's a missing tag in any of the dependencies. I thought that made sense because an error like this might confuse the rest of the dependency graph, but I could instead try to emit both diagnostics.
How does the wording of the error message sound? Should I switch to using the PackageReference's .path instead of using its description? I modeled it off DependencyResolverError.incompatibleConstraints, but the original error just uses the path.

Hey @aciidb0mb3r since you posted the original bug I wonder if you'd be the person to look at this or if you'd know the right person?

Sorry for the delay, I plan to take a look soon but I am a little occupied at the moment.

Not a problem! Just wanted to make sure you saw it.

Sorry it took so long. This is awesome, thanks a lot! I just made some minor tweaks to the original patch.

@swift-ci smoke test

Thanks!


Add text on how the event can be shared

Add to this page here:
https://pymc-data-umbrella.xyz/en/latest/2022-07_sprint/sprint_parties/community_partners.html#community-partners
Source page:  https://github.com/pymc-devs/pymc-data-umbrella/blob/main/2022-07_sprint/sprint_parties/community_partners.md

[x] add link to tweet:  https://twitter.com/DataUmbrella/status/1538875289652142080
[x] add link to LinkedIn post:  https://www.linkedin.com/feed/update/urn:li:activity:6944647561630662656

Closed #81 via #92.


Use deployment state for the horizontal

Fix for #352 #411 and #419

Love it, @tetienne! Do you want to notify users already to validate this fix, or do you want to make some changes first?

I tested it quickly this morning. It seems ok. We can indeed notify them.

I keep this PR as draft. I think I can still clean the code.

Wondering, if instead I can test this: "uiClass": "Awning",

@tetienne this one is still a draft, could it be merged?

@tetienne this one is still a draft, could it be merged?

I have to double check the code is still OK with my awning.

Works fine with my own device.

Ready to be merged


Width in samples

Hi, new to this area, could you explain what 'width in sample' indicates?
audiowaveform  uses

samples_per_pixel:

Number of audio samples per waveform minimum/maximum pair

Does it correspond to that? (not even sure if waveformjs outputs pairs). Also what does framecount indidcate?
Sorry for the dumb questions, just pretty confused atm.

I think it does correspond to that. However waveformjs output does not necessarily imply pixels; it's as if the audio file is resampled to "width in samples" total number of samples, each sample being one waveformjs number. Does that make sense?

I think so. Do you know if it is output as pairs? Audiowavform outputs pairs:

The next stage is to compute the minimum and maximum sample values over groups of N input samples such that each N input samples produces one pair of minimum and maxmimum points in the output.

No it's only one number per sample. If waveformjs allows pairs then it's changed since last I looked at it.

Thanks for that.


Update documented minimum for loginWindowWidth and loginWindowHeight

The documented minimum value for loginWindowWidth and loginWindowHeight may need to be updated.  When setting these  settings to their minimum value of 100 the cloud login webview is displayed full screen.  Increasing these values to 200 does show as expected with the webview being displayed very small.  The effective minimum value appears to be somewhere between 100 and 200.

minimum value was 101. Changed to 150.


fix: matomo possible fix

Description
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr559.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr559.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr559.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tapahtumat-pr559.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://liikunta-pr559.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://harrastukset-pr559.dev.hel.ninja :laughing::tada::tada::tada:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr559.dev.hel.ninja :rocket::rocket::rocket:


Schema Registry support for Python Client

Right now Python client does not support schema registry to produce & consume encoded messages

Fixed in #3391, #3511 and #3557


Cleanup hook fired at the beginning instead of at the end when using nested states

Operating system
Ubuntu 22 & Manjaro
Helmfile Version
0.144.0
Helm Version
3.9.0
Bug description
I have a helmfile with a global "cleanup" hook and a "postsync" and "postuninstall" hook per-release. Everything works like a charm for both "sync" and "destroy" commands and the "cleanup" hook is always executed after all the releases have been processed.
However, when I start using nested states, the global "cleanup" hook for the "destroy" command gets executed at the beginning of the process, right after logging into the registry.
I honestly don't understand this behavior and I don't see it documented anywhere.
Any insight will be welcome,
Thanks in advance
Steps to reproduce
https://github.com/teksuo/helmfile-nested-hook-bug
Relevant discussion
No response

@teksuo Hey! Thanks a lot for reporting. It's very unfortunately this isn't documented anywhere, but it's the expected behavior as of today.
Helmfile is designed around the idea that each sub-helmfile (i.e. nested state) is independent of the parent helmfile.yaml, and should be able to run directly. If the global cleanup was designed to be called after the nested states, it encourages the opposite. The same applies to environment values- environment values aren't automatically inherited down to the nested states, just to encourage you to make those independently runnable.
I'd guess you won't like it, but the best practice here is to repeat your global hooks config within every nested state. You'd want to make it DRY, so I'd suggest considering extracting the global hooks part from the helmfiles and load it by using {{ readFile }}.


Add draining contract

Phoenix.Socket returns {:socket, :draining} when phoenix is draining connections. Supporting this inside websock allows us to keep the socket transport agnostic. Companion PR will be opened on websock_adapter

I'm going to merge and release this so we can update deps to it in websock_adapter & bandit


Use tf.while_loop in iterative attacks

Transition all iterative attacks to tf.while_loop to prevent redundant graph construction.
This resulted in a 50x speed up using the Madry attack on a ResNet constructed from a Estimator model_fn.

WIP in #441


Design: Capture better data about individuals experiencing homelessness

We need to capture better data about those experiencing homelessness to facilitate case investigation and contact tracing, and better understand the impact of the pandemic on the homeless population (especially in congregate settings like homeless shelters).
Context from Sparkle:

I talked to the lead at CDC for capturing data for those experiencing homelessness in regards to the COVID response today. (This is going to be an issue for SURE in CO in the shorter term.) The issue being that if you don't have a permanent address, it can be difficult for public health to follow up with you AND public health is particularly interested in flagging positive cases in congregate settings like homeless shelters.
At the very least. we should consider adding options around "i live in a homeless shelter", " i live in a hotel/motel", and "i live in an encampent/outside" as alternatives to the address field. As this would help public health capture data around these populations.
It might also be good to then provide the option to share the contact information of different person who can also be contacted. As we are thinking about the patient experience...this might be something we want to add...
Some volunteer testing orgs are using their own address for homeless individuals, which is useless from a contact tracing perspective
Phone number is also an issue because when funds are limited you may not have a consistent phone anymore and may be limiting the minutes you use, etc. We were told to capture as much information as we can (phone number, email, etc.) BUT having the option to have an alternate contact was recommended.
Note from Alicia: An alternate contact would also be really useful in the K12 scenario to capture parent/guardien information too

Possible solutions:

Add a “no permanent address” checkbox near where we ask for address --> then surface additional options if that is checked. Right now address is a required field.

Guidance:

From Jeremy: There are a lot of guides out there for how to properly ask about homelessness, since due to stigma many people won't respond depending on how the question is worded.
Example: https://nhchc.org/wp-content/uploads/2019/08/ask-code-policy-brief-final.pdf

Duplicate


Feature/patty

updated no match page!

this can merge to develop instead of feature/front-end now since I merged front-end to develop (I cannot approve now that I edited the branch to merge to)


Rename SingleMotorExtensorJustForConnerBecauseHeWantsToRebuildTheRobotOnAWhim

Rename the SingleMotorExtensorJustForConnerBecauseHeWantsToRebuildTheRobotOnAWhim to something better

It will stay in the code until the end of the season


Open Open source

Hi, first let me say thanks for this wonderful plugin. If I understand it correctly, you don't have time to maintain this anymore, which while being a shame is understandable. So I propose making this an Open Open source project, where contributors gain commit access and can help maintaining this.
Thoughts?
http://openopensource.org/

Hi,
you're right, I can not maintain BashSupport any more. Even fixing bugs and keeping up with the IntelliJ platform takes a major part of my available time.
I'm willing to accept pull requests and to grant commit access to authors who contributed good code before.
But since there hasn't been a single line of code from another party in the last years, I don't think that this will speed up the development. There just seems to be a very few people working on IntelliJ platforms...

Closed for now. If there are contributors I'll be more than happy to accept their contributions and to grant access to the repository.


Crash on ModMenu open

Hi,
I'm getting a crash when opening Modmenu. Seems to only happen after a while in the world.
Noticed yigd in the logs along with a few mods by XanderIsDev.

[22:06:47] [Render thread/ERROR]: Error from mod 'yigd'
java.lang.ArrayIndexOutOfBoundsException: Index -1 out of bounds for length 129

It's a kitchen sink pack so probably just an incompatibility, but wanted to see if you had any ideas.
Thanks.
latest.log

Sorry for not replying sooner, I've not really felt like modding lately. I believe I can be of assistance now however. I realize this was reported 3 weeks ago now, but are you still having problems with this? Looks like the config file might be broken or something. If you've not initially changed anything in the yigd configs, you can try to just delete the yigd.toml file and let it regenerate. Do let me know if you still want help if there's anything I can do


Migrate DNS and NS management to Engineering, esp. Snowplow, including process docs for maintaining

Migrated from GitLab: https://gitlab.com/meltano/handbook/-/issues/65
Originally created by @afolson on 2022-03-01 00:37:57

I've moved DNS hosting from SiteGround to Route 53 and need to update the handbook. This should also include documenting the sp NS records per https://gitlab.com/meltano/marketing/website/-/issues/11#note_855914523

@afolson I see that Route 53 is documented in the handbook. I'm not sure what you mean above re: the sp NS records.  There is no reference in the linked issue to that.  There is a reference to here that says to make sure to document the differences for sp and why -- though there is no detail given. I will note that the NS records for sp are different, though:

sp.meltano.com.		21600	IN	NS	ns-1020.awsdns-63.net.
sp.meltano.com.		21600	IN	NS	ns-1191.awsdns-20.org.
sp.meltano.com.		21600	IN	NS	ns-2013.awsdns-59.co.uk.
sp.meltano.com.		21600	IN	NS	ns-355.awsdns-44.com.

and

meltano.com.		21600	IN	NS	ns-1488.awsdns-58.org.
meltano.com.		21600	IN	NS	ns-1953.awsdns-52.co.uk.
meltano.com.		21600	IN	NS	ns-309.awsdns-38.com.
meltano.com.		21600	IN	NS	ns-709.awsdns-24.net.

@magreenbaum - I'm assigning this to you but you can also delegate to another team member if someone else is able to pick up. This is one area where @afolson jumped in to help us get this set up, but we really should be owning in Engineering long-term. This was originally created solely as a 'process docs' issue int he Handbook repo, but feel free to open an infra-side issue as well if consolidation is helpful in long-term management.

@tayloramurphy @pnadolny13 You all set Snowplow up right? sp.* was just delegated to snowplow to manage DNS or something right? How is that managed?

Cc @WillDaSilva, @pnadolny13 in case they have additional context.

@magreenbaum I've invited you to the snowcatcloud channel in slack which has more info on the snowplow domain

Thanks @tayloramurphy for adding me to that channel. So we're just setting our NS records for sp.meltano.com to allow snowcatcloud to host the zone in their AWS route 53 is my understanding.

@magreenbaum As part of this effort I'd like to make sure we at least consider all of our domains.  The complete list is here.  Ideally we would have all of these with a single registrar (not, alas, possible), and a single DNS provider.

@therebbie sounds good. I'll update the docs for this issue as needed and then move the rest to an internal issue (around migrating to a single DNS provider).

Heads up @cjohnhanson, I've removed my assignment.


Hangs in REPL mode

Sometime taiko hangs in REPL mode, after issuing openBrowser command and does not allow to do anything. After pressing Enter or Ctrl + C the control comes back but none of the commands work after that.
Running in headful mod gives the following error
Error: unexpected server response (500)
   at ClientRequest._req.on (/usr/local/lib/node_modules/taiko/node_modules/ws/lib/WebSocket.js:653:21)
   at emitOne (events.js:116:13)
   at ClientRequest.emit (events.js:211:7)
   at HTTPParser.parserOnIncomingClient (_http_client.js:543:21)
   at HTTPParser.parserOnHeadersComplete (_http_common.js:112:17)
   at Socket.socketOnData (_http_client.js:440:20)
   at emitOne (events.js:116:13)
   at Socket.emit (events.js:211:7)
   at addChunk (_stream_readable.js:263:12)
   at readableAddChunk (_stream_readable.js:250:11)

Versions
Node - 8.11.4 LTS
Taiko - 0.1.0

After some debugging found out that if CRI is not able to connect to chrome it tries to connect again and again which cause the REPL to hung.
Ref - https://github.com/getgauge/taiko/blob/ddd7db3a57be5b6e9a1922d85928050077f583ed/lib/taiko.js#L42
The stacktrace -
{ Error: connect ECONNREFUSED ::1:9222
   at Object._errnoException (util.js:992:11)
   at _exceptionWithHostPort (util.js:1014:20)
   at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1186:14)
 code: 'ECONNREFUSED',
 errno: 'ECONNREFUSED',
 syscall: 'connect',
 address: '::1',
 port: 9222 }

Fix should be available in 271f059

Steps to replicate :

Don't map localhost to ip <IP_ADDRESS>
Try running taiko from REPL.

Working as expected with commit - a5da76d97ef5a1e6ca17534424747b96003c8b13


The 1660 graphics card does not work well with local sd

My laptop configuration：
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz   2.60 GHz
16.0 GB (15.9 GB 可用)
nVidia 1660
System Info：
Windows 11 home
22H2
I've deployed stable-diffusion-webui locally.  Use the following command to start: python launch.py --share --api .  Set the value  DEMO_STABLE_DIFFUSION_ADDRESS=<sd share url> in  docker/jarvis/.env .  When I was running autotest in webset, the cpu, memory and hard disk usage was 100%.

How much is your GPU memory ? Depending on the specific model you are using, Stable Diffusion can consume between 4GB and 10GB of video memory. So, you may need a more powerful graphics card.
Certainly, you can also optimize it by following these methods:

use --xformers when startup Stable Diffusion webui
set "enable_hr": Fasle, in stable_diffusion.module.py to disable upscale


Cannot reasonably sort in reverse order

The set of methods and classes does not allow for some trivial tasks, see example below:
In SQL I would use this simple query:
SELECT * FROM log WHERE log.contest_id = 'CxkmY77sD' ORDER BY log.utc_date_time DESC

In Dexie, I am always stuck in the last step:

db.log.where( { contest_id:  'CxkmY77sD' }).orderBy( 'utc_date_time').reverse() cannot be used, because where({...}) returns Collection, and Collection does not implement orderBy() (which, by the way, I would like the backend to do, i.e. I need that where() return Table, not Collection).

db.log.where( { contest_id:  'CxkmY77sD' }).sortBy( 'utc_date_time').reverse() also cannot be used, because Collection.sortBy() does not return Collection, it requires callback and passes Array as the callback argument. I do not want to sort in reverse order using the Array reverse() method, because there could be quite many items in the array and I would prefer the sort be handled before the program receives the array.

The method design is mutually inconsistent, skipping from one object type to another, and ignores basic operations. In fact, it would be helpful if the methods sortBy() and orderBy() had an optional parameter that would indicate sort in reverse order - this would be logical and most straightforward. A library should not require inventing workarounds for tasks that are supposed to be addressed by the library itself.

You can do db.log.where({ contest_id: x}).reverse().sortBy('utc_date_time'').

it would be helpful if the methods sortBy() and orderBy() had an optional parameter that would indicate sort in reverse order - this would be logical and most straightforward.
I couldn't agree more.
 .sortBy('utc_date_time desc')  would be much easier.


[multistage] Refactor StageMetadata from pinot-query-planner to pinot-query-runner module

Refactor StageMetadata from pinot-query-planner to pinot-query-runtime module

Codecov Report

Merging #10791 (4bcf0ff) into master (e302318) will decrease coverage by 20.74%.
The diff coverage is 0.00%.

:exclamation: Current head 4bcf0ff differs from pull request most recent head 31b2c9d. Consider uploading reports for the commit 31b2c9d to get more accurate results

@@              Coverage Diff              @@
##             master   #10791       +/-   ##
=============================================
- Coverage     34.43%   13.70%   -20.74%
+ Complexity      462      439       -23
=============================================
  Files          2159     2105       -54
  Lines        116044   113557     -2487
  Branches      17569    17272      -297
=============================================
- Hits          39961    15564    -24397
- Misses        72604    96720    +24116
+ Partials       3479     1273     -2206

Flag
Coverage Δ

integration1
?

integration2
?

unittests2
13.70% <0.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../pinot/query/planner/DispatchablePlanFragment.java
0.00% <ø> (ø)

...va/org/apache/pinot/query/runtime/QueryRunner.java
0.00% <ø> (ø)

...pinot/query/runtime/plan/DistributedStagePlan.java
0.00% <ø> (ø)

...ot/query/runtime/plan/OpChainExecutionContext.java
0.00% <ø> (ø)

...e/pinot/query/runtime/plan/PlanRequestContext.java
0.00% <ø> (ø)

...apache/pinot/query/runtime/plan/StageMetadata.java
0.00% <0.00%> (ø)

.../query/runtime/plan/serde/QueryPlanSerDeUtils.java
0.00% <0.00%> (ø)

.../runtime/plan/server/ServerPlanRequestContext.java
0.00% <ø> (ø)

.../pinot/query/service/dispatch/QueryDispatcher.java
0.00% <0.00%> (ø)

... and 769 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Should hide resource details from the overview page in Try

Should hide everything except maybe URL

I am not sure if we need to.
Today, when we give them access via MSA they also RBAC read access to the resourcegroup so they can log into the portal and see these details.
In short they have access to the data via API and also the resourcegroup if using MSA , so displaying it here wont really expose any more information.
@lindydonna  Is it okay to leave this as is ?

@paulbatum and I thought it was a bit weird to show these details, but I had forgotten about RBAC access with MSA.
Paul, what do you think?

OK fair, closing.


Update metric-related docs with the respect to recent metric updates in Kyverno

The metric design was recently upgraded in Kyverno to deal with cardinality explosion.
Refer: https://github.com/kyverno/kyverno/pull/2157
Update the metric-related docs to comply with the new metrics.

Milestone 1.4.2, yes?

Milestone 1.4.2, yes?

Yep


handle usage stats

An attempt (like a working one) to add getting execution statistics via time.
https://www.man7.org/linux/man-pages/man1/time.1.html
Example:
{
   "run":{
      "stdout":"hello world\n",
      "stderr":"",
      "code":0,
      "signal":null,
      "output":"hello world",
      "stats":{
         "elapsed_time":"0.20s",      <-- Elapsed real time
         "cpu_time":"0.00s / 0.01s",  <-- Total num of CPU-seconds that the process spent in user/kenrel mode
         "max_mem":"9636Kb"           <-- Max resident set size of the process
      },
      "exec_time":221
   },
   "language":"python",
   "version":"3.12.0"
}

Closing this PR since usage stats were handled in the newest version of Piston using Isolate. Thanks though!


"value XXX is not a member of object ammonite.predef.XXX"

Motivation
I’d like to split my predef file into multiple files, as suggested by the Ammonite docs.  Specifically, I’d like to put all Java-related imports in a file ~/.ammonite/java.sc and call that file from ~/.ammonite/predef.sc.
How to reproduce

Create a predef file in ~/.ammonite/java.sc with the contents:

import java.util._

Call this in the predef.sc file:

import $exec.java

Start Ammonite.

Loading...
Compiling ~/.ammonite/java.sc
java.sc:1: value util is not a member of object ammonite.predef.java
import java.util._
            ^
Compilation Failed

Expected behavior
The import should work.

Your java script object is shadowing the java package; name your script something else

Oh yes, I’m aware of why, mechanistically, this doesn’t work. That’s not the issue here.
I would argue this behavior violates the Principle of Least Surprise. Why? Predef files are Scala files (minus the import $… syntax), and so users expect them to behave like Scala files. Specifically, Scala files named foo.sc don’t silently create a package named foo. Even package object files explicitly declare package object foo. In short, users (understandably) don’t expect this behavior.

Pinging again. What are your thoughts?

Funnily enough I encountered this same problem, although it took me a while to recognize that it was the same problem -- I had wanted to learn about Shapeless and thought of using Ammonite as a quick way to get a sandboxed environment up and running quickly. Silly me named the file shapeless.sc, and it took me a good half hour to figure out why I couldn't import anything from import shapeless._.
The more you know :shrug:


add commands to create ceph quotas and directories

add commands to creat ceph dirs and quotas for new users

something's off with the circleci tests - it's happening in multiple PRs, so not an issue here


Fix unintentionally replacements

Looks like there were used search/replace which changed words that weren't intended.
This reverts the words which seems to have been unintentionally changed.

i use
regex to all files for typo Cal
([^oi])Cal([^maleipc]) -> $1Kyaru$2


Add 2 tests for getPercentages.

We have the private function created by Laura, called getPercentages(..). This morning we discussed adding unit testing for each of Laura's functions. I used '// @ts-ignore', yet it is not a good practice. Should we test by making the function protected and then add an extended class that calls the protected function? Or should Laura add all of her functions and test everything at the end through the public function, yet the concept of unit testing loses somehow its meaning? :/

if this works you can PR to master too, here you do not need our approvals ;)


Enhance the java doc of dubbo-config module.

Enhance the java doc of dubbo-config module.
Related project: https://github.com/apache/incubator-dubbo/projects/2

https://github.com/apache/incubator-dubbo/issues/2884

I can do this module

Ok, looking forward to your contribution.