Database Cleaner starting too soon

I'm having an issue with truncation happening in the middle of an integration test. It deletes test records, and then test continues on until an ActiveRecord:NotFound occurs. Excerpt from log pasted below.
[1m (0.1ms)  BEGIN
  User Exists (0.2ms)  SELECT  1 AS one FROM `users`  WHERE `users`.`slug` = 'fb4b3e44ddf4' LIMIT 1
  User Exists (0.1ms)  SELECT  1 AS one FROM `users`  WHERE `users`.`email` =<EMAIL_ADDRESS>LIMIT 1
  User Exists (0.1ms)  SELECT  1 AS one FROM `users`  WHERE `users`.`remember_token` = 'tuyFEjBZgrQVJVQOK7pFiQ' LIMIT 1
  SQL (0.2ms)  INSERT INTO `users` (`created_at`, `email`, `first_name`, `last_name`, `password_digest`, `prefs`, `remember_token`, `slug`, `updated_at`) VALUES ('2015-01-15 04:13:26'<EMAIL_ADDRESS>'Tuna', 'Wiper 2', '$2a$04$pmPj9mzauNHbiUszUbk4teCFDFKSRHMK6Zmy1MyczfR0uu5hPtNEe', '--- !ruby/hash:ActiveSupport::HashWithIndifferentAccess\ndonation_emails: true\n', 'tuyFEjBZgrQVJVQOK7pFiQ', 'fb4b3e44ddf4', '2015-01-15 04:13:26')
   (0.8ms)  COMMIT
   (0.1ms)  BEGIN
  Invitation Exists (0.2ms)  SELECT  1 AS one FROM `invitations`  WHERE `invitations`.`slug` = '3c66c8586438' LIMIT 1
  SQL (0.2ms)  INSERT INTO `invitations` (`created_at`, `email`, `slug`, `trip_group_id`, `updated_at`, `user_id`) VALUES ('2015-01-15 04:13:26'<EMAIL_ADDRESS>'3c66c8586438', 1, '2015-01-15 04:13:26', 2)
   (0.5ms)  COMMIT
   (0.1ms)  SELECT @@FOREIGN_KEY_CHECKS
   (0.1ms)  SET FOREIGN_KEY_CHECKS = 0
   (6.1ms)  TRUNCATE TABLE `addresses`;
   (6.3ms)  TRUNCATE TABLE `administrations`;
   (7.9ms)  TRUNCATE TABLE `bank_accounts`;
   (5.6ms)  TRUNCATE TABLE `buddies`;
   (8.1ms)  TRUNCATE TABLE `campaigns`;
   (12.5ms)  TRUNCATE TABLE `donations`;
   (15.6ms)  TRUNCATE TABLE `error_logs`;
   (80.1ms)  TRUNCATE TABLE `event_logs`;
   (9.7ms)  TRUNCATE TABLE `grants`;
   (78.7ms)  TRUNCATE TABLE `guides`;
   (59.3ms)  TRUNCATE TABLE `invitations`;
   (160.3ms)  TRUNCATE TABLE `invoices`;
   (7.1ms)  TRUNCATE TABLE `organizations`;
   (6.1ms)  TRUNCATE TABLE `transaction_logs`;
   (7.1ms)  TRUNCATE TABLE `trip_groups`;
   (9.4ms)  TRUNCATE TABLE `trips`;
   (10.8ms)  TRUNCATE TABLE `users`;
   (0.1ms)  SET FOREIGN_KEY_CHECKS = 1
Started GET "/signin" for <IP_ADDRESS> at 2015-01-14 23:13:27 -0500
Processing by SessionsController#new as HTML
  Rendered layouts/_h1.html.erb (0.1ms)
  Rendered sessions/_session_form.html.erb (1.3ms)
  Rendered layouts/_meta.html.erb (0.1ms)
  Rendered layouts/_shim.html.erb (0.1ms)
  Rendered layouts/_scripts.html.erb (0.2ms)
  Rendered layouts/_brand.html.erb (0.2ms)
  User Load (0.4ms)  SELECT  `users`.* FROM `users`  WHERE `users`.`remember_token` IS NULL LIMIT 1
  Rendered layouts/_nav.html.erb (1.3ms)
  Rendered layouts/_header.html.erb (2.1ms)
  Rendered layouts/_content.html.erb (0.2ms)
  Rendered layouts/_links.html.erb (0.2ms)
  Rendered layouts/_footer.html.erb (0.3ms)
Completed 200 OK in 7ms (Views: 6.0ms | ActiveRecord: 0.4ms)
Started POST "/sessions" for <IP_ADDRESS> at 2015-01-14 23:13:27 -0500
Processing by SessionsController#create as HTML
  Parameters: {"utf8"=>"✓"<EMAIL_ADDRESS>"password"=>"[FILTERED]"}, "commit"=>"Let me in!"}
  User Load (0.4ms)  SELECT  `users`.* FROM `users`  WHERE `users`.`email` =<EMAIL_ADDRESS>LIMIT 1
  Rendered layouts/_h1.html.erb (0.1ms)
  Rendered sessions/_session_form.html.erb (0.9ms)
  Rendered layouts/_meta.html.erb (0.1ms)
  Rendered layouts/_shim.html.erb (0.0ms)
  Rendered layouts/_scripts.html.erb (0.2ms)
  Rendered layouts/_brand.html.erb (0.2ms)
  User Load (0.3ms)  SELECT  `users`.* FROM `users`  WHERE `users`.`remember_token` IS NULL LIMIT 1
  Rendered layouts/_nav.html.erb (0.8ms)
  Rendered layouts/_header.html.erb (1.2ms)
  Rendered layouts/_content.html.erb (0.1ms)
  Rendered layouts/_links.html.erb (0.1ms)
  Rendered layouts/_footer.html.erb (0.3ms)
Completed 200 OK in 6ms (Views: 3.9ms | ActiveRecord: 0.7ms)
Started GET "/i/3c66c8586438/new" for <IP_ADDRESS> at 2015-01-14 23:13:27 -0500
Processing by CampaignsController#new as HTML
  Parameters: {"invitation_id"=>"3c66c8586438"}
  Invitation Load (0.3ms)  SELECT  `invitations`.* FROM `invitations`  WHERE `invitations`.`slug` = '3c66c8586438' LIMIT 1
Completed 404 Not Found in 2ms

ActiveRecord::RecordNotFound (ActiveRecord::RecordNotFound):
  app/controllers/application_controller.rb:54:in `find_invitation_by_invitation_id'
  test/integration/campaign/campaign_new_share_test.rb:13:in `block in <class:IntegrationTest>'

Here is how database cleaner is being invoked in minitest:
 def setup
    DatabaseCleaner.strategy = :transaction
    DatabaseCleaner.clean_with :truncation
    DatabaseCleaner.start
  end

  def teardown
    DatabaseCleaner.clean
  end

Strange - any ideas?

I have the same issue. Could not find what triggered truncation in cucumber test. After crossing all buy-checkout procedure, something starts truncating before test is finished.
Side effect of experiments is that I found that 'sleep' of any lengtht in controller will start truncating procedure.

My issue was related to the minitest_bang gem. Reading my test log, I noted that anything running before truncation began was being created using let!(). This occurs immediately when test begins, prior to any before blocks. Therefore, when the pre-test truncation ran in the before block, it was erasing everything created by let!(). I resorted to removing the bang and using let(), manually invoking objects in a before block.
Instead of this:
let!(:person) { Factory :person }

I did this:
let (:person) { Factory :person }

before { person }


Reported shelljs vulnerability through<EMAIL_ADDRESS>- use<EMAIL_ADDRESS>instead

I work in a high security environment and I wanted to use your library. Unfortunately it relies on<EMAIL_ADDRESS>that has a known shelljs vulnerability as reported here:
https://snyk.io/test/npm/standard?tab=issues
The good news is that the later version (4.7.2) of eslint does not have that vulnerability:
https://snyk.io/test/npm/eslint
Hopefully this is something that can be changed for this project.

Yes, I have the same problem, with us not really being able to use standard until it changes this dependency due to company policy surrounding insecure dependencies.
Any chance this dependency update will happen anytime soon? Cheers.
Details here: https://www.bithound.io/github/standard/standard/master/dependencies/npm#filter-insecure-dep

Hi folks!
A beta of the 11.0.0 version of standard has been released, which upgrades eslint to its latest version, which should resolve this issue.
Could you give it a try? npm install standard@beta

🗣 Standard 11 is released! Run npm install standard@latest --save-dev to update to the latest version. This will also update the version in package.json ✨
Changelog: https://github.com/standard/standard/blob/master/CHANGELOG.md#1100---2018-02-18


All checkpoint downloads unavailable

All checkpoint links lead to the same page:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<Error>
PublicAccessNotPermitted
<Message>Public access is not permitted on this storage account. RequestId:677b1eb8-501e-0054-079c-93e2cc000000 Time:2023-05-31T08:48:21.6368816Z</Message>
</Error>

Yeah this is ridiculous

thanks for reminding me. It is because the public access has been turned off per policy. will move the ckpts to huggingface.

@jwyang Thanks for the reply! Do you know when the checkpoints will be available on huggingface? The downloads are currently still unavailable.

@jwyang Do you know when the FocalNet-DINO will be available on hugging face? The uploaded checkpoints are still classification backbones.

@jwyang , FocalNet-Huge ImageNet22K weight seems to be missing from the huggingface while all the other architectures are available (e.g. Base, Large). Is this intentional?

This is not intentionally, I also do not understand that and will upload them to huggingface.


Handful of Fixes

Sorry this is a bit messy but here's what was done:

Turn get_log.py into just a plain Python script (no ROS) because it doesn't need rclpy
Remove unused search_freq_bw_log function in get_log.py
Remove outlier filtering in get_log.py as this was causing many empty CSV files to be produced
Fix a bug in dropped message counting
Fix a problem caused by FastRTPS not forwarding the QoS profile from publishers (it is not required that the DDS provider does this, BTW, but Cyclone does it anyway)

@LawrenceXu13467 When you have time.


Zarr Extension?

Following up on the discussion in https://github.com/radiantearth/stac-spec/issues/713, I thought it would be good to raise a standalone issue about how STAC may interact with the Zarr format. Zarr is a storage protocol for multidimensional arrays + metadata. (Think an alternative to HDF). It began with a python implementation, but, using the Zarr V2 Spec, implementations now exist in Java, C++, Julia, and JavaScript.
Zarr can store data in anything that resembles a key-value store: both filesystem (keys are filenames) and cloud object store (keys are object names) work well. Consequently, zarr works well as a cloud-optimized format for storing geospatial data, and we are using it extensively in the Pangeo project (e.g. http://catalog.pangeo.io/).
Zarr stores its metadata in json files and stores the arrays in binary chunks (i.e. fragments), which are optionally compressed. So the layout of a zarr group may look something like this:

where the dark boxes are json and the light boxes are binary data. In this way, Zarr resembles Parquet, but aimed at ND-arrays rather than tabular data.
If Zarr were simply a standalone file format, analogous to COG, it would be reasonable to catalog Zarr using STAC Items. However, Zarr arrays consist of many different files / objects, none of which is meaningful on its own. In that sense, Zarr is more akin to a STAC Catalog or Collection. Zarr arrays also tend to be aggregated at a higher level than individual images / scenes. This notebook, for example, shows how Xarray represents a particular Zarr store:

In this example, all of the timesteps from the original netcdf files have been concatenated in a single Zarr array (which consists of thousands of individual chunks). We have a data cube, rather than a data square. (Note that the number of dimensions in Zarr is arbitrary, and we have examples with up to 6 dimensions, e.g. scenario, ensemble, time, height, lat, lon).
We are putting more and more data into the cloud in Zarr format (e.g. CMIP6), and we would like to be able to catalog this data using STAC. However, the lack of clarity about how to refer to Zarr data in STAC is a barrier.
I have a few main questions for this community:

Looking closely at the STAC Item spec, the main challenge appears to be the requirement that the item have a single datetime property. Since we are generally storing all the timesteps from a given product, it doesn't make sense to specify a single datetime. Is there any chance of an option to specify a datetime range at the Item level?

Alternatively, we could avoid the Item problem by referring to Zarr via a Collection or Catalog. Would it make sense to refer to a Zarr array or group via a Collection? The Standalone Collection concept appears to allow for a collection with no links to Items.

Another issue is that the Zarr spec already defines a convention for storing metadata (conveniently, in .json files). It would be redundant to duplicate all this metadata in STAC Item or Collection. But if STAC were made aware of Zarr via an extension, processors could easily crawl and this metadata directly. Does that idea sound feasible?

If we can resolve these issues, I believe we can move forward with generating STAC catalogs for all of our cloud-based Zarr data.
I also note that this discussion is somewhat orthogonal to the ESM collection spec discussion in https://github.com/NCAR/esm-collection-spec/issues/21. One should be able to catalog Zarr with STAC, regardless of whether or not one is using the ESM data model.

Short of time at the moment, so just a heads up to some issues:

Looking closely at the STAC Item spec, the main challenge appears to be the requirement that the item have a single datetime property. Since we are generally storing all the timesteps from a given product, it doesn't make sense to specify a single datetime. Is there any chance of an option to specify a datetime range at the Item level?

I'll support you on this for sure. I already tried to get this into STAC, but was rejected: #613. Hopefully it's getting on the agenda again.

2. Alternatively, we could avoid the Item problem by referring to Zarr via a Collection or Catalog. Would it make sense to refer to a Zarr array or group via a Collection? The Standalone Collection concept appears to allow for a collection with no links to Items.

I feel like Zarr groups would mostly be collections or catalogs and Zarr arrays would be items, but that's more a feeling than anything profound.

3. Another issue is that the Zarr spec already defines a convention for storing metadata (conveniently, in .json files). It would be redundant to duplicate all this metadata in STAC Item or Collection. But if STAC were made aware of Zarr via an extension, processors could easily crawl and this metadata directly. Does that idea sound feasible?

This sounds a bit like a use-case for #757.

Looking closely at the STAC Item spec, the main challenge appears to be the requirement that the item have a single datetime property. Since we are generally storing all the timesteps from a given product, it doesn't make sense to specify a single datetime. Is there any chance of an option to specify a datetime range at the Item level?

I'll support you on this for sure. I already tried to get this into STAC, but was rejected: #613. Hopefully it's getting on the agenda again.

So I'm still not sure about having a default item have a single value and a time range on the same footing, where implementors can just pick either and clients then have to know about both and be able to handle either. I think we should do one by default - we originally did have a range, but it felt weird there too to shoehorn everything into a range.
What I am very much for is extensions being able to say 'for this extension a range is all that makes sense', and zarr and datacubes are perfect examples of that. I could see something like a special value of 'datetime', like it is 'null' or says 'range' or something, which indicates that a single datetime is meaningless, but that the date time range values are present.
Or maybe we just somehow allow extensions to override how the datetime works, and can say for their data the date time range is required, and the datetime field is optional. I know that probably gets messy with the json schema, and perhaps it's a bit looser in actual validation, but the spec defines it more strongly.
In short - the original idea was that extensions should refine their meaning of the time, and I think it makes sense to flesh things out so datacube and zarr extensions can make the range the required thing, and the single datetime less important.
But I do think it's important the core spec doesn't start with 'you can use a single time or a range' - that we have a default recommendation: Pick a single datetime. If you can't then use or define an extension that uses a range.

As for general responses on zarr:
I think the key thing for us to figure out in STAC is whether a datacube (zarr or otherwise, thinking netcdf fits too) that has tangible assets should be defined as a Collection or an Item.

Collection we'd need to add 'assets' at the collection level.
Item we'd need a stronger / clearer way to define it with a range.

These both feel like critical things to figure out for 1.0-beta1 (not saying we necessarily add each, but we need to decide and show how to handle this and other similar use cases).
I have a similar feeling that zarr arrays likely makes sense as items, but like Matthias I'm not certain, have not worked with zarr enough. But it feels like that's the level it could be useful to 'search' on. Like you find a specific model / model run, just like you find a specific 'image' from an larger catalog. I don't know that there'd need to be 'items' defined for every array (though could if it's easy) - you could just pick key ones to expose. I liked the idea discussed on the call that you might have a number of 'collections' for CMIP, like all X type of simulations. And then I think items below that makes sense, as I'm hesitant to have big hierarchies of collections (though I suppose it is possible).

Another issue is that the Zarr spec already defines a convention for storing metadata (conveniently, in .json files). It would be redundant to duplicate all this metadata in STAC Item or Collection. But if STAC were made aware of Zarr via an extension, processors could easily crawl and this metadata directly. Does that idea sound feasible?

This sounds a bit like a use-case for #757.

I think it's a wee bit different, but definitely similar. For that one I'd propose a new JSON structure for those adding more metadata. I think this one is simpler - the zarr extension would just require that there's an asset with a role 'zarr-metadata' (or whatever makes sense to call it) and it provides the link to that. That's in line with assets in general - you can refer to metadata, and I think it'd make good sense for zarr to define and (possibly) require a role / file type that has that.

I think the key thing for us to figure out in STAC is whether a datacube (zarr or otherwise, thinking netcdf fits too) that has tangible assets should be defined as a Collection or an Item.

Collection we'd need to add 'assets' at the collection level.
Item we'd need a stronger / clearer way to define it with a range.
We have PRs for both now: Collection Assets #800, Item Date Ranges #798

I wanted to check in on this issue. I apologize for losing track of it somewhat. (That happened with a lot of projects this spring / summer).
Since both #800 and #798 have been merged, it should be possible for us to represent our zarr stores in STAC. That's what @charlesbluca has been working on in https://github.com/charlesbluca/pangeo-datastore-stac/.
I'd love to push this forward as part of the STAC data sprint next week. Please help us figure out where contributions are needed.

I think we need to finalize the esm-collection-spec PRs, especially the written docs. I think there were one or two open questions to be answeed by you guys. What do you need from us to finish them? Let me know if there's anything I can help with.

Thanks @m-mohr. Yes we definitely stalled on our end, for various reasons. Honestly I think the biggest thing we are missing is a high-level view of how we as a community / project are going to be interacting with STAC going forward. There are lots of moving pieces that have to move together.

I think we need to finalize the esm-collection-spec PRs, especially the written docs.

I think there may be two related but distinct issues here.

esm-collection-spec transition to STAC (discussed in https://github.com/NCAR/esm-collection-spec/issues/21)
how to reference a single, standalone zarr store from STAC. This need not involve esm-collection-spec, which is a higher-level concept. That's what this issue is about.

Over in https://github.com/charlesbluca/pangeo-datastore-stac, we are working on a STAC catalog that exposes individual Zarr groups in a hierarchy. Our problem seems to be that we don't know whether it is "right." Since we are not (yet) using STAC on the client side, we have no way to check if the catalog is usable. In https://github.com/charlesbluca/pangeo-datastore-stac/issues/1 we are discussing validators. If you could provide some advice on how to test or validate our catalog, even informally / non-rigorously, ideally from python, that would help us bring some closure to that step of the project.
The next step would be to try to plug these catalogs into one of the pretty STAC front ends so we can browse the catalog interactively. We would then work towards rewriting / replacing our existing Pangeo catalog website, an intake / flask app living at https://github.com/pangeo-data/pangeo-datastore-flask, to something javascript based.

Let's try to have a session on stac + zarr at the sprint next week to push things forward. I was talking to @abarciauskas-bgse and she's going interested in STAC plus zarr, and will be working to add STAC to https://registry.opendata.aws/mur/ soon. That might actually be an easier dataset for us to all start on, since it looks a bit more focused - thinking through the huge zarrs hurts my brain. She's also going to give an 'intro session' on zarr, which I'm hoping to attend that will hopefully get me a bit more up to speed to be able to contribute to the conversations.

I tried submitting our catalog to the STAC index and it kind of worked!
https://stacindex.org/catalogs/pangeo
I consider this a big win!

Nice! It doesn't support showing collection assets yet (https://github.com/radiantearth/stac-browser/issues/38) so it doesn't show anything to download, but other than that, it's cool!


Custom Summarization Runtime API

Data Plane API - Pull Request
Custom Summarization
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your engagement record.

Link to engagement record issue:

Is this review for (select one):

[] a private preview
[x] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous Open API document (swagger) if applicable, and the root paths that have been updated.

Design Document:https://microsoft-my.sharepoint-df.com/:w:/g/personal/ylxiong_microsoft_com1/ET5gzsTjgTVGoBXAN2BG5ucBwAi3GB7gC_IKVFVvuMHUUg?e=5vLTuv
Previous Open API Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

All the breaking changes are in response-only models, so based on https://aka.ms/AzBreakingChangesPolicy these are acceptable in a new api-version.


Add stylings for shopping lists index page

Add stylings for shopping lists index page

One last functionality required we did not create it :
Recipes list:
If the user is the owner of the recipe, should allow the user to delete it.


Helldivers 2 "Not Linked" status.

Attempting to link an account for Helldivers 2 drop seems to only function if you've already claimed the drop if I understand correctly. Twitch Drops Miner refuses to mine the drop unless it sees the account status as "Linked" however- is there any way to force the bot to mine for the drop despite not being seen as linked?

Hello.
No, there's no way to enable mining campaigns you don't have the account linked to. This makes the Helldivers 2 game unsupported by this miner.
This is also a duplicate of #439 and #432.


Remove EnsurePSANotPrivileged

Full revert of [1] and partial revert of [2].
[1] https://github.com/openshift/hypershift/pull/3107
[2] https://github.com/openshift/hypershift/pull/3137
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes # OCPBUGS-30651
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[x] This change includes unit tests.

/retest

/retest

I don't see how the failure logs have any relation to this PR.

/retest

Per https://ibm-argonauts.slack.com/archives/C01C8502FMM/p1710414602696739, the test problem may be resolved.

/retest

/approve

/retest

/retest

/retitle [release-4.15] OCPBUGS-30651: Remove EnsurePSANotPrivileged

/retest


New getIP

Tested with launching a bridge container and use the hack method to get the ip.

This getip logic should really only be invoked for InstanceActivate and nothing else.

@ibuildthecloud  PR updated according to your review

LGTM


Do not install icon cache upon staged install

Upon installation, updating icon cache is relevant so that new icons can be found by the system.
Upon a staged installation though, i.e. when DESTDIR is set to a staging installation area, it is not relevant to update the icon cache as this cache is not the actual target system's cache.
Staged installation is typically used when creating a package of the application. Cache update is a problem well known to packagers which typically handle it by wrapping cache update commands into scripts to be executed at post (un)installation time - on the real target system.
See also:

#703
The tutorial referenced in the ticket which actually mentions this tricky staged install case

Thank you for setting me straight (I misunderstood the purpose of this bit), and thank you for the fix!


Is there Xamarin binding for this

Report
Plugin Version
Any
ℹ Please replace these two lines with the plugin version.
e.g. via 4.1.1
NA
On what Platform are you having the issue?
Android through Xamarin forms
What did you do?
Switching to segment
We have been using Appsflyer Xamarin nuget. So now we are switching to Segment.
We noticed there is no Xamarin counterpart for that.
So what should be the way to go ?
Can we use Segment library along with Appsflyer.
Are there plans to have Xamarin library for this repo?
If not how can we route installation/attribution events through Segment.
I notice we have AppsFlyerLib.Instance.Start in the Appsflyer Xamarin library , do we need to have AppsFlyerConversionDelegate instead of that for this approach.

Hi @pmahend1,
Thank you for your message!
Currently, there is no plan to have a Xamarin library for this repo.
However, using Segment library along with AppsFlyer in Xamarin should not be a problem.
I'm not sure what you mean with the last question, but the start method and ConversionDelegate are not the same.
Please contact support if you have more questions on how to implement it!

So my question is about the approach . I am looking the code you have in this repo..
Call AppsFlyerLib.Instance.Init(appsFlyerkey, new AppsFlyerConversionDelegate(this), this.Application);
public class AppsFlyerConversionDelegate : Java.Lang.Object, IAppsFlyerConversionListener
    {
        AppCompatActivity activity;

        public AppsFlyerConversionDelegate()
        {
        }

        public AppsFlyerConversionDelegate(MainActivity activity)
        {
        }

        public void OnAppOpenAttribution(IDictionary<string, string> p0)
        {
        }

        public void OnAttributionFailure(string p0)
        {
        }

        public void OnConversionDataFail(string p0)
        {
        }

        public void OnConversionDataSuccess(IDictionary<string, Java.Lang.Object> attributionData)
        {
            Dictionary<string, object> valueMap = new Dictionary<string, object>();
            valueMap.Add("ad_group", attributionData["adgroup"]);
            valueMap.Add("name", attributionData["campaign"]);
            valueMap.Add("source", attributionData["media_source"]);
            Dictionary<string, object> properties = new Dictionary<string, object>();
            properties.Add("provider", "AppsFlyer");
            foreach (var item in attributionData)
            {
                properties.Add(item.Key, item.Value);
            }

            properties.Remove("media_source");
            properties.Remove("adgroup");
            properties.Add("campaign", valueMap);
            Segment.Analytics.Client.Track("Install Attributed", properties);//
        }
    }

Will this be equivalent code we need to have in C# to accomplish this.

Thanks @af-margot I have emailed to the support email you mentioned.


chore: update tag to digest

Resolve #677
Signed-off-by: Yi Zha<EMAIL_ADDRESS>

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

Comparison is base (4569f3f) 64.05% compared to head (31b24ea) 64.05%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #837   +/-   ##
=======================================
  Coverage   64.05%   64.05%
=======================================
  Files          40       40
  Lines        2323     2323
=======================================
  Hits         1488     1488
  Misses        709      709
  Partials      126      126

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


Broken syntax highlighting for multiline function args in typescript

Do you have a question? Please ask it on Stack Overflow with vscode tag
(Use Help > Report Issues to prefill these)

VSCode Version: 1.14.0-insider d5e10a589ff68d026ba9cf57485d3004149a9c8a
OS Version: MacOS Sierra 10.12.5

Steps to Reproduce:

open vscode
paste this snippet:

export const toPdf = (renderer: Renderer) => <P>(
  document: Doc<P>,
): Promise<string> => {
  return renderer(document.html, document.config);
};

// compared to
export const toPdf = (renderer: Renderer) => <P>(document: Doc<P>): Promise<string> => {
  return renderer(document.html, document.config);
};

Syntax is fine when first arg appears right after opening parenthesis:

But when you take first arg to next line, here's what happens:

Everything below line 94 turns white.

This issue was moved to Microsoft/TypeScript-TmLanguage#481


Label position

Hi,
I try to explain my "problem" ....
How you can see I have this gauge (in fact are many more) and I have difficult to understand to which gauge label refer ....
I always watch the lower label, not the upper ...
Well, is possible to have the label under the gauge?

It is of course possible to do. But it's not an easy task. As the label is part of svg and all the elements are positioned carefully to maintain the look for many different configuration possibilities the rearrangement will take a lot of work.
Unfortunately I have a ton of other responsibilities to cover before I could do this.


Fix documentation typo: screenWindowWidth is float, not V2f

Website preview: https://openexr--1778.org.readthedocs.build/en/1778/

Looks good to me


About dataset

Hi author, I would like to ask, can you share the code that handles the data？

Hi, the dataloader is in the data folder


Fix compiling static attributes

Hi, this will revert static expressions, this should rather be:
if ($attribute->hasStaticMember('value') || $attribute->getValue() instanceof TextElement)

As far as I understand, based on AttributeNodeCompiler, $attribute->getValue() can be TextElement or ExperssionElement.
Can you give an example of pug code then attribute value will be string?

Furthermore, compiler already check hasStaticValue, so we can only check instanceof TextElement

It might happen on hooking the compile node event or if you have custom(s) assignment_handlers. You can manually add/yield an attribute in such events or interceptors:
$attribute = new AttributeElement('src', '/foo/bar.png');
$img->getAttributes()->attach($attribute);

So removing the hasStaticMember can cause a regression in existing apps.

Sorry for the delay. I miss the last commit. I will check this change in phug/phug and I think it will be fine.

Tested on phug/phug and it breaks the attributes_mapping option.
@rodion-k Can you give me an example of what exactly would be fixed by this PR. A complete minimal case of PHP rendering with the Pug code and actual vs. expected output.
Else it will still be a no-go for now.

Pug:
$renderer = new Phug\Renderer(['debug' => false]);
$renderer->compile('input.form-control(name="foo" value="bar")');

Actual:
# $pugModule init
<input<?= (is_bool($_pug_temp = $pugModule['Phug\Formatter\Format\BasicFormat::attributes_assignment']([], ['class' => 'form-control', 'name' => 'foo'], ['value' => 'bar'])) ? var_export($_pug_temp, true) : $_pug_temp) ?> />

Expected
<input class="form-control" name="foo" value="bar" />

@rodion-k this case should not returns static HTML code, it seems static but it's dynamic under the hood since:
Options can intercept attributes with events or attributes_mapping. Class can be merged. You can have some extension that add a class="foo" attribute to input tag. Then it should not render class="foo" class="form-control" but it should render class="foo form-control".
So make class compiling into static HTML would many features no longer available.
The compile method is public only to ease integration with framework caching, it aims to produce a PHP code able to render the HTML but it's not meant to get some predictable code and less likely HTML ready-to-use code.
What is your high level goal in this fix?

The goal is performance. While profiling I found thousands calls of $pugModule functions.

And then realized what part of that they do can be done at compile time. If we should always run attribute_assigment module, why there is condition hasNonStaticAttributes in this line?

hasNonStaticAttributes will be taken into account for example an hook (programmatically created element, options or interceptors) provides static attributes.
$attribute = new AttributeElement('width', '12');
$iframe = new MarkupElement('iframe');
$iframe->getAttributes()->attach($attribute);

$formatter = new Formatter();

echo $formatter->format($iframe);

This output <iframe width="12"></iframe> (without rendering). This is a very particular case. With the whole process from a pug code input to the compiled output you should always fall in the dynamic case as it does for pugjs.

It's an hard decision, but I cannot merge this PR as is. It would need a condition such as "no hook and no attributes_mapping option set".


Visual Studio 2015 support

FYI, running into errors during OpenSSL build when trying VS 2015 build (both x86 & x64)
        link /nologo /subsystem:console /opt:ref /debug /out:out32\constant_time_test.exe @C:\Users\Ali\AppData\Local\Temp\nm7651.tmp
constant_time_test.obj : error LNK2019: unresolved external symbol __iob_func referenced in function main
out32\constant_time_test.exe : fatal error LNK1120: 1 unresolved externals
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\amd64\link.EXE"' : return code '0x460'
Stop.

Microsoft (R) Program Maintenance Utility Version 14.00.23026.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Building OpenSSL
        link /nologo /subsystem:console /opt:ref /debug /out:out32\constant_time_test.exe @C:\Users\Ali\AppData\Local\Temp\nm77C6.tmp
constant_time_test.obj : error LNK2019: unresolved external symbol __iob_func referenced in function main
out32\constant_time_test.exe : fatal error LNK1120: 1 unresolved externals
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\amd64\link.EXE"' : return code '0x460'
Stop.

For Perl I am using strawberry-perl-<IP_ADDRESS>-64bit-portable.zip

Might be something that is fixed in the latest 1.0.1 release https://github.com/openssl/openssl/releases/tag/OpenSSL_1_0_1p
You can just change the tag checkout in the build script to OpenSSL_1_0_1p here https://github.com/realXtend/tundra-urho3d/blob/master/tools/Windows/BuildDeps.cmd#L297
Might be a good idea at the same time to update curl (why we build openssl in the first place) to https://github.com/bagder/curl/releases/tag/curl-7_45_0 again branch to curl-7_45_0 here https://github.com/realXtend/tundra-urho3d/blob/master/tools/Windows/BuildDeps.cmd#L355

If that error comes just from building a test executable, sounds like it could be skipped?

I'll try updating OpenSSL and Curl first. But yeah, completely insignificant test biz could be skipped, if possible.

Updating OpenSSL to 1.0.2d fixed the build. Updating Curl caused the build to fail with VS 2015, but the current Curl version builds OK.

I think OpenSSL keeps two version tracks 1.0.1 and 1.0.2, afaik it has been like this for years. I would suspect 1.0.2 has some API changes that certain projects might not work with. Did you try the latest 1.0.1 + latest Curl? Maybe they used something that no longer worked with 1.0.2? Dunno...
In general should always try to use latest OpenSSL when they arrive. So should be fine to update for all VC. Linux uses PolarSSL https://github.com/realXtend/tundra-urho3d/blob/master/tools/Linux/Ubuntu/Build.bash#L263-L265 that Curl also supports. Somehow it was just less hassle. We should update that one too to latest (and curl) if someone has time.

Using latest ssl libraries is always the right call, but more critical for projects that do HTTPS hosting. Usually security bugs are on the server side, we are using OpenSSL purely to support Curl HTTPS GET for resources at the moment. Ofc client chiphers and other stuff can get updated so it's important to be up to date.

@jonnenauha Yes, latest OpenSSL 1.0.1p failed to build. Quick googling would suggest that this error occurs when doing static lib build, but not when doing dynamic lib build. Buliding latest Curl failed with OpenSSL 1.0.2d.

Websocketpp says they support TLS. Naturally the devil is in the details once we actually try running it.

Should I commit the VS 2015 with OpenSSL update to 1.0.2d or should we leave this pending for now?

I think you can commit if it also works on older VS. If you could try 2010 works. Leave the curl as is then.
It took a fair bit of time to get curl linked with static OpenSSL, similar is done in linux with PolarSSL.  Hopefully we don't need to go to dynamic linking due to this.

I'll test with 2013.

BTW didn't find an obvious way for disabling building tests for OpenSSL. I'll test HTTPS biz by using TXML from Dropbox and if it works, I'll commit the OpenSSL update and VS 2015 support.

Lasse was getting the same error for VS 2013 + OpenSSL 1.0.1 build so I decided to pull the trigger on this.

VS2010, 2013 and 2015 deps all seem to be building fine. Had to update CMake though, curl couldn't build with CMake 3.1.0.


Detailed example with logging, database access, environment variables, and basic middleware

I'm currently learning sanic (am a long-time aiohttp user). I was reading the beginner Issues and ran across #72. So I created this example of accessing (GET, POST) a passworded redis cache using aioredis, and added document logging (where log files are placed in location specified by APP_LOGFILE).
Created detailed example of using sanic. Adds configurations based on various environment variables, handles database access (using aioredis), uses middleware to check for db object and attach it to request object, and logs events to a logfile (which is set using environment variables).
I hope this satisfies #72. If not, I can go back and make more changes.

So Ive made some updates to the example based on suggestions by @go2sh


Ensure metric is reported for unexpected exceptions during auth

In https://github.com/stargate/stargate/pull/756 a new metric was added for clear auth errors but as @dimas-b pointed out "in case of unexpected exceptions during auth, the CQL client often does not get any clear error message and often just times out".
We need to dig deeper to catch when these sort of errors happen and report a metric for it.

@kathirsvn is this something we should keep open, or close?

@jeffreyscarpenter I think we should keeps this open. I remember seeing this behavior. When there's some uncaught exception, the cql client ll not get clear error message.


python311Packages.pytensor: 2.25.1 -> 2.25.2

Description of changes
Changelog: https://github.com/pymc-devs/pytensor/releases/tag/rel-2.25.2
cc @bcdarwin @ferrine

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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 327569 run on aarch64-darwin 1

6 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

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

2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

4 packages built:

python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

Result of nixpkgs-review pr 327569 run on x86_64-darwin 1

6 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

Result of nixpkgs-review pr 327569 run on aarch64-linux 1

2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

4 packages built:

python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist


'Never': different behavior with and without function type declaration

🔎 Search Terms
Never, return type never
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
https://www.typescriptlang.org/play?#code/DYUwLgBAhgXBDOYBOBLAdgcwgHwgVzQBMQAzdEQgKFEgCM5FVMd8jTyqaIBjB5dLLgLEyaCpUrcA9mkTQSEALwQAFAEolAPggBvShAMQwACyRSA7hDGWAokjNJ1lAL6SZc2guXq4YgG4gSFq6+oYmZpbWEHYOTq7SspDcJHDqwf6BSqoaitp6hkamFlYgtvZSjmouEmAAngAOIEYgiACCAGJZAErgeEhoACoNIAA8dY1SClAkmpQA9HOGAHoA-JTjTWAtYABCnco9YH2Dw2PDkxCeswvLaxvNiADC+xCHx0ONZxMKydeLBqsJJQUAoVABCKAaabqADclCgADpQJgTBIQaowbQNJ5YZRaEiQCjjBA0aCwdwNMlcdwCUSIEA
💻 Code
let a: string | undefined
let b: string | undefined
let c: string | undefined

const af = () => {
    throw new Error()
}
const bf = (): never => {
    throw new Error()
}
const cf: () => never = () => {
    throw new Error()
}

if (!a) af();
a.length // Error

if (!b) bf();
b.length // Error

if (!c) cf();
c.length // Fine

🙁 Actual behavior
Typescript does not check 'never' in condition statements but does if the whole function is declared
🙂 Expected behavior
The 'never' return type should always behave in the same way
Additional information about the issue
No response

This is working as intended. See #32695 and many many others (e.g. #55351).


AlertDialog should not inherit a global menu bar

Currently AlertDialog inherits a menubar. A complete but short demo is below. I think the window for the alert should be made configurable, and the menubar should not be shown. Here is how the demo looks at the moment:

Here the demo that produces this output:
package com.thomaskuenneth

import androidx.compose.desktop.AppManager
import androidx.compose.desktop.Window
import androidx.compose.material.AlertDialog
import androidx.compose.material.Button
import androidx.compose.material.Text
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.ui.window.Menu
import androidx.compose.ui.window.MenuBar
import androidx.compose.ui.window.MenuItem

fun main() {
    AppManager.setMenu(MenuBar(
            Menu("File", MenuItem("Quit", onClick = {
                AppManager.exit()
            }))
    ))
    Window(title = "AlertDialogDemo") {
        val show = remember { mutableStateOf(false) }
        Button(onClick = {
            show.value = true
        }) {
            Text("Show AlertDialog")
        }
        if (show.value) {
            AlertDialog(onDismissRequest = {},
                    title = {
                        Text("A title")
                    },
                    text = {
                        Text("A text")
                    },
                    confirmButton = {
                        Button(onClick = {}) {
                            Text("A button")
                        }
                    })
        }
    }
}

Fixed in version 0.3.0-build138.

Fixed in version 0.3.0-build138.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Excel workbook with macro enabled not supported

If an excel workbook with macro enable is used I get the an error that I included below. I used macros in my excel workbook for all kind of custom functions and other automation scripts that help write consistent forms. I have a macro that insert a repeat group with a regular group nested in it for exemple. And another that increment the version when I save. This is a feature request and not a bug but I am not sure how to tag it as such

G:....\Survey Instruments>xls2xform wtp.xlsm wtp.xml
Traceback (most recent call last):
File "c:\program files\python37\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\program files\python37\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "c:\program files\python37\Scripts\xls2xform.exe_main.py", line 9, in
File "c:\program files\python37\lib\site-packages\pyxform\xls2xform.py", line 137, in main_cli
enketo=args.enketo_validate)
File "c:\program files\python37\lib\site-packages\pyxform\xls2xform.py", line 19, in xls2xform_convert
json_survey = xls2json.parse_file_to_json(xlsform_path, warnings=warnings)
File "c:\program files\python37\lib\site-packages\pyxform\xls2json.py", line 1003, in parse_file_to_json
workbook_dict = parse_file_to_workbook_dict(path, file_object)
File "c:\program files\python37\lib\site-packages\pyxform\xls2json.py", line 986, in parse_file_to_workbook_dict
raise PyXFormError("File was not recognized")
pyxform.errors.PyXFormError: File was not recognized

@khalifeserge does the PR #254 address your issue?
Also, does us opening up .xlsm extension make systems that store and make these files available to other users vulnerable to viruses as well as aid in the spreading of viruses? Should we avoid processing these files in general because this increases the security risks to users since they can always save the file with .xlsx when they need to convert it to XForm?
@yanokwa any thoughts on my concerns above?

@ukanga asking the hard maintainer questions 😂
I bias towards allowing xlsm because it does make form design easier for folks. xlrd's readme says they safely ignore macros so I think as far as processing files we are fine. And if such a large library allows xlsm, we can probably do it to.
But systems like Ona that store those files and make them available for download might want to have some virus checking (e.g., like Google does with Gmail).


Migrate LGTM.com installation from OAuth to GitHub App

Hi There,
This project is still using an old implementation of LGTM's automated code review, which has now been disabled. To continue using automated code review, and receive checks on your Pull Requests, please install the GitHub App on this repository.
Thanks,
The LGTM Team

Ok, thank you. Done.


Builder in world Keyboard shortcuts

We need to change the current keyboards shortcuts because they are colliding with the current browser shortcuts
Current shortcuts:
https://docs.google.com/spreadsheets/d/1SAu9YAtGD_Hx6-9VvJ-1JWchFXqq7u_lWt5PvsZ1peY/edit#gid=0
Extra info that can be helpful:
https://stackoverflow.com/questions/11941180/available-keyboard-shortcuts-for-web-applications

@ShibuyaMartin @agusferreira we should be moving this task to the product board


Update mock-data and comments

Thanks @piehld. Looks good. Feel free to merge when the checks are complete.


Fix the flaky test testExtractDatePartWithTimeType()

Description
Flaky test:
ExtractTest > testExtractDatePartWithTimeType() FAILED
    org.opentest4j.AssertionFailedError: expected: <53> but was: <1>
        at app//org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
        at app//org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
        at app//org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177)
        at app//org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:635)
        at app//org.opensearch.sql.expression.datetime.ExtractTest.datePartWithTimeArgQuery(ExtractTest.java:90)
        at app//org.opensearch.sql.expression.datetime.ExtractTest.testExtractDatePartWithTimeType(ExtractTest.java:98)

Caused by function extract(WEEK FROM ...) actually returns the number of the ISO 8601 week-of-week-based-year:

A week is considered to start on a Monday and week 1 is the first week with >3 days. In the ISO week-numbering system, it is possible for early-January dates to be part of the 52nd or 53rd week of the previous year, and for late-December dates to be part of the first week of the next year. For example, 2005-01-02 is part of the 53rd week of year 2004, while 2012-12-31 is part of the first week of 2013.

So the current expected number 53 in the test day 2024-12-30 is incorrect.
The fixing is skipping the testing in December and January.
Related Issues
Resolves https://github.com/opensearch-project/sql/issues/3224
Check List

[x] New functionality includes testing.
[ ] New functionality has been documented.
[ ] New functionality has javadoc added.
[x] New functionality has a user manual doc added.
[ ] API changes companion pull request created.
[x] Commits are signed per the DCO using --signoff.
[x] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

gentle ping @YANG-DB @dai-chen @penghuo @noCharger @ykmr1224 .


add workflow to build and publish docker image

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


Add docs for interfacing with DBus

See https://github.com/AndrewVos/vbar/pull/5#issuecomment-379054273

Closing because we don't use dbus anymore


Increase TCP keepalive interval

The Pathfinder client library uses TCP keepalive to track the "liveness" of the server it is connected to. The interval is set to 1 s, which allows libpaf to detect a dead server or a loss of network connectivity in 3-4 s.
The interval setting should be relaxed, since a 1 s interval causes a lot of traffic on domains with a lot of clients.

Replaced by
https://github.com/Ericsson/paf/issues/26
https://github.com/Ericsson/paf/issues/17


Improvements for eslint (and try switching to prettier)

This PR modifies the eslint config to extend default rules for the base linter and the @typescript-eslint plugin, and then it switch from tsfmt to prettier and integrates prettier into the eslint config. This way, formatting issues now show up as linter issues.
I also fixed the lints reported by the stricter config and finally ran prettier on the output.
I don't know if y'all really want to switch from tsfmt to prettier but I figured I'd give it a try. One reason to consider switching, aside from the eslint integration, is it looks like the tsfmt repository may not be maintained anymore (https://github.com/vvakame/typescript-formatter).

I made some changes to the formatting based on your feedback:

I removed duplicated types from some expressions (e.g., expr<T>(...) where T was inferrable)
I tried to remove most of the return types on arrow functions
I left the return annotations for top-level functions (could disable that rule entirely though)
I tried to remove verbosity around Cmd without resorting to any (but could go back if preferred)

I had to leave the new (class ... as is since it seems prettier really wants to parenthesize class expressions. There may be a way to disable that rule but I couldn't find it.
Same for multi-line operator expressions. I just used String.prototype.concat for that text blurb. I know it's supposed to be slower than + but since it's only for a few lines it should be okay.

We intentionally use build-in VS Code formatter over prettier

Thought which has not occurred to me before: it probably makes sense to just copy-paste config from https://github.com/Microsoft/vscode-languageserver-node/

We intentionally use build-in VS Code formatter over prettier

Could you please elaborate on this, @matklad ?

"built in" in my opinion outweighs any benefits of prettier.

Hmm, I don't use any built-in formatter, I just run npm run fix before the ultimate commit... So from my side, it doesn't matter what's under the hood

If you would rather not merge this PR because you find the use of recommended eslint rules or the prettier formatting rules too onerous, that's okay.
If you would like me to split out some of the other changes made, I can do that too.
Otherwise I'm happy to close this out.

We intentionally use build-in VS Code formatter over prettier

@matklad the built-in formatter seems reasonable but it still looks like tsfmt is no longer actively maintained. The last release was in 2018 and it apparently has compatibility issues with newer typescript configurations (see issues about newer flags). It seems likely that it will eventually need to be replaced or removed from the npm scripts.

Yeah, I just prefer to keep the nodejs side of things as simple as possible, and generally try to stray away from extra tooling. Given the size of the TypeScript codebase, we are not at the point where complex non-built-in tooling makes sense, imo.


no way to enable high-level io_uring object

Unless I'm missing something (very possible), there's no way to enable the high-level io_uring object after using IOURING_SETUP_R_DISABLED. I'd like to implement io_uring_enable(3) unless this capability does already exist. Please let me know if you'd like a different name or anything.
Thanks for the great work on iouring!

io_uring_enable_rings() exists for this, but looks like it hasn't been documented yet.

i'll throw you a man page for it this evening unless you'd prefer to write it!

Please do!

btw, since this function hasn't been a release yet AFAIK, maybe s/rings/ring? unless there's something i'm missing.

I think you could argue both ways on ring vs rings - there are two rings there, SQ and CQ. Both are effectively disabled, even though just having the SQ ring disabled obviously means that the CQ ring is too.

btw, since this function hasn't been in a release yet AFAIK, maybe s/rings/ring? unless there's something i'm missing.

just fyi it's been there a long time - so can't update it now. since at least 2.0: https://github.com/axboe/liburing/blob/liburing-2.0/src/include/liburing.h#L144


Bug: Safari, Video Player does not work

``Flutter 3.3.10```
Working on other browsers Chrome, Firefox, Edge etc.
On Safari we get the following effect:
https://user-images.githubusercontent.com/6940726/209938894-391d0292-56e3-48b8-9ca8-6a16f865bb60.mov

Fix was just pushed out might, be related to new bug: https://github.com/flutter/engine/pull/35948

@ollydixon Hello. Can you try with the master channel of Flutter and see if it reproduces?

@exaby73 sure, I will soon. I managed to narrow the bug down. It's when you have two clip-rects up the widget tree.

If it still reproduces in master, please provide a complete and minimal, reproducible example so that I may verify it on my end :)

@exaby73 yes just writing a small app to replicate it, strange thing is I'm having trouble replicating it. Something far up the widget tree in my app is causing the issue. Just isolating it. :-)

@exaby73 yes it's on my list to try to debug again, might take a bit of time :-)

@ollyde Hi, I'm facing the similar issue with videos in Safari. Have you found any workaround?

@MaksGinger fixed in the latest hotfix

@MaksGinger I had to get rid of rounding on some boxes only for Safari, I couldn't find the issue specially.

@ollyde I checked this on video_player sample code but didn't see the issue on the Safari. Verified on video_player: ^2.5.2.

Demo
https://user-images.githubusercontent.com/104349824/222108180-9732b3a1-1bbb-4431-a58d-833a2d5c1258.mov

What is your plugin version? Please provide a completed and minimal reproducible code sample so that we may verify this. Also, please provide the output of flutter doctor -v as well.
Thank you!

Hello, @huycozy! I've created a reproducible code https://gist.github.com/MaksGinger/76a0ca16d8f30f17aa4bbb3d52ab0bda
I have video_player: ^2.5.2.
This is my flutter doctor -v output:
`flutter doctor -v
[✓] Flutter (Channel stable, 3.7.5, on macOS 13.2.1 22D68 darwin-arm64, locale
en-RU)
• Flutter version 3.7.5 on channel stable at
/Users/maksimilianyakovlev/Development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision c07f788888 (6 days ago), 2023-02-22 17:52:33 -0600
• Engine revision 0f359063c4
• Dart version 2.19.2
• DevTools version 2.20.1
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/maksimilianyakovlev/Library/Android/sdk
• Platform android-33, build-tools 31.0.0
• Java binary at: /Applications/Android
Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14B47b
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
11.0.13+0-b1751.21-8125866)
[✓] IntelliJ IDEA Community Edition (version 2021.2.2)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 63.3.3
• Dart plugin version 212.5744
[✓] VS Code (version 1.75.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.42.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-arm64   • macOS 13.2.1 22D68
darwin-arm64
• Chrome (web)    • chrome • web-javascript • Google Chrome 110.0.5481.177
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!`
https://user-images.githubusercontent.com/32202184/222132528-9300a192-e56a-4728-b6ab-acf1e40d7d6c.mov

Thank @MaksGinger for providing the sample code. But as far as I can see, the issue is caused by ClipRRect and when I going further with it, it is related to canvaskit renderer as it doesn't reproducible with html renderer.

Demo video (no ClipRRect widget, canvaskit renderer ✅)
https://user-images.githubusercontent.com/104349824/223025252-92aa372c-bdac-48a6-9a0a-5314046d14cc.mov

Demo (with ClipRRect, html renderer ✅)
https://user-images.githubusercontent.com/104349824/223025366-c5f12d6e-1146-41f8-905e-b1b119bf16e2.mov

✅: No Issue ❌: Issue reproduced
And it seems to be hitting https://github.com/flutter/flutter/issues/79550. Please try on html render and check the linked issues.

@huycozy html render is not an option on Safari, it breaks loads of other features (like even icons don't work)

@ollyde I know building on a specific render is not a solution, but I just to check and find the closest related issue (if any) to this issue. Can you please confirm the result from the preliminary investigation above?

@huycozy switching to HTML rendering doesn’t address this issue; of it works (I tried before posting this ticket)

@huycozy , Yes, you're right html rendering works, but it would be nice to have the ability to use ClipRRect with video player on Safari using canvaskit. So waiting patiently for this https://github.com/flutter/flutter/issues/79550 to be solved

There was a @MaksGinger a push 2 hot fixes ago to fix it. But this is a rare edge case with scrolling. I'm scrolling with ClipRRect on Safari and it works on 99% of the platform, just one screen !

@ollyde I'm not sure I can understand your above comment. Can you be more specific on it and provide a recorded video for the issue (to better visualize the problem)? (It should be reproducible on a minimal sample as @MaksGinger provided above)

@huycozy MaksGinger has provided an sample.

@MaksGinger confirmed the HTML renderer worked as mentioned at https://github.com/flutter/flutter/issues/117777#issuecomment-1456117843. Once you believe the HTML renderer doesn't work for your case, please provide more information so that we may verify the issue.
To be more precise, please provide the information indicated below:

The output of flutter doctor -v (please retry on the latest stable and master channels)
Safari browser version
A recorded video of the issue would be appreciated

@huycozy I'm not sure what you're going on about, this bug has nothing to do with the HTML renderer which isn't viable for Safari websites created in Flutter.

@ollyde What I'm trying to do is isolate the issue. As I mentioned above at https://github.com/flutter/flutter/issues/117777?notification_referrer_id=NT_kwDOBjhAgLQ1MjAxMDE0NjkyOjEwNDM0OTgyNA#issuecomment-1455476469, this issue is caused by canvaskit renderer. Also, confirming this can be worked on HTML to make sure this issue is related to (or duplicate of) https://github.com/flutter/flutter/issues/79550.
For more detail, please see how video_player is built in Web: https://github.com/flutter/packages/blob/main/packages/video_player/video_player_web/lib/video_player_web.dart
Hope this will be clearer.


Support .secret.local

Originally posted by @ciriousjoker in https://github.com/simondotm/nx-firebase/issues/37#issuecomment-1151347554

Does anybody use .secret.local ?
If so, please let me know your use cases, so I can figure out how best to integrate support for it in the plugin

We use Gcp's secret manager to store secrets and load them at runtime. During development, we want to run the cloud functions in an emulator, so we use the .secret.local file to provide the secrets during development. These local secrets sometimes grant access to a sandbox, sometimes to a production environment. Every dev who needs access fills this file with the secrets they need/want, so we don't commit it.
Right now our workaround is to cp the file during the build pipeline.

Yes, the file is copied into the dist folder, like this:
{
  "command": "npx cpy ./apps/<project>/.secret.local ./dist/apps/<project> --flat"
},

Ok I think I have a solution for this.
I'm going to add an environment folder to firebase applications, which can contain common files such as:

.env
.env.<project>
.env.local and
.secret.local (will be git ignored)

Function project.json configs will then be updated to have a glob in the assets ruleset, to always copy any files from this folder to dist for every function. This way every deployed or emulated function has the same common env vars.
        "assets": [
          "apps/functions/test/src/assets",
          { "glob": "**/*", "input": "apps/firebase/environment", "output": "."}
        ],

Nx appears to not use .gitignore or .nxignore rules for globs, so this seems to work well.
It may mean that .secret.local gets deployed to cloud functions, but since that is only consumed by the emulator I dont think it will have any effect on deployed runtimes and will be just ignored in the cloud.

@ciriousjoker - do you take any steps to prevent .secret.local being deployed, since it is in your dist folder?

Thinking we can add .secret.local to functions.ignore in the firebase.json if its an issue

Fixed in PR #137


Pygoat on Flask fwk

Hello,
When i can expect pygoat on Flask framework ?. In addition can we have write-up vulnerabilities ?
@adeyosemanputra @gaurav618618

We are currently very short on contributors will definitely think about that when we have good amount of contributors for sure. Me and @adeyosemanputra are also currently busy with other projects so sorry we cant tell you the exact time period .

will working and starting now @dugdug36 @gaurav618618

we done with release V.1.3


Add support for MSSQL's OPENJSON WITH clause

Resolves issue #1494

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

264 of 303 (87.13%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 89.376%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/query.rs
20
22
90.91%

src/parser/mod.rs
38
41
92.68%

tests/sqlparser_mssql.rs
206
240
85.83%

Files with Coverage Reduction
New Missed Lines
%

src/ast/query.rs
1
86.43%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.02%

Covered Lines:
30876

Relevant Lines:
34546

💛  - Coveralls

🚀


There was an error while loading the crate

Spark link
No response
Plugin Version
2.1.2
Server Version
1.20.4
Server Platform
Paper
Describe the bug
here are the logs
https://mclo.gs/Tx6JB1z#L542
Crate Type
None
Steps to reproduce

start the server
error
plugin disabled
can't run the plugin, obviously

Additional context
No response
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[ ] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by CrazyCrew.

Nova.yml has a null ItemStack, send your Nova.yml

[23:01:25 INFO]: Broken Crates: Monthly.yml,Nova.yml,Solar.yml<dark_gray
solar: https://mclo.gs/OOtsm8e
nova: https://mclo.gs/ClF3573
monthly: https://mclo.gs/KHGUWCx

Purchase things from the actual source on BuiltByBit instead of downloading leaked files from sites like nullforums.
I won't fix config issues with leaked setups. The issue is invalid materials used but that's all I am going to say.


Names of technologies

There is a quite big list of technologies involved in the nomenclature, and there may be some unconsistancies coming from the fact that technologies names come from different sources and horizons.
The objective of this pull request is to define the list of technologies to be used now (knowing that it can evolve in the future), in particular avoid duplication of technologies names.
I also remember a discussion that I cannot find any more aiming at simplifying the yaml files which are becoming very big with lots of variables 'duplicated' per technology. The idea was that instead of having a huge list eg
Capacity|Electricity|techno1 ...., to use Capacity|Electricity| (eg  ,  . Then we would need to maintain lists of technologies for each category.
Would like to have your opinions @erikfilias @danielhuppmann @tburandt  and all others of course
I have no idea if this would be compatible with the tools for checking the nomenclature...
List of technos existing now in the nomenclature for power system is:
1/ thermal power
Nuclear
Biomass
Biomass|w/ CCS
Biomass|w/o CCS
Biomass|Traditional    (from ehighway)
Biomass|New and imported   (from ehighway)
(note that there exist 2 potential segmentations for biomass as namings come from different scenarios)
Coal
Coal|w/ CCS
Coal|w/o CCS
Lignite (from ehighway)
Fossil
Gas
Gas|w/ CCS
Gas|w/o CCS
Gas|Fossil
Gas|Fossil|w/ CCS
Gas|Fossil|w/o CCS
Gas|Biomethane
Gas|Biomethane|w/ CCS
Gas|Biomethane|w/o CCS
Gas|Synthetic Methane
Gas|Synthetic Methane|w/ CCS
Gas|Synthetic Methane|w/o CCS
Oil
Oil|w/ CCS
Oil|w/o CCS
3/ variable renewable
Solar
Solar|CSP
Solar|PV
Wind
Wind|Offshore
Wind|Onshore
Hydro|Run of River
4/ I dont know how to classify
Hydrogen
Geothermal
Non-Biomass Renewables
Ocean
Other
5/ storage
5-1 short term storage
Hydro|Pumped Storage
Energy Storage System
5-2 seasonal storage
Demand response|Load shifting
Demand response|Load curtailement
Hydro|Reservoir
The idea of the classification is to link it with the list of needed variables....
Would appreciate your feedback

this is a great idea, and it is related to a proposal I made earlier (#29) - I just need a few free days to implement this...

Thanks @danielhuppmann , I remembered this but could not find it, sorry for opening yet a new issue!

@HauHe In which file did you see this? In technology.yaml, eg you can find Capital Cost|Electricity|, and the list of  is here https://github.com/openENTRANCE/nomenclature/blob/master/nomenclature/definitions/variable/fuel_types.yaml
you can also find in the same directory list of ,  and
Seems to me this was completed but I may be wrong....
(and we still can add new categories to "Fuel" and others......
Best!

@sandrinecharousset, thanks for your fast response and sorry for my slightly inprecise formulation.
The description of the Capital Cost that I posted above comes from here: https://github.com/openENTRANCE/nomenclature/blob/master/nomenclature/definitions/variable/technology/README.md
What I'm looking for is a list of terms to use for the placeholder <Specification> not the <Fuel>.
I see that in the list you shared there are some technologies like Gas|OCGTdefined, but as the name of the list says it is rather fuel types than technology types. Therefore, I thought that this issue is about creating a yaml file perhaps called technology_types similiar to the fuel_types.
Best,
Hauke


dont rely on existence of short version of the macros, define the lon…

…g with API calls instead
relates to https://github.com/ros/ros_comm/pull/1235

PR job didnt get triggered but I have faith, merging

:pray:


How to use a custom initial model?

I tried replacing the "MODEL_NAME" in the "Settings and run" section with the path to my own chkpt file on google drive, but it doesn't seem to work. I can't seem to get it to work with anything other than the default "CompVis/stable-diffusion-v1-4"
Did I misunderstand or do something wrong?

The ckpt needs to converted to diffusers version using this script first.
https://github.com/ShivamShrirao/diffusers/blob/main/scripts/convert_original_stable_diffusion_to_diffusers.py


Adding support for -> centered content <- like markdown language

-> must be at the start of each line.
trailing spaces after <- are ignored.
embedded md are supported like -> centered _underline_ here <- or ->#centered level 1 heading here<-.

Sorry, but we can't accept this PR, it conflicts with recent changes done to Hoedown.
Even then, development of version 3 has stalled, only bugfixes are accepted.


ansible-lint

This role has some ansible-lint issues:
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/handlers/main.yml:7
  command: systemctl daemon-reload

ANSIBLE0002 Trailing whitespace
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/meta/main.yml:25

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/kernel_check_and
Task/Handler: Install Xorg packages for backported kernels (very optional)

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/kernel_check_and
Task/Handler: Reboot instance

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/kernel_check_and
Task/Handler: Wait for instance to come online (10 minute timeout)

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:56
Task/Handler: Run dmsetup for Ubuntu 16.04

ANSIBLE0006 curl used in place of get_url or uri module
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:68
Task/Handler: Alternative | Add Docker repository key

ANSIBLE0012 Commands should not change things if nothing needs doing
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:77
Task/Handler: Return dpkg architecture

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:157
Task/Handler: meta flush_handlers

ANSIBLE0011 All tasks should be named
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:160
Task/Handler: pause seconds=1 __file__=/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml __line__=160

ANSIBLE0016 Tasks that run when changed should likely be handlers
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:160
Task/Handler: pause seconds=1 __file__=/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml __line__=160

ANSIBLE0010 Package installs should not use latest
/home/hurricanehrndz/gitrepos/ansible-roles/docker.ubuntu/tasks/main.yml:165
Task/Handler: Install {{ _python_packages | join(', ') }} packages with apt

I think these need to be addressed.

I'm don't think any of these linter remarks need to be addressed, except for the whitespace issue and that is not critical.


Release/2.0.4 release

Coverage increased (+0.08%) to 70.134% when pulling aaf36e4d14498469680b31b612d6e4603cab9cd3 on release/2.0.4-release into c634ae2189a651e3293752fd60609a237714e153 on master.

Coverage increased (+0.08%) to 70.134% when pulling aaf36e4d14498469680b31b612d6e4603cab9cd3 on release/2.0.4-release into c634ae2189a651e3293752fd60609a237714e153 on master.


[Bug] Route trip creation fail

403 indicates that your tokens expired. This is probably related to the recent changes we made to auth0 requiring people to sign in again.

I got try that. All other modules of admin view was ok when i was creating
route last night
On Fri, 27 Oct 2017 at 7:08 AM, LoneRifle<EMAIL_ADDRESS>wrote:

403 indicates that your tokens expired. This is probably related to the
recent changes we made to auth0 requiring people to sign in again.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/333#issuecomment-339826069,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU35tbcQRBShXqLXSJxiRvO-hOYYNHks5swRDQgaJpZM4QH2nR
.

I'll probably need you to demonstrate how to reproduce the bug then..

I am on leave today, but what u can do is, go to C208 and edit that trip
inside. This bug will consistently happening. I try a few times last night
On Fri, 27 Oct 2017 at 7:25 AM, LoneRifle<EMAIL_ADDRESS>wrote:

I'll probably need you to demonstrate how to reproduce the bug then..
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/333#issuecomment-339829004,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU3ws5GEuPs5lW6IGg86NyygydtE2Eks5swRUEgaJpZM4QH2nR
.

Did you mean C201? There is no route corresponding to C208.

Sry, try c228 instead
On Fri, 27 Oct 2017 at 7:34 AM, LoneRifle<EMAIL_ADDRESS>wrote:

Did you mean C201? There is no route corresponding to C208.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/333#issuecomment-339830402,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU3wiG8LwihO5n3FVKsvq6A1WioU5Hks5swRcMgaJpZM4QH2nR
.

Unable to reproduce, trips created without issue

Might be to do with auth0 stability issues that are currently ongoing, and as reported on our monitoring tool


[type:fix]update openGauss sql for tcp plugin

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

#4857

Codecov Report

Merging #4880 (043a0be) into master (5a635ca) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4880      +/-   ##
============================================
- Coverage     64.51%   64.46%   -0.06%
+ Complexity     8251     8243       -8
============================================
  Files          1166     1166
  Lines         34232    34232
  Branches       3067     3067
============================================
- Hits          22085    22066      -19
- Misses        10389    10409      +20
+ Partials       1758     1757       -1

see 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update ODK 2 to ODK-X in Sync-endpoint login screen

This is on https://github.com/odk-x/sync-endpoint-web-ui

@maprehensive @linl33
I've made the changes.
Can I move forward with a PR?

Please send a pull request to odk-x/sync-endpoint-web-ui. Thanks.


Documentation about grpc polling engines

Converted a recent presentation I gave into markdown.
The document talks about gRPC polling engine interface, implementations and gives an overview (in reasonable detail) about the two most commonly used polling engines (grpc epoll1 and grpc epollex)

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,588      Total (=)      1,964,588

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,806      Total (=)     10,886,806

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,588      Total (=)      1,964,588

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,807      Total (>)     10,886,803

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,588      Total (=)      1,964,588

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,801      Total (<)     10,886,802

 No significant differences in binary sizes

The windows iomgr implementation is based off io completion ports, also named iocp.

Great docs Sree, thanks for that!

Thanks for doing this, Sree!
Minor nit: Suggest moving these to the doc/core subdirectory.

Thanks @jtattermusch and @markdroth . I moved the docs to under 'doc/core' directory.
@vjpai - I made the changes you suggested. PTAL

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,588      Total (=)      1,964,588

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,952      Total (>)     10,886,946

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,588      Total (=)      1,964,588

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,947      Total (<)     10,886,952

 No significant differences in binary sizes

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,600      Total (=)      1,964,600

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,886,954      Total (<)     10,886,956

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


The 'search' property should accept 'null' or 'undefined' value

When there is nothing to search, the 'search' property of the component should accept null or undefined value to signify that nothing should be searched and the component should return the child 'as is'.
Currently, React throws
Warning: Failed prop type: The prop `search` is marked as required in `Highlighter`, but its value is `null`.
warning.js:33
    in Highlighter

when null is passed to the 'search' prop.

I think you can pass false if there is nothing to search.
<Highlight search={highlightRegexp || false}>{text}</Highlight>

From looking at the code, you can indeed pass false or even '' and it would act the same as not having received the search prop
    search: PropTypes.oneOfType([
      PropTypes.string,
      PropTypes.number,
      PropTypes.bool,
      RegExpPropType
    ]).isRequired,

  hasSearch: function() {
    return (typeof this.props.search !== 'undefined') && this.props.search;
  },

However, the undefined case is explicitly being supported in the hasSearch function, but then made fail by propTypes.
Moreover (typeof this.props.search !== 'undefined') && this.props.search is the same as this.props.search for the purposes of this code.
Removing the isRequired from the propTypes would spare me from having to transform
{...props} into {...props} search={props.search || false}
Actually the very existence of the function hasSearch opposes the usage of isRequired


Splitscreen window only works on primary monitor.

I'm having a small issue with the mod.
In my setup, I have my TV hooked up to my computer as a secondary monitor. This way, I usually start the game on my PC, then either drag it to the TV or it launches directly on there. The reason I simply do not set the TV as duplicate screen (which would work as an workaround for this issue), is because it has a higher resolution then my PC monitor does.
But I can't seem to get the splitscreen window on another monitor that isn't the primary one.
How to reproduce:

Launch minecraft with the mod installed
Drag the minecraft window to the desired monitor (not the primary)
Press F11

Expected behavior:

Minecraft resizes to splitscreen format on the desired monitor

Actual behavior:

Minecraft resizes to splitscreen format on the primary monitor

Set your TV as the primary monitor. You can then turn off the built-in display (from within your OS) and then Windows (or any other OS) will allow you to use the higher resolution.


put funcation result as a name

.e.g Complex-read12
result: SUM(loan.loanAmount) -> sumLoanAmount

And re-check the results from divided list to tuples

And re-check the results from divided list to tuples

Leave it to solve in the alpha test
https://github.com/ldbc/ldbc_finbench_docs/issues/50


🦟[Bug Report] v2.8.4 violating the 2% crash rate

The Play Store has an alarm showing crashes with the v2.8.4 of Litewallet.
Ask
Can we confirm this is closed per the beta release v2.9.0?
Can we explain what the actual issue is?

Crash Report

Information only


Documentation - Using GitHub App with Action

Description
We need some introductional docs on how to setup and use this Action using a GitHub App.
Related:

#50

#51

+cc @4bg0P
Propose Solution

[ ] New Document
[ ] Reasoning for using GitHub App versus a PAT
[ ] Full example workflow

Using peter-murray/workflow-application-token-action?

Hi @GeekMasher, where within the 'docs' folder are you expecting this to live? I managed to get authentication with a GitHub app working recently using V1.6.3 of your action 🎊

@angusjellis Just seen this comment and PR ❤ Let me review it today or next week and hopefully get it merged soon

To be honest, maybe you can also just embed this video from a coworker of yours: https://youtu.be/1MhmeuaaqRY?t=496
This is how I got into this repository.


test: behaviors-message_on_lock_failure (2024-02-21 22:15:00.880865+00:00)

automatic test cleanup


Summary
Shiny is currently not linker friendly
API Changes
None
Intended Use Case
Linker is nice for shrinking app size
Notes
Related to #160
In hanselman.forms I am trying to turn Link All on.
Getting: System.InvalidOperationException: 'A suitable constructor for type 'Shiny.Jobs.JobManager' could not be located. Ensure the type is concrete and services are registered for all parameters of a public constructor.'
Had to add Shiny.Core to the list to skip in linking. Could probably throw a preserve on there to make sure it works properly.
Additionally, I had to add a [Preserve(AllMembers = true)] to my IJob implementation

Currently blocked by #190

Hey @aritchie.
First of all, thank you for this great library. Are you planning to support linking for Shiny in the near future? Currently we have 2 apps with where we have just Shiny.Core via Shiny.Prism package integrated. We are unable to link both the apps on Android even if we add Shiny.Core to the list to skip in linking process. Because of this our android apps with not so many features inside have size more >80MB. We also tried to add Shiny.Prism and Shiny.Core to the list, nothing helps. Are we missing something in the configuration oder someting has to be done in Shiny.Core? Thanks in advance for any information.
Michal

@michalhoblik this is not a priority at all for me right now.


Cookbook installation failing at Recipe: kafka::_configure. [ undefined method `root_logger' for #Mash:0x00000004fc62b0 ]

Hi,
I am using v2.2.0 [ from Supermarket], it was working good since last week. But, all of sudden, getting the below issue. And cookbook is failing to install.
kafka cookbook version: v2.2.0
chef version: 13.0.118
_> Below is the log:

Recipe: kafka::_configure

directory[/opt/kafka/config] action create (up to date)
template[/opt/kafka/config/log4j.properties] action create =======================================================================
Error executing action create on resource 'template[/opt/kafka/config/log4j.properties]' =======================================================================
Chef::Mixin::Template::TemplateError

undefined method `root_logger' for #Mash:0x00000004fc62b0_

Thanks,
Balasekhar

Alright, so this is because of Chef 13 and would explain why it worked perfectly fine last week but now not so much. Forgot that Chef 13 was slated for this week (and not next), and I'm currently looking into backporting support for Chef 13.
Should hopefully have a release out later today.

Release got delayed a bit, but there's a 2.2.2 release on Supermarket that should (hopefully) work with Chef 13 (don't know if there are any corner cases...).
If you run into further issues feel free to re-open this issue.

Hi @mthssdrbrg,
Thanks for the prompt response.
We are facing different issue now. Kafka service failed to restart.
Error executing action start on resource 'service[kafka]'
_> Below is the log:

===========================================
Error executing action start on resource 'service[kafka]'
       Mixlib::ShellOut::ShellCommandFailed
       ------------------------------------
       Expected process to exit with [0], but received '1'
       ---- Begin output of /etc/init.d/kafka start ----
       STDOUT: * Starting kafka
   ...done.
       STDERR:
       ---- End output of /etc/init.d/kafka start ----
       Ran /etc/init.d/kafka start returned 1_

Thanks,
Bala

Hm, well, that could be for a myriad of reasons. Do you get any logs? What's your configuration like?

Hi @mthssdrbrg ,
Kafka cookbook with v2.2.0 with chef 12.19.36 worked well till yesterday. It's failing when I tried to converge [ using Test Kitchen CI ].
Below is the log:
         * service[kafka] action start[2017-04-18T09:21:05+00:00] INFO: Processing service[kafka] action start (kafka::_coordinate line 14)

           ================================================================================
           Error executing action `start` on resource 'service[kafka]'
           ================================================================================

           Mixlib::ShellOut::ShellCommandFailed
           ------------------------------------
           Expected process to exit with [0], but received '1'
           ---- Begin output of /etc/init.d/kafka start ----
           STDOUT: * Starting kafka
       ...done.
           STDERR:
           ---- End output of /etc/init.d/kafka start ----
           Ran /etc/init.d/kafka start returned 1

           Resource Declaration:
           ---------------------
           # In /tmp/kitchen/cache/cookbooks/kafka/recipes/_coordinate.rb

            14: service 'kafka' do
            15:   provider kafka_init_opts[:provider]
            16:   supports start: true, stop: true, restart: true, status: true
            17:   action kafka_service_actions
            18: end

           Compiled Resource:
           ------------------
           # Declared in /tmp/kitchen/cache/cookbooks/kafka/recipes/_coordinate.rb:14:in `from_file'

           service("kafka") do
             action [:enable, :start]
             supports {:start=>true, :stop=>true, :restart=>true, :status=>true}
             retries 0
             retry_delay 2
             default_guard_interpreter :default
             service_name "kafka"
             enabled true
             running nil
             masked nil
             pattern "kafka"
             declared_type :service
             cookbook_name "kafka"
             recipe_name "_coordinate"
             kafka_init_opts {:env_path=>"/etc/default/kafka", :source=>"sysv/debian.erb", :script_path=>"/etc/init.d/kafka", :permissions=>"755"}
           end

           Platform:
           ---------
           x86_64-linux

Thanks,
Bala

I'm unfortunately unable to reproduce this locally. Though it should be said that if you're just using the .kitchen.yml in this repository the Chef run converges successfully and it (attempts) to start Kafka, but the process will actually die after a timeout as it attempts to talk to a non-existing ZooKeeper.
Do you get any logs in /var/log/kafka? What's your configuration like?

A log file got created in "/var/log/kafka" with name kafka-gc.log.
Log is as below:
Java HotSpot(TM) 64-Bit Server VM (25.121-b13) for linux-amd64 JRE (1.8.0_121-b13), built on Dec 12 2016 16:36:53 by "java_re" with gcc 4.3.0 20080428 (Red Hat 4.3.0-8)
Memory: 4k page, physical 1016284k(100020k free), swap 0k(0k free)
CommandLine flags: -XX:+CMSClassUnloadingEnabled -XX:+CMSScavengeBeforeRemark -XX:+DisableExplicitGC -XX:InitialHeapSize=1073741824 -XX:+ManagementServer -XX:MaxHeapSize=1073741824 -XX:MaxNewSize=87244800 -XX:MaxTenuringThreshold=6 -XX:NewSize=87244800 -XX:OldPLABSize=16 -XX:OldSize=174489600 -XX:+PrintGC -XX:+PrintGCDateStamps -XX:+PrintGCTimeStamps -XX:+UseCompressedClassPointers -XX:+UseCompressedOops -XX:+UseConcMarkSweepGC -XX:+UseParNewGC
I am converging in AWS on t2.micro instance.

It is working with m4.large. I guess it is the issue with configuration.

Aha, then I suspect it's because the default heap size is set to 1G, and it (t2.micro) has exactly that much RAM, which mostly will have the JVM fail to start properly.


[Question]: About spawnThread's thread name

Hello @mrousavy,
Thanks for this library
To my understanding, spawnThread runs function in // custom thread as you mentioned.
My Question:

What is the name of the thread spawnThread is using?

if named, does it have a dedicated thread?

else does it create new thread for each functions?

The name of a thread is only a concept by Xcode/Android Studio. The Thread does not have a name, it's a normal C++ thread. It's a pool of 1 (for now), so the parallel thread is re-used.

Okay Thanks for your answer.


Update Twitter/X sharing meta tags

closes #1610
Per the docs:

twitter:url is not a thing
we should be using name instead of property for the tag keys

Thank you @fturmel!


Add CMR and access metrics

This PR adds a notebook for generating s3 access metrics + plots using Athena and other notebooks for generating metrics for CMR queries.

I think after we merge this, we should immediately remove the CMR queries since MAAP CMR is going away. But I'm going to share those with the CMR team.

Thanks for reviewing @wildintellect - just noting that I plan to address these comments late this week or next week


Common: add Ruby script to generate unified config.yml and iglu-resolver.json from runner.yml and loader.yml

Script should have zero dependencies.

Example of how this script operates:
storage_loader_config.yml:
:aws:
  :access_key_id: ADD HERE
  :secret_access_key: ADD HERE
:s3:
  :region: ADD HERE # S3 bucket region
  :buckets:
    :jsonpath_assets: # If you have defined your own JSON Schemas, add the s3:// path to your own JSON Path files in your own bucket here
    :enriched:
      :good: ADD HERE # Must be s3:// not s3n:// for Redshift. This is the same as the :enriched:good: bucket specified for EmrEtlRunner
      :archive: ADD HERE # Where to archive enriched events to
    :shredded:
      :good: ADD HERE # Must be s3:// not s3n:// for Redshift. This is the same as the :shredded:good: bucket specified for EmrEtlRunner
      :archive: ADD HERE # Where to archive shredded types to
:download:
  :folder: # Not required for Redshift
:targets:
  - :name: "My Redshift database"
    :type: redshift
    :host: ADD HERE # The endpoint as shown in the Redshift console
    :database: ADD HERE # Name of database
    :port: 5439 # Default Redshift port
    :table: atomic.events
    :username: ADD HERE
    :password: ADD HERE
    :maxerror: 1 # Stop loading on first error, or increase to permit more load errors
    :comprows: 200000 # Default for a 1 XL node cluster. Not used unless --include compupdate specified

eer_config.yml:
:logging:
  :level: DEBUG # You can optionally switch to INFO for production
:aws:
  :access_key_id: ADD HERE
  :secret_access_key: ADD HERE
:s3:
  :region: ADD HERE
  :buckets:
    :assets: s3://snowplow-hosted-assets # DO NOT CHANGE unless you are hosting the jarfiles etc yourself in your own bucket
    :log: ADD HERE
    :raw:
      :in: ADD HERE
      :processing: ADD HERE
      :archive: ADD HERE    # e.g. s3://my-archive-bucket/raw
    :enriched:
      :good: ADD HERE       # e.g. s3://my-out-bucket/enriched/good
      :bad: ADD HERE        # e.g. s3://my-out-bucket/enriched/bad
      :errors: ADD HERE     # Leave blank unless :continue_on_unexpected_error: set to true below
    :shredded:
      :good: ADD HERE       # e.g. s3://my-out-bucket/shredded/good
      :bad: ADD HERE        # e.g. s3://my-out-bucket/shredded/bad
      :errors: ADD HERE     # Leave blank unless :continue_on_unexpected_error: set to true below
:emr:
  :ami_version: 3.6.0      # Don't change this
  :region: ADD HERE        # Always set this
  :jobflow_role: EMR_EC2_DefaultRole # Created using $ aws emr create-default-roles
  :service_role: EMR_DefaultRole     # Created using $ aws emr create-default-roles
  :placement: ADD HERE     # Set this if not running in VPC. Leave blank otherwise
  :ec2_subnet_id: ADD HERE # Set this if running in VPC. Leave blank otherwise
  :ec2_key_name: ADD HERE
  :bootstrap: []           # Set this to specify custom boostrap actions. Leave empty otherwise
  :software:
    :hbase:                # To launch on cluster, provide version, "0.92.0", keep quotes
    :lingual:              # To launch on cluster, provide version, "1.1", keep quotes
  # Adjust your Hadoop cluster below
  :jobflow:
    :master_instance_type: m1.small
    :core_instance_count: 2
    :core_instance_type: m1.small
    :task_instance_count: 0 # Increase to use spot instances
    :task_instance_type: m1.small
    :task_instance_bid: 0.015 # In USD.  Great work


perf(*): use cycle aware deep clone everywhere

Summary
This is continuation of https://github.com/Kong/kong/pull/10970. It tries to reuse caches on declarative whenever possible. It does not clone the keys anymore by default.
This time we use cycle aware deep cloning more aggressively across the codebase.
In my testing this reduces memory usage of workers considerably, e.g. in dbless/dataplanes
I see following numbers:
Before #10970: 79 MB
After #10970: 70 MB
After (this PR): 60 MB
About 24% decrease (may be even bigger in EE).
When I start Kong with:
KONG_DATABASE=off \
KONG_ROLE=data_plane \
KONG_CLUSTER_CONTROL_PLANE=cp.test:8005 \
KONG_CLUSTER_CERT=cluster.crt \
KONG_CLUSTER_CERT_KEY=cluster.key \
kong restart -p dp

This means that if we decide to include https://github.com/Kong/kong/pull/10798 the privileged worker's memory usage with all plugins is now 30 MB.

This PR should perhaps be reduced to just declarative_config as there is the most effect, and we should check that we place this sharing of cycle cache only on safe places, where it has the most effect.

Superseded by https://github.com/Kong/kong/pull/10980, so closing this.