Add more tests which use ast_plus_one_deps_strict_deps_unused_deps_error toolchain

Description
ast_plus_one_deps_strict_deps_unused_deps_error toolchain has more coverage and first class
This PR moves //scala:ast_plus_one_deps_strict_deps_unused_deps_error toolchain to the prod section since we now promote it as the alternative but leave users up to stitching everything themselves. I also added reference to it in the readme.
Additionally it builds and tests test/... with this toolchain to surface issues like #1029.
Motivation
Better coverage for AST and other combo and better alignment with our approach.
Open issues

Should we choose a more concise name for the toolchain? Right now it's honest and complete but I'm not sure we should stick to it (wasn't sure what the abstraction is OTOH).
I needed to add quite a few unused_dependency_checker_ignored_targets exclusions to get the build passing. I'd really love for us to understand if they are working as intended, bugs we can fix or bugs we'll have to live with.
We might be able to merge this PR even with these exclusions but I'm willing to do that only if it's clear who's owning the rest of the inquiry.

@Jamie5 I'd love your review here

probably obvious but I'll just add that I added the above unused_dependency_checker_ignored_targets only after I saw they are required (that is the build fails with them and without them on different dep combos)

linting fails. Pushing a fix. Otherwise green

Looks reasonable to me. I may be able to look at the false positives to see what's going on (don't have much context on these rules but hopefully can at least get something useful out of it). Not sure of timeline though, but if there's a tracking thing I can deal with it at some point. As for name maybe exact_deps? Since the goal of ast/+1 is to include only the deps found in the source, and the strict_deps error mode/unused_deps error mode also make we don't have random unneeded deps, nor not including deps which are found in the source. So I guess the high level is having deps exactly match up with the source. (only thing that may not fully is plus-one as opposed to transitive, but eh)

Great.
With respect to false positives- I’ll open 3 PRs for each of the issues so you can see them?
With respect to the name- “minimal_source_deps”? “exact_source_deps”? Because the big diff between direct and plus one is that we’re trying to focus about source deps and between transitive and plus one is that we’re trying to be minimal

Sounds good re opening PRs (or issues is fine too). Re name exact_source_deps I slightly prefer because we're not quite going for minimal or we wouldn't care about strict_deps, and in the transitive vs plus one, we're trying to be minimal about deps implicitly brought in, not necessarily source deps. But again not a strong opinion here.

I also don't have a really strong opinion but I think this discussion has a bit more value in it.
If you feel you've exhausted it feel free to say so :)
I think we're trying to fit a few different dimensions here which makes naming hard :)
WDYT about minimal_direct_source_deps? The reason I feel uneasy about exact is the use-cases of more than one level (i.e. deep inheritance) which means that you will need to bring in a dep you don't need source wise. This is why I feel minimal tries to give a bit more context there.
WDYT?

oh and re PRs vs issues- The reason why I'm thinking PRs is so that you can have easy repros. If this isn't valuable then I'll just open issues. LMK.

Hmm minimal_direct_source_deps sounds reasonable to me. I guess PRs might be helpful then if it's not much trouble for you.


flax still uses deprecated jax.tree_util.register_keypaths api

Error message every time I import from flax:
/venv/lib/python3.10/site-packages/flax/core/frozen_dict.py:169: FutureWarning: jax.tree_util.register_keypaths is deprecated, and will be removed in a future release. Please use `register_pytree_with_keys()` instead.

Hi, are you using the latest version of flax? This shouldn't happen after 0.6.7.

looks like I'm still on 0.6.7, will try updating


Add support for relative distribution URIs in the Wrapper task

Fixes #25252
Support relative URIs (URIs with no scheme specified)

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

@bot-gradle test and merge

WARN: The milestone of this PR is 8.2 RC2, but the base branch version is 8.3

@bot-gradle cancel

I've cancelled the pre-tested commit build.

Pre-tested commit build failed.

@bot-gradle test and merge


Ommit showing images that are filtered from client

Some sites filter requests to content linked from other sites, in order to reduce bandwidth.
This issue affects linker behaviour, because the images are first fetched by linked, but afterwards they are broken in the user-agent.
I propose requesting images in the first place with a referer header.

This is happening with this link: http://demo-social-stream.dit.upm.es/links/92

This is weird. Most of the time my browser does not load the image, but sometimes....

This also happens if I type the image URL at the browser (i.e. fetch without a referrer): Most of the time I get a 403, but sometimes I just get to see it.
OTOH if I try hotlinking the image:

<title>Image hotlinking test</title>

Try hotlinking the image:

It works! Please notice that this is similar to what linkser does when determining whether to validate an image.
This leads me to believe that said site has some weird referrer policy, maybe with a random element to increase the fun. If this is the case, we should probably blacklist the site and avoid including images from its domain. They obviously don't want us hotlinking their content (which makes me wonder why they are using OpenGraph in the first place).

My bad, hotlinking this pic does not work from any of my sites either (yet sometimes I see it in social_stream demo).
I still don't see why linkser reports it. We'll probably have to debug. Do you have examples from any other domain?

Not so far. Things working properly in the rest of cases. Maybe it is their fault, because I also can see it sometimes. We can close this until we have more examples.


Resume swipe after user touches on image

I have getting an problem. Whenever i  click on any image while the layout is swiping, swiping stops and i have to resume it manually. How to solve this?

Well I think this is a feature that was added so if the user wants to navigate the slideshow themselves, the autoswipe doesn't interrupt them. To disable this, you can user a custom slider layout class that extends SliderLayout and Overrides onInterceptTouchEvent with an empty function.
The class would look something like this
public class CustomSliderLayout extends SliderLayout {

    public CustomSliderLayout(Context context) {
        super(context);
    }

    @Override
    public boolean onInterceptTouchEvent(MotionEvent e) {
        return false;
    }
}

Then you'd just use this class in your layout.
       <com.yourdomain.yourpackage.CustomSliderLayout
            android:id="@+id/explore_details_slider"
            android:layout_width="match_parent"
            android:layout_height="200dp"
            custom:indicator_visibility="invisible" />

Thankyou so much sir. I will try that. I just wanted the layout to pause
when clicked otherwise i needed it to resume automatically. Thanks again !!
On २०१७ जनवरी २, सोम at ८:४४ अपराह्न Nicholas Mattiacci <
<EMAIL_ADDRESS>wrote:

Well I think this is a feature that was added so if the user wants to
navigate the slideshow themselves, the autoswipe doesn't interrupt them. To
disable this, you can user a custom slider layout class that extends
SliderLayout and Overrides onInterceptTouchEvent with an empty function.
The class would look something like this
public class CustomSliderLayout extends SliderLayout {
public CustomSliderLayout(Context context) {

    super(context);

}

@Override

public boolean onInterceptTouchEvent(MotionEvent e) {

    return false;

}

}
Then you'd just use this class in your layout.
   <com.yourdomain.yourpackage.CustomSliderLayout

        android:id="@+id/explore_details_slider"

        android:layout_width="match_parent"

        android:layout_height="200dp"

        custom:indicator_visibility="invisible" />

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/daimajia/AndroidImageSlider/issues/278#issuecomment-269982946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM-Jvcmtjzd7fTZK5omMXb7Ndw8z9aq6ks5rORDfgaJpZM4LHhkG
.

I try implement your way @RishavChudal , but got an error.
Logcat:
android.view.InflateException: Binary XML file line #9: Binary XML file line #9: Error inflating class com.xdec.banner.CustomSliderLayout
MainActivity.java
CustomSliderLayout sliderLayout;

private void setupViews() {
        sliderLayout = new CustomSliderLayout(this);
        sliderLayout = (CustomSliderLayout) findViewById(R.id.slider);
}

activity_main.xml
    <com.xdec.banner.CustomSliderLayout
        android:id="@+id/slider"
        android:layout_width="match_parent"
        android:layout_height="300dp"/>

Please help.

@nizzam I think when you make a custom library from other's library, you just need to have/copy all the files/codes from the pre-built library in your custom class/classes and then you can edit as you like. After doing that, it shouldnt give any error in inflating your custom class.
Hope I gave you a good suggestion.

public void onSliderClick(BaseSliderView slider)
{
sliderLayout .startAutoCycle();
}


1r1s cmd

@alulsh this PR merges your init-command branch and my 1r1s branch, modifies the default cfn-config cli interface for lambda-cfn init and sketches out some places for the toboggan deploy. I'd like to rebase this before merging to clean up the git history, but want to get your eyeballs on it asap.

Nice work deleting integrating my PR in https://github.com/mapbox/lambda-cfn/pull/48 as well as the advanced features of cfn-config @zmully! Having one integrated lambda-cfn command rather than multiple ones will be easier for users and as well as maintainers.
Discussion point: allow for setting CFN_CONFIG_BUCKET and CFN_CONFIG_TEMPLATE_BUCKET environment variables in lib/cli.js. We're not currently checking for this environment variable and instead only accepting a value passed in via a flag or the default cfn-configs-$AWS_ACCOUNT_ID-region. I find it annoying as a user to have to pass in the values for these every time. I also may not want to use the default naming convention we have in place here. Is it a high level of effort to add checking for these environment variables in lib/cli.js?
Currently the help docs for the lambda-cfn CLI mention setting this environment variable and the tests for lib/cli.js check for this value (see https://github.com/mapbox/lambda-cfn/pull/51/files#diff-da8fe3f2391da380ddef00afcc0f93f0R29). If we decide we don't want to use these environment variables at all then we should remove it from both the help text and the tests.
Otherwise I didn't see any major red flags here, only minor issues. I left line notes but will also summarize some of my findings here.

Use strict equality (===) and inequality (!==) wherever possible instead of abstract/loose comparison (==, !=) operators.

I left line notes for the instances of abstract checks I saw in lib/cli.js but I also saw several instances in lib/cfn.js that I did not leave line notes for. I'd search for == in lib/cfn.js and replace with === wherever possible.

Delete files no longer needed - lib/deploy.js, bin/lambda-cfn-deploy.js, etc
Use console.error() and console.log() in bin/lambda-cfn.js (see https://github.com/mapbox/lambda-cfn/pull/51/files#r132270052) for friendlier, cleaner messages to users

We should also re-institute a linter for consistent code style but we should do this in a separate PR after merging this one.

Per discussion with @zmully in scrum, I added a linter to this PR in https://github.com/mapbox/lambda-cfn/pull/51/commits/4baba826b3312f2a4458a2d31eec87f220af64b8 rather than do it in a separate step later. This is so @zmully could clean up code style and linting issues while doing other code clean up.
Some notes about the addition of linting:

I went with eslint rather than another linter because I only wanted to add one module and wanted to keep the set up simple. I didn't want multiple modules or plugins.
I borrowed and adapted an .eslintrc.js file from another project I've worked on (https://github.com/dcfemtech/hackforgood-waba-map/blob/master/.eslintrc.js) as a starting point. The list is quite long though @zmully so we can definitely remove any problematic rules that hurt more than help.
The current eslint rules are a mix of consistency and legibility issues (no tabs, 2 spaces for indents, single quotes) as well as a couple static analysis rules for potential bugs or unused code (e.g. no-unused-vars)
I had to disable the single quotes only rule in a few lines of lib/init.js since package.json files are required to use double quotes. I got this usage from https://stackoverflow.com/a/34771423.

@zmully - did a review of your function spec doc and found a couple minor issues, but otherwise looks great!
I also added docs on support for the CFN_CONFIG_BUCKET and AWS_REGION environment variables. We may also want to consider adding support for a CFN_CONFIG_TEMPLATE_BUCKET environment variable as well, or we can table this for a later release.


Arrow is not visible when item is selected

Check Atom Material Icon Settings, This basically happens for all dropdowns with an arrow.

This problem seems to come from the icon plug-in. The same problem exists when I use the default light theme.

Do you mean default phpstorm thing ? Because i dont have any icon plugin installed still issue persists.. That means its something wrong with the dropdown colour pallet of this theme!!

I mean, when the icon plug-in is used, the Github3 theme behaves the same as the default Intellij light theme. The following figures are for comparison.
What they looks like before installing the plugin:

And what they looks like after installing the plugin:

What could be the fix? If you dont have time guide me.. I will raise PR

When I uninstalled the icon plug-in, this problem disappeared.
To be compatible with the icon plug-in, the solution I can think of is to modify the background color of the selection.
If you have a better plan, PR is welcome.

Actually it was problem with icon plugin when i removed it it just works .. But still thnx


App stops responding (Only Android)

App becomes Non-Responsive
What's happening?

App stops responding and crashes in Android
How can it be reproduced?

When the code is present, app crashes sometimes (only Android, works fine in iOS). When I comment the code, app works fine.
Build details?

Android

@moaazsidat

after following the installations instructions my app doesn't launch. I have added some maven repositories too because react-native-camera dependency was crashing

Same here. App works fine on iOS device but on Android version 7.0 if the app is sent to the background it will stop responding. Tested on Android Oreo 8.0.0 works fine.

@Taglohner same here，works fine on IOS ，but on Android Oreo8.0 not work for me

The app stops responding whenever making live changes to the screen.
Other apps also stop responding and I have to restart my Android device.


host_exerciser: fix buffer comparison in interrupt test mode

Buffer comparison varies by FPGA configuration. Stop calling direct memory comparison in interrupt test mode. The caller of he_interrupt() already does the proper test.
Tests run:
Gen5x16 FPGA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.641%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
15826

Relevant Lines:
24483

💛  - Coveralls


Delete DeltaConnection

After PR https://github.com/microsoft/FluidFramework/pull/3904 is in, remove DeltaConnection - it is simple pass-through wrapper that does nothing.

Vlad did this in #3937


Units for y axes to group them together

Summary
My only problem with this awesome library so far, is that having multiple series of which some but not all are sharing the same unit makes you chose between

using only one y axis with no labels or units displayed
using multiple y axes, hide all but one with the same unit and

using fixed min and max values for all y axes with the same unit
implement a function that finds minY and maxY depending on the data (filtered by minX and maxX) and apply these to all axes using the same unit

This could be solved relatively easy by grouping axes together and use the existing functionality that dynamically calculates the minY and maxY but instead of doing this per y axis, doing it using a new property that can be used on multiple y axes.
API Changes
A new property for y axis configuration called unit.
{
  options:{
    yaxis:[{
      seriesName:'Living room',
      unit:'temp',
      show:true,
      showAlways:true,
      labels:{
        formatter:val => (typeof val === 'number' ? val.toFixed(1) + '°C' : '')
      }
    }, {
      seriesName:'Bed room',
      unit:'temp',
      show:false,
      labels:{
        formatter:val => (typeof val === 'number' ? val.toFixed(1) + '°C' : '')
      }
    }, {
      seriesName:'Outside',
      unit:'temp',
      show:false,
      labels:{
        formatter:val => (typeof val === 'number' ? val.toFixed(1) + '°C' : '')
      }
    }, {
      seriesName:'Wind',
      unit:'wind',
      show:true,
      showAlways:true,
      opposite:true,
      labels:{
        formatter:val => (typeof val === 'number' ? val.toFixed(1) + 'km/h' : '')
      }
    }]
  }
}

Intended Use Case
This would be a simple way to use the same unit for multiple series without losing the benefit of dynamic min and max values when zooming or hiding series. This also seems the problem mentioned in #91 (that is closed for some reason, probably because "The design decision to keep the y-axis correlate with the series was taken pretty early on. So, a big chunk of code will need a rewrite if I change this.")
I think my solution might be a good way to solve that problem without having to rewrite that much code. For my specific needs this code (slightly simplified and adapted version because I'm using Vue3) did the job:
function setMinMax() {
  var {minX, maxX} = chartContext.w.globals, units = {};
  data.forEach((series, i) => {
    var unit = options.yaxis[i].unit;
    units[unit] = units[unit] || {};
    if (chartContext.series.isSeriesHidden(series.name).isHidden) {
      return;
    }
    series.data.forEach(d => {
      if (d.x < minX || d.x > maxX) {
        return;
      }
      if (units[unit].minY === undefined || d.y < units[unit].minY) {
        units[unit].minY = d.y;
      }
      if (units[unit].maxY === undefined || d.y > units[unit].maxY) {
        units[unit].maxY = d.y;
      }
    });
  });
  Object.values(units).forEach(unit => {
    unit.minY = Math.floor(unit.minY);
    unit.maxY = Math.round(unit.maxY + 1);
  });
  options.yaxis.forEach(yaxis => {
    yaxis.min = units[yaxis.unit].minY;
    yaxis.max = units[yaxis.unit].maxY;
  });
  chartContext.updateOptions({
    yaxis:options.yaxis
  });
}

Ofc I simply could use that code for every chart that has this problem but a general solution for everyone would improve this library even more. Thanks a lot for your great work, for reading this and considering an implementation.

For those who are wondering, like me, how to set min and max for each yaxis without above function:
yaxis: [         {             showAlways: true,             min: 0,             max: 100,             axisTicks: {                 show: true,             },             axisBorder: {                 show: true,                 color: '#008FFB'             },             labels: {                 style: {                     colors: '#008FFB',                 }             },             title: {                 text: "Temperature (˚F)",                 style: {                     color: '#008FFB',                 }             },             tooltip: {                 enabled: true             }         },         {             min: 0,             max: 100, show: false         },         {             min: 0,             max: 100, show: false         } ]
each obj is order dependent, you can tune min max by trying whatever suits your data requirements.


EXO-AtpPolicyForO365 Issue when not Enabled

Details of the scenario you tried and the problem that is occurring
Running a configuration with EXOAtpPolicyForO365 on a tenant that is not configured for it throws the following error:
The term 'Get-AtpPolicyForO365' is not recognized as the name of a cmdlet
We should either modify this resource to provide additional information to the user on how to configure the tenant properly to enable ATP or create anew resource that will allow the configuration to enable ATP all together.
Verbose logs showing the problem
N/A
Suggested solution to the issue
N/A
The DSC configuration that is used to reproduce the issue (as detailed as possible)
N/A
The operating system the target node is running
N/A
Version of the DSC module that was used ('dev' if using current dev branch)
N/A

This should now be handled via the EXOOrganizationConfig resource


[CLOUD_NATIVE] Cache data not deleted when execute drop table force stmt

Steps to reproduce the behavior (Required)
Create some cloud_native table with default cache settings, load data into the table, then execute drop table xx force
Expected behavior (Required)

remote object storage table path deleted
local disk cache deleted

Real behavior (Required)

[done] remote object storage table path deleted
local disk cache not fully deleted, only a small fraction of disk space released.

Before drop the table...

After drop the table...

ON be node, I can find data directory remains there on storage/starlet_cache path..
StarRocks version (Required)
Latest version build from main branch on 07-20.

@sduzh please take your time to review this issue, thanks

Yes, you're right, with the current implementation, dropping a table does not guarantee that all cache files on the local disk will be deleted. This is because, with the current cache implementation, there is no straightforward way to determine which table a cache file on disk belongs to.

Yes, you're right, with the current implementation, dropping a table does not guarantee that all cache files on the local disk will be deleted. This is because, with the current cache implementation, there is no straightforward way to determine which table a cache file on disk belongs to.

Thanks for your explanation:-)
Two more questions:

Did you have any plan to solve this problem?  Since it would introduce unnecessary local disk cost.
For current implementation, will the cache files of dropped tables be deleted when cache TTL time reached?

We do have plans to work on this optimization, but to be honest it is not high on our priority list right now. So we don't have a specific timeline for it at the moment. Also, there is no longer a TTL in place, so deletion based on TTL will not happen.

Yes, you're right, with the current implementation, dropping a table does not guarantee that all cache files on the local disk will be deleted. This is because, with the current cache implementation, there is no straightforward way to determine which table a cache file on disk belongs to.

Thanks for your explanation:-)
Did you have any plan to solve this problem? Since it would introduce unnecessary local disk cost. For current implementation, will the cache files of dropped tables be deleted when cache TTL time reached?

We do have plans to work on this optimization, but to be honest it is not high on our priority list right now. So we don't have a specific timeline for it at the moment. Also, there is no longer a TTL in place, so deletion based on TTL will not happen.


Purplecoat.js

https://ellekasai.github.io/purplecoat.js/

I think we'll pass on this one. Thanks for the suggestion.


RT4669: dgst can only sign/verify one file

Check arg count and print an error message.
This is the 1.0.2 version of #1530

+1

I did +1 this, but I take it back temporarly on the grounds that this change looks very different from the master one...  I'll get back when I've had a chance to look around..

Ok, I've looked at it and at #1530, and it seems to me that the latter should be possible to apply fairly easily on 1.0.2 as well.  I find the #1530 change much easier to read and understand than this one.
So, -1 on the current solution here, please make it more like #1530.

okay, updated.

And why not place the change in about the same spot as in #1530?  Line 245?

Because I could not follow the twisty flags and such, and thought that putting it right before the argc loop was the most sensible.

Ok, would you please align #1530 and this PR then?  I don't really care if you put this change just after the argv parsing look or just before the file fetching loop (which is what you call "the argc loop"), just that they are the same.  That will make future maintainance easier, if such is needed.

No.  I think each diff makes sense.  You disagree, fine, let someone else do it ☺

Ok.  #1606


crypto/internal/bigmod: fix panic [freeze exception]

#60411 was only filed today, as it was blocked by an internal security investigation. The CL (http://go.dev/cl/498035) was sent & reviewed, but just slipped the deadline for the freeze. The change is small, tested, and reviewed, so I don't think it poses a significant risk.
cc @golang/release

My understanding is that actual bug fixes (and doc changes) are OK at the moment.  Freeze exceptions are required for new features.
(Please correct me if I am wrong.)

Oh I completely misread the change in the freeze policy, and this makes complete sense 🤦, ignore me!


Sync support windows changes

This PR updates the minimum OTP badge version and Erlang/OTP and Elixir versions for asdf.

Thanks for this, @kianmeng.
@starbelly, if you wanna further automate this, I propose Renovate, e.g. as per https://gist.github.com/paulo-ferraz-oliveira/68e04915037d1e78eb815a90f491f97d. I use it in many of my repos and it works like a charm.

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


I encountered some problems during test

(maskrcnn) wuyi@nclab:~/github/maskscoring_rcnn/tools$ python test_net.py
Traceback (most recent call last):
File "test_net.py", line 12, in
from maskrcnn_benchmark.engine.inference import inference
File "/media/dat1/users/master/2019/wuyi/github/maskscoring_rcnn/maskrcnn_benchmark/engine/inference.py", line 20, in
from maskrcnn_benchmark.structures.boxlist_ops import boxlist_iou
File "/media/dat1/users/master/2019/wuyi/github/maskscoring_rcnn/maskrcnn_benchmark/structures/boxlist_ops.py", line 6, in
from maskrcnn_benchmark.layers import nms as _box_nms
File "/media/dat1/users/master/2019/wuyi/github/maskscoring_rcnn/maskrcnn_benchmark/layers/init.py", line 8, in
from .nms import nms
File "/media/dat1/users/master/2019/wuyi/github/maskscoring_rcnn/maskrcnn_benchmark/layers/nms.py", line 3, in
from maskrcnn_benchmark import _C
ImportError: /media/dat1/users/master/2019/wuyi/github/maskscoring_rcnn/maskrcnn_benchmark/_C.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _Z20ROIPool_forward_cudaRKN2at6TensorES2_fii
I'm confused，can you  help me with it?

Have you install maskrcnn_benchmark "python setup.py build develop"

Yes，i have done that.

I also read the Troubleshooting.md from maskrcnn-benchmark.
My nvcc is “Cuda compilation tools, release 7.5, V7.5.17”,but my cuda is 8.0,Could it be caused by this?I don't have root access，I don't know how to modify nvcc.

I think the problem may be cuda 8.0. You can install cuda 9.0 locally and add to the env.

@WY51BE hello, I am also confused about this problem, have you solved it? thanks in advance.


Switch log file location to local ytermusic files

When I run the program from ~/ that is where the log is saved, /var/log/ytermusic would be a better location or %APPDATA%\ytermusic on windows

I've changed the log directory to the cache dir since the directiories crate doesn't support log dirs. If you have any other better idea...


Referenced object [objectname] was not found.

I'm using Deploy Azure Data Factory by SQLPlayer extension to Azure DevOps and very happy with it!
I have separated Data Factory objects into "common" and project specific so I can develop and deploy those independently. I have built separate build pipelines and releases for the "common" and the project where I filter which objects I want to deploy in build pipeline PowerShell task by name (like dataset/ds_common* and so on). So when release for "common" is triggered it gets artifacts from build pipeline with only "common" objects. The project objects have dependencies on "common" objects but my idea is to deploy "common" before project so the objects project is dependent on should already be deployed to the data factory.
Build pipeline for the project is packing only project specific artifacts. By the time the project release is triggered "common" objects are already deployed to the data factory. However I still get "Referenced object ["common" objectname] was not found." in the project release.
I was able to fix the issue by including even "common" objects in the project artifacts but excluding them from deployment in the project release pipeline (Include/Exclude Filtering Text section of the task). I guess there is some kind of validation happening before the objects are deployed to a data factory that produces the error even so the object error is referencing to is already deployed to the data factory. Is there any way to disable this validation?

You will be able to set the following option to replace an error by warning, hence the process will continue.
$opt.IgnoreLackOfReferencedObject = $true

Previous behaviour:

Behaviour when the mentioned option is set to true:

This feature will be deployed in the nearest release.


Add more projects to docs/projects

Added projects:

Koululaskuri
Abitti OpenAccess

related to: #47

@raikasdev Is this text correct? Please review it.

It is correct, but would like it to be more "formal" for the association homepage.


Update ResourceNotificationServiceLogging

Fixes #4686
Microsoft Reviewers: Open in CodeFlow

Is there any chance that we'll get two resource notifications for a resource being processed in parallel. I'm wondering if there is potential for resource states to rapidly oscillate and then we don't fire them.

Is there any chance that we'll get two resource notifications for a resource being processed in parallel. I'm wondering if there is potential for resource states to rapidly oscillate and then we don't fire them.

The publish method takes a lock on the state, so all updates for a resource get effectively sequentialized anyway, so while races might result in "extra" transitions being logged, I'm not sure any can be missed, e.g.

no state -> state1
state2 update comes in
state1 update comes in
state2 and state1 updates race such that state1 gets the lock first
no update is logged for state1 update as resource is already in that state
state2 update is logged as transition from state1 to state2

If in step 4 state2 update gets the lock first, then state1 to state2 transition is logged, and then state1 update logs transition from state2 to state1 (the "extra" update).
If we wanted to solve that we could use a Channel or similar but ultimately calls to Publish still race each other until the sync point.


Add SDK for EventHub and ServiceBus

Can one of the admins verify this patch?

Hi @sazeesha, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (sazeesha). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@markcowl : I have created a new PR here.

@azuresdkci add to whitelist

@sazeesha Please split this up into separate PRs for ServiceBus and EventHub

Can someone review this?

Thank you!


Created the save_prediction option

created the save prediction option inside the badminton.py

Hey!
Just a correction, the input image is not always a path, it can be a loaded array also.
So you would need to check for that too.
And just a suggestion : Try saving it using opencv, by making a copy of original image(it is already done as out_img)


Update HH dataset on helpful only subset

This PR updates the scripts to use the helpful only subset of the anthropic's HH dataset.
https://huggingface.co/datasets/trl-internal-testing/hh-rlhf-helpful-base-trl-style

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Having trouble running the sample in multitenant mode.

VS2013.4: The sample page says that it can be run in any tenant, but when I run the sample locally and log in with<EMAIL_ADDRESS>I get this error:
Additional technical information:
Correlation ID: 0c3e4797-929e-47ed-9fd7-309e3e084e94
Timestamp: 2015-09-30 21:45:28Z
AADSTS50020: User account<EMAIL_ADDRESS>from external identity provider 'live.com' is not supported for application 'd43fe016-2ce2-4686-ad31-9169be137d72'. The account needs to be added as an external user in the tenant. Please sign out and sign in again with an Azure Active Directory user account.
VS2015: The HTTPS link won’t even work when I tried to test it locally.

Any ideas?

Did you add your hotmail account as a guest user in your tenant?

Also by request, I meant the HTTP request that is sent to Azure AD.  You can find it by copy+pasting it out of the URL bar before attempting to sign in, or by capturing a trace in Fiddler or similar

I'm using the application in its default configuration, trying to use the multitenant mode. The user I used to sign in with is the service administrator of my default AAD tenant, so I don't need to add it as a guest account, right? The instructions don't say anything about adding a guest account.

The URL at the sign-in page is: https://login.microsoftonline.com/common/oauth2/authorize?client_id=d43fe016-2ce2-4686-ad31-9169be137d72&response_mode=form_post&response_type=code+id_token&scope=openid+profile&state=OpenIdConnect.AuthenticationProperties%3DwpZT0M3Mz3WQWeGrH8rpkAAz2fDFaQVX4kZKqjl26c0SDWZLVa0pVU0pj-lv9oKAdvYMjS3RDM00lokMGVSANPftTEYLY8bJQNjWAixGFkw&nonce=635794053963374959.NjMyYzQ3NWEtNDQzNi00Yzc1LWE1NzQtZGQwMzg4ZmE1YWJkNjQ5NzM5NmUtMzllZi00M2JlLThiM2EtNjJkY2Q1NDYwOTZh

To confirm - which tenant is the application registered in?

If I use the default settings, I suppose the application is registered in the Microsoft tenant, right?

Ohhhhh I totally forgot that this app already has default settings. My fault.  Ok no, the default app is registered in my own tenant actually.  I get it now.
MSA guest account users will not work with multi-tenant apps.  Your MSA user could be a guest in N AAD tenants - and when the sign in request goes to the /common endpoint, AAD cannot determine which tenant to sign the user into.  That's why you get that error.  If you sign into the app with a work account in your tenant, it should work fine.
I'll add a comment to the readme about MSA users not being supported.

a38ded23d9122a8b7662bb4e306aff12c6199c6f

If you try it and it works, please close the issue.

It works now. Thanks!


Remove pilots

As an airline manager, I can remove pilots from our workforce, so that I can remove them from the flights.

Implemented


fix fonarcloud report

Description

Bugs
Vulnerabilities
Code Smells
Duplications

Not Code Coverage
Related links

https://sonarcloud.io/projects

not finish

https://github.com/yurake/k8s-3tier-webapp/runs/598042432?check_suite_focus=true
[jaxrs-activemq-quarkus] native build error
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]    classlist:  10,895.37 ms,  2.05 GB
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]        (cap):   2,046.87 ms,  2.05 GB
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]        setup:   4,338.88 ms,  2.05 GB
16:01:59,979 INFO  [org.jbo.threads] JBoss Threads version 3.0.1.Final
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]   (typeflow):  49,740.18 ms,  3.32 GB
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]    (objects):  37,344.82 ms,  3.32 GB
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]   (features):   1,116.74 ms,  3.32 GB
[jaxrs-activemq-quarkus-1.0.4-SNAPSHOT-runner:3098]     analysis:  90,085.78 ms,  3.32 GB
Error: No instances of sun.security.provider.NativePRNG are allowed in the image heap as this class should be initialized at image runtime. To see how this object got instantiated use -H:+TraceClassInitialization.
Detailed message:
Trace: Object was reached by


Allow using JDA without gateway connection

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[x] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This allows using JDA without any gateway connection.
Example
fun main() {
    val jda = JDABuilder.createRest(token).build()

    val route = Route.Messages.SEND_MESSAGE.compile(CHANNEL_ID)
    val body = DataObject.fromJson("""{"content": "Testing"}""")
    RestAction.makeAction(jda, route, body)
            .map(RestPayload::toString)
            .queue(::println)
}

I believe we close the response body before calling the rest action callback so a handler setup needs to be implemented to properly use this.

To be considered:

Remove the RateLimiter abstract class, we only need one implementation
Add an interface for the RateLimiter to make it easier to analyze usage
Add a way to track bucket statistics such as queue size

This PR is getting discontinued for now.


Future of Z-Wave

There has been a lot of awesome work been done lately on Z-Wave by @armills, @andrey-git and @turbokongen to ensure stability and reliability.
Based on #6813 and the recent addition of Z-Wave nodes, I think that we should talk about creating a shared vision on how our ultimate Z-Wave integration looks like. We can then start taking steps towards making that a reality.
Here are just some of my thoughts

Full test coverage #6437 (almost there!)
A dedicated Z-Wave panel in the frontend:

Call services to add/remove node or heal network (v1). We already have the services, just need buttons.
Show Open Z-Wave log (v1). We already have the log, just need an API call.
Explore current nodes and all their info (v2)
Configure nodes? (v3)

What else?

Nodes neighbour connectivity map

+1 on configuring nodes, especially performing associations between nodes. There are a lot of Z-wave switch controls (like the Aeotec 4 button) coming on the market and having to shut down home assistant to go launch and mess with ozwcp (and it's less than intuitive interface for associations) is a pain.

Hi Paulus, looks nice (goed werk!), like to see also a connectivity map with possibility to adjust neighbours.

@ViperRNMC I don't think neighbours are settable.

No, Neighbours are not settable. They are handled in the controller. Heal node or network is the best that can be done to fix neighbours. If it does not help, look at range, interferrence or faulty devices.

I would love to do the panel, but I have big issues with understanding the polymer coding part.

I'd personally love to not have to shut HA and fire up ozwcp to edit parameters on devices or check their values. Integrating that into HA would make things much more usable "out of the box".
Also firmware updates for ZW+?

@raccettura OZW does not support OTA, so as long it doesn't, we can't either.

@raccettura Also note that there is already a {print,set}_config_parameter services.

Love to see configuration of Node/Device so don't have to shutdown HA.
Also real time activity when I am in that Zwave panel not sure that above mentioned item "Show Open Z-Wave log (v1). We already have the log, just need an API call." will cover this.
Some kind of trigger (state machine?) when any of the node is dead or come alive.
Nice work guys!
Thanks a lot.

@turbokongen I can help with a foundation of a panel. I also know that @andrey-git is getting pretty solid in Polymer so should be able to help.

I would love the possibility to set my names on the z-wave devices. I had to re include all my zwave devices the other day and it was a pain in the butt to search and replace all entity names.
The possibility to override the name from OZW from kitchen_lamp_3_4 to just kitchen lamp would help a lot.

Made a PR to address the last comment.

With the new hass release, you can click in the z-wave panel and say "stop z-wave network" then i made  a custom toggle, that runs below script, which turns on ozwcp panel. Run the toggle again and it turns off and you can turn back hass zwave again. Might not be the end result but since i was configurating a bunch of settings this made my life a little easier. (works on the hassbian image)
#!/bin/bash

BIN_FOLDER=/srv/homeassistant/src/open-zwave-control-panel/
if [ -z $2 ]; then
        PORT=8888
else
        PORT=$2
fi

test -f ${BIN_FOLDER}ozwcp || exit 0
case "$1" in
    start)
            echo -n "Starting ozwcp"
            cd ${BIN_FOLDER}
            nohup ./ozwcp -p${PORT} > /dev/null 2>&1 &
            echo $! > ${BIN_FOLDER}ozwcp.pid
            echo ""
            ;;
     stop)
            echo -n "Stopping ozwcp"
            pkill ozwcp
            rm -f ${BIN_FOLDER}ozwcp.pid
            echo ""
            ;;
     toggle)
            if [ -f "${BIN_FOLDER}ozwcp.pid" ];then
                    pkill ozwcp
                    rm -f ${BIN_FOLDER}ozwcp.pid
                    echo "Stopped ozwcp (was running)"
                    echo ""
            else
                    cd ${BIN_FOLDER}
                    nohup ./ozwcp -p${PORT} > /dev/null 2>&1 &
                    echo $! > ${BIN_FOLDER}ozwcp.pid
                    echo "Started ozwcp (was stopped)"
                    echo ""
            fi
            ;;
        *)
            echo "Usage: ./ozwcp_toggle.sh start|stop|toggle <portnumber>"
            exit 1
            ;;
esac

Has there been any work or desire expressed for supporting multiple controllers? I have two zwave.me sticks and would like to have them both available in homeassistant.

@ziadsawalha I think the only big hurdle with multiple controllers would be the zwave services/API. We'd need a new parameter to the services that specify which network you want to control.

@ziadsawalha Any advantage to using 2 sticks at the same time (at the same place)? Why not use just one?

@andrey-git I'm using them to support devices on different frequencies (US and European devices).


WIP: Add an _R prism which is useful with modifyRoute

Write it and rewrite the existing rPrism in terms of some more general optics.
This will hopefully aid future attempts to put the most general mechanisms in
dependent-sum itself.

@Ericson2314 Did you mean to leave all that commented code in there?

Yes the code works and I mean to leave it to help with the follow-up work / explain the comments.


Move setup-python block before poetry install

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

Comparison is base (637f0b5) 61.03% compared to head (6e77098) 60.47%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
- Coverage   61.03%   60.47%   -0.57%
==========================================
  Files           9        9
  Lines        1404     1404
==========================================
- Hits          857      849       -8
- Misses        547      555       +8

Flag
Coverage Δ

integrationtests
?

unittests
60.47% <ø> (-0.57%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add person,student and teacher classes

In this feature we're adding three classes with their own methods:

Person:

is_of_age? (private)
can_use_services? (public)

Student (inherits from Person):

play_hooky

Teacher (also inherits from Person):

can_use_services? (overrides)

Thank you very much! Merging right away


Flatbuffer_conversion changes for Transpose

Refactor transpose in flatbuffer_conversions.cc to prepare for porting kernel to micro.

@patriklaurell Can you please resolve conflicts? Thanks!

@patriklaurell Can you please resolve conflicts? Thanks!

Ready for review @petewarden


Use source in the example for sourcing Zsh shell integration

A few people were copying that snippet and were facing issues such as "permission denied", and (after a subsequent chmod) "syntax error".

From the CONTRIBUTING.md:

Pull requests should be associated with a previously accepted issue.

I decided to ignore this for a single-word change, since creating a discussion then waiting for it to be promoted to an issue seemed pointless for such a minor change.

This is actually a dupe of #3511 (I forgot to check closed PRs), I'll leave it up to someone else to decide which one to merge.
And no, this is not a dupe of #3332, that fixes a different file.


Flaky Test: v3discovery.TestCheckCluster

pull-etcd-unit-test Test: go.etcd.io/etcd/server/v3/etcdserver/api/v3discovery.TestCheckCluster/2_retries_for_getClusterSize
Test failed 1.5% (2/134) of the time
failure logs are:

commit
started
log

d28b489ab
2024-04-18 15:04:03 +0000 UTC
https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/pull-etcd-unit-test/1780975546570641408

ebaf40ca8
2024-04-18 14:44:51 +0000 UTC
https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/pull-etcd-unit-test/1780970720042422272

Please follow the instructions in the contributing guide to reproduce the issue.

pull-etcd-unit-test Test: go.etcd.io/etcd/server/v3/etcdserver/api/v3discovery.TestCheckCluster/no_retries
Test failed 1.5% (2/134) of the time
failure logs are:

commit
started
log

d28b489ab
2024-04-18 15:04:03 +0000 UTC
https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/pull-etcd-unit-test/1780975546570641408

ebaf40ca8
2024-04-18 14:44:51 +0000 UTC
https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/pull-etcd-unit-test/1780970720042422272

Please follow the instructions in the contributing guide to reproduce the issue.

auto close due to no updates for 1 days


Migration change, ecto.rollback and flush problem

This situation happen to me when working with reversible migrations, so instead of down, execute/1 and up I was simply using change and execute/2. I noticed that one part requires first to be finished, so I was in need to add flush() call. With ecto.migrate there was no problem. In ecto.rollback commands were althrough in reverted order, but not as I imagined …
When I had first part of migration, flush() call and second part of migration then after using ecto.revert I got reversed first part of migration, flush() call and reversed second part of migration. Should it not be like: reversed second part of migration, flush() call and reversed first part of migration instead? That's how I understood it after reading documentation.
In order words flush() in ecto.rollback task works like flush() in ecto.migrate task, but each part (separated by flush call) have reversed commands and the parts are executed with not reversed order (like in ecto.migrate).
This is really easily reproducable by such simple migration:
defmodule Example.Repo.Migrations.Test do
  use Ecto.Migration

  @migrate_first "select 'first part of ecto.migrate';"
  @migrate_second "select 'second part of ecto.migrate';"
  @rollback_first "select 'first part of rollback.migrate';"
  @rollback_second "select 'second part of rollback.migrate';"

  def change do
    execute(@migrate_first, @rollback_second)
    flush()
    execute(@migrate_second, @rollback_first)
  end
end

Output with flush() call commented:
$ mix ecto.migrate
05:14:25.060 [info]  == Running<PHONE_NUMBER>3147 Example.Repo.Migrations.Test.change/0 forward

05:14:25.063 [info]  execute "select 'first part of ecto.migrate';"

05:14:25.064 [info]  execute "select 'second part of ecto.migrate';"

05:14:25.067 [info]  == Migrated<PHONE_NUMBER>3147 in 0.0s

$ mix ecto.rollback
05:14:25.166 [info]  == Running<PHONE_NUMBER>3147 Example.Repo.Migrations.Test.change/0 backward

05:14:25.166 [info]  execute "select 'first part of rollback.migrate';"

05:14:25.167 [info]  execute "select 'second part of rollback.migrate';"

05:14:25.168 [info]  == Migrated<PHONE_NUMBER>3147 in 0.0s

Output with flush() uncommented:
$ mix ecto.migrate
05:13:22.580 [info]  == Running<PHONE_NUMBER>3147 Example.Repo.Migrations.Test.change/0 forward

05:13:22.583 [info]  execute "select 'first part of ecto.migrate';"

05:13:22.584 [info]  execute "select 'second part of ecto.migrate';"

05:13:22.587 [info]  == Migrated<PHONE_NUMBER>3147 in 0.0s

$ mix ecto.rollback
05:13:22.682 [info]  == Running<PHONE_NUMBER>3147 Example.Repo.Migrations.Test.change/0 backward

05:13:22.682 [info]  execute "select 'second part of rollback.migrate';"

05:13:22.683 [info]  execute "select 'first part of rollback.migrate';"

05:13:22.684 [info]  == Migrated<PHONE_NUMBER>3147 in 0.0s

I may be wrong, but for me it looks like a bug. What do you think about it?

Definitely a bug, thanks for the report. However, I am not sure if we cacn fix it. flush is usually used when you have non-migration commands in your migration and those likely won't work on change+rollback. So perhaps the best option is to make flush() a non reversible command.

@josevalim The point is that my non-migration commands actually requires flush() (or equivalent solution) for having fully reversible migrations. :-)
I'm making some changes in database and I need to fetch something using Ecto.Migrator.repo() in the middle of migration.
My library is generating migrations and if ecto.rollback would work like ecto.migrate then it would be really amazing. This is only one thing which stops me for now from finishing a release candidate of my new library. I have almost complete documentation and now I'm just stuck on last tests.
If there would be a patch then I would be the first who would test it in real use case. As it blocks me from finishing my current library I'm more than ok to help with that.
btw. I made a small research and gave summary in other issue in this repo, but nobody replied to it. Hope I did not made any mistake. :-)

I'm making some changes in database and I need to fetch something using Ecto.Migrator.repo() in the middle of migration.

That's my point. Whatever you do with Ecto.Migrator.repo won't be included in the reverse/flush instructions correctly. For us to correctly reverse custom code, we would need to reverse the Elixir code in the migration, and that's a non-trivial problem. :)

Is there really no way to fix that other than making it non reversible command? If no then I can make a test PR and maybe I could somehow help with implementation fix.

Sure, you could try to implement flush as you describe. However I think once you use it in practice, it won't work as you expect. :) But practice beats theory, so do give it  try!

@josevalim

That's my point. Whatever you do with Ecto.Migrator.repo won't be included in the reverse/flush instructions correctly. For us to correctly reverse custom code, we would need to reverse the Elixir code in the migration, and that's a non-trivial problem. :)

Ah, sorry - now I see. Yeah, I missed that repo calls are not a part of command list …
I'm did not take a look at that part of ecto_sql source code, but from what I understand it's like Ecto.Migrator stores a list of commands which are called after change/0 function finishes. So in short: "Migrator has many commands at run-time".
What came to my mind is (let's call it) step/1 macro which would be exactly between Migrator and commands, so "Migrator would have many steps at compile-time and for each step Migrator have many commands at run-time". Instead of list of commands Migrator could optionally store a list of quoted expressions. ecto.rollback would firstly reverse steps list and for each step Migrator would collect commands. Then rest is easy as such commands could be reversed like now they are in normal migrations. Basically it would work something like sub-migrations. Since all of them would be quoted we would be able call flush() between Code.eval_quoted/3 calls and therefore simply use Ecto.Migration.repo() separately in each step.
defmodule Example.Repo.Migrations.Test do
  use Ecto.Migration

  def change do
    step do
      # first part
    end

    step do
      # Migration.Ecto.repo()
      # second part
    end
  end
end

The point here is that we would be able to do something like:
defmodule Example.Repo.Migrations.Test do
  use Ecto.Migration

  def change do
    MyLib.Migration.first_part(args)
    MyLib.Migration.second_part(args)
  end
end

each of such functions would call step/1 macro.
For sure I could generate multiple migrations for one thing, but going forward we don't need flush() at all and I think that's not the point.
It may require lots of work, but I believe that it would definitely increase experience of working with migrations. Please let me know what do you think about it.

Actually, I think the solution is simpler. You most likely don't want to run the repo operation on rollback, so you just need to do something like this:
if direction == :up do
  ...run the command...
end

This takes care of the stray Repo operation, now we just need to change flush to flush properly.

Alternatively, you can move the flush to inside the conditional above:
if direction == :up do
  flush()
  ...run the command...
end

Then everything should work today.

@josevalim I'm already using it for another part (inserting rows), so I already thought about it. Unfortunately it would not work in my use case.
I thought about my idea and for now this scenario looks best for me:

Making a flush() non reversible as you said (a quick fix for this issue)
Add steps support for Migrator (new feature for having reversible migrations with Ecto.Migration.repo() calls)
Deprecate (of course not "right now") non reversible flush() in favor of step/1 (migration would be flushed automatically between steps, so there is no need for flush() API any more).

I'm sure that you have more important things now, but I would like to ask if such scenario looks good for you (i.e. if someone would implement it then would it have a chance to merge) or maybe I missed something. If in theory it's ok for you then I could start work on implementation for 2nd point.

Unfortunately it would not work in my use case.

Can you please expand on why?

Can you please expand on why?

@josevalim Well … I still would like to make a little surprise, so I would describe it in general.
I need to fetch from database few things and depending on them I would generate an SQL code (use in Ecto.Migration.execute/2) for providing an automated migration and that fetch needs to be inside migration.
Imagine that you have 2 functions: func_a and func_b. func_a is working like a rollback of func_b and func_b is working like a rollback of func_a. Both functions run Ecto.Migration.execute/2. Only func_b requires flush() (or equivalent), so the problem is when I have func_a in change which in rollback it works like func_b which requires flush(), because when in change there is func_b then on rollback I have behaviour of func_a which does not requires flush().
I can make them non reversible, but it requires more work from developer and more code for extra up and down migration (my_lib.gen.migration task). Of course I can make also separate migration files, so both func_a would be always in separated change, but that's not the point.
Hope I described it enough.

I see. Going the step routes is ok but I think the proposal can be simpler. You can introduce:
step do
  Repo.all(...)
end

That is a regular migration commant except stores its contents to be executed later (think of it as execute but for code) . But step wouldn't accept migration commands inside. So you can't call execute inside but you can just call Repo.query!, which has the same effect anyway.

@josevalim So in this form it's ok to make PR or there is something more to discuss?

Please do a PR and let's discuss based on the code.

Closing in favor of: #158 and #159


Compatibility issue with mono version of MSBuild on Mac

On Mac OS X, the mono version of MSBuild sets the OS property to Unix, but the CoreCLR version of MSBuild sets the OS property to OSX. This can cause build errors for some projects. For example, this is blocking build of mscorlib.dll on Mac.

@JohnChen0 It looks like our Mono build and our .NET Core build both claim OSX. Mono's xbuild, on the other hand, does report Unix on OS X. It seems like the MSBuild behavior is better, since I don't know how you distinguish between OS X and Linux otherwise.
I would prefer to keep the current MSBuild behavior. Do you feel otherwise?

CoreCLR build on Mac is currently using Microsoft.Build.Mono.Debug <IP_ADDRESS>-prerelease from myget.org, and the MSBuild.exe included in this package sets OS to "Unix" on Mac OS X. Is this MSBuild.exe really xbuild instead of MSBuild?
I'm OK with keeping the current MSBuild behavior, and work it around in CoreCLR build. Just want to make sure we're not introducing a compatibility problem that many people will need to work around.

AFAIK the current Mono behavior is that way for historical reasons. From http://www.mono-project.com/docs/faq/technical/#how-to-detect-the-execution-platform :

Notice that as of Mono 2.2 the version returned on MacOS X is still 4 (== Unix) for legacy reasons, too much code was written between the time that the MacOSX value was introduced and the time that we wrote this text which has lead to a lot of user code in the wild to not cope with the newly introduced value.

xbuild just uses Environment.OSVersion.Platform that's why it ends up being Unix on OSX and Linux.
A lot of code out there seems to rely on this, but I guess breaking this behavior is better in the long run for MSBuild.

I talked to @ValMenn and @cdmihai about this today, and it sounds like the packaged Mono version of MSBuild had a bug (since fixed) that caused us to always report Unix. I didn't see it because the code that returns the string is unchanged, but methods it calls now work right.
I don't think we have a significant compatibility concern here--if someone is moving from xbuild to MSBuild, I think it's reasonable to ask them to update conditions to the more expressive variety. Good to check though--if you're reading this and disagree, please comment with your reasoning.

Given how common the use of $(Platform) is in existing cross-platform targets and project files, IMO this effectively makes MSBuild incompatible with xbuild on Mac.
Maybe it's worth the pain, but I'd be very wary of this.

For CoreCLR build (dotnet/CoreCLR#1934) we will work around this. Thus I'm closing this issue.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

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


eco_navi NoneType

I am working on a fix.
Logger: homeassistant.components.switch
Source: helpers/entity_platform.py:364
integration: Switch (documentation, issues)
First occurred: 06:54:59 (4 occurrences)
Last logged: 07:34:13

Error while setting up panasonic_cc platform for switch
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 364, in _async_setup_platform
    await asyncio.shield(awaitable)
  File "/config/custom_components/panasonic_cc/switch.py", line 30, in async_setup_entry
    if device.support_eco_navi:
       ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/functools.py", line 995, in __get__
    val = self.func(instance)
          ^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/panasonic_cc/panasonic.py", line 274, in support_eco_navi
    return self._details.features.eco_navi
           ^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'features'

I have some bug fixes I haven't checked in yet


How to change the font on the hover panel?

Default is Courier New I guess, which looks blurry on Windows most of the time.

I didn't find any settings about fonts -- maybe I'm missing.
Any way to cure my eyes? XD

AFAIK, there is no way to control the font of the hovers, or many other UI elements in vscode. I would recommend opening an issue with the vscode repo and asking for this ability.


Market time Timezone

I am trying to get U.S. market times to show on a chart. I have tried both of these solutions:
date-solution
date-fns-tz-solution
My chart keeps rendering with a +7 hour time difference. Has anyone experienced this issue?

Timezones are discussed in more detail here: https://tradingview.github.io/lightweight-charts/docs/time-zones
The easiest way to change what is displayed is to adjust the data before it is given to the chart. You could manually add or subtract the 7 hours from the timestamp for each data point.
There are also formatters which you can use to customise:

The time displayed on the time axis: https://tradingview.github.io/lightweight-charts/docs/api/interfaces/TimeScaleOptions#tickmarkformatter
The time displayed on crosshair label: https://tradingview.github.io/lightweight-charts/docs/api/interfaces/LocalizationOptions#timeformatter


NPM run prod error

Laravel Mix Version: 1.6.1
Node Version (node -v): v8.9.0
NPM Version (npm -v): 5.5.1
OS: Mac os x high sierra

Description:
When running npm run dev everything works, but when running npm run prod I get the following error:
npm run prod

> @ prod /websites/laravel
> npm run production

> @ production /websites/laravel
> cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js

/websites/laravel/node_modules/schema-utils/dist/validateOptions.js:40
    throw new _ValidationError2.default(ajv.errors, name);
    ^

false
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ production: `cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ production script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the @ prod script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /.npm/_logs/2017-11-07T10_09_46_249Z-debug.log

This is my package.json
{
  "private": true,
  "scripts": {
    "dev": "npm run development",
    "development": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "watch": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --watch --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "watch-poll": "npm run watch --watch-poll",
    "hot": "cross-env NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
    "prod": "npm run production",
    "production": "cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js"
  },
  "devDependencies": {
    "cross-env": "^5.1",
    "laravel-echo": "^1.3.2",
    "laravel-mix": "^1.6.1",
    "less": "^2.7.3",
    "less-loader": "^4.0.5",
    "pusher-js": "^4.2.1"
  },
  "browserslist": [
    "> 1%",
    "last 2 versions"
  ]
}

Here the NPM log
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/Cellar/node/8.9.0/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'prod' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'preprod', 'prod', 'postprod' ]
5 info lifecycle @~preprod: @
6 info lifecycle @~prod: @
7 verbose lifecycle @~prod: unsafe-perm in lifecycle true
8 verbose lifecycle @~prod: PATH: /usr/local/lib/node_modules/npm/bin/node-gyp-bin:/websites/laravel/node_modules/.bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Applications/VMware Fusion.app/Contents/Public:/usr/local/MacGPG2/bin:/usr/local/sbin:~/.composer/vendor/bin
9 verbose lifecycle @~prod: CWD: /websites/laravel/
10 silly lifecycle @~prod: Args: [ '-c', 'npm run production' ]
11 silly lifecycle @~prod: Returned: code: 1  signal: null
12 info lifecycle @~prod: Failed to exec prod script
13 verbose stack Error: @ prod: `npm run production`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:280:16)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at EventEmitter.emit (events.js:214:7)
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at ChildProcess.emit (events.js:214:7)
13 verbose stack     at maybeClose (internal/child_process.js:925:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
14 verbose pkgid @
15 verbose cwd /websites/laravel
16 verbose Darwin 17.2.0
17 verbose argv "/usr/local/Cellar/node/8.9.0/bin/node" "/usr/local/bin/npm" "run" "prod"
18 verbose node v8.9.0
19 verbose npm  v5.5.1
20 error code ELIFECYCLE
21 error errno 1
22 error @ prod: `npm run production`
22 error Exit status 1
23 error Failed at the @ prod script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

mmm found the issue I think...
I have this is my webpack.mix.js file
let mix = require('laravel-mix');

/*
 |--------------------------------------------------------------------------
 | Mix Asset Management
 |--------------------------------------------------------------------------
 |
 | Mix provides a clean, fluent API for defining some Webpack build steps
 | for your Laravel application. By default, we are compiling the Sass
 | file for the application as well as bundling up all the JS files.
 |
 */

mix.options({
    processCssUrls: false,
    purifyCss: false,
    postCss: [
        require('postcss-discard-comments')({
            removeAll: true
        })
    ],
    uglify: {
        comments: false
    }
});

When I comment out the uglify block then it works again...

Had the exact same issue, removing my custom uglify settings did the trick. How would one go about this with 1.6.0 now though?

The same thing is happening to me.

Also having this happen when running npm run production

+1

Update: looking at the release notes Uglify seems to require the previous settings to be wrapped in a "uglifyOptions" object now: https://github.com/webpack-contrib/uglifyjs-webpack-plugin#uglifyoptions (as seen by https://github.com/JeffreyWay/laravel-mix/pull/1301/files).
This still doesn't change the fact that I'm getting an exception originating from vendor.js on page load.

Confirmed @spaceemotion's suggestion does indeed fix this.

uglify options was changed in version 1.6
https://github.com/JeffreyWay/laravel-mix/pull/1301/files#diff-9d9a6cd82f41984872a66a3ab0d440c4R256
New syntax should be
mix.options({
uglify: {
    uglifyOptions:{
        compress:{
        }
    }
}
});

then this page needs changing:
https://github.com/JeffreyWay/laravel-mix/blob/master/docs/options.md

seems like that page is just showing which options are available and the data type of the option, but a pull request may be accepted to make things a little more clear.

I had this issue recently, maybe you are running out of memory. Check it out. In my case, was a problem with swap on Ubuntu, so I just restarted my server to get it working.

@ricardovigatti  You are right.

rm -rf node_modules
rm package-lock.json yarn.lock
npm cache clear --force
npm install

Hello, resolve the error as follows.

Deleted the node_modules folder
Delete the package-lock.json

Edit the package.json file and change the version of "sass-loader" to version "^7.0.0"

Run "npm run prod".

and that's it. ;)


Missing Azure Government Endpoints

The URLs for Azure Government/GCC High should be added:
https://autologon.microsoft.us < Please verify this one as I guessed it based on what I know about the Azure Government DNS endpoints (it did resolve correctly)
https://device.login.microsoftonline.us

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

ID: b9c51404-0a1b-c22e-e220-ce4126ff88d0
Version Independent ID: bd3aab43-4202-bb69-bd1a-580e0464f357
Content: Configure hybrid Azure Active Directory join for federated domains
Content Source: articles/active-directory/devices/hybrid-azuread-join-federated-domains.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@SanDeo-MSFT Can you verify the government endpoints for hybrid Azure AD join in this Issue?

The analogous Government endpoints I've seem to have found are:
https://enterpriseregistration.windows.net -> enterpriseregistration.microsoftonline.us
https://login.microsoftonline.com -> login.microsoftonline.us
https://device.login.microsoftonline.com -> device.login.microsoftonline.us
https://autologon.microsoftazuread-sso.com -> autologon.microsoft.us

For Azure Government these are the three key URLs
https://enterpriseregistration.microsoftonline.us
https://login.microsoftonline.us
https://device.login.microsoftonline.us

@miketheitguy
I have added this to my backlog for updates to the devices docs that are currently in progress. Going to close out this issue for now but will comment back when the changes go live in the future. #please-close


Vim-Plug error with nerdTree autocommand example

Installing nerdTree with vim-plug
s:lod_cmd throw an error while executing
autocmd StdinReadPre * let s:std_in=1
autocmd VimEnter * if argc() == 0 && !exists("s:std_in") | NERDTree | endif

in vimrc when starting up vim
The details of the problem can be found
Here

In this particular case, exec 'NERDTree' will fix the issue, but just don't use on option.
https://github.com/junegunn/vim-plug/wiki/faq#when-should-i-use-on-or-for-option
on sets up a proxy command of the same name, but without -bar option, so in this case, it doesn't recognize the following | endif and fails.
vim-plug cannot confidently apply -bar to the proxy command as it has other side-effects. Fixes something, breaks another.

@junegunn, does your suggestion for @soulomoon fix his problem (i.e., not using the on option)? Since the code in our README is correct, I'd rather not change it to accommodate this particular case. It seems more like an issue of using vim-plug in the recommended way.
@soulomoon, did @junegunn fix your error?

Yes, it should.

@junegunn @lifecrisis
Yes, removing the on  option fix the problem


Invalid \uxxxx in a parsed string throws range_error

If a json string is parsed and contains a string literal with a \u that is not followed by hex digits, a range_error is thrown by from_hex_digit, but the surrounding code doesn't expect that.
The parse_string() function, which calls the string decoding, only handles decode_error, not range_error. Probably this function should handle range_error in the same way, or string_decode() should already convert it into a decode_error when parsing the \u escape.
The exception can for example be triggered by this:
"\"\\uzzzz\""_json

Agreed -- it makes more sense to have from_hex_digit throw a decode_error here.

Fixed.


Connection Limit?

Is there a connection limit to the number of icecast streams that you can connect to? I'm working on building a monitoring service where i connect to the icecast stream and then wait for the end event, which I then do some operations to alert the admins.
I'm able to trigger an end event message for about the first 5 streams on the list.  The rest don't seem to be connecting.
Here is an example of what I'm trying to do:
// var sources is coming from the icecast stats
// go through the sources and set them up to be monitored
for(var $i = 0; $i < 50; $i++){
    var stream = sources.source[$i].listenurl;
    icecast.monitor(stream);
}

// my icecast.monitor function
// receives the streamurl / listenurl and tries to connect
exports.monitor = function(url) {
	console.log('Monitoring: '+url);
	icecast.get(url , function(res){

		// get the metadata
		res.on('metadata', function(metadata){
			var parsed = icecast.parse(metadata);
			console.error(parsed);
		});

		res.on('data', function(data){
			// console.log(data);
		});

		// when something goes wrong
		res.on('end', function(data){
			console.log('oh shit something went wrong with');
		});
	});
};

a dirty way to increase socket connections is to patch the icecast/lib/client.js by adding i.e.:
http.globalAgent.maxSockets = 20;
It would be great to have a possibility to pass a config to set the amount of allowed socket connections or is it already possible in a better way as the above patch?
greetings from berlin

Send an agent: false option in the icecast.get() function in order for a new http.Agent to be used for every connection. Then this problem should not occur.

Thank you for the responses.  I haven't worked on this in a while, but I'am still heavily involved with using IceS and IceCast.  I'll let you know how it goes.


Help us improve our documentation

Are the readme and contributing docs clear? Are there directions you don't understand? Help us better communicate the project!

link to Make your own buttons is broken in the README

Thanks @manasvil! Fixed!

The readme and much of the contributing docs are clear.  I see that you all are still working on getting some of the material up. Looking forward to checking out the slides once they are available!


Yuzu Game Cover Not Appearing In Deck Mode

When using my-game-cover.png, the agape install script successfully finds and generates the cover file but the Steam Deck's deck mode does not show the cover. The PNG file I'm using is 603x993 and 1,037,664 bytes. Does it need to be a smaller file?

No, the size shouldn't cause any issue. You mean in the file manager? After following the instructions here, you should view the file with the thunar file manager to generate the icon, after that it should appear everywhere.

Oh I see, I must have overlooked that. I will close this issue. Thank you!


Bkash payment gateway module

Dear vai.
Bkash payment gateway whmcs module not working. Please solve it.

Please describer what error you are getting?
Aslo, please WHMCS Version, and PHP version write here.

Whmcs Version 8.5
Show error: Invalid response from the bKash Server.
On Wed, 14 Dec 2022, 11:50 am Aminul Islam, @.***>
wrote:

Please describer what error you are getting?
Aslo, please WHMCS Version, and PHP version write here.
—
Reply to this email directly, view it on GitHub
https://github.com/NawabHost/bkash-checkout-for-whmcs/issues/16#issuecomment-1350433096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4GO2XC74M42T6DVOYDTUV3WNFNYZANCNFSM6AAAAAAS5F3R74
.
You are receiving this because you authored the thread.Message ID:
@.***>

Please see this: https://github.com/NawabHost/bkash-checkout-for-whmcs/issues/11#issuecomment-1132766567


ctrl a a strange choice for bind - breaks ability to move cursor to beginning of the line

Is your feature request related to a problem? Please describe.
I want to retain the ability to use the normal ctrl-a and ctrl-e features of zsh and bash etc. which move the cursor the beginning and end of the line respectively. By binding to ctrl-a by default, I can no longer move the cursor the the beginning of the line when not using inshellisense.
Describe the solution you'd like
Bind to a key which is not used by zsh and bash etc.  Or make it configurable.
Describe alternatives you've considered
The file ~/.inshellisense/key-bindings.zsh contains
bindkey '^A' __inshellisense

so I edited this file and changed the A to O which fixed the problem, but then I lose whatever ^O used to do. Perhaps a CTRL plus SHIFT key combination would be better - how do I specify SHIFT in bindkey?
Additional context
The ability to move to the beginning of the line and end of the line etc. are all broken inside inshellisense anyway, but I guess that's another issue....

There is a todo to support emacs/vi navigation flows within inshellisense, so that should address the additional context. I'll look at making the binding customizable or at least providing a few options during the bind operation

Implementation aside, I recommend that you also pick a good default that is not surprising or annoying for people who know and use the readline/emacs bindings. Those are common in many places, not only bash. Despite being a heavy nvim/vim user, I use many of those myself. Other projects have taken choices which clash annoyingly with them (e.g. tmux uses Ctrl-b for its prefix, and docker uses Ctrl-p, both fairly common).
Some that (IMHO) might be less likely to be used are:

Ctrl-j. This is basically the same as Return. I use this as prefix in tmux, and I've never found it necessary.
Ctrl-t. This transposes two characters. It's a bit niche.

I don't recommend giving Ctrl-o away, @abulka. This is extremely useful to repeat a sequence of commands.

@suy I've now changed my inshellisense shortcut to Ctrl-j and since learned the amazing usefulness of Ctrl-o as described e.g. here. To be honest, I think Ctrl-o is going to be of more value to me than inshellisense!

Closing as the ctrl-a binding was removed in 0.0.1-rc.5


Add and review unit tests

Add a set of unit tests and review the existing ones.

Added unit test to replay a transaction and extract libfuc statistics using RPC protocol. #36


OSErrors when using send media key

Describe the bug
A clear and concise description of what the bug is.
I noticed when trying to send media keys I'm getting OSErrors in the log. If I reload the integration sending keys works for awhile and eventually it stops and I get OSError.
It looks like Smartthings sees the TV as offline when it's not so the keys don't send which may be the cause of the OSError? Are the keys sent locally or to smart things API?
Expected behavior
If applicable, a clear and concise description of what you expected to happen.
Send media keys to always send when the TV is on.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment details:

Environment (HASSIO, Raspbian, etc): Home assistant Docker container
Home Assistant version installed: 2024.4.0
Component version installed: 0.13.1
Last know working version:
TV model: Q90CA

Output of HA logs
Paste the relavant output of the HA log here.
2024-02-25 16:46:00.455 DEBUG (SyncWorker_14) [custom_components.samsungtv_smart.api.samsungws] Sending key KEY_ENTER
2024-02-25 16:46:00.455 DEBUG (SyncWorker_14) [custom_components.samsungtv_smart.media_player] Error in send_command() -> OSError
2024-02-25 16:46:00.602 DEBUG (MainThread) [custom_components.samsungtv_smart.api.smartthings] {'deviceId': 'mytvid', 'state': 'OFFLINE', 'lastUpdatedDate': '2024-01-16T00:29:41.710Z'}
2024-02-25 16:46:05.267 DEBUG (MainThread) [custom_components.samsungtv_smart.api.smartthings] {'deviceId': 'mytvid', 'state': 'OFFLINE', 'lastUpdatedDate': '2024-02-15T06:39:50.790Z'}
2024-02-25 16:46:05.468 DEBUG (MainThread) [custom_components.samsungtv_smart.api.upnp]
2024-02-25 16:46:05.669 DEBUG (MainThread) [custom_components.samsungtv_smart.api.upnp]

Additional context
Add any other context about the problem here.

Ok, I reopen.
Sorry but I'm very busy with my job on last months so my activity on this repository is very limited.
Please next time remember just to give an "up" on the issue when you receive stale notification to avoid auto-closure.

I'm also having this issue, but running HAOS directly on a raspberry pi.

Environment (HASSIO, Raspbian, etc): HAOS on Raspberry Pi 4b
Home Assistant version installed: 2024.10.3
Component version installed: 0.13.5
Last know working version: unknown, only just trying this now
TV model: QE55S90C

Example debug log output
2024-10-20 23:32:02.944 DEBUG (SyncWorker_15) [custom_components.samsungtv_smart.api.samsungws] Sending key KEY_CHUP
2024-10-20 23:32:02.946 DEBUG (SyncWorker_15) [custom_components.samsungtv_smart.media_player] Error in send_command() -> OSError
2024-10-20 23:32:02.994 DEBUG (MainThread) [custom_components.samsungtv_smart.api.smartthings] {'deviceId': '46926259-97a8-3916-d744-067be2082a93', 'state': 'ONLINE', 'lastUpdatedDate': '2024-10-20T22:16:44.752Z'}

Not stale.


Update 2-enforcement.md

#3.1 Protecting the integrity of the Sansar Marketplace
It is harmful to the marketplace when a user of sansar uploads an Avatar, accesory, or 3d model that is not there own. it is unfair to an artist to compete with other people calling themselves artist that dont make their own art. No matter if the excuse is that the item is for free or is for people that are new users that dont know how to build. You as a user of sansar after November 12th 2022 cannot upload any model to the sansar marketplace that is not there own creation. Your own creation is not an edit to someone else creation and is forbidden. Made from scratch in standard 3d software means your creation by you or partnership between two or more artists in collaboration are allowed only.
if another concerned sansar user can bring solid evidence of this violation of sansar law. Evidence must be looked into by sansar moderation with full severity of violation. We as 3d artist that make models off of pure imagination and in spirit of generating original content should not have to deal with people that dont want to be original and do the same. 3d modeling can be learned and it just takes time. It is a requirement to upload to the store to take the time to learn. Violating this law harms sansar because its a loss to sansar because talented artist may not be part of sansar due to competing stolen mesh model which doesnt allow sansar to be at its biggest potential in the 3d art gaming industry.
you can bring in whatever 3d mesh for your personal experience for use but cannot sell in the marketplace
1st volation is a verbal warning and enforcement is removal and offending user understanding why it was removed. (documented public enforcement)
2nd violation is suspension of marketplace for one month. no upload no cashout of monies (documented public enforcement)
3rd violation complete bann of store which mean a complete bann of your avatar and account (documented public enforcement)

What about the usage of CC0 licensed (a license given by the creator for anyone to use it as they wish even commercially, some info https://creativecommons.org/share-your-work/public-domain/cc0/) meshes, textures and materials, it's extremely common for people to use multiple CC0 licensed materials for example on their original model work, unless you are going out and taking HD photographs with an expensive camera yourself to create your texture maps, you more often than not relying on the work of others who give express permission to do so.
In terms of models, CC0 models often lack uv unwraps or texture work as well as rigging, could it be considered a collaborative effort, for example, the raptor avatars me and nick sell on our store are a CC0 model whose owner even gave express permission to sell on places like sl marketplace, we mostly did this as a learning process, we both spent many many hours of work onto these raptor avatars with unique textures and rigging, although we do not do this sort of thing anymore, at the time it was a good learning process and people enjoyed these avatars very much, we both felt we added significant amount creativity and unique value to an otherwise blank unusable model so we charged for them based on the value we added.
How does this situation and licensing issue fit into all this?
Otherwise I mostly agree with you and our store is pretty dedicated to making things from scratch as of present.

no

Adding in store content guidelines is an interesting idea.  I'm not sure it really fits though.
Currently the magna-carta is solely about our public social behavior.
I suppose a question would be has anyone ever had a moderation action against them for content uploaded to the store?
Obviously Sansar has removed content from the store, but has Sansar ever banned someone for uploaded content?

Adding in store content guidelines is an interesting idea. I'm not sure it really fits though. Currently the magna-carta is solely about our public social behavior.
I suppose a question would be has anyone ever had a moderation action against them for content uploaded to the store? Obviously Sansar has removed content from the store, but has Sansar ever banned someone for uploaded content?

I think that should be the focus of this section. The magna-carta should not change the terms of service, but can describe enforcement action related to breaking TOS.

This is the part of the TOS that we CAN change -- If we are restricted to Social --- we can set our own standards. This is one of the aspects of social moderation, it deals with equitable competition practices. One may argue that unequitable competition on the marketplace dilutes the social value of "fair play" users and it can shun them from the platform by allowing the behavior.

Ok I think I can manage most of that at first reading what your saying. I been at work just got out

After rewriting the other pull requests I realized I don't think you wanted me to put this in 2.5 but I already had done it for this one.  I might need clarification where these need to be going specifically

Would you mind turning this into an issue? That way someone else can write it for you and get your feedback. That may be the quickest way to get this implemented.


Update redis4cats-effects to 0.12.0

Updates dev.profunktor:redis4cats-effects from 0.11.1 to 0.12.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.profunktor", artifactId = "redis4cats-effects" } ]

labels: library-update, semver-minor

Superseded by #61.


docs: remove entire spec and leave only readme that points to Cometbft spec

Description
According to the proposal of @rootulp, the entire spec are removed now and only a readme is left which points to the Cometbft spec.
PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use
unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments

Hi @rootulp, please review this PR based on our discussion here: https://github.com/celestiaorg/celestia-core/pull/1496
Thanks

This might add unnecessary extra steps when pulling changes from upstream.

While pulling upstream changes for https://github.com/celestiaorg/celestia-core/pull/1495#issue-2539434463 I learned that it's not that difficult to resolve merge conflicts in docs but IMO it will be easier if those files aren't present in this repo.


JWT Configuration with Jaas

I like to try the payed service at https://jaas.8x8.vc but I don't understand how I need to setup the authentication. I found the App ID but I can't find the JWT token secret. Is there some trick or is this service not supported?

@jespino Can you help with this?

We have a PR open to integrate with JaaS. I will keep you posted when we get the new release out with those changes.


Hass.io Full Snapshot Error - Reduced size to kb

Home Assistant release (hass --**0.65.4**):
Python release (`python3 --3.6):
Snapshot
Full Snapshot is only 740kb
Take into consideration for review
Aditional info:
After the update to 0.65.4 I created a Full Snapshot of Hass.io. When trying to save them locally, I noticed that the snapshot only has 740kb. Created another snapshot, this time a partial one, the result is the same size. Same problem occurs even if the password protection check is applied.

/move to home-assistant/hassio


VNET integration

Enable complete Azure SDN support for containers:
[x] Add "azure" to network policy options: Feature is currently disabled by default. Can be enabled by specifying networkPolicy: "azure" in orchestrator properties.
[x] Allocate IP addresses for containers: Pre-provision IP addresses for containers from Azure VNET CA space. These addresses are allocated to containers on demand.
[] Install Azure VNET and IPAM CNI plugins.

@ofiliz,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

CC @sharmasushant

@colemickens @anhowe @amanohar Thanks for the great feedback.
I grouped changes related to the extra validation and pre-existing issues together and made a separate commit. I'd appreciate it if you do NOT squash these commits, to clarify the reasoning for future reviewers.
Hopefully all issues are addressed now and we can merge this soon :)

(I don't understand why you made the changes in this commit: a3e61bb007881e7491b1daaf816f71d6fdd018e7 I preferred it the other way. Those orchestrator specific options are eventually going to have lots of things inside. A quick check against nil seems clearer than checking for any of the fields being set.)

@colemickens I agree, I also liked the nil check. However, that solution would cause a regression where acs-engine crashes when a KubernetesConfig is not specified. Basically I had to insert tens of "if Orchestrator.KubernetesConfig != nil" checks everywhere, even in some part files. Too much code churn. I agree the nil check is better, but we should do it in a separate PR.

Got it, I concur.

@acs-bot test this please

@acs-bot test this please

@acs-bot test this please

Hitting merge... it looks green to me.


fix: Incremental where clause generation from triggering TypeError

This is an attempt to implement the third option listed in possible fixes for the bug described in #1676.
Closes #1676

:books: Documentation preview :books:: https://meltano-sdk--1688.org.readthedocs.build/en/1688/

@edgarrmondragon @kgpayne I am in need of your sage wisdom and advice.  Codecov is showing query = query.where(replication_key_col >= start_val) is not covered by testing.  I was not able to find a previous test that I could change to cover.  I might have missed it though.  I have two questions:

Was there a previous covering test?  If there was a previous test, please point me in its direction.
If this is a new test where would you like it to live. My best guess would be in the test_tap_sqlite.py tests that I think are pulled from tap_test.py.

Thanks 🙏 in advance.

@BuzzCutNorman this is a new test afaict. The code that currently seeds the data for the test_tap_sqlite.py (which makes the most for the new test to live in) is https://github.com/meltano/sdk/blob/46f4d6a467933fbde000b3ddf886309d96d6e521/tests/samples/conftest.py#L21-L33
so you'll probably want to add a column that acts as replication key and a test that uses a custom catalog.


SendBlock() loop index underflow

openchain/peer/handler.go function sendBlock() has a loop to process blocks in reverse order high to low.
This loop uses a uint var as the index so decrementing below 0 makes the var jump to uint.MaxValue which causes the loop to continue forever which causes an out of memory error.
Changed the loop condition to check that the index stays between upper and lower bounds.

Closing this PR since I need to re-fork my repo after obc-peer became public. Will re-submit in a bit.


Box shadow is appling after border box

Please make sure you are testing with the latest release of html2canvas.
Old versions are not supported and issues reported for them will be closed.
Please follow the general troubleshooting steps first:

[X] You are using the latest version
[ ] You are testing using the non-minified version of html2canvas and checked any potential issues reported in the console

Bug reports:
Please replace this line with a brief summary of your issue AND if possible an example on jsfiddle.
Specifications:

html2canvas version tested with: 1.0.0-rc.7
Browser & version: Google chorme version 87.0.4280.141
Operating system: Windows 10

I would like to export a card like this:

The css to generate this card is the following:
.card {   height: 100%;   border-radius: 25px;   box-shadow: 0 0 12px 0 rgba(0, 0, 0, 0.09);   background-color: #FFF;   border: 1px solid rgba(0, 0, 0, 0.05) }
The result after render with html2canvas is the following:

It looks like the border-radius of 25 px is applied before the box-shadow anyone found a solution to this issue?
Thanks in advance

I have same issue

did you find a workaround for this?

Duplicate of https://github.com/niklasvh/html2canvas/issues/1856


fix list of filters for New_Contributors

The filters should be displayed as a list. This PR adds the necessary formatting.

relates to #329

done


Create akhaura.txt

Please validate the request.
Thank you.

@shsbd Could you please request a free classroom license by email to<EMAIL_ADDRESS>We cannot add the foundation to this repository because it's neither a university, nor a college. Also, we cannot add an educational institution which provides a significant number of the students with 100% online education. This is why I close this request without merging. Thank you for your understanding.


-lpthread in link_flags reverses link order

Upgrading from 76ce37e977a304acf8948eadabb82c516320e286 to ef31d2cedcbb1d41dc86bb94981bca823b3408d1 we noticed failures in our tests:
Error fetching symbol 'pthread_getspecific' from libdl: libpthread_shim.so: undefined symbol: pthread_getspecific

This line moves -lpthread to the front of link flags, which causes certain symbols (e.g. pthread_getspecific) be removed by the linker and become unreadable from dlsym.
            # For single-platform builds, we can statically link the bundled
            # libraries.
            link_flags.extend([
                "-L{}lib".format(toolchain_path_prefix),
                "-l:libc++.a",
                "-l:libc++abi.a",
                "-l:libunwind.a",
                # Compiler runtime features.
                "-rtlib=compiler-rt",
                # To support libunwind.
                "-lpthread",
                "-ldl",
            ])

Is there a specific reason why pthread needs to be part of link flags and not link_libs? If so, is there a way to expose a configuration knob which would allow us to configure the opposite?

Thanks for the issue. I think it's just legacy. Can you please send a PR to do the right thing? 😄

https://github.com/grailbio/bazel-toolchain/pull/103

Closed by #103.

I reverted the changes proposed in this issue, because going forward, I think this issue is better solved through the user specifying the new link_libs attribute in the repository rule. Please keep in mind when you upgrade bazel-toolchain in your workspace. Please see 7580bbe44efaf6e41b1b77efc7f890ad6b46494b.


Bug in plyara.utils.detect_dependencies() with rule reference in last condition

Looks like a classical off-by-one: If a reference to another rule is in the last condition, plyara.utils.detect_dependencies() is empty instead giving the referenced rule. Just switching the last 2 lines of the conditions makes it work.
import plyara
import plyara.utils

def rule_test(rule):
    parser = plyara.Plyara()

    mylist = parser.parse_string(rule)

    for r in mylist:
        print(r['rule_name'])
        print(plyara.utils.detect_dependencies(r))

rule='''
private rule WINDOWS_UPDATE_BDC
{
condition:
    (uint32be(0) == 0x44434d01 and // magic: DCM PA30
     uint32be(4) == 0x50413330)
    or
    (uint32be(0) == 0x44434401 and
     uint32be(12)== 0x50413330)    // magic: DCD PA30
}

rule SndVol_ANOMALY {
	strings:
		$s1 = "Volume Control Applet" fullword wide
	condition:
		filename == "sndvol.exe"
      and uint16(0) == 0x5a4d
'''

rule_cond1 = '''
      and not 1 of ($s*)
      and not WINDOWS_UPDATE_BDC
}
'''

rule_cond2 = '''
      and not WINDOWS_UPDATE_BDC
      and not 1 of ($s*)
}
'''

rule_test( rule + rule_cond1 )
print("-----")
rule_test( rule + rule_cond2 )

Output:
$ python3 bug.py
WINDOWS_UPDATE_BDC
[]
SndVol_ANOMALY
[]
-----
WINDOWS_UPDATE_BDC
[]
SndVol_ANOMALY
['WINDOWS_UPDATE_BDC']

the missing 'not' is the problem in plyara/utils.py
            elif previous_term in ('and', 'or', ) or next_term in ('and', 'or', ):

that's a nasty one, maybe include a test case

Thanks for the report. Could you add that test case to the PR? Also, can you enable editing on the PR as well?

editing is already enabled:

sorry, don't have much time at the moment


UnknownAttribute Error for ActiveRecord <IP_ADDRESS>

Hi 👋
I have a small, single file, Sinatra app with 1 model NfcTag, with 2 attributes guid and number.
After following the install guide, creating a migration and migrating my sqlite3 database, whenever I save/create my NfcTag object, ActiveRecord raises a UnknownAttribute Error.
The weird thing is; when I use a debugger, I can see that the NfcTag class has both guid and number defined. Yet when saving, it suddenly forgets that they are there.
For clarity, here is a gist with my exact setup/files: https://gist.github.com/eirvandelden/172548c72aff03181418c2c04516ebd5
I'm not yet sure if this is a bug in sinatra-activerecord, some incompatibility with ActiveRecord or a bug in my setup.

Just a hunch, does the .sqlite3 file has the guid and number column?

Yes, they are present: