⚠ gorzdrav.org has degraded performance

In 6620e32, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 2480 ms

Resolved: gorzdrav.org performance has improved in 5819429.


chore: fix symbol error

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


07/26/2018 05:46 : Create Pull Request for branch prTesting_2018072605 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix DCO CLI example in CONTRIBUTING.md

Signed-off-by: Tyler Ohlsen<EMAIL_ADDRESS>Description
Fixes the example flag passed in the DCO CLI example to not have added whitespace.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

start gradle check


could not find copy/paste mechanism

Tried all four solutions and they don't work on linux
Installed xsel and xclip still got same error
could not find a distribtion for gtk and PyQt4

Using a CentOS 8.2.2004 inside Virtualbox, commandline only.
As above installed both xsel and xclip,.
As above, when attempting to install either (py)gtk or PyQt4, I get
"Could not find a version that satisfies the requirement  (from versions: )
No matching distribution found for '
When running pip3 install --user pygtk I get
Complete output from command python setup.py egg_info:
********************************************************************
* Building PyGTK using distutils is only supported on windows. *
* To build PyGTK in a supported way, read the INSTALL file.    *
********************************************************************
Is there a way for pyperclip to not require X11?


Question: SignalR JWT authorization

Describe the bug
Due to the limitation of the Browser WebSocket, we cannot set custom headers to the connection so I had to find another way to secure it. I don't like to set the authorization token as a query parameter because it can be intercepted by malicious people and used to access also the other APIs. So I was thinking if I cannot secure the WebSocket with an HTTP header I can put under authorization only the POST /negotiate call.
I successfully did that by implementing my own HttpClient and passed to the client HubConnectionBuilder. But now I don't know how to put the /negotiate call under the authentication on the server side.
On my Startup.cs, under AddJwtBearer option, I'm listening for the call to /negotiate but I don't know how to validate the JWT token... Is there another way to validate authentication rather than the Authorize attribute? Or has someone a better solution?
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 2.2 with Angular 7

I can put under authorization only the POST /negotiate call.

This isn't sufficient to secure your application. The connection ID returned by the /negotiate call is not designed to be used as a security token.

Thanks for your answer. Maybe I badly explained myself. I don't want to use the connection Id returned by the /negotiate call to secure my API. I already use an JWT token and I set it as an Authentication header to the /negotiate throw my custom HttpClient.
Server side I want to validate my token inside the Startup.cs. The only way that I found to validate an JWT token (server side) is throw the Authorize attribute. But I can't use it inside the Startup.cs.

I was referring to the fact that you only wanted to secure the /negotiate call. It's not sufficient to secure the /negotiate call, you also need to secure the actual WebSocket request. It's possible for a client to simply connect a WebSocket and spoof a connection ID. It's not something we'd expect a client to do, but the connection ID isn't designed as a security token (for precisely this reason).

@anurse Ok, you are right. Is there any way to secure also the WebSocket with a JWT token without passing it as a query parameter? The token can be sniffed from the WebSocket URL and used to access also the other API.
If not I was thinking to use a Cookie to secure the /negotiate and the WebSocket. The JWT will be used for the other API. But I don't think is an elegant solution. What do you think about?

If you're using https/wss then the URL should not be sniffable. https://security.stackexchange.com/questions/176164/is-it-possible-to-sniff-https-urls

Oh, nice to hear that. Thank you @BrennanConroy. So, if the query parameters cannot be sniffed throw https/wss the problem is solved. Thanks again.

The token can be sniffed from the WebSocket URL and used to access also the other API.

The token can always be sniffed if the request is sent plain-text (non-https). Even when it's in a header, it's still just part of the plaintext request. It is essential that you always use https when dealing with user authentication.


[📚] [Firebase Crashlytics] Handling uncaught errors

The guidance here is no longer compatible with Firebase Crashlytics 0.2.0 because the FirebaseApp must first be initialized -- you get this exception:
[VERBOSE-2:ui_dart_state.cc(166)] Unhandled Exception: [core/no-app] No Firebase App '[DEFAULT]' has been created - call Firebase.initializeApp()
#0      MethodChannelFirebase.app (package:firebase_core_platform_interface/src/method_channel/method_channel_firebase.dart:118:5)
#1      Firebase.app (package:firebase_core/src/firebase.dart:52:41)
#2      FirebaseCrashlytics.instance (package:firebase_crashlytics/src/firebase_crashlytics.dart:31:62)
#3      main (package:kozzeter/main.dart:21:46)
#4      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:233:25)
#5      _rootRun (dart:async/zone.dart:1190:13)
#6      _CustomZone.run (dart:async/zone.dart:1093:19)
#7      _runZoned (dart:async/zone.dart:1630:10)
#8      runZonedGuarded (dart:async/zone.dart:1618:12)
#9      _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:225:5)
#10     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:301:19)
#11     _RawReceivePortImpl._handleMessage (dart:isolate-patch<…>

You also get this exception when trying to use FirebaseCrashlytics.instance.log() before Firebase.initializeApp() was called.
And you really have to await the Firebase.initializeApp() before being able to call FirebaseCrashlytics.instance.log().

You might want to run Firebase.initializeApp() in your main():
https://github.com/FirebaseExtended/flutterfire/issues/3271#issuecomment-677703143

Can confirm that adding Firebase.initializeApp() to main() works
You may also see an exception like:
Unhandled Exception: ServicesBinding.defaultBinaryMessenger was accessed before the binding was initialized.
If you're running an application and need to access the binary messenger before `runApp()` has been called (for example, during plugin initialization), then you need to explicitly call the `WidgetsFlutterBinding.ensureInitialized()` first.

Add the method call WidgetsFlutterBinding.ensureInitialized(); to your main can solve this
Your main() will look something like:
void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await Firebase.initializeApp();

  runZonedGuarded(() => runApp(new App()), FirebaseCrashlytics.instance.recordError);
}

For the record, I was able to get this to work.  The purpose of this issue is to ensure that the documentation gets updated.


Avoid connections to the server to only display a notification (i.e. deploy:starting)

Some servers with strict SSH policies block the script if we connect too many times.

Sadly, this is a limitation of envoy. If we wanted to fix that, we'd most likely have to write how the compilation of the Envoy.blade.php file is done so that we can have pre/post task callbacks that would be 100% php.


Incorrect syntax near the keyword 'as' in creation of catalog

Hi
I'm trying out my model wity the latest release of dbt-sqlserver i.e. 1.8.2 and the model runs but I cannot create documentation.
When I run dbt docs generate it fails with the error Incorrect syntax near the keyword 'as'.
Looking at the log the issue seems to come from the macro sqlserver__get_catalog.
With my model the cols CTE compiles to
cols as (
    select
        c.object_id,
        c.name as column_name,
        c.column_id as column_index,
        t.name as column_type
    from sys.columns as c with (nolock)
    left join sys.types with (nolock) as t on c.system_type_id = t.system_type_id
)

Moving the table aliasing as t to be before  {{ information_schema_hints() }} here with (nolock) resolves the issue with the query not running for me.

https://github.com/dbt-msft/dbt-sqlserver/blob/master/dbt/include/sqlserver/macros/adapter/catalog.sql#L99 This line here right?

Yes, I just created a PR to change it. #557


Content management training

Create an expanded presentation on content management principles to be given to teams who adopt the design system. We have already given a brief presentation to DCC staff that will form the nucleus of these slides.

Draft slides. I will review after the July 27 analytics training with DCC.

Sent slides to Christina at DCC. This work is done for now, but we will revisit and port these principles to a website when one is developed.


Check taskrun status when displaying logs

Fixes
https://issues.redhat.com/browse/RHTAPBUGS-126

Description
Page crash happens when pipelinerun.status is not available.
This results in another error when the modal tries to update itself but the modal provider is removed from the tree.

Add null check for status.taskRuns of pipelinerun when displaying logs modal.
Move error boundary up the tree to cover modal provider.

Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review
https://user-images.githubusercontent.com/20013884/233167332-85f41be2-0bf6-4250-8948-faf40b1e28c4.mp4

How to test or reproduce?

Browser conformance:

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Edge

/approve
/lgtm

/retest


Add AssembleDiagonal to VectorMass and VectorDiffusion

This PR adds the AssembleDiagonalPA method to the VectorMassIntegrator as well as VectorDiffusionIntegrator.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

https://github.com/mfem/mfem/pull/1257
@jandrej
@v-dobrev
@barker29 + @YohannDudouit
01/28/20
⌛due 02/18/20
⌛due 02/25/20

@jandrej, can you address @YohannDudouit's comments above?
Maybe mention also in CHANGELOG?

✅. They will be mentioned in an upcoming miniapp where there is actual usage for them. Do you still want me to add them to the CHANGELOG?

Currently there is this line in CHANGELOG for the new release:

Diagonals of partially assembled operators can now be computed efficiently.
See the new methods AssembleDiagonal in BilinearForm, AssembleDiagonalPA in
BilinearFormIntegrator and the implementations in fem/bilininteg_*.cpp.

I guess that's general enough to cover this contribution, so maybe no need to mention it?

Agree.

@YohannDudouit, your review is due

Merged in next for testing...

Re-merged in next for testing...


Error On Running Examples

I am trying to integrate splinter DB into an application of mine. I tried running the intro examples, and I encountered this error.
Assertion failed at src/clockcache.c:2234:clockcache_get_internal(): "SUCCESS(status)".
Could this be because the DB file does not exist? If yes, is there any way to run splinter DB without the DB file?

I narrowed it down to a pread call which is happening in laio_read

Hi Aaditya,
Thanks for the bug report.
Is one of the examples failing, or is it your code that's failing?  I just
built and successfully ran the examples in the current main branch.
What OS are you compiling and running on?
If it's your own code that is encountering the error, can you send me an
example that you think should work but is not working?
It is not currently possible to run splinter w/o a database file.  However,
splinterdb_create should create the file for you.
Best,
Rob
On Fri, Nov 17, 2023 at 2:22 PM Aaditya Rangarajan @.***>
wrote:

I narrowed it down to a pread call which is happening in laio_read
—
Reply to this email directly, view it on GitHub
https://github.com/vmware/splinterdb/issues/600#issuecomment-1817191187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA45FRKX66OWWX7VHGHBOEDYE7PRRAVCNFSM6AAAAAA7QGSWQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGE4TCMJYG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Rob,
I am just trying to run splintered_intro_example on Ubuntu 18.04. I tried debugging it and narrowed it down to src/platform_linux/laio.c, in the laio_read function. The pread() is throwing an error. (It says operation not supported, but I am running the ext3 file system).
I am just trying to create a splinterDB instance and insert elements into it, using the same code as shown in the example.
Best,
Aaditya.

It looks like this is due to the fact that ext3 does not support
fallocate.  There is an uncaught failure in the fallocate call earlier when
splinter is creating the database.
I've pushed a PR for splinter to catch the error.
This won't solve your problem on ext3, though.  It will just result in a
more meaningful error message.
Can you move to ext4?
Best,
Rob
On Fri, Nov 17, 2023 at 5:20 PM Aaditya Rangarajan @.***>
wrote:

Hi Rob,
I am just trying to run splintered_intro_example on Ubuntu 18.04. I tried
debugging it and narrowed it down to src/platform_linux/laio.c, in the
laio_read function. The pread() is throwing an error. (It says operation
not supported, but I am running the ext3 file system).
I am just trying to create a splinterDB instance and insert elements into
it, using the same code as shown in the example.
Best,
Aaditya.
—
Reply to this email directly, view it on GitHub
https://github.com/vmware/splinterdb/issues/600#issuecomment-1817307627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA45FRK7NK75TMM5KKZVVEDYFAEPPAVCNFSM6AAAAAA7QGSWQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGMYDONRSG4
.
You are receiving this because you commented.Message ID:
@.***>

Oh sure, thank you! Yes, we will move to ext4. Thanks!
Aaditya.


DeviceOrientationEvent.requestPermission()

Hello,
In order to make this plugin work on iOS, you need to ask user permission for device orientation capture.
To do that, we need to insert the following code in the requestSession action.
navigator.xr.requestSession('immersive-vr', {
requiredFeatures: thisXRMananger.gameModule.WebXR.Settings.VRRequiredReferenceSpace,
optionalFeatures: thisXRMananger.gameModule.WebXR.Settings.VROptionalFeatures
}).then(async (session) => {
if (typeof(DeviceOrientationEvent) !== "undefined" && typeof(DeviceOrientationEvent).requestPermission === 'function')
{
DeviceOrientationEvent.requestPermission()
.then(response => {
if (response === 'granted') {
session.isImmersive = true;
session.isInSession = true;
session.isAR = false;
thisXRMananger.xrSession = session;
thisXRMananger.onSessionStarted(session);
}
}).catch(function (error) {
thisXRMananger.BrowserObject.resumeAsyncCallbacks();
thisXRMananger.BrowserObject.mainLoop.resume();
});
} else {
session.isImmersive = true;
session.isInSession = true;
session.isAR = false;
thisXRMananger.xrSession = session;
thisXRMananger.onSessionStarted(session);
}
});

It seems that the DeviceOrientationEvent permission is required for iOS devices.
I'm not sure if it should be added in WebVRDevice
https://github.com/immersive-web/webxr-polyfill/blob/main/src/devices/WebVRDevice.js
or in CardboardXRDevice
https://github.com/immersive-web/webxr-polyfill/blob/main/src/devices/CardboardXRDevice.js

It seems that the DeviceOrientationEvent permission is required for iOS devices.
I'm not sure if it should be added in WebVRDevice
https://github.com/immersive-web/webxr-polyfill/blob/main/src/devices/WebVRDevice.js
or in CardboardXRDevice
https://github.com/immersive-web/webxr-polyfill/blob/main/src/devices/CardboardXRDevice.js

The src/devices/WebVRDevice wraps native WebVR 1.1 implementations, and src/devices/CardboardXRDevice uses the rendering polyfill, cardboard-vr-display -- confusing, I know.
There's an open issue for permission change request for iOS there that's related to this. I don't have an iOS device to test this FWIW


POC of client-side jakarta namespace transformation of testsuite/doma…

…in deployments

@darranl I've fixed the test failures I was seeing with earlier iterations of this. As this is just a POC and you're aware of its branch, I'm going to go ahead and close it now.


[CLI] trl env for printing system info

What does this PR do?
$ trl env

Copy-paste the following information when reporting an issue:

- Platform: macOS-14.5-x86_64-i386-64bit
- Python version: 3.10.14
- PyTorch version: 2.2.2
- CUDA device: not available
- Transformers version: 4.44.2
- Accelerate version: 0.30.1
- Accelerate config:    not found
- Datasets version: 2.19.1
- TRL version: 0.11.0.dev0
- HF Hub version: 0.24.6
- PEFT version: 0.10.0

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


scraper fails to scrape resource page properly

encountered an AWS CF resource page, from which the scraper managed to scrape only first two properties and completely ignored all the rest
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigateway-restapi.html

Yeah the AWS docs HTML is really inconsistent at times. In fact I'm amazed the scraper works as well as it does sometimes. I'll take a look though, hopefully this page is just a fluke and they haven't changed their docs page structure too much.

thanks, no rush really and not a showstopper by no means - i've added this just so it won't get lost


the compression and decompression logic

Just to confirm if my understanding is correct.  Currently, Mersenne31Dft supports interpolating a Mersenne31 poly into a Mersenne31Complex.
This ticket will further compress the Mersenne31Complex poly to a Mersenn31 Poly and two Mersenne31Complex parameter($\tau;c_0$).
So we will provide a new structure for storing the matrix and a vector of $\tau;c_0$ to replace current
RowMajorMatrix?

After double-checking the code and some experiments, I think the existing DFT (Mersenne31Dft) has already compressed the interpolation somehow, and https://eprint.iacr.org/2023/824 seems cannot do better in terms of space.
The current implementation folds the base field value to the complex field and gets a complex poly of degree n/2.
If we do the compression logic, I think we will have a base value poly of degree n  and two complex coefficients. The space taken seems the same. So, our goal is to implement the compression/decompression logic to see if it can be more efficient than the current one.
@dlubarov, please let me know do we just want another option to implement DFT.

I think the limitation with the existing code is that it gives us compressed output for a single [I]FFT, but doesn't give us a way to perform a full low-degree extension (chaining two FFTs together) with compressed output. https://eprint.iacr.org/2023/824 would enable that.

I am still thinking about the work of this ticket. I misunderstood it as a DFT process, but it is more like an LDE method that extends the Mersenne31 poly to a Complex field.
BTW, while I am checking the code of LDE, I find something seems wrong. In lde/native
        let g = Domain::two_adic_generator(bits);
        let subgroup = cyclic_subgroup_known_order::<Domain>(g, 1 << bits).collect::<Vec<_>>();
        let weights = barycentric_weights(&subgroup);

        let lde_bits = bits + added_bits;
        let lde_subgroup = cyclic_subgroup_known_order::<Domain>(g, 1 << lde_bits);

If g is a two_adic_generator of order 2^bits, I think lde_subgroup is just the original subgroup? shouldn't g be regenerated?

Good point, I made https://github.com/Plonky3/Plonky3/pull/170 with a fix

We've also been having second thoughts about this approach to Mersenne31 integration (doing a "MersenneFFT" with complex roots, compressing the output, etc.). @ulrich-haboeck and Shahar have been looking into a different approach for supporting Mersenne31, which is still being written up. It should lead to cheaper constraint evaluation, and might also be cleaner in some ways.

Closing this as we have a different plan for Mersenne integration; we can keep #56 to track progress with the new scheme.


Choose language on welcome page or switch it

Is there a way to switch the installer language???

The package uses the default language of your application.


[Bug] The paginator shows more than one page when the first page is not full

Is there an existing issue for this?

[X] I have searched the existing issues

What projects are you seeing the problem on?
OpenChallenges
Current behavior
The default page size is 20. When making a query that leads to less results than 20, the paginator shows page numbers besides 1. Click on page 2 for instance shows an empty page.

Expected behavior
In the situation described above, the paginator should only show page number 1.
Anything else?
No response
Commit ID
a60ca8b
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

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

@rrchai said that he can reproduce the issue.


Revisit processing variant without names in the repeat expansion pipeline

Follow up from #247.
The repeat expansion pipeline has two ways to determine whether a variant is a repeat expansion event:

By looking at the variant's name and trying to decode the HGVS-like notation
By looking at the difference of provided REF and ALT alleles

This means that variants without a name, in general, could and should still be processed by the repeat expansion pipeline. In particular, the variant cited in the original issue contains allele sequences, which means it should be treated as a repeat expansion variant:
<SequenceLocation Assembly="GRCh38" AssemblyAccessionVersion="GCF_000001405.38" AssemblyStatus="current" Chr="9" Accession="NC_000009.12" start="117724889" stop="117724890" display_start="117724889" display_stop="117724890" variantLength="12" positionVCF="117724889" referenceAlleleVCF="T" alternateAlleleVCF="TACACACACACAC" />

@tskir I've done some more digging on this, my sense is that we can use the REF and ALT, but I don't know a good algorithm to determine the repeating unit, which will make it hard to distinguish between trinucleotide_repeat_expansion and short_tandem_repeat_expansion.  (It's entirely possible this is included somewhere else and I just didn't see it, in which case... whoops, my bad.)  If there's a reliable way of doing so then it could be a good option to have available.
A viable possibility for the variants mentioned in the original ticket is to use HGVS identifiers that are in the measure but not provided as variant names, which can be parsed in the same way we do variant names.  There are generally multiple of these in a measure and I'm guessing we need to use a RefSeq one, but I'm not sure beyond that.
<AttributeSet>
  <Attribute Accession="LRG_320" Type="HGVS, genomic, LRG">LRG_320:g.25489CA[28]</Attribute>
</AttributeSet>
<AttributeSet>
  <Attribute Accession="NG_011475" Version="2" Change="g.25489CA[28]" Type="HGVS, genomic, RefSeqGene">NG_011475.2:g.25489CA[28]</Attribute>
</AttributeSet>
<AttributeSet>
  <Attribute Accession="NC_000009" Version="12" Change="g.117724891CA[28]" Type="HGVS, genomic, top level" integerValue="38">NC_000009.12:g.117724891CA[28]</Attribute>
</AttributeSet>
<AttributeSet>
  <Attribute Accession="NC_000009" Version="11" Change="g.120487169CA[28]" Type="HGVS, genomic, top level, previous" integerValue="37">NC_000009.11:g.120487169CA[28]</Attribute>
</AttributeSet>
<AttributeSet>
  <Attribute Type="HGVS, protein" />
</AttributeSet>

I'm writing up a PR with this second approach, but I'm curious what your thoughts are on the first approach, as well as which HGVS I should be using.

In case concrete examples are helpful, here are the seven in question in chr_pos_ref_alt form:
11_88291569_T_TGAAAGAAAGAAAGAAAGAAA
15_72340841_T_TCACACACACACA
15_72340841_T_TCACACACACACACACA
9_117724889_T_TACACACACACAC
7_101247300_A_AAAAAAAAATATAT
4_38792853_T_TTATATATATATATATATATATATATATATATA
20_31663045_T_TATATATATATATACACACACATATATATACATATATACATATATATACACACACATATACACACACACACACACACAC

Hi, @apriltuesday.
Regarding the mentioned issue:

Making use of REF and ALT alleles would probably doable. The issue is that inferring a repeated event involves an additional step that requires, most likely, an external tool that does so. I found resources like the Tandem Repeats Finder (TRF) that, for what I've seen, revolves around multiple sequence alignments (MSA) shifting the sequence to find 1-6 nucleotide k-mers. These tools, nevertheless, tend to work well taking into account the context (adjacent sequences). I have not used them with the alternative allele alone, but with a considerable sequence length, they could do us a favour.
Using HGSV identifiers seems a bit more straightforward and easier. Regarding what type of HGSV to use, I would first take a look at what identifiers are consistently given within all ClinVar XMLs (at least those without the Name node). As far as I have seen, most are standard and interchangeable (to uniquely identify a gene), so a priori it doesn't seem to be a big issue. If it serves of inspiration, NCBI only accepts sequence identifiers, CCDS identifiers (unique version and number for annotated genes), or LRG sequence identifiers (created specifically for clinical reporting by manual curation). We could make use of regular expressions to match short tandems (e.g. LRG_320:g.25489CA[10]). Nevertheless, if the tandems are slightly different or interrupted (e.g. LRG_320:g.25489CA[5]CG[2]CA[3]) we would end up again having to resort to MSAs.

Thanks @M-casado, this is useful input!  I agree that probably using REF and ALT properly is something best left as a last resort.
For parsing HGSV, I probably should've mentioned for context that we do have regexes to parse these already in the code, though I don't think it handles things like LRG_320:g.25489CA[5]CG[2]CA[3] flawlessly.  If the types are essentially interchangeable I'm inclined to just choose one arbitrarily, among the variants in question it looks like HGVS, genomic, top level is always present, and LRG and RefSeqGene occasionally present (see ClinVar nomenclature here).

I had a look at this paper describing an algorithm for SSR detection. It has the advantage of having a python implementation that we could leverage but unfortunately it seems to work from precomputed list of repeat which would not work in our case.


help text wrapping creates too long lines

If the help for a command is multiple lines long then the wrapping ends up creating lines that are a few characters too long. The issue is that the indent has to be subtracted from the width in https://github.com/alecthomas/kingpin/blob/5be72af01b3953d98c31938060ff8a3aa0c8f900/usage.go#L121

Can you provide a screenshot or some code that reproduces this? On what OS does this occur?


Possible Brick?

I closed my N3DS while on the base menu of the app, and when I opened it back up, it was frozen. I shut it off, and now it won't turn back on at all, except for the blue LED

It's pretty much impossible for FBI to have caused a brick that way.

Then what might be wrong with it?


Running wiremock mocks in multiple integration test overlaps responses.

Question
Hello,
I'm experiencing the following issue.
Each time I fire my integration tests with wiremocks running in gradle, I'm getting a number of different responses. The bad responses come from other stubs that are used in different tests.
My configuration as of now requires that I ran each test using mocks with a different port for Wiremock

@AutoConfigureWireMock(port = xxxx)

This required that the setup needs to be splited, and we are not able to have some default configuration.
My setup:
I ran each test with the following configuration:

@ExtendWith(SpringExtension.class)
@SpringBootTest(webEnvironment = WebEnvironment.MOCK)
@AutoConfigureWireMock(port = xxxx)

Setting port = 0, does not prevent this issue.

Which version of Spring Cloud Contract do you use?

IMO this is a duplicate of https://github.com/spring-cloud/spring-cloud-contract/issues/706

Wow, your quick :D,

compile "com.github.tomakehurst:wiremock:2.18.0"
compile "org.springframework.cloud:spring-cloud-starter-contract-stub-runner:1.2.4.RELEASE"
compile "org.springframework.cloud:spring-cloud-contract-wiremock:1.2.4.RELEASE"

I;m also running JUnit 5.

Yeah, that's a duplicate of #665. So you'll have to wait until we release  2.1.0 or use a different port or use @DirtiesContext.

DirtiesContex fix the problem. Thanks :)


Add Number.parseFloat to es2015 lib.d.ts

TypeScript Version:  2.2.2  and 2.3.1-insiders
Code
Number.parseFloat('1.2')

Expected behavior:
No error. This function was added in es2015: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/parseFloat
Actual behavior:
test.ts(1,8): error TS2339: Property 'parseFloat' does not exist on type 'NumberConstructor'.

Other methods to make sure exist:
Number.isFinite()
Number.isInteger()
Number.isNaN()
Number.isSafeInteger()
Number.parseFloat()
Number.parseInt()

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

It looks like parseFloat and co have been in es2015.core for two years... what am I missing?
https://github.com/Microsoft/TypeScript/blame/de313ff1bd11daf484aa421a5d9fde954b8e6fc7/src/lib/es2015.core.d.ts#L241

@mjbvz confirm?

Yes, looks like the originally reporting user and I were on the es3 lib


Bot doesnt support the new array.at method in JavaScript

Recently the array.at method was released in some ES version however the bot doesnt support that.

this is likely a node.js v17 feature. it'll be updated in due time.

Thanks for the reply

closing since resolved


[ECS] Extraneous fixer is unexpectedly added

Hi,
I came across weird issue. Whats worse, it only happens on Travis, and I was not able to reproduce it locally.
The configuration on Travis is:

PHP 7.2.6
ECS 4.3.0
no Travis cache is being used

When I use this easy-coding-standard.yaml file (which just imports easy-coding-standard.yaml from lmc-eu/php-coding-standard):
imports:
    - { resource: '%vendor_dir%/lmc/coding-standard/easy-coding-standard.yaml' }

it works as expected, and running ecs show now prints "52 checkers from PHP_CodeSniffer" and "98 checkers from PHP-CS-Fixer". But if I edit the easy-coding-standard.yaml like this:
imports:
    - { resource: '%vendor_dir%/lmc/coding-standard/easy-coding-standard.yaml' }

services:
    PhpCsFixer\Fixer\PhpUnit\PhpUnitTestAnnotationFixer:
        style: annotation

ie. just add PhpUnitTestAnnotationFixer, now ecs show prints "52 checkers from PHP_CodeSniffer" and "100 checkers from PHP-CS-Fixer" (+2!). Besides PhpUnitTestAnnotationFixer, I get for some reason also PhpUnitTestClassRequiresCoversFixer now being shown and applied by the check command, even though it is not part of the imported easy-coding-standard.yaml, neither have I added it.
Do you have any idea what could cause this?

Ok, it seems its too late...
This was caused by the "PR" build on Travis, which is run on "merged" branch. And the easy-coding-standard.yaml in master branch actually contains this extra fixer.
Sorry, this is my fault :rofl:

I've read only the last message. I'm glad it works, no troubles :+1:


Allow dates in natural-language formats

Instead of YYYY-MM-DD, users should be able to use phrases like "yesterday" or "two Tuesdays ago," as in git.
Likely the best way to incorporate this functionality: https://github.com/mojombo/chronic/

I can do this, should be a good task to start with!

Perfect, thanks so much! Let me know if you need any help!

@natejlong sorry if you were still working on this, but it had been a while and I wanted to get the feature out the door so I went ahead and implemented it. Let me know if you have any questions, and I hope you can contribute in the future!

Hey @JacobEvelyn , sorry about that; other work has been taking up my time. I will absolutely contribute in the future, I was actually planning my own similar system when I happened to come across yours. Thanks for your work!


[WIP] Qt 5.12.9

I'm trying to compile welle.io for raspberry pi and stumbled upon this great project. But I want to compile it with 5.12 LTS. I adjusted your code to work, but right now I'm only focusing on Raspbery Pi 3, only full init and disregarded backward compatibility right now, hence WIP.
My current status is:

Upgraded Crosscompiler to lenaro 7.5 (I have to upgrade the download script)
Upgraded raspi image to buster
Qt 5.12.9 builds successfully
Qt modules build successfully

Still working on it

Hello Christian,
Wow thanks a lot for this awesome contribution! I've skimmed through your changes and it looks very promising. Ping me again when you have stabilized your changes and I'll gladly review and test your PR.
We might have some back and forth about a few fixes but I definitely want to merge your work :+1: !

Hello Christian,
Wow thanks a lot for this awesome contribution! I've skimmed through your changes and it looks very promising. Ping me again when you have stabilized your changes and I'll gladly review and test your PR.
We might have some back and forth about a few fixes but I definitely want to merge your work :+1: !

Yes, it is very crude right now, as I just figured out how everything is working. Have to do some cleanup later. I managed to build my intended application welle.io and it is running on my raspberry so the build is definitely working.

Yes, it is very crude right now, as I just figured out how everything is working. Have to do some cleanup later. I managed to build my intended application welle.io and it is running on my raspberry so the build is definitely working.

Okay, i did some cleanup and removed the requirements for welle.io. I tested it with rpi3 and 5.12.9. I will try to compile the old QT versions tomorrow. But I think we can start to talk about the fixes you mentioned.
I'd like to hear your opinion about docker builds. Let's say I want to build welle.io (as an example). It would be awesome to use docker for this, but i can't use the provided -minimal sysroot because I have to add packages and QT modules. The way to do this would be to add modules to qt-modules.txt and adjust prepare-sysroot-full.sh to add packages. Then compile everything on my local machine and create a docker container with my local compiled resources, right? The current docker file is meant to be used in a different way? It retrieves the systroot by downloading it and the sysroot is initially created by a jenkins buildjob if i see this right. It would be nice to add some scripts to automatically create a custom docker image with my customizations to use it in a automatic build environment.
But maybe we should first try to merge the PR as it is and then make this docker stuff a PR on its own.

Okay, i did some cleanup and removed the requirements for welle.io. I tested it with rpi3 and 5.12.9. I will try to compile the old QT versions tomorrow. But I think we can start to talk about the fixes you mentioned.
I'd like to hear your opinion about docker builds. Let's say I want to build welle.io (as an example). It would be awesome to use docker for this, but i can't use the provided -minimal sysroot because I have to add packages and QT modules. The way to do this would be to add modules to qt-modules.txt and adjust prepare-sysroot-full.sh to add packages. Then compile everything on my local machine and create a docker container with my local compiled resources, right? The current docker file is meant to be used in a different way? It retrieves the systroot by downloading it and the sysroot is initially created by a jenkins buildjob if i see this right. It would be nice to add some scripts to automatically create a custom docker image with my customizations to use it in a automatic build environment.
But maybe we should first try to merge the PR as it is and then make this docker stuff a PR on its own.

Alright, I've reviewed the change set. If you could do some minor tweaks, it would be perfect:

Add the executable permission to docker/build.sh
Update the README.md with the commands to build the Docker image
Remove the -j flags on the make commands. It was my mistake, but it's actually useless, because omitting it will not limit the number of jobs

After that, I'll merge your PR and I'll take the time to clean up the repo to refer to the Docker build as the main one. Again, thanks for sharing your work :)

I will update the README tomorrow


Alpine county, California

Part of "finish California" sprint.

Another Parcel Quest county... via (src):
Maps & Property Information
We currently do not publish Assessor maps, Recorded Maps or property information
such as values or ownership on this website yet, but some information is available on
the Parcel Quest website. See the links below for online info or visit our office and we
will help you any way we can.

Office hours are Monday through Friday 8 a.m. to 12 p.m. and 1 p.m. to 5 p.m.
excluding holidays.

At the Assessor / Recorders Office public counter you can:
View and purchase Assessor Parcel Maps (APN Maps), Official Recorded Maps and
Survey Maps. Copies are available for a fee

For map & property information, come by or contact the Assessors office at:

99 Water St.
P.O. Box 155
Markleeville, CA 96120
Phone:<PHONE_NUMBER>

Data cost sheet (doesn't mention GIS data):

http://alpinecountyca.gov/DocumentCenter/View/504

Also (probably old info):
Zach Wood
Alpine Planning Department
17300 St Rt. 89
Markleeville, Ca 96120
530.694.2140 x 437
<EMAIL_ADDRESS>

$1300 - https://www.alpinecountyca.gov/DocumentCenter/View/52
Think it is too much - thoughts @ingalls ?
Thanks!

e-mailed<EMAIL_ADDRESS>to make sure that is current (*believe is dated 2005) The license seems restrictive too... http://alpinecountyca.gov/documentcenter/view/53
I'll see what we can get!

They will send the address points for free! Just got off the phone with Zach:
Zach Wood
Planner III
530 694 1371
<EMAIL_ADDRESS>We just have to explain why sharing it freely would benefit the general public/ or the county.

Awesome!

On Dec 10, 2018, at 11:03, Justin Meyers<EMAIL_ADDRESS>wrote:
They will send the address points for free! Just got off the phone with Zach:
Zach Wood
Planner III
530 694 1371
<EMAIL_ADDRESS>We just have to explain why sharing it freely would benefit the general public/ or the county.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

from previous source mentiod tonight: https://gis2.cdfa.ca.gov/server/rest/services/Parcel_DM/MapServer/1
**Will wait to get data from county - assuming it would be more current and correct - will compare as soon as they ship


Met error at initialization when get organization from Claude

When I trying to use it, get an error at initialization, seems Claude blocked program to access API directly, anybody can help on this?
Error at: getOrganizations
SyntaxError: Unexpected token '<', "
)
at parseJSONFromBytes (node:internal/deps/undici/undici:6662:19)
at successSteps (node:internal/deps/undici/undici:6636:27)
at node:internal/deps/undici/undici:1236:60
at node:internal/process/task_queues:140:7
at AsyncResource.runInAsyncScope (node:async_hooks:206:9)
at AsyncResource.runMicrotask (node:internal/process/task_queues:137:8)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

I tried by calling via Postman, it can work normally, while always get 403 when trying by Pythong progrem, even I exactly copied all header parameters from Psotman. I also tried request.session, it doesn't work, code is as below (I marked cookie as *).
import requests
import datetime
import uuid
import re
now = datetime.datetime.now()
twelve_hours_ago = now - datetime.timedelta(hours=12)
print(twelve_hours_ago.strftime('%a, %d %b %Y %H:%M:%S GMT'))
datatime=twelve_hours_ago.strftime('%a, %d %b %Y %H:%M:%S GMT')
uuid_obj = uuid.uuid4()
uuid_str = str(uuid_obj)
url = "https://claude.ai/api/organizations"
session = requests.session()
main_url = 'https://claude.ai/'
Getting the cookie
session.get(main_url)
print(session.cookies)
cookiejar_str = str(session.cookies)
pattern =r'(__cf_bm=.*) for'
match = re.search(pattern, cookiejar_str)
cf_bm_value = match.group(1)
print(cf_bm_value)
strcookie=cf_bm_value + '; sessionKey=sk-ant-*****************'
print(strcookie)
payload = {}
headers = {
'Date':datatime,
'Content-Type':'application/json',
'Connection':'keep-alive',
'Cookie': cf_bm_value,
'Host': 'claude.ai',
'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/114.0.1823.82',
'Accept': '/',
'Accept-Encoding': 'gzip, deflate, br',
'Upgrade-Insecure-Requests': '1',
'sec-ch-ua': '"Not.A/Brand";v="8", "Chromium";v="114", "Microsoft Edge";v="114"',
'sec-ch-ua-mobile': '?0',
'sec-ch-ua-platform': '"Windows"',
#'Postman-Token': 'uuid_str',
}
response = requests.request("GET", url, headers=headers, data=payload)
print(response)
print(response.text)

Still working for me, regardless, not sure what I could to to the API to make it work better so I'll close unless you have suggestions.

Yes, I finally identified it's an network issue, and now, it can be normally executed in Node (the program @Explosion-Scratch created can run normally now, thanks). while still cannot run correctly in Python program, I don't know why...


Incorrect path

The correct path to Acronyms is Settings>Microsoft Search>Answers, not Settings>Microsoft Search>Acronyms

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

ID: 8fd51764-80a7-b404-50bd-b492b193dab2
Version Independent ID: 62a6858b-81a3-bafa-4bcc-3bb2a951c654
Content: Manage Acronym answers in Microsoft Search
Content Source: MicrosoftSearch/manage-acronyms.md
Service: mssearch
GitHub Login: @jeffkizn
Microsoft Alias: jeffkizn

We are in the process of rolling out a new look to the UI which your are seeing. It is not yet available to all customers, world wide. We are timing the update to the documentation to coincide with the completion of the rollout as it will include deep links to specific sections that will only work on the new UI and did not want any customer to have broken links in the doc.
In the meantime, I hope you found the new UI discoverable so the discrepancy in documentation didn't matter and that you like the new look.

We have completed the rollout of the new UI and updated the documentation to reflect it.


log_link: fix the log_link test failure on adsp boards

The log_link tests are meant to work with 2 mock log backends.
But Intel ADSP platform has an extra default backend named
"log_backend_adsp" when CONFIG_LOG_BACKEND_ADSP=y, which will
mess things up.
Fix it by disabling the default backend.
Signed-off-by: Ming Shao<EMAIL_ADDRESS>

In prj.conf I tried to disable all potential default backends. Would that work if ADSP is added there?

In prj.conf I tried to disable all potential default backends. Would that work if ADSP is added there?

I tried adding CONFIG_LOG_BACKEND_ADSP=n to prj.conf. It can pass.
But I can't explain why the log can still be output even no backend is there.

OK, I think it's safe to simply put the CONFIG_LOG_BACKEND_ADSP=n in the prj.conf.
Because all the test logs are from the ztest fx. And ztest fx use printk().
https://github.com/zephyrproject-rtos/zephyr/blob/78809549ee4cf7de3150ff4c3887011825584547/subsys/testsuite/include/zephyr/tc_util.h#L25
And on ADSP platforms, the printk() is based on the winstream_console.
https://github.com/zephyrproject-rtos/zephyr/blob/78809549ee4cf7de3150ff4c3887011825584547/drivers/console/winstream_console.c#L41-L47
Which is stuffed into the printk implementation by overriding the weak symbol below:
https://github.com/zephyrproject-rtos/zephyr/blob/78809549ee4cf7de3150ff4c3887011825584547/lib/os/printk.c#L45-L54
I will re-push it.

@nordic-krch  re-pushed.


Use low-level Process method with SCAN commands

Motivation
I am currently working on a project that requires to use the HSCAN redis command on a cluster.
I would like to use the low-level Cmdable.Process(cmder) method.
With the current implementation, I cannot access the Scanner of a ScanCmd. I cannot do:
cmd := redis.NewScanCmd("HSCAN", "x")
cluster.Process(cmd) // assuming cluster is set...
scanner := redis.Scanner{ScanCmd: cmd)

// iterator.Next() eventualy calls the scanner.client.process method, which triggers a nil pointer panic: there is no way to set the client attribute of a scanner.
for scanner.Iterator().Next() {
    // it.Val()
}

This PR provides a NewScanner function, that takes a cmd and a processor.
// NewTestScanner creates a scanner whose internal process function is
// hijacked by the provided processor.
func NewScanner(cmd *ScanCmd, process func(cmd Cmder) error) Scanner {
	return Scanner{
		client: &cmdable{
			process: process,
		},
		ScanCmd: cmd,
	}
}

With this modification, we could achieve the previous goal by changing one line in the previous snippet:
cmd := redis.NewScanCmd("HSCAN", "x")
cluster.Process(cmd) // assuming cluster is set...
scanner := redis.NewScanner(cmd, cluster.Process)

// The scanner.client is defined, and the process attribute is the cluster.Process method. The panic
is avoided. Beside, the process method can be changed for a mock, easier for tests.
for scanner.Iterator().Next() {
    // it.Val()
}

Can you please explain why clsuster.HScan("x", 0, "", 0) does not work for you?

It does work. However, I would like to use the Process method.
Using the Process method for any kind of command would allow me to use a simple Processor interface.
Beside, I don't understand how to use the current NewScanCmd with the current API since I cannot access its scanner.

@moxar I think you are right, but I fixed this slightly different in https://github.com/go-redis/redis/pull/470. Thanks for creating PR anyway!

Thanks a lot!


Cleaning up Windows compilation.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Fixed a dumb issue - Jenkins is now churning on it at https://grpc-testing.appspot.com/view/Pull requests/job/gRPC_pull_requests/5618/

Green enough.


Test dynamic table shrinks correctly

Test that it correctly resizes the dynamic_entries to the largest size that will fit the new max_size, and sets the _current_size accordingly
Some b'TestValue' and b'TestName' values were switched

This might have actually uncovered a bug…
Per test_size, the length of the three headers added to dynamic_entries gives a _current_size of 147.
Unless I misunderstand, .pop() should remove the entry from the stack, in _shrink, and so, setting the max_size to 146 should remove one entry from dynamic_entries, giving a len() of 2.
The test fails, because dynamic_entries still has a len() of 3: https://travis-ci.org/python-hyper/hpack/jobs/166339022#L243

@dshafik I'll take a look at this when I get off work. cc @Lukasa

@dshafik Confirmed that this is a bug, a tables max_size should never be less than it's current size.

@dshafik I can't reproduce the bug you mention, the test case you proposed though is a good one and should definitely be brought in as a test case.

@dshafik @Lukasa I've found the problem, it is in your code you use the attribute max_size where the attribute is actually maxsize. Change that and you should see the table shrink properly. I still agree with adding this test for shrinking though.

Thanks @dshafik! :sparkles: Do you mind updating to the current master so I can merge? (In future, you may find it more convenient to either grant maintainers access to your fork or to push your branch directly to this repo so that other contributors can do that update for you. =D )

@Lukasa I made the necessary change to be merged. Test is now passing.

@SethMichaelLarson Did you mean that comment to go on this issue?

@Lukasa Yeah, I made the change to @dshafik's test to make it pass. The commit shows up in the PR. So this should be ready for merge.

@SethMichaelLarson Oh, I see. The issue isn't that the change isn't there, it's that this change needs to have the current master branch merged into it.

@Lukasa Ah, my apologies for the misunderstanding. I have write access to @dshafik's repository so I'll rebase and get this PR ready to merge then.

@Lukasa Ah, apologies for the misunderstanding.

@SethMichaelLarson No need to apologise! =D

@Lukasa Should be rebased onto master now…

LGTM

Thanks @dshafik and @SethMichaelLarson! :sparkles:


move remote authentication snippets into project file

Closes #28548
Side note: I hope it's OK that I'm picking up these issues 😅

Internal previews

📄 File
🔗 Preview link

aspnetcore/migration/inc/remote-authentication.md
Remote Authentication

@timdeschryver  thanks for the PR, much improved. Changes should be live in10 minutes.


Demo for video processing

I want to create a video classification demo where I record a fixed frame length video (in browser), and then give feed it into an onnx model. Could you tell me how to do so? as in the problem boils down to taking the video input into Float32Arrays

@euler16 Could you help me out with this?  I'm trying to swap out the YOLO demo's model with my own custom-trained YOLOv5  model, having a hard time with it.


Set up readthedocs

I've created haskell-graphql-api.readthedocs.org.
Still to do:

[ ] Give @teh maintainer access (teh needs a readthedocs username)
[ ] Add badge & link to README.md
[ ] Confirm that webhook actually works (NB: I didn't go for the automated process because I'm not comfortable giving other services admin access)

Signup up to RTD as "teh" and donated a small amount to them.


Please invite me to the GitHub Community Organization

Name
PaladinKnightMaster
Discord Username (if applicable)
Cody_Christ#6150

Welcome to the community PaladinKnightMaster  :)


NavController.replace() doesn't set the view controller on new view

When using NavController.replace() to replace the current view with a
new view the new view's navController isn't set. This fixes that, by
leveraging the NavController's push method, instead of going directly
to the viewStack, it also moves the isLastViewPopped logic into the
updatePublicFields method.

great first PR, thanks for fixing this.


feat(deps): update webonyx/graphql-php into latest version

BREAKING CHANGE: The webonyx/graphql-php from now on requires PHP version 7.4 and higher. So do we from now on.

:tada: This PR is included in version 2.0.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


typescript vite config file error

when I try to put this command:
import svgrPlugin from 'vite-plugin-svgr'

in the vite.config.ts file, I get this error from the compiler:
Module '".../node_modules/vite-plugin-svgr/lib/index"' can only be default-imported using the 'allowSyntheticDefaultImports' flag
[index.d.ts(9, 1): ]()This module is declared with using 'export =', and can only be used with a default import when using the 'allowSyntheticDefaultImports' flag.

although the error is not preventing the plugin to run but would still like to know how to get this resolved, thanks.

You can solve it by：
tsconfig.json or  tsconfig.node.json
{
  "compilerOptions": {
    "allowSyntheticDefaultImports": true
  },
  "include": ["vite.config.ts"]
}

You can solve it by：
tsconfig.json or tsconfig.node.json
{
  "compilerOptions": {
    "allowSyntheticDefaultImports": true
  },
  "include": ["vite.config.ts"]
}

I put that into the tsconfig.node.json file and I no longer see the error. I first thought this was unnecessary because it was already in the tsconfig.json file, thanks.


Make CanvasItem's "drawing outside of NOTIFICATION_DRAW" error a macro

The premise is simple:

Imagine this but 25 times.
I also took the chance to improve upon the error message itself. At least, attempted to.
Please give suggestions for a better macro name, I do not know the pattern.

I do not have the capacity to test any of this so I just stripped the DEBUG_ENABLED

I also took the chance to improve upon the error message itself. At least, attempted to.

This should resolve #75376. But maybe the error message should also explicitly state what this and CanvasItem::current_item_drawn are when the error happens? Directly pointing out the problematic nodes should make debugging / understanding what's the problem much easier.

I do agree and I tried to do that. Problem is I just couldn't find a nice "spot" to explicitly state both.

CanvasItem::current_item_drawn will always be nullptr when drawing is false though

Yeah, that one cannot be done. But the node name can.

Risks causing unhelpful editor paths indeed, like parenting errors at times root/@EditorNode@17034/@Panel@13/@VBoxContainer@14/@HSplitContainer@17/@HSplitContainer@25/@VSplitContainer@27/@TabContainer@29/Scene/@P

I thought about it and I cannot fit the name anywhere. I wish there was a standard way to mention it.
vformat("Node (%s) cannot draw outside of its drawing routine. Drawing is only allowed inside this node's `_draw()`, functions connected to its `draw` signal, or when it receives NOTIFICATION_DRAW.", )
But this is bad already.

Of course that doesn't mean errors shouldn't be more explicit, and the Errors tab should be MUCH IMPROVED for this, but for this message any longer and it's just plain irritating.
This behavior is already documented but perhaps it needs to be even more explicit, as what the "drawing routine" is isn't exactly clear on the surface.

Applied both suggestions. Note that the ERR_THREAD_GUARDs are not formatted the same way.

CanvasItem::current_item_drawn will always be nullptr when drawing is false though

Not really, you can call draw_x methods on a different CanvasItem than current_item_drawn and it will result in such errors, as such other CanvasItem is not drawing at that point (drawing is non-static contrary to current_item_drawn).
That's exactly when the current error message (pre this PR) would confuse people when they try to do something like:
extends Node2D
func _draw():
	# When this is called `current_item_drawn` is `self`.
	some_other_canvas_item.draw_circle(...) # Any `draw_x` method called on non-self canvas item.

	# Error: "Drawing is only allowed inside NOTIFICATION_DRAW, _draw() function or 'draw' signal."
	# User: What do you mean?! It is in `_draw()`...

The error being changed to:
"Drawing is only allowed inside this node's `_draw()`, functions connected to its `draw` signal, or when it receives NOTIFICATION_DRAW."

already makes the error message correct/clearier. :+1:

Regarding further improvement - yeah, I agree it's out of scope of this PR. I'm also not sure what would be a good way to refer to the problematic nodes. Path seems too long, just names too vague, Node::to_string which includes ObjectID not "friendly enough" and still it doesn't provide info the path does, etc.

BTW it could be also considered to have slightly different error messages depending on the case, like:
ERR_FAIL_COND_MSG(current_item_drawn == nulltpr, "..."); // Not in drawing routine at all.
ERR_FAIL_COND_MSG(current_item_drawn != this, "..."); // In drawing routine of a different CanvasItem.

(drawing bool would be no longer needed if the error conditions would be changed like that.)
Wondering if something like this could make things clearer even if not directly specifying what canvas item(s) this happened for. But also not sure how doubling the checks would affect performance.
Just a thought, not saying it should be done in here. This PR is fine as is.

Speaking of the much prior topic, if only checking for drawing in debug builds has a good chance of breaking things in release builds. I have to question how bad things can break when the raw RenderingServer methods try to draw outside of the routine, as well? If that's even possible?

Thanks!


WIP: RHOARDOC-1614-fix: File name and cleanup

@tradej I need to do a final check of the links and cross references and will then remove the WIP

Can one of the admins verify this patch?

ok to test

@huntley, sure.
On a side note, there seem to be quite a lot of conflics, can you please rebase the branch, resolve the conflicts and then force push once you're done? It doesn't make much sense to review the PR before that is done. Thanks.

@tradej Added parameter for the examples catalog to document-attributes. Would you prefer document-attributes-launcher.

@huntley Thanks for resolving the conflicts and for creating the attribute.
@inoxx03, can you please review the PR?


Updating list_sinks() to Iterator pattern.

NOTE: Has #2651 as diffbase. Unit test changes will be forthcoming.

@tseaver @daspecster PTAL this is ready to go

This LGTM as well.


Only OpenAI's API is supported?

Hi, could you share more details about your goal and which model provider, aside from OpenAI, you're looking to use?

Hi, could you share more details about your goal and which model provider, aside from OpenAI, you're looking to use?

I'm using the Qwen Plus model. My goal is to convert fragmented JSON into the standard JSON format. I wonder if there is a Transformer that can be used to do this conversion. I just need to input the non-standard JSON strings.

Thank you so much for your efforts. However, I have switched to another library which has been of great help to me in solving my problem.The lib name is wtf-json

Hi, could you share more details about your goal and which model provider, aside from OpenAI, you're looking to use?

I'm using the Qwen Plus model. My goal is to convert fragmented JSON into the standard JSON format. I wonder if there is a Transformer that can be used to do this conversion. I just need to input the non-standard JSON strings.

Indeed, the handler is designed to handle a specific data stream format provided by OpenAI. Before passing the data downstream, it will need to be updated. For now, this can be accomplished by understanding the internal structure of the handler and formatting it accordingly. The JSON closer is a generic component that can be reused. But, it will require some customization for your specific use case.

Thank you so much for your efforts. However, I have switched to another library which has been of great help to me in solving my problem.The lib name is wtf-json

Nice, thanks for sharing. Glad it solves your use case.


PmagPy in Python3

Hey all PmagPy developers, to avoid the sun setting of the scientific python libraries on Python2 and to future proof PmagPy from the problems this will bring @Swanson-Hysell, @moonshoes87, and I have developed a fork of PmagPy master which we converted to Python3. Making this change will grantee that PmagPy keeps the support it needs from the numerous scientific libraries it relies on as well as by the newer GUI library wxpython Phoenix which while a bit more unstable than the old wxpython is better built and under current development.
That said this will require that all developers obtain a Python3 environment either using Anaconda, Enthought, or just Python. It should be noted that Canopy does not yet fully support Python3 so those using Mac or Windows are recommended to try Anaconda. Once you've set up your Python3 environment you will need wxpython Phoenix just like in the old PmagPy you can pip install this as follows:
On Mac or Linux:
 sudo -H pip install --upgrade --pre -f https://wxpython.org/Phoenix/snapshot-builds/ wxPython_Phoenix

On Windows (don't forget to replace PATH_TO_PIP with the actual path):
C:\PATH_TO_PIP\pip.exe install --upgrade --pre -f https://wxpython.org/Phoenix/snapshot-builds/ wxPython_Phoenix

Once you've done that and if necessary added PmagPy to your path and pythonpath just like in Python2, or run the new dev_setup.py script you should be able to run the GUIs and command line programs as usual. Note: As of this moment the command line programs should be cross compatible with both Python2 and 3, but all developers will need to continue development in Python3 or there will be serious errors and the GUIs are strictly Python3 only because of the difference between the GUI libraries.
One of the other large things to address is that this change does require that all developers change their code to Python3 code something that should not be overly daunting as there are only a few changes between the two languages. These are documented in the (mostly complete) notebook Py2toPy3.ipynb in data_files/notebooks/ and can also be found all over the internet like here.
So this merge will likely have many conflicts as a lot of development has happened both on this fork and in the PmagPy main repo, so master will be unstable or some time. For those that wish to continue to use the Python2 version of PmagPy while master is in flux there should now be a Python2 branch off PmagPy master. This should cause no change to the binary releases of the GUIs as once frozen it doesn't matter what language we are using, and @moonshoes87 has come up with a clever solution to having pip know which PmagPy to download when you run pip install pmagpy to minimize impact on those that are just users.
Let me know what you all think and what happens when you all give it a whorl.

@Caoimhinmg, thanks for all the work you've done!  I've pulled all those changes into a local PmagPy branch, so here are the steps you need to follow to try the new code.

Install Anaconda.  You can create an environment with Python 3 or have Python 3 as your default for Anaconda.

git checkout Caoimhinmg-master

If you already have PmagPy in your path, you should be ready to go.  Try it out!

Hello Kevin and Lori,
So I was thinking of removing all of my PmagPy Python 2.7 stuff on my mac
(Enthought Python and my current PmagPy git directory) and starting over my
installation from scratch using Anaconda and ending up with the current
master branch installed which is Python 3. Going forward I will use that
branch and make my changes there. Do you see any pitfalls in this approach?
The future standalones will be built with the master branch, not the
Python2 branch, correct? I remember reading about a release
branch(different from master?), but don't see that branch on the PmagPy
github.
Nick
On Mon, May 1, 2017 at 12:35 PM, Kevin Gaastra<EMAIL_ADDRESS>wrote:

Merged #175 https://github.com/PmagPy/PmagPy/pull/175.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/PmagPy/PmagPy/pull/175#event-1064410741, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABODRdyV9Ve8MovMovkGyOsuxse_cRB6ks5r1jQegaJpZM4NJg2i
.

Hi Nick,
Your plan sounds like a good one, although you should not need to remove your git PmagPy directory.  You don't even need to uninstall Enthought Python, you just need to install Anaconda Python and give it priority (by commenting out Enthought Python from your .bashrc or .bash_profile).
With Anaconda, you will be able to switch easily between Python 2 and Python 3 if needed.  Likewise, with the developer install, you can switch easily between the Python 2 PmagPy branch and master.  So you will be able to access both pretty simply.
Having PmagPy in your PATH via the developer install means that your system will access whichever version of a program your git branch is pointing to.  So, to switch between using PmagPy with Python 2 and Python 3, you will need to do just two things.
Let's say you have Anaconda installed with python 3 as your default Python, and that you have the developer install already.  If you need to switch back to using Python 2.7 and the Python 2 compatible PmagPy, you can follow these steps:
# create a Python 2 conda environment (one time only)
conda create python2_env python=python2.7
# switch to using that python 2 environment
source activate python2_env
# install any needed packages into this environment (one time only)
# for example:
conda install numpy
pip install --upgrade --pre -f https://wxpython.org/Phoenix/snapshot-builds/ wxPython_Phoenix
# switch to the Python2 branch (must be in your PmagPy git directory)
git checkout Python2

To go back to your defaults of Python 3 and PmagPy for Python 3:
source deactivate
git checkout master

As far as the standalones are concerned, we just created up-to-date versions for all platforms from the Python 2 branch.  Once master is stable again, you are correct, we will start making new releases off of master.
And as far as the release branch is concerned, that is just a temporary branch that I create when it is time to do a new release.  What this does is freeze development while I (or whomever else) does some testing on that branch to make sure everything is ok, while still allowing other developers to continue making changes to master.  After making a new release, the release branch gets deleted.
Let me know if you have any additional questions!  Hopefully this should be pretty straightforward.


missing license header and updates to pkg/*

Changes

Added missing license header in files under pkg/*
Updated vendor/*

:broom: Update or clean up current behavior

/kind cleanup
/assign @cdlliuy

/ok-to-test

@cdlliuy you need to /approve and /lgtm this to merge it. Thanks.

/lgtm

/approve


Docs.MS Release Updates for Azure.Storage.Blobs.Batch

Update docs metadata and targeting for release of Azure.Storage.Blobs.Batch

Docs Build status updates of commit c652131:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/storage.blobs.batch-readme-pre.md
:white_check_mark:Succeeded
View

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Fix strawberry docs schema#handling-execution-errors

Description
Fix strawberry docs schema#handling-execution-errors
Implementation of Logger has been changed, but it has not been updated in the document.
That's why I updated the document.
You can see docs in this link
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[x] Documentation

Issues Fixed or Closed by This PR

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊
You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67
And don't forget to join our discord server: https://strawberry.rocks/discord 🔥


Added first version of assisted_installer

This docker image will be used for preforming the installation of the cluster nodes.
The bm-inventory will send the each node the install command  with the relevant role (master, worker, bootstrap)

the node with the bootstrap role will pivot to master once we have 2 ready master nodes.

We need to understand if we should change it to python
On one hand, in python it will be much more readable, but on the other hand, the bootstrap is also bash.
Considering that will need to pass it sometime in the review process of OpenShift, maybe they prefer bash?, Maybe it has less dependencies?
What do you think?
I think it should be python/go
Also lets think how to test this?
Do we want/need UTs?
Definitely need UT and system test


Old dependencies

https://github.com/DataWhisker/x-mcp-server/blob/eec04e7880d04c717749717c087477d01d42047a/package.json#L19C6-L19C31
The SDK that you've used is very old; The latest version is 1.0.4

upgraded MCP SDK to version 1.0.4


Bug fixes and aliases

It should be self.pull(args).
Also added 'ls' alias for 'list'.

The ls alias is because ollama has the "rollama ls" alias for "ollama list", it's also kinda ingrained into a lot of people's muscle memory to type ls.

LGTM


Support for EMR 6.7.0

Hi, EMR 6.7.0 latest should be supported.
https://docs.aws.amazon.com/emr/latest/EMR-on-EKS-DevelopmentGuide/emr-eks-6.7.0.html

Hi @zhuyulin27,
I created a PR to support 6.7.0 release. Could you help review the PR?
Thanks!

Any updates? What about later releases?


Update xynafactory pom.xml

version needs to be updated to <IP_ADDRESS>

merged into 9.0.3.x: 66783d04986a4bb41b9af09616c4da661a970141


Cannot sign in to Jira

Expected Behavior
Clicking the Continue button upon first sign-in, has any effect.
Actual Behavior
No effect whatsoever. No entry in the Auth debug console
Steps to Reproduce the Problem

enter jira host
enter user name and password
click Continue

Specifications

Chronos-timetracker version: 2.7.3
OS: Win10

@Treehopper Could you try the last version - 2.8.1?

Actually it worked after I restarted the application. When I check the version now it displays 2.8.1. I assume it updated without me noticing.
Sorry, Google gave me a link (I don't remember) that made it seem liked 2.7.3 is the latest version.
I think this can be closed.

@Treehopper Thanks, feel free to create a new issue with a feature request or bug report.


Reformat with ruff, and fix linting warnings

Ran ruff format and ruff check --preview, and fixed most of the warnings.
I used ruff check --preview --unsafe-fixes --fix where possible.
(This is comparable to black, isort, pyupgrade, flake8, and various other linters.)
This PR is pretty huge due to the auto formatting and switching from single quotes (') to double ("). The individual commits show the steps a little bit at a time.

I think I'm pretty happy with this one now... It still works great on both macOS and Linux, and I've silenced all the warnings from ruff, flake8, pylint, etc.
The only remaining warnings are in the get_processes() function, for which I thought we might use https://github.com/giampaolo/psutil instead. I'll make a separate PR for that.


Prometheus now publishes number of clients connected since server started

#206
Total ip/ports that have connected since server started are published in prometheus now for each service.

This change is

Seems as though similar PRs did not have a test suite would love guidance on what sufficient testing looks like for these Prometheus related tasks.
Counter from Prometheus had a conflict with the custom metrics_utils Counter so had to rename. LSP wasn't showing that Counter was unused can remove the prometheus counter and revert back to the old naming to clean this up.

Made all of the above changes except not fully understanding why the getter is not needed.

Also added publishing timestamp for server starting, can move to a separate PR if preferred.


runners: add fedora38

It's that time of the year again.

We should really do #996 instead.

This is blocking Fedora Ci for the latest osbuild release.


Pass --fail-rerun option on the main branch

Treat regrtest exit code 5 (EXITCODE_RERUN_FAIL) as WARNINGS, rather than FAILURE.
Rename ONLY_MASTER_BRANCH to ONLY_MAIN_BRANCH.
Add MAIN_BRANCH_NAME and JUNIT_FILENAME constants.
Always pass the branch to Test() steps.
Test steps use lists for test options, rather than strings.
Remove unused SlowSharedUnixBuild and FreezeBuild factories.

I'm waiting for https://github.com/python/cpython/pull/108896 to be merged.
cc @zware

I restarted the server manually:
$ ps aux|grep buildbot.tac
buildbot 2533539 50.9  5.8 1285740 119816 ?      Sl   08:45   2:03 /srv/buildbot/venv/bin/python3.9 -c from twisted.scripts import twistd; twistd.run() --no_save --logfile=twistd.log --python=buildbot.tac

I'm now curious if the cron job will kill this process to spawn a new one.

I'm now curious if the cron job will kill this process to spawn a new one.

15 min later, it didn't:
$ ps aux|grep buildbot.tac
buildbot 2533539 13.4 15.5 1452288 316140 ?      Sl   08:45   3:53 /srv/buildbot/venv/bin/python3.9 -c from twisted.scripts import twistd; twistd.run() --no_save --logfile=twistd.log --python=buildbot.tac

I scheduled a job on FreeBSD on the main branch and it worked successfully! https://buildbot.python.org/all/#/builders/1223/builds/128
make buildbottest 'TESTOPTS=-j2 --junit-xml test-results.xml -j4 --fail-rerun ${BUILDBOT_TESTOPTS}' TESTPYTHONOPTS= TESTTIMEOUT=900
 in dir /buildbot/buildarea/3.x.ware-freebsd/build (timeout 1200 secs) (maxTime 14400 secs)
 watching logfiles {}
 argv: [b'make', b'buildbottest', b'TESTOPTS=-j2 --junit-xml test-results.xml -j4 --fail-rerun ${BUILDBOT_TESTOPTS}', b'TESTPYTHONOPTS=', b'TESTTIMEOUT=900']
(...)
./python -E  ./Tools/scripts/run_tests.py -j 1 -u all -W --slowest --fail-env-changed --timeout=900 -j2 --junit-xml test-results.xml -j4 --fail-rerun
(...)
1 re-run test:
    test_tarfile

Total duration: 5 min 27 sec
Total tests: run=40,945 skipped=1,320
Total test files: run=461/463 failed=1 skipped=15 resource_denied=3 rerun=1
Result: FAILURE then FAILURE

test_tarfile failure is known and unrelated

I scheduled a Windows job: https://buildbot.python.org/all/#/builders/146/builds/6354
The --fail-rerun option is passed as expected!
Tools\buildbot\test.bat -p x64 -j4 --junit-xml test-results.xml --fail-rerun --timeout 900
 in dir D:\buildarea\3.x.bolen-windows10\build (timeout 1200 secs) (maxTime 14400 secs)
 watching logfiles {}
 argv: [b'Tools\\buildbot\\test.bat', b'-p', b'x64', b'-j4', b'--junit-xml', b'test-results.xml', b'--fail-rerun', b'--timeout', b'900']
(...)
D:\buildarea\3.x.bolen-windows10\build>"D:\buildarea\3.x.bolen-windows10\build\PCbuild\amd64\python_d.exe"  -u -Wd -E -bb -m test  -uall -rwW --slowest --timeout 1200 --fail-env-changed -j1 -j4 --junit-xml test-results.xml --fail-rerun --timeout 900

My change works as expected: Result: FAILURE then SUCCESS, the build is marked as ORANGE 22:37 warnings test (warnings) WARNINGS.
1 re-run test:
    test_threading

Total duration: 20 min 31 sec
Total tests: run=40,689 skipped=2,242
Total test files: run=463/463 skipped=37 resource_denied=1 rerun=1
Result: FAILURE then SUCCESS
D:\buildarea\3.x.bolen-windows10\build>exit /b 5
program finished with exit code 5

The orange color is a hint that something went wrong. Here is problem is that test_default_timeout was re-run:
Test the barrier's default timeout ... Warning -- Unraisable exception
Exception ignored in thread started by: <function Bunch.__init__.<locals>.task at 0x00000290E578B7D0>
Traceback (most recent call last):
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 48, in task
Warning -- Unraisable exception
Exception ignored in thread started by: <function Bunch.__init__.<locals>.task at 0x00000290E578B7D0>
Traceback (most recent call last):
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 48, in task
Warning -- Unraisable exception
Exception ignored in thread started by: <function Bunch.__init__.<locals>.task at 0x00000290E578B7D0>
Traceback (most recent call last):
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 48, in task
    f()
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 1020, in f
    f()
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 1020, in f
    f()
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\lock_tests.py", line 1020, in f
    i = barrier.wait()
        ^^^^^^^^^^^^^^
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 709, in wait
    i = barrier.wait()
        ^^^^^^^^^^^^^^
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 709, in wait
    i = barrier.wait()
        ^^^^^^^^^^^^^^
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 709, in wait
    self._wait(timeout)
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 749, in _wait
    self._wait(timeout)
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 749, in _wait
    self._wait(timeout)
  File "D:\buildarea\3.x.bolen-windows10\build\Lib\threading.py", line 749, in _wait
    raise BrokenBarrierError
threading.BrokenBarrierError:
    raise BrokenBarrierError
threading.BrokenBarrierError:
    raise BrokenBarrierError
threading.BrokenBarrierError:
ERROR

This issue is tracked as: https://github.com/python/cpython/issues/108987

For comparison, I scheduled a FreeBSD job on the 3.11 branch: https://buildbot.python.org/all/#/builders/1218/builds/35
On the 3.11 branch, tests are not run with --fail-rerun as expected.
make buildbottest 'TESTOPTS=-j2 --junit-xml test-results.xml -j4 ${BUILDBOT_TESTOPTS}' TESTPYTHONOPTS= TESTTIMEOUT=900
 in dir /buildbot/buildarea/3.11.ware-freebsd/build (timeout 1200 secs) (maxTime 14400 secs)
 watching logfiles {}
(...)
./python -E  ./Tools/scripts/run_tests.py -j 1 -u all -W --slowest --fail-env-changed --timeout=900 -j2 --junit-xml test-results.xml -j4
== CPython 3.11.5+ (heads/3.11:f1f85a42ea, Sep 14 2023, 04:56:23) [Clang 14.0.5 (https://github.com/llvm/llvm-project.git llvmorg-14.0.5-0-gc12386
== FreeBSD-13.2-RELEASE-amd64-64bit-ELF little-endian
== Python build: debug
== cwd: /buildbot/buildarea/3.11.ware-freebsd/build/build/test_python_60282æ
== CPU count: 4
== encodings: locale=UTF-8, FS=utf-8
Using random seed 879875
0:00:00 load avg: 1.30 Run tests in parallel using 4 child processes (timeout: 15 min, worker timeout: 20 min)


sqlalchemy.text() used for qmark queries in SQLite

Hello,
I am using sqlc v1.27.0, sqlc-gen-python 1.2.0 and sqlalchemy 1.4.54.
Example query:
-- name: GetUser :one
SELECT * FROM users
WHERE id = ?

Generated code:
GET_USER= """-- name: get_user \\:one
SELECT id FROM users
WHERE id = ?
"""

class Querier:
    def __init__(self, conn: sqlalchemy.engine.Connection):
        self._conn = conn

    def get_user(self, *, id: Any) -> Optional[models.User]:
        row = self._conn.execute(sqlalchemy.text(GET_USER), {"p1": id}).first()
        if row is None:
            return None
        return models.User(
            id=row[0],
        )

This code does not execute. Running
users.get_user(id=1)

gives error
ProgrammingError: (sqlite3.ProgrammingError) Incorrect number of bindings supplied. The current statement uses 1, and there are 0 supplied.

This happens because sqlalchemy.text() expects bound parameters to use where id = :id style. By using qmark style bound parameters (where id = ?), this query is not recognized to be a prepared statement and is probably run as-is, and SQLalchemy does not send parameter values with it. Reference: https://docs.sqlalchemy.org/en/14/core/tutorial.html#using-textual-sql
I cannot find a workaround for this.  If you want to use sqlalchemy.text(), you have to use :parameter syntax.

I ran into the same issue. Looks like the problem is known for quite some time #3. According to the language support page of sqlc, SQLite is not supported.
Would it be possible to throw an error when using the engine: sqlite? I think this would be preferable to having code generated that does not work.

Sure, throwing an error would be better than nothing, but fixing it would be even better of course. I think the fix should be fairly simple, since the generated Python code has a kwarg named after the SQL field id:
def get_user(self, *, id: Any)

So the rendering code should have knowledge somehow of this identifier, and can use it to correctly render named placeholders instead of question marks. If someone can point me towards a piece of code where it uses this knowledge to build the function header, I can try to prepare a fix.

For anyone running into the same issue: I found a workaround.
Change your prepared statements to use @placeholder syntax:
-- name: GetUser :one
SELECT * FROM users
WHERE id = @id

and set the engine to postgresql instead of sqlite. The generated code will still work for SQLite because of SQLAlchemy.


macchina: 1.1.3 -> 1.1.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Macchina-CLI/macchina/releases.
meta.description for macchina is: "A fast, minimal and customizable system information fetcher"
meta.homepage for macchina is: "https://github.com/Macchina-CLI/macchina"
meta.changelog for macchina is: "https://github.com/Macchina-CLI/macchina/releases/tag/v1.1.4"
Updates performed

Rust 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

/nix/store/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4/bin/macchina passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 1.1.4 with grep in /nix/store/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4

found 1.1.4 in filename of file in /nix/store/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4

directory tree listing: https://gist.github.com/9a4ea9a5040dd4234c55b5b97e25cd1e

du listing: https://gist.github.com/07ec6ec5586a16cd5fcf7e77bc980036

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
macchina

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4 \
  --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 macchina https://github.com/r-ryantm/nixpkgs/archive/93d5531b29a9464033b5742dc988dfa4a9413699.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/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4
ls -la /nix/store/1rla6g6idzz5mmbrixqwprjd9cxbqfv6-macchina-1.1.4/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:

macchina

Maintainer pings
cc @414owen for testing.

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

1 package built:

macchina


Add IndyChecksumAdvisor to allow add-ons to determine when content checksums are calculated

The old ChecksummingTransferDecorator implementation from galley didn't provide much flexibility
in terms of turning checksumming on/off, and its configuration was a mass of confusion. We don't
need to calculate checksums, except in two cases:

when we generate a file, such as maven-metadata.xml
when we store or download a file via Folo tracking

This change takes advantage of a refactored ChecksummingTransferDecorator, which introduces a
ChecksummingDecoratorAdvisor interface. Two of these are passed into the decorator's constructor,
one for reads and another for writes. Indy extends this concept with IndyChecksumAdvisor. Where
ChecksummingDecoratorAdvisor returns a ChecksumAdvice value, IndyChecksumAdvisor returns one of
these values wrapped in Optional. This allows the Indy advisor to abstain on making a decision
for content it doesn't care about, while asserting certain advice for other content. Additionally,
the Indy interface it intended to be injectable and additive. The most onerous advice available
for any piece of content is selected (where most onerous is calculating and writing the checksum
file).
Indy collects these advisors into two lambda implementations of ChecksummingDecoratorAdvisor,
which implement the selection of the most onerous advice. If no advisors are available or contribute
to the decision-making for a file, then checksums are only written when a generated file is written.
Folo provides the first implementation of IndyChecksumAdvisor, in order to furnish the checksums
for the tracking records it maintains.

retest this please


nyc fails to parse ES6 ava.config.js file

Link to bug demonstration repository.
See the Travis build failure in honestbleeps/Reddit-Enhancement-Suite#4841.
ava.config.js contains the following:
/* @flow */

export default {
	files: [
		'**/__tests__/*.js',
	],
	require: [
		'@babel/register',
	],
};

Expected Behavior
cross-env NODE_ENV=test nyc ava should succeed.
Observed Behavior
nyc reports an error with parsing export default:
$ cross-env NODE_ENV=test nyc ava
✖ Error loading ava.config.js
/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/ava/lib/load-config.js:17
ReferenceError: default is not defined
    at loadConfig (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/ava/lib/load-config.js:17:5)
    at Object.exports.run (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/ava/lib/cli.js:24:10)
    at Object.<anonymous> (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/ava/cli.js:10:23)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Module.replacementCompile (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/nyc/node_modules/append-transform/index.js:58:13)
    at Module._compile (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/pirates/lib/index.js:83:24)
    at Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at /home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/nyc/node_modules/append-transform/index.js:62:4
    at newLoader (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/pirates/lib/index.js:88:7)
    at Object.<anonymous> (/home/travis/build/honestbleeps/Reddit-Enhancement-Suite/node_modules/nyc/node_modules/append-transform/index.js:62:4)

Forensic Information
Travis, node.js image. Related: #916.

I was able to reproduce a conflict between the way ava uses babel and @babel/register being loaded from outside ava.  I've opened a ticket at avajs/ava#2089 to see if they have ideas for how to make this less of an issue.
Good news I was able to work around this issue: add "ignore": ["ava.config.js"] to your babel configuration.  It is specifically the @babel/plugin-transform-modules-commonjs being run on ava.config.js from outside ava's control which causes this failure.

That's good to know, thanks for debugging the issue! Since this seems like an issue with ava and not nyc, I'm going to close the issue.


[SPARK-16522][MESOS] Spark application throws exception on exit.

This is backport of https://github.com/apache/spark/pull/14175 to branch 2.0

@mgummelt, @srowen

Test build #63504 has started for PR 14575 at commit f06ab15.

Test build #63504 has finished for PR 14575 at commit f06ab15.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63504/
Test PASSed.

Merged to 2.0. @sun-rui you can close the PR now; the bot won't close it.


Views in sample app not showing any beacons

Does this application need any major change to test in India? I can see a blank page for Map, Radar, and Chart.
My Current location is as per log: Last known location set to Latitude: 13.092544 Longitude: 80.1626726
I am using IBeacon Minor value of 3 beacons. Minor values set to 100, 200 and 300.
Sample part:
case 100: {
beaconLocation.setLatitude(13.092441779956898);
beaconLocation.setLongitude(80.16278815262923);
beaconLocation.setAltitude(20);
break;
}
Now I can see that application is not loading Map image .
Also when I check the log for the following I can see infinity value.
fromLocation.getLatitude(), fromLocation.getLongitude(), fromLocation.getAltitude(),toLocation.getLatitude(), toLocation.getLongitude(), toLocation.getAltitude()
Infinity::-Infinity,0.0,-Infinity,Infinity,0.0
Can you please help.

The chart view shows all beacons that match the current UUID filter. You can either adjust that filter to accept your beacons, or configure your beacons to advertise frames that match the current filter implementation (IBeacon frames with UUID 03253fdd-55cb-44c2-a1eb-80c8355f8291).
The map and radar views will only show something meaningful if you have a LocationProvider assigned to each of your beacons. The dummy app currently does that in the BluetoothClient.
To get a background image in the map view, adjust the BeaconMapFragment and replace the image drawable with a floor plan of your location. Calculating the reference locations is quite complex, though. You can read about that in the wiki.

Sorry I failed to add the code in insert code. Please check now.
As per the LOG the execution jumps to else if part. So I can't see the user location in the Map view.

And how is that related beacons not showing up in the view?
The IndoorPositioning singleton uses beacons that have a LocationProvider to calculate the geolocation of the device that received the beacon advertising frames.
The fact that the code always jumps to the else if part means that you're not receiving location updates from IndoorPositioning, most likely because you didn't configure location providers for your beacons (or the provided locations are so wrong that the multilateration is impossible).


Unable to write data into a file (-f &-F commands)

I am trying to use below command and I want to write all the data into an external file how ever nothing is working not getting how to use -f , --file & -F, --file-field commands
twurl /1.1/followers/list.json?cursor=-1&screen_name=pvkrajj --file C:\Users\pvkraa\Desktop\tmp.json

The -f options are for use when taking data from a file to POST to the Twitter API. They are not intended for saving data into a file.
You should use the standard terminal redirect syntax for your operating system. On Windows, this is the > operator. Read more here.


[Feature Request] Add Compatability with Damage Log

I'd like to request compatibility with the Damage Log Module.
This may or may not be related to #2 .
Issue: Chat cards added by this module are seemingly not recognized by Hide GM Rolls.
I've already contacted the author of the module as well, please see this post.
If you could either give some insight into how compatibility could be added by the Author of Damage Log, or if you yourself could take a look would be highly appreciated.
Thank you for this wonderful module, makes DMing so much more convenient!

Hey @SirMotte sorry for the delay, haven't had much time for personal projects this year. When you say that chat cards are not recognised, can you elaborate - ie, what did you expect to happen, and what happened instead?

I just got this warning:
Nothing seems obviously broken though.

Hey @SirMotte sorry for the delay, haven't had much time for personal projects this year. When you say that chat cards are not recognised, can you elaborate - ie, what did you expect to happen, and what happened instead?

Hey sPOiDar, thanks for replying. I no longer use damage log, since I have switched over to Midi Qol. It's Author has already implemented similar solutions and compatibility with this and the Quick Reveal Module.
Please see this post. https://gitlab.com/tposney/midi-qol/-/issues/1116
Unless you still want to support the module mentioned above, even though I won't use it anymore, I will close this thread for now.
Thank you!


endpoint: split up code into separate files

This makes committing / updating code, and reading the code, easier, as it is more logically organized.
Read per-commit.

This change is

test-me-please

Coverage increased (+0.009%) to 45.427% when pulling cc82360527fdda6f793c72e7403dbd5587f61d61 on pr/ianvernon/more-endpoint-cleanup into a3dfaf8b13f5fa38f79570139c0cce1ec4d077df on master.


Fix backwards incompatible change: include a prometheus service by default

In https://github.com/cert-manager/cert-manager/pull/6459 the prometheus service was made conditional both on .Values.prometheus.enabled and .Values.prometheus.servicemonitor.enabled. This however conflicts with the original intentions of the author of this Service: https://github.com/cert-manager/cert-manager/pull/1942#discussion_r309007374.
For that reason, we decided to still include the service when only .Values.prometheus.enabled is set.
We do however remove the service when .Values.prometheus.podmonitor.enabled is true and aim to add an extra option in the future to disable this service.
Kind
/kind bug
Release Note
NONE

/cherry-pick release-1.14


i18n(zh-cn): Add analyze-bundle-size.mdx

Description (required)
Add analyze-bundle-size.mdx
Related issues & labels (optional)

#9974
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

zh-cn/recipes/analyze-bundle-size.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.