Update article with 10054 error number

Learn Build status updates of commit b23ced4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

support/sql/connect/tls-exist-connection-closed.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Sorry, meant to do this in the other repository. I'll resubmit.


I want additional tests set up for the VSCode extension

As a VSCode extension developer, I want additional tests set up for the VSCode extension so that I can test the extension more thoroughly.
AC

[ ] At least 2 smoke and/or integration tests are added for the VSCode extension.
[ ] I can run VSCode extension tests and VSCode will not be downloaded/executed as part of that.
[ ] The pipeline is updated to run all VSCode unit tests before running functional tests.

Notes
From a conversation with @Coffe2Bits:

[... It] looks like we should theoretically be able to run unit tests without having to run the VSCode instance but we’d have to do extra setup because this file downloads (if needed) and runs the VSCode instance. :man_shrugging:

Evaluate testing more of the connectivity of the VSCode extension, i.e. services, LSP server, etc.

https://code.visualstudio.com/api/working-with-extensions/testing-extension
Reference for testing extensions from @cvchaparro


bugifx: navigation/NativeLink props correction

Describe the bug
Props not handled properly in NativeLink

Closing as done.


Is this a complete solution???

Why does Teams need inbound ports open?
What if another user on the machine launches Teams after the script is executed?
What if a new user logs on?
What if it were deployed through ConfigMgr as part of a TS running as SYSTEM?
If ran as part of a logon script, then a user would have to launch Teams and the first call will fail.  The rules are placed after the exe is discovered which would be the next logon.
All the solutions seem like a bandaid.
This is why Microsoft should stop designing apps to install into the user profile.  If Teams installed into Program Files, this would not be an issue.  We asked for an MSI but it only puts the stub setup into Program Files.  This is not what people meant when they asked for an MSI.
WHY IS THIS NOT FIXED?????

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

ID: 6457d204-d229-9154-28fd-ca9bbb4bddcd
Version Independent ID: f5d82846-41cb-3548-6997-ce23bd8f89c6
Content: Get clients for Microsoft Teams - Microsoft Teams
Content Source: Teams/get-clients.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@mniccum  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@mniccum thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Stream an audio file using react-native-tcp

Hi ! Thank you for reporting an issue, but we would like to remind you, we have a trouble shooting page in our wiki. You may want to take a look on that page :p

please provide the version of installed library and RN project.
a sample code snippet/repository is very helpful to spotting the problem.
issues which have been tagged as 'needs feedback', will be closed after 2 weeks if receive no feedbacks.

======
I am  reading text files properly and sending them as they are, but this mp3 file appears to be 170% times bigger than the original when using this configuration.
      socket.on('data', (data) => {
        this.updateChatter('Server Received: ' + data);
        RNFetchBlob.fs.readStream('/storage/sdcard1/Music/elpisito.mp3'/*'sdcard/Audio/tone.mp3'*/, 'ascii')
        .then((stream) => {
          let data = ''
          stream.open()

        //  socket.write("HTTP/1.0 200" + "\r\n" + "Content type: audio/mpeg" + "\r\n")
          stream.onData((chunk) => {
            data += chunk
//            console.log(stream)
  //          console.log(chunk)
            socket.write(chunk)
          })
          stream.onEnd(() => {
            console.log('LECTURA TERMINADA')
            //socket.write(data)
            socket.end()

          })
            })

      //  var RNFS = require('react-native-fs')
        console.log('here1')
      /*  RNFS.readFile('/storage/sdcard1/Music/elpisito.mp3').then(data =>{
          socket.write(data)
        })*/
      })

@franrios , thanks for reporting this issue, I'll look into it and update my progress here 👍

@franrios , after some test, I think the readStream is working correctly.
I've noticed in the code snippet you've provided
stream.onData((chunk) => {
            data += chunk
            socket.write(chunk)
})

may having some problem since the chunk is an Array when reading stream using ascii encoding. You may replace it with this
stream.onData((chunk) => {
-            data += chunk
-            socket.write(chunk)
+            data = data.concat(chunk)
+            socket.write(chunk.join(''))
})

I think this will give you correct file size 😏
Besides, the ascii encoding is somehow slower than other encoding, please refer to the Performance Tips and #118 😄

Thanks for your response @wkh237, at the end I needed to use 206 Partial content status code so I  used socket.write(chunk,'ascii') in order to allow streaming instead of a full download.

@franrios , does it working properly ? If it's not please feel free leave any feedback 😄


Draft: test_walkfail.sh: implement src walk fail test

Implement a test where dsync --delete is run, and the walk of the
source tree is caused to fail by making a directory unreadable.
This test is currently expected to fail.
dsync currently has a bug described in
https://github.com/hpc/mpifileutils/issues/587
After the source walk fails, dsync continues with the dest walk
followed by the copies that the deletes.
Because the source walk failed, the list of files and directories
on the source is incomplete. Files or directories that are already
on both source and destination, but which were not walked on the
source, are missing from the source list.  Those files or directories
are then deleted from the destination tree.

After https://github.com/hpc/mpifileutils/pull/596 is merged, I'll rebase this, and there will only be one commit - "test_walkfail.sh: implement src walk fail test"


Fix warnings "only available on iOS 10_0 or newer"

Hi! Thanks for your PR.
There seems to be a lot of extraneous changes in this commit. We will be merging another PR which will specifically suppress those warnings.


chore: create notify server

Overview
Creating the mono repo sub-project for the notification server
closes #6697
Changelog

used poetry's new command to create the project.
added zmq as dependency
created a main.py
created a service file to run the server

Review requests

Name. Should we use a different name than the bland notification-server?

Risk assessment
None

Are we purposefully opting out of the yarn subpackage tracking? Means we'll have to have a custom extra task in make bump but other than that I'm pretty into it, honestly

Nope.


Retro-cap old versions of AbstractPlotting to PlotUtils@0

Does this look correct and could we get this merged?

Bump!


Get added to official list of OpenAPI implementations

RapiDoc should be listed alongside other OpenAPI specification user interfaces.The world needs to know about it! :smile:

I've created https://github.com/OAI/OpenAPI-Specification/pull/2683 to add RapiDoc to that list :)

@afranke - thanks for the suggestion
@waldyrious - Thanks much for going ahead and adding it to the list !!!

Update: the PR was merged earlier today, so this issue is now resolved! :tada:


Creating a task with bufferTask method result on INVALID_ARGUMENT

Environment details

which product (packages/*):  @google-cloud/tasks
OS: Linux/Ubuntu
Node.js version: v20.8.1
npm version: 10.1.0
google-cloud-node version<EMAIL_ADDRESS>
Steps to reproduce
import "dotenv/config";

import { v2beta3 } from "@google-cloud/tasks";

const PROJECT_ID = process.env["PROJECT_ID"] ?? "";
const LOCATION = process.env["LOCATION"] ?? "";
const QUEUE_ID = process.env["QUEUE_ID"] ?? "";
const KEY_FILE = process.env["GOOGLE_APPLICATION_CREDENTIALS"] ?? "";

const client = new v2beta3.CloudTasksClient({ keyFile: KEY_FILE });
const queuePath = client.queuePath(PROJECT_ID, LOCATION, QUEUE_ID);
await client.bufferTask({ queue: queuePath });

Result
(personnal values replaced by XXXXXX on purpose)
node:internal/process/esm_loader:40
      internalBinding('errors').triggerUncaughtException(
                                ^

Error: 3 INVALID_ARGUMENT: "x-goog-request-params" header is either missing or misformatted. "x-goog-request-params" must contain "name=projects/XXXXXXXXX/locations/XXXXXXXX/queues/XXXXXXXXXXX"
    at callErrorFromStatus (/home/david/project1/node_modules/@grpc/grpc-js/build/src/call.js:31:19)
    at Object.onReceiveStatus (/home/david/project1/node_modules/@grpc/grpc-js/build/src/client.js:192:76)
    at Object.onReceiveStatus (/home/david/project1/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:360:141)
    at Object.onReceiveStatus (/home/david/project1/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:323:181)
    at /home/david/project1/node_modules/@grpc/grpc-js/build/src/resolving-call.js:99:78
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
for call at
    at ServiceClientImpl.makeUnaryRequest (/home/david/project1/node_modules/@grpc/grpc-js/build/src/client.js:160:32)
    at ServiceClientImpl.<anonymous> (/home/david/project1/node_modules/@grpc/grpc-js/build/src/make-client.js:105:19)
    at /home/david/project1/node_modules/@google-cloud/tasks/build/src/v2beta3/cloud_tasks_client.js:212:29
    at /home/david/project1/node_modules/google-gax/build/src/normalCalls/timeout.js:44:16
    at OngoingCallPromise.call (/home/david/project1/node_modules/google-gax/build/src/call.js:67:27)
    at NormalApiCaller.call (/home/david/project1/node_modules/google-gax/build/src/normalCalls/normalApiCaller.js:34:19)
    at /home/david/project1/node_modules/google-gax/build/src/createApiCall.js:84:30
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 3,
  details: '"x-goog-request-params" header is either missing or misformatted. "x-goog-request-params" must contain "name=projects/XXXXXXXXX/locations/XXXXXXXX/queues/XXXXXXXXXXX"',
  metadata: Metadata {
    internalRepr: Map(2) {
      'endpoint-load-metrics-bin' => [
        Buffer(27) [Uint8Array] [
           49,  48, 167,  39, 214, 245, 148,
          109,  64,  57,   0, 144, 240,  41,
          106, 246,  71,  64,  73, 170,  78,
           92, 129,   9, 130, 197,  63
        ]
      ],
      'grpc-server-stats-bin' => [
        Buffer(10) [Uint8Array] [
          0, 0, 14, 32, 42,
          2, 0,  0,  0,  0
        ]
      ]
    },
    options: {}
  }
}

Node.js v20.8.1

Thank you for the report, I'm going to follow up internally and will get back to you.

It seems like the currently released client library will not properly handle BufferTask without a few tweaks. As a workaround, please use the HTTP fallback instead of gRPC, and supply some task ID since the current client library won't be able to request an auto-generated task ID.
This code worked for me:
const {v2beta3} = require("@google-cloud/tasks");

async function main() {
  const client = new v2beta3.CloudTasksClient({fallback: true}); // enable HTTP fallback instead of gRPC
  // this example uses gcloud application default credentials;
  // if you have `GOOGLE_APPLICATION_CREDENTIALS` set you don't need to pass any extra auth parameters

  const PROJECT_ID = await client.getProjectId(); // this gets project ID from authentication credentials
  const LOCATION = 'us-west1'; // use whatever location you need
  const QUEUE_ID = 'testqueue'; // make sure you enabled HTTP overrides for this queue:
  // gcloud beta tasks queues create testqueue --http-uri-override=host:google.com,path:/ --http-method-override=get --location=us-west1

  const queuePath = client.queuePath(PROJECT_ID, LOCATION, QUEUE_ID);
  const response = await client.bufferTask({ queue: queuePath, taskId: 'test' }); // note: we pass some task ID

  console.log(response);
}

main();

We'll release a new version of the library with the fix.

So just to update this issue, after talking to the Cloud Tasks team, based on their explanation, the BufferTask RPC will remember how it was called (with GET, or POST, etc.) and its parameters. This is impossible to achieve in the client library. Also, it won't work via gRPC at all. The decision was made to remove BufferTask from the client libraries since the only valid use case of it is via the HTTP endpoint.


1e-10 == 1.

The examples clip the outputs to an upper bound of 1. - 1e-10 before passing them to binary_crossentropy() (eg here). This value can't be represented as float32 and is always rounded to 1, which might be the reason for the NaNs you're witnessing occasionally.

Indeed thanks! I fixed it in 9f7b97b6934d3bedec7916b01511a88c14b741f2 by increasing the clipping value from 1e-10 to 1e-6.


fix(plugin/rate-limiting): use redis:eval() to improve rate-limiting accuracy

Summary
fix rate-limiting accuracy reported on issue: https://github.com/Kong/kong/issues/5311
There is a low impact change to the current code and this implementation has better accuracy on redis policy
it is same fix made in PR https://github.com/Kong/kong/pull/8227, but I remove sliding-window feature. Kong have no plans to add a sliding window to this plugin, Sliding window is an enterprise-only feature at this point
Checklist

[x] The Pull Request has tests
[ ] There's an entry in the CHANGELOG

Full changelog
[it removes the race condition problem getting keys on redis. it uses the return of the increment command as the usage counter ensuring atomicity, different from the current one, which allows multiple requests to get the same value of the redis key while the counter increment is done asynchronously]
[it has an even better performance than the current plugin using the redis policy, as it does only 1 operation per request in redis (eval) instead of the two operations per request that the current one (get and incrby)]
Issue reference
Fix https://github.com/Kong/kong/issues/5311
KAG-981

Thank you for your contribution. Can you please have a look at my comments? It would be nice to have a test that exposes the failing behavior in the previous implementation and demonstrates that the fix works. If that is too difficult to write, can you please point out how the existing tests exercise the functionality that you've changed? Thank you again, -Hans

@hanshuebner ,
I tried to write the test in lua using coroutine to simulate the calls in parallel, but I couldn't. As I need to get the response code of each call, it seems to me that the coroutines were running sequentially and the error did not occur.
But I created a scenario using the k6 load test where you can compare the result before and after the fix.
I put it in this repo here: https://github.com/giovanibrioni/kong-plugin-rate-limiting-temp

i've been trying to solve this problem for over 1 year.  I already use this fix in my applications and it has been battle tested receiving more than 4 billion transactions per day in a productive environment.  I'd like to help the rest of the community and I've made all the adjustments requested, but this slowness is annoying me.  if the bot closes my PR I won't open another one.

@giovanibrioni I'm sorry that this is taking longer than you hope.  We can't really merge the code without an automated test, however.  I understand that writing such a test is difficult and we're going to discuss internally whether we can help with that.

And for the test, I understand this scenario is hard to simulate in our test framework, So I think maybe we just need to add some unit tests in https://github.com/Kong/kong/blob/master/spec/03-plugins/23-rate-limiting/02-policies_spec.lua, and make sure existing integration test passed, that should be enough. @hanshuebner what are your options?

i've been trying to solve this problem for over 1 year. I already use this fix in my applications and it has been battle tested receiving more than 4 billion transactions per day in a productive environment. I'd like to help the rest of the community and I've made all the adjustments requested, but this slowness is annoying me. if the bot closes my PR I won't open another one.

I'm sorry about this, let's expedite the progress of this PR and strive for a quick merge.

@ADD-SP , Could you take a look at this PR and give some comments?

I think the expiration time of KEY can be optimized. Let's try to imagine a scenario here:

Let's say we have a KEY with a TTL of 1s and 200ms left before the KEY expires, and let's say a request from Kong to Redis now takes 600ms round trip, which means a single trip takes about 300ms. What happens is that the request doesn't reach Redis until after the KEY has expired, so Redis re-creates the KEY with the same name and returns 1 to the request, causing the card control to fail.
The solution to this problem is as simple as ensuring that the TTL is greater than the limited period. So I think adding a jittered time delay to the EXPIRATION of KEY would be better.

Rebased


[Azure Search] CS1061: 'IIndexesOperations' does not contain a definition for 'GetClient'

Just migrated from Microsoft.Azure.Search Version="5.0.2". Everything is fine except this string:
ISearchIndexClient indexClient = ServiceClient.Indexes.GetClient("IndexName");
CS1061	'IIndexesOperations' does not contain a definition for 'GetClient' and no accessible extension method 'GetClient' accepting a first argument of type 'IIndexesOperations' could be found (are you missing a using directive or an assembly reference?)
Setup (please complete the following information):

Microsoft.Azure.Search Version="10.0.0"

@DrJuriko How did you upgrade? Did you use NuGet, or did you change version numbers manually? Have you tried a full clean & rebuild?

Yes, exactly, what I did: upgraded project from nuget, save it to git, delete old project folder from local and theg get all sources again with clone repository from git. This is my project file items:
 <ItemGroup>     <PackageReference Include="Autofac.Extensions.DependencyInjection" Version="5.0.0" />     <PackageReference Include="AutoMapper" Version="9.0.0" />     <PackageReference Include="AutoMapper.Extensions.Microsoft.DependencyInjection" Version="7.0.0" />     <PackageReference Include="Microsoft.ApplicationInsights" Version="2.11.0" />     <PackageReference Include="Microsoft.ApplicationInsights.WorkerService" Version="2.8.0" />     <PackageReference Include="Microsoft.Azure.Search.Data" Version="10.0.0" />     <PackageReference Include="Microsoft.Azure.Search.Service" Version="10.0.0" />     <PackageReference Include="Microsoft.CodeAnalysis.Analyzers" Version="2.9.4">       <PrivateAssets>all</PrivateAssets>       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>     </PackageReference>     <PackageReference Include="Microsoft.CodeAnalysis.FxCopAnalyzers" Version="2.9.4">       <PrivateAssets>all</PrivateAssets>       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>     </PackageReference>     <PackageReference Include="Microsoft.Extensions.Hosting" Version="3.0.0" />     <PackageReference Include="Microsoft.VisualStudio.Azure.Containers.Tools.Targets" Version="1.9.5" />   </ItemGroup>
Are you sure that GetClient method is still here?

GetClient still exists, but it's defined in an assembly that you're not referencing. I see Microsoft.Azure.Search.Data and Microsoft.Azure.Search.Service, but not Microsoft.Azure.Search itself. It's recommended that you just reference Microsoft.Azure.Search as this should automatically bring in all the dependencies.
Try that and let me know if it works.

Yes, thank your time, after Microsoft.Azure.Search install all were fine...

@drjuriko Glad it’s working for you. Thanks for confirming.


RuntimeError: cannot reshape tensor

@rewicks @mjpost  Dear contributor,
I came across an error when I train the model,
Traceback (most recent call last):
File "trainer.py", line 419, in
main()
File "trainer.py", line 93, in main
determiner=determiner)
File "trainer.py", line 342, in run_epoch
for i, (contexts, factors, labels, context_strings) in enumerate(self.training_set.batchify(batch_size)):
File "/opt/yansongpeng/Mirli_Bert/Mirli_Bert/ersatz/dataset.py", line 214, in batchify
context = context.view(len(data), -1)
RuntimeError: cannot reshape tensor of 0 elements into shape [0, -1] because the unspecified dimension size -1 can be any value and is ambiguous
My training set is,
▁to ▁help ▁us ▁understand ▁what ▁has ▁been ▁happen ing . ||| ▁A t ▁the ▁start ▁of ▁the ▁crisis , ▁many ▁people |||
Would you please give me a hand? Thanks!

Hi! It looks like the generated line doesn't have a label?
A generated line in the training data should look like:
, 00 0 ▁G old ? ||| ▁S A N ▁F ||| <eos>
Could you share the line from the original file that created this?
Also, as a temporary fix, you should be able to append <eos> to this line (assuming end of sentence is the intended label)
Are all of the lines in the training file missing the label?


[1.9.x] clean ReaderProxy cache on acked_changes_set [6959]

Port of #893

Build status:

Linux
Mac
Windows
Performance

Build status:

Linux
Mac
Windows
Performance

The test that failed on Linux is being fixed by #902


Functionality to report nearby drive-thru testing centers

Users should be able to submit info about local resources like clinics, drive-thru testing centers with appropriate address which can then be made available to other users in a certain radius around that area.

In what format? I Suggest json.
Please specify the data you want.
Like: country, lat, lon, type, address, phone,
where type would be: clinic, drive-thru, lab, other.

This will be part of the Healthcare Authorities in the near future release.  Each can provide a locale-specific website to provide information for their constituents.


OS-A-D integration rebased

Rework of the RPC deploy to consume OSAD as a sub-repository.
Introduces an RPC deploy script.
RPC Upgrade concerns excluded from this PR to break down the size of the reviews.

:+1:

reworking the README somewhat to integrate the actual commands to be used, per comment by @git-harry

Looks good to me, but I haven't tested it yet.

:+1:

:+1:

who's got two thumbs and is merging?

^ That guy!


Timer thread dies on android an is not resurrected

I am working with the Timer utility on an android application. LibGdx version is 0.9.9 (latest stable).
The issue is caused (again) by android's weird policy with keeping static objects over app restarts.
The following is a description what causes the problem to occur:

The app launches for the first time
Timer is creating its TimerThread
The thread registers itself as a LifecycleListener
The app is left (works best for me when I turn off the screen and wait for the device to be pattern-locked)
When the app is entered again, onCreate is called, leading to a new initialize()ation => Gdx.app is new, with a new list of LifecycleListeners
Timer.thread is not null, for it "statically survived" => It is not being created in Timer.instance.start (the method itself probably isn't even called, because instance survived, too)
The timer's LifecycleListener methods are not invoked either, because the listener is not registered any more

=> The thread remains dead
For fixing the problem, I suggest setting Timer.thread to null in Timer.TimerThread.pause. If you are concerned about increased unnecessary TimerThread instantiations, one might as well keep a reference to the actual Thread which is created in TimerThread.resume, allowing to poll the thread's state in Timer.instance
Additional information:
Strangely, I was not able to reproduce this issue when being connected to the pc (i.e. a power source). So I suggest using adb over network when debugging
The problem occurred with all of our team members, which is 6 persons with at least 3 different device types.

Using libgdx 1.5.4 I'm running into the exact issue mentioned by @skunktrading above me. Are there any plans to solve this soon?

@asherbar, the proposed solution has already been integrated. If you are still having a problem, open a new issue with an example showing the issue.

Done.

I'm still struggling with this on libgdx 1.9.8. We use gdx-pay, and when the google play native overlay is closed, my game resumes but my timers are dead. The same happens when the screen is locked then unlocked, or navigating to a different app etc.


feat: Support enabling of features for directories

Purpose
This PR allows enabling of features once the directory is created. The BTP CLI command to enable the features after the creation of the directory is integrated into the UPDATE function of the directory resource to allow the user to enable the features in an update command.
Resolves #471
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code via automated test

go test ./...

What to Check
Verify that the following are valid:

Automated tests are executed successfully

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

[x] The PR is assigned to the Terraform project and a status is set (typically "in review").
[x] The PR has the matching labels assigned to it.
[x] The PR has a milestone assigned to it.
[x] If the PR closes an issue, the issue is referenced.
[x] Possible follow-up items are created and linked.

Remark: No need to wait for the bug fix mentioned in the assigned issue. The mentioned issue does not appear when using the Terraform provider. The error message is only a temporal (probably) glitch in the cockpit when the removal of the AUTHORIZATION feature is executed.


Remove the sercom::v2::Pad type and redefine the v2::spi::Pads type

The sercom::v2::Pad type was a simple wrapper around a gpio::v2::Pin
type. Its purpose was to ensure that each Pin was properly configured
to act as the corresponding SERCOM Pad. However, one of my goals in
designing the v2::spi API was to avoid the need for users to manually
convert Pin types to Pad types. As that module progressed, it became
clear that the v2::Pad type served no real purpose. All the same
constraints could be imposed using a slightly different approach to
type-level programming.
Remove the v2::Pad type and modify the sercom::v2::pad module
accordingly. Update the v1::Pad type as well. Rename some of the
type-level items in the v2::pad module and improve the documentation.
Update the sercom::v1 peripheral APIs to accept both v1::Pad types
and the equivalent of a v2::Pad, which is simply a properly configured
v2::Pin.
Finally, redefine the type parameters of v2::spi::Pads to be
OptionalPads rather than OptionalPinIds, and provide a new
PadsFromIds to make up for it. This change increases consistency in
the definition of the spi::Pads type, and it allows me to remove a
significant amount of trait boilerplate needed to make it all work.

@jbeaurivage, this should be complete. Your v2::uart module might need some minor refactoring, but it shouldn't be too bad. It wasn't much for me on the v2::spi.

@quentinmit, this also updates your BSP. Could you please take a look and make sure I didn't introduce any bugs? I fixed the SERCOM4 vs. SERCOM2 problem.

@bradleyharden would you like to wait for comments from the people you've asked to take a look, or should I look it over / merge?

Let me see if @jbeaurivage can review it today. Otherwise, we can merge it.
I very much hope this is the final form of SERCOM pads. There has been a lot of churn, as I had discussions with people and started porting BSPs to it. I think it has improved tremendously, but there have been some bumps.

@bradleyharden, you got my official LGTM. I think this has come very far. The latest changes make everything a lot clearer. There's still a lot of typelevel complexity, but not to a point where it's off-putting, especially for the end-user, and the docs help a lot.

@sajattack, I think this is ready for you


Revert back the executable path.

Fixes #134

@onema Can you take a look?
You changed the executable path from composer/bin/coveralls to bin/coveralls and it fails when invoked from the vendor directory. See #133
I committed a quick fix at https://github.com/satooshi/php-coveralls/commit/410d235bde2aa3f93271b39bbc75f82c58b15a11 but then I can not execute it from git clone #134 .
I believe this PR will support all the following cases:

Install php-coveralls as a composer dep and run from vendor/bin/coveralls
Git clone php-coveralls and run it from repo/composer/bin/coveralls
Install and run as a phar

Thanks!

That is odd. I'll take a look at it.

@onema I also added some improvements suggested in #124

I looked into this, and I was not able to reproduce the issue you describe in #134.

#134 is still reproducible for me.
$ git clone git@github.com:satooshi/php-coveralls.git
$ cd php-coveralls
$ composer install --no-dev
$ bin/coveralls -v
You need to set up the project dependencies using the following commands:
curl -s http://getcomposer.org/installer | php
php composer.phar install

Don't you see the error too?
Anyways, bin/coveralls is nicer, so I moved it back again. The change is now much smaller.
@onema Can you take another look?

I'll do.

Thanks for the additional description btw :)

FYI, travis is back to normal. One of our builds show this issue too:
https://travis-ci.org/satooshi/php-coveralls/jobs/95218240

@onema
Because of #134, currently we don't have coveralls webhook for other pull requests. I'd appreciate if you could take a look. Thanks!

okay, I think I understand what is going on.
You closed #133 with the following commit https://github.com/satooshi/php-coveralls/commit/410d235bde2aa3f93271b39bbc75f82c58b15a11, I believe lines 6 and 7 where the only ones required to fix the issue, not lines 4 and 5.
Before making any further changes I would like to identify when this file is called, then we can make the appropriate changes without braking anything else :)
These are the ones I can identify:

When executed directly from the bin directory: php bin/coveralls; uses '/../vendor/autoload.php'
When executed from the .phar; uses '/../vendor/autoload.php'
When installed as a global dependency using composer global require satooshi/php-coveralls:dev-master; uses '/../../../vendor/autoload.php'

Anything else?

Ah yes, @onema sorry, my bad!
I think that's all :)

@onema OK to merge?

This looks good. If you can verify that phar works as intended after these changes we should be good to go!

Great thanks for looking!!


Voxel Resolution for Tail datasets are in microns instead of nanometers

I want to multiply the voxel resolution Tail datasets by 1000 so that they are in nanometers.  But I don't have access to them in the admin console. For consistency we should always enter the voxel resolution in  nanometers since that's what the admin console requests.  Like below

The 'voxelres' entries for the Tail dataset (MyToken1, MyToken2 and MyToken3) are 0.585 x 0.585 x 5.  They should be 585 x 585 x 5000.
http://openconnecto.me/ocp/ca/MyToken1/info/
http://openconnecto.me/ocp/ca/MyToken3/info/
http://openconnecto.me/ocp/ca/MyToken2/info/

Done.


[REQ][python] Improve generated new input type for anyOf schema

Bug Report Checklist

[x] Have you provided a full/minimal spec to reproduce the issue?
[x] Have you validated the input using an OpenAPI validator (example)?
[x] Have you tested with the latest master to confirm the issue still exists?
[x] Have you searched for related issues/PRs?
[x] What's the actual output vs expected output?
[ ] [Optional] Sponsorship to speed up the bug fix or feature request

Description
When having an anyOf modifier for a schema that is used in a array schema, the resulting python class does not have the correct typing.
Given the following schemas
    ClassificationToReturn:
      properties:
        name:
          type: string
          minLength: 3
        displayName:
          type: string
        exportName:
          type: string
          nullable: true
          minLength: 1
        attributes:
          items:
            $ref: "#/components/schemas/ClassificationAttribute"
          type: array
    ClassificationAttribute:
      $ref: "#/components/schemas/SingleValueClassificationAttribute"
    SingleValueClassificationAttribute:
      properties:
        name:
          type: string
          minLength: 1
          pattern: ^[^.]+$
        displayName:
          type: string
          minLength: 1
        exportName:
          type: string
          minLength: 1
      required:
        - name
      type: object
      additionalProperties: false
    MultiValueClassificationAttribute:
      properties:
        name:
          type: string
          minLength: 1
        maximumCount:
          type: integer
          format: int32
          minimum: 0
      required:
        - name
      type: object
      additionalProperties: false

the created class (in classification_to_return.py) has the correct typing information
class AttributesTuple(
    typing.Tuple[
        classification_attribute.single_value_classification_attribute.SingleValueClassificationAttributeDict,
        ...
    ]
):

    def __new__(cls, arg: typing.Union[AttributesTupleInput, AttributesTuple], configuration: typing.Optional[schema_configuration.SchemaConfiguration] = None):
        return Attributes.validate(arg, configuration=configuration)
AttributesTupleInput = typing.Union[
    typing.List[
        typing.Union[
            classification_attribute.single_value_classification_attribute.SingleValueClassificationAttributeDictInput,
            classification_attribute.single_value_classification_attribute.SingleValueClassificationAttributeDict,
        ],
    ],
    typing.Tuple[
        typing.Union[
            classification_attribute.single_value_classification_attribute.SingleValueClassificationAttributeDictInput,
            classification_attribute.single_value_classification_attribute.SingleValueClassificationAttributeDict,
        ],
        ...
    ]
]

@dataclasses.dataclass(frozen=True)
class Attributes(
    schemas.Schema[schemas.immutabledict, AttributesTuple]
):
    types: typing.FrozenSet[typing.Type] = frozenset({tuple})
    items: typing.Type[classification_attribute.ClassificationAttribute] = dataclasses.field(default_factory=lambda: classification_attribute.ClassificationAttribute) # type: ignore
    type_to_output_cls: typing.Mapping[
        typing.Type,
        typing.Type
    ] = dataclasses.field(
        default_factory=lambda: {
            tuple: AttributesTuple
        }
    )

However, when we use an anyOf in ClassificationAttribute
      ClassificationToReturn:
        properties:
          name:
            type: string
            minLength: 3
          displayName:
            type: string
          exportName:
            type: string
            nullable: true
            minLength: 1
          attributes:
            items:
              $ref: "#/components/schemas/ClassificationAttribute"
            type: array
      ClassificationAttribute:
        anyOf:
          - $ref: "#/components/schemas/SingleValueClassificationAttribute"
          - $ref: "#/components/schemas/MultiValueClassificationAttribute"
      SingleValueClassificationAttribute:
        properties:
          name:
            type: string
            minLength: 1
            pattern: ^[^.]+$
          displayName:
            type: string
            minLength: 1
          exportName:
            type: string
            minLength: 1
        required:
          - name
        type: object
        additionalProperties: false
      MultiValueClassificationAttribute:
        properties:
          name:
            type: string
            minLength: 1
          maximumCount:
            type: integer
            format: int32
            minimum: 0
        required:
          - name
        type: object
        additionalProperties: false

we will get the wrong typing information in classification_to_return.py
class AttributesTuple(
    typing.Tuple[
        schemas.OUTPUT_BASE_TYPES,
        ...
    ]
):

    def __new__(cls, arg: typing.Union[AttributesTupleInput, AttributesTuple], configuration: typing.Optional[schema_configuration.SchemaConfiguration] = None):
        return Attributes.validate(arg, configuration=configuration)
AttributesTupleInput = typing.Union[
    typing.List[
        typing.Union[
            schemas.INPUT_TYPES_ALL,
            schemas.OUTPUT_BASE_TYPES
        ],
    ],
    typing.Tuple[
        typing.Union[
            schemas.INPUT_TYPES_ALL,
            schemas.OUTPUT_BASE_TYPES
        ],
        ...
    ]
]

@dataclasses.dataclass(frozen=True)
class Attributes(
    schemas.Schema[schemas.immutabledict, AttributesTuple]
):
    types: typing.FrozenSet[typing.Type] = frozenset({tuple})
    items: typing.Type[classification_attribute.ClassificationAttribute] = dataclasses.field(default_factory=lambda: classification_attribute.ClassificationAttribute) # type: ignore
    type_to_output_cls: typing.Mapping[
        typing.Type,
        typing.Type
    ] = dataclasses.field(
        default_factory=lambda: {
            tuple: AttributesTuple
        }
    )

The AttributesTuple should have the type information ClassificationAttribute or at least both of the possible types.

openapi-json-schema-generator version
Version 4.5.1

OpenAPI declaration file content or url

Generation Details
Used the cli command to generate the client.

Steps to reproduce

Related issues/PRs

Suggest a fix

The code generation is functioning as expected here. The anyOf refs are not known as possible types. For them to be known they would have to be examined at one higher schema level and compared against type info. Currently the generation only knows about input and output types for array and object, types defined in type, and ref only. Because anyOf is not analyzed for type info, the only info comes from absent type info which allows any type.
If you want different functionality here, please consider filing a PR with thorough test cases.

Also, should your anyOf be a oneOf so it matches only one of those schemas rather than one or more?

Thanks for the information. Is it possible to have a short meeting or could you point out the necessary steps for such functionality? We would like to invest some time to add such feature as our use case prefers to work with these classes and it would be nice to have proper type hints. However, we cannot really judge how long it may take.

Also, should your anyOf be a oneOf so it matches only one of those schemas rather than one or more?

You are right. Our openAPI.yaml is also auto-generated which sometime causes semantically wrong specification like this.

Adding this is non-trivial.
In the latest version of the generators, returned instances only subclass one class. So to do this one would need to:

perhaps generate interfaces for all object types and have the object output classes implement them
accumulate type + interface/dict output class info from all subschemas
for use cases like this, the code will know from introspecting the calculated type constraints that only dicts are allowed. And we know that the oneOf interfaces or output types are allowed

Doing this is difficult because any number of keywords can exist in the same schema including:

type
oneOf
allOf
etc and type and interface information need to be gathered from them and correctly combined

So a new CodegenSchema field that could store this may look like typeToInterfaces: Map<String, List>
Interfaces can come from refed locations so one would need to store the ref location and the name of the generated interface class. Some examples:
typeToInterfaces = {"string": null, "boolean": null} # fora a schema that allows strings and numbers
typeToInterfaces = {"object": [{SingleValueClassificationAttributeInterface, MultiValueClassificationAttributeInterface}]} # for your use case
This property would need to be added to CodegenSchema.
When are you free to discuss this this week and next week?

Hey, I am free this week. How should we proceed? I can write you a mail to figure out a date. A coworker would potentially join the meeting.

A simpler solution here is to handle this specific cas eonly where a schema only has oneOf defintion and no other json schema keywords. In this case, you could modify the Schema.validate method to do the following:

run validation on the payload and get back the list if [schema_containing_oneof, specific_oneof_schema]
and then use the last class specific_oneof_schema, as the class that is produced (instantiated)
other needed changes (for only this simple oneOf use case )

modify new input to accept union of oneoOf typeddicts and map output classes
modify new output to product union of oneof output classes

@Marcelo00 how did you decide to proceed forward here?
It has been ~ 1 month since we met about this.

If I don't hear back here I will close this issue as inactive.

Hey, I am sorry for not coming back to you. We tried out your suggestion of using speakeasy and it turns out that I fulfills our requirements. Thanks again for your time investing in this project and your suggestion. Really appreciate your help.


⚠ Samsung has degraded performance

In 8b2c6c9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2791 ms

Resolved: Samsung performance has improved in 0a63df1.


Cannot run (Linux Mint)

python3 run
up to date: scheduler.py
up to date: clock.py
up to date: main.py
up to date: wifi.py
up to date: localPTZtime.py
up to date: ds3231_port.py
up to date: test.py
up to date: helpers.py
up to date: config.json
up to date: pico_temperature.py
up to date: buttons.py
up to date: time_set.py
up to date: display.py
up to date: util.py
up to date: pomodoro.py
up to date: mqtt.py
up to date: rtc.py
up to date: apps.py
up to date: temperature.py
up to date: speaker.py
up to date: constants.py
up to date: configuration.py
Uploaded code

Traceback (most recent call last):
  File "/home/jiri/.local/bin/ampy", line 8, in <module>
    sys.exit(cli())
  File "/usr/lib/python3/dist-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1656, in invoke
    super().invoke(ctx)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3/dist-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/jiri/.local/lib/python3.10/site-packages/ampy/cli.py", line 99, in cli
    _board = pyboard.Pyboard(port, baudrate=baud, rawdelay=delay)
  File "/home/jiri/.local/lib/python3.10/site-packages/ampy/pyboard.py", line 147, in __init__
    raise PyboardError('failed to access ' + device)
ampy.pyboard.PyboardError: failed to access /dev/cu.usbmodem14101

Is there any other way how to run clock?

I have used Thonny to upload all files and run main.py from there. All ok! :-)
I also see that "run" is python script, so I will have a look on that why there is a problem with my serial port.
Great job! I will prepare fork of this repo.

I use run as a bash script, so I am not sure. To be honest I ported this repo over to Rust and built it mostly again with that anyway. A fork and continued development in that is probably recommended!

I am quite new to Python. I see in your code a lot of stuff I have never used. There is probably no documentation available, right?
My goal was to reuse parts of your code (like display driver) and build own clock.
My clock shall be used by child with hear loss.
My current requirements:

clock battery powered
** keep display off by default because of that, activate it via button press
alternatively show also temperature received from internet (ThingSpeak server)
use "power" output to activate vibration motor to wake up person instead of using speaker
allow clock configuration via REST API or via other interface (just a level of comfort)

Maybe you will have some advice for me how to start. If not, I will keep trying to understand your code.
I have never used Apps/scheduler, so it will be challenging for me.

Hi, yeah no docs available. I never got round to fully finishing off the project.
The apps concept is that one app can essentially take resources (like telling the buttons how to behave) and update the display.
The scheduler concept is just a small wrapper around uasyncio. Each task gets its own event loop which just runs a loop that calls some code every x milliseconds. In case of the display, that is the update leds function.
To disable the display, you would just need to remove and add a new task to the scheduler when a button press happens. You would obviously need to be scanning for the button press.
It's hard to be more exact unless I know exactly which areas and what parts you don't understand, but I'm happy to try and explain where I can.

Hello Dom!
Thanks for quick reply and thanks for giving me more details about your
work.
I will have a look on your code more deeply trying to play with that.
I may be late with my next reply as I have opened too many projects in
parallel. :-)
I guess that you have also some piece of code that checks button states.
So maybe this is where I start with. Also I will read about uasyncio.
Talk you later!
Sany
On Thu, Dec 21, 2023 at 6:49 AM Dom Needham @.***>
wrote:

Hi, yeah no docs available. I never got round to fully finishing off the
project.
The apps concept is that one app can essentially take resources (like
telling the buttons how to behave) and update the display.
The scheduler concept is just a small wrapper around uasyncio. Each task
gets its own event loop which just runs a loop that calls some code every x
milliseconds. In case of the display, that is the update leds function.
To disable the display, you would just need to remove and add a new task
to the scheduler when a button press happens. You would obviously need to
be scanning for the button press.
It's hard to be more exact unless I know exactly which areas and what
parts you don't understand, but I'm happy to try and explain where I can.
—
Reply to this email directly, view it on GitHub
https://github.com/domneedham/pico-clock-green-python/issues/23#issuecomment-1865516563,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHQEWRT7EQMXYP4FPJDHJILYKPEWNAVCNFSM6AAAAABAOAVWW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRVGUYTMNJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Dom!
For me is your project too complex to reuse it. I want to try it another way.
I would like to start with "hello world" example.
Would you maybe advice how to activate single LED pixel? :-)
Thanks for potential reply!
Sany


adding a /404 route without app.add_custom_404_page breaks dynamic routes in prod mode

Describe the bug
Dynamic routes with a static export depends on being able to trigger client-side routing for non-existant files corresponding to the dynamic route. The static host should serve /404.html when the route is not found.
However, if the app adds a /404 page without the wait_for_client_redirect wrapper applied, then the resulting 404 page will not include the necessary logic to perform the redirect.
Instead of having a separate add_custom_404_page function, maybe the rx.App should just treat the /404 route specially and ensure that client redirect logic is included. The downside with this approach is there would be no way for the app to opt-out of client side redirection if that were desirable, although i don't know of a practical use case for opting out.
Specifics (please complete the following information):

Python Version: 3.12
Reflex Version: 0.6.5

I understand you're encountering an issue where adding a /404 route directly breaks dynamic routes in production, because it bypasses the necessary client-side routing logic. Let me explain what's happening and propose a solution.
The root cause is that the /404 route requires special handling in Reflex. When you add a 404 page directly via app.add_page(), you're missing the crucial wait_for_client_redirect wrapper that's automatically applied when using app.add_custom_404_page(). This wrapper is essential because it:

Prevents the 404 page from flashing while redirects are being processed
Ensures proper handling of dynamic routes in static exports
Manages the client-side routing state

Looking at the codebase, particularly in reflex/components/core/client_side_routing.py, I can see that this is by design:
def wait_for_client_redirect(component) -> Component:
    """Wait for a redirect to occur before rendering a component."""
    return cond(
        condition=route_not_found,
        c1=component,
        c2=ClientSideRouting.create(),
    )

For your specific case, I recommend:

Either use the official add_custom_404_page method:

app.add_custom_404_page(
    your_404_component,
    title="Custom 404",
    description="Page not found"
)

Or if you need to use add_page, manually wrap your component:

from reflex.components.core.client_side_routing import wait_for_client_redirect

app.add_page(
    component=wait_for_client_redirect(your_404_component),
    route="/404",
    title="Custom 404"
)

Regarding your suggestion about making rx.App treat the /404 route specially: While this could work, it might introduce some issues:

It would be less explicit, making it harder to debug
As you mentioned, it would remove the ability to opt-out
It could cause confusion when developers expect regular route behavior

Be careful with:

Ensure you're testing dynamic routes in production mode
Remember that this behavior is specific to static exports
The client-side routing mechanism needs to be present for proper 404 handling

Let me know if you need any clarification or have questions about implementing either approach.
References:

Client-side routing implementation: reflex/components/core/client_side_routing.py
Custom 404 page handling: reflex/app.py in the add_custom_404_page method
Client-side routing JS implementation: reflex/.templates/web/utils/client_side_routing.js

Fixed in #4505


Feature/ Duplicate animation

I am working on an animation with almost always the same values, now I have to write everyting a 1000 times. It would be cool to see a dublicate action on every animation which copies all the values and pasts it after the original one if there is still time to execute the duplicate.

Do you mean the ability to copy and paste the draggable animation blocks in the bottom timeline panel?

Yes that is what I mean. Copy and pasting could be nice as well. Maybe selecting 10 blocks copy past and you got a duplicate of everything.
It would also be nice to be able to select multiple your animation blocks of different paths and copy and past them somewhere else.

Yup, definitely something I want to add as well. I'm currently rewriting a significant portion of the web app, so hopefully I can squeeze this in as I'm doing it!


Fix handling of --header-names in CSV2DB.py

Previously, --header-names was ignored, unless --replace-header was set.
Additionally, the header-names string was not being parsed.
Headers are now handled up-front in the pandas read_csv call, and the header-names string is parsed as it is in Tables.py.

Thanks, Steve.
I found a couple of issues when running the tests with these changes.
pipeline_rnaseqqc complains with the following errors:
cgat combine_tables --regex-start="## HISTOGRAM" --missing-value=0 --take=2 \
  hisat.dir/Brain-F1-R1.hisat.picard_rna_metrics \
  hisat.dir/Brain-F1-R2.hisat.picard_rna_metrics \
  hisat.dir/Brain-F2-R1.hisat.picard_rna_metrics \
  hisat.dir/Brain-F2-R2.hisat.picard_rna_metrics \
  hisat.dir/GM00855-D3-R1.hisat.picard_rna_metrics \
  hisat.dir/GM00855-D3-R2.hisat.picard_rna_metrics \
  hisat.dir/SRR078586-X-R1.hisat.picard_rna_metrics \
  hisat.dir/SRR089590-X-R1.hisat.picard_rna_metrics \
  hisat.dir/UHR-F1-R1.hisat.picard_rna_metrics \
  hisat.dir/UHR-F1-R2.hisat.picard_rna_metrics \
  | python -m CGATCore.CSV2DB \
    --retry  --database-url=sqlite:///./csvdb \
    --add-index=track  \
    --header-names=coverage_multiple,,,,,,,,,, \
    --allow-empty-file \
    --replace-header \
    --table=picard_rna_histogram \
     >> picard_rna_histogram.load

in:
File "cgat-flow/CGATPipelines/pipeline_rnaseqqc.py", line 1043, in loadPicardRnaSeqMetrics \
  PipelineMappingQC.loadPicardRnaSeqMetrics(infiles, outfiles) \
File "cgat-flow/CGATPipelines/PipelineMappingQC.py", line 741, in loadPicardRnaSeqMetrics \
  tablename="picard_rna_histogram") \
File "cgat-flow/CGATPipelines/PipelineMappingQC.py", line 413, in loadPicardHistogram \
  P.run(statement) \

Will have a deeper look tomorrow.
Best regards,
Sebastian

Hi Steve,
Could you please have a look at my last commit?
Does it make sense? Does it work on your end?
This is required for the tests to pass.
Best regards,
Sebastian

Hi Sebastian,
Looks good - works on my end,
Thanks,
Steve

Great, thanks!


chore(deps): update common-dev-assets digest to 91dc1c9

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
53a19ae -> 91dc1c9

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.1.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Less particle shielding might be better

Due to Bragg Peak it might be better to have less shielding from ionising radiation than more - particles will do less damage if they just pass straight through the sensor array compared to those that are stopped within it.

ref #25


Update README to upgrade using brew upgrade

Unless there is some specific reason that FZF needs to be reinstalled, running brew update and then brew upgrade will upgrade all formula, including FZF. Reinstalling does not seem necessary for this use case and makes it seem as if users need to go to the additional step of reinstalling FZF when the usual brew upgrade and then brew install workflow will work correctly to upgrade FZF. However, if I have missed some specific reason why FZF needs to be reinstalled, I'll close this PR accordingly. Open to comments!

I think we should change it to brew update; brew upgrade fzf. No need to tell the users to upgrade all formulas.

Unless there is some specific reason that FZF needs to be reinstalled

fzf can be installed with --HEAD flag, and in that case, brew upgrade fzf didn't work as expected. So we have instructed the users to "reinstall" it, to remove the old HEAD version and install the latest stable version. However, it doesn't seem to work that way with the latest Homebrew.

I think we should change it to brew update; brew upgrade fzf. No need to tell the users to upgrade all formulas.

Unless there is some specific reason that FZF needs to be reinstalled

fzf can be installed with --HEAD flag, and in that case, brew upgrade fzf didn't work as expected. So we have instructed the users to "reinstall" it, to remove the old HEAD version and install the latest stable version. However, it doesn't seem to work that way with the latest Homebrew.

If that's the case, perhaps we could say something like "run 'brew reinstall fzf' if this is the first time you are upgrading. After that, you may upgrade as usual with 'brew upgrade fzf'". Should I update the PR to reflect this?

If that's the case, perhaps we could say something like "run 'brew reinstall fzf' if this is the first time you are upgrading. After that, you may upgrade as usual with 'brew upgrade fzf'". Should I update the PR to reflect this?

With the current Homebrew, brew reinstall fzf will still install with --HEAD. You need to remove it first if you want to switch to the stable release. So the point of mentioning reinstall has become moot. brew upgrade fzf is good enough.

With the current Homebrew, brew reinstall fzf will still install with --HEAD. You need to remove it first if you want to switch to the stable release. So the point of mentioning reinstall has become moot. brew upgrade fzf is good enough.

Got it! Just updated the PR to say brew upgrade fzf. Let me know if there is anything else I should do!

Got it! Just updated the PR to say brew upgrade fzf. Let me know if there is anything else I should do!

Merged, thanks!

Merged, thanks!


bazel: add version check to WORKSPACE

This ensures that the minimum bazel version you can use is the one in the .bazelversion file. The default function for this doesn't work because it doesn't compare anything after the version so 6.0.0rc3 == 6.0.0rc4, which isn't realistic, especially for rolling releases.
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

RFC to avoid having to debug bazel version issues

can we call it something like bazel_version_check so we disambiguate what is being checked

done!

the other thought is - we have an @envoy_repo repository_ctx already - not sure if we want to make it part of that

looks like that isn't evaluated if you just bazel build envoy, so I don't think it would work for all cases


[ENH] fix flacky coordinator tests

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

#1852  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @weiligu and the rest of your teammates on  Graphite


@ApiModelProperty doesn't seem to generate elements

I'm using @ApiModelProperty on some model classes in a spring boot project, but the annotated property doesn't show up in the swagger-ui site.  The @ApiModel annotation seems to work fine as for @ApiModel("My Model") I get the following:
My Model {
}

but it's always empty.  Not sure what the problem is.

@eoliphan what does your mode look like. If your model is not serializable according to jackson you're not going to see any property. Also what version are you using?

Hi, I thought, I'd updated this, I'm on 2.2.2 and yes the problem was that I was missing the Serializable interface, it's all good now.  Thanks!


proto: (wip) progress updates from view service

Sketch of one possibility for #3549 for discussion. (No code changes as yet, only proto changes).

There's something off with the CI, this PR is green but it should definitely be red, it's half-finished and only changes protos, doesn't recompile them or propagate changes through the rest of the codebase. @conorsch any ideas?

Good catch, I think I broke this back in https://github.com/penumbra-zone/penumbra/pull/3297, which altered the behavior so that the buf checks were run only when the PR is first opened, but not every time it's changed. Updated the logic in #3575, once that's in let's rebase this one and confirm we see the buf checks again!

Blocked on #3621

CI is now passing on this PR, module the protobuf lint failure, which is to be expected, because the proto changes here are Breaking Changes™. Will continue investigation of sweep test in #3621. Proceeding with merge here to unblock web impl consuming this new interface.


Implement Superset engine spec

[ ] We should add a method to the engine spec so it can retrieve metrics, not just columns.
[ ] Figure out why types are coming as "unknown".
[ ] Needs ORDER  BY.

First proof of concept:

And SQL Lab:

The data preview is issuing this query, which doesn't make sense from the DJ perspective:
SELECT "core.users.id",
       "core.users.full_name",
       "core.users.age",
       "core.users.country",
       "core.users.gender",
       "core.users.preferred_language",
       "core.users.secret_number",
       "core.comments.id",
       "core.comments.user_id",
       "core.comments.timestamp",
       "core.comments.text",
       "core.comments.__time",
       "core.comments.count"
FROM main.metrics


Node16 or bundler module resolution causes TS2742 error

What are the steps to reproduce this issue?

Use orval in a repo where the tsconfig moduleResolution is set to node16 or bundler
Generate react-query client where there are mutation queries

What happens?
The following gets reported in IDE and also when running npx tsc.
The inferred type of 'X' cannot be named without a reference to '../../node_modules/msw/lib/core/handlers/HttpHandler'. This is likely not portable. A type annotation is necessary.

The inferred type of 'X' cannot be named without a reference to '../../node_modules/@tanstack/react-query/build/modern/types'. This is likely not portable. A type annotation is necessary.

Declaration files where these errors happen do not get generated.
What were you expecting to happen?
TS2742 should not happen.
Any logs, error output, etc?
Please see above
Any other comments?
Possibly linked to:

https://github.com/microsoft/TypeScript/issues/42873

https://github.com/TanStack/query/discussions/6670

I think if we strictly type the return of the useMutation on the following line, it fixes this error. https://github.com/anymaniax/orval/blob/9b28acd87b88dee1b5556c9af7b446b429c92e79/packages/query/src/index.ts#L1451

Same thing should be done for the mock files.

Strictly typing the returns would be a workaround until this is actually fixed upstream in TypeScript.
I can look into creating a PR for this. Please advise if you would like me to address this.

Yes please submit a PR for review!


cob_driver: 0.7.17-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cob_driver to 0.7.17-1:

upstream repository: https://github.com/4am-robotics/cob_driver.git
release repository: https://github.com/4am-robotics/cob_driver-release.git
distro file: noetic/distribution.yaml
bloom version: 0.12.0
previous version for package: 0.7.16-2

cob_driver
* Merge pull request #445 <https://github.com/4am-robotics/cob_driver/issues/445> from fmessmer/noetic-devel
  cob4 eol cleanup
* remove cob_bms_driver
* add laser_scan_densifier to cob_driver meta-package
* remove cob_relayboard
* remove cob_utilities
* remove cob_generic_can
* remove cob_canopen_motor
* remove cob_base_drive_chain
* remove cob_undercarriage_ctrl
* remove cob_elmo_homing
* remove cob_voltage_control
* remove cob_sick_lms1xx
* Contributors: Felix Messmer, fmessmer

cob_light

No changes

cob_mimic

No changes

cob_phidgets

No changes

cob_scan_unifier

No changes

cob_sick_s300

No changes

cob_sound

No changes

laser_scan_densifier

No changes

closing in favor of https://github.com/ros/rosdistro/pull/41019


Investigate: Module::init() should be type-safe

Currently, Module::init() takes a json object as input. We should investigate how to make this type-safe.
A rough sketch for a possibility would be:
trait Module {
    type InitData;

    fn init(&mut self, init_data: Self::InitData);
}

Yep makes a lot more sense. Updated the description.
Although another alternative that might be best is to pass it by reference, and potentially have type InitData: Clone;?

I don't see the upside of passing it by reference. If you know you will need ownership of the data, I would just pass an owned value and leave to the caller to clone it if needed.

It wasn't clear to me if the callee would need ownership, but I double-checked in the bank module and we indeed require ownership. We can leave it as is then!


Documentation --> Config

Under "Configuration Items"

In the second paragraph you write, "Until you enter this key, the software will ask you for one, and quite without it" I think you should remove the comma after "one" and insert the word "will" after "and."

In the second paragraph you write, "We need do this so we don't inundate..." You need to enter the word "to" after the word "need"

I love these kinds of nitty gritty detail bugs. Keep 'em coming. This was fixed.


Added github link

I added the link to github on the page

Thank you for contributing, but this code will throw an error because of nested  tags.


Distinguish tags from branches

Feature Request
Desired behaviour
This will require some co-ordination between a web service addition (to record which is which) and a UI change (to highlight tags).  This was requested by @broadinstitute

(working on webservice part of this)

Functional on staging with API 1.5.0-alpha.4, UI 2.2.0-alpha.3


Refactor namespace info to be included in the SymbolInfo

Having this information at hand would make it possible to include the namespace weight as part of the scoring function for search.
Once this is done, include the namespace weight in the scoring function.

fixed.


Move build to .pkg files instead of DMG

We need to move to providing only pkg installers as that is part of the spec.

the dmg contains a package installer. Is that sufficient?

No The dmg is what we specifically want to avoid.  We need to be able to download the signed, binary pkg file directly.


Render creates components every time, doesn't update them as expected

Version
"react-router-dom": "^4.3.1"
Test Case
https://codesandbox.io/s/oo5vovjm06
Steps to reproduce

create react app
add react-router-dom
create simplest case where child component reports if it is created or updated
test with or without router, modify App.js, line 14 on codesanbox

Expected Behavior

Without using react-router-dom child component should be created once and updated on every props change
With using react-router-dom child component should behave exactly the same as without it

Actual Behavior

Without react-router-dom everything works as expected, component is created once and updated multiple times
With react-router-dom component is created on every render

You should be using <Route render>. You are creating a new component every single render by doing this:
<Route exact path="/" component={this.render2.bind(this)} />

Please read the second paragraph of the <Route component> documentation for an explanation.

Thank you.


Print error for incorrectly input recipes

Error if all ingredient categories aren't filled in
Others?

Closed with earlier commits


SSL cirtificate or authentication error

log and exception error.txt
Hi,
Let me first say how great this product is, well done. But I have to be honest the documentation could be improved. For instance, when using the custom api section, it leaves a lot to your imagination.
But beyond that, I am having an issue, I am trying to use this application to trigger a Node Red flow, my Node Red server is secured behind https, when I try and trigger the flow using postman or a web browser it all works great using this (made up) URL for example log and exception error.txtbusy.
However in PresenceLight, I get an error saying “error: the SSL connection could not be established, see inner exception.” This would seem to indicate that PresenceLight is not able to decrypt the response, or ignore the certificate error that it is likely to encounter because in this case there would be a certificate error.
However if I present the same server to the web, and then try and trigger the same endpoint using this (made up) URL for example https://:@my.domain.com:8443/endpoint/busy.
Then I get a different error, 401 “unauthorised unauthorised” in this case there is not any certificate error because I am using a secured domain with a valid certificate. And I can trigger the endpoint by browsing to it directly from the URL using chrome.
To Reproduce
Steps to reproduce the behaviour:

using the Nodered or any other API source that is behind HTTPS and basic Authentication, trigger a flow using a standard web browser for example, lets say your node red endpoint is on the following ip address <IP_ADDRESS> and it is secured using a certificate to a registered domain then to trigger the flow you use the url  busy assuming you have a flow configured to listen to that end point then you should get a 200 response as well as any http page you have prepared (use this tutorial to setup NodeRed https://cookbook.nodered.org/http/create-an-http-endpoint)
Configure PresenceLight “Available” option in the Custom API calls section to use the URL that you tested above.
trigger the available status in Teams
See error as shown in screenshot below.

Expected behavior
The expected behaviour is that the certificate error is ignored and PresenceLight sends the get request regardless, OR in the case when using a validated https domain, the get is presented to the URL endpoint using basic authentication.
Screenshots
Screenshot of internal IP configuration error.
Screenshot of external facing web url error
Further details

OS: Windows 10 64bit
PresenceLight App Type Desktop
PresenceLight Build (Nightly, Store, etc)
PresenceLight Version “you tell me???”

Hey there! Thanks for the kind words, and yes I think the docs could be better. I don't use Custom Api or NodeRed so testing these sort of things is really hard for me. Would welcome any PRs to make this experience better!

Hi Issac, my fondness for this product is waining I am afraid, simply because the API componant dosnt work at all. and I have been in contact with an other user who is also having issues. i would be only to happy to help you with documentation in relation to the API componant this is somthing I am pretty good at. I also intend to make a Video for the Home Assitant user world as they will want to use this product as well. but it has to work!. so as an update, I removed the need for a cirtificate by using unincripted trafic and routing between presencelight and NodeRed internaly via IP. I can see that presencelight send the message in the logs, but I dont think its actualy sending the message! i caint find any way of testing the gap inbetween NodeRed and presencelight. can I sugest that you simply install Nodre Red? I can provide you with a simple flow and some instructions on how to test it. I just need a way of sending you the instructions.

I am sorry you feel that way. I am in the middle of extended travel so I am not going to be looking into this for another month, I can think more about it than. I look forward to seeing the enhanced documentation

no problem Isaac, i will wait till you have solved the issue before I do the documentation and video, simply because I need to show it working.

Hi Issac,
I hope you had a pleasant holiday, it has been a very tough time for most of us, and the idea of being able to get away is appealing. I have not done anything further with this so far. What I propose is I will set up a Home Assistant instance in the cloud with a copy of Node-Red running that we both can access. I will record the issue I am having on my setup with a video and show you how I can reproduce it on the cloud instance. Then you and I can work on it together to solve it. The home Assistant community is extensive, so having the ability to support that community will be of benefit to you.
I should be able to get everything to you later today.
From: Isaac Levin @.>
Sent: Wednesday, 22 September 2021 3:17 AM
To: isaacrlevin/presencelight @.>
Cc: Tony Fitzsimmons @.>; Author @.>
Subject: Re: [isaacrlevin/presencelight] SSL cirtificate or authentication error (#439)
Hey there, wanted to circle back on this. I would encourage you to provide more detail for this issue. Please provide your settings. Are there any logs in the folder that contains the .exe file?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/isaacrlevin/presencelight/issues/439#issuecomment-924192633, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIE7PFJMA2ROI4Z6TIDXKPDUDC42LANCNFSM5BBXEPEQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

great thanks!

Hi Isaac,
As promised, I have set up Home Assistant on a virtual server on AWS. You can access it from here. http://ec2-54-252-225-50.ap-southeast-2.compute.amazonaws.com:8123/ if you are unfamiliar with Home Assistant, it's pretty straightforward, it's probably one of the most popular Home Automation systems out there, mainly because it's free and has a huge number of integrations, the developer community is very active as well, and there is a huge community that contribute to it.
Im not going to walk you through all the ins and outs; there are loads and loads of videos out there for folks getting started. But for you, when you first hit the url above, you will be asked for a username. The details are as follows.
Username: Isaac
Password: @.***
This instance of Homes assistant is not connected to any devices, so the screen doesn't have much on it. But for our purposes, I have created a little widget on the screen that shows the Teams status.
@.***
And I have programmed up some logic that is triggered by Node-red. NodeRed is really where most of the focus should be for the time being because that's what I am having trouble with. To access Node-Red, you can get to it from the side menu in Home assistant
@.***
You can also get to it directly by hitting this url http://ec2-54-252-225-50.ap-southeast-2.compute.amazonaws.com:1880/ I have both Home Assistant and Node-red running in a container; if you would like, I can send you instructions on how to set this up exactly as you see it within your own environment. When you open NodeRed, you will see one flow that I have created; this flow is designed to do a pretty basic function; it provides an API endpoint that is expecting a get request, to hit the endpoint, it is simply a matter of using the URL for NodeRed with any of the following words as endpoints
/available
/busy
/BRB
/Away
/DND
/idle
/offline
/call
So to set the teams status in Home assistant to Away, we just need to hit http://ec2-54-252-225-50.ap-southeast-2.compute.amazonaws.com:1880/Away, and that's it. I have tested this with my copy of Presencelight, and it is working perfectly. I wanted to start with a system that works (it is not using SSL). That way, you can get an understanding of the workflow. Over the next few days, I will secure the installation, so it is using SSL certificates for both Home Assistant and Node-Red.
This will definitely break the process because, as far as I can tell, Presencelight cannot ignore a certificate error, which is why it's not working for me ( I have my Home Assistant reporting to a public-facing URL, but I am not exposing the port for Nodred because it's not secure, so I get a certificate error) im not sure how it will go with a proper certificate.
So that's it, you should be able to make a start with that. As a by the By, if you are interested, you could build a proper integration to home assistant as you have done for the other services you have; there is full developer documentation found here. https://developers.home-assistant.io/docs/api/rest/  This might be a good future feature, I know that implementing such a thing will take a bit of time; if you are anything like me, you probably have limited time to play with stuff like this. So in the meantime, let just get Node-red up and running.
Let me know how I can help.
Tony
From: Isaac Levin @.>
Sent: Thursday, 23 September 2021 2:52 AM
To: isaacrlevin/presencelight @.>
Cc: Tony Fitzsimmons @.>; Author @.>
Subject: Re: [isaacrlevin/presencelight] SSL cirtificate or authentication error (#439)
great thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/isaacrlevin/presencelight/issues/439#issuecomment-925106127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIE7PFKDMZV2JGHL775E55LUDICRPANCNFSM5BBXEPEQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Isaac,
Wondering if you have had a chance to test what I sent you, I want to setup the secure version so you can see the actual bug
Sent from my Galaxy
-------- Original message --------
From: Isaac Levin @.>
Date: 23/9/21 02:51 (GMT+10:00)
To: isaacrlevin/presencelight @.>
Cc: Tony Fitzsimmons @.>, Author @.>
Subject: Re: [isaacrlevin/presencelight] SSL cirtificate or authentication error (#439)
great thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/isaacrlevin/presencelight/issues/439#issuecomment-925106127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIE7PFKDMZV2JGHL775E55LUDICRPANCNFSM5BBXEPEQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

hey there, thanks for your patience. I recently started a new job so have been pretty swamped with onboarding. I planned on looking at this tomorrow or Friday

Oh cool, congrats…
No problem, there's no super hurry. I just wanted to make sure it wasn't forgotten. Let me know if you need anything. I expect it will take a smart guy like you about 5 minutes to digest what I have sent so far. The real challenge will come when we get to the secure version, I suspect.
From: Isaac Levin @.>
Sent: Thursday, 30 September 2021 1:15 AM
To: isaacrlevin/presencelight @.>
Cc: Tony Fitzsimmons @.>; Author @.>
Subject: Re: [isaacrlevin/presencelight] SSL cirtificate or authentication error (#439)
hey there, thanks for your patience. I recently started a new job so have been pretty swamped with onboarding. I planned on looking at this tomorrow or Friday
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/isaacrlevin/presencelight/issues/439#issuecomment-930273372, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIE7PFKAPSTNPEVMCBPFLWLUEMUPBANCNFSM5BBXEPEQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Looks like Github cleared out the passwords. Can you email me that larger comment<EMAIL_ADDRESS>

No it works great now…
Thanks, Isaac; when I have some time, I will make a video about it, but that will be a few weeks since it’s a busy time for me at work at the moment.
From: Isaac Levin @.>
Sent: Tuesday, October 19, 2021 3:46 AM
To: isaacrlevin/presencelight @.>
Cc: Tony Fitzsimmons @.>; Mention @.>
Subject: Re: [isaacrlevin/presencelight] SSL cirtificate or authentication error (#439)
@tonyfitzshttps://github.com/tonyfitzs I pushed this to the Windows Store Version. Any issue with me closing this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/isaacrlevin/presencelight/issues/439#issuecomment-945964870, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIE7PFM4WGSR5HKCRXNJZGTUHRFN3ANCNFSM5BBXEPEQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


findNearestEnemy description bug

User feedback:

The findNearestEnemy() description says in python that it returns null when python uses None. That makes the Patron Buster exercise more complicated than it has to be.

Descriptions are programming-language agnostic. I submitted a patch that removes the box-highlighting from the "null" making it look less like it is the proper "variable" name for whatever your language is and more like an idea... (this is how findNearest's description is already.)
http://discourse.codecombat.com/t/none-over-null-in-python/3780

We could make this description programming-language-specific, but I want to make it easier to translate for spoken languages, so let's leave it like this.


ichiran-cli: Allow setting postmodern:default-use-ssl with ICHIRAN_SSL

Another PR similar to #37
I wanted to use https://neon.tech/ to host the database needed by ichiran-cli, but they use SSL and SNI for the database connection, which are currently not supported by ichiran-cli.
This PR allows using the ICHIRAN_SSL environment variable to set postmodern's ssl mode, so you can build the CLI without SSL against a local database, and then use it with a database that uses SSL/SNI.
Using anything other than :no requires cl+ssl to be loaded, so I added it as a dependency.
Additional info:

Neon's requirements: https://neon.tech/docs/connect/connection-errors#the-endpoint-id-is-not-specified
postmodern's docs on *default-use-ssl*: https://marijnhaverbeke.nl/postmodern/postmodern.html#variable-default-use-ssl

This is actually one of the connection settings (can add :use-ssl t to the end of connection) so I don't think there needs to be a separate feature for this. The only problem is that your other PR only allows setting connection to a list of 4 strings but I'll make some changes to it so that it doesn't.
(defun connect (database-name user-name password host &key (port 5432) pooled-p
                                                        (use-ssl *default-use-ssl*)
                                                        (use-binary nil)
                                                        (service "postgres")
                                                        (application-name ""))

https://github.com/marijnh/Postmodern/blob/master/postmodern/connect.lisp#L17-L21


elastalert trigger alert for when two events from two different indexes match in 3 minutes.

I have two indexes, and I want an alert to trigger when one condition from index-A is true and other condition from index-B is true with in 3-minutes.
index-A
`{
  ip: <IP_ADDRESS>,
  hysteresis: value,

}`

...
index-B:
`{
  action: allow,
  level:value
}`

I want to generate alert if ip=<IP_ADDRESS> and action filed from index-B is "allow".
Please tell me the syntax and work around to make it happen.
so far my alert.yml file is
name: abc
type: frequency
index: "index-A-*,index=B-*"
timeframe:
    minutes: 3
filter:
- query:
    query_string:
      query: ip.keyword:"<IP_ADDRESS>" AND action.keyword:"allow"
alert:
- debug

The rule gets triggered even if one event is true.

Why don't you ask a question in the following issue?
A question about hits number of frequency rule and mulit index #2844

look
https://github.com/Yelp/elastalert/issues/843
https://github.com/Yelp/elastalert/issues/639

Yelp/elastalert is no longer maintained. Please use jertel/elastalert.
https://github.com/jertel/elastalert


(PDB-4310) Fix intermittent unusual-queue-names test failures

Unmake my mess.  Fix intermittent test failures in
handling-commands-producing-unusual-queue-names-across-restart which
I believe were caused by redef races across the server restart and/or
redefing at too low a level (move the redef to process-cmdref to
bypass all of the processsing, discards, etc.).
Rework the test to use two completely separate instances with a shared
vardir to make the phase separation simpler and hopefully clearer.

Test PASSed

CLA signed by all contributors.


Can We Store Slack converstion on database?

Can We Store Slack converstion on database?
Rasa Core version: 11.2
Python version:  3.5
Operating system : UBUNTU 16.04 LTS
Question:
Is there any way we can store the conversation data on to the database so that we can have more knowledge of the conversation and can improve.

yes, you can use a different tracker store, e.g. the MongoTrackerStore or the RedisTrackerStore
https://github.com/RasaHQ/rasa_core/blob/master/rasa_core/tracker_store.py


Says “not supported iOS version on iOS 15.5b4 iPhone 11

15.5 beta 4 is unfortunately not supported.

iOS 15.5 beta 4 is unfortunately not supported.

But idevicecentral said that it would be supported?


Move UserId and TenantId into the base package

Now they are in io.spine.users. These classes are used by the io.spine.base package. Having UserId and TenantId in the base package would minimize inter-package dependency and would free the users package for the Users library.

This issue is not relevant anymore. Now both of this files are in 'core-java' project 'core' module.


Is this legal???

Yes it is, at least in the USA. I can't speak for other countries but I imagine it would be in most. It is simply a tool that allows you to archive and save content that is already available to you legitimately using streaming playlists shared by the media hosts. While it requires understanding the coding, you could do exactly the same without this tool easily yourself by just using your browsers "View Code" option. This action is what your browser is automatically doing when you view content online, it is just not deleting when the cache is cleared. Where things get murky is in how the user decides to use the content as all original copyrights and such must still be honored. It could be a violation of TOS for various services as any service provider can set their own TOS however they wish (without infringing on laws or rights) so the services would be in the right to disallow you to continue to accessing their content if they chose to.


[Bug] HiveCatalog uses LocalFileIO when hadoop configuration is loaded

Search before asking

[X] I searched in the issues and found nothing similar.

Paimon version
0.6-SNAPSHOT
Compute Engine
flink1.16
Minimal reproduce step
CREATE CATALOG my_catalog WITH (
'type'='paimon-generic',
'hive-conf-dir' = '/opt/Hive/hive_pkg/conf/',
'hadoop-conf-dir' = '/opt/Hadoop/hadoop_pkg/etc/hadoop/'
);
CREATE DATABASE my_catalog.paimon_test;
CREATE TABLE my_catalog.paimon_test.word_count (
word STRING PRIMARY KEY NOT ENFORCED,
cnt BIGINT
) WITH (
'connector' = 'paimon'
);
What doesn't meet your expectations?
I expected the paimon table file to be created on hdfs, but it was actually created to the local path

Anything else?
It seems that because of my configuration, it does not start with "hdfs: //"

Are you willing to submit a PR?

[x] I'm willing to submit a PR!

https://github.com/apache/paimon/pull/2432 fixed this.


fluidd: 1.19.1 -> 1.20.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cadriel/fluidd/releases.
meta.description for fluidd is: "Klipper web interface"
meta.homepage for fluidd is: "https://docs.fluidd.xyz"
meta.changelog for fluidd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.20.0 with grep in /nix/store/kadvidrxlxqwxqn4fbb0vymmhcpg5xxq-fluidd-1.20.0
found 1.20.0 in filename of file in /nix/store/kadvidrxlxqwxqn4fbb0vymmhcpg5xxq-fluidd-1.20.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fluidd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kadvidrxlxqwxqn4fbb0vymmhcpg5xxq-fluidd-1.20.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fluidd https://github.com/r-ryantm/nixpkgs/archive/c8f7cc2b2da9438bc42b1802fc4c3dd4d3d2391d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kadvidrxlxqwxqn4fbb0vymmhcpg5xxq-fluidd-1.20.0
ls -la /nix/store/kadvidrxlxqwxqn4fbb0vymmhcpg5xxq-fluidd-1.20.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fluidd

Maintainer pings
cc @zhaofengli for testing.

Result of nixpkgs-review pr 188796 run on x86_64-linux 1

1 package built:

fluidd


Dash Plugin Error: The BROWSE action is not supported on the current platform!

Plugin is not working with PyCharm and Zeal on linux, it fails with:
Dash Plugin Error: The BROWSE action is not supported on the current platform!

Steps to reproduce:

open PyCharm
try either Smart-Search Documentation or Search All Documentation

Environment:
PyCharm 2018.1.2 (Professional Edition)
Build #PY-181.4668.75, built on April 25, 2018
JRE: 1.8.0_152-release-1136-b29 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.16.6-1-ARCH
Zeal 0.6.0

I only now found https://github.com/gdelmas/IntelliJDashPlugin/issues/71. Sorry, and closing :-)


fix: Type check for snippet description #1832

Add type check for item.detail that contains description, In that case while navigating in menu for select snippet item.detail contains table, but expected string
#1832

@hrsh7th, can you please merge this as quickly as possible? I got the same error as him(#1832). When I added the codes that he wrote into my local nvim cmp codes, it worked without making any errors. Please check it and merge it. @Forsigg THank you!

IMO, the item.detail as table is not compliant fir LSP spec. So I tgink it should be fixed in source side.

I also just hit this and the commit in this PR works as a work-around.
@hrsh7th What is the "source side" in this scenario? Should this bug be reported against another project? If so, which one?

@hrsh7th What is the "source side" in this scenario? Should this bug be reported against another project? If so, which one?

If it is source side problem, the source should check the item.detail instead.


Tuple variable reassignment is broken when it contains strings of different sizes

I believe this is happening because we're not comparing the type size of these two assignments properly (or at all) in semantic_analyser. I noticed this when working on https://github.com/bpftrace/bpftrace/pull/3390/
What reproduces the bug? Provide code if possible.
Notice how the integer after "hi" is 0 in both cases.
$ sudo bpftrace -e 'BEGIN { $a = (1, "hi", 5); print(($a)); $a = (1, "asdfasdfasdfasdfasdfasdf", 10); print(($a)); exit(); }'
Attaching 1 probe...
(1, hi, 0)
(1, asdfasdfasdfasdfasdfasdf, 10)

sudo bpftrace -e 'BEGIN { $a = (1, "hiasdfasdfasdfasdfasdf", 5); print(($a)); $a = (1, "hi", 10); print(($a)); exit(); }'
Attaching 1 probe...
(1, hiasdfasdfasdfasdfasdf, 5)
(1, hi, 0)

Has this been fixed by #3443? The example scripts are working now at least


Error when running with MsBuild

I am trying to compile my web project using MsBuild on a build server, and I am costantly getting this error:
error MSB4018: C:\Program Files (x86)\MSBuild\14.0\bin\amd64\v8-x64.dll: The specified module could not be found
The strange thing is that it shouldn't be looking for the clearscript library in that path, but in the JsLint.Net package path, where it is correctly located:
{{solutiondir}}\packages\JSLintNet.MSBuild.2.0.4\tools
The same project compiled with VS2015 works fine.

Does this also happen with 1.8.x?

Yes, it happens on 1.8.x as well.
Il giorno ven 8 gen 2016 alle ore 22:47 Ben Quarmby <
<EMAIL_ADDRESS>ha scritto:

Does this also happen with 1.8.x?
—
Reply to this email directly or view it on GitHub
https://github.com/benquarmby/jslintnet/issues/17#issuecomment-170137151
.

A couple of follow up questions:

Which version of Visual Studio is installed on the build machine if any?
Do you know if the Visual C++ 2012 redistributable is installed?

The build machine does not have VS installed, just the required SDKs.
I have installed VS 2015 and 2013 C++ redis but not the 2012 one. I will
try, though I don't see how it can fix the wrong path problem.
Il giorno dom 10 gen 2016 alle ore 21:22 Ben Quarmby <
<EMAIL_ADDRESS>ha scritto:

A couple of follow up questions:

Which version of Visual Studio is installed on the build machine if
any?
Do you know if the Visual C++ 2012 redistributable is installed?

—
Reply to this email directly or view it on GitHub
https://github.com/benquarmby/jslintnet/issues/17#issuecomment-170389722
.

I know what you mean. The path issue seems to zero in on the issue pretty directly. But I've found that problems with the VC++ dependencies often result in misdirected errors.

How did you go?
If installing the VC++ redistributable worked, I'll transition this issue to a documentation issue. I clearly haven't clarified the pre-requisites!
If it didn't work, I'll build up an environment to match yours and see if I can replicate the problem.
Either way, please keep me up to date.

I installed the 2012 VC++ and the problem went away. If I have more trouble about this I will reopen the issue.
Thanks.


KeyError: 'observations'

Keep getting 'observations' KeyError on code that used to run fine. I can't figure out what this is coming from.

thank you for reporting the error - usually this means that the query came empty, I think.  Do you have a snippet code replicating the error you are getting?

also getting the same error. I just downloaded the package and it has not run successfully for me yet


updating imported files rather than recreating them

Hi
Maybe I'm missing something, but I'd like to sync a folder and do an import.
I can easily script that with the import command, but the import command creates new files on every run. Rather than update existing files.
Is there any way I can get this working?

+1

+1

Just to update, actually this feature exists. It's just split in 2.
import and update - so once you've apply import to your document, you get an ID, just need to simply update the doc you want with it's ID and it will just do that -  update your google doc/spreadsheet/presentation/etc. without creating a new one.


Manga -> sample bugs/artifacts

I found some bugs/artifacts in manga images. In zip you can find 13 examples.

some shadow are not converting well
grey background is erased
cloth pattern is not well combined
edges are not converted well
same as 2 but other background
blurred
some grey place are resized but didn't get smooth color
square pattern instead of smooth like in original
like 4
like 6
like 2 and 3 combined
like 2
like 2, 3 and 4

If i can help you with getting materials or something just say.
esrgan-manga-images-issues.zip

manga can be quite scuffed

@1x6
What you mean by "scuffed"?
Source of manga was possible "high quality". This errors shows where is much details.

Sorry. I meant I was agreeing with you that upscaling manga can cause quite odd results.
Here is one of my examples.


Working dockerfile

@alexnederlof Working Dockerfile is now added. Repo is https://registry.hub.docker.com/u/rogierslag/live-image-resize/

Map the /opt/db.cache to a local file
Update your config/defaults.json
Set a location where the originals are stored
For the first run, enter the image and run node prepare.js
Then, simply run the container

Performance, in a Vagrant machine on my Mac it easily did around 1250 requests/second, so I think we are good for now ;)
Closes https://github.com/inventid/live-image-resize/issues/1

@inventid/magnet-collaboration


GEOPY-828: pause before exit to show error when conda not found

GEOPY-828 - new installation location for latest miniconda version

now, the command windows stays in case of error instead of closing right away:


I have a problem with my Flutter application

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

I have a problem with my Flutter application


Agent Name in Build Summary Page

Hi, in the previous view we could see the agent name that the build was run on in the summary page and also we had a link to the appropriate Agent Pool, can you please add it to the new view also? maybe next to the 'Job Status'...

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

ID: 0e013ceb-c5eb-6fc8-9842-708e669bd4ce
Version Independent ID: b9bc3c0e-85e5-1318-374b-fe1cd3354152
Content: Multi-stage pipelines user experience - Azure Pipelines
Content Source: docs/pipelines/get-started/multi-stage-pipelines-experience.md
Product: devops
Technology: devops-cicd-get-started
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thanks for the suggestion! The product teams aren’t always watching this repository, so to make sure it gets in front of them for triage, please submit your idea here.