[CWS] implement user path discarders on Windows

What does this PR do?
This PR adds windows discarders on .path in addition to .device_path.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Json parse error

ruby exploit.rb http://<IP_ADDRESS>:8080/pnns
/data/data/com.termux/files/usr/lib/ruby/3.2.0/json/common.rb:216:in parse': unexpected token at '' (JSON::ParserError)         from /data/data/com.termux/files/usr/lib/ruby/3.2.0/json/common.rb:216:in parse'
from exploit.rb:69:in parse_users'         from exploit.rb:85:in display_users'
from exploit.rb:135:in `'

What's the version of Joomla? Also, I'll probably won't be able to do anything without being able to test on the target unfortunately.

I tried it on
Joomla v.4.3.4
Joomla v.4.2.9
Joomla v.4.1.0
Using termux

It's normal it doesn't work for 4.3.4 and 4.2.9 as the vulnerability impacts 4.0.0 <= Joomla <= 4.2.7

Ok, thanks


Request help making react Twitter Feed component

Category
[ x ] Enhancement
[ ] Bug
[ ] Question
Request
This is more of a request for help making a react component that might end up here so I'm posting in this repo instead of the general one.
Old non-react Twitter feed repo
I'm looking for a twitter feed component and found one in OliverCC that is not react, is pretty old and I was not able convert it into a regular component on my own.
My attempt converting it
Since I've never really built a new react component from scratch myself (only copied/modified), I tried to hack one together without getting any outright errors.
My biggest hurdle... wiring up the widget (I think)
But I do not know if it's correct and I do not know where or how to put this line of code which calls the js widget library:  twttr.widgets.load();
Summary gist of my logic
My gist with a summary of my conversion
If anyone has any ideas, I would be greatful :)
Thanks

I just want to add ... that all the methods either react-twitter-embed or OliverCC seem to just be wrappers to make embed code and call twitter .js file.  It doesn't seem any more than going to twitter and copy embed code / add to page with script editor.
The reason I say this is b/c twitter embed doesn't support IE 11 so if you have a requirement to support that then none of those methods will work either. I've seen a hack out there but that did not work for me.
https://help.twitter.com/en/using-twitter/twitter-supported-browsers
We officially support versions of the following browsers from the past year on twitter.com:
Edge (https://microsoft.com/edge)
Safari (https://www.apple.com/safari)
Chrome (https://www.google.com/chrome)
Firefox (https://www.mozilla.org/firefox)
Any browser based on Webkit / Chromium engines (Opera, Samsung Internet, UCBrowser, etc.)

Hey guys!
I've added a sample web part to display Twitter feed: https://github.com/SharePoint/sp-dev-fx-webparts/tree/dev/samples/react-twitter.
If there is an actual demand - I can add the control from there to this repo.

Cool thanks @AJIXuMuK , I will give it a shot.
@tom-daly , Yes I can get the embed code from twitter using I went to https://publish.twitter.com/ ...
But I'm trying to integrate the feed into an existing SPFx webpart so unless I'm mis-understanding, I can't just add the code to a script editor.
As a general comment, if I could get it to work only with Modern Browsers (ie not IE - no pun intended), I'm ok with that.  I can just put a message in the webpart showing to click the link because you are using an unsupported browser.


switch package declaration to ensure_packages

Switch this declaration from package to ensure_packages for module compatibility.

i strongly recommend to not use any ensure_* functions. you only get in trouble with them. specially when you think about containment, chaining, ordering...

Would you be opposed to declaring https://forge.puppetlabs.com/saz/sudo as a dependency and then using:
class { 'sudo':
  purge               => false,
  config_file_replace => false,
}

in place of Package['sudo']? This would allow folks to require sudo in other places without resource conflicts...

... assuming they don't mind using saz/sudo of course...

i'll discuss this tomorrow with some colleagues of mine... someone other had asked the same in the past...

Like robert mentioned: dont use create_resources.
Can you send a pull request for using saz-sudo?

so we agree to close this one and you prepare one with saz/sudo?

@rwaffen Yep sounds good. Thanks!


Docs on unitary synthesis

Closes #99.
Explains what unitary matrices are, how they can be mapped to circuits and how this process can be used to optimize circuits.

also could you please add this page to the toc under the build operators heading pls
?

I think all comments should be address with 309ce2c 🙂

@abbycross @Cryoris should the title of this page be "Synthesize unitary operations" or "Synthesize unitary matrices"? I see it's one in the sidebar and the other in the page title

ok cool thanks for confirming @Cryoris ! I've updated accordingly


Async example

Hi, I'm having trouble to get it working with async load of autocompletions.
I think it would be nice if you could provide an example of an async Presenter with a progress / failure management. This join the request from #12

This is not an Autocomplete issue but standard RecyclerView management.
If you can show a progress in your activity RecyclerView, than you can show it with Autocomplete. Autocomplete will just present what you give.
For example (just one of the many ways) your adapter can return 1 for getItemCount() while loading, and display a loading indicator there.

Are you using the RecyclerViewPresenter or your own?

For now I'm using RecyclerViewPresenter, but I might write my own to customise it after I get the minimal behaviour working

I understand your problem, I think it was introduced in 1.0.3.
We dismiss the popup if height == 0. So right now you should provide a 'loading' item of some sort, or add some padding to the recyclerView. Anything that might give the view some height.

Hahah spent the day on this one .. got it working with 1.0.2.
Gonna try with a more complex layout that should not be empty. Thanks !

Maybe it can be fixed on our side, I'll take a look

Not able to solve this easily. We should introduce a 'loading state' and it takes some work.
For now I have bumped version to 1.1.0 since the change was more breaking than I thought.
I have stated more clearly in README that height == 0 means that the popup will be dismissed.


Simple property assignments in KTS

Description
Follow up #2105.
https://blog.gradle.org/simpler-kotlin-dsl-property-assignment
Checklist
Before submitting the PR, please check following (checks which are not relevant may be ignored):

[ ] Commit message are well written. In addition to a short title, the commit message also explain why a change is made.
[ ] At least one commit message contains a reference Closes #<xxx> or Fixes #<xxx> (replace<xxx> with issue number)
[ ] Tests are added
[ ] KtLint format has been applied on source code itself and violations are fixed
[ ] CHANGELOG.md is updated
[x] PR description added

Documentation is updated. See difference between snapshot and release documentation

[ ] Snapshot documentation in case documentation is to be released together with a code change
[ ] Release documentation in case documentation is related to a released version of ktlint and has to be published as soon as the change is merged to master
[ ] In case of adding a new rule, it needs to be added to experimental rules documentation

https://blog.gradle.org/simpler-kotlin-dsl-property-assignment

states:

It’s important to note that the feature is still marked as incubating. Changes to the feature are possible, although unlikely. We don’t recommend using this feature in published plugins until it’s marked as stable (likely in Gradle 8.3).
Lazy property assignment is supported in IntelliJ 2022.3 (2023.1.1 or newer is recommended) and Android Studio Giraffe (or newer).

I have some doubts whether we already should try this out in our build or that we should wait until the stable release. Is there a pressing need, not to wait?

Either way, I also filed it to https://github.com/detekt/detekt/pull/6251.

In Gradle 8.3, this feature is still marked as unstable.

Yeah.

Some of this feature had been used in https://github.com/pinterest/ktlint/blob/460c6358cdf15bc02d1921b572e8b23c1b4a722c/build-logic/src/main/kotlin/ktlint-kotlin-common.gradle.kts#L31
Maybe it's time to land this.

It's stable after #2294 merged.


[Stepper] Add Alternative Label

Add Alternative Label in Stepper, Material
Linked issues
#91337
Before

Screenshots
Horizontal

Vertical

[important!] Remained issues

Padding, Align

Up and down number-icon

Reproduce code
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Stepper Alternative Label',
      home: MyStatefulWidget(),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  const MyStatefulWidget({Key? key}) : super(key: key);

  @override
  State<MyStatefulWidget> createState() => _MyStatefulWidgetState();
}

class _MyStatefulWidgetState extends State<MyStatefulWidget> {
  int _index = 0;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Stepper(
        type: StepperType.horizontal,
        // type: StepperType.vertical,
        currentStep: _index,
        onStepCancel: () {
          if (_index > 0) {
            setState(() {
              _index -= 1;
            });
          }
        },
        onStepContinue: () {
          if (_index < 3) {
            setState(() {
              _index += 1;
            });
          }
        },
        onStepTapped: (int index) {
          setState(() {
            _index = index;
          });
        },
        steps: <Step>[
          Step(
            isActive: _index == 0,
            title: const Text('Step 1 title'),
            content: Container(
                alignment: Alignment.centerLeft,
                child: const Text('Content for Step 1')),
            label: const Text('Step 1'),
            subtitle: const Text('Step 1 subtitle'),
          ),
          Step(
            isActive: _index == 1,
            title: const Text('Step 2 title'),
            content: const Text('Content for Step 2'),
            subtitle: const Text('Step 2 subtitle'),
            label: const Text('Step 2'),
          ),
          Step(
            isActive: _index == 2,
            title: const Text('Step 3 title'),
            content: const Text('Content for Step 3'),
            subtitle: const Text('Step 3 subtitle'),
            label: const Text('Step 3'),
          ),
          Step(
            isActive: _index == 3,
            title: const Text('Step 4 title'),
            content: const Text('Content for Step 4'),
            subtitle: const Text('Step 4 subtitle'),
            label: const Text('Step 4'),
          ),
        ],
      ),
    );
  }
}

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thanks for the confirmation on tests, @KKimj. If you're new to widget tests, the existing Stepper widget tests contain many great examples of meaningful tests :)

@craiglabenz
Hello, I add new test for lable ( Alternative Label ) with stepper.
Please check, and give me some comments.
Thanks, HAGW!

@craiglabenz
Hello, how are you doing?
Can you give me a new feedback?
Thank you.

@KKimj
I am So. So. sorry for disappearing for so long. Somehow, the alert for this PR snuck past me and I never caught it.
In my last comment, I wrote "it would be nice to add another one that confirms the styling logic you've written is behaving as anticipated", which I now realize was a little unclear. The key function I would like to see tested is _labelStyle, since it accepts an index and merges that with the user's progress through the stepper and the Dark Mode / Light Mode status before ultimately yielding a TextStyle. The internals of that function are pretty complicated, so I think we should have unit tests to verify that a Stepper with X steps, and currently at step Y, is rendering each label correctly. There a few scenarios to cover there - steps before, on, and after the active step; in both light mode and dark mode.
Does all of that make sense?
PS - I promise not to disappear this time!

Hello!! LTNS!!
I understand what you say.
I will update test-codes soon.
Something like below.

Let, X := currentStepIndex, Y := arbitraryStepIndex (except current)

set X = 0.
Check Styles of X and Y.
set X = 1.
Invoke setState().
Check Styles of X and Y.

P.S
Happy New Year!

Yes, that looks like the right pseudocode. Thanks so much and 🌔 Happy New Year 🌔 to you, too!

@KKimj Is this a PR you are interested in doing further work on? It's perfectly understandable if you don't have time right now, and if that's the case we can save what you've done so far and put it aside so that if someone else wants to work on this they can start from your PR. But if you do have the time to work on this, that would be great!

@Hixie
Hello!
I always had it in mind. I want to finish all the work within April. How does that sound?

@craiglabenz
Hello..! It's been so long. How have you been?
I pushed a commit for test about TextStyles of Label Widget.
Please Review my commit.
Thanks!

That test looks good, @KKimj. However, I just tested the code myself directly, and the padding / alignment issue does still seem to be in play. Is this something you're also looking to address?

@craiglabenz
Hello!
Now I realize again that there is no alternative label in the Vertical type, StepperType.vertical.

https://material.io/archive/guidelines/components/steppers.html#steppers-types-of-steppers

Anyway, I found a way to solve the padding / alignment issue.
The method is to modify the function '_buildVerticalHeader', forcing width to 24.0, according to the Material Guide.

Code diff here, (It was not pushed.)

An example UI can be similar to the following.

But it's too small, and as I said above, it's dubious because alternative label is not in the Vertical type. I'd like to ask for advice on how to do it.

Thanks, Have a good day!

Oh dear, we're running out of pixels below the circle. To be honest, I'm not sure that it makes sense to put anything in this spot, given the screenshots you shared and how cramped the area is. The fact that only such short strings will fit is a bit of a problem, in my opinion.
Taking a step back - what informed your desire to land this change? Is there possibly another way that need could be satsified?

@craiglabenz
I was motivated for the horizontal type.
Is it right to exclude it from the vertical type?
Or is it right to include even more width?
But this would violate the width or margin value of the Material design.

I think it's probably fair to have this be horizontal-only, and to spell that out in the relevant docstrings.

Thank you!
Then, when the label attribute is given in the vertical type, how should I handle it?
Should I use an asset or something?

My ideal solution would simply be to do nothing in the vertical case. Have the new attribute's docstring mention that it is only honored in the horizontal alignment and then make no changes to the vertical build methods.

Thank you for letting me know about your Insight!
I just did some new push.
Please check. Thank you!

I like that this is currently horizontal-only, but I'm also seeing a spacing issue there. Note that before adding a label, everything is appropriately centered:

But then after adding a label, the horizontal connectors shift and are no-longer centered. fwiw, I think this is largely the core challenge of this PR - you can no longer as easily predict the heights of each component to know how to keep the horizontal connectors centered within the circle.

The horizontal connectors look centered within the entire column of circle-and-label, but my hunch is that it should remain centered within the circle.

@craiglabenz
This is what you're expecting, right?

That centering looks good! I forget, is the elevation coming from the Stepper itself, or have you wrapped it in something in this demo? If it comes from the Stepper itself, then some padding below the label would help.

@craiglabenz
Hello!!
I Pushed some commits.
Please review!

And also, Please check out these corner cases, which are related to fontSize.

@craiglabenz
Wow, thank you.
Is there anything I need to do for customer_testing-linux, CI that has failed?

I strongly suspect that failure is a false-negative, but we'll have to see if it happens again.


update: codeql action

Applied mkpr on .github/workflows/codeql_analysis.yml
cat .github/workflows/codeql_analysis.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
109

Relevant Lines:
109

💛  - Coveralls


New version: Mozilla.Thunderbird 115.6.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@stephengillie Why was this PR merged because it purged all languages from manifest.

@krystofkrticka because all the other locale installers were causing CI internal errors on "installer scan" step, because of the huge number of installers to scan (4 per locale).

If you wait long enough the mod will override it becuase this causes issues to people using other locale then english.

@krystofkrticka

Looking at other PRs, other packages have no problems providing only English
Language can be changed in Thunderbird after install

Yes but if you install applications for users as system administrator on fleet of machines its kind of annoying to change it for every users because at my work I have quite high percentage of users which are unable to speak or read in English so they are kind of dependent on computers communicating with them in their native language. And deploying languages using GPO is very annoying in mozillas applications because you have to change it every time there is an update.

Hi @krystofkrticka,
I haven't been checking for modifications to language files before. I'll start this going forward.


fix(sec): upgrade org.fusesource.mqtt-client:mqtt-client to 1.15

What happened？
There are 1 security vulnerabilities found in org.fusesource.mqtt-client:mqtt-client 1.14

CVE-2019-0222

What did I do？
Upgrade org.fusesource.mqtt-client:mqtt-client from 1.14 to 1.15 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
The specification of the pull request
PR Specification from OSCS

Included in Included in https://github.com/apache/storm/pull/3463


Integration component test issues after v0.5.31

After the v0.5.31 release, I started getting an error when running tests. The first  (alphabetically) integration component test will always fail to render. Event if it's a very simple test that just renders a div it will fail, so it's consistent (in my app).
When using ember exam with multiple partitions I get the same error for different partitions.
But always just the first integration component test (or first ones when using partitions).
Tracked it down to this line:
https://github.com/switchfly/ember-test-helpers/compare/v0.5.30...v0.5.31#diff-6131c3a14dd0889855bcf084535928fbR250
This is what renders to the dom instead of the component's content
{ type: 'log', text: '\'<!---->\'\n' }

It works fine again when I replace this line:
 context._element = element = document.querySelector('#ember-testing > .ember-view');

with:
context._element = element = Ember.$('#ember-testing > .ember-view');

PhantomJS: 2.1.1
Ember: 2.4.5
ember-cli: 2.7.0
ember-cli-qunit: 2.1.0

I'm also seeing this, reverting back to 0.5.31 resolved this.

From my eyes the only difference between those two selectors is that the jQuery one will select all elements that match and the document.querySelector one will only match the first...
Can someone create a demo repo showing the issue?

I'll throw one together tonight

Thank you!

@rwjblue I'm now unable to reproduce it, I'll keep an eye out and will report back if I can reproduce+isolate it..

Kk. I was wondering perhaps if it was relate to Ember version and/or whether or not the component under test was tagless.

@rwjblue this happened while upgrading from ~2.6 to 2.8.  We are also on ember-mocha so unsure if that had anything to do with it.
The tests that were failing with this were not tagless components, though we had test failures because of that new assertion but I consider that unrelated to this issue.

After looking at this again, I found the issue on my app.
There was a DOM element leaking from Acceptance Tests that wasn't removed properly.
Since the new selector only returns the first ".ember-view" it didn't match the one it expected, hence the error on the new version.
Also, the reason this was only breaking the first test is that the HTML gets cleaned up after the component test is done.
https://github.com/switchfly/ember-test-helpers/blob/master/lib/ember-test-helpers/abstract-test-module.js#L138
teardownTestElements() {
  document.getElementById('ember-testing').innerHTML = '';
  ...
}

I fixed the issue by making sure the element leaking was removed properly on ATs.
Thanks for the input

@calderas that might explain why I wasn't able to reproduce it an example app.  Thanks for reporting back, I'll be sure to look for any components that are leaking elements.

@jasonmit @calderas can we close this? or is this still an issue?

No longer an issue, thanks!


fix: missing command

Summary
Command $cluster = Get-VCFCluster | Where-Object { $_.id -eq ($workloadDomain.clusters.id) } was removed on accident, also updated a couple variables

Type

[x] Bugfix
[ ] Enhancement or Feature
[ ] Code Style or Formatting
[ ] Documentation
[ ] Refactoring
[ ] Chore
[ ] Other
Please describe:

Breaking Changes?

[x] Yes, there are breaking changes.
[ ] No, there are no breaking changes.

Test and Documentation

[x] Tests have been completed.
[ ] Documentation has been added or updated.

Issue References
Closes #101

Additional Information

I am unable to validate the commit due to other issues as well.
line: 196 foreach ($clusterid in $mgmtClusterIds) when the cluster in which sfo-m01-vc01 vm exists is higher than other cluster in this list, the function will shutdown sfo-m01-vc01 vm before other clusters in the domain shutdown down.
line 349: if i sort the cluster list to have the main cluster be the last one on the list, the script ends at line 349 and the startup json is not created, therefore i am unable to powerup the environment through the script.
Please make the above changes and re-run the module E2E based on step 6 in https://vmware.github.io/powershell-module-for-vmware-cloud-foundation-power-management/documentation/#usage-examples

line: 196 foreach ($clusterid in $mgmtClusterIds) when the cluster in which sfo-m01-vc01 vm exists is higher than other cluster in this list, the function will shutdown sfo-m01-vc01 vm before other clusters in the domain shutdown down. i am not sure understand this comment. I am really not sure what the issues are, i can't replicate these issues

ah i found the issue, line 199

Merging this one after discussions with @joisika. He will review the current state and initiate some in-depth testing for the next release.


chore: use prettier

Fixes #90 and updates Tailwindcss to fix dependency compatibility errors

The greetings action is failing because of some error in the action - so I have added reviewpad which also adds many more features

The greetings action is failing because of some error in the action - so I have added reviewpad which also adds many more features

I think you should add ReviewPad in a separate PR.

Removed Reviewpad - @Panquesito7

There are too many conflicts, I think I'll just make a new pr


Shortened links aggregation ?

Assume we have two playlists with multiple videos and we want to join them easily.
(Of course we could copy over all intervals together, but this is trivial...)
What about providing shortened forms to yt-looper (goo.gl ids), then comes some magic blah blah and we have both list merged together with full playlist in editor ?
:trolleybus: :neckbeard:

Maybe we could revive #21 somehow along with this idea ? :sparkles:

How about syntax: &expand=<short_link1>+<short_link2 ?
URL normalizer would evaluate short links and simply replace
&expand=http://goo.gl/ycH3Px+http://goo.gl/LKbVEc

with
&v=ZuHZSbPJhaY&t=1h1s;1h4s&v=lWqJTKdznaM&t=58;68

what about: #http://goo.gl/ycH3Px&http://goo.gl/LKbVEc ?
This wouldn't limit us in any way! We could switch to different url shortener without any changes in aggregation code.
:neckbeard:

I am okay with #http://goo.gl/ycH3Px&http://goo.gl/LKbVEc, it is short and makes sense (replace short url with intervals it represents).
Normalizer will simply look for http://goo.gl/ before currently existing rules, so there will not be any issues.

Voila! :rocket:
https://yt.aergia.eu/#http://goo.gl/ycH3Px&http://goo.gl/LKbVEc


Marker rendering clipped

Hi, I was trying to render some points (attached bellow), but the marker was rendering cutted.
I try with mapnik 2.0.3 and 3.0.0. I was attaching one file to see, and some links to reproduce the experiment.
1 - antenas.geojson : should be putted in root of node server project (http://liveweave.com/2FJglb)
2 - The project of sample server (app.js) (http://liveweave.com/irl8CG)
3 - stylesheet.xml (http://liveweave.com/ZUuwIS (use the css area in this case))
4 - A HTML to consume the map of server. (maybe its necessary to configure the IP) (http://liveweave.com/51tYbg)
To test, download the server project, execute the following command in a terminal:
$ git clone git@github.com:mapnik/node-mapnik-sample-code.git
$ cd node-mapnik-sample-code
paste the antenas.geojson  and stylesheet.xml on this folder
replace the tile/pool/app.js  by app.js in the link
On the root folder, now, execute:
npm install
npm install mapnik
npm install generic-pool
node tile/pool/app.js ./stylesheet.xml 8586
open the HTML file in the brownser. The initial position and zoom is fixed to point the problem.
I already have tested to use those parameters in the stylesheet.xml file:
shield-clip:false  /  avoid-edges:true   /  marker-clip:false
I try to increase the bufferSize on server map creation tile, too.
Nothing works.
After applly bufferSize greater than 16, it does not affect the map visual.
Ps. I use liveweave to create the links, so ever copy the content of HTML part in the tool.

Have you tried playing with the extent of your map, I see you set it to the bounding box of the tile requested.

Doh, sorry thats not going to fix anything. Having a brain fart kind of day.

@everton1 https://github.com/mapnik/mapnik/issues/2577


SNOW-1617549: throw warning on python 3.8

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
<!---
In this section, please add a Snowflake Jira issue number.
<p>Note that if a corresponding GitHub issue exists, you should still include
the Snowflake Jira issue number. For example, for GitHub issue
https://github.com/snowflakedb/snowpark-python/issues/1400, you should
add "SNOW-1335071" here.
--->
Fixes SNOW-1617549

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.
[x] I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines

Please describe how your code solves the related issue.

throw one time warning when importing snowpark library on python 3.8
effect:
on python 3.8, this warns deprecation message exactly once
Python 3.8.16 (default, Mar  1 2023, 21:19:10)
[Clang 14.0.6 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from snowflake.snowpark import functions
<stdin>:1: DeprecationWarning: Snowpark Python Runtime Support for Python 3.8 will reach end-of-life on March 31, 2025. To ensure continued support, please upgrade your runtime environment to Python > 3.8 before this date. For more details, please refer to https://docs.snowflake.com/en/developer-guide/python-runtime-support-policy.
>>> from snowflake.snowpark import functions
>>> from snowflake.snowpark import functions
>>> exit()

on python 3.11, no deprecation warning
Python 3.11.9 (main, Apr 19 2024, 11:44:45) [Clang 14.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from snowflake.snowpark import functions
>>> from snowflake.snowpark import functions
>>> from snowflake.snowpark import functions
>>> from snowflake.snowpark import functions
>>> exit()

@sfc-gh-pspaeth thanks!
I've flatten the python version part as per your comment, I will leave the rest untouched -- those words are copied from an internal doc to communicate the deprecation.

@sfc-gh-deliu yes, to be precise, it's whenever snowpark module is imported, the following examples will be able to trigger the warnings:
import snowflake.snowpark
from snowflake import snowpark
from snowflake.snowpark import functions
from snowflake.snowpark.functions import sum

however, the following examples won't because snowpark is not imported
import snowflake
import snowflake.connector

@sfc-gh-jdu updated


ApplyBatchCommand Fix

Use SqlParameter and item.SqlValue instead of item.Value as it can cause a data-type swap/mismatch if item.Value is null

FWIW, it fixed my specific problem.  There will be more vetting to be certain all is working and I didn't break something else.


automate loading world into ramdisk

automate enabling/disabling running the server on a RAM disk (similar to how msm used to). should also dump the world periodically back to disk in case of unexpected events (power loss, system crashes etc).

Check out the options in the readme for Adjusting global server properties
The options mscs-enable-mirror and mscs-mirror-path as well as their "default" counterparts can be enabled for this exact feature. Then you will want to use the sync command (See Command reference) to periodically sync to disk probably using a cron job.

@m4lfuncti0n, are the mscs-enable-mirror and mscs-mirror-path options and the sync command enough to get you what you need? Otherwise, please describe the missing feature in a little more detail.

Since I we haven't heard back, I'm going to assume that @m4lfuncti0n is happy with the features already in place for loading and syncing a world server to a ramdisk, and close this issue.


PR Proposal: update to latest Packer SDK and modern plugin support

I wanted to gauge interest in a PR for updating to the latest Packer SDK and enabling modern plugin support for this repository. It would be helpful for various projects if we could use the GoSS provisioner with modern Packer usage.
Please let me know. Alternatively, I can fork this project and perform light maintenance it if there is no further interest. Thanks!

This has been solved by #69


Missing transaction type for spendings of watch-only addresses

I added Masternode yEeSSUjQKPVJKZYeMRtaRiSoUqYuid3Wuv as "watch-only" address
When this Masternode gets paid I can see it as type "Mined" in the "Transactions" tab
When this Masternode spends some DASH (e.g. transaction de0721f68246864a9c475517240c5c67d777b11a92eb4aca366f3a391733eb4d) I see the sum of the payments from that transaction as one outgoing payment listed, but without any transaction type

Would be nice to have a transaction type there to make use of the type-filter.

Deprecated by Evolution GUI


added a couple of rules and create_triggers

Made some better extractions...

fair enough -- forthcoming.
FYI @MihaiSurdeanu these came out of the intervention doc that was sent around

It looks like GitHub is having some kind of problem.  It won't merge.  I'll check back periodically.


The topic reference href="#null" could not be found when publishing to PDF

With the attached DITA project which is valid I'm getting an error in the console when publishing to PDF:

[xslt] file:/C:/Users/radu_coravu/Desktop/chunk-map-stuff/maps/abc.ditamap:7:78: [PDFX005F][FATAL]: The topic reference href="#null" could not be found. Please correct the reference, or set the scope or format attribute if the target is not a local DITA topic.

Uploaded the ZIP to our web site, somehow I could no longer update it using GitHub's facilities:
https://www.oxygenxml.com/forum/files/chunk-map-pdf.zip

Same issue here when chunking topics. preprocess.chunk.skip saves the day but, unless I missed it, it's not in the docs.

I tried to publish again to PDF my original samples using DITA OT 2.4.4 and I cannot seem to reproduce the problem anymore. What DITA OT version are you using? Do you obtain the problem using my samples or only using yours?

With my own samples, I confirm that the issue is not found in DITA-OT 2.4.6 or 2.5.


Relax Copy bound to Clone

This issue was mentioned in #118. Thus I felt it would be good to make a separate issue for it, as I do not think it makes sense to handle it as part of that. Big integers or rationals would be an interesting use case, for example.

@vbarrielle While we are doing some breaking changes: Maybe we could change the constructor methods of CsMatBase so new does not do any sorting and we add a new method new_sort which acts as new does today?

@vbarrielle While we are doing some breaking changes: Maybe we could change the constructor methods of CsMatBase so new does not do any sorting and we add a new method new_sort which acts as new does today?

@mulimoen are you sure you wanted to comment on this issue? It looks like you wanted to comment on #232.

@mulimoen are you sure you wanted to comment on this issue? It looks like you wanted to comment on #232.

But this particular issue could be solved for 0.10.

But this particular issue could be solved for 0.10.

It is a big stopper here too, the new constructors would not require elements being Clone nor Copy

It is a big stopper here too, the new constructors would not require elements being Clone nor Copy

I see, what would the behavior of the non-sorting new constructor be? Error returning, same as the other structure checks?

I see, what would the behavior of the non-sorting new constructor be? Error returning, same as the other structure checks?

I think error returning is the cleanest, we would need this constructor anyway. We can always rename and add a panicking one if we find this cumbersome.

I think error returning is the cleanest, we would need this constructor anyway. We can always rename and add a panicking one if we find this cumbersome.

Ok, let's include the new and new_sorted constructors for 0.10 then. As you say, a panicking constructor can always be added later. I'm goind to add a separate issue as in my mind this one was more about removing the Copy bound that's present on most linear algebra functions.

Ok, let's include the new and new_sorted constructors for 0.10 then. As you say, a panicking constructor can always be added later. I'm goind to add a separate issue as in my mind this one was more about removing the Copy bound that's present on most linear algebra functions.

Fixed by #276.


Graphiql height is not 100%

When I open Graphiql it looks like this

It looks like that html and body tags need to have their height set to 100% so the layer #graphiql-container can expand too.
I've tried also opening in safari and in chrome on incognito window (without any extension running) and the same happens.

+1

Are you running graphiql via express-graphql?
I realized that I had just released a breaking change in a point release, so I republished to account for the breaking change. I'll make the change to express-graphql now as well.

Ok, I could easily reproduce this issue when trying to update http://graphql-swapi.parseapp.com/. I've got  new versions of both graphiql and express-graphql on npm which should fix this, and have unpublished the offending ones so people don't accidentally run into this when they npm install!
Thanks again for the fast report

Reinstalled node_modules and it shows fine now. Thanks for the quick fix and the awesome work!

Most excellent, thanks for confirming the fix!

Hello! I've got the same behaviour using build.sh on master, what was the fix to this?

Hey @leebyron, I have the same issue but I don't use express-graphql.
I work in a company with closed network and express-graphql uses cdns.
I use webpack with css and style loaders for the css.
I've made a project with the problem. It's very simple and I will be happy if you help me. I'm simply lost

Hey again,
after investing almost two days into figuring out what the hell is going on, it appears that the problem is the id of the div i try to render into.
In your example it is set to 'graphiql' and in my project it is 'app'.
when chaning my app to 'graphiql' it worked. and when changing in your example 'graphql' to 'unique' it didn't work anymore.
I am shocked

I am infinitely sorry. Of course it won't work since the way css works.
maybe you can write about it in the readme or bind the css to the component, please?

I'm glad you figured out what was going on! I'll find a way to make the css bindings more appropriate so that this kind of issue is harder to get yourself into in the future.


docs: update structure

introduce new documentation structure https://github.com/inveniosoftware/cookiecutter-invenio-module/issues/56
document user perspective
document signals (if not done)
document example app (if not done)

Closed by 81efd1d58ed534c79fbf1ac3d1674dfdf5b98cfe


Promises Chaining

Hello @Thomvis!  Great project!  I love the simplicity and how the code works.  However, I didn't notice anything in the documentation that would allow for promises chaining like they have in PromiseKit.    I also looked at the code as well.  The andThen() function only supports side effects and not the same idea as chaining promises and I didn't see anything else in the code that would support that functionality.
Is there something I missed or is promises chaining not part of the design of BrightFutures?  Or maybe it's something that's part of the roadmap?  Thanks.

Hi! Thanks for checkout out BrightFutures.
I think what you're looking for is map and flatMap. There's an example at the start of the readme:
User.logIn(username,password).flatMap { user in
  Posts.fetchPosts(user)
}.onSuccess { posts in
  // do something with the user's posts
}.onFailure { error in
  // either logging in or fetching posts failed
}

Here we're chaining the result of the logIn function to Post.fetchUser. You can find more examples in the tests as well. Is this what you're looking for?

Hi @Thomvis
Thanks!  That's exactly what I was looking for!  I guess it never occurred to use a flatMap for promise chaining.  I like this way better as it gives more power by using functional composition.  Great stuff.  Thanks for your help.


[Feature Request] Super Sonic

A great feature would be to add Super Sonic a some sort of transformation that allows the user to fly for example. Maybe a different moveset taking advantage of Sonic's other moves in Frontiers (I.E Wild Rush, Loop Kick, etc...).Would be pretty sick!

Agree, would be great to see super sonic be part of the mod!

https://github.com/DS-GAMING-SE/Sonic/pull/4

@DS-GAMING-SE Shouldn't this Issue be closed since he has already been added and enabled?

@DxsSucuk This is actually in the mod?

Yes @lolkitre123 it has been inside for a long time, I made a PR around Sep 2023 where I readded the stuf DS commented out. But since then DS has put in the effort to balance it and make it functional compare to the PR I made. And with the 2.0.0 release he just sent out, Super Sonic is now out there!


Bugfix/json error handling fix

Add's a new exception which handles multiple downstream errors or returns as before if there is only one error.

@TJC Updated to include that extra case. Is this okay now?

LGTM 👍


Jade Kit not giving rewards

Only gave
12x Rare Candy
16x Duskball
5x Wishing Pieces

This was indeed broken. Fix is being pushed in just a bit.


Receiving data multiple time and string value is not accurate

Hello
I have an issue in reading data which i write from arduino side. For example i'm trying to send a string value like "P152" but in android side it receives in pieces like "p","152","p152" or "P1","52","p152" like this, also there are some delay. if my string length is "AP152Q524" then it took around 28-30 second to fetch complete string.
I don't understand what i do to deal with this so please help me..
Thank you.

Which chip you are using?

I have same Problem. what's Problem?

I am also facing the same issue? Can anyone help me on this ?

+1

I am also facing same issue @felHR85 I am receiving my packets in parts in delay of 20 milliseconds , I am not able to parse my full packet without using custom function to merge packets with delay less then 20 milliseconds.


Updated Readme.md file

Description

Contribution Checklist:

[✔] The pull request only addresses one issue or adds one feature.
[✔] The pull request does not introduce any breaking changes
[ ] I have added screenshots or gifs to help explain the change if applicable.
[✔] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Changes:
The logo is centered.
Headers are more prominent with larger font sizes, Key features are emphasized.
Links are more organized in bullet points.
Support and testimonials sections are styled for better engagement.
More concise and clear instructions for contributing.
Please see here for more information.

Hey @Ratheshan03

The logo is centered.

The logo is intentionally kept left as it aligns with the rest of the readme.

Headers are more prominent with larger font sizes, Key features are emphasized.

This is also intentional. I'd like the header to be lighter and smaller.

Links are more organized in bullet points.

Agreed on this. You can raise a new PR with this change.

Support and testimonials sections are styled for better engagement.

Agreed on this. You can raise a new PR with this change.

More concise and clear instructions for contributing.

Agreed on this. You can raise a new PR with this change.
Closing this PR. Please raise a new one. Thanks @Ratheshan03 !


CSE: Imported rune constants appear to be bound to strings; formating issues

Currently we have:

The environment shows "< RUNE >" instead of just . This makes it look like the value is a string.
Also note that the string "< RUNE >" exceeds the boundary of the frame box. This problem appears to be solved for real strings:

This is handled in the latest version.


Allow the back gesture to not override any other gesture that is not in the same direction

Basically I have a bunch of dismissible views, but when I use this plugin with the whole page set as the swipe back gesture, I am unable to dismiss my views in the other direction. Would it be possible to make it so that the gesture does not override these (meaning the right to left gesture) and only the left to right gesture?
Thank you! Love the plugin btw

Honestly I don't have deep understanding how does gesture disambiguation works in Flutter.
The way I've made this package is

Forked CupertinoPageRoute class from Flutter source
Replaced constant back gesture width (20.0) with configurable one

Swipe detection in CupertinoPageRoute is implemented this way:
Stack(
  fit: StackFit.passthrough,
  children: <Widget>[
    widget.child,
    PositionedDirectional(
      start: 0.0,
      width: dragAreaWidth,
      top: 0.0,
      bottom: 0.0,
      child: Listener(
        onPointerDown: _handlePointerDown,
        behavior: HitTestBehavior.translucent,
      ),
    ),
  ],
)

e.g. Listener in on top of the page content. So it captures horizontal swipes that occurs on its area.
There exists an issue about CupertinoPageRoute's swipe precedence. OP wants to make Slider draggable from the very left edge of the screen

should be fixed on #7


Allow 10,000 more log enteries

if u check the commits its not doing more than 10000 enteries its running in a loop

@vijaye-statsig


Login and Signup use cases

Implementing the use cases for the login and signup

Unlinked PR #17 as there is a TODO left, which needs to be done after other use cases, are implemented


Document routeTableResourceGroup

New cloud provider option routeTableResourceGroup is added from https://github.com/kubernetes/kubernetes/pull/75580 (in master branch).
We should update documents for it.
/kind documentation

Added in #163.


Support React Native 0.74

Details

Closes #221.
Related Issues

GH_LINK
Manual Tests

Linked PRs

Do not merge this until E/App upgrades to RN 0.74.

Asked about this in slack: https://expensify.slack.com/archives/C06BDSWLDPB/p1720805115823789

Is there an estimate for when this can get merged?

We will merge this PR once E/App gets upgraded to 0.74.

@tomekzaw ready for merge, E/App upgraded to 0.75

conflicts

@roryabraham Conflicts resolved

@tomekzaw when we can expect a release with this change?

@pleerock Each merged PR to live-markdown gets automatically released. This PR was released in 0.1.122.
There's also a PR that adds support for RN 0.75 (https://github.com/Expensify/react-native-live-markdown/pull/428) and I would love to merge it by tomorrow.

thanks @tomekzaw !


Unable to compile.

Trying to run the simulator I get the following errors.
TTLockHelp.h Line 42 @interface TTLockHelper : NSObject
Error: No type or protocol named 'TTSDKDelegate'
TTLockHelp.h Line 68  optionType:(OprationType)optionType
Error: Expected a type
Any help.


Lambda's Context Object is not being passed on

Hi,
I'm having an issue with Voxa library on AlexaSkill.js not passing on the context object that comes as a parameter for all Lambda handlers:
https://github.com/mediarain/voxa/blob/ec0d98d61f161970dcf52c6073aa42815b5ea2b3/lib/AlexaSkill.js#L93
We need to be able to change a property in the context item, specifically: context.callbackWaitsForEmptyEventLoop = false so that Lambda function will returns immediately as callback is called. Without this property, if we still have any nodejs event in the loop (e.g: shared resources connecting to redis/elasticCache, DB Connection), the function will just freeze and timed out (see: http://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-model-context.html#nodejs-prog-model-context-methods).
Is it possible that this context item being exposed to the AlexaSkill object?

Hi @nkahoang, you have access to that property from the alexaEvent object,
Example:
skill.onState('state', (event) => {
  event.lambdaContext
})

However, in general it's better to just use promises and let voxa handle this. Voxa will wait for all promises to complete before finishing the lambda if you return them from your controllers
skill.onState('someState', (event) => {
   return saveToDBPromise()
    .then(() => {
      return { reply: 'SomeView'}
    })
})

Hi @armonge ,
Thank you for that. We will test the lambdaContext object via event.lambdaContext.
This is not really because we are not using Promise. In fact, we are using them on every onState / onIntent. We we did a trace and the final promise did resolve (trace in cloudwatch log) and found out that the code managed to reach the onBeforeReplySent event.
In my opinion by the fact that many packages such as cache-manager-redis and other (we had this issue using firebase with Lambda) caches the connection internally; and killing / opening new connection manually causes delays and additional resource usage. We've found an article regarding this behaviour for MongoDB as well (https://www.mongodb.com/blog/post/optimizing-aws-lambda-performance-with-mongodb-atlas-and-nodejs).
But as we haven't tested this extensively (we will do now that we know we can access the context directly from event object), we might not have known the side effects. It seems like all of Voxa's middlewares are being handled fine (we use Opearlo and Voicelabs) as we found their traces in the log after setting context.callbackWaitsForEmptyEventLoop=false. If you have know any problems with the usage please let us know.

Hi @armonge ,
We have just tested and found that the alexaEvent object (or event) has the lambdaContext property but it is null. We tested by both checking if alexaEvent.lambdaContext is not undefined and confirmed by json encode the alexaEvent object out to lambda cloudwatch log.
Can you please assist us? Below is the raw log
var a = {
session:
{ attributes: {},
sessionId: 'SessionId.xxxxxxxxxxxxx',
application: { applicationId: 'amzn1.ask.skill.xxxxxxxxxxxxxxxxxx' },
user:
{ userId: 'amzn1.ask.account.xxxxxxxxxxxxxxxxxxxxxxxxxxx',
accessToken: null },
new: true },
request:
{ intent: { name: 'GetInfo', slots: [Object] },
requestId: 'EdwRequestId.xxxxxxxxxxxxxxxxxxx',
type: 'IntentRequest',
locale: 'en-AU',
timestamp: '2017-11-27T22:32:21Z' },
context:
{ AudioPlayer: { playerActivity: 'IDLE' },
System: { application: [Object], user: [Object], device: [Object] } },
version: '1.0',
lambdaContext: undefined,
intent:
Intent {
name: 'GetInfo',
slots:
{ date: [Object],
movie: [Object],
specialTime: [Object],
concept: [Object],
genre: [Object],
time: [Object],
cinema: [Object] } },
model:
Model { ... },
opearlo:
VoxaOpearloEventRider {
name: 'CheckCinema',
alexaEvent: [Circular],
variables: {},
ignoreState: false } }```

@nkahoang really weird, how does your look lambda handler look like? are you using
exports.handler = skill.lambda();
```
?

I'm available to chat over slack on http://alexaslack.com/ , you can DM me, or you can also check https://gitter.im/voxa-rain/voxa

Oh wait, i think you actually found a bug! seems like we're not passing the context to the AlexaEvent constructor on https://github.com/mediarain/voxa/blob/master/lib/AlexaSkill.js#L100, that would cause the undefined. Sorry about that, i'll fix and push a new version tomorrow, also create some tests so this wont happen again

@nkahoang we pushed a new version that should fix the issue about lambdaContext being undefined. Let me know if that helps

Hi @armonge ,
We've just tested with the latest version and the lambdaContext is confirmed to be working via alexaEvent.lambdaContext! Thanks!


JSON.parse() console error

Please help me

Are you on the latest version? Or a specific branch?

n the latest version

yes i am on latest version

That error would usually come from a locale issue. What do you currently have set in your config/app.php locale variable?

config/app.php locale variable are same as before i haven't touched that file

Another thing I'm seeing in the console is that you're getting a 404 on api/posts. In a browser window, can you hit the api directly and see if you get results like these here?

this is what is got

Thank you so much
you can close this issue


Python 3.12, pyproject.toml, distutils and a new build system

Currently python-flint builds using a setup.py and also uses distutils and numpy.distutils (see #4). As of Python 3.12 the stdlib distutils module is removed and also numpy is removing its own distutils module. This means that python-flint needs to move to a new build system somehow. Also setup.py should be removed and replaced with pyproject.toml and it probably makes sense to do that at the same time.
There are a few options for build system:

Try to use setuptools instead of distutils.
Switch to meson and meson-python.
Use cmake

Various Python projects seem to be choosing one of these three options. For pure Python projects there are other options like poetry, hatch, etc. Some of those can handle extension module building but I am not sure how well any of them works. The people who seem to know most about packaging extension modules seem to suggest either meson or cmake. In particular meson is being used by SciPy now and will be for NumPy soon (if not already).
To me switching to meson seems like the best long term option. I made a start on that here:
https://github.com/mesonbuild/meson-python/discussions/410
I discussed lots of things there like editable installs etc that make things a bit more complicated but the most important thing is just to have a build system that can produce wheels either for local install or upload to PyPI. Restricting to just producing wheels means that it would probably not be too hard to get something working with meson but the work I started here just needs to be carried through:
https://github.com/oscarbenjamin/mesontest
I don't know much about how using cmake would work here or what the potential upsides/downsides are.
Potentially the quickest fix for the removal of distutils is just to replace distutils with setuptools. It is probably not hard for someone to test that:

Replace distutils with setuptools in setup.py
Fix any immediate problems until you have a local build
Push up to a PR in this repo and see if CI can build for all platforms and architectures.

One thing that might be a problem with setuptools is the use of MinGW compilers on Windows but this is tested in CI so we should know if it is working or not. My understanding is that numpy.distutils has special support for MinGW that is not in distutils or setuptools. If that means that the Windows build fails with setuptools then it probably means that python-flint just can't use setuptools.
Switching somehow to MSVC rather than MinGW would be good but I have already explored the options for doing that with GMP and I think we just have to give up on that. What might be possible is using MinGW to make a fat build of GMP and then using MSVC to compile everything else or perhaps just for the python-tools extension module. I don't know enough about MSVC to investigate that option so someone else would need to explore that. If we could get MSVC to pick up the MinGW compiled GMP then that might make it more possible to use setuptools or probably makes things easier in some sense for any of the build system choices.
Along with the new build system we should replace setup.py with pyproject.toml which is how the build system should be configured for the project in modern Python packaging. Other things can also be moved into pyproject.toml:

Build-time dependencies (gh-24) - I don't see how to add these in setup.py.
All of the cibuildwheel configuration variables that are currently in the CI config and some of the scripts in bin/.

It possibly makes sense to create a pyproject.toml before attempting to use a new build system because there needs to be a pyproject.toml in order for any standardised build system to be used and for cibuildwheel to work without setup.py.

For pure Python projects there are other options like poetry, hatch, etc. Some of those can handle extension module building but I am not sure how well any of them works.

Poetry-core has undocumented support for defining a .py file that contains an arbitrary build hook which you then use to bring your own build system (typically setuptools). I don't think it adds anything here.
Hatchling has the concept of build plugins which is sort of the same thing except documented -- you write your own build system, or wrap an existing one, and then you write code to tell hatchling that the resulting files were created.
Realistically in both cases you are going to end up using something like meson, autotools or cmake anyway. I'm skeptical that either one provides value over just using the build system directly.

What might be possible is using MinGW to make a fat build of GMP and then using MSVC to compile everything else or perhaps just for the python-tools extension module. I don't know enough about MSVC to investigate that option so someone else would need to explore that. If we could get MSVC to pick up the MinGW compiled GMP then that might make it more possible to use setuptools or probably makes things easier in some sense for any of the build system choices.

gmpy2 supports Cython by providing a C-API. I have a report that someone has successfully used MSVC to compile a Cython extension and link to the MinGW versions of GMP, MPFR, and MPC included with gmpy2. See https://github.com/aleaxit/gmpy/issues/320#issuecomment-1017948914

Note also that currently on Windows python-flint and all DLLs in the wheel link against msvcrt.dll rather than ucrt. I have an old PR https://github.com/flintlib/python-flint/pull/41 that tried to use the MinGW ucrt toolchain but somehow it didn't work. If using MSVC then either it needs to use msvcrt.dll as well or we need to figure out the ucrt build issue.

I have successfully built gmpy2 with (a version of) the MinGW ucrt toolchain. It appeared to work but hasn't been extensively tested.

That setup would make it so that if pip install python-flint (or pip install . etc) needs to build from source then meson could build GMP etc which is nice but not critical: we can just tell users to build the dependencies first or arrange to build them first when building the wheels (which is already how it works now with setup.py).

I include the compiled Windows dependencies in gmpy2's GitHub repository. They are not included in the standard source distribution. To compile from source, you'd either need to install a previous version of gmpy2 or git clone ....
I don't know the impact of sharing libraries between python-flint and gmpy2 and sage-math and ??? I suspect there will be some subtleties. Especially with library-wide settings (MPFR and precision, user-definable memory allocation memory hooks for GMP that are automatically used by MPFR and MPC, should mpbitcnt be long or long long on 64-bit Windows). And then add a future with no GIL and multiple interpreters.
Slight different topic, GMP 6.3 with support for Mac OSX and some other enhancements has just been released. Are you planning to use that version? I think an update to MPFR is also planned.

@oscarbenjamin I think your mesontest is pretty close to the finish line already, right? With the DLL loading on Windows being the main issue to the immediate problem at hand of being able to publish working wheels for Python 3.12?
There are more topics discussed in https://github.com/mesonbuild/meson-python/discussions/410 that will require time to unravel and make polished, but being able to vendor a DLL and load it on import should be doable.

I don't know the impact of sharing libraries between python-flint and gmpy2 and sage-math and ??? I suspect there will be some subtleties. Especially with library-wide settings (MPFR and precision, user-definable memory allocation memory hooks for GMP that are automatically used by MPFR and MPC, should mpbitcnt be long or long long on 64-bit Windows). And then add a future with no GIL and multiple interpreters.

I'm really not sure how viable this is and I don't know enough of the internals of GMP and MPFR here to know what the problems might be. Specifically in python-flint's case MPC is not used and MPFR is not used directly but somehow Flint uses it and I am not sure what parts are used or what they are used for.
It would definitely be nice to be able to share work here and avoid duplicating shared libraries at runtime if e.g. gmpy2 and python-flint are used simultaneously. Being constrained to support the same versions of GMP and MPFR as gmpy2 would not be a problem for python-flint as long as gmpy2 is managing any patches or whatever are needed for them. If gmpy2 could handle the GMP build so that python-flint could use MSVC then that would be very nice.
I think that Flint itself requires that GMP be built with 32 bit limbs on a 32 bit system and 64 bit limbs on a 64 bit system regardless of OS or sizeof(long) etc. So if Flint is there than I think that some ways of building GMP are ruled out.

Slight different topic, GMP 6.3 with support for Mac OSX and some other enhancements has just been released. Are you planning to use that version?

It is already used in the 0.4.x releases:
https://github.com/flintlib/python-flint/blob/71bf11054576978a6bdb48cf3d054d3af301e765/bin/build_variables.sh#L16-L21
YASM and MPIR are not used but the other versions listed there are what is built and bundled in the wheels.

I think your mesontest is pretty close to the finish line already, right?

Yes, I think so. Someone just needs to take the time to add all the pieces, test and fix on every OS, make it all work in CI, update dev docs etc. If no major problem emerges during that then hopefully it is just a question of someone finding the time to work it through (I don't have time right now).

With the DLL loading on Windows being the main issue to the immediate problem at hand of being able to publish working wheels for Python 3.12?

The immediate problem is definitely solvable if only in a hacky way. Ultimately we are talking about building one single extension module. All distutils is really doing for us putting together a single command line call to gcc. Unfortunately cibuildwheel hides that command line but it is just something like:
gcc -I python_include_dir src/pyflint.c -o build/flint/_flint.pyd

If we really have to then it is not hard to manually reproduce everything that distutils does in python-flint's case.

Now that Python 3.12 is released this becomes more urgent.

Latest update. I have compiled GMP, MPFR, and MPC using mingw64 and linking them to mscvrt. I then followed the documentation at https://github.com/GBillotey/Fractalshades to create a MSVC compatible .lib files. I was able to compile gmpy2 with MSVC and link to the DLLs compiled by mingw64. The gmpy2 tests passed.
There are more tests that I want to run. I plan to bundle all the required files into the Window binary wheels so it should be as simple as install gmpy2 and using MSVC.
I modify the GMP to make mp_bitcnt_t == long long' instead of long. This provide 64-bit addressing for bit indexing but does impact external code that assumes mp_bitcnt_tislongon Windows. The GMP docs do say thatmp_bitcnt_tcould becomelong long` in the future. Does this matter to flint?
Cython files should also be compatible with the GMP, MPFR, and MPC libraries.
Could this be a viable solution for flint?

I compiled gmpy2 with MSVC and linked to the DLLs compiled by mingw64. The gmpy2 tests passed.

Nice!
Do you have this running in CI at all or is it just done locally?

I modify the GMP to make mp_bitcnt_t == long long' instead of long

Currently python-flint has:
https://github.com/flintlib/python-flint/blob/dbbf8bbb597c8e2dbbfe083f71976620a463627b/src/flint/flintlib/flint.pxd#L31
So I guess some changes would be needed in some places to build against that.

Could this be a viable solution for flint?

Maybe. I want everything running in CI so we will need to figure out how to use MSVC in CI. Probably that's just a case of removing the hacky workarounds that patch in mingw64 though:
https://github.com/flintlib/python-flint/blob/dbbf8bbb597c8e2dbbfe083f71976620a463627b/bin/cibw_before_all_windows.sh#L8-L14

Looks like these are the instructions for turning mingw64 DLL into MSVC files:
https://stackoverflow.com/questions/9946322/how-to-generate-an-import-library-lib-file-from-a-dll

In gh-100 I found that it was possible to use setuptools with python-flint but that setuptools and MinGW did not work for Python < 3.12. There is some explanation in this Cython issue: https://github.com/cython/cython/issues/4470#issuecomment-1361506709
That was enough though to get out a new release today of python-flint 0.5.0 with wheels for 3.12 because we already have a working setup for Python < 3.12. Basically we now have:
https://github.com/flintlib/python-flint/blob/9252d179da5116f70b5449c44408768b3b73e183/setup.py#L9-L19
I would still like to solve the other problems though:

I think it would still be better to build python-flint on Windows with MSVC.
python-flint needs a pyproject.toml including with build dependencies specified.
I think that it would still be better to move to meson-python rather than setuptools so that we can build Flint built as part of installing python-flint from source.

Do you have this running in CI at all or is it just done locally?

@oscarbenjamin, unfortunately, it's not automated yet.  Our CI has minimal M$ testing: https://github.com/aleaxit/gmpy/blob/ac1e151b8a818c32c04bf0cb9c9ee270b81b9471/.github/workflows/pip_install_gmpy2.yml#L81-L106
Also, wheels from the build_wheels.yml aren't used for PyPI (hence we have https://github.com/aleaxit/gmpy/issues/438).

Maintainer of Hatchling here! Let me know if I can help at all

An update on this:
As of gh-108, gh-110 and gh-112 python-flint now has a pyproject.toml with configuration for setuptools and cibuildwheel.
In gh-129 I have a working meson-python build to replace the current setuptools build.
Building with meson-python and pkgconfig is nice because it provides proper support for detecting versions of the C libs and setting up paths without so much manual modification of environment variables like LD_LIBRARY_PATH. It also transparently handles building with MinGW when run from an msys2 environment as is done when building the Windows wheels.
The meson-python build also provides fast parallel builds with the ninja backend as well as fast incremental rebuilds for a development checkout. With meson-python we can have an editable install that detects changes in the Cython code and performs a fast rebuild automatically when flint is imported from Python. For example (with gh-129):
$ rm -r build # delete this when setting PKG_CONFIG_PATH
$ PKG_CONFIG_PATH=$(pwd)/.local/lib/pkgconfig pip install --no-build-isolation --editable .
Obtaining file:///home/oscar/current/active/python-flint
  Checking if build backend supports build_editable ... done
  Preparing editable metadata (pyproject.toml) ... done
...
Successfully installed python-flint-0.6.0
$ time python -m flint.test -qt
Running tests...
flint.test: all 49 tests passed!
...
real	0m0.826s
$ vim src/flint/types/fmpz.pyx # edit the code
$ time python -m flint.test -qt # automatic rebuild
Running tests...
flint.test: all 49 tests passed!
...
real	0m9.122s

The only outstanding problem with gh-129 is that pkgconfig does not work with Flint < 3.1.0 because the flint.pc file was incorrect. That is a bug that is fixed in Flint 3.1.0 (https://github.com/flintlib/flint/pull/1647) so python-flint should either:

Drop support for Flint < 3.1.0
Find a workaround for the buggy flint.pc

Otherwise gh-129 just needs docs and then the next steps would be deciding how to handle building the dependencies (https://github.com/mesonbuild/meson-python/discussions/410#discussioncomment-8900296) and probably making a development frontend with spin:
https://github.com/scientific-python/spin

The only outstanding problem with gh-129 is that pkgconfig does not work with Flint < 3.1.0 because the flint.pc file was incorrect.

This problem is fixed: https://github.com/mesonbuild/meson-python/discussions/607
Many thanks to @rgommers, @eli-schwartz, and other meson-python folks for helping me to understand meson and get python-flint's meson-python configuration working!

Fantastic work! Very exciting to see this reach fruition -- I'm delighted to have been of help.

There were two outstanding items I wanted to address before closing this issue:

Coverage measurement of Cython code (https://github.com/cython/cython/issues/6186, https://github.com/scientific-python/spin/issues/182).
Documenting the new development workflow.

The coverage issue was fixed locally in gh-188 although it would be better to upstream it. At least part of it got merged so far (https://github.com/cython/cython/pull/6341).
The workflow documentation was added in gh-208:
https://python-flint.readthedocs.io/en/latest/workflow.html
Everything else is fixed:

We have a new build system (meson)
Better compiler support, detection of library versions, faster parallel builds etc.
We have a development workflow that is reasonable using spin and meson.
We have pyproject.toml and build requirements plus version constraints.
Python 3.12 is fine and 3.13 seems to be fine as well (first time any of python-flint's dependencies has given us a non-breaking release!).

I keep discovering other things that have auto-magically fixed themselves somehow since using meson like now building with PyPy just works or this horribleness is apparently not needed any more:
https://github.com/flintlib/python-flint/blob/a217c71a9b6fcc81aa5fda04830aca5fb8e7388a/bin/cibw_before_build_windows.sh#L13-L42
I haven't checked yet but it wouldn't surprise me if building with MinGW UCRT works now or that building dependencies with UCRT works and then building python-flint afterwards with MSVC works or something. (These are both things that I tried and failed at before, along with building for PyPy.)
Thanks everyone! I'm going to close this now.


chore(avm-simulator): track new ulogs and messages in trace alongside other side-effects

#6748  👈
master

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

Join @dbanks12 and the rest of your teammates on  Graphite

stale


Find an alternative to Heroku

On Nov. 28 2022, Salesforce stopped providing free Heroku services.
Need An alternative for Heroku on GCP, Azure, AWS, etc.

https://www.techtarget.com/searchsoftwarequality/news/252524336/Heroku-to-end-free-tiers-creating-platform-void-for-devs
Heroku to end free tiers, creating platform void for devs
Heroku will cease offering its free tiers this November, leaving developers to choose from an array of alternatives that don't quite match up, according to industry experts.


Switch Right and Left Menu sides

I'm trying to switch the left menu, to pop out on the right side (instead of on the default left side).  Can you please advise on how to adjust the code to do this.  Thanks.

I am facing this issue. Please reopen. I want to give RTL support to my application and SWReveal had only left when in english.

Hi, Can you please reopen this. I am having this issue too. Need to support RTL languages with the Swrevealcontroller.

Try with https://github.com/iDevelopper/PBRevealViewController

Hi iDeveloper, I am not currently able to give as much support on the class as I used to do (health issues), but please stop pushing your own controller for developer code issues that are anyway not related with the SWRevealViewController. This controller is very well supported on the web and used by thousands, so it's not right that you just tell people to replace it by your own. If you really think that yours is superior at least give some credit to me, as your code is blatantly based on my code.

Hi @John-Lluch , I am very sorry about this. I don't think mine is superior. But I saw that you will not answer to issues anymore. I will add some credit to your controller I use in some of my applications. I hope that health is better, this is the most important. Thanks a lot.

Thanks for your reply. Having choice is good, so your controller is very welcome. My point is just that some of the developer issues are not this controller fault. Regards.

Thank you guys both for your efforts. Appreciate the work you have done, and all the help.


ticdc(redo, sink): return correct error in redo writer & fix default retryer

What problem does this PR solve?

Issue Number: close #11744
What is changed and how it works?
This issue is introduced in #11011. If redo encoding worker closed since context canceld(ref the fix), the external module is not aware of the internal error anyway. Then the checkpoinTs/resolvedTs may continue to advance normally even if the redo writer has been closed.
Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`Fix a bug where the redo module could not report errors correctly`.

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest


[5.8] Handle DATABASE_URL env variable for database connection

Hi,
Many cloud providers use an url env var to configure the database connection (often DATABASE_URL), and many ORM (in many languages) handle them too.
The PR aims to handle theses url.
This is inspired by the doctrine way to handle them.
This is just a draft, coming directly from my projects, feedback and comments are welcomed.
If someone in the Laravel core team confirms me that this is a wanted feature and that it will be merged, I will be glad to spend more time on it.
Roadmap:

[ ] Add tests
[ ] Edit database.php config file to use that class
[ ] Add a redis part
[ ] See something else?

Ideas for usage (to challenge, could be easier):
// database.php
[
	'default' => env('DATABASE_URL') ? 'url' : env('DB_CONNECTION', 'pgsql'),
	'connections' => [
		// ....
		'url' => UrlParser::parse(env('DATABASE_URL')),
	],
	// ....
]

Thanks,
Matt'

Which cloud providers?

Heroku does this, for example

Please remove all the nullable type-hints. They are useless. Thanks.

@taylorotwell Sorry I don't understand what you mean: do you want that I remove all the type-hint or just the nullable part?
1 - ?string  $alias => string $alias
2 - ?string  $alias => $alias
Because in the first case, it will not work as I'm actually waiting for a nullable string, and doing checks on the null value.
Do you confirm that you're interested by adding and merging this feature, and so I can invest more time on it? If you need more examples of cloud providers, or ORM, I can look for and link them to you.

So does Heroku not provide the traditional host name / port? You can only use this specifically formatted URL?

I'm fine in general with supporting it if Heroku provides no other way to connect to databases.

This url is pretty standard, and heroku (for example) set this url in your project env variables, and rotate it automatically.
It's also easier to manage in other hosting ways, CI, and local projects because it's only one variable to set/copy instead of several.
I've seen Rails use that one too.
And icing on the cake, database software handle it to directly open your database on a click 😉!

Is there somewhere the format this URL is standardized / documented?

I didn't found anything really agnostically standardized, but each language/framework has its documentation:

https://www.postgresql.org/docs/current/libpq-connect.html#LIBPQ-CONNSTRING
https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-reference-jdbc-url-format.html
https://www.doctrine-project.org/projects/doctrine-dbal/en/2.9/reference/configuration.html#connecting-using-a-url
https://eggerapps.at/postico/docs/v1.5.6/postico-url-scheme.html
https://node-postgres.com/features/connecting#connection-uri

At the end, it's the classic url format (same than for http and ftp protocoles or any software)
scheme://username:password@host:port/path?query
so in a database context:
driver://username:password@server:port/database?additional_parameters
Examples:
mysql://localhost:4486/foo?charset=UTF-8
pgsql://user:secret@localhost/mydb
sqlite:///somedb.sqlite
The behavior will be clearer when test will be added.

Gotcha.

Feel free to add tests.

Closing pending inactivity but I do think this is kind of a nice idea if anyone else wants to pick it up.

Hey folks, I've made another PR : https://github.com/laravel/framework/pull/28308!


[byte_slice] Clean up docs and methods

Makes progress on #1692

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

Project coverage is 89.05%. Comparing base (782c12f) to head (26fe155).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1700      +/-   ##
==========================================
+ Coverage   88.98%   89.05%   +0.06%
==========================================
  Files          16       16
  Lines        5830     5818      -12
==========================================
- Hits         5188     5181       -7
+ Misses        642      637       -5

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


Add more status methods

Add #closing?, #closed? and aliases #open? -> #connected?.
@arempe93

Changes Unknown when pulling 43a5eb41776fa571077bfe2c99ceb02ed1f5bd34 on syndbg:add_more_status_methods into ** on arempe93:master**.

hey @syndbg, thanks for the contribution!

@arempe93 No probs. Thank you for the fast review. Btw, are you going to release a new version in rubygems?

I will release a patch version when I get the chance today
On Mon, Mar 28, 2016, 10:25 AM Anton Antonov<EMAIL_ADDRESS>wrote:

@arempe93 https://github.com/arempe93 No probs. Thank you for the fast
review. Btw, are you going to release a new version in rubygems?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/arempe93/bunny-mock/pull/6#issuecomment-202415109


fix(batch-write-single-table.request): unprocessedItems check

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

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update page weights for content/en/docs/concepts/architecture

contributes to https://github.com/kubernetes/website/issues/35093
@reylejano @natalisucks @bradtopol
Updates page weights in content/en/docs/concepts/architecture

/lgtm
/approve

Hi @a-mccarthy, looks like the deploy preview failed. Sometimes making an empty commit will kick Netlify and make it retry.


Cache encoded X509CRL to allow disk persistence

Handles https://github.com/Jasig/cas/issues/522

An alternative that is much more straightforward would be caching the encoded bytes of the CRL, which is accessible via X509CRL#getEncoded(). Since byte arrays are perfectly serializable, this would work for both memory and persistent caches.

Looks good. +1


ESP-Modbus不使用定时器进行3.5T符号时间时第一个数据包没有回复 (IDFGH-12105)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
V5.1
Espressif SoC revision.
ESP32-S2
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
CMD
Development Kit.
Custom Board
Power Supply used.
USB
What is the expected behavior?
失能Modbus stack use timer for 3.5T symbol time measurement后，modbus功能应该不受影响
What is the actual behavior?
失能Modbus stack use timer for 3.5T symbol time measurement后，esp32-s2上电后主机发送的第一个modbus数据包没有回复
Steps to reproduce.
失能Modbus stack use timer for 3.5T symbol time measurement后就会出现这个问题
Debug Logs.
No response
More Information.
最终定位到mbrtu.c文件中的 void eMBRTUStart( void )函数
`void eMBRTUStart( void )
{
ENTER_CRITICAL_SECTION(  );
/* Initially the receiver is in the state STATE_RX_INIT. we start
* the timer and if no character is received within t3.5 we change
* to STATE_RX_IDLE. This makes sure that we delay startup of the
* modbus protocol stack until the bus is free.
*/
eRcvState = STATE_RX_INIT;
vMBPortSerialEnable( TRUE, FALSE );
vMBPortTimersEnable(  );
EXIT_CRITICAL_SECTION(  );

}将其修改为void eMBRTUStart( void )
{
ENTER_CRITICAL_SECTION(  );
/* Initially the receiver is in the state STATE_RX_INIT. we start
* the timer and if no character is received within t3.5 we change
* to STATE_RX_IDLE. This makes sure that we delay startup of the
* modbus protocol stack until the bus is free.
*/
eRcvState = STATE_RX_INIT;
vMBPortSerialEnable( TRUE, FALSE );
#if CONFIG_FMB_TIMER_PORT_ENABLED
vMBPortTimersEnable(  );
#else
pxMBPortCBTimerExpired();
#endif
EXIT_CRITICAL_SECTION(  );
}`
我只测试了modbus rtu功能，但我想在ascii模式下应该也有这个问题

@Zhicon ,
Thank you for the issue.
The Modbus RTU starts from STATE_RX_IDLE stage and it skips the incoming data received before the 3.5T.

失能Modbus stack use timer for 3.5T symbol time measurement后，modbus功能应该不受影响
失能Modbus stack use timer for 3.5T symbol time measurement后，esp32-s2上电后主机发送的第一个modbus数据包没有回复

When the CONFIG_FMB_TIMER_PORT_ENABLED is not enabled the stack ignores the first incoming packet. This behavior is confirmed as incorrect.
Your proposed code can be used to handle the Init state correctly.

我只测试了modbus rtu功能，但我想在ascii模式下应该也有这个问题

The ASCII mode has different logic and is able to start from IDLE state and can process the first packet correctly.
The fix is planned to be implemented in next release.

The issue solved with commit 8165909be4328fd6ce14e94dbd4932bf2a0ec4a9.


src/qt/README.md is unclear/incomplete regarding "Setup Qt Creator"

I'm going through the gui/src/qt/README.md to set up a development environment in Ubuntu.
I had 2 issues:

I got errors for every "#include <qt/. . .>". I believe this is because creator has no sense of where the qt directory is when it's the absolute path of the project (step 4), as the root directory of the project is named "bitcoin-qt". Removing "qt/" from relevant includes, or importing the project with absolute path of gui/ or gui/src resolves this.

I got "file not found" errors for all of the Qt core #includes (QApplication, QDebug, QSettings, etc). I resolved this by adding "QT += core" to my project file. Perhaps you could provide an example project file for beginners.

For what it's worth, I had identical behavior in Arch Linux.

cc @jarolrod

Just went through the whole build process and qtcreator set up on a clean install of Ubuntu 20.04.1. Tested that I can run the executable from qtcreator, I can make changes and build with qtcreator. Cannot replicate the errors described.

Here are some notes on setting up the project in qt creator:

you must build bitcoin before importing the project into qtcreator
when building bitcoin and you get to the configure stage, add the --enable-debug flag
At Step 4, you must enter the absolute path to the qt director: /home/{USER}/gui/src/qt
Leave the default include files as is! You may need to re-select the forms folder to ensure the ui files are included (if you wish to edit those), that's all.
Make sure that you go to Projects -> Manage Kits and verify that the correct values are selected as described in step 7

Performing these steps should get you up and running. After verifying these steps, please let us know if this is still a problem.

I tried with a clean install of Ubuntu 20.04 (virtualbox 6.1.18). I am able to build and run without issues in qtcreator. However, I have the same errors for almost every include:

This is my full system history:
   1  reboot     2  sudo apt install gcc make     3  cd /media/ben/     4  ll     5  cd VBox_GAs_6.1.18/     6  ll     7  ./VBoxLinuxAdditions.run      8  sudo ./VBoxLinuxAdditions.run      9  shutdown now    10  sudo apt install git    11  git clone git@github.com:bitcoin-core/gui.git    12  git clone https://github.com/bitcoin-core/gui.git    13  cd gui/    14  ll    15  sudo apt-get install build-essential libtool autotools-dev automake pkg-config bsdmainutils python3    16  sudo apt-get install libevent-dev libboost-dev libboost-system-dev libboost-filesystem-dev libboost-test-dev    17  sudo apt install libsqlite3-dev    18  sudo apt install libminiupnpc-dev libnatpmp-dev    19  sudo apt-get install libzmq3-dev    20  sudo apt-get install libqt5gui5 libqt5core5a libqt5dbus5 qttools5-dev qttools5-dev-tools    21  sudo apt-get install libqrencode-dev    22  ./contrib/install_db4.shpwd   23  history    24  export BDB_PREFIX='/home/ben/gui/db4'    25  ./autogen.sh    26  ./configure BDB_LIBS="-L${BDB_PREFIX}/lib -ldb_cxx-4.8" BDB_CFLAGS="-I${BDB_PREFIX}/include" --enable-debug    27  make -j6    28  sudo apt install qtcreator    29  qtcreator &
After that, import existing project, title "bitcoin-qt", path /home/ben/gui/src/qt. Selected the forms folder and added to git. Compilers (GCC), debugger (GDB) and executable are set.
This is what my end of my ./configure output looks like:
`Build Options:
with ecmult precomp     = yes
with external callbacks = no
with benchmarks         = no
with tests              = yes
with openssl tests      = no
with coverage           = no
module ecdh             = no
module recovery         = yes
module extrakeys        = yes
module schnorrsig       = yes
asm                     = x86_64
bignum                  = no
ecmult window size      = 15
ecmult gen prec. bits   = 4
valgrind                = no
CC                      = gcc
CFLAGS                  = -O2 -fvisibility=hidden -std=c89 -pedantic -Wall -Wextra -Wcast-align -Wnested-externs -Wshadow -Wstrict-prototypes -Wundef -Wno-unused-function -Wno-long-long -Wno-overlength-strings -W -g
CPPFLAGS                =
LDFLAGS                 =
Options used to compile and link:
external signer = no
multiprocess    = no
with libs       = yes
with wallet     = yes
with sqlite   = yes
with bdb      = yes
with gui / qt   = yes
with qr       = yes
with zmq        = yes
with test       = yes
with bench      = yes
with upnp       = yes
with natpmp     = yes
use asm         = yes
ebpf tracing    = no
sanitizers      =
debug enabled   = yes
gprof enabled   = no
werror          = no
target os       = linux
build os        = linux-gnu
CC              = gcc
CFLAGS          = -pthread -g -O2
CPPFLAGS        =  -DDEBUG -DDEBUG_LOCKORDER -DABORT_ON_FAILED_ASSUME   -DHAVE_BUILD_INFO -D__STDC_FORMAT_MACROS -DPROVIDE_FUZZ_MAIN_FUNCTION
CXX             = g++ -std=c++17
CXXFLAGS        =  -O0 -g3 -ftrapv  -fstack-reuse=none -Wstack-protector -fstack-protector-all -fcf-protection=full -fstack-clash-protection  -Wall -Wextra -Wformat -Wformat-security -Wvla -Wswitch -Wredundant-decls -Wunused-variable -Wdate-time -Wsign-compare -Wduplicated-branches -Wduplicated-cond -Wlogical-op -Woverloaded-virtual -Wsuggest-override  -Wno-unused-parameter -Wno-implicit-fallthrough -Wno-deprecated-copy    -fno-extended-identifiers
LDFLAGS         = -lpthread  -Wl,-z,relro -Wl,-z,now -Wl,-z,separate-code -pie
ARFLAGS         = cr`

In project->manage kits, what do you see under Qt versions?

Under Manage Kits...->Kits->Desktop (default)-> Qt version: Qt 5.12.8 in PATH (System)
There's also an option for Qt 5.12.8 in PATH (qt5)

@benliepert confirming I can replicate the issue with includes, all of the errors are corrected by using gui/ as root instead of qt/ as root

I'm editing my initial comment to avoid confusion, as I was wrong about resolving #2: it seems some Qt constructs aren't found/known but I can't see a consistent pattern

@hebasto I think this can be closed.
Upon the removal of the bitcoin-qt.pro file in https://github.com/bitcoin/bitcoin/pull/21695, it can be said that we do not actively promote the use of Qt Creator as an IDE. That being said it does provide a convenient way to make changes to the *.ui files, and the current instructions will get you to a point where you can do so.


attempt to detect buffer overflows in Texture::setImage()

We verify that the buffer given to setImage() is at least as large as needed for the given region to transfer; at least based on the size given.
This might help catch b/330407429.

We verify that the buffer given to setImage() is at least as large as needed for the given region to transfer; at least based on the size given.
This might help catch b/330407429.

You might want to add in the commit message
BUGS=330407429
so that it'll post to the bug (but not close it).


Creation of Webserver code

This code is a bit of a beta and was brought in and corrected from here a different fork.  I will try to clean it up more but for now openchats (plural) is the one that is working and I will try to do work on openchat webserver later.  I also created an example for gptneo-xlarge.

Thank you for PR.
I want to talk about this framework with you.
Could you tell me your email?

Sure, haroldhannon at gmail dot com.  I am avoiding the bots :D

Did you get my mail?


Added docs about the PokemonHunter

Added missing docs about the Pokemon Hunter

@pogarek 😄

👍

merged #5941

done. I need to read about hunter - not using it now. :)

It's a kind of "poor mans sniper" 😉

:-) it is supposed to replace another task ? or as additional one ?

It's an additional one. I think it was created when the "nearby pokemon" were availible in the game. This task will use that data to look for Pokemon around the current location of the bot. Seems to work as a charm!

I'll test it then. thanks

Yup. Has been using it since day one when someone implemented it.


Command died with <Signals.SIGPIPE: 13> message

I am using SSM plugin as proxy command for SSH to execute commands on EC2 instances. This is mostly used in jenkins pipelines.
Latest version of SSM plugin (session-manager-plugin-1.2.398.0-1.x86_64) is running on Centos7, installed with official rpm.
Occasionally (pretty rarely), I get this in output:
Command '['session-manager-plugin', '{"SessionId": "xxxxx", "TokenValue": "xxxxxxxxx", "StreamUrl": "wss://ssmmessages.eu-west-1.amazonaws.com/v1/data-channel/xxxx?role=publish_subscribe&cell-number=xxxxx", "ResponseMetadata": {"RequestId": "xxxx", "HTTPStatusCode": 200, "HTTPHeaders": {"server": "Server", "date": "Tue, 10 Jan 2023 12:10:51 GMT", "content-type": "application/x-amz-json-1.1", "content-length": "1059", "connection": "keep-alive", "x-amzn-requestid": "xxxx"}, "RetryAttempts": 0}}', 'eu-west-1', 'StartSession', 'xxxx', '{"Target": "i-xxxxxx", "DocumentName": "AWS-StartSSHSession", "Parameters": {"portNumber": ["22"]}}', 'https://ssm.eu-west-1.amazonaws.com']' died with <Signals.SIGPIPE: 13>.
Command doesn't fail, just getting this message where http status code is 200.
Any idea what's causing this? Or any way to hide the message (since it reveals some sensitive data) if it's not causing any issue?

Hi, I had a similar problem using the SSM plugin (1.2.463.0-1) as ProxyCommand for SSH to connect on EC2 instances.
To use ProxyCommand, i had to execute my session manager command inside quotes, like that :
ProxyCommand sh -c "aws ssm..."
I sometimes had to use other quotes inside these commands, it is possible that these quotes create execution problems.

I had the same problem, started to work after i allowed all protocols for egress in the security group of the instance.


Gracefully handle office locations w/ no names

Sometimes the office names aren't present in the API response for the House.  Just need to check for this so we don't see something like:  "222-333-4567 ()"

Do we have any examples?

It's dependent on the data, but try to find an issue with a house representative.  I found one w/ zip 77429 and I posted the picture to slack.
Sent from my iPhone

On Mar 8, 2017, at 1:03 PM, Ben Gottlieb<EMAIL_ADDRESS>wrote:
Do we have any examples?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hmm, every issue I'm seeing is either Senate- or Governor-only, or just hits a House subcommittee.

Fixed by #122


Problem when running inside cordova

I have no clue why, but the following image does not get any exif info when used inside cordova.
When using the same code on desktop browser the data is fetched correctly.

How can I debug this?

Seems like this was not related to this library after all... Closing.

@HarelM can you share how to fix your problem? because i also using cordova.

The problem was a bug in a cordova plugin where the exif information was lost when using the gallery.
The following line solved it:
https://github.com/IsraelHikingMap/Site/blob/e38e4b47d333bb86168e02b2731d4b10556a6b43/IsraelHiking.Web/sources/application/directives/image-capture.directive.ts#L65
Relevant issues:
https://github.com/IsraelHikingMap/Site/issues/932
https://github.com/apache/cordova-plugin-camera/issues/329


birth and death simulation

We might want to come up with a general way to do birth and death in simulations. This overrides the Simulation.run method to implement that.

Pull Request Test Coverage Report for Build 43

4 of 6 (66.67%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 86.508%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

dworp/environment.py
3
5
60.0%

Totals

Change from base Build 42:
-0.5%

Covered Lines:
218

Relevant Lines:
252

💛  - Coveralls


Fix #36: Prevent Invalid System/AuxiliaryCode Calls

Fix to Issue #36

Added 2 PrereqCheckFailure types to account for invalid calls to the System and AuxiliaryCode classes
Added new feedback for these new PrereqCheckFailure types in FeedbackGenerator
Added relevant tests

Had an idea about making this more sustainable. Will work on it more so it's not ready for review yet.

OK I've made some adjustments but I'm not sure they're actually all that effective or helpful in terms of scalability for how things are right now. I'm open to suggestions! Otherwise, this should be ready for review now.


Not keeping the reference to the Disruptor prevents cleanup when resetting Harness client

https://github.com/harness/ff-dotnet-server-sdk/blob/0528d4296dc7c40dd625db0fcb3712dbf71a85c2/client/api/analytics/AnalyticsManager.cs#L100
If we create a new CfClient to reset everything, the long-running event processor associated with the Disruptor created here keeps running. It won't get any new events, but it can't be cleaned up by GC. Ideally the reference to the Disruptor would be kept so when MetricsProcessor.Stop is called, it can call the Disruptor Halt method.

RingBuffer code was removed in 1.1.8, closing ticket


[EWT-278] Build also against JDK 20

Description
This PR is about building also against the latest available JDK version (20).
As part of these changes, I'm switching to the Eclipse Temurin distribution as suggested here.
I won't create a release for this change.
Type of change
Please select multiple options if required.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code where necessary
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the relevant documentation

LGTM 👍
Could you just please merge the main branch?


Stores double quotes as part of a string

In one part of my app I am storing a string in localStorage to correctly set the app theme on page load
const newTheme = !theme || theme === 'default' ? 'night' : 'default';
store.set('fccTheme', newTheme);

When I come to retrieve the theme from localStorage in a script tag I have to do
if (typeof localStorage !== 'undefined') {
    const fccTheme = localStorage.getItem('fccTheme');
    if (fccTheme === '"night"') {
      document.body.classList.add('night');
    }
   }

From what I can work out
const fccTheme = localStorage.getItem('fccTheme');
typeof fccTheme === 'string' // true
fccTheme === 'night' // false
fccTheme == 'night' // false
fccTheme === '"night"' // true

Hi!
LocalStorage only stores string values (it can't store numbers, objects, arrays, etc). One of the key benefits of store.js is the ability to store any type of value, and it does it using JSON to serialize values to a string (which is then stored in the underlying localStorage, sessionStorage, etc). It also has the benefit of using a storage mechanism that works across all browsers, which means that your site will work properly in eg IE7, FF4 and Safari Private mode, whereas your current approach would break in all those cases.
So what you should do is use store.get("fccTheme") instead :)
If you insist on accessing it using localStorage directly, you should definitely set the value using localStorage directly as well.
I also suggest using the new v2.0 version which supports more browsers and has added functionality: https://raw.githubusercontent.com/marcuswestin/store.js/v2-dev/dist/cross-browser.min.js
Cheers!
Marcus