Add S3Cache

Is there interest in merging a cache backed with S3? If so, I will complete work on this PR.
At the moment it's incomplete:

[x] Custom bucket
[x] Key prefix
[x] has, get, set
[x] add
[x] delete
[x] clear
[x] timeout
[x] delete_many
[ ] inc, dec
[x] get_dict, get_many
[x] set_many
[x] Test

Not sure if that makes much sense - a cache is usually meant to be something very fast, and I don't think S3 is a great choice for that. AFAIK it's more something people use to put bigger files etc, but not the kind of data snippets one usually puts in a cache..

The performance of S3 is probably better than you expect. Keys can handle
thousands of reads and writes per second. Time to first byte is 100-200ms.
Both these data points are from here:
https://docs.aws.amazon.com/AmazonS3/latest/dev/optimizing-performance.html
S3 is in many ways similar to the filesystem cache. It would be useful in
situations where objects are relatively large and the cache miss time is
relatively high. For example, systems which deal with converting media
files, such as on-demand image resizing or audio re-encoding from master
copies.
S3 works in places where the filesystem cache does not, for example in AWS
compute resources. Most particularly Lambda, but also EC2/ECS/EKS, where
the filesystem cache could only work via EFS.
On Tue, 4 Jun 2019 at 20:32, Adrian<EMAIL_ADDRESS>wrote:

Not sure if that makes much sense - a cache is usually meant to be
something very fast, and I don't think S3 is a great choice for that. AFAIK
it's more something people use to put bigger files etc, but not the kind of
data snippets one usually puts in a cache..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pallets/cachelib/pull/6?email_source=notifications&email_token=AAC4U5PWQRGWI3HJ5VFBGHTPYZACJA5CNFSM4HS2MT22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW4EQTQ#issuecomment-498616398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC4U5JXW6IVNNEIEI5OVPLPYZACJANCNFSM4HS2MT2Q
.

ok fair enough, for large cached objects it makes more sense

This is ready for review. Caveats / differences from base cache implementation:

I didn't implement inc/dec because it seemed pointless
TTL is implemented by setting the Expires header. I added a parameter to automatically clean up stale data, but real production use should probably rely on a S3 bucket lifecycle rule to clean up old objects.

I've added pytest for S3Cache but it's not hooked up to tox or Travis or what have you. I don't think that's in scope for this PR. You can run the tests with python -m pytest.

Also, this PR only works on Python 3. I assume that's fine? Also, it would be nice to bump the version of this package to 1.0.0 before merging this PR, and merge it as 1.1.0 (so the version numbers are useful).

Finally, testing cache expiry could be sped up with https://github.com/spulec/freezegun (I won't do it as part of this PR, this is just a note for the future).

@ThiefMaster @alexjurkiewicz Wondering would it be possible to merge this? :)
Superset's documentation is mentioning an s3cache library but that package is not compatible with the werkzeug's abstract class anymore. Since most of the caching methods were moved here, It would be awesome to have S3 cache option back without hacking around this functionality. :)
https://superset.incubator.apache.org/installation.html?highlight=cache

Sorry, someone else will have to pick up the torch here, I'm not interested
in working further on it.
On Thu, 12 Mar 2020, 5:15 pm Ronny Pfannschmidt<EMAIL_ADDRESS>wrote:

@RonnyPfannschmidt requested changes on this pull request.
In cachelib/s3.py
https://github.com/pallets/cachelib/pull/6#discussion_r391419340:

@@ -0,0 +1,191 @@
+# -- coding: utf-8 --
+import datetime

+try:

import cPickle as pickle
+except ImportError:  # pragma: no cover
import pickle

+try:
this bit has to go in the init, else the complete package fails
without boto3 installed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pallets/cachelib/pull/6#pullrequestreview-373300032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC4U5IMBT7FIP6YY3KHFC3RHB4YLANCNFSM4HS2MT2Q
.

I don't think S3 is a good choice for cache.


Added a few tests for sub queries used for tables and where conditions

Codecov Report

Merging #6 (aca5164) into develop (8bcaf5b) will increase coverage by 0.28%.
The diff coverage is n/a.

:exclamation: Current head aca5164 differs from pull request most recent head 51dee4b. Consider uploading reports for the commit 51dee4b to get more accurate results

@@              Coverage Diff              @@
##             develop       #6      +/-   ##
=============================================
+ Coverage      97.61%   97.89%   +0.28%
  Complexity       267      267
=============================================
  Files             11       11
  Lines            714      714
=============================================
+ Hits             697      699       +2
+ Misses            17       15       -2

Impacted Files
Coverage Δ

src/QueryBuilder/WPDBAdapter.php
97.70% <0.00%> (+0.91%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8bcaf5b...51dee4b. Read the comment docs.


Improper outputs generated for ME-EPI

While at #2133, I've noticed that we are generating single desc-preproc_bold.nii.gz files, however, AFAIK, we are not producing the BOLD signal corresponding to the combination of echoes.
In other words, we should be generating one preproc file per echo (please correct me if I'm wrong @emdupre, @tsalo), or else actually compute the final combination. Given the current state of things, I would incline for generating the different echoes.
This also stands for surface and CIFTI-2 outputs.
/cc @mnarayan @shotgunosine

We should in fact be generating the optimal combination ! Unless something has changed to remove the T2* map interface.
The echo-idx flag allows users to avoid this and instead return individual echoes.

We should in fact be generating the optimal combination !

Is this a fact or an indication? If we are generating the optimal combination this issue can be closed. In that case, sorry for my ignorance.

I can confirm that we previously generated the optimal combination. As to whether we are still generating it with the latest code base, I'll be able to look at the workflows later tonight. From my recollection of #2109, we were still generating the optimal combination when that was merged.

Awesome, thanks for calling out this false alarm so quickly. I'll let you close the issue whenever you can confirm the implementation :)

On a related note, I tried to use the following command --t2s-coreg and --echo-idx $no and I am not sure exactly how the individual echoes are processed in this case. Ideally I am looking for individual echoes processed using the same transforms as the optimal combination image when --t2s-coreg alone is performed.
If these two options cannot be used together that would be good to know.

The connections all seem to be there, @oesteban , so I think this can be resolved:
https://github.com/poldracklab/fmriprep/blob/3701780e73a0cb7c32ef203dea176e1426555e04/fmriprep/workflows/bold/base.py#L505
@mnarayan you're right that these two can't be used together ! The way that we implement echo-idx is that we subselect the relevant echo from the original MEEPI list:
https://github.com/nipreps/niworkflows/blob/9b0f64659017f869530d9a33cf2a7b5c87b1e8c4/niworkflows/utils/bids.py#L184
I'd say that we should update the documentation but we've now removed the t2s-coreg option from master (see the discussion around #2109 ). Which release are you using, @mnarayan ?

Thanks much!
Do you think @emdupre and @mnarayan this deserves discussion on a new issue, at least noting the documentation misses?

I suppose that's my question 😄 I'm not entirely familiar with the new release cycle, so I'm wondering if there's a way in which we could still release documentation updates for an older version. I'm happy to follow your suggestion here, @oesteban !

@emdupre I've been using the v20.0.1/v20.0.7, so I wasn't aware of the change.
@oesteban I will open a new issue about this, as I have more questions anyway.


Post Ticket

Hi Shameem,
Wonderful ticket system, very clean interface.
For some reason, nothing happens when I submit a new ticket. Do you think I might be missing something? I use PHP 7 with latest MySQL.
Any help from you is much appreciated.

Thanks for reporting the issue. You need to downgrade your PHP as it support 5.6 only.


Customizable question mark

Automatically casting to Series<'K, float> when writing frame?ColName is not ideal for all applications. Perhaps make it possible to change the behaviour by opening a different namespace?

Closing old issue


⚠ GitHub has degraded performance

In 6cc62c9, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: GitHub performance has improved in 1f79280 after 1 hour, 15 minutes.


Use event priorities instead of relying on plugin load order

I use YAML inheritance so I can separate the time and overhead from with-pdf and print-site from regular execution. This results in these plugins occurring first in the list.
MkDocs 1.4 added the ability to declare event priorities, so it should be easy to add @plugins.event_priority(-100) to each event to enforce proper event order.
https://www.mkdocs.org/dev-guide/plugins/#event-priorities

That doesn't garantuee that is plugin is last.
To separate from normal execution, set the enabled option to false for both these plugins. All plugins now have this option.
Then you can use an environment variable to turn them on. See https://www.mkdocs.org/user-guide/configuration/#environment-variables
So something like:
enable_print=True && mkdocs build

I've also documented this in the options section: https://timvink.github.io/mkdocs-print-site-plugin/options.html

Closing as won't fix


How to set envirmental variables

1-
npm install dotenv --save
2-
Add the line require('dotenv').config()
3-
Add a file .env with :
CLIENT_ID=
CLIENT_SECRET=

@Jiasuo Please be more verbose.
Add the line where?
Add the file where?

You can log in  https://developers.facebook.com/
and create a demo app over their.It will give you CLIENT_ID, CLIENT_SECRET
And paste in  server .js
Similarly it can be done for other types of third party authentications.


Data Access should have authenticationMethod

Example:
authenticationMethod: API key, HTTP Basic, OAuth, No authentication.
Credentials are never distributed with specs.

Add new element


Bump galera storageRequest to 5G

Use a more production valid storageRequest in the sample files for galera.

not sure if this is to high for ci, but lets see


Update port-map to not use google code

Unfortunately the only url I could find was a latest download. I still think this is an improvement from having a google code url.
See #10461

Thank you for the contribution. It was merged directly as 033571897f661d1ba327a29ae3f283a9e5b476ab to keep commit history cleaner. Your contribution is still credited to you.


[WIP] Produce test set sample from labelled dataset to benchmark prediction

Take top 50/(200) most frequently visited pages (February 2019), pair up against all of the pages in the labelled dataset from our taxonomy days (~206K items) and save out pairs and node data (tfidf vectors)

Closes #10 and produces data for #11


derived_from_specimen takes different values in GDC and PDC

PDC specimen of type "sample": Initial specimen
GDC specimen of type "sample": Initial sample

fixed by Commits ff580e7 and ac7d995 to integration branch


Support Solr 5, Solr 6

Task checklist:

[x] Determine API for getting Solr information.
[x] Query Solr info, get version
[x] If version >= 5.x, copy basic_configs/conf to new directory location.
[x] Copy and rename managed_schema to schema.xml
[x] If necessary, create data folder. I think Solr will do this automatically, but need to confirm.
[ ] Fix configuration to  <schemaFactory class="ClassicIndexSchemaFactory"/>, removing existing element.
[ ] Test with 5.0, 6.3

Works fine with 4.1 and 4.10

Moving this to Roadmap.  5.0 has a significantly different API (e.g cores are created from command line), and the distribution no longer ships with a "collection1".  I'd like to see if there is a way to find out from the URL where the Solr file system is located, even if no core is loaded yet, Then I could implement a process similar to the one with 4.x, except that I will need to copy the conf file from server\solr\configsets\basic_configs\conf, and will need to rename (or directly use) manged-schema instead of schema.xml.

Did some exploring of the Solr API, and you can access the version and path via http://localhost:8983/solr/admin/info/system.
With this in hand, we can query the Solr version, and if 5.0 or above, we can generate the instanceDir from the basic_configs directory.

Task checklist:

[ ] Query Solr info, get version
[ ] If version >= 5.x, copy basic_configs/conf to new directory location.
[ ] Copy and rename managed_schema to schema.xml
[ ] If necessary, create data folder. I think Solr will do this automatically, but need to confirm.
[ ] Test with 5.0, last 5, 6.0, last 6.

As an interim step, throwing meaningful error if Solr 5:

Index creation appears to work with 5.0, but I am getting this error when I try to build the index:
ERROR: [doc=sitecore://master/{975ca216-d3ea-43a9-af69-964daab8bc11}?lang=da&ver=1&ndx=sitecore_master_index] unknown field 'name_t_da'

Receiving this error when rebuilding "sitecore_master_index" on Sitecore 8.2 with Solr 5.0.0:
Job started: Index_Update_IndexName=sitecore_master_index|#Exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> SolrNet.Exceptions.SolrConnectionException: <?xml version="1.0" encoding="UTF-8"?> <response> <lst name="responseHeader"><int name="status">400</int><int name="QTime">1</int></lst><lst name="error"><str name="msg">This IndexSchema is not mutable.</str><int name="code">400</int></lst> </response>  ---> System.Net.WebException: The remote server returned an error: (400) Bad Request.    at System.Net.HttpWebRequest.GetResponse()    at HttpWebAdapters.Adapters.HttpWebRequestAdapter.GetResponse()    at SolrNet.Impl.SolrConnection.GetResponse(IHttpWebRequest request)    at SolrNet.Impl.SolrConnection.PostStream(String relativeUrl, String contentType, Stream content, IEnumerable1 parameters)
--- End of inner exception stack trace ---
at SolrNet.Impl.SolrConnection.PostStream(String relativeUrl, String contentType, Stream content, IEnumerable1 parameters)    at SolrNet.Impl.SolrConnection.Post(String relativeUrl, String s)    at SolrNet.Impl.SolrBasicServer1.SendAndParseHeader(ISolrCommand cmd)
at Sitecore.ContentSearch.SolrProvider.SolrBatchUpdateContext.AddRange(IEnumerable1 group, Int32 groupSize)    at Sitecore.ContentSearch.SolrProvider.SolrBatchUpdateContext.Commit()    at Sitecore.ContentSearch.SolrProvider.SolrSearchIndex.PerformRebuild(Boolean resetIndex, Boolean optimizeOnComplete, IndexingOptions indexingOptions, CancellationToken cancellationToken)    at Sitecore.ContentSearch.SolrProvider.SolrSearchIndex.Rebuild(Boolean resetIndex, Boolean optimizeOnComplete)    --- End of inner exception stack trace ---    at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)    at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)    at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)    at Sitecore.Reflection.ReflectionUtil.InvokeMethod(MethodInfo method, Object[] parameters, Object obj)    at Sitecore.Jobs.JobRunner.RunMethod(JobArgs args)    at (Object , Object[] )    at Sitecore.Pipelines.CorePipeline.Run(PipelineArgs args)    at Sitecore.Pipelines.DefaultCorePipelineManager.Run(String pipelineName, PipelineArgs args, String pipelineDomain)    at Sitecore.Jobs.Job.ThreadEntry(Object state)

And with 6.3.0, I get this error when rebuilding sitecore_master_index:
Job started: Index_Update_IndexName=sitecore_master_index|#Exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> SolrNet.Exceptions.SolrConnectionException: <?xml version="1.0" encoding="UTF-8"?>
<response>
<lst name="responseHeader"><int name="status">400</int><int name="QTime">0</int></lst><lst name="error"><lst name="metadata"><str name="error-class">org.apache.solr.common.SolrException</str><str name="root-error-class">org.apache.solr.common.SolrException</str></lst><str name="msg">This IndexSchema is not mutable.</str><int name="code">400</int></lst>
</response>
 ---> System.Net.WebException: The remote server returned an error: (400) Bad Request.
   at System.Net.HttpWebRequest.GetResponse()
   at HttpWebAdapters.Adapters.HttpWebRequestAdapter.GetResponse()
   at SolrNet.Impl.SolrConnection.GetResponse(IHttpWebRequest request)
   at SolrNet.Impl.SolrConnection.PostStream(String relativeUrl, String contentType, Stream content, IEnumerable`1 parameters)
   --- End of inner exception stack trace ---
   at SolrNet.Impl.SolrConnection.PostStream(String relativeUrl, String contentType, Stream content, IEnumerable`1 parameters)
   at SolrNet.Impl.SolrConnection.Post(String relativeUrl, String s)
   at SolrNet.Impl.SolrBasicServer`1.SendAndParseHeader(ISolrCommand cmd)
   at Sitecore.ContentSearch.SolrProvider.SolrBatchUpdateContext.AddRange(IEnumerable`1 group, Int32 groupSize)
   at Sitecore.ContentSearch.SolrProvider.SolrBatchUpdateContext.Commit()
   at Sitecore.ContentSearch.SolrProvider.SolrSearchIndex.PerformRebuild(Boolean resetIndex, Boolean optimizeOnComplete, IndexingOptions indexingOptions, CancellationToken cancellationToken)
   at Sitecore.ContentSearch.SolrProvider.SolrSearchIndex.Rebuild(Boolean resetIndex, Boolean optimizeOnComplete)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Sitecore.Reflection.ReflectionUtil.InvokeMethod(MethodInfo method, Object[] parameters, Object obj)
   at Sitecore.Jobs.JobRunner.RunMethod(JobArgs args)
   at (Object , Object[] )
   at Sitecore.Pipelines.CorePipeline.Run(PipelineArgs args)
   at Sitecore.Pipelines.DefaultCorePipelineManager.Run(String pipelineName, PipelineArgs args, String pipelineDomain)
   at Sitecore.Jobs.Job.ThreadEntry(Object state)

Looks like issue is this element:

Thanks, Stack Overflow! http://stackoverflow.com/a/31721587/402949

Tested 8.1 against Solr 6.3, and 8.2 update 1 against Solr 4.10, Solr 5.0 and Solr 6.3.   Testing consisted of rebuilding the sitecore_master_index and confirming that: (A) documents appeared in Solr, and (B) content editor search worked. I also rebuilt all indexes for the SC 8.2 + Solr 6.3 instance.
As noted in #41, Sitecore 8.0 is not working with Solr installation. This looks like a preexisting issue.


Prepare release 1.18.1

What does this PR do?
1.18.1 release with spammy telemetry removed.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1361 (4b29c3e) into release/1.18.0 (e3b23a7) will increase coverage by 0.02%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           release/1.18.0    #1361      +/-   ##
==================================================
+ Coverage           81.86%   81.88%   +0.02%
==================================================
  Files                 362      362
  Lines               12903    12903
  Branches             2164     2164
==================================================
+ Hits                10563    10565       +2
- Misses               1680     1682       +2
+ Partials              660      656       -4

see 13 files with indirect coverage changes


Update pubspec.yaml

input_signature.dart:12:67 - This API is available since SDK 2.14.0, but constraints '>=2.12.0 <3.0.0' don't guarantee it.

Thanks!


Unity Constants generator enhancements

The MakeSafeForCode function currently does not handle dashes inside the layer names, a regex to strip all special characters from layer names makes it more robust.
Added comments to the Layer incides/bitmasks that show up in intellisense.

Looks like good changes. Thanks for the contribution!


[Request] Expose cached offsets and heights

Is your feature request related to a problem? Please describe.
VListHandle exposes a cache object with really useful details about the virtualized rendering — but unfortunately the keys are obfuscated in production and the CacheSnapshot type is opaque. I think there are valid use cases for consumers to access this data and would love to see it exposed in a way I know I can rely on.
One example use case — true sticky items.
Virtualization makes position: sticky challenging, because the sticky element may get scrolled outside of the rendered range and disappear. The official Sticky demo skirts around this with pseudo-virtualization — each group is virtualized to ensure that the header will always be rendered. However this creates limitations, such as making it impossible to scrollToIndex(...) on any particular row (#241).
However, if we know the virtual position of all the rows, we can actually have true sticky headers by rendering them next to the virtualized items. I've got an example in this branch — load it up in Storybook and you can see that to the user it looks exactly the same as the official Sticky demo. But now, each row is individually virtualized, meaning we can be more efficient with our overscan and use imperative functions like scrollToIndex(...) as expected.
However, this technique relies on grabbing offset data from the cache. I feel hesitant doing that, especially when in production it's clearly not meant for human consumption.
Is there any reason not to expose the inner data of cache so we can rely on it? Thanks!
Describe the solution you'd like
VListHandle.cache should not have its keys obfuscated in production, and should have transparent types and documentation.
Describe alternatives you've considered
We can already use the data in VListHandle.cache — but since it's not exposed to users I don't know if it's safe to assume it will always be available in that format.

@inokawa Thanks for the feedback!
Have you verified that a rangeExtractor-like solution would actually enable sticky positioning in virtua? I've been testing with the current implementation (setting a huge overscan) but can't seem to make stickiness work. I think we'd need a custom Item because by default it's position: absolute which prevents my rendered components from sticky-ing to the actual scrolling context. Even with custom Item, we have to overwrite position to sticky and use top for the stuck position — which means we need to adjust the offset with margin, or translateY, and it seems to bug out whenever I try.
I can kind of fake it by creating a fixed-height container within the custom Item, with a much larger-height sticky bounding box that overflows outside of that:
const Item = forwardRef<HTMLDivElement, CustomItemComponentProps>(
  ({ children, style, index }, ref): ReactElement => (

 {/* The virtualized item will be this height */}

 {/* The sticky content will scroll within this box */}

            {children}

  ),
)

In any case, the vanilla version of this still only enables "global stickiness" instead of "ranged stickiness". Meaning we can set a "top: X" but if we don't know the offset of the next sticky header we can't set a "bottom: Y". That means all the sticky items will simple stack on top of one another (like the tanstack example) rather than "moving out of the way for one another" (like the virtua example). If Virtua doesn't offer an API for items to know the position of other items, I suppose the best solution would be inspecting the DOM, and hoping the component is updated when cached positions change to re-calculate the DOM positions.

I checked your branch again and read your comments, I understood what you meant, trickiness of positioning of sticky items and the difference of "global stickiness" and "ranged stickiness".
Probably adding getItemOffset(index: number): number like method to VListHandle is good for this case. It's just a exposed internal store._getItemOffset but it will be much safer than direct access to cache.

VirtualizerHandle.getItemOffset was added in 0.27.4.


Isometric wall textures (make platform look platformier]

13/08/2024
title; Isometric wall textures

possible reference

i don't think this is very needed tbh


typo in the repository title

The current title is:
EIP 820 - Pseudo-introspaection using a registry contract
introspaection should be introspection
Update: the same typo is in the description field of package.json

@d10r thanks for spotting that typo!


Improvements to explore resources section on Home page

Description

Add sub title "Latest on the blog" to first column and list titles only of latest 3 blogs.
Add title "Webinar" more prominently for middle column and list title, date and "Register now" CTA.

Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Hi @Hasmin-AC  It would be like this

Apart from increasing the sub header titles if possible, this looks good to go from me. Thanks!


rotating doesn't work

It does something, but it's not what I would call a rotation

#21 changed the situation so that it just prints instead of doing its weird thing


[iOS] Sign in with Apple requirement

As announced at WWDC this year it looks like Apple will be requiring iOS apps to have a "Sign in with Apple" button if they offer a 3rd-party sign in option (like Facebook or Google).

It will be required as an option for users in apps that support third-party sign-in when it is commercially available later this year.

https://developer.apple.com/news/?id=06032019j
Use case
This is more a precaution than an issue I am having currently.
It should help avoid any issues with iOS Flutter apps in the future being rejected from the Apple store with some planning and preparation around this issue.

Proposal
I guess one of these two approaches could be used to help tackle any potential issue in the future:

Updating all of the sign in packages READMEs on how to add this button and create a new package using the "Sign in with Apple" button

or

Possibly add it automatically with existing sign in packages if the Flutter app is being run on an iOS device

I've listed the two most popular packages for sign in functionality below however there might be others with pub.dev which would also have to be updated.
https://pub.dev/packages/google_sign_in
https://pub.dev/packages/firebase_auth

If someone would like to submit a PR for option #1 to flutter/plugins, we would consider hosting it in the first-party plugins repository.
I don't know if it makes sense to add this functionality automatically since there are many cases where you might not need it.

This is certainly a required package, as Flutter is predominantly used for service apps and a signing in via OAuth is a regular affair in such applications.

I started work on a plugin this week. Should have an initial prototype by the weekend.

Thanks, Tom! That's great news for everyone.

Very early days and half-implemented but you can see an example here: https://github.com/tomgilder/flutter_sign_in_with_apple/blob/master/example/lib/main.dart
API is still evolving and package not ready to be published yet.

Is there any progress for this item? This is very important for developers who do not have mac but still wants to publish apps for ios.

Published the initial version of the package, 0.0.1: https://pub.dev/packages/apple_sign_in
Tested with iOS 13 beta 3, logs in successfully and gets credentials. Still a few things to implement, see the readme.
Still need to decide if there's any value in adding support for Apple's JavaScript framework for other platforms to that package.

Great work. Starred and watching the repo. Thank you.

Will there be any UI elements for this? Apple has set out a good guideline for the button UI (https://developer.apple.com/design/human-interface-guidelines/sign-in-with-apple/overview/)

Should this issue be moved to the https://github.com/FirebaseExtended/flutterfire Repo now since it affects firebase_auth?
I think for the UI elements @json469 took care in https://github.com/dmjones/flutter_auth_buttons/pull/13 so they should soon be in the flutter_auth_buttons package. What's missing is the implementation in Firebase ( https://github.com/firebase/firebase-ios-sdk/issues/3145 ) and following up the implementation in firebase_auth (which is now in the flutterfire Repo) is missing.

Sorry for the lack of updates, have been a busy bee.
Just published 0.1.0 of apple_sign_in, the API should now be stable. It includes a Flutter "Sign In with Apple" or "Continue with Apple" button (only in English, for now).
Would love to get the package battle-tested, please let me know if anyone's used it successfully (or unsuccessfully) in production.

Sorry for the lack of updates, have been a busy bee.
Just published 0.1.0 of apple_sign_in, the API should now be stable. It includes a Flutter "Sign In with Apple" or "Continue with Apple" button (only in English, for now).
Would love to get the package battle-tested, please let me know if anyone's used it successfully (or unsuccessfully) in production.

Awesome! Thanks, I am going to try using this with custom firebase tokens this week/weekend and and get back to you!

Sorry for the lack of updates, have been a busy bee.
Just published 0.1.0 of apple_sign_in, the API should now be stable. It includes a Flutter "Sign In with Apple" or "Continue with Apple" button (only in English, for now).
Would love to get the package battle-tested, please let me know if anyone's used it successfully (or unsuccessfully) in production.

Thank you! I've successfully used it in combination with firebase custom auth token using cloud function. Works beautifully. Production looks good on Android. Can't confirm Apple yet but will let you know if any problems there.

How did you use it on Android? This package is only for iOS?

Hi @epiphanatic what steps are you taking to do this? Is it:

Get credentials using apple_sign_in
Send credentials to cloud function
Call admin.auth().createCustomToken(uid)
Return token to mobile device
Call signInWithCustomToken(token)

If that is what you are doing what did you use for uid and does signInWithCustomToken exist in flutter's firebase_auth?

@joserocha3 yep, that's exactly what I'm doing. For uid: when I get the apple credentials from
final AuthorizationResult result = await AppleSignIn.performRequests([       AppleIdRequest(requestedScopes: [Scope.email, Scope.fullName]) ]);
I then use result.credential.user.
I printed the entire result and best I can tell that is Apple's form of uid (I have no experience with ios). It has the format of number.hash.number.
However, as of ios 13.2 this plugin may not work. I was trying to get something into production with it, which was fine during 13.1, but it was rejected just as 13.2 became a requirement and they said the button didn't do anything. I haven't tested since then (I just pulled the plugin out to get it through) as I've not gotten around to it. But then again, I've had so many problems with Apple it could be just them...

@epiphanatic I'll give it a go this coming week and test with 13.2. Maybe we can pinpoint an issue and @tomgilder can provide a fix. Thanks!

Haven't tested with 13.2 - will take a look asap

@tomgilder @joserocha3 I just tested with 13.2, it's fine. As suspected, it is/was just Apple...

hai, anyone can help me to findout how to regist to firebase using apple id sign in, sorry for my bad english. thanks

Hi , im need a Sign in with apple to deploy my app, im using this plugin and realy works, im using firebase cloud function to generate the token and use _auth.signInWithCustomToken but the firebase user that returns is empty, why this happen or is there another way to use firebase?

hi @dacxjo, can you tell me how to use firebase cloud function to generate the token, im still newbie by the way. thanks

@erizakaria Hi, i dont now if my approach is the best but to my situation it works.
What i did, is create a "callable" cloud function , you can create one on the cloud console , make sure that you select the right project , and enable for the project the Identity and Access Management (IAM) API, which is required to generate custom tokens.
This is what i have in my cloud function:
const admin = require("firebase-admin");
const functions = require("firebase-functions");
admin.initializeApp();

exports.appleToken = functions.https.onCall(async (data, context) => {
  let uuid = data.uuid;
  let token = "";

  if (uuid != "") {
    await admin
      .auth()
      .getUserByEmail(data.email)
      .then(async user => {
        await admin
          .auth()
          .createCustomToken(user.uid)
          .then(tkn => {
            token = tkn;
          });
      })
      .catch(async err => {
        if (err.code === "auth/user-not-found") {
          await admin
            .auth()
            .createUser({
              uid: uuid,
              email: data.email,
              displayName: data.name
            })
            .then(async userRecord => {
              await admin
                .auth()
                .createCustomToken(userRecord.uid)
                .then(tkn => {
                  token = tkn;
                });
            });
        }
      });

    return {
      token: token
    };
  } else {
    return null;
  }
});

and my package.json
{
  "name": "sample-http",
  "version": "0.0.1",
  "engines": {
    "node": "8"
  },
  "dependencies": {
    "firebase-admin": "^8.0.0",
    "firebase-functions": "^3.2.0"
  }
}

as you can see , first check if the user already exist to generate a token with his uid but if not the user is created and after the token is generated and returned.
Back in your flutter app , you'll have to install the cloud_functions plugin .
Create an instance of a callable function:
 final HttpsCallable callable = CloudFunctions.instance.getHttpsCallable(
    functionName: 'your_cloud_function_name',
  );

And when the apple_signin plugin returns your user data (email,uuid,name) , then you call the cloud function that should return the token , in which you will call the _firebaseAuth.signInWithCustomToken:
 try {
          await callable.call(<String, dynamic>{
            'uuid': result.credential.user,
            'name': result.credential.fullName.givenName,
            'email': result.credential.email
          }).then((HttpsCallableResult res) async {
            await _auth
                .signInWithCustomToken(token: res.data['token'])
                .then((authRes) {
              print(authRes);
            });
            Navigator.of(context, rootNavigator: true).pop();
            notifyListeners();
          }).catchError((err) {
            print(err);
          });

This is what i did, i don't know if this is going to help you but is what works to me. And sorry if i don't explain well, english is not my first language.

@erizakaria i forgot to say that you can monitor the cloud functions behavior from the firebase console , in the left menu "Functions"

@epiphanatic @joserocha3 I've also used the apple_sign_in and can confirm it works but created a custom auth solution using backend Node.js code to generate the token and then using signInWithCustomToken to complete the firebase side.  My observations and maybe you guys can comment :
1- Email is blank in identifier field in Firebase authentication tab (Can be resolved by adding this inside callable function)
2- Provider field in Firebase authentication tab is blank
3- Provider field in FirebaseUser after call from signInWithCustomToken   is Firebase and not Apple - I'm guessing this is because we have opted for a custom auth solution and not provided an equivalent  AuthCredential
4- Since FirebaseUser provider field is now Firebase it means your Node function will fail if there is another Firebase user with the same email address. This is incorrect since if there is a Facebook, Twitter, Google, Github user with the same email address - your node function will pass because the provider is different.
Do you guys get the same observation and more importantly - is there a way to change the Provider ID/Description so its not 'Firebase' (In Firebase User)  and 'BLANK' in Authentication console (Ultimately I'm guessing google needs to Update Firebase Auth to provide an AppleAuthProvider

@dacxjo The approach you are using in your cloud function is somewhat flawed. The issue you have is that you assume that the email address is unique. In Firebase as you know, there is the ability to allow multiple accounts per email address. If this is enabled, based on your current logic , two users with the same email address and different providers will have access to the same data ...
i.e
Assume you have a firebase user called<EMAIL_ADDRESS>- and this user is authenticated already on Firebase and has the provider set to Firebase.
Now assume you also enable apple sign in, and a new user also called<EMAIL_ADDRESS>wants to sign in via apple sign in. Note - these two users are completely different. They just share the same email address but have different conceptual providers :
<EMAIL_ADDRESS>Provider==> Firebase
<EMAIL_ADDRESS>provider==> Apple
Based on your logic of checking for a valid email user -> your code will confirm that<EMAIL_ADDRESS>** already exists and will then return the custom token for that user and then allow that user to be authenticated and be allowed to log in.
The issue is that the<EMAIL_ADDRESS>user is the apple user (And should have a completely different user area) compared to<EMAIL_ADDRESS>(which is the Firebase user). In your example , user data areas (assuming you set this up based on email/uid) would be shared across two or more entirely different users.
The issue is compounded because a Custom Auth solution will not have valid Provider information for Apple Sign in

@therealjohnsummer
This is the firebase default auth behavior
Prevent users from creating multiple accounts using the same email address with different authentication providers
Are you suggesting that it should not be that way?

@AirborneEagle Not sure how you deduced I was suggesting this should not be the case by my last post. I tried to be specific by stating :

In Firebase as you know, there is the ability to allow multiple accounts per email address.

I was simply trying to illustrate that the solution presented by @dacxjo is only suitable in the default firebase auth set up. If one changes the default to :

Which provides added flexibility for social provider login and allows logons such as :
<EMAIL_ADDRESS> -> using facebook provider
<EMAIL_ADDRESS> -> using google provider
<EMAIL_ADDRESS> -> using twitter provider
<EMAIL_ADDRESS> -> using firebase provider
which then allows a custom logon solution to allow more flexibility when creating user accounts which share the same email address such as :

etc

@therealjohnsummer In my firebase console i have enabled Allow creation of multiple account with the same email address to share the account data when the email is exactly the same. I know that this is not the best behavior but , this is a workaround because I didn't find a reasonable way or tutorial to link accounts providers with firebase auth using the linkWithCredentials method.

@dacxjo There is no need to use LinkWithCredentials. As can be seen with my last post - 4 distinct users with same email log in and each have separate user folders. Your approach and more importantly your Node.js cloud function would not facilitate this.

@dacxjo There is no need to use LinkWithCredentials. As can be seen with my last post - 4 distinct users with same email log in and each have separate user folders. Your approach and more importantly your Node.js cloud function would not facilitate this.

Not sure if I understand correctly, but why do someone should create separate user folders if they want to link the existing one with the newly "created". In my use case I need to save the user folder with all faves, stuff and all... so I need this functionality, two different users (well they are really one with different signin methods) to share one account folder.

@i6x86  Firestore authentication allows social provider logon as well as Firebase logon. The latter allows one to enter ANY email address whatsoever. That email address could be the same email address as a valid user on your system who say uses gmail. If I then use a known gmail address of a user on your system and attempt to create a user account using this email, if you follow the approach @dacxjo uses - you will end up piggy backing of the other users account - assuming you allow the creation of multiple email accounts and also do not require that an email address has been checked (User taps on the email link to verify their email address via the email sent from Firebase).

@therealjohnsummer ok I think I get it. I didn't saw it because I use only Facebook and Gmail and they both are prompt for password so if you don't have access to the user device you can not log in with their account. I think it would be the same with Apple. Am I right or there's something I miss?

Is it working now? Is there anyone who have successfully implemented Apple Sign-In in Flutter iOS App using Firebase Authentication?

@batbileg289 I am using the method proposed by @dacxjo  and it works fairly well.

Why do we need to use cloud functions?
Is there any known bug in library? If so, is there anyone who can fix that bug? :)

For iOS there's currently a PR in the FlutterFire Repo: https://github.com/FirebaseExtended/flutterfire/pull/1526

Because the method does not yet exist in the firebase_auth plugin.
https://github.com/FirebaseExtended/flutterfire/issues/1434

@TheJulianJES @lgArlequin Thanks

Does anyone get done the Android side? I think we'll appreciate some good example.

For iOS there's currently a PR in the FlutterFire Repo: FirebaseExtended/flutterfire#1526

Any idea, when it will be available in Flutter pub store? As of now, I can see it is 0.15.2, which does not have the changes.

Sorry for the lack of updates, have been a busy bee.
Just published 0.1.0 of apple_sign_in, the API should now be stable. It includes a Flutter "Sign In with Apple" or "Continue with Apple" button (only in English, for now).
Would love to get the package battle-tested, please let me know if anyone's used it successfully (or unsuccessfully) in production.

@tomgilder
Hi Tom, I submit success with apple_sign_in plugin, however I must fix button text for Vietnamese language.
https://apps.apple.com/us/app/lsicheck/id1471614412?ls=1
Thank you very much!

For iOS there's currently a PR in the FlutterFire Repo: FirebaseExtended/flutterfire#1526

Any idea, when it will be available in Flutter pub store? As of now, I can see it is 0.15.2, which does not have the changes.

You can update to 0.15.3 to get Apple sign-in support.

Marking this closed since I don't think there's anything more for the Flutter team to do here, but please let me know if I'm mistaken and I'll reopen.

Marking this closed since I don't think there's anything more for the Flutter team to do here, but please let me know if I'm mistaken and I'll reopen.

Thanks a lot @collinjackson
I am getting this error after upgrading to version 0.15.3. Can you please help, if anything I am missing here?

Xcode's output:
↳
/Users/soumensaha/Softwares/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.15.3/ios/Classes/FLTFirebaseAuthPlugin.m:493:38: error: no known class method for selector
'credentialWithProviderID:IDToken:rawNonce:accessToken:'
credential = [FIROAuthProvider credentialWithProviderID:provider
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/soumensaha/Softwares/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.15.3/ios/Classes/FLTFirebaseAuthPlugin.m:502:38: error: no known class method for selector 'credentialWithProviderID:IDToken:rawNonce:'
credential = [FIROAuthProvider credentialWithProviderID:provider
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

Marking this closed since I don't think there's anything more for the Flutter team to do here, but please let me know if I'm mistaken and I'll reopen.

Thanks a lot @collinjackson
I am getting this error after upgrading to version 0.15.3. Can you please help, if anything I am missing here?
Xcode's output:
↳
/Users/soumensaha/Softwares/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.15.3/ios/Classes/FLTFirebaseAuthPlugin.m:493:38: error: no known class method for selector
'credentialWithProviderID:IDToken:rawNonce:accessToken:'
credential = [FIROAuthProvider credentialWithProviderID:provider
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/soumensaha/Softwares/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.15.3/ios/Classes/FLTFirebaseAuthPlugin.m:502:38: error: no known class method for selector 'credentialWithProviderID:IDToken:rawNonce:'
credential = [FIROAuthProvider credentialWithProviderID:provider
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

I have the same problem 0.15.3 also

Same error here, after update to the last version

I solved this error as most of the time I do it ...
Run pod update in the IOS folder

@collinjackson could you also update the firebase_auth documentation to describe how to use or add the option for apple sign in?


Tutorial adding command

Codecov Report

Merging #7 (b3c819e) into master (0dd686b) will increase coverage by 0.14%.
The diff coverage is 79.36%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage     72.15%   72.30%   +0.14%
- Complexity      399      417      +18
============================================
  Files            70       73       +3
  Lines          1232     1289      +57
  Branches        125      131       +6
============================================
+ Hits            889      932      +43
- Misses          311      323      +12
- Partials         32       34       +2

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/Remark.java
75.00% <75.00%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
97.05% <100.00%> (+0.04%)
:arrow_up:

...va/seedu/address/logic/commands/RemarkCommand.java
100.00% <100.00%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

...c/main/java/seedu/address/model/person/Person.java
97.82% <100.00%> (+0.20%)
:arrow_up:

.../java/seedu/address/model/util/SampleDataUtil.java
25.00% <100.00%> (+5.00%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0dd686b...b3c819e. Read the comment docs.


md-ink-bar doesn't calculate initial width properly when min-width of md-tab-link is overriden

while it's perhaps not on spec to have a tab without min-width, I think it's not an unreasonable thing to do for a consumer.. also it works fine after you click on a tab
see http://plnkr.co/edit/u5M39n994lJriqMABMod?p=preview
@andrewseguin @jelbourn maybe?

Issue has been fixed by #3037


Clarify Active Directory Install subtitle

Small correction to the subtitle regarding the Active Directory, to clarify that we are installing the nuget not Active directory itself.

@jptarqu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Provide token parsing for storeContextMenus, Fixes #300

*Specifically for colChooser right now

Note: This will be a breaking change for any apps using storeContextMenus as the constructor takes and object now.


[feat] Add sparse vector support to knowhere

Work Items:

[x] Search w/ & w/o filter

[x] columnar brute force sparse vector index
[x] wand based sparse vector index

[x] other feature

[x] range search
[x] iterator
[x] index mmap
[x] get vector by id
[ ] etc...

[x] Unit Test

[x] Add brute force search method to provide ground truth
[x] Add tests for search/filtered-search/serialize/deserialize/
[x] Add tests for growing segment(add-search and add-search again)

[x] E2E test

all features in comments are now implemented, closing


Error: Unexpected EOF

Hi,
this plugin is not working for me and I don't quite understand why (maybe because I'm on windows?).
I don't get a response from ollama when using this plugin, so the response modal is always empty. Enabling debug output, I get these logs:
curl --silent --no-buffer -X POST http://localhost:11434/api/chat -d "{""messages"": [{""role"": ""user"", ""content"": ""Regarding the following text, hi:\n""}], ""model"": ""mistral"", ""stream"": true}"
Response data:
{ '{"error":"unexpected EOF"}' }
Response data:
{ "" }

When I try to run the same curl command in git-bash, the output is this:
$ curl --silent --no-buffer -X POST http://localhost:11434/api/chat -d "{""messages"": [{""role"": ""user"", ""content"": ""Regarding the following text, hi:\n""}], ""model"": ""mistral"", ""stream"": true}"
{"error":"invalid character 'm' looking for beginning of object key string"}

I don't quite understand why the json body contains doubled quotes. When I remove them and surround the body with single quotes, than it works:
curl --silent --no-buffer -X POST http://localhost:11434/api/chat -d '{"messages": [{"role": "user", "content": "Regarding the following text, hi:\n"}], "model": "mis
tral", "stream": true}'
{"model":"mistral","created_at":"2024-03-17T16:18:22.8339767Z","message":{"role":"assistant","content":" Hello"},"done":false}
...

Don't know if this is really the cause of the issue or if the "unexpected EOF" comes from something else. But that's what I was able to figure out so far.
Here's a screenshot of this issue in action:

Seems like my initial hunch with the double quotes "" is wrong, considering this is apparently the way to escape them in windows cmd (according to #4).
So I realized from mentioned issue, that neovim executes the curl in windows cmd, instead of git bash, and if I execute the command there I get an empty response:
curl --silent --no-buffer -X POST http://localhost:11434/api/generate -d "{""messages"": [{""role"": ""user"", ""content"": ""Regarding the following text, hi:\n""}], ""model"": ""mistral"", ""stream"": true}"
{"model":"mistral","created_at":"2024-03-17T17:02:41.9377754Z","response":"","done":true}

I guess I will have to debug my setup with ollama in cmd.

Hi @Kahitar ,
Thanks for opening this issue. Unfortunately, I have no access to a windows machine, therefore I cannot debug this problem. Would you mind modifying the escaping mechanism to investigate?

Issue
When using the application, only the first letter or two of the response is being streamed before it stops. This appears to be a timeout issue.

Questions

Which configuration option controls the timeout for streaming responses?
Is it related to the curl -m [timeout_seconds] parameter?
If there's no existing option to control this, could such an option be added?

I'm new to reading code, so apologies if this is a basic question. I've been experimenting with CLI-based LLM applications to test ollama functionality and prompt development. This application is exactly what I've been looking for! great work -i t's incredibly useful and well-designed.


Parser Error: class class_name hide a global script class " when read tres file with flag bundle

Godot version
4.2dev
System information
Windows10
Issue description
I want to save and load Resource file in godot 4.2, I used resources for that. in main resources i have another resources which is initiated, the issue was when use resourcesaver.save() with no flag_bundle_set the resource inside main resource doesn't write inside .tres file, so i must use resourcesaver.save(ResourceSaver.FLAG_BUNDLE_RESOURCES) to save it. it saves successfully but when I want to load it with resourceloader.load or load I got this error: " Parser Error: class class_name hide a global script class "
class_name is my resource class which another resource is inside that.
Steps to reproduce

use ResourceSaver.save  with FLAG_BUNDLE_REOURCES
2.
USE RESOURCELOADER.load for load that tres file
Minimal reproduction project
ResourceSaver.save(a,"user://_myfile.tres",ResourceSaver.FLAG_BUNDLE_RESOURCES)
var c:Resource
if(ResourceLoader.exists("user://_myfile.tres")):
c=ResourceLo
FlagBundle.zip
ader.load("user://_myfile.tres")

Please provide a proper MRP:

A small Godot project which reproduces the issue, with no unnecessary files included. Be sure to not include the .godot folder in the archive (but keep project.godot).
Drag and drop a ZIP archive to upload it. Do not select another field until the project is done uploading.
Note for C# users: If your issue is not Mono-specific, please upload a minimal reproduction project written in GDScript or VisualScript. This will make it easier for contributors to reproduce the issue locally as not everyone has a Mono setup available.

FlagBundle.zip

@AThousandShips , I attached Minimal reproduction project


network/consensus: use new dbft StopTxFlow callback

It makes sense in general (further narrowing down the time window when transactions are processed by consensus thread) and it improves block times a little too, especially in the 7+2 scenario.
Related to #2744.

Depends on and will be updated after nspcc-dev/dbft#63.


compose: Fix compose box getting close on copying code.

This commit adds an exception in click-handlers.js so that
when mouse clicks on .copy-codeblock, the composebox is not
closed.
Fixes #19130

Testing plan:
Tested this manually on chrome browser.
GIFs or screenshots:

@alya FYI

Nice, merged after a small tweak to the comment explaining why we want this to focus compose, and not just do nothing, thanks @Signior-X!

Awesome, thanks!
On Wed, Jul 7, 2021 at 10:48 AM Tim Abbott @.***> wrote:

Nice, merged after a small tweak to the comment explaining why we want
this to focus compose, and not just do nothing, thanks @Signior-X
https://github.com/Signior-X!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/19154#issuecomment-875804105, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAP6IUTJCAOQFKMZPCAUNJLTWSHNVANCNFSM4767SINQ
.


Roll Flutter Engine from 1947b95c5060 to b2f142788838 (13 revisions)

https://github.com/flutter/engine/compare/1947b95c5060...b2f142788838
2023-03-29<EMAIL_ADDRESS>Supress warning for missing super call onBackPressed cl/516851539 (flutter/engine#40757)
2023-03-29<EMAIL_ADDRESS>[Impeller] migrate Gaussian shaders to half precision. (flutter/engine#40752)
2023-03-29<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from A7rAmHDuuQ89yV9Fw... to mxMrU7aA6avJRQ9qf... (flutter/engine#40760)
2023-03-29<EMAIL_ADDRESS>Revert "Listen to WM_CLOSE message on Windows to allow framework to cancel exit (#40493)" (flutter/engine#40739)
2023-03-29<EMAIL_ADDRESS>Roll Dart SDK from 69867ba60bb7 to 5606c2c7e1fc (3 revisions) (flutter/engine#40758)
2023-03-29<EMAIL_ADDRESS>Roll Skia from 145d93ee3f4f to 001ba6e28f99 (2 revisions) (flutter/engine#40754)
2023-03-29<EMAIL_ADDRESS>[Impeller] Enable playgrounds using a runtime instead of a build time flag. (flutter/engine#40729)
2023-03-29<EMAIL_ADDRESS>[Impeller] Start a more generic stroke tessellator for path (flutter/engine#40690)
2023-03-29<EMAIL_ADDRESS>[Impeller] Use NSAutoReleasePool for Metal playgrounds. (flutter/engine#40748)
2023-03-29<EMAIL_ADDRESS>Roll Skia from f415e93eafc8 to 145d93ee3f4f (1 revision) (flutter/engine#40751)
2023-03-29<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from JH1CUJ3kBRCcQn2AI... to whJeWVcM-8gVZRdVR... (flutter/engine#40750)
2023-03-29<EMAIL_ADDRESS>[web] remove obsolete object caches; simplify native object management (flutter/engine#40617)
2023-03-29<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from n85YWHBBE4HYwce_E... to A7rAmHDuuQ89yV9Fw... (flutter/engine#40749)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from JH1CUJ3kBRCc to whJeWVcM-8gV
fuchsia/sdk/core/mac-amd64 from n85YWHBBE4HY to mxMrU7aA6avJ
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Closing to get a new role that picks up a revert.

Commit queue failed; closing this roll.


Configuring Signed bot commits

This PR amends the scripts used to commit and raise PRs for dependabot configuration file to create signed commits.

Added git-commit-and-pull-request.sh script which uses gh cli to create a branch and commit code changes via the API since this approach allows github actions bot to sign the commit.
Configures the environment variables to utilise default github token.

I've rebased and tested. It generates the file as it should without errors.


Screencast gif is too big

Screencast is a nice thing, but 3+ MB is too big. Please try to make a smaller one about 1MB if possible. You could reduce the size and length.

screencast is 831kb in size now.


Support Swift 3.0 ?

Hi, @CNKCQ I believe it already supports Swift 3.
pod 'RxDataSources', '= 1.0.0-beta.3'


fastfetch: 2.2.0 -> 2.2.1

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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.11 Release Notes (or backporting 23.05 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 265033 run on x86_64-linux 1

1 package failed to build:

fastfetch

error: builder for '/nix/store/zj6xy82c2wh641yrsr2jhn23b6gj6fk6-fastfetch-2.2.1.drv' failed with exit code 2;
       last 10 log lines:
       > make[1]: *** Waiting for unfinished jobs....
       > lto1: some warnings being treated as errors
       > make[3]: *** [/build/ccShElcS.mk:11: /build/cc6X8c9m.ltrans3.ltrans.o] Error 1
       > lto-wrapper: fatal error: make returned 2 exit status
       > compilation terminated.
       > /nix/store/p58l5qmzifl20qmjs3xfpl01f0mqlza2-binutils-2.40/bin/ld: error: lto-wrapper failed
       > collect2: error: ld returned 1 exit status
       > make[2]: *** [CMakeFiles/flashfetch.dir/build.make:425: flashfetch] Error 1
       > make[1]: *** [CMakeFiles/Makefile2:139: CMakeFiles/flashfetch.dir/all] Error 2
       > make: *** [Makefile:156: all] Error 2
       For full logs, run 'nix log /nix/store/zj6xy82c2wh641yrsr2jhn23b6gj6fk6-fastfetch-2.2.1.drv'.
error: 1 dependencies of derivation '/nix/store/ffi4bpc36n9ki3d0sdc6m1skxw6rl75n-review-shell.drv' failed to build

Result of nixpkgs-review pr 265033 run on x86_64-linux 1
1 package failed to build:
* fastfetch

error: builder for '/nix/store/zj6xy82c2wh641yrsr2jhn23b6gj6fk6-fastfetch-2.2.1.drv' failed with exit code 2;
       last 10 log lines:
       > make[1]: *** Waiting for unfinished jobs....
       > lto1: some warnings being treated as errors
       > make[3]: *** [/build/ccShElcS.mk:11: /build/cc6X8c9m.ltrans3.ltrans.o] Error 1
       > lto-wrapper: fatal error: make returned 2 exit status
       > compilation terminated.
       > /nix/store/p58l5qmzifl20qmjs3xfpl01f0mqlza2-binutils-2.40/bin/ld: error: lto-wrapper failed
       > collect2: error: ld returned 1 exit status
       > make[2]: *** [CMakeFiles/flashfetch.dir/build.make:425: flashfetch] Error 1
       > make[1]: *** [CMakeFiles/Makefile2:139: CMakeFiles/flashfetch.dir/all] Error 2
       > make: *** [Makefile:156: all] Error 2
       For full logs, run 'nix log /nix/store/zj6xy82c2wh641yrsr2jhn23b6gj6fk6-fastfetch-2.2.1.drv'.
error: 1 dependencies of derivation '/nix/store/ffi4bpc36n9ki3d0sdc6m1skxw6rl75n-review-shell.drv' failed to build

Welp... so much for just a patch version bump.. lol

Do you need the full logs or maybe the concrete error messages?

Do you need the full logs or maybe the concrete error messages?

Nah I see the logs now. Just trying to decipher and figure out what needs to change.

Looks like I get the same build error on the current 2.2.0 version in master...

Looks like last upgrade also broke darwin. Gonna try fixing that, too...


Voice.start causes ios app running in simulator to exit

Here is what my component looks like:
import React, { Component } from 'react';
import {
StyleSheet,
Text,
View,
Button
} from 'react-native';
import Voice from 'react-native-voice';
export default class VoiceTest extends Component {
onPressStart(e){
  try {
Voice.start('en-US');
} catch (error) {
 console.log('Not working')
 }
  }
 render() {
   return (
    <View>
  <Text>This is working</Text>
  <Button onPress={this.onPressStart} title='Start'></Button>
    </View>
   )
 }
}
If I run the app in ios simulator and click on the Start button, the app closes immediately. Nothing logged in the console or in the terminal.

Thank you! That solved it.


Should guideline sections be collapsed when opening the document?

@ypriverol With the new TOC, should guideline sections still be collapsed initially?

Yes!!!!


Please take my knees

hi, @jimkyndemeyer, this plugin is pretty awesome :+1:   thank u so much！

Thanks. Appreciate it :)
The completion and error highlighting builds on top of codemirror-graphql, so they deserve a large part of the credit.


npsp:Too many query rows: 50001

Error occurs during mass opportunity record insert (~115 records). Appears to be based on the cumulative fields being updated on the contact object (one to one model). Have de-activated Opportunity trigger (RLLP_OppRollup_TDTM) as a workaround. Also tried setting the trigger to async but issue still remains.
Thanks!

what happens if you reduce your batch insert size to much smaller (say 10 opps)?
is this with NPSP 2.0 or 3.0?
is this a recent change in behavior (ie, this scenario recently worked and now doesn't?)
if using 3.0, is Custom Opportunity Naming turned on?

Hi David,

This is based on a mass insert via the iATS Brickwork application (batch size (200 records) cannot be adjusted).
3.0
iATS Brickwork has a journal download which runs via nightly scheduled Apex. The maximum number of opportunity records inserted per batch run is 200. The number varies and for the date of the issue, ~115 opportunity records were inserted.
No

Let me know if you would like me to send the debug log. I am the lead developer for the iATS Brickwork application. We haven't noticed this error before. Thanks!

Opportunities have a lot of associated code, whenever I did data imports I would always reduce the batch size to avoid hitting limits. Why can't you adjust the batch size in your application?

Hi Tanner, with NPSP there is a lot of functionality and code that gets executed on opportunity insert.  while we strive to make our code efficient as possible, we have never set our bar to be that we should handle batch sizes as large as 200.  We've always told people who run into limits that they should reduce their batch size into the 10's, not 100's, when bulk loading data.
Now if you are running into a specific scenario that is failing, and you have reviewed our code and found one of our trigger handlers to be too inefficient, we are open to seeing how it could be improved.  But it is going to require some upfront work on your side to identify what you think the issue is.  @kbromer let me know if you disagree. thanks.

Thanks David. This is a isolated issue so we'll internally look in to just reducing the batch size if the NPSP is installed or the trigger is active. Thanks for the detailed response!


ReCAPTCHA couldn't find user-provided function: vueRecaptchaApiLoaded

Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.

If you use build tool you must do:
require('vue-grecaptcha')

Vue.use(VueGrecaptcha, {
  // options
})

Then it will global and will work.
This will not work: import VueGrecaptcha from 'vue-grecaptcha', I know this issue and will fix it later.


Update accessing-the-api.md

"the creator need to share" or "the creator needs share"

This change is

"needs" is correct


[ssr] ModuleLoader linker fails with multiple concurrent entrypoints

Given two modules that both import lit, this approach works fine:
new ModuleLoader(...);
for (const mod of modules) {
  await loader.importModule(module, referrer);
}

But this approach consistently fails:
new ModuleLoader(...);
await Promise.all(modules.map((module) =>
  loader.importModule(module, referrer));
);

with an error like:
Error: request for '@lit/reactive-element' is not yet fulfilled
        at SourceTextModule.link (node:internal/vm/module:198:17)
        at ModuleLoader.importModule (/home/aomarks/work/wireit-lit/packages/labs/ssr/src/lib/module-loader.ts:107:7)
        at async Promise.all (index 1)
        at AsyncEventEmitter.<anonymous> (/home/aomarks/work/wireit-lit/packages/labs/eleventy-plugin-lit/src/index.ts:85:7)
        at async Promise.all (index 0)
        at async Eleventy.executeBuild (/home/aomarks/work/wireit-lit/packages/labs/eleventy-plugin-lit/node_modules/@11ty/eleventy/src/Eleventy.js:971:7)

Decent chance there is a timing issue in the ModuleLoader class https://github.com/lit/lit/blob/908f451d1444955571206588646efd04457d8d02/packages/labs/ssr/src/lib/module-loader.ts#L68

cc @justinfagnani I spent a while poking around at ModuleLoader, but couldn't identify the problem here. I'll dig into this more deeply probably tomorrow, but just pinging in case this seems familiar to you, or something comes to mind.

cc @justinfagnani @kevinpschaaf I spent a while poking around at ModuleLoader, but couldn't identify the problem here yet. I'll dig into this more deeply probably tomorrow, but just pinging in case this seems familiar to you, or something comes to mind.


Get access data (kubeconfig or equivalent)

Is there any way to get the access data? (E.g. cluster_ca_certificate, client_certificate and client_key) . If it is not the case, then I am willing to submit a pull request with a data source (https://developer.hashicorp.com/terraform/plugin/framework/data-sources).

Seconded, how do you utilise the kubeconfig output?

In my projects I'm decoding the yaml in a local variable:
locals {
  kubeconfig = yamldecode(k0s_cluster.cluster.kubeconfig)
}

and I'm using it in the provider configurations, for example:
provider "kubernetes" {
  host                   = local.kubeconfig.clusters[0].cluster.server
  cluster_ca_certificate = base64decode(local.kubeconfig.clusters[0].cluster.certificate-authority-data)
  client_certificate     = base64decode(local.kubeconfig.users[0].user.client-certificate-data)
  client_key             = base64decode(local.kubeconfig.users[0].user.client-key-data)
}

Let me know if this works for you 😄

Ah thanks. Yes It worked for me :)


Identify four more use cases and find people to work on them

We want to keep the momentum going with documenting use cases. At the 2018 TDWG meeting we decided a total of six use cases seemed like a good goal. We have two use cases documented so far. We need to identify four more uses and find people to work on them. These two things probably go hand in hand and it might even make the most sense to identify the right people first and let them determine the use case they are most excited to address.

One use I case I want to work on at some point (for our work at INBO) is camera trap data. We’ll first express our data in CTMS (camera trap metadata standard) and then try to map that to Darwin Core. That mapping should (just like OTN, Movebank) be useful to a wider range of users than just us. But it will be a while before I can tackle this.

I know of someone who has deployed cameras on marine mammals. i'll ask them to share a sample dataset w/ me for classification. we should still find a 'classic' camera trap dataset though, that'll be much more commonly reported.

We have a "classic" camera trap dataset, which we plan to standardize to Darwin Core later this year.

For Camare deployments on mammals I could probably find some Antarctic examples from researcher that are willing to provide data.
Are we talking just camera footage or also combined wit accelerometer date and such?

i mean, eventually the more complicated, the better, but for an example case we probably want just the camera data plus positional information for the moving camera.

Are private projects ok also s use cases? I'm working on a system that picks out bird calls (nocturnal flight calls / nocmig) from audio recordings using AI, which will eventually become machine observations.
Since this is my personal project, I would not publish the data directly as DwC, but can feed it into Finnish Biodiversity Information Facility (where I work), which could then share it to GBIF.

@mikkohei13 I don't see any reason your use case wouldn't fit. Let's add it!


Rich Text raw don't get the new data

Describe the bug
We manage to implement the Live Preview SDK and all goes ok , with normal fields, but not with Rich Text Fields.
We don't get the new data updated on raw but yes on json.

We are using renderRichTextfrom https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby-source-contentful/src/rich-text.js#L5
renderRichText needs raw object.
Is there an option to get the data updated in raw object?
Thank you

Hey,
thanks for reporting the issue, we are aware that there are some issues with Gatsby (especially gatsby-source-contentful) and we're working on a better integration for that (https://github.com/contentful/live-preview#integrating-with-gatsby)

@YvesRijckaert us I understand you will not fix this bug? Because with media files in Rich text there is a big bug.
Thank you

@YvesRijckaert  us I understand you will not fix this bug? Because with media files in Rich text there is a big bug.
Thank you

Hi @liviutodoran
Live updates for Gatsby is still under active development


Modifiable "Eye of Ender"

Hi! I noticed that "Eye of Ender" cannot be edited and I implemented edit.
Also PR closes issue #75
Didn't find a better icon of "Eye of Ender" and with absence game packs maybe looks different from others icons :)
Bellow some screenshots of the changes
Without game paks:

With game paks:

Changing this value is not enough to unlock the end portal.
I would recommend to create a unlock portal button which change those values in the strongholdProgress field :

strongholdDeepwoodBrookUnlocked
strongholdHighblockHideawayUnlocked
strongholdOldTownHallUnlocked
strongholdSunkenWatchtowerUnlocked
strongholdTheTrialOfTheNamelessUnlocked
strongholdTheUndercroftUnlocked

I don't know exactly how the property EyeOfEnder is used, but if those boolean are not set to true, I was not able to put the enderpeal in the portal.

Changing this value is not enough to unlock the end portal. I would recommend to create a unlock portal button which change those values in the strongholdProgress field :

Indeed, without these fields to true, the game does not allow put your eyes of ender in the portal
I implemented "Unlock Portal" button. She sets these fields to true values
The button is displayed if save loaded and at least one "stronghold*Unlocked" field is set to false

I've tried another one trainer, there's an 'unlimited emerald' option, it set the number of eyes of ender to 9999999, now i just want set it to 0, will it be available in the next update?

I've tried another one trainer, there's an 'unlimited emerald' option, it set the number of eyes of ender to 9999999, now i just want set it to 0, will it be available in the next update?

I think If @CutFlame will accept PR then the functionality will appear in the new release 😄


In the coco model, when I use resnet100 and resnet50 to generate h5 files to pb files respectively, why can both be generated ok?

In the coco model, when I use resnet100 and resnet50 to generate h5 files to pb files respectively, why can both be used?

Hi, @buaacarzp Sorry but I am not being able to understand your issue clearly.
Mask_RCNN has used ResNet 101 as backbone. All I have done is to convert the Mask_RCNN  (which by default uses ResNet 101) Keras model to Tensorflow frozen and serving model. If that answers your question.

hey, I am so happy to receive your response. when I set the" self.BACKBONE = "resnet101"" ,the .pb file was successfully created, and I set the " self.BACKBONE = "resnet50" ,the pb file was successfully created too. my question is that how can I test the pb model is right? can I use the pb file which I created to inference the model?
because I want to generate teh pb file and the pbtxt file to use the opencv  by c++ to run the maskrcnn. so,now I counld not to guess that my pb file is right, could you please help me ?
thank you.
<EMAIL_ADDRESS>From: Bendang
Date: 2019-06-06 20:25
To: bendangnuksung/mrcnn_serving_ready
CC: buaacarzp; Mention
Subject: Re: [bendangnuksung/mrcnn_serving_ready] In the coco model, when I use resnet100 and resnet50 to generate h5 files to pb files respectively, why can both be generated ok? (#8)
Hi, @buaacarzp Sorry but I am not being able to understand your issue clearly.
Mask_RCNN has used ResNet 101 as backbone. All I have done is to convert the Mask_RCNN (which by default uses ResNet 101) Keras model to Tensorflow frozen and serving model. If that answers your question.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Glad to help. To test 'mask_frozen_graph.pb'  for inference please refer https://github.com/bendangnuksung/mrcnn_serving_ready/issues/5#issuecomment-485647139

hey,now I have a trouble in there.
first of all,  what's the use of serving model?
the second ,why I meet a wrong  when I use the image to test the  file of "forward.py" to test the "mask_frozen_graph.pb'" ? here is my code to call the forward.py:
`from forward import ForwardModel
import cv2
import skimage
is_coco=True
def get_config1():
if is_coco:
import coco
class InferenceConfig(coco.CocoConfig):
GPU_COUNT = 1
IMAGES_PER_GPU = 1
    config = InferenceConfig()

else:

config = mask_config(NUMBER_OF_CLASSES)
    pass

return config

my_config=get_config1()
forward_model = ForwardModel('./frozen_model/mask_frozen_graph.pb', my_config)
#forward_model = ForwardModel('./serving_model/1/saved_model.pb', my_config)
list_of_image_arrays=cv2.imread("./zp.jpg")
skimage.transform.rescale(list_of_image_arrays, (4,4))
#if list_of_image_arrays.ndim != 3:
list_of_image_arrays = skimage.color.gray2rgb(list_of_image_arrays)
# If has an alpha channel, remove it for consistency
#if list_of_image_arrays.shape[-1] == 4:
list_of_image_arrays = list_of_image_arrays[..., :3]
results = forward_model(list_of_image_arrays)but  it give me a wrong message:ValueError: operands could not be broadcast together with remapped shapes [original->remapped]: (3,2) and requested shape (2,2)
`
I am true that my image is RGB, why ...... I am so sad and confused.... please help me ,handsome boy.

Serving model can be used by tensorflow_model_server to deploy models on production.
forward_model expects a list of images. i.e it requires input shape of (N_Images, height, width, channel)

here is the code I use to call the forward_model
from forward import ForwardModel
import cv2
import numpy as np

frozen_graph_path = "frozen_model/mask_frozen_graph.pb"

# assign your own Config class
from config import mask_config
my_config = mask_config(5)

forward_model = ForwardModel(frozen_graph_path, my_config)

def test_one_image(image):
    images = np.expand_dims(image, axis=0)
    results = forward_model(images)
    return results

if __name__ == "__main__":
    test_image_path = 'XYZ.jpg'
    image = cv2.imread(test_image_path)
    r = test_one_image(image)
    print(r)

1. Serving model can be used by `tensorflow_model_server` to deploy models on production.

2. `forward_model` expects a list of images. i.e it requires input shape of (N_Images, height, width, channel)

here is the code I use to call the forward_model
from forward import ForwardModel
import cv2
import numpy as np

frozen_graph_path = "frozen_model/mask_frozen_graph.pb"

# assign your own Config class
from config import mask_config
my_config = mask_config(5)

forward_model = ForwardModel(frozen_graph_path, my_config)

def test_one_image(image):
    images = np.expand_dims(image, axis=0)
    results = forward_model(images)
    return results

if __name__ == "__main__":
    test_image_path = 'XYZ.jpg'
    image = cv2.imread(test_image_path)
    r = test_one_image(image)
    print(r)

I feel so touched that you reply me, you are a goodness boy, but when I use your code to test the coco.h5,  this model have 81 classes, so I change the 5 to 81, but it give me a wrong message:
ValueError: Cannot feed value of shape (1, 94) for Tensor 'import/input_image_meta:0', which has shape '(?, 93)'

How can I solve this problem? I feel it have a little distance to  the victory, please help me again. I want to make friends with you, it's my honer.

I solve this problem by change the 81 to 80 ,  I feel it is successful ,because it give me this message:
{'rois': array([], shape=(0, 4), dtype=int32), 'class': array([], dtype=int32), 'scores': array([], dtype=float32), 'mask': array([], shape=(1440, 1080, 0), dtype=float64)}]

I want to ask you a  queston, can you explain to me why it print the 'rois','class','scores','mask'?　 and why the four value  is None.  it just print eh shape and the dtype?

can you help me .... why my print have  not  values,only a shape size inside.
<EMAIL_ADDRESS>From: Bendang
Date: 2019-06-12 16:01
To: bendangnuksung/mrcnn_serving_ready
CC: buaacarzp; Mention
Subject: Re: [bendangnuksung/mrcnn_serving_ready] In the coco model, when I use resnet100 and resnet50 to generate h5 files to pb files respectively, why can both be generated ok? (#8)
Serving model can be used by tensorflow_model_server to deploy models on production.
forward_model expects a list of images. i.e it requires input shape of (N_Images, height, width, channel)
here is the code I use to call the forward_model
from forward import ForwardModel
import cv2
import numpy as np
frozen_graph_path = "frozen_model/mask_frozen_graph.pb"
assign your own Config class
from config import mask_config
my_config = mask_config(5)
forward_model = ForwardModel(frozen_graph_path, my_config)
def test_one_image(image):
images = np.expand_dims(image, axis=0)
results = forward_model(images)
return results
if name == "main":
test_image_path = 'XYZ.jpg'
image = cv2.imread(test_image_path)
r = test_one_image(image)
print(r)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorry for the late reply. Looks like you have used my_config = mask_config(5), you have to use your own config which you have used above.  my_config = get_config1()

Sorry for the late reply. Looks like you have used my_config = mask_config(5), you have to use your own config which you have used above.  my_config = get_config1()

dear bendangnuksung , I understand your meaning of that, but I use the coco model from , it contains 81 classes in that trained model, so I am true that I set the my_config = mask_config(80) , and I use a image fromhttps://github.com/matterport/Mask_RCNN/tree/master/images  to inference the pb model is right, but here is no output,and it print the   {'rois': array([], shape=(0, 4), dtype=int32), 'class': array([], dtype=int32), 'scores': array([], dtype=float32), 'mask': array([], shape=(1440, 1080, 0), dtype=float64)}], without the information of the mask and  the class, so I am confused  of it .
my config like the https://github.com/matterport/Mask_RCNN/blob/master/mrcnn/config.py,  the model I download is the maksrcnn_coco.h5, how can I overcome this problem? please help me . this problem I have tryed many times , but it didn't work.

Sorry for the late reply. Looks like you have used my_config = mask_config(5), you have to use your own config which you have used above.  my_config = get_config1()

Can you send me your model file ?  thank you very much

Sorry for the late reply. Looks like you have used my_config = mask_config(5), you have to use your own config which you have used above. my_config = get_config1()

I got it, thank you very much , thanks, now  it print the
[{'rois': array([[ 94, 141, 380, 590],
       [261,   0, 341,  91],
       [260, 536, 365, 629],
       [282, 611, 351, 640],
       [256,  72, 286,  96],
       [247,  89, 313, 145],
       [278, 599, 299, 610],
       [262,   4, 289,  80]], dtype=int32), 'class': array([6, 3, 3, 3, 3, 3, 1, 3], dtype=int32), 'scores': array([0.9999424 , 0.9876124 , 0.98480034, 0.9665876 , 0.96335596,
       0.94175243, 0.74342126, 0.7117303 ], dtype=float32), 'mask': array([[[False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        ...,
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False]],

       [[False, False, False, ..., False, False, False],

Next, I want to ask you  how to generate a pbtxt file ,because I want to use the opencv ,DNN module to  load my model and inference , please help me .

Sorry but I don't know how to generate pbtxt file to be used in opencv. A user said it was was not  able to read by opencv  link: https://github.com/bendangnuksung/mrcnn_serving_ready/issues/7#issuecomment-491193911

Sorry but I don't know how to generate pbtxt file to be used in opencv. A user said it was was not able to read by opencv link: #7 (comment)

I can’t understand your answer...  why there many people want to generate the pb file but they didn’t load the pbtxt file to use the opencv dnn module? I don’t understand it...

Sorry for the late reply. Looks like you have used my_config = mask_config(5), you have to use your own config which you have used above. my_config = get_config1()

I got it, thank you very much , thanks, now it print the
[{'rois': array([[ 94, 141, 380, 590],
       [261,   0, 341,  91],
       [260, 536, 365, 629],
       [282, 611, 351, 640],
       [256,  72, 286,  96],
       [247,  89, 313, 145],
       [278, 599, 299, 610],
       [262,   4, 289,  80]], dtype=int32), 'class': array([6, 3, 3, 3, 3, 3, 1, 3], dtype=int32), 'scores': array([0.9999424 , 0.9876124 , 0.98480034, 0.9665876 , 0.96335596,
       0.94175243, 0.74342126, 0.7117303 ], dtype=float32), 'mask': array([[[False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        ...,
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False],
        [False, False, False, ..., False, False, False]],

       [[False, False, False, ..., False, False, False],

Next, I want to ask you how to generate a pbtxt file ,because I want to use the opencv ,DNN module to load my model and inference , please help me .

how to show the result masked image ?
@buaacarzp @bendangnuksung

Closing as the original issue has been solved.


Create settings file

Added function that creates settings file - compiles, however errors at
runtime with "file in use".
Also moved the LookupCustomer code to an async method.
Only merge in the LookupCustomer bit.

Can't merge only a section of code.


Update to Purescript 0.9.1

The dependencies and import statements need to be updated for Purescript 0.9.1

I started working on this here https://github.com/kika/purescript-express
But I'm blocked by purescript-node-fs module which depends on purescript-datetime, which, in turn, is being redesigned https://github.com/purescript/purescript-datetime/issues/33 and will not be updated to 0.9.1 in its current form. As soon as they release datetime I'll continue on the 0.9.1 update.

Nice! Do you know if this repo is still maintained or if the maintainer is just mia?

History of PRs shows that they are being merged, so the repo seems to be alive, just not actively developed. My ToDoServer implementation was accepted back in December.

@Risto-Stevcev @kika Yes, that's right. I'm not into purescript anymore, but can always review and accept occasional PR. If somebody wants to take ownership and develop the project more actively I'll be happy to transfer it.

@dancingrobot84 I'd be happy to take over the project unless @kika really wants to do it

We're two steps closer, purescript-datetime was updated and I've just submitted PR for purescript-node-fs : https://github.com/purescript-node/purescript-node-fs/pull/30
I'll work on express using my local copy of node-fs meanwhile.
@dancingrobot84 I can take over the module, no problem. I use it for work and my product depends on it (not much currently, but I have plans to move more code from plain vanilla coffeescript+express to purescript). If you don't mind reviewing and accepting PRs I'm good with this too.

@Risto-Stevcev @kika Ok, then let's leave it how it is now until the time I can't review PRs anymore.

We're almost there. Currently the only thing left is https://github.com/bodil/purescript-test-unit/pull/19
And people are working on it.

All modules we were waiting for were updated and my fork compiles and runs. The only issue is tests. They were written using purescript-test-unit 3.0.0 which is very old and the API changed even before 0.9.1 release. I made some changes and resolved some minor problems, but work still needs to be done, as of now it is not able to pass Travis build because of the tests. Unfortunately, I can't devote much time now and finish the port of tests to the new 7.0.0 release of purescript-test-unit.
But all the examples run and so far run successfully, so my fork could be picked up by anyone who needs functional purescript-express on 0.9.1


Add jsDelivr hits badge

Hey, I noticed that handlebars.js gets 19M hits per month on jsDelivr, so I thought you might like this badge.

There seem to be no objections...


[Question] i can eval, but can not pretrain and instruction tuning

Question
If i run with deepspeed, got this error:
RuntimeError: The server socket has failed to listen on any local network address. The server socket has failed to bind to [::]:29500
If i run python with 1 gpu, got this error:
RuntimeError: FlashAttention only support fp16 and bf16 data type
Some of my environmental configurations:
Ubuntu
cuda 11.7
flash-attn==2.0.4
deepspeed==0.9.5
gpu:A100

I can run pretrain task with 1 gpu.  Just disabled flash_attention and reduced train batch_size.
The above problem is still not resolved

I can run pretrain task with 1 gpu. Just disabled flash_attention and reduced train batch_size. The above problem is still not resolved

Hi, have you solved this problem?


Dear Svante, can you please ad a 3D-object to your app?

hello again, I just wanted to let you know that I have created a new branch and I am currently working diligently on the issue and hoping to get great results shortly.


New package: Gigabyte.GigabyteControlCenter version <IP_ADDRESS>

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

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

Microsoft Reviewers: Open in CodeFlow

What is wrong?

Automatic Validation ended with:

(Automated response - build 586.)

Automatic Validation ended with:

(Automated response - build 588.)

Automatic Validation ended with:

2024-01-17T08:12:36.9962121Z ##[error] One or more errors occurred. (The file was not found. Verify git path must match with the casing of the file in disk. Not found file on 'manifests\g\Gigabyte\GigabyteControlCenter\<IP_ADDRESS>\Gigabyte.GigabyteControlCenter.installer.yaml') The file was not found. Verify git path must match with the casing of the file in disk. Not found file on 'manifests\g\Gigabyte\GigabyteControlCenter\<IP_ADDRESS>\Gigabyte.GigabyteControlCenter.installer.yaml'

(Automated response - build 589.)

This package installs and launches normally on a Windows 10 VM.

@stephengillie

Automatic Validation ended with:

2024-01-17T19:53:18.1739013Z ##[error] One or more errors occurred. (The file was not found. Verify git path must match with the casing of the file in disk. Not found file on 'manifests\g\Gigabyte\GigabyteControlCenter\<IP_ADDRESS>\Gigabyte.GigabyteControlCenter.installer.yaml') The file was not found. Verify git path must match with the casing of the file in disk. Not found file on 'manifests\g\Gigabyte\GigabyteControlCenter\<IP_ADDRESS>\Gigabyte.GigabyteControlCenter.installer.yaml'

(Automated response - build 602.)

@stephengillie

Hello @rakleed,
The package manager bot determined there was an issue with the pull request. Make sure the manifest files are under the manifests\partition\publisher\appname\version directory and only one package version is being modified in your PR. The partition of the path must be the first letter of the publisher in lower-case.
Example: Path: manifests / m / Microsoft / WindowsTerminal / 1.6.10571.0 / WindowsTerminal.yaml
For details on the error, see the details link below in the build pipeline.
You may also try the Windows Package Manager Manifest Creator
Template: msftbot/validationError/pullRequest

I didn't change WindowsTerminal. What is the problem?

Hello @rakleed,
The package manager bot determined there was an issue with the pull request. Make sure the manifest files are under the manifests\partition\publisher\appname\version directory and only one package version is being modified in your PR. The partition of the path must be the first letter of the publisher in lower-case.
Example: Path: manifests / m / Microsoft / WindowsTerminal / 1.6.10571.0 / WindowsTerminal.yaml
For details on the error, see the details link below in the build pipeline.
You may also try the Windows Package Manager Manifest Creator
Template: msftbot/validationError/pullRequest

/AzurePipelines run

/AzurePipelines run


brew install not working with current code

What happened:
Tried to brew tap and brew install but brew install failed with following error.
curl: (22) The requested URL returned error: 404 Not Found
Error: Failed to download resource "gardenctl"
Download failed: https://github.com/gardener/gardenctl/releases/download/v0.25.0/gardenctl-darwin-amd64

What you expected to happen:
brew install should be executed successfully and install gardener cli.
How to reproduce it (as minimally and precisely as possible):

execute following commands

brew tap gardener/tap
brew install gardenctl

Anything else we need to know:
no
Environment:
MacOS Catalina

/assign

this is due to our testing with GithubAction with gardenctl, i will revert the changes
/CC @AndreasBurger @dansible @zanetworker


[Project Zomboid]

Project zomboid apparently no longer uses the beta branch of the appid 10800 any more, rendering the pzserver script defunct.
According to this link:
https://steamcommunity.com/sharedfiles/filedetails/?id=514493377
the new appid is 380870.
There also appears to be a new server startup script included in the server base.

It also not longer requires a Steam account with ownership of the game to obtain the server - it can be retrieved from SteamCMD with the anonymous user

I am unable to resolve this currently as I'm away and wont be able to do this for a few weeks. If anyone is able to fix this before then I can merge it.

I'll try and take a look at this today.


Change the number of words displayed in 'suggestions'

There are too many suggested words displayed when coding, and the number of suggested words can't be changed now. I hope I can change the suggested words to arbitrarily number.

You can drag the lower border of the suggestions box up to reduce the number of suggestions shown.

@Chikapro Is this with latest stable (1.51)? Did you try what @ArturoDent suggested?

@ArturoDent @jrieken Yes I tried and it works now, thanks very much!


Add robot names

in Mobile_Detect.php on line 524:
    protected static $utilities = array(
        // Experimental. When a mobile device wants to switch to 'Desktop Mode'.
        // http://scottcate.com/technology/windows-phone-8-ie10-desktop-or-mobile/
        // https://github.com/serbanghita/Mobile-Detect/issues/57#issuecomment-15024011
        // https://developers.facebook.com/docs/sharing/best-practices
        'Bot'         => 'YandexBot|YandexImages|YandexVideo|YandexMedia|YandexBlogs|YandexFavicons|YandexWebmaster|YandexPagechecker|YandexImageResizer|YandexDirect|YaDirectFetcher|YandexCalendar|YandexSitelinks|YandexAdNet|YandexMetrika|YandexNews|YandexNewslinks|YandexCatalog|YandexAntivirus|YandexMarket|YandexVertis|YandexForDomain|YandexBot|Googlebot|facebookexternalhit|AdsBot-Google|Google Keyword Suggestion|Facebot|YandexBot|bingbot|ia_archiver|AhrefsBot|Ezooms|GSLFbot|WBSearchBot|Twitterbot|TweetmemeBot|Twikle|PaperLiBot|Wotbox|UnwindFetchor',
        'MobileBot'   => 'Googlebot-Mobile|AdsBot-Google-Mobile|YahooSeeker/M1A1-R2D2',
        'DesktopMode' => 'WPDesktop',
        'TV'          => 'SonyDTV|HbbTV', // experimental
        'WebKit'      => '(webkit)[ /]([\w.]+)',
        // @todo: Include JXD consoles.
        'Console'     => '\b(Nintendo|Nintendo WiiU|Nintendo 3DS|PLAYSTATION|Xbox)\b',
        'Watch'       => 'SM-V700',
    );

it's only Yandex bots added
YandexBot|YandexImages|YandexVideo|YandexMedia|YandexBlogs|YandexFavicons|YandexWebmaster|YandexPagechecker|YandexImageResizer|YandexDirect|YaDirectFetcher|YandexCalendar|YandexSitelinks|YandexAdNet|YandexMetrika|YandexNews|YandexNewslinks|YandexCatalog|YandexAntivirus|YandexMarket|YandexVertis|YandexForDomain|YandexBot

Facebook robot also ++

@jrean, it could be good idea to present facebook bots names and paste them here

facebookexternalhit/1.1 (+http://www.facebook.com/externalhit_uatext.php)
facebookexternalhit/1.1
Facebot
https://developers.facebook.com/docs/sharing/webmasters/crawler

The list of potential robot names could get quite large, I don't know it it would be the best solution. Might be even better of Mobiledetect could solve this in their code base.

So, it's better to create an issue for mobiledetect?)

I would definitely love to hear their take on this matter.

I saw that MobileDetect implemented a couple of new robots, is it enough to close this issue?


[Candidate 2v2 Freezing Bursa ZH v1] Oil Derricks at the top cannot be driven around, but at the bottom can be

Observed behavior
Oil Derricks at the top cannot be driven around, but at the bottom can be.
Expected behavior
Mirrored Oil Derricks offer equal room for unit mobility.
Screenshots

Unclear to me if this is a balance and blocker issue. The Oil Derricks are in the center, so perhaps about equal chance for both teams.

fixed

Works in principle, but there is no a little patch of water on the short, creating a smaller gap for units to drive through.


Use ConstraintLayout in dynamiclinks quickstart

Our dynamiclinks quickstart has a single screen which uses nested LinearLayouts. It would be nice if we could simplify it with a ConstraintLayout.

Hi @thatfiredev, can I take this one?

Sure @EdNgulele , assigned to you


LightManager Not Working Properly in Large Maps

CARLA version: 0.9.15
Platform/OS: Ubuntu 22.04
In my experiments I noticed that LightManager is only able to access and control (turn on/off, change color etc.) some of the street/building lights in Town15 and not all of them, where the lights are unaffected by changes applied through the LightManager.
It seems the problem comes from the fact that in Town15 the light IDs are all set to -1, and clicking Update Light IDs seems to have resolved the issue by assigning a unique ID to each light.
Another issue is that a lot of building lights seem to have been misclassified as street lights. I think this can be resolved by changing the light type in BP_Building from Street to Building.
BTW any plans on adding building lights to Town12?

Same way with the buildings, check your lights have a BP Lights component set to street light.