Installation Error in Colab

I can't install flash-attn in Colab. How to fix this?
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Collecting flash-attn
  Downloading flash_attn-1.0.7.tar.gz (2.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.0/2.0 MB 22.0 MB/s eta 0:00:00
  Installing build dependencies ... done
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
  Getting requirements to build wheel ... error
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

same in Kaggle.  I tried to install it from GitHub but then I cannot import flash-attn

same in Kaggle. I tried to install it from GitHub but then I cannot import flash-attn

Same here. I've tried that too.

https://github.com/HazyResearch/flash-attention/pull/283


How to draw a rawarray Image

Right now I have to save my rawarray(image) first but I want to show it without saving to the drive.
var image_as_raw_array = []#is filled with an png image from www
var path = "res://cache.png"
if(f.open( path, File.WRITE )==0):
    f.store_buffer( image_as_raw_array )
    f.close()

    var img = ImageTexture.new()
    img.load( path )

    preview_sprite.set_texture( img )

Is there a way without "img.load( path )" something like img.load( rawarray )?
http://www.godotengine.org/qa/744/how-to-draw-a-rawarray-image

For storing a Image in a file you can use File::store_var and File::get_var.
But if you want create a Image from RawArray now is not pratical. But there's this Image constructor:
Image(int p_width, int p_height, int p_mipmaps, Format p_format, const DVector<uint8_t>& p_data);

I can bind this constructor GDScript, but i will need increase the maximun number of arguments a constructor can have, it's easy and will not afect anything.

Is it possible to read the width and height from the rawarray?
It works with "img.load( path )" so why not with img.load( format, rawarray )

In libgdx I used the Pixmap class to generate and manipulate Image/texture in game. Very fast and very cool to create procedurally generated content.
With new Texture(pixmap) you got the texture to draw.
Perhaps we could have something similarly in godot.

@AlexHolly, oh i'm sorry i I misunderstood the question, you can transform transform a RawArray in a Image using bytes2var.
@puppetmaster- Image and ImageTexture. Exposing the additional constructor, maybe, will expand it's possibilities.

I think bytes2var only works if the bytes where created by godot. My rawarray is a .png image from www
I try anyway.

bytes2var and var2bytes would (de)serialize a godot Texture, not a .png file :smile:

The idea of creating an Image directly from a RawArray looks very useful. At least I can see it working for my case, where I get images from a database. Currently I need to save it to a file locally and load it again as an Image.

Note that this works just fine for C++ (as evidenced by assetlib, even though AL would cache them).

If I could open this up again.
I was able to expose the constructor to gdscript by by editing the variant_call class
I added the ("data", Variant::RAW_ARRAY):
_VariantCall::add_constructor(_VariantCall::Image_init1,Variant::IMAGE,"width",Variant::INT,"height",Variant::INT,"mipmaps",Variant::BOOL,"format",Variant::INT,"data",Variant::RAW_ARRAY);
and adding *p_args[4]:
r_ret=Image(*p_args[0],*p_args[1],*p_args[2],Image::Format(p_args[3]->operator int()),*p_args[4]);
and adding then the argument:
const String& p_name5="", const Variant::Type p_type5=Variant::NIL)
to the add_constructor function.
So now gdscript allows me to use my RawArray from udp to create the Image.
However it errors because of a size difference. The "data" size that the image expects is much greater than the size of the RawArray.
So my question is what do a I need to do to my RawArray before it becomes ready to be image data?

Looking to utilize this as well.

@ArthurAttout There is Image::load_{jpg,png}_from_buffer.


Powershell succeeds when others don't

I've noticed this since I started using GitLFS.. but the powershell from GitHub Desktop will work and succeed at tasks when all other methods crash and burn.
Typically I'll do my work in SourceTree, but some major tasks that seems to suck/fail on.  So I'll switch to GitHub Desktop, and sometimes that'll do it better, sometimes not.  However, going to the powershell in GitHub Desktop seems to -always- work when the others fail.   Is there a reason for this?   Also, is there a reason GitHub Desktop itself would fail when its powershell succeeds?  Wouldn't it be doing the same commands under the hood?
Just trying to understand, as I think a lot of the "bugs" i experience using GitLFS might just be some config issue that powershell configs better when its doing things.  If i can figure that out, the whole experience overall might be bettter

So I'll switch to GitHub Desktop, and sometimes that'll do it better, sometimes not

@Leviathan1753 there's a log file found at %LOCALAPPDATA%\GitHub\TheLog.txt that will have some extra details around what might be occurring.
Also, we published an update to Git LFS 1.0.2 during the week. This might be related, but I'd love to understand what differences there are between the GUI and running commands in PowerShell.

Sorry about the delay, somehow missed this notification.   I'll see if I can trigger the difference again and report back.  Unfortunately, it's not always a surefire way that I've found to trigger the difference.  Usually is apparent when we're doing major merges or other messy work.  I've yet to find a truly repeatable case, which is why I was a bit hesitant making this report in the first place.  I'll let you know what I find with some logs.

Hi, I'm going through old issues that I've missed. I'm closing this until I can see some specific details.
However, there is some recent discussion in https://github.com/github/git-lfs/issues/1195 about similar issues. Git LFS provides better ways to download objects by bypassing git's internal filters, but it means there are manual commands you have to type, or that 3rd party tools like GH Desktop have to support.


Yosys Optimization Issue: Process Stuck at OPT_CLEAN Pass

Version
Yosys 0.30+48
On which OS did this happen?
Linux
Reproduction Steps
Hello,
I encountered an issue while using Yosys to synthesize a Verilog file.
During the synthesis process of my Verilog file rtl.v, the procedure gets stuck at the OPT_CLEAN pass and does not proceed further.
I have minimized the test case as much as possible to help with troubleshooting. Attached is the Verilog file (rtl.v) that triggers this issue. Below is a screenshot of the synthesis log right before the process gets stuck.

design.file.zip
Thank you in advance for your attention to this matter.
I look forward to hearing from you regarding this issue.
Expected Behavior
The synthesis process completes successfully within a reasonable time frame.
Actual Behavior
The synthesis process gets stuck at the OPT_CLEAN pass and does not complete.

This is almost certainly the same problem as #4478 given the very large negative offset from peepopt.


Bug Report About Auth

I have defined a redirectTo variable in AuthController beside use AuthenticatesAndRegistersUsers. But when I logged in(successfully), it still redirect to '/home' url.
As the document said

When a user is successfully authenticated, they will be redirected to the /home URI, which you will need to register a route to handle. You can customize the post-authentication redirect location by defining a redirectTo property on the AuthController.

I run my program in homestead latest, and my Laravel version is 5.1(Updated on 6.9)
Here's my code or AuthController:

<?php

namespace App\Http\Controllers\Auth;

use App\User;
use Validator;
use App\Http\Controllers\Controller;
use Illuminate\Foundation\Auth\AuthenticatesAndRegistersUsers;

class AuthController extends Controller
{

    use AuthenticatesAndRegistersUsers;

    protected $redirectTo = '/';

    public function __construct()
    {
        $this->middleware('guest', ['except' => 'getLogout']);
    }

    protected function validator(array $data)
    {
        return Validator::make($data, [
            'name' => 'required|max:255',
            'email' => 'required|email|max:255|unique:users',
            'password' => 'required|confirmed|min:6',
        ]);
    }

    protected function create(array $data)
    {
        return User::create([
            'real_name' => $data['name'],
            'email' => $data['email'],
            'password' => bcrypt($data['password']),
        ]);
    }
}

I've tried.
When I defined $redirectTo variable to '/', it always redirect me to '/home', but when I define it to something else, it effects.

use AuthenticatesAndRegistersUsers;

protected $redirectTo = '/test2';

Route::get('/test2',function(){
    return "Ok Ok";
});

My route file is like this:

Route::get('/', 'WelcomeController@index');

Route::get('/home', 'HomeController@index');

So I think the bug is
When I define $redirectTo to '/', it doesn't effect and still redirect to '/home' as the trait expect.

Please ask on the forums.


Changelog file is outdated

The current information in the changelog is outdated. It includes 0.15 which is likely to be irrelevant for developers today. I'd propose to delete the file (will be in git history anyway) or add links to other changelog files that contain information relevant to core.
https://github.com/elm/core/blob/master/changelog.md

Same here. I can't find a Changelog for 1.0.5.

Removed the file. Thanks for the report!


Venues examples

Added venues example notebook and requirements file

Hey Aisling @PolarisX7, just did the review, overall it looks good to me, just two things that might need to be improved:

there are 2 repeated checking final parameters that might be better deduped.
this is just a suggestion, might be worth finding someone who is good at editing to review again to polish some of the wording before publishing.


[Menu] component different position in production (Chrom)

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
I don't have a live example.
I am using version 4 of material-ui, but i tried also version 5
I have at the bottom left of my page a button which opens the menu.
Here's the code:
...
import { Menu, MenuItem } from '@mui/material';
...

                  <Avatar name={nomeCognome} size="34" round textSizeRatio={1.65} style={{ cursor: 'pointer' }} />

              <Menu
                  anchorEl={anchorEl}
                  id="simple-menu"
                  open={open}
                  onClose={this.handleClose}
                  anchorOrigin={{
                    vertical: 'top',
                    horizontal: 'right',
                  }}
              >
                  <MenuItem onClick={this.openProfiloUtenteDialog}>
                          {nomeCognome}
                  </MenuItem>
                  <MenuItem onClick={this.handleOnClickLog}>
                      <Trans id="Credit log" />
                  </MenuItem>

                  <MenuItem onClick={this.handleBack}><Trans id="Back" /></MenuItem>
                  <MenuItem onClick={this.handleLogout}><Trans id="Logout" /></MenuItem>
              </Menu>

Current behavior 😯
While if I open the page on a normal tab of Chrome, the menu will almost disappear at the bottom, as shown in the screenshot. And until a few days ago it worked ok

Expected behavior 🤔
In local and private navigation of Chrome the menu opens up perfectly above the button, also on other browsers (Firefox, Safari).
Here's the screenshot

Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

My very bad guys, sorry 🙏🙏🙏
I use a zoom function add a zoom style to the body, so the menu component enlarge as well


NameError: name 'create_agent' is not defined

Rasa version: 2.6.0
Rasa SDK version (if used & relevant): Not relevant
Rasa X version (if used & relevant): Not applicable
Python version: 3.7
Operating system (windows, osx, ...): Google Colab
Issue: When following the tutorial there is an error in the chat(model_path, endpoints) function. The error is related to the create_agent function that has not been correctly imported in jupyter.py. I solved by manually adding
from rasa.core.agent import Agent, create_agent in jupyter.py.
I also found this condition:
if typing.TYPE_CHECKING:
    from rasa.core.agent import Agent, create_agent

Is it necessary and above all, does it works as expected?
Error (including full traceback):
NameError: name 'create_agent' is not defined

thanks for raising @MorenoLaQuatra, I'll work on a fix for this.

if this is on the google colab environment, using rasa version 1.10.3, solved the "NameError: name "create_agent" is not defined" for me... should be a convenient workaround for now.

Thank you for pointing out the workaround.

@TyDunn this actually seems to have been fixed by this PR  referenced above - is it safe to close this ticket?


Fix bug where the fork stack in cycle detection pops undefined

fixes: #1303
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Confirmed with Romeo that it's okay to merge so we can get this into the next dev container image.


Preserve

Hi team, I added:

[ ] preserve data management file
[ ] preserve and load book and label related data

Status: Approved ✔
Hi Micronaut @Fahim-Khan-P
Great work!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

No linter errors ✔
Route Works✔
Implement classes :heavy_check_mark:
Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @tonnymuchui  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


How to draw a circle

I want to draw a simple geometric figure

That's a good one!
You can take a Container and set its border_radius to a half of size, for example:
import flet
from flet import Container, Page

def main(page: Page):
    page.add(Container(bgcolor="blue", width=100, height=100, border_radius=50))

flet.app(target=main)

So what if the hollow circle? By way of superposition?

Something like this

You can take a Container and set its border_radius to a half of size,

What if the width != height..
How can we make a circle from that? That will obviously not be the half again. 🤔
And btw, what is the max border radius of a container(i.e the value at which any further increase in border Radius will have no effect)?
And eventually, how can it be calculated..? 🤔

A new Container.shape property was added in the recent release.


[BUG]:  Answers are not saved when using Azure OpenAI

What would you like to see?
After reloading, the answers are blank:

I have tested switching to OpenAI, and it doesn't have this problem, only when using the azure version this happens.

Just patched, if running locally or from repo you can re-pull. Otherwise when the docker build completes the new image will have the fix.

Thank you! works now on my end


Statistics don't get logged

Version: 1.11.1.
The websocket/pusher functionality works just fine... but no matter if in the local environment or on our dev environment, no matter if using SQL Server or MySQL, statistics do not get logged.
The migrations are there, they've been run, and every setting is on its default. In fact, the project is just a fresh Laravel project with laravel-websockets installed.
I know that in \BeyondCode\LaravelWebSockets\Statistics\Logger\HttpStatisticsLogger::save() we do get to this part:
            $this
                ->browser
                ->post(
                    action([WebSocketStatisticsEntriesController::class, 'store']),
                    ['Content-Type' => 'application/json'],
                    stream_for(json_encode($postData))
                );

But then it all gets swallowed up there. I put a \Log::warning('here'); at the top of \BeyondCode\LaravelWebSockets\Statistics\Http\Controllers\WebSocketStatisticsEntriesController::store(), but it does not get called. There's no other exception/message logged anywhere.
I don't really understand this "browser" thing of how it works to debug it.

Well I had the same issue. But after 1 day trying to fix I just gave up and updated to laravel-websockets 2.x.
It started working and now I'm using 2.0.0-beta.33. And don't forget to copy the new websocket.php to your config/websocket.php. Somehow even after I upgraded it still wasn't logging in the dashboard, but at least I managed to workaround.
Here my fix after upgrading to 2.x.

@rennokki may I ask why you closed this? Do statistics work for you in the latest stable version of this library (1.11.1)?


feat(swc-angular-plugin): add support for latest @swc/core

@yjaaidi could you please check it?

👌


Matlab core function any support

🚀 Feature Request
Matlab core function any missing in mat2py, yet.
🔈 Motivation
"mean" function is implemented, but just passes without execution of code.
📝 Expected Results
%% Test basic functionality
A = [0 1 2; 2 3 4; 0 4 2; 4 2 6]

Any = any(A)
assert(isequal(Any, [1, 1, 1]))

Any1 = any(A == 1)
assert(isequal(Any1, [0, 1, 0]))

AnyAll = any(A, 'all')
assert(isequal(AnyAll, 1))

AnyDim2 = any(A == 3, 2)
assert(isequal(AnyDim2, [0; 1; 0; 0]))

🛰 Alternatives
📎 Additional context
To my mind this function should be equal to the numpy version:
return np.any(args)

It has already been implemented, but not imported because it is not in __all__ of mat2py.core. I have plan to solve all of those importing issue but for now you can just import it yourself like here did.

Temporary implementation by from mat2py.core.ops import _any works for me and passes my unit test:
    def test_any_function(self):
        A = M[[0, 1, 2], [2, 3, 4], [0, 4, 2], [4, 2, 6]]
        An = _any(A)
        assert(np.array_equal(An, M[[1, 1, 1]]),"standard check without additional arguments")
        Any1 = _any(A == 1)
        assert(np.array_equal(Any1, M[[0, 1, 0]]), "check with comparison, but without additional arguments")
        AnyAll = _any(A, 'all')
        assert(np.array_equal(AnyAll, 1), "additional argument 'all'")
        AnyDim2 = _any(A == 3, 2)
        assert(np.array_equal(AnyDim2, M[0, 1, 0, 0,]), "advanced check with dimension argument")


feat(en(): Revert "feat(en): Revert "feat(en): Fix operator address assignment for ENs""

Reverts matter-labs/zksync-era#977

@slowli this is the revert of revert of your PR :)


Polymorphic GlobalIdMap with ZooKeeper implementation

Add a GlobalIdMap implementation using the ZooKeeper Ebb.
This PR contains the following features and fixes required to make the above functional:

Polymorphic "GlobalIdMapBase"
Support for multi-inheritance types in the Messageable template
constexpr hash(string) method for generating deterministic static global EbbIds
correct znode argument passing in ZooKeeper

:umbrella: The latest upstream changes (presumably 0e086f2) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 1b2f4b8) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 922be49) made this pull request unmergeable. Please resolve the merge conflicts.

@BetsyRT try

:hourglass: Trying commit f8e6dcc with merge 7f8e140...

:broken_heart: Test failed - runtests

@BetsyRT try

@BetsyRT clean

@BetsyRT retry

:hourglass: Trying commit f8e6dcc with merge cde4db0...

:broken_heart: Test failed - runtests

@BetsyRT try

:hourglass: Trying commit 17f9faf with merge e010dfa...

:broken_heart: Test failed - runtests

@BetsyRT retry

:hourglass: Trying commit 17f9faf with merge b8abbaf...

:broken_heart: Test failed - runtests

@BetsyRT retry

:hourglass: Trying commit 17f9faf with merge 6377345...

:sunny: Test successful - runtests
State: approved= try=True


Add L2 solver for FUGW

Add solver when all the divergences in FUGW are L2-squared (instead of KL)

Tests seem to fail because if timeouts, however your last change seems minimal so I don't know what's going on.
Maybe you could push an empty commit to restart the tests.
If they fail again, could you make sure they run fast on a non-GPU setup?

Merging!


W1D2T2 - Micro-tutorial 7

Just a detail that was a bit confusing... In Micro-tutorial 7, in the step-by-step, it says after filtering to take the average, even though it is to take the final value.

Good catch! This has been fixed in the final notebooks.


Error while compiling with NGC

Hi! First of all thanks for an awesome component!
It worked fine until I compiled my project with NGC and got next error:
Error: ENOENT: no such file or directory, open '/Users/.../node_modu
les/angular-2-dropdown-multiselect/src/multiselect-dropdown.metadata.json'

I checked that folder and also could not find any *.metadata.json files there. They just do not exist in the module...
Version of your lib I'm using is: v1.1.0
Can you please help how to resolve the problem, 'cause I saw you have merged before PR related for it but still...
Thanks

PR #155 will fix this issue (I verified on my temporary package based on that PR).

Merged and published.


Err 137 with v2.1.1 in Gatsby 1.9.241

Hello,
I have reviewed all the open & closed issues, however, haven't found any with the solution to my issue.
My package.json has the following relevant info:
...
  "dependencies": {
    "bloomer": "^0.6.3",
    "bulma": "^0.6.2",
    "font-awesome": "^4.7.0",
    "gatsby": "^1.9.241",
    "gatsby-link": "^1.6.39",
    "gatsby-plugin-favicon": "^2.1.1",
    "gatsby-plugin-react-helmet": "^2.0.8",
    "gatsby-plugin-sass": "1.0.19",
    "nuka-carousel": "^4.4.0",
    "react-helmet": "^5.2.0",
    "react-responsive": "^5.0.0",
    "react-scroll": "^1.7.10"
  },
...

My gatsby-config.js is as follows:
module.exports = {
  siteMetadata: {
    title: 'ph - Crypto wave',
  },
  plugins: [
    'gatsby-plugin-react-helmet',
    'gatsby-plugin-sass',
    {
      resolve: `gatsby-plugin-favicon`,
      options: {
        logo: './src/favicon.png',
        injectHTML: true,
        icons: {
          android: true,
          appleIcon: true,
          appleStartup: true,
          coast: false,
          favicons: true,
          firefox: true,
          twitter: false,
          yandex: false,
          windows: false,
        },
      },
    },
  ],
  pathPrefix: '/ph_website',
};

However, I am getting the npm exit status 137 with code ELIFECYCLE
What am I doing wrong and facing this error?
Thanks,

Im closing this as I used an alternative method of appending a favicon as I couldn't get this to work.

Hi! Thanks for the report. I have no idea why you've got this error.
Maybe if you update Gatsby to version 2 and this plugin to the last version you can make it work.

Thanks for the feedback, in the end I just went ahead with ico file with the existing version. That resolved the requirement. Next time, will definitely use Gatsby 2. Thanks again.


telink: b92: fix pwm fault pulse

Adjust the execution position of gpio function disable .

@fengtai-telink You can merge this PR

ok, think, i will merge it .


Remove page from menu items for performance reason.

Fix #4276
Fix #4189

@mkoistinen are you ok with this?

Coverage decreased (-0.08%) to 87.735% when pulling 977111b0644c447e0dd3bbe813af38c76d9db10f on yakky:feature/remove_page into c0ad45f3f4989098e4cf81c65819a7e69d4a2de4 on divio:support/3.0.x.

Coverage decreased (-0.08%) to 87.735% when pulling 977111b0644c447e0dd3bbe813af38c76d9db10f on yakky:feature/remove_page into c0ad45f3f4989098e4cf81c65819a7e69d4a2de4 on divio:support/3.0.x.


Fix KeyDecodingStrategy.custom signature

When trying to provide test coverage for KeyDecodingStrategy.custom we've stumbled upon an API, which doesn't make much sense. The corresponding case for JSONDecoder involves creation of a dummy AnyKey structure to return a new synthesised coding key, which isn't guaranteed to match a coding key of the decoded type. It's also unclear how intValue of this dummy structure could be used.
All other cases of KeyDecodingStrategy delegate to functions with String -> String signature and my opinion this is the main application of KeyDecodingStrategy.custom: convert string keys to other string keys with a supplied transformation.
This is a breaking change though, so I'm still a bit unsure whether we should follow JSONDecoder or rely on common sense with this.
@regexident could you please provide your feedback, maybe you had some experience with this? Thanks.

Hah, if it wasn't for .custom (and the exact issue you're describing) the whole key transformation madness within XMLKeyedDecodingContainer could removed and the transformations be applied right within XMLParserDelegate's func parser(_:didStartElement:namespaceURI:qualifiedName:attributes:).
I stumbled upon this today, trying to remove all those unnecessary allocations caused by the late transformation.
There are quite a few places where Codable's API has made some sub-optimal choices.
For the sake of user-friendlyness though I guess we should try to keep the public API as close to JSONCoding, as possible. :/

thanks @regexident, that's super helpful!


[Autocomplete] onChange called twice when autoSelect and blurOnSelect are both true

When using the Autocomplete component with both autoSelect and blurOnSelect the onChange callback is invoked twice on each selection.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When the autoSelect and blurOnSelect props are both true on Autocomplete the onChange callback gets called twice on each selection (click or enter key).
It calls onChange only once when:

Take advantage of the autoSelect flag and hover over an item and then click outside the select to close.
When you clear the file using the X button.

Expected Behavior 🤔
Regardless of prop choices the onChange callback shouldn't be called more than once.

Steps to Reproduce 🕹

Can reproduce here:
https://codesandbox.io/s/condescending-sun-mee9h?file=/src/Demo.js
Steps:

Use AutoComplete component.
Add the autoSelect prop, make sure it's true.
Add the blurOnSelect prop, make sure it's true.
Add the onChange handler callback function.

Context 🔦
Honestly just stumbled across this in one of our own implementations. Sharing the cause so others don't need to spend too much time debuging.

Your Environment 🌎

Tech
Version

Material-UI
v4.9.10

React
v16.13.1

Browser
Chrome v80.0.3987.163

TypeScript
Nope

@wasinsandiego I'm not aware of any way to solve this problem. Any idea? We could close as "won't fix" otherwise.

@oliviertassinari Sorry, times is crazy so I let this slip. Thank you for the quick reply 🙏
Happy to look into it when things settle down for me. If it's something thats more effort than it's worth in the immediate future would a note in the docs be welcome?


making changes to tx builder model

Adding a signature to a conway era transaction causes a called Result::unwrap()on anErr value: CorruptedTxBytes error.
let signed_tx_cbor = tx
    // called `Result::unwrap()` on an `Err` value: CorruptedTxBytes
    .add_signature(witness_public_key, witness_vector)
    .unwrap()
    .tx_bytes;

This error comes from line 685 in model.rs of the txbuilder.
If you change the babbage call into a conway call then it does not throw the error.
This PR changes the babbage to conway on all the things. Also, it adjusts the witness building as pushing to a set is not possible and requires coverting back to a vector then back to an empty set.
This PR passes all current tests. There is no open issue on this topic as it brought up in discord.

I also commented out a debug statement that clutters the terminal screen.


difference between bind and property name

Hi I cannot clearly distinguish the following differences.
From the parent-to-child binding section, we can use one of these..
@bind-Year="parentYear" or Year="@parentYear".
Please let me know. Thanks.

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

ID: 8b1b6ad8-13ba-0195-8829-ade3b1b0b9cb
Version Independent ID: 6609ea68-a918-f3b9-8c9c-004a4436e350
Content: ASP.NET Core Blazor data binding
Content Source: aspnetcore/blazor/data-binding.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @junpark12 ...
@bind-Year="parentYear" is the proper syntax with the @bind directive. Year="@parentYear"should only fill the value ofparentYearinto the value of an attribute namedYearand not actually bind it (but I haven't even tested that, since it isn't supported). Use the@bind` directive syntax.


update black version in pre-commit-config

Motivation and Context
Formatting was updated in #1110, but version should also be bumped in the pre-commit config or formatting won't be correct for developer commits.
How Has This Been Tested
?
Types of changes

Docs change / refactoring / dependency upgrade

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

You also need to update the black version here I believe.

You also should select a fixed version here : https://github.com/facebookresearch/habitat-lab/blob/main/.pre-commit-config.yaml#L103 If you set 22.3.0 it will fix the errors, but I think you should set it to 23.1.0 to be consistent

Thanks @aclegg3 ! You also need to update the black version here I believe.

To be clear you are suggesting that we lock the black version in all locations instead of letting it upgrade automatically with future releases like it is now?

To be clear you are suggesting that we lock the black version in all locations instead of letting it upgrade automatically with future releases like it is now?

Yes, or at lest putting a maximum major version so that CI pre-commit does not just randomly break once in a while.


Windows 10 run error

I downloaded the Wondows version from https://flutter.gskinner.com/#g-download
When I run the program an enter an email address in the registration dialog, the complete keyboard is not useable as soon as I type the @ Character.
It feels like SHIFT and ALT are permantently pressed as with the CAPS-Lock key.

I think this is a dupe, we can not reproduce here. Are you using a non-english keyboard by chance?
https://github.com/gskinnerTeam/flutter-folio/issues/3

Sorry for answering that late. I use a German keyboard layout.


Copy Publish Artifact: "Contents" input markdown is incorrect

On the "Copy and Publish Build Artifacts" task, the Contents input markdown is incorrect.  It says "Supports multiple lines or minimatch patterns" which is not accurate.

What's inaccurate? The task carries two implementations so something may have regressed or never reached parity on a specific platform.

Sorry, I should have been more clear.  The input field is a textbox, so it does not support multiple lines like the markdown suggests.

The Contents input is "type": "multiLine" and I'm able to enter multiple lines. Are you not able to enter multiple lines? What browser/OS are you using?
Or are you saying it doesn't look like a multi-line input? I can't tell on my laptop - I'm on a small high-res monitor and Windows manipulates the size of inputs so they don't look really small - it does a good job most of the time, but i can't tell what the UI is really supposed to look like.

Interesting.  I wonder why multiLine is appearing as a textbox.  I'm using the latest Chrome with a production account.  See below:

@yaananth ?

That is just being displayed as textbox, but indeed it's textarea, will get styling fixed in current sprint. Next deployment should fix the look of it.


How to control the collision and path of the object moving to the desired point.

I am developing a game like Pachinko of Japan, currently I am stuck in the calculation based on the results of the server returned. I tried applying the MatterAttractors plugin to create marbles that move to the right place where the server wants to but it's pretty bad and not as natural as the real physical collision. I'm trying to use paperjs for creating marble paths based on given results and ignoring collisions between marbles. However, this is really bad when ignoring the collision. Do you have a way to control the path of objects when collisions occur? Please tell me a little about it!
Thanks you so much!

Have a look at the manipulation example which shows how you can move bodies exactly as needed. As for changing the outcome of collisions, well that's pretty fundamental stuff so it's very hard to change.

Thanks for your help!


Add support for npm ci

Currently JFrog CLI has support for npm install via the jfrog rt npmi command. The npm ci command offers a number of advantages in automated build environments including faster build times. To continue to use JFrog CLI for collecting dependancies during NPM build, it would be good to support npm ci.

Keeping it popping fresh.

I'd like to see support for this too.

Hi @scottrus and @CC84
I would like to update you that we have added the support for npm-ci. This is already included in our latest release (version 1.26.1). You can find the documentation regarding this here.
We will appreciate your feedback regarding this.

I can confirm that jfrog rt npm-ci works and runs npm ci. I think this issue can be closed. Note there is #705 about some cosmetic documentation issues between npm-install and npm-ci.

This is resolved for me.


People might use bookmarks with numeric text, conflicting with the regular mainline navigation method

fixed in v2.1.47


How to enable floating signature help

I 'm using your config. Can you suggest me the way to enable feature like this

Solved by using lspsaga.nvim.


ci: change to use sha from branch name

This PR fixes #879

Looks good, can you make a PR from a forked repo to test?

Do we test after merging this PR?

Ah, I didn't know how to fork by target branch. #881

Yep, we can test it before merging!
🟢  https://github.com/openameba/spindle/actions/runs/7327227031


docs: Fix Php Doc

Fix the php doc on object method from object to array.

This isn't correct. The return type is an object because the second param of json_decode is set to false.

I agree with you, but finally the method can return objects and array.
json_decode('{"a": "b"}', false) => object
json_decode('[{"a": "b"}]', false) => array of object


I can't run the .deb for some reason

when I try to run it it just say an error for me, here is the error:
[dark@LinuxPC linux-shimeji-master]$ ./linux-shimeji_0.0.1-3_all.deb
./linux-shimeji_0.0.1-3_all.deb: line 1: syntax error near unexpected token `newline'
./linux-shimeji_0.0.1-3_all.deb: line 1: `!<arch>'
[dark@LinuxPC linux-shimeji-master]$

Hi,
you need to install the package with
sudo dpkg -i linux-shimeji_0.0.1-3_all.deb
After the installation you can start it with
linux-shimeji
or by using the icon in Application.


Github-profile searchbox

Describe the project you want to add with tech stack

The project contains a serchbox which can give the details of an github id  within seconds.The user has to only input the github id of the person he/she wants to see.
Can you assign this task to me?
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 Can you assign this task to me?


Docs:  How does s2i differ from something like the fabric8 docker plugins?

One thing that is not clear to me - and perhaps could be addressed in the documentation.
How does using s2i differ from using something like the fabric8 maven docker plugin?
It looks like both can produce the same result:  They both take source code and build a final docker container with that source. The fabric8 approach builds the artifact outside of a container (no builder container to speak of), but in theory it creates repeatable builds.   The plugin injects the artifacts (war file, etc.) into the final container - much like s2i does.
Why would I want to use one vs the other?

as far as i know the maven docker plugin is still ultimately writing a dockerfile and executing a docker build, which is not what s2i does, the advantages of the s2i approach (not doing a docker build) are enumerated in the readme here:
https://github.com/openshift/source-to-image/blob/master/README.md#goals
in particular see the sections on speed and security.
the maven plugin also presumes you are working in a maven tool chain, s2i is agnostic to such things.
i'm also pretty sure the maven docker plugin is targeted at producing images for fabric, again s2i is agnostic.
So if fabric is your target, it makes sense to use it, if not, s2i is probably more flexible/appropriate.
@jstrachan maybe you can offer more background on the fabric8 maven plugin side.

I'm still not connecting the dots...
What does "not doing a docker build" mean?
Doesn't s2i execute docker commands to build an image?
If I examine the nginx example using docker history, I don't think I see a difference in the image layers  vs, what I would get if I just built the container directly.
What would really help me (and perhaps others?) is to show the same image being built two different ways (old school, and using s2i) - to illustrate the difference.
I do get the build vs. runtime image thing (no build tools needed in final runtime image). This seems to be the key value prop?
Maybe a more complex example (tomcat?) would illustrate the use case better.

Write a dockerfile that does:
FROM centos(or some tomcat image)
ADD mysource
RUN mvn package
RUN mv mywar /tomcat/deployments
You will get an image with 3 new layers on top of what centos had.
In addition, you can write more docker commands in that dockerfile like:
USER root
RUN some_superuser_command
RUN yum install
etc to perform privileged actions during your "build" process.
When you use s2i, only one additional layer is added, we are not writing a dockerfile, we are not performing a docker build. We are starting a new container, running commands within that container (the assemble script) and committing that container as a new image (which adds the 1 new layer).
And the assemble script can be enforced to run as a controlled(non-root) user so unless you have permission to sudo, you're not performing privileged actions, so if you use s2i with a source repo and a known builder image (assuming you don't override the assemble script), you are asserting more control over the types of actions that were taken to create the application image, compared to writing a Dockerfile where someone can yum install/curl download anything from anywhere.
The build vs runtime thing is definitely not the key value proposition, that's actually a brand new feature we've just delivered.
I actually just did a community briefing that covers s2i and the value proposition in detail, the recording should be posted soon, i'll put the link to it in this issue once it goes live, I think it may address more of your questions (and we can link it from the s2i doc).
@dmueller2001 can you link the s2i community briefing here once it's available?

Ahh , OK - so looking more carefully at the nginx example I can see one layer added to the base image.
Those first 2-3 paragraphs might be helpful in the README

@wstrange here's the briefing i mentioned:
https://www.youtube.com/watch?v=flI6zx9wH6M
i'll add it in the readme.

Awesome - watching it now!


[eta-tour]: Bug - [description-here]

Tour execution was stuck on the page below
Source Code:
module Lain where

main :: IO ()
main = putStrLn "Alex Iskold 🗽's first module!"

Output:
Hello
<img width="1121" alt="screen shot 2017-09-08 at 11 22 15 pm" src="https://user-images.githubusercontent.com/1530204/30236672-a6ffde7e-94ec-11e7-94f1-2e64ad663e72.png">

Thanks for reporting this! The culprit is that image next your Twitter handle, though it should properly tell you that instead of hanging. Need to filter those out when adding them to the editor. Does it work after you remove the image?


use ShardingArgumentCollection in HaplotypeCaller

I introduced a new argument collection to help share arguments between HaplotypeCaller and HaplotypeCaller spark called ShardingArgumentCollection.
To avoid causing a difficult rebase for other people I didn't change AssemblyRegionWalker to use it when I introduced it in (https://github.com/broadinstitute/gatk-protected/pull/616).  It should be made a top level argument collection and used in there as well as in HaplotypeCallerSpark

Issue moved to broadinstitute/gatk #2875 via ZenHub


Patch Envelopes: Add OpenAPI spec and update docs

This PR adds OpenAPI spec of Patch Envelopes to docs, so that Edge Apps developers could use generators for clients interacting with Patch Envelope endpoints of metadata server.
Also this PR updates docs for Patch Envelopes

@eriknordmark I see only one complaint in yetus here and it's not related to this PR. I created another fix for this #3751

The summary at the end has:
yetus: docs/api/patch-envelopes.yml#L126blanks:end of line
yetus: docs/api/patch-envelopes.yml#L127blanks:end of line
yetus: docs/api/patch-envelopes.yml#L20blanks:end of line
yetus: docs/api/patch-envelopes.yml#L30blanks:end of line
yetus: docs/api/patch-envelopes.yml#L98blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L17markdownlint:MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
yetus: docs/PATCH-ENVELOPES.md#L3blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L3markdownlint:MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
yetus: docs/PATCH-ENVELOPES.md#L49blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L53blanks:end of line
Annotations
10 errors and 1 warning
yetus: docs/api/patch-envelopes.yml#L126
blanks:end of line
yetus: docs/api/patch-envelopes.yml#L127
blanks:end of line
yetus: docs/api/patch-envelopes.yml#L20
blanks:end of line
yetus: docs/api/patch-envelopes.yml#L30
blanks:end of line
yetus: docs/api/patch-envelopes.yml#L98
blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L17
markdownlint:MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
yetus: docs/PATCH-ENVELOPES.md#L3
blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L3
markdownlint:MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
yetus: docs/PATCH-ENVELOPES.md#L49
blanks:end of line
yetus: docs/PATCH-ENVELOPES.md#L53
blanks:end of line


Add alias

Description
Starting an explainer for this one, as noted in the original ticket by @svonworl in https://github.com/dockstore/dockstore/issues/4422 it looks like there is something odd about the "entries" property in Collections (and thus Categories which are a subclass).
Currently, collections have entries (json ignored) and collection entries (serialized as "entries" in swagger, but disappear in openapi). The suspicion is that there is an issue with properties named "entries" , another theory is that we have clashing property names i.e. collection entries is being renamed and clashes with actual "entries"  property
So we have a few options

In this PR, return collectionEntries twice, once as "entries" for backwards compatibility and once as "collectionEntries" which will show up in openapi.yaml
Return collectionEntries not as a "entries" property but as "collectionEntries", this will break backwards compatibility with the existing UI, but presumably no CLI users use it anyway

I kinda prefer the second option, but wanted to get opinions before I go on a large renaming and backwards compatibility breaking exercise. Tagginng @svonworl who worked on this, @hyunnaye who is using this now
Review Instructions
Describe if this ticket needs review and if so, how one may go about it in qa and/or staging environments.
For example, a ticket based on Security Hub, Snyk, or Dependabot may not need review since those services
will generate new warnings if the issue has not been resolved properly. On the other hand, an infrastructure
ticket that results in visible changes to the end-user will definitely require review.
Many tickets will likely be between these two extremes, so some judgement may be required.
Issue
https://ucsc-cgl.atlassian.net/browse/DOCK-1890
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[ ] Check that you pass the basic style checks and unit tests by running mvn clean install
[ ] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[ ] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[ ] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[ ] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[ ] Do not serve user-uploaded binary images through the Dockstore API
[ ] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[ ] Do not create cookies, although this may change in the future
[ ] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

If we go for Option 2, we might rename CollectionEntry to something like CollectionElement or CollectionItem, since a Collection is more than a set of Entries.
And, while we're breaking things, maybe we make a list of other naming issues, elsewhere in the API, and possibly fix them too (especially if they're in similarly-ancillary parts of the product).

Better solution in https://github.com/dockstore/dockstore/pull/5563


Error after installing react-stl-viewer

Hi guys,
I just installed the react-stl-viewer to test it, but I somehow get the following error:
Failed to compile.
./node_modules/react-stl-viewer/lib/index.esm.js
Module not found: Can't resolve 'three' in 'CUSTOM_PATH/node_modules/react-stl-viewer/lib'

Not sure if this is on my site or if react-stl-viewer just did not install the three.js component.
Looking forward to your help!

Hi!
react-stl-viewer is expecting you to install yourself three, as it is declared as a peer dependency. As three is really big, react-stl-viewer does not include that library in its bundle.
If you install three yourself and declare it as a dependency in your package.json you should be ready to go

Perfect, thanks for the quick reply! Now everything is working!

Hi! react-stl-viewer is expecting you to install yourself three, as it is declared as a peer dependency. As three is really big, react-stl-viewer does not include that library in its bundle. If you install three yourself and declare it as a dependency in your package.json you should be ready to go

Hi.

For example I have next.js application with static page, and inside a product details page, that is generated statically, clientside I want to resolve my stl files in list of images for media for product with your plugin.
Next.js allow dynamic import any library
https://nextjs.org/docs/advanced-features/dynamic-import#with-external-libraries
So, how to deal with three.js in this case?
Thanks!

I am pretty sure that it is not possible to make react-stl-viewer work with SSR, so you should use the StlViewer only client-side.
I guess one way of achieving this is to dynamically import this library, with that also three should be imported dynamically


Replace KoanfValueStore with MapStore to fix config deep-merge

Summary
Turns out that Koanf doesn't actually do deep-merging of objects.
Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
fix, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Update documentation.
[x] Update tests.

Hi @ccremer, randomly stumbled upon this issue. koanf does deep merges recursively. Would you be able to describe the issue so that it can be fixed in koanf?

Hi, I'm curious how you randomly landed here :)
The testdata in the unit test should provide you start.
Given the following YAML:
:globals:
  features:
    testing:
      enabled: true
      anotherKey: value

.github/workflows/release.yaml:
  features:
    testing:
      enabled: false

In this example, :globals is the dict that defines base values. When merging the dict of .github/workflows/release.yaml into one, I'd expect features.testing.enabled=false in the result as the dict under .github/workflows/release.yaml overrides :globals, which was the case. so far so good.
However features.testing.anotherKey was completely missing in the resulting map.

@knadh did that answer your question?

Whoops, lost track of this. I recreated your exact test case:
func TestMerge2(t *testing.T) {
	src := map[string]interface{}{
		"globals": map[string]interface{}{
			"features": map[string]interface{}{
				"testing": map[string]interface{}{
					"enabled": false,
				},
			},
		},
	}

	dest := map[string]interface{}{
		"globals": map[string]interface{}{
			"features": map[string]interface{}{
				"testing": map[string]interface{}{
					"enabled":    true,
					"anotherKey": "value",
				},
			},
		},
	}

	Merge(src, dest)
	fmt.Println(dest)
}

$ go test
map[globals:map[features:map[testing:map[anotherKey:value enabled:false]]]]

anotherKey is not lost in the result. Swapping src and dest don't cause the key to disappear either.

hm, interesting!
Maybe I'll do another round of investigation later, maybe the root cause was something else.

@knadh I'm picking up here and I've found the culprit.
I wasn't using maps.Merge, but k.Unmarshal, like this:
data := make(map[string]interface{})
err := k.Unmarshal(":globals", &data)
// ...
err := k.Unmarshal(".github/workflows/release.yaml", &data) // I was expecting data to be merged

However, the 2nd unmarshalling removed anotherKey.
I guess this was a design oversight and nothing wrong with koanf after all.
If the 2nd overwrite is using Cut() instead, it works.
So my map is now retrieved with:
merged := root.Cut(":globals")
// ...
err := merged.Merge(root.Cut(".github/workflows/release.yaml"))
// ...
return merged.Raw()


Improvements to PyPI packaging

The new version of PyPI and Warehouse added support for PEP 566,
which adds the option "long_description_content_type" to specify the
format of the long description used on the PyPI page. This removes
the need to convert it to RST using pypandoc, which (for this package
at least) was not formatting correctly. Additionally, I fixed up a few other things while I was at it.
Summary:

Remove need to convert README to RST
Ensure LICENSE gets included in the package
Cleaned up setup.py a bit.

The PyPI page for this project is currently raw RST, hence the PR out of the blue 😄

This is great, thank you!


Install php module

Hello,
first, thanks for your nice work on this docker file.
I am new to docker. Switching from Vagrant to Docker.
I tryed to install the php_mysqli.dll into docker, by including this line into /etc/php/development.ini
extension = php_mysqli.dll
After rebuild this seems not working. php -m don´t shows the module.
Changes on the development.ini are working. I tryed to edit max_input_vars to 5001 and this showed up.
My question is, how to activate / install needed extensions.
Thanks for your help.
regards
Torsten

Hi, the mysqli PHP extension is already available.
What are you exactly trying? What is the goal? Changing max_input_vars is done in another way. (Maybe @hhoechtl can post an example?)

@totemusik JFYI: https://github.com/webdevops/TYPO3-docker-boilerplate/blob/develop/documentation/CUSTOMIZE.md#install-moreothercustom-php-extensions

I close the issue, as there where no feedback.


Liveshare mostly broken for the guest user using Golang

Describe what happened:
Intellisense is broken most of the time for the guest user when developing in Go. Syntax highlighting is present, though.
Other things we are seeing are "ghost errors" - red squiggly underlining where the host has none.
What was your system configuration?

VS Code 1.62.0
LiveShare v1.0.5090
Go 1.17.2
MacOS 12.0.1
We are experiencing the same thing now in Golang, on VS Code 1.62.0 & LiveShare v1.0.5090.

Steps to Reproduce / Scenario:
Be the guest to a Liveshare session with the above settings.

@daytonellwanger do we have a general sense on our Go language support?

We’re not able to prioritize this issue (relating to Go language support) over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it is for you.


reademe, quickstart is outdated

"go get" is no longer supported outside a module since go1.18
use "go install" instead.
"go install github.com/minio/minio-go@latest"
error: package github.com/minio/minio-go is not a main package
"go install github.com/minio/minio-go/v7@latest"
error: package github.com/minio/minio-go/v7 is not a main package

You use go get to add it to your module. The readme is fine.

This is an SDK you need to depend on it, so you need to go get under your project.
go install is meant for standalone programs like mc or minio


feat: v6.0.0

Restructure of the API with a focus on a leaner bundle and more straightforward interface.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ed15e96). Click here to learn what that means.
The diff coverage is n/a.

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

@@            Coverage Diff            @@
##             master     #171   +/-   ##
=========================================
  Coverage          ?   99.68%
=========================================
  Files             ?       12
  Lines             ?      316
  Branches          ?       52
=========================================
  Hits              ?      315
  Misses            ?        0
  Partials          ?        1

Continue to review full report at Codecov.

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

Closing in favor of the v6 branch


Update SQL Server Trace Flag.md

SQL Server 2019 CU29 added a handful!

Thanks, @BrentOzar, amazing work by the SQL Server team! Adding new workarounds—thankfully, they’re documented!


backfeed quote tweets

requested by @kartikprabhu on irc. quote tweets can be identified by the quoted_status_id field. more docs:

https://support.twitter.com/articles/20169873
https://dev.twitter.com/overview/api/tweets
http://indiewebcamp.com/quotation#Silo_Examples

looks like we can find them by searching for the id of the quoted tweet. for bridgy, we can search for the user's last N tweets by ORing the ids together, e.g. https://twitter.com/search?q=564316935001628672 OR<PHONE_NUMBER>57380608
TODO: are there corresponding features in any other silos? cc @tantek @kevinmarks @aaronpk for their silo knowledge.

+1 to this, especially as I am trying out emoji likes as tweets. e.g. https://twitter.com/kevinmarks/status/662041840938422274 of https://twitter.com/benwerd/status/662002025270349825 that is the POSSE copy of http://werd.io/2015/creating-a-usable-text-editor-in-a-web-browser-is

+1 to this as someone who's tweets get quoted once in a (rare?) while, like:
https://twitter.com/kfury/status/661738224969121792
And of course @KevinMarks doing his reacjweet thing:
https://twitter.com/kevinmarks/status/661802300407545856

it's happening, kyle, just watch http://emojitracker.com

what are quote tweets? reposts with content? replies with a cite? how should we translate a quote tweet to mf2?

i don't think the mf2 vocab is a settled issue yet, but i'm fine with whatever markup you go with.
for the record, in my mind it's definitely a repost + content, just like Tumblr reblogs and Facebook shares (which can have commentary but are primarily about highlighting the original). I believe @tantek argues that repost cannot have additional content, but retweets are basically the only implementation I can think of that doesn't allow you to add commentary.

yup, agreed that the real world examples mostly do allow commentary.
one additional consideration is that many webmention handlers currently facepile reposts, probably even if they have content, so their display would hide it. we could maybe encourage people to not facepile reposts with content.

When I repost something, my intent is to share the original content as a post with equal weight as one I wrote myself.
Twitter shows a facepile and count of pure reposts, and quoted tweets are not part of that count.
I am of the opinion that a quoted tweet, or repost with content, is not the same as a pure repost. It is more along the lines of just sharing a link and adding a comment, where the rendering of the post also includes the original content.

thanks for the input! do you think the same of facebook and google+ shares with messages? (which they encourage you to include.)

I'm currently leaning toward backfeeding quote tweets as replies, not reposts, because many webmention handlers render reposts in facepiles, so the comments in them would be hidden.
i don't feel strongly, and I'd happily switch if enough facepile implementations start fully rendering reposts that have content.
feedback is welcome!

That makes sense to me; the facepile code could look for the pure emoji
ones if necessary
On 5 Dec 2015 1:57 p.m., "Ryan Barrett"<EMAIL_ADDRESS>wrote:

I'm currently leaning toward backfeeding quote tweets as replies, not
reposts, because many webmention handlers render reposts in facepiles, so
the comments in them would be hidden.
i don't feel strongly, and I'd happily switch if enough facepile
implementations start fully rendering reposts that have content.
feedback is welcome!
—
Reply to this email directly or view it on GitHub
https://github.com/snarfed/bridgy/issues/504#issuecomment-162250687.

They're not replies, so could you just add a link to them without in-reply-to?

a link to them without in-reply-to

+1 this

They're not replies, so could you just add a link to them without in-reply-to?

sure! that could work too.
honestly though, to be fair, we don't know what they are. twitter definitely doesn't say explicitly. so we kinda get to choose for ourselves.
i know some webmention handlers suppress mentions similarly and just render then as "X mentioned this." fewer than facepile responses, maybe, but still some, and that's not ideal either.
i dunno.

They can be replies, they are like top posted ones in email
On 5 Dec 2015 3:34 p.m., "Aaron Parecki"<EMAIL_ADDRESS>wrote:

They're not replies, so could you just add a link to them without
in-reply-to?
—
Reply to this email directly or view it on GitHub
https://github.com/snarfed/bridgy/issues/504#issuecomment-162256921.

Twitter calls them "quoted tweets" but doesn't show a counter of how many times a tweet has been quoted.
I would say don't add any special property until we have a better idea of what that property might be. If you don't add anything except a plain link to the tweet (and more accurate would be to also include the full text of the quoted tweet), then webmention receivers will treat it just like any other URL linked to their post.
You could add h-cite markup around the text of the quoted tweet which would be an accurate representation of what's happening.
For example, this tweet https://twitter.com/joshmarinacci/status/666413402789535744
Here's how I imagine it showing up after parsing the microformats: https://gist.github.com/aaronpk/5cb3306d6ba0605b5425

agreed, i ended up coming around to the same conclusion last night. thanks for writing it up, @aaronpk! the tweet and markup example are definitely helpful. plus, i'd like to encourage pure mention handlers to look at content length and show the whole content if it's under a certain size, and this is definitely good ammo for that.

I like that answer with the embedded h-cite - I should make my fragmention
auto-linker do that too
http://www.unmung.com/?rawtext=Thus+we+could+turn+a+link+like+this%3A+https%3A%2F%2Fkartikprabhu.com%2Farticle%2Fmarginalia%23In%2520the%2520case%2520of%2520digital%2520content%2C%2520the%2520artifact%2C%2520once%2520created%2520and%2520published%2C%2520is%2520not%2520static.+into+an+inline+embed&maxurllength=0&embed=on
On 6 Dec 2015 9:16 a.m., "Ryan Barrett"<EMAIL_ADDRESS>wrote:

agreed, i ended up coming around to the same conclusion last night. thanks
for writing it up, @aaronpk https://github.com/aaronpk! the tweet and
markup example are definitely helpful. plus, i'd like to encourage pure
mention handlers to look at content length and show the whole content if
it's under a certain size, and this is definitely good ammo for that.
—
Reply to this email directly or view it on GitHub
https://github.com/snarfed/bridgy/issues/504#issuecomment-162330007.

Thinking about what to do with replies to quote tweets. I think we've concluded that quote tweets are basically equivalent to "mentions" (i.e. the same as someone tweeting a link to a post on your site). And so replies to a quote are not considered replies to the original, and will not be backfed.

:+1:

Well, they are replies too, except that twitter shows them as top posts. It
doesn't thread them though, so they can be discussion terminating.
On 20 Dec 2015 11:16 a.m., "Ryan Barrett"<EMAIL_ADDRESS>wrote:

[image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/snarfed/bridgy/issues/504#issuecomment-166148160.

one remaining issue: the mentioned link is added to the outer h-entry, but it should really be the u-url of the contained h-cite.

  this is a quote tweet https://twitter.com/kylewm/1234

    this is the thing being quoted

Deployed... It doesn't seem to have broken anything, but it also doesn't seem to have worked. It's not finding new tweets, e.g. searching for
https://twitter.com/dissolve333/status/677170470152593408
should find
https://twitter.com/kylewmahan/status/677171403544616960
Here's the query. Confirmed via the API console that Twitter does return that quote tweet. Not sure yet why Bridgy isn't backfeeding it. and now i have to go do Christmas family stuff :(
679698503644352512+OR+677937737186459648+OR+677809810084110336+OR+677172635407634432+OR+677170470152593408+OR+668510418902487040+OR+664964044860686336+OR+664561829201375232+OR+663777396206776322+OR+663008355120758784+OR+662665847509504000+OR+659391428343300096+OR+654295558962720768+OR+652490299374092288+OR+651858246257389568+OR+644705523988889601+OR+644242456846856192+OR+644242104869195780+OR+644241717701378048+OR+644241008432037888

i think that's because bridgy includes since_id in its search, so it will only find quote tweets created since the deploy. totally ok with me, fwiw.

Oh good point!
On December 24, 2015 10:20:55 PM PST, Ryan Barrett<EMAIL_ADDRESS>wrote:

i think that's because bridgy includes since_id in its search, so it
will only find quote tweets created since the deploy. totally ok with
me, fwiw.

Reply to this email directly or view it on GitHub:
https://github.com/snarfed/bridgy/issues/504#issuecomment-167199660

Yay! New quote tweet from me to Ben went through:
2015-12-25 15:39:29.495670 I Webmention from https://brid-gy.appspot.com/post/twitter/dissolve333/680412194941476864 to https://ben.thatmustbe.me/note/2015/12/18/1/

woo yay congrats! a Christmas miracle! thanks @kylewm! 🏅🎉🎄

Re-reading this thread per Bridgy Publish feature request issue 720, note that we have a separate explicit post type for quotation posts, with explicit markup:
https://indieweb.org/quotation#How_to_markup
Not sure if/when/how many people are publishing or consuming, but wanted to at least provide an update here in case we want to consider updating what Bridgy backfeed uses as markup to represent backfed quote tweets.

done! thanks for the nudge.
example updated markup (simplified) from https://brid.gy/post/twitter/dissolve333/680412194941476864 :

  ...
  ...

    ...

    New reacji UI, backfed reacji from twitter now working, and all likes are now converted to reacji hearts!… (btmb.me/s/Da)

  ...


Reusable Agent In Different Scenarios

Hi,
I create a deal watching chain along the lines of RSS Agent (Input By Category X) -> Website Agent (Scrape Additional Info) -> Event Formatting Agent (Aggregate some fields to make a "key") -> De Duplication Agent (Remove duplicates based on key) -> Trigger Agent (Some more generic filtering) -> Data Output Agent (RSS Output).
I'd like to duplicate this chain for different RSS Agent input URL representing a different category, but I don't think I can re-use the already configured agents - like the event formatter and de-duplicator which would function in exactly the same way - in between.  To accomplish that I need to clone all the agents, change rss input feed url, rename all cloned agents, re-link with new names.  Maybe I'm mistaken, I haven't been using huginn for long.
I think if events were tagged with Scenario name, and output event is routed based on scenario name, then re-use of agents could be made possible.

I don't think that's possible now.  It would be cool to be able to clone a Scenario (not possible now).

A workaround that I've found is to export the scenario to a JSON file, and then change it's guid and re-import it back to Huginn.
Makes it seem like implementing this would be quite easy, then ;)


Update restart policies to match on-fail to remove always

Long story short: containers shouldn't be present after a system reboot, and this has caused some issues for users. Note that this issue has been affecting some users in the original project.

https://github.com/amazeeio/pygmy/issues/40

yup - dnsmasq appears to be the main culprit for me!

@tobybellwood, are you able to test the change on branch issue/184 before it gets a PR?


Does not respect samesite cookie setting

Expected Behavior
In the settings, I set:
SESSION_COOKIE_SAMESITE = "Stict"

this is respected by the Django session framework and I'd expect django-user-sessions to do so, too.
Current Behavior
The picture should be sufficiently clear:

For web deployments, I also have set
SESSION_COOKIE_SECURE = True

I presume this is not respected either, but can't confirm this at the moment because this setting is turned off for local development and testing, as we don't have ssl enabled there.
Steps to Reproduce (for bugs)

Set the setting SESSION_COOKIE_SAMESITE = "Strict" (or any other value)
Login
Confirm in the web storage explorer of your devtools that this setting is ignored

Your Environment

Browser and version: Firefox 74.0
Python version: 3.8.1
Django version: 3.0.4
django-otp version: Not installed
django-user-sessions version: 1.7.1
Link to your project: Irrelevant

Thank you for your contribution!


active twitch account failure

open the exe，and click loggin turn to twitch active page when i copy active codes then this page tell that someting error
how to solve？pls

Hello o/
There is no immediate solution at this time. I'll study login issues once #462 can be resolved, or I'll run into said login issue myself. For now, since #462 is still in effect, I'd recommend not trying to use the miner for the time being.

wish you solve it soon  ..good luck!

Please check the latest master build to see if that resolved your issue.

Is this issue still happening to you?

I've tested the login method a few times over the past week or two, and found no issues. I'm closing this due to lack of further information provided.


test: add playwright testing to start-basic

This adds some playwright tests to start-basic to ensure that it's working correctly.

pnpm run test:playwright runs the tests locally
pnpm run test:playwright:replay runs the tests with the Replay browser

To view the replays, join the TanStack router team. Here's an example replay of one of the new tests.
https://www.loom.com/share/d31ef881d2894cdd9e566e56ce9a223d

@jasonLaster you can close this in favour of https://github.com/TanStack/router/pull/1673


support binding to a given address only

Also, defaults to listening on localhost by default. Set --address <IP_ADDRESS> to
listen on all interfaces.
fixes #55

The failing check seems to fail on master too:
https://travis-ci.org/75lb/local-web-server/branches

Hi, thanks for this PR - it highlighted that local-web-server was suddenly broken in node v4 and v5 thanks to misuse of semver in the Koa project.
Thanks for the new feature - I started looking at it but have run out of time, for now. Will look again tonight. The first issues i noticed were lack of basic validation:
$ ws -a
serving at http://null:8000

$ ws -a server.home
events.js:163
      throw er; // Unhandled 'error' event
      ^

Error: getaddrinfo ENOTFOUND server.home
    at errnoException (dns.js:28:10)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:73:26)

Will look more later.

I confess that I have not done much input validation.

$ ws -a
serving at http://null:8000

This seems like something the command line parser should handle. Is there a way to mark the argument to -a as required?

$ ws -a server.home
events.js:163
throw er; // Unhandled 'error' event
^
Error: getaddrinfo ENOTFOUND server.home
at errnoException (dns.js:28:10)
at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:73:26)

While verification for this can be implemented, it requires a DNS lookup. Maybe it would be better to only allow IP addresses. They can easily be validated. Is there a real use-case for listening on a hostname? It's usually either <IP_ADDRESS> or <IP_ADDRESS>, or then a specific IP address assigned to a local interface.

don't worry, I will see to this feature. I working on version 2 of local-web-server as we speak, it is long overdue. I'll let you know once it's in beta!


Spool cover and tensioner not connecting

I printed the tensioner and spool cover pieces and they are not connecting properly. In the video he was able to put them on without issue. I was able to get them together, but they broke in the process. They were printed with a resin printer. Is that the problem? I tried using a 3D printer but it was having issues.

I haven't printed the parts in resin (but was thinking of giving that a go) but make sure you've sanded any elephant footing etc off.  Also consider volumetric shrinkage when curing - can be up to 5 or 6%!!  If I can find time I might print a v4 set in resin and see how it goes too.

Another think to keep in mind (and I'm not sure how this translates to SLA printers, but) you want to make sure that you keep print the cover with the side that has a lip AWAY from the bed, because that's the part that is the most sensitive to elephant's foot.

I'm having the same issue, on the previous prototype they snapped in for me but for this print I've only gotten 1 out of 10 to snap-in. It might just be of my own doing but I'm not sure what would be causing it. I even printed them with the lip away from the bed as Lucas suggested and still didn't seem to help much.

I found that if I clipped the ring on the tensioner before curing there wasn't the issue of snapping.  It requires a little more fiddling around to get the spring hook in, but once you start winding there was no difference.
If that isn't feasible, maybe take a sharp knife/scalpel and cut a notch in the lip of the tensioner and in the lip of the ring and then "screw"/twist the cover on rather than snapping it on.


go: use non-versioned buildGoModule to support out-of-band compilers

If you set languages.go.package to a version of the go compiler that doesn't yet exist in the nixpkgs input, you'll get an evaluation error on buildGoXXXModule because the function for that specific version doesn't exist.
I couldn't find anything in the docs preventing us from using the base buildGoModule and override the go argument as we already do.
Fixes #1568.

This works. The bigger issue is that older tools fail to compile with newer versions of go. That makes updating just languages.go.package non-viable if we have to recompile all the tools against this version.


Issue with prive command line tool

I am getting the following error:
site-packages/prive/run.py", line 15, in
from prive.threat_models import TargetedAuxiliaryDataMIA
ImportError: cannot import name 'TargetedAuxiliaryDataMIA' from 'prive.threat_models'
I think this is just the refactoring from the last commit and should be fixable? @fhoussiau

I believe the latest push has changed this, could you please check it now works? Thanks!

Getting there, it now gets through a few more stages. Running:
prive --datapath privacy-sdg-toolbox/prive/tests/data/texas --genpath privacy-sdg-toolbox/prive/bin/raw --runconfig privacy-sdg-toolbox/prive/configs/config.json
Give me the new following error:
File "/Users/andrewelliott/miniconda3/envs/privTool/lib/python3.10/site-packages/prive/run.py", line 114, in main
test_labels, predictions = threat_model.test(
TypeError: LabelInferenceThreatModel.test() got an unexpected keyword argument 'replace_target'
I think this attribute is now in the threat model init rather than the test function?


Update safeguarding_for_device_users.md

Context
Changes proposed in this pull request
Guidance to review

Added via https://github.com/DFE-Digital/get-help-with-tech/pull/349, closing.


update jxcore+v8-<IP_ADDRESS> OSX package's sha

same as #106 https://github.com/jxcore/jxcore/issues/774#issuecomment-170566013


Adding a new event - Serverless Workshop on Amplify and AppSync

Issue #, if available:
NA
Description of changes:
I am hosting and organising a Serverless Workshop on 5th September, 2020 on Amplify and AppSync and would like to add it to be listed in the list of events. Link to the workshop details - Serverless Workshop
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hey @dabit3, any specific reason why my MR was approved, but not merged? And is now closed?
Thanks!


WIP version of blacklist

Fixes #9
Currently, JWTManager does not check any token for being blacklisted.

Thanks for your contribution!

@NovemberOscar

Thanks for your hard work again :)
but your PR is different from what I thinking.
https://gist.github.com/NovemberOscar/e75822ac896c548b0de9eb2f9febf6a4
it must be like this. (I've issue)
please implement like this and open new PR if you want.
Thanks.


Getting a timeout when trying to view new Vigicrues dataset

If we try to configure the dataset as tiled, the visualization becomes slow

A first problem on using tiling here is that the probe geometry (polygon) does not match the observation geometry (bbox) so that sometimes an observation intersects a tile while the probe does not. In the last case the layer is thus updated by adding the bbox as a new feature.


won't play mp3

Hi,
I installed your app because I needed an mp3 player, but Just Player would not play the mp3s I had. I downloaded the mp3s from libro.fm and then, copied them to the phone. Fyi, another app played them, but 2 other apps did not.

Model: moto g power (XT2165DL)
OS: Android 12
Version 0.168

https://github.com/moneytoo/Player#are-there-any-media-formats-it-cannot-play

Just Player focuses on playing videos so audio only playback isn't officialy supported

Duplicate of #55


refactoring: use of cl_object_collection_iterator

object cl_object_collection_iterator is not a steampunk relasesed object #632
suggest implementing the iterator functionality in a new zcl_excel_iterator class, and using that instead

and cl_object_collection

Merged → Closing.

only first part done, todo: use the new classes

For information, I realize now that it breaks the ascendant compatibility, my code based on abap2xlsx doesn't compile anymore, I need to replace cl_object_collection_iterator everywhere. Not a big deal but we'd better have released a new major version i.e. contains incompatible changes, so that the client is aware to review all code that is using abap2xlsx.

@sandraros what's the exact NetWeaver ABAP version where you ran into issues?


Using hotswap lib to add/remove pages w/o restart

... because NodeBB restarts are annoying and take too long.

Requires nodebb/nodebb@2ddcaa9 or newer, hence the nbbpm change to require ^1.0.4


Add netty ByteBuf to network layer

Adding netty ByteBuf to network layer and add a new config option to enable or disable it.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@795ff13). Click here to learn what that means.
The diff coverage is 60.43%.

@@            Coverage Diff            @@
##             master    #1294   +/-   ##
=========================================
  Coverage          ?   72.26%
  Complexity        ?     6332
=========================================
  Files             ?      459
  Lines             ?    36295
  Branches          ?     4591
=========================================
  Hits              ?    26230
  Misses            ?     8853
  Partials          ?     1212

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com.github.ambry/account/AclService.java
0% <ø> (ø)
0 <0> (?)

...github.ambry/network/BoundedByteBufferReceive.java
93.33% <ø> (ø)
13 <0> (?)

.../com.github.ambry.network/BlockingChannelInfo.java
85.48% <ø> (ø)
16 <0> (?)

.../com.github.ambry/tools/admin/ServerAdminTool.java
0% <0%> (ø)
0 <0> (?)

...ithub.ambry.messageformat/MessageFormatRecord.java
79.03% <0%> (ø)
28 <0> (?)

...ithub.ambry.utils/NettyByteBufDataInputStream.java
0% <0%> (ø)
0 <0> (?)

...java/com.github.ambry.network/SSLTransmission.java
70.89% <100%> (ø)
74 <0> (?)

...in/java/com.github.ambry.network/SocketServer.java
85.12% <100%> (ø)
15 <0> (?)

....github.ambry.router/TrackedRequestFinalState.java
100% <100%> (ø)
4 <3> (?)

...java/com.github.ambry/rest/RestRequestMetrics.java
100% <100%> (ø)
4 <0> (?)

... and 19 more

Continue to review full report at Codecov.

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


problem in navbar

Describe the bug
There is a big problem in navbar
Expected behavior
It must be visible clearly
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

please assign me this issue

@thestarsahil please assign me this issue


sfpowerkit:source:profile:reconcile removes a valid recordtype layout

Describe the bug
When running sfpowerkit:source:profile:reconcile, a specific recordtype layout assignment keeps getting pulled from the profile despite the layout and record type being in the org.
In my profile, I have a page layout assignment for a record type:

In the org, the layout exists:

as well as the record type (found with SOQL SELECT Name, DeveloperName,  SobjectType FROM RecordType where SobjectType = 'Campaign':

Running the reconcile command with trace, I see:
Found in Org? Campaign.AC_Campaign false
To Reproduce
Steps to reproduce the behavior:

sfdx sfpowerkit:source:profile:reconcile -n Admin -u [org]
Look at what was removed from the profile

Expected behavior
The record type page layout assignment should not be removed from the profile
Screenshots
See above
Desktop (please complete the following information):

OS: MacBook 12.3.1
Node Version: v16.13.2
Version of sfpowerkit[e.g 1.45.0]: 4.2.5
Salesforce Org Version [ In case of pre release window ] Winter'22 (54.0)

Additional context

Any/all profiles with the same layouts assignments are removed.
If I put the layout assignment back into the profile, it deploys without any errors and the assignment is visible in the org.

Thanks for raising the issue @MarcDBehr. We will start looking into this.
Have you successfully used the reconcile command with a different version of powerkit before?

@Caitlyn-Mills This is a new record type and layout, so I can't say that it worked previously. There are other record type layout assignments in the same profiles and they are working fine from what I see.

I'm able to reproduce this issue on v4.2.5., Also, I compared the reconcile results between 4.0.3 and 4.2.5 for same Org and Profile, I found the below differences:

Please test if the same issue occurs on version 4.2.6

@Caitlyn-Mills Couple of questions:

The PR only appears to update the version number. Am I not seeing the other changes?
I don't remember how to test the patch release (it has been a bit since I last did that). Can you remind me how to do this?

Please test if the same issue occurs on version 4.2.6

I have tested this in 4.2.6 and still able to reproduce the issue.

@Caitlyn-Mills Couple of questions:

The PR only appears to update the version number. Am I not seeing the other changes?
I don't remember how to test the patch release (it has been a bit since I last did that). Can you remind me how to do this?

Hey @MarcDBehr, There were some changes merged without updating the patch version, so I was unclear if your issue was due to something that was in the code prior to this code being merged. To test the patch release you just have to update the version of sfpowerkit through the cli.
I am still doing some testing on this issue, it seems there must be some very specific conditions to be able to replicate.
Thanks for the patience. Hopefully we resolve this one shortly.

@sumanthreddybolledla  Can  you test the behaviour in 4.2.7

Hi @azlam-abdulsalam , I have tested this in 4.2.7, it is working fine now. I checked both retrieve and reconcile, both have the tags which were missing in previous version..


Can -Wx be overridden by a file-specific "#pragma warning"?

The pragma warning is supposed to be able to raise or lower warning/error level for specific codes.
https://docs.microsoft.com/en-us/cpp/preprocessor/warning?view=vs-2019
If I use -Wx to elevate all warnings to errors generally, can a specific warning be knocked back down to a warning in a single file using a "#pragma warning" command?

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

ID: abd121e9-da7d-891c-c881-773cb9756aec
Version Independent ID: cc4a6be7-8eaa-6f09-c32a-631ae4f551bf
Content: /WX (Treat Linker Warnings as Errors)
Content Source: docs/build/reference/wx-treat-linker-warnings-as-errors.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@SRM256
We try not to repeat ourselves too often, so the answer to your question is found in another article: Pragmas and compiler options. The short answer is that the most local directive takes precedence.


Possibly over-privileged service account in scheduled-asset-inventory-export-bq

On line 39 of the scheduled-asset-inventory-export-bq/main.tf
the robot service account is granted projectIamAdmin.
"roles/resourcemanager.projectIamAdmin" = ["serviceAccount:${module.project.service_accounts.robots.cloudasset}"]

Granting project-wide admin is generally discouraged. Is such a high privilege really needed?

@lcaggio can you take a look? if that's really needed, we might want to limit it via delegated role grants.

@lcaggio can you tal?


Registration Emails with OTP Are Marked as Spam

Describing the bug
Emails containing OTP for registration are being delivered to the spam folder instead of the inbox.
To Reproduce
Steps to reproduce the behavior:
Attempt to register a new account on the chat app.
Enter your email address and submit the registration form.
Check the spam folder of the email account used for registration.
Observe that the OTP email is present in the spam folder.
Expected behavior
The OTP email should be delivered to the inbox, not the spam folder.
Screenshots
[Add screenshots of the email in the spam folder if applicable.]
Desktop (please complete the following information):
OS: [e.g. Windows 10]
Browser: [e.g. Chrome 91]
Version: [e.g. 91.0.4472.124]
Smartphone (please complete the following information):
Device: [e.g. Samsung Galaxy S21]
OS: [e.g. Android 11]
Browser: [e.g. Chrome]
Version: [e.g. 91.0.4472.124]
Additional context
The email service provider used for sending OTP emails is Brevo.
The email domain used for sending OTP is<EMAIL_ADDRESS>

only some emails show in direct inbox

now is not show in spam folder


Incorrect relation parsing when specific columns with constraining

Assume there are User and Post models.
The relation is User hasMany Post.
When executing following codes, it will result in
Illuminate/Database/Eloquent/RelationNotFoundException with message 'Call to undefined relationship [posts:posts] on model [App/User].'
User::with(['posts:posts.id,user_id,name' => function ($query) {
  $query->where('name', 'apple');
}])
  ->first()

$user = User::first();

$user->loadMissing(['posts:posts.id,user_id,name' => function ($query) {
  $query->where('name', 'apple');
}])

What's your Laravel version?

Fairly sure you can just pass in posts:id,name instead of posts:posts.id,name.

Hi @36864,
This is an example to reproduce the bug and thanks for reminding me that I forgot to add user_id column.

@36864 There are cases where have to specify the table name, e.g. with BelongsToMany and
HasManyThrough relationships.

I'm not sure this is supposed to work.
Since you are already using a closure, you can just add $query->select('posts.id,user_id,name');.

Closing this issue because it's already solved, old or not relevant anymore. Feel free to reply if you're still experiencing this issue.


Clang format improvements

PR Checklist

[x] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
[ ] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
[x] You have signed the CLA.
[ ] All existing tests are passing: https://github.com/NativeScript/ios-runtime/blob/master/CONTRIBUTING.md#contribute-to-the-code-base
[ ] Tests for the changes are included.

What is the current behavior?
You have to manually apply patches created by clang-format
What is the new behavior?
Ask whether to automatically apply the patch and add it to the index, if the console invoking git commit is interactive

run ci


Write a F# program to implement selection sort

Description
Write a F# program to implement selection sort
Selection sort is a sorting algorithm that selects the smallest element from an unsorted list in each iteration and places that element at the beginning of the unsorted list.
Pseudocode
procedure selection sort
   list  : array of items
   n     : size of list

   for i = 1 to n - 1
   /* set current element as minimum*/
      min = i

      /* check the element to be minimum */

      for j = i+1 to n
         if list[j] < list[min] then
            min = j;
         end if
      end for

      /* swap the minimum element with the current element*/
      if indexMin != i  then
         swap list[min] and list[i]
      end if
   end for

end procedure

Example
Input  : [8, 5, 2, 6, 9, 3, 1, 4, 0, 7]
Output : [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/implement-selection-sort/ImplementSelectionSort.fs
Commit the changes
Create a pull request

!assign


Send notification to project supporters when a project update has been added

related to #809

Fix #817 as well

This PR and https://github.com/Giveth/notification-center/pull/32 should get merged at same time


Suggestions to get user friendly guidelines

Greetings,
I worked with a customer who was not able to scrap metrics with Prometheus following this web page. We realized that the documentation is lacking of some useful steps that might help to users with no previous experience using this feature.
Below are the suggestions you could analyze to be added to this webpage:
I suggest making the doc more prescriptive by adding some steps like the following:
1. Determine the endpoints exposing a Prometheus metrics by running “kubectl get services -A”. Pick the ones that have the term “metrics” in their name.
2. Exec into a pod running in the cluster that has curl installed and check that the endpoints determined in the previous step returns Prometheus metrics. Exece'd into the omsagent and run the following commands:
○ curl  argo-rollouts-metrics.argo-rollouts:PORT/metrics
○ curl  istio-operator-metrics.istio-gateway:PORT/metrics
3. Update the ConfigMap per the instructions in the Configure Container insights Prometheus Integration by updating the following line with the endpoints determined above:
○ kubernetes_services = [ http://argo-rollouts-metrics.argo-rollouts:8090/metrics,  http://istio-operator-metrics.istio-gateway:8383/metrics]
4. After applying the change make sure that the the omsagent logs return:
config::configmap container-azm-ms-agentconfig for settings mounted, parsing values
config::configmap container-azm-ms-agentconfig for agent settings not mounted, using defaults
config::configmap container-azm-ms-agentconfig for npm metrics found, parsing values
config::configmap container-azm-ms-agentconfig for settings mounted, parsing values for prometheus config map
config::configmap container-azm-ms-agentconfig for MDM metric settings mounted, parsing values
config::configmap container-azm-ms-agentconfig for metric collection settings mounted, parsing values
5. Run the following Kusto query against the cluster to make sure that the metrics from the two endpoints above are now available in Log Analytics:
InsightsMetrics
| where Namespace contains "prometheus"
| extend tags=parse_json(Tags)
| summarize count() by Name
Which should return the following metrics:
	controller_clientset_k8s_request_total              70
	controller_runtime_reconcile_time_seconds_bucket              44
	controller_runtime_reconcile_time_seconds_count              4
	controller_runtime_reconcile_time_seconds_sum              4
	controller_runtime_reconcile_total              4
	go_gc_duration_seconds             45
	go_gc_duration_seconds_count 9
	go_gc_duration_seconds_sum    9
	go_goroutines   9
	go_info 9
	go_memstats_alloc_bytes           9
	go_memstats_alloc_bytes_total 9
	go_memstats_buck_hash_sys_bytes              9
	go_memstats_frees_total            9
	go_memstats_gc_cpu_fraction   9
	go_memstats_gc_sys_bytes        9
	go_memstats_heap_alloc_bytes              9
	go_memstats_heap_idle_bytes  9
	go_memstats_heap_inuse_bytes              9
	go_memstats_heap_objects       9
	go_memstats_heap_released_bytes              9
	go_memstats_heap_sys_bytes   9
	go_memstats_last_gc_time_seconds              9
	go_memstats_lookups_total       9
	go_memstats_mallocs_total       9
	go_memstats_mcache_inuse_bytes              9
	go_memstats_mcache_sys_bytes              9
	go_memstats_mspan_inuse_bytes              9
	go_memstats_mspan_sys_bytes              9
	go_memstats_next_gc_bytes      9
	go_memstats_other_sys_bytes  9
	go_memstats_stack_inuse_bytes              9

Thank you in advance for your assistance and help reviewing this recommendation and I hope it will help others.
Regards,
Thaylor Carcachi

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

ID: 1eac8408-d575-12b6-e660-1b53787fd303
Version Independent ID: 52bce78a-3aef-29b9-77a7-c1c547c48914
Content: Configure Container insights Prometheus Integration - Azure Monitor
Content Source: articles/azure-monitor/containers/container-insights-prometheus-integration.md
Service: azure-monitor
Sub-service: containers
GitHub Login: @bwren
Microsoft Alias: bwren

Thanks for the feedback! We are currently investigating and will update you shortly.

@tcarcachi, We appreciate you taking time to share your valuable feedback.
The tutorial aims to provide key concepts and general instructions for configuring scraping of Prometheus metrics with Container insights.
The doc meant to show something more involved, but still allow users (/readers) to learn things beyond a specific configuration. It provides step-step process on Prometheus scraping settings (/for querying and reviewing metric data), highlights the perquisites, guides on how to and configure, deploy and apply ConfigMaps.
And, also as mentioned in the doc, ‘By integrating with Azure Monitor, a Prometheus server is not required. Just exposing the Prometheus metrics endpoint through exporters or pods (application), and the containerized agent for Container insights can scrape the metrics for you.’
Having mentioned above, I’ll have also shared your feedback with the author for further review.
@bwren, Requesting your comments on this feedback.

tcarcachi, Your feedback has been shared with the content owner for further review.

I'll take this up with engineering team and take their recommendation on adding these details. Saving them into a work item. Thanks!
#please-close


Names for DeferredOperations

We can currently construct calculated parameters out of existing parameters, by way of the DeferredOperations framework. But to make this useful in loops and plots, you need to separately provide name and label:
cond = (meter.volt - meter.voffset) * preamp.sens * 25812.8 / source.volt
cond.name = 'cond'
cond.label = 'Conductance (e^2/h)'

Loop(...).each(cond)

So the questions are:

Should we try and automatically generate a name, so you can use these in loops without error no matter what, or is it only allowed with an explicit name?
Do we want an inline syntax, such as Loop(...).each((meter.volt / source.volt).named('cond')) so people can create parameters completely inline, or is it OK to require these to be separate statements? I suppose if we implement #26 then these can be kwargs to Get.

@MerlinSmiles  I wonder, how much easier this is that really using a new parameter ?

@giulioungaretti I wonder the same.
just adds the amount of data in the dataset instead of having an on the fly calculation, as I wont want to nuke the raw data.
If its about me you can close this thing...

We have deprecated deferred operation


global vars, default values are positioned in a wrong sequence

The compiler is positioning the config file before the default values from the component repositories.
Error:
None, None, 'found undefined alias %r' % utf8(alias), event.start_mark
ruamel.yaml.composer.ComposerError: found undefined alias 'cream_default_var_ce_cpu_model'
in "./.temp/runtime.yaml", line 59, column 23
To Reproduce
Just run the compiler giving a regular input file. And you will get the error
Expected behavior
Should put this config above and gives the augmented file

#46


[WFLY-10943] Upgrade to Hibernate ORM 5.3.6.Final

https://issues.jboss.org/browse/WFLY-10943

Note that ORM is not pushed to JBoss Nexus but to Bintray and then synced to Central. I forced the sync to Central and it's supposed to be synced but I can't find it yet in the Central UI.
Hopefully, CI will be able to get it.

I get "Could not find artifact org.hibernate:hibernate-core:jar:5.3.6.Final in jboss-public-repository-group (http://repository.jboss.org/nexus/content/groups/public/)"


Log immediate rtt crash

Backend was not initialized to work in synchronous mode if LOG_IMMEDIATE was enabled. That causes use of rtt_lock which uses mutex. That lead to assert in application.
Fixes #14191.

Codecov Report

Merging #14453 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master   #14453   +/-   ##
=======================================
  Coverage   51.98%   51.98%
=======================================
  Files         308      308
  Lines       45505    45505
  Branches    10543    10543
=======================================
  Hits        23655    23655
  Misses      17050    17050
  Partials     4800     4800

Impacted Files
Coverage Δ

subsys/logging/log_output.c
47.58% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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