Hold/Unhold video issue

When we create video call via xmpp jingle (as server we use openfire) it works fine. But when one of participants set call on hold after unhold no video on the both ends.
Please help.

I tried to debug it and here is my findings:
seems there is video stream but camera isn't started.
Direct show error code is: 0x80004005

I mean camera doesn't start after unhold

I found the solution and fixed it by us (not all users).
Please check this issue for details: https://github.com/jitsi/libjitsi/issues/362

Should be fixed by updating libjitsi with commit from this pull request: https://github.com/jitsi/libjitsi/pull/429

@ibauersachs please review

Please review this pull request to libjitsi: https://github.com/jitsi/libjitsi/pull/429
Thre is the fix for this


Duqtools compability

Some suggested changes for better compatibility with duqtools:

[ ] Single source variables.yaml, currently duplicated in both code bases. I'm thinking duqtools use the imas2xarray one by default.
[ ] Combine common code in H5Handle/ImasHandle -> BaseHandle

The interface for HDF5 data is much simpler than IMAS/MDS+, so the amount of shared code is very little. For example, IDSMapping is redundant for this tool, and ImasHandle can be a lot shorter.


Support content negotiation for IDP requests

Merging this into feat/idp-architecture branch instead of main due to breaking changes in some IDP classes. Going to push future architecture changes there as well.
Now uses RequestParser on input data and ReponseWriter for all requests handled by our handlers.
The one issue here is that we still need to pass the original request object to the OIDC library in case it needs to handle something. Fortunately our default BodyParser doesn't actually read data from the input request and re-uses it in the Representation it creates, so the input request isn't actually modified. Should this ever change we either need to do the parsing later (but we still want to parse preferences early for error handling), or create a mock request with the correct data for the OIDC library.
The default response data is now JSON, on which we apply a converter to do content negotiation. For this I created a new converter that takes a JSON Representation and uses that JSON as input parameter for rendering a template. The metadata object gets used to pass the template file paths to the converter. This is a bit ugly perhaps but I think it is the cleanest way to handle this. Another unfortunate issue is that this is not a TypedRepresentationConverter since its supported output types depend on the input Representation. This means we can't use it in the ChainedConverter for now. A solution would be to update the TypedRepresentationConverter interface so its functions can take a Representation or other arguments as input.
The position of the converter in the config is also because of #834 .
The templates field for every IDP handler entry is now an object with the key being the content-type, making it easy to add other template types in the future if needed.
This is an example JSON output when registering:
{
    "webId": "http://localhost:3000/joachimvh/profile/card#me",
    "email"<EMAIL_ADDRESS>    "oidcIssuer": "http://localhost:3000/",
    "podBaseUrl": "http://localhost:3000/joachimvh/",
    "createWebId": true,
    "register": true,
    "createPod": true,
    "apiVersion": "0.1"
}

The plan in the future is to add an extra controls field (or other name) in every response object that links to all the relevant pages.
When doing a GET request you always get the following for now:
{
    "errorMessage": "",
    "prefilled": {},
    "apiVersion": "0.1"
}

Probably want to clean that up in the future. Not sure what to add instead besides the controls. A list of all supported fields and a description perhaps.
The integration tests currently still use the HTML responses, but now they could be completely rewritten to use the JSON API instead.
Still need to move the input parsing to the main handler so the IDP handlers don't have to do it themselves yet and we can support JSON input besides form data.
It's also getting to the point where it might make sense to split the main handler up in a few separate classes, perhaps one for the route handling, and another one for operation handling, but I think it might be best to first get to a state where we support the requests we want to support.

Should this ever change we either need to do the parsing later (but we still want to parse preferences early for error handling),

Can/should we guard against this? Setting the typings such that the request is read-only?

Should this ever change we either need to do the parsing later (but we still want to parse preferences early for error handling),

Can/should we guard against this? Setting the typings such that the request is read-only?

The request being read is the issue. What would break this solution is if the body parser reads out the stream. The problem is that the OIDC library also wants to read out the stream so then it would error since you can't read a stream twice. Technically we could create a complex class that prevents a stream from being read unless we want it to, but that would be overkill for this situation. It's not needed now, and if it is ever needed there are simpler solutions.

Can this be merged?

Can this be merged?

I assumed there would be some replies to my comments and was waiting on that before finishing up this PR 😅 . Unless the reply everywhere is "you're right, you do what you think is best!"

All good! I had approved the PR, so that's a sign that you can do what's best 🙂


update go to 1.18

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/23955

/unhold

/lgtm


Data is empty on every google account besides my own?

When I sign into my account (The developer) my get(api/me) returns the proper response. However, when i choose any other account besides mine. It returns no data. Any hints?
So, this happens when I locally host the Satellizer demo. I was wondering if this is a problem with my authenticator or my setup, etc.
Object {data: Object, status: 200, config: Object, statusText: "OK"}
data: Object{
__v: 0
_id: "568d342c1f838615b5cab9a8"
displayName: "Andrew Aquino"
google: "103727224997038097181"
picture: "https://lh5.googleusercontent.com/-88c-pgLtI_k/AAAAAAAAAAI/AAAAAAAAAMY/U4KxmfTh1Dw/photo.jpg?sz=200"}
VS.
Object {data: "", status: 200, config: Object, statusText: "OK"}
data: ""
headers: (name)
status: 200
statusText: "OK"

I did a further investigation with the satellizer demo, and the issue still persists. I am curious as to whether or not this problem is related to my google authentication setup or the fact that this is locally hosted.
Cheers,
Andrew

After further further investigation, the issue is with MongoDB, I currently have an issue in trying to solve it! Can anyone shed some light?
Cheers,
Andrew

@andjosaquino I have not experienced this issue. Does this occur only with Google, and have you tried other providers, also email and password users?
I suspect it's a MongoDB issue, as you pointed out.
It hasn't happened to me in years, but I vaguely remember back in 2013 something like that happening to me. Usually clearing the database and restarting computer helped.


chore(*): fix incorrect tslint comments

Description:
Fix incorrect tslint comments (tslint:disable:max-line-length --> tslint:enable:max-line-length), so that the max-line-length rule is properly enforced.  (Luckily, there was only 1 violation :smiley:)

Coverage increased (+0.0003%) to 97.698% when pulling 4becdd0b5be2822f035bc80be24412211fcb4221 on gkalpak:patch-5 into 31cf2bfa1707e090140cf0091de418720c260677 on ReactiveX:master.

@gkalpak can you better outline the issue that this fixes?

@blesh, I have updated the commit message and PR description.

@blesh looks like a goof on my part, this change looks good from that perspective.

Coverage increased (+0.0003%) to 97.702% when pulling 37b6e3939e46d92fd507d4868862338868e2f79a on gkalpak:patch-5 into d4533c4090a238cadffb4bc444709e7375c1f827 on ReactiveX:master.

Coverage increased (+0.0003%) to 97.702% when pulling 37b6e3939e46d92fd507d4868862338868e2f79a on gkalpak:patch-5 into d4533c4090a238cadffb4bc444709e7375c1f827 on ReactiveX:master.

Yeah, this is a whoopies!


[Bug]: JFrog Artifactory doesn't fetch<EMAIL_ADDRESS>from npm.

Version
29.4.2
Steps to reproduce

Use JFrog artifactory.
Try to install<EMAIL_ADDRESS>
Expected behavior
It should fetch from npm and cache into JFrog.
Actual behavior
Error message

Additional context
No response
Environment
node: 16.19.0
npm: 8.19.3

same here

Yes we are also facing the same issue. Need help on this ASAP.

I think you should talk with JFrog support.
Does not seem like there is an issue with Jest or Npm, because npm install<EMAIL_ADDRESS>works with no problem.

Yeah, not much we can do about artifactory's mirroring (or whatever they do). As long as it's correct on npm everything that can be done on our side is done.


Enable OpenPoints to be installed as a package

Hi,
I want to use OpenPoints as a library within my project. It would be nice if OpenPoints was able to be installed in its entirety as a package. This would entail:

Defining all non-cuda requirements in setup.py or setup.cfg
Installing all c++ extension modules from setup.py
Adding OpenPoints to pip (and in the future, Anaconda) to be installed by downstream libraries

Let me know your thoughts, thanks!

Thanks for this suggestion. But I have no experience in this. Could you kindly direct me to an example?

Hi!
I recommend taking a look at the following libraries for examples on setting up setup.py:
https://github.com/torch-points3d/torch-points-kernels/blob/master/setup.py
https://github.com/mit-han-lab/torchsparse/blob/master/setup.py
Let me know if you have any questions, I am happy to help. I am one of the core contributors on torch-points3d and I'm looking to use OpenPoints as a basis for the next version of the framework. Feel free to reach out!

I would maybe recommend following the format of torch-points-kernels, where the cuda c++ code is in a "cuda" folder and then you can setup all of the extensions under the global setup.py, like so:
# /setup.py
setup(
    name='openpoints',
    ext_modules=[
        CUDAExtension('openpoints.pointnet2_cuda', [
			'cuda/pointnet2/pointnet2_api.cpp',
			'cuda/pointnet2/ball_query.cpp',
			'cuda/pointnet2/ball_query_gpu.cu',
			'cuda/pointnet2/group_points.cpp',
			'cuda/pointnet2/group_points_gpu.cu',
			'cuda/pointnet2/interpolate.cpp',
			'cuda/pointnet2/interpolate_gpu.cu',
			'cuda/pointnet2/sampling.cpp',
			'cuda/pointnet2/sampling_gpu.cu',
		],
		CUDAExtension('openpoints.chamfer_cuda', [
              'cuda/chamfer/chamfer_cuda.cpp',
              'cuda/chamfer/chamfer.cu',
          ])
    ],
    cmdclass={'build_ext': BuildExtension}
)

And then, the convention is to have all of the python code under a folder same as the name of the package.
Project folder Structure:

+ cuda
+ openpoints
	+ dataset
	+ loss
	+ models
	+ optim
	+ scheduler
	+ transforms
	+ utils
- setup.py
- README.md

This way, the code can be imported into a downstream dependency after being installed by doing something like:
import openpoints.dataset


Misleading Error message in DialogContext#repromptDialog

Describe the bug
In the DialogContext's repromptDialog(), there is an error whose message says it has happened in DialogSet.reprompt(), which is misleading, as it has happened inside DialogContext#repromptDialog.

Additional context
I did not search for that kind of message, so it is possible that there are other cases like that.

This could be fixed in #2478, but I don't think it would make sense to keep that message like that until then...

@Veigaribo - Thanks for the report


Rename ETH_MACIIDR field TD to MD to align with RM0090

Fixes #69
Signed-off-by: Daniel Egger<EMAIL_ADDRESS>

LGTM, thanks!


feat: add calculation and application model fields as notes (hl-1550)

Description :sparkles:
Override apart default GET / list() endpoint and add rejected and calculation comments as notes.

Include manual calculation comment into notes (calculation.override_manual_benefit_amount_comment)
Include accepted/rejected comment into notes (log_entry with transition to rejected or accepted)
Send handler language preference to backend so that correct django translations are loaded

force Finnish for now

API branch is deployed to platta: https://helsinkilisa-pr3549.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3549.dev.hel.ninja :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3549.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr3549.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-handler-pr3549.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr3549.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3549.dev.hel.ninja :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3549.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr3549.dev.hel.ninja :laughing::tada::tada::tada:


Save a process, create index within the module

bcftools concat supports the --write-index flag, so an extra tabix process is superfluous.
https://github.com/CenterForMedicalGeneticsGhent/nf-cmgg-germline/blob/bddaca3e32e98535626053afe627bbb64852e36a/subworkflows/local/vcf_concat_bcftools/main.nf#L30-L35

This option creates .csi index files, which is not supported by all tools :cry:

NOOOOOooooooo....;


docs: scope link

I will update to match https://github.com/ublue-os/main/pull/547 a bit more closely.


Auto close intermittent failure without activities for X months

The list is crazy long:
2328 bugs found.
https://bugzilla.mozilla.org/buglist.cgi?keywords=intermittent-failure%2C &keywords_type=allwords&list_id=14511130&o1=casesubstring&o2=notregexp&o4=notsubstring&v2=\[(test|stockwell) disabled.*\]&v4=[no-nag]&f1=keywords&o3=greaterthan&v3=180&resolution=---&f4=status_whiteboard&query_format=advanced&f3=days_elapsed&f2=status_whiteboard&product=Core&product=DevTools&product=External Software Affecting Firefox&product=Firefox&product=Firefox Build System&product=Firefox for Android&product=Testing&product=Toolkit&product=WebExtensions
I am pretty sure that most of them are no longer relevant

Examples:
https://bugzilla.mozilla.org/show_bug.cgi?id=1467484
https://bugzilla.mozilla.org/show_bug.cgi?id=1467481
@calixteman wdyt?

The intermittent bot already does this, after 3 weeks of inactivity. @jmaher said they could be reduced to 2 weeks.

A lot of them haven't been touched for a while:
https://bugzilla.mozilla.org/show_bug.cgi?id=481369
https://bugzilla.mozilla.org/show_bug.cgi?id=1206551
https://bugzilla.mozilla.org/show_bug.cgi?id=1230810
do you know why?

I would like to consider 2 weeks- I don't know how many bugs make it to 2 weeks and then get a comment.  If that number is low (<10%), then I would vote for 2 weeks. Also @Archaeopteryx would be a good person to confirm with.

I personally think there is little value to a bug that sits for 1+ year
telling you a test is disabled with no activity.  You can always search the
history of bugzilla if you want more context, but typically the test needs
to be rewritten or updated and the old context from years ago isn't as
valid.
Once I had some needinfo's for 2+ years and cancelled them and the
developer got back to me immediately and said he planned to address those
when he had time.  We can propose a time window (6 months, 12 months,
18...) and see what would happen.
I also find it rare that once a test is disabled that it gets re-enabled- a
more effective route would be having a dashboard for development teams to
see what tests are related to their code and which of those are disable or
partially disabled (say on debug only, or android only), that would help
give an accurate picture, yet it would be another tool for them to use.
On Thu, Feb 7, 2019 at 5:20 AM Sylvestre Ledru<EMAIL_ADDRESS>wrote:

A lot of them haven't been touched for a while:
https://bugzilla.mozilla.org/show_bug.cgi?id=481369
https://bugzilla.mozilla.org/show_bug.cgi?id=1206551
https://bugzilla.mozilla.org/show_bug.cgi?id=1230810
do you know why?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/relman-auto-nag/issues/310#issuecomment-461362449,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD0fLdLiZvV6rYZSkL8zgN_EN-DjrpKks5vK_4FgaJpZM4Z-P8x
.

we had a discussion in the work week to do this- but we were going to switch single tracking bugs  to be meta, which then we don't want to add patches, needinfo, etc. on the bug; only when there is an actionable pattern would we create a new bug to do work in, and leave collecting stats in the meta bug.
so when do we close the meta bug, there is no specific mention, but I would like this issue to be used for closing an intermittent meta bug after X months of inactivity.
I propose 3 months/13 weeks/90 days (pick one)

@jmaher @Archaeopteryx OK for you to reduce this to 2 weeks?

2 weeks seems short.  In a near future (Q4) where we don't file/reopen/update bugs unless they have enough failure instances, would 2 weeks be enough?
In general I like the idea of closing in 2 weeks and as we change things for how we file/update bugs we could revisit the closing rules?

We can try 2 weeks instead of 3 and adjust later.

All right. Let's close this one since we're already closing intermittent bugs without activities, and I'll open a new issue about reducing the number of weeks from 3 to 2.


fix(near): CW-7273 refactor near protocol

Task linked: CW-7273 [SDK] Review Near Protocol pull request


In gRPC-C++ podspec, copy the certificate to bundle resources

In GRPC.podspec, the root certificates file from etc/roots.pem is copied to the bundle resources, so that it can be loaded at runtime for SSL to function properly. However, roots.pem is not preserved in gRPC-C++.podspec, so a project depending on that pod won't be able to establish SSL connections. Having the project maintain its own copy of the certificates file and keep it in sync with gRPC repo is undesirable.

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

libgrpc.so

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

  [ = ]       0        0  [ = ]

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

libgrpc++.so

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

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

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

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,021,771      Total (>)     11,021,759

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

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

libgrpc.so

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

  [ = ]       0        0  [ = ]

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

libgrpc++.so

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

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 2,015,303      Total (=)      2,015,303

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,115,426      Total (=)     11,115,426

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

Basic Tests MacOS [opt] - #16201

Basic Tests Multi-language Linux - #15308


feat(cmd): add Chart Version to helm list output

What this PR does / why we need it:
This PR implements the proposal in #8503 to add the "Chart Version" to the helm list output.
Closes #8503.

cc: @bacongobbler


[QUESTION] Adding named space and Conditional drop down in Laravel-Excel version 3.1

Prerequisites

[X] Have read and understood: https://github.com/Maatwebsite/Laravel-Excel/blob/3.1/.github/SUPPORT.md
[X] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[X] Checked that your question isn't already asked before.
[x] Filled in the entire issue template

Versions

PHP version: 7.2
Laravel version: 5.7
Package version: 3.1.14

Description
Hi, I have been trying to add named ranges into an excel sheet before exporting via download.
I am using AfterSheet event class to add named ranges before finally exporting the file
Below is my little snippet which I have put in registerEvents function
AfterSheet::class => function (AfterSheet $event) {
    $event->sheet->addNamedRange(
        new \PhpOffice\PhpSpreadsheet\NamedRange(
            'StandardList',
            $event->getSheet(),
            'A1:A12'
       )
   );
}

Additional Information
I need to have a conditional dropdown in my exported excel file
I am referring to snippets mentioned in below links
https://stackoverflow.com/a/30471766
https://stackoverflow.com/a/24745039
I was able to figure this out in version 2.1 but since version 3.1 is very different from version 2.1 I am not able to find a solution.
I have spent a lot of time to figure out how to add named ranges in an excel spreadsheet and use it to make a conditional dropdown. I have read your documentation as well.
Can you please help me to figure out how can I achieve this?

Hi, I am waiting to hear from your end. I have already spent a lot of time figuring this out. Can you please help me with this? It is very important for me to figure this out.

Hi, I have the same question in https://github.com/Maatwebsite/Laravel-Excel/issues/2229
Sample:
I used this in:
public function registerEvents(): array
{

    $event = $this->getEvent();

    return [
        AfterSheet::class => function(AfterSheet $afterEvent) use ($event){
            ...... // HERE
        }
    ];
}

// Option YES NO
$sheet->SetCellValue('BA3','YES');
$sheet->SetCellValue('BB3','NO');

// Set to 'BA3:BB3' the name "yes_no"
$sheet->getParent()->addNamedRange( new NamedRange('yes_no', $sheet->getDelegate(), 'BA3:BB3') );

// Create a dropdown list validation
$cellValidation = $sheet->getCell('C27')->getDataValidation();

$cellValidation->setType(DataValidation::TYPE_LIST);
$cellValidation->setErrorStyle(DataValidation::STYLE_INFORMATION);
$cellValidation->setAllowBlank(false);
$cellValidation->setShowInputMessage(true);
$cellValidation->setShowErrorMessage(true);
$cellValidation->setShowDropDown(true);
$cellValidation->setErrorTitle("Input error.");
$cellValidation->setError('Value is not in this list.');
$cellValidation->setPromptTitle('Chose from list');
$cellValidation->setPrompt('Please, chose a value for dropdown list.');

// Adding to a cell

$sheet->getCell('A10')->setDataValidation(clone $cellValidation);
$sheet->getCell('A10')->getDataValidation()->setFormula1('yes_no');

And thanks for the links to Stack, I come here for the same conditional example.

@kidjapa Can I see the entire code?

Hi, I have figured it out. I was able to add NamedRanges into my excel with below code:
AfterSheet::class => function (AfterSheet $event) {
	$event->sheet->getDelegate()->getParent()->addNamedRange(
		new \PhpOffice\PhpSpreadsheet\NamedRange(
			'StandardList',
			$event->sheet->getDelegate(),
			'A1:A12'
		)
	);
}


Reapply staking refactoring

Closes #879, #355, #1102
Description:

❤ Thank you!

Wait for gaia-8000

@faboweb does this addresses #1042 as well ?

I think yes. We refactored the staking endpoints so that there is not "fractions" anymore. It is just floating point numbers delivered from the LCD.

As discussed with Billy I reapply these changes now. They will not work with Gaia-7005!


Material Design color hue out of current palette for theming

Bug, feature request, or proposal:
I am actually not sure, probably a request, but might be a bug as well.
What is the expected behavior?
When creating a theme, you are expected to pass a default color, plus optionally a lighter and darker hue. You enter a palette name, and codes for default, lighter and darker colors. Those codes are supposed to mean a color hue inside of the named palette.
What is the current behavior?
If you want a lighter or darker hue to be part of a palette other than currently selected one, I am not sure but it does not seem that you can proceed.
What are the steps to reproduce?

Go to Material Color tool
Choose yellow (#ffee58) as either primary or secondary color
Proposed dark hue will be #c9bc1f
Mentioned darker hue is not in the Yellow palette in Material Color Palettes
When theming a Material Angular app, you can provde $mat-yellow as color palette, but you cannot define your darker hue as a number (100-900, A100-A700) based on your current color palette, because it is out of it.

What is the use-case or motivation for changing an existing behavior?
Defining a theme that is actually based on Material Design recommendations.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 4, Angular-material 2.

The material palettes are based on these, as you noted. I can't speak for the inconsistency that the color picker tool shows, but you can create your own palettes and use them:
$app-custom-yellow: (
  50: #fffde7
  100: #fff9c4
  ...
  contrast: (
    ...
  )
);

Good idea, thanks

As mentioned, our palletes are based on the link given above. Closing as they are currently in sync so this is as expected.


j: fix build

Motivation for this change
It looks like the build of J was broken when we switched to GCC 10. This makes the error a awrning.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 113316 at 637fcf25 run on aarch64-linux 1

1 package built:

j

3 suggestions:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/development/interpreters/j/default.nix:37:3:
   |
37 |   buildPhase = ''
   |   ^

warning: missing-phase-hooks
checkPhase should probably contain runHook preCheck and runHook postCheck.
Near pkgs/development/interpreters/j/default.nix:54:3:
   |
54 |   checkPhase = ''
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/interpreters/j/default.nix:67:3:
   |
67 |   installPhase = ''
   |   ^


Create a ValidRange class, extending Range

Right now if I do this:
var range = new Range<int>(5, 2);

That works, the object just sets Start to 2, and End to 5. There are cases where we want a validated range - that is, start should be less than or equal to end, and throw an exception otherwise.
Therefore:
◾Unseal Range
◾Create ValidRange that inherits from Range and adds that validation in the constructor.

I'm closing this. Don't see a reason to add it.


⚠ GitLab Server has degraded performance

In 7cd1b16, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9287 ms

Resolved: GitLab Server performance has improved in 25ff7ce.


[5.2] Fixed isset bug for unloaded relationships

isset does not work on relationships that have no been loaded
This commit fixes that

Please add a test case to prove the bug. Integration would be preferred, but both unit and integration if you have time.

Will do
https://www.catlab.eu/email/www.catlab.be
Thijs Van der Schaeghe
Alpha Cat
+32 473 72 68 79 <+32473726879> -<EMAIL_ADDRESS>Catlab Interactive - Pannestraat 75 - 9000 Gent, België - www.catlab.be -
facebook https://www.facebook.com/catlab.interactive?fref=ts - twitter
https://twitter.com/CatLabGames
On 28 April 2016 at 13:58, Graham Campbell<EMAIL_ADDRESS>wrote:

Please add a test case to prove the bug. Integration would be preferred,
but both unit and integration if you have time.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/13354#issuecomment-215401652

@GrahamCampbell could you let me know what else is needed?

@daedeloth good work mate. I think only thing @GrahamCampbell might ask you again is to rebase your commits into a single commit.

might ask you again is to rebase your commits into a single commit.

Nah, Taylor can just squash on merge.

Please don't rename the test file.

It now missmatches the classname.

Woops not sure how that happened; fixed

I question whether this is a bug. Regardless, it could be breaking behavior if people are using isset() to check if relationships have been loaded, and not whether they merely exist. Open to other thoughts but going to close for now since at the least it could break existing applications.

Well that was an anti climax...
Would people actually be interested to check if relationships have been loaded?
Without this isset I don't have any way to check if the relationship exists.
If you don't want to merge it in 5.2, at least merge it in master...

@taylorotwell can we get a bit of a conversation going here? Problem I'm currently having is that I cannot see any proper way around this.


document how to exit the Pachyderm shell

Provide a link to the page with an issue:
https://docs.pachyderm.com/latest/deploy-manage/manage/pachctl_shell/
Describe what is wrong:
The documentation doesn't mention how to exit the shell.
Describe the suggested changes:
Control-D will exit it.
A future release will feature an "exit" command
Anything else we need to know?
Environment:

Kubernetes version (use kubectl version):
Pachyderm CLI and pachd server version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS:
Other:

Note that starting in 1.11 the shell will say "Type 'exit' or press Ctrl-D to exit." when you start it up, so you can probably just replicate that message.


rfc(dm): add rfc doc of enhancing task manageability

What problem does this PR solve?

Issue Number: ref #4484
What is changed and how it works?
as title
Check List
Tests

No code

Code changes
Side effects
Related changes
Release note
`None`.

Codecov Report

Merging #4458 (c672295) into master (e469fb3) will decrease coverage by 0.4707%.
The diff coverage is 41.8848%.

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

Flag
Coverage Δ

cdc
59.1878% <43.0379%> (-0.9996%)
:arrow_down:

dm
52.0828% <41.0714%> (-0.0208%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4458        +/-   ##
================================================
- Coverage   55.8077%   55.3369%   -0.4708%
================================================
  Files           495        495
  Lines         61246      61299        +53
================================================
- Hits          34180      33921       -259
- Misses        23614      23974       +360
+ Partials       3452       3404        -48

could you please take another look for this pr ? thanks 🧡🧡🧡
/cc @D3Hunter

/run-all-tests

/run-dm-integration-tests

maybe we can put the rfc(or detailed description) in an issue, not in codebase. it's not something our customers'll read, in that case write docs or blogs and in fact you must write it anyway, and dm is not a huge and complex project like JVM or TiDB which may such comprehensive thing to track features.
but for dm, i think an issue with detailed description for tracking purpose is enough.

Great job and LGTM!

/merge


can't boot the game

when I try to boot the game with play button, it shows alexanders window and game window few second
then both of them disappears.
After uninstalling alexander game worked well.
Tried turning off window defender, re-installing game.
Don't know how to fix it.

Try this one at here: https://github.com/Soreepeong/XivAlexander/issues/161#issuecomment-933263637
Also, as this probably is the same issue with the other one (#161), closing as duplicate for now. Please add to the other issue if you have anything to attach.


Display error in bash on windows

When using WSL the prompt and status bar aren't displayed correctly. Rather than modifying the color of the relevant characters, the color code strings are displayed:

Note: the last line is the prompt, after loading the plugin.
Any ideas about what's causing this?
The same issue appears when I use the WSL standard terminal, as well as others like HyperTerm

no idea, maybe asking on xonsh gitter might help https://gitter.im/xonsh/xonsh, it might be related to readline/prompt-toolkit differences maybe.

@rmreq can you give me the value of your $PROMPT variable before loading powerline?

Thanks, but it appears to be working now. I was using the Hyper terminal emulator, which seems to have a ton of display errors that mysteriously come and go. I suspect that was the cause.


Unable to login to harbor registry

Describe the bug: (a clear and concise description of what the bug is)
It happens quite often that, I am not able to login to harbor registry by using User name and CLI secret obtained from harbor UI.
Steps to reproduce the behavior:

Go to harbor service in otomi
Login with OIDC
Go to User Profile (right top corner)
Copy < User_Name> and <Client secret>
Wait 60 seconds (until JWT expires)
Open you terminal
Try to login to registry and observe error

$ docker login harbor.<cluster-domain> -u < User_Name > -p <CLI secret>
Error response from daemon: Get https://harbor.demo.eks.otomi.cloud/v2/: unauthorized: authentication required

This issue has been reported by me at harbor repository: goharbor/harbor#14172
Workaround: a user needs to logout from otomi and login once again and then obtain CLI secret from harbor
Expected behavior: (a clear and concise description of what you expected to happen)
A can always use CLI secret for obtaining access to image registry

The otomi.io docs should describe this limitation, and a Task added to expose a refresh token in Keycloak. Then this issue can be closed.

Docs updated and Task created: https://github.com/redkubes/otomi-tasks/issues/22
closing this


service names -> Title Case

Pull Request Type

[ ] Feature
[x] Bug Fix

Summary of Bug Fix(es)
Previous Behaviour
Description of the bug and it's impact
Service names were shown as non-separated strings
New Behaviour
Description of the bug fix and it's impact
Now we show as Title Case with white space between words.

Still checking for other uses...


Update jquery.floatThead and adapt to breaking changes

See https://github.com/mkoryak/floatThead#130

:mushroom:


Add a FormType constructor.

Added another constructor to allow create a Form passing FormType as parameter.

Thanks again!


D3 Add missing optional index parameter

@gcastre sorry, I can't find any document or code for your code review.
https://github.com/mbostock/d3/blob/3abb00113662463e5c19eb87cd33f6d0ddc23bc0/src/svg/axis.js#L111
do you know any documents?

@vvakame Sorry for the undocumented PR.
The line you have linked is the default function for formatting axis ticks which is used when you don't specify one.
cf: https://github.com/mbostock/d3/blob/3abb00113662463e5c19eb87cd33f6d0ddc23bc0/src/svg/axis.js#L29
Now as you can see here: https://github.com/mbostock/d3/blob/3abb00113662463e5c19eb87cd33f6d0ddc23bc0/src/svg/axis.js#L47 the tickFormat is given to the d3 text function, which lead us to https://github.com/mbostock/d3/wiki/Selections#text

if value is a function, then the function is evaluated for each selected element (in order), being passed the current datum d and the current index i

That's why you can have the index on the tickFormat function.

@gcastre okey, thanks mate!


(BUG) assigning vip_tweak does not work

related https://github.com/ecommerceberlin/exhibitors/issues/110


Flag takes a lot of time to glow on hover

Describe the bug
Flag on the upper left corner takes a lot of time to glow on hover. It should be fixed.
Expected behavior
The glow up of the flag should be immediate as soon as a hover is there.
Environment (please complete the following information):

OS: Windows
Browser: Chrome

@QAZIMAAZARSHAD Please let me change this. Assign me this under LGMSOC'21

Can I work on this issue under LGMSOC'21

Can I work on this issue under LGMSOC'21

Look for some other issue, this is already assigned.

Goodevening sir, i have fixed this issue. Can i upload it or should i work on some other issue? If i can upload then can you guide me?

Goodevening sir, i have fixed this issue. Can i upload it or should i work on some other issue? If i can upload then can you guide me?

If you have solved the issue, make pull request.
First, update your repo, fetch upstream then make changes then make PR.

@QAZIMAAZARSHAD But this issue was assigned to me.

@QAZIMAAZARSHAD But this issue was assigned to me.

Yes, and it is still assigned to you, I have not merged any PR for this issue.

@QAZIMAAZARSHAD I opened a PR #177 solving this issue


Popup Style not working

I have added the following section to Styles.xaml

However my popup doesn't use these values for the Color, it comes up as an off white color: Red=0.827451, Green=0.827451, Blue=0.827451, Alpha=1
Is this a bug or am I doing something incorrect to set the consistent style for popups?

That is a very good question! @pictos do you have any thoughts on this?

I need to double check that, but from the top of my head, Popup is an Element and not VisualElement, so it doesn't have support for Style, but need to double check that. @PureWeen can you confirm this?

That is correct, Popup is an Element, not a VisualElement, but IMO that seems like an oversight.  Some "Controls" adhere to Styles but some don't?  I'm sure there are technical considerations but these are the types of inconsistencies that made Xamarin such a nightmare, hopefully MAUI will have better consistencies, things like Style working 90% of the time doesn't seem like the correct solution.

You can try to set the style inside the Layout on the Popup... Something like:
<Popup>
    <Grid>
        <Grid.Resources>
                <Style TargetType="Label">
                        <Setter Property="FontSize" Value="20"/>
                </Style>
        </Grid.Resources>
        <Label Text="Bla" />
    </Grid>
</Popup>

About your thought, I would say to open a discussion on .NET MAUI repo, since this change should happen there, and it's a valid request.

Thanks @pictos I'll open a discussion there.

One more question I anticipate will be the response from the MAUI repo, is why is Popup based on an Element and not a VisualElement if it should have Styling capabilities?

If I understood your first comment, you want to be able to apply Styles on all Maui elements, right? So that is related to more stuff than a popup.
To give you the full context around the popup.
Today popup is based as an Element because it can't be a ViewHandler the reason is to be a ViewHandler the control should inherit from what represents a View on each platform, (UIView on apple, View on Android, etc) the popup doesn't have that so it should be an ElementHandler and because of that an Element on the Maui world

Moved to this MAUI discussion


fix(ast): missing visit JSXElementName enum

Current dependencies on/for this PR:

#2546
#2547  👈
main

This stack of pull requests is managed by Graphite.

Merge activity

Mar 1, 8:03 AM EST: @Boshen started a stack merge that includes this pull request via Graphite.


Python 3.9 Support

[x] Add action tests
[x] Add badge.

We will wait till all dependencies support Python 3.9


Nest all transform-specific arguments in an explicit 'config' parameter.

Previously these were implicitly lifted from the base dictionary.
We may want to consider allowing that again, but it is easier to loosen restrictions than tighten them up once this is declared stable.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @damccorm


⚠ DDoS-Relay EU-C-fra1-02 has degraded performance

In debbd67, DDoS-Relay EU-C-fra1-02 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 177 ms

Resolved: DDoS-Relay EU-C-fra1-02 performance has improved in 3ee4637 after 21 minutes.


java.lang.NullPointerException

Hi,
I recently try GO-plotter for drawing spectra, and it crashed into this error when loading data. Could you please fix this bug?
Many thanks.

Thanks for your feedback!
I checked the source code and found that the exception was caused by missing selection ion information in the precursor section of mzML file. I have updated the software and removed mandatory requirement of precursor information. Please find the updated GP-Plotter software in the release page, and please contact us if there is any questions.

Hi,
Thanks! I got a new error called java.lang.IllegalArgumentException: Failed to Find Scan Number in mzML Spectrum Id: index=2894. Does it mean that the mzmL file should not contain spectrum id not in the identification list.

Also, I  tried  only one spectrum with two identification results to plot mirro images. The loading seems to be successful but no content is displayed. The file I used is attached below.
plotter.zip

I think I get what the problem is: the mzML was converted from mgf, right? It is not recommended, because many spectrum information was lost in mgf, and caused the above first and second errors in mzML reading (please see picture below, scan number 12517 only exists in mgf spectrum title). The scan number was missing in mgf, and spectrum title was also re-assigned in mgf to mzML conversion. Information is more comprehensive when mzML is converted from Raw file.
For your convenience, I updated GP-Plotter again in the release page to support mgf file reading, even though mzML is more standard. In the updated software, the scan number is obtained from title part "file name.scan.scan.charge", like "MouseLiver-Z-T-3.12517.12517.4" in plotter_mgf_test.mgf.
When reading the plotter_mgf_test file, the result was empty. The reason is quite simple: the spectrum file name is "MouseLiver-Z-T-3" in pGlyco txt, but the name of mgf file is "plotter_mgf_test", thus the software can not find where the spectrum is. The spectrum file name is based on the information in result file, because some search engines support multi-file searching. To fix this bug, you can simply change the name "plotter_mgf_test.mgf" to "MouseLiver-Z-T-3.mgf".
I hope the above answers can help you.

Hi,
Thank you! I think it's almost done but I got one weird issue that I can't save my image. I have the "save as" page by clicking "save" but there is no picture in the saving folder.

Does software path or image save path contains special character? Please avoid using Chinese character in the path of software and image, because Chinese and English characters are different encoding ways.

There is no Chinese character or other special character there.

Is python already installed? You can press "window⊞" key, enter "cmd" and press enter key. In cmd page, enter "pip list" and press enter key. If python environment is ok, you will see information like below picture:

I have got all these packages. Some packages have different version, like setuptools 63.4.1 but most are similar. I have much more packages than these.

When you run GP-Plotter, is there any stop or error at "Python Package Validatation" page ?

it is like this

And this page can vanish automatically like it goes successful.

The "Python Package Validation" page can close automatically, indicates that python environment and necessary packages exist on your computer. I re-tested GP-Plotter on my computer, my laptop and two lab computers, and it worked well......So it is really wired.
Below zip file contains spectrum data of "MouseLiver-12517" in json. The source python code for image generation is in the configuration folder of GP-Plotter ("configuration\gp-plotter\1.0.0\python\py"). You can change the json path in "plot_mirror.py" and run the code. If there is any error in image generation, you can see detailed information in cmd page.
spectrum json file.zip

I see the problem. It got an error " File "E:\tools\GP-plotter\configuration\gp-plotter\1.0.0\python\py\plot_mirror.py", line 6, in
from py import mirror_spectrum
ImportError: cannot import name 'mirror_spectrum' from 'py' "
The Python interpreter searches for modules in a specific order based on the search path. The order of paths in sys.path determines the search order.  I got another py in the sys.path and it result into error.
I change the "py" name into "py_plotter" and fix the bug. Maybe you can change the py into something else. Otherwise, it might contradict anaconda even there is the working path set.
Again, many thanks for your help!

Thanks for your suggestion! The py folder name will be changed soon. And updates will be released later in today. Thanks a lot!

I found that the netural loss (H2O, NH3) peaks for B/Y ions and peptide precursor ions are not matched.
For example, for the glycopeptide [CVVHYEJSTVPEKK 1.Car. (N(N(H(H(N(H(G))))(H(N(H(G))))))) ], the HexNeuGc(-H2O) has a mz of 452.14 but is not matched. The CVVHYEJSTVPEKK (2+) (845.42 m/z) is not matched too.
All the ions not matched are:
452.1395569 2618.0759277344
655.2207031 1388.4241943359
836.9050903 2818.5266113281
845.4172974 9855.546875
937.9440308 3029.2897949219
938.4454346 4688.0502929688
1039.984619 1399.3388671875
1672.780518 5002.8823242188
1689.816406 9887.896484375
1874.924805 1187.0699462891
1875.853394 3091.0866699219
The default setting is :

Could you please fix the problem or direct me to the source code python file that I can self-define the ion type?

The software was coded in Java language with dependency of multiple self-developed packages, python is only used for image generation, thus it is hard to share the source code to you.
NH3/H2O neutral loss ions are only enumerated and matched for peptide ions and not for B/Y ions, The reasons are below:

The common oxonium ions contain those with neutral loss other than NH3/H2O, for example, HexNAc_C2H4O2 (m/z=144.064). There is an oxonium ion list for GP-Plotter and you can find the list in configuration folder (configuration\oxoniumIon.csv) and self-define the list to include other oxonium ions like HexNeuGc(-H2O).
Neutral loss ions are not considered for Y ions, there are two reasons: 1) m/z of fucose-containing ions is close to the isotopic peak of H2O/NH3 loss peak, for example, [Hex(2)-NH3 +1 Da] = Hex(1)Fuc(1). And it is necessary to avoid incorrect ion annotation; 2) matching ammonium loss peak might be inappropriate for specific type glycans, like oligo-mannose glycans. Thus only neutral loss peaks of N-glycan core structure are matched in the software.
The ion matching issue for precursor neutral loss and Y0 ion [CVVHYEJSTVPEKK (2+) (845.42 m/z)] have been fixed in updated version.
I hope the above answers can help you.

Sorry that new bug comes for this update.

Regarding the Y ions, 1 dalton often exceeds 20 ppm. Furthermore, the Y ion carries a peptide segment and the first monosaccharide HexNac (with N,H,O). Theoretically, deamination and dehydration should exist (although not necessarily connected with the loss peak of the fractured monosaccharide). From experimental observations, it appears that there are still many such ions present.
I understand that this may increase the possibility of mismatches. If you don't want to alter the existing results, could you provide me with a separate version that includes the loss peak of the Y ion at your convenience? You can contact me privately at<EMAIL_ADDRESS>Thank you.

Sorry for the late reply.
The error was caused by a missing property for precursor neutral loss peak, and it has be fixed.

+1 Da is not the matter of mass tolerance. For Y ion, it contains an intact peptide and a fragmented glycan part, thus poses large mass value. Therefore, the mono-isotope Y peak is usually not the most abundant one in isotopic peak cluster (as the example in below picture) and even is low intensity in MS2 spectrum. +1/+2 Da isotopic peaks are also considered and matched for glycopeptide identification and ion annotation.

The nitrogen atom in HexNAc is at the linkage position and is not likely to be lost during fragmentation. In addition, for amino acid, only RKNQ can loss ammonium theoretically because of -NH2 group in their side chain (this is the reference). To avoid mis-matching of ions, I didn't consider Y neutral loss peaks in the software.

Anyway, I also packaged a version of GP-Plotter to enumerate and match Y ions with neutral losses. The software has been sent to your personal email and please check it.


[Sema] Requestify inheritsSuperclassInitializers

This PR splits off the computation of whether a class inherits its superclass' designated and convenience inits into a request. This leaves addImplicitInheritedConstructorsToClass just with the job of synthesizing designated init overrides.

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 1b70794e959d349d1bd1f255cd3fc191c4fde3dc

@swift-ci please test OS X platform

@swift-ci please test source compatibility

@swift-ci please test source compatibility release

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 1b70794e959d349d1bd1f255cd3fc191c4fde3dc

Build failed
Swift Test Linux Platform
Git Sha - 1b70794e959d349d1bd1f255cd3fc191c4fde3dc

@CodaFi If it's good with you, I'll go ahead and merge this now and look into removing the isRecursiveValidation() check in a follow-up.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@hamishknight Merge away!  But please squash this so the merge commit doesn't show up in the history.

@swift-ci please smoke test Linux platform

Looks like you picked up the SourceKit failure
@swift-ci please smoke test Linux platform

@swift-ci please smoke test Linux platform


Create doublylinkedlist.py

PR accepted @Mystery-2-Dev
Successfully merged to HacktoberFest_22 repo.
🚀🚀👍


ngAnimate (v1.2.0rc1) : ngRepeat shows stale items

When updating an array that is used for an ngRepeat, the list will momentarily show what appears to be the previous items together with the new items before updating with the new items. This only happens when ngAnimate is a dependency for the app.
Please see http://plnkr.co/edit/fqrI0w1j0HqtV3CTzsFI?p=preview
When you click on the different lists in the above example, you will see how it momentarily shows a combination of the previous and current lists before showing the current one.
Watch carefully, it happens very fast!
This buggy behavior will not occur when you remove ngAnimate.
This bug is serious because it makes ngAnimate virtually unusable for apps that use ngRepeat due to the flashing.

I, it seems that I'm still facing this issue with latest angular version. Whenever I import ng-animate in my module I got this problem. Any idea ?

Same here,
example: http://dev.curveradio.com/#/home in the "you just heard" div

Yes, this is serious. I'm also facing this issue with Angular 1.3.5. Ended up removing 'ngAnimate' module from the app.

@bimal1331 Just add a css class like .repeat-modify to your ng-repeat elements, that you are experiencing the issue with. Then, in your app config block, you can add:
// Let ngAnimate know which elements to not handle
$animateProvider.classNameFilter(/^((?!(repeat-modify)).)*$/)

Then you can continue using ngAnimate in rest of your app.

Thanks @kbdaitch for the suggestion.

having the same issue with angular 1.3.15 and bootstrap .form-control class on the repeated items.
@kbdaitch solution seems to solve this, but i'm not sure why animation works by default for repeater :(?
is anyone looking into this?

I have the same issue with Angular 1.3.16 on Foundation for Apps framework. @kbdaitch's solution works for me, and the elements are not being combined.

@kbdaitch's solution definitely works. Thanks!

Any test with angular 1.4 without having to modify something ?

Seeing this in 1.4.1. The repeated elements have no transitions or keyframe animations.
Will try and get a Plunker example going.

Plunker example ~ http://plnkr.co/edit/uReg3o5JQ03zrVNN9t8P?p=preview

Had the same problem with 1.4.0 and @philBrown suggestion fixed it.

The problem is that .thumbnail (in bootstrap) sets a transition value on it for hover effects. When ngAnimate comes across thumbnail for animation-purposes it thinks that it will trigger itself for the animation.
In addition to the other methods of side-stepping this, another solution is to simply do:
.thumbnail.ng-animate { transition: none; }

You can also place the thumbnail as its own element within the element containing the ng-repeat directive.
Unfortunately there's nothing we can do to detect this due to the nature of specificity. Angular 2.0 will have a better approach to this, but Angular 1.x can't be changed.

I will be locking this issue since anything involving CSS detection may result in a misinterpretation of detected transition and/or keyframe styles.
So if you have a CSS style like:
div { transition:0.5s linear all; }
div:hover { outline:5px solid red; }

Then anytime a div is animated via ngAnimate it will assume that a transition of 0.5s linear all applies to the enter/leave/move or class-based animation. This in turn results in a waiting delay during which the elements may appear on screen when they are not supposed to be.
The unfortunate thing here is that some frameworks like boostrap and gumby spinkle transition values onto elements in a global way.
The solutions to fix this are to:

Place transitions yourself onto elements that you know you want to animate through ngAnimate.
Move elements that already contain a default transition or keyframe to be a child of an element that is managed by Angular
Use $animateProvider.classNameFilter() to set a regex that only allows specific elements with specific classes to be animated.
Use .some-class.ng-animate { transition:none; keyframe:0s none; } to cut out animations on any elements/selectors that contain a global transition and/or keyframe.

If you do happen to have an issue that shows up despite using any of these solutions then please open a new issue.


runtime error: index out of range when makeMove()

Environment :
go version go1.16.2 linux/amd64
In wsl2 Ubuntu20.04
In ##step03##, when I move the 'P' to this position, the program panics an error like the title.

I print the logs in function makeMove()
       rest code
       ----
	case "LEFT":
		newCol = newCol - 1
		if newCol < 0 {
			newCol = len(maze[0]) - 1
		}
	}
	log.Println("len of maze: ", len(maze))
	log.Println("len of maze[0]: ", len(maze[0]))
	log.Println("newRow: %d\n", newRow)
	log.Println("newCol: %d\n", newCol)
	if maze[newRow][newCol] == '#' {          //This is line 124
		newRow = oldRow
		newCol = oldCol
	}
    ----
    rest code

then It prints the message  below
2021/11/09 17:48:25 len of maze:  24
2021/11/09 17:48:25 len of maze[0]:  28
2021/11/09 17:48:25 newRow: %d
 11
2021/11/09 17:48:25 newCol: %d
 27
panic: runtime error: index out of range [27] with length 22

goroutine 1 [running]:
main.makeMove(0xb, 0x0, 0x4e3d37, 0x4, 0xb, 0x0)
        /mnt/e/github/pacgodemo/main.go:124 +0x37c
main.movePlayer(...)
        /mnt/e/github/pacgodemo/main.go:135
main.main()
        /mnt/e/github/pacgodemo/main.go:183 +0x145
exit status 2

I don't know why the length is 22.
Anyone else can help me? Thanks

Can you please check that the len of line 11 is the same of line zero? The problem you are facing is that probably your maze doesn't have the space characters filling the empty space on the other size of the maze. The way the code was written it relies on the TXT file for the maze to be uniform.
In summary, after the last # on line 11 make sure the rest of the line is filled with spaces.

@danicat
Thanks for your

@danicat
Thanks for your detailed explanation. Forgive my naive issue.
I have resolved by unable the trimTrailingWhitespace function of vs code.


Printing image in --preview option of PDF images

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
I'm trying to get an image preview of PDF files while looking for them. This approach is highly inspired in what is done in this example, but it prints an image of the first page instead of showing texts.
My humble try to do so is
function fpp()
{
  # open PDF using fuzzy finder with preview
  rg --type 'pdf' --files | fast-p | fzf  --read0 --reverse -e -d $'\t'  \
      --preview-window down:80% --preview '
        pdftoppm -f 1 -l 1 -singlefile -jpeg -tiffcompression jpeg -- {1} | viu -
        '
}

where I am using the viu program to print the image and pdftoppm to corvert the cover of the pdf file to jpg. The result is

If a replace viu by viu -b, I get

Which is okay but I looking for a non-pixel image. Is it possible? If it is, how can I do that?
PS: viu program prints seamlessly on terminal, which is kitty.

No, it is not possible. fzf doesn't just print the output of the program, but it extracts ANSI escape sequences for colors, and trims or wraps the long lines so that the output fits on the preview window of fzf, whose dimension and position are unknown to the terminal emulator.


Benjamin Bulb in java

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes I have.
Description
(Write your answer here.)
Checklist

[X] I've read the contribution guidelines.
[X] I've checked the issue list before deciding what to submit.
[X] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
For issue #1668 Completed the code in java.

@HarshCasper Kindly review this PR and add DWOC label for it to be considered as a contribution.
Thanks!

@HarshCasper Kindly review this PR and add DWOC label for it to be considered as a contribution.
Thanks!

Are you still working on this ? @shivangi00

Are you still working on this ? @shivangi00

Are you still working on this ? @shivangi00

I've made the needed changes. If these are satisfactory, then this PR will be completed.

Are you still working on this ? @shivangi00

I've made the needed changes. If these are satisfactory, then this PR will be completed.

Hello @shivangi00
I merged the PR but still found some pressing issues that were not addressed. Like the README changes were not done. Can you please again make a PR with the same code and apt README changes?
Please link this PR to your PR Descrption.

Hello @shivangi00
I merged the PR but still found some pressing issues that were not addressed. Like the README changes were not done. Can you please again make a PR with the same code and apt README changes?
Please link this PR to your PR Descrption.

Hello @shivangi00
I merged the PR but still found some pressing issues that were not addressed. Like the README changes were not done. Can you please again make a PR with the same code and apt README changes?
Please link this PR to your PR Descrption.

Okay I will make a new PR with the required changes.

Hello @shivangi00
I merged the PR but still found some pressing issues that were not addressed. Like the README changes were not done. Can you please again make a PR with the same code and apt README changes?
Please link this PR to your PR Descrption.

Okay I will make a new PR with the required changes.


test: add  workflow dispatch options and fix scheduled workflow

Checklist

[ ] Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md, if applicable.
[ ] The values-schema.yaml file and test/** fixtures have been updated to reflect code changes, if applicable.
[ ] The OpenApi Schema from redkubes/otomi-api project is compatible with definitions from values-schema.yaml file, if applicable.
[ ] Helm releases are meeting otomi's baseline security policies, if applicable.
[ ] Helm chart and helmfile changes are tested against upgrade scenario, if applicable.

@j-zimnowoda do you have any idea if all this duplication in the file is needed?

Yes, it is needed because it defines two different interfaces


Update authentication.md

Type

[X] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
None
What is in this Pull Request ?
Spelling update --AllowPrelease to --Allow Prerelease

Thank you!


KAFKA-17672 Run quarantined tests separately

Introduce new quarantinedTest that excludes tests tagged with "quarantined". Also introduce two new build parameters "maxQuarantineTestRetries" and "maxQuarantineTestRetryFailures".

@chia7712 looks like #17336 fixed the issue

@chia7712 can you take a look?

I realized we should not have quarantinedTest task explicitly fail the build like we do with test. This is because we are explicitly telling Gradle not to cache this task
    outputs.upToDateWhen { false }
    outputs.cacheIf { false }

This means we can also stop forcing ignoreFailures to true in the CI build. If we have more than 20 flaky failures in the quarantinedTest task, it will fail the overall Gradle run.

I assumed the quarantined tests should NOT impact the CI flow for PR

Since we're defining this new task separately, we have some flexibility. We can have it fail after some high threshold (like 20 that I arbitrarily chose), or we can prevent it from ever failing the build.
In principle, I guess it's best to not fail the build and rely on the quarantined test reports to ensure we don't accumulate too many quarantined tests or have them for too long.


CI: put test junits in a separate dir

Description
Per title, to remove the 'crowding' from the top level artifacts dir.
Checklist

[x] Investigated and inspected CI test results

Testing Performed
CI is sufficient

Does this handle the non-bash junit file? That might not be in the scope of these changes but wanted to make sure

Does this handle the non-bash junit file? That might not be in the scope of these changes but wanted to make sure

The non bash ones that I've noticed are all in there own location.


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed

JavaScript heap out of memory.
Using test report parser dotnet-rtx

I faced the the same issue:
Processing test results for check run Test Results

  <--- Last few GCs --->

  [579:0x60ffc10]    37812 ms: Scavenge 2038.1 (2071.0) -> 2037.8 (2071.7) MB, 7.5 / 0.0 ms  (average mu = 0.[24](https://github.com/KyberNetwork/integration-test/actions/runs/6180778211/job/16777756267#step:8:26)6, current mu = 0.214) allocation failure
  [579:0x60ffc10]    37829 ms: Scavenge 2038.8 (2071.7) -> 2038.6 (2076.5) MB, 7.7 / 0.0 ms  (average mu = 0.246, current mu = 0.214) allocation failure
  [579:0x60ffc10]    37864 ms: Scavenge 2041.6 (2076.5) -> 2041.0 (2077.5) MB, 10.2 / 0.0 ms  (average mu = 0.246, current mu = 0.214) allocation failure

  <--- JS stacktrace --->

  FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
   1: 0xb090e0 node::Abort() [/runner/externals/node16/bin/node]
   2: 0xa1b70e  [/runner/externals/node16/bin/node]
   3: 0xce1a20 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/runner/externals/node16/bin/node]
   4: 0xce1dc7 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/runner/externals/node16/bin/node]
   5: 0xe994[35](https://github.com/KyberNetwork/integration-test/actions/runs/6180778211/job/16777756267#step:8:37)  [/runner/externals/node16/bin/node]
   6: 0xe99f16  [/runner/externals/node16/bin/node]
   7: 0xea843e  [/runner/externals/node16/bin/node]
   8: 0xea8e80 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/runner/externals/node16/bin/node]
   9: 0xeabd75 v8::internal::Heap::HandleGCRequest() [/runner/externals/node16/bin/node]
  10: 0xe[39](https://github.com/KyberNetwork/integration-test/actions/runs/6180778211/job/16777756267#step:8:41)[41](https://github.com/KyberNetwork/integration-test/actions/runs/6180778211/job/16777756267#step:8:43)7 v8::internal::StackGuard::HandleInterrupts() [/runner/externals/node16/bin/node]
  11: 0x11e5935 v8::internal::Runtime_StackGuard(int, unsigned long*, v8::internal::Isolate*) [/runner/externals/node16/bin/node]
  12: 0x15d9e59  [/runner/externals/node16/bin/node]


Automatic structural conversion can break type-safety at generic call-sites

Automatic structural conversion causes different type-checking results at the call-sites of generic functions.
In this case, the generic constraints are not taken into account:
interface A<T1, T2> {}
interface B<T1, T2> {}

function foo<T1, T2, T3>(x:T1, f:A<T1, T2>) {}

var addOne : A<string, string> = {}
var addOne2 : B<string, string> = {}

foo(1, addOne) // <-- ERROR HERE
foo(1, addOne2) // <-- FINE HERE (BUT SHOULD SHOW ERROR TOO?)

I would expect both of the bottom lines to have errors.

BTW this was discovered in http://www.typescriptlang.org/Playground
I assume v1.7

See #5451 or any other issue referencing the fact that empty types are interchangeable.
If you added a member to A / B that consumed T1 or T2 you'd see an error.


Bump deck.gl and mapbox related dependencies

This PR upgrades deck.gl to v8.9 with the official support of type definitions. We no longer need to run the postinstasll script from @danmarshall/deckgl-typings.
Some related changes are as followed:

Bumps mapbox-related dependencies to match the deck.gl version.
Bumps @ng-bootstrap to v10.0.0 to fix a bug that prevents NgbModule from loading after the version changes.
Minor refactoring on scatterplot visualization.

to be merged after #2025.


Layout changes

Adding .img-responsive class with js to best fit for images
Commented out modernize

Can one of the admins verify this patch?

Could we ignore the .sass-cache directory in .gitignore? Or this hidden directory is needed?

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8892/
Test PASSed.

Merged build finished. Test PASSed.

It doesn't look like your github account is correctly set up with your email. Could you please set up your account with the directions here: https://help.github.com/articles/setting-your-email-in-git/
Thanks!

@gpang I've changed my email address. Is there any problem?
@uronce-cc  Should I close this one and make a new pull request with .sass_cache/ ignored ?

@aunglinn The .sass_cache directory still exists.

I'm going to create a new pull request. Thanks @uronce-cc .


Fixes SnowplowFlutterTracker.initialize

Just noticed while integrating the latest commit on Main into our application that initialising a SnowplowFlutterTracker is currently broken and only tries to initialise when it was already initialised, instead of initialising when it wasn't initialised yet. 🤦
This PR fixes this and adds some tests.

I'll add some tests to .close as well.

Thanks! Nice catch.


Falco Hot Reloader

Description of Request
@Issif @Kaizhe @Dentrax and I developed a little Go binary that continuously monitors Configmap which includes rules and configuration of the Falco. Whenever the binary sees that there is a change about the rules, it will send a SIGHUP signal to the Falco's container within the same Pod. By doing that Falco can be able to reload itself and continue to work with the new configurations.
Motivation
For motivation see the proposal in Falco repository - PR falcosecurity/falco#1692

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


CI: Use jruby-<IP_ADDRESS>

This PR updates the CI matrix to use latest JRuby, <IP_ADDRESS>.
JRuby <IP_ADDRESS> release blog post

Hi @olleolleolle
I think, we should keep jruby-9000 as well — for regression testing on the oldest supported version.

@nepalez Here's what jruby-9000 resolves to in rvm stable:
$ java -Xmx32m -version
java version "1.8.0_151"
Java(TM) SE Runtime Environment (build 1.8.0_151-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.151-b12, mixed mode)
$ javac -J-Xmx32m -version
javac 1.8.0_151
$ ruby --version
jruby <IP_ADDRESS> (2.5.0) 2018-12-06 6d5a228 Java HotSpot(TM) 64-Bit Server VM 25.151-b12 on 1.8.0_151-b12 [linux-x86_64]
$ rvm --version
rvm 1.29.7 (latest) by Michal Papis, Piotr Kuczynski, Wayne E. Seguin [https://rvm.io]
$ bundle --version
Bundler version 2.0.1
$ gem --version
2.7.6

Perhaps it's not very useful?


I am trying to uploading a 15kb js file through github UI, its show error " upload file less than 25mb "

I am trying to uploading a 15kb js file through github UI, its show error " upload file less than 25mb "
its a simple nodejs file.
please solve it.

I don't really understand what you're trying to say. You tried to upload a JS file to LibreSignage? Through GitHub's UI? Also, I don't think LibreSignage
First of all, you can't upload JS files to a LibreSignage server because it's not an allowed filetype. And what does GitHub's UI have to do with that?


Problem running example_control_loop.py

I've attached a photo of the error. It occurs from the rtde_config.ConfigFile(config_filename) on line 43 of the program.

Any suggestions of points in the right direction are appreciated

It's likely because you're running example_control_loop.py from main folder instead of examples folder.
Version 2.7.2 is also fixing minor issue in control loop example.


MaterialAlertDialogConfiguration cannot set ButtonTextColor

🐛 Bug Report
Right now MaterialAlertDialogConfiguration allows users to set:
MessageFontFamily, MessageTextColor, TitleFontFamily, TitleTextColor, and ButtonFontFamily.
But there's no property to set button text color, which I would argue is even more important than the buttons' font family.
Unless I'm missing something of course 🤷
Configuration
Version: 1.5.3
Platform:

[x] :iphone: iOS
[x] :robot: Android
[x] :checkered_flag: WPF
[x] :earth_americas: UWP
[x] :apple: MacOS
[x] :tv: tvOS
[x] :monkey: Xamarin.Forms

https://github.com/BaseflowIT/XF-Material-Library/blob/develop/XF.Material/UI/Dialogs/Configurations/MaterialAlertDialogConfiguration.cs
This is the file that I'd expect the property to be added.

This is a property called TintColor. I'm embarrassed.


ML 2.1 : What is Learning? Why Machine Learning works? (A)

Welcome to 'ML' Team, good to see you here
This issue will helps readers in acquiring all the knowledge that one needs to know about What is Learning? Why Machine Learning works?.
To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", this issue is only for Audio content.
Domain : Machine Learning
Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Supervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Audio

All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Audio
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I want to contribute to this issue :
Name-Bhaswati Roy
Domain-Machine Learning
Batch-6
Issue Number-2775
Type-Audio
Contribution Approaches-what is learning & ml, importance of data in ml, how machine learns from data,

Sarmistha Sarna
Batch :01
https://github.com/ron352
type : audio
@abhisheks008  can I work on this?

@ron352 Definitely!


[RadioGroup] Prevent radio activation on manual focus

Fixes #938

Easier to review with hidden whitespace changes

@benoitgrelard this is ready for a second look whenever you're free 🙏


cards showing black background??

HI,
My cards are now all showing a black background... What could be the reason for this? Ive had a dark blue setting after 21 hours, but set it to default now. still showing black.

code:
light.gamer_den:
  state_card_mode: break-slider
  stretch_slider: true
  extra_data_template:
    if (attributes.color_temp) return attributes.color_temp + ' C' ; return null;
  hide_control: false
  show_last_changed: false
  theme: happy
  #confirm_controls_show_lock: true
  slider_theme:
    min: 0
    max: 255
    pin: true
    off_when_min: true
    report_when_not_changed: true
  extra_badge:
    - attribute: brightness
      unit: Br
#    - attribute: color_temp
#      unit: Co

light.keyboard:
#    extra_data_template: >
#    if (attributes.color_temp) return attributes.color_temp + ' C' ; return null;
  state_card_mode: break-slider
  stretch_slider: true
  hide_control: false
  show_last_changed: true
  theme: happy
  #confirm_controls_show_lock: true
  slider_theme:
    min: 0
    max: 255
    pin: true
    off_when_min: true
    report_when_not_changed: true
  extra_badge:
    - attribute: color_temp
      unit: Co
    - attribute: brightness
      unit: Br

---edit---
i sus[ect it had to do with the fact that on my phone i had selected a night theme.. Strange this had effect on my macbook with the default theme?

@hvbhome Themes are not device independent. If you change the theme on one device it will change on everything else. You are using your devices to access a webpage running on a local server after all.

Ok Thanks, i was thinking that too, but it only showed on the lights page, nowhere else...


ci: test all supported node LTS versions

Fixes #8

Edit: I couldn't get v14 to run, it seemed to have a problem importing "sibling" packages linked by lerna:
packages/valhalla/index.ts:13:8 - error TS2307: Cannot find module '@routingjs/core' or its corresponding type declarations.

    13 } from "@routingjs/core"


 It would be nice for Inliner to either retry on 503 or for me to set an option to limit the number of requests being made at once.

I worked around the issue by "saving as" "webpage complete" with chrome, then inlining that file.
Still, a way to limit the number of synchronous requests would be nice.

Makes sense - I'd take a PR if you want to take a shot.


throw new Error('Not running'); // error message from dgram_legacy.js

I don't got more atm but I'm getting this error soon after a torrent is removed, there's two in total, the other one is pending.
I have a working repo locally but it's not something I can reproduce elsewhere, the issue is probably external but there doesn't seem to be a way for me to handle the error, so the application crashes.
dgram.js:461
    throw new Error('Not running'); // error message from dgram_legacy.js

Error: Not running
    at Socket._healthCheck (dgram.js:461:11)
    at Socket.send (dgram.js:285:8)
    at RPC.send (C:\Projects\tordl\node_modules\k-rpc-socket\index.js:132:15)
    at RPC.query (C:\Projects\tordl\node_modules\k-rpc-socket\index.js:176:8)
    at RPC.query (C:\Projects\tordl\node_modules\k-rpc\index.js:142:17)
    at DHT._sendPing (C:\Projects\tordl\node_modules\bittorrent-dht\client.js:94:13)
    at DHT.addNode (C:\Projects\tordl\node_modules\bittorrent-dht\client.js:84:8)
    at Wire.<anonymous> (C:\Projects\tordl\node_modules\webtorrent\lib\torrent.js:582:23)
    at emitOne (events.js:90:13)
    at Wire.emit (events.js:182:7)

What version of webtorrent are you using?
What version of node.js are you using?

cc @mafintosh

@feross yeah, sorry. Latest 0.72.2 (first thing I did was to bump/update the version of webtorrent)
Latest node version, 5.5

Can you reliably trigger this error? If you can share code that triggers it reliably, it would really help to debug the issue.
Also, please try using webtorrent 0.74.0, since behavior might have changed.

Attempted a naive fix and released as v0.74.1.
Going to assume this is fixed now. If you see it again, please leave a comment here and I will re-open this issue.

@feross I'll look into. I had a working repro with a couple of torrents. I'll see if I can put together something. I'll also check if the bug is there before or after your fix.
Thanks for this.

No dice but a different stack trace.
dgram.js:461
throw new Error('Not running'); // error message from dgram_legacy.js
^
Error: Not running
at Socket._healthCheck (dgram.js:461:11)
at Socket.address (dgram.js:371:8)
at RPC.address (C:\Projects\tordl\node_modules\k-rpc-socket\index.js:119:22)
at RPC.address (C:\Projects\tordl\node_modules\k-rpc\index.js:107:22)
at DHT.address (C:\Projects\tordl\node_modules\bittorrent-dht\client.js:330:20)
at Torrent._onWire (C:\Projects\tordl\node_modules\webtorrent\lib\torrent.js:585:31)
at emitTwo (events.js:92:20)
at Swarm.emit (events.js:172:7)
at Peer.onHandshake (C:\Projects\tordl\node_modules\bittorrent-swarm\lib\peer.js:189:14)
at Wire. (C:\Projects\tordl\node_modules\bittorrent-swarm\lib\peer.js:136:10)

This was probably fixed a while ago. There were lots of fixes of this sort in bittorrent-dht a while ago.


Card default background color should be transparent

Additionally, same for shadow.  Shadow rendering also slows the component.
Current approach to remove default color is by passing
cardStyle: {
			backgroundColor: 'transparent',
			shadowColor: 'transparent'
		}

Well done!

transparent colors can cause perf problems, as described in https://launchdrawer.com/i-made-react-native-fast-you-can-too-9e61c951ce0


chef-client 13.0.113 compatability fixes

Stromweld, thank you for the change in code. How can we get this commited? It is breaking my ability to create AMI with packer and chef.

@L2G is there anything we can do to expedite getting this merged and pushed to the community supermarket?

@kenhen93 in the short term you can clone my fork with this fix and push it to your chef-server.
https://github.com/Stromweld/timezone-ii.git


Stable API

This is the main proposal for 1.0.0 milesone

Current 0.x.y, match the Dart semversion.
Which means, each minor version may or may not break APIs. But any patch version does not.
In 1.0.0, any patch or minor version will never break APIs.


Add accessibility to RMQ Animations

Use UIAccessibilityIsVoiceOverRunning() to detect if voiceover is on and if so, don't actually animate the object(s), just change the state(s) and run the callbacks.
From @amirrajan's RubyMotion #Inspect2015 talk - if you animate controls, they're not available for swiping in voiceover mode for the blind.
cc/ @austinseraphin - we missed you this year at Inspect!

Beautiful!


Code Reformatting

There are some major code changes. Mainly in the DQN utils.py file and the Common buffers.py files. Changed PrioritizedReplay to use deque and added a helper function in utils.py to add customizability to use fully connected and noisy layers together. There's some reformatting done here and there other than variable name changes, blank line and line length corrections.
Still have to go through a lot of code. Let me know if changes along these lines are fine.

Yeah, that's fine. Also, how exactly do you improve code coverage? I could try doing it while reformatting if it helps.

Code coverage is the percentage of code tested. So to increase, it we've to add more tests. I think it would be better if that could be done as a separate PR though

By adding tests for functions, classes, etc. You can check how much of each file is covered here: https://codecov.io/gh/SforAiDl/JigglypuffRL https://codecov.io/gh/SforAiDl/JigglypuffRL

On 25-Apr-2020, at 1:38 PM, Zeus3101<EMAIL_ADDRESS>wrote:
Yeah, that's fine. Also, how exactly do you improve code coverage? I could try doing it while reformatting if it helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/SforAiDl/JigglypuffRL/pull/93#issuecomment-619339915, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH72FJ7QLPCP3JJS5TEJ7K3ROKLBNANCNFSM4MQOGIUQ.

Ok nvm, I thought it could be improved by adding documentation.

Any other changes? I was considering revamping the save/load to make it use get_hyperparams instead. Should I instead do it in another PR?

I guess it could be implemented better with the Trainer class so maybe after that?

Are you still working on this?

Few key changes:

Changed Prioritized Buffer to use deque
Added noisy_mlp function, analogous to the mlp function. Makes customising Noisy layers easier and cleaner
Reformatted code

Push a useless commit to build Travis CI and Codecov again

Will merge this today. Can you resolve the conflicts. Created due to merging #106

Is this done?

Yeah

You can merge it now

Hmm codecov fails. I guess thats happening cause of the extra cuda statements somwhere (and travis not being able to run them).


Bad link to / has confusing error message

hyperlink apparently strips leading slashes which however results in a confusing error message when an index.html file in the base-path is missing:
% mkdir /tmp/test
% echo 'test' > /tmp/test/foo.html
% hyperlink /tmp/test
Reading files
Checking 1 links from 1 files (1 documents)
/tmp/test/foo.html
  error: bad link

Found 1 bad links

Notice that it doesn't say what the bad link is.

tempted to fix this by prepending / when formatting the error message


Restyle grid layer legend & display conditionally

Overview

restyle grid layer legend to use circles rather than boxes for the
legend items
display the grid layer legend only for zoom levels at which the grid
layer is shown

Connects #798
Demo

Testing Instructions

serve this branch with vector tiles switched on and the default facilities data loaded
verify that the legend now displays only at zoom levels for which the grid layer is also shown and that it disappears when the marker layer is displayed

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Looking at this now.

Consider also showing the circle size changes in the legend. Maybe something like:

Sounds good! I'll add this before merging.


Doubt regarding dataloader file

I have a small doubt in the file dataloader.py under the dataloader directory. In the function augment_loader, the train_label makes sense i.e., if train_text1 and train_text2 belong to same parent, then it is 1, otherwise it is -1. But, in the function train_unshuffle_loader, I don't get the use of train_label. Can you please explain that?

train_label is used to evaluate the clustering acc or NMI if the ground truth cluster labels are available. It's only used for evaluation.


ClangImporter: Mark synthesized struct constructors implicit

Just like https://github.com/swiftlang/swift/pull/61203 made sure ClangImporter synthesized struct accessors were marked implicit, synthesized constructors need to be marked implicit as well to avoid crashes when emitting coverage maps.

@swift-ci please test

@swift-ci please test

FYI this looks like the same issue as rdar://139486938, cc @Xazax-hun

FYI this looks like the same issue as rdar://139486938, cc @Xazax-hun

Wow! Indeed, the repro I have seems to be fixed by this PR. I'll assign the rdar to you in case you don't mind.

FYI this looks like the same issue as rdar://139486938, cc @Xazax-hun

Yeah, I was trying to reproduce a different compatibility suite regression and hit this instead.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test source compatibility


Button click question

Steps to reproduce:

Press a button (but don't release it)
Move the mouse outside the button
Move back the mouse inside the button
Release while inside the button

I was expecting that this will still generate a valid click event because the initial press and final release are on the same button.
However, it does not register a click event. Is this expected?
EDIT: I have modified the question and issue description, as the click outside events I previously mentioned were normal (for other buttons in the same gui).

In Druid, button for click event button should be pressed and released on this without leave node zone. I will add this to documentation. Thanks!
But, yea, this is the bug, click_outside event should be not trigger in this case.
Edit: oh, see. You say it work as expected for click_outside. But anyway I'll check it!


The editor has crashed

Description
As soon as I start atom live server it says editor is crashed

Actual behavior: Editor crashes
Please help me in starting live server

First thing to do would be opening the issue in the right repository. This is an issue with atom live server, so open it there. If the maintainers decide the bug is with Atom, they can then raise the issue on core.

@2sagarraut2  Here is the place it should be raised.


Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
c3444
Other environment information
No response
Actual behavior
我想在本地跑下这个项目，但是安装依赖的时候，开始用3.10版本，显示必须大于等于3.9 小于310。然后用3.9版本，显示必须大于3.10，我应该使用哪个python版本
Expected behavior
No response
Steps to reproduce
无

Additional information
No response

3.11

你本地跑起来了嘛？

Thank you for following! We intend to create an international community, so we encourage using English for questions and answers to help others with similar queries. 😊


need to review editor comments and take action

Chris has left a lot of Editor notes on Mondo terms when he created the ontology. The list of terms with editors notes is here.
This is a long list, but over time, we should review these comments and take appropriate action.
related to: https://github.com/monarch-initiative/mondo/issues/3784

(Initial) Action items:

[ ] Create a new ticket for the obsoletion candidates in this tab. Create a markdown table from the spreadsheet, see instructions here.
[ ] Create a new ticket for the merge candidates in this tab (create a markdown table using the link above

[ ] need to look up the (potential) terms to be merged with and add ID and label to the spreadsheet above
[ ] Nicole or Sabrina should review
[ ] next, add the merge candidates to this template

[ ] add the obsoletion candidates to this template

@sabrinatoro I can work on this, you should focus on other tickets. I'll try to work on this a bit each month, it's a really big ticket.

Can I ask that someone take a look at the MONDO:0011940 & MONDO:0000070 Merge that has been pending for a while? (scheduled for obsoletion on or after
2022-10-01)
Also Mycobacterium tuberculosis, protection against should be split into new record, with the OMIM 'included concept' tag. (related to #5507 )

Hi @kanems - thanks for pointing this out, there is not a ticket associated with that merge for some reason. I'll create one and I'll aim to address this before the next release.


Options not working if called from instance

I have remodal used quite a bit on a site I am building and in some cases allowing remodal to work automatically I use, in other cases where I want to trigger it on a condition I would generate an instance.
In cases where options are supplied as a data attribute, I notice the options not working. Specifically closing window : data-remodal-options="hashTracking: false, closeOnOutsideClick: false"
If remodal responds naturally the options seem to work.

I am having this same issue

user: @ralivav in issue #214
pointed out that when you create an instance, you can pass it options:
var modalSms = this.$('[data-remodal-id=modalSms]').remodal({             'hashTracking': false,            'closeOnOutsideClick': false }); modalSms.open();

Note for others running into same issue: your markup must not have the remodal class on it to begin with. If it does the options will not apply!


Clarification of document

In paper, it says

For Wikipedia we extract only the text passages and ignore lists, tables, and headers

I wonder if these text passages extracted from a document are separated as individual documents or they are recombined to be a document?

We treat every Wikipedia article as a single document. So for example you might have two sentences in different sections of the article be considered SentenceA and SentenceB for the purposes of next sentence prediction. Although it probably doesn't matter much either way.


Deleted vacancy page

I really don't think we need to do this/we should do this.
For a user to see this page, the following needs to happen

They create a job application
The job application receives applications and we send a email and a notification with a link
Somebody in the organisation deletes the job application
At some time after this the user clicks the link in an old email, or they click a link in an old notification (we don't need to render notifications for trashed vacancies)
Rather than a generic "Page not found", we take them to this page

This currently comes at the cost of an additional query to the database every time a user visits the vacancy job applications dashboard.
Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-2125
Changes in this PR

Update existing trashed vacancy page
Add publisher trashed vacancy page (to be rendered when clicking link to vacancy job applications dashboard in email/notification)

Context
https://ukgovernmentdfe.slack.com/archives/C01BB5D318W/p1616429664001900

Review app deployed to https://teaching-vacancies-review-pr-3279.london.cloudapps.digital


feat: add si8 , s16, s64 atomic types and corresponding atomic attributes

Added si8 , s16, s64 atomic types and corresponding tests to import/export types test files.
Added si8 , s16, s64 atomic attribute types. I added corresponding tests to the project.mlir test file since project uses literals, so I thought that was the easiest way to test for it. Let me know if I should change this @ingomueller-net

TODO: When this PR gets merged, I will add import/export tests to the project.mlir / project.textpb files testing the si8, si16, si64 attribute types! Will do this in this PR or a separate PR depending on the situation.


Confusing logs when using Keras model

I've loaded a trained model with its weights and config file, like shown in https://github.com/JetBrains/KotlinDL/blob/master/docs/importing_keras_model.md
There are some logs with INFO tag, which look a bit confusing for person, who is not yet comfortable with Keras/Tensorflow:

Maybe change that kind of logs to DEBUG?

I agree, then maybe the number of parameters loaded could be added to the no weights loading for ... prints (or as separate prints). Might be good to move this to the discussions section.


Allow use of names for positional-args

Current behavior is to use numbered args in a justfile:
set positional-arguments

@foo bar:
  echo $0
  echo $1

Given bar is labelled and just will report on it if it is missing from the recipe, some syntactic sugar might allow for it to be referenced by name in the recipe, e.g:
set positional-arguments

@foo bar:
  echo $0
  echo $bar

It might just reduce the learning curve of the format a little more for new users but also help make the recipe more reliable if that value need be repeated on any other lines without having to keep in mind the enumeration.

set positional-arguments does not prevent the use of just variables

ah, cool. Much nicer.
Feels like there's some skew in definition here between arg/variable but probably not worth going into.
Thanks for pointing that out!


Animations with hideExpression function

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x] support request

Would it possible to provide a demo or some advice on how you would use the hideExpression function with Angular animations?
For example applying some type transition effect to the fieldGroup.
Thanks in advance.

I think you can do it by using a custom wrapper and change the animation state based on hide property.

it would be nice if you can add an example in our demo.

Hello, I need animation in hiddenExpresision, but i cant to create custom wrapper.  What could I do?
You could create a property in the json:
transitionShow: slide1
transitionHidden: slide1
I can do it if you indicate me where to modify it and make a PR
thanks

have you thought anything about animation, friend?

In a simple field, you can not, right? it would be good use:
{
key: 'firstName',
type: 'input',
animation: true,
templateOptions: {
required: true,
type: 'text',
label: 'First Name',
},
},

possible but you have to wait until tomorrow :)

thanks.

Animations do not work when hidden. The same thing happens with all.
what's going on?
https://stackblitz.com/edit/ngx-formly-animation-yhjg6e?file=app/app.component.ts

you have to disable the default hide behavior
  // disable hide default behavior
  styles: [`
    ::ng-deep formly-field {
      display: block !important;
    }
`]

see https://stackblitz.com/edit/ngx-formly-animation-zzcq4s

great

see https://stackblitz.com/edit/ngx-formly-animation-zzcq4s

yeahhhhh.
But just one more details. with both do not work. (field and fieldgroup)
https://stackblitz.com/edit/ngx-formly-animation-sreafs?file=app/app.component.ts

in style add :host selector:
styles: [`
  :host::ng-deep formly-field {
    display: block !important;
  }
`]

the wrapper is inside formly-field, here is another way to solve it:
https://stackblitz.com/edit/ngx-formly-animation-bdjyij
by adding disable-default-hide class into animated field
.disable-default-hide {
  display: block !important;
}

But when the element is hidden, the animation does not work
This is a example
https://stackblitz.com/edit/ngx-formly-animation-m3txne?file=app/animation-wrapper.component.ts

you should move disable-default-hide into the main css or use ViewEncapsulation.None

You could show an example. I can not make it work

Oh ok ok, now  this is not necessary:  wrappers: ['animation'], but I have to use: animation: true , not only in the wrappers field but also in the input field. right?

hello, I'm testing animations with text (template) but they do not work
https://stackblitz.com/edit/angular-dstmcq?file=src%2Fapp%2Fanimation-wrapper.component.ts

you have to wrap it inside a fieldGroup see https://stackblitz.com/edit/angular-dstmcq-k7rwwc


Rewrite for kubernetes manual install guide for CNI

A couple things to address, but otherwise looks fine.
Do we want to talk about IPAM at all?

Comments are addressed. I don't think this is the right place to talk about IPAM.

It's actually pushed now.

LGTM


Native windows port #519 (mingw 32/64 + msvc 2015/2017)

Hi
Here is CMake based native windows port of leveldb's master.

I could build this on Windows with mingw compiler. Thank you.

very nice, !build successfully with vs2017

We've landed the official Windows support, based on code from Chrome. Going to close this PR now. Sorry we couldn't get to it sooner!