Title suffix

This PR implements #11288.
How to test
Open a map, change the title, and check the title doesn't contain CartoDB as suffix. Then do the same for a dataset.

Acceptance: This is working for the dataset view, but in the map view I'm still getting the CartoDB title.

I focused on the metadata modal and I forgot about the rename map action we have in the header. Fixed.


Expand coverage of mitogen/-test docker images

Thanks for creating a PR! Here's a quick checklist to pay attention to:

Please add an entry to docs/changelog.rst as appropriate.

Has some new parameter been added or semantics modified somehow? Please
ensure relevant documentation is updated in docs/ansible.rst and
docs/api.rst.

If it's for new functionality, is there at least a basic test in either
tests/ or tests/ansible/ covering it?

If it's for a new connection method, please try to stub out the
implementation as in tests/data/stubs/, so that construction can be tested
without having a working configuration.

fixes #815

I expect CI to fail for now. All my testing so far is locally in Tox, I've not uploaded new Docker images anywhere. Before I do that I'll need to make sure the existing images are tagged in a way we don't lose access to them.

I've already uploaded the regenerated Dociker images, this is their first test in CI. Tests on my laptop using Tox have worked.


Prevent dragStart event getting fired when click event happens

Issue:
when i click the actions in grid Item it automatically trigger the dragStart event and preventing the click functionality getting  called sometimes.
Expectation:
Need the PR fix #1411  mentioned here / any alternatives. please merge it the stable branch and release it. Because this is majorly impact the usability of react-grid-layout in desktops and touch devices (ex. iPad).

Note: we use this library in production that really affect the user experience.

Environment:
"react-grid-layout": "^1.3.0"

I have the exact same problem.

We are also experiencing issues as described about above when users are using our app on an iPad. Users cannot click on any buttons inside react-grid-layout items but instead think the user it dragging the item the whole time. We also see this happening sometimes within HubSpot's dashboard items. Some items that can be clicked are being registered as dragging events. Are there any recommended solutions to distinguish between clicking and dragging events on buttons/links inside react-grid-layout items on an iPad?

Trying to get around this as well.

Same issue. Click events inside the item are not working, because the grid activates drag mode on click.

I have the same issue.

I got round the problem by setting the draggableHandle & draggableCancel props to the elements within the item. I defined the header area as draggableHandle, where the user can then drag and drop. The actual content area was provided with the class from draggableCancel.
draggableHandle='.dragHandle'
draggableCancel='.notDraggable'

Please check the documentation - Grid Layout Props
 // A CSS selector for tags that will not be draggable.
// For example: draggableCancel:'.MyNonDraggableAreaClassName'
// If you forget the leading . it will not work.
// .react-resizable-handle" is always prepended to this value.
draggableCancel: ?string = '',

So you can add a className={'MyNonDraggableAreaClassName'} to any element of the gid item and use any onClick handlers without being interrupted by the RGL.


[Proposal] Improve service invocation

In what area(s)?
/area runtime
Describe the proposal
This is to track the work to improve service invocation for gRPC. The detail spec will come soon.

[x] Document service invocation spec
[ ] Make user-facing HTTP/gRPC API behaviors consistent
[ ] Support standard headers - related to #1371
[ ] Support all http verbs for http service invocation

Dupe with https://github.com/dapr/dapr/issues/1409


Fix getVarchar value length  #332

#332

Coverage decreased (-0.1%) to 52.98% when pulling 7b15e675018b16de4436dd273e702870503bd453 on eric-haibin-lin:buf-fix into 56b6a6be45775af556d3b7eafdc4602ee3c9031a on cmu-db:master.

I just merged Yingjun's fix in another PR. Can you rebase on top of that? Thanks!

Coverage increased (+0.006%) to 53.123% when pulling 4e91f92c57b9d089cfb25eccbdc6ef0ce3a0c5d6 on eric-haibin-lin:buf-fix into b4503ee44fd313ba5382faa957cdd788391874ca on cmu-db:master.


Image thumbnails not available

I have been having an issue with thumbnails not displaying when I upload a larger image (over 1000px on the smallest side) unless I allow php to have more memory. Thumbnails that are generated and visible are of low quality compared with the original image (lots of jpeg artifacts). I'm using GD currently and not sure if I can move to ImageMagick with my current host.
I'd be fine with upping the php memory if the thumbnails looked better.
I'm not sure why 4 small square jpegs would swamp my php memory.
Your theme is my favorite I've used so far, and I'd hate to have to move to a different one

You didn’t ask any question, but “... unless I allow php to have more memory” sounds like you answered the question you didn’t ask yourself. You didn’t say how much you needed to give it, I’d say 512MB sounds reasonable for larger jpegs. Image processing requires memory, give it as much as it needs.

You're right, I guess I didn't ask a question.
Thanks for figuring out what I was trying to ask. :) From years of
Photoshop, I realize that image processing takes a lot of ram, I  didn't
think that just making a small jpg would be all that memory intensive.
On Mon, May 7, 2018 at 12:51 PM, Thomas Kuther<EMAIL_ADDRESS>wrote:

You didn’t ask any question, but “... unless I allow php to have more
memory” sounds like you answered the question you didn’t ask yourself. You
didn’t say how much you needed to give it, I’d say 512MB sounds reasonable
for larger jpegs. Image processing requires memory, give it as much as it
needs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tkuther/piwigo-bootstrap-darkroom/issues/163#issuecomment-387129075,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlQHzNNpLvwZRQEao6EZPFTNzh18XmsQks5twHufgaJpZM4T1QBh
.


Update to point to the GitHub repository

Update to point to the GitHub repository instead of the now defunct KerbalStuff.

Hi, a few things:

To fix the break you need a comma after resources and before depends.
The version will be pulled from the GitHub tag which currently contains a leading v, previous versions of the mod did not contain that. Use "x_netkan_version_edit": "^[vV]?(?<version>.+)$" to extract the pure numerical version from the tag.
The resources.repository will be set automatically, no need to hardcode it.
You'll probably want to update resources.homepage, due to the forum migration the link currently points to the main TAC Life Support thread, rather than any particular post.

Since you're updating your own metadata, you may want to consider self-hosting it so you can make changes on your own. Let me know if that's something you're interested in.

Build breaking because it can't download dependencies (cause Kerbal Stuff). Checked manually, looks fine, merging. Thanks and let me know if there are any issues.


[GithubIssues] Add examples for count github pull requests closed by label

This PR adds two more examples, as described by the title. It seems this functionality was always supported, but not documented.
For example, with the current API this works already:
https://img.shields.io/github/issues-pr-closed/badges/shields/service-badge

Closes #2536.
It also changes the pullrequests search tag to pullrequest for consistency.

Warnings

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @MonliH!

  Generated by :no_entry_sign: dangerJS against c4a37e49727fc41eed5716479e395437308b0193

How can I rerun tests? (I forgot to add [Github] to the title)

How can I rerun tests? (I forgot to add [Github] to the title)

You won't have access in Circle to manually re-run the workflow, but no worries. In cases like this we'll often do so on your behalf, and you can always update your branch either by rebasing or merging in latest changse

I think we'll need two additional example listings for closed issues with labels to be able to close #2536 in addition to the closed PRs which are now covered

Alright, will add those examples too.


Resumable, parallel downloads

~Currently we stream downloads to final dst and this means the file is corrupt if the download fails. Generally we assume the file existing means it is cool so it breaks shit.~
No, actually, we want *resumable downloads.

We got parallel downloads. Resumable is ideal. Moving to libtea


Fix cloud config default value and update naming convension

Codecov Report

Merging #1895 (988be21) into master (b1c75a7) will decrease coverage by 58.51%.
The diff coverage is 0.00%.

:exclamation: Current head 988be21 differs from pull request most recent head 8cafdea. Consider uploading reports for the commit 8cafdea to get more accurate results

@@              Coverage Diff              @@
##             master    #1895       +/-   ##
=============================================
- Coverage     74.00%   15.49%   -58.52%
+ Complexity     9644     2276     -7368
=============================================
  Files           689      689
  Lines         52966    52966
  Branches       6592     6592
=============================================
- Hits          39198     8206    -30992
- Misses        11715    44287    +32572
+ Partials       2053      473     -1580

Impacted Files
Coverage Δ

...main/java/com/github/ambry/config/CloudConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/cloud/VcrServer.java
0.00% <0.00%> (-84.41%)
:arrow_down:

...c/main/java/com/github/ambry/server/AmbryMain.java
0.00% <0.00%> (ø)

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 543 more

Continue to review full report at Codecov.

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


[Pods] DCOS-10183: Fix network labels destructuring

This PR addresses the issue of using network as labels because of a bug in destructuring.

LGTM! 🚢


Fix hovering methods not working correctly

Fixes #7020
#6764 probably removed a bit too much and caused that when hovering a method in the inherited section, the detailed representation of the method is shown at the wrong place.
This is fixed by adding back blur(0);.

This should use position: relative instead of filter: blur(0). Using filter does apply position: relative (lol wtf??), but the former is better to solve this bug.


Naming complaints with babel plugin

If I add the recommended code in my babel.config.js:
{
  plugins: [
    "@typescript-nameof/babel"
  ]
}

I get this:
Parsing error: Cannot find module '@typescript-nameof/babel-plugin-babel'
…
If you want to resolve "@typescript-nameof/babel", use "module:@typescript-nameof/babel"

If I make the suggested change my code works, but I'm wondering whether this is something that is better addressed in the plugin and the docs?

Thank you very much for pointing this out!
I will have this changed in the docs asap
Thanks for your interest in this project

I have updated the documentation properly now.
Test cases have been added for making sure that running the babel compiler using the package name (prefixed with module:) properly.
Thank you very much for reporting 😄


fix(breaking-change): update for nodes 18 and 20

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

all tests now pass, merging this


[FancyZones editor] Spacing issue

ℹ Computer information

Windows build number: Microsoft Windows [Version 10.0.19041.329]
PowerToys version: 0.20 release candidate
PowerToy module: FancyZones

📝 Provide detailed reproduction steps (if any)

Start editing a 2 column grid layout, with spacing=16
Add four more vertical splitters
Click apply
Start dragging a window to a zone

✔ Expected result
The zones are displayed correctly.
❌ Actual result
There's a noticable gap on the right side of the screen

@ivan100sic

Add four more vertical splitters

Can you clarify where the splitters should be added?

@enricogior
Anywhere, it doesn't matter, you will end up with a six-column layout. This is still present in 0.21.1.

Fixed in 0.23.0 https://github.com/microsoft/PowerToys/releases/tag/v0.23.0


Unhelpful error message when not importing the correct packages

If I run the following code
using Integrals # Here I forget to include Cuba

f(x, p) = sum(sin.(x))
domain = (ones(2), 3ones(2)) # (lb, ub)
prob = IntegralProblem(f, domain)
sol = Integrals.solve(prob, CubaVegas(), reltol = 1e-3, abstol = 1e-3)

I get the error message:
ERROR: MethodError: no method matching __solvebp_call(::IntegralProblem{…}, ::CubaVegas, ::Integrals.ReCallVJP{…}, ::Tuple{…}, ::SciMLBase.NullParameters; reltol::Float64, abstol::Float64)
An arithmetic operation was performed on a NullParameters object. This means no parameters were passed
into the AbstractSciMLProblem (e.x.: ODEProblem) but the parameters object `p` was used in an arithmetic
expression. Two common reasons for this issue are:

1. Forgetting to pass parameters into the problem constructor. For example, `ODEProblem(f,u0,tspan)` should
be `ODEProblem(f,u0,tspan,p)` in order to use parameters.

2. Using the wrong function signature. For example, with `ODEProblem`s the function signature is always
`f(du,u,p,t)` for the in-place form or `f(u,p,t)` for the out-of-place form. Note that the `p` argument
will always be in the function signature regardless of if the problem is defined with parameters!

I don't know the internals of SciML well enough to understand why I get this error, but perhaps this situation could be added to the list in the error message? Or is there a better solution?

What version of Integrals and SciMLBase?

Integrals: "4.3.0"
SciMLBase: "2.26.0"
I get this error in a blank environment with just Cuba.jl and Integrals.jl installed

Oh I see, because the extension isn't loaded. Yeah we can make a better error message for that case.


Insert new member to the list

I'm a Baloise employee and have to get a membership to the Baloise-incubator for our Code Camp. I've read and understood our Open Source Guidelines and agree to act accordingly.
Config changes proposed in this pull request

For new organization members
I acknowledge that

[x] I have read and understood the Open Source Guidelines
[x] I agree to act accordingly (with due dilligence) to our guidelines
[x] I have no open questions regarding the topics covered (please delete the "open questions" section)

@joachimprinzbach do I wait or do I have to write to somebody with the write access to merge this pull request?


Generate random values based on column type

Until #6 is implemented, return a constant number of rows (1000) for every table.

Done in d3f5bd1e5f3028ae0a3bbf7e9507a2dcfee13244


Use Travis CI

There are some good tests within the library. We should use Travis CI to make sure that there are no test regressions.

Thanks, add travis.yml for test


Button Text Clipping

🐛 Bug Report
Hello, I am trying to render larger text inside a button, but when it renders it clips the text in half. I have tried changing properties such as size and margin, but the render still getting cut both in IOS and Android
To Reproduce
Use code below
<Button style={{ width: '80%',borderRadius: 15, position : 'absolute', bottom : '10%'}} textStyle={{ fontSize: 20, textAlignVertical: 'center', marginTop: 10}}   >
Get Started!
</Button>

Expected behavior

Hi! You need to set lineHeight by passing it to textStyle prop if you would like to increase it. Because it is provided by Eva by default. I also recommend you setting it by costomizing mapping. See default values here

Thank you so much!


Testing on environments with no postgres root password set is currently not possible

I have a configuration that I am wondering if it is possible/wise to support for with profile. I am currently setting up postgres with no postgres password, and I have noticed that this makes it impossible to run this STIG check. Is this on purpose, since it is considered insecure to run in this configuration, or for a different reason?

It is required to have password for OS user as well DB superuser for security.


Support for php 8.1

Hi!
I would like to know if you have plans to support MonetDB-PHP for php 8.1.
I see that there hasn't been any activity this year and I would like to know the status of the project.
Thanks!

It will support it after the weekend.

@federicoemartinez Hi, the PHP versions 8.x are now supported by the newest version of the library: 1.2.1
You can upgrade to it with:
composer require tbolner/monetdb-php


Is it possible to have a default rule?

Is it possible to have a default rule? means if we have multiple rules in YML file. and No one rule condition is satisfied(True) in that case we should have a default rule which would be execute. just like a conditional statement if our any one rule is satisfied then default rule will not be execute and if no one rule is satisfied in that case default rule will be execute?

in skipOnFirstAppiedRule mode you can define a low priority rules to solve the problem.


FatFS using SDHC is hardcoded to 1-bit mode in diskio.c

Hi there,
I noticed slow SD card writes when using the SDHC library. Probing the data lines with an oscilloscope revealed that only the DATA0, CLK, and CMD lines were active. The DAT1 .. DAT3 lines were permanently high during data writes.
Browsing through the code and using the debugger, I found that the data width is hardcoded with the enumeration MXC_SDHC_LIB_SINGLE_DATA in diskio.c. The relevant line for the write function is shown below. The matching read function has the same hardcoded enum. I am using FF15.
https://github.com/analogdevicesinc/msdk/blob/5955fcf103fdb42f5a424dcc08eedbb942b520b1/Libraries/SDHC/ff15/source/diskio.c#L199
After manually changing the hardcoded enums to MXC_SDHC_LIB_QUAD_DATA in diskio.c, I do see all 4 data lines wiggling, and the SD card write speeds increase as expected.
Is there a reason for the SDHC to be hardcoded to 1-bit mode when using FatFS/can the ability to use all 4 data lines be added to the MSDK?
Thanks!

What chip are you using? I do not know the full history, but not all of our chips support QSPI operation, for ease it was most likely set to that to not have to deal part to part variation. I will add this to my list and will update this sometime soon.

We are using MAX32666. The current code is on a FTHR board, but we will eventually move it to a custom PCB. Thanks!

Thanks @JordanAceto, this is just a legacy hard-coded config.  We can and should expose this as a config option to the API.

Ah looks like @EricB-ADI is already on it in #1045


Fixing bug found in discussion #2987

Requirements
Mark the following tasks as done:

[x] Checked the codebase to ensure that your feature doesn't already exist.
[x] Checked the pull requests to ensure that another person hasn't already submitted the feature or fix.
[ ] Adjusted the Documentation.
[ ] Updated CHANGELOG.md
[ ] Added tests to ensure against regression.

Description of the Change
When you use the withValidator() method and call the $validator->errors()->add() method to generate a custom error as instructed in the documentation, if the error doesn't exists from a previous validation, it will raise an Undefined index exception. This small fix solves the problem as suggested by @patrickbrouwers .
Why Should This Be Added?
Fixes an exception when adding new errors to the validator
Benefits
You can add new errors to the validator
Possible Drawbacks
No tests
Verification Process
Test on local machine
Applicable Issues
Discussion #2987

Can you try to add a regression test for this + update the changelog

Can you try to add a regression test for this + update the changelog

@jmunozpcoco see my above question

@jmunozpcoco see my above question

Hey Patrick! So sorry for the delay, I'll catch up this weekend. I'm overflowed with work right now.


Update requiretype.py

This is clearer IMHO. Also more efficient due to short circuit evaluation.

You're right, good catch.
Thanks!


may line title needed？

if there more than one line, maybe it need a line title to let user know what the line means.
what do you think?

Hi, nice idea, I will try to add that to current implementation, thanks.

Yeah if we could display a "Label" on a Line without the points this would be great.


cache downloaded videos

Don't re-download videos that were just downloaded; ytgif should store videos in a cache and re-use them if available. This will make iteration much faster.
This means that there probably must be an option to delete the cache, since the downloaded video could be quite large. What's the right place to store them? /tmp/ytgif.cache or something?
Relates to #1 - we can probably push slicing to a step in between downloading and encoding as gif, or include it in the encode as gif step.

fixed in 680745a2348315eb486c9dae21c599074bc74be3


Bug - 404 error on shareNote

Hey guys,
looks like an awesome plugin, but its not working for me.
Error: Request failed, status 404
    at new t (app.js:1:710073)
    at Aw (app.js:1:710265)
    at app.js:1:710942
    at app.js:1:235505
    at Object.next (app.js:1:235610)
    at a (app.js:1:234349)

Inside Func     shareNote I keep on getting inside the catch block .catch(this.handleSharingError);
Title and body is looking good, seems like serverUrl is making some trouble?

I tried with various configurations inside plugin options, but It keeps failing
I also created a new empty Vault without other plugins, does not work here, too :(

Do you have any advice for me?

Hey Jonas!
Thank you so much for bringing this to my attention. It looks like the note storage service was down, which is why you are receiving the 404.
I just rebooted it and it should be working!


implement match_diff for Pathspec

This implements a higher-level binding for git_pathspec_match_diff.
It was a very straightforward implementation thanks to the way you set it all up. I just copied the match_tree method and substituted a Diff instead. Hope I didn't miss anything. It's working great on my end.
This enables me to, for example, add many pathspecs to a DiffOptions, compute a diff once (which I imagine is expensive) and then for each of those pathspecs I can see if it's contained in the resulting Diff using match_diff (I do this because say I add 3 pathspecs "A", "B", and "C" to the DiffOptions, I can't just assume that the diff contains them all, it could be that it only contains "A" and "B" but not "C", at least, that's my understanding).
If there's a better way to do that please let me know. Although, doing this instead of what I was previously doing already gave me a huge performance boost.

Thanks!


I gues

More of a question than an issue..
I was trying the plugin with Apple Sign-in and it requires the redirect_uri to be a domain not a localhost or an IP address. Just want to confirm I'm not misunderstanding and basically the plugin will not work in this case right? Any other alternatives apart from using Swift or something like objc2-authentication-services ?

Correct, this plugin serves solely the oppsite use-case. It was created for oauth implementations that require, or work better, with localhost redirect_uris (initially this was about deep links vs localhost, not a https requirement, but it still applies).
I don't know much about apple's requirements but if don't need cross-platform support than what you linked is probably best, though a swift solution via swift-rs would probably also work if it can link against that framework.
If you need cross platform support then you probably won't get around a small website i guess.


[exploreV2] variety of changes / improvements

supporting getProps in field definition, simpler semantics, less field-specific rules scattered in the codebase
fixed the loading spinner
made the first load async instead of using bootstrap data
changed a few components to align with react-bootstrap
cosmetic changes

overall removed 50 lines of code while adding important features and making the flow of data a big easier to track

@ascott @vera-liu

i'll review this monday.

I'm breaking this down into a dozen or so PRs, keeping open while the process is ongoing for reference.


mitmproxy crashes when I try to copy binary content

Problem Description
When I try to copy request content that has a binary format, mitmproxy crashes.
Coping binary data to the clipboard may be ridiculous but I just want to report this crash.
Traceback (most recent call last):
File "mitmproxy/master.py", line 54, in run_loop
File "urwid/main_loop.py", line 287, in run
File "urwid/main_loop.py", line 385, in _run
File "urwid/main_loop.py", line 1494, in run
File "urwid/compat.py", line 58, in reraise
File "asyncio/events.py", line 80, in _run
File "urwid/raw_display.py", line 416, in
File "urwid/raw_display.py", line 515, in parse_input
File "urwid/main_loop.py", line 412, in _update
File "urwid/main_loop.py", line 513, in process_input
File "mitmproxy/tools/console/window.py", line 316, in keypress
File "urwid/container.py", line 1123, in keypress
File "mitmproxy/tools/console/statusbar.py", line 201, in keypress
File "mitmproxy/tools/console/statusbar.py", line 149, in keypress
File "mitmproxy/tools/console/statusbar.py", line 169, in prompt_execute
File "mitmproxy/tools/console/statusbar.py", line 115, in execute_command
File "mitmproxy/tools/console/commandexecutor.py", line 18, in call
File "mitmproxy/command.py", line 273, in execute
File "mitmproxy/command.py", line 259, in call_strings
File "mitmproxy/command.py", line 129, in call
File "mitmproxy/command.py", line 303, in wrapper
File "mitmproxy/addons/cut.py", line 136, in clip
File "mitmproxy/utils/strutils.py", line 46, in always_str
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xf9 in position 3: invalid start byte
mitmproxy has crashed!
Please lodge a bug report at:
https://github.com/mitmproxy/mitmproxy/issues
Steps to reproduce the behavior:

Ctrl+L
@all request.content
Enter

System Information
Mitmproxy: 7.0.2 binary
Python:    3.9.5
OpenSSL:   OpenSSL 1.1.1k  25 Mar 2021
Platform:  Linux-5.11.0-27-generic-x86_64-with-glibc2.31

Thanks for reporting this, there are several related / duplicate issues already. #3965


Fix publish workflow

Codecov Report
Base: 42.45% // Head: 43.45% // Increases project coverage by +0.99% :tada:

Coverage data is based on head (2165c12) compared to base (0035ac2).
Patch coverage: 80.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   42.45%   43.45%   +0.99%
==========================================
  Files           7        7
  Lines         862      856       -6
==========================================
+ Hits          366      372       +6
+ Misses        496      484      -12

Impacted Files
Coverage Δ

src/lib.rs
17.02% <80.00%> (+1.23%)
:arrow_up:

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

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


Update Keras to note support for CNTK backend

Keras supports Tensorflow, Theano AND Microsoft Cognitive Toolkit (formerly CNTK). It's noted correctly in other parts of this repo (see https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/machine-learning/data-science-virtual-machine/use-deep-learning-dsvm.md). See also the Keras docs (https://keras.io/backend/), which call out support for CNTK.
As a side note, the Microsoft Cognitive Toolkit page (https://www.microsoft.com/en-us/cognitive-toolkit/) notes that CNTK is the former name, but CNTK remains in popular usage throughout much of the Azure Docs. Indeed, it is even referred to as CNTK in the Microsoft Cognitive Toolkit site. :-D  Is there guidance on fixing these other references? Can we continue to use the name CNTK in official documentation?

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

@StevenBorg Thank you! @gopitk approved your changes.
#sign-off

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

#sign-off


config dupes

from settings.json:
{
        "version": 1,
        "geth": {
                "url": "https://api.gastracker.io/web3",
                "type": "remote"
        },
        "chain": {
                "id": 61,
                "name": "mainnet"
        },
        "terms": "v1",
        "rpcType": "remote",
        "chainId": 61
}

rpc type configured twice, as well as chainId. do we need both?

No. Remove your settings.json


clickable table in docstat

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Sat, Oct 14, 2017 at 6:08 PM, Mark Wendell Cabuang <
<EMAIL_ADDRESS>wrote:

Merged #27
https://github.com/aomwen/Document-Tracking-System-BSIT3A/pull/27.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aomwen/Document-Tracking-System-BSIT3A/pull/27#event-1293350225,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeMFU9_vHgV7-s1PmHhs-NetTARSTBTSks5ssIgDgaJpZM4P5VGm
.


fix(database): remove postgres pid error from log

This is the error in jenkins:
...
database: expecting initialization id: none
database: no existing database found or it is outdated.
database: no backups found. Initializing a new database...
postgres: could not write external PID file "/var/run/postgresql/daemon.pid": Permission denied
2015-05-11 11:51:14 UTC LOG:  could not open usermap file "/etc/postgresql/main/pg_ident.conf": No such file or directory
2015-05-11 11:51:14 UTC LOG:  database system was shut down at 2015-05-11 11:51:13 UTC
...

This does appear to have cleaned up the PID error:
database: expecting initialization id: none
database: no existing database found or it is outdated.
database: no backups found. Initializing a new database...
2015-05-13 00:43:08 UTC LOG:  could not open usermap file "/etc/postgresql/main/pg_ident.conf": No such file or directory

Code LGTM.

@carmstrong yes. This only changes the PID file location inside the postgres directory.

This is just shifting things around internally so code LGTM.

:ship:


Fix spelling and naming mistake

Percentage-columns.js exported MixedGrid instead of PercentageColumns
Spelling mistake in inforce -> enforce

Thank you kindly :)


CSS multicol data is mixed with fragmentation

Hello, I'm the co-editor of the multicol specification. Looking at the data for multicol, pretty much all of the issues detailed as multicol problems are actually fragmentation issues. The break-before, break-after, break-inside properties are part of the CSS Fragmentation specification, and general fragmentation properties not specifically for multicol. See: https://www.w3.org/TR/css-multicol-1/#break-before-break-after-break-inside
There doesn't seem to be a more general page for fragmentation (which applies to print contexts as well as multicol), though the properties are there due to the MDN data https://caniuse.com/?search=break-inside.
Would there be interest in fragmentation data separate from multicol? If so I could submit a PR for that. I think it would be clearer for people looking at multicol if the true support for that spec was indicated.

Hi Rachel! Yes, excellent point. I do think that could be valuable and a PR would absolutely be welcome. Thanks in advance!

@Fyrd great - I'll look at putting that together. :)


Added current file extension to eslint tmp file

Since eslint acts different upon file extension this change allows to simple set in a .vimrc file something likelet b:formatters_vue = ['eslint_local'] for vuejs
and eslint will take a .vue temporary file instead of previous temporary .js

Looks good to me. Thanks!


[zh] sync 2023-04-24-openapi-v3-field-validation-ga.md

Add zh text to this blog:
content/zh-cn/blog/_posts/2023-04-24-openapi-v3-field-validation-ga.md

See preview please.

/approve


Delay node: add option to send intermediate messages on separate output

[x] New feature (non-breaking change which adds functionality)

Proposed changes
Backlog item: https://trello.com/c/1jAoRESp
This changes the 'drop intermediate messages' checkbox of the Delay node (in rate limit mode) to a select box. The options are:

Queue intermediate messages - the default behaviour, equivalent to the old checkbox being unticked
Drop intermediate messages - equiv to the old checkbox being ticked
Send intermediate messages on 2nd output.

If this last option is selected, the node gains a 2nd output. Any message that would previously have been dropped will now get sent on that output.
This is useful for rate limiting flows that need to do some work with the 'reject' messages. For example, rate limiting an HTTP API, where the 'dropped' requests need to be responded to with a 503.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.01%) to 67.681% when pulling 1c186416995add363c89466c3263b24c786b0bea on output-discarded-rate-limit into<PHONE_NUMBER>b8423484e70c64fb7f762c6ff6a1ca on dev.


Purchase OSD through the Google Cloud Marketplace

Which service is this feature request for?
OpenShift Dedicated
What are you trying to do?
A customer who has selected GCP as their preferred cloud should be able to pay for OpenShift Dedicated through GCP, take advantage of their committed spend agreement with Google, and consume on-demand.

Link to the OpenShift Dedicated (OSD) product listing on the Google Cloud Marketplace
https://console.cloud.google.com/marketplace/product/redhat-marketplace/red-hat-openshift-dedicated

OSD, now available on Google Cloud Marketplace, allows organizations to easily get started with a turnkey application platform to build, deploy, and run containerized applications in Google Cloud.
Customer benefits:

Flexible billing: Customers can use a pay-as-you-go pricing model for consumption-based pricing, in addition to annual contracts through private offers.
Google Cloud committed spend: Customers can leverage their committed spend with Google when purchasing OSD, optimizing costs.
Seamless procurement: OpenShift Cluster Manager (OCM) in the Red Hat Hybrid Cloud Console is now integrated with Google Cloud billing for an easy procurement and adoption experience.


[QUERY] Service Bus: SendVia and Deduplication

Hi
I haven't been able to find any information in the documentation about how deduplication works when a queue is used as a transfer entity.
Say I have a queue "Queue A" with deduplication enabled with a window of 2 minutes, and I send a message to that queue. Directly after I send a message to another queue via "Queue A". Will the deduplication system for "Queue A" look at the message id of that message as well and potentially filter it out? Or will it bypass the deduplication system for "Queue A" completely?

Thank you for your feedback.  Tagging and routing to the team best able to assist.

@johnknoop it should bypass the dedup system and go through Queue A onto the actual destination.
Are you seeing a different behavior?

Hi @axisc
Thanks! No I'm not seeing a different behaviour, but I just wanted to make sure :)


Batch selection

Sorry but I just can't get it to work you did reply on play market but I can't figure it out. I can't add folders for example folder1 folder2 I cannot select the folders together and add them but I can go in folder 1 a select the photos 1 at a time in it to add but that's very time consuming when it would be great to select all the folders and it add all photos inside it cause I'm trying to to add up to 35 folders with about 200 photos to each folder so it would be nice to batch select the folders and it add all the photos inside to the widget or allow adding all photos inside folder 1 then hitting back and add folder 2 photos so on long press the folder albums to add just like with images long press and it check them all and then add. You suggested it already works but I cannot get it to I only can open folder one and individually select each picture to add and it's very time consuming then just adding a batch operation I'm not finding any app anywhere to work for me and you are so close oh and selecting all folders in my own gallery app and hitting the share button will not pull up your app to make the widget to share to. I'm using Samsung Galaxy ultra android if that helps please let me know what I'm doing wrong or if what I'm asking for just is not possible because I just cannot figure it out sorry good luck with your app because the play store needs a good one and you about have it for me thanks

@JoeInagonY even though I've replied to the email I'll post the same reply here in case anybody else has the same issue, this way they may find the answer.
If I understand correctly, some of the things you are trying are not possible because of how the system photo picker works. You have a few options and I'll try to explain them with examples, one by one. I'm also attaching explainer videos as they might help as well. There are two types of widgets:

Photo-based

You need to select photos from your device using your preferred photo picker and gallery app. How you get to select photos, including bulk selection is up to those apps. Normally you're able to select multiple photos from a single folder, but not from multiple folders at once. So in this case, if you wish to select photos from multiple folders then you have to open the picker multiple times. I recorded a video showing some of these operations, although you might be familiar with most of them.

Folder-based

You need to select a single folder, then all of its photos will be automatically added to the widget, even if you add more photos to that folder later. I also recorded a video demonstrating this.
Also, you mentioned that you're trying to select photos from 35 different folders with roughly 200 photos each for a grand total of 7000 photos. Unfortunately that's currently not possible, even folder-based widgets are currently limited to folders with at most 500 photos, otherwise Android takes a really long time to load them so I'm imposing this limit which I might evaluate increasing in the future but there's little that I can do about the performance since I depend on the system to provide the photos.

Demo:
https://github.com/fibelatti/photo-widget/assets/3277516/e2f0496a-fbc4-4b36-af9e-f7f2761337be


Update pawn.json

Fixed typo in JSON format (missing ,), updated regular expressions to match recent release asset versions properly.

Merged, thanks 👍


fix: Corrected the act of clean with broom, for disarm trap skill exp…

…erience and stamina.

感谢纠错


VS Code opens a new window even if window.openFilesInNewWindow is set to off and even if code -r is used

Does this issue occur when all extensions are disabled?: Hard to verify, the issue only occurs after VSCode has been open for a while.

VS Code Version:

Version: 1.72.2 (Universal)
Commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075
Date: 2022-10-12T22:16:30.254Z (1 wk ago)
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.1.0
Sandboxed: No

OS Version: macOS 13.0 Beta (22A5373b)

Steps to Reproduce:

Open VSCode, use for several hours. Set window.openFilesInNewWindow to "off".
Use code -w -r (eg. via EDITOR variable) in the terminal

Results:
A new instance and window opens, which is several times slower than opening a tab in the existing window.
This is exactly the same problem as described in https://github.com/Microsoft/vscode/issues/28497 - but the workaround described there doesn't work, and the new window doesn't open in the 'vulnerability-scanner'.
Expected results:
The setting and CLI flag are respected and the existing window is reused.

Does this only reproduce when run from the integrated terminal?

@bpasero no, I don't use the integrated terminal, STR were all done with iTerm2.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


i am upgrading react native version 0.75.2 T0 0.76.5 But i am gtting issue unable to run project / I am using javaScript

Description
37 |
38 | type Registry<TEventToArgsMap: {...}> = {

39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
|      ^
40 | };
41 |
42 | /**
BUNDLE  ./index.js

ERROR  SyntaxError: /Users/harishpalsande/Desktop/projects/native/imeuswe-mobile-native/node_modules/react-native/Libraries/vendor/emitter/EventEmitter.js: Unexpected token, expected "]" (39:5)
37 |
38 | type Registry<TEventToArgsMap: {...}> = {

39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
|      ^
40 | };
41 |
42 | /**
BUNDLE  ./index.js

ERROR  SyntaxError: /Users/harishpalsande/Desktop/projects/native/imeuswe-mobile-native/node_modules/react-native/Libraries/vendor/emitter/EventEmitter.js: Unexpected token, expected "]" (39:5)
37 |
38 | type Registry<TEventToArgsMap: {...}> = {

39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
|      ^
40 | };
41 |
42 | /**

Steps to reproduce
upgrading react native version 0.75.2 T0 0.76.5
React Native Version
0.76.5
Affected Platforms
Runtime - Android, Runtime - iOS
Output of npx react-native info
System:
  OS: macOS 15.1.1
  CPU: (8) arm64 Apple M3
  Memory: 120.39 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.17.0
    path: ~/.nvm/versions/node/v20.17.0/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.9.0
    path: ~/.nvm/versions/node/v20.17.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.16.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 24.0
      - iOS 18.0
      - macOS 15.0
      - tvOS 18.0
      - visionOS 2.0
      - watchOS 11.0
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.2 AI-242.23339.11.2421.12483815
  Xcode:
    version: 16.0/16A242d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.11
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli":
    installed: 15.0.1
    wanted: 15.0.1
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.76.0
    wanted: 0.76.42
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
37 |
  38 | type Registry<TEventToArgsMap: {...}> = {
> 39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
     |      ^
  40 | };
  41 |
  42 | /**
 BUNDLE  ./index.js

 ERROR  SyntaxError: /Users/harishpalsande/Desktop/projects/native/imeuswe-mobile-native/node_modules/react-native/Libraries/vendor/emitter/EventEmitter.js: Unexpected token, expected "]" (39:5)

  37 |
  38 | type Registry<TEventToArgsMap: {...}> = {
> 39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
     |      ^
  40 | };
  41 |
  42 | /**
 BUNDLE  ./index.js

 ERROR  SyntaxError: /Users/harishpalsande/Desktop/projects/native/imeuswe-mobile-native/node_modules/react-native/Libraries/vendor/emitter/EventEmitter.js: Unexpected token, expected "]" (39:5)

  37 |
  38 | type Registry<TEventToArgsMap: {...}> = {
> 39 |   [K in keyof TEventToArgsMap]: Set<Registration<TEventToArgsMap[K]>>,
     |      ^
  40 | };
  41 |
  42 | /**

Reproducer
d
Screenshots and Videos
No response

I managed to fix this issue by replacing presets: ['module:metro-react-native-babel-preset'], with presets: ['module:@react-native/babel-preset'], in both babel.config.js & .babelrc


bugfix: broken link

Use github raw link for downloading file, otherwise it's an HTML page.

Codecov Report

Merging #63 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #63   +/-   ##
=====================================
  Coverage       0%     0%
=====================================
  Files          68     68
  Lines        2640   2640
  Branches      438    438
=====================================
  Misses       2640   2640

Continue to review full report at Codecov.

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

https://github.com/alibaba/Dragonfly/issues/52


Update documentation about DNS server config

Please update the documentation to reflect the current situation:
For SSTP, up to two DNS servers are pushed from the gateway (only VNG supports SSTP, VWAN not).
For IKEv2 and OpenVPN only 1 DNS server is pushed by the gateways in both case VNG and VWAN. A 2nd DNS Server has always to come from the client config. The Azure VPN Client is not capable to configure DNS servers. So with the Azure VPN client you can have only one DNS server.
To change/remove the DNS config on vWAN, you would have to raise a support ticket. It would not change on P2S Config change/Scale unit change.
With VNG the DNS config comes from the VNet, which is configurable by the customer.
Please update the documentation to reflect this major gaps.

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

ID: e9ef898a-0b72-e018-8a3d-6f14f62e1119
Version Independent ID: b62b5b8f-bc88-0868-a2f6-1020a5097460
Content: VPN Gateway: VPN client for OpenVPN protocol P2S connections: Azure AD authentication
Content Source: articles/vpn-gateway/openvpn-azure-ad-client.md
Service: vpn-gateway
GitHub Login: @anzaman
Microsoft Alias: alzam

Hi @TravisCragg-MSFT , @anzaman . Could you please give an update?

I'm using vWAN and have contacted Microsoft Support to have the DNS addresses added to the vWAN configuration but there has been no noticable change at my end - my Azure VPN clients still aren't provided a DNS server address. Neither the Azure VPN client, IPCONFIG, or the AzureVpnCxn.log log has any reference to DNS servers. If I add the DNS servers into the clientconfig area of the XML before the import, those DNS server addresses are not carried over to the rasphone.pbk entry (however the DNS Suffix configuration is carried across). I've also tried setting/adjusting the IpDnsAddress, IPDnsFlags, IPNameAssign, etc flags in the in %LOCALAPPDATA%\Packages\Microsoft.AzureVpn_8wekyb3d8bbwe\LocalState\rasphone.pbk file which is where the imported XML lands, however this has no impact either.
AzureVpnCxn.log log has no reference to DNS server settings either which pretty much confirms vWAN isn't trying to provide DNS server addresses to clients.
Lastly, I've even tried deploying a new P2S service after Microsoft added the DNS configuration but this new P2S service doesn't provide DNS server addresses to clients either.
Any assistance or ideas would be much appreciated!
Azure VPN Client version is 1:1152:28:0

for me both methods work - the first DNS server gets pushed, the second one I am able to configure via client XML. What is your OS version? Are the configured DNS servers reachable from the client? Can you use them as DNS servers when you specify them manually in nslookup? Did you try it from another, fresh installed client?

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Remove useless if check of max value of Integer

What is the purpose of the change
When execute getWeight method, whether uptime is larger than Integer.MAX_VALUE is unnecessary.
Brief changelog
modify:
AbstractLoadBalance.java
add:
AbstractLoadBalanceTest.java
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4886 into master will increase coverage by 0.05%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4886      +/-   ##
============================================
+ Coverage     63.99%   64.04%   +0.05%
- Complexity      451      452       +1
============================================
  Files           769      769
  Lines         33161    33159       -2
  Branches       5230     5229       -1
============================================
+ Hits          21221    21237      +16
+ Misses         9522     9508      -14
+ Partials       2418     2414       -4

Impacted Files
Coverage Δ
Complexity Δ

...o/rpc/cluster/loadbalance/AbstractLoadBalance.java
63.15% <0%> (+20.3%)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75% <0%> (-12.5%)
0% <0%> (ø)

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
51.42% <0%> (-5.72%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
64.75% <0%> (-2.46%)
0% <0%> (ø)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
62.5% <0%> (+0.62%)
29% <0%> (ø)
:arrow_down:

...pache/dubbo/remoting/transport/AbstractServer.java
48.95% <0%> (+1.04%)
0% <0%> (ø)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Check proj segment_azimuth calculation vs. Matlab

https://github.com/brendanjmeade/celeri/blob/fb75921d14e4364284a022ce90c340b3a4c1e2eb/celeri/celeri.py#L796

I will check this as part of a broader celeri-Blocks cross-validation


Update setup / run scripts based on new sound generation

Update setup / run scripts based on new sound generation

@am1tyadav  I merged this PR, but I have a question, do you intentionally not want to use the latest commit from rg_sound_generation submodule ?

no wrong comment I'll delete them

@wassfila I don't think I created a PR, but I had left a message to Jesse to use my commit to test if everything works. I still have to add a few things for it to be fully ready for the main branch


Code cleanup: Isolate constants for code consistency.

The binary path for the database binaries are hard coded into the code. It would cleaner and easier to change if they were made into a constant and that constant is referenced.
The mongodb isolated some constants instead of hardcoding them into the function calls.
var (
DefaultHost        = "<IP_ADDRESS>"
DefaultPort        = "27017"
DefaultMongoBinDir = "/var/vcap/packages/shield-mongo/bin"
)
This should be done for the other plugins as well.

A better solution is to migrate pathing concerns into the BOSH deployment for shield-agent, and remove the hard-coded paths from the endpoints.

Good idea


Update README.markdown with some stylistic changes

Minor changes:

Fixed: brand name references to "XCode" that should be "Xcode". (source: https://developer.apple.com/xcode/)
Fixed: brand name references to "OSX" that should be "OS X". (source: http://www.apple.com/osx/)

Thanks!


Eliminate redundant model sent to driver

The PR improves Arcs startup time particularly backing store preparation (syncing data from database, etc) 700+ ms.

close


Readme is confusing

There are some grammar and clarity issues with the README.md. Some of the lines I found very confusing. For example:

alex is also as an AMD, CommonJS, and globals module, uncompressed and compressed

I think you should add "available" to this line. It looks like you missed it completely.
I say we should take some time and proofread our files a bit.

I say we should take some time and proofread our files a bit.

I say we shouldn’t be passive aggressive.

I say we should take some time and proofread our files a bit.

I say we shouldn’t be passive aggressive.

I'll take that into consideration next time. :angry: :arrow_backward: grrr aggressive

Haha, great!
Thanks for the suggestions!


At https://github.com/cucumber/cucumber/wiki/Gherkin#gherkin-syntax, I see

When you run the feature the trailing portion (after the keyword) of each step is matched to a Ruby code block called Step Definitions.

I'm trying to understand this better. This is my first exposure to Gherkin, from the link at https://github.com/cucumber/cucumber-js#features.

Must the code block be Ruby? I arrived here from the JS port.
Where is that code located?
Should "Step Definitions" be "Step Definition" in that quote?

Hi Dan,
Sorry this is all so confusing, and thanks indeed for feeding back to us. Try this link instead: https://cucumber.io/docs/reference#step-definitions
We've been gradually trying to migrate the chaotic wiki docs over to the website, but at the moment things are between two places.


[HOPS-5] Hops ACLs.

https://hopshadoop.atlassian.net/browse/HOPS-5
Adding extended acls to HopsFS
See design document: "Hops ACLs" and eventually readthedocs in the ACLs section
depends on: hopshadoop/hops-metadata-dal#105 and hopshadoop/hops-metadata-dal-impl-ndb#131

Can one of the admins verify this patch?

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.


Coverage decreased (-1.0%) to 82.798% when pulling 084aa35cd1b8b1913f18a19f3f3ef343db7c9e65 on opeongo:arrayLiteral into 04d2b6546d047e4c5d3ce9400dfe92f041441194 on JSQLParser:master.

I will look into it. At the moment I am nearly full time offline. You know holidays.  :) Cheers Tobias

Hi Tobias,
I'm just wondering if you have any interest in this feature, or is it a
dead end?
On Fri, Jul 15, 2016 at 9:40 AM, Tobias<EMAIL_ADDRESS>wrote:

I will look into it. At the moment I am nearly full time offline. You know
holidays. :) Cheers Tobias
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JSQLParser/JSqlParser/pull/306#issuecomment-232953415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMgvTW3LO_u9hmHv9dVY538RFg_f3Kkoks5qV43rgaJpZM4JL4ed
.

Sorry. I overlooked your pr after beeing back from holidays.
What I do not like about your change is this kind of token "array whitespace* [". I have to admit, that I do not have a proper solution at hand. Thats the reason your pr is not merged. At the moment I do not think, that a "correct" (in my eyes ;)) solution and the "bracket" quotation of identifiers can coexist. Maybe one could construct a solution using syntactic or token lookahead .. Tobias

I agree that it would be aesthetically more pleasing if token did not use
the '[' character, but I was not able to figure out how to do it
otherwise.  I have a limited experience in developing parsers and I don't
fully understand all of the more advanced features of javacc.  Can you give
me a more specific suggestion to try out, or can you point me to some
sample code that deals with a similar problem?  If I had an avenue to
explore I would give it another go.
On Mon, Oct 3, 2016 at 6:29 PM, Tobias<EMAIL_ADDRESS>wrote:

Sorry. I overlooked your pr after beeing back from holidays.
What I do not like about your change is this kind of token "array
whitespace* [". I have to admit, that I do not have a proper solution at
hand. Thats the reason your pr is not merged. At the moment I do not think,
that a "correct" (in my eyes ;)) solution and the "bracket" quotation of
identifiers can coexist. Maybe one could construct a solution using
syntactic or token lookahead .. Tobias
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JSQLParser/JSqlParser/pull/306#issuecomment-251245895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMgvTeHewqOQzCoamQ2A9sxAlkGF96q1ks5qwYG0gaJpZM4JL4ed
.

I did some more work with javacc and lexical states for another project, and learned a bit ore how to use them.  I have updated my code so that it uses a lexical state change instead of the  "array whitespace* [" idiom.
Unfortunately I am a bit github challenged and I was not able to merge your master in to my development tree so that I could sort out the conflicts.  My changes are based on a version of your code that I checked out in the summer, and it appears to have conflicts.  Sorry about that.

Oops, I did not mean to close the pull request.

Hi Tobias
Is there anything I can do to move this along?  Is is a matter of me not getting a clean pull request together, or is the solution still not right?

Your corrected solution looks more promising. I did not see a single test. Please add some and I will do the merge and coflict resolving. Cheers Tobias

This PR has come a long way.
Two points from me: first of all I merge PRs using githubs merge & squash.  Second, could you upgrade your PR to the actual SNAPSHOT. There were some API changes that seem to collide with your changes.

Hi @opeongo, I tried to rebase your PR to the current SNAPSHOT (5896d8387bde6cd0cf822a67354954c046bf1864) and added the feature to use "ARRAY[...]" as PrimaryExpression (to use it as SelectItem). You find both patches attached (array-as-PrimaryExpression.patch.zip).
The file rebase-to-5896d8387bde6cd0cf822a67354954c046bf1864.patch contains your PR (based on SNAPSHOT). The file array-as-PrimaryExpression.patch contains my feature extension.
Is there anything else I can do to help getting this feature merged?

should work now with removal of sqlserver bracket quotation


Reduce log level for least useful logs

Re-worked from https://github.com/GoogleCloudPlatform/bigdata-interop/pull/231 with some review fixes

/gcbrun


Streams missing map methods after FL prefix update

#1352 introduced the FL prefix change to streams, however Stream.map should be exported as before as well.  The prefixing is just for libraries like Lodash/Ramda to safely dispatch.
https://github.com/lhorie/mithril.js/blob/rewrite/util/stream.js#L19

Based on the discussion that led to the change in the FantasyLand spec, I decided to export map as a function:
https://github.com/lhorie/mithril.js/blob/rewrite/stream/map.js


Add import/export RTs to virtual network resource/datasources

Inclusion of import/export RT policies in the resource 'apstra_datacenter_virtual_network,' mirroring the addition made to 'apstra_datacenter_routing_zone' as a response to issue https://github.com/Juniper/terraform-provider-apstra/issues/406.
(I apologize for not mentioning this earlier and for requesting both additions separately).

Note to myself: import/export RT shows up in the GUI when you select type: "VXLAN"


Release talker & talker_flutter 2.4.0

Codecov Report

Merging #95 (b0b1c1f) into master (a7bc29d) will decrease coverage by 9.36%.
The diff coverage is 58.18%.

:exclamation: Current head b0b1c1f differs from pull request most recent head 4043962. Consider uploading reports for the commit 4043962 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   99.48%   90.12%   -9.36%
==========================================
  Files          12       13       +1
  Lines         194      233      +39
==========================================
+ Hits          193      210      +17
- Misses          1       23      +22

Impacted Files
Coverage Δ

...ackages/talker/lib/src/extensions/talker_data.dart
100.00% <ø> (ø)

...alker/lib/src/utils/filter/base_talker_filter.dart
100.00% <ø> (ø)

packages/talker/lib/src/talker.dart
84.29% <42.42%> (-15.71%)
:arrow_down:

...kages/talker/lib/src/settings/talker_settings.dart
80.95% <77.77%> (-19.05%)
:arrow_down:

...ackages/talker/lib/src/addons/original_addons.dart
100.00% <100.00%> (ø)

.../src/models/talker_data/talker_data_interface.dart
100.00% <100.00%> (+4.76%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Existing whitelisting policy doesn't appear in view policy page

Steps to reproduce:

Add policy for app whitelist
Publish and apply policy to devices.
Click on policy and explore app restrictions.

Closing as this is fixed.


add time types and functions

Um, wut? FILETIME and SYSTEMTIME are already defined elsewhere? They are in winbase.h, and I didn't find definitions in winbase.rs....

Ok, so FILETIME is in minwindef.rs, and SYSTEMTIME is in minwinbase.rs. Should they stay there, or be moved to winbase.rs?


Bump @api3/ois to v1.4.0

Also requires zod bump to ^3.20.0 to maintain compatibility.

@martinkolenic I don't think this needs to be tested as this was a simple dependency update. This is verified by our tests.


[YUNIKORN-1261] Add pre-upgrade hook

I believe YUNICORN-1261 is caused because we are missing a lifecycle hook for upgrades:
https://helm.sh/docs/topics/charts_hooks/#the-available-hooks

By adding a pre-upgrade hook could we be triggering a resource exists error causing the upgrade to fail? Or are we relying on the default hook deletion policy before-hook-creation?
One last question: has this been tested and how?

@wilfred-s I apologize- I put this up-for-review prematurely.
I think YuniKorn is in a highly similar place to Spark Operator in terms of Helm hooks:
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/pull/1624
What we really need is first-order support for resource ordering. Unfortunately, the hooks have weird semantics that make it hard for us to get the desired behavior.
I am going to wait for the conversation in https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/pull/1624
to resolve, and then we can decide whether we want to adopt this same approach within YuniKorn. The Spark Operator only has 3 instances of the hook, whereas I see 12 in YuniKorn, so this might be more heavy-handed.


Add resources for python

@khageshwor congratulations your PR is merged.

@khageshwor congratulations your PR is merged.


docs: added contributors

Motivation

Added the all-contributors boilerplate and rc
Description

Great to engage community

@all-contributors please add @phklive for code


User location button click event listener

Hi, is it possible to handle the click event on the locate user button? I need to do some distance calculations after it is being clicked.
Thanks

I can not provide click event of locate buttons.
Because it shown by native Google Maps SDK and I can not handling button click.

I think it could be possible https://developers.google.com/android/reference/com/google/android/gms/maps/GoogleMap.OnMyLocationButtonClickListener

Thanks @deextercz ,
I found in iOS side!

https://developers.google.com/maps/documentation/ios-sdk/reference/protocol_g_m_s_map_view_delegate-p.html#ac0e0171b811e839d9021800ca9fd33f4

I released 1.8.0-beta2 what contains Map.MyLocationButtonClicked event.
Also added sample for it.

https://github.com/amay077/Xamarin.Forms.GoogleMaps/blob/master/XFGoogleMapSample/XFGoogleMapSample/BasicMapPage.xaml.cs#L76-L84

No problem?
Okay closed it.


Major version bump to 5.0.0

Bumped package versions to 5.0.0 for breaking changes in #223

Could you add a note to root repository README.md outlining what major versions of @itwin/imodels-access-backend and @itwin/imodels-access-frontend support which major versions of iTwin.js?I think something like this would work (numbers are fake)
iTwin.js version 	@itwin/imodels-access-* version

=a.b.c <x.y.z 	0.1.2
=x.y.z < ... 	0.3.4

I don't think there should be too many rows in this table, if there appears to be, let's discuss, maybe we can think of something more granular.
One way to determine the ranges would be to go through the file history of imodels-access-backend package.json file. The lowest version of iTwin.js we should mention is 3.0.0.

Added the table; here are corresponding PRs:
^3.0.0: https://github.com/iTwin/imodels-clients/commit/5924c5c5de830d22263e53cbaeded087a5769758
^3.3.0: https://github.com/iTwin/imodels-clients/commit/4afad2b5690ccb942edbf66a15eca0be181180aa and https://github.com/iTwin/imodels-clients/commit/e4a925fe5fd48c00a5e8161da30272af79873070
^3.6.0: https://github.com/iTwin/imodels-clients/commit/93a74d62504c59045f990fd0025c0422c241b35e and https://github.com/iTwin/imodels-clients/commit/52206238a863cb3ddad9e9abdec5700f8bf7ede6
^4.0.0: https://github.com/iTwin/imodels-clients/commit/a0017b1b697cd03d23cf73d3a4301fb07e4a4887 and https://github.com/iTwin/imodels-clients/commit/fb39c5afce0ac1f93eb69fa999a8cfdc9b7e6881
I thought about using another table format (screenshot below), but I think the proposed one is more convenient for consumers.


Show Community Top Bids only when NSFW enabled

The Issue
As an immediate step to make the Explore page friendlier for all audiences, only show the Community Top Bids if the NSFW setting is enabled.
Do we want to hint at its existence / explain how to enable on the explore page?
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Please make sure that there is some cue to new users that this exists and do not just silently hide it.


here.com does no longer provice App Codes

When followong the link in the readme, creating the app id still works. However here.com changed something and you can only choose to generate either "API Keys" or "OAuth 2.0 Access Key ID".
Neither one of those seems to work.
Which one to choose?

Same problem here. this ist how here.com API Wensite looks now:

Sorry about that.. They removed the App id & App code combination just 2 days after release of the new adapter version..
Im going to release a new version (approximately this weekend) which uses the REST API Key..

Fixed in Version 0.1.1 - please delete all instances of the adapter before doing the update!


Remove unused u_inputTexture uniform in panorama_to_cubemap.frag

This uniform is not used in the shader code.

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


Bad file descriptor (C:\Users\runneradmin\AppData\Local\Temp\tmpw8fxpxar\build_deps\bundled_libzmq-src\src\epoll.cpp:73

This is a pyzmq bug

[X] This is a pyzmq-specific bug, not an issue of zmq socket behavior. Don't worry if you're not sure! We'll figure it out together.

What pyzmq version?
26.0.3
What libzmq version?
Not sure
Python version (and how it was installed)
3.11 via Windows
OS
Windows 11
What happened?
I believe this is bundled with Jupyter. Spinning up the Jupyter server fails with the error above when first notebook is opened. Downgrading to 25.1.2 solved the issue.
Code to reproduce bug
No response
Traceback, if applicable
No response
More info
No response

Did you install Python via the Windows Store? If so, this is #1981 . In either case, can you test with the appropriate wheel for you from the latest build here and let me know if it fixes things for you?

Thanks I'm not sure what this means. I can try to increment to 26.2 and check if it resolved if you want.
python was installed from windows store. This is a work machine and the mandatory security software that it runs will not allow to install other python distros. I imagine that this is the same for many machines. Microsoft has made some changes to the python distro that can cause issues. Recommend to that windows store python distros to your testing procedures.

Unfortunately, Microsoft Store installations are not available in any testing environments to my knowledge, so that is not feasible. But I have managed to reproduce this issue and confirm it was fixed by #2024, so if you can confirm it does so for you, that would be great! Or if it does not, that would be super helpful as well.

It should be feasible but you will need to spin up windows VM and use some Powershell. I tried 26.0.3 and 26.2.0 just now and wasn't able to reproduce. Sorry. Anyway thanks for working it.

Thanks so much for testing!
I wish it was that easy to set up a vm, but it took me months to find one that actually reproduced the bug, and only in incredibly slow x86 emulation on arm. I don't know why the Store usually isn't available and no cloud provider I have access to supports VMs with the store available. And even the store alone is not sufficient to reproduce the bug, it needed a username with non-ascii characters as well.

If you happen to have a chance and can test an install install from one of these wheels (download and unzip, then pip install --force-reinstall path/to/pyzmq-...whl), I think I have fixed the underlying libzmq problem in https://github.com/zeromq/libzmq/pull/4734.


Document how to mount a volume when using Docker

Current Behavior
Currenty if I mount a directory like in the following docker-compose it is very likely that the container won't go up because it has no permission to write into those directories
version: '3.7'

services:

  rdf4j:
    image: eclipse/rdf4j-workbench:3.7.4
    privileged: true
    restart: always
    environment:
      JAVA_OPTS: '-Xmx4g -Xms4g'
    ports:
      - 8080:8080
    volumes:
      - ./data:/var/rdf4j
      - ./logs:/usr/local/tomcat/logs
    restart: unless-stopped

Expected Behavior
The docker container should go up without errors
Steps To Reproduce

Create a directory somewhere (e.g. /tmp/rdf4j) and cd into it (e.s. cd /tmp/rdf4j)
mkdir data ; mkdir logs
create a file docker-compose.yml with the following content

version: '3.7'

services:

  rdf4j:
    image: eclipse/rdf4j-workbench:3.7.4
    privileged: true
    restart: always
    environment:
      JAVA_OPTS: '-Xmx4g -Xms4g'
    ports:
      - 8080:8080
    volumes:
      - ./data:/var/rdf4j
      - ./logs:/usr/local/tomcat/logs
    restart: unless-stopped

Run docker-compose up
The container complains about missing permissions (about writing into those 2 directories)

Version
3.7.4
Are you interested in contributing a solution yourself?
Yes
Anything else?
Document to change permission to 777 and add a sticky bit to the created folders. That is:

chmod 777 data logs
chmod +t data logs

Thanks for reporting this @fpompermaier - adding these instruction to the README of the docker dir would (as a first step) be very useful I think.


Make cutout compatible with keras layer

Description
Fix cutout in keras layer (graph and symbolic tensor).
Type of change

[x] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[x] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Passes existing and new tests.

@fsx950223
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

No performance regression on switching implementation I believe.
https://colab.research.google.com/drive/1MlPkCziMK4msWIPC-L9sJvEePqtE-rCg?usp=sharing


Fixing tests

Fixes #ISSUE_NUMBER
Proposed Changes

this should fix the tests, but it changes the logic using which the values from the dataframe are extracted.
fixed the order of return for segmentation penalty (if training_loop is ingesting in the order "penalties, class_weights", that is the order the functions should return)
added epsilon to fix corner case of division by zero
updated nested splits to create more informative training data

Checklist

[ ] I have read the CONTRIBUTING guide
[ ] My PR is based from the current GaNDLF master
[ ] Non-breaking change (would not break existing functionality): please provide as many details as possible for any breaking change
[ ] Function/class source code documentation added/updated
[ ] Code has been blacked for style consistency
[ ] If applicable, version information has been updated in GANDLF/version.py
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file
[ ] Usage documentation has been updated, if appropriate
[ ] History has been updated, if appropriate
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation
[ ] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3,4]

This should fix the test, but it changes the logic using which the values from the dataframe are extracted.

Codecov Report

Merging #6 (0c2afd5) into master (7508fee) will increase coverage by 0.29%.
The diff coverage is 95.93%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   91.25%   91.54%   +0.29%
==========================================
  Files          91       93       +2
  Lines        5157     5395     +238
==========================================
+ Hits         4706     4939     +233
- Misses        451      456       +5

Flag
Coverage Δ

unittests
91.54% <95.93%> (+0.29%)
:arrow_up:

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

Impacted Files
Coverage Δ

GANDLF/compute/loss_and_metric.py
96.66% <ø> (ø)

GANDLF/parseConfig.py
75.17% <71.42%> (+0.63%)
:arrow_up:

GANDLF/models/unet_multilayer.py
91.83% <91.83%> (ø)

GANDLF/utils/generic.py
86.88% <94.44%> (+3.16%)
:arrow_up:

testing/test_full.py
98.26% <96.20%> (-0.21%)
:arrow_down:

GANDLF/models/light_unet_multilayer.py
97.95% <97.95%> (ø)

GANDLF/compute/inference_loop.py
84.28% <98.00%> (+3.20%)
:arrow_up:

GANDLF/compute/training_loop.py
83.88% <100.00%> (+0.14%)
:arrow_up:

GANDLF/data/ImagesFromDataFrame.py
88.88% <100.00%> (-0.48%)
:arrow_down:

GANDLF/data/inference_dataloader_histopath.py
100.00% <100.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


can't discover device

Any hint on how to discover the device? I have connected a bluetooth 5.0 adapter to the pc but after some scan no device is found.
Debugging into the BlueClient _client.DiscoverDevices() is returning an empty list

nevermind, was a windows issue


Audio Not Transmitting from Raspberry Pi to Browser in Webrtc Bot

Thank you for this great project.
I've been working on setting up a Webrtc bot following the provided instructions. The project works wonderfully overall. However, I've encountered an issue with audio transmission from the Raspberry Pi to the browser.
I can successfully talk on my PC, and the Raspberry Pi is playing the sound. However, the audio is not transmitted from the Raspberry Pi to the browser. The camera and controls work as expected.
I've tested the audio recording on the Raspberry Pi using the arecord command, and it works fine. I can record and playback without any issues. I have also tried the gen2brain/malgo example code capture which was also working fine.
.env file:
srv_url = "wss://api.roboportal.io:8001"

public_key = "xxxxxx-xxxxxxxxx-xxxxxxxxx-xxxxxx"
secret_key = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"

stun_urls = "stun:stun.l.google.com:19302"
video_codec_bit_rate = 1000000
frame_format = "RGBA"
video_width = 800
video_frame_rate = 30

audio_input_enabled = true
audio_output_enabled = true

output_mode = console
port_name = "/dev/serial/by-id/..."
baud_rate = 115200

bot_box_ipc_port = 5555
robot_ipc_port = 5556
robot_ipc_host = <IP_ADDRESS>

camera_multiplexor_i2c_bus = 1

debug = true

I am using a generic USB audio card for capturing and playing the audio, which I have set as the default card in /usr/share/alsa/alsa.conf
My setup:

Raspberry Pi 4B
OS: 32bit Raspbian GNU/Linux 12 (bookworm)
go version 1.21.4

Let me know if you need more details. Appreciate your help!

Thanks for reporting this, @kadharsh.
We'll need a few days to ensure there is no regression and that the software is functioning correctly. We'll get back to you after confirming.

Hi @malykhin
Thanks a ton for the quick response!  Take your time, and I appreciate your efforts. Feel free to reach out if you need any more info or testing on my end.

Hi @malykhin
Thanks a ton for the quick response!  Take your time, and I appreciate your efforts. Feel free to reach out if you need any more info or testing on my end.

Hi @kadharsh
I've checked the microphone and it works well.
The mic that I tested (it's just the one that I was able to get quickly).
Steps I used to set it up:

Getting the record device details:

Configuration:

Setting the gain:

Hi @malykhin
I've reviewed my setup, and I seem to be following similar steps, but unfortunately, the audio is not coming through on the other side. Am I missing something?
Here is the configuration I'm using:
1 .env file is the same
2. record device details and OS:

3. Alsa Configuration

4. alsamixer

5. pion debug log

But no audio output from the browser, the speaker option in the video player is blacked out.

I've tested the microphone by recording a small audio file using this command arecord -f S16_LE -r 44100 audio_file.wav, and that's working.

The setup looks good to me.
Could you please try using headphones instead of speakers on your computer? I've noticed that my browser disables WebRTC audio if feedback occurs.
Could you please share the link to the audio device you're using? I can test it on my end.

I tried using headphones with my PC, but unfortunately, the issue persists. The USB device I am using is Plextone GS3. Additionally, I attempted to use the microphone of the USB webcam Logitech C270, and the problem remained the same.

I've identified the issue – it was related to ALSA. After installing pulseaudio, the miniaudio started selecting it as the backend, resolving the problem. Now, the audio works in both directions. Additionally, I encountered distorted audio, and I solved it using this patch: https://github.com/pion/mediadevices/issues/500
Thank you @malykhin for your time and support; I appreciate it.


Phraseology Improvements

Changes various pieces of phraseology to correct FAA phraseology (for the most part). A lot of work done for making the speech synthesis sound good without the need for the ui_log to display messages like United six niner four, runway two eight right, cleared for takeoff, and instead shows UAL694, runway 28L, cleared for takeoff, but reads it aloud correctly. You'll also notice that aircraft will call up to the different facilities we simulate based on the situation, eg. a departure will say San Francisco Tower, UAL394, holding short of 28R, and an arrival will say NorCal Approach, BAW588 with you at 10000. This reflects reality, as the names of the approach, departure, and tower facilities are not always the same, as pictured below.

PS - Sorry to the reviewer if the code in this PR is a bit scattered and difficult to rummage through. I tried to make it presentable but there were just a lot of tiny changes in a whole bunch of different places.
Give her a listen! :smiley: http://erikquinn.github.io/atc

i love it!!!
BUT: everytime i try to assign an altitude, ot says altitude not understood... maybe that is because im on my surface, which i doubt, or i am just having one of these days  where nothing works, which is more likely, or it has something to do with the changes....
also she pronounces wind as "waind", as you would say in reWIND, which is weird...

is there a way to give the waypoints an additional attribute, so that she pronounces them correctly??? like ARCHI for example, she spellt that out in phonetic letters rather then saying it...

@Maverick283 I took another look, and it is printing and reading altitudes back to me properly. Maybe try an incognito chrome tab (so the cache is definitely cleared of old stuff) on a real computer. If that works, and the surface still doesn't, that might be a separate bug itself.
But yes, I too noticed that she says "waind" as in "winding a clock", but it's only about 30% of the time in takeoff clearances. I couldn't figure out what was causing her to choose between the two pronunciations, but with fresh eyes I just now figured out to just change wind to wynd in the pronounced sentence only, and that seems to have fixed it (see 6300b9d).
As for the fixes, she tries with some, and just spells out others. I'm not really sure there, that one will be more tough, but seems like a problem for the future.
Changes are live at http://erikquinn.github.io/atc

Okay, on the laptop altitude assignment is no problem! Will see if I figure out what caused it on the Surface, but this has obviously nothing to to with this pull request ;)
Wind is pronounced fine now, good idea to change it!
Something else I struggle understanding right now: Even though this is not merged yet, when I play at http://zlsa.github.io/atc/ I can enable Speech output already, BUT: It is in German! This is actually quite funny as she has that unbelievable strong german accent, yet it is not how it should be.
Any ideas what could have caused this?

As mentioned in #368, there seems to be a issue with the "heading" a.k.a. "h" command.
See a video here

@Maverick283 it has been a feature for a pretty long time, and has always had the German accent, which led to weird pronunciations like mentioned in #338, which are fixed in PR #364.
Hold this PR while I look into the heading issue.

Resolved the turn issue in 48f3aeb, and edited some old commits for tiny fixes (eg apparently KSAN tower is called "Lindbergh Tower", which I didn't bother to look up, a spelling mistake, and similar unimportant changes).
Due to there being a conflict between this and #364, I merged #364 into this PR, applied the intended merge resolution, and then closed #364.
Gave it a more thorough test this time (which led to all the nitpicky changes). Should now be good to go. Test if you like at http://erikquinn.github.io/atc (fyi my gh-pages also has #356 merged in, relating to more realistic climb performance).
Ready to merge

I agree, this is looking and working good!!! Didn't find any more bugs, good work @erikquinn!

Nice, one little bug though: if you enter a wrong command there´s a double comma with a space between.

@harp71 Good catch. Will be fixed in #383. :smiley:


External Annotations Format/s Support, instead LibraryModels

Greetings! 🤗
https://www.lastnpe.org founder ⛲ here.
Coming today from https://github.com/enola-dev/enola/issues/845.
Just wondering if NullAway had ever considered External Annotations (such as Eclipse EEA and also old Wiki, or that Annotation File Format from Checker Framework Support), instead of LibraryModels?
Or perhaps someone hacked some sort of LibraryModels generator kind of thing? I dreamt of something like that in https://github.com/lastnpe/external-annotations-esperanto, many moons 🌝 ago - but never did it.
/CC @cpovirk

Hi @vorburger!  Short answer: I think it shouldn't be too hard to write a LibraryModels implementation that just parsed external annotations and then returned the right nullability info from its methods.  This could be built outside of NullAway and plugged in as a service loader, and we'd be happy to help with fixes within NullAway if needed to accommodate this.  Our cycles are extremely limited so we're unlikely to have time to build this ourselves anytime soon.
Longer answer: a key motivation for NullAway has always been fast running times, so NullAway can run on every build.  When we were first writing NullAway we looked at supporting Checker Framework stub files, but we were concerned about the time required to parse many stubs.  So we went with LibraryModels at first, and eventually created a binary astubx format that is fast to parse.  Right now we are actively working on building the support required to translate the annotations in https://github.com/jspecify/jdk to an astubx file so NullAway users can use those annotations; see https://github.com/uber/NullAway/issues/950.  Given our limited cycles, and our longer-term effort to better support the JSpecify annotation semantics, we figured it best to focus on supporting the annotations provided in the JSpecify project.

Or perhaps someone hacked some sort of LibraryModels generator kind of thing?

I don't think any kind of code generator is needed necessarily.  What I would try at first is to just write an implementation of LibraryModels that parses info from the external annotation files and returns it.  If performance is an issue, one could try lazily parsing only the relevant parts of the external annotation files.  Only if that was too slow would I resort to something like code generation, but at that point I might just consider generating astubx.


Only parse 64 bit int

I ran into an issue while implementing balloon hashing in javascript using this as a reference.
The issue resolved down to getting different values for other, and that was because the python implementation generates a bigint from the entire buffer, rather than reading a 64 bit integer as in the C reference implementation.
Note: This commit breaks all the unit tests, so if those test vectors are known good then the code below is a mistake. But if they're not this may be an issue.

For various reasons, see https://github.com/RustCrypto/password-hashes/pull/232, we are not following the C reference implementation.
There are already a bunch of libraries out there that have used this implementation with the test vectors so breaking that isn't really an option unless it's for a good reason.
I don't remember all the details, but looking at the paper it's not exactly clear how this should be done. I'm trying to retrace the steps I originally took in my mind and it seems to me that how it's currently done discards the least amount of data, which is probably why I decided to go that route.
If you are interested, you could adjust the reference implementation to use this algorithm, which was approved by the author: https://github.com/henrycg/balloon/issues/1#issuecomment-1568345655.


I can't compiled the example

Hi I follow te README but whe I compile I can't link the main and neither the libtensorflow_all.so
I have a lot of undefined reference. I copy and paste a couple:
main.cc:(.text+0x13c): undefined reference to tensorflow::SessionOptions::SessionOptions()' /usr/local/lib/libtensorflow_all.so: undefined reference to tensorflow::Tensor::ComputeFlatOuterDims(tensorflow::gtl::ArraySlice, long long)'
What I did wrong?
Thanks

i got the same issue. @contactmat did you fix it?


dart language not support

@Ivorzk If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


Feedback for "Executing ABAP Test Cockpit (ATC) Check Runs"

https://help.sap.com/docs/BTP/65de2977205c403bbc107264b8eccf4b/d8cec788fc104ff9ad9c3757b4dd13d4.html
Where can I find Object Set Library (OSL) schema?

Thank you for your feedback! We’ll look into it and come back to you if we have any questions.

Hi @wozjac, the documentation for the Object Set Library is not yet available. We are planning to deliver it in the near future.


CSUB-504: Testing creditcoin-cli & other related changes

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #1087 (86977d5) into dev (e700a76) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1087   +/-   ##
=======================================
  Coverage   74.50%   74.50%
=======================================
  Files          74       74
  Lines       10674    10674
=======================================
  Hits         7953     7953
  Misses       2721     2721

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Improve map layouts (correct bench tile types + all stations included)

Description
Task: Improve map layouts (correct bench tile types + all stations included)
Feature: #459
This task is about finalising the layouts for each level to make sure we are using the right tiles.
Dependencies
No dependencies
Milestones
List of steps that need to be completed for this task.

[x] Choose the correct bench tile types (Oct 8)
[x] Make sure all stations are included each day/include if not (Oct 8)
[x] Replace them in text file (Oct 8)
[x] Make sure it works (Back-end works - check with stations team)(Oct 9)

Completion Deadline: Oct 9
Documentation

Map Layouts

Member

@Sarahkhalek

Closed with #542


Relax generator and speedup tests

Description
This PR fixes the slowdown in tests  introduced in #3288
Checklist

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[x] New tests are added if needed and existing tests are updated
[ ] Any changes are noted in the changelog
[x] Code is formatted with fourmolu (which can be run with scripts/fourmolize.sh)
[ ] Cabal files are formatted (which can be done with scripts/cabal-format.sh)
[ ] Self-reviewed the diff

maybe leave a comment trail about why we are dividing by 15?

Experimentally decided to be a decent value


Map translation design v2

v1 design
v1 demo
We need to think about the user experience and make a v2.
Some things we learned from discussing the demo:

Users will probably upload a bunch of maps at once, and there will be previously uploaded maps.
We will want to take all the strings from all the maps and combine/deduplicate them so translators only translate a string once.
We will store the already translated maps and let the users see them in a list.

Until Svetlana makes wireframes, a developer can do this:

We need a UI for listing the maps. We'll show this first. We have designs in Figma for the user's maps, so that design could just be tweaked and moved to the first screen.
The map list will have a button to add a map. This is where the user will click to add a map. We have designs for that flow, but we need to change them because the user will only upload a map and not view it or do any translation at that time. When a map is added, we parse the file and add all text strings to the database. We don't add a string if it already exists. Use the key-terms/phrasebook local graph API.
We need a button/link on the new map list screen that takes a translator to a list of strings from all maps. This will show the strings that have been stored in the database. On this screen the translator chooses a target language and can translate the words. Figma has designs close to this.
Next we need to update the first screen to let the user select a language to see if any maps have been translated into that language.

i think we should change /map-list to /maps


TaskManager has too many responsibilities

It stores tasks, updates them and splits them into meaningful blocks. Way too many for a single class.

It has basically no responsibilities after 65d60c6be1ff6473e2f716320ff217e3e94254e8


Update HA template and User guide

Update the docker compose template and user guide

Coverage decreased (-2.8%) to 61.591% when pulling 7fa42b81189af377eec8352f0cefc95a4db41046 on yixingjia:HA15Docs into 04bad55134c8a38f0ba08f92498b0cf659c7783f on vmware:master.


GRPC.Core.Xamarin android emulator x86_64 channel not created

Creating channel on x86_64 android emulator returns null, x86 works fine.
new Channel("localhost:8081", ChannelCredentials.Insecure);

Grpc.Core Xamarin / Android is no longer actively supported.

The Grpc.Core implementation is now in maintenance mode and we're only going to focus on critical and security fixes.
Supporting Xamarin simply presents too much maintenance overhead and it's also difficult to test with the test infrastructure available to us
The Xamarin support has always been classified as "experimental", see https://github.com/grpc/grpc/tree/v1.46.x/src/csharp/experimental

See the recent update on gRPC C# for more details. https://grpc.io/blog/grpc-csharp-future/
Note that in practice, running Grpc.Core on Xamarin might still be possible (there is nothing in principle that would prevent it from working), but it will be completely at your own risk and it's simply not one of the use cases the grpc team will help you with as Grpc.Core on Xamarin is now considered as "not supported" by us (but you might still be able to get help from the community).


cmake error in linux

CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 3.5 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

-- Including Wayland support
-- Including X11 support
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- Documentation generation requires Doxygen 1.9.8 or later
CMake Error at CMakeLists.txt:39 (file):
  file COPY cannot find "/home/user/EclipseEngine/assets": No such file
  or directory.

-- Configuring incomplete, errors occurred!

Hi, thank you for positing this issue. This will be fixed soon as it is due to assets folder not available as it doesn't get pushed as the folder is empty. Also, this issue occurs on all platforms.