resolve does not import tags

Detailed description
given 2  specs A,B:
spec A contains
paths:
  /stuff:
    $ref: B.yaml#/paths/~1thingies

spec B conatins
tags:
  - name: ext
    description: description

paths:

  /thingies:
    get:
      tags:
          - ext

speccy resolve A.yaml
includes the thingies path, but it does not take the tag with it's description into the merged file.
Context
in an environment where there is one spec to be presented to users that merges several specs coming from different teams, this feature is vital in the compilation of the one and only spec
Possible implementation
Simply add the tags in the result

The #/paths/~1/thingies fragment defines the section of the referenced document which is 'in scope' and forms the result of the reference (the exception is if there are more $refs to follow in the in-scope section).
The tags array is out-of-scope for the reference and is therefore not seen by the resolver. OAS does not allow direct $refs to tags. The "referencing" of a top-level tag by the operation-level tags array does not work sufficiently like a $ref for this to be included in a JSON Reference resolver.
Additionally to that, Speccy is no longer maintained, so you should probably be looking at an alternative, such as boast from oas-kit or Spectral if you need just a validator/linter.


no quests for immersive petroleum mod

It looks like it has some cool stuff and it has a big multiblock but it has no quest or at last I could not find any

there are quests for immersive petroleum


test gpt2-xl

Goal
Test how many tokens we can fit with gpt2-xl and 32GB V100.
Note:
I changed the main code a bit to limit prefix to half of maximum length.
This seems to break test. So we might not want to merge it.
But I can run some a simple training fine. So it should be ready for the goal of this branch.

Do you want to merge it on master? (to know if you need a review :smile: )

I don't think it's necessary to merge this, at least for now. 🙂

This PR served its purpose. Code is not merged into another branch. Closing this now.


Revisit our config for 'Naming' Cops

This has the effect of turning on two cops:

Naming/AsciiIdentifiers
Naming/FileName

Please see the commits for more details. Trying to enable Naming/PredicateName
uncovered quite a few issues, so I'm going to leave that one for now.

Update: turning on Naming/FileName isn't so great for Smart Answers, which deliberately uses a lot of hyphens in its filenames. That does seem like a special case, and we can always disable this cop for the offending directories in that repo (or remove the hyphen requirement).

Can we enable Naming/AccessorMethodName by default and just disable it for the specific cases you mentioned?

@AlanGabbianelli thanks for taking a look 😃. In your comment, are you meaning to refer to Naming/FileName? I haven't mentioned Naming/AccessorMethodName in this PR, but it's the same problem as Naming/PredicateName - there are quite a few (as in lots!) of issues with it, so I'd like to tackle enabling them/that in a single version upgrade.

Hey @benthorner, no, I mean Naming/AccessorMethodName. In https://github.com/alphagov/rubocop-govuk/commit/c927eab9dcc5db30fa7f6e66989e919387323994 you added an explanation for it as a comment and left the cop disabled.
Instead of that explanation, can we enable this cop by default and only disable it manually for the specific cases you mentioned in the comment such as def set_political_and_government(edition)? In a later version bump would be fine too if they are too many to change in this PR.

@AlanGabbianelli oh, I see, sorry. Personally I don't think it's a particularly valuable Cop, and we shouldn't really enable it if our only plan is to do # rubocop:disable in lots of places, which creates more clutter. My thinking is that the Cop itself is buggy: it should check the method signature to see if what it's asking for is actually appropriate. And even if it's not, using name= vs set_name doesn't actually make much difference to me.
I also had a quick look around our apps: doesn't look like we're using the name= style anywhere; in general, we just use attr_writer or attr_accessor to create simple name= methods.
Still content to revisit, though - I just don't see it as a quick win now, and there is at least some argument for keeping it as it is. We also argue against that argument in the future 😉.


@vue/cli-plugin-typescript: please upgrade dep fork-ts-checker-webpack-plugin to 7.2.x

What problem does this feature solve?
@vue/cli-plugin-typescript is currently on
"fork-ts-checker-webpack-plugin": "^6.4.0",

The latest version is 7.2.3 which fixes some issues. I currently can't upgrade to Vue CLI 5 because of those issues.
What does the proposed API look like?
.

Any update on this? I can build locally and in a local docker container, but I'm getting the "RpcIpcMessagePortClosedError" error that's breaking my circleci pipeline. I haven't figured out what's different that it would succeed in one environment and fail in another, but it seems like this dependency update request would fix it.

Wanted to follow up with what fixed my issue. May not help anyone else, but it was very unintuitive, and I still don't understand why the fix worked. We used circleci for our pipelines and upgrading the resource class to "extra large" solved the problem. (https://circleci.com/product/features/resource-classes/)
I see discussion about memory in this thread, but it didn't seem to be a solution
https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/issues/674
Anyway, hope this helps someone.


Provide authentication

Basic auth like bitcoind would be a good start, but several options could be supported.

Now that requests are CORS enabled by default (#4), we need to make it optional once auth is enabled.


[FEATURE REQ] Reference architectures to build web applications on Azure

Provide reference architectures to help spring developers to better build the Spring MVC application on Azure
References

Build web and mobile applications
Java CI/CD using Jenkins and Azure Web Apps
Serverless web application
Host a RESTful API with CORS in Azure App Service

@chenrujun why is this blocked?

@chenrujun why is this blocked?

@stliu
Originally, you told me to forget the task about reference architecture article, build Grafana dashboard first. I though they are different task. Now I know it's my misunderstanding. I'll change the status from blocked to In Progress.

Closing this issue now for the changes have been merged.


Update SGX documents

This commit mainly simplifies the description about building a
debug and hw mode enclave.
Signed-off-by: Jia Zhang<EMAIL_ADDRESS>

To run the wasm app in the docker image, you can add the following command:
ENTRYPOINT ["/run/rune/wasm_app.aot"]


Add Binary Search in C language

@arpit456jain  @RounakNeogy  @RishikaGhosh  @arpitkatiyar1999  Please check and tell me if any improvement is there in the code.


HI, Your Documentation is not up to date

@chiefapp which part of the documentation need update? Any help is appreciated.


Exec tachikoma update<PHONE_NUMBER>0141

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io

Coverage remained the same at 68.0% when pulling 99f8e042131305b35082cc2b9665091f54adf90c on tachikoma/update-20150410150141 into 0d53cb0f12e0d67b9053cb1732e3358ff0eea435 on master.


Functionality lmited

i do a make build, make run then login as admin.  If I add a project manually that I know to build properly on the public readthedocs it adds the project but the build remains in the "Triggered" state...is celery not functioning?

config/etc/default/rtd-config.py contains a message queue (redis), elastic search and postgres config, however those things are not part of this container.
config/bin/rtd-start.sh does appear to be starting celery as part of this container
If you the container directly, I think you will need to configure those three services on the right names. Does it work for you from docker-compose?

Hi @frodopwns, as suggested by @monkeypants, you should run redis, postgresql and elasticsearch containers
The easiest way is to use docker-compose, I updated the README.md file with newer instructions


Not downloading yet downloads via web link

Provide environment information
which python; python --version; python -m pip list | grep gdown
/usr/bin/python
Python 3.9.2
What OS are you using?
lsb_release -a No LSB modules are available. Distributor ID: Debian Description:    Debian GNU/Linux 11 (bullseye) Release:        11 Codename:       bullseye
Describe the Bug
gdown does not download files stored but the link works and lets you download with web view. Have tried --fuzzy and removing the usp=sharing link to no affect. E.g.
https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing
Last option downloads a microsmall file of no relevance.
gdown --fuzzy https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing
zsh: no matches found: https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing
gdown https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing
zsh: no matches found: https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing
gdown https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
/home/xxx/.local/lib/python2.7/site-packages/gdown/parse_url.py:39: UserWarning: You specified a Google Drive link that is not the correct link to download a file. You might want to try --fuzzy option or the following url: https://drive.google.com/uc?id=None
url="https://drive.google.com/uc?id={}".format(file_id)
Downloading...
From: https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
To: /home/xxx/storage/DB1-ENC/Ingest/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
71.4kB [00:00, 1.16MB/s]
~/storage/DB1-ENC/Ingest/ gdown --fuzzy https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
Downloading...
From: https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
To: /home/xxx/storage/DB1-ENC/Ingest/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
71.4kB [00:00, 1.42MB/s]
Expected Behavior
Video to download, in thie case, 906.9Mb worth. It doesn't.
To Reproduce
See above.

I had problem downloading zip files from links copied from browser GoogleDrive page in form "https://drive.google.com/file/d/{FILEID}/view?usp=sharing"
Then, I tried convert them to direct "download link form" to the form "https://drive.google.com/uc?export=download&id={FILEID}" and it worked
ref: https://www.howtogeek.com/747810/how-to-make-a-direct-download-link-for-google-drive-files/
import gdown
url = "https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing"
new_url = "https://drive.google.com/uc?export=download&id=" + url.split("/")[-2]
gdown.download(new_url, "myfile.mkv"), quiet=False)

@DarrenPIngram It is working for me. Can you double-check?
% gdown --fuzzy 'https://drive.google.com/file/d/1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7/view?usp=sharing'
Downloading...
From: https://drive.google.com/uc?id=1q-I95pr2FhnRWN_byNIKr4jY3YbZqSC7
To: /home/wkentaro/Nothing to Declare (2021).mkv
  6%|██▎                                    | 54.5M/907M [00:12<03:07, 4.54MB/s]

@DarrenPIngram Working for me. Maybe the quotes are missing?
% gdown --fuzzy 'https://drive.google.com/file/d/1nbdVPtrHLSjSTPuEVCGP028BIy5CfLhJ/view?usp=sharing'
Downloading...
From: https://drive.google.com/uc?id=1nbdVPtrHLSjSTPuEVCGP028BIy5CfLhJ
To: /Users/wkentaro/Takeaway Titans - S01E02.mp4
  0%|                                      | 6.82M/2.37G [00:02<15:07, 2.61MB/s]


Ehcache interacts with the Expiry policy to supress firing events for values that immediately expire

It was expected that interaction with cache expiration was a concern for Store implementations.
To satisfy the expectations of jsr107 I needed to add some expiry checks in the cache itself. Specifically to cover the case of new cache values that are immediately expired (see org.ehcache.Ehcache.newValueAlreadyExpired(K, V, V)). When new values are already expired there should be no cache events fired. Since cache events are implemented in the Cache we need to deal with expiry.

Goal of this task is to define if we want to do some changes on this or leave code as is.

Not much can be done right now.
As the cache currently holds the responsibility of events and JSR-107 mandates that no events are fired for immediately expired values, we have no choice but to in advance look at the expiry before letting the store handle that.
However, in the context of #165, there have been discussion about pushing down the complete eventing system down to the store. If that becomes true, then we should indeed get rid of that code as everything could be handled at the store level.
I will thus close this issue and reference it from a comment on #165 to make sure it is taken into account if we go down that path.


copyFromAssets relative path causing issues

I noticed an issue while working with this lib. If loading my app in a nested path, the copyFromAssets command, which runs when the app loads, is failing. It looks like it's trying to load the JSON file from a relative path.
https://github.com/jepiqueau/jeep-sqlite/blob/a7566249cddabddb737281fe0ed4f6c3f7e168e8/src/components/jeep-sqlite/jeep-sqlite.tsx#L1518
I also found #12 which seems like it had the same issue in a different location. I'm guessing doing the same, prefixing the path with /, would resolve this issue.

@rain2o What do you mean by  'If loading my app in a nested path' did you test it by adding a '/' in front

@rain2o this is fixed in 2.5.4


Update containers

Ckan:

Recommendation blocked by proxy-ip and minor layout change.
Improved resource type comparison with resource action buttons.
Added some translations.
Drupal:
Guide page content should be wider and also it should fill all the page when there isn't menu presented

Test needs an update as its looking for wrong text string.


Converts artwork page to palette v3

https://user-images.githubusercontent.com/112297/120709545-f7fcc180-c48a-11eb-9b9d-625c0caca7fe.mov

Post Launch

[ ] Better use space for primary artwork image (optimize images for device)
[ ] Appropriately sized placeholders for images (lazy loading)
[ ] Rebuild view in room
[ ] Rebuild lightbox

This is ready but I'll get a review app of this deployed.

OK. This should be green now and I feel confident but I'm not about to merge this at 6pm on a Friday. So: first thing Monday 🚀


Enhanced the navbar's UI #517

**Resolved ** #517 issue
Changed the **Login Icon **
Added Spacing between navbar elements
Removed the Hovereffect for better UI

Please preview the fixes and merge it

Please preview the fixes and merge it

Please preview the fixes and merge it

fix merge conflicts

navbar is completely inconsistent across pages , kindly follow PR guideline s, fix it across all pages , record a video

navbar is messing up on leading blog page , and also once we login/ sigup

@ANSHIKA-26  ok I will try to enhance all of them

@ANSHIKA-26 , I have made the necessary changes to the Navbar for all the landing pages.
https://github.com/user-attachments/assets/e10581d1-e4d1-4379-8148-40133ece3e59

resolve merge conflicts

change the hover effect to a decent underline effect , not this oval hover

resolve merge conflicts

How can I resolve it

you had a nice underline effect earlier , i wanted that on all pages , why would you change it to an oval hover , it looks too big

Can I enhance that hover effect instead of that underline effect .I will try to make it more user friendly

can u please assist me how can i resolve the conflicts.

this is what i want for hover , you might find the code in some old PR as well

@ANSHIKA-26 ,can u please assist me how can i resolve the conflicts.

im gonna close this PR , please, resync and then make a new PR , keep your code somewhere since you might loose it if you refork as well :)

@Deepak0yadav due to your lack of response , and not completing the PR on time , this issue has been assigned to others , kindly do not work on it anymore


Fix behavior modality in upload manifest

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:
Always adds the behavior modality, instead of checking stream_modalities which had behavior removed as a modality
What issues or discussions does this update address?

resolves #1114
resolves #1123

Describe the expected change in behavior from the perspective of the experimenter

none

Describe any manual update steps for task computers

none

Was this update tested in 446/447?

[x] tested in 447

@XX-Yin This is ready for review, and replaces #1116

@alexpiet
The way to check the modality has two issues:

The video modality will not be added to the manifest file as the video streams will only be added to the session streams when there is an end time detected.
The current code would only generate manifest file in the first trial and could miss modality started after first trial.

I know that you thought the point 2 is not an issue. Please make changes to the point 1 to make the modality check more robust. I suggest we directly check the folder empty for each modality instead of using the session streams.

Why is the video only added to the stream modalities when there is an end time, but the FIP is added before there is an end time?

It's a history issue. You can also change the video modality to be like the FIP
        if hasattr(self, 'fiber_photometry_start_time'):
            Obj['fiber_photometry_start_time'] = self.fiber_photometry_start_time
            if hasattr(self, 'fiber_photometry_end_time'):
                end_time = self.fiber_photometry_end_time
            else:
                end_time = str(datetime.now())
            Obj['fiber_photometry_end_time'] = end_time

@XX-Yin what about this update

@XX-Yin what about this update

This works.

Can you also remove the camera_start_time and fiber_photometry_start_time when starting a session(like I did in https://github.com/AllenNeuralDynamics/dynamic-foraging-task/pull/1116/files#diff-b3cf4ca4fc9cb05d0a854615b8178bb10c1805e102fa1685fd60bd66bec9d34aR4052)?  Otherwise, it may interact with the next session.
And don't delete the fiber_photometry_start_time at here https://github.com/AllenNeuralDynamics/dynamic-foraging-task/pull/1116/files#diff-b3cf4ca4fc9cb05d0a854615b8178bb10c1805e102fa1685fd60bd66bec9d34aL3628. Otherwise, it may not generate the fiber photometry data stream after the new session button is clicked (e.g. after saving).
Can you also make changes in the fiber photometry data stream in the GeneratedMetadata.py to keep consistent with the behavior video data stream?

Those seem like separate issues from the upload manifest modality.

At least the first one is directly associated the manifest file as the code is using the Session Metadata to get the modalities.

But I'm not setting the start time for the camera or photometry in either the metadata or the window object.

If the previous session has the camera_start_time and camera_end_time and the current session doesn't have the video stream and we don't restart the GUI, the video stream will be added to the current session. I think the root issue is from the session metadata, but it will also influence the modality detection in the manifest file.

I just tried it, and that does not happen

Although that may be a result of this bug I just found: https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/1123

I just tried it, and that does not happen

Did you click the start recording (not the preview)?

No, I recorded video through the FIP workflow. I just fixed #1123. I will check in 446 and just record video

Commented on the wrong PR:
Ok, you are right that if you run a session with the camera, and then start a new session without the GUI, then it incorrectly lists behavior-video as a modality in the manifest and in the session metadata. I'll make a separate issue, since I don't have time to fix that right now. Lets merge this fix in, since it will effect every session
here is the issue: https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/1124

@XX-Yin do you have any remaining concerns?

@alexpiet We can first merge this and fix other issues


Is there a good way to collaborate on a dust project when using the front end?

When using the front end, all of the code blocks (I believe) are saved in the sqllite database. This prevents us having a central code repo with all of our code blocks. The only way we have been able to collaborate is by copy and pasting the code from code blocks. Is there a solution for this?

Hi! We're working on introducing the notion of Workspace for teams to collaborate on apps and datasources together.
ETA 1-2 weeks.

We are now ready to roll-out shared team workspaces. Feel free to ping us at<EMAIL_ADDRESS>to get into the preview of that feature :+1:

(if you're using the hosted version, I can help you set that up of course)


SoundNet Supervised Upper Bound

Use an SVM on CNN/MaxPool layer outputs to get the supervised upper bound for each of the datasets.

I was able to train an svm on the same dev set used for ESC-10 and UrbanSound8K and make predicitons. Still need to move the code over to its own file and get a 20 iteration average like we have done for the affinity coding predictions.


Limit game name to 140 characters, inspired by twitter char count (#405)

Added a 140 character limit for the game name with a character counter located to the top right of the text input.

(As ystros suggested, we want to prevent full texts of War & Peace as game titles)
Also added CSS adjustments to keep the text within its container, wrapping accordingly

I'm not sure the character count is overly necessary, but I'm not sure it hurts anything either.  I will implement the limit server side, don't worry about that


Test pipeline

Test

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: OlenaKashuba:x: admmastersYou have signed the CLA already but the status is still pending? Let us recheck it.


Back and forth sweep

Plottr cannot plot sweeps where the dependent variable is swept back and forth in the same measurement

solved in the new version of plottr. It supports data that cannot be gridded easily (which is the case for the back/forth sweep)


Override container runtime for commands

Closes #179

@eemcmullan I also think that this should not cherry-picked because this is introduced in my most recent fix which itself isn't cherry-picked (it is for 0.4)


feat(guides): Create New Definitions Page

Pull Request
Purpose
To add a new guides page containing definitions of common words and terms.
Approach

Create a new guides page containing definitions of common words and terms.
Add this page to the navbar

Open Questions and Pre-Merge TODOs

[x] Complete first pass of definitions

Requirements

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

When this gets closer to being out of draft, if you guys want me to give it a once over let me know.

Perhaps add the following additions

Thanks - had to tweak some of the formatting etc but merged.

When this gets closer to being out of draft, if you guys want me to give it a once over let me know.

Thanks for this - it’s now out of draft so I’d be happy for you to give it a look over.

When this gets closer to being out of draft, if you guys want me to give it a once over let me know.

Thanks for this - it’s now out of draft so I’d be happy for you to give it a look over.

Sure, will try to get to it today.

Going to wait for the above review from @nuxencs to be addressed before I do a pass at this if you guys don't mind, given that two reviews simultaneously can make things complicated.

Going to wait for the above review from @nuxencs to be addressed before I do a pass at this if you guys don't mind, given that two reviews simultaneously can make things complicated.

Think we're good whenever you're ready and willing - the only unresolved questions are the ordering (which I can do at the very end), and the two open comments above.

Going to wait for the above review from @nuxencs to be addressed before I do a pass at this if you guys don't mind, given that two reviews simultaneously can make things complicated.

Think we're good whenever you're ready and willing - the only unresolved questions are the ordering (which I can do at the very end), and the two open comments above.

Sure - the ordering would definitely be easier to accomplish with a force push or something than in a review, so I'll leave that for you since it's your PR.

Sort of a lot, but there's a lot here. If you want to tweak it feel free, and if you want my help with thoughts and adjustments just reply and we can work it out.
Overall, it's not as bad as the 17 suggestions make it seem, but you asked for me to review it and you all know how picky and shit I can be when it comes to grammar :)
Let me know what you think.

Thanks for this - appreciated. Have accepted most, and tweaked a couple.

Sort of a lot, but there's a lot here. If you want to tweak it feel free, and if you want my help with thoughts and adjustments just reply and we can work it out.
Overall, it's not as bad as the 17 suggestions make it seem, but you asked for me to review it and you all know how picky and shit I can be when it comes to grammar :)
Let me know what you think.

Thanks for this - appreciated. Have accepted most, and tweaked a couple.

Of course, I assumed a few might need tweaking.

yeah that looks good to me

I'll go over it again, on a scale from 1-10 how picky do you want me to be? Last time I'd say I was like a 6 or 7....

I'll go over it again, on a scale from 1-10 how picky do you want me to be? Last time I'd say I was like a 6 or 7....

I'm happy with a 6-7 really - if there's nothing really egregious or confusing then I'd prefer to get it merged and we can dial it in as time goes on.

Let me run through it and double-check one last time, Give me 15m

Thanks everyone who contributed to this one!


Unvalidated data passing from validation

Laravel Version: 5.6.12
PHP Version: 7.2.3

Description:
Steps To Reproduce:
Create validation with this rule
'number.*' => 'required|integer',
and post string value without array like this <input name="number" value="string" >
after try to get all validated data from request you will have number field passing as a validated value but it should not pass or there should be a error because it is not integer.

Please add a code example with actual data so that we can reproduce it.

I reproduced with the following, testing for an expected 422 status:
Request
<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class StoreNumber extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'number.*' => 'required|integer',
        ];
    }
}

Controller
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests\StoreNumber;

class TestController extends Controller
{
    public function create(StoreNumber $request)
    {
        return response()->json(['success' => true]);
    }
}

Test
<?php

namespace Tests\Feature;

use Tests\TestCase;
use Illuminate\Foundation\Testing\WithFaker;
use Illuminate\Foundation\Testing\RefreshDatabase;

class StoreNumberTest extends TestCase
{
    /**
     * Testing number validation.
     *
     * @return void
     */
    public function testStoreNumberValidation()
    {
        $response = $this->json('POST', '/numbers', ['number' => '']);

        $response->assertStatus(422);
    }
}

And got this output on terminal:

@jonpemby In the controller get validated data with $request->validated(); and you will see number field is there as a validated data

@taliptako I'm testing for response failure (which is what we want it to do) but we're getting response success (as you've described).
If I var_dump the request->validated() in a manual browser test, I get the following:

Is this what you're experiencing?

@jonpemby Question is why number field show up as a validated data  ?

@anishdcruz: This issue was discussed in #23612.

@staudenmeir i can see that issue was close. Is there any workaround for this issue.

It will be fixed in 5.7: #23708
If you don't have any array rules (foo.*.bar), you can use this workaround:
$data = $request->only(array_keys($rules));

Okay, so after investigating what's going on, it looks like there's some (perhaps) unintentional filtering happening in this method:
Illuminate\Validation\ValidationRuleParser.php
/**
     * Define a set of rules that apply to each element in an array attribute.
     *
     * @param  array  $results
     * @param  string  $attribute
     * @param  string|array  $rules
     * @return array
     */
    protected function explodeWildcardRules($results, $attribute, $rules)
    {
        $pattern = str_replace('\*', '[^\.]*', preg_quote($attribute));

        $data = ValidationData::initializeAndGatherData($attribute, $this->data);

        foreach ($data as $key => $value) {
            if (Str::startsWith($key, $attribute) || (bool) preg_match('/^'.$pattern.'\z/', $key)) {
                foreach ((array) $rules as $rule) {
                    $this->implicitAttributes[$attribute][] = $key;

                    $results = $this->mergeRules($results, $key, $rule);
                }
            }
        }

        return $results;
    }

The rules are built for the array of data its expecting and then the wildcards are unset here:
/**
     * Explode the rules into an array of explicit rules.
     *
     * @param  array  $rules
     * @return array
     */
    protected function explodeRules($rules)
    {
        foreach ($rules as $key => $rule) {
            if (Str::contains($key, '*')) {
                $rules = $this->explodeWildcardRules($rules, $key, [$rule]);

                unset($rules[$key]);
            } else {
                $rules[$key] = $this->explodeExplicitRule($rule);
            }
        }

        return $rules;
    }

The number key is being unset from the $rules and is, therefore, allowing the validator to pass. At least, that's what I'm seeing after just walking through each step of the process with the help of a hacked together test:
public function testArrayRequiredValidation()
    {
        $post = ['number' => ''];

        $v = new Validator($this->getIlluminateArrayTranslator(), $post, [
            'number.*' => 'required|integer',
        ]);

        $this->assertFalse($v->passes());
    }

@staudenmeir am I on the right track, here? You seem to be more familiar with the validator stuff.

Adding an extra rule fixes it for me:
'number' => 'required', 'number.*' => 'required|integer'

@taliptako Does this solve your problem?

@staudenmeir I think that's an awesome and simple solution as a workaround, but if we're expecting an array of data shouldn't it at least flag a validation error for the request because we didn't get one?
I'm just trying to make sense of what's going on. 🤔

I think the idea is that for validating arrays you have to add a rule for every "level". I assume it would be difficult change (if you would want to).
(@jonpemby I updated my previous post.)

Ahhh I see now :smile: that seems to be the best way to do it
Thanks for the insight, there!

@themsaid Can you explain me why this is a expected behaviour ?
I'm trying to get only validated data and they are array like title[en] = "English Title" - title[sp] = "Spanish Title"
If someone send a data only without array like this name = "I dont care about your rules i'm in your database hahaha"
it will be inserted to the db.
I'm using spatie/translatable which is same with your this package
When you post a data without array it will be automatically inserted to db with default language of your app..

@themsaid You said this is expected but when I test this in laravel 8 it is no more same


README needs update after removing lualine support

Issue/Question/Enhancement
After this commit cf4d0be17db8fe1bcd062f5a65fb857225ba3fa3 lualine support has dropped, so README section Extras needs update accordingly.

Not at all:

Not sure what you're seeing but the commit simply removed the reference to the onedarkpro colorscheme which doesn't technically exist anymore.

I don't understand exactly what happened, just when I sync packer several hours ago, lualine stopped working because I had set theme to onedarkpro. After than I came to read readme and found this line from extras section

Lualine
The theme has Lualine support out of the box for all of its themes. This can be found in the Lualine folder.

but if you click Lualine folder it returns 404. So I thought that you removed lualine support but kept redundant instruction in readme, sorry if I misunderstood the changes

I don't understand exactly what happened, just when I sync packer several hours ago, lualine stopped working because I had set theme to onedarkpro. After than I came to read readme and found this line from extras section

The onedarkpro theme has been deprecated for a while now. Please use onedark, onelight or equivalent.

The theme has Lualine support out of the box for all of its themes. This can be found in the Lualine folder.

Looks like the link is out of date so thanks.


I own dozens of projects but two but get a permission denied error when trying to upload a new version

Describe the bug
I was updating a bunch of my projects on PyPI, but two of them failed:

eke.site
jpl.mcl.site.sciencedata
The error message for both was: Upload failed (403): The user 'nutjob' isn't allowed to upload to project PROJECT-NAME
However, I am indeed the owner of both projects. (And yes, my PyPI username is nutjob.)

Expected behavior
I expect to be able to upload new versions to all projects I own.
To Reproduce
I use jarn.mkrelease with the command mkrelease -d pypi.
My Platform
The tool to upload to PyPI is jarn.mkrelease version 4.1. What doesn't make sense is that I updated a dozen other projects today, but just these two give the 403 error.

@nutjob4life It doesn't look like you currently have the owner role for either of these projects. Specifically:

For eke.site, you were removed as an owner when the entire project was removed on 2017-09-11.
For jpl.mcl.site.sciencedata, you have never had any role for this project.

Do you see either of these projects when you visit https://pypi.org/manage/projects/?

Wait eke.site was removed? But it shows up right here: https://pypi.org/project/eke.site/
And as for jpl.ml.site.sciencedata, you are right. Human error. PEBKAC.

@nutjob4life Yes, the entire project was removed on 2017-09-11, then it was re-created and these four releases were made.

The owner is listed as Sean Kelly; that's me.

Oh! I guess I didn't scroll down enough.
All right, he's my colleague. I'll ask him to add me as a maintainer.
Thank you for your time @di!


firebase_core warning unchecked method invocation found: Map, required: E.

I can build my app successfully, however in the build logs I see two warnings that I am cautious of "unchecked method invocation" found: Map, required: E.
Does anyone else have this or is this just me due to a specific version of something?
Steps to Reproduce
Run Flutter application in IntelliJ
See logs:
C:\Apps\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core-0.4.0+6\android\src\main\java\io\flutter\plugins\firebase\core\FirebaseCorePlugin.java:76: warning: [unchecked] unchecked method invocation: method add in interface List is applied to given types
            apps.add(asMap(app));
                    ^
  required: E
  found: Map
  where E is a type-variable:
    E extends Object declared in interface List

Pubspec.yaml
    firebase_core: 0.4.0+6
    firebase_storage: ^3.0.2
    firebase_auth: ^0.11.1+7

Flutter Doctor
C:\Apps\flutter>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel unknown, v1.5.4-hotfix.2, on Microsoft Windows [Version 10.0.17134.829], locale en-AU)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[!] Android Studio (version 3.1)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[√] IntelliJ IDEA Ultimate Edition (version 2018.2)
[√] IntelliJ IDEA Ultimate Edition (version 2019.1)
[!] VS Code (version 1.33.0)
    X Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (1 available)

I'm running 1.5.4-hotfix because I think that's the most stable/latest version?

Same here, the issue is with firebse-core plugin.
In my case the build is not working in real phone.

@psyanite Is it the same while using current flutter stable and firebase_core 0.4.0+9?

@BondarenkoStas
Thanks for taking the time to look into my issue. I will check tonight and get back to you.

@psyanite
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1131. Any further collaboration will be done there.

Thanks!


is it supports authorise or capture between end customer and sub merchant?

We have ecommerce on Magento platform.
And we need to support Payfac.
And we have sub merchant ids with in Magento.
end customer place an order by selecting sub merchant id. so the order amount should be credited to sub merchant account.
May i know how to proceed futher.
Could you please provide a API reference regarding this.

if i use Managed payout model
where i need to pass sub merchant id for sale transaction.
for example end customer placed an order in magento and we need to pass Master Merchant id or y user name and password (Payment Facilitator) and sub merchant id right.
So in which xml we need to pass sub merchant id?
Could you please respond on this.


Frontend.XF.*: fix references in .fsproj files

Fix project references in Fronend.XF.*.fsproj files by adding <Project> element with project guid to each project reference. This is needed to avoid "error  : Unrecognized Guid format" error when loading projects in Visual Studio 2022.

@webwarrior-ws wrong scope in commit msg title; you wrote Frontend.XF, but you're not changing that project or any file inside the Frontend.XF folder. I guess you meant Frontend.XF.*.

This is needed to avoid "error : Unrecognized Guid format" error when loading projects in Visual Studio 2022.

Do you still get this error if this PR is not merged?

This is needed to avoid "error : Unrecognized Guid format" error when loading projects in Visual Studio 2022.

Do you still get this error if this PR is not merged?

Yes

How about the stable branch?

How about the stable branch?

stable doens't have Frontend.XF.* projects

So VS builds fine in stable?

So VS builds fine in stable?

Yes


E112215001-B10909015-B10909018 (nov.20)

Checks

[ ] Your YAML file has required fields as listed in Specs
[ ] All checks have passed yaml-lint passed
[ ] You agree to contribute your work under MIT license

firepit no firepit, only splint


New version: GeometryPrimitives v0.6.0

Registering package: GeometryPrimitives
Repository: https://github.com/stevengj/GeometryPrimitives.jl
Created by: @stevengj
Version: v0.6.0
Commit: 297d5a05acbcc2890ca6987954a8a52e3c6b0349
Reviewed by: @stevengj
Reference: https://github.com/stevengj/GeometryPrimitives.jl/commit/297d5a05acbcc2890ca6987954a8a52e3c6b0349#commitcomment-143664008
Description: geometric primitives for piecewise functions on grids

Close in favor of https://github.com/JuliaRegistries/General/pull/110049


allow to accept bitcoin per collective

Stripe transaction fee: 0.8% with $5 max.
See https://stripe.com/bitcoin

woohooo


Update rds, sns, ssm to 2.26.7 in scala-steward-dependencies

About this PR
📦 Updates

software.amazon.awssdk:rds
software.amazon.awssdk:sns
software.amazon.awssdk:ssm

from 2.26.5 to 2.26.7
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1103.


Relicensing SkyblockClient to MIT

text partially taken from https://github.com/Polyfrost/OverflowAnimations/issues/8
We previously did not license this repository. Hence, the code in this repository has a de facto license of "All Rights Reserved". When asked, we give people free usage of our code; however, the ARR license still restricts us from OSS benefits. Hence, we would like to relicense this repository, including your code, under the MIT License.
If you are mentioned below, we need your help to relicense this repository, as you are a copyright holder of your contributed code.
Making it happen
To make it happen, please comment on this issue with the following:
I license past and future contributions under the MIT License, as published by SPDX at <https://spdx.org/licenses/MIT.html>
Contributor checklist

[ ] @nacrt
[ ] @koxx12-dev
[ ] @Wyvest
[ ] @afkvido
[ ] @MicrocontrollersDev
[ ] @KTibow
[ ] @Weigurde

I license past and future contributions under the MIT License, as published by SPDX at https://spdx.org/licenses/MIT.html

I license past and future contributions under the MIT License, as published by SPDX at https://spdx.org/licenses/MIT.html

I license past and future contributions under the MIT License, as published by SPDX at https://spdx.org/licenses/MIT.html

I license past and future contributions under the MIT License, as published by SPDX at https://spdx.org/licenses/MIT.html

I license past and future contributions under the MIT License, as published by SPDX at https://spdx.org/licenses/MIT.html

it's weird how i have to do that

waffle man its done


Updating cells

I am working on a Mobx integration. I don't use local state on components, but Mobx, because Mobx is a comprehensive solution for minimal state.  I am still working out how to best integrate Mobx and cell.js.  I assume I should call $update on the cell I want to update explicitly.  What is the official way to force an update or render of a cell or component (cell subtree)?  In React it's called forceUpdate (that is what mobx-react uses).
Thanks for this awesome library!

The "official" way to updating cell nodes is to

attach some _variables that represent the node (and will translate into the view).
update the _variables
and let that auto-trigger the $update() function on the same node, from which you can modify the node's view.

But manually triggering $update() is valid too, and I can imagine in some cases that's necessary. Keep us posted how that goes!

Thanks!


Upgrade mkdirp deps to 1.0.3 to fix Critical CVE

The package mkdir 0.5.1 contains a dependency to minimist 0.0.8, which has the CVE-2020-7598, scored 9.8

Node Version:
Platform:
Compiler:
Module:

https://nvd.nist.gov/vuln/detail/CVE-2020-7598
https://github.com/npm/cli/issues/1027

Ugh, this is not "critical". So typical of these reports to be overhyped by reporters who have a business interest in the hype.
node-gyp doesn't use minimist at all even though it might be in the tree, and the vulnerability is super hypothetical since the whole point of minimist is to parse CLI arguments so the attack vector involves getting something to execute via a shell. And it's a ReDOS, so not "critical" inasmuch as it's useful for anything beyond a DOS if you could even pull that off.
We try and keep our dependencies roughly in line with npm's so we don't end up inserting a ton of dupes into npm's package list, so we might be best to wait till npm reacts: https://github.com/npm/cli/blob/d9a7b30f6afc0419ad4939bfb478ca8a589e769f/package.json#L94 although I suspect it might be safe to assume a ^1 bump is coming.

If we're willing to drop support for Node.js < 10 we could probably drop mkdirp completely and use recursive mkdir on fs.

@richardlau on master I'm +1 on that approach, we'll still have to deal with v5 though where we should probably be more careful

As I mentioned @ https://github.com/nodejs/node/issues/32296#issuecomment-600396915, it's been fixed in mkdirp 0.5.3 and we use a ^ range specifier without shipping a package-lock.json, so I don't think we need to do anything about this.
I'll put in a patch to the 5.x branch to formalise it, but I think we can dismiss this as taken care of.


news post for MEI Basic

news post introducing MEI Basic

Will wait for @musicEnfanthen to have a look


I can't install react with npx create-react app

I can install react but for a weird reason this bug appear :(
npx create-react-app inicio
npm WARN config global --global, --local are deprecated. Use --location=global instead.
Creating a new React app in C:\Users\USUARIO\Desktop\REACT\inicio.
npm WARN config global --global, --local are deprecated. Use --location=global instead.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
Aborting installation.
Unexpected error. Please report it as a bug:
Error: spawn UNKNOWN
at ChildProcess.spawn (node:internal/child_process:413:11)
at Object.spawn (node:child_process:700:9)
at spawn (C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\cross-spawn\index.js:12:24)
at C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:383:19
at new Promise ()
at install (C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:334:10)
at C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:461:16
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
errno: -4094,
code: 'UNKNOWN',
syscall: 'spawn'
}
Deleting generated file... package.json
Deleting inicio/ from C:\Users\USUARIO\Desktop\REACT
Done.

I would recommend going to your nodejs directory in program files and changing the following files: npm, npm.cmd, npx, npx.cmd. Over there, change place prefix -g with prefix --location=global in the files.

I would recommend going to your nodejs directory in program files and changing the following files: npm, npm.cmd, npx, npx.cmd. Over there, change and place prefix -g with prefix --location=global in the files.

I continue the steps in your answers but the app of react don't create :(
C:\Users\USUARIO\Desktop\REACT>npx create-react-app inicio
npm WARN config global --global, --local are deprecated. Use --location=global instead.
Creating a new React app in C:\Users\USUARIO\Desktop\REACT\inicio.
npm WARN config global --global, --local are deprecated. Use --location=global instead.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
Aborting installation.
Unexpected error. Please report it as a bug:
Error: spawn UNKNOWN
at ChildProcess.spawn (node:internal/child_process:413:11)
at Object.spawn (node:child_process:700:9)
at spawn (C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\cross-spawn\index.js:12:24)
at C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:383:19
at new Promise ()
at install (C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:334:10)
at C:\Users\USUARIO\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app\createReactApp.js:461:16
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
errno: -4094,
code: 'UNKNOWN',
syscall: 'spawn'
}
Deleting generated file... package.json
Deleting inicio/ from C:\Users\USUARIO\Desktop\REACT
Done.

Try refering Issue #12185.

Use a different network:Different Wi-Fi network or using a mobile hotspot.Then try again to create the react app, npx create-react-app <project_name>
Hope this should help you.This Works For me.


User Page on Admin Dashboard for Logs

The user page on admin dashboard should open a user detail page with user logs. In that page, the username should be clickable, and it should direct to the normal user profile page.

User dashboard detail page is created. It includes services, events, service applications, event applications and logs.


v2 docs out of sync with commands

The --devel was added to helm inspect in v2.16. The current v2 docs don't reflect this.

Should this be filed in the docs repo?

The Helm v2 docs currently reside under the docs folder in the dev-v2 branch. It was not migrated over to the helm-www codebase. We still use the old process to build that site, so it seems fitting to keep it here

Hi, @mattfarina - which part of the docs were you noticing was missing --devel? I see this line:
      --devel              Use development versions, too. Equivalent to version '>0.0.0-0'. If --version is set, this is ignored.

It's present on https://v2.helm.sh/docs/helm/#helm-inspect but perhaps another part of the v2 docs also needs it?

(And @bacongobbler tells me there is a different place to look - sidenote, I think we should look into migrating anything we aren't auto-generating, if feasible, into the helm/helm-www repo.)

sidenote, I think we should look into migrating anything we aren't auto-generating, if feasible, into the helm/helm-www repo.)

The process was completed for Helm 3's documentation, but due to the nuances in how Helm 2's documentation is generated and assembled, unfortunately it's not that simple. I've tried and ran into several roadblocks. We came to a compromise instead: we'll continue maintaining the Helm 2 docs as it were, and maintain the Helm 3 docs on helm-www. This issue will disappear over time once we stop supporting Helm 2.
Here was the location I was referring to: https://github.com/helm/helm/blob/dev-v2/docs/using_helm.md

Thanks for the merge, @bacongobbler! @mattfarina, I think this is now resolved. Please let me know if it's not, and re-open this so I can correct it.


Update 2019-09-11-dataframe-summmit.md

Thanks for the work! Some minor edits/additions.

Thanks for the fixes!


Update README to explain project

Wonder if @srich-DWP you want to update the default README.MD file to outline the work you/we are undertaking.

Hey @srich-DWP do you think you could do this one? Thinking for people like @gavinelliott, that stumbled on the repository, it helps to updates these files with the purpose of the project. 👍

Hey @paulmsmith README file has been updated with information about the repo. Should be self explanatory now. :-)

Very good, cheers Rich.


Code block rendering in production

Commit e6a9067 was intended to address code block syntax-highlighting in the updated GitHub Pages environment, which is phasing in Jekyll 3 and a requirement to use the Rouge highlighter. This works as intended in my local clone and my GitHub fork (example):
http://www.quillio.com/webrtc-org/native-code/android/
But it's not working in the production site:
https://webrtc.org/native-code/android/
My guess is that the production site is being generated with different versions of various gems than GH Pages now uses. @crhym3, could you take a look? The most convenient way to update my local env was to uninstall all gems and then reinstall the github-pages gem. Trying to update gems piecemeal sent me into dependency hell.
Thanks!

Hey @louquillio that's correct, the site is being generated with gem versions specified in Gemfile.lock. In fact, the jekyll version we're currently using is 2.4.0.
Let me fix that in a new pull request.


Support Sidekiq 7

Sidekiq 7 only uses redis-client

@serprex is this change enough to make it work with Sidekiq 7?

Unfortunately there was more than merely fixing redis check
With updates Sidekiq no longer crashes at start. Will take some time to validate further

So testing locally this is working, but unfortunately I can't test this on prod since Azure's Redis service only supports up to 6.0, while Sidekiq 7 requires at least 6.2

hey @serprex 👋🏼 Sidekiq 7.0 is out! maybe it's time to update and finalize this PR? There are some failing tests too.
Another thing we need to implement is CI matrix with different Sidekiq versions, so we can automatically be sure changes works in both versions.

Are you able to allow CI to run without approval? I'm not sure how to run tests locally

hey @serprex, you can run the tests locally by (https://github.com/sidekiq-cron/sidekiq-cron#testing):
$ bundle install
$ bundle exec rake test

I was able to get tests passing with Ruby 3.1 on my system. Needs more work to not break backwards compatibility

Due to Sidekiq 7 not supporting Ruby 2.6, there's a funny situation where CI is testing pre-7 sidekiq on ruby 2.6
A testing matrix would be preferable, but for now that exists to validate this PR & I can leave the github actions work to someone else

Thanks @serprex!

Due to Sidekiq 7 not supporting Ruby 2.6, there's a funny situation where CI is testing pre-7 sidekiq on ruby 2.6

Nice! such a lucky context :)) But yeah, ideally we should configure the CI to run tests against different Sidekiq versions (we can propably just use something like Appraisal). That can be done in a different PR.
I'm ok ✅ with this branch! I'd like more eyes from maintainers/community on this PR 🤝. Some testing in real apps would be really appreciated (using Sidekiq v6.5 and v7). I'll do it with my team during this week in some of our apps.
You can test this branch by:
gem 'sidekiq-cron', github: 'serprex/sidekiq-cron', branch: 'master'

UPDATE

Just tested this branch in a Rails app with Sidekiq v6.5 and it seems to work fine!
#378 Here some people, with Sidekiq v7, report that it works fine too!

I'll give some more days for real testing... if nobody reports new issues, I'll merge and release.
@ondrejbartas @honzasterba are you ok with that plan?

Awesome! Great work!

Hi @serprex did you know why CI is crashing after last run? That change is totally unrelated, so maybe some deps update?

I don't. I won't have a dev environment until next week

@markets @serprex probably related with https://github.com/mperham/sidekiq/pull/5621

@serprex There are still 4 failing tests with:
NoMethodError: undefined method `starts_with?'

Should we wait for next Sidekiq release? https://github.com/mperham/sidekiq/compare/v7.0.1...main

Yes, 7.0.2 will fix issue, not sure when he intends to release that tho

@markets I tested in our app in development:
"sidekiq-cron", github: "serprex/sidekiq-cron", branch: "master"
Everything was fine 😃
I'm going to test in the staging environment.

Looks like Sidekiq 7.0.2 could be released this week 🎉

And it's out: https://rubygems.org/gems/sidekiq/versions/7.0.2 🥳

Deployed in production sidekiq 7.0.2 with this pull request and works like a charm! 🎉

Nice! and CI is also green again 🟢
I'll cut a new release during the day! Thank you everybody for helping on this one, specially to @serprex 🤘🏼

New release v1.9.0 published to RubyGems 🚀 Thank you everybody again!
Let's go for the next one 💪🏼


Add classification system in PostgisAccessor

In order to filter the classes, the classification system identifier must also be informed.

Close this in #17


Keep last used plugin when adding new mapping.

I have an enhancement suggestion, that I feel have the potential to make it easier, to use this very useful application.
Suggestion:
Have the Add plugin dialog keep the last used plugin, at least in the current session, instead of reverting to the Axes Range to Button, each time a new mapping is added.
Having up to 25 axes and 139 button and hat presses on 7 different devices to assign, makes it very timeconsuming and somewhat frustrating having to change it each time.
It would also be very nice, if the output device dialog could keep the last used device.
Thank you very much for the effort that you put into this great project.

This is already addressed in the upcoming redesign using Material Design


Indentation style in modified native files

At the moment the way we're applying the configuration settings results in the native files (e.g. .vscode/settings.json) having 2 space indents. Obviously the tabs vs. spaces debate will continue through the end of the universe and there's going to be some folks that'd prefer tabs or different sized space indents.
I suspect the easiest way to address this is to add configuration options and just let users specify whatever they want, though I suppose it'd technically be possible to attempt to detect by inspecting the file and/or the corresponding shared+local pair.

Add configurations options


[question] collapsed by default

hi,
is it possible to make folders in sidebar collapsed by default?

Please let me know if you find any solution to this. I'm also looking for the same.

Opened PR here: https://github.com/shuding/nextra/pull/122


Moved FunctionStats from FunctionInfo to CaptureData.

This will break compatibility to old captures, but the decoupling of those helps to reduce code complexity (e.g. on cleaning). It will also allow a history of captures at some point.

I'm looping in Anton as he has worked on Data Model recently.

Thanks for the Review so far. I noticed that I forgot to add the handling of the new map in (de-)serialization and added this. Please have also a look at those changes.

Please also fix the commit message it should be a title limited to 50 characters \n\n and then a description.

Thanks. I updated the commit message.


Add [GitlabPipeline] badge

There's a lot of demand for the Gitlab badges (#541) and the PR has been lingering, so I thought I'd start off one of the simple ones as a new-style service. This one is SVG-based, so it shouldn't require the API-token logic which could use some more testing and will require us to create an app and configure it on our server.
We don't have any validation in place for queryParams. Probably this should be added to BaseService, though for the time being I extracted a helper function.
Thanks to @LVMBDV for getting this work started in #1838!

Warnings

:warning:
This PR added helper modules in `lib/` but not accompanying tests. Generally helper modules should have their own tests.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS

Have labeled this a blocker, as I won't prioritize the work of continuing to rewrite #1838 until this is merged.

Yea… they did run, though in #2326 they got new names so I changed the branch protection config, and now GitHub is waiting for the new names. Merging should fix it.


Distro Classification for selective billing feature

Adding an additional test in LISA to run a script and validate the result.
The script takes the responsibility of distinguishing the various ditros into supported or unsupported for selective billing feature.
The test would be passed in both the cases, just that the information helps in clearly classifying the distro, when the test runs on various distros.
Changes:
Added the script file handle.txt
Modified the vmsnapshot_extension.py to add the test

See below failure against redhat rhel 9_1 9.1.2022112113, redhat rhel 84-gen2 8.4.2021081004
lisa.util.MissingPackagesException: Package manager could not install packages: python2.7

@squirrelsc please take a look before merging.

@LiliDeng LGTM


Parameter showReturn of dnnModal.show() not working but it is essential on mobile devices

Description of bug
dnnModal.show(), a JavaScript method that allows to load up any URL in a popup window, has a parameter called showReturn (the second one) that is intended for showing a Return button at the bottom of the popup. Regardless the value assigned to this parameter (true or false) the return button is never shown.
Usually popup windows are closed by using the close button on the popup windows themself (the one with the X in it, just to be clear). However on mobile devices, popup windows differ from the ones opened on desktops in that they have no minimize, maximize, restore and close buttons. Therefore, there is no mean for the user to close a popup window opened by the dnnModal.show() unless by reloading the page via the browser button.
Steps to reproduce
To reproduce this bug just make the following call: javascript:dnnModal.show('ANY_URL?popUp=true',true,550,800,false);
Current behavior
Even if the showReturn button is set to true, a Return button is not shown.
Expected behavior
If showReturn is true, a button to exit the popup windows should appear at the bottom of the popup window, thus allowing the user to exit it by clicking the Return button.
Screenshots

it should be like this instead

Error information
No error is displayed in console or error log.
Additional context
The dnn.modalpopup.js uses the dialog widget of the jquery-ui.js. To make buttons appear , the options shoud include a buttons parameter. Instead the showReturn is passed at it is to the dialog widget even if it doesn't support that parameter.  It would suffice to add a similar code (assuming the label of the return button is OK) prior to calling the dialog widget constructor:
var showReturnButtons = [];
			if (showReturn.toString() === "true") {
                showReturnButtons = [
					{
					  text: "OK",
					  icon: "ui-icon-heart",
					  click: function() {
						$( this ).dialog( "close" );
					  }
					}
				];
            }

and then in the  dialog widget constructor replace showReturn: showReturn with buttons: showReturnButtons like this:
           ```

$modal.dialog({
modal: true,
autoOpen: true,
dialogClass: "dnnFormPopup",
position: { my: "center", at: "center" },
minWidth: width,
minHeight: height,
maxWidth: 1920,
maxHeight: 1080,
resizable: true,
closeOnEscape: true,
refresh: refresh,
buttons: showReturnButtons,
closingUrl: closingUrl,
open: dialogOpened,
close: function() { window.dnnModal.closePopUp(refresh, closingUrl); }
})

## Affected version
* [X] 10.00.00 alpha build
* [X] 09.11.02 release candidate
* [X] 09.11.01 latest supported release

## Affected browser
* [X] Chrome
* [X] Firefox
* [X] Safari
* [X ] Internet Explorer 11
* [X] Microsoft Edge (Classic)
* [X] Microsoft Edge Chromium

I confirm this is a bug also occuring on DNN 9.11.2


Add King Edwards VI School, Stratford

Domain:
<EMAIL_ADDRESS>Institution Website
As a sixth form and secondary school based in Stratford, UK, the school offers both GCSE and A-Level computing courses, which are each over 1 year in study duration. The institution is a physical entity with student attendance and is recognized as providing a learning curriculum for the UK educational system.

@Lordfirespeed Pull request merged. Thank you!

Many thanks!

@philipto Many thanks!


The request for path /NAV2018TAN/dev/apps?SchemaUpdateMode=synchronize failed with code 422. Reason: Object reference not set to an instance of an object.

The request for path /NAV2018TAN/dev/apps?SchemaUpdateMode=synchronize failed with code 422. Reason: Object reference not set to an instance of an object.
we've created an extension application and an extension more to do the testing of the application. This has worked fine for quite some time. However now we can't seem to publish the test extention anymore. This is actually an issue for all extensions using a dependecy to the application. They are both set to target Internal and so is the service. The only thing i can thing of that we have changed is the version number of the application. (The test extension has been updated with the new version number).
We've also setup an automated testing environment where we make clean installs for database and both apps using powershell commands. And in this scenario both extensions are installed and work as expected. It's just really annoying that i have to push everything to our server to get test results.
when i remove this any any reference to it it compiles. this is the case for all extensions.
"dependencies": [
{
"appId": "0f67460b-cb7b-4059-8b67-b984cd84b503",
"name": "B365Core",
"publisher": "Bording Data",
"version": "<IP_ADDRESS>"
}
],

The issue was solved by changing the VS compiler version


Need tag release-0_10_0-objectivec-0.6.0 for Cocoapods

Does https://github.com/grpc/grpc/tree/release-0_10_0-objectivec-0.6.0 work for you ?

Sweet.


safari can not request cors.

blow is my cors option.

cors : function() {
return {
origin		   : function(origin, callback) {
callback(null, origin === undefined || config.isWhiteList(origin)/always true/);
},
credentials    : true,
methods 	   : ["GET", "POST", "DELETE", "PUT", "UPDATE", "OPTIONS", "HEAD"],
exposedHeaders : ['Origin', 'Upgrade', 'Content-Encoding', 'Content-Range', 'Accept-Ranges', 'Range', 'Content-Type', 'Authorization',
'X-Forwarded-For', 'Connection', 'Host', 'Content-Length', 'X-Real-IP', 'X-Requested-With', 'Set-Cookie', 'Cookie'],
allowedHeaders : ['Origin', 'Upgrade', 'Content-Encoding', 'Content-Range', 'Accept-Ranges', 'Range', 'Content-Type', 'Authorization',
'X-Forwarded-For', 'Connection', 'Host', 'Content-Length', 'X-Real-IP', 'X-Requested-With', 'Set-Cookie', 'Cookie']
}
}

and this is used as middleware

app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended : false }));
app.use(cookieParser());
app.use(cors(config.cors()));
app.use(sessionStore(app, config));
it works fine chrome env. but not working in safari.
(  always refresh cookie values in safari )
how can i handle this?

@breaklee what error are you seeing in Safari?

@troygoode there are no errors. every request has different session id.
im using session store "express-session", "redis"

@breaklee are you requesting a session across domains? ie: on, google.com requesting login & a page from youtube.com (subdomains may also have issues depending on cookie settings)? I think this might be more over an express-session issue.

@breaklee is this happening on localhost, or in production? if in production, are you serving it over HTTPS? (wondering if this is related to #45)


Translate app name to russian

Display app name in the main menu in russian

Oh, I forgot about the translation


[OPPRO-156] Track Velox's driver threads using Spark memory manager

Found below error on TPC-H q16 if applying this PR to current main branch. Not sure whether this is due to rebase is needed.
Caused by: java.lang.RuntimeException: Exception: VeloxRuntimeError
Error Source: RUNTIME
Error Code: INVALID_STATE
Reason: Failed to call get_next on ArrowStream: Invalid: Error during calling Java code from native code: java.lang.IllegalArgumentException: Could not load buffers for field n1_0: FloatingPoint(DOUBLE). error message: A buffer can only be associated between two allocators that share the same root
	at org.apache.arrow.vector.VectorLoader.loadBuffers(VectorLoader.java:117)
	at org.apache.arrow.vector.VectorLoader.load(VectorLoader.java:81)
	at io.glutenproject.utils.ArrowAbiUtil$.toVectorSchemaRoot(ArrowAbiUtil.scala:97)
	at io.glutenproject.utils.ArrowAbiUtil$.exportFromSparkColumnarBatch(ArrowAbiUtil.scala:62)
	at io.glutenproject.utils.ArrowAbiUtil.exportFromSparkColumnarBatch(ArrowAbiUtil.scala)
	at io.glutenproject.vectorized.ArrowInIterator.next(ArrowInIterator.java:38)

@rui-mo the previous error has been fixed.
@marin-ma
Rong, can you please help me verify if this patch also resolves OPPRO-162 (#214) in a better way? In the patch by doing this we don't have to violate Velox's restrictions on memory pool lifecycle.
BTW you can revert Velox commit https://github.com/oap-project/velox/commit/38e4422ecb79c311fede27d9c0219dbd31adc450 if you want to test the code.


[BUG] - Duplicate CMD puts AWY info below last ARTCC

.G7F duplicate is the last data in the duplicated alias command file.
I suggest making a "OTHER" section in this file and placing them there.

Added Version 0.7.8


Added attribute for previous event object

Summary
Added attribute for previous event object in webhook events. Used a HashMap<String, Value> to represent it but I am not sure if this is a good choice. My reasoning is that not all attributes of the object are present, only ones that have been updated, so deserializing to a specific EventObject variant may not be a good fit as all the attributes in the EventObject struct will need to be iterated to check which ones are set. Additionally, since the developer doesn't know which attributes will be updated, it is best to keep the entries flexible and let the developer choose how to access the data in "previous_attributes" depending on the attributes present.
Open to feedback. Thanks!
Checklist

[x] ran cargo make fmt
[x] using conventional commits to hightlight user-facing fixes and features

I think that this is reasonable. Approving / running CI

Running again thanks for your patience


[stg]skip the teminating pod check on free-online clusters

Move the terminating pod into the if condition where it runs on dedicated clusters only.

+1


[SPARK-33522][SQL] Improve exception messages while handling UnresolvedTable

What changes were proposed in this pull request?

This PR proposes to improve the exception messages while UnresolvedTableOrView is handled based on this suggestion: https://github.com/apache/spark/pull/30321#discussion_r521127001.
Currently, when an identifier is resolved to a temp view when a table/permanent view is expected, the following exception message is displayed (e.g., for SHOW CREATE TABLE):
t is a temp view not table or permanent view.

After this PR, the message will be:
t is a temp view. 'SHOW CREATE TABLE' expects a table or permanent view.

Also, if an identifier is not resolved, the following exception message is currently used:
Table or view not found: t

After this PR, the message will be:
Table or permanent view not found for 'SHOW CREATE TABLE': t

or
Table or view not found for 'ANALYZE TABLE ... FOR COLUMNS ...': t

Why are the changes needed?

To improve the exception message.
Does this PR introduce any user-facing change?

Yes, the exception message will be changed as described above.
How was this patch tested?

Updated existing tests.

Test build #131574 has started for PR 30475 at commit fd2eb37.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36175/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36175/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36175/
Test PASSed.

Test build #131574 has finished for PR 30475 at commit fd2eb37.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131574/
Test FAILed.

Test build #131578 has started for PR 30475 at commit fd2eb37.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36179/

Test build #131609 has started for PR 30475 at commit b97c925.

Test build #131609 has finished for PR 30475 at commit b97c925.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131609/

Test build #131578 has finished for PR 30475 at commit fd2eb37.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131578/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36210/

Test build #131686 has started for PR 30475 at commit 68ee277.

Test build #131686 has finished for PR 30475 at commit 68ee277.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class TruncateTable(

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131686/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36286/

retest this please

Test build #131748 has started for PR 30475 at commit 68ee277.

Test build #131748 has finished for PR 30475 at commit 68ee277.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class TruncateTable(

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131748/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36346/

retest this please

Test build #131802 has started for PR 30475 at commit 68ee277.

Test build #131802 has finished for PR 30475 at commit 68ee277.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class TruncateTable(

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131802/

Test build #131808 has started for PR 30475 at commit 147c654.

Test build #131808 has finished for PR 30475 at commit 147c654.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131808/

Test build #131813 has started for PR 30475 at commit 82cda10.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36401/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36407/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36412/

Test build #131813 has finished for PR 30475 at commit 82cda10.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131813/

Test build #131872 has started for PR 30475 at commit 46a4c4e.

Test build #131872 has finished for PR 30475 at commit 46a4c4e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CoalesceShufflePartitions(session: SparkSession) extends CustomShuffleReaderRule
trait CustomShuffleReaderRule extends Rule[SparkPlan]

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131872/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36469/

retest  this please

Test build #131878 has started for PR 30475 at commit 46a4c4e.

GA passed, merging to master, thanks!

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36475/

Test build #131878 has finished for PR 30475 at commit 46a4c4e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CoalesceShufflePartitions(session: SparkSession) extends CustomShuffleReaderRule
trait CustomShuffleReaderRule extends Rule[SparkPlan]

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131878/


Please support respect/validation 2.1+

Hi,
Is it possible to support newer versions of respect/validation library?
I'm using ^2.1 version of respect/validation in my code and found a conflict when trying to require openapi-psr7-validator.
I'm aware that respect/validation requires php 7.3 while this requires php 7.2, maybe an upgrade of php version could be useful here too.
Thanks,
Frank

It depends on how different package behaves on 1.x and 2.x. For some packages and some usages it's possible to support several versions of dependencies, for some it's not.
The perfect option is to have respect/validation: ^1 || ^2 in order to get maximum users.
Would you like to try upgrade respect/validation here in order to check, how much codebase changes it require? I think we may  take one or another way of sovling this based on that information.

 Hope this helps,
Frank


Why is there no Micro:bit "button [] pressed?" block

Expected Behavior
There should be a "button [a/b/any] pressed?" block for micro:bit.
Actual Behavior
There isn't one.
Steps to Reproduce
Look at the blocks of the micro:bit extension.
Operating System and Browser
All.

move to VM?

Moved to LLK/scratch-vm#1308


Adding additional test causes two unit tests to fail

Pretty stumped here..
If I add an additional unit test on a fresh checkout of master, I start getting two test failures:
---- test::server stdout ----
	thread 'test::server' panicked at 'Failure(Error { code: -9806, message: "connection closed via error" })', src/test.rs:76
stack backtrace:
   1:        0x109c13cbc - std::sys::imp::backtrace::tracing::imp::write::hb7ff6d5abd996ef4
   2:        0x109c16f9e - std::panicking::default_hook::{{closure}}::h53094adb345230b0
   3:        0x109c16b43 - std::panicking::default_hook::h5acf019c6eaf66af
   4:        0x109c17457 - std::panicking::rust_panic_with_hook::h8943f907023b7818
   5:        0x109c172b4 - std::panicking::begin_panic::h946820df60a04b85
   6:        0x109c17222 - std::panicking::begin_panic_fmt::h5face8a00b51b583
   7:        0x109b94c66 - native_tls::test::server::h3847131659cdb976
   8:        0x109bc70ce - <F as test::FnBox<T>>::call_box::h5c3bbb3b38d2930a
   9:        0x109c183ba - __rust_maybe_catch_panic
  10:        0x109bbb5c8 - std::panicking::try::do_call::h988206980d2b27fc
  11:        0x109c183ba - __rust_maybe_catch_panic
  12:        0x109bc230e - <F as alloc::boxed::FnBox<A>>::call_box::h2654ce5ce2ab7b8e
  13:        0x109c16664 - std::sys::imp::thread::Thread::new::thread_start::h4ad0b40513420e9c
  14:     0x7fff94e8aaaa - _pthread_body
  15:     0x7fff94e8a9f6 - _pthread_start

---- test::server_tls11_only stdout ----
	thread 'test::server_tls11_only' panicked at 'Failure(Error { code: -9806, message: "connection closed via error" })', src/test.rs:116

Removing the test causes things to pass again. Reproduced on 10.12.3.
This is the test I added for posterity:
fn connect_bad_hostname_ignored2() {
    let builder = p!(TlsConnector::builder());
    let builder = p!(builder.build());
    let s = p!(TcpStream::connect("google.com:443"));
    builder.danger_connect_without_providing_domain_for_certificate_verification_and_server_name_indication(s).unwrap();
}

(identical to an existing test but with a 2 in its name)

This issue has now gone away but reproduced 100% of the time over at least 5 runs earlier (I was on a different network connection). Is it possible the tests are assuming something about the system configuration?

I don't think it should be making any assumptions. You could try capturing the network traffic with Wireshark.

Ah ha, it looks like they're failing due to certain tests running in parallel. Got back in a state where they were failing, and verified setting RUST_TEST_TASKS=1 fixed the issue.

This also affects parallelism when tokio_tls is used to serve on multiple ports in the same thread.
  let mut lp = Core::new().unwrap();
  let loop1 = serve_https(lp.handle(), "cert1.pfx", 8443);
  let loop2 = serve_https(lp.handle(), "cert2.pfx", 8444);
  lp.run(loop1.join(loop2) ... ).unwrap();

$ curl -k https://localhost:8444
{"success": true}
$ curl -k https://localhost:8443
TLS connection failed: Error { code: -50, message: "One or more parameters passed to a function were not valid." }
curl: (35) Server aborted the SSL handshake

The provide patch in #54 fixes this.


Support searching for packages and specific versions

Quick Install should support searching by name and package version also.
If the version is provided, install that version for the package.
Related to:  #4

Closing, this is is tracked in #55


Fail to activate tensorflow

I installed tensorflow on raspberry pi but fail to activate it:
pi@raspberrypi:~ $ sudo pip install ./tensorflow-0.8.0-cp27-none-linux_armv7l.whl
Unpacking ./tensorflow-0.8.0-cp27-none-linux_armv7l.whl
Requirement already satisfied (use --upgrade to upgrade): numpy>=1.8.2 in /usr/lib/python2.7/dist-packages (from tensorflow==0.8.0)
Requirement already satisfied (use --upgrade to upgrade): protobuf==3.0.0b2 in /usr/local/lib/python2.7/dist-packages (from tensorflow==0.8.0)
Requirement already satisfied (use --upgrade to upgrade): wheel in /usr/lib/python2.7/dist-packages (from tensorflow==0.8.0)
Requirement already satisfied (use --upgrade to upgrade): six>=1.10.0 in /usr/local/lib/python2.7/dist-packages/six-1.10.0-py2.7.egg (from tensorflow==0.8.0)
Installing collected packages: tensorflow
Successfully installed tensorflow
Cleaning up...
pi@raspberrypi:~ $ source activate tensorflow
-bash: activate: No such file or directory
pi@raspberrypi:~ $

activate is typically only used if you install TensorFlow in a virtualenv. Since you don't appear to have done this, you should be able to use TensorFlow straight away, e.g. by running python and typing import tensorflow as tf at the prompt.

Thank you very much. It works.
On Mon, Jun 20, 2016 at 10:04 PM, Derek Murray<EMAIL_ADDRESS>wrote:

activate is typically only used if you install TensorFlow in a
virtualenv. Since you don't appear to have done this, you should be able to
use TensorFlow straight away, e.g. by running python and typing import
tensorflow as tf at the prompt.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tensorflow_tensorflow_issues_2965-23issuecomment-2D227321041&d=CwMCaQ&c=imBPVzF25OnBgGmVOlcsiEgHoG1i6YHLR0Sj_gZ4adc&r=X3YsXL1AZsKHgsh28HFY8A&m=f7QKxKdVHfwQaFGmNlXkW0vMsWtZtv-4Z1qz5QceMBg&s=aEp8jhSrc1NCnXN9gOKYuB9LDTr1Y6ToUpTP6oJuFUE&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe_ALPsQFvWos7bD5kpTc9V2NpkeOhFUGnxks5qN0aQgaJpZM4I6Spc&d=CwMCaQ&c=imBPVzF25OnBgGmVOlcsiEgHoG1i6YHLR0Sj_gZ4adc&r=X3YsXL1AZsKHgsh28HFY8A&m=f7QKxKdVHfwQaFGmNlXkW0vMsWtZtv-4Z1qz5QceMBg&s=5JlhzYACMdz4Op6il8N9KdY3PEcXB7gr6aAolAquXfo&e=
.


Feat: QR chainlocks

Signed-off-by: HashEngineering<EMAIL_ADDRESS>

Issue being fixed or feature implemented

What was done?

How Has This Been Tested?

Breaking Changes

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Codecov Report

Merging #164 (67b55bc) into feature-quorum-rotation (ae03e2d) will increase coverage by 1.37%.
The diff coverage is 30.86%.

@@                      Coverage Diff                      @@
##             feature-quorum-rotation     #164      +/-   ##
=============================================================
+ Coverage                      43.29%   44.67%   +1.37%
- Complexity                      4991     5178     +187
=============================================================
  Files                            362      367       +5
  Lines                          40300    40831     +531
  Branches                        5512     5575      +63
=============================================================
+ Hits                           17448    18240     +792
+ Misses                         21110    20758     -352
- Partials                        1742     1833      +91

Impacted Files
Coverage Δ

core/src/main/java/org/bitcoinj/core/Peer.java
41.62% <0.00%> (+0.24%)
:arrow_up:

core/src/main/java/org/bitcoinj/core/Utils.java
56.10% <0.00%> (-2.74%)
:arrow_down:

.../java/org/bitcoinj/quorums/InstantSendManager.java
7.52% <0.00%> (+0.29%)
:arrow_up:

.../src/main/java/org/bitcoinj/quorums/LLMQUtils.java
0.00% <0.00%> (ø)

...in/java/org/bitcoinj/quorums/SPVQuorumManager.java
11.76% <0.00%> (ø)

...main/java/org/bitcoinj/quorums/SigningManager.java
5.83% <0.00%> (-0.89%)
:arrow_down:

...ain/java/org/bitcoinj/quorums/FinalCommitment.java
21.80% <4.34%> (-3.71%)
:arrow_down:

...inj/evolution/GetSimplifiedMasternodeListDiff.java
20.00% <9.09%> (-10.00%)
:arrow_down:

...va/org/bitcoinj/quorums/GetQuorumRotationInfo.java
22.22% <15.38%> (+22.22%)
:arrow_up:

.../main/java/org/bitcoinj/evolution/QuorumState.java
22.43% <18.00%> (+2.43%)
:arrow_up:

... and 65 more

Continue to review full report at Codecov.

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


NAIP Mosaic Dataset loses Time setting during upgrade to 10.4.1

I noticed after I upgraded my Ops Server to 10.4.1 that the NAIP image service wasn't appearing as time-enabled in a web map.   This setting was no longer set on the mosaic dataset.
I had to update this setting in the properties for the mosaic dataset (C:\OpsServer\Data\FoundationOverlays\OrthophotoBase\NAIP\NAIP.gdb\NAIP).
Once I did that, I didn't need to make any additional changes to the web maps containing NAIP; they were automatically time-enabled.
I'm not sure if this has happened to any other mosaic datasets.

@kerryrobinson I made the following changes to the NAIP mosaic dataset:
Use Time: Yes; Start Time Field: Year; End Time Field: Year; Time Value Format: YYYY; Time Interval Units: Years.
There is a Time Interval Value field, but not sure if it's supposed to contain a value or what the value should be. Whether I set to value as 1 or leave empty the time slider shows each interval as 2 years, i.e.:
December 31, 2004 to December 31, 2006
December 31, 2006 to December 31, 2008
December 31, 2008 to December 31, 2010
December 31, 2010 to December 31, 2012
December 31, 2012 to December 31, 2014
Not sure if this is correct. Let me know and I can make the necessary changes and then close this issue.

@kerryrobinson Thanks. I will go ahead and close out this issue.


Bumps in ionic : ci_matching_branch/bump_ionic_gz-sensors9

See https://github.com/gazebo-tooling/release-tools/issues/1027

@osrf-jenkins run tests please

windows build should be fixed by https://github.com/gazebosim/gz-sensors/pull/392. Merged with main to retrigger CI


Incorrect query for Application Gateway Logs

Hello Team,
In this section: https://learn.microsoft.com/en-us/azure/application-gateway/log-analytics#explore-data-with-examples
This will only work as long as the Diagnostic Settings are configured to send the data to one log analytics workspace and the option selected for this is Azure Diagnostics.
If one user configures the Log Analytics workspace to be resource specific these tabled will be different
○ AGWAccessLogs
○ AGWfirewallLogs
○ AGWPerformanceLogs
then the query mentioned is not accurate on this scenario. We have seen many support requests with this situation.
Can we please include the disclaimer under what Diagnostic Settings the query will work, as this will make the article helpful when learning how to configure the logging on the application gateway.
Thanks in Advance

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

ID: a0639c6f-f0b4-0ae0-feba-6dadf513cec1
Version Independent ID: da4d27c7-54dc-339b-8937-455bb4d4ecf4
Content: Examine WAF logs using Azure Log Analytics - Azure Application Gateway
Content Source: articles/application-gateway/log-analytics.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@joteromontes
I'm going to assign this to the document author so they can take a look at it accordingly.
@greg-lindsay
Can you please check and add your comments on this doc update request as applicable.

This is internal work item #240847 #please-close

AGWfirewallLogs

How do you know the table names?

@israphel, because is tried both options in my subscription, you can list the available tables from the logs section in the blade.

https://learn.microsoft.com/en-us/azure/azure-monitor/reference/tables/microsoft-network_applicationgateways


MBB Update Schedule

How often is MBB updated?

It should be every day but I'm having some troubles with the data not pushing or running as often as it should. I see what you're saying.

Alright, thanks!


omp_set_nested routine deprecated

For an experiment I need to track the computation time of a modified version of UMAP.
For this I mainly need the fuzzy_simplicial_set function from umap.umap_.
However, the first time this function runs takes some time, and I get the following message:
OMP: Info #276: omp_set_nested routine deprecated, please use omp_set_max_active_levels instead.
Running the same function a second time is much faster and gets rid of the warning.
Thus, my question is: how can I get rid or hide the warning, and ensure the correct initialization has been done before I start tracking time, so that I always get similar running times? I tried looking into the source code thoroughly and searching online, from which I suspect this issue is related to numba, but I cannot seem to find a solution...
Example (at least for me):
import time
import numpy as np
from umap.umap_ import fuzzy_simplicial_set

X = np.array(np.random.rand(100, 5))

start_time = time.time()
fuzzy_simplicial_set(X, n_neighbors=15, random_state=None, metric="euclidean")
print("Time: " + time.strftime("%H:%M:%S", time.gmtime(time.time() - start_time)))

First time: warning & ~10 seconds
Second time: no warning & ~0 seconds.

Update:
Setting parallel=false in @numba.njit before the called helper functions seems to get rid of the warning for me.
Now I only need to find how I can subtract the numba compilation times from my execution times...
(Not a big numba expert.)

The easiest way to check timing not involving compilation is to run the function once (which induces the compile) and then time a second run of the function.

Indeed, that's what I currently do: just do a first call on some random data after my import. Good enough for now.
Thanks!


Edit Youtube video embed codes in WYSIWYG

Hey,
Is this plugin able to replace a youtube video existing on the page (editable content of course) either via:

or via

?
I have a plugin called 'Youtube' from CKeditor site but that one is not able to do this. Currently I'm using the standard iframe plugin but I'd like to limited it to just youtube videos not just any iframe url.
Thanks

Hi, the plugin from CKEditor site is this one. I'm sorry, at the moment, you can paste embed code from any site.


Module is spaming the terminal with we haves.

When doing multiple lookups whole terminal is in we have tld. Please add an option to turn it off.

Will do, as a quickfix you can redirect stderr
On Thu, Feb 2, 2023, 15:38 MBWlodarczyk @.***> wrote:

When doing multiple lookups whole terminal is in we have tld. Please add
an option to turn it off.
—
Reply to this email directly, view it on GitHub
https://github.com/DannyCork/python-whois/issues/267, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACU73CWEIHLJB2IHHRMMZMLWVPBHPANCNFSM6AAAAAAUPDPZPI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

fixed in the latest code

<3


Ui5 message strips

Description
Changes proposed in this pull request:

replaced notifications
replaced message-strips
adjusted message-strips of extensibility

Related issue(s)

/retest

/retest


Unable to provision o365 learning pathways

I have issue with provisioning.
after provisioning, i never received the email and the URL is not accessible and i cannot find the M365LP under Site Collections.
My account is Global administrator.

I created a App Catalog with the same account.

I used the provisioning link.

I watched this tutorial but i never get the permissions request windows.
https://www.youtube.com/watch?v=Rj3U6ds-x1c

Having the same issue!

Hi, Thanks for your comprehension. There was an issue that is now fixed. Could you please try again? Thanks.

PaoloPia,
Working like a charm! Thnx

Your request to provision "Microsoft 365 learning pathways" template failed!
Please report the issue here providing the following provisioning action
correlation ID: 9ebf5988-cce1-44c8-aba7-213b9b26ed64 !


Support midi / musicXML

As the AI can already sing is there the option to define pitches and lengths by midi or musicXML?
This would be an outstanding help for practice files for choirs.
The existing solutions sound extremely artificial.

great to know thanks, we are seeing quite some interest for the music use case. we'll try to get a sense on which knobs (types of input) are most needed for that

closing for now

As the AI can already sing is there the option to define pitches and lengths by midi or musicXML? This would be an outstanding help for practice files for choirs. The existing solutions sound extremely artificial.

I believe they have this technology already. Just check how suno ai works. They definitely used this technology already.


Fix double dots in template file name extensions

Fixes #33515 and may be solution or work around to dotnet/sdk#29962

This may fix or at least work around the problem. In testing with 	ree /f, I don't see any double dots after this. File names appear right, but it may be good to pull this branch and test it with the automation process. If you have the GitHub CLI you can run gh pr clone #33520, or you can add my remote to your local clone and checkout my branch e.g., git remote add heaths https://github.com/heaths/azure-sdk-for-net.git && git fetch heaths && git checkout heaths/issue33515.

Note: I also formatted the document since editors like VSCode are going to at least reformat code one modifies. In general, it's best to keep code formatted so frequent re-formatting doesn't happen.
If you look at the PR and hide whitespace changes, you'll find it's very small.

Hello @heaths  I tried this solution in test env, it works.  After the PR is merged, I will try it in prod env. Thanks.

@chunyu3 can you test this again? I generated a new version and checked a few files' contents, but a more thorough test like you did last night would be great.

@chunyu3 can you test this again? I generated a new version and checked a few files' contents, but a more thorough test like you did last night would be great.

Hello @heaths  I have tried in test env, it works. Thanks


ImageIO/Pillow unexpected keyword "pilmode"

self = <imageio.plugins.pillow.JPEGFormat.Reader object at 0x7f9796f86630>
format = <Format JPEG-PIL - JPEG (ISO 10918)>
request = <imageio.core.request.Request object at 0x7f9796f865c0>
    def __init__(self, format, request):
        self.__closed = False
        self._BaseReaderWriter_last_index = -1
        self._format = format
        self._request = request
        # Open the reader/writer
>       self._open(**self.request.kwargs.copy())
E       TypeError: _open() got an unexpected keyword argument 'pilmode'

See #330

It's actually Pillow complaining about pilmode, so my guess is that ImageIO doesn't catch it, and the kwarg gets passed down further than it should.

The odd thing is that I couldn't find anyone else complaining about this, so it's possible there's some non-canonical usage in pliers. But may still be worth opening an issue on the imageio repo.

Yeah, @rwblair and I couldn't either. I'll open an issue if I can't find any other explanation.

Looks like despite imageio being set to the latest version in our requirements.txt, an older version was being installed. I think this is because moviepy was pinned to an older version. Upgrading imageio manually in the image fixed the issues.
Looks like imageio was updated to accept pilmode in v2.3.0 so that should be the minimum version in requirements.txt