Change site architecture to use Jekyll

This removes Node.js as a dependency for building the site, and allows
standalone updates to the sites.json file to render to the site without
doing any building of HTML files.
To account for these differences, the CI build has changed to utilize
a mixture of Ruby packages/scripts and a single call to a NPM package
(via npx) to perform similar validations and linting.
Written in collaboration with PyroSamurai<EMAIL_ADDRESS>Closes jdm-contrib/justdelete.me#194

FYI, I have cloned this branch to my gh-pages branch, and you can view the generated site here: https://nightfirecat.github.io/justdelete.me/

A  pull request wasn't really had in mind but can you assure this pull request will being in working order after the merge?

So long as https://nightfirecat.github.io/justdelete.me is working fine (that is, is behaving identically to http://justdeleteme.xyz), it should also be working fine after merging.

hmm, okay. I'm not too worried. I'll go ahead and merge it. We'll fix anything that breaks right away, anyways. Please head over to telegram chat afterwards as I have some things I wanted discuss before we pushed this.

Well, hold off on merging till we discuss first then. :)


Update CI

Proposed Changes
ci.yaml

update versions
update set-output to env logic (set-output is deprecated)
change Codenotary logic as the old Github action uses Github API --> rate limit problems

deploy.yaml

update versions
update set-output to env logic (set-output is deprecated)
change Codenotary logic as the old Github action uses Github API --> rate limit problems

cloudflare_release.yaml --> release_updater.yaml

update set-output to env logic (set-output is deprecated)
change release updater logic as the old Github action uses Github API --> rate limit problems

I also updated the workflow to the latest version which I use to keep my Github binaries up to date.
This version brings two new features:

generic update approach to update more than one Github release (not really needed as we only have cloudflare/cloudflared in use)
completely automate the update process
-- create PR (as different User [PAT] or authorized Github App --> triggers workflow run for PR without manually closing & reopening)
-- set PR to auto merge if all status checks pass (CI + Review)
-- add approval review

I commented out all the new optional features as PAT (from second account) or Github app needs to be set up first. I also don't know if you want to have this level of automation 😉

I did a test in a separate repository with the PR version for release updates and everything went smooth. As it is not possible to run workflows from PRs, we should trigger a run after merging. However, I do not expect any major problems... I am also available tonight to troubleshoot any issues that arise if needed.

#275 👍🏻

@elcajon you did close and re-open the PR, I thought with the new system this was not necessary anymore to trigger the Actions?

This would only work if we change the token to Github App or PAT. The current implementation behaves exactly as the previous one.

ah got it, sorry for missing that. Let's change it, I am happy with both approaches (for a PAT we could use my brenner-bot). Which approach do you recommend?

I would also recommend to use a PAT from your bot account. I guess you need to add the account as contributor and save a PAT as secret but then it should work.


Fix code smell for array list declaration in LoginService

Hey @mepox
I have raised PR for the issue #60
Have made the suggested changes to fix the code smell.

Hi @ujwalkumar1995 ! The PR looks good. Thanks for your contribution! Ignore the checks, this will be merged soon.


:sparkles: Add interactive cmd prompt for permission claims

Summary
This PR introduces an interactive command prompt to edit the status of open claims for apibindings. The user can provide - Yes/No/Skip to accept/reject/skip any action on a permission claim respectively.
Use cases:

Edit permission claims related to a specific APIBinding:

➜  kcp git:(claims/edit) ./bin/kubectl-kcp claims edit apibinding cert-manager
Running interactive prompt. Enter (Yes/No/Skip)
Accept permission claim for Resource: secrets (APIBinding: cert-manager) > yes

Edit permission claims related to all apibindings in a workspace:

➜  kcp git:(claims/edit) ✗ ./bin/kubectl-kcp claims edit apibinding
Running interactive prompt. Enter (Yes/No/Skip)
No open claims found for apibinding: "apiresource.kcp.dev-1wked"
Accept permission claim for Resource: secrets (APIBinding: cert-manager) > Yes
No open claims found for apibinding: "tenancy.kcp.dev-59phz"

Use the edit command, but there are no open permission claims:

➜  kcp git:(claims/edit) ✗ ./bin/kubectl-kcp claims edit apibinding
Running interactive prompt. Enter (Yes/No/Skip)
No open claims found for apibinding: "apiresource.kcp.dev-1wked"
No open claims found for apibinding: "cert-manager"

What happens after editing?

The APIBinding's spec.AcceptablePermssionClaim gets updated based on the user input. For example:

➜  kcp git:(claims/edit) ✗ k get apibinding cert-manager -o yaml
apiVersion: apis.kcp.dev/v1alpha1
kind: APIBinding
metadata:
  ...
spec:
  permissionClaims:
  - all: true
    resource: secrets
    state: Accepted
  reference:
    workspace:
      exportName: cert-manager-stable
      path: root:cert-manager-service
status:
  appliedPermissionClaims:
  - all: true
    resource: secrets
  boundResources:
  - group: cert-manager.io
    resource: certificaterequests
    schema:
...

PS: With the updates added to the permission claims spec (like sub-resources, verbs), the prompt's UI will need to change accordingly.
Design doc: https://docs.google.com/document/d/1J31wXY1-2aCyyGFjUlusKoHDzV-zktAmRdIMjMf4OR0/edit
Related issue(s)
Fixes #

A question I don't see addressed in the design doc - will there be a way for users to programmatically accept permission claims? I know there's danger there and we might not want to make it too easy, but I'm anticipating someone asking.

Agreed, for now my understanding was anyone who can access the workspace where APIBinding was created can change the state of the claim. Or maybe the right wording is, any user who can edit the apibinding can accept/reject respective claim.

That's probably sufficient; if someone wants to automate things, they can use Go or wrappers around kubectl to do it. This command can be purely interactive. 👍

@varshaprasad96 so sorry for losing sight of this PR! Are you still available to work on it, or would you be interested in trying to find someone to take it over for you?

@ncdc I can work on this sometime this week and rebase with the latest changes.

I still think we want this / something like this.
/remove-lifecycle stale


android.permission.CALL_PHONE

Why does the library need the permission android.permission.CALL_PHONE ?

It doesn't. This permission was brought in a recent pull request or
something - I can't actually remember. I'll remove it tonight.
Sorry!

Fixed! Sorry for the inconvenience.


Panel 31

Creates NCL panel_31 example.
Resolves issue #476
Original NCL Plot:
https://www.ncl.ucar.edu/Applications/Images/panel_31_lg.png
Python Plot:

It looks like my changes to the layout did work (see the plot build here). I will add a couple more minor changes (comment updates and spacing) and add my overall review!


Moved base form styles from cf-core to cf-forms

This doesn't do anything other than move the styles so they're organized in one place. It's a first step in a larger atomic re-write of our form styles. This change is bound to be contentious and therefore deemed important enough to do separate from everything else.
Changes

Moved base form styles to cf-forms

Testing

checkout and run npm run cf-link
open new tab and cd to your sandbox repo
in the sandbox tab run npm link cf-forms
in the sandbox tab run npm run build
in the sandbox tab run npm start
navigate to the cf-forms examples and ensure everything looks as it did (mostly inputs)

Review

@mistergone
@cfarm
@Scotchester

Notes

@Scotchester has voiced that he's not in favor of this change because they are basic html elements not needing classes. While I agree, I find it's super confusing that form elements are the only ones styled in two places and believe this is a usability improvement for new (and even existing) developers.

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[x] New functions include new tests
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Here's a preview of why this will make a difference. With base styles and cf-input combined, we end up with
.a-text-input {
    .u-webfont-regular();

    // Browser Reset
    appearance: none;

    display: inline-block;

    box-sizing: border-box;
    padding: unit( 6px / @base-font-size-px, em );
    border: 1px solid @input-border;
    width: 100%;

    background: @input-bg;
    vertical-align: top;

    &:focus,
    &__focused {
        border-color: @input-border-focus;
        outline: 1px solid @input-border-focus;
    }
}

Which will allows us to easily extend the styles to other states and situations, instead of having to use classes in some places and element selectors in others ex: https://github.com/cfpb/capital-framework/pull/385#discussion_r75884973

Looks good to me!

It seems like this is about two distinct issues:

Going to work on cf-forms and not knowing that base styles are set in cf-core.

It seems like this could easily be solved with a prominent code comment notifying the user of that fact.
It's not really true that this is the only place this happens. For example, basic list styling happens in core, but fancy list stuff happens in cf-typography.

Wanting to have a class on inputs for the purpose of … well, I'm not entirely sure.

Which will allows us to easily extend the styles to other states and situations

What possible other situations would we use this styling for?

instead of having to use classes in some places and element selectors

What's wrong with that? It already happens with stuff like lists. It's simple CSS. Are there places where it's causing actual cascade/specificity/overriding issues that I"m just not seeing? I don't totally understand the discussion that you referenced in https://github.com/cfpb/capital-framework/pull/385#discussion_r75884973

I still don't like having to use a class for every usage of a basic HTML element. If I'm never going to use the element without that class, why force myself to remember to add the class every time, or else get a default-browser-styled element?

So, currently still a :-1: from me. But open to being convinced.

It seems like this could easily be solved with a prominent code comment notifying the user of that fact.

But why? Why change it in two places when it's far easier in one? What's the selling point besides "we've always done it like this"? As far as I've seen we don't ever use cf-core without using cf-forms, so I don't see the value in keeping the separated.

It's not really true that this is the only place this happens. For example, basic list styling happens in core, but fancy list stuff happens in cf-typography.

Lists a much different use case since they are often used without any attributes whereas inputs will always have attributes because they require a name, id, and type as well as often having a placeholder and value depending on the type.

What possible other situations would we use this styling for?

There are already three instances for inputs, error, warning, and success. To keep with the atomic principles, the first three should be .name__state instead of .name.state. That's not possible and a lot messier to do with a bunch of elem[attr] selectors.
What's wrong with that? It already happens with stuff like lists. Where? There are no attr selectors in lists as far as I know so there are no specificity issues when trying to use a class instead.

If I'm never going to use the element without that class

See above, inputs are never used without markup attributes. Adding a class isn't a big deal and it's probably automated 9 times out of 10 with templates.

But why? Why change it in two places when it's far easier in one? What's the selling point besides "we've always done it like this"? As far as I've seen we don't ever use cf-core without using cf-forms, so I don't see the value in keeping them separated.

I said "comment" not "duplicated code" :) There would be no changing things in multiple places. The selling point is that is just seems natural (to me) to do basic element styling all in the same place.

Lists are a much different use case since they are often used without any attributes whereas inputs will always have attributes because they require a name, id, and type as well as often having a placeholder and value depending on the type.

I don't see how this is relevant to where the Less code lives, which is what I was talking about when making that particular point.

What possible other situations would we use this styling for?
There are already three instances for inputs, error, warning, and success. To keep with the atomic principles, the selectors should be .name__state instead of .name.state or elem.state. That's not possible and a lot messier to do with a bunch of elem[attr] selectors.

What's wrong with that? It already happens with stuff like lists.

Where? There are no attr selectors in lists as far as I know so there are no specificity issues when trying to use a class instead.

And finally we come to the point where I begin to understand that attribute selectors are the actual problem, causing specificity issues. I wasn't making that connection, and it was never stated explicitly. In fact, attribute selectors have not been mentioned at all prior to your last post :)
I don't like that we have to do it, and if it were just a personal project, I would deal with the specificity issues to conform to my vision of the "ideal" (in terms of code organization and not requiring classes on basic elements), but I get it now. :+1: for making it somewhat simpler for a team dealing with it.

Will do.
As for the two places thing, I didn't mean make the same changes in two places, I meant if you were adjusting the inputs later in life, and it turned out a base adjustment also messed with cf-forms, you're now making changes in two places.


Friend object has no uuid in Player.friends.records

def main():
    ign = input('Please enter your ign: ')
    uuid = UUIDLookup().resolve_by_name(ign).id
    if not uuid:
        print("That player was not found. Restarting")
        return main()
    player = Player()
    player.fetch_player(uuid)
    print(player.friends.records)

This is my code when I print the records its a list of Friend objects that have no uuidsender and uuidreceiver. I can get both of those using raw_data. Output in file.
output.txt

I expect the cause is simply an incorrect key name here:
https://github.com/Preocts/hyapi/blob/main/src/hyapi/models/playerfriends.py#L22-L23
I can look into it this weekend. Also open to a pr for updating those key names if your raw_data confirms that is the issue.

Yeah, those names should be camelCase.  uuidSender and uuidReceiver
https://api.hypixel.net/#tag/Player-Data/paths/~1player/get

Thanks!


Docs update: more boards; Verilog CFU handshake details.

Signed-off-by: Tim Callahan<EMAIL_ADDRESS>

Documentation updates look good to me!


Incorrect headers

Hi,
Thanks for this module. I had a problem with the headers however. The default content-type isn't set correctly and the common header ('Accept': 'application/json, text/plain, */*') is not included. I followed the axios code in lib/defaults.js and lib/core/dispatchRequest.js to come up with my own simplified version:
import axios from 'axios';
import {isURLSearchParams, isObject, merge} from 'axios/lib/utils';
import qs from 'qs';

declare module 'axios' {
    interface AxiosRequestConfig {
        curlCommand?: string;
    }
}

const filterKeys = new Set(['delete', 'get', 'head', 'post', 'put', 'patch', 'common']);

axios.interceptors.request.use(config => {
    try {
        const {method, url, params, data} = config;
        let {headers = {}} = config;
        let dataArg = '';
        const setsContentType = Object.keys(headers).some(key => key.toUpperCase() === 'CONTENT-TYPE');
        if(isURLSearchParams(data)) {
            dataArg = ` --data '${data}'`;
            if(!setsContentType)
                headers['Content-Type'] = 'application/x-www-form-urlencoded;charset=utf-8';
        } else if(isObject(data)) {
            dataArg = ` --data '${JSON.stringify(data)}'`;
            if(!setsContentType)
                headers['Content-Type'] = 'application/json;charset=utf-8';
        }
        headers = merge(
            headers.common || {},
            headers[method] || {},
            headers,
        );
        config.curlCommand = `curl -X ${method.toUpperCase()} ${
            Object.keys(headers).filter(key => !filterKeys.has(key))
            .map(key => `-H "${key}:${headers[key]}"`)
            .join(' ')
        }${
            dataArg
        } "${url}${
            params && Object.keys(params).length ?
                `?${qs.stringify(params)}` :
                ''
        }"`;
    } catch (err) {
        console.error(err);
    } finally {
        return config;
    }
});

I confirm the content-type header is for sure wrong, json requests (axios default) are sent with form-urlencoded content-type.

I'm having the issue in v2.0/v1.3.7. I'll willing to help if someone can offer some guidence. :)

friendly ping @anthonygauthier :)

So I know this is an old issue, but I've been trying to reproduce this locally without any success. Any time I make a request to a service using axios the generated cURL command lists the right headers. One potential thing that might be happening is the presence of another interceptor modifying the headers field after curlirize already generated the command?

it seems only happen with the non-get requests.
just made a simple repro: https://github.com/aladdin-add/repros/tree/repro-axios-curlirize-issue25

I also had the same issue in NestJS so I added the content-type during the module registration as follows and it correctly added the content-type header in the curl string.
...
import { HttpModule } from 'nestjs-http-promise';
...
HttpModule.register({
      headers: {
        'Content-Type': 'application/json',
      },
}),
...


Add tests that management commands start

This test caught a few bugs where refactoring had made management
commands fail (and would have caught a few more recent ones).
Ideally we'd replace this with a more advanced test that actually
tests that the management command do something useful, but it's a
start.
Thanks to @walterheck for the suggestion (see #282)!

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

:+1: FFTP

Merged, thanks for the review!


Adjust conformance test for ogcapi

I don't know what changed server wise, but the number of conformance items seems to have changed.

pep8 failure not related to this PR.


Annoying output close

If prettier.openOutput is set to true, an error will correctly cause it to open. However when the error is resolved, this will close the output view as well as the integrated terminal one. Which is pretty annoying!

The default is set to true To end the annoyance -- Include"prettier.openOutput": false,  inside your VSCode user settings.


About get flops

I would like to know how to get the flops of the model
When using get_flops.py in tools to get flops, there is an error here
  File "/mmdet/models/detectors/two_stage.py", line 88, in forward_dummy
    roi_outs = self.roi_head.forward_dummy(x, proposals)
TypeError: forward_dummy() missing 1 required positional argument: 'roi_labels'

This is def forward_dummy in two_stage.py:
    def forward_dummy(self, img):
        """Used for computing network flops.

        See `mmdetection/tools/analysis_tools/get_flops.py`
        """
        outs = ()
        # backbone
        x = self.extract_feat(img)
        # rpn
        if self.with_rpn:
            rpn_outs = self.rpn_head(x)
            outs = outs + (rpn_outs, )
        proposals = torch.randn(1000, 4).to(img.device)
        # roi_head
        # roi_labels = torch.zeros(1000)
        roi_outs = self.roi_head.forward_dummy(x, proposals)  # roi_labels
        outs = outs + (roi_outs, )
        return outs

refinemask adds roi_labels in def _mask_forward, refine_roi_head.py during training
I tried defining  roi_labels with all 0
        roi_labels = torch.zeros(1000)
        roi_outs = self.roi_head.forward_dummy(x, proposals, roi_labels)

but have error
  File "/mmdet/mmdet/models/roi_heads/mask_heads/refine_mask_head.py", line 561, in forward
    instance_logits = self.stage_instance_logits[idx](instance_feats)[torch.arange(len(rois)), roi_labels][:, None]
IndexError: tensors used as indices must be long, byte or bool tensors

when I change the type to bool
  File "/mmdet/mmdet/models/roi_heads/mask_heads/refine_mask_head.py", line 561, in forward
    instance_logits = self.stage_instance_logits[idx](instance_feats)[torch.arange(len(rois)), roi_labels][:, None]
IndexError: The shape of the mask [1000] at index 0 does not match the shape of the indexed tensor [100, 1, 14, 14] at index 1

How to implement the calculation, hope you can help me!
I don't understand the dimension and type of the roi_labels
# this is forward_dummy in mmdet/models/roi_heads/standard_roi_head.py
    def forward_dummy(self, x, proposals):
        """Dummy forward function."""
        # bbox head
        outs = ()
        rois = bbox2roi([proposals])
        if self.with_bbox:
            bbox_results = self._bbox_forward(x, rois)
            outs = outs + (bbox_results['cls_score'],
                           bbox_results['bbox_pred'])
        # mask head
        if self.with_mask:
            mask_rois = rois[:100]
            mask_results = self._mask_forward(x, mask_rois)
            outs = outs + (mask_results['mask_pred'], )
        return outs

It has not been supported to run get flops on RefineMask. I will fix the bug when I have time to do that.

Thanks a lot, looking forward to it

@zhanggang001 hello, is there any progress on this part of the code?


chore: replace auto-generate public ip with <IP_ADDRESS>

Motivation
Most people test the OSS repo locally. By defaulting to the public IP, the Terraform cluster fails to finish booting because it can't be accessed.
We'll provide better docs on the config after our OSS refresh.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#650  👈
#649
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Support tokio for platforms without multi-threading support

Description

@langyo
This pull request consists of the following changes:

Supports tokio for platforms without multi-threading support.

Checklist

[x] I have self-reviewed and tested this pull request.
[ ] I have added tests for my changes.
[ ] I have updated docs to reflect any new features / changes in this pull request.
[ ] I have added my changes to CHANGELOG.md.

After some consideration, I think it is not possible to replicate the behaviour described in test test_spawn_local_within_send.
Since it is not possible to automatically register a LocalSet for tasks spawned by tokio::spawn to enable prokio::spawn_local when a task is not running in a dedicated prokio worker thread, This results in hard to explain behaviour and contradicts how prokio works with multi-threading support.

use prokio::Runtime;

let runtime = Runtime::default();

runtime.spawn_pinned(|| async {
    tokio::spawn(async {
        prokio::spawn_local(async {
            // Despite running without prokio runtime,
            // this does not work without dedicated worker thread.
        });
    });
});

For now, I think it is better to assume outside-runtime behaviour for prokio.
Where a LocalSet is explicitly required to use LocalServerRenderer.
Since most web frameworks in tokio uses Send futures, I do expect issues with most web frameworks.
However, the only way to solve this properly is to wait std::thread support for wasm32-wasi.


✨ [ENHANCEMENT] - Enhancement for Issue Templates

Description
We already have some issue templates in place, but we need to make it easier for contributors to raise issues. To achieve this, we are planning to enhance all the issue templates.
Tasks
For each individual issue template, the enhancement includes:

Adding a detailed description.
Including a screenshot (if applicable).
Implementing a task bar for contributors to express their willingness to work on the issue.
Incorporating an additional information field.

Additionally, we will provide a link to star the repository.

@Pinaka-Pani-18 Can i do this

Hey @Pinaka-Pani-18 ! I would love to work on this Issue as well.


Fixed indent issues

Using two annotations or two labels on the service field causes just the first one to be taken into account.

Can you bump the charter version?

Thanks!


mixer produces errors during graceful shutdown of adapter pods

Bug description
istio-policy pods call out to our custom adapters for policy checks. We notice that every time an adapter pod is killed (either due to a deployment or an eviction), a small set of requests from mixer fail.
We tried adding a prestop hook of sleep 30 to the adapter pods to give mixer a chance to update its endpoints but that seems to have no effect. My hypothesis is that these are persistent grpc connections that only fail when the adapter pod closes the connection. (I haven't dug into how this works so I may be totally offbase).
Is this a known issue and, if so, is there a workaround to get zero-downtime deployments of adapter pods?
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
100% of policy checks succeed during a rolling update of adapter pods.
Steps to reproduce the bug
Will update shortly
How was Istio installed?
helm charts
Environment where bug was observed (cloud vendor, OS, etc)
AWS
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.
Version (include the output of istioctl version --remote and kubectl version)
client version: version.BuildInfo{Version:"1.1.2", GitRevision:"2b1331886076df103179e3da5dc9077fed59c989", User:"root", Host:"35adf5bb-5570-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.1"}
citadel version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}
galley version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}
pilot version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}
policy version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}
sidecar-injector version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}
telemetry version: version.BuildInfo{Version:"1.1.8", GitRevision:"145b18a441045d6fad33d7916380d8642c7bf21d-dirty", User:"root", Host:"11387264-87af-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-14-g145b18a"}

Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.8", GitCommit:"4e209c9383fa00631d124c8adcc011d617339b3c", GitTreeState:"clean", BuildDate:"2019-02-28T18:40:05Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Errors are of the form:
2019-07-18T23:14:27.866106Z	error	api	Check failed: performing check operation failed: 1 error occurred:
* rpc error: code = Canceled desc = context canceled

Looks like https://github.com/istio/istio/issues/9992#issuecomment-466229569 is relevant to this case, we will try out this feature

Thanks for reporting this! I think the retry happens on transport error between mixer proxy filter to mixer server and I doubt it could help mitigate the transport error between mixer server and oop adapter. But yeah please try it out, meanwhile I think we should really add retry into mixer server.


Use the TIG stack for data visualisation

Description
Implement TIG stack for local hosting of data visualisation
Acceptance Criteria
Link to IoT HackMD

Done, but TIG stack not included in this repo.

Closed with #25


Modify Function Room.create return type, from Promise<Room> to Promise<Room|null>

Now,  function Room.create() return type as follows:
public static create(contactList: Contact[], topic?: string): Promise<Room> {

I suggest it return Promise<boolean> to detect whether it can create room successfully
Related logs as follows:
If it can create room successfully, return as follows:
Bot createDingRoom() new ding room created: %s Room {
  domain: null,
  _events: {},
  _eventsCount: 0,
  _maxListeners: undefined,
  id: '@@356afb5f9c9491eb34158dd1f11b99e136d104c9055106545869cbba1b0fec1c' }

If it cannot create room successfully, throw the following exception:
02:22:13 ERR PuppetWebBridge roomCreate(@ac4db07084d16fe10555569c660e5b8526d162bdc0acca2e93dd72d322f22b4a,@35f68307d4ec6b6e0eabead00e446127) exception: undefined
02:22:13 WARN PuppetWeb roomCreate(@ac4db07084d16fe10555569c660e5b8526d162bdc0acca2e93dd72d322f22b4a,@35f68307d4ec6b6e0eabead00e446127, 网球黑龙江1群) rejected: undefined
02:22:13 ERR Room create() exception: [object Object]

Could you explain the pros and cons between the two design?
A good purpose should go with good reasons.

After I thought more, maybe it should return Promise<Room | null> will be better.
If it cannot create a room, we want to get the fail status and don't see more error.
Like Contact.find() we will return Promise<Contact | null> instead of throw Error if it cannot find a contact.

Yes, that's the right answer after you think more.
Please think more before post question/suggestions, at least explain your ideas with pros and cons in detail.

But now it returns  and throw error, maybe is should return <Room | null> instead of throw error.

I agree your purpose.
Please change the title of this issue to better reflect your purpose, and I'll label it as a enhancement

If we can not find a contact by name, it's not an error because the reason will be there's no contact with that name.
However, if we can not create a room, there must be something wrong.

In CASE 1 we should return null, in CASE 2 we should throw an exception.
So my question is: when will null be returned by Room.create() is necessary, and why?

Close this issue due to inactivity for almost two weeks.

We should not return null.
We are planning to throw an Exception instead.
See also: https://github.com/Chatie/wechaty/issues/683#issuecomment-317378166


Testing the new pr

Testing if the new copr repo is created.

/packit test

/packit test

/packit build

The issue in packit-service was fixed. Not needed anymore.


implement detect_clear function

This PR adds a clearsky.detect_clearsky function. The function is a port of the matlab library's pvl_detect_clear_times. The main differences are:

no support for unequal times -- you'll need to resample your irregular data before passing it to the function, or you can implement the feature yourself!
automatically determines sample_interval
requires a reference clear sky series instead calculating one from a user supplied location and UTCoffset. I suggest that users that want functionality closer to the matlab function should implement a Location.detect_clearsky method.
parameters are controllable via keyword arguments instead of hardcoded
option to return individual test components

@cwhanse provided me with a short test file and expected output from the matlab function. We do not think that it covers all cases, but it's the best we have for now. Cliff has also been a big help in tracking down some issues with my initial implementation and helping me understand how the function works. Thanks!
I hope to merge this by the end of the week, so please let me know sooner than later if you have comments or concerns. Maybe an 0.4.3 release with just this and #283?
to do:

[x] test for NotImplementedError for irregular data
[x] probably need tests for more of the arguments
[x] test that a RuntimeWarning is generated if the clear sky scaling for loop fails to break
[x] ensure that documentation example works
[x] update documentation to make input names more general. State that it was designed for GHI but can work with other inputs if the filter settings are adjusted.
[x] update whatsnew (waiting on #283 for git simplicity)

Some examples from the NREL MIDC OASIS irradiance sensor here at the U. Arizona campus.
location = pvlib.location.Location(32.229961, -110.95388, altitude=741, name='OASIS GHI', tz='US/Arizona')

# data available at http://www.nrel.gov/midc/ua%5Foasis/
# import not shown here
ghi = data.resample('1min', how='first')
cs = location.get_clearsky(ghi.index)

clear_samples = pvlib.clearsky.detect_clearsky(ghi, cs['ghi'], cs.index, 10)

cs['ghi'].plot(label='clear sky profile')
ghi.plot(label='measured ghi')
ghi[clear_samples].reindex(ghi.index).plot(label='detected clear')
plt.legend(ncol=3)
plt.ylabel('irradiance W/m**2')
plt.ylim(0, None)
plt.title(location.name)

Hi, I'm trying to replicate the code for a college assignment, but I can't get a similar result. I'm actually just getting errors. I would like to know if it is possible to help me? (And yes, I'm new to programming).

Hi @erased99, this github repository is for the development of pvlib-python.  Questions about PV modeling or programming should go to a forum like StackOverflow, for example using the pvlib tag: https://stackoverflow.com/questions/tagged/pvlib


RuntimeError: probability tensor contains either inf, nan or element < 0

I was able to do finetuning before but after the recent update I am getting this error:
RuntimeError: probability tensor contains either inf, nan or element < 0
The same error happens when I try to do inference with a previously fine tuned model.

Did you pull the latest changes?
What script did you run, what arguments did you pass?
Did you make any changes to the script?

Sorry for not providing the details. I pulled the latest changes and ran the adapter_v2.py script. I did not change the script just changed the number of epochs. The fine tuning runs but after optimizer.step() the loss becomes nan. I tried reducing the learning rate but it does not help. My dataset size is very small.
Also I am training on 4 v100 so no bfloat16 support. I am running in 16 bit.

16-true. The training runs fine and I am able to use the generate script. However I want to experiment multiple prompt and really want to try the chat/interactive mode. Since there is no script to try the fine tuned model in chat mode, i tried to add the adapter to the model in the chat script but it throws error that the probability tensor has inf or nan values. I tried to make the generate script interactive as well but same thing happens :(

bf16-true will most likely fix it


✨ Add support for the ObservedGeneration in HelmChartProxy Status field

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #144

/ok-to-test

Please not, that update test can be implemented as real reconcile test with setup-envtest and will be fixed as part of the #146

/retest

/lgtm
Could you squash your commits before merging?

Could you squash your commits before merging?

Yes, sure. Will do it.
Thanks a lot.

/lgtm
/approve
Nice work!


[PRED-1395] Improve logging

Ticket: https://datarobot.atlassian.net/browse/PRED-1395
Super small change to improve logging:

log version of the package
use better language for retries and log retry attempt count

Also, I noticed we don't ignore files such as the script output, logs, and shelves, so I've added those to .gitignore
@datarobot/predictions

Coverage increased (+0.1%) to 83.062% when pulling 9786a707c744dbd77e81e1563e5be69e70836baf on roman/PRED-1395 into a6b3884e89af0de31fcb55c8fbb5957040bce704 on master.

Coverage increased (+0.07%) to 83.006% when pulling f0e21ccceabdf15f3efbd3bf1e1ea5240a3f3ff4 on roman/PRED-1395 into a6b3884e89af0de31fcb55c8fbb5957040bce704 on master.

Add an Enhancements section to the changelog and include this - then I think you're in good shape to ship.

Coverage increased (+0.01%) to 82.95% when pulling 2ac927473f1f5da6f83d6d8a34471339375b850a on roman/PRED-1395 into a6b3884e89af0de31fcb55c8fbb5957040bce704 on master.


Collection: Fix type of interval array, should be string instead of number

Related Issue(s): n/a
Proposed Changes:

Update Collection Temporal Extent interval field definition to be a string rather than a number.  The description and all our examples are RFC3336 strings, but the type was number. Also replaced the confusing term "timestamp" with "datetime" per RFC3336 term.

PR Checklist:

[X] This PR has no breaking changes.
[X] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
[ ] API only: I have run npm run generate-all to update the generated OpenAPI files.

@m-mohr updated the bbox description to be clearer and fixed the examples to be doubly-nested


Update waf-cdn-create-portal.md

clarification of only CDN from Microsoft are supported

@teresayao : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@teresayao : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@teresayao: I'm sorry - only the author of this article, @vhorne, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@vhorne @teresayao Is my edit for grammar correct? I'm not sure I'm interpreting "with" correctly in the original commit--"with" can mean either "that has" or "by using"... Thanks.

@vhorne Glad I asked! Thank you. I've updated the PR with your revision. I'll merge the PR now.


allow multiple deploy conditions and targets

l think I mean:

master -> netlify
devel -> ghpages

or something

similar to #203


Can't push changes to branch

Hi @iamtekson
I cannot seem to push changes to a branch. I am able to create it on the GitHub UI (I just did, it's called sphinx-docs. But when I try to push my changes to it, I get:
21:12:54.285: [geoserver-rest] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain origin refs/heads/sphinx-docs:refs/heads/sphinx-docs --set-upstream
remote: Permission to gicait/geoserver-rest.git denied to iboates.
fatal: unable to access 'https://github.com/gicait/geoserver-rest.git/': The requested URL returned error: 403

Am I maybe missing a permission?

Nevermind, it is because Github has disabled username/password auth for pushing


Feature/linting

PR abstract
Currently the code is "NOT" formatted.
The new github action "code_formatter.yaml" fails because it's not formatted yet.
The followings are the PR changes:

Remove setup.cfg, requirements.txt and MANIFEST.in
Update setup.py (this is required for old envs) and add pyproject.toml
Add a code formatting action and move pycodestyle and flake8 from the default action to the code formatting action
Implement pre-commit
Add a document to format the codes

Known issue
To my understanding, the code will contain circular imports if the code is formatted.
One of the solution is to NOT format some of the codes.
But this means that code_formatter.yaml fails.
Maybe we can add ignored options for these imports.
One of the other solutions are to import when needs.
But some of them are not solved because it's used for the superclass.
Anyway we may need to fix the architecture.

NOT formatted commit is 57c3441.
It failed with a code format action (black and flake8 cause errors).
Formatted commit is b9a8741.
It failed with CI (circular imports).

ありがとうございます！ 循環参照をまず直すのが大事そうな気がしますね… 今日この後進め方を議論できればと思います．


About MOTA calculation, followed the https://github.com/xinshuoweng/AB3DMOT.git version.

Thank you for your response. I have used the suggested code and made modifications based on the following suggestions. I successfully ran the evaluation code, but the MOTA value is still negative.
I removed objects with fewer than 5 radar points (both in the ground truth and model predictions).
I set the confidence threshold to 0.078483.
I merged the labels for bicycles and the person riding them together (following the suggested method).
Additionally, I made a video my video to show the ground truth and model detection results. In the video, the ground truth is displayed as bounding boxes (for visualization purposes only; in the evaluation code, the calculations are still based on radar points), and the model predictions are represented using different colors.
I compared this video with the one you provided RaTrack video and I noticed that in my video, there are a lot of false positives compares to yours, for example, between the 0:14 and 0:31 minute marks,, whereas in your video, between the 0:24 and 0:30 minute marks, there aren’t as many false positives.
Would it be possible to share the detection results used in your video? This could help me pinpoint where the discrepancies are arising.
Thank you for your continued assistance.
my email<EMAIL_ADDRESS>

Download
https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."


communicate to DIO technical resources on the depreciation of the core CSS library

Options
-could create README file in the folder to communicate with teams.
As a: Developer consuming or thinking about consuming the core CSS library
I want: Know that the library is now deprecated
so that: So that I can avoid trying to use it in the future and can fix any dependencies in my app that may exist.
Acceptance criteria

[ ] create a README file that explains to teams that the Core CSS library was depreciated and explains why this course of action was taken
[ ] A statement is authored (could be the same as the content of the README file) is distributed in the following channels:

[ ] Open sprint review on February 10th 2022
[ ] The next developer guild meeting
[ ] The developer guild channel

Presentable outcomes

[ ] This is announced during the next open sprint review

Sprint Ready Checklist

[x] Acceptance criteria defined

[x] Team has defined steps to satisfy acceptance criteria

[x] Acceptance criteria is verifiable / testable

[ ] External / 3rd Party dependencies identified

Related issues:
GitHub issue: [If applicable a "# link" to the GitHub issue]
Resources:
Mockups: [If applicable URL to or the name of the mockup(s) in Figma]
Testing URL: [If applicable URL to the testing branch]

No doing.


[Bug]: single backtick is not recognizable as code block

Is there an existing issue for this?

[X] I have searched for an existing issue

Describe the bug
single backticks are not rendered as code
Steps To Reproduce

create a presentation with this contentAt the command prompt, type `nano`.

check the rendered version

Expected Behavior
according to https://www.markdownguide.org/basic-syntax/#code single backtick should be rendered as code
Actual Behavior
I cannot see any difference in the font

Examples, Code Samples, or Screenshots
No response
Environment
- **OS**: Ubuntu 22.04
- **ownCloud Web**: ownCloud Web UI 8.0.0
- **oCIS**: Infinite Scale 5.0.0-RC4

What browsers are you seeing the problem on?
Firefox, Chrome

There is a difference in the font (it's best seen in letter "a") but I agree that the difference should be clearer. Maybe it's helpful to use a monospace font with serifes similar to the one that is used here in github?


Move to a Lerna project

The idea is to turn this project into a Lerna project and split SVGR into several modules.

[x] @svgr/cli
[x] @svgr/core
[x] @svgr/webpack
[x] @svgr/rollup

As referred in #3880, we should not include prettier dependency in @svgr/core if possible.

v2 is published 🎉


How to get Enum Text in C#

I know there is way to get Enum Text with typescript, but how to get the text with C#.
Serenity.EnumFormatter.format(Quality.Enum.InspectionModel, data.LastInspectionRule.InspectionModel)

I have found it
Serenity.EnumMapper.GetText(approveItem.ResponsibleRole)


Refactor SignallingWebServer to be consumable as an extensible libarary

We want to update the SignallingWebServer scripts so they can be interfaced with as a library.
This will more easily allow users to simply integrate the functionality they need from the library into their own infrastructure, along with the benefits described in the referred to issue above.

Moved from https://github.com/EpicGames/PixelStreamingInfrastructure/issues/10

Behold: https://github.com/EpicGamesExt/PixelStreamingInfrastructure/blob/master/README.md#npm-packages


Names for addreses ??

We need to make an aliases (namees) for addreses

adress book

This has been asked for several times and should definitely be implemented!

This is under discussion, if not actually in progress, for "gen1" release coming soon.


Allow validating env vars without prefix

I'd like to validate env vars coming from .env as well as from the system.
System environment:
EXPORT SENTRY_AUTH_TOKEN=sensitivevalue

.env:
VITE_APP_VAR=publicvalue

vite.config.ts:
export default defineConfig(({ mode, command }) => {
  process.env = {
    ...process.env,
    ...loadEnv(mode, process.cwd(), "VITE_APP_")
  };

  return {
    envPrefix: "VITE_APP_", // Default env prefix
    plugins: [
      // Validate environment variables from .env file prefixed with VITE_APP_
      // that are used in the application and will end up in the bundle
      ValidateEnv({
        validator: "builtin",
        schema: {
          VITE_APP_VAR: Schema.string()
        }
      }),
      [
        // Validate sensitive environment variables from system that doesn't have prefix
        // and should never end up in the bundle
        {
          ...ValidateEnv({
            validator: "builtin",
            schema: {
              SENTRY_AUTH_TOKEN: Schema.string()
            }
          }),
          apply: "build"
        },
        {
          ...sentryVitePlugin({
            authToken: process.env.SENTRY_AUTH_TOKEN // Use sensitive env variable here
          }),
          apply: "build"
        }
      ]
    ]
  };
});

Above won't work because we're only able to validate vars prefixed with envPrefix (VITE_APP_), hence SENTRY_AUTH_TOKEN will always report as missing.
Do you think there's a way to achieve this? Would you accept a PR, and if so, any pointers how you'd like it done?

What do you think of https://github.com/Julien-R44/vite-plugin-validate-env/pull/26 ?
It loads all environment variables, enabling validation, but only exposes variables (through define) if their keys begin with the configured envPrefix.


php(-nts)-xdebug: Update to version 3.1.2-8.1

Updates xdebug to PHP8.1, since php version goes to 8.1 now.

Please create a new branch from master and then make a PR, otherwise it's difficult to review.

Closing this now.

Please create a new branch from master and then make a PR, otherwise it's difficult to review.

Right, thanks for the input, I'll do that.


Gallery Option does not List gallery app

Summary
The gallery option list all documents and and shows images recent images. But it does not list the default gallery app or Google photos app which can have different folders for easy selection of picture.
Code to reproduce
The issue is because the Intent is set to open a document
I was able to list the gallery apps by use the below intent
        val galleryIntent = Intent(Intent.ACTION_GET_CONTENT)
        galleryIntent.addCategory(Intent.CATEGORY_OPENABLE);
        galleryIntent.addFlags(FLAG_GRANT_READ_URI_PERMISSION);
        galleryIntent.addFlags(FLAG_GRANT_WRITE_URI_PERMISSION);
        galleryIntent.setType("image/*")

Android version
Android 11
Impacted devices
All

Why did you close your PR? Did you find another solution?

The gallery app is very important, without this option I will not use this library. Because as a user of the app I'd be very annoyed


hovering effect on images

Description
i would like to create hovering effect on images which will make it more interesting and nice.
kindly assign it to me and also add labels.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@Codewithmeowmeow As of now, we do not need it. We will let you know in the future if there is a need. Please search for another issue and raise it, which I'm happy to assign to you.


Topbar not dropping down

Thanks for viewing.
First attempt with reverie and foundation and cannot get dropdown menu to drop down. Was working originally and can't work what I did to stop it working. Where can I start looking for problems with this function?
Many thanks

Yes I get this problem every install, you have to put a line of code in that sais on hover of the top li the dropdown is visible, not hidden as in the standard setup, a real pain, the drop down should work automatically without tweaking a lot of people use it.
.top-bar-section ul li.has-dropdown:hover .dropdown {   visibility: visible;}
regards G

Thanks Graham!


Gauge amount of work necessary to embed AGR pathway and/or ribbon widgets into AmiGO

We would like to see what compatibility issues there might be in getting some of the new widgets that have been written by GO for external groups embedded into AmiGO, namely the pathway widget and the ribbon.
AmiGO is a very very old custom JS framework and we ran into some issue the first time we tried this (with react?).
The idea would be to gauge how easy it would be to drop some react or web components in there and have them behave properly.
Closing this ticket would be a ballpark estimate of difficulty.
@paaatrick

As discussed at the managers' call; adding for software essentials as it is research for gauging the possibilities of future projects. Noting that this is the research, not the actually final embedding (which we would want more user feedback for).

@tmushayahama Are https://github.com/geneontology/wc-ribbon and https://github.com/geneontology/wc-gocam-viz the correct resources for these two widgets?

@paaatrick For the AmiGO side of things, it might be helpful to get a brief tour from me, either during one of the GO software calls (most Thursdays at 1pm) or as a one-on-one/small group at our convenience.

That sounds good. I'm available during the GO software call time so if that makes one less meeting for you I'm all for it.
(Also hello 👋  from my work GitHub account)

So far I can say that updates to the build process to enable integrating a typical, off-the-shelf Web Component or React component are pretty minimal and straightforward. I have a branch that shows a proof-of-concept with a few run of the mill third party components placed on the gene product details page here: https://github.com/geneontology/amigo/compare/issue-650-component-integration-demo
The key ingredient here is adding the babelify transform to browserify. It allows browserify (via Babel) to transform ES Module syntax (an increasingly common way for components to be distributed) and JSX syntax (although not strictly necessary for integrating React components it makes life easier and it's basically free once Babel is in the toolchain).
I tried using wc-gocam-viz as the Web Component injected into the page by AWebComponentThing.js, but I ran into a bit of trouble. The issue might be in the way that wc-gocam-viz is bundled and distributed. I'll need to take a closer look at that.
cc: @kltm

Talking to @pkalita-lbl , it looks like there might be a couple of strategies to move forward with what he's done so far that would 1) likely work and 2) not take too much time (discounting appropriate data loading for the moment). That said, there may be packaging changes that would make it all more natural, as mentioned above. Next steps would be exploring the cleaner path a little before making a final judgement for the roadmap.

I pushed one more change to the issue-650-component-integration-demo branch which places the wc-gocam-viz widget on the gene product details page. There were two minor hurdles there:

I think version 0.0.49 of wc-gocam-viz was published incorrectly to NPM. The files referenced by the main and module fields of its package.json are missing from the installed package. To work around this I used version 0.0.48 instead. This is probably not a major thing to fix in wc-gocam-viz -- just a matter of documenting or even automating the correct publishing workflow.
In AmiGO (or any client that intends to require or import the package instead of using <script> tags) using the show-legend=true option is problematic because of how the legend image asset is located. It's possible for a client to use Stencil's setAssetPath and to copy the image file from node_modules to somewhere that's served statically. That seems like a lot of overhead for a client though, and I'd say we should look into solutions from the wc-gocam-viz side.

Closing this issue because I think we've met the goal of understanding the level of effort to bring GO widgets into AmiGO.
Given the work that has been done in the branch related to this issue, the amount of work to required to add the pathway and/or ribbon widgets into AmiGO is relatively small. Assuming there are requirements about where to place the widgets (which pages and roughly where within those page layouts), the technical work would be:

Start with the changes in the demo branch and strip out some of the "exploratory" changes
Fetch any required information (i.e. GO-CAM IDs) from Solr on the backend and pass them to the frontend
Make front end changes to pick up required information from Solr; configure and place widgets according to requirements

1 and 3 are fairly small efforts. 2 is a little more unknown to me, but I assume there are some existing patterns to follow.
I should also say that all of my experimenting was done with the wc-gocam-viz widget. I assume that working with the wc-ribbon widget would be roughly similar.


Feature : Added events search (#14)

Fixes #14
This PR adds events search feature to the calendar
Changes/Updates

Added SearchHeader component to the project.
Removed directly passing events.json to FullCalendar component. events.json is now manually imported to App.jsx.
Due to manual import, devs are now required to move events.json from root directory to src directory in development. Same has been updated in the Readme.md file.

Features

The search works once the user clicks the search button after passing a search string in the input.
Search feature checks for availability of search string in all the event's title and description.
When search is in use, only the events matching the search string are displayed throughout the calendar.
If the search string doesn't match any any string, no events are displayed in the calendar.
Once the search box is cleared, the calendar resets and loads all available events.

Screenshots

Calendar

Calendar (when searched for an event)

Dark Mode

Mobile view

Suggestions and improvements
Since this is the first iteration of search in the project it might not be perfect, thus any improvements and suggestions are very much appreciated

Hi @gautamjuyal looks and works great for a 1st iteration! I think we can start with this. Would like an additional review from @maoo and/or @mimiflynn if possible. But I would be happy to roll this out and then we can work on improvements.

Sounds good. Thanks for the review!

Hey @gautamjuyal I had to revert this as it broke the build. It cannot resolve ./events.json probably because the file is added to the dist folder and not the root. See https://github.com/finos/calendar/actions/runs/7462231946/job/20304079088#step:4:20.

@TheJuanAndOnly99 I had my doubts around this. Do you think changing the import to this should work?
import events.json from './dist/events.json

We can try but in the action we would have to do npm run getEvents  before doing npm run build


chore: remove the outdated cloudinary from this repo

Purpose
A while back we replaced the "original" Cloudinary with a new improved version that supported external references (i.e. "Cloudinary 2". At the time we kept the original Cloudinary around, in case we needed to fallback to it: https://github.com/contentful/marketplace-partner-apps/pull/262
That time has long since passed and we can now safely remove the old Cloudinary and keep Cloudinary 2.
Approach

Remove the old cloudinary
Follow up PR will rename (just separating to make things a bit less confusing)

Testing steps

Breaking Changes

Dependencies and/or References

Deployment

@andipaetzold TY good catch! 😅


Misleading tagline

Right now (30-4-2017) when one gets on training.plone.org below the "Plone trainings" header there is a line saying:
Plone training events are a great way to expand your skills. Plus it's a excellent way to meet the community.
This gives the feeling that if it's you are not in a training event you should not look at the training materials listed below.
Is that the feeling we want to give to random developers that happen to come across it? Like our GSOC students?

The claim has since then changed to A collection of trainings developed and created by the Plone Community. 🎉


Add support for REDUCE

Hi all
I was trying to translate a Cypher statement that contained the REDUCE instruction, but the parse method fails saying that the REDUCE expression is not supported. Any change this get added?
The statement can be found here: https://neo4j.com/graphgist/movie-recommendations-with-k-nearest-neighbors-and-cosine-similarity
The need to conversion risen because I'm trying to implement that k-NN logic and cosine similarity on Azure Cosmos DB.

Hello @GiancarloLelli,
thank you for suggestion.
Currently, it is not planned for implementation, but contributions are always welcome.

Hi @dwitry
I would be willing to help, but I need some hints :)

Hello @GiancarloLelli ,
Great to know!
For high level overview see:

Implementation
Development
How to contribute

I've also just created new page How to implement new Cypher feature?.
Feel free to ask more specific questions in this issue, I will be happy to assist when I have time available.


Upload of images for team members, should allow cropping

Using one of the cropping Javascript libraries to crop the uploaded image before it actually is stored in the database, giving some control of the image upload and not just large files.

Being handled under the dev-cropper branch.


Missing device configuration: Assa Abloy YRD-ZWM-1 Door Lock

Checklist

[X] It is not in the configuration DB

[X] It was not merged recently or has a pending PR

Which device is missing?
Assa Abloy YRD-ZWM-1
Manufacturer ID
0x129
Product Type
0xc002
Product ID
0x0601
Firmware Version
1.78
Is the device listed on the Z-Wave Alliance website?
No response
Do you have a manual?
No response

This is the document for a similar, but different module by the same manufacturer if it helps
YaleRealLivingAssureSL_Z-WavePlus_SystemIntegratorsGuide.pdf

The Yale locks are pretty similar, but their configs differ in small details, so it would be important to have the correct manual for this one.

Also...
@zwave-js-bot config help wanted

This appears to be the manual from the actual lock I'm using. Although this lock is branded Yale and mine is Lockwood, it is otherwise identical
YaleRealLivingAssureSL_Z-WavePlus_SystemIntegratorsGuide (1).pdf

I've taken a look at this as part of a recent merge of all Yale locks, but I can find no information on this device. If OP wants it merged they're going to have to do some investigating/testing.


Fix parameterised $NETWORK_NAME in interfaces-setup.sh

The ip-dhcp-host command still hard codes the network name to default. We should use the $NETWORK_NAME variable.

/recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

All the Zuul jobs have passed independently over many attempted batch runs.  Let's override to stop rolling the dice in the hopes for a perfect run with no hiccups.

/override rdoproject.org/github-check

All the Zuul jobs have passed independently over many attempted batch runs. Let's override to stop rolling the dice in the hopes for a perfect run with no hiccups.

There was kind of an agreement to not use override and instead fix the jobs. I think we have no other option but to revert https://github.com/openstack-k8s-operators/edpm-image-builder/pull/19 to make the baremetal job stable.

recheck

/approve


WFLY-18267: Upgrade HAL to 3.6.11.Final

https://issues.redhat.com/browse/WFLY-18267

Thanks @hpehl


Automatic commit: Move 'spa-dox-create-decision' from QA to Production

Also, please resolve conflicts.


Update to latest Angular 13 & Remove unused dependencies

References

Fixes a large number of recent security alerts from dependabot (at least 10+ by my count) by updating our existing dependencies

Description
NOTE: While this PR looks large, it's almost entirely the yarn.lock file which has changed. Other changes are very minor & easy to code review.
This PR cleans up our dependencies as follows:

Updates us to the latest Angular 13 release (to patch Angular for bugs/security issues). We've already been running Angular 13 so this is a minor upgrade
Updates other dependencies to most recent patched versions which are compatible with Angular 13.  Some key updates are:

Updates to Sass 1.33.0 -- required to upgrade ngx-ui-switch to v13.  This required minor fixes to a few of our Sass stylesheets.
Updates to latest Bootstrap 4 release (to support Sass 1.33.0)

Removal of a large number of unused dependencies.   Unused dependencies were determined by running npx depcheck (https://www.npmjs.com/package/depcheck) and then double checking each result by searching if they are called by our codebase.  Any which seem 100% unused were removed.

Instructions for Reviewers

Verify all tests pass
Pull down this PR & verify no changes to build/install process.  Spin up the UI and verify that nothing seems broken.
NOTE: the new version of ngx-scroll-to has strict requirements for which versions of yarn & node it supports.  Therefore, if you are using an older version of either, you may get an error that says The engine "node" is incompatible with this module.  If you see that, simply use yarn install --ignore-engines to ignore the error (or upgrade your Node or Yarn, alternatively).

@artlowel : Would you or someone on your team be willing to take a look at this one?  It's easy to code review & mostly just needs more testers.  It's important though as it fixes a lot of dependabot notified security vulnerabilities (by just updating our dependencies in minor ways).

NOTE: This appears to currently have a few e2e test failures which are unrelated to these changes (they are accessibility failures). I'll get those fixed on Monday as they seem to be minor accessibility checks failing.

Thanks @artlowel !  Merging this immediately so that we can see the full impact on the dependabot security notices.  If we notice any oddities we can clean them up as needed in follow-up PRs


Issue with HP models not getting Known Models selected in console.

Hi All
I am on the latest version 6.4.9 (hotfix) and I notice that previously I had Hewlett-Packard as manufacturer for all HP models and the new version shows only HP as manufacturer. I re-downloaded all drivers into SCCM so they now show as HP for Manufacturer, but I had to manually pick all the models that we use. None of the known HP devices are being picked as known models like the Lenovo devices. Am I missing a setting somewhere? Any help would be appreciated. Thanks in advance.
Rich

I have the same issue HP models not auto selecting in version 6.4.9. Also tried the BETA 6.5.0 and same results

I have the same issue HP models not auto selecting in version 6.4.9. Also tried the BETA 6.5.0 and same results

I got mine working again. I had to remove all the HP drivers that were in SCCM and re-download them via the tool. It is now picking up the HP models. Also the manufacture name changed from Hewlett Packard to HP. Try deleting the driver packages in SCCM and re-downloading them in the Driver tool.

Had to re-add all the driver packages back into sccm and the tool picked them up again.


Support whitespaces and fuzzing when resuming searches

I have been using the resume search functionality very happily over the last months 🎉 Now I have finally gotten around to fixing two minor related annoyances.

Resume search did not work when the resumed query contained a whitespace.
Fuzzing was not applied to a resumed query.

This fixes both. It works nicely in my local testing. Finally I can resume my function bar searches 😀
Happy holidays everyone ❄ ☃

Great, thanks so much! Happy holidays :)


Enakhra's Lament - Bugs out after left arm door

Trying to re-sync quest journal or logging in/out doesn't help either.

Experiencing the same issue, I queued the door before the cutscene, consuming the limbs. The plugin doesn't sync after the end of the cutscene. Had to go up a floor for it to sync correctly.

Should be resolved with https://github.com/Zoinkwiz/quest-helper/commit/6c03404378139db84d084db84d16960d91d6cb59


Rename WaylandInputDevice to WaylandSeat

The name of this class seems a bit off. It's not a single input device:
it's a collection of related input devices tied to a particular wl_seat.

LGTM Merged


try catch syntax

Hi team,
I am trying to execute a script which has try & catch blocks.
try {
api https://api.github.com/
result = api_result
echo result
}
catch(e) {echo e}
above snippet is my script.
While executing script I am getting the below errors.
ERROR - number of step { does not tally with with }
ERROR - automation aborted due to above  //due to above step it is failing.
Can anyone help me
to run a script with try and catch block.

Hi, the issue is that TagUI will realise you are trying to use Javascript code at the try{ line in your code. However, it will not recognise that the closing } is part of the Javascript code. The main issue here is that once TagUI detects the end of Javascript code, it will pass the code as a 'block' for casperjs to execute. You can use js begin and js finish to wrap around your code, but you will not be able to run TagUI commands in TagUI script within those blocks.
To change TagUI to handle such cases is likely not to be on our agenda, since it is very difficult to make sure we can have the produced Javascript code be correct, as well as behave as expected when executed by casperjs for a try block with both javascript and tagui code intermixed. You can still use try/catch with Javascript, but the code within must be fully written in Javascript to work correctly.
Also as a sidenote, the original motivation behind not having error catching and resolving in TagUI was that if an automation script encounters an error, it is better to abort and warn about the issue, than to continue and potentially take actions that are unintended (like clicking in and filling in wrong information on a form, etc.).

Hi @Aussiroth, I also have a similar kind of requirement where i need to catch the exception.  In real time scenarios, when there is an error/exception, we need to perform some compensation job (reverting the steps performed till then etc.), not only simply exit the flow. To achieve this kind of behavior we should be able to catch the exception so that developer will implement the compensation job in catch block.
I understand that we can still use try/catch with Javascript, but we can't use TagUI steps in that try/catch block. Can you suggest is there any workaround with which we can achieve try/catch functionality?
Thanks,
Ramu

Closing this for now, but adding on, you can try the following to make it easy for such situations, where flow_present is a normal TagUI script with the steps to be done if the element is present, and flow_missing is the script of steps to be done if the element is missing. For some complex use cases, this can make the main script more readable and easier to maintain.
if present('identifier')
{
    tagui flow_present
}
else
{
    tagui flow_missing
}

if present('identifier')
    tagui flow_present
else
    tagui flow_missing


sver can't work with a path with trailing separator

sver can't work properly with the followings cases.
I fixed it with removing it. This is my first time writing Rust. I'm not sure this PR is good in Rust..
sver list can't list path with the separator at the end
# file structure
- lib1
  - sver.toml
- lib2
  - test2.txt

# lib1/sver.toml
[default]
dependencies = [
     "lib2/",
]

% sver list lib1
lib1/sver.toml # lib2/test2.txt should be included!

sver validate reports that the test data of https://github.com/mitoma/sver/issues/62 is invalid
% sver validate
[NG]	lib/sver.toml:[default]    # This should be valid!
		invalid_dependency:["lib/:prof1", "lib/:prof2"]
		invalid_exclude:[]
[OK]	lib/sver.toml:[prof1]
[OK]	lib/sver.toml:[prof2]

Sorry. Previous review comment was not enough.
It seems that cli args and sver.toml must be parsed according to different rules.
Please wait while I sort it out.

@mitoma
Thank you for your review and noticing other problems.
I'll check them.
Cloud you tell me one edge case?
If the expression is ``(empty string), how does it work?
It is valid(current directory) for cli args and invalid for sver.toml?
Or it is valid for sver.toml too?

I think the case goes like this.

expression
cli args
sver.toml

"" (empty)
invalid
valid ( ≒ :default )

@kna
I fixed it another way. Can you please confirm?
https://github.com/mitoma/sver/tree/path_with_separator_at_end_2

@mitoma

https://github.com/mitoma/sver/tree/path_with_separator_at_end_2

It is perfect for me! Thank you. I'll close this PR.
By the way, sver works like this on https://github.com/mitoma/sver/tree/path_with_separator_at_end_2

expression
cli args
sver.toml

//
invalid
valid

(reason: trim_end_matches removes matched patterns repeatedly)
I think it's okay to be an error if you deal strictly with it.
How do you think?

Thank you for your confirmation.
Multiple trailing slashes are removed on Linux, so I think this behavior is natural.
$ ls /tmp | sha256sum
a78ffd5c1ea0c6cd7812e5ddab3bb0f86df0fbf863fbf502df4e17ea0029688c  -
$ ls /tmp/ | sha256sum
a78ffd5c1ea0c6cd7812e5ddab3bb0f86df0fbf863fbf502df4e17ea0029688c  -
$ ls /tmp////// | sha256sum
a78ffd5c1ea0c6cd7812e5ddab3bb0f86df0fbf863fbf502df4e17ea0029688c  -

@mitoma
I see. I agree with you!


time and date plugins can not click to select a certain selection in android brower?

I has tried in different devices; it doesn't work!
please fix it !
thanks

Calendar is deprecated now and a new version is about to be released based on inline-calendar.


[CP][beta channel] Please merge b225582593e3a6603abc06de23da8aab35171ddd into the recent 2.14 beta branch

commit(s) to merge: b225582593e3a6603abc06de23da8aab35171ddd
merge instructions: clean merge into the dev branch at version 2.14.0-377.0.dev, a beta branch at this point is not yet available in order to verify the clean merge
What is the issue: A new feature for streaming profiler sample buffers was added in c9e322dd43dc5a6911aab92aa79e23b6eafd14e, Subsequent tests have revealed some racy logic in the eviction code and makes use of the profiler functionality unstable. The change above turns off the streaming feature.
What is the fix: This change turns off streaming of profiler sample buffers.
Why cherrypick: The new feature of streaming profiler sample buffers has some racy logic in the eviction code and this leads to the profiler functionality being unstable. The profiler is used by a lots of flutter developers and flakiness in this code would impact all these developers.
Risk: Low, turns off a feature
Link to original  issue(s): https://github.com/dart-lang/sdk/issues/46825
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @bkonyi

lgtm

Merged to beta 2.14.0-377.4.beta (efcdca04aac66e37d579d179866189385e63cd9a).


Add building of release/test builds

Add launch.json configs for building release executables and running tests
Did a quick check on Linux and it isn't work just at the moment

It's also curious that on Windows, if I debug in release mode, I do see the breakpoints getting hit. This doesn't work when compiling in debug mode. The variable names are mangled though.

I've checked that the .xctest file is present after compilation, but it's not an executable.

Great so windows do it differently to linux. I guess we'll have to find out how we can trigger tests correctly on Windows. Temporarily I'll disable.


LUA Stack when loading  US Flatlands Map 4x <IP_ADDRESS>

Hi,
i have only the map and ContractBoost in the mod drawer and did not load macDon Pack.
LUA stack while first time loading, it never reaches 100% and stalls at "loading vehicles"
2024-12-16 16:20 -- ContractBoost :: loadMapData
2024-12-16 16:20 -- ContractBoost:MissionBalance :: settings updated.
2024-12-16 16:20 -- ContractBoost:MissionBalance :: scaleMissionReward
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: rewardFactor:1.5
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 1: baleMission | updated 2200 => 3500
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 2: baleWrapMission | updated 300 => 500
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 3: plowMission | updated 2800 => 4200
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 4: cultivateMission | updated 2300 => 3450
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 5: sowMission | updated 2000 => 3000
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 6: harvestMission | updated 2500 => 3750
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 7: hoeMission | updated 1500 => 2250
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 8: weedMission | updated 2000 => 3000
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 9: herbicideMission | updated 1500 => 2250
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 10: fertilizeMission | updated 1500 => 2250
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 11: mowMission | updated 2500 => 3750
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 12: tedderMission | updated 1500 => 2250
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 13: stonePickMission | updated 2200 => 3300
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 14: deadwoodMission | skipped, not found on map
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 15: treeTransportMission | skipped, not found on map
2024-12-16 16:20 ---- ContractBoost:MissionBalance :: Mission 16: destructibleRockMission | skipped, not found on map
2024-12-16 16:20 -- ContractBoost:MissionBalance :: scaleMissionReward complete.
2024-12-16 16:20 -- ContractBoost:MissionBorrow :: fillMissionTools
2024-12-16 16:20 Error: Running LUA method 'update'.
G:/LS25/Modordner/US-Flatlands-4x/FS25_ContractBoost/scripts/MissionBorrow.lua:45: invalid argument #1 to 'pairs' (table expected, got nil)

Please let me know if you need the full log :-)
Map DL: https://forbidden-mods.de/filebase/file/57477-us-flatlands-map-4x/

@matze0301 thank you for taking the time to submit a report!
One missing piece from the full log that is critical for me to help you - what version of Contract Boost are you running? (this sounds similar to an issue that was already reported with version <IP_ADDRESS>, and fixed in <IP_ADDRESS> which was finally released on ModHub.

not a bug
the problem here is that map maker in their hasty conversion has disabled ALL MISSION VEHICLES, which is not in any way a problem of the Contract Boost mod.
you can easily fix it if you want, by unzipping that map, opening the map/map.xml file, and finding this block:
<!--<missions vehicleFilename="$dataS/missionVehicles.xml">
        <deadwoodMission treeType="DEADWOOD">
            <spots filename="map/missions/deadwoodSpots.i3d" />
        </deadwoodMission>
        <treeTransportMission treeType="TRANSPORT">
            <spots filename="map/missions/transportSpots.i3d" />
        </treeTransportMission>
        <destructibleRockMission>
            <spots filename="map/missions/destructibleRocksSpots.i3d" />
            <marker filename="$data/shared/assets/marker/rockMissionFlag.i3d" />
        </destructibleRockMission>
    </missions>-->

and updating it to look like this:
<missions vehicleFilename="$dataS/missionVehicles.xml" />
<!--        <deadwoodMission treeType="DEADWOOD">
            <spots filename="map/missions/deadwoodSpots.i3d" />
        </deadwoodMission>
        <treeTransportMission treeType="TRANSPORT">
            <spots filename="map/missions/transportSpots.i3d" />
        </treeTransportMission>
        <destructibleRockMission>
            <spots filename="map/missions/destructibleRocksSpots.i3d" />
            <marker filename="$data/shared/assets/marker/rockMissionFlag.i3d" />
        </destructibleRockMission>
    </missions>-->

then zipping it back up, which will fix the root issue (the lack of contract vehicles).
That said, I've also pushed / released <IP_ADDRESS> which should prevent against incorrect map configurations causing an issue with the Contract Boost mod.

Wow, thank you for finding the problem and providing a solution :-)


Fitting for X' and X" fails

Elements of the v, Xp and Xpp can be NaNs, and lmfit does not handle this

Implemented search in "fit_Xp_Xpp_standalone" to check for NaNs and remove them from the arrays before the fitting commences.

Implemented search in "fit_Xp_Xpp_standalone" to check for NaNs and remove them from the arrays before the fitting commences.

Changed the solution to remove rows with NaN-values already at loading the data

Changed the solution to remove rows with NaN-values already at loading the data


fix: Image, video, and file transfer is not possible in open channels

fix: accepting parameter type to Array or File in the useFileUploadCallback

@liamcho Sorry, I missed the target repo,, pls approve it one more time 🙏


sslv3, tls 1.0 and tls 1.1 disable using ansible

I am finding it difficult to find the module to disable sslv3, tls 1.0 and tls 1.1 settings on the ssl vserver.

Hi @pavankalyanvskp , can you please elaborate the issue you are facing?

Unfortunately the ssl configuration options are not supported in the current implementation of citrix_adc_lb_vserver and citrix_adc_cs_vserver.
We will be adding this functionality when we revisit the implementation of these modules.
For now there are two solutions to the problem of enabling/disabling the ssl option for a target ssl vserver.

Use citrix_adc_nitro_request with the sslvserver NITRO object to set the appropriate options.
Use the standard Ansible shell module along with the ssh_citrix_adc connection plugin to issue the appropriate nscli commands

I will detail both solutions.
Using citrix_adc_nitro_request
We will be updating the appropriate attributes of the sslvserver NITRO object.
The full documentation for this object can be found here
First we need to identify the exact attribute name we need.
This can be done by going over the documentation or using the following playbook, which will print all the attributes that can be retrieved from the target ssl vserver.

- hosts: citrix_adc
  gather_facts: False

  tasks:

    - name: Get ssl vserver
      delegate_to: localhost
      register: get_result
      citrix_adc_nitro_request:
        nsip: "{{ nsip }}"
        nitro_user: "{{ nitro_user }}"
        nitro_pass: "{{ nitro_pass }}"

        operation: get

        resource: sslvserver
        name: 'test_ssl_cs_vserver'

    - name: Show sslvserver NITRO object
      debug:
        msg: "{{ get_result.nitro_object }}"

The output of the debug task will be something like the following
TASK [Show sslvserver NITRO object] ***************************************************************************************************************************
ok: [citrix_adc_vpx121] => {
    "msg": [
        {
            "ca": false,
            "cipherredirect": "DISABLED",
            "cleartextport": 0,
            "clientauth": "DISABLED",
            "dh": "DISABLED",
            "dhcount": "0",
            "dhekeyexchangewithpsk": "NO",
            "dhkeyexpsizelimit": "DISABLED",
            "dtlsflag": false,
            "ersa": "ENABLED",
            "ersacount": "0",
            "hsts": "DISABLED",
            "includesubdomains": "NO",
            "invoke": false,
            "maxage": "0",
            "nonfipsciphers": "DISABLED",
            "ocspstapling": "DISABLED",
            "polinherit": "0",
            "priority": "0",
            "pushenctrigger": "Always",
            "redirectportrewrite": "DISABLED",
            "sendclosenotify": "YES",
            "service": "0",
            "sessreuse": "ENABLED",
            "sesstimeout": "120",
            "skipcaname": false,
            "snicert": false,
            "snienable": "DISABLED",
            "ssl2": "DISABLED",
            "ssl3": "ENABLED",
            "sslredirect": "DISABLED",
            "sslv2redirect": "DISABLED",
            "strictsigdigestcheck": "DISABLED",
            "tls1": "DISABLED",
            "tls11": "DISABLED",
            "tls12": "ENABLED",
            "tls13": "DISABLED",
            "tls13sessionticketsperauthcontext": "1",
            "vservername": "test_ssl_cs_vserver",
            "zerorttearlydata": "DISABLED"
        }
    ]
}

Then it is a matter of identifying the attribute we want changed and using the citrix_adc_nitro_request module to update this particular attribute or set of attributes we are interested in.
For example if we wanted to change just the tls1 attribute we would use the following playbook.
    - name: Update ssl cert
      delegate_to: localhost
      citrix_adc_nitro_request:
        nsip: "{{ nsip }}"
        nitro_user: "{{ nitro_user }}"
        nitro_pass: "{{ nitro_pass }}"

        operation: update

        resource: sslvserver
        name: 'test_ssl_cs_vserver'
        attributes:
          vservername: "test_ssl_cs_vserver"
          tls1: "DISABLED"

Using the standard Ansible shell module and ssh_citrix_adc connection plugin to issue nscli commands.
This method is very simple. It just requires us to issue the nscli commands through the playbook.
Please refer to the connection plugin documentation to make sure your environment is set up to correctly for the plugin to work.
For setting an sslvserver option the following playbook could be used
- hosts: citrix_adc
  remote_user: nsroot
  connection: ssh_citrix_adc
  gather_facts: False
  vars:
    ansible_python_interpreter: /var/python/bin/python

  tasks:
    - name: Set ssl vserver option
      shell: nscli -s -U :nsroot:nsroot "set ssl vserver test_ssl_cs_vserver -tls1 ENABLED"
      register: nscli_output

    - name: Show nscli output
      debug:
        msg: "{{ nscli_output }}"

For setting multiple ssl options multiple shell tasks must be used.
I hope this helps you out. If you have any further issues please re raise the issue.

can someone please help me with an ansible playbook module to disable tls 1.0, 1.1 and Enable tls 1.2 on CentOS. Thank you


Update README.md with Telemetry page

Updated readme to link the Telemetry page in the documentation

thanks @maryamsulemani97


landing page: fix bug where get-link button doesnt copy

Related to #1318

Copy on get-link now works for Chrome, but Safari still has the same issues likely because of async issues due to the creation of the link...

Created #1324 to fix the issue in Safari


Unable to assign vlan group to prefix

Issue type
[ ] Feature request
[x] Bug report
[ ] Documentation

Environment

Python version:  3.5.3
NetBox version:  2.2.2

Description
Recently upgraded from 2.0.7 to 2.2.2.
After upgrading, I can no longer set a vlan group on a prefix. I can select the appropriate vlan group but this doesn't persist when i click 'Update'

This means I'm unable to associate that IP prefix with a vlan. I've tried disassociating existing prefixes from a vlan, and am then unable to add them.

Hello,
It seems to be a problem with django-filter 1.1.0. Downgrade to version 1.0.4 fix the problem for me.

I'm having a similar issue.  I downgraded django-filter to 1.0.4 and it doesn't help.  Unsure if these issues are related, so I figured I'd tack on to this before opening a new issue.

After upgrading, I can no longer set a vlan group on a prefix.

NetBox has never supported assigning a prefix to a VLAN group. That wouldn't make sense. A prefix can only be assigned to a single VLAN. The VLAN group field is only there to help choose a VLAN.

@jeremystretch Apologies, my description of the problem was incorrect. The net result is the same, though - I was unable to select a vlan and associate it with a prefix.
@maximebrun thank you, downgrading django_filters ("pip3 install django_filter==1.0.4") resolved the problem for me.

Back on django_filters 1.1.0, as this problem appears resolved in version 2.2.4 #1671  - see #1649


BitmapData.draw() doesn't work properly with mask in HTML5

New stencil buffer mask works well for me but seems like it's not taken into account during BitmapData.draw call.
Here is a simple example:
//addChild(holder);

var rect:Sprite = new Sprite();
rect.graphics.beginFill(0x000000);
rect.graphics.drawRect(0, 0, 400, 300);
rect.graphics.endFill();

var circle:Sprite = new Sprite();
circle.graphics.beginFill(0x00ffcc);
circle.graphics.drawCircle(0, 0, 100);
circle.graphics.endFill();

circle.x = 150;
circle.y = 150;
rect.mask = circle;
holder.addChild(rect);
holder.addChild(circle);

bitmapData = new BitmapData(Std.int(holder.width), Std.int(holder.height));
bitmapData.draw(holder);

var bmp:Bitmap = new Bitmap(bitmapData);
addChild(bmp);

Flash output:

HTML5 output:

Using latest versions of openfl 6.5.0 and lime 5.8.2
BitmapDataIssue.zip

Hi does this maybe fixes your issue ?
https://github.com/openfl/openfl/pull/1770

@NenadBojkovski no it doesn't fix it. i've just installed the latest version of openfl with your pull request merged and i have the same problem with html5.

I've noticed that during the bitmapData.draw some graphics commands are missing from renderMask() in CanvasGraphics.hx
https://github.com/openfl/openfl/blob/656a5c2352ab40444ebddf422ff73651f00ed440/src/openfl/_internal/renderer/canvas/CanvasGraphics.hx#L1462
Maybe this could be the problem.

With the solution from #1831 I'm getting this result :

Joshua has fixed the issue in 8.0.1 release


Asm sept explore

Hello @aarmey! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file lineage/Viterbi.py:

Line 100:80: E501 line too long (110 > 79 characters)

In the file lineage/tHMM.py:

Line 114:80: E501 line too long (110 > 79 characters)


Adding pagination logic for fetching users in GET /users/search/

Issue Description
In the https://dashboard.realdevsquad.com/users/discord/ all the users are loaded at once and there is no pagination logic
Expected Behavior

Only some users should be fetched depending on how many users can be loaded before scrolled.
Current Behavior

All the users are being fetched at once as soon as the page is loaded
Screenshots

NA
Reproducibility

[x] This issue is reproducible
[ ] This issue is not reproducible

Steps to Reproduce

Go to https://dashboard.realdevsquad.com/users/discord/

Severity/Priority

[ ] Critical
[ ] High
[ ] Medium
[x] Low

Additional Information

Checklist

[x] I have read and followed the project's code of conduct.
[x] I have searched for similar issues before creating this one.
[x] I have provided all the necessary information to understand and reproduce the issue.
[x] I am willing to contribute to the resolution of this issue.

Is this issue taken?
I'd love to work on this


Fix a very minor race condition in cargo fix.

There is a very rare race where a "fix" message was getting posted after Message::Finish, and thus being dropped without being printed. This is caused by the diagnostic server disconnecting the client before posting Message::FixDiagnostic. The solution is to keep the client on the line until after the message is posted.
Fixes some errors in fixes_missing_ampersand seen in CI:
https://travis-ci.org/rust-lang/cargo/jobs/474384359
https://travis-ci.org/rust-lang/cargo/jobs/429809800
I also moved the done check to the beginning of the loop because every time the server was shut down it was needlessly trying to parse an empty string (and failing).

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

Should this have some kind of test? Not that I know how to make one...

r=me, after the test chat.

To make sure I understand the race here, the problem is that we closed the client connection too early, which allow the client (the cargo fix process) to continue and finish. This in turn then finished the entire compilation queue, posting a Message::Finish. All of that happened before the server thread was able to post FixDiagnostic, causing a message to not get printed. The solution here is to block the client thread until we've actually sent the FixDiagnostic message along its way into a queue?
If that's all the case I think it's fine to avoid a test here, spurious failures are showing we're already covering this :)

The test is fixes_missing_ampersand.  I can't think of a way to force the race to happen naturally, and it's fairly rare.  I ran it for several hours in a loop on my machines, and on CI, and it seems to be fixed.  You can simulate the error with this PR and the following:
diff --git a/src/cargo/util/diagnostic_server.rs b/src/cargo/util/diagnostic_server.rs
index e7e1aff1..39140504 100644
--- a/src/cargo/util/diagnostic_server.rs
+++ b/src/cargo/util/diagnostic_server.rs
@@ -266,6 +266,8 @@ impl RustfixDiagnosticServer {
             if let Err(e) = client.read_to_string(&mut s) {
                 warn!("diagnostic server failed to read: {}", e);
             } else {
+                drop(client);
+                std::thread::sleep(std::time::Duration::from_millis(1000));
                 match serde_json::from_str(&s) {
                     Ok(message) => on_message(message),
                     Err(e) => warn!("invalid diagnostics message: {}", e),
@@ -274,7 +276,6 @@ impl RustfixDiagnosticServer {
             // The client should be kept alive until after `on_message` is
             // called to ensure that the client doesn't exit too soon (and
             // Message::Finish getting posted before Message::FixDiagnostic).
-            drop(client);
         }
     }
 }

and run cargo test --test testsuite -- fixes_missing_ampersand which will give a 100% failure rate.

I understand the race here

Yes, everything you said is correct.

@bors: r+

:pushpin: Commit 7584dced280f09fa30d7a101a145b09854f90832 has been approved by alexcrichton

:hourglass: Testing commit 7584dced280f09fa30d7a101a145b09854f90832 with merge d4a04daaa1a85d6bd907f3a7be2f7286672d27ab...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing d4a04daaa1a85d6bd907f3a7be2f7286672d27ab to master...


Retrieve list of CloudWatch Alarms for an instance via PowerShell

Is there a PowerShell cmdlet that will list all configured CW Alarms for an instance?

No. Also, I think this is too specific of a usecase for us to add this  as a custom powershell cmdlet.

The aws console does it, so why would it be to specific??

DescribeAlarms should list all the alarms and you can probably filter it by whether there are associated instances.
AWS Console has that functionality because the console team implemented the feature.

You can probably do something like:
Get-CWAlarm -Region <region> | Where-Object {(ForEach-Object -InputObject $_.Dimensions -Process { $_.Name -eq "InstanceId" -and $_.Value -eq "<instanceid>"}) -contains $true }

That does work, but when you are dealing with a enterprise environment and there are thousands of alarms; this takes a very long time to run. Is there anyway the CW team can expose the describeAlarmsForResourceIds method to AWS services? Or add the ability to filter the describeAlarms method?

@sstevenkang : i tried your command but it gives me no output though i have CW alarm configured on my instances.Any help

@sstevenkang it's been two years, and I gave a very good reason why we would not want to pull all of the alarms down first and then filter locally. It takes a very very long time to pull all of our alarms down for all of our instances in our VPC, to then find the few for a single instance.

@johnwc To provide what you want we need the service to add the feature in their API. I would recommend letting them know about your need via their forums. https://forums.aws.amazon.com/forum.jspa?forumID=138 There backlog is driven based on customer asks and the asks are more impactful if they come directly from the customer.

can some one explore the below cmd:
Get-CWAlarm -Region  | Where-Object {(ForEach-Object -InputObject $.Dimensions -Process { $.Name -eq "InstanceId" -and $_.Value -eq ""}) -contains $true }
Does it works for you ?

@prakashst205  That doesn't look like valid powershell. What are you trying to accomplish with the command?

Yes i found in the earlier script property name is not available either it is namespace .
Actually i having n number of ec2 instances , i wanted to extract all the cloud watch alarm for each instances with the instance id.
I tried get-cwalalarm but it gives alarm name but not instance id which also i m looking for.

@prakashst205 To retrieve all the CW alarms for a single instance, you must pull back ALL CW alarms and then filter on them locally. If you are in a large org, the Get-CWAlarm call may take a while to return all the CW alarms.
$InstanceId = "<your instance id>"
Get-CWAlarm | Where-Object {(ForEach-Object -InputObject $_.Dimensions -Process { $_.Name -eq "InstanceId" -and $_.Value -eq "$InstanceId"}) -contains $true }

@normj I know you mentioned to go over to the forums to ask for this feature, but is there any way that you can forward this issue link to someone over in that area so that can see what we have to do in order to list the CW alarms for just one instance?


Both Cookie Buttons firing the same action?

I am a beginner trying to get the script working but it appears that, using the op-out method from the autogenerated code, pressing either button leads to the same result.
Here is the code I currently have:
(the "true2" message is shown both when pressing the Accept and Reject button)
Could it be that the variable only stores whether a button was pressed, but not which one? Is there a correct one? How do I check the pressed button?
<script<EMAIL_ADDRESS>data-cfasync="false"></script>
<script>
window.cookieconsent.initialise({
  "palette": {
    "popup": {
      "background": "#eb6c44",
      "text": "#ffffff"
    },
    "button": {
      "background": "#f5d948"
    }
  },
  "type": "opt-out",
  "content": {
    "message": "Diese Website verwendet Cookies, um Ihnen die bestmögliche Nutzererfahrung zu bieten.",
    "dismiss": "Cookies erlauben",
    "allow": "Cookies erlauben",
    "deny": "Cookies blockieren",
    "link": "Weitere Informationen",
    "href": "link to my data privacy site"
  },
  onInitialise: function (status) {
 var type = this.options.type;
 var didConsent = this.hasConsented();
 if (type == 'opt-out' && !didConsent) {
  document.cookie = '_ga_opt_out=true; expires= Thu, 31 Dec 2099 23:59:59 UTC; path=/';
  window['ga-disable-UA-XXX-X'] = true;
      alert("true1");
 }
},
onStatusChange: function(status, chosenBefore) {
 var type = this.options.type;
 var didConsent = this.hasConsented();
 if (type == 'opt-out' && !didConsent) {
  document.cookie = '_ga_opt_out=true; expires= Thu, 31 Dec 2099 23:59:59 UTC; path=/';
  window['ga-disable-UA-XXX-X'] = true;
      alert("true2");
 }
},
onRevokeChoice: function() {
 var type = this.options.type;
 if (type == 'opt-out') {
  document.cookie = '_ga_opt_out=false; expires= Thu, 31 Dec 2099 23:59:59 UTC; path=/';
  window['ga-disable-UA-XXX-X'] = false;
    alert("true3");
 }
},

});

</script>

I solved it.
It happened because the "didConsent" does not save the choice of button. It just saves whether any button was pressed.
The documentation of the callback loop should be adjusted accordingly.
You need to check for the "status" flag to make it work, which saves the pressed button.

I solved it.
It happened because the "didConsent" does not save the choice of button. It just saves whether any button was pressed.
The documentation of the callback loop should be adjusted accordingly.
You need to check for the "status" flag to make it work, which saves the pressed button.

Any change you could show us some working code please? Having the same issue! Thanks


thread 'main' panicked at 'range end index 422 out of range for slice of length 421', src/main.rs:228:31 (Client project)

Hi!
I can't execute with success the instruction: cargo run --bin main -- --cluster mainnet
Then compiling I get this error:
thread 'main' panicked at 'range end index 422 out of range for slice of length 421', src/main.rs:228:31
What can be done to fix this error?
Thanks!

Got the same error as well