Removing the TPS transform in batch mode

Perhaps we should remove the TPS transform (and calculation of the warped mesh) when using batch processing. There is no reason to do the calculation and it increases runtime immensely when using hundreds/thousands of landmarks. See relevant lines below:
https://github.com/SlicerMorph/SlicerMorph/blob/f6c0ff337ba791d8f5bd7a4196de23116d556a39/ALPACA/ALPACA.py#L1108
Thoughts? @smrolfe @muratmaga

Make sense to me. But I will defer to Sara.
From: Arthur Porto @.>
Sent: Tuesday, June 21, 2022 5:04 PM
To: SlicerMorph/SlicerMorph @.>
Cc: Subscribed @.***>
Subject: [SlicerMorph/SlicerMorph] Removing the TPS transform in batch mode (Issue #221)
Perhaps we should remove the TPS transform (and calculation of the warped mesh) when using batch processing. There is no reason to do the calculation and it increases runtime immensely when using hundreds/thousands of landmarks. See relevant lines below:
https://github.com/SlicerMorph/SlicerMorph/blob/f6c0ff337ba791d8f5bd7a4196de23116d556a39/ALPACA/ALPACA.py#L1108
Thoughts? @smrolfehttps://github.com/smrolfe @muratmagahttps://github.com/muratmaga
—
Reply to this email directly, view it on GitHubhttps://github.com/SlicerMorph/SlicerMorph/issues/221, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKXQZMSZAYHHV4PBEOEVIXDVQJJ5JANCNFSM5ZOJUTXQ.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>


State the minimum version of PHPUnit in CONTRIBUTING.md

run into the same as this; https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/899

:+1:

:+1:

Thank you @SpacePossum.

:+1:  thank you @keradus

@keradus When is this 1.4 release coming?

I have no rights to release it.

Yeh. We really need to get that sorted out.


Update home.js

Limit podcast episode titles to one line. A small change alongside patch-4 (which I'm not clever enough to know how to combine in the web version of Github).

What do you mean limit to one line? does it cut off the message? Do you have a picture which illustrates it?

Here's what it can look like if you have overly long podcast episode titles.

This patch adds an overflow-ellipsis and limits episode titles (only) to one line-height. It has no effect on the message below.

It is assumed that podcasters will have episode titles that can be told apart from the first 30-40 characters; it would be very poor practice were they not to in any case (for typical podcast apps only show 30-40 characters).

That is an improvement! Is there a way to see the entire title? perhaps by hovering over it? Curious how you would implement that? Would a a click to expand be better?

You can relatively simply add a 'hover' action, but that doesn't work on mobile. It's also not very accessible.
This title has been set by the podcaster that is reading these boosts; and the first 30-40 characters should be enough to make the episode clear. If you'd be able to clarify the usecase for seeing the full episode title, I'd be grateful.

I guess as long as you can identify which episode it is that should be enough. It does seem unlikely that 30-40 characters would not be enough to identify the podcast. I suppose someone might want to read the title. What is the canonical mobile alternative to tooltips on hover?


Minor refactoring of authentication

Review app deployed to https://teaching-vacancies-review-pr-3118.london.cloudapps.digital


Error: Webpack Compilation Error

Hi, I have installed wick-a11y in my cypress project and on using getting the following error. Could you please help me with it ?
Module not found: Error: Can't resolve 'cypress-axe' in '/Users/path-to-my-project/node_modules/wick-a11y/src'
Node version: v18.14.0

Delete the node_modules folder and pacjage-locked.json in the root, and do
the npm install of your Cypress project.
Did you have installed previously cypress-axe in your project? If so, no
need remove from your cypress framework package.json, wick-a11y install it
as peer dependency.
On Sun, Dec 8, 2024, 2:05 PM Zohaib Hassan @.***> wrote:

Hi, I have installed wick-a11y in my cypress project and on using getting
the following error. Could you please help me with it ?
Module not found: Error: Can't resolve 'cypress-axe' in
'/Users/zohaibhassan/Documents/office/fe-app-manager/node_modules/wick-a11y/src'
—
Reply to this email directly, view it on GitHub
https://github.com/sclavijosuero/wick-a11y/issues/13, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6VPSCLNQTIASAGHYO3RTUL2ES7BDAVCNFSM6AAAAABTHTYQKWVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4ZDKNJVGUYTEMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@sosafe-zohaib-hassan , did you got the issue resolved?


Very Nice CMS

Yes, I too like mdwiki and I like this one too! I can have an infinite number of content areas by just making lots of .md files and placing them in any div in any generic web template, cool! Point to note though, I had to place the javascript line right in the div if I was wanting it to read a .md file. If I placed the .js line at the top or bottom of the html file, it did not work! Do you know why?

Can you post the a link to a gist or the actual html page and one of the markdown files please?

This is what I had to do
		Javascript must be enabled.</div>
		<script src="mdcms.js"></script>
  </div> ````

ah! I see the problem. I was loading javascripts after the rest of the page, in accordance with "Best Practices" I see that you want to load the javascript in the head, so I've made mdcms trigger once the window is loaded instead. This seems to work in my testing. Please try it and reopen this if it doesn't work for you.


Replace log_process_action to append_info_to_payload

This implementation replace log_process_action to append_info_to_payload  that way the status should be present in ActiveSupport::Notifications's payload and at the log
It's based on what AR is doing:
https://github.com/rails/rails/blob/92703a9ea5d8b96f30e0b706b801c9185ef14f0e/activerecord/lib/active_record/railties/controller_runtime.rb#L35
No test change was needed but one test was include.

Is there any appetite for landing this change? Happy to help get it across the line if anything outstanding.

Would love to see this re-reviewed.

I hope this PR will be merge.

@rafaelfranca @lucasmazza
Is there anything that I may help to merge this PR?

Seems like this actually broke my logging.
I noticed that logs in my app started reporting Completed 401 Unauthorized when the action failed with an unhandled exception. The client still receives 500 Internal Error, but the log reports 401.
I'm checking how to fix it

@rafaelsales I not sure it's related, we released this on v4.4.0

@tegon sorry, I had a typo in the version numbers of my comment. Just fixed.
If I revert this particular commit, the previous behavior is established. I'm building a test case to reproduce.

@tegon Here's the test case to reproduce - this commit was made on top of the latest master
https://github.com/rafaelsales/devise/commit/302c53e9a8a2b6aad5e369605891740b5c72dc7e
I tried to dig in a little bit and I found out that in the following snippet of this PR, the payload[:exception] always returns nil even if a public action raises an error, so the 401 status is added to the payload.
        def append_info_to_payload(payload)
          super
          payload[:status] ||= 401 unless payload[:exception]
        end

I did some investigation on this. Cross-linking my thoughts in the Lograge issue tracker: https://github.com/roidrage/lograge/issues/67#issuecomment-601604344


Copy Blob in sdk?

is there any copy blob functionality built into the SDK currently?
https://docs.microsoft.com/en-us/rest/api/storageservices/fileservices/copy-blob
I might just be totally missing the package, but I can't seem to find a way. Any help would be much appreciated. thanks.

nevermind, i found the new repo... my bad


Jinja Not Rendered in schemachange-config.yml

I need to parameterize my change-history-table based on the target environment and cannot successfully get this Jinja to resolve when ever I deploy.
change-history-table: {% if TARGET_ENV != "PROD" %}{{TARGET_ENV}}_{% endif %}EDA_ENGINEERING.SCHEMACHANGE.CHANGE_HISTORY
This works successfully in my change scripts, and I see that in cli.py the config file should be getting passed to the Jinja engine. Not sure why this works elsewhere, just not in my config file.

does this correction work?
change-history-table: {% if TARGET_ENV != "PROD" %}{{ TARGET_ENV }}_{% endif %}EDA_ENGINEERING.SCHEMACHANGE.CHANGE_HISTORY
The mandatory spaces around the variable reference and the curly braces were missing from your posted comment.  This could just be a transcription error.

No, unfortunately the jinja exception is still raised that TARGET_ENV is undefined. Removing that jinja from my config file resolves the exception, but not my need to parametrize the environment :)

I believe we have a 🐔🥚.
VARS that would contain the value for TARGET_ENV is loaded after the yaml is passed to the parser.
could you perhaps use something like this line: https://github.com/Snowflake-Labs/schemachange/blob/e8fee3dfb36af38e689a71764096fbe1b9c73fa2/demo/citibike_jinja/schemachange-config.yml#L6
as an alternate route to introduce that information into the workflow?
change-history-table: {{env_var('TARGET_ENV', '')}}EDA_ENGINEERING.SCHEMACHANGE.CHANGE_HISTORY
Then Set the value of TARGET_ENV as an environment variable with the value : TEST_ DEV_ STAGE_  as needed?

Hello @ATidmore
Did @MACKAT05's suggestion address your need to parameterize the yaml file ?
@MACKAT05, Could you raise the PR corresponding to this issue?

For reference this is my clunky solution implementation to much the same problem...

the Yaml order doesn't matter we could probably include a more explicit example of handling this in an example yaml  with the "snowflake-role", "snowflake-account","snowflake-database",  "change-history-table"   clustered together wrapped in an if statement.

Thanks @MACKAT05  - I was coming along in my understanding of Jinja and env vars (and DBT and Schemachange etc. etc.) over the past few months, so coupling an environment variable with some jinja should do the trick.


more informative schema check

Currently, and XML issue will simply say
"Traceback (most recent call last):
File "", line 1, in
File "/Users/whitford/miniconda3/envs/openSMOG/lib/python3.9/site-packages/OpenSMOG/OpenSMOG.py", line 170, in loadSystem
self.loadXml(Xmlfile)
File "/Users/whitford/miniconda3/envs/openSMOG/lib/python3.9/site-packages/OpenSMOG/OpenSMOG.py", line 306, in loadXml
raise ValueError("The xml file is not in the correct format")"
We should try to write out the message indicating what is wrong.  This is probably easy to add, but I don't know the syntax.  No rush, for now.  If one uses smog2, then they should comply, as long as they are using the right version of OpenSMOG.

I modified this, so that the actual XML error is printed.


Update READ ME file to include -keepclassmembers class com.shockwave.** { *; }

In the release version of my app. I was getting NoSuchMethodError for PdfPasswordException while reading a password-protected file.
Including  -keepclassmembers class com.shockwave.** { *; } in proguard with -keep class com.shockwave.** solved the problem.

It sounds like you neglected to implement an error handler for a pdf with a password,
The code below would be more of help to you.
.onError(new OnErrorListener() {
@Override
public void onError(Throwable t) {
if (t instanceof PdfPasswordException) {

I have used the same logic in the code. My problem only got solved after inserting that line in proguard.

Readme file says you should use proguard rule otherwise it will not work in release build,..

The advice as specified in the README file right now is to add:
-keep class com.shockwave.**

This is insufficient because it doesn't keep any members of any of the classes. The following are basically equivalent and both work:
-keep class com.shockwave.**
-keepclassmembers class com.shockwave.** { *; }

or
-keep class com.shockwave.** { *; }

I recommend using the latter for simplicity, and I recommend the README should be changed to say this instead.
(Disclaimer: I'm not claiming this is the most optimal set of rules - there may be a way to keep less un-minified and still work.)


cmd/go: confusing compiler errors because of gomod.go

When compiling the following inside a Go module project (containing go.mod in a parent directory):
package main

var string = "variable named 'string'"

func main() {}

you get the following very confusing error message:
dirbaio@jupiter:~/reby/backend/nexus$ go build weirderror/main.go
# command-line-arguments
/tmp/go-build698262353/b001/_gomod_.go:7:22: string is not a type

It seems this is because there is an autogenerated file trying to declare a variable of type string.
dirbaio@jupiter:~/reby/backend/nexus$ go build -x weirderror/main.go
WORK=/tmp/go-build835159733
mkdir -p $WORK/b001/
cat >$WORK/b001/_gomod_.go << 'EOF' # internal

		package main
		import _ "unsafe"
		//go:linkname __debug_modinfo__ runtime/debug.modinfo
		var __debug_modinfo__ string
		func init() {
			__debug_modinfo__ = "0w\xaf\f\x92t\b\x02A\xe1\xc1\a\xe6\xd6\x18\xe6path\tgithub.com/rebythecode/backend/nexus/weirderror\nmod\tgithub.com/rebythecode/backend\t(devel)\t\n\xf92C1\x86\x18 r\x00\x82B\x10A\x16\xd8\xf2"
		}
	EOF
cat >$WORK/b001/importcfg << 'EOF' # internal
# import config
packagefile runtime=/usr/lib/go/pkg/linux_amd64/runtime.a
EOF
cd /home/dirbaio/reby/backend/nexus/weirderror
/usr/lib/go/pkg/tool/linux_amd64/compile -o $WORK/b001/_pkg_.a -trimpath $WORK/b001 -p main -complete -buildid _XG4Y2LGaLdfLVCNTPfu/_XG4Y2LGaLdfLVCNTPfu -goversion go1.11.1 -D _/home/dirbaio/reby/backend/nexus/weirderror -importcfg $WORK/b001/importcfg -pack -c=4 ./main.go $WORK/b001/_gomod_.go
# command-line-arguments
/tmp/go-build835159733/b001/_gomod_.go:7:22: string is not a type

This does not happen when trying to build in a non-modules project.

I presume there could be other ways to break it, such as declaring __debug_modinfo__, or assigning to it in one's init function. I'm not sure what a good fix here could be.

@rsc I think it's better to add the info in a way similar to link's -X flag or gc's buildid recording.

Can you still reproduce this with a go command built from source?
https://golang.org/cl/134915 should have fixed the string is not a type error. (See #27584.)

You're right @bcmills, the particular error seems like a duplicate. I wonder if we should still be worried about the __debug_modinfo__ name, although that seems extremely unlikely to cause issues.

It's theoretically a problem, but empirically much less likely than shadowing string.

I just checked. The error is not present in current master.


Create graph UI

Changes:

Briefly describe changes
-Can upload multiple csv files
-Can edit color and graph type independently of each graph
-Can read csv files of two columns with any column title
-Improved the UI
-Improved the code

I have:

[x] Set target branch to the correct branch, usually dev.
[x] Rebased source branch from the target branch.
[x] Ensured code is formatted and linted.
[x] Cleaned up git history.
[x] Ensured commit messages describe what was changed and why.
[x] Ran tests locally after rebasing and checked output.

Use rebase when integrating features from dev into a feature branch. Revert 5e6855b1430944bb88e0886bda057d8f02a1b952 and use rebase.


Graph API Access

Hi, are there any plans to allow this report data to be downloaded via Graph API's as per the other admin reports? This and Visio are currently missing but would be useful to have. Thanks.

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

ID: 88b68222-3eac-0447-de7f-9b7ba70f1794
Version Independent ID: 88b68222-3eac-0447-de7f-9b7ba70f1794
Content: Microsoft 365 admin center Project activity  - Microsoft 365 admin
Content Source: microsoft-365/admin/activity-reports/project-activity.md
Service: microsoft-365-business
GitHub Login: @camillepack
Microsoft Alias: camillepack

@davegreyuk Thank you for your feedback. Currently, we do not have updates on when this will be available. However, you contact to leave your feedback on the main feedback page at https://feedbackportal.microsoft.com/- So that our dev or product engineers will be able to respond.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @davegreyuk Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. You can contact Microsoft support via the need help section on the admin center. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


webots_ros2: 2022.1.0-1 in 'galactic/distribution.yaml' [bloom]

Increasing version of package(s) in repository webots_ros2 to 2022.1.0-1:

upstream repository: https://github.com/cyberbotics/webots_ros2.git
release repository: https://github.com/ros2-gbp/webots_ros2-release.git
distro file: galactic/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.2.3-1

webots_ros2
* Adapted controllers to communicate with Webots R2022b.
* Added feature to import URDF on the fly.
* Add PointCloud2 support for RangeFinder.

webots_ros2_driver
* Added an URDF importer feature to spawn robots from URDF files.

webots_ros2_importer
* Upgraded to urdf2webots 2.0.0

webots_ros2_msgs
* Added URDF robot messages

webots_ros2_universal_robot
* The 'robot' and 'moveit_demo' scenarios now show a use case of the new URDF importer.

There is a problem with this release. See https://github.com/ros/rosdistro/pull/34630#issuecomment-1256638495


the 4'th menu element

suggest a fourth menu item to be placed in the top-left corner

[ ] if we are doing a map thin, it can be the map with all NFTs

MAP ADDED


Enable stdpar

How do I enable stdpar on gpu? I'm trying to get maxwell_gpu_s.cpp to work.
I did some debugging and the check appears to be this line, which doesn't help since it's called from inside a CPU version of std::for_each.
https://github.com/NVIDIA/stdexec/blob/main/include/nvexec/stream/common.cuh#L56

Nevermind, I got it.
https://twitter.com/seanbax/status/1642567493188288514
Now on to the nvexec stuff.


Server crashing with Iron Furnaces

Server crashed twice when using furnaces. Attached is one of the crash reports. The other is pretty much identical.
Expected Behavior is not to crash.
Iron Furnaces Version 2.6.7 for Minecraft version 1.16.5. FastSuite version 1.0.0 for Minecraft version 1.16.5.
crash-2021-04-29_16.59.57-server.txt

I'll add this here, quick glance seems like an interaction with Polymorph
https://gist.github.com/Gbergz/a3772c2f2c07881d0f33b8c304eece1a

I can't for the life of me see why that would be happening.  I just ran a stress test with about 500k recipes and it was unfazed.
However upon further inspection and some thinking there's not really a benefit to me overriding the getAllMatches as it always has to iterate the entire list anyway, so I'll just let it be vanilla.

af52837193d88f9b71b827a2b56eaa5dcff608fe


Remove the need for unsafe-inline in CSP header.

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
Since src/main/webaspp/index.html uses inline scripts the CSP header needs to include unsafe-inline when serving draw.io.
Describe the solution you'd like
To increase the security and tighten the CSP header all inline scripts  in index.html need to be replaced with dedicated JavaScript files.

You can just include the hashes in the CSP.

You are right. Hashes solve my problem. Thanks!


fix: schema name is optional for future stream_grant

make test
CGO_ENABLED=1 go test -race -coverprofile=coverage.txt -covermode=atomic  ./...
?       github.com/Snowflake-Labs/terraform-provider-snowflake  [no test files]
ok      github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/datasources  2.258s  coverage: 3.6% of statements
?       github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/db   [no test files]
?       github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/helpers      [no test files]
ok      github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/provider     3.233s  coverage: 25.5% of statements
ok      github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources    13.633s coverage: 46.9% of statements
ok      github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/snowflake    0.350s  coverage: 46.8% of statements
?       github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/testhelpers  [no test files]
ok      github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/validation   0.218s  coverage: 31.8% of statements
?       github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/version      [no test files]

@bennylu2 thank you for your contribution!


Coffeescript Interpreter Oddities

I'm writing a coffeescript script for hubot and I'm running into several problems.

Nested If statements are ignored completely,
Numbers are sometimes being arbitrarily interpreted as Strings.

I'm using the latest version of Coffeescript and If I run the exact same code on http://coffeescript.org/ or on other sites and apps these issues do not occur

This sounds weird, can you provide some code example?

Hello,
Sorry for the delayed response, the code was stored on an laptop that died suddenly, I was only recently able to pull some of the files off of it.
I found and attached the version of the code that I got to work through restructuring the nested IF statements in the "#Rule of 10" section and explicitly casting all the Numbers that were randomly evaluating as Strings. I'll do some more digging through the old laptop tomorrow, if it deigns to boot up, and see if I can find the version that was giving me the trouble.
Hubot L5R Roller (Latest).txt
Retroactive documentation
Just so it's easier to look through; the script is designed replicate rolling dice in the Legend of the Five Rings RPG system.
rolled = The number of dice you will roll
kept = The number of rolled dice that you are allowed to add together to determine the total of the roll
bonus = a value (positive or negative) that will be added to the total of the kept dice
additional variables are for the eventual inclusion of other features of the system that didn't get added before we had to abandon Rocketchat and move to a different program.
Rule of 10
Only ten dice are allowed to be rolled at once but it is often the case that your character will have bonuses that bring their rolled and kept dice above this limit. Rolled dice in excess of this limit are turned into kept dice until you are both rolling and keeping 10 dice, then extra dice get turned into a static bonus to the roll.


Fixup README example on unbound variable and indent

Thanks @ikuo !
You code has been merged here but for some reason Github can't see it.

@YoEight thank you for merging!


Use 'parameters' for ami, ec2 type, name of bucket

Typed this in while trying to follow Nathan (Sentia):
construct in stack, extend stack class, init add my args
or cfn parameters of commbinatie
voor **kwars in def init
instance_type: str
instance_type = cfnParameter(self, 'instanchet_type_parameter', defualt- instance_type)
instance_type = ec2.InstanceType(instance_type_param.value_as_string)
user_data = ec2.UserData.for_linux()
user_data.add_commands("""
script
""")

Extended the stack class with instance_type, ami_name, ami_owner, bucket_name.
Defined the instance_type and the bucket_name as a CloudFormation parameter.
Did NOT do the same to the ami_name and ami_owner as these are used to look up the AMI. Looking up with parameters gives errors about TOKENs. You try to use the parameter values, before they can be resolved.
This could be fixed, maybe, by defining an image directly with the AMI ID (and region and account?), but this is error prone and not easy to understand.
Decided not to travel this path.
If someone wants to use the CF template with a different AMI, they can edit "ImageId" in the template before deploying.
The "ImageId" has to be available in the region used.


BUG - Not available after couple of days. 2nd time in two weeks

After a couple of days the power outage is stuck on a random value. It keeps displaying that value and does not change anymore. Other values are working from the growatt integration keep working.
It's the 2nd time in a couple of weeks where i just started with it. After first time i deleted the device and reconfigured the integration, working again after it.
Any tips to solve or troubleshoot this? Without keep reconfiguring this after x days?

There's several extensive conversations on this going on in other places:
https://github.com/home-assistant/core/issues/100874
https://github.com/muppet3000/homeassistant-growatt_server_api/issues/15
I've even updated the main readme for this plugin see "IMPORTANT": https://github.com/muppet3000/homeassistant-growatt_server_api
I'm going to close this as a duplicate of the others.


Hard to use the library for URL validation

I wanted to validate an externally-input URL using Boost.URL.
The naïve implementation that I assumed would be sufficient wasn't:
auto result = boost::urls::parse_uri(url);
if (!result.has_value())
    // handle error

That apparently lets me parse the URL https:// and get a result.
After that I tried
if (!result.has_value() || !result->has_authority())
    // handle error or missing authority

The super-interesting is that this returns has_authority=true
Is this a bug, am I using the wrong API for URL validation, or do I need to use host().empty() instead?
Using boost 1.81.

Can you show the string you are parsing?

That apparently lets me parse the URL https:// and get a result.

Sorry, should have just put that straight in the example. I'm parsing incomplete inputs, e.g. https://

That is not an incomplete input. It has an authority with a host that happens to be empty. Hosts are allowed to be empty strings.

For example look here:
https://www.boost.org/doc/libs/master/libs/url/doc/html/url/ref/boost__urls__url_base/set_host.html#url.ref.boost__urls__url_base.set_host.bnf

reg-name is "zero or more of..." which means it can be empty. So host can be empty.

Ah. I understand I guess it makes a world of sense for e.g. file:///. It is, however, not a particularly useful HTTP URL. Is there any example of using Boost.URL for more specific URL validation? Would you consider the below test somewhat solid?
auto result = boost::urls::parse_uri(url);
if (!result.has_value() || !result->host().empty())
    // handle error or missing authority

The library is correct. https:// is a valid URL.
Have a look at the URL types: https://www.boost.org/doc/libs/master/libs/url/doc/html/url/urls/parsing.html

Boost.URL can't know what you consider useful, you have to define that yourself. Make a formal specification of what you require to be in the URL and then use Boost.URL to check if those conditions are met. Boost.URL only guarantees that it will check whether or not the string is syntactically valid according to the ABNF grammar you choose when you select the parsing function.


The size of the budget inspector is now saved and restored across sessions and when moving between an account and the budget. Note that the size is local to the computer and not your YNAB account so you can have different sizes on different computers.

Github Issue (if applicable): #113
Trello Link (if applicable):
Forum Link (if applicable):
Explanation of Bugfix/Feature/Enhancement:
Added code to store the size of the budget inspector (actually saving the size of the content section) and restore it across sessions and when moving between an account and the budget. The inspector size is now retained when the collapse menu button is clicked.
Recommended Release Notes:
The size of the budget inspector is now saved and restored across sessions and when moving between an account and the budget. Note that the size is local to the computer and not your YNAB account so you can have different sizes on different computers.

My man @dbaldon tackling the old issues! Get it!

OG issues?

Thank you!


[DOCFIX] Add doc for mounting s3 location at a nested folder

Described how to mount s3 using the mount command with options.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16060/
Test PASSed.

@apc999 thanks for the review.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16066/
Test PASSed.


Cannot Resolve symbol 'Properties'

Hi Team,
I am trying out few things in Flutter.I have updated Android studio today to 3.5.1.I was able to use my git code and run my app in device.I tried to configure app for firebase and added lines as per firebase instructions.Firebase said my app is communicating with firebase .I ran the app once again and it started to throw this error.I believe this error is related to gradle but not sure how to make this work.
I can see below error:
settings.gradle highlights Properties and File in red color saying cannot resolve symbol 'Properties'
include ':app'

def flutterProjectRoot = rootProject.projectDir.parentFile.toPath()

def plugins = new Properties()
def pluginsFile = new File(flutterProjectRoot.toFile(), '.flutter-plugins')
if (pluginsFile.exists()) {
    pluginsFile.withReader('UTF-8') { reader -> plugins.load(reader) }
}

plugins.each { name, path ->
    def pluginDirectory = flutterProjectRoot.resolve(path).resolve('android').toFile()
    include ":$name"
    project(":$name").projectDir = pluginDirectory
}

**build.gradle
buildscript {
    ext.kotlin_version = '1.2.71'
    repositories {
        google()
        mavenCentral()
        jcenter()
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:3.2.1'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
        //classpath 'com.google.gms:google-services:4.3.2'
    }
}

allprojects {
    repositories {
        google()
        jcenter()
    }
}

rootProject.buildDir = '../build'
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
    project.evaluationDependsOn(':app')
}

task clean(type: Delete) {
    delete rootProject.buildDir
}

gradle-wrapper.properties
#Fri Jun 23 08:50:38 CEST 2017
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip

Flutter info
Flutter 1.9.1+hotfix.4 • channel stable • https://github.com/flutter/flutter.git
Framework • revision cc949a8e8b (9 days ago) • 2019-09-27 15:04:59 -0700
Engine • revision b863200c37
Tools • Dart 2.5.0

Android version:
3.5.1

@sagar90k Which firebase plugin do you use?
Please try to change maven to
  maven {
    url 'https://maven.fabric.io/public'
  }

as it's said in https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_crashlytics.
Also, I'm afraid there will be error with google-services. If it will be, try to change the version as per this comment https://github.com/flutter/flutter/issues/38163#issuecomment-520429066.

Thanks for the reply!
Meanwhile I cloned my project from git once again and ran it .App was launched in device.
This cloned project doesnt have anything to do with firebase.When I open the settings.gradle, it highlights same thing 'Properties' and 'File' in red color in settings.gradle file.
Next, I stopped main.dart and once again ran  the main.dart and it still built the apk and ran successfully. Weird behaviour.
Now I dont have anything related to firebase in project , here is my content.
*build.gradle
buildscript {
ext.kotlin_version = '1.2.71'
repositories {
google()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.2.1'
    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
}

}
allprojects {
repositories {
google()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}
gradle-wrapper.properties
#Fri Jun 23 08:50:38 CEST 2017
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https://services.gradle.org/distributions/gradle-4.10.2-all.zip
Next thing I did is I invalidated cache and restarted just to check if everything works fine after restart and it worked all great
Now if i want to follow steps for firebase once again then I think I will run into same problem again.Can anyone tell like what is causing this problem?Or what should be the best practice here so as to remove that red colored Properties and File in settings.gradle?

@sagar90k I may be wrong, but I think it was absence of maven (not mavenCentral). Please try again following the instruction and keeping google-services v4.2.0. But anyway, what the firebase plugin you use?

Meanwhile i have just added firebase json and updated the given lines in gradle files as per firebase page.I deliberately used 4.3.1 instead of 4.3.2 for google services.It still worked and app was launched.We can close this for now.Thanks for the help!


Cannot npm install package because invalid peer dependencies

Steps I used:

Checked out repo
npm install - got an error.
npm list for more details.

Slightly more details here: https://github.com/mkay581/scroll-js/pull/6#issuecomment-125766526
My environment:
$ npm --version
2.11.2
$ node --version
v0.12.5

The error:
npm ERR! peer invalid<EMAIL_ADDRESS>/path/to/build-tools/node_modules/phantomjs

Thanks, this uses an older version of mocha-phantomjs package which requires a phantomjs explicit dependency no later than version 1.9.15. but you ended up installing the latest (1.9.17), which gives that error.
To fix this, I could have just locked the version of phantomjs to 1.9.15, but instead of doing that, I thought it would be more efficient to just remove the phantomjs dep altogether. Since this package doesnt need it explicitly anyway, and the new version of mocha-phantomjs no longer requires explicit phantomjs packages.Your issue here is similar to this issue.
Anyway, the fix can be found in latest 1.6.0 of this package which is now published. I'll hop over and add it to the scroll-js package so that it uses it.

FYI, here is the commit that fixes this: c3550c2838aa1b17f739b6d24f6904449ba02845

Nice.
Thanks!

did this fix your npm install issue? if so, i'll close.

Yep, looks like it did.  Thanks!

cool thanks


Missing docstrings - metric

deepreg/model/metric.py

[ ] Missing module docstring
[ ] Missing class and method docstrings: MeanWrapper
[ ] Missing class and method docstrings: MeanDiceScore
[ ] Missing class and method docstrings: MeanCentroidDistance
[ ] Missing class and method docstrings: MeanForegroundProportion

close this issue as those classes have been removed, after using dictionary for inputs and outputs


[WIP] feat(makefile): add operator-sdk-cli support

as the image exists on quay, it's useful to be able to override it and use it instead of the binary

why would this fail?
error
time="2020-12-15T10:17:53Z" level=error msg="error disabling `generate kustomize manifests` interactive mode: unable to find the content to be replaced"
exit status 1
make: *** [generate] Error 1
Makefile:39: recipe for target 'generate' failed
Error: Process completed with exit code 2.

link to raw input
https://pipelines.actions.githubusercontent.com/rCxJNnmnPxceTulPiKjy4D10BPNEoNpJWKjKpuP5HwWzF9JoSn/_apis/pipelines/1/runs/5647/signedlogcontent/4?urlExpires=2020-12-15T10%3A31%3A12.0618534Z&urlSigningMethod=HMACV1&urlSignature=zLalsEE8voE8lv8K4lQJOQIqscRC4XgEfuoAvAJKxUk%3D

seems that the CLI is running as root, which makes this command fail

sure, thanks 👍
and btw, what did you think of the change?

@camilamacedo86 it seems the error I get in the CI is persistent when I run make generate
ERRO[0000] error disabling `generate kustomize manifests` interactive mode: unable to find the content to be replaced

same error error creating the bundle: unable to find the content to be replaced
not sure what is causing it

I can fix these, but I am not sure how to fix the underlying issue..
I'll close it once we get to the bottom of this.
/lifecycle frozen

Maybe not, then
/remove-lifecycle stale

I guess this is not getting traction, so I'll close it for now
Hopefully this will be picked up and merged at some point


Updated Readme + Contributing docs

This just tweaks the README.md and CONTRIBUTING.md docs a bit + updates them with post-launch details (This is part of https://github.com/flowtype/flow-typed/issues/254).
Note that assumes the flow-typed install command (no flags) feature has landed, so it shouldn't land until after that does.

Aw, it looks like @GitHub missed a critical aspect of accept/request changes: That when an update comes back after an RC, it should reset :(

Hey @jeffmo, quick q. Do you recommend to check in lib typedefs (i.e., flowtyped dir) into version control? Why or why not?

@migueloller: Yes, we recommend checking in libdefs.
Libdefs are tagged at flow/library version when installed, but libdefs themselves -- even for the same flow/library version -- can improve over time. There may be a bug or an improvement to accuracy, etc.
So in the scenario where a libdef was improved, it's possible that that improvement may cause new Flow errors that weren't previously present in your repo (similar to what happens when you upgrade Flow: Flow gets smarter with each release, so it may find new issues it wasn't able to before). And thus, we want to make sure your flow errors are consistent over time for each checkout of the library.
If/when you want to upgrade a libdef, simply update it and check that update (plus any potential fixes) in for all of your library's contributors to benefit from as well.

Thanks for asking that question, I'll add the above to the README here :)


Add automation for Spirit Turtle's Environmental Balance

Also add aura for Pacifying Aura, add Constrict ability, and add damage links for Unbalancing Stomp.

Also needs spells adding, but that can be follow-up PR.


Feature/usenet

Category

Feature

Overview

Integrate Sabnzbd

Issue Number (if applicable)

Related issue: #248

Screenshot (if applicable)

If you've introduced any significant UI changes, please include a screenshot.

I noticed that "Get api key here" redirects me to <hostname>/sabnzbd/. I think it should open <hostname>/sabnzbd/config/general/ as it does with all BitTorrent clients.

I noticed that "Get api key here" redirects me to <hostname>/sabnzbd/. I think it should open <hostname>/sabnzbd/config/general/ as it does with all BitTorrent clients.

To be honest that's outside the scope of this integration I'd say since that is the same case for pretty much every other integration.

Sorry to ask for this but can you change the way the other integrations get the selected service by making a function "getServicesByType" that returns a list of services. This way we can return an error if no service of the wanted type is present and iterate if multiple services are present (for example multi sonarr instances with calendar)

That's outside the scope of this PR.

I agree with @JannesV . This seems way out of scope. Can we merge this, so 0.10 can go out?
Or is there anything else we need to merge?


General cert name

Apple recently changed their APNS handling and new certificates are named "Apple Push Services" instead of "Apple Production IOS Push Services". When I now try to import one of these Pusher refuses to import them.

+1

I added the 'Simplified' certificate type in commit 49ca3dd. @whiskey could you see if that fixes the issue? Thanks.

The new certificates can now be selected but on connection I get a SSL handshake error:
[10:08:24.081366 Pusher NWAppDelegate.m:334] [warn] Unable to connect: SSL handshake failed (-9806)
Let me know if you need further testing or review.

Thanks for checking that. It looks like APNS handles connections differently for this new kind of certificate. The SSL handshake fails because of either the connection is reset (ECONNRESET) or a broken pipe (EPIPE), see NWSSLConnection.m for more details. I currently don't have such a certificate to further investigate the matter, but will get back to you once I do.

For the sake of better error reporting, I added this errSSLClosedAbort (-9806) error in commit 504ab4d.

I had the same issue but the new certificate type works for me in version 0.6.4.

@leonardvandriel, I'm trying to use production certificate to send push notification but its not showing up so I couldn't select it, and even tried importing .p12 wouldn't help.

any plan to release new build for Apple's updates? @leonardvandriel

@hemang-azilen, which particular Apple's update are you referring to? I did a small update (0.6.4) that should add some support for the new certificates. I'm currently not planning to add support for the HTTP/2 APNS.

@leonardvandriel , I'm referring to this, https://developer.apple.com/news/?id=12172015b
Please have a look.

Thanks, I'm aware of this. This update consists of a set of changes. Are you referring to the Simplified Certificate Handling?

Yes, may be. NWPusher was working fine before with either development or production certificate. Now its only working with Development.

Could you describe how it's not working? Maybe an error message or code.

I see, you say it's not showing up. This is probably because you're using certificates that can be used for both production and development, which aren't showing up as such anymore. There's a pull request that deals with this (https://github.com/noodlewerk/NWPusher/pull/11). I'm looking into merging this. In the mean time you could give that a try.

I'm getting following message inside NWPusher when tried to import .p12 from a production certificate (which I'd created after Apple changed their way of creating production certificates).

@leonardvandriel should I update my pull request to be compatible with most recent codebase?

Thanks all for your patience. I've merged @666tos pull request and support for both production and sandbox on simplified certificates is now available.
@666tos Thanks for picking this up.
@hemang-azilen Can you verify the issue is now resolved?

@leonardvandriel, that's great, thanks for this! Its working now! Bingo!! :+1:


Update build-configuration.md (add nx.dev)

Hey @waylun - I see that the build directory you used includes the apps directory. I'm not familia with NX, but looking at the docs, it appears like that may be non-standard? Any thoughts there?

Hey @waylun - I see that the build directory you used includes the apps directory. I'm not familia with NX, but looking at the docs, it appears like that may be non-standard? Any thoughts there?

Hi @signalnerve, thanks for the quick response. So NX is a monorepo that can host multiple applications in one repo, so the apps directory may include as default for build distribution.

Hey all - we've gotten some feedback internally from the team that NX isn't a good fit for this section, because it's a monorepo tool (not a framework). Going to close this!


how to set the ACL parameter

anybody know how to set the format of ACL when to call acreate function

zookeeper.h

zookeeper.h
/** This is a completely open ACL*/
extern ZOOAPI struct ACL_vector ZOO_OPEN_ACL_UNSAFE;
/** This ACL gives the world the ability to read. /
extern ZOOAPI struct ACL_vector ZOO_READ_ACL_UNSAFE;
/* This ACL gives the creators authentication id's all permissions. */
extern ZOOAPI struct ACL_vector ZOO_CREATOR_ALL_ACL;
so you can register these struct to lua.

from zklua.c(line 320) function _zklua_parse_acls, we can see ACL is a table in Lua, and you can pass N ACL, each ACL is consist of three items, the code is like below
local acl = {      {            perms = 31,          scheme = "world",         id = "anyone"     }    } ret, err = zklua.create(zh, "/zklua/test", "test", acl, 0)
the normal return code of the create is 0 and error code is descripted in https://github.com/apache/zookeeper/blob/master/src/c/include/zookeeper.h#120.
Hope to help.


Provide clickable buttons in header to jump to section headers

Perhaps not necessary. Page is rather clean looking and small.


CI/CD codespell check always fails with false positives

Self Checks

[X] This is only for bug report, if you would like to ask a question, please head to Discussions.
[X] I have searched for existing issues search for existing issues, including closed ones.
[X] I have searched for existing pull requests search for existing pull requests, including closed ones.

Environment
N/A - github related
Steps to reproduce

Initiate a PR :recycle:
Look at checks :ballot_box_with_check:

Prospective Issue

crate for rust packages is being mistaken for a typo for create
codespell also thinks ratatui is misspelled

✔ Expected Behavior

❌ Actual Behavior

Our test project will mainly shift to Rust and Java in the follow-up, and this test task will be cancelled first.


Open Cloud Support

@Vorlias, I'm going to mark this ready for review. Currently, we need to think of a way for asset downloading through open cloud (or w/o a cookie) before this can be downloaded.

This should fix it 🤞

Looking through it now, sorry for the long wait

What version of Rust did you do this on? seeing a few syntax errors popping up in CI as well as on my local machine.

I used 1.66.1.

I'll go ahead and fix those errors, I thought it worked but I must be wrong


WAF Proxies, Keep refreshing, not solving the captcha

Greetings and Really thank you for your library,
It seems that when use proxies and try to solve the captcha, it was detected and keep retrying to solve it.
How to fix that? Thanks

Please send the website link

Please send the website link

What do you mean?

This is not a problem for a few people affecting the general situation.
https://github.com/zfcsoftware/puppeteer-real-browser/issues/80
Since this has been discussed, I will close this.

This problem appeared in a few people. In general, not many people have the problem. zfcsoftware/puppeteer-real-browser#80 Since this has been discussed, I will close this.

hi, considering it was fixed in that repo, could you fix it in here too? looking at the dependencies this one doesnt use puppeteer-real-browser

Hello, I will fix it soon.

This problem appeared in a few people. In general, not many people have the problem. zfcsoftware/puppeteer-real-browser#80 Since this has been discussed, I will close this.

hi, considering it was fixed in that repo, could you fix it in here too? looking at the dependencies this one doesnt use puppeteer-real-browser

I fixed it.


Unexpected behavior when reflecting on constructor that has parameter that sets instance variable using syntactic sugar

Given the constructor below which uses syntactic sugar to assign a constructor argument to an instance variable.
class Sugar {
  Sugar({this.foo = 222});

  final int foo;
}

When I invoke the constructor using newInstance without any arguments, then the default value of the constructor parameter does not seem to be used. In the example above, after calling newInstance without any arguments, the instance variable foo is equal to null. I'd expect the instance variable foo to be equal to 222 after calling newInstance without any arguments.
Also, when I get the default value of the constructor parameter, it returns null. I'd expect it to return 222.
I believe the issue is around the syntactic sugar used in the constructor parameter.
I wrote a test to show the failures based on my expectations:
https://gist.github.com/fertrig/8b1c50deb09c6422779fe5014bdd8aeb
Are my expectations correct? Is there a workaround?

Sounds like a bug. (I'm on vacation, but I'll take a look next week).

So quickly a week can evaporate when part of it is vacation. ;-)
The case where a named constructor argument has a default value which is used (because no parameter with that name is passed at a call site) wasn't covered by the current tests. But I just added it (here), and that test does succeed.

Turns out that an initializing formal constructor parameter (this.something) which is optional is not of type DefaultFormalParameter in the Dart analyzer program model. So these parameters are not considered when the default value code is generated for the other optional parameters.
I don't know how to obtain that default value at this point (or even how to discover that a given initializing formal parameter has a default value), so I'll have to return to this later.

Awaiting a review, you may be able to use the fix in the branch 'fix_170_may19' by putting this into pubspec.yaml:
  reflectable:
    git:
      url: git://github.com/dart-lang/reflectable.git
      ref: fix_170_may19

This is expected to be published as 2.0.11.

Closing: The branch is now gone and 2.0.11 has been published.

Thank you so much, @eernstg!
Invoking the constructor now works as expected.
However, when I inspect the parameter with the initializing formal, its parameter mirror reports that it has a default value but it also says that its default value is null.
If a constructor parameter is initialized like this?
Foo({ this.bar = 222 });

Is the parameter bar considered to have a default value? Its parameter mirror is currently reporting that hasDefaultValue is true, but that defaultValue is null. This seems inconsistent.
One could also argue that hasDefaultValue should be false since this initializer is syntactic sugar.
Here's is a gist with a test that fails showing the issue:
https://gist.github.com/fertrig/8b1c50deb09c6422779fe5014bdd8aeb

Thanks, @fertrig!
Argh, that is another bug in the same area, defaultValue should return 222. A fix is available in branch fix_170b_may19, to be published as 2.0.12.

2.0.12 was just published (and branch fix_170b_may19 was deleted).


Fix compilation errors with musl libc

error.h is a GNU extension and is not provided by musl. Replaced with fprintf.
NULL is defined as a value of type std::nullptr_t, reinterpret_cast from std::nullptr_t to void ** is not allowed[1]. But static_cast is allowed.

[1] reinterpret_cast conversion - cppreference.com

Closing as cherry-picked

Closed wrong PR


Support gossipsub 1.2.0

https://github.com/libp2p/specs/pull/548 - this has been merged and IDONTWANT is officially part of the spec - the feature is formally only enabled in gossipsub 1.2.0, meaning we must start supporting that version as a protocol in order to receive IDONTWANT messages from other clients (putting the fields in 1.1 was a hack).

Is it enough to create a new version here https://github.com/vacp2p/nim-libp2p/blob/0911cb20f4a0b880451938988f24cb077cfd8b1b/libp2p/protocols/pubsub/gossipsub/types.nim#L22 and add it here https://github.com/vacp2p/nim-libp2p/blob/0911cb20f4a0b880451938988f24cb077cfd8b1b/libp2p/protocols/pubsub/gossipsub.nim#L197 ?

Gossipsub-v1.2 specification suggest that we MAY cancel outstanding IWANT if we receive the message from some other peer
This specification also suggests that we SHOULD cancel any queued IWANT reply if we receive IDONTWANT from the requesting peer. IMO, we have this available in this draft PR


Support Ruby v 3.0.0p0

Issue Description
I've been updating our CI images and the latest one packs Ruby v3 by default, which causes crashes due to what I assume is an old dependency on google-api-client-0.38.0. I think a later version has support for this (https://github.com/googleapis/google-api-ruby-client/pull/900).
Complete verbose output
Resolving dependencies...
google-api-client-0.38.0 requires ruby version ~> 2.4, which is incompatible
with the current version, ruby 3.0.0p0

Exited with code exit status 5

Library
Version

fastlane
2.181.0

fastlane-plugin-firebase_app_distribution
0.2.7

@lupajz Thanks for the report. Looks like we should probably update that dependency, however I'm not able to reproduce this with the latest versions of fastlane (2.187.0) and the plugin (0.2.9), when using Ruby 3.0.0p0. Could you try updating both and let me know if you're still experiencing the issue?

@lfkellogg Thank you :+1: ! It seems to work after updating the Fastlane. Somehow the CI was running some older version of it.


AOT and snapshots not supported

We've been trying to figure out how to make this plugin work for us for a really long time, and after fiddling with this, we ended up getting it working on our end by removing --env.aot and --env.snapshot form our build process, so it seems this plugin doesn't support either options being applied to the release build

I can imagine snapshots are a no-go, but I wonder why --env.aot would not work. It's simply different content being written in the "app" folder mentioned here, right?

I can imagine snapshots are a no-go, but I wonder why --env.aot would not work. It's simply different content being written in the "app" folder mentioned here, right?

in a normal angular app that is what's happening IIRC, not sure if nativescript acts the same way in that aspect though ?


Please add a GitHub sponsor button!

Description
After beating my head hard against for openapi-client-axios due the performance problems associated with a runtime configuration of the OpenAPI doc, combined with ES Module import issues due to everything being shoved into namespaces that aren't exported. The project also seems to have been more or less abandonned.
I was absolutely delighted to find a near drop in replacement that doesn't require runtime initialization and it simplified so much of my code. Not to mention the three shaking benefits inherit with the piece meal approach brought forward by this library.
There's also the issue that the other library ingests dependencies which are unlicensed.
So thank you for this great library, keep up the great work, and please add a GitHub sponsor button to this repository!

Thanks for the kind words @jscarle! Please leave any feedback you have as you start using the package(s) more!

@jscarle I created a sponsorship profile, thank you for the suggestion! https://github.com/sponsors/mrlubos


[proxy] Add the password hack authentication flow

See https://github.com/neondatabase/cloud/issues/1620#issuecomment-1165332290

Looks good, thanks!
I've set up a proxy from this branch alongside the local console and confirm that it works as desired: the usual password works with SNI, and an altered password works when SNI is disabled.
With this double base64 encoding it is hard to encode the password by hands. What do you think about using non-base64 encoded password? I think URL encoding is only needed when one uses postres://... type connection string but not needed in case of host=... pass=... type of string. So we can get away with a more transparent option.
Let's also impose a stricter schema on json just project and password (I would also suggest field password instead of token).


Use Self where possible

Retry of #433

Ok fixed and merged.

Why was this closed?

it was merged, pekka's merge-pr.py script sometimes ends up not showing it as merged in github, but you can see the commit hash above https://github.com/tursodatabase/limbo/commit/97647ff0568f2a160bf3f639440a592a18881fab

Ah ok.

Actually, this one I edited manually to drop the vendored changes and merged by hand because it was about to bit-rot again as we merge other PRs.


SQLCipher doesn't work on windows

When trying to use sqlcipher_flutter_libs package with sqlite3 and drift on windows, the build fails. Android, iOS and macOS are working. On windows I get the following errors:
libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertOpenStore referenced in function capi_open_store [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertCloseStore referenced in function capi_find_key [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertEnumCertificatesInStore referenced in function capi_find_cert [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertFindCertificateInStore referenced in function capi_find_cert [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertDuplicateCertificateContext referenced in function capi_load_ssl_client_cert [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertFreeCertificateContext referenced in function capi_find_key [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-e_capi.obj) : error LNK2019: unresolved external symbol __imp_CertGetCertificateContextProperty referenced in function capi_cert_get_fname [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_ioctlsocket referenced in function BIO_socket_ioctl [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_getsockname referenced in function BIO_sock_info [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_getsockopt referenced in function BIO_sock_error [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2001: unresolved external symbol __imp_getsockopt [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_ntohs referenced in function BIO_get_port [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2001: unresolved external symbol __imp_ntohs [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_select referenced in function BIO_socket_wait [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_gethostbyname referenced in function BIO_gethostbyname [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2001: unresolved external symbol __imp_gethostbyname [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_WSAStartup referenced in function BIO_sock_init [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2019: unresolved external symbol __imp_WSACleanup referenced in function bio_sock_cleanup_int [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_conn.obj) : error LNK2019: unresolved external symbol __imp_WSAGetLastError referenced in function conn_read [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock.obj) : error LNK2001: unresolved external symbol __imp_WSAGetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2001: unresolved external symbol __imp_WSAGetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_sock.obj) : error LNK2001: unresolved external symbol __imp_WSAGetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2001: unresolved external symbol __imp_WSAGetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_htonl referenced in function WspiapiLegacyGetAddrInfo [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_htons referenced in function BIO_lookup_ex [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_inet_addr referenced in function WspiapiParseV4Address [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_inet_ntoa referenced in function WspiapiLegacyGetAddrInfo [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_gethostbyaddr referenced in function WspiapiLegacyGetNameInfo [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_getservbyport referenced in function WspiapiLegacyGetNameInfo [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_getservbyname referenced in function BIO_lookup_ex [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_addr.obj) : error LNK2019: unresolved external symbol __imp_WSASetLastError referenced in function WspiapiGetAddrInfo [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_sock.obj) : error LNK2001: unresolved external symbol __imp_WSASetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_conn.obj) : error LNK2001: unresolved external symbol __imp_WSASetLastError [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_sock.obj) : error LNK2019: unresolved external symbol __imp_recv referenced in function sock_read [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_conn.obj) : error LNK2001: unresolved external symbol __imp_recv [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_sock.obj) : error LNK2019: unresolved external symbol __imp_send referenced in function sock_write [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_conn.obj) : error LNK2001: unresolved external symbol __imp_send [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_accept referenced in function BIO_accept_ex [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_bind referenced in function BIO_bind [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_closesocket referenced in function BIO_accept_ex [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_connect referenced in function BIO_connect [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_listen referenced in function BIO_listen [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_setsockopt referenced in function BIO_connect [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bio_sock2.obj) : error LNK2019: unresolved external symbol __imp_socket referenced in function BIO_socket [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] libcrypto64MDd.lib(libcrypto-lib-bss_conn.obj) : error LNK2019: unresolved external symbol __imp_shutdown referenced in function conn_close_socket [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\Debug\sqlite3.dll : fatal error LNK1120: 34 unresolved externals [C:\Users\elviz\OneDrive\Desktop\New folder (2)\app\build\windows\plugins\sqlcipher_flutter_libs\sqlite3.vcxproj] Exception: Build process failed.

It looks like statically linking OpenSSL fails. Did you install OpenSSL development libraries on your Windows build machine? I'll also take another look at this, but it will take a few days until I have access to a Windows VM again.

Yes, I did.
openssl version
OpenSSL 3.1.1 30 May 2023 (Library: OpenSSL 3.1.1 30 May 2023)

I fixed this by adding following lines into ./windows/CMakeLists.txt
include_directories("C:\\Program Files\\OpenSSL-Win64\\include") link_directories("C:\\Program Files\\OpenSSL-Win64\\lib") link_libraries(libssl libcrypto)

Did you install OpenSSL manually or through a package manager like Chocolatey? I think there's a way to add OpenSSL to PATH so that it gets found by CMake by default.

I installed it through choco, and it was added to PATH automatically.

CMake Error at C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.25/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY
OPENSSL_INCLUDE_DIR)
Call Stack (most recent call first):
C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.25/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.25/Modules/FindOpenSSL.cmake:613 (find_package_handle_standard_args)
flutter/ephemeral/.plugin_symlinks/sqlcipher_flutter_libs/windows/CMakeLists.txt:27 (find_package)
Error: Unable to generate build files
may i know how to fix it i have ready insall openssl

I have the same issue, it keep tell me the following:
CMake Error at C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
  system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY) (found
  version "3.3.0")
Call Stack (most recent call first):
  C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.28/Modules/FindOpenSSL.cmake:668 (find_package_handle_standard_args)
  flutter/ephemeral/.plugin_symlinks/sqlcipher_flutter_libs/windows/CMakeLists.txt:27 (find_package)

I have openssl already been installed.
How can I resolve this issue?

@simolus3 I am seeing the same issue as @saadinstitute with a fresh install of Windows 10. I used chocolatey to install the SSL libs, but it seems that CMAKE can't find it. I tried setting an environment variable for the OPENSSL_ROOT_DIR in the user env, the system env, and setting a variable with that name in the windows CMAKE file. I also tried uninstalling VS2022 for VS2019 as I remember seeing a warning about VS2022 not being supported.
I am unfortunately not that familiar with how to resolve an issue like this, especially from windows. Most of the help you find online assume Linux for a CMAKE issue. I verified a non-encrypted Drift project does compile on the system. Any suggestions on other tests to make or configuration to change to let if find the SSL libraries?

I had the same problem - after some digging it appears to be an issue with latest Chocolatey OpenSSL versions (from 3.2+).
If you install version 3.1.1 that works:
choco install openssl --version 3.1.1 -y

@elvinasn Have you fixed the problem? I've tried all these solutions but none of them work, still getting the same errors.

I managed it to work by adding what @elvinasn said AND by adding sqlcipher_library_windows package.
@simolus3 Is this some weird workaround or it's intended to work this way? I feel like I'm "cheating" but @elvinasn's comment fixed those linking errors and sqlcipher_library_windows fixed an error where sqlcipher.dll was not found.

Sorry, I'm not very familiar with Windows development. I think that CMake should be able to find OpenSSL when installed through Chocolatey, at least it behaved like that in my VM. But I also don't know what could cause that to break.
Obviously the solution isn't great, but if it works and the compiled app also runs on other Windows machines without the workaround, I don't see any potential for this to go wrong or be compiled/linked in a broken way.

Sorry, I'm not very familiar with Windows development. I think that CMake should be able to find OpenSSL when installed through Chocolatey, at least it behaved like that in my VM. But I also don't know what could cause that to break.
Obviously the solution isn't great, but if it works and the compiled app also runs on other Windows machines without the workaround, I don't see any potential for this to go wrong or be compiled/linked in a broken way.

I went through sqlcipher_library_windows and I see that it's keeping required DLLs in it's assets folder, and it copies them manually to the same place where executable is generated.
Do you think it would be better to create a separated plugin for Windows platform only, where we can write a POST BUILD script that will copy all DLLs where binary is generated? With this approach we don't need to copy DLLs on other platforms since they don't need them. I can definitely do that since it's more lightweight than the mentioned package, which helps in this case.

Hello, I just ran into the exact same issue and I keep getting this error:
[ +864 ms] CMake Error at flutter/ephemeral/.plugin_symlinks/sqlcipher_flutter_libs/windows/CMakeLists.txt:36 (target_link_libraries):
[        ]   Target "sqlite3" links to:
[        ]     OpenSSL::Crypto
[        ]   but the target was not found.  Possible reasons include:
[        ]     * There is a typo in the target name.
[        ]     * A find_package call is missing for an IMPORTED target.
[        ]     * An ALIAS target is missing.

I installed OpenSSL just normally using chocolatey and have the newest version currently available on there (3.3.1). I've tried the fixes people suggested in here and nothing seemed to be working. Are there other things you guys did to fix this? I would really appreciate some help. Thanks!

Is there any progress on this issue? This will affect whether I continue to use.

Hello, i have tried multiple version of openssl via choco and by third party I'm i have pass with all errors listed in this issue With No solution


Put 'Zephyr/Arduino 101 Build Test' to allowed failures temporarily.

JerryScript-DCO-1.0-Signed-off-by: László Langó<EMAIL_ADDRESS>

Related issue: #2268
I could not found a proper fix for problem. Unfortunately I don't have much time for this issue, so I suggest to put it into allowed failures until it is fixed.

LGTM


[Storage] GA folllow-up items

[ ] (minor) We should remove ThirdPartyNotices.txt from blob & queue repo
[ ] (minor) Rollup.config.js still generate browser bundle by default, should remove it.
[ ] When file removes browser bundle from npm, it should do above 2 checks too.
[ ] Samples in general readme.md in storage folder should update to @azure/logger package, instead of customized log interface implementation

@ramya-rao-a

Remove the gulpfile that zips the browser bundle too.


Rendering is totally botched up in Internet Explorer

A quick fix here might be to add a setting that disables the django-material-admin templates for certain browsers.  I'm Ok telling my users that they should upgrade to Chrome or Edge.  But for my use case, I can't tell folks that Internet Explorer simply isn't supported.  (For whatever reason, some corporations actually force their employees to use Internet Explorer.)


[Request]: Project version

Describe the request
Get the version number from the PowerShell manifest or let the user specify one.
PowerShell version
7.2
Platform
Windows 11 Pro or Enterprise

This will be added in v0.5.0

Added in v0.5.0


Refactored subpackages not appearing in HTML docs

I triggered a build of the docs for the integration branch, and noticed that classes and methods corresponding to refactored sub-packages are completely missing e.g. resqpy.lines :

Found some answers here: https://stackoverflow.com/questions/38765577/overriding-sphinx-autodoc-alias-of-for-import-of-private-class
Looks like two-step fix:

Ensure any "inner" submodules are private
Overwrite the __module__ attribute of each class in the subpackage __init__ to __name__.

On the "integration" version of the docs, some subpackages still need to be fixed:
https://resqpy.readthedocs.io/en/integration/
These show the "inner" submodules which should ideally be private:

[ ] resqpy.derived_model
[ ] resqpy.organise
[ ] resqpy.property
[ ] resqpy.surface

And these are missing:

[ ] resqpy.fault (entirely absent)

Reopening as not quite fixed for "property" submodule: some inner empty modules are still displaying the docs:
https://resqpy.readthedocs.io/en/integration/_autosummary/resqpy.property.property_collection.html
I think this could be fixed by making the inner submodules non-public.

This issue has mostly been dealt with.  Remaining to be resolved (in latest readthedocs):

time_series package
duplication of classes in surface package
(structure of grid package documentation, subject of a separate issue)


Fix a bug where flows crash when they're closed

I’m inadvertently getting concurrency in my test. Will rework and re-open this PR.


[DFT] Building multiple DFT backends at once causes linking conflicts

Summary
When building multiple DFT backends at once, each backend defines a symbol for the many oneapi::mkl::dft::compute_forward(...) and oneapi::mkl::dft::compute_backward(...) compilation time dispatch overloads, so at linking time either one can be chosen.
To get around this I tried linking just one of the two backends, but then compilation failed because of an undefined reference to oneapi::mkl::dft::detail::descriptor<...>::commit(oneapi::mkl::backend_selector<other_backend>). This is because both ENABLE_MKLCPU_BACKEND  and ENABLE_MKLGPU_BACKEND are defined in the preprocessor so both commits are defined here -
https://github.com/oneapi-src/oneMKL/blob/develop/include/oneapi/mkl/dft/detail/descriptor_impl.hpp#L63 .
I tried getting around this by providing a definition of commit but made it a weak symbol so it can be overridden when the appropriate backend is linked. This led to problems when running the tests because https://github.com/oneapi-src/oneMKL/blob/develop/tests/unit_tests/main_test.cpp#L113 is still picking up both devices.
I believe other interfaces get around this by having a backend_selector<backend::BACKEND> parameter on all the functions, like here https://github.com/oneapi-src/oneMKL/blob/develop/include/oneapi/mkl/blas/detail/blas_ct_backends.hxx. I think the best course of action might be a spec change to include a backend_selector on compute_forward and compute_backward.
Version
This on develop, I specifically tested githash 005c3c79a44974fc8c3adebfa71d94b0fe005f33
Steps to reproduce
Build with both ENABLE_MKLCPU_BACKEND and ENABLE_MKLGPU_BACKEND set to 1.
Observed behavior
When running test_main_dft_ct I get this output:
finlay@devpc-c310:~/perf/oneMKL/build$ ./bin/test_main_dft_ct
Run this program with --terse_output to change the way it prints its output.
[==========] Running 488 tests from 3 test suites.
[----------] Global test environment set-up.
[----------] 240 tests from ComputeTestSuite/ComputeTests_in_place
[ RUN      ] ComputeTestSuite/ComputeTests_in_place.COMPLEX_SINGLE_in_place_buffer/sizes_8_batches_1_Intel_R__UHD_Graphics_750__0x4c8a_
Skipping test because: "oneMKL: DFT/compute_forward: function is not implemented Not implemented for MKLCPU"
...

Expected behavior
The MKLGPU compute_forward should be called.

One suggestion for implementing the current specification would be to use the descriptor to choose which compute_forward to dispatch to i.e.
void compute_forward(descriptor d, args...) {
  switch(d.get_backend()) {
#if ENABLE_MKLGPU_BACKEND
    case mklgpu: compute_forward_mklgpu(args...); break;
#endif
#if ENABLE_MKLCPU_BACKEND
    case mklcpu: compute_forward_mklcpu(args...); break;
#endif
  }
}

My concern with this is that is missing the intention of the specification author.

One suggestion for implementing the current specification would be to use the descriptor to choose which compute_forward to dispatch to i.e.
void compute_forward(descriptor d, args...) {
  switch(d.get_backend()) {
#if ENABLE_MKLGPU_BACKEND
    case mklgpu: compute_forward_mklgpu(args...); break;
#endif
#if ENABLE_MKLCPU_BACKEND
    case mklcpu: compute_forward_mklcpu(args...); break;
#endif
  }
}

My concern with this is that is missing the intention of the specification author.

This is for the compile time dispatching functionality? Is the "intent" you are referring to here that there should be no runtime dispatch?

@lhuot Yes to both.

I believe RNG would be in the similar situation for their oneapi::mkl::rng::generate API.
Did you check what they are doing?
@aelizaro would you have any insights to share on how RNG handles compile time dispatching for the multiple backends for the generate API?

@lhuot, @FMarno, if I understand correctly, we don't have such issue in RNG as we take backend_selector parameter for compile-time dispatching only in engine's constructor: https://github.com/oneapi-src/oneMKL/blob/4c7e7eb7227bf131d6326d3dd84f2bfc57ecf3b9/include/oneapi/mkl/rng/engines.hpp#L72 and use pimpl to create an exact implementation we need (e.g. mklcpu::create_philox4x32x10(selector.get_queue(), seed) will return derived class for mklcpu implementation)
also, to avoid having multiple oneapi::mkl::domain::foo() functions they all marked as static inline in oneMKL Interfaces headers: https://github.com/oneapi-src/oneMKL/blob/develop/include/oneapi/mkl/blas/detail/blas_ct_backends.hxx

@lhuot, @FMarno, if I understand correctly, we don't have such issue in RNG as we take backend_selector parameter for compile-time dispatching only in engine's constructor:
https://github.com/oneapi-src/oneMKL/blob/4c7e7eb7227bf131d6326d3dd84f2bfc57ecf3b9/include/oneapi/mkl/rng/engines.hpp#L72
and use pimpl to create an exact implementation we need (e.g. mklcpu::create_philox4x32x10(selector.get_queue(), seed) will return derived class for mklcpu implementation)
also, to avoid having multiple oneapi::mkl::domain::foo() functions they all marked as static inline in oneMKL Interfaces headers: https://github.com/oneapi-src/oneMKL/blob/develop/include/oneapi/mkl/blas/detail/blas_ct_backends.hxx

This is similar than the DFT descriptor constructor. But then when calling the generate APIs (that have the engine as input and not the backend selector) don't you have different implementation depending on backend for this API or is the implementation the same for any backend?

I've just noticed that I can't actually find backend_selector in the oneMKL specification. @aelizaro Do you know how it was decided which functions would have a backend_selector?

@FMarno, as I remember, it is not part of oneMKL specification (same as interfaces which take backend_selector instead of sycl::queue), it is just an extension of oneMKL Interfaces to handle compile-time backend selection case - it has just minimal support for tag-dispatching and accessing the sycl::queue itself

#342 removed the duplicate function signatures, so this should be solved now. I confirmed this by running the tests for MKLCPU and MKLGPU from the same binary!


Update improve-readability-with-high-contrast-text.chinese.md

I improved the readability of the Chinese content.

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Update documentation

This PR

updates the README to cover version 0.5.0

@moxiegirl Thank you for reviewing, I have updated the easy parts and rephrase the two sections that weren't clear.

@moxiegirl Updated.


add version back to cli options

The switch from structopt to clap seems to have introduced a regression in the CLI arguments for version.

can we do a new release with this fix?


Update links/paths in manpage, hacking.md, usage.md to reflect brew-cask structure

Old github.com links were invalid
Old Cellar-based structure was invalid
Man page regenerated, including content from #15643

Refs #15381

LGTM :+1:


[enhancement] Add a label checker to GitHub Actions

Description
PR rules related to labeling are currently not enforced. Labels are required by the development guideline for oneDAL. This adds a GitHub Action to PRs to verify that at least some label has been set to guide reviewers.
Changes proposed in this pull request:

Add a Github Action check for pull requests which checks the labels.

Will re-run CI, and use attempt 2 to compare to attempt 1 to show CI system addition is fully operational.


Optional fields are suggested with field access when attempted on a record union

Description:
$title.
This happens only for a union - for a single record type only the required fields are suggested.

Steps to reproduce:
import ballerina/io;

type Foo record {
    string s;
    int i?;
};

type Bar record {
    string s;
    int i?;
};

public function main() {
    Foo f = { s: "hello world", i: 1 };
    Foo|Bar fb = f;

    io:println(fb.);
}

Affected Versions:
$ ballerina -v
Ballerina 1.0.0-beta
Language specification 2019R3
Ballerina tool 1.0.0

Closing since the fix has been merged


[titiler.extensions] remove stamen basemap

https://github.com/developmentseed/titiler/blob/dc9e0c7a753700031799aaa4ade593dd0f0d247c/src/titiler/extensions/titiler/extensions/templates/cog_viewer.html#L328-L358
https://github.com/developmentseed/tipg/issues/104

done in #691


Hide Animation performs differently on 32 bit and 64 bit devices

@lauraskelton @devinfoley Hi,
I've discovered today that the hide animation performs differently on 64 bit devices than on 32 bit.
It seems that on 64 bit devices the interpolation of hide animation returned a fraction value that was translated to true if it was nonzero, but on 32 bit it was always 0 or 1, never a fraction.
I found that @encode(BOOL) and @encode(char) returned same on 32 bit but not on 64, and NSCFBoolean has a char type.
Please see my pull request. I hope it sticks to the original idea of the method.
Thanx,
Amir.

Great find, thanks!


Search is broken for some words

When I try to search for a word (such as Bluesky). I get a "no posts found" message.

Can anyone else reproduce it?

Chrome on MacBook, just now

Thanks for confirming it.

Probably a stopword.
Same for "not".

This is related to the old search service and it not handling taken down accounts as gracefully as it could. Searches that result in a page full of posts from a taken down account have those posts filtered out, but then the search page is effectively empty and the client doesn't request the next page since it thinks the search is empty. Should be better with the new post search coming in the next update.

The issue is still happening.

This issue is still here.

Same for london: finds accounts, but no posts
https://bsky.app/search?q=london

Work is being done on this issue according to https://github.com/bluesky-social/social-app/issues/2359.

This seems to have been fixed.

seems to have been fixed

You get search results for London now? Not me, neither on desktop nor ios.

I do for Bluesky/bluesky.

Just to add: #london with the hashtag works.

Seems to be fixed? https://bsky.app/search?q=london

That's great to know. I am now going to close this issue.


feat: update bug_report.yml

Description
Updates bug_report.yml
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Maybe the template should warn to remove API keys from the config.json?


⚠ Minecraft has degraded performance

In ea6af0b, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1171 ms

Resolved: Minecraft performance has improved in ffcde52.


Updated .gitignore for Vagrant environments

Added the necessary folders and files to the .gitignore so contributors don't commit their Vagrant Environment

Thanks!

No worries :D Really want to start contributing to CodeIgniter4 so if there's anything you need help on let me know and I'll try my best to help 😄

Always will welcome help! :) There are some issues here on GitHub that are marked help wanted that haven't been claimed by anyone, yet. There's usually a couple of un-resolved issues from users laying around that you can help out with.
If you are feeling really brave there's SQLite and MSSQL drivers that have to be ported over.
If you just want to start with smaller stuff, it's always helpful to go through the and verify accuracy of documentation against what the code will actually do, or writing tests for any place missing tests, etc.
Basically - dive right in and start submitting PR's no matter how large or small. On anything other than small items, though, I'd prefer if you open a new issue (or comment on an existing one) and we can discuss your plan for you, just to make sure we're all on the same page.

Just a quick little change as well the URL when in the repo does not work as a link - I think this is because there is a '/' at the end of the URL if you remove the '/' then it becomes a link 😃


docs: Add @xhluca to contributor list

Pull Request overview
Regarding #1194
Adding myself (Xing Han Lu, @xhluca) to the contributor list in scores.md
Let me know if there's anything else needed!
cc @KennethEnevoldsen

Wonderful will get this merged in - I can deal with the merge conflict as well

Thank you!