Add SNC_GENERATE_{MACOS,WINDOWS}_BUNDLE

On most arches, it makes no sense to generate macos and/or Windows bundles.
This commit adds SNC_GENERATE_{MACOS,WINDOWS}_BUNDLE variables which will
only be set x86_64.

/hold
At this point it's a proof of concept I'd like to run through CI/show to @mtarsel

snc.sh runs successfully on ppc64le with this patch. Thanks for sharing, this logic is cleaner :)
When executing the createdisk.sh I got this error...
+ ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i id_ecdsa_crc<EMAIL_ADDRESS>-- 'sudo rm -f /var/lib/kubelet/config.json'
Warning: Permanently added 'api.crc.testing,<IP_ADDRESS>' (ECDSA) to the list of known hosts.
+ ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i id_ecdsa_crc<EMAIL_ADDRESS>-- 'sudo find /var/log/ -iname "*.log" -exec rm -f {} \;'
Warning: Permanently added 'api.crc.testing,<IP_ADDRESS>' (ECDSA) to the list of known hosts.
+ [[ none != \n\o\n\e ]]
+ mkdir crc-tmp-install-data//packages
+ sudo yum install -y --downloadonly --downloaddir crc-tmp-install-data//packages hyperv-daemons
Last metadata expiration check: -1 day, 2:31:51 ago on Wed Feb 10 13:47:58 2021.
No match for argument: hyperv-daemons
Error: Unable to find a match: hyperv-daemons
[root@pkvmci824 snc]#

which I pushed in a sloppy PR previously here: https://github.com/code-ready/snc/pull/323/commits/869a2f44135860dfe12aa93acdac0ca23c6ad861
So something like this should be added
if [ -n "${SNC_GENERATE_WINDOWS_BUNDLE}" ]; then
    prepare_hyperV()

which will handle that OKD_VERSION check and the hyperV v-sock support.

So something like this should be added
if [ -n "${SNC_GENERATE_WINDOWS_BUNDLE}" ]; then
    prepare_hyperV()

which will handle that OKD_VERSION check and the hyperV v-sock support.

Indeed, thanks, I've taken your preexisting patch for this, and slightly tweaked it to give proper credit for this change. I can squash it in the initial commit if you prefer.

/unhold

Great - thank you very much for pushing that. This patch works on ppc64le

oh and thanks a lot for giving me credit in the patch too. I don't have a preference if it's squashed or not. Going forward, I'll use this approach for ppc64le support so I'm sure there will be more patches :)


New package: SimpleGA v0.1.0

Registering package: SimpleGA
Repository: https://github.com/MonumoLtd/SimpleGA.jl
Created by: @chrisjldoran
Version: v0.1.0
Commit: 0f1acff4ca4bbaa90949881d704365bf967aa53e
Reviewed by: @chrisjldoran
Reference: https://github.com/MonumoLtd/SimpleGA.jl/issues/12
Description: Geometric algebra in Julia.

Can you add some text to the README (seems like the docs are pretty good)? Happy to merge then.

I have added text to the README file. Not sure how this slipped past.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4369

Questions ?
Please refer to the Backport tool documentation


Improve access of __annotations__

Changes
This is a continuation of #216, but with signed, verified commits.
Tests
One test was added (in #216).  The last run in #216 passed all tests.

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

@nfx GitHub was complaining about unsigned commits so I resynced my fork, which closed #216.

@nfx Ok; the line endings had indeed been modified.  I have now rebased with auto handling of line endings and the diff is showing a relatively small difference.

Codecov Report
Patch coverage: 50.00% and project coverage change: +0.06 :tada:

Comparison is base (6f93548) 53.24% compared to head (c5a6867) 53.31%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
+ Coverage   53.24%   53.31%   +0.06%
==========================================
  Files          32       32
  Lines       19151    19197      +46
==========================================
+ Hits        10197    10234      +37
- Misses       8954     8963       +9

Impacted Files
Coverage Δ

databricks/sdk/core.py
67.24% <50.00%> (-0.15%)
:arrow_down:

... and 8 files with indirect coverage changes

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


Revert "global serializer cache"

Reverts AltSchool/dynamic-rest#72

@aleontiev - Can we merge this? I realized global serializer caching just isn't going to work because context is dynamic, but DRF caches it at the instance level. So when serializers are re-used across requests, serializers could be using the wrong cached context.

:+1:, sorry about the delay


Question: Can the version of go be update to 1.17

This will address:
CVE-2021-38297
CVE-2022-23806

@timflannagan

This likely gets tackled for free in #264

@timflannagan It's still showing 1.16 for go
https://github.com/openshift/operator-framework-olm/blob/9f98954e6adec65c4b820742cf2f49238ae21197/go.mod#L3

Yep - looks like the root module still needs to be bumped. cc @perdasilva


layer._extent_data inconsistent between image and points/shapes/tracks

🐛 Bug
The values for _extent_data are calculated differently for the image layer and points/shapes/tracks. In the image layer, the maximum value for the extent is the maximum index + 1 (i.e., the result of data.shape) and in the points/shapes/tracks layer it is the maximum index (i.e., the results of np.max(data). It appears that our Dims model assumes the maximum value of the extent is maximum_index + 1 so the range of the slider is incorrectly calculated for the points/shapes/tracks layers (it is 1 step less).
Is layer._extent_data defined as (min_index, max_index + 1 ), where min_index and max_index are the layer data array indices?
To Reproduce

create 3D data with indices from 0 through 4 for axis 0 for image, points, shapes, and tracks layers

view those data in napari in 2D mode (axis slider is the 0th axis). Note that the axis slider correctly goes from 0-4

view the layer._extent_data property for each layer

viewer.layers['image']._extent_data
Out[1]:
array([[   0.,    0.,    0.],
       [   5., 1000., 1000.]])

viewer.layers['points']._extent_data
Out[2]:
array([[   0,  236,    0],
       [   4,  636, 1000]])

viewer.layers['tracks']._extent_data
Out[3]:
array([[   0,  236,    0],
       [   4,  636, 1000]])
viewer.layers['shapes']._extent_data
Out[4]:
array([[ 0., 10., 10.],
       [ 4., 20., 20.]])

If we  delete the image layer, the axis slider now goes from 0 - 3.

code

test_im = np.random.random((5, 1000, 1000))

tracks_data = np.asarray([
    [1, 0, 236, 0],
    [1, 1, 236, 100],
    [1, 2, 236, 200],
    [1, 3, 236, 500],
    [1, 4, 236, 1000],
    [2, 0, 436, 0],
    [2, 1, 436, 100],
    [2, 2, 436, 200],
    [2, 3, 436, 500],
    [2, 4, 436, 1000],
    [3, 0, 636, 0],
    [3, 1, 636, 100],
    [3, 2, 636, 200],
    [3, 3, 636, 500],
    [3, 4, 636, 1000]
])

shapes_data = [
    [
        [0, 10, 10],
        [0, 10, 20],
        [0, 20, 20],
        [0, 20, 10]
    ],
    [
        [4, 10, 10],
        [4, 10, 20],
        [4, 20, 20],
        [4, 20, 10]
    ]
]

with napari.gui_qt():
    viewer = napari.view_image(test_im, name='image')
    viewer.add_tracks(tracks_data, name='tracks')
    viewer.add_points(tracks_data[:, 1:], name='points')
    viewer.add_shapes(shapes_data, name='shapes')

Expected behavior
The values for the 0th dimension in layer._extent_data should be the same for the image
Environment
napari: 0.3.8.dev14+g504ef2f
Platform: Linux-5.4.0-48-generic-x86_64-with-debian-bullseye-sid
Python: 3.7.7 (default, May 7 2020, 21:25:33) [GCC 7.3.0]
Qt: 5.14.2
PyQt5: 5.14.2
NumPy: 1.19.1
SciPy: 1.5.2
Dask: 2.24.0
VisPy: 0.6.4

GL version: 4.6.0 NVIDIA 450.66
MAX_TEXTURE_SIZE: 32768

Plugins:
- napari_plugin_engine: 0.1.7
- ome_zarr: 0.0.14
- svg: 0.1.4

Additional context
This is related to https://github.com/napari/napari/issues/922

The values for the 0th dimension in layer._extent_data should be the same for the image

Thanks for this write up @kevinyamauchi, I will probably need to ponder this one more, and am curious what @jni thinks, but am I write in thinking that this can be fixed by dropping all the +1 but making sure that any rounding happening on the sliders always makes sure to include the last slice for dims?

I'm not sure about the fix, but the reason that it's different for images vs points et al is that images use 0-based half-open indexing, so we wanted to be able to read the dims extent from the layer extent. There are no such concerns with vector quantities, but then the dims slicing needs to know that the extent of a points layer and the extent of an image layer might mean different things. And of course this all becomes a bit hairy once you add a scale to the image.
I guess it might make sense to adopt a convention that extent is not half-open, even for images.

that it's different for images vs points et al is that images use 0-based half-open indexing, so we wanted to be able to read the dims extent from the layer extent

Ah yes, good point, especially now with #1660 merged that makes sense to me

Thanks for the quick responses, @sofroniewn and @jni!

the reason that it's different for images vs points et al is that images use 0-based half-open indexing, so we wanted to be able to read the dims extent from the layer extent.

Got it - the rationale makes sense to me.

I guess it might make sense to adopt a convention that extent is not half-open, even for images.

At the moment, I'm not sure if I have a strong preference if the extents should all be half-open intervals or all be closed intervals, but I agree that they should all be the same (i.e., don't do special casing on the dims).
It looks like to make the change we would have to:

Update the layers so the extents are all the closed interval. I think this would only be the Image and Labels layer.
Update the Dims.nsteps calculation.
Update the docs

Am I missing anything? Does that seem right?

Am I missing anything? Does that seem right?

I think that is right, it looks like we just need to add a -1 to the image shape here https://github.com/napari/napari/blob/master/napari/layers/image/image.py#L332, right? (labels inherits)
I think that will be what we need.
Consider:
with napari.gui_qt():
    viewer = napari.Viewer()
    viewer.add_image(np.random.random((11, 11)), colormap='red', opacity=.5, name='half')
    viewer.add_points([[0, 0], [10, 10]], size=0.5, opacity=.5)

Both of these layers should have an extent of 0 to 10, which means doing a -1 on the shape for the Image layer. Looks like we already have a +1 on nsteps, will we need another one? https://github.com/napari/napari/blob/master/napari/components/dims.py#L107

@kevinyamauchi I have created a PR to fix this in #1696


[master] Upgrade to latest dependencies

Cron. -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced via:
./hack/update-deps.sh --upgrade --release v0.21
./hack/update-codegen.sh

Details:
bumping knative.dev/eventing 81ce898...ea452b5:
  > ea452b5 go 1.15 in go.mod, drop k8s 1.17 e2e tests (# 4795)
  > ce099cf new cloudevent extension attribute with dispatch error (# 4760)
  > 83ecc7c Set failure policy to ignore for webhook (# 4793)
  > ae17ca3 Adding HPA for eventing-webhook (# 4792)
bumping knative.dev/reconciler-test c46f01f...fa758f8:
  > fa758f8 Update common github actions (# 75)
  > 57ec0b5 upgrade to latest dependencies (# 76)

/lgtm
/approve

/lgtm
/approve


distance_to for plain old Ruby object

In a Rails project I am working on I have a class that is not an Active Model but does include a latitude and longitude. (It fetches data from an API and makes it available as if it were an Active Model). I would like to use distance_to with this class and I have found that I can do it with:
require 'geocoder/stores/base'

class Branch
  include Geocoder::Store::Base

  attr_reader :lat, :long

  def initialize
    @lat = 55.8642
    @long = -4.2518
  end

  def self.geocoder_options
    {
      latitude: :lat,
      longitude: :long
    }
  end
end

branch = Branch.new
puts branch.distance_to('London')

However, this isn't documented anywhere (or I am missing it) so I am wondering if this is a horrible hack and there is a correct way to do it. Thanks.

A colleague pointed out that I could use Geocoder::Calculations.distance_between to create my own distance_to method instead of including Geocoder::Store::Base. I guess that is the right way to do it.


Fix env var about GRATAN_DB_DATABASE and GRATAN_DB_USERNAME

Fix a bug about specifying MySQL database using env variable
Fix README about how to use GRATAN_DB_USERNAME env variable
https://github.com/codenize-tools/gratan/blob/10d55bcebe096360f1462fc1187442489f85aac8/bin/gratan#L22

Hi, @winebarrel :smiley:
This PR is simple bugfix. Can you check it? 🙏


Change Zookeeper Cluster CRD Spec Labels

Zookeeper cluster CRD spec.pod.labels presently does not function as intended. StatefulSet pods receive all their labels from spec.labels. This is outlined in https://github.com/pravega/zookeeper-operator/issues/511.
This change makes it so that any labels which are provided in the pod labels are merged into the spec.labels of the Zookeeper cluster CRD. This also prioritizes any labels provided by the Solr Operator, thus not allowing you to override any important labels like 'app' or 'release' when using the spec.pod.labels in the SolrCloud CRD.
This PR addresses https://github.com/apache/solr-operator/issues/490

Thanks again!


width <= height crop

Is it possible to allow the crop area only with width <= height?

width <= height
↓
width/height <= 1
↓
aspectRatio <= 1


Upgrade weston to 1.7.0

Hello,
This commit upgrades weston to 1.7.0, also changes the remote back to git.baserock.org

I did a rebase on 1.7.0 locally (I hosted this repo here because Baserock's git was not responding at some point), but we can probably switch back to your version now. Thanks!


fix: Ensure correct protocolVersion is retrieved

For InferenceServices using the old predictor schema, no
matching runtime is found because GetProtocol generally
returns a protocol even if none is specified in the spec.
This adjusts the logic a bit to pull the ProtocolVersion from the actual spec.
Fixes: #173

Verified with Kserve including the webhook fix.

/lgtm


Expand/Collapse bug in code edit area

Bug in code edit area with function Expand / Collapse code .
This bug continuously from 1.5 to 1.6 version. in one moment after preset on "+" for expand code , "+" not change to "v" and after press to "+" don't have effect. This expand block of code don't collapse and don't have inner expand and collapse. after im press F5 and all work perfectly, but in one moment bug return.
This problem affects a certain part of the code, I can not collapse the code above a certain place, below this point everything is working perfectly
ps : I'm edit js/node code, if this information help

Hey @FreezeLogic,
could you most some minimal amounts of code that show this behavior?

Yes i have same problem too

Very likely fixed by https://github.com/adobe/brackets/pull/12521 which will be in 1.8.


perf: Speed up proto unmarshaling using vtprotobuf plugin

Description of change
Speed up marshaling / unmarshaling of check(http) requests & responses by using
optimized methods generated by vtprotobuf plugin and some use of pooling.
Results: ~6% speedup (total (un)marshaling: ~15% → ~8%), 10% reduction in allocs.
Drive-by: Expose heap profiler endpoint.
Resolves https://github.com/fluxninja/aperture/issues/2212
Checklist

[x] Tested in playground or other setup
[ ] Screenshot (Grafana) from playground added to PR for 15+ minute run


Rdk 34348 trace control plugin document update

Documentation enhancement for TraceControl Plugin.
Jira link: https://ccp.sys.comcast.net/browse/RDK-34348

@kabildev634 - Pls check and update. This is pulling other source changes.


[FLINK-8283] [kafka] Stabalize FlinkKafkaConsumerBaseTest::testScaleUp()

What is the purpose of the change
This PR sits upon other FlinkKafkaConsumerBaseTest changes from #5188 and #5200 .
Only the last commit d53409c is relevant.
As it seems, the previous parameters for testScaleUp() was too aggressive and took up too many resources, which cause the test resource to be terminated before the test could finish.
This PR adjust the testScaleUp() arguments by testing the same behaviour, while consuming less resources.
Previous settings were: initialPartitions: 5 -> scale to 15, with 1000 Kafka partitions on restore
Changed to: initialPartitions: 5 -> scale to 8, with 30 Kafka partitions on restore
Brief change log

Modify test arguments for testScaleUp()

Verifying this change
The testScaleUp() should still work fine.
I'm also running several Travis iterations of the tests on my local branches, to see if the instability persists.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? n/a

The following 10 local Travis runs suggests that the stalling tests on the connectors build no longer remains after this change:

https://travis-ci.org/tzulitai/flink/builds/319936358
https://travis-ci.org/tzulitai/flink/builds/319936375
https://travis-ci.org/tzulitai/flink/builds/319936388
https://travis-ci.org/tzulitai/flink/builds/319936405
https://travis-ci.org/tzulitai/flink/builds/319936419
https://travis-ci.org/tzulitai/flink/builds/319936440
https://travis-ci.org/tzulitai/flink/builds/319936454
https://travis-ci.org/tzulitai/flink/builds/319936469
https://travis-ci.org/tzulitai/flink/builds/319936480
https://travis-ci.org/tzulitai/flink/builds/319936556


Zoom limits

Hi folks,
(if the case was already please send me there, thanks)
Are any limits for the zooming of the map in Mapsui?
What I have here now is max zoom level 19 (Scale = 1:1065), and I need 24 (1:33).
I tried:
Map.ZoomLimits = new MinMax(1, 24);
but, this gives me some weird view
What should be in MinMax? (documentation pleeeaaaaassssse ;) )
I tried also:
MinMax(-100, 20000)
MinMax(-1000, 50000)
etc.
but still max zoom is too small (is not enough close, not enough details I can see on the map).
(in Map.ZoomLimits  Max is MinZOOM and Min is MaxZOOM I think :| )
I'm not sure if this what I can see in GeoServer (OSM maps) is true or graphic fake zoom. In GS I can zoom till scale = 1:2.
And how about ZoomMode in that issue?
Thanks for the help!
Łukasz
P.S.
Level values based on ThinkGeo zoom levels, Scale from GeoServer

About min and max. These refer to the values used. Since these values are in units-by-pixel and not pixel-by-unit this could be counter intuitive.
As @charlenni said, the baselayer determines the number of levels. So if you use a baselayer with more levels this is solved.
You can create a baselayer with more zoomlevel through the constructor of the schema. The 'GlobalSphericalMercator(0, 22)' below.
        private static HttpTileSource CreateTileSource()
        {
            return new HttpTileSource(new GlobalSphericalMercator(0, 22),
                "http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png",
                new[] { "a", "b", "c" }, name: "OpenStreetMap",
                attribution: OpenStreetMapAttribution);
        }

In this sample the tilelayer will try to do requests to levels that do not exist, which is awkward (but works for the end user), but if you have a tile source with more levels this will have the proper effect.

If you use a proper wmts you can initialize the baselayer with the metadata and you don't have to do anything special for it. I will send you a sample.

🥇
WMTS is the solution.
Thanks!


Revamp/login page

Proposed Changes

new login-page view - compatible with design,
made a fluid layout so on smaller devices page is still readable
added bootstrap to styles and used grid layout
added logo to assets
the user is now able to see a password on login and register
login page starts now with sign up form

Tablet:

Iphone:

Please add some screenshots to the Pull Request message :)

Done :+1: Forgot about Lorem Ipsum text, so will need to update this. Plus validation works, but there are some issues with visibility of validation. I think this issue will go as another branch ;)


Introduce SharedBuf trait for Bytes VTable

The goal of this PR is to continue the work of PR #567.  This updates the code to be more conformant to the idioms of Rust and the Tokio project.
The work in #567 introduces a new trait which is used as a contract for internal and external structs to make themselves usable as the internal buffer for Bytes.
This should solve the current issues that request a mechanism for allowing a Bytes struct to own arbitrary 3rd party buffers.  see:  https://github.com/tokio-rs/bytes/issues/437 https://github.com/tokio-rs/bytes/issues/526 https://github.com/tokio-rs/bytes/issues/571
This would allow an arbitrary buffer to be converted into a Bytes buffer (using the new Bytes::from_shared_buf method) and later be converted back to the original type, consuming the buffer (using Bytes::into_shared_buf method)
An example of this can be found here
Specific changes from #567:

Renames the trait BytesImpl into SharedBuf.
Modularizes the implementations of SharedBuf and moves them into their own module.
Changes some types to ensure that theSharedBuf trait can be implemented by 3rd party devs.
Adds an example "3rd party" SharedBuf implementation with tests.

Due to re-organizing the SharedBuf implementations, this makes things a bit harder to diff,  but the functionality of Bytes itself is almost completely unchanged.  All tests pass, and the benchmarks are equivalent within a reasonable deviation (using cargo benchcmp)
For more precise deltas between the code, you can look at the commit progression in #567

change to_vec to into_vec and takes &mut AtomicPtr<()> instead of &At…
add will_truncate to Bytes vtable to prevent special casing promotabl…
add Bytes::with_impl and related trait
migrate existing vtable code to BytesImpl trait

I've addressed the formatting, and will address the loom issue.  The miri issue is an interesting one, I'm having trouble divining the origin of that one.

Thanks for the PR. We will be reviewing it soon. Given the size, it will take a moment to get through it.


Bug: Please remove reference to Maui.Essentials

https://github.com/Azure/azure-mobile-apps/blob/1ed7821ac0dc2267a7a3c0a519027d266d78847e/sdk/dotnet/src/Microsoft.Datasync.Client/Microsoft.Datasync.Client.csproj#L30
This shouldn't be required in a cross-platform library and forces maui.essentials as a dependency on any application targeting net6-X

Running a quick test here if you remove that line and remove Xamarin.Essentials as a dependency from the classic Xamarin targets it still compiles just fine which tells me it's not actually used and should be removed as a dependency.
That said this really highlights an issue with the dependency chain where when installing the Datasync Client you would never know that you "need" UseMauiEssentials as it does not get passed through to the generated NuGet into the consuming project.
cc: @davidortinau @Redth


Retrieve field values used for interpolation

Hello,
For the purpose of understanding dispersal differences obtained between Parcels and another Lagrangian model, I would like to retrieve the different velocity values that parcels uses for its interpolation scheme.
More precisely, as explained in the reference paper https://gmd.copernicus.org/articles/12/3571/2019/gmd-12-3571-2019.html, I am looking for the values that are read on a C-grid at each time step, to compute the fluxes and interpolate u,v,w at the particle position.
I tried to look for the appropriate function that does the interpolation in the source code but I am a bit loss between the different files and functions. Could you highlight for me where those values are read in the code ?
Thanks a lot,
Hugo

You don't need to dig into the source code, you can just use code from the AdvectionRK4 method (make sure your ParticleType has custom Variables uvel and vvel.
(u, v) = fieldset.UV[time, particle.depth, particle.lat, particle.lon]
particle.uvel = u
particle.vvel = v

If the interp_method is set to cgrid_interpolation, then this will yield exactly the right values, see also the interpolation tutorial.

Yes, I have used this option, the thing is I don't only want the values that are interpolated, but also the values that are read on the grid and used in the computation of the interpolation

In that case, have a look at the AnalyticalAdvection kernel (and its tutorial here), where exactly this is done:

the grid indices are found on lines 127-128
the velocities (actually fluxes) are computed on lines 187-290

Hope this helps

Thank you very much !

How would you manage to print values from a certain kernel to the console or to a file ?
For instance if I try to retrieve u1 from Advection_RK4, none of print or sys.stdout methods work. I get the type of following error :
Cannot convert 'sys' used in kernel to C-code
From what I see in the documentation, this comes from the fact that kernels are translated into C and therefore, sys or other methods can't work properly.
Is there an alternative I could use to retrieve values of interest that are computed in a certain kernel, that would be compatible with parcels ?
Many thanks for your tremendous help, hope you are not too bored of my requests !
Hugo

You can use print statements in Kernels directly, see e.g. these test kernels
Alternatively, you can always use ScipyParticles instead of JITParticles to circumvent running in C. This will allow you to use whatever python code you want


Code Smell: components/moral

Description
Task: Code Smell: components/moral.
Feature: [Code Smells] (#455)
Fix all 68 code smells located in the main game folder, specifically the /game/components/moral/ folder.
Fix all 40 code smells located in the test game folder, specifically the /game/components/moral/ folder.
Dependencies
N/A
Milestones
List of steps that need to be completed for this task.

[x] MoralConsequences.java (Oct 10)
[x] MoralDayFour.java (Oct 10)
[x] MoralDayOne.java (Oct 10)
[ ] MoralDayThree.java (Oct 10)
[ ] MoralDayTwo.java (Oct 10)
[ ] MoralDecision.java (Oct 10)

Test Files:

[ ] MoralDayFourTest.java (oct 10)
[ ] MoralDayOneTest.java (oct 10)
[ ] MoralDayTwoTest.java (oct 10)
[ ] MoralDecisionTest.java (oct 10)

Completion Deadline: Oct. 10
Documentation

...

Member

Caleb Murphy (@Caleb-J-Murphy)

Completed in a seperate ticket


Zoom Issues

So when using the plugin with default settings, it will fit the whole image, and not zoom the large image:

Original images:

If anyone knows how to fix this, it's working fine on some larger images:

Maybe is a CSS problem?


Run fetch operations threaded per reader

Improves the execution speed of queries with > 1 shard per node.
For example:
250 iterations
select * from uservisits limit 1000
Before: mean: 281.639 +/- 60.773
After:  mean: 139.511 +/- 17.454

Note that this was measured on an otherwise idle node. If all CPU cores
are busy or mostly busy this change won't be as noticeable.

@dobe please review

changelog entry about that fetching is now multithreaded etc
otherwise ack

ack


css for audio button

There are limited css changes that can take effect for the audio button, e.g. not all colors can take effect.

This was a great start--superseded by #52


unsupportedURICharacter(60) error when running on Linux

I have a simple websocket client that uses Vapor Engine. The client connects to a wss:// endpoint.
The client works fine on macOS.
However, when I compile and run it on Ubuntu, it doesn't connect and I get the following cryptic error:
unsupportedURICharacter(60)
Any ideas what might be the problem?

@gmosx no idea off the top of my head, any more info or context about what exactly is going on here or a way to possibly recreate it? If we could make a unit test to find it, that would definitely help us start narrowing down.

Feel free to re-open when we get more info.


cesium from vscode

Why is there no intelligent prompt for the relevant APIs in cesium in vscode? It's a bit tiring to manually complete a long string of words every time. Do you have any good suggestions

Using typescript!

I don't quite understand. Do you mean using TS for code writing and development? Or what kind of TS file is imported?

There are hints that I use typescript for development

我去，大哥，我刚开始学cesium，从最基础的html引入一个cesium.js文件来进行展示，但是vscode不提示拼全，太烦了呜呜

你需要一个typescript项目，配合cesium开发会很nice! typescript不光是针对cesium,而是针对所有的GIS库，各种GIS库API类型出参入参都得去翻文档，开发速度很缓慢。如果是初学没关系，后期实际项目建议使用

好的，蟹蟹哥，方便留个vx给小弟吗

CesiumJS is a big project, so we use GitHub for feature requests and bug tracking exclusively. In the future, please take any questions to the Cesium Forum where there are members of the community and developers from the team who can help. Thanks!


dynamic allocation error even though spark.shuffle.service.enabled = true

Problem:
I want to connect to a spark standalone cluster using dynamic allocation.
According to this page, dynamic resource allocation is "disabled by default and available on all coarse-grained cluster managers, i.e. standalone mode, YARN mode, and Mesos coarse-grained mode." The page goes on, "In standalone mode, simply start your workers with spark.shuffle.service.enabled set to true."
This does not appear to work with sparklyr.
Setup:
Running spark 2.4.3 using a docker swarm cluster. It is a standalone cluster, at the moment using two worker containers and one master container. Using rstudio server Version 1.2.1335 in a docker swarm service based on rocker/tidyverse:latest.
I can connect to the spark cluster using rstudio and sparklyr 2.4.3. This works using:
spark_connect(master = "spark://master:7077"
But, I cannot connect to the spark standalone cluster from rstudio using sparklyr with: config$spark.dynamicAllocation.enabled <- "true".
Throws an error:
ERROR SparkContext: Error initializing SparkContext.
org.apache.spark.SparkException: Dynamic allocation of executors requires the external shuffle service. You may enable this through spark.shuffle.service.enabled.

Now, I did enable shuffle service in the spark docker containers, by adding to the spark-defaults.conf file:
spark.shuffle.service.enabled           true
And I suspect that worked, because when I look at the docker logs for the workers, I see:
INFO ExternalShuffleService: Starting shuffle service on port 7337 (auth enabled = false)
Has anyone gotten dynamic allocation to work using sparklyr and a standalone cluster? Not sure what I should do to narrow down the problem from here. Suggestions welcome. Thanks!

Found the solution: Include this in the config passed to spark_connect:
config$spark.shuffle.service.enabled <- "true"
Apparently, including config$spark.dynamicAllocation.enabled is not enough. The documentation makes it sound like spark.dynamicAllocation.enabled = true is something that should only be set on the workers, prior to using spark_connect.


Technical specifications of Cannon's engine?

Hey Schteppe,
I'm writing a report and doing a presentation about my project so far (the one I've been asking all these questions for), and I figured it'd be good to know at least a tiny bit about cannon's back-end.
If you could just give me keywords, or direct me to wikipedia articles, that'd be great. I've read very briefly about penalty/constraint/impulse based methods.
Cheers :)

All right!
Cannon was originally made as a lab at this course at Umeå University: http://www8.cs.umu.se/kurser/5DV058/VT15/index.html
Cannon is a rigid body physics engine. It uses the SPOOK stepper, and its equation is derived from verlet. Constraints are formulated using Jacobians, and to solve them, a built in Gauss-Seidel solver is used. Friction is based on a scaled box model.
A key document is this one: http://www8.cs.umu.se/kurser/5DV058/VT15/lectures/SPOOKlabnotes.pdf
Regarding the SPOOK stepper stuff, please refer to "Ghosts and Machines" by Claude Lacoursière. See the bibliography section in the pdf above.
Let me know if you need something more. And good luck with the report!

Great - this is perfect, I'll get reading!
Thanks!


remove issue mention from create RP, issue is solved

URL for relevant page?
http://docs.influxdata.com/influxdb/v1.0/query_language/database_management/#create-retention-policies-with-create-retention-policy
https://github.com/influxdata/influxdb/issues/3634 has been closed, so the text needs updating (although the fix should be confirmed, first)
What products and version are you using?
Where did you look before opening the issue?

I believe this is a dup of https://github.com/influxdata/docs.influxdata.com/issues/573, and it's fixed in 1.1.0: https://github.com/influxdata/influxdb/pull/6597.


Fix for 2 vulnerable dependency paths

clone-org-repos currently has a 3 vulnerable dependency paths, introducing 3 different types of known vulnerabilities.
This PR fixes vulnerable dependencies.

ReDOS vulnerability in the hawk dependency.
remote memory exposure  vulnerability in the request dependency.

You can see Snyk test report of this project for details.
This PR changes Package.json to upgrade request to the newer 2.74.0 version, and will fix the vulnerability listed above.
You can get alerts and fix PRs for future vulnerabilities for free by watching this repo with Snyk.
Note this PR fixes all the vulnerabilities introduced trough request dependency, in order to be vulnerability free you will need to upgrade cli dependency as well.
Stay Secure,
The Snyk Team

Coverage remained the same at 80.916% when pulling 734508bb45baa3cc636a6fa6a69b481c6510fc94 on snyk-community:snyk-community-patch-1 into e648a09348cda1c91e049c4feffce171d0cc09ce on tegon:master.


crashes 1.19.4

https://mclo.gs/X9SMmtY
just dies, plz fix I need this for my modpack

Hi, Featured Servers 1.19.2-1.6 doesn't support 1.19.4.
I've uploaded a new release on modrith: https://modrinth.com/mod/featuredserversfabric/version/CvyfYeu6


[ETHOSN] Minor corner case fixes

Minor issues in corner cases from static analysis and a code standard
violation fix.
@lhutton1

Thanks @Leo-arm!


Unable to use avr-gcc

I'm unable to build the following trivial AVR example:
## main.c
#include <avr/sleep.h>

int main()
{
    sleep_mode();
}

On other Linux distros, after installing the AVR dependencies (e.g. apt-get install build-essential gcc-avr binutils-avr avr-libc), the source can simply be built with avr-gcc -mmcu=atmega8 main.c.
On NixOS, the following fails because of missing include and library paths:
nix-shell -p avrgcc avrlibc --run 'avr-gcc -mmcu=atmega8 main.c'

After manually setting the extra paths,
nix-shell -p avrgcc avrlibc --run '
  libc=$(nix-build --no-out-link "<nixpkgs>" -A avrlibc)
  avr-gcc -mmcu=atmega8 -I ${libc}/avr/include -L${libc}/avr/lib/avr4 main.c
'

I'm still getting an error:
/nix/store/22ds38v5pj07wg21gargk3pbrm02zicy-avr-binutils-2.31.1/bin/avr-ld: cannot find crtatmega8.o: No such file or directory

Note that crtatmega8.o exists in the libpath: ls ${libc}/avr/lib/avr4/crtatmega8.o
How can I build this example?
Would it be possible to make the NixOS build process as painless as in other distros?
Environment
I'm using version 18.09.git.8cfce96 (the current 18.09 channel version)

I don't know if this does enough, but you could try adding something like,
avrgcc-wrapper = pkgs.stdenv.mkDerivation rec {
      name = "avrgcc-wrapper-${version}";
      version = pkgs.stdenv.lib.strings.getVersion pkgs.avrgcc.name;
      nativeBuildInputs = [ pkgs.makeWrapper ];
      builder = pkgs.writeText "builder.sh" ''
        source $stdenv/setup
        mkdir -p $out/bin $out/lib
        for exe in gcc g++; do
          makeWrapper "${pkgs.avrgcc}/bin/avr-$exe" "$out/bin/avr-$exe" --add-flags "-B${pkgs.avrlibc}/avr/lib -isystem ${pkgs.avrlibc}/avr/include"
        done
        ln -s ${pkgs.avrgcc}/lib/* $out/lib/
      '';
    };

to your config.nix. If I do that, I can compile your test program but I haven't tried running it.
If this does work, we should add it to nixpkgs as I agree with your assessment that the status quo seems incomplete.

Thanks a lot, by adding -B${pkgs.avrlibc}/avr/lib the build now succeeds!
You wrapper derivation can probably be simplified to:
avrgcc-wrapper = pkgs.stdenv.mkDerivation rec {
  name = "avrgcc-wrapper-${version}";
  version = pkgs.stdenv.lib.strings.getVersion pkgs.avrgcc.name;
  nativeBuildInputs = [ pkgs.makeWrapper ];
  buildCommand = ''
    mkdir -p $out/bin
    for exe in gcc g++; do
      makeWrapper "${pkgs.avrgcc}/bin/avr-$exe" "$out/bin/avr-$exe" --add-flags "-B${pkgs.avrlibc}/avr/lib -isystem ${pkgs.avrlibc}/avr/include"
    done
    ln -s ${pkgs.avrgcc}/lib $out
  '';
};

But I don't feel competent to add it to nixpkgs, so I'm leaving this PR open for now.

Awesome! Do you know if we need to wrap more of the avr- executables, or is it enough to have gcc and g++ (along with a tool for flashing the chip)?

It's enough for all AVR-related workflows I can think of.

The little bits of ceremony of having a file for builder (rather than a string) and sourcing $stdenv/setup to get coreutils on $PATH are needed (I think) when building as part of nixpkgs.

Great, thanks for the patch!
buildCommand is a shorthand for running a custom command in a stdenv  environment, it perfectly fits your use case.
Another nit: Why are you linking the contents of $avrgcc/lib and not the lib dir itself?

I think this issue can be closed. Executing nix-shell -p pkgsCross.avr.buildPackages.gcc --run 'avr-gcc -mmcu=atmega8 main.c' with the above main.c file on Linux creates an a.out file. This was probably fixed in #48286.
Reopen if this is still an issue.

Can confirm that attinys are broken, atmegas are not.

still an issue (fails when trying -mmcu=attiny2313a), the -B fix still works

Still happening:
$ nix-shell --pure -p pkgsCross.avr.buildPackages.gcc --run 'avr-gcc'
avr-gcc: error: unrecognized command-line option '-iframework'

my setup:
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.5.0, macOS 14.5`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.18.4`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

version:
 nix-shell --pure -p pkgsCross.avr.buildPackages.gcc --run 'avr-gcc -v'
Using built-in specs.
Reading specs from /nix/store/6kipcsjy6ypm8xl1h2sp0b2s8pwssfcc-avr-gcc-13.2.0/lib/gcc/avr/13.2.0/device-specs/specs-avr2
COLLECT_GCC=/nix/store/6kipcsjy6ypm8xl1h2sp0b2s8pwssfcc-avr-gcc-13.2.0/bin/avr-gcc
COLLECT_LTO_WRAPPER=/nix/store/6kipcsjy6ypm8xl1h2sp0b2s8pwssfcc-avr-gcc-13.2.0/libexec/gcc/avr/13.2.0/lto-wrapper
Target: avr
Configured with: ../gcc-13.2.0/configure --prefix=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-gcc-13.2.0 --with-gmp-include=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-gmp-with-cxx-6.3.0-dev/include --with-gmp-lib=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-gmp-with-cxx-6.3.0/lib --with-mpfr-include=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-mpfr-4.2.1-dev/include --with-mpfr-lib=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-mpfr-4.2.1/lib --with-mpc=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-libmpc-1.3.1 --with-native-system-header-dir=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-libc-avr-2.1.0/avr/include --with-build-sysroot=/ --with-gxx-include-dir=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-gcc-13.2.0/include/c++/13.2.0/ --program-prefix=avr- --enable-lto --disable-libstdcxx-pch --without-included-gettext --with-system-zlib --enable-static --enable-languages=c,c++ --enable-multilib --disable-libquadmath --disable-shared --enable-plugin --disable-libcc1 --with-as=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-binutils-wrapper-2.41/bin/avr-as --with-ld=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-binutils-wrapper-2.41/bin/avr-ld --with-headers=/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-avr-libc-avr-2.1.0/avr/include --enable-__cxa_atexit --enable-long-long --enable-threads=single --enable-nls --with-avrlibc --build=aarch64-apple-darwin --host=aarch64-apple-darwin --target=avr
Thread model: single
Supported LTO compression algorithms: zlib

nix shell ".#pkgsCross.avr.stdenv.cc" --command avr-gcc -mmcu=atmega8 main.c works
because there's already this https://github.com/NixOS/nixpkgs/blob/83132a9a1bcddcb19d98c2fdb6f5912f77f73b00/pkgs/build-support/cc-wrapper/default.nix#L680-L683
There was also this which tried to simplify it to use /avr/lib commit PR but was reverted PR
@rwese Your issue is a darwin issue, a fix for only one package can be found in https://github.com/NixOS/nixpkgs/pull/273047

Made a new issue specifying what doesn't work, so this doesn't have to be kept open https://github.com/NixOS/nixpkgs/issues/334906


(User error!) Photos with 1 Live photo (pic+video) and a separate video with same filename: Separate video file gets overwritten.

Before submitting a bug report, please ensure you are running the most recent version of osxphotos and that the bug is reproducible on the latest version
$ osxphotos  --version
osxphotos, version 0.67.10
Python 3.11.5 (main, Aug 24 2023, 15:23:30) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
macOS 12.7.4, x86_64
$ sw_ver
ProductName:	macOS
ProductVersion:	12.7.4
BuildVersion:	21H1123

Describe the bug
Not really a big issue. I have in Photos loaded:

the Live Photo (with file.HEIC and file.MOV components
but also the video componente loaded separately (go figure!) with file.MOV

On export, osxphotos:

it first exports the separate file.MOV with uppercase extension
ant then goes and exports the Live Photo pair (with lower case extensions!):

file.heic
file.mov

But as the NAS filesystem is not case sensitive it returns no error but simply silently overwrite file.mov

To Reproduce

osxphotos export Live Photos and have a Live Photo and a separate movie in the Photos database with the same name.

Expected behavior

Again, I will remove the operate video file from Photos and not really a big issue for me.
May be more relevant for other cases/users when the same filename may create a conflict here.

Screenshots
2024-05-12 05:45:28.697327 -- Exporting XXX_113503.MOV (XXX_113503.MOV) (57266/100514)
2024-05-12 05:45:28.819834 -- Writing metadata with exiftool for XXX_113503.MOV
2024-05-12 05:45:28.945077 -- Exported XXX_113503.MOV to /Volumes/photo/AAA/BBB/XXX_113503.MOV
2024-05-12 05:45:28.951871 -- Exported updated file /Volumes/photo/AAA/BBB/XXX_113503.MOV
2024-05-12 05:45:28.952328 -- Touched date on file /Volumes/photo/AAA/BBB/XXX_113503.MOV

(...)

2024-05-12 06:39:31.771080 -- Exporting XXX_113503.heic (XXX_113503.heic) (82302/100514)
2024-05-12 06:39:31.880613 -- Writing metadata with exiftool for XXX_113503.mov
2024-05-12 06:39:32.013204 -- Exported XXX_113503.heic to /Volumes/photo/AAA/BBB/XXX_113503.mov
2024-05-12 06:39:32.019895 -- Exported updated file /Volumes/photo/AAA/BBB/XXX_113503.mov
2024-05-12 06:39:32.020202 -- Skipped up to date file /Volumes/photo/AAA/BBB/XXX_113503.heic
2024-05-12 06:39:32.020479 -- Touched date on file /Volumes/photo/AAA/BBB/XXX_113503.mov

Desktop (please complete the following information):
$ osxphotos  --version
osxphotos, version 0.67.10
Python 3.11.5 (main, Aug 24 2023, 15:23:30) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
macOS 12.7.4, x86_64
$ sw_ver
ProductName:	macOS
ProductVersion:	12.7.4
BuildVersion:	21H1123

Additional context
N/A

Good News @RhetTbull ! I've decided to test with the importapalooza branch on a test library and voila...  the new code treat this case correctly:
2024-05-12 11:47:29.477041 -- Exporting 2 photos to /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo...
2024-05-12 11:47:29.528476 -- Exporting XXXX_113503.heic (XXXX_113503.heic) (1/2)
2024-05-12 11:47:29.539713 -- Writing metadata with exiftool for XXXX_113503.heic
2024-05-12 11:47:29.979116 -- Exported XXXX_113503.heic to /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503.heic
2024-05-12 11:47:29.982970 -- Writing metadata with exiftool for XXXX_113503.mov
2024-05-12 11:47:30.085588 -- Exported XXXX_113503.heic to /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503.mov
2024-05-12 11:47:30.087874 -- Exported /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503.heic
2024-05-12 11:47:30.088575 -- Exported /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503.mov
2024-05-12 11:47:30.090232 -- Exporting XXXX_113503.MOV (XXXX_113503.MOV) (2/2)
2024-05-12 11:47:30.093627 -- Writing metadata with exiftool for XXXX_113503 (1).MOV
2024-05-12 11:47:30.198827 -- Exported XXXX_113503.MOV to /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503 (1).MOV
2024-05-12 11:47:30.201038 -- Exported /Users/MSP/Documents/GitHub/osxphotos/working/LivePicsVideo/XXXX_113503 (1).MOV
Exporting 2 photos ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
2024-05-12 11:47:30.203041 -- Processed: 2 photos, exported: 3, missing: 0, error: 0
2024-05-12 11:47:30.203377 -- Elapsed time: 0:00:00
2024-05-12 11:47:30.206449 -- Cleaning up lock files

Nothing in the importapalooza branch should have affected export so I checked against 0.67.10 and was not able to replicate this behavior.  In my test the second file with the same name was properly incremented.
Exporting 2 photos to /Users/rhet/Desktop/export...
Exporting IMG_1853.mov (E304B775-6A09-4A61-98D9-B0EA73EC22AD.mov) (1/2)
Writing metadata with exiftool for IMG_1853.mov
Exported IMG_1853.mov to /Users/rhet/Desktop/export/IMG_1853.mov
Exported /Users/rhet/Desktop/export/IMG_1853.mov
Exporting IMG_1853.HEIC (17922D2B-D91F-4C87-8AA3-B9559EBEB4E6.heic) (2/2)
Writing metadata with exiftool for IMG_1853 (1).HEIC
Exported IMG_1853.HEIC to /Users/rhet/Desktop/export/IMG_1853 (1).HEIC
Writing metadata with exiftool for IMG_1853 (1).mov
Exported IMG_1853.HEIC to /Users/rhet/Desktop/export/IMG_1853 (1).mov
Exported /Users/rhet/Desktop/export/IMG_1853 (1).HEIC
Exported /Users/rhet/Desktop/export/IMG_1853 (1).mov
Exporting 2 photos ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
Processed: 2 photos, exported: 3, missing: 0, error: 0

I've re-ran:

version
Photos DB
Remark
Comments

0.67.10
test Photos database
you're right
it does export 3 pics: the Live photo pair and the MOV.

0.67.10
Main Photos database
(with --name and limited settings) expecting some quirk behaviour
still exported: 3 pics: the Live photo pair and the MOV.

0.67.10
Main Photos database
full settings (--directory, --update, etc)
But the log for the actual export on 2024-05-12 does state only 2 pics and actually only the pair is on the exported Album folder

0.67.10
Main Photos database
full settings (--directory, --update, etc)
Will delete the destination files to force a re-export of these pics and re-check


Ecto master tracking branch

This branch tracks changes made in ecto master branch. We'll merge it when a new release emerges from said branch.

Coverage remained the same at 96.393% when pulling 0c0afd9cb99555dbf2260ba450797276e378ff31 on ecto-master into 17f39db2a221dcc415e470694887d13e1a8e98fd on master.

Coverage remained the same at 96.393% when pulling 0c0afd9cb99555dbf2260ba450797276e378ff31 on ecto-master into 17f39db2a221dcc415e470694887d13e1a8e98fd on master.

Coverage remained the same at 96.393% when pulling 61df2819ca85f76f0d00e0adad0a9a0a930315b0 on ecto-master into 17f39db2a221dcc415e470694887d13e1a8e98fd on master.

Coverage remained the same at 96.393% when pulling 10a5fda747d3ecbd3ad40af306907eb5886085da on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage remained the same at 96.393% when pulling 10a5fda747d3ecbd3ad40af306907eb5886085da on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage remained the same at 96.393% when pulling f71f8ea569ddb1ee376dcad793ab6f0cc1da023b on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling 7d5ed09daf03f29a390fcfe211fa2ba668f78ccf on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling 7d5ed09daf03f29a390fcfe211fa2ba668f78ccf on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling 7d5ed09daf03f29a390fcfe211fa2ba668f78ccf on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling 7d5ed09daf03f29a390fcfe211fa2ba668f78ccf on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling d990dc358f2825cf37fb53bac5d5d90cee5800d3 on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling d990dc358f2825cf37fb53bac5d5d90cee5800d3 on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling d990dc358f2825cf37fb53bac5d5d90cee5800d3 on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling d990dc358f2825cf37fb53bac5d5d90cee5800d3 on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

Coverage increased (+0.01%) to 96.405% when pulling 8a447b755b3062e782f59431eb46e66241503e19 on ecto-master into 913b20dad83640f8a88ff0771a0e14c3cd29f468 on master.

https://github.com/elixir-ecto/ecto/compare/ad54edaf195d8cc7f333ac7464017c3cbc752f58...master
:(

these commits are all that needs to be coerced into this branch.


[BUG] Found multiple configuration files error | CLion

Describe the bug
I have a diagram.json and wokwi.toml at the root of my project in CLion, yet whenever I start the simulation, I get the following error, Found multiple configuration files: file:/... Specify the concrete one in the Settings. The issue is that I have just one wokwi.toml, yet I get this error. More interestingly the error specifies two file paths; one for each conflicting config file, but they are 100% identical and lead to the same file.
To Reproduce
Steps to reproduce the behavior:

Create diagram.json and wokwi.toml in online editor
Download files to project root
Go to Wokwi tab
Click on "Start simulation"
See error balloon

Expected behavior
Simulation starts as normal using the configuration files at project root.
Additional context
My CLion project is not setup for platformio or anything else of the sort, I am simply looking to simulate my firmware on an esp32-s3.
Any help appreciated!

For context, here are the two config files that I downloaded from the Wokwi website to my project root.
diagram.json
{
  "version": 1,
  "author": "David Goldstein",
  "editor": "wokwi",
  "parts": [
    {
      "type": "board-esp32-s3-devkitc-1", "id": "esp",
      "attrs": {"flashSize": "16", "psramSize": "4", "psramType": "octal"}
    }
    ],
  "connections": [ [ "esp:TX", "$serialMonitor:RX", "", [] ], [ "esp:RX", "$serialMonitor:TX", "", [] ] ],
  "dependencies": {}
}

wokwi.toml
[wokwi]
version = 1
elf = "./build/release/build/adafruit_metro_esp32s3/firmware.elf"
firmware = "./build/release/build/adafruit_metro_esp32s3/firmware.bin"

Thanks for your bug report, I will look into it!


daemon mode for lbry-desktop

Is your feature request related to a problem? Please describe.
videos take too long to download and/or buffer too much to be practical/popular.
Describe the solution you'd like
As a predecessor to a more robust service node infrastructure, like mentioned in #3413, it would be nice if lbry-desktop had a daemon mode so that the user could close the application window, but lbry would continue sharing in the background. It would be nice to have a scheduler (share midnight -8am, for instance), and upload bandwidth limit setting. Bonus points if LBRY issues LBC rewards based on sharing amount periodically. Being able to set a share folder would be nice too, so you could choose which files get shared in daemon mode, if you can't already specify that with normal mode. That would allow you to only use bandwidth for files you really care about sharing. Maybe a "selfish mode" setting where only the files in your channels get shared, without the need for special folders.
Describe alternatives you've considered
Waiting for full blown service node infrastructure and/or using a client that can't play a video without buffering constantly. :)

The app already continues to run when you close it. The other various suggestions already have separate issues in their respective repos

Ok. I should have checked that. Sorry for the noise.
Just for the record, i think that "hide" when user clicks "close/quit" is an anti-pattern if it's not exposed as an option to the user.

It will be in the next release actually!

that's great! thanks


cdkScrollable to table with nzScroll not working

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-ivy-q7gpfc
Steps to reproduce
Make a table with nzScroll property and add a popover component in the table.
What is expected?
Popover component stays where the element is when scrolled.
What is actually happening?
Popover element moves with scroll.

Environment
Info

ng-zorro-antd
13.1.1

Browser
Any

@vthinkxie Any update on this?

have same issue, any new update?


Add Mimic Start command

Describe the solution you'd like
The command should allow the user to start a sentence so that mimic can finish it.
Example usage:
mimic!start I like -> mimic's response: I like ....
This should use the global server chain.

Implemented


Typo in topics

(Cloudyn / Resources /) "Clouydn Regional Availability" should be "Cloudyn Regional Availability"

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

ID: d2ccf81c-f627-3733-21a6-470647d8b00f
Version Independent ID: 5f79db74-f8f4-616f-ca77-ccc58c806ab9
Content: Frequently asked questions for Cloudyn in Azure
Content Source: articles/cost-management/cost-mgt-faq.md
Service: cost-management
GitHub Login: @bandersmsft
Microsoft Alias: banders

@mikavirkkunen Thanks for catching that error. I'll update.

#in-process

Fixed.
#please-close


SDK project.assets.json - multi-target project framework being set to wrong target framework

Description
project.assets.json lists a multi-target project as targeting net461 even though netstandard2.0 is set in project reference
Repro steps
unzip the attachment ProjectAssetsJson.zip
MSBUILD version  - 16.2.37902.0

go to root folder of project
run .paket\paket.exe install
run msbuild /t:cleanobjbins
run msbuild /t:restore
run msbuild /t:build

Expected behavior
TestingHost/obj/project.assets.json - Dependency-MultiTarget would have framework value as ".NETStandard,Version=v2.0"
TestngHost/bin - this folder should have

Dependency-MultiTarget.dll - built for .netstandard2.0
Depdendency_NetStandard.dll

Actual behavior
TestingHost/obj/project.assets.json - Dependency-MultiTarget framework is set to ".NETFramework,Version=v4.6.1"
TestngHost/bin - this folder has

Dependency-MultiTarget.dll - built for .netstandard2.0 (Verified using ILSpy)
Dependency_Net46.dll (should have been Dependency_NetStandard.dll)

Known workarounds
None

i was able to reproduce this without using paket. Using nuget restore has similar behavior. here is a open issue on nuget - https://github.com/dotnet/core/issues/2571
closing this now.


Docs missing collapsible sandbox example

https://danielcaldas.github.io/react-d3-graph/docs/index.html#collapsible
On the collapsible section on the docs it states:
"To see an example of this behavior you can access this sandbox link that has a specific set up to experiment this feature"
But this sandbox link is no where to be found. Could this be added please so we can see the use of the collapsible feature?

https://danielcaldas.github.io/react-d3-graph/sandbox/index.html?data=marvel
The Marvel dataset example contains the ability to collapse the 3 category nodes (Teams, Heroes, Villains):

Thank you both @terahn and @TamjidTayeb in fact the example I intended to link in the docs is the Marvel diagram. I'll update the documentation soon with the link.


og:image must be frame of large GIFs

And/or something else is wrong here that this image doesn't properly show a preview image on social media.
https://spee.ch/f/jeremy-toy-factory.gif

That's a 28mb GIF; we have other issues open about resizing, closing in favor of https://github.com/lbryio/spee.ch/issues/60


⚠ Bofrost.de has degraded performance

In c26f10f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1055 ms

Resolved: Bofrost.de performance has improved in e07bd30.


Unwanted auto-indentation (to the left/decreased indentation) while coding in React and ReactNative

VSCode Version: 1.44
OS Version: 10.0.18362 N/A Build 18362

Steps to Reproduce:

Create a JavaScript file(for a ReactNative Component), for eg. ResultsList.js
import React
Create a functional component
Inside the return, statement paste the following:

return (
  <Text></Text>
 )

Now add the following by typing(not pasting) in between the opening and closing  tag:
{}
such that it should look like: {}
as shown in the gif below:

You will notice that the whole line gets auto indented (decreased indentation, i.e. to the left), which is not expected in this case.

Does this issue occur when all extensions are disabled?: Yes

Any updates on this?

The default autoIndent setting value is full, which will keep the current line's indentation, honor language defined brackets, invoke special onEnterRules defined by languages, and lastly honor indentationRules defined by languages. The catch here is indentations rules are the most powerful one but also problematic sometimes as it's regular expression based rules instead of being real context-aware.
We will explore how we can improve regex based onEnter and indentation rules but for now, please use "editor.autoIndent": "advanced".


refactor: Move non-public utilities to their own file

Two reasons why I want this:

The function moved from main has to be exported for tests. But when it's in main it's also visible to anyone using the package. It should be a private function.
In upcoming changes I'm doing I need more utility functions. Makes sense to not grow main that much bigger.

Codecov Report

Merging #99 (e397a85) into main (1dba38f) will increase coverage by 20.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             main       #99       +/-   ##
============================================
+ Coverage   80.00%   100.00%   +20.00%
============================================
  Files           2         2
  Lines          35       122       +87
  Branches        7        18       +11
============================================
+ Hits           28       122       +94
+ Misses          7         0        -7

Impacted Files
Coverage Δ

src/main.ts
100.00% <100.00%> (+22.58%)
:arrow_up:

src/util.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

This sounds like a good plan! Like it! 👍

Do we want to add the utils file to the typedoc exclude btw? It's not meant for public consumption


Set required properties for some objects in v140 schema

Closes #12
Hi all,
what do you think? Mandatory properties for some objects in metadata? Yes, no, maybe :wink:
Let's discuss!
Context: (i) https://github.com/OpenEnergyPlatform/metadata/issues/9#issuecomment-551908556, (ii) https://github.com/OpenEnergyPlatform/metadata/issues/9#issuecomment-551925124

Commit for discussing required properties: https://github.com/OpenEnergyPlatform/metadata/pull/11/commits/89d94ea891285be31096d704428e751f814b36dc
Commit for discussing to not allow non-defined (additional) properties: https://github.com/OpenEnergyPlatform/metadata/pull/11/commits/581dde677e88d8a55568ffdc25087860a5e9bee2

BTW: all changes on the schemas v130 and v140 are compatible with the given templates and examples of v130 and v140 (no changes where necessary or made to the templates or examples), so if all given metadata (v130 and v140) complies with the example and/or template of the specific version, they should validate against the schemas out-of-the-box, no changes needed! Hopefully :wink:

As per our discussion at the meeting in Berlin today, all no additional properties (or sub properties) should be allowed, except for sub properties of "_comment".

It was my understanding that no fields should be required. At least @Ludee and I has always communicated on this premise. Any requirement might put off potential users (and we'd rather have someone fill out a single field than no field), so we thought there should be none whatsoever.

@4lm I would say that the PR is not related to the issue you indicated this PR should close : the issue is "Additional properties should not be allowed" and the PR is about required properties. I'd suggest to remove the commits about the required properties and keep the ones about forbidding additional properties

Sorry for the confusion, @Bachibouzouk I will do it like you suggested!

Mhh, I wanted to rename the branch name to better reflect the topic of the PR. This is not working, GitHub closed the PR in didn't move the PR to the new branch feature/don-t-allow-additional-properties. So I restored the old branch and reopened this issue. And will remove the above mentioned commit on the old branch ...

Done :) Please review!


Mission control should prevent adding another database with same db alias name

Done. Coming in v0.17.0


feat: docker service improvements

Describe your changes

It adds support for using .dockerignore as an exclude filtering for tars created and sent to cloud build

addresses issue https://github.com/avast/wanna-ml/issues/114

Adds the cloud build link to stdout

addresses feature request https://github.com/avast/wanna-ml/issues/71

Better documents DockerService & refactors it a little

Issue ticket number and link

https://github.com/avast/wanna-ml/issues/114
https://github.com/avast/wanna-ml/issues/71

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] If it is a core feature, I have verified thorough tests.

Codecov Report
Attention: Patch coverage is 50.00000% with 39 lines in your changes are missing coverage. Please review.

Project coverage is 74.49%.  Consider uploading reports for the commit aa79efd to get more accurate results

Files
Patch %
Lines

src/wanna/core/services/docker.py
50.00%
28 Missing :warning:

src/wanna/core/utils/io.py
35.29%
11 Missing :warning:

Additional details and impacted files
@@                 Coverage Diff                 @@
##           joao/fix-import     #119      +/-   ##
===================================================
- Coverage            74.54%   74.49%   -0.06%
===================================================
  Files                   57       58       +1
  Lines                 2852     2866      +14
===================================================
+ Hits                  2126     2135       +9
- Misses                 726      731       +5

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


generateExecutableReflection needs to give more information

Currently it doesn't tell you what is wrong. It needs  to give appropriate information about the callable that is failing e.g. in generateExecutableFromString the exception message should include the string that was attempted to be executed.

I don't believe this is currently an issue.


Only show service bindings if requester has read access to bound app

As an app dev, I can only see bindings in spaces that I have access to for shared service instances #153456495
NOTE: This PR builds on top of #1027, which should be merged first. The actual changes on top of #1027 can be viewed in this diff.
Context
In our initial work for service instance sharing, we allowed service bindings to be viewed by anyone who has access to view the binding's service instance. That means anyone who can see the service instance because it was shared with them can also see the service bindings. We've now decided to restrict this so that a user can only see service bindings if they have read access to the app of the service binding.
This change was motivated by the behavior of the CLI during cf service <service-name>, where it reasonably assumes the app guids that come back from /v2/space/:guid/service_instances/ can be queried for more information about the app.
Impact

Users who run the CLI command cf service <service-instance-name> for a service instance that has bindings to an app they don't have access to will no longer get a 403 Unauthorized error.
API consumers who hit GET /v2/service_bindings, GET /v2/space/:guid/service_bindings, etc., will only see service bindings for which they have read access to the associated app.

Code Changes:

Make the service binding user visibility filter use the app's visibility filter, not the service instance's.
Added a user visibility filter to AppModel. This is the same logic as the visibility filter used by the ProcessModel. (Since the service binding is attached to an AppModel, not a ProcessModel, we thought the easiest and cleanest way forward would be to add a user_visibility_filter to the AppModel. If this is a pattern we don't to continue into the v3 models, let us know.)

PR

[X] I have viewed signed and have submitted the Contributor License Agreement
[X] I have made this pull request to the master branch
[X] I have run all the unit tests using bundle exec rake
[X] I have run CF Acceptance Tests on bosh lite

Thanks,
SAPI Team (@jenspinney and @samze)

Note: Travis CI test failure seems like a flake and unrelated to these code changes.

Hi SAPI friends,
We're looking at your PR right now, and are a little bit confused about the expected initial behavior. You said:
In our initial work for service instance sharing, we allowed service bindings to be viewed by anyone who has access to view the binding's service instance. That means anyone who can see the service instance because it was shared with them can also see the service bindings.
And you also said:
Users who run the CLI command cf service <service-instance-name> for a service instance that has bindings to an app they don't have access to will no longer get a 403 Unauthorized error.
The first explanation implies that our pre-PR users should be able to see the service bindings in this scenario. The second explanation implies that the users should have been receiving a 403. We're seeing the former behavior right now. Your code changes seem clear to us, so we're looking to confirm the acceptance criteria for the feature as well.
Let us know what you think!
Raina and Maryam
CF CAPI Team

Hi @rainmaker and Maryam,
Sorry for the confusion. The reason for receiving 403 when using the CLI command cf service <service-name> in the pre-PR state of the world is because the CLI essentially does the following:

Get me all service bindings for this space (i.e., GET /v2/space/:guid/service_bindings).
For each service binding in the returned list, grab the associated app's guid and look up more detailed information about that app, like the name, etc. (i.e., GET /v2/apps/:guid)

In the pre-PR code, step 1 returns all service bindings that are associated with a serivce instance that the user can view (including service instances that they've shared with other spaces where the user doesn't normally have read access). The CLI then tries to look up the details for the apps and returns a 403 error because it can't query the app details (the user doesn't have view permission in that space).
With the code in this PR, step 1 will no longer return those entries at all. The returned list of service bindings will only include bindings where the user should have enough permission to issue a /v2/apps/:app_guid request without getting a permissions failure.
Does that answer your question?
Thanks,
Jen

@jenspinney Thanks for the context, that's helpful!


[release-2.7] Add backup label to globalnet configmap

This is a cherry-pick of #871

/approve
/cc @sridhargaddam

/lgtm


Support PKCS #7 certificates

Hi,
We're having an issue using cert-chain-resolver with a Letsencrypt certificate.
mallmann:cert-chain-resolver> ./cert-chain-resolver cert.crt
Invalid certificate.
mallmann:cert-chain-resolver>

Looking into is, it appears that this certificate points to two parents: http://cert.int-x3.letsencrypt.org/ and http://apps.identrust.com/roots/dstrootcax3.p7c.
The error happens when we try to parse the second one.
In the file ./src/github.com/zakjan/cert-chain-resolver/certUtil/io.go, inside func DecodeCertificate, the call to x509.ParseCertificate(data) returns the following error:
asn1: structure error: tags don't match (16 vs {class:0 tag:6 length:9 isCompound:false}) {optional:false explicit:false application:false defaultValue:<nil> tag:<nil> stringType:0 timeType:0 set:false omitEmpty:false} tbsCertificate @2

Looks like the certificate from identrust is in DER form. What is the best way to handle this?
If there is anything we can do, we'll be glad to develop and post a PR. We just need a few pointers =)
Thanks!

Hi,
Thanks for reporting this issue! Note the line if IsPEM(data) {, the DER format is already supported. This certificate is in another PKCS #7 format.
I guess we can use this library https://godoc.org/github.com/fullsailor/pkcs7 I will be happy to accept a PR:)

Right.
PEM vs DER: I'm a n00b. Now I get the differences between formats and encodings =)
I made a quick fix for our use here. In case of parsing error we retry using fullsailors's pkcs7. Works nicely, but I'm not sure it's the best approach. I'll submit the PR and we can discuss there, ok?

Ok :) I'll keep this issue opened.

Merged in #15


Virus total detections

Hi, i want to try this fork but i tried to Analise the windows files in virustotal website i uploaded and virus total have some Trojan detection (maybe false positives).
For me it's sufficient for avoid using Covid coin i recommend to solve this issue.

Sorry for the late response.  Almost all of the Chia forks are being reported as having Trojan.  I am looking into submitting the app for approval. Luckily with the code on Github for everyone to see it is pretty hard to hide something malevolent.


The git in conda-forge on Windows in our shell is broken (was: Using git when conda environment is loaded can fail)

Loading the lf LFortran environment: https://github.com/lfortran/lfortran/blob/b50db1203c2cff09c621d3a16de771dd57d97b07/environment_win.yml can fail like this:

I think it is probably caused by the fact that some DLL is loaded first, and the (system) git depends on some other one.

I don't even know which git is being used, I am guessing the first one:
$ where git
C:\Users\ondrejcertik\AppData\Local\miniforge3\envs\lf\Library\mingw64\bin\git.exe
C:\Users\ondrejcertik\AppData\Local\miniforge3\envs\lf\Library\bin\git.exe
C:\Program Files\Git\cmd\git.exe

Similar issue with m2-bison:

This one is a tough one --- we need Bison for LFortran.


Provide a UI for viewing, editing and deleting the Project

Acceptance Criteria:

Users should be able to create a project when deploying.
Users should be able to view all the projects on the dashboard.
Users should be able to view all details of a single project.
Users should be able to view all the deployments that happened for a project.
Users should be able to view all the settings of the project.

Delete of the project has been left off, but this feature has been implemented!!!


Drag and drop for custom cellTemplate?

Hi, @revolist . I created my custom cell by using cellTemplate and VGridVueTemplate.
It lost the drag icon and can not drag and drop anymore. How can enable drag and drop row again?

Hi all,
I started with RevoGrid in Vue 2 project, no custom cell, all out of the box.
Drag control invisible in my case (unable to dragg at all).
Maybe I'm doing something wrong?

For custom cells you need to repeat events like they are in original cell render. Or use your own d&d plugin.


building react-dnd from source and importing into project doesnt seem to work

Describe the bug
I've cloned the react-dnd git tree locally on my machine and did 'yarn run build', which completes successfully , but when I try to use my locally built packages in my app react-dnd doesn't seem to work at all.
It appears as though all the source files exist and the code is running (i can set breakpoints inside of react-dnd) , but ultimately once I start the page I can't initiate drags.
The app works correctly if I blow away my node_modules and do an 'npm install' and install react-dnd from the npm registry.
I noticed that one of the differences is that in my app the dragSource has draggable=true when i use the NPM registry version, but draggable=false when I use the version that I compile from my local react-dnd tree.
(This is with no changes to my react-dnd source tree)
Does anyone have any insights into why this might be happening?

when i use my locally built version, the following breakpoint hits on load in HTML5Backend.js on line 453:
    return function () {
      _this.sourceNodes.delete(sourceId);

      _this.sourceNodeOptions.delete(sourceId);

      node.removeEventListener('dragstart', handleDragStart);
      node.removeEventListener('selectstart', handleSelectStart);
      **node.setAttribute('draggable', 'false');**
    };

when i blow things away and run with the npm registry version of react-dnd, this line never gets triggered on load.

it appears as though this is a regression in master. i reverted back to v7.4.5 and am not having this issue.

Were you using connectDragSource() on a ref or a JSX element? There was an issue a couple of days ago on master where it was busted when using it on JSX

return this.props.connectDragSource(
<Container />
)

Can you check this again with the latest on master?

yeah ill try that later this week and close if appropriate
On Wed, Jun 5, 2019 at 12:15 PM Chris Trevino<EMAIL_ADDRESS>wrote:

Can you check this again with the latest on master?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-dnd/react-dnd/issues/1371?email_source=notifications&email_token=AAEA57AI3JUS4QEBVC2AKELPY77FRA5CNFSM4HSJ26V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXASFKY#issuecomment-499196587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEA57CVW2WRVPOMIGK7FTTPY77FRANCNFSM4HSJ26VQ
.

Cool, thanks @mx2323

@darthtrevino that appears to have fixed it! great stuff.


Components topic code example updates

Fixes #12892
RE: Update blazor docs to reflect directive attributes changes (aspnet/AspNetCore.Docs #12756)

WRT the temporary (Pre6) need to place an @ on a field/property value of an @bind directive. I looked at Implement directive attribute feature for components (aspnet/AspNetCore #6364), but none of the description discusses the scenario.
For example in the Format strings section of the Components topic, updates didn't add the @ to the value for StartDate ...
<input @bind="StartDate" @bind:format="yyyy-MM-dd" />

Same thing in the ParentYear examples (@bind-{property} syntax) ...
<ChildComponent @bind-Year="ParentYear" />

... and should devs apply the @-value temporary rule to @bind-value:event/bind-{property}:event? It was left on the updates as ...
<ChildComponent @bind-Year="ParentYear" @bind-Year:event="YearChanged" />

What's the difference between the following, if any:
<input @bind="@CurrentValue" />

<input @bind-value="@CurrentValue" @bind-value:event="onchange" />

Is the former just a shorthand way of setting up onchange binding? If they are the same, it isn't considered better to have Just One Consistent Way:tm: :smile: to do it (the latter approach, which comports with other on{event} binding)?

@guardrex, looks like you're missing a detail about this. https://github.com/aspnet/AspNetCore.Docs/issues/12766 refers to @ not being needed for @onclick, @onfocus etc. @bind doesn't have any bug and doesn't have to start with @.

What's the difference between the following, if any:
<input @bind="@CurrentValue" />
<input @bind-value="@CurrentValue" @bind-value:event="onchange" />
Is the former just a shorthand way of setting up onchange binding? If they are the same, it isn't considered better to have Just One Consistent Way™ 😄 to do it (the latter approach, which comports with other on{event} binding)?

There is no difference in behavior between the two in your example. The first one is just a short version. The second format is useful if you want to specify an event other than onchange like oninput. Like in the code example that you edited. <input @bind-value="@CurrentValue" @bind-value:event="oninput" />

you're missing a detail about this

Thanks ..... yes .... it all makes sense now. :smile: ...... i.e., I got 😵 on it.

WRT my point number 2 .... do you think we should call that out in the topic? I could draft some text here for you to look at.

@ajaybhargavb "it all makes sense" ... famous last words for my tombstone. :smile: RIP
I went back to the original PR #12756.
At https://github.com/aspnet/AspNetCore.Docs/pull/12756/files#diff-5cf16565c6014e97ab5fd792aba7b93dR24 ... it preserves the @ on _italicsCheck.
However, at https://github.com/aspnet/AspNetCore.Docs/pull/12756/files#diff-aec5790e12a05d20a3cda92a708f6679R123, it removes the @.
It looks like what we need to do here is take that @ off of the _italicsCheck in the sample app's HeadingComponent.razor.

Also note that the @ is on CurrentValue for @bind-value here in that PR ... https://github.com/aspnet/AspNetCore.Docs/pull/12756/files#diff-aec5790e12a05d20a3cda92a708f6679R142 ...
<input type="text" @bind-value="@CurrentValue" @bind-value:event="oninput" />

... and my original change on this PR was to make the text match (but for @bind, which seems incorrect) ...
Using `@bind` with a `CurrentValue` property (`<input @bind="@CurrentValue" />`) is essentially
equivalent to the following:

So, I assume that there's a difference between @bind and @bind-value in these contexts.

So, I assume that there's a difference between @bind and @bind-value in these contexts.

Nope. Both @bind and @bind-value should be expecting a C# value (meaning doesn't have to start with @). Please correct it if I missed anything in my original PR

ok ... I'll shape this PR up and ping u back.

@ajaybhargavb kk ... that should get it. 🤞


Doc update

This is just to get the documentation in master up to date.

No code changes, so merging now


Register the package

Hi @philippberens,
I think the package is ready to be registered in the default Julia package registry, so that user can just use Julia package manager, and type add CircStats to install the package.
Since i don't have access to the repository, the easiest way for you to register is go to Juliahub register page, and it should be easy to complete the whole process manually.
Thanks,
Alex

Ok, I did that.


Error in buildProvider configuration in web.config after installing nancy.viewengines.razor

I created an empty webapplication and installed the nuget packages nancy.hosting.aspnet and nancy.viewengines.razor.
The buildProvider element in the web.config now says: type="Nancy.ViewEngines.Razor.BuildProviders.NancyCSharpRazorBuildProvider, Nancy.ViewEngines.Razor.BuildProviders".
The assembly is called Nancy.ViewEngines.Razor (without the .BuildProviders).

No? That is not telling Nancy to use the Nancy razor engine to render views, it's letting Visual Studio know that it should use the specified build providers to enable Intellisense support
Are you experiencing any issued with the code?

Oops, my mistake, Visual Studio 2010 underlined the last part of the type definition. I reopened the web.config and everything was fine (my intellisense is sometimes confused). The app is working fine.

Thank you for the update. Once you've built your solution once, the Nancy.ViewEngines.Razor.Buildproviders.dll file will be copied (with the help of post-build events) to your \bin folder and that will help Visual Studio enable Intellisense in the views.


update cifar100 results

The standard deviation of the current version method looks not small on CIFAR-100... Let's improve it and try more on large-scale datasets for the journal extension.


fine-tuning on cross-dataset

Hello,
Thank you for this very interesting contribution.
I am trying to use maple on custom datasets, and I understand that the cross-dataset scenraio uses the contexts learned from Imagenet pre-training and infers on the test splits of the novel dataset without finetuning on potential shots of the novel dataset.
Is my understanding correct? and if so how do I finetune on the novel dataset?
Thank you

Hi @aymane399,
Sorry for the delayed response.
Regarding your first question, yes your understanding it correct. For cross-dataset setting, we train only a single model using source dataset and then evaluate/test it directly on downstream novel datasets.
Regarding your second question, you can finetune on the novel dataset by simply replacing the dataset name in the run script.
For example:
For imagenet training, we use the following commands for all 3 seeds:
bash scripts/maple/xd_train_maple.sh imagenet 1
bash scripts/maple/xd_train_maple.sh imagenet 2
bash scripts/maple/xd_train_maple.sh imagenet 3
Now, if you want to instead train the model on novel/cross-dataset, we can simply change the name of dataset. For example, training MaPLe on EuroSAT can be executed as follows:
bash scripts/maple/xd_train_maple.sh eurosat 1
bash scripts/maple/xd_train_maple.sh eurosat 2
bash scripts/maple/xd_train_maple.sh eurosat 3
I hope that is clear now. Kindly let me know if there is any other question.
Thank you and kind regards!


fzy-tmux: Use named pipe for conveying exit code

On a couple of occasions I've seen the following error being reported by
the fzy-tmux script:

head: cannot open '/tmp/tmp.qUbDwrPLxQ/ret' for reading: No such file or directory
/usr/bin/fzy-tmux: line 41: exit: : numeric argument required

The issue seems to be that PATH_FIFO_RET is not yet created but we
already try to read from it, causing the read to fail. With this change
we use a named pipe which we create before the actual tmux invocation,
preventing this case from happening.
Credit for the fix goes to @zsugabubus.
Closes: #143

Thanks!


http://backbonethis.com/ does not respond

today it is answering a blank page.
Cheers

The domain expired. I will host it on a new location soon.


DatabaseGeneratedOption.None required for Primary Guid Id

Describe the bug
We are using Guids for our Ids, and if my understanding is correct the default convention for EF Core is to auto generat it on the client when calling SaveChanges. For the entities that I now do want to Bulk Insert I'm manually setting the Id in code.
But when I call BulkInsert it skips the Id column and I get an SQL exception:
 Microsoft.Data.SqlClient.SqlException (0x80131904): Cannot insert the value NULL into column 'Id', table 'master.dbo.StudentReportImports'; column does not allow nulls. UPDATE fails.
The statement has been terminated.
   at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at Microsoft.Data.SqlClient.SqlCommand.InternalEndExecuteNonQuery(IAsyncResult asyncResult, Boolean isInternal, String endMethod)
   at Microsoft.Data.SqlClient.SqlCommand.EndExecuteNonQueryInternal(IAsyncResult asyncResult)
   at Microsoft.Data.SqlClient.SqlCommand.EndExecuteNonQueryAsync(IAsyncResult asyncResult)
   at Microsoft.Data.SqlClient.SqlCommand.<>c.<InternalExecuteNonQueryAsync>b__210_1(IAsyncResult result)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location ---
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteNonQueryAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteNonQueryAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteNonQueryAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.ExecuteSqlRawAsync(DatabaseFacade databaseFacade, String sql, IEnumerable`1 parameters, CancellationToken cancellationToken)
   at N.EntityFrameworkCore.Extensions.DabaseFacadeExtensionsAsync.ExecuteSqlAsync(DatabaseFacade database, String sql, Object[] parameters, Nullable`1 commandTimeout, CancellationToken cancellationToken)
   at N.EntityFrameworkCore.Extensions.DabaseFacadeExtensionsAsync.ExecuteSqlAsync(DatabaseFacade database, String sql, Nullable`1 commandTimeout, CancellationToken cancellationToken)
   at N.EntityFrameworkCore.Extensions.BulkOperation`1.ExecuteMergeAsync(Dictionary`2 entityMap, Expression`1 mergeOnCondition, Boolean autoMapOutput, Boolean insertIfNotExists, Boolean update, Boolean delete, CancellationToken cancellationToken)
   at N.EntityFrameworkCore.Extensions.DbContextExtensionsAsync.BulkInsertAsync[T](DbContext context, IEnumerable`1 entities, BulkInsertOptions`1 options, CancellationToken cancellationToken)
   at N.EntityFrameworkCore.Extensions.DbContextExtensionsAsync.BulkInsertAsync[T](DbContext context, IEnumerable`1 entities, Action`1 optionsAction, CancellationToken cancellationToken)

If I add this attribute [DatabaseGenerated(DatabaseGeneratedOption.None)] to the Id property then it works as it should.
Expected behavior
I would expect it to work without the attribute since it is the default convention in EF Core to not create database generated values for Guid Ids.

KeepIdentity = true does not help.
The CloneTable part gets the Id column without the attribute but not the Merge statment.

Looked some more into it and it seems like the problem only exist on the async version.
if(keepIdentity)
{
    columnsToInsert = columnsToInsert.Union(TableMapping.GetPrimaryKeyColumns());
}

This part is missing from the async version of ExecuteMergeAsync

SpaceOrge,
I made some recent fixes to the development branch and I believe it fixed this issue already. You are welcome to submit a pull request, but you may need to adjust your code now and do another commit.


Supported OS Versions clarifications

Make logical link/heading to 'Supported OS Versions'
Integrate unclear section 'Architecture' into existing topics above
Add Unsupported OS list

cc @Petermarcu @danmosemsft

Any reason we don't just add the community distro's just to the list with the "C" label as community supported?
Perhaps we should create a status page or issue for each distro to list current known issues and things rather than having to update the main roadmap as bugs get fixed, etc?

@Petermarcu I think it would be great to distinguish distros which already have something existing + community support vs. those which are still "in progress" (e.g. Arch Linux).
I'll take a stab at that, then then ping you for additional feedback ...

@Petermarcu tables merged - what do you think now? Let me know if you have suggestions for further tweaks.

I think we can go with this.


Cannot close miniterm

VSCode Version: Version 1.11.0-insider (1.11.0-insider)
OS Version: Mac OSX Sierra

Steps to Reproduce:

I've got pyserial and miniterm installed together with ESP32-IDF (https://github.com/espressif/esp-idf)
If I want to compile and flash and run a console I need to issue

make flash montitor
on a terminal prompt. This works both, on native terminal and integrated terminal. But I can't finish the terminal after debugging. The key combination is CTRL-[, which is CTRL+ALT+6 on a Mac. This closes the miniterm window in native terminal, but not in integrated terminal in VSCode

@neilyoung do you know what ctrl+[ is meant to do in your case exactly? xterm.js appears to act the same as gnome-terminal in a regular bash prompt.
You could try ctrl+c (SIGINT) or ctrl+\ (string terminator).

I'm sorry, my fault. It is actually CTRL+]
Description here: http://esp-idf.readthedocs.io/en/latest/idf-monitor.html?highlight=miniterm

Please provide additional repro steps if you're still having this issue.


Setter function fails silently if called with incorrect type, assignment ignored without warning

Godot version:
3.3 stable

OS/device including version:
Windows 10

Issue description:
If the input type is defined for a setter function, it fails silently if given another type . No error is thrown, no warning given, and the assignment is completely ignored. Note that this only occurs if the setter is called implicitly, see steps below.

Steps to reproduce:
Upon defining a class like so:
class_name ObjectTest

var test_var : int = 10 setget test_set, test_get

func test_set(input_string : int):
	print("Set invoked")
	test_var = input_string

func test_get():
	print("Get invoked")
	return test_var

And calling from another script like so:
func _ready():
 	var obj = ObjectTest.new()
 	obj.test_var = "Hello"  # IGNORED
	print(obj.test_var)

The output is:
Get invoked
10

Note that "Set invoked" is not shown, so the setter has not run at all.
Project zip:
setter_bug.zip

Fixed by #54117.


Update ACFloatingTextfield.swift

Change framerror size height to 15, since current label text is cut

Thanks! i will look forward to this ASAP.
On Wed, 5 Apr 2017 at 5:02 PM, dav3harry<EMAIL_ADDRESS>wrote:

Change framerror size height to 15, since current label text is cut
You can view, comment on, or merge this pull request online at:
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/16
Commit Summary

Update ACFloatingTextfield.swift

File Changes

M Source-Swift/ACFloatingTextfield.swift
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/16/files#diff-0
(2)

Patch Links:

https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/16.patch
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/16.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/16, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANMZ106rWR9Np-B068k0i1CdjEryxdSSks5rs3vegaJpZM4M0HOa
.

--
Thanks & Regards
Er. Abhishek Chandani

Remove the placeholder label color for error state


Missing Azure DevOps related page

Several customers use Microsoft Azure DevOps as their developer platform and it would be valuable to also provide support for those scenarios.

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

ID: 9659eab1-b7f7-d163-ff0a-577b7496b5c8
Version Independent ID: 0892de76-78f8-4c1c-c09c-60af3f275320
Content: Handle AKS node upgrades with GitHub Actions - Azure Kubernetes Service
Content Source: articles/aks/node-upgrade-github-actions.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

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

@pmsousa
Would you kindly explain on the question in order to make it clearer what is missing from Azure DevOps in relation to the pages, So that we can help you even more.

Hey, @SaibabaBalapur-MSFT.
So, we have this page on handling AKS node upgrades using GitHub actions. Several of our customers also/only use Azure DevOps and I think a docs page should also be provided to fill the gap for those customers also.
Looking at this content, it seems like a page for DevOps pipelines would be easy to add.
Does this clarify a bit what I ment?

@pmsousa
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@MGoedtel
Can you please check and add your comments on this doc update request as applicable.

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The suggested scenario has not been added to the article since the creation of this issue, and the timeline for resolution may vary based on resourcing. This article is something that we've had on our roadmap and we have an internal work item where I've incorporated your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


change import syntax

Fixes #215

Will it take more memory?

Not noticably. LLVM stuff (creating LLVM IR, optimizing IR, writing IR to object files) takes up much more memory than the rest of the compilation, and this doesn't affect the LLVM IR much.


Add support for IKEA LED1925G6 and LED1949C5

Hi.
Please add support for two new IKEA lights, measured with Shelly Plug S.
Regards,
Chris

Nice!


iOS - Bindable property on BackgroundGradientStartColor BackgroundGradientEndColor doesnt update

Hello,
I try to bind BackgroundGradientStartColor and BackgroundGradientEndColor on my ViewModel, to change the Color runtime while some events occours.
In Android id works fine,
but in iOS the PancakeView doesn't change the colors.
I dont know if other properties have the same problem
Xamarin Forms 4.0 sr2
My PancakeView is in a ViewCell
All packages update to latest version.
Thanks

Hello,
trying to help in some way, I make a sample project
In this sample I make a listview with 3 cells.
In each cell there is a button. The button change the model property: StartColor, EndColor and BorderColor.
The results:
Android: all is working as expected
iOS: Only BorderColor change in the cells...background gradient color fails to change
Thanks!
pankake.zip

Before using your PancakeView, I used one made by me
Now I am using PancakeView because is more complete comparing mine, but after finding this problem, I checked if it was present in my old renderer
Curiously it was also present, here's how I solved it:
Before InsertSublayer, i search and remove the existing one (if present):
            var gradLayer = Layer.Sublayers.Where(x => x.GetType() == typeof(CAGradientLayer)).FirstOrDefault();
            if (gradLayer != null)
                gradLayer.RemoveFromSuperLayer();

            NativeView.Layer.InsertSublayer(gradientLayer, 0);

I dont know if its the best way to do it

Seems like it could work. I'll try it out soon, or feel free to PR the change in :)

Sorry, I am not very practiced with github...
I made a clone of repository but the solution give me some compilation error
Maybe is better to wait for your update :)
Thank you

Try out 1.0.25-beta, it should solve this.

Unfortunately it still doesn't works


ERROR: Unable to extract uploader id;

Linux Mint 21
youtube-dl https://www.youtube.com/watch?v=SzP2VFieJM
give
[youtube] SzP2VFieJMo: Downloading webpage
[youtube] SzP2VFieJMo: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
youtube-dl is up-to-date (2021.12.17)

sudo youtube-dl -U
youtube-dl is up-to-date (2021.12.17)

#30839 !


feat: add Linux support

Issue #, if available:
Description of changes:
Adds Linux support to Finch by creating a new "native" code path that doesn't use Lima and instead relies on the system packages for nerdctl/containerd/etc. being properly configured.
TODO in this PR:

[x] Fix e2e tests
[x] Add environment variable overrides for nerdctl config and buildkit socket paths

Because this diff is already huge, TODO in followup PR(s):

Add e2e tests for Linux

Right now, they're all excluded from building to avoid fixing lint errors and refactoring

Increase Linux/native specific unit test coverage: https://github.com/runfinch/finch/pull/1058
Fix semantics around what toggling the soci snapshotter or ecr-credential-helper actually means on Linux (e.g. toggling environment variables or config files)
Fix what logs a support bundle should include on Linux (e.g. dumping journalctl)

Testing done:

[x] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

squash the commits may be unless they have actual feat that are separable

will finch run all its command as sudo?

squash the commits may be unless they have actual feat that are separable
We do that automatically on merge

will finch run all its command as sudo?
Finch itself doesn't have to run commands as sudo, but right now since we don't support rootless, user's will need to either be root, use setuid, or run sudo finch for Finch to read it's config file and execute nerdctl commands successfully

LGTM in general, can you address the test failures?

LGTM in general, can you address the test failures?

Re-ran the tests