Internal set up of ParallelStencil

This PR introduces environment!(:gpu, Float64, 2) to dynamically set up ParallelStencil inside the Diffusion modules and replaces @init_parallel_stencil. The first argument should be :gpu or :cpu. There is no longer the need to load GeoParams and/or ParallelStencil after loading MTK.jl. E.g. in Example2D_paper:
using MagmaThermoKinematics
environment!(:gpu, Float64, 2)   # initialize parallel stencil in 2D
using MagmaThermoKinematics.Diffusion2D # to load AFTER calling environment!()

using CairoMakie    # plotting
using Printf        # print
using MAT, JLD2     # saves files in matlab format & JLD2 (hdf5) format
using Parameters
using Statistics
using LinearAlgebra: norm

thanks a lot - this is great!


Fill a usability gap in iris.analysis.Trajectory

When I used iris.analysis.Trajectory I was a bit surprised that it did not have an interpolate method. It seemed to me the obvious use of Trajectory was to set up points for the trajectory and then interpolate a cube along the trajectory, all in one class. Instead I found that not only could I not interpolate from the Trajectory, but the Trajectory's points were not even in the same format as the trajectory module's interpolator function.
Given that this could easily be improved, I went out and did.  I've added Trajectory.interpolate that calls the trajectory module's interpolate function and a private helper method to Trajectory that translates the trajectory points to the format expected by the interpolate function. The interpolate method also adds a DimCoord to name the anonymous dimension that's otherwise created by the trajectory interpolation.

Ping @corinnebosley

@dkillick You got a licence header to fix, and it looks like there might be a problem with _get_interp_points.  Not sure yet whether it's the function or the test which is squiffy, but it's one or the other.

@corinnebosley what's odd is that the get_interp_points error you mention is only happening for py35... It looks like an ordering issue and there's no way i'm going to try and fix that down as you just end up chasing your tail 😒 I'll see what I can do.

Note: in case you hadn't seen it,
the error in your Python 3.5 "minimal" test is explained here ...

What's new and a rebase, and I think you're good to go.
I've not put this in the v2.0.0 milestone as it doesn't hugely matter if it gets in for a major release number - we could add this feature in v2.0.1 if we had to...

An alarming segfault in https://travis-ci.org/SciTools/iris/jobs/330392251 😱. 🤞 these are bad dependency builds, and re-running solves.
Trivial license header issues.

Rebase; license header issues tackled.

Another small step forward!

Thanks @pp-mo 🎉


⚠ SkyfallMC Minecraft Server has degraded performance

In 1193a6f, SkyfallMC Minecraft Server (play.skyfallmc.ga) experienced degraded performance:

HTTP code: 200
Response time: 1979 ms

Resolved: SkyfallMC Minecraft Server performance has improved in d455665.


Does CloudAPAuthEnabled for Chrome 111 replace the requirement for Windows Accounts?

It seems like the CloudAPAuthEnabled feature of Chrome 111 can be used instead of deploying the Windows Accounts extension. Can you confirm and if confirmed by product group update the documentation and guidance?
More info here: https://chromeenterprise.google/policies/?policy=CloudAPAuthEnabled

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

ID: 625bb1b2-81fd-9804-c0b5-3cd2904decd9
Version Independent ID: e4468b05-200e-24ae-75e9-29fb2b3a6ddc
Content: Conditions in Conditional Access policy - Microsoft Entra
Content Source: articles/active-directory/conditional-access/concept-conditional-access-conditions.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@JonesMikael
Thanks for your feedback! We will investigate and update as appropriate.

#label:"awaiting-product-team-response"

Any feedback from the product team yet on this?

FWIW, the CloudAPAuthEnabled feature is working in my tenant for about 95% of Chrome 113.x (I don't have anybody on earlier Chrome versions to test) users. I see Managed and Compliant fields filled under Device Info for requests coming from Chrome browsers with this policy enabled. There are a few outliers where it is not working yet, but that seems to be the exception not the rule.

We appreciate this feedback! Glad to hear it is working for a majority of your users @Lauwnch.
Our engineering team is still evaluating the full breadth of testing and outlier scenarios before we update documentation.

#please-close


feat(tfrun): Add cleanup of past TerraformRuns

#167 needs to be merged into main first.

Codecov Report

Merging #168 (9ced629) into main (c0abbca) will decrease coverage by 1.42%.
The diff coverage is 66.77%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
- Coverage   69.04%   67.63%   -1.42%
==========================================
  Files          23       28       +5
  Lines        1586     2039     +453
==========================================
+ Hits         1095     1379     +284
- Misses        425      575     +150
- Partials       66       85      +19

Files
Coverage Δ

api/v1alpha1/common.go
100.00% <ø> (ø)

internal/annotations/annotations.go
100.00% <ø> (ø)

internal/controllers/terraformlayer/conditions.go
62.31% <ø> (-2.79%)
:arrow_down:

internal/controllers/terraformpullrequest/layer.go
84.00% <100.00%> (+0.43%)
:arrow_up:

internal/lock/lock.go
93.02% <100.00%> (+1.84%)
:arrow_up:

internal/burrito/config/config.go
48.75% <0.00%> (-0.62%)
:arrow_down:

internal/controllers/terraformlayer/controller.go
38.02% <40.00%> (-9.60%)
:arrow_down:

internal/controllers/terraformlayer/states.go
77.41% <73.33%> (+9.28%)
:arrow_up:

internal/controllers/terraformrun/pod.go
92.18% <82.35%> (ø)

api/v1alpha1/terraformlayer_types.go
16.66% <0.00%> (-83.34%)
:arrow_down:

... and 5 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Gitlab: Fix posting prun status on GitOPS comments

We were not posting the status on GitOPS comments, this is because we were not catching the Note events as defined by the Gitlab provider.
Better functional tests to check the actual comments has been posted properly on GitOPS comments for gitea.
This has been backported as well for release-v0.24.x in https://github.com/openshift-pipelines/pipelines-as-code/pull/1585
Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/retest

lgtm


slight formatting adjustments for svp

This isn't ideal but I'm not sure how much more time to spend on it.

html version now looks kind of better... using a new package though.


GH74: Adds MSpec support

This adds support for MSpec unit tests

Support NET4 and X86.
Doesn't support NET2 (assuming no one uses it anymore).
The mspec console currently doesn't support dotnet core.

@holytshirt,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@holytshirt, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

done
On 11 October 2017 at 15:52, Gary Ewan Park<EMAIL_ADDRESS>wrote:

@gep13 requested changes on this pull request.
In src/Cake.Common/Tools/MSpec/MSpecRunnerUtilities.cs
https://github.com/cake-build/cake/pull/1873#discussion_r144036515:

@@ -0,0 +1,22 @@
+using System.Collections.Generic;

Can you add the license header here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cake-build/cake/pull/1873#pullrequestreview-68636194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFgFOjO33rSyPT-nV1kppWg-QHAjJsVks5srNY3gaJpZM4P1lsT
.

Pleasure doing business with you 😊
From: Gary Ewan Park
Sent: 11 October 2017 16:06
To: cake-build/cake
Cc: Toby Henderson; Mention
Subject: Re: [cake-build/cake] GH74: Adds MSpec support (#1873)
@gep13 approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I can't see why the OSX Cake build has failed, says I'm "Not authorized." I signed up to bitrise but still can't seem the error

@holytshirt CI failed to restore packages. I've restarted it for you.

Relates to #74

@holytshirt your changes have been merged, thanks for your contribution 👍

Awesome thanks!


added UUID column to the managed image table

added an additional column to the managed image table (UUID). Changed compoundParent to 2 due to column order

Thanks Chris (aka @resoluteCoder)


Small fix to ensure Assignment field indexes start at 1

This addresses the comment https://github.com/docker/swarmkit/pull/1544#discussion_r82338554.
cc @stevvooe

Current coverage is 54.78% (diff: 100%)

Merging #1604 into master will decrease coverage by 0.07%

@@             master      #1604   diff @@
==========================================
  Files            84         84
  Lines         14174      14174
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           7776       7765    -11
- Misses         5381       5385     +4
- Partials       1017       1024     +7

Powered by Codecov. Last update c7ade46...91bc273

LGTM

ping @docker/core-swarmkit-maintainers - let's merge this soon to minimize compatibility issues

LGTM, though I think if this was just left as 2 it wouldn't make a difference, right? 1-10 are all the same performance-wise, right?

1-10 are all the same performance-wise, right?

Don't consider performance when making field id selections.
Leaving at 2 is fine, but it always makes you wonder what happened to 1. Is he okay?

LGTM


Saving throws not showing up

For some reason saving throws are not showing up.
The only problem i see i might be doing is switching from tidy sheets to this one doesn't bring saving throws over?

Yeah must be some interference because it works fine for me.
On Nov 26, 2022 at 1:12 PM -0800, heyzoms @.***>, wrote:

For some reason saving throws are not showing up.
The only problem i see i might be doing is switching from tidy sheets to this one doesn't bring saving throws over?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

For some reason saving throws are not showing up.
The only problem i see i might be doing is switching from tidy sheets to this one doesn't bring saving throws over?

I am also having this issue - were you able to find a solution? I've tried disabling all other modules and activating just this module, but it still doesn't work. Here is a screen shot of what it looks like.

Thanks for responding. First, thanks for this awesome module. It's so much easier to reference all features and abilities on one page than the standard character sheet.
A recommendation - would it be possible to see Saving Throws as a separate line? Same as it's seen on DnDBeyond, like this:

Oh you know what, thanks for that screenshot. Maybe they did disappear… I’ll look into it. I agree it would be better.
On Feb 7, 2023 at 12:05 PM -0800, SaintLoda @.***>, wrote:

Thanks for responding. First, thanks for this awesome module. It's so much easier to reference all features and abilities on one page than the standard character sheet.
A recommendation - would it be possible to see Saving Throws as a separate line? Same as it's seen on DnDBeyond, like this:
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Addressed in 1.4.4.


Create linter for status files

Remove references to files and directories that no longer exist
Ensure all failures have associated issues
Etc

@whesse, do our status file tools cover these cases? Should we keep this open if we plan on deprecating status files anyway?

We have a presubmit on status files about normalizing them, but no additional work should be done on them except implementing the plan to remove them.

I did add a bunch of validation to status files after this bug was filed, but @whesse is right that we don't intend to do any more.


Adding reverse lookup function to LookupExtractor.

This PR adds to the LookupExtractor unApply  function in order to perform reverse lookup.
Part of this PR is to clean most of the unused import.

@cheddar and @drcrallen please checkout the new API, i want your opinion before i start working on the self cached version of JDBC.

can we clean unused imports in a separate PR ?
Its good to separate unrelated things in separate PR's helps in backporting things to different branches.

Aside from wanting to add the bulk methods as well, I'm :+1: on this.  The primary change is the LookupExtractor and then there's a bunch of changes to the only known current extension.
I agree with separating the cleaning of imports.  We should either do that in its own PR or opportunistically as those classes are touched.
For this PR, I could see an argument for adding the usages of the new interface method as well.  I.e. the broker's optimizations that can be enabled by this.  I can also see an argument for doing that in its own PR.  I'm ok with either of them.

@cheddar i will send two PRs one for making LookupExtractor abstract class and add bulk functions and second for the use case of unApply.

@nishantmonu51 moved out the cleaning import.

@fjy please checkout this

@b-slim I have a few comments. Most are minor.

@fjy and @cheddar please check this out.

:+1:

@fjy build passed ! can u please pull this

@b-slim : https://travis-ci.org/druid-io/druid/jobs/97728605

@drcrallen  this should fix it https://github.com/druid-io/druid/pull/2110/files or i can add a while loop as you have done for other tests, let me know what work better.


Loss linear is None

Hi, I have trained with my dataset. I trained with 20k images and everything was fine but I'm training with 70k images and linear losss are NONE.
Why ?

Oh, I see . I trained with labels so loss linear is not None. If we train without labels, loss linear will be None, right ?

Yes @Holmes2002 the linear probe needs labels to train so if your dataset doesn't include labels it won't train those pieces.


Preserve indentation in checklists

Hey! Nice plugin; just imported all of my notes into Obsidian with nary a hitch.
Only issue I had was that checklist notes with indented items didn't have their indentation preserved after import.
For example, the following:
[ ] Item 1
    [ ] Item 1.1
[ ] Item 2

Was imported into Obsidian as:
[ ] Item 1
[ ] Item 1.1
[ ] Item 2

It wasn't much of an issue for me since I only had two or three notes where this applied, but could be an annoyance for others.

Thanks for creating an issue and me know!
I’ll have a look soon and see if I can fix it up.

Actually, I just checked one of my files that had this issue. Seems the takeout doesn't include checklist indentation at all. I have a file that looks like this:

But the JSON for it looks like this:
{
  "listContent": [
    {
      "text": "And of course:",
      "isChecked": false
    },
    {
      "text": "Custom portfolio site",
      "isChecked": false
    },
    {
      "text": "Ascalon web interface",
      "isChecked": false
    }
  ]
}

The HTML export doesn't have any indentation either. Guess it's a no-go.

That's a shame. Thanks for checking. I'll close this issue and add a note in plugin that this isn't supported.


test(batch): More local some tests

What's changed and what's your intention?
Move all basic tests to support both local and distributed mode.
Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

Blocked on https://github.com/singularity-data/risingwave/pull/3242


Release / 4.5.2

Milestone: https://github.com/Emurgo/yoroi-mobile/milestone/10?closed=1
Initial change set

https://github.com/Emurgo/yoroi-mobile/pull/1836
https://github.com/Emurgo/yoroi-mobile/pull/1840
https://github.com/Emurgo/yoroi-mobile/pull/1842
https://github.com/Emurgo/yoroi-mobile/pull/1843

This release should resolve these three:

https://github.com/Emurgo/yoroi-mobile/issues/1838
https://github.com/Emurgo/yoroi-frontend/issues/2510
https://github.com/Emurgo/yoroi-frontend/issues/2578

Release: https://github.com/Emurgo/yoroi-mobile/releases/tag/4.5.2


Assignment with user-assigned managed identity at management group scope : The request did not have a subscription or a valid tenant level resource provider

Hi,
I have created a blueprint at management scope. I want to assign blueprint as stated in the document.
But I have this error:
('blueprint_assign_response', u'{"error":{"code":"MissingSubscription","message":"The request did not have a subscription or a valid tenant level resource provider."}}')
managed identity has blueprint operator permission.
Any advice would be appreciated.

Can you share how you are creating the blueprint assignment? You need to provide a specific subscription that you want the blueprint to be applied to. Assigning blueprint at management scope is only for restricting CRUD permissions for the blueprint assignment itself. It still functionally operates like a subscription-scoped blueprint assignment.

Hi,
Assignment creating via api:
        "identity": {
            "type": "userAssigned",
            "userAssignedIdentities": {
                "/subscriptions/{subscription_id}/resourcegroups/{rg}/providers/Microsoft.ManagedIdentity/userAssignedIdentities/{managed_identity}": {}
            }

        },
        "location": "",
        "properties": {
            "description": "blueprint",
            "blueprintId": "/providers/Microsoft.Management/managementGroups/{management_group}/providers/Microsoft.Blueprint/blueprints/{blueprint_name}",
            "scope": "subscriptions/{subscription_id}",
            "parameters": {

            },
            "resourceGroups": {
                "ResourceGroup": {
                    "name": "",
                    "location": "",

                }
            }
        }
    })

blueprint_assign_url = "https://management.azure.com/providers/Microsoft.Management/managementGroups/{management_group}/providers/Microsoft.Blueprint/blueprintAssignments/xwx?api-version=2018-11-01-preview"

blueprint_assign_response = requests.request(
        "PUT",
        blueprint_assign_url,
        headers=blueprint_header,
        data=blueprint_payload)```


Update bundle config reference for error plugin

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets
---

Documentation
https://github.com/php-http/HttplugBundle/pull/419

License
MIT

What's in this PR?
Updates the Symfony config reference for the added error plugin configuration
Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
[X] Documentation pull request created (if not simply a bugfix)

can you please also update the list at https://docs.php-http.org/en/latest/integrations/symfony-bundle.html#list-of-services ?

Just to be safe, because the plugin is already in the list. It should be moved to the last section: "These are plugins that are disabled by default and only get activated when configured.", right?

Just to be safe, because the plugin is already in the list. It should be moved to the last section: "These are plugins that are disabled by default and only get activated when configured.", right?

yes indeed. oh, so we even had it in the list but forgot to add configuration for it? all the better you fixed the situation. thanks a lot!


Update create-issue-from-file GHA 2.3.2 to 3.0.0

Depends on https://github.com/submariner-io/enhancements/pull/40.

@dfarrell07 could you rebase this PR?


[Question] Is there anyway to slow down the flow of items, or get them to flow in stacks?

Is it possible to move items in stacks/groups of 4 instead of one at a time

It would be handy as I have set my hoppers to be able to pull out items 4 at a time

Alternatively is it possible to slow down the flow?

Keep up the great work!

I am also interested in having a mechanism to slow down the item flow. In my case I have an iron farm which produces tons of iron ingots per minute, and I autocraft them to iron blocks, the pipes are so fast, that the autocrafter will break.
So a property to set the item flow speed would be really helpful

I am also interested in having a mechanism to slow down the item flow. In my case I have an iron farm which produces tons of iron ingots per minute, and I autocraft them to iron blocks, the pipes are so fast, that the autocrafter will break. So a property to set the item flow speed would be really helpful

Pipes are meant to be used alongside Hoppers for situations like those
I didn’t want to outright completely replace them


xml open update

Description

Open only specific nc file(s) that are listed in the xml file, instead of opening all nc files in the directory
When xml was generated using relative path, in particular in the same directory where data is, assume nc files are in the current directory (solution suggested by @pochedls)

Closes #454
Closes #459

Checklist

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

If applicable:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass with my changes (locally and CI/CD build)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)

Codecov Report
Patch coverage: 41.66% and project coverage change: -0.49 :warning:

Comparison is base (cbef082) 100.00% compared to head (d2d4f5f) 99.51%.

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##              main     #460      +/-   ##
===========================================
- Coverage   100.00%   99.51%   -0.49%
===========================================
  Files           14       14
  Lines         1425     1435      +10
===========================================
+ Hits          1425     1428       +3
- Misses           0        7       +7

Impacted Files
Coverage Δ

xcdat/dataset.py
95.73% <41.66%> (-4.27%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

When xml was generated using relative path, in particular in the same directory where data is, assume nc files are in the current directory (solution suggested by @pochedls)

After trying to write a unit test to cover this case, I found that substituting directory="" with the current working directory (".") will introduce issues.

The current directory (".") is relative to where xcdat is ran

If the user doesn't run xcdat in the same directory as the data, this will raise OSError: no files to open
I am confident there is a high possibility that users will fail to do this

This substitution behavior seems to be handling very specific I/O use cases, especially with the deprecated CDML format

I don't think this logic should be implemented in xcdat if it is only applicable to a small subset of potential xcdat users (who are CDML users)
We should eventually be moving away from CDML since CDAT is being deprecated

My proposed solution:

CDML users should supply the string path explicitly if no directory attribute is found in the XML

I think they should know where the .nc files are generated?
Alternatively, CDML users can implement directory attribute substitution logic in their own code instead of baking it into xCDAT

When xml was generated using relative path, in particular in the same directory where data is, assume nc files are in the current directory (solution suggested by @pochedls)

From my understanding, we are trying assuming .nc files are in the same directory as the .xml file. After trying to write a unit test to cover this case, I found that substituting directory="" with "." introduces issues:

The current directory (".") is relative to where xcdat is imported/ran, not the supplied XML file

If the user doesn't run xcdat in the same directory as the data, this will raise OSError: no files to open

This substitution behavior is handling specific I/O cases for CDML users

My proposed solution: CDML users should supply the string path explicitly if no directory attribute is found in the XML
Example:
xml_path =  "/path/to/data/xml"

# Breaks because to `directory` key
xc.open_mfdataset(path)

# User assumes or knows files are in the same directory as the XML
xc.open_mfdataset("/path/to/data/*.nc")

@tomvothecoder I like the idea, but I am not clear where I can include this logic under the current structure. Would you mind adding it into the code in this PR?

My proposed solution: CDML users should supply the string path explicitly if no directory attribute is found in the XML
@tomvothecoder I like the idea, but I am not clear where I can include this logic under the current structure. Would you mind adding it into the code in this PR?

What I mean by my proposed solution is that the user implement the try and except statement in their code if they are using XML files to open up datasets. This would catch instances where the directory key is potentially blank in the XML file(s).
xcdat shouldn't assume a specific directory (e.g., current directory or same directory as XML) if this key is blank since it might not always be correct.

Didn't this PR arise in part to address a blank directory attribute in the xml?
Can't we just add something like this here:
os.path.join(os.path.dirname(xml_path), '') # wrap dirname with join to ensure you have the trailing slash
If the xml is missing the directory attribute it means that it was created with data from the same directory.

I think for this particular PR, I'm fine with adding the cdms_filemap parsing and assuming datasets are in the same directory as the XML file if it has directory="" (although we get the same OSError if the files are not in there).
In hindsight, I think CDML/XML support is more ideal for a PCMDI utilities package. We're starting to integrate features that are too specific to an end-of-life software (CDAT/CDML) and for a smaller user group. I'm being cautious about how much support we have for CDML in xCDAT because it is much more difficult to remove code than to add code, especially in a mature package.

Actually on second-thought, I still have concerns about this scope of this feature. Let's talk about it at our next meeting since isn't the highest priority item.

Path forward:

Scope this feature
Figure out timeline

deprecate this feature by end of the year when CDAT/CDML gets deprecated?

Review this PR

We have feature request guidelines here that we overlooked https://xcdat.readthedocs.io/en/latest/contributing.html#feature-request

I have noted in the past that you could create your own xml/cdml file without cdscan, which could be a nice feature (e.g., handling spliced datasets), but it is true that

Climate Data Markup Language (CDML) is the markup language used to represent metadata in CDMS

Does this mark the beginnings of the xcdml format? We will see.

Much to the dismay of @lee1043 and @pochedls  I have decided to veto CDML support due to this reason.
Next steps will be:

Close this PR
Add a deprecation warning when opening CDML/XML files with xcdat in v0.6.0 -- this is standard practice and the user can silence them
Remove CDML support in the subsequent release (maybe v0.7.0) -- this timeline should parallel CDAT's end-of-life at the end of this year

Suggested Alternative
I suggest we drive #492 forward, which discusses the idea of developing a package with more specialized features that meet the needs of LLNL Climate Program staff.
This new package can provide an API to parse CDML files for .nc filepaths (PR #460 and #385). I'm aware that it will take time for some users and packages (e.g.,  pcmdi_metrics) that rely on the CDML files hosted at LLNL to transition away from them. I don't think a path forward has been scoped for CDML users yet (maybe use xsearch?).


Need to get the correct DPI value on Android

Hey guys,
Looks like there is a number of different ways to measure DPI on Android
Lime uses DisplayMetrics.xdpi , but a number of devices have this value completely wrong
For example Lint marks usage of "mm", "in" and other android metrics that rely on xdpi and ydpi with warnings - http://stackoverflow.com/questions/20631221/why-lint-shows-warning-when-using-in-inch-or-mm-millimeter-units-as-dimensio/22707894#22707894
For example my Nexus 4 gives xdpi of 160 instead of expected 320, but densityDPI gives correct value of 320. It looks like it's a Cyanogen mod bug, since the latest stock Lolipop Nexus 4 gives correct 320 in xdpi and ydpi. But what's important is that I have never noticed wrong sizing of UI on my nexus 4 in years, all because of the fact that all android UI sizing logic always relies on DisplayMetrics.densityDPI instead of xdpi/ydpi and only when I used DPI in Lime I've noticed that the value is wrong.
Here Google http://developer.android.com/guide/practices/screens_support.html recommends using "dp" units which are based on .densityDPI and .density and also shows code examples using .density.
To clarify why there are 2 different ways to access dpi -

xdpi/ydpi gives exact DPI value of a device.
density and densityDPI give a value of a nearest dpi bucket android os assigns a device to (120, 160, 240, 320, 480 or 640).

So if a device has DPI of 330, densityDPI would return 320. I haven't done much Android development, but I think that is intended behaviour, though I'm not fully sure why.
I propose to change a way Lime calculates DPI to a most commonly and widely used approach -DisplayMetrics.densityDPI.
Of course it's wrong to change usage of API just because some phones/roms give a bugged value, but it's also important to use API in the most common way.
What do you guys think?

How maximum different percent can be from DisplayMetrics.densityDPI and right DisplayMetrics.xdpi ?

@FreakMurderer in theory - half of the bucket. Not sure what's the biggest one in practice.
PS: There are very few valid reasons to use xdpi/ydpi, one of them is virtual reality, for stuff like google cardboard. Another example would be making a ruler for real-life objects...

Even with newer SDL release versions, I believe this is still unimplemented. I think we'll have to use JNI and get this value ourselves
http://stackoverflow.com/questions/3166501/getting-the-screen-density-programmatically-in-android

I can confirm the (sdl) value is incorrect on android...

Sorry for circling back on such an old issue.
We started using JNI to calculate DPI on Android ourselves.
https://github.com/openfl/lime/blob/develop/templates/android/template/app/src/main/java/org/haxe/lime/GameActivity.java#L36-L47
lime.system.Display should have a dpi property that represents our best estimate of the real DPI of the device, openfl.system.Capabilities.screenDPI works more like the density mentioned in the original issue, where it averages to a close "DPI level" to use for sizing UI. We hope that both give developers the tools they need.
If you think we should modify our behavior for calculating Android DPI, open a pull request with a proposal :smile:
Thanks everyone


first devBranch commit.

added .env to gitignore, not even sure if that's going to be needed for rails dotenv

adding the .env ignore to master


Allow encoding ASN.1 objects using indefinite length encoding

I have a use case where I communicate with a remote system (over which I have no control) which crashes when it gets definite-length encoded ASN.1 objects for ASN.1 objects which exceed a certain length.
So this PR adds a UseIndefiniteLengthEncoding property to the DerAsnType class which allows you to force the encoder to use indefinite length encoding if you want to.

Discussion continues in #18


ld: library not found for -lstdc++

I have found an error when I install on Mac Mojave when I use pip to install
clang: warning: libstdc++ is deprecated; move to libc++ with a minimum deployment target of OS X 10.9 [-Wdeprecated]
    ld: library not found for -lstdc++

My solution is export MACOSX_DEPLOYMENT_TARGET=10.14 before pip install ...


解决当table的容器已经滚动到最右边，table的列总长度大于table长度出现滚动条，右边滚动到type为selection的列，

checkbox选择框只展示一半，选择头部的全选，出现头部向左边移动，body的内容没有变的错位bug
Please make sure these boxes are checked before submitting your PR, thank you!

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

a lot of pull requests and no merges...


Assignment 6: report

[x] Link to our GitHub repository with Readme
[x] Link to our deployed project: https://anonymous-feedback.snejugal.ru
[x] Link to our GitHub project board (https://github.com/orgs/InnoSWP/projects/22/views/2)
[x] A summary of:

[x] sprint goal
[ ] sprint retrospective
[x] sprint review
[ ] customer feedback

[x] Brief explanation of the MIT license: #96

240747


Section "Map the temporary cdnverify subdomain" shouldn't direct you to permanant endpoint

The section "Map the temporary cdnverify subdomain" says you can jump ahead to "Map the permanent custom domain" if you don't have production traffic, but if you skip over "Associate the custom domain with your CDN endpoint" it won't work and you'll get a weird error message. The first section should clarify that you always need to add the custom domain to your CDN endpoint no matter what option you choose.

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

ID: b32fbbf3-178d-a0ef-7c3a-87df74a8eb16
Version Independent ID: 2475b34a-88e8-5b45-723d-ee0a62230b8a
Content: Tutorial - Add a custom domain to your Azure CDN endpoint
Content Source: articles/cdn/cdn-map-content-to-custom-domain.md
Service: azure-cdn
GitHub Login: @asudbring
Microsoft Alias: allensu

@dbrennan Thanks for your comment! We will review and provide an update as appropriate.

@dbrennan , Appreciate your patience. Mapping temporary subdomain is always been an option. You can skip that step if you are not in production. Please let us know at<EMAIL_ADDRESS>if you still face any error while mapping custom domain for deeper investigation.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


2.15.4 Release

Summary
Hotfix branch for the 2.15.4 release.
This is intended to support the upcoming consensus release by updating the submodule.
A small bugfix is also included:
https://github.com/algorand/indexer/pull/1459

Approving because passed catchup and validation tests.


Godot 3.5 support

There are multiple errors preventing this plugin to work on anything other than Godot 3.2. I would love to port this plugin to 3.5 but run into some errors I can't start to solve because the implementation is fairly big. Is there some way to contact you directly and ask some questions about the structure of the plugin?

Hey Sander. I am Calneon#3773 on Discord, that's probably the best way to get in touch.

I just tried the demo project on the latest Godot version (v3.5.1), and it worked correctly for me. I did get an error when re-creating the NavigationPolygon, it seems checking for null in plugin.gd line 24 is no longer enough and is_instance_valid() must be used instead. But after making that change I can generate a new navmesh.


[BrokenLinksH2] removing broken links

Global effort to fix broken links
@rick-anderson
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. Please review within five business days and merge, or comment in the PR with any changes you'd like to see. Thanks!

@StephenBrentPeters  can you start putting all the files in a single PR?
aspnet/ajax/cdn/jquery-datatables/cdnjquerydatatables182.md
aspnet/ajax/cdn/jquery-datatables/cdnjquerydatatables105.md
aspnet/ajax/cdn/jquery-datatables/cdnjquerydatatables104.md

aspnet/ajax/cdn/jquery-datatables/cdnjquerydatatables190.md

aspnet/ajax/cdn/jquery-datatables/cdnjquerydatatables191.md
That should have been 1 PR, not 5


Fix issue with infinite loop

Remove the new Error thrown when update is not passed, as user can
now pass in other types of actions like onenter (responsibility of
the user to implement an action for the new value)
Update readme
Fix issue where user would need to specify both checked and value for checkboxes

Closes #11
Should also close #12, as the action will now only fire if _sanitizedValue !== rawValue (set in didInitAttrs)

cc @rwjblue

@homu r+

:pushpin: Commit 5e6c6f0 has been approved by poteto

:zap: Test exempted - status


[Bug] 我已換了新api key 也不行

昨天到現在，一直出現下面這段代碼
我已換了新api key 也不行
希望有人能解答，感謝
        Attention Required! | Cloudflare               Please enable cookies.   Sorry, you have been blocked You are unable to access api.openai.com

Why have I been blocked?

This website is using a security service to protect itself from online attacks. The action you just performed triggered the security solution. There are several actions that could trigger this block including submitting a certain word or phrase, a SQL command or malformed data.

What can I do to resolve this?

You can email the site owner to let them know you were blocked. Please include what you were doing when this page came up and the Cloudflare Ray ID found at the bottom of this page.

 Cloudflare Ray ID:<PHONE_NUMBER>030f0c •  Your IP: Click to reveal 2a06:98c0:3600::103 •  Performance & security by Cloudflare

HK ip的好像会被vercel透传IP？看#3286

前天 之前一直用vercel，是可以的，昨天開始不能用，有什麼解決方法？謝謝

更新你的代码到最新版本，#3286


Support for other container runtimes than Docker

Problem
I'm trying to use scaphandre on an Openshift/OKD cluster that is not using Docker as the container runtime but CRI-O.
Scaphandre seems to only support docker for its --containers option with the prometheus submodule.
For instance that's what I get on an OKD test cluster (4.12.0-0.okd-2023-02-04-212953), running CRI-O 1.25 and Kubernetes 1.25 on Fedora CoreOS 37 nodes:
root@scaphandre-bnl6l:/app# scaphandre -v 3 prometheus --containers
Scaphandre prometheus exporter
Sending ⚡ metrics
scaphandre::exporters::prometheus: 2023-02-15T15:06:27: Starting Prometheus exporter
Press CTRL-C to stop scaphandre
scaphandre::exporters: Couldn't connect to docker socket. Error: /var/run/docker.sock not found.

Solution
That would be great if scaphandre was somehow compatible with other container runtimes (I think there is a CRI standard nowadays?)  so that it can work in any kind of K8s cluster.
I've got no clue how we could do that however TBH :)

Hi !
Support for CRI-O can be added for 1.1 release. Would you be keen to test it once it's being developed ?

Hi !
Support for CRI-O can be added for 1.1 release. Would you be keen to test it once it's being developed ?

Hello,
Of course, I would gladly do that.

Hi,
Can we expect this to be ready for 1.1 release by september 22?


Add IdentityMetadata Routes

Readiness checklist

[x] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

@sebbi08 is your feedback also incorporated well enough?

LGTM


nixos/incus: add support for software TPMs

Was previously broken due to a missing runtime dependency.

Error: Failed to start device "vtpm": Failed to validate environment: Required tool 'swtpm' is missing

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@ofborg test incus.virtual-machine

Probably related to the virtualisation.tpm implementation. CC @RaitoBezarius as the author of that feature.

Probably related to the virtualisation.tpm implementation. CC @RaitoBezarius as the author of that feature.

LGTM from my end.


Improved DateTimeSymbol Capabilities

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=601

Wasn't clear if user needed runtime or compile time capabilities. In either case, reviewers provided solutions to both with no response from user.


URI injection

Hello,
When I try to do URI injection your tool fails automatically. Would it be possible to add this feature to your tool?
For example, sqlmap gives you the option to choose for URI injection.
Thanks.

For now you can try inserting custom injection marker within the uri it should scan


renovate: Don't exclude github.com/{cilium,vishvananda}/netlink anymore

The netlink library was excluded from renovate updates in d6fe7de12cd1 ("renovate: exclude github.com/{cilium,vishvananda}/netlink"), because at that point Cilium was using its fork, but then in 9f0be9ecbe73 ("vendor: Update vishvananda/netlink/") it was switched back to the upstream version. Update the renovate configuration accordingly to no longer exclude netlink from updates.
The d6fe7de12cd1 commit message says that it should be switched back, but it seems it hasn't been done yet. I think it needs to be backported to v1.14 too?

/test


Open API to public?

Perhaps some feasibility research would be best to start out with, as the costs involved with serving up these search results will eventually become unmanageable.
I'd like for users of my modular system to use a command-line utility to search npms and return results based upon keywords. Currently, they must use the npmjs.com search which tears them away from the terminal.

Perhaps some feasibility research would be best to start out with, as the costs involved with serving up these search results will eventually become unmanageable.

We have not make it publicly available yet because it's not finalised and it's undocumented. This means that we may change the responses structure until then.

I'd like for users of my modular system to use a command-line utility to search npms and return results based upon keywords. Currently, they must use the npmjs.com search which tears them away from the terminal.

One of our goals is to build a CLI program, similar to npm search that uses our API. Would you be interested in contributing?

We have not make it publicly available yet because it's not finalised and it's undocumented. This means that we may change the endpoints and responses structure until then.

This could be solved elegantly with an intermediate SDK layer that both the official client and all other clients can connect with (Including the CLI below).

One of our goals is to build a CLI program, similar to npm search that uses our API. Would you be interested in contributing?

Perhaps. I have a lot of experience with CLI applications and testing them, but we'd need to agree upon an approach before we go butting heads against each other :)

For example, the project will need to agree upon standards for:

linting rules
test runner
assertion style
quality services

bithound, codeclimate, etc.

workflow management (gitflow?)
test structure

one assertion per test? multiple assertions per test?
BDD or TDD?

This is just the beginning of the conversation, really.

Perhaps a new issue for discussion of standards is in order.

We should create a new issue in the npms-api repository itself and discuss there.

See: https://github.com/npms-io/npms-cli/issues/1


python312Packages.dissect-shellitem: 3.6 -> 3.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.dissect-shellitem is: Dissect module implementing a parser for the Shellitem structures
meta.homepage for python312Packages.dissect-shellitem is: https://github.com/fox-it/dissect.shellitem
meta.changelog for python312Packages.dissect-shellitem is: https://github.com/fox-it/dissect.shellitem/releases/tag/3.7
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.7 with grep in /nix/store/xwabfb968jzr55ys3a23zf92zsvlbj76-python3.12-dissect-shellitem-3.7
found 3.7 in filename of file in /nix/store/xwabfb968jzr55ys3a23zf92zsvlbj76-python3.12-dissect-shellitem-3.7

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
acquire
python311Packages.acquire
python311Packages.dissect
python311Packages.dissect-shellitem
python311Packages.dissect-target
python312Packages.acquire
python312Packages.dissect-shellitem
python312Packages.dissect-target

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xwabfb968jzr55ys3a23zf92zsvlbj76-python3.12-dissect-shellitem-3.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.dissect-shellitem https://github.com/r-ryantm/nixpkgs/archive/7d8314e11be6ff6e7f0169635775bd888f3a3a35.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7d8314e11be6ff6e7f0169635775bd888f3a3a35#python312Packages.dissect-shellitem

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xwabfb968jzr55ys3a23zf92zsvlbj76-python3.12-dissect-shellitem-3.7
ls -la /nix/store/xwabfb968jzr55ys3a23zf92zsvlbj76-python3.12-dissect-shellitem-3.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

python312Packages.acquire
python312Packages.acquire.dist

12 packages built:

acquire (python311Packages.acquire)
acquire.dist (python311Packages.acquire.dist)
python311Packages.dissect
python311Packages.dissect-shellitem
python311Packages.dissect-shellitem.dist
python311Packages.dissect-target
python311Packages.dissect-target.dist
python311Packages.dissect.dist
python312Packages.dissect-shellitem
python312Packages.dissect-shellitem.dist
python312Packages.dissect-target
python312Packages.dissect-target.dist

Maintainer pings
cc @fabaff for testing.

Superseded by #294244


Merged docs/commit-message.md into contribution-guidelines.md

commit-message.md duplicated a section in contribution-guidelines.md. I went ahead and merged the two.
Merge #29 before this

LGTM otherwise

Yeah, but the GitCop footer was pointing to commit-message.md, and now that this file is gone, the link gives a 404 not found.
I will fix that.

Oh I'm sorry. that's my mistake. Apologies-- hard to keep track of all the dependencies on things.
@chriscool i'm ok to revert this, too. The "single file to look at" for figuring out what's wrong with the commit was pretty useful, i think. maybe we can just link to it from the original contribution-guidelines.md.

@jbenet no problem I already fixed the GitCop footer to point to:
https://github.com/ipfs/community/blob/master/contribution-guidelines.md#commit-messages
But note that there is still some duplication as the commit-messages section now talks about the Signed-off-by: as does the sign-and-license-your-work section in the same document:
https://github.com/ipfs/community/blob/master/contribution-guidelines.md#sign-and-license-your-work
I will let you decide what is best.

@chriscool Darn, I missed that. I'll quickly submit another pr


Supporting a TF lite shared library target

Describe the problem
Would the TF project be open to supporting a TF lite shared library target (as is done already with libtensorflow.so and libtensorflow_cc.so)?
I believe many people would benefit from this, based on recent related issues:
https://github.com/tensorflow/tensorflow/issues/18060
https://github.com/tensorflow/tensorflow/issues/17826
https://github.com/tensorflow/tensorflow/issues/16219
I am happy to do the upfront work, but I would require assistance from TF devs for e.g. correct build configuration and on-going support.
Source code / logs
We could add a tf_cc_shared_object target to tensorflow/contrib/lite/BUILD:
tf_cc_shared_object(
    name = "libtensorflow_lite.so",
    framework_so = [],
    linkopts = tflite_linkopts() + ["-s"],  # Strip library.
    visibility = ["//visibility:public"],
    deps = [
        ":framework",
        "//tensorflow/contrib/lite/kernels:builtin_ops",
    ]
)

We could also add a header-only target to define the headers for use with libtensorflow_lite.so (although this won't be useful until issue 5192 has been resolved).
As mentioned above, I'd need some input to decide the correct build config (e.g. linkopts, use of framework_so).

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

These fields are not applicable, as this is a feature request. I might have used the wrong template.

Making the shared library target is easy, but we probably want to be careful about which symbols should be exposed.

Hi @martis-chromium, we're actively working to publish a fully functional, pure C API, and as part of this will support a proper shared library target and various language bindings for that API. We don't have an exact ETA, but expect some related activity in the coming weeks.

Ok, good to know - thank you.

In the meantime, I'd like to (locally) produce a TF lite shared library and limit its size somewhat.
Is using a linker version script to only expose symbols containing the substring "tflite" an acceptable (broad-stroke) solution?
I am able to compile and run the demos using this approach, but I wanted to double-check that I'm not suppressing some symbols I'll need later.

You can certainly try that, though you may get mixed results. One example of a stripped binary for TFLite is the JNI library @ https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/lite/java/BUILD#L172. That build rule is even more aggressive about stripping.

Nagging Assignee @jdduke: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

A preview of what the shared library looks like is the new :litbensorflowlite_c.so target. We'll be adding an analogous target with C++ bindings in the near future.

@martis-chromium how do you build the demo? I am having exactly the same problem as #18060

Are you waiting on a response from me? I'm not really in a position to help @tensorbuffer with his problem.

Nagging Assignee @jdduke: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jdduke: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jdduke: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jdduke: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@Nimitz14 here's how I did it after reading a couple of issues around here
diff --git a/tensorflow/lite/BUILD b/tensorflow/lite/BUILD
index be84fc5db1..fadf4069e7 100644
--- a/tensorflow/lite/BUILD
+++ b/tensorflow/lite/BUILD
@@ -342,6 +342,21 @@ cc_test(
     ],
 )

+cc_binary(
+    name = "libtflite.so",
+    deps = [
+        "//tensorflow/lite/kernels:builtin_ops",
+        "//tensorflow/lite:builtin_op_data",
+        "//tensorflow/lite:framework",
+        "//tensorflow/lite:schema_fbs_version",
+        "//tensorflow/lite:string",
+        "//tensorflow/lite:string_util",
+        "//tensorflow/lite/schema:schema_fbs",
+
+    ],
+    linkshared=1
+)
+
 # Test the serialization of a model with optional tensors.

 # Model tests

You can certainly get away with less entries in deps.
This issue is along with #22643 make the development on Android a big headache.

Here's mine:
diff --git a/tensorflow/lite/BUILD b/tensorflow/lite/BUILD
index f8bb719..7fe6227 100644
--- a/tensorflow/lite/BUILD
+++ b/tensorflow/lite/BUILD
@@ -340,6 +340,20 @@ cc_test(
     ],
 )

+cc_binary(
+    name = "libtensorflowlite.so",
+    linkopts=[
+        "-shared",
+        "-Wl,-soname=libtensorflowlite.so",
+    ],
+    linkshared = 1,
+    copts = tflite_copts(),
+    deps = [
+        ":framework",
+        "//tensorflow/lite/kernels:builtin_ops",
+    ],
+)
+

Hi, TF developers
I also find that it will be super convenient if there is a bazel target for C library. Looking forward to that!
@Nimitz14, for now I find a makefile under tensorflow/lite/tools/make. And if we make it under tensorflow root path, like: make -f tensorflow/lite/tools/make/Makefile TARGET=android TARGET_ARCH=armv7 TARGET_TOOLCHAIN_PREFIX=/ndk_standalone_tc/p19/arm/bin/arm-linux-androideabi- all -j6.
It should generate a static library under tensorflow/lite/tools/make/gen/android_armv7/lib/ named libtensorflow-lite.a .
Thanks again :+1:

Nagging Assignee @jdduke: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hey all, a new shared library target that has both C and C++ bindings is now available.
You can build this (for Android) as follows:
bazel build //tensorflow/lite:libtensorflow.so \
    --config=android_arm --cxxopt='--std=c++11' -c opt

This should have a smaller binary size as it strips most unnecessary symbols. Sadly, bazel doesn't have very good support for generating monolithic static (.a) libraries unless you use a custom build rule that manually assembles the objects via ar.

bazel build //tensorflow/lite:libtensorflow.so
--config=android_arm --cxxopt='--std=c++11' -c opt

bazel build //tensorflow/lite:libtensorflowlite.so
--config=android_arm --cxxopt='--std=c++11' -c opt

Ah, good catch, updated!

bazel build //tensorflow/lite:libtensorflowlite.so
--config=android_arm --cxxopt='--std=c++11' -c opt

What does it take to make this work for the x86/x86_64 configurations? --config=android_x86 is rejected as invalid by bazel.

bazel build //tensorflow/lite:libtensorflowlite.so
--config=android_arm --cxxopt='--std=c++11' -c opt

What does it take to make this work for the x86/x86_64 configurations? --config=android_x86 is rejected as invalid by bazel.

You can use bazel build //tensorflow/lite:libtensorflowlite.so  --config=android --cpu=x86 --cxxopt='--std=c++11' -c opt

The size of these shared libraries is ~50MB. Contrary to what @jdduke talked about here, saying it would 1-1.5MB. Is there something extra one has to do for that to happen?

@RuABraun can you provide the exact build command you used to generate the library? Using bazel build //tensorflow/lite:libtensorflowlite.so --config=android_arm64 --cxxopt='--std=c++11' -c opt I'm getting < 2MB.

I'm using the same. Afterwards:
~/git/tensorflow-android$ ll -h bazel-out/arm64-v8a-opt/bin/tensorflow/lite/libtensorflowLite.so
-r-xr-xr-x 1 rab rab 53M May 28 14:51 bazel-out/arm64-v8a-opt/bin/tensorflow/lite/libtensorflowLite.so*

During configure I said no everything except the android stuff, I'm not using clang. This is on the latest commit of master (last commit from 5 hours ago). Struggling to see where things could have diverged (unless I should be using clang?).

Hi, I'm trying to utilise a smaller tflite library in my Android app which led me to this thread. I managed to build the libtensorflowlite.so file, but have no idea on how to integrate it into my project. And I can't find any mention of it in the official website. Is there any guide to point me towards? Thank you.

Hey all, a new shared library target that has both C and C++ bindings is now available.
You can build this (for Android) as follows:
bazel build //tensorflow/lite:libtensorflowlite.so \
    --config=android_arm --cxxopt='--std=c++11' -c opt

This should have a smaller binary size as it strips most unnecessary symbols. Sadly, bazel doesn't have very good support for generating monolithic static (.a) libraries unless you use a custom build rule that manually assembles the objects via ar.

If it helps, here is a shell script that will compile the android distribution, and then make static libraries from the .a files in the build.
#!/bin/sh

# Script to make a tensorflowlite static library from the generated archives

# Run it from your local tensorflow directory

configs=(android_arm android_arm64)

for config in "${configs[@]}"
do
  bazel build --cxxopt='--std=c++11' -c opt --config=$config //tensorflow/lite/java:tensorflow-lite
done

# See: How to build tensorflowlite as a shared library
#   https://github.com/tensorflow/tensorflow/issues/20905#issuecomment-468756051
# See: How to combine libraries using ELF AR
#   https://stackoverflow.com/a/23621751/45114

# Step 1. Edit this to point to an 'ar' command in your Android toolchain that can
# handle the output archive files. I'm running on a mac so I use darwin
AR="${ARG_NDK_PATH}/toolchains/x86_64-4.9/prebuilt/darwin-x86_64/x86_64-linux-android/bin/ar"

# This is a function that outputs an "MRI" script as commands to the 'ar'
# command that can build the resulting library
write_mri() {

  # See: How to combine libraries using ELF AR
  #   https://stackoverflow.com/a/23621751/45114

  echo "create ${result}"

  for line in $(find $dir -type f -name \*.a)
  do
    echo "addlib ${line}"
  done

  echo "save"
  echo "end"
}

for dir in bazel-out/android-*
do
  mkdir -p "${dir}/lib"
  result="${dir}/lib/libtensorflowlite.a"
  rm -f $result
  write_mri | $AR -M
  echo "made: ${result}"
done

@michaeljbishop this very helpful however when I try to link against the generated libraries I the following errors:

@jdduke
I am on Ubuntu 16.04.6 LTS
I launched docker with:
'sudo docker run -it --rm tensorflow/tensorflow:devel bash'
Then './configure' in tensorflow_src (with defaults)
And then use the command:
bazel build //tensorflow/lite:libtensorflowlite.so
--config=android_arm --cxxopt='--std=c++11' -c opt
But I get the error:
ERROR: /tensorflow_src/tensorflow/lite/kernels/internal/BUILD:615:1: C++ compilation of rule '//tensorflow/lite/kernels/internal:audio_utils' failed (Exit 1)
Target //tensorflow/lite:libtensorflowlite.so failed to build
Furthermore when I rerun it I get:
ERROR: /tensorflow_src/tensorflow/lite/experimental/resource_variable/BUILD:6:1: C++ compilation of rule '//tensorflow/lite/experimental/resource_variable:resource_variable' failed (Exit 1)
Target //tensorflow/lite:libtensorflowlite.so failed to build
And I will get different fails almost arbitrarily.

@amitDaMan when you say you used the defaults, did you at least point it to your Android SDK/NDK install location? That is a required part of the configure script if you're building with --config=android_arm.

@jdduke  That solved it, thank you for your speed-y assistance

Hi @jdduke, I followed your instruction and successfully built  //tensorflow/lite:libtensorflowlite.so with size < 2M. However, when I link it with my executable and run on android, my tflite model loaded fine but I got "segmentation fault" when executing builder(&interpreter). I am using tensorflow 1.15.0 and flatbuffer 1.10. Any ideas ?

@zhaojiaTech were you using the same compiler and STL version for both the libtensorflowlite.so library and your own C++ code? If not, that might cause issues, and you may be better off using the C API which avoids these kinds of ABI incompatibilities.
I should note, we're hoping to introduce a helper build script which will also package all the necessary C++/C headers to make the shared libraries easier to use.

@jdduke hey. I am trying to run the c++ lib on iOS. Not using the C API/Swift/ObjC. Do you have any tips on how to do this? Building the c++ lib and linking it seems straightforward. But I am wondering how to setup the c++ lib to use the metal delegate?

@scm-ns this thread is quite long already, can you file a separate bug if you're having trouble with iOS + C++ + Metal?

hi, what if i what to build a lib for arm v6 linux?

Hi @jdduke ,
I followed your instruction and successfully built //tensorflow/lite:libtensorflowlite.so with size < 2M. However, when I run on android with 64-bit Architecture, my tflite model is allegendly terminated succesfully but with 0 [ms] runtimes and no output. When I used a different platform with 32-bit ARM based architecture everything works fine. I am uses tensorflow 1.14.0 and flatbuffer 2.0. Any idea what can be the problem?

@michaeljbishop Thanks for the script building a static build of tflite. However I keep getting undefined reference error, it seems that the static build does not include all needed implementations?
Error while executing process /home/andrey/Android/Sdk/cmake/3.6.4111459/bin/cmake with arguments {--build /home/andrey/data/bokeh/code/bokeh_android_tmp/app/.cxx/cmake/debug/arm64-v8a --target bokehapi}
[1/1] Linking CXX shared library ../../../../build/intermediates/cmake/debug/obj/arm64-v8a/libbokehapi.so
FAILED: : && /home/andrey/Android/Sdk/ndk/android-ndk-r17c/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++  --target=aarch64-none-linux-android --gcc-toolchain=/home/andrey/Android/Sdk/ndk/android-ndk-r17c/toolchains/aarch64-linux-android-4.9/prebuilt/linux-x86_64 --sysroot=/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sysroot -fPIC -isystem /home/andrey/Android/Sdk/ndk/android-ndk-r17c/sysroot/usr/include/aarch64-linux-android -D__ANDROID_API__=24 -g -DANDROID -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -Wa,--noexecstack -Wformat -Werror=format-security -std=c++11 -fopenmp -O2 -fvisibility=hidden -fvisibility-inlines-hidden -fomit-frame-pointer -fstrict-aliasing -ffunction-sections -fdata-sections -ffast-math   -std=c++11 -frtti -fexceptions -O0 -fno-limit-debug-info  -Wl,--exclude-libs,libgcc.a -Wl,--exclude-libs,libatomic.a -nostdlib++ --sysroot /home/andrey/Android/Sdk/ndk/android-ndk-r17c/platforms/android-24/arch-arm64 -Wl,--build-id -Wl,--warn-shared-textrel -Wl,--fatal-warnings -L/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/libs/arm64-v8a -Wl,--no-undefined -Wl,-z,noexecstack -Qunused-arguments -Wl,-z,relro -Wl,-z,now -shared -Wl,-soname,libbokehapi.so -o ../../../../build/intermediates/cmake/debug/obj/arm64-v8a/libbokehapi.so CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o CMakeFiles/bokehapi.dir/src/main/cpp/bokehapi_jni.cpp.o  -lvulkan ../../../../src/main/jniLibs/arm64-v8a/liblibpng.a ../../../../src/main/jniLibs/arm64-v8a/liblibjpeg.a ../../../../src/main/jniLibs/arm64-v8a/libIlmImf.a ../../../../src/main/jniLibs/arm64-v8a/liblibjasper.a ../../../../src/main/jniLibs/arm64-v8a/liblibtiff.a ../../../../src/main/jniLibs/arm64-v8a/liblibwebp.a ../../../../src/main/jniLibs/arm64-v8a/libIlmImf.a ../../../../src/main/jniLibs/arm64-v8a/libcpufeatures.a ../../../../src/main/jniLibs/arm64-v8a/liblibprotobuf.a ../../../../src/main/jniLibs/arm64-v8a/libtegra_hal.a ../../../../src/main/jniLibs/arm64-v8a/libtensorflowlite.a -ljnigraphics -lz /home/andrey/Android/Sdk/ndk/android-ndk-r17c/platforms/android-24/arch-arm64/usr/lib/liblog.so ../../../../src/main/jniLibs/arm64-v8a/liblibjasper.a ../../../../src/main/jniLibs/arm64-v8a/liblibtiff.a ../../../../src/main/jniLibs/arm64-v8a/liblibwebp.a ../../../../src/main/jniLibs/arm64-v8a/libcpufeatures.a ../../../../src/main/jniLibs/arm64-v8a/liblibprotobuf.a ../../../../src/main/jniLibs/arm64-v8a/libtegra_hal.a -ljnigraphics -lz /home/andrey/Android/Sdk/ndk/android-ndk-r17c/platforms/android-24/arch-arm64/usr/lib/liblog.so -latomic -lm "/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/libs/arm64-v8a/libc++_static.a" "/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/libs/arm64-v8a/libc++abi.a" && :
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `std::__ndk1::default_delete<tflite::Interpreter>::operator()(tflite::Interpreter*) const':
/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/include/memory:2233: undefined reference to `tflite::Interpreter::~Interpreter()'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `std::__ndk1::default_delete<tflite::FlatBufferModel>::operator()(tflite::FlatBufferModel*) const':
/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/include/memory:2233: undefined reference to `tflite::FlatBufferModel::~FlatBufferModel()'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `BokehAPI::loadModel(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&)':
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:22: undefined reference to `tflite::DefaultErrorReporter()'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:22: undefined reference to `tflite::FlatBufferModel::BuildFromFile(char const*, tflite::ErrorReporter*)'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `std::__ndk1::default_delete<tflite::FlatBufferModel>::operator()(tflite::FlatBufferModel*) const':
/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/include/memory:2233: undefined reference to `tflite::FlatBufferModel::~FlatBufferModel()'
/home/andrey/Android/Sdk/ndk/android-ndk-r17c/sources/cxx-stl/llvm-libc++/include/memory:2233: undefined reference to `tflite::FlatBufferModel::~FlatBufferModel()'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `BokehAPI::loadModel(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&)':
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:31: undefined reference to `tflite::ops::builtin::BuiltinOpResolver::BuiltinOpResolver()'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:32: undefined reference to `tflite::InterpreterBuilder::InterpreterBuilder(tflite::FlatBufferModel const&, tflite::OpResolver const&)'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:32: undefined reference to `tflite::InterpreterBuilder::operator()(std::__ndk1::unique_ptr<tflite::Interpreter, std::__ndk1::default_delete<tflite::Interpreter> >*)'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:32: undefined reference to `tflite::InterpreterBuilder::~InterpreterBuilder()'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:32: undefined reference to `tflite::InterpreterBuilder::~InterpreterBuilder()'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `BokehAPI::runBokeh(unsigned char const*, int*)':
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:36: undefined reference to `tflite::Interpreter::AllocateTensors()'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/src/main/cpp/BokehAPI.cpp:61: undefined reference to `tflite::Interpreter::Invoke()'
CMakeFiles/bokehapi.dir/src/main/cpp/BokehAPI.cpp.o: In function `~MutableOpResolver':
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/.cxx/cmake/debug/arm64-v8a/../../../../src/main/cpp/include/tensorflow/lite/mutable_op_resolver.h:57: undefined reference to `vtable for tflite::MutableOpResolver'
/home/andrey/data/bokeh/code/bokeh_android_tmp/app/.cxx/cmake/debug/arm64-v8a/../../../../src/main/cpp/include/tensorflow/lite/mutable_op_resolver.h:57: undefined reference to `vtable for tflite::MutableOpResolver'
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

@andreydung can you talk more about your static lib needs? Is there a reason the shared library is insufficient (assuming this is for Android)?

Hi @jdduke ,
I've been following this thread for a while.
One of the reason why I (not sure about others) want to build static lib is that it will have smaller size in terms of built app.
Right now, each arch has the tensorflowlite.so with the size about 1.6MB (armv7) to 2.4 (arm64).
I wonder the static lib will help.

@jdduke I would assume that the need for a static library here is the same basic reason for needing any static library: speed and size optimization. I only want what I absolutely need and I don't want the overhead of dynamic lookups.

I would assume that the need for a static library here is the same basic reason for needing any static library: speed and size optimization. I only want what I absolutely need and I don't want the overhead of dynamic lookups.

The overhead of calls to the shared library should be minimal relative to the cost of running inference (i.e., any marginal overhead in non-inlined calls will be dominated by the cost of executing the model), so I don't think you'd see any observable performance impact between static or shared library usage of TensorFlow Lite (unless you were to use different compiler optimization flags in your library vs the shared library).

I would assume that the need for a static library here is the same basic reason for needing any static library: speed and size optimization. I only want what I absolutely need and I don't want the overhead of dynamic lookups.

The overhead of calls to the shared library should be minimal relative to the cost of running inference (i.e., any marginal overhead in non-inlined calls will be dominated by the cost of executing the model), so I don't think you'd see any observable performance impact between static or shared library usage of TensorFlow Lite (unless you were to use different compiler optimization flags in your library vs the shared library).

I appreciate that the cost is negligible compared to the cost of inference. My point is that it's an unnecessary cost regardless.

@jdduke I would assume that the need for a static library here is the same basic reason for needing any static library: speed and size optimization. I only want what I absolutely need and I don't want the overhead of dynamic lookups.

Another use case is creating one's own shared library which embeds tensowflow inside.

Anyone kowns how to specific it in cmake? Reduce the libtensorflow-lite.a size?


The program just stops. It seems that the problem is with this line "_lock.acquire(timeout)"

Hi all,
After running a few hours, the program just stops. It seems that the problem is with this line "_lock.acquire(timeout)". As shown in "https://github.com/mila-udem/platoon/blob/master/platoon/channel.py#L390", the timeout is "None" as default.
From here (http://semanchuk.com/philip/posix_ipc/), "A timeout of None (the default) implies no time limit. The call will not return until its wait condition is satisfied."
I don't know what the problem is yet.
Anyone has encountered this before?

Everything you said is right but what do yu mean it just stops?
Did you start both the worker and the controller?
Did you redirect the output? Are they running in the background on in different shells?
Are you using the launcher instead?

I am using the launcher. I did not redirect the output.
It runs correctly for a while, sometimes even over 5000 batches (sometimes around 2000 batches.). Then it just stops (the process is gone). There is no error in controller or worker err and out files.
In order to locate where the error happens, I added a few print lines in the code.
def sync_params(self, synchronous=True):
print('actually start sync')
if synchronous:
print('start lock')
self.lock_params()
print('lock done')
print('start update')
self.update_fn(self.shared_params)
print('update done')
if synchronous:
print('start unlock')
self.unlock_params()
print('unlock done')
def lock_params(self, timeout=None):
    print ('start lock acquire')
    test=self._lock.acquire(timeout)
    print(test)
    print ('lock acquired')

I found that when the error happens, the fours worker processes are at the following location.
Two of then just going through "self.unlock_params()". printed "unlock done"
One is start to acquire the lock. printed "start lock acquire"
One is updating. printed "start update"
I am using a machine with 2 CPUs (Intel®Xeon E5-2620 V4) and 4 GPUS (TitanX). The system is Ubuntu server. I tried both python 2.7 and python 3. I also tried using only two gpus, but the error still occurs.
Any advice?
Thanks a lot.

Not sure why yet, but after I changed the training batch input strategy, it can work now.  In the last version, all the subprocesses may access the same data. Now I changed it to different subprocesses access different data.


fixed SwiftLint warnings in date tests

I shut down some annoying linting warnings!

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

Thank you @nathanbacon!


Make the limit of emotion product stream sliders configurable

1. Why is this change necessary?
The number of products which are shown in an emotion product stream slider is limited by the default limit (25), see here:
https://github.com/shopware/shopware/blob/406925fbb6a8e98beb94e71ceca1fd473a6edc31/engine/Shopware/Bundle/EmotionBundle/ComponentHandler/ArticleSliderComponentHandler.php#L225-L231
But one cannot change this limit, although it might be wanted.
2. What does this change do, exactly?
Add the possibility to configure this limit, by showing the necessary configuration fields.
3. Describe each step to reproduce the issue or behaviour.
Try to show a long product stream in a shopping world.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/SW-23779
5. Which documentation changes (if any) need to be made because of this PR?
https://docs.shopware.com/en/shopware-5-en/products-and-categories/product-streams#stream-in-a-shopping-world
maybe this picture should be changed.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-25022
Please use this issue to track the state of your pull request.

Thanks for your Contribution! :blue_heart:


[VS Code] Sharing Terminal: RPC connection closed.

<!--
For Visual Studio problems/feedback, please use the Report

Closing this as a duplicate of #1353.


use_inline_resources breaks this cookbook on chef11 and older chef12 clients

Merging on request...

cool thanks @sbotman
And thank for the style correction comment 😄


For Graphql, For OData in the feature

For now, we need a strong query function， we can use
var db = new ApplicationDbContext();
db.Users.FromSql("query string")
db.FromSql("query string")
But, we need
db.Users.FromOData("expression")
db.FromOData<User>("expression")

db.Users.FromGraphql("expression")
db.FromGraphql<User>("expression")

@seven1986 This request could mean one of two things:

Somebody creates EF Core providers for OData (see #7759) or graphql and then these methods would be the natural equivalent of FromSql. We have no plans at this time to create these providers--see #7759--but others in the community could certainly do so, and we would provide guidance as much as possible.
These are extension methods that take some kind of raw OData or graphql expression and translate it into an expression tree that can be handled by EF. Note that this may be tricky or even prohibitively difficult for non-trivial expressions. We don't plan to implement these, but again others in the community could give it a try. We would not ship these with the core EF packages--they would be better delivered as third-party packages containing extension methods.


[editor] Smart SQL formatter indentation improvements with sql-formatter

Next step would be to publish/pull the one from https://github.com/romainr/sql-formatter
which contains a prettier semi colon indentation.

:+1:


[ASK] Pull from Docker Hub - tls: oversized record

Hi,
I try to pull an image from docker hub and get an error
-> tls: oversized record received with length 20527.
I work behind a corporate proxy and I think that might be in relation, but can't figure out how to parametrize it. I tried to set env variable (HTTP_PROXY, etc) without succes.
So questions are :

Is this error caused by Proxy ?
If so, how can I make this work ?

I work with Docker Toolbox on windows,
Client :
Version 1.11.1.
API 1.23.
Server:
Version 1.11.2
API 1.23
Thanks in advance.

please keep in mind that the github issue tracker is not a general support forum, but for tracking bugs and feature requests; please consider using either one of;

forums.docker,com
the #docker IRC channel on freenode
StackOverflow

I'll close this issue here, because this is not a bug, but feel free to comment after I closed


Repository structure

See if you can do a little cleanup of your repository.  I see an environment.yml like file in the env directory of the module directory.  These are typically stored in the root of the repository.  Also, your tech review PDF should be in the doc directory.  I will write a separate issue for improving your design documents.
I only saw one branch with any real code.  Maybe that is an oversight on my part, but make sure you are using version control and committing and pushing your code regularly.

@dacb we have done some pretty heavy revisions to the repo structure; is there any more work we need to do for the organization or am I good to close this issue?


distribution.md: Broken link

Hi.
The following link is broken: https://developer.microsoft.com/microsoft-edge/webview2
Expected behavior: It is supposed to be the language-agnostic link to the WebView2 page. For example, a German user clicking on it is expected to be taken to https://developer.microsoft.com/de-de/microsoft-edge/webview2/.
Actual behavior: The affected link redirects to: https://developer.microsoft.com/en-US/microsoft-edge/

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

ID: 8a805f15-55ba-4ef4-aeda-22188a040c66
Version Independent ID: 88de0955-526f-a4c9-efcc-d952281850e1
Content: Distribute your app and the WebView2 Runtime - Microsoft Edge Development
Content Source: microsoft-edge/webview2/concepts/distribution.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Thanks @skycommand for filing this. This is a known issue that hasn't been fixed yet unfortunately. We have a dev team working on this at the moment, so hopefully this should be resolved soon.
@mikehoffms thanks!

I confirmed that clicking the link at top of Issue,
https://developer.microsoft.com/microsoft-edge/webview2
brings me to correct deep-nested page per original URL, auto-inserting my own "user's default lang":
https://developer.microsoft.com/en-us/microsoft-edge/webview2/


Add back deletability support for graph.user/photo and graph.group/photo

Since DELETE is now supported for the above resources... we need to revert some changes made via this PR: https://github.com/microsoftgraph/msgraph-metadata/pull/392

Initial issue: https://github.com/microsoftgraph/msgraph-metadata/issues/387

DELETE support announcement: https://github.com/microsoftgraph/msgraph-metadata/issues/387#issuecomment-1764826865

Documentation ref:

Related to: https://github.com/microsoftgraph/msgraph-metadata/issues/576

I notice we still have  challenge on this, for user photo and group photo
This blocks us on: https://github.com/microsoftgraph/msgraph-sdk-dotnet/issues/2247 across SDKs
The API supports this:

updating profile photo
deleting profile photo

Reopening this and wishing it is prioritized as it might be a challenge to users once Exchange Online PowerShell UserPhoto cmdlets are deactivated following this
cc @irvinesunday  @adhiambovivian  @petrhollayms

the conversion lib (Yoko) has never supported DELETE operations for media content paths, like the ones listed above. This might have to be a feature request on Yoko, while also considering that other non graph.profilePhoto /.../$value paths might have the DELETE operations generated as well, unless there exists an appropriate annotation limiting this.
cc: @millicentachieng


Bug in CSG scaling

Bugs

Bug repro on playground

Expected result: I want to have a hollow cylinder. When i'm applying the scaling on the inner mesh it works. But when I apply the scaling on the innerCSG it doesn't work

Current result: Scaling doesn't apply correctly and the CSG subtract themself.

The scaling in CSG is applied to the exported mesh and not to the CSG itself.
if you want to do that (as you already wrote in your playground), you would need to scale the input mesh, or export the CSG to a mesh and create a new CSG.
You will notice that there is also position and rotation values. both are applied to the exported mesh but not the vertex data of the mesh, which is what being used to create the polygons for the CSG.


Iconography additions

[x] wifi
[x] wifi-off
[x] wifi-no-internet
[x] wifi-not-connected
[x] battery versions
[x] heart
[x] more-horizontal (uploaded)
[x] more-vertical (uploaded)
[ ]...

@egunther39
Please review: https://drive.google.com/drive/folders/1GAZzdbx7tU9JRbehaWVzNHbmLt4qBE1Z
Can the wifi dot have a pixel-perfect value, e.g. 4*4 px?

Icons to be added:

[ ] 'view-templates.svg'
[ ] '3D-cube.svg'
[ ]

@egunther39 can this be closed? Address remains, 3D, tables?


Fixes scrolling the navigator to the active item

Bug/issue #, if applicable: 99279409
Summary
Fixes a bug where the navigator does not scroll to the active item, if deeply in to the tree.
Dependencies
NA
Testing
Steps:

Navigate to a deep item in your docs navigator
Assert that when you refresh, i scrolls to the active item

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

Could you briefly explain what was actually going wrong in the code for a deeply nested item just for my own clarification?

this.getChildPositionInScroller(element) returns 0 if element is falsy, because we cannot figure out the position if we dont have an actual element. Now I just added an extra element &&  to make sure it is there.


Will TypeScript definitions be released on npm?

For now we can copy the .d.ts file into our project, but longer term it would be nice to deploy the .d.ts file along with the npm project.  The most recent version on npm (2.2.2) does not include them.
Will they be deployed to npm?
Thanks!

Thanks!

This fix open new issue: https://github.com/dancormier/react-native-swipeout/issues/230


Sema: Teach getReferencedAssociatedTypes() to look through typealiases [3.1]

Now that NameAliasTypes desugar to interface types, it is possible
to have a protocol requirement type contain a NameAliasType which
contains an associated type:
protocol P {
associatedtype Element
typealias Elements = [Element]
func process(elements: Elements)
}
In Swift 3, the typealias would be desugared at name lookup time
in this case, but this is no longer the case, as a result associated
type inference stopped working in this example.
Fixes https://bugs.swift.org/browse/SR-3641.

@swift-ci Please test

@DougGregor Mind reviewing this for 3.1?

Ah, of course. LGTM!


Feature Request: Throw warning when unembedding results in too many chain breaks

Proposed syntax:
resp = unembed_response(response, embedding, bqm, warning_threshold=.1)

In this case a UserWarning will be raised if more than 10% of the chains are broken.

@jking-ca

relates to Add chain stats to response #197

This warning would include some description of strategies to address the problem, possibly a documentation link.

Alternatives:

throw a warning only when 100% of the samples have at least one broken chain
add a new data vector / record field to the resp that says what percentage of chains were broken in each sample.


feat: added onConnectionLost function to WebSocket config

Fixes / Enhancements
onConnectionLost function to WebSocket
Docs

Added documentation about auth headers and onConnectionLost function

Fixes https://github.com/zino-hofmann/graphql-flutter/issues/1323

P.s I will fix version

P.s I will fix version

Run also make dep && make, the rules are equal for everyone. as it is for me that I must push code well-formatted, it is also for you

But his is general change not ment only for tokens :)

ok commits are squashed

Mh why your commit is signed by me? this looks odd

Have no idea.... I just squashed them and that's it. I understand your
awareness about git history but Can we skip all these 'git history'
things... at the end code is most important :)
Meanwhile I have made one more PR branch from yesterday's one (which also
may make git history not the coolest but did that because i needed all
changes).
Would you mind reviewing it and releasing - those really blocks me from
releasing our APP.
Thanks :)
On Mon, 24 Jul 2023 at 13:02, Vincenzo Palazzo @.***>
wrote:

Mh why your commit is signed by me? this looks odd
—
Reply to this email directly, view it on GitHub
https://github.com/zino-hofmann/graphql-flutter/pull/1364#issuecomment-1647695046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF5A2U3YJBEJW4QW7HLXRZI5XANCNFSM6AAAAAA2UOFCPE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Pagarbiai,
Vytautas Pranskunas

Have no idea.... I just squashed them and that's it. I understand your
awareness about git history but Can we skip all these 'git history'
things... at the end code is most important :)

No, because if there is some bugs I need to fix your mess :)

Would you mind reviewing it and releasing - those really blocks me from
releasing our APP.

If the code and the git history is not good enough I will not, sorry!

Ok as i see that your goal is not to maintain this project but to mess with
git. I am not good at git whatsoever and do not have time for it.. Most
likely i will use my own fork to have these features on board because i
even do not now ehre to start fighting with git. Those changes are too
trivial to talk about them for so long. :)
On Mon, 24 Jul 2023 at 13:41, Vincenzo Palazzo @.***>
wrote:

Have no idea.... I just squashed them and that's it. I understand your
awareness about git history but Can we skip all these 'git history'
things... at the end code is most important :)
No, because if there is some bugs I need to fix your mess :)
Would you mind reviewing it and releasing - those really blocks me from
releasing our APP.
If the code and the git history is not good enough I will not, sorry!
—
Reply to this email directly, view it on GitHub
https://github.com/zino-hofmann/graphql-flutter/pull/1364#issuecomment-1647748504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF6Y2EUIBOFLZZH7XHDXRZNPFANCNFSM6AAAAAA2UOFCPE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Pagarbiai,
Vytautas Pranskunas

p.s. i can do it in only way - if you write me commands that i have to
write because fighting with git for me not acceptable at this point in time
On Mon, 24 Jul 2023 at 13:51, Vytautas Pranskūnas @.***>
wrote:

Ok as i see that your goal is not to maintain this project but to mess
with git. I am not good at git whatsoever and do not have time for it..
Most likely i will use my own fork to have these features on board because
i even do not now ehre to start fighting with git. Those changes are too
trivial to talk about them for so long. :)
On Mon, 24 Jul 2023 at 13:41, Vincenzo Palazzo @.***>
wrote:

Have no idea.... I just squashed them and that's it. I understand your
awareness about git history but Can we skip all these 'git history'
things... at the end code is most important :)
No, because if there is some bugs I need to fix your mess :)
Would you mind reviewing it and releasing - those really blocks me from
releasing our APP.
If the code and the git history is not good enough I will not, sorry!
—
Reply to this email directly, view it on GitHub
https://github.com/zino-hofmann/graphql-flutter/pull/1364#issuecomment-1647748504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF6Y2EUIBOFLZZH7XHDXRZNPFANCNFSM6AAAAAA2UOFCPE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Pagarbiai,
Vytautas Pranskunas

--
Pagarbiai,
Vytautas Pranskunas

so Vincenzo,
Either please help me to put those features to package that you
maintain either you can transfer them manually and do with git history all
things that you like :)
What would you say?
Cheers
Vytautas
On Mon, 24 Jul 2023 at 13:56, Vytautas Pranskūnas @.***>
wrote:

p.s. i can do it in only way - if you write me commands that i have to
write because fighting with git for me not acceptable at this point in time
On Mon, 24 Jul 2023 at 13:51, Vytautas Pranskūnas @.***>
wrote:

Ok as i see that your goal is not to maintain this project but to mess
with git. I am not good at git whatsoever and do not have time for it..
Most likely i will use my own fork to have these features on board because
i even do not now ehre to start fighting with git. Those changes are too
trivial to talk about them for so long. :)
On Mon, 24 Jul 2023 at 13:41, Vincenzo Palazzo @.***>
wrote:

Have no idea.... I just squashed them and that's it. I understand your
awareness about git history but Can we skip all these 'git history'
things... at the end code is most important :)
No, because if there is some bugs I need to fix your mess :)
Would you mind reviewing it and releasing - those really blocks me from
releasing our APP.
If the code and the git history is not good enough I will not, sorry!
—
Reply to this email directly, view it on GitHub
https://github.com/zino-hofmann/graphql-flutter/pull/1364#issuecomment-1647748504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF6Y2EUIBOFLZZH7XHDXRZNPFANCNFSM6AAAAAA2UOFCPE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Pagarbiai,
Vytautas Pranskunas

--
Pagarbiai,
Vytautas Pranskunas

--
Pagarbiai,
Vytautas Pranskunas


Conversation

Traditional Chinese section , starting form translating current English corpus.

@Kennyl For long term maintenance of corpus data, we moved corpus into separate repository, https://github.com/gunthercox/chatterbot-corpus, could you please submit pull request over there?
Still some minor work is in progress

Ok


Request: Add Option to Display Block Height and Other Bitcoin Statistics

The ability to cycle through other Bitcoin metrics in addition to the price would be great. Ideally, a user could choose from a list of various stats and have it cycle through the selected data points.
Metric examples would be like on Clark Moody's Dashboard  or Mempool.space

Hi @corbimos , at the moment there are no plans to extend the platform to support the metrics you mentioned.


Fix Error Message for Optional Fields

Codecov Report
Base: 70.44% // Head: 70.44% // No change to project coverage :thumbsup:

Coverage data is based on head (1a64d1a) compared to base (5d9100e).
Patch coverage: 20.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #250   +/-   ##
=========================================
  Coverage     70.44%   70.44%
  Complexity      610      610
=========================================
  Files            97       97
  Lines          2003     2003
  Branches        236      236
=========================================
  Hits           1411     1411
  Misses          496      496
  Partials         96       96

Impacted Files
Coverage Δ

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

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

...java/seedu/address/logic/commands/FindCommand.java
25.00% <0.00%> (ø)

...eedu/address/logic/commands/FindPersonCommand.java
100.00% <ø> (ø)

...eedu/address/logic/commands/FindModuleCommand.java
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.