Feature Request: Add some info to the DDNS widget - hostname.domain.com

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
The feature is not realated to a problem. Just a couple of suggestion to try and improve the widget.
Describe the solution you like
This is two fold:

Add the actual hostname and domain of the DDNS that is registered to the widget. ie. add hostname.no-ip.org. I know some services authenticate with a username rather than a hostname, so, maybe add text box to the Dynamic DNS: Settings, so, the hostname.domain.com could be entered and displayed on widget, just like the IP address is.

For those of us that are using MORE than one DDNS service, we're not able to resize the individual boxes within the widget to minimize the blank space. We can resize the overall widget but not the smaller rectangles.

The pic should illustrate what I mean:

A clear and concise description of what you want to happen.
I think I mentioned that above...but I am available to clearing if needed.
Describe alternatives you considered
I haven't tried any alternatives.
Additional context

In my original post above, I used FF (version 130.0)
However I tried Chrome (version 128.0.6613.120) and it formats "correctly" with no wasted space.

So, looks like a "bug" when displaying in Firefox?
Still would like to see an option to display the hostname and domain of each DDNS.

You can if you expand the size of the widget. Then a new row with the hostnames appear.

You can if you expand the size of the widget. Then a new row with the hostnames appear.

damn...didn't even know that was hidden there! Thanks for that.
I'd still like see the host name be brought into sight when the widget is 1 column.
Also, for example, services like HE.net use a username... and duckdns use the hostname, it'd be nice if I could add some text that would include the FULL hostname and domain at that service.

Im sure there are a lot pf dyndns services who all do things a bit different.
The scope of the widget was to show the API output of the ddclient. There are probably many other configurations where the API output is incomplete and the widget will show incomplete data.
The domains are hidden when not expanded because I thought it would be nice having a small widget that only shows the most crucial information if not expanded.
Of course, if you have any suggestions here, PRs to improve the widget are welcome. Without having the dns providers to test, its impossible to improve the widget more.

@Monviech For sure, I'd like to see the hostname.domain.com displayed with the crucial info. IMO, that's pretty crucial info. Also, not sure why FF and Chrome are displaying the widget slightly differently, FF seems to have more wasted space, and Chrome is displaying how I think you meanted it to be display, concise and tight.

@Monviech For sure, I'd like to see the hostname.domain.com displayed with the crucial info. IMO, that's pretty crucial info. Also, not sure why FF and Chrome are displaying the widget slightly differently, FF seems to have more wasted space, and Chrome is displaying how I think you meanted it to be display, concise and tight.

The info in the widget is also incorrect if you fail over to another host with a different ip for the wan (no wan carp virtual ip). the other host may update that dns.  when the other node comes back up and pre empts, it never checks or updates the ip.  it just assumes what it updated last was what the value is.  is there a way to force an update when switching to master?  i tried force from command line and it still doesn't update


Look at N-API to see if it would help our binding portablitiy

Still experimental with 8, but it looks like they're trying to integrate the Nan approach (with some other stuff) into the platform https://nodejs.org/api/n-api.html
/cc @mgol  and @saper who were talking about this in Slack too

I've been following N-API closely. My bet is the that the community is blocked until nan adds support. There is currently no tracking issue with nan yet.

I do not think that squeezing N-API into NAN is possible.
I am looking right now if our low-level uv callback stuff can be converted into N-API, which offers some callback and async job handling.

N-API is meant to be mostly a replacement for NAN in that N-API is designed to be non-leaky about V8 or engine details and provide a flat, stable ABI surface. An N-API enabled native module binary shouldn't have to be re-compiled to continue working with future versions of Node.js. But I'm not going to go completely into the sales pitch here, you can read about it. 😁
We (N-API team) have built a preliminary port of node-sass on top of N-API as part of our effort to understand the API surface used by native modules. We used this port and the port of some other modules to determine which areas to focus on converting to N-API. This port has all of the NAN and V8-specific code removed and replaced by N-API equivalents.
I doubt this port meets the quality level to go in as-is, but it would certainly be a springboard if someone on node-sass was serious about doing the conversion to N-API. 😄 Please take a look if you're interested. I will be happy to answer questions about the port, help with coding it, point to documentation/code, etc.
https://github.com/boingoing/node-sass/tree/napi

@boingoing if Nan is replaced with the napi convention, would we still be able to produce builds compatible with at least the current LTS versions of Node (4+) or is this only going to work for 8+

The N-API enabled module binary can be loaded by older Node versions as well (4 and 6) by relying on an NPM package. There are more details in the documentation of the package:
https://github.com/nodejs/node-addon-api

Thank you @boingoing this is very helpful. From what I can see some libuv specific code is still left there (but that's no V8 and even not C++).
It looks fairly complete and maybe even works, nice! :)

My bet is the that the community is blocked until nan adds support.

I'm one of the developers on the N-API project. Unfortunately, NAN cannot ever support N-API directly. NAN is only a partial abstraction over the V8 APIs: when you use NAN, you're still using a lot of V8-specific types and other constructs. N-API is a higher-level, more complete abstraction: when using N-API, you shouldn't need to even include v8.h at all or reference any types in the v8:: namespace. We realize that the magnitude of the differences will make porting existing native addon code more difficult, but that was a conscious tradeoff.
We discussed the possibility of branding the N-API C++ library as "NAN 3.0" with @kkoopa, the top contributor to NAN. (He has also been reviewing some of the N-API code.). While NAN has some precedent of breaking changes between major versions, this is more of a complete redesign than a "breaking change", so that branding didn't seem appropriate. Anyway, Node.js will need to continue to support native addons that use the V8 APIs for the foreseeable future, because some addons might not ever be updated to N-API since the changes are non-trivial. NAN should continue to support those addons the same way it does now.

Sorry I was looking at an older version of this page and didn't see @boingoing's replies until after I posted mine. It looks like we've said mostly the same things!

From what I can see some libuv specific code is still left there (but that's no V8 and even not C++).

Right, at this time it is not a goal for N-API to cover all of the libuv APIs. Currently, it just covers its async APIs, partly to provide a clear migration path from Nan::AsyncWorker. It's possible N-API could expand to cover more or all of libuv in the future, but there is no immediate need, because libuv APIs are very stable (unlike V8 which tends to make breaking changes often) and libuv is still used by most other forks of Node, even when the JS VM is swapped.

Regarding the node-sass N-API fork @boingoing linked above, we might do some things differently now that the N-API project is more mature than we originally developed the fork. In particular, that fork is mostly using the C APIs, while we think most native addons will prefer to use the C++ APIs, because they are more concise and more similar to the V8/NAN style.
I'd assume you don't want to immediately take a dependency on a feature that is still has "experimental" status. The best case scenario from our point of view might be if the node-sass team could eventually maintain a branch that uses N-API, that is kept mostly up to date with development occurring in node-sass/master. I guess it could be staging for a future node-sass v5.x, or however you all want to manage it. But I understand you might not be ready for that just yet. Of course we'd appreciate feedback about anything we can do to improve the development or migration experience.

Frankly, were libsass a C library, I would be happy to switch to 100% C for portability. We have lots of reports from various users where the binding does not even load and recompiling magically helps. I find it hard to keep compatibility with C++ runtimes working for everybody alike.

Noticed this issue is still hanging-around. Is there anything we can do to help or any feedback on the particulars of the N-API?
We have a weekly working group meeting every Thursday at 1:30 Eastern / 10:30 Pacific US time if you're interested in sitting-in. We would be able to spend some time going over any issues you have adopting N-API either technical, philosophical, or whatever. 😀
Here are just a couple more notes for N-API.
We typically track N-API-specific issues in the abi-stable-node repo. Issues can also be opened in the ordinary nodejs/node repo, as well, with the n-api tag but abi-stable-node is really only used for discussions about N-API.
https://github.com/nodejs/abi-stable-node
There is official documentation for N-API in the node docs.
https://nodejs.org/dist/latest-v8.x/docs/api/n-api.html
We did mention the c++ wrapper for N-API which provides a very Nan-like interface and lives in a separate repo from nodejs/node (similar to the Nan shipping story). This can be used to bring N-API compatibility to versions of node without N-API, as well.
https://github.com/nodejs/node-addon-api

Hello, sorry for the silence. I've been on the job search and recently hurt my back 😱
I'm definitely interested in pushing this forward. We have some big changes landing in LibSass very soon which will results in node-sass@v5 and v6. I would N-API to land in node-sass@v5. I'll create a meta issue documenting the plan for v5 and v6 this weekend. I'll be sure to cc @boingoing

Now that Node 10 has been released, N-API is no longer experimental. So if WebAssembly is not the way to go, maybe we should give N-API a try.
@xzyfer Regarding your question on uv_queue_*, does this package aswer your concerns: get-uv-event-loop-napi-h?

@nschonni also mentioned using https://github.com/nodejs/node-addon-api as a N-API shim for older versions of Node.
This is certainly a step towards WASM.

For those following along at home I've picked up work on this again in #2440. It's getting pretty close (I think).
I'm hoping to get N-API support out in a v5 beta.

As previous, It seem the best time to migrate is around the corner. Node.js remove version dependencies


Azure.ResourceManager.Resources.SubscriptionData returning wrong values

I am trying to get all the subscriptions defined in my tenant. I'm using the following code:
       var armClient = new ArmClient(new DefaultAzureCredential());

        var subs = armClient.GetSubscriptions();

        await foreach (var s in subs.GetAllAsync())
        {
            Console.WriteLine($"{s.Data.Name} --- {s.Data.SubscriptionGuid}");
        }

But for some reason what is getting printed to console is:
aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee --- example.sub.1
It is returning GUID for Data.Name property and name for Data.SubscriptionGuid property.
I'm using <IP_ADDRESS>-beta.4

I believe this is fixed : https://github.com/Azure/azure-sdk-for-net/pull/25704


Redirect to home page after invalid route

Description

Right now, the page just errors with a wrong route.
That's fine, but it would be more convenient if we just redirect to the main route.

Hi! I would love to take a look on that issue if possible :)

@Dezert01 actually, this was a mistake. This has already been resolved by the linked issue. I don't know why it wasn't automatically closed. Apologies

This can be reproduced by going to antalmanac.com/added/xyz. Just any almanac.com/xyz doesn't error


tests/jdbc: Fix tests that assert negative row counts

For example testDeleteFromPartitionedTable could fail if executed with
JDBC.

Igtm


TypeError: _lodash is undefined

Hi,
when trying to run let yawn = new YAWN(yaml);
i'm getting this error TypeError: _lodash is undefined

How did you import YAWN ? I had the same issue and fix it by importing like this :
import Yawn from "yawn-yaml/cjs";

import Yawn from "yawn-yaml/cjs"; - This resolved the _loadash is undefined error
But it gives a console error saying-
ERROR in ./node_modules/yaml-js/lib/yaml.js
Module not found: Error: Can't resolve 'fs' in '/node_modules/yaml-js/lib'

I have come to a problem like this Cannot read properties of undefined (reading 'isString'), and it's came out in the construcor function.

Thanks, bro. I did as you said, and it works.

How did you import YAWN ? I had the same issue and fix it by importing like this :
import Yawn from "yawn-yaml/cjs";
Thanks, bro. I did as you said, and it works.

How did you import YAWN ? I had the same issue and fix it by importing like this :
import Yawn from "yawn-yaml/cjs";

Thanks, bro. I did as you said, and it works.

v2 should not have this problem


Update MD file on new branch 2017-14-08_04-11-36

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

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


Fix potential leak, use modules, various cleanups

Just basic maintenance.

Might help address #28.


[Bug] Double logging when get_logger(name) used in the same file

Search before asking

[X] I searched the issues and found no similar issues.

Component
Library/core
What happened + What you expected to happen
Logging of messages twice is seen.
This is due to the fact that get_logger(name) does not check for previously existing loggers and adds another StreamHandler even if the logger was pre-existing.  The work around is to not use the same name when calling get_logger().  THis is happening because many xyz_transform.py files call get_logger(name) more than once in the same file.
Reproduction script
:~/git/data-prep-kit/transforms/universal/noop/python$ make run-cli-sample
/Library/Developer/CommandLineTools/usr/bin/make RUN_FILE=noop_transform_python.py \
		RUN_ARGS=" --data_local_config \"{ 'input_folder' : '../test-data/input', 'output_folder' : '../output'}\" " .transforms.run-src-file
source venv/bin/activate;	\
	cd src;				\
	python noop_transform_python.py --data_local_config "{ 'input_folder' : '../test-data/input', 'output_folder' : '../output'}"
13:09:08 INFO - Launching noop transform
13:09:08 INFO - noop parameters are : {'sleep_sec': 1, 'pwd': 'nothing'}
13:09:08 INFO - pipeline id pipeline_id
13:09:08 INFO - job details {'job category': 'preprocessing', 'job name': 'noop', 'job type': 'pure python', 'job id': 'job_id'}
13:09:08 INFO - code location None
13:09:08 INFO - data factory data_ is using local data access: input_folder - ../test-data/input output_folder - ../output
13:09:08 INFO - data factory data_ max_files -1, n_sample -1
13:09:08 INFO - data factory data_ Not using data sets, checkpointing False, max files -1, random samples -1, files to use ['.parquet'], files to checkpoint ['.parquet']
13:09:08 INFO - orchestrator noop started at 2024-08-07 13:09:08
13:09:08 INFO - Number of files is 1, source profile {'max_file_size': 0.0007181167602539062, 'min_file_size': 0.0007181167602539062, 'total_file_size': 0.0007181167602539062}
13:09:09 INFO - Sleep for 1 seconds
13:09:09 INFO - Sleep for 1 seconds
13:09:10 INFO - Sleep completed - continue
13:09:10 INFO - Sleep completed - continue
13:09:10 INFO - Completed 1 files (100.0%) in 0.040456779797871906 min
13:09:10 INFO - done flushing in 1.5020370483398438e-05 sec
13:09:10 INFO - Completed execution in 0.0404723326365153 min, execution result 0

Anything else
No response
OS
MacOS (limited support)
Python
3.11.x
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Fixed in PR (#481


Add support for Numeric to SQLite

As far as I can tell, there is no "internal" representation of this that we can deserialize to (like we did with PgNumeric). As such, we should probably have this deserialize to a string, as that's the only type that we can reasonably implement FromSql for without losing precision. We can add ToSql impls for f32 and f64 and/or the various integer types if we think it's useful enough.

Upon further investigation, SQLite appears to have no visible difference in behavior between numeric and double precision floats. If anybody knows this to be incorrect, please let me know.


Remove syntax error in release script

I think I'll just change the release script of vscode extension and slang-torch for now.


Prepare for 8.2.0 Release

🎈 Release
Preparation for 8.2.0 release.
Comparison to 8.1.1: https://github.com/gazebosim/gz-gui/compare/gz-gui8_8.1.1...gz-gui8

Needed by <PR(s)>
Checklist

[ ] Asked team if this is a good time for a release
[ ] There are no changes to be ported from the previous major version
[ ] No PRs targeted at this major version are close to getting in
[ ] Bumped minor for new features, patch for bug fixes
[ ] Updated changelog
[ ] Updated migration guide (as needed)
[ ] Link to PR updating dependency versions in appropriate repository in gazebo-release (as needed):

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

update CMakeLists.txt and package.xml?

update CMakeLists.txt and package.xml?

I rushed to create the PR :(. Should be fixed now.

update CMakeLists.txt and package.xml?

I rushed to create the PR :(. Should be fixed now.

Probably safe to re-enable auto-merge (squash)?


Korean Docs

I want to make a document for Koreans. However, it is difficult to understand the document structure of this project. If you give me some tips, I will make it easier to create more documents.
I have previously translated electron-vue documents.
I think this project should be used by many people.
good luck!

This might be of help.


Challenge clarification for "errors" exercise

ultimate_rust2/exercise/errors
Based on the description:
        // Challenge: Change main() so that it returns a Result, and instead of handling the error
        // that play_time returns, use the try operator to only handle the success condition. How
        // does the output of the program change?

the following change is made:
fn main() -> Result<()> {
    ...
    for dolphin in &dolphins {
        let responses = play_time(dolphin)?;
        println!("{} did a FABULOUS PERFORMANCE!", dolphin.name);
        for response in responses {
            println!("  {}", response);
        }
    }
    Ok(())
}

It changes the output of the program to:
Error: The dolphin's name is too long and annoying to say
Did I understand the challenge correctly and was it the correct solution?
Or anything should be done to explicitly ignore the error Result returned by play_time() and yet to consume the Ok Result?
Please advise.

Yes, that's exactly the right output after making the change.  You are not the first person to ask this question, so I will go and update the text to make it clear that this is the expected solution!


add max unshield balance

Add maximum amount to unshield in one transaction.
Usually z-nomp users have to modify the well-known if in paymentProcessor.js
add this to config with a default value

I believe this has been removed.

i have it in here https://github.com/faneaatiku/s-nomp/blob/master/libs/paymentProcessor.js#L298
And as far as i can see, pool always shields coinbase but forgets to unshield them. this leads to a delay in paying the miners in certain cases.

Interesting. I thought someone submitted a PR to remove that. It's up to you. The best implementation would be to make it configurable. I wouldn't be opposed to removing it altogether.

Yeah. Configurable with 0 meaning no limit.


Changelog fix

Codecov Report

Merging #1717 (7376157) into main (2c50b02) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1717      +/-   ##
==========================================
- Coverage   94.08%   94.07%   -0.01%
==========================================
  Files         259      259
  Lines       49684    49684
==========================================
- Hits        46743    46742       -1
- Misses       2941     2942       +1

Impacted Files
Coverage Δ

.../termdict/sstable_termdict/sstable/block_reader.rs
80.35% <0.00%> (-1.79%)
:arrow_down:

src/store/index/mod.rs
97.83% <0.00%> (-0.55%)
:arrow_down:

src/postings/stacker/expull.rs
99.05% <0.00%> (+0.47%)
:arrow_up:

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


Fix [Feature Request] Dropdown in ButtonGroup(#12815)

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Fix [Feature Request] Dropdown in ButtonGroup(#12815)
Now ButtonGroup support Dropdown which be place as ButtonGroup's last-child.

For implement below scenario.

Demo Link


Write web server benchmarking

There is some research to be done here to find out what the best practices are for web server benchmarks and then list them here before actually running them.

There is some research to be done here to find out what the best practices are for web server benchmarks and then list them here before actually running them.

Apache Bench seems to be popular. Here are the Rust web frameworks listed by arewewebyet:

actix-web
rocket (note: only available on nightly due to 2 unstable features)
warp
tide
gotham
~iron~ seems inactive based on GitHub activity
~nickel~ same as above
~thruster~ just middleware


Rust on Windows does not add lib prefix

When I compile the library with Rust on Windows it didn't add the "lib" prefix to library name.
I tested on Linux Mint also but Rust on Linux did.
Library names:
Windows: faster_path.dll
Linux:  libfaster_path.so

Can you open up an irb window and tell me what RUBY_PLATFORM returns?

Of course. Here: x64-mingw32
Also I was fixed this issue with this sloppy line.
ffi_lib "#{File.dirname(__FILE__)}/../target/release/"  + (RUBY_PLATFORM =~ /mswin|mingw|cygwin/ ? 'faster_path.dll' : 'libfaster_path.so')

Where did you get the list for Windows platform aliases?

I believe this is the link where i get.
http://stackoverflow.com/questions/4871309/what-is-the-correct-way-to-detect-if-ruby-is-running-on-windows

Alright the master branch should now support Windows https://github.com/danielpclark/faster_path/commit/2bf47f6188b1831b926e667d977d6402dfff124b .  Can you test it before I release it as a new gem version?

No cygwin can not be a linux. It's a linux simulator on Windows.
I agree with using Gem.win_platform? but if you want to check without dependency I think you should go with RUBY_PLATFORM =~ /mswin|mingw|cygwin/. I believe these 3 alias are suffice enough.

There's no dependency required at all for Gem.  It's built in.

Oh, unless they're no MRI?

I think you should check the Ruby engine also but i don't know how FFI supports.
Here is the nokogiri's engine check line.
if defined?(RUBY_ENGINE) && RUBY_ENGINE == "jruby"

I'm not sure what the file name will be with jRuby.  I think I'd rather wait for a jRuby user to open an issue and have details available.
Looking into Gem.win_platform? I see that Jekyll https://github.com/jekyll/jekyll/issues/3721 uses it.  And one other issue raised online had something about Java detection and another person said this issue was already raised & fixed for rubygems.
I'd rather trust the implementation that's been thoroughly vetted and that way I don't have to maintain future Windows possibilities myself.  Other then that I'll wait till it becomes an issue some one needs to raise as it doesn't look like there will be a problem.

I tested on Windows with Ruby 2.3.1p112 (2016-04-26 revision 54768) [x64-mingw32] and it works as should. Thanks again.

Alright!  New gem version coming out! :smiley:


docs: add readme for github action

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Allow usage of the inject HOC outside of an  component

Summary
Allow usage of the injectStripe HOC outside of an <Elements/> component in order for components to access the Stripe client that <StripeProvider/> provides (when not using Stripe Elements).
Motivation
I'm using this library for Stripe elements in one part of my app and it's great. In another part of my app I'm using Stripe directly to create a bank account (which is not provided by Elements). I'm already using <StripeProvider/>. I know that I could access the Stripe client directly via context but I lose (or have to duplicate) the logic that deals with the context differences between sync vs async (mainly this).
Alternatives
Provide an additional inject HOC that has no Elements specific logic.
Scope
The inject HOC.
Risks
Users who intend to use the inject HOC with Elements may not get alerted when setting up the components incorrectly.

Hey @peeinears! Thanks for opening this API review up.
I believe this should be a use case that is already supported with the existing API surface area. In particular, you should be able to follow this section in the README under "Advanced integrations":
Using an existing Stripe instance

In some projects, part of the project uses React, while another part doesn't. For example, maybe you have business logic and view logic separate. Or maybe you use react-stripe-elements for your credit card form, but use Stripe.js APIs directly for tokenizing bank account information.

I believe this is exactly the use case you describe above.
The way this would work is higher up in your component tree, you'd create a stripe instance and save it into component state or store it in something like a Redux store. You'd use that stripe instance both to create the <StripeProvider /> and to pass to wherever you create bank tokens.

@jez Woops, missed that! Yeah, I guess I was thinking I wouldn't have to use the <Elements/> component at (since I'm not using Elements in this part of the code). Are there any implications with putting <Elements/> right below <StripeProvider/> as opposed to using it just immediately above the components that use Stripe Elements?

If you're asking about performance, it shouldn't matter where in the hierarchy the <Elements /> component is.
There's a bit of subtly around where <Elements /> and injectStripe are with respect to each other. In particular, the component you call injectStripe on must not reference <Elements />. But generally you'll know if you did this right because it will fail loudly otherwise.
The only other thing is that if for whatever reason you have, for example, two card elements on the same page, when you call createToken we won't know which one to use if they're both under the same <Elements /> component. In that case, you'd need to push the <Elements /> container down far enough so that you can wrap each <CardElement /> in its own <Elements /> container.
If this is hard to visualize, see the demo:
https://github.com/stripe/react-stripe-elements/blob/master/demo/demo/index.js#L338-L343
we have to do this because on the demo page, there's both a <CardElement /> and <CardNumberElement />, both of which can create tokens or card sources, so when you call createToken we don't know which one to use.
(But it's fine to mix and match, having both a <CardElement /> and an <IbanElement /> under the same <Elements /> group, because only the <CardElement /> can create card tokens, and only the <IbanElement /> can create sepa_debit sources.)

Got it, thanks. That makes sense.
It still feels unintuitive to me that I would need the <Elements/> component for inject even when I'm not using Elements, especially knowing that <Elements> doesn't add any value in that case (correct me if I'm wrong). So I still think the requested update could be useful.
That said, I realize this is the Elements library, and understand if this non-Elements use case is not high priority or within scope. I defer to y'all!
Thanks for your help and consideration.

Without <Elements />, you'd have to manually keep track of all the element refs that get created when mounting, say, a <CardElement />. In the non-React version of Elements, you have to keep track of these element references, and pass them as arguments to createToken. In the react version, we can automatically keep track of them for you, so you don't have to pass them to createToken.
If you never mount a <CardElement /> or other individual element, I don't think you need react-stripe-elements at all. You can just use Stripe.js directly in your page.

Toootallly. I am using react-stripe-elements elsewhere, so it's already in my bundle, and since it comes with <StripeProvider/> and inject (which I like!) I figured I'd use them. Just saw an opportunity to make the provider/inject pattern more generalized. Thanks :)

👍 closing because I think this is mostly resolved / won't do.
Thanks!


httpx: 1.2.8 -> 1.2.9

Diff: https://github.com/projectdiscovery/httpx/compare/refs/tags/v1.2.8...v1.2.9
Changelog: https://github.com/projectdiscovery/httpx/releases/tag/v1.2.9
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

1 package built:

httpx


Screen Freeze

https://user-images.githubusercontent.com/68264080/139390216-94e9ea33-b9e0-43f0-a6b5-1450e1dbcd75.mp4

Verified this issue on dev app v2.0.01(374)


packbuilder: allow users to set number of threads

libgit2 provides the option for the pack builder to be multi-threaded during deltification. On repositories of any significant size, such as the Linux kernel, threading provides a significant performance benefit. Allow users to specify the thread count used by a pack builder.
Opinions about the best interface here are welcome. The interface accepts 0 for "number of CPUs" and returns the number of threads actually set, which will always be 1 if threading is disabled. I'm unsure if we want an Option<u32> instead (with None as 0) or if specifying 0 as the value is better. Also, I considered checking the return value and coercing it into a Result if threads are disabled, but there isn't a really good value here (GIT_EINVALID? GIT_ENOTFOUND?).
I'm happy to revise (including the documentation) with whatever's decided if there's a better way forward.

Thanks!

I think mirroring the C interface is pretty reasonable here, so should be all good!


Crash on opening html file

Atom can not read html files and crashes. Here is one example page that does make this editor crash (it is the only editor I know that crashes on loading that html):
http://news.google.com/news?cf=all&hl=en&ned=us

Thanks for the report -- this seems to be a duplicate of https://github.com/atom/atom/issues/979 because that HTML file contains a super long line.


zabbix: removed unnecessary patch

I've found that fping3 couldn't be applied on Zabbix 3.0.3 for CentOS 7. I suppose that it should be excluded from spec.

Thanks!


add versioned operator docs

https://issues.redhat.com/browse/ISPN-11808
This adds links to versioned docs for the Operator on master. Depends on this change to the site: infinispan/infinispan.github.io#109

merged, thanks!


rubygems.org license notice

The following might be a good idea for something to add to the gemspec, because it would provide an indication of the copyright status of the gem on rubygems.org:
s.license = 'Unlicense'

The contribution notes in the README explicitly prohibit contributing changes to the gemspec.  If not for that, I would have simply submitted a pull request for this.

Never mind.  I should have looked at the gemspec.  The actual problem is that a version built on the current gemspec does not appear to have been pushed to rubygems.


Commas in FROM: cause "undefined method `formatted' for Mail::UnstructuredField"

After nearly an hour of hair-pulling, I've discovered that if the from: display name contains a comma, Mailgun throws an exception NoMethodError (undefined method 'formatted' for #<Mail::UnstructuredField:0x007fba8303c8f8>).
Examples:
Acme Manufacturing, Inc<EMAIL_ADDRESS># <-- Throws Exception
Acme Manufacturing Inc<EMAIL_ADDRESS># <-- OK

Actually, I forgot that a comma is a delimiter for multiple addresses, so I'll have to figure out a way to escape it.

Heh, more than a year later, and I found myself in this pitfall again!
Although not specific to this project, here's a tip: %Q["#{addressee_name}" <#{email_address}>]

I just came across this too.

This seems to happen whenever the from: and to: fields are not formatted correctly (extra commas and periods seem to be the common ones)


RStudio: 0.98.110 -> 1.1.216

This fixes incompatibilities introduced by a new R version in d16c38a260ef41ae648f994918f491437fb7c75c
It also fixes #25315 as a result.
Motivation for this change
Upgrade to latest version of RStudio and also fix a broken RStudio build due to an upgrade in R.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The Travis CI build is failing for something that looks like it's hitting multiple different PRs. Should I be worried about that?


kubernetes-pod-fails-with-crashloopbackoff 🔴

When I do pull the python to kubernetes, the pod get fails and shows crashloopbackoff.

I would try asking over at the Docker Community Forums, Docker Community Slack, or Stack Overflow. And providing the commands you ran and any relevant files or logs, since these repos aren't really a user-help forum

Just a random guess but unless you are running a specific python file that is a service of some sort, then the default of this image is to run an interactive python shell that would exit immediately in a default kubernetes pod because there is no TTY (i.e. an interactive user).
https://github.com/docker-library/python/blob/50850bb077be358026bfc240adcbcbc850a5bd33/3.9/bullseye/Dockerfile#L100


Bit overflows unchecked resulting in data integrity failure

Description:
When generating Snowflake IDs using the rusty-snowflake library, there is an issue with the parsing of Snowflake IDs when the worker ID exceeds the maximum value. The current implementation incorrectly interprets worker IDs greater than the maximum value as a different worker ID, leading to incorrect results. This issue also applies to the sequence component of Snowflake IDs, but it is less likely to cause issues for most developers.
Steps to Reproduce:

Generate a Snowflake ID using the rusty-snowflake library with a worker ID exceeding the maximum value (e.g., 1024, 1025, etc.).
Parse the generated Snowflake ID using the library's parsing function.
Compare the parsed worker ID with the original worker ID used during Snowflake ID generation.

Expected Behavior:
The parsing function should accurately extract the worker ID from the Snowflake ID.
Actual Behavior:
When the worker ID exceeds the maximum value, the parsed Snowflake ID has a different worker ID than the original. For example, worker ID 1024 is parsed as worker ID 0, worker ID 1025 is parsed as worker ID 1, and so on.
Relevant Code Snippet:
use rusty_snowflake::Snowflake;

let worker_id: u64 = 1025; // Worker ID exceeding the maximum value of 2^10 - 1 (1024)

let snowflake = Snowflake::new(worker_id);

let parsed = Snowflake::parsed(snowflake.to_id());

assert_eq!(snowflake.worker_id, parsed.worker_id, "Worker ID should be {}, but it received {}", worker_id, parsed.worker_id);

Additional Information:
This issue affects the correctness of Snowflake ID parsing when worker IDs exceed the maximum value. It may lead to incorrect identification of the originating worker ID and potential data inconsistency or misattribution. Although the sequence component is also affected, it typically has a smaller range and is less likely to cause issues in most scenarios.
Proposed Solution:
To address this issue, the Snowflake ID parsing logic should be modified to handle worker IDs exceeding the maximum value correctly. This could involve adjusting the parsing algorithm or validating the worker ID before interpreting it in the parsing process.
Impact:
The incorrect Snowflake ID parsing issue affects applications relying on the rusty-snowflake library for unique ID generation. It may cause misidentification of worker IDs and potential data integrity issues if worker IDs are not correctly assigned and interpreted.
Severity:
This issue can be classified as a medium severity as it may impact applications relying on accurate worker ID parsing for Snowflake IDs. However, the impact can vary depending on the specific use case and how worker IDs are managed and interpreted in the application.
Affected Versions:
This issue affects all versions of the rusty-snowflake library.
Workaround:
Currently, there is no known workaround for this issue within the rusty-snowflake library. Users are advised to ensure that the worker ID does not exceed the maximum value until a fix is provided in a future release.

I'm currently working on a fix to this, but the implementation will result in breaking changes. The current version (0.2.0) is not safe, and shouldn't be used until this has been resolved.


[Release] sdk/resourcemanager/recoveryservices/armrecoveryservicesbackup/4.0.0

https://github.com/Azure/sdk-release-request/issues/4714

The breaking changes are introduced and approved in this PR.

@microsoft-github-policy-service rerun

@Alancere , let's wait until the service response on the breaking change question in JAVA release request.

/check-enforcer override


Added support for <IP_ADDRESS>.1

part of https://github.com/Proemion/ts2-psp-merge/issues/34546

@Proemion/devops could someone merge please?


npm WARN<EMAIL_ADDRESS>requires a peer of element-theme-chalk@* but none is installed. You must install peer dependencies yourself.

npm WARN<EMAIL_ADDRESS>requires a peer of element-theme-chalk@* but none is installed. You must install peer dependencies yourself.

需要安装什么依赖 ？？ 按照官方文档 运行下来   报错 怎么解决？

Could be a problem with the last version of Node? I have the same issue, with node 9.7.1

having same issue with node 9.2.0

gyp ERR! not ok
Build failed with error code: 1
npm WARN<EMAIL_ADDRESS>requires a peer of element-theme-chalk@* but none is installed. You must install peer dependencies yourself.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node scripts/build.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/logan/.npm/_logs/2021-02-24T07_37_12_898Z-debug.log


Add Channel Voice Message: Note-On

Add Note-On channel voice messages.

First full Channel Voice message implemented.


"sfdx force:source:deploy --sourcepath my/path/to/files" command is not working as expected.

Summary
When runing the command in the title of this issue, output an html with title of "internal server error" and local code is not deployed to the sandbox org.
Here is the HTML that outputs:

<title>An internal server error has occurred</title>

An internal server error has occurred

An error has occurred while processing your request. The salesforce.com support team has been notified of the problem. If you believe you have additional information that may be of help in reproducing or correcting the error, please contact Salesforce Support. Please indicate the URL of the page you were requesting, any error id shown on this page as well as any other related information. We apologize for the inconvenience.

Thank you again for your patience and assistance. And thanks for using salesforce.com!

An internal server error has occurred

An error has occurred while processing your request. The salesforce.com support team has been notified of the problem. If you believe you have additional information that may be of help in reproducing or correcting the error, please contact Salesforce Support. Please indicate the URL of the page you were requesting, any error id shown on this page as well as any other related information. We apologize for the inconvenience.

Thank you again for your patience and assistance. And thanks for using salesforce.com!

Error ID: 532018934-1114435<PHONE_NUMBER>)

Steps To Reproduce:

This is step 1.  Open vscode.
This is step 2.
This is step 3.  Right click on the file, then select the option "SFDX: Deploy this source to org."

Expected result
sourpath files deploy succesfully to the authorized organization.
Actual result
Throws an error containing the html descripted in the summary.
Additional information
I install the cli via npm. The error start ocurring when I update the sfdx cli from version 7.80.0 to 7.92.0.
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):  sfdx-cli/7.92.0 linux-x64 node-v14.16.0
OS and version:  Ubuntu 20.04

can you set try to use the soap API instead by setting SFDX_REST_DEPLOY=false in the environment?
The change to use the newer REST metadata api has caused some bugs when it doesn't return valid json.

can you set try to use the soap API instead by setting SFDX_REST_DEPLOY=false in the environment?
The change to use the newer REST metadata api has caused some bugs when it doesn't return valid json.

This do the magic trick!  for using the command in vscode, also want  to add that using directly the command in the command line, will allow you to perform the deploy with REST api but the gui of vscode wont perform it with REST. Thanks a lot @mshanemc !

I'm observing similar issue:
$ sfdx force:mdapi:deploy -u uat -w -1 -d deploy/03_23_21/src --checkonly
*** Deploying with REST ***
ERROR running force:mdapi:deploy:

        Illegal Request

        You have sent us an Illegal URL or an improperly formatted request.

Yes!  The mdapi:deploy command has a soap option, but source:deploy doesn't.  That's why you either need to force it to use REST via environment variable or config:set.
We're working with the metadata team to get this resolved without manual steps.

@mshanemc This is still broken without the manual workaround, correct?  If so, could I suggest leaving this ticket open, perhaps flagging 'owned by another team' if the work is with the metadata team?  The default search in github is 'is:open'; some folks may miss the workaround if the ticket is closed.


Various Text Sizes for Captions

It would be very useful to specify the font size for captions on a per-caption basis instead of all captions having the same size.

Hi @DerKorb - thanks for the feature request.
Looking at PR #419 - we will need a way of knowing what the caption size should be. I don't believe the current default of "9" is sufficient. Ideally we could have the ability for these 3 scenarios:

change all captions to a specified size
have various sizes
smarter default size based on the drawing


[🐞]  Error while deploying Qwik project with Cloudflare Pages adapter

Which component is affected?
Qwik Runtime
Describe the bug
I am trying to deploy a Qwik project using the Cloudflare Pages adapter, but I encountered an error while running npm run deploy. After selecting the project, I received the following error message:
✘ [ERROR] Could not resolve "..serverentry.cloudflare-pages"
_worker.js:1:22:
  1 │ ...{ fetch } from "..\server\entry.cloudflare-pages"; export defaul...
    ╵                   ~~~~~~~~~~

You can mark the path "..serverentry.cloudflare-pages" as external to exclude it from the bundle, which will remove this error.
✘ [ERROR] Build failed with 1 error:
_worker.js:1:22: ERROR: Could not resolve "..serverentry.cloudflare-pages"
Reproduction
https://stackblitz.com/edit/qwik-starter-rqzdwk?file=README.md
Steps to reproduce

Set up a Qwik project with the Cloudflare Pages adapter.
Run npm run deploy.
Select the project.
Expected Result

The project should be successfully deployed without any errors.
Actual Result
The deployment process fails with the error message mentioned above.
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (20) x64 12th Gen Intel(R) Core(TM) i9-12900H
    Memory: 15.41 GB / 31.70 GB
  Binaries:
    Node: 16.15.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.1555.0), Chromium (112.0.1722.68)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @builder.io/qwik: 1.0.0 => 1.0.0
    @builder.io/qwik-city: ^1.0.0 => 1.0.0
    undici: 5.22.0 => 5.22.0
    vite: 4.3.3 => 4.3.3

Additional Information
Please let me know if there are any known workarounds or any additional information I can provide to help troubleshoot this issue. Thank you!

damn windows...


PostgrestFilter.hasPathRecursive returns false if there is an array within an array

Describe the bug
When there are nested arrays, updating the cache using useUpsertItem causes the cache to disappear.
https://github.com/psteinroe/supabase-cache-helpers/blob/main/packages/postgrest-core/src/postgrest-filter.ts#L115-L142
To Reproduce

Create and retrieve a query with nested arrays

{
    "recipes": [
        {
            "title": "hoge",
            "recipe_instructions": [
                {
                    "id": "1",
                    "title": "hogehoge"
                }
            ]
        }
    ]
}

Update the cache using useUpsertItem
This bug causes the cached data to disappear

Expected behavior
Return true if valid
Additional context
The data should be in recipe_instructions[0].title, but it cannot be retrieved due to the following path
basePath: recipes,
objectPath: recipe_instructions.title

Can you please provide both the query and the mutation you use?

query
supabase
    .from("saved_recipe_groups")
    .select(
      `id,
      title,
      created_at,
      user_id,
      is_deleted,
      saved_recipe_group_items(
        saved_recipe_id,
        is_deleted,
        saved_recipes(
          id,
          recipe_id,
          recipes(
            id,
            title,
            image_url,
            servings,
            recipe_ingredients(id, name, quantity)
          )
        )
      )`,
    )
    .eq("user_id", userId)
    .eq("is_deleted", false);
    ```

useUpsertMutation

useUpsertMutation

useUpsertMutation(
supabase.from("saved_recipe_group_items"),
["saved_recipe_group_id", "saved_recipe_id"],
saved_recipe_id,         is_deleted,         saved_recipes(           id,           recipe_id,           recipes(             id,             title,             image_url,             servings,             recipe_ingredients(id, name, quantity)           )         ),
);

UseUpsertItem to update the cache of save_recipe_groups.
Useoptions of merge.
Except for saved_recipe_group_items, the value of current (first argument) is used as is.

useUpsertItem

const updateItem = useUpsertItem({
primaryKeys: ["id"],
table: "saved_recipe_groups",
schema: "public",
merge: (current, insert) => {
return {
...current,
saved_recipe_group_items: mergeArrays(
current.saved_recipe_group_items ?? [],
insert.saved_recipe_group_items ?? [],
),
};
},
});

@psteinroe
uery and mutation are ↓.

query
supabase
    .from("saved_recipe_groups")
    .select(
      `id,
      title,
      created_at,
      user_id,
      is_deleted,
      saved_recipe_group_items(
        saved_recipe_id,
        is_deleted,
        saved_recipes(
          id,
          recipe_id,
          recipes(
            id,
            title,
            image_url,
            servings,
            recipe_ingredients(id, name, quantity)
          )
        )
      )`,
    )
    .eq("user_id", userId)
    .eq("is_deleted", false);

useUpsertMutation
useUpsertMutation(
    supabase.from("saved_recipe_group_items"),
    ["saved_recipe_group_id", "saved_recipe_id"],
    `saved_recipe_id,
        is_deleted,
        saved_recipes(
          id,
          recipe_id,
          recipes(
            id,
            title,
            image_url,
            servings,
            recipe_ingredients(id, name, quantity)
          )
        )`,
  );

UseUpsertItem to update the cache of save_recipe_groups.
Useoptions of merge.
Except for saved_recipe_group_items, the value of current (first argument) is used as is.

useUpsertItem
 const updateItem = useUpsertItem({
    primaryKeys: ["id"],
    table: "saved_recipe_groups",
    schema: "public",
    merge: (current, insert) => {
      return {
        ...current,
        saved_recipe_group_items: mergeArrays(
          current.saved_recipe_group_items ?? [],
          insert.saved_recipe_group_items ?? [],
        ),
      };
    },
  });


FileNotFound error during photo retrieval

Python exited the script with "FileNotFoundError: [Errno 2] No such file or directory:...", even though I've verified the file(photo) is there and viewable on the group page.  Is there a way to retry the retrieval, picking up where it left off, or exluding certain photos or albums if they are recurrently causing errors?

Tried rerunning the script, ended with same error at same file.  The file exits and I'm able to download it manually, but how do I resume or complete the archiving process?  How can I skip over this file?

FileNotFoundError implies a file attempted to be opened on your local system but failed. Unsure where this would have happened without further debug information.
There's been several changes to the code in the last 20 days, including functionality to skip over already downloaded content. Could you try updating and see if it works now?

Python 3.8 32bit W7Pro x64  Error archive_files Couldn't access Files functionality for this group.  This error repeats for files, photos, db, links, calendar form the open group vegoilfuel

@tssmith2002 This implies that the cookie credentials you are providing to the script are incorrect.

If it a open group then you shouldn't need the Cookie, right?

No, with open groups you'll only get the email archives without authenticating, and nothing else.

Thanks, I am having some difficulty in getting the cookies out but that is probably the issue for me.


Adding an Editorial page

Under GSSoC 22
Adding an Editorial page that contains a detailed solution for questions.
The solution will be in a markup language.
**Screenshots

@HazemAbdo try to send the pull request within 2 days


Delete button for admin

Description

Types of changes

Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-134

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-134


time management and position complexity

recently i saw a GitHub commit of the chess engine Simbelmyne (rating ~3000), see https://github.com/sroelants/simbelmyne/pull/272/commits/36b8d6833e541c56a7b40d9ad7f09b03798102b8
it's about "soft & hard time" concerning time control .. i wanted to share this with you, because i'm attracted to the idea, then i saw you already implemented such function, or is it the same ? Can you explain in words how it functions in practice and why you chose to use it ?
regarding time control i was also thinking about the "complexity of the position" - once i saw some function for that, someone created a value for this, but i forgot where i found it .. i think smart time control is very important, whatever the strength of the engine : when it fails to play the 'right' move in a 'crucial' position, the game may be lost while no good continuations can be found ..
i imagine it's hard to get your head around this, let it sink in.

another thing regarding this : when using the MultiPV function (mpv) of some engines, the cp score of the top-5 bestmoves at each search depth may be close - many moves are possible and at each following depth bestmove may have changed .. now here comes my idea : in such case another evaluation function could be used, one which distinguishes on different grounds. I'm not suggesting you should implement mpv (for this, or anyhow, although it's a nice addition), for me mpv is a way to better see all alternative moves and how they can be rejected at climbing depths.

yes time management can be indeed very complex and the implementation I have Is very basic though I think it does the job for now. Hard time limit is the one you should not be exceeding, while a soft limit is put in place to make sure we dont just aimlessly go into a deeper search that will be stopped halfway through (at least in Aramis soft time limit is checked before starting new iterative deepening depth). You are right, this Implementation is very simple and does not take into account complexity of a position where spending maybe 1 extra second on a certain move would win you the game. I have seen around some time-management that considers how stable the evaluation result is as a way to estimate the complexity of the position and hence give more or less time to the engine to think. For now I think I have enough Improvements lined up that would contribute to the engine's play a lot more than time-management but one day I will definitely revisit this and look at more complex soft bound calculation but from my tests, my engine does not flag and does not have un-reasonable chunks of time left when the game ends.

another thing regarding this : when using the MultiPV function (mpv)...
MPV is in my eyes a very advanced technique and I have no Idea how it works, and how the search and pruning are affected, I’m sure you can find someone more experienced than myself that will give you a good answer to your question

..MPV is in my eyes a very advanced technique and I have no Idea how it works, and how the search and pruning are affected, I’m sure you can find someone more experienced than myself that will give you a good answer to your question..

right .. but doesn't your engine have some top-(5) best-move-list internally ? I mean, during search when 'current bestmove' is replaced with a better one, the best-move-history list can be considered as such top-x ? Then, when bestmove changes (a lot) at climbing depth, another evaluation can (also) be consulted, maybe leading to a more clear bestmove selection in such position. I'm not aware someone has ever implemented such idea .. special tuning is needed to develop this evaluation setup, and probably many test positions, but i think it can be done.

btw. i understand you consider other code issues more important at this moment in time.

I would need to look into this again, but yes this would be possible to keep track of the best moves overall. though because of pruning and alpha beta keeping track of if a move is better than another, and not by how much (null-window search) some of the scores given to the second, third, fourth etc. moves might be completely understating how truly bad or neutral they are. I’m not sure how top engines accurately score moves other than the first one, might be worth looking into it

btw in v1.5.0 of Aramis I have added best move and eval stability which have gained +70 ELO compared to previous versions.  I will do further tuning of the values to hopefully gain some more but I have to thank the much stronger chess engine Alexandria as I am using a very similar implementation for time management.

I didn't know 1.5 exists .. i just tried to compile by your commands in the README but it fails :
~/Apps/Cmake/cmake-3.29.2-linux-x86_64/bin/build$ ../cmake --build .
[  3%] Building CXX object CMakeFiles/ChessEngine.dir/engine/main.cpp.o
[  6%] Building CXX object CMakeFiles/ChessEngine.dir/chess/movegen/movegen.cpp.o
[ 10%] Building CXX object CMakeFiles/ChessEngine.dir/chess/board.cpp.o
[ 13%] Building CXX object CMakeFiles/ChessEngine.dir/chess/movegen/update.cpp.o
[ 17%] Building CXX object CMakeFiles/ChessEngine.dir/engine/init.cpp.o
[ 20%] Building CXX object CMakeFiles/ChessEngine.dir/chess/misc.cpp.o
[ 24%] Building CXX object CMakeFiles/ChessEngine.dir/engine/search/search.cpp.o
[ 27%] Building CXX object CMakeFiles/ChessEngine.dir/engine/UCI.cpp.o
[ 31%] Building CXX object CMakeFiles/ChessEngine.dir/engine/hashtable.cpp.o
[ 34%] Building CXX object CMakeFiles/ChessEngine.dir/engine/eval/evaluation.cpp.o
[ 37%] Building CXX object CMakeFiles/ChessEngine.dir/engine/search/movesort.cpp.o
[ 41%] Building CXX object CMakeFiles/ChessEngine.dir/test/benchmark_tests/perft_tests.cpp.o
[ 44%] Building CXX object CMakeFiles/ChessEngine.dir/test/benchmark_tests/search_and_eval.cpp.o
[ 48%] Building CXX object CMakeFiles/ChessEngine.dir/test/benchmark_tests/puzzle_tests.cpp.o
[ 51%] Building CXX object CMakeFiles/ChessEngine.dir/test/debug_tests/evaluation_tests.cpp.o
[ 55%] Building CXX object CMakeFiles/ChessEngine.dir/test/debug_tests/move_tests.cpp.o
[ 58%] Building CXX object CMakeFiles/ChessEngine.dir/test/debug_tests/bitmask_tests.cpp.o
[ 62%] Building CXX object CMakeFiles/ChessEngine.dir/engine/logger/logger.cpp.o
[ 65%] Building CXX object CMakeFiles/ChessEngine.dir/test/debug_tests/game_scenario.cpp.o
[ 68%] Building CXX object CMakeFiles/ChessEngine.dir/chess/see.cpp.o
[ 72%] Building CXX object CMakeFiles/ChessEngine.dir/test/debug_tests/see_tests.cpp.o
/home/roelof/Apps/Cmake/cmake-3.29.2-linux-x86_64/bin/test/debug_tests/see_tests.cpp: In function ‘void Test::Debug::see_test()’:
/home/roelof/Apps/Cmake/cmake-3.29.2-linux-x86_64/bin/test/debug_tests/see_tests.cpp:31:51: error: variable ‘std::istringstream inputStream’ has initializer but incomplete type
   31 |             std::istringstream inputStream(command);
      |                                                   ^
gmake[2]: *** [CMakeFiles/ChessEngine.dir/build.make:356: CMakeFiles/ChessEngine.dir/test/debug_tests/see_tests.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/ChessEngine.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2

I didn't look into this yet.

it is a development version so I haven't checked proper compilation on windows and linux systems. Once it is released it should work properly


hang backing up firmware, flash works

Teckin SP20 smartplug
The stage in start_flash that backs up the existing firmware would hang after about 7000 bytes transferred. Ctrl-C out to a prompt and then running the next step, firmware-picker, would continue as normal providing a re-flashed module.
Reproduced over 4 separate plugs. Other model plugs ran same install of start_flash all the way through as expected.

Hi @chriscook8, thanks for your report.
It is a known issue with some devices, perhaps due to dirty power supplies.

Did you manage to complete a backup for any of the SP20s?

No, none of them backed up. I don't need a resolution, just reporting an issue and offering to help troubleshoot if need be. I suppose a "do you want to skip the backup step?" prompt in the script would have sped things up for me after I discovered the issue. Thanks

Do you have any other stock SP20s for testing?

I suppose a "do you want to skip the backup step?" prompt in the script would have sped things up for me after I discovered the issue

Not a bad idea

Sorry, they are all happily Tasmota now.... thanks to tuya-convert!

I'll look into this but it will be difficult to verify without one of these devices. If anyone willing to help can reproduce this, please comment.

hi, i have the same issue atm with this guys
https://www.action.com/de-de/p/lsc-smart-connect-smarter-multicolor-spot-2/
i have succesfully flashed 2/4 with tasmota, the other 2 are stuck in firmware backup

Could you share your firmware backups please?

20191206_215833.zip

Those are both empty files. Check in the backups directory for the 2/4 devices that succeeded in backing up.

20191206_224152.zip

Thanks but still doesn't look like a complete backup. A full backup should be exactly 1MB.
When you say success, you mean backup success or flashing success? They should all be flashable, as discussed above you can manually continue the process if the backup fails.

20191206_233259.zip
Now this one should be what you were looking for

interesting, after i shortened the distance of the devices, the backup seems to work (or is it just luck?). Now im stuck in flashing. everytime i enter 2 for flash tasmota.bin i get "Not found: /flash" as error

Thanks @tobit6795 I will take a look at your backup and see if I can reproduce the issue.
The error you got when trying to flash was due to running an older version of the intermediate firmware with a more recent version of tuya-convert. This probably means you attempted to flash this device with an older version of tuya-convert but never proceeded to /flash3.
Glad you were able to sort it out!

Im having the same issue with 1 of 2 sp20s and cant get past the backup. Just says speed 0 and eta of 20+ hours.

@mkono87 see #459


Use modern algorithm when generating p12 file. Fixes: #4

I updated the code to use the new modern encoder made available in https://github.com/SSLMate/go-pkcs12/pull/48
Previously symfony-cli was using old algorithm, newer nodejs does not support that. It was causing webpack encore (and vite too) to fail if symfony's pfx was used. Main issue: https://github.com/symfony/webpack-encore/issues/1187
After this is merged and a new version is tagged I can open another PR in https://github.com/symfony-cli/symfony-cli to use new version and also update symfony doc to no longer need workaround NODE_OPTIONS=--openssl-legacy-provider

@tucksaun Can you have a look at this PR? Should fix https://github.com/symfony-cli/cert/issues/4 I suppose

@fabpot LGTM (but can't test it at the moment)

@tucksaun Can you have a look at this PR? Should fix #4 I suppose

Yes It fixes #4. I forgot to add it to title.

@fabpot LGTM (but can't test it at the moment)

Thanks for taking a look. If it helps, here how I tested it.
In /home/gittu/symfony-cli's go.mod file I added replace github.com/symfony-cli/cert => /home/gittu/symfony-cli-cert so It uses my fork. Then I deleted ~/.symfony5/certs/default.p12 and run server:start with symfony-cli. I check with openssl command and try to use default.p12 with node. Here's the result of both system one and my one.

Output from system one (installed via apt).
↪ ...backend (master *) ➤ rm ~/.symfony5/certs/default.p12
rm: remove regular file '/home/gittu/.symfony5/certs/default.p12'? y
↪ ...backend (master *) ➤ symfony server:start

 [WARNING] The local web server is optimized for local development and MUST never be used in a production setup.

 [OK] Web server listening
      The Web server is using PHP FPM 8.2.14
      https://<IP_ADDRESS>:8005
                                                                                Stream the logs via symfony server:log
↪ ...backend (master *) ➤ openssl pkcs12 -info -in ~/.symfony5/certs/default.p12 -noout
Enter Import Password:
MAC: sha1, Iteration 1
MAC length: 20, salt length: 8
PKCS7 Encrypted data: pbeWithSHA1And40BitRC2-CBC, Iteration 2048
Error outputting keys and certificates
408712696C7F0000:error:0308010C:digital envelope routines:inner_evp_generic_fetch:unsupported:../crypto/evp/evp_fetch.c:349:Global default library context, Algorithm (RC2-40-CBC : 0), Properties ()
↪ ...backend (master *) ➤ yarn dev
yarn run v1.22.19
$ vite
error when starting dev server:
Error: unsupported
    at configSecureContext (node:internal/tls/secure-context:279:15)
    at Object.createSecureContext (node:_tls_common:117:3)
    at Server.setSecureContext (node:_tls_wrap:1354:27)
    at new Server (node:_tls_wrap:1218:8)
    at new Http2SecureServer (node:internal/http2/core:3144:5)
    at createSecureServer (node:internal/http2/core:3323:10)
    at resolveHttpServer (file:///home/gittu/backend/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:48574:16)
    at async _createServer (file:///home/gittu/backend/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:59984:11)
    at async CAC.<anonymous> (file:///home/gittu/backend/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/cli.js:764:24)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Output from my one.
↪ ...backend (master *) ➤ rm ~/.symfony5/certs/default.p12
rm: remove regular file '/home/gittu/.symfony5/certs/default.p12'? y
↪ ...backend (master *) ➤ ~/symfony-cli/symfony-cli server:start

 [WARNING] The local web server is optimized for local development and MUST never be used in a production setup.

 [OK] Web server listening
      The Web server is using PHP FPM 8.2.14
      https://<IP_ADDRESS>:8005
                                                                                Stream the logs via symfony-cli server:log
↪ ...backend (master *) ➤ openssl pkcs12 -info -in ~/.symfony5/certs/default.p12 -noout
Enter Import Password:
MAC: sha256, Iteration 2048
MAC length: 32, salt length: 16
PKCS7 Encrypted data: PBES2, PBKDF2, AES-256-CBC, Iteration 2048, PRF hmacWithSHA256
Certificate bag
Certificate bag
PKCS7 Data
Shrouded Keybag: PBES2, PBKDF2, AES-256-CBC, Iteration 2048, PRF hmacWithSHA256
↪ ...backend (master *) ➤ yarn dev
yarn run v1.22.19
$ vite
Forced re-optimization of dependencies
Port 5173 is in use, trying another one...

  VITE v5.0.10  ready in 219 ms                                                                                ➜  Local:   https://localhost:5174/build/
  ➜  Network: use --host to expose
  ➜  Vite ⚡Symfony: vite-plugin-symfony: v6.3.2, pentatrion/vite-bundle: vv6.3.2
  ➜  press h + enter to show help

</details>

Check the openssl output of line "PKCS7 Encrypted data:". Also node server fails vs successfully starts https.

I've released a new version of Symfony CLI, can you double-check that it works well for you?

@fabpot Yes, I just upgraded via apt. The new version is generating the modern p12 file :)

Maybe we need some docs about how to upgrade an existing p12?

Yes, there's already a note at
https://github.com/symfony/symfony-docs/blob/50a296fc648774e3cab0c60331f664727424fcb7/frontend/encore/dev-server.rst?plain=1#L78-L85
Should I open a PR there informing users to upgrade to latest version of symfony-cli and delete old ~/.symfony5/certs/default.p12?

Yes, there's already a note at https://github.com/symfony/symfony-docs/blob/50a296fc648774e3cab0c60331f664727424fcb7/frontend/encore/dev-server.rst?plain=1#L78-L85 Should I open a PR there informing users to upgrade to latest version of symfony-cli and delete old ~/.symfony5/certs/default.p12?

Let's update the recommendation to use the latest CLI version and remove the old p12 file. Can you submit a PR there? Thank you for your help on this topic.

Yes I opened a PR in docs repo. https://github.com/symfony/symfony-docs/pull/19369


Fixed the way completed task count is computed and some refactoring

Signed-off-by: Yinan Li<EMAIL_ADDRESS>

@zliu41 can you review?

LGTM


Presence of export affects narrowing

🔎 Search Terms
presence of export narrowing typescript typecheck type checking export statement
🕗 Version & Regression Information

This changed between versions 5.4.5 and 5.5.3

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.5.3#code/MYewdgzgLgBAHgLhgBQIYCcoEtUBsA8ASgKajoAm+YArgLYBGx6ANDAN6pLTpZgDmAXwB8QmAF52AgNwAoGbmKws4mAFZZWAGYAKOAG0sAXQCU7GTHgHDAOlQqA5AAtiuXCHuyLoSLE0r9RrIA9EEWYTAAegD8MgJyITBQ6ACeMNRgALSgtLTEYNj8iY5YEDC4vMQyCcRwAA4gmDDe0Glg6C6oUMTkACrFhRIALFJAA
💻 Code
const x: Partial<Record<number, {a: string}>> = {};

let i = 5;
if(x[i]) {
  x[i].a = 'hello';
  const f = x[i];
//      ^?
}

// try un-commenting this line
// export const unrelatedThing = 4;

🙁 Actual behavior
The type narrowing for an index type did not work unless an export was included.
🙂 Expected behavior
The export should not affect the compilation or typechecking of a file at all.
Additional information about the issue
This could possibly be a bug with just the TS playground; I did not test it on my machine.

As an additional note, if you change let to const, then there is no error. The problematic behavior only occurs with let.

Behavior is intentional; implemented in #57847, and see #58972

Hmmm, okay. This comment summarizes it well. Adding export actually encapsulates the file, making static analysis possible.


Implement RFC0009: Remove yj binary

RFC
Summary
Remove the yj binary from all stacks' build images
Stack Adoption

[x] Jammy Static
[x] Jammy Tiny
[x] Jammy Base
[x] Jammy Full

Hey @paketo-buildpacks/stacks-maintainers,
here's the implementation tracking issue for the yj removal RFC.

With the PRs merged above for all jammy stacks, this is complete.

@robdimsdale unfortunately, I think the stack version bumps should have been a minor bump, since we removed a package which could break folks. Do you think we should cut new minor releases to make this clear?

@sophiewigmore I think they should either be a major or a patch. Depending on whether you view yj as part of the surface area of a stack or not.
I don't, hence the minor. We could could majors if you want though.

Why don't you view it as a part of the surface area of the stack? I think it would be if it was in the stack before! Also, I don't think any of our stacks are at a 1.0.0 release yet, which would make a minor bump the best option IMO

Maybe it would be more accurate to say I don't consider the yj library part of the supported surface area of the stack.
My understanding is that yj was added to help simple shell-based buildpacks interact with TOML files. I don't think anyone actually uses it for that feature as all of our buildpacks have TOML parsing libraries in golang. I view its existence as a bug, not a feature. So I view its removal as a bugfix, not a breaking change.
Also, pre v1.0.0 semver doesn't really exist, so I interpret that to mean we have flexibility to choose our own versions.
Personally I don't think the majority of Paketo consumers ever look at stack versions because they're always hidden inside a builder.
All of that being said, I don't really feel strongly about this. If you think a minor bump is more clear to consumers, I have no objections to you re-releasing all the stacks with a minor bump.


Export TEOS10

Looks like I forgot to do this in #731. Will include with v0.29.0 (#738).

Are we going to move this to SeawaterPolynomials.jl?

Are we going to move this to SeawaterPolynomials.jl?

Yes. My plans was to merge PR #731 to make sure both nonlinear equations of state work in Oceananigans.jl then have a separate PR where they're nuked and imported from SeawaterPolynomials.jl (same tests should still pass).


Sankaku giving wrong file extensions (Image not found)

The thumbnail shows up but the image itself won't load
When trying to download a GIF the log displays it as a SWF
When trying to download a JPG the log displays it as a MP4 and then displays Image not found
With gelbooru or other boorus it works fine as intended.

I can confirm this. !(https://dl.dropboxusercontent.com/u/3907397/img/Grabber_2016-12-15_05-23-23.png)

Reproduced here too.
Only apply to "chan.sankaku" not "idol.sankaku".
I'll start working on a fix right away.

Could you push an update for this? It's very annoying.

You can update your model.xml file with this one, it should fix the issue until the next release.

Where is it located?

On Windows?
C:\Users\<USER>\AppData\Local\Bionus\Grabber\sites\Sankaku

Someone is too anxious for naughty bits... Thanks for fixing this Bionus. Confirmed working.

confirmed working, thanks


Shard ingest flush queue by username and metric name

...to match the underlying sharding of the index.  Part of #254.
The idea being, if we do hotspot a partition in the index, then lets not hotspot it with all 50 flush threads.  At least this will give the other series a chance to flush, and hopefully get us closer to saturating the provisioned capacity.

:+1:


Only use the username if it is actually provided

Coverage decreased (-0.3%) to 91.143% when pulling 43cf3de86121a0e1d3cfdf363c82af656ba0ef4f on cobusc:dont-use-unset-username into cde53c53157355d40cdcd3681b33782b1728eff0 on kencochrane:master.


Update metrics to the latest version

Latest version changed groupId to io.dropwizard.metrics from com.codahale.metrics

NetflixOSS » Hystrix » Hystrix-pull-requests #117 SUCCESS
This pull request looks good

Thanks, @beckje01


support init system s6 (error: unable to mkdir /var/run/s6)

lots of docker images throws

s6-mkdir: warning: unable to mkdir /var/run/s6: Permission denied

is there a way to solve this issue securely? For example emby/embyserver image.

This on itself is not surprising. /var/run is owned by root and only root can write to it.
Looking a bit closer, it seems that those images use init system/supervisor s6.
As a quick'n'dirty fix you can run x11docker with --user=root. However, this disables all container security features of x11docker.
I'll look closer on how/if x11docker could support s6.

For example emby/embyserver image.

Can you give me other examples, please? I could not find the Dockerfile for emby/emby-base, but need a well documented example to investigate.

of course i will add them here.
lots of emby version i tried has the same base so i will check the transmission ones. For example this one :

https://github.com/linuxserver/docker-transmission/blob/master/Dockerfile
which uses this one as a base:

https://github.com/linuxserver/docker-baseimage-alpine/blob/master/Dockerfile

I have added support for s6-overlay with option --init=s6-overlay.
Please update x11docker to master version and try out. There might be still some issues I did not see on my first test runs. Please report if you find something.
Try e.g.
x11docker --init=s6-overlay emby/embyserver
x11docker -ti --init=s6-overlay emby/embyserver sh
Currently x11docker only supports the specialized s6-overlay.
Maybe I'll add support for s6 in general some day. On first attempts I did not succeed, I would have to dig deeper into s6.

i tried with x11docker version: 5.5.1 and no luck.

s6-mkdir: warning: unable to mkdir /var/run/s6: Permission denied

You have tried the latest stable release.
The s6-overlay implementation is in 5.5.2-beta in master branch.
Run x11docker --update-master to install it.

thanks a lot. it works.
i wanna ask you 2 off topic question if you don't mind.

x11docker WARNING: User me is member of group docker.
That allows unprivileged processes on host to gain root privileges.. >

How can i mitigate this with out kicking out me user out of docker group?

x11docker WARNING: Option --init=s6-overlay slightly degrades container isolation.
It adds some user switching capabilities x11docker would disable otherwise.
Services started as root might try to abuse their privileges

What are exact security implication of this warning and how can i harden it?

How can i mitigate this with out kicking out me user out of docker group?

You cannot, unfortunately.
Possible alternatives without group docker:

Always type in your root/sudo password.
For often used commands: Create small scripts (with chmod +x) in /usr/local/bin like:

#! /bin/sh
x11docker --init=s6-overlay emby/embyserver

Make entries in /etc/sudoers to allow running them with sudo without a password.


OPCUA_Integration_Core missing

Hai,
       i already added opcua package from nuget but still  "OPCUA_Integration_Core" is missing, how can I resolve this, from where can I get it.

It has been quite a while since I worked on this and cannot look it up currently, but I would suggest looking at the Onvif Interace project too (https://github.com/thephez/onvif-interface). I possibly used part of that as the "core". Unfortunately I cannot look into this more at the moment but figured I would mention this in case it could be helpful.


Bug when running "flutter doctor"

I found a bug on Linux systems. Specifically using Ubuntu 22.10
when running
$ flutter doctor

I get the following output:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.5, on Ubuntu 22.10 5.19.0-31-generic, locale
    en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (version 2022.1)
    ✗ Unable to find bundled Java version.
[✓] VS Code (version 1.75.1)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

! Doctor found issues in 1 category.

but when running Android applications, the application runs smoothly on my device and on emulators with no sort of errors.
I think this is a bug on the stable Snap release for flutter.
DISCLAIMER: This command ran on an Ubuntu 22.10 terminal. Also I am using the Snap flutter --classic release from Snapcraft.

When you are already at the Android Studio installation root folder, you will see two folders: "jbr" and "jre".
Copy all the files inside the "jbr" folder to the "jre" folder, then try it again.

When you are already at the Android Studio installation root folder, you will see two folders: "jbr" and "jre".
Copy all the files inside the "jbr" folder to the "jre" folder, then try it again.

Copying jbr to jre fixes the problem with flutter doctor, but it isn't a good solution. Android Studio properly recognizes it in the jbr folder. Copying the folder uses disk space unnecessarily, and since Android Studio isn't aware of the folder, the new jre folder won't be updated. It is likely that eventually (after Android Studio upgrades/patches) flutter doctor will report that the jre version is not correct for Android Studio, even though everything is fine because the jbr directory has been updated. Creating a symlink for jre that points to jbr is a better workaround (ln -s jbr jre) which avoids these problems.
That said, I agree with the bug report that it is something that should be fixed rather than worked around on each installation. The ideal fix is to have flutter doctor recognize the jbr folder as an alternate location for the JRE.

@theunfactoringguru The fix for this is already available on latest master as part of #106674 but not in stable yet. There's a hotfix request raised by @gnprice #121505 which when approved and merged, the fix will be available in next hotfix stable version. I suggest you to follow-up in cp issue for further updates. Will close this as a duplicate. If anybody disagrees, write in comments and I'll reopen it.

Before the fix in next Flutter hotfix stable version, here is the solution for
MacBook M1:
Step 1: Change directory to the subdirectory (Contents) of the Android Studio forder:
cd /Applications/Android\ Studio.app/Contents/
Step 2: Create a symbolic link named jre and point it to jbr:
ln -s jbr jre

Yeah the simbolic link worked with the issue
On Linux if using Jetbrains Toolchain
# the directory "221.6008.2211.9514443" may be unique to others I do not know
$ cd .local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9514443/
$ ln -s jbr jre


Add Sound Effects for GUI

[ ] GUI
[ ] Receipt counting
[ ] Storehouse events

[ ] Rally
[ ] Inspector
[ ] Layoffs
[ ] Strikes

[ ] Harvesting with machines
[ ] Harvesting with workers
[ ] Market sell
[ ] Market buy
[ ] Market contract signed


chore(main): release 1.11.0

:robot: I have created a release beep boop
1.11.0 (2024-07-26)
Features

add opengraph markup to enable rich previews in messaging apps (34fba59)
add postgres extension (6f7718f)
upgrade to laravel 11 (0cc5b97)

Bug Fixes

correct .ini path so that environment variable overrides are honoured (b60bc4a)
prevent server error on upload page (720091b)

Miscellaneous Chores

tidy up tag alignment in output (f44f642)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mdshack/shotshare/releases/tag/v1.11.0 :sunflower:


Silk.NET.Core.Loader.DefaultPathResolver does not check the directory of its own assembly

The DefaultPathResolver normally assumes that the library is either located in the domain's base dir or the main module's dir (which should usually be the same anyway), but this becomes a problem when Silk is loaded from a different directory. One solution would be to just set the current directory to Silk's directory... or at least that would've been the case if the resolver checked the current directory.
I believe the DefaultPathResolver should also check the directory of its own assembly. Otherwise, it should at the very least check the current directory. This will allow developers to place Silk in directories separate from the exe.

For context, I am using Silk in a plugin for another application, and since the plugin has its own directory, Silk cannot find its native libraries


[spike] view only accounts

https://www.notion.so/shapeshift/View-Only-addresses-d33511af79ab44a3b53efc58c6c0354c?pvs=4

LOE for implementation
Can we generalize to allow the connection of multiple wallets when some are active / inactive?
most basic option would just be adding a view only addy that couldn't be used (and maybe safest)
ideal state would be to allow any number of view only wallets to be entered into the new account management flow that are view only.
any portfolio view will be consolidated with all connect wallets (view only and fully functional)
time box to ~8 hours

Spike: https://gist.github.com/gomesalexandre/a6cc77196f5e0b65dcd9fc65fd9f3665
PoC: https://github.com/shapeshift/web/pull/7286
No real estimates as of now, as this brings more product issues and questions to the table than it solves.


DepthAI example: module 'cv2.dnn' has no attribute 'DictValue'

Running the depthAI example with the latest patch version of opencv-python caused an error on my Windows machine. Downgrading to the version specified in this ticket fixed the issue.
pip install -r requirements.txt
.\depthai_combination_install.sh
python .\depthai_combination.py

Thanks for reporting! In this case, I think we will just leave this ticket open until a fix to the issue is released in OpenCV.


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Upgrade from 20230718 to 20231026 fails silently with exit code 1.
Is MSYS2 not meant to be upgraded in this way, since it has an internal pacman-based updater?
Steps to reproduce
Install 20230718. Attempt to upgrade to 20231026.
Actual behavior
Upgrade fails with exit code 1 and no additional info.
Expected behavior
Successful update, or not listed as an available update if this is not a supported means of updating an existing MSYS2 installation.
Environment
Windows Package Manager v1.6.3133
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2715
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3133.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
InstallationLog.txt says:
TargetDirectoryInUse : Error : The directory you selected already exists and contains an installation. Choose a different target for installation.

Thanks for your excellent service 👏

Is MSYS2 not meant to be upgraded in this way, since it has an internal pacman-based updater?
Right

I tried to prevent it from being updated by WinGet by setting RequireExplicitUpgrade: true in the manifest but sometimes it doesn't work. You can try winget pin add MSYS2.MSYS2 to pin the package.

Just ran into this myself. Is exit code 1 unique to the "the directory already exists and contains an installation" error? I wonder if the fileInUse expected error code tied to exit code 1 might provide something slightly more user friendly. It currently just fails and doesn't provide a pointer to the install log or anything.
 pwsh> winget upgrade --all
Name  Id          Version  Available Source
-------------------------------------------
MSYS2 MSYS2.MSYS2 20230526 20231026  winget
1 upgrades available.

Installing dependencies:
(1/1) Found MSYS2 [MSYS2.MSYS2] Version 20231026
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Successfully verified installer hash
Starting package install...
Installer failed with exit code: 1

 pwsh>

Just ran into this myself. Is exit code 1 unique to the "the directory already exists and contains an installation" error? I wonder if the fileInUse expected error code tied to exit code 1 might provide something slightly more user friendly. It currently just fails and doesn't provide a pointer to the install log or anything.
 pwsh> winget upgrade --all
Name  Id          Version  Available Source
-------------------------------------------
MSYS2 MSYS2.MSYS2 20230526 20231026  winget
1 upgrades available.

Installing dependencies:
(1/1) Found MSYS2 [MSYS2.MSYS2] Version 20231026
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Successfully verified installer hash
Starting package install...
Installer failed with exit code: 1

 pwsh>

No. The exit code 1 indicates a general installation failure according to the docs of Qt Installer Framework, which is the installer technology used by MSYS2 installer.

I think MSYS2 does not need upgrades as an installer. You can upgrade it via pacman -Suy inside its shell.
So does it mean we don't need this as a winget application?

You can still do first-time install, or even install another copy of MSYS2 if needed. 🤔

You can still do first-time install, or even install another copy of MSYS2 in a different directory if needed. 🤔

Then you have a nagging winget upgrade notifier in your hand, which doesn't work.

Not THEN. WinGet will always notify you as long as it detects a copy of MSYS2 from the registry.
I am thinking about modifying the package so it can still install new copies but never detect and update them. RequireExplicitUpgrade: true appears to be working only when it was installed through WinGet, and WinGet will still try to update them if it is installed by downloading and running the installer manually.
(AFAIK the counterpart on Chocolatey upgrades by invoking pacman directly, which can not be implemented here at the moment)

You can also install/extract MSYS2 from tar file.

Currently WinGet doesn't support extracting tar file (as it doesn't rely on 7z for extracting archives). See https://github.com/microsoft/winget-cli/issues/2899
The counterpart on Scoop installs in this way though but I don't know how they tackle the upgrading issue as it persists only the home directory.

I am thinking about modifying the package so it can still install new copies but never detect and update them. Currently I am using RequireExplicitUpgrade: true but it appears to be working only when it was installed through WinGet, and WinGet will still try to update them if it is installed by downloading and running the installer manually.

When we have the 1.6.0 schema out, maybe UpgradeBehavior: deny can help with this one... :eyes:

any update?

Just got this issue.
Is there a way to pin a previously installed package to get rid of the winget update notification?

I mangled to pin the current installed version with:
# winget pin add -q 'MSYS2.MSYS2'

Found MSYS2 Installer [MSYS2.MSYS2]
Pin added successfully


SW-5123 Modules Store

Skeleton structure for stores and permissions for modules
Implemented datetime constraints on tables, and module store

#1855  👈
main

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

Join @tommylau523 and the rest of your teammates on  Graphite

Merge activity

Apr 17, 7:33 PM EDT: @tommylau523 merged this pull request with Graphite.


Potential Optimization: pack/unpack more booleans at a time

The current pack/unpack booleans operation process 8 booleans at a time. It might be easier (for compilers) to take advantage of SIMD instructions if the code processes more booleans at a time. Should run some tests and profile and verify.

Using a strided strategy to pack 1024 bits at a time would provide close to 2X runtime improvement, but mysteriously increase unpack by 3X. This result is from GCC and Clang gives a smaller difference, but the trend is consistent. As a result, PR #123 only introduces strided boolean packing.
The attached program is my implementation of strided pack and unpack with timing code.
pack.txt

Update: I also tested using std::bitset<64> to convert 64 bits at a time. Its performance is close to unstrided implementation and slower than strided implementation, so not deploying for now.
void pack_booleans_bitset(std::vector<uint8_t>&    dest,
                          const std::vector<bool>& src )
{
  assert(src.size() % 64 == 0);

  auto bit64 = std::bitset<64>();
  unsigned long long ull;

  auto src_itr = src.cbegin();
  for (size_t dest_idx = 0; dest_idx < dest.size(); dest_idx += 8) {
    for (size_t i = 0; i < 64; i++) {
      bit64.set(i, *src_itr);
      ++src_itr;
    }
    ull = bit64.to_ullong();
    std::memcpy(dest.data() + dest_idx, &ull, 8);
  }
}

Closing since I don't feel this detail needs more attention.


#266 - Move SpringData proxy configuration to store specific NativeImageHint.

Closes: #266

Merged via 4c07772b1aa94e4f393e82ce7cde6541a71a7e7e thanks.