Inaccurate Statement on Non-UI-Supported Linked Services

[Enter feedback here]

Advanced authoring: For other linked service types that are not in above list, you can parameterize the linked service by editing the JSON on UI:
In linked service creation/edit blade -> expand "Advanced" at the bottom -> check "Specify dynamic contents in JSON format" checkbox -> specify the linked service JSON payload.
Or, after you create a linked service without parameterization, in Management hub -> Linked services -> find the specific linked service -> click "Code" (button "{}") to edit the JSON.
Refer to the JSON sample to add parameters section to define parameters and reference the parameter using @{linkedService().paramName}.

This is not the case for Azure Functions linked services. You can add parameters to the linked service itself through the JSON editor, but it is impossible to pass parameters from Azure Functions Activities into the linked service, even through the JSON. I believe there's a bug internally to fix this. Essentially the Data Factory UI will automatically strip any parameters from the Pipeline JSON definition for Azure Functions activities.
There needs to be a warning here to let us know about this bug -- I wasted a couple of business days trying to figure this out, because the documentation said it should be doable.

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

ID: 2bf75315-da09-2cf9-6db5-d12fcc804283
Version Independent ID: bf4deb1b-fe88-a9cb-3605-0500a60fb882
Content: Parameterize linked services - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/parameterize-linked-services.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @chez-charlie
Microsoft Alias: chez

@tcc-sejohnson, Thanks for the feedback! We are currently investigating and will update you shortly. Apologies for any inconvenience.

@chez-charlie Could you please validate feedback and update document as appropriate.

@tcc-sejohnson  thanks for reporting.
If you look at doc. Azure Functions is not listed for GUI based parameterization. We have not tested it. It is mainly because of Azure Functions payload contract with ADF.
It also implies that you should not be paramteterizing more than what we is required  in GUI. We will improve with a note/warning once we discuss with Eng.
#please-close

@ssabat
Thank you! Please do add a warning.

Advanced authoring: For other linked service types that are not in above list, you can parameterize the linked service by editing the JSON on UI

There is no implication here that you should not be parameterizing services not listed. In fact, it says in plain English that you should be able to parameterize services that are not supported through the UI by using JSON. If that's not the case, this paragraph needs to be removed.


Support image relocation for docker and oci image types

Image relocation
Before installing a bundle, the user may wish to relocate the images referenced by the bundle to point at a suitable registry.
A highly desirable property of image relocation is that the image digest of the relocated images are the same as those of the original images. This gives the user confidence that the relocated images consist of the same bits as the original images.
Using your own registry has other advantages:

You can control when those images are updated or deleted.
The registry can be hosted on a private network for security or other reasons.

Restriction
This feature is restricted to images with type "docker" and "oci".
Relocating image names
An image name consists of a domain name (with optional port) and a path. The image name may also contain a tag and/or a digest. The domain name determines the network location of a registry. The path consists of one or more components separated by forward slashes. The first component is, by convention, a user name providing access control to the image.
Let’s look at some examples:

The image name docker.io/istio/proxyv2 refers to an image with user name istio residing in the docker hub registry at docker.io.
The image name projectriff/builder:v1 is short-hand for docker.io/projectriff/builder:v1 which refers to an image with user name projectriff also residing at docker.io.
The image name gcr.io/cf-elafros/knative-releases/github.com/knative/serving/cmd/autoscaler@sha256:deadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeef refers to an image with user name cf-elafros residing at gcr.io.

When an image is relocated to a registry, the domain name is set to that of the registry and the path modified so that it starts with the user name that will own the image.
Draft externals
duffle relocate [INPUT-BUNDLE] [OUTPUT-BUNDLE] --repository-prefix [string]

Relocates the images referenced by a bundle and creates a new bundle with an updated image map. Relocation is restricted to images with type "docker" and "oci". Images of other types are not relocated.
The --repository-prefix flag determines the repositories for the relocated images. Each image is tagged with a name starting with the given prefix and pushed to the repository.
For example, if the repository prefix is example.com/user, the image istio/proxyv2 is relocated
to a name starting with example.com/user/ and pushed to a repository hosted by example.com.
Implementation notes
The path of a relocated image may:

Include the original user name for readability
Be “flattened” to accommodate registries which do not support hierarchical paths with more than two components
End with a hash of the image name (to avoid collisions)
Preserve any tag in the original image name
Preserve any digest in the original image name.

For instance, when relocated to a registry at example.com with user name user, the above image names might become something like this:

example.com/user/istio-proxyv2-f93a2cacc6cafa0474a2d6990a4dd1a0
example.com/user/projectriff-builder-a4a25a99d48adad8310050be267a10ce:v1
example.com/user/cf-elafros-knative-releases-github.com-knative-serving-cmd-autoscaler-c74d62dc488234d6d1aaa38808898140@sha256:deadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeef

The hash added to the end of the relocated image path should not depend on any tag and/or digest in
the original image name. This ensures a one-to-one mapping between repositories. In other words, if:
x maps to y

where x and y are image names without tags or digests, then
x:t maps to y:t (for all tags t)

and
x@d maps to y@d (for all digests d).

The intention is to use https://github.com/google/go-containerregistry for accessing OCI image layouts and remote repositories.

I think the use case you described is exactly what https://github.com/docker/cnab-to-oci does (@simonferquel please jump in if I'm wrong).
You can pull the bundle from a registry, then push it to another (private or not) registry, and cnab-to-oci will update the image references. It even support image promotion pattern.
With @simonferquel we planned to integrate this library to duffle (it's already integrated in docker-app), had no time yet to do it.

I think the use case you described is exactly what https://github.com/docker/cnab-to-oci does (@simonferquel please jump in if I'm wrong).
You can pull the bundle from a registry, then push it to another (private or not) registry, and cnab-to-oci will update the image references. It even support image promotion pattern.
With @simonferquel we planned to integrate this library to duffle (it's already integrated in docker-app), had no time yet to do it.

We have a use case where the bundle is not stored in a registry. Will that also be supported by cnab-to-oci?

We have a use case where the bundle is not stored in a registry. Will that also be supported by cnab-to-oci?

I don't understand the use case described earlier then, it says you want to relocate to another registry.
What do you want to relocate then, if it is not towards another registry?
Or maybe you mean you will relocate the images, but not the bundle itself?
That's not how cnab-to-oci works. See it more as pushing the thick bundle (bundle + the images) to a registry, as a whole.

Or maybe you mean you will relocate the images, but not the bundle itself?

Correct! So it seems that cnab-to-oci won't support our use case. Agreed?

Yes, the goal of cnab-to-oci is to be able to publish bundles with guaranteed immutability from the bundle metadata down to the actual component and invocation images.
It also guarantees that if you have pull rights for the bundle, you also have pull rights for the invocation image and component images.

We have a use case where the bundle is not stored in a registry.

@glyn could you describe this use case?
It's interesting for cnab-to-oci as we may have all the building blocks to do it, and just need to add a top-level function to handle this relocation.

Thanks @simonferquel. I agree that goal is laudable and maybe our use case will change in the future to align with that goal.

@glyn could you describe this use case?

@silvin-lubecki We currently ship distributions of our software as a zipped archive plus a CLI program. Users relocate images to a (typically) private registry using the CLI. There are a number of steps involved and the process is a little complex to document and for the user to follow.
In the future, we are considering shipping a CNAB instead, either as a thin bundle for users with a DMZ, from which they can relocate images (but not the CNAB itself -- see below) from public repositories to a (typically) private registry, or as a fat bundle for users with no DMZ, in which case they can relocate from an OCI image layout in the bundle to a (typically) private registry.
We expect the user to use a standard CNAB runtime, such as duffle, to perform this relocation as well as installation. (The purpose of this issue is to enable duffle to perform this style of relocation.)
We don't currently have a registry in which to host a CNAB and, as CNAB is still being developed, we don't want have the user push the CNAB to their private registry. From a registry perspective, it should just look very similar to today: images are pushed to a private registry and our software pulls the images from the private registry.

I took a look at FixupBundle and the associated command line. These perform a many-one mapping of repositories, whereas we currently use a one-one mapping and would like duffle to do the same (see "Implementation Notes" above).
The main downside of a many-one mapping is that the mapped image names are effectively indistinguishable from each other from a human readability point of view and certainly not easy to relate to the corresponding original image names. In our case, when we relocate kubernetes configuration files containing a number of image names, it's important to distinguish between mapped image names and preserve a readable connection to the original image names. This ensures that the relocated configuration files remain readable by users.
Note that the mapping we currently employ ensures the same authentication/authorisation property as mapping to a single repository: the mapped repositories are authenticated with and authorised to a single user.

@glyn This is definitely a valid issue. One thing that we plan to do (at least for docker-app bundles) is to populate a well known annotation on kubernetes/swarm resources that we create with the original image name.
In the future, we'll add support for the Docker CLI so that things like docker stack ps reports both the real image and the original one.

Let's discuss your comment on PR 671 here rather than in the PR:

Hello [Glyn], as we discussed in the issue, I think we should try to improve cnab-to-oci Fixup instead.
If I understand it correctly, the only blocking point you saw in cnab-to-oci is that we mount/copy all the images in the same repository, and in your case you need to push all the images to their "own" repository? My guess is that this can be easily implemented in the current Fixup command, it is just a matter of where we push the images, but all the logic of pushing is already there.
@simonferquel and I would be pleased to help you write this PR against cnab-to-oci 👍
A second argument for not merging this PR as-is, is that it adds a dependency on github.com/google/go-containerregistry, which I think will "duplicate" with the containerd library used to push/pull bundles to a registry. @radu-matei any thoughts on that point?

There are a number of requirements for duffle relocate which I’d want to make sure are satisfied by an implementation based on cnab-to-oci:

Must preserve content digest of each image during relocation.
Must support a one-one repository mapping with similar usability to that provided by the current PR.
Must support copying images from OCI image layouts to remote registries so that we can relocate thick bundles.
Must be able to relocate a bundle which is not stored in a registry. Isn’t this contrary to the over-arching design of cnab-to-oci?
Should have the same authentication configuration as docker CLI.

Must preserve content digest of each image during relocation.

This is the case, except when using platform filtering (which is off by default)

Must support a one-one repository mapping with similar usability to that provided by the current PR.

That is where your PR would come in :)

Must support copying images from OCI image layouts to remote registries so that we can relocate thick bundles.

Not yet supported, although we want to do that. (we already use containerd under the hood, and it is actually really easy to do

Must be able to relocate a bundle which is not stored in a registry. Isn’t this contrary to the over-arching design of cnab-to-oci?

For now the implementation tries to implement an end-to-end whole bundle relocation. However, we'd like for your PR to refactor the code to bring a little bit of flexibility here (while still providing easy push/relocation APIs for bundle-level operations)

Should have the same authentication configuration as docker CLI.

Already the case
I am convinced the amount of work for bringing this flexibility to cnab-to-oci is actually quite low. Under the hood, the image relocator already works image per image, it is just not exposed (yet).

Ok, that all sounds reasonable, although it might take a while to finalise. May I suggest a practical way forward? cnab-to-oci could provide an equivalent to the registry package of github.com/pivotal/image-relocation and then we could simply re-base the implementation of duffle relocate on that package. (The NewLayout and Add methods are not needed to implement duffle relocate, but will be useful when duffle wants to support creating thick bundles containing OCI image layouts.)


refactor: get bus_demand via Scenario method instead of import

Pull Request doc
Purpose
Get bus_demand via the Scenario.get_bus_demand() method instead of importing the get_bus_demand function, which is being removed in https://github.com/Breakthrough-Energy/PowerSimData/pull/581. Closes https://github.com/Breakthrough-Energy/PostREISE/issues/327.
Testing
Tested manually. With the PowerSimData branch for the PR installed:
>>> from powersimdata import Scenario
>>> from postreise.analyze.transmission.congestion import calculate_congestion_surplus
>>> calculate_congestion_surplus(Scenario(3873))
UTC
2016-10-01 00:00:00<PHONE_NUMBER>36
2016-10-01 01:00:00<PHONE_NUMBER>81
2016-10-01 02:00:00<PHONE_NUMBER>19
2016-10-01 03:00:00<PHONE_NUMBER>91
2016-10-01 04:00:00<PHONE_NUMBER>19
                           ...
2016-10-31 19:00:00<PHONE_NUMBER>30
2016-10-31 20:00:00<PHONE_NUMBER>73
2016-10-31 21:00:00<PHONE_NUMBER>10
2016-10-31 22:00:00<PHONE_NUMBER>75
2016-10-31 23:00:00<PHONE_NUMBER>60
Freq: H, Length: 744, dtype: float64

Time estimate
1 minute.

Tests are failing due to black upgrade and functions calls from calculate_congestion_surplus which is as expected.

Tests are failing due to black upgrade and functions calls from calculate_congestion_surplus which is as expected.

calculate_congestion_surplus fails because MockAnalyze doesn't have get_bus_demand! I need to add that to https://github.com/Breakthrough-Energy/PowerSimData/pull/581 as well.

Tests have been updated to reflect the new expected behavior from Scenario.get_bus_demand for a Scenario in Analyze state. With the https://github.com/Breakthrough-Energy/PowerSimData/pull/581 branch checked out, tests pass locally.


Add QMidiIn class + allow sending/receiving System Exclusive messages

Here it is, the (at least by myself :P) long-awaited QMidiIn class.
I have tried the changes with a (as of yet unreleased and) small test program which interfaces with my Yamaha reface DX where it also retrieves certain data via System Exclusive messages (e.g. the voice name or a change in the FM algorithm used) -- worked fine across all three platforms.

I also really should break the QMidiEvent class out of QMidiFile, and make that the one way to pass MIDI messages to QMidiIn/QMidiOut. The byte-banging just feels like a hack to me now.

Yup.  In a related note: one thing I thought that would be nice was to emit QMidiEvents instead of quint32s, but I quickly gave up fighting with the compiler as I just wanted to see the basic stuff working for now...  Should be part of a later PR imho

Oh and another thing -- should this library still be compatible with Qt 4, or can we drop that and make Travis build it with Qt 5?

Yeah, we can drop Qt 4 at this point; I don't see any reason to keep that around.

Looks good aside from the one comment.

Thanks for the contribution!!


Our Fitness

Project title
I want to add a project of GYM website "OurFitnessCenter" .
Project Description
website will contain 4 sections home , services , about us and contact us .
Stack:

[x] Html
[x] CSS
[x] JavaScript

/assign

/assign

@gurjeetsinghvirdee please assign

@gurjeetsinghvirdee please assign

Assigned


Gtt 1522

Description
I added a script and examples to insert examples to the deployed app.

added examples folder
added lambda to insert entities into the database and copy files into datasets bucket
added one examples dashboard
added readme on how the script works and how to add more.
updated deployment script to handle language
updated cft template for new paramter

Testing
Added unit tests for deployment script.
Ran deployed lambda multiple times to verify in AWS
Setup command line debugging to test locally
Ran deploy.sh several times to check cdk deployment.
Checked inserted examples in app to verify charts are they and look as expected.
Still need to test with the performance-dashboard-on-aws.template . CDK synth templates dont work with parameters as expected
Example Dashboard

Go to https://d2h803e1awe3zo.cloudfront.net/admin/dashboards
Open the dashboard
Verify preview works fine
Publish the dashboard
Verify public view works fine
Verify the https://d2h803e1awe3zo.cloudfront.net/ link to the dashboard works too.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mabreuortega
It looks good! Couple of questions:
1. in ./.gitignore, we are ignoring node_modules.  Why is there a node_modules folder in the top level folder

2. in widget.ts, why is the change with showWithTabs needed for this story?

I reverted the gitignore changes, this was added by VSCode.
Regarding the widget.ts this property was used as boolean but the type in the API was a string. I check and everything is working normally due to the fact that the client have it as a boolean already.


improve sample data / improve docs

Could you please improve the json sample file for product import with some example attributes and the nesting of those?
Please improve docs in terms of "what" can I actually do with this tool. Importing product types, okay, are nested types possible? And please add something like "how to use this tool".
Thanks a lot.

in progress and duplicate to https://github.com/sphereio/sphere-product-import/issues/13

@emtii please have a look at https://github.com/sphereio/sphere-product-import/tree/documentation
and suggest improvements if any

Perfect - thx


Job - fail to create job because of wrong value of spec.template.spec.restartPolicy

rancher/rancher:master-3411-head
Steps:

install rancher in HA mode
go to the dashboard
navigate to Job page
create a Job

Result:

failed to create the job because of the following error

Job.batch "test1" is invalid: spec.template.spec.restartPolicy: Unsupported value: "Always": supported values: "OnFailure", "Never"

but there is no place in the creation workflow to set the value of restartPolicy

here is the payload of the POST request

The bug fix is validated on master-616259e0274b26300169f9c06aa0945b4f31e840-head
The steps are the same as the issue provides. The job is created successfully.


Should we be making AIRR releases citable?

There is a mechanism for this linking Github and Zenodo.
https://guides.github.com/activities/citable-code/

Do we want to consider this? It looks like the AIRR Community has a Zenodo record and @bussec manages this.
https://zenodo.org/communities/airr-community/records?q=&l=list&p=1&s=10&sort=newest
The process one follows is enables github hooks to Zenodo, then when you tag a release it is automatically published to Zenodo.
We do this for iReceptor releases (e.g. https://zenodo.org/records/7430516)

Seems like an extra layer to me, as we'll have a v2 paper to cite.

Not sure I agree - this isn't a question of whether we should make the v2.0 release citable, it is whether we should make all versions citable. We will (we hope) have a paper that matches the v2.0 release, but there is no real mechanism to properly cite any of the intermediate releases. From a FAIR software (or even FAIR standards) and reproducible research perspective, how can I cite that I used v1.4.1 of the python library. Or that my data is in a format that adheres to v1.4.1 AIRR specification (not 1.0, not 1.2, not 2.0).
Yes, you can point to PyPi or a github release tag, but neither are the equivalent of having a DOI/PID for that release.

What would be the benefit of citing a specific version as opposed to just stating the version when you cite the paper? Would this be instead of citing the paper?

For the record: We have been doing this for the last 6 years ;-) see here DOI:10.5281/zenodo.1185414. Somehow the automatic process broke between 1.3.1 and 1.4, I will fix this with the next days.

lol. Well, that settles that then. :)

For the record: We have been doing this for the last 6 years ;-) see here DOI:10.5281/zenodo.1185414. Somehow the automatic process broke between 1.3.1 and 1.4, I will fix this with the next days.

Excellent...

I now had a closer look at this, but I could not figure out why the v1.4.1 and v1.5.0 releases did not trigger the archiving routine, potentially the token expired or was accidentally deleted. I then created a new webhook for the AIRR Standards repo which has been successfully tested and I used it to manually push the v1.4.1 and v1.5.0 releases to Zenodo (using https://github.com/zenodo/zenodo/issues/1463 as a cheat sheet). So I think this issue is solved now, we should however confirm that the next release does trigger the archiving again.

@bussec can we close this? Create a new issue if next release doesn't work...

IIRC, this was resolved.


fix e2e UI tests incorrectly running tests with flights when no notebook name specified and reduce number of test jobs

Description
Fix e2e UI tests incorrectly running tests with flights when no notebook name specified and reduce number of test jobs.
1.) When the e2e UI tests are run without a notebook flag but with a flight, such as in the release build, we don't create a path filter that excludes the tests using flights, so we end up running all UI tests with all flights instead of just the ones for the specified flight, which leads to test failures.
2.) The number of notebook jobs is very high, but it seems that most of those jobs spend more time in the setup (eg 5-10 mins) than running the notebook itself (which varies from 2-8 minutes).  I think it's more prudent to reduce the number of jobs and compute used when we seem to be wasting most of the time in the setup, which could instead be run for multiple notebooks.  Hence, I created 2 groups of notebooks - so that we still parallelize the jobs but significantly reduce the number of jobs, especially since prior to this PR we spent more time in the setup code than running the notebook itself.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1577 (70901f6) into main (5eae3bc) will increase coverage by 8.83%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1577      +/-   ##
==========================================
+ Coverage   88.03%   96.87%   +8.83%
==========================================
  Files         109       12      -97
  Lines        5291      160    -5131
==========================================
- Hits         4658      155    -4503
+ Misses        633        5     -628

Flag
Coverage Δ

unittests
96.87% <ø> (+8.83%)
:arrow_up:

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

Impacted Files
Coverage Δ

responsibleai/responsibleai/_tools/__init__.py

raiwidgets/raiwidgets/error_handling.py

...leai/responsibleai/modelanalysis/model_analysis.py

raiwidgets/raiwidgets/__init__.py

..._flask/environments/credentialed_vm_environment.py

raiwidgets/raiwidgets/error_analysis_constants.py

...sponsibleai/responsibleai/managers/base_manager.py

.../responsibleai/modelanalysis/constants/__init__.py

...nsibleai/responsibleai/_managers/causal_manager.py

...nalysis/erroranalysis/_internal/version_checker.py

... and 87 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 5eae3bc...70901f6. Read the comment docs.


Update Beta yields - enhancements and bugs

New feature: turn off recombination fluctuation
We add an option to produce simulations turning off recombination fluctuation.
Handy to study response decoupling rec fluctuation from other effects.
Bug fix: no electrons if create_S2 is False
In the implementation of BetaYields, the treatment of create_S2 was missing. Now we correctly set to zero the production of electrons in the plugin if the create_S2 of the interaction is set to False.
TODOs

[ ] Introduce an energy threshold for using the BetaYield ( like only >10keV ) - it's impossible to make a function that goes well also there, and nest gives good behavior - show plot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 80.506%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fuse/plugins/micro_physics/yields.py
5
7
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
2610

Relevant Lines:
3242

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

7 of 9 (77.78%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 80.506%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fuse/plugins/micro_physics/yields.py
7
9
77.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
2610

Relevant Lines:
3242

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

8 of 10 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 80.518%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fuse/plugins/micro_physics/yields.py
8
10
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
2612

Relevant Lines:
3244

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

34 of 36 (94.44%)  changed or added relevant lines in 1 file are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.08%) to 80.432%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fuse/plugins/micro_physics/yields.py
34
36
94.44%

Files with Coverage Reduction
New Missed Lines
%

fuse/plugins/micro_physics/yields.py
13
92.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
2606

Relevant Lines:
3240

💛  - Coveralls

Ciao @cfuselli, when you find some time, could you update the topmost description of your PR to account for the recent changes and the merged parallel branch? Thanks for the good work!


Added bower.json

Hey!
First, thanks for this amazing library. It's been really useful for the project that I'm working on.
 This was requested a long time ago as you can see at #50.
I extracted most of the content from package.json and created a bower.json.
After accepting, it will be necessary to release a new version (1.0.5) because bower download the latest version by default. 1.0.4 doesn't have the bower.json file, so...
Please feel free to suggest modifications so we can move forward.
Best,
Vítor Avelino.

1 year)

After being shamed that it's taken me a year, I've merged this, and made other changes. I'll be pushing a version to bower shortly.


Feature/send payment message

based on https://github.com/trustlines-protocol/clientlib/pull/364
closes https://github.com/trustlines-protocol/clientlib/issues/346

Codecov Report

Merging #365 into master will decrease coverage by 0.42%.
The diff coverage is 27.90%.

@@            Coverage Diff             @@
##           master     #365      +/-   ##
==========================================
- Coverage   59.85%   59.43%   -0.43%
==========================================
  Files          21       21
  Lines        1395     1420      +25
  Branches      287      295       +8
==========================================
+ Hits          835      844       +9
- Misses        492      509      +17
+ Partials       68       67       -1

Flag
Coverage Δ

#integration
32.95% <9.30%> (-0.31%)
:arrow_down:

#unit
59.43% <27.90%> (-0.43%)
:arrow_down:

Impacted Files
Coverage Δ

src/Event.ts
69.13% <0.00%> (ø)

src/TLNetwork.ts
83.56% <ø> (ø)

src/typings.ts
100.00% <ø> (ø)

src/Payment.ts
27.34% <10.00%> (-1.61%)
:arrow_down:

src/Messaging.ts
26.53% <16.66%> (+0.34%)
:arrow_up:

src/extraData.ts
82.60% <83.33%> (+6.41%)
:arrow_up:

src/utils.ts
66.44% <100.00%> (+0.44%)
:arrow_up:

Continue to review full report at Codecov.

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


Using large number of concurrent requests

Thank you for the interesting project! 👍
I tried running it against the local service and found that concurrent requests aren't working as expected. The command ./SlowSloth -u http://localhost:9001/courses -m GET -c 100 -delay 10  doesn't create the expected number of connections.
Is there a way to get more details regarding the current test's state while it is running?  So far this is the only thing I'm receiving
Total active connections: 100
Service availability: truee
Total RAM usage: 12 MB

Many thanks 🎄

Hey @rootart,
Thank you for your feedback. If you are using a system like Mac or Linux, you can instantly see the open connections on the port where the application is running on your system by using the following command. I will examine in detail if there is a specific problem with Localhost. Here is an example for you:
sudo lsof -i :9001 | wc -l | awk '{print $1-1}'

Please contact me for more details and information.
Cheers.


feat: ensureInitialize feature

Description
Hi @felangel
Would you mind if dart_frog have a feature or method to ensure all dependencies are initialize correctly before server running serve, cause i have a method to initialize database on my main.dart (custom entrypoint). on development application is working correctly, but when i run from docker using docker compose to make containers my dart_frog application and my postgre database, on first running dart_frog application can't connect to my database, i must run command docker-compose up twice to make dart_frog application and database connected. I try to add delay, but isn't solved my issue
Requirements

[ ] Checklist of requirements to be fulfilled

Additional Context
This my main.dart

This my app.dart to initialize postgres database and do migrate a models to database

This my method to get connection from postgres database

And this my docker-compose

Add any other context or screenshots about the feature request go here.

Hi @ekokurniadi 👋
Thanks for opening an issue!
The custom entry point allows you to execute whatever code you want before calling serve so you should be able to await any connection initialization there (just like in your snippet). Why is the database connection not established? Did you try to debug your application to see when running in docker whether establishing the connection is failing?
I’m not sure I understand the problem. Can you provide a bit more context?

On first running using docker compose up i got some error, the application can't connect to the database, i must run docker-compose up twice to make application and database connected

This doesn’t seem like a dart frog issue and it sounds more like you need to adjust your docker-compose to use restart on failure with a health check. Check out https://stackoverflow.com/questions/31746182/docker-compose-wait-for-container-x-before-starting-y/41854997#41854997 for more info.

i see, i'am sorry about that, i will try to improve my docker compose before. Thank you

No problem! 👍
Closing for now but feel free to comment if you have any follow up questions.

Ok, now my application working nicely on production. Thank you 👍 🥇


⚠ Bofrost.at has degraded performance

In b9f851b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1260 ms

Resolved: Bofrost.at performance has improved in d4b6988 after 30 minutes.


Make it compatible with Bun.

Fast server, slow runtime. Doesn't that mismatch? Basically, uWS is the only package that complaining. But otherwise, it'd be damn good to run hyper on bun.

ahh, it was requested already.


Add API call to convert channels attributes to nodes

Issue
It is possible to either miss or bypass the channels attribute on an input
If this occurs this can fail validation due to input / output type being found to be different types.
(Note: There also appears to be issues with how this attribute can sometimes not propagate properly
with interface inputs and outputs.)
Suggestion
Add a convenience utility to convert implicit channels attributes to explicit nodes (such as an extract,
swizzle etc)

Thanks for this report, @kwokcb, and I'll just add a note that this will be addressed by the design changes in MaterialX 1.39, which includes the replacement of channels attributes with explicit nodes.

Thanks for this original report, @kwokcb, and channels attributes have been removed from MaterialX 1.39 in https://github.com/AcademySoftwareFoundation/MaterialX/pull/1804.


Simple as is: thank you!

Wonderful too for production and devops, thank you very much for taking so much effort into this!

Thanks!


[Text Analytics] Enable RecognizeHealthcareEntitiesTestWithAssertions tests

Describe the bug
Enable the RecognizeHealthcareEntitiesTestWithAssertions in text analytics once the service is deployed to westus2.

Fixed on https://github.com/Azure/azure-sdk-for-net/pull/19248


go.mod: update the Go dependencies so that we can use venafi-connection

The venafi-connection-lib library uses more recent versions of the dependencies in the Venafi Kubernetes Agent. When I ran
export GOPRIVATE=github.com/jetstack
go get github.com/jetstack/venafi-connection-lib@latest

most of the dependencies in this project got updated. This commit captures these changes.

Closing in favor of #552.


Hitting chocolatey rate limits

I'm getting this error when running the startup command on windows low priority nodes
Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (429) Too Many Requests."
At line:167 char:3
+   $downloader.DownloadFile($url, $file)
+   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : WebException

ERROR: The system cannot find the file specified.
D:\Users\PoolAdmin1469656965\AppData\Local\Temp\chocolatey\chocInstall\chocolatey.zip

Looking at the Chocolatey troubleshooting docs I see that there is per IP rate limits which appear to be the cause of this error.
Is there a solution to this?

The pool allocation mode is batch service. And there were around 50 VMs in the pool at the time of the error. I can easily see it reaching 20 per minute in that scenario. I'll try again tomorrow and see if the error happens repeatedly.
From what I can tell the only thing chocolatey is used for is to install python. So I guess I can work around the issue by installing python through another mechanism and then following the generic instructions in the readme?

I mean on the pool are you using cloudServiceConfiguration or virtualMachineConfiguration?
Yes a workaroung would be to install python some other way.
There is also a branch where I rewrote this in go, still need to be fully tested but then it would just require you to download an single file and run it.

Sorry, I misunderstood. The pool is using virtualMachineConfiguration

Should be solved with the go rewrite


grequests merge with requests

@kennethreitz Is there any throughts on merging grequests back into the main, requests repo and or also adding support for Asynio/Trollius.   It's something I would be willing to work on if there was support from you as the founder/main developer for both projects.
Sorry for the comment here wasn't sure the best place to discuss this.

grequests originated within the requests codebase as "requests.async". Many things have been extracted out of requests to make the core library as lean as possible, while providing all the hooks and correct abstractions to allow for these pieces to be provided by third-parties (see-also: oauth, kerberos auth).

Makes sense

Closing — but more comments welcome!


Release

:tada: This PR is included in version 1.72.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Please add LICENSE file

Hello!
Not sure how all the pieces you've used are licensed, nor how you'd like to license this project. However, it is pretty much unusable without a license.

Sure! Will do! Thanks! I will go MIT


replaced Nil with Empty for list context

From IRC discussions as of today I think we should assing () here instead of Nil or Empty.

will change to () and do new pull request


Add a way to increment the version for dirty checkouts

Is your feature request related to a problem? Please describe.
If the HEAD commit is tagged, then gotagger reports the version unchanged regardless of whether there are uncommitted changes. While this may be desirable in some situations, there's no way to adjust this behavior.
Describe the solution you'd like
A flag and config option that lets me control whether and how to increment the version if there are no new commits but the checkout is 'dirty'.

Fixed in 5931142ad209c8fc8f4541155c8d39788ec57d40


Add subcommand APP for mesheryctl

Description
This PR fixes #3510
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3520 (ea13c32) into master (ee57bf8) will decrease coverage by 6.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3520      +/-   ##
==========================================
- Coverage   17.32%   10.47%   -6.86%
==========================================
  Files          37       37
  Lines        3209     3209
==========================================
- Hits          556      336     -220
- Misses       2511     2806     +295
+ Partials      142       67      -75

Flag
Coverage Δ

gointegrationtests
?

unittests
10.47% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
1.27% <0.00%> (-50.43%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.93% <0.00%> (-39.26%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.49% <0.00%> (-22.10%)
:arrow_down:

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 ee57bf8...ea13c32. Read the comment docs.

Please sign-off on your commit(s). To do so for the commits in this PR, execute - https://github.com/meshery/meshery/pull/3520/checks?check_run_id=3078282571
To configure your system to always sign-off, see the instructions in the contributing.md readme.

Please sign-off on your commit(s). To do so for the commits in this PR, execute - https://github.com/meshery/meshery/pull/3520/checks?check_run_id=3078282571
To configure your system to always sign-off, see the instructions in the contributing.md readme.

Not able to sign it with following
git commit --amend --no-edit --signoff
git push --force-with-lease origin feature/suhaniii/3510

@suhaniii, kindly reply in slack

@suhaniii, kindly reply in slack

Hi @Anita-ihuman , Thanks for the checking up :) , I was able to pass DCO this morning.

@leecalcote @Anita-ihuman  @navendu-pottekkat This PR needs reviewers. Adding reviewers is disabled for me.

@leecalcote I think we already have this subcommand under exp by the name application in mesheryctl.


Autocomplete is inserting space between double colons

Just updated from 0.4.9 to latest version 0.4.15 and started having autocompletion problems.
For a field written like so:
Field::

The autocomplete would enter an extra space between the collons, like so:
Field: :

That specific field has been setup to be a multi-file field. Not sure what triggers the issue, but will add that I had some metadata place in a callout like so:
> [!note] My Callout
>
> Field::

As a temporary solution I downgrade to 0.4.10, and the issue got resolved. I tried 0.4.11 and the issue started popping up again.
As a side note, I noted that not all field had autocomplete working (when a Dataview query is assigned). Some field I have setup via a global fileClass and some are tag specific. It's a little bit annoying, btw I am running version 0.4.10 and have this side issue.

As a temporary solution to the issue relating to global fileClass, I have manually added Parent fileClass to each fileClass.
This works for me:

While the following stopped working:

@tbogdal thanks for sharing, same issue for me. Can you please share how you downgrade to previous version?

I have the same issue with autocomplete. In my case I found out that adding three dashes (but not with front matter) triggers this behavior. When I delete any three dashes (which I use to separate sections in my notes) then the autocomplete issue does not appear.

good catch. Should be ok with 0.4.16.

Thanks for the quick update. It works now again in my case. Great work!


Make PR artifacts more useful

Keep data about new failures, new passes and full csv files that were
used for comparison.

Are these the ones the "compared test results" table is build from ? If so, could we embed links in the table to the artifacts that can further explain what was happening ?

Yes, the files are the original csv files (before pr and after pr) and in addition to that, pre filtered csv files that only show new failures and new passes

Nice, so could we craft URLs to this information in the generated pull request comment that contains the results table ?

Converting to draft as adding the url requires some experimentation.

Oh, right. We won't be able to test it here as the workflow that posts the comment has to be already on the main branch. Let me add some logs to CI that generate the link and test it there. If the URL is generated correctly, then I will add the code that appends it to the comment separately.


The plt.show () command is not inserted

Can you paste a link to the CircleCI: build_docs artifact for this function once it's complete? You might need to login to CircleCI (using GitHub credentials) to be able to see the "Artifacts" link, where the doc build for this PR will live once it finished.

Currently none of the plotting examples have plt.show() but it is generated in the documentation http://scipy.github.io/devdocs/generated/scipy.signal.windows.general_cosine.html#scipy.signal.windows.general_cosine

@larsoner https://circleci.com/gh/scipy/scipy/6904#artifacts/containers/0

Thanks @mohmmadd


Disable deletion token

At times, I'd like to be able to deploy this in a way that doesn't provide a popup for a deletion token, just saving and providing a URL.
Would you be willing to allow a config option to disable the deletion token creation/popup?

That sounds nice, I will implement it. Thank you!

I pushed that feature to the develop branch to test it a few days until I'll fully release it. If you want to set it up for yourself, you may use the ghcr.io/lus/pasty:staging Docker image + tag ^^
Thanks for your suggestion. If you have any further problems/suggestions etc. feel free to open another issue!


Check file system watching support on Windows ReFS

cc: @gradle/execution

The new file Windows system with the nice snapshotting etc. features seems to work with a couple test failures: https://ge.gradle.org/s/4wjzwyqvafqc2
Looks like on ReFS we also get a MODIFIED before the REMOVED event when a directory is removed (similar to what NTFS does when removing files).
All the other tests are passing.


Missing Javascript language and its identifier

Version: 1.45.1 (user setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08:27:35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Description
I can't find Javascriptlanguage in Select Language mode. ANYWHERE. I find all the extensions to use it better but, can't find the javascript option.
Vs code shows .js files as plain text and doesn't show javascript file identifier.
this happened after suddenly chrome has crashed and I reinstalled chrome.

Does this issue occur when all extensions are disabled?: Yes

In the extensions view search @builtin javascript. Is the JavaScript and TypeScript language features extension disabled?

hi, sorry for the delay. No, nothing has been disabled. the problem was my corrupted hard drive I guess. files were being corrupted and I didn't know that a failed hard drive can corrupt parts of data randomly. my system got shut down and i have to replace the hard drive. now there is no problem. i feel the issue with VS code is also related to it.


Issue243 less verbose warning for missing sub package

Improvements

The warning message is much more concise (1 header, 1 line per subpackage, 1 bottom line)
=> users have the minimum info without pollution
The error messages are still stored internally in tofu.dsub and accessible to developers, but just not displayed
=> when an optional subpackage import failed, a developer can still why

Example on ITER:
In  [1]: import tofu as tf
/home/ITER/vezined/ToFu_All/tofu/tofu/__init__.py:99: UserWarning:
The following subpackages are not available:
    - tofu.mag
  => see tofu.dsub[<subpackage>] for details on error messages
  warnings.warn(msg)

In  [2]: tf.dsub['mag']
Out [2]: 'pywed not available => no tofu.mag'

Issues
Fixes, in devel, issue #243

Hello @Didou09! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Comment last updated at 2019-11-05 14:10:40 UTC

Codecov Report

Merging #244 into devel will decrease coverage by <.01%.
The diff coverage is 56.41%.

@@            Coverage Diff             @@
##            devel     #244      +/-   ##
==========================================
- Coverage   43.95%   43.94%   -0.01%
==========================================
  Files          70       70
  Lines       21278    21289      +11
==========================================
+ Hits         9352     9355       +3
- Misses      11926    11934       +8

Impacted Files
Coverage Δ

tofu/imas2tofu/__init__.py
80.95% <100%> (-5.42%)
:arrow_down:

tofu/version.py
100% <100%> (ø)
:arrow_up:

tofu/mag/__init__.py
33.33% <11.76%> (-18.85%)
:arrow_down:

tofu/__init__.py
91.66% <89.47%> (+12.25%)
:arrow_up:

Continue to review full report at Codecov.

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

Had been closed without being merged the first time...

done, sorry for that 🙈


Mobile video buffer

Video Buffers on Mobile and not on PC
When watching a video on plyr the video does not buffer on the PC but it buffers when on mobile. It buffers so much that it is irritating. Any solutions for that.
I am using hls js.

same problem please how can solve this problem in mobile browser


Add required indicators to fields

Added required indicator in sql migration extension experience:

For subscription field
For database backup fields

Linked issue: https://github.com/microsoft/azuredatastudio/issues/25919
Task link: https://dev.azure.com/msdata/Database Systems/_workitems/edit/3563160
Fix:

@akshika898 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
30720

Relevant Lines:
68916

💛  - Coveralls

@akshika898 , can you add before pictures?

@akshika898 , can you add before pictures?

Added


fix: move two trivial theorems over booleans to Bool/Basic

This eliminates an unnecessary dependency from Num.Bitwise to Bitvec.Defs.

bors merge


Some of the operators in the model are not supported by the standard TensorFlow Lite runtime and are not recognized by TensorFlow. If you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CONV_2D, TANH. Here is a list of operators for which you will need custom implementations: DEPTH_TO_SPACE.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
# Copy and paste here

Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@jichangsheng ,
Can you please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version.Thanks!

As the error suggests you have to build custom operation for DEPTH_TO_SPACE
See https://www.tensorflow.org/lite/guide/ops_custom

@jichangsheng ,
Can you please try @ymodak solution and let us know your feedback?Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

It worked and successfully produced the .tflite file.
But when I want to use this file with tf.lite.Interpreter(.tflite), An error raised as
‘ValueError:Didn’t find custom op for name ‘DEPTH_TO_SPACE’ with version 1，Registration failed ‘
--
发自我的网易邮箱手机智能版
在 2019-08-13 18:16:07，oanush<EMAIL_ADDRESS>写道：
Closed #31227.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Bump github.com/layer5io/meshkit from 0.2.11 to 0.2.14

Description
This PRupdates Meshkit to v0.2.14
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Move err.Error() to long description fields

Making changes.


CMake Tools exported API is undefined

Brief Issue Summary
I'm trying to revive an old extension that I wrote for one of my projects and I came upon this roadblock.
When trying to access the cmake tools API from another extension the exports variable (or result of activate() I tried both) is undefined.
Looking through the code I found
// TODO: Return the extension API

at https://github.com/microsoft/vscode-cmake-tools/blob/develop/src/extension.ts#L1755
Which I guess means that this is not currently implemented.
Quick blame shows that it was broken in f6c49d3855bcdd38fc7fc3c3e6a9abd1b73fc4f8 ~3 years ago, so I imagine that's not a very popular feature. Still it would be nice if it got fixed, since the actual API implementation seems to be in place.
Thank you for delivering this amazing extension and have a wonderful day!

Hi @enbyted, yes, the API was removed before we assumed ownership of the extension.  It needs some major work before we can re-enable it.  We are tracking the request in #494, so I will close this as a duplicate.  If you click on the 👍 in that issue that will help us track the demand for this feature.

Ah, I'm sorry for creating a duplicate, I tried to search for existing issue, but got way too many results to sift through.
Thanks for the pointer, vote added!


AWS EC2 needs Helm

During the works on the testdriver, we noticed that the AWS EC2 template needs Helm installed.

False alarm, sorry, the error was on a feature branch, so no hotfix required.


Inconsistencies in job's state

kue version: 0.11.6
I'm experiencing a weird phenomena with some of our jobs. I have jobs in {q}:jobs:active ZSET that have their state set to failed.
I've tried to figure out how this is possible but I couldn't. My first suspect was that there was some external restart of the process during the job.state() function but the MUTLI is used there so it shouldn't cause any inconsistencies.
There is this queue.checkActiveJobTtl() mechanism that runs every second and in our case on some events we have a lot of these inconsistent jobs and these get processed every second which is causing an unnecessary load on our servers.
The simplest solution would be to add:
job._state = 'active';

here: https://github.com/Automattic/kue/blob/master/lib/kue.js#L245 however on one server I've noticed that we have inconsistency with jobs in the "incative" box (these are in inactive ZSET but their state is set to "failed")

Finally I know what's the problem.
So it's the refreshTtl function that is putting these jobs back to active list: https://github.com/Automattic/kue/blob/master/lib/queue/job.js#L346
This refreshTtl function is called when progress is set.
The thing is that we don't always wait for the progress callback to be called
So from time to time, a job finishes but later the progress (thus refreshTtl) runs and it adds the job back to active zset.

Unfortunately refreshTtl and Job.prototype.progress do not accept callbacks so it's impossible to fix it on our side.


docker registry delete download counts on push

if you push a docker image to same existing tag, download counts are erased.
to reproduce:

push a docker image with tag :latest
delete image local and download image -> download count shows 1
push a new image with tag :latest
download count is 0 again

gitea version: v1.17.0 - main

@KN4CK3R is this intended?

No, fixed in #20728


ValueError: Box has no area

When I use my data,This error occurred.Is that because of the problem with my data?

you don't care this error or delete that line code.


[2.0.0.rc3] Messages with quotes get truncated from Discord to Game

Heyo, a fun new one! Discovered that messages "using quotes" mid-sentence when sending from Discord to game get truncated before the first quote mark. Example verified. Messages from game to Discord are not affected. Parentheses do not trigger the bug.
Trevor — Today at 9:43 PM
testing "this part in quotes might not go through" testing
TrevorBot — Today at 9:43 PM
Server: [Trevor] testing
Got a player in-game to send the same message. Came thru to Discord just fine.
charname: testing "this part in quotes might not go through" testing

I've replicated and identified the root cause, and it's messy. Adding quotes is breaking the syntax of the commands the bot sends to the game. The only current solution I'm finding is to remove quotes from messages.
I've applied this change immediately, as I suspect that this could open up some other nasty issues.
This is obviously not an ideal fix. I'm looking into some ways that we might be able to work around it.

I've replicated and identified the root cause, and it's messy. Adding quotes is breaking the syntax of the commands the bot sends to the game. The only current solution I'm finding is to remove quotes from messages.
I've applied this change immediately, as I suspect that this could open up some other nasty issues.
This is obviously not an ideal fix. I'm looking into some ways that we might be able to work around it.

Just enough knowledge here to be dangerous but miss important fundamental concepts... at the risk of teaching you how to suck eggs, would it be possible / useful to treat the message internally as HTML so you can use something like " and similar tokens? Does Discord accept HTML strings?

Discord takes messages in the form of Markdown, i.e. the same as it's used here. Unfortunately this will not work, as this issue falls in the hands of the game rather than Discord. (I.e. if you type a message with quotes into the game console, it will do the exact same thing that you've replicated in Discord)
My first attempt to fix this was to use an escape character(''), which Discord and most common software recognizes. Unfortunately, 7 Days to Die does not, which doesn't leave us with many options.


issue #94 - initial changes for supporting VS Code outline panel for cucumber feature files

@aslakhellesoy ,
I have added initial changes for supporting VS Code outline panel for Cucumber feature file. The way it works is: whenever a .feature file is opened, it shows Scenarios and Feature on outline panel - see the screenshot below. I will add support for other keywords in feature file such as Background, Rule etc, once this approach is approved.

🤔 What's changed?
Added a new class for VS Code document symbol provided.

Adding the initial changes for supporting VS code outline panel for Cucumber feature file.
⚡ What's your motivation?
The outline panel is really a nice and cool feature to be implemented with our plugin.

🏷 What kind of change is this?
It is a new feature.

:bank: Refactoring/debt/DX (improvement to code design, tooling, documentation etc. without changing behaviour)
:bug: Bug fix (non-breaking change which fixes a defect)
:zap: New feature (non-breaking change which adds new behaviour)
:boom: Breaking change (incompatible changes to the API)

♻ Anything particular you want feedback on?
It supports only Feature, Scenario keyword in .feature file. I will add support for other keywords: Rule, Background etc.

📋 Checklist:

[x] I agree to respect and uphold the Cucumber Community Code of Conduct
[ ] I've changed the behaviour of the code

[ ] I have added/updated tests to cover my changes.

[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[ ] Users should know about my change

[ ] I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

This text was originally generated from a template, then edited by hand. You can modify the template here.

@aslakhellesoy ,
It is my pleasure!! Thanks so very much for all your insights!!
I totally agree with you this feature should be part of Cucumber Language Service. Also, the we must use a actual parser!
I will be interested in giving this approach a shot :) No problem, it happens.
Could you please create an issue on Cucumber Language Service for this feature and assign that to me? Also, if you have other ideas for new features, please feel free to assign to me :)
Cheers!

Sure thing. See https://github.com/cucumber/language-service/issues/98


Coq poller crash after internet connection glitch

My internet connection had a temporary glitch, and this crashed the Coq python poller, with the following backtrace:
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 603, in urlopen
    chunked=chunked)
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 387, in _make_request
    six.raise_from(e, None)
  File "<string>", line 2, in raise_from
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 383, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/lib/python3.7/http/client.py", line 1321, in getresponse
    response.begin()
  File "/usr/lib/python3.7/http/client.py", line 296, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python3.7/http/client.py", line 257, in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
  File "/usr/lib/python3.7/socket.py", line 589, in readinto
    return self._sock.recv_into(b)
ConnectionResetError: [Errno 104] Connection reset by peer

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 641, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/lib/python3.7/site-packages/urllib3/util/retry.py", line 368, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/usr/lib/python3.7/site-packages/urllib3/packages/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 603, in urlopen
    chunked=chunked)
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 387, in _make_request
    six.raise_from(e, None)
  File "<string>", line 2, in raise_from
  File "/usr/lib/python3.7/site-packages/urllib3/connectionpool.py", line 383, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/lib/python3.7/http/client.py", line 1321, in getresponse
    response.begin()
  File "/usr/lib/python3.7/http/client.py", line 296, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python3.7/http/client.py", line 257, in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
  File "/usr/lib/python3.7/socket.py", line 589, in readinto
    return self._sock.recv_into(b)
urllib3.exceptions.ProtocolError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "poller_coq.py", line 91, in <module>
    Poller_Coq().run()
  File "/home/armael/Prog/proving-contest-backends/poller.py", line 104, in run
    response = requests.get(url, verify=True, headers=self.headers)
  File "/usr/lib/python3.7/site-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python3.7/site-packages/requests/adapters.py", line 498, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))

What is the best way of handling this kind of failure? Typically I'd like the poller to restart itself if that happens.

A quick fix would be to catch the error in the polling loop of poller.py and just try again. Is that fine, or would you prefer some more sophisticated solution?
If a submission was polled by a judge, the restapi reserves that submission for some time. After that the submission is free again and gets assigned to the next judge that polls.
So if a poller fails (because of some strange issue, i.e. internet connection glitch) it will just wait until the submission is available again (by looping the polling loop) and then try judging again.

A quick fix would be to catch the error in the polling loop of poller.py and just try again. Is that fine, or would you prefer some more sophisticated solution?

That's perfect.

see 007afb2

Thanks!


Snowflake Cookbook Coginiti Pro Collection

Hello,
I have created a Coginiti Pro collection with the book code. It is equivalent to add a folder to Coginiti, IO guess it is a matter of preference. One difference to note is that Catalog code can be referenced by queries as code snippet reusable and sharable by multiple queries. It can also be parameterized.
https://1drv.ms/u/s!AsIn9sExmmCdk5xF8EONcRUstkZu2A?e=Tq65U6

Hi @paddelia
Thank you for your contribution, I am sure this will be useful to the other users of the book.


Upgrading Elasticsearch from 0.10.0 to 1.12.0

I'm trying to upgrade an old version of Elasticsearch to the new one by updating the version number, and I am getting the following error:

Open Distro Security not initialized.

To Reproduce
Steps to reproduce the behaviour:

Updated the image number in my Docker compose file.
Since I got the 'not initialized' error by going to https://localhost:9200/, I tried to run the securityadmin.sh migration suggested in the docs.
I get the following error:

sh-4.2# ./plugins/opendistro_security/tools/securityadmin.sh -migrate -cd ./plugins/opendistro_security/securityconfig/ -icl -nhnv -cacert config/root-ca.pem -cert config/kirk.pem -key config/kirk-key.pem
Open Distro Security Admin v7
Will connect to localhost:9300 ... done
Connected as CN=kirk,OU=client,O=client,L=test,C=de
Elasticsearch Version: 7.10.0
Open Distro Security Version: <IP_ADDRESS>
Contacting elasticsearch cluster 'elasticsearch' and wait for YELLOW clusterstate ...
Cannot retrieve cluster state due to: null. This is not an error, will keep on trying ...
Root cause: MasterNotDiscoveredException[null] (org.elasticsearch.discovery.MasterNotDiscoveredException/org.elasticsearch.discovery.MasterNotDiscoveredException)

Try running securityadmin.sh with -icl (but no -cl) and -nhnv (If that works you need to check your clustername as well as hostnames in your TLS certificates)
Make sure that your keystore or PEM certificate is a client certificate (not a node certificate) and configured properly in elasticsearch.yml
If this is not working, try running securityadmin.sh with --diagnose and see diagnose trace log file)
Add --accept-red-cluster to allow securityadmin to operate on a red cluster.

Configuration (please complete the following information):

I'm using Docker Compose with the following settings:

environment:
- network.host=<IP_ADDRESS>
- http.cors.enabled=true
- http.cors.allow-origin=*
- http.cors.allow-credentials=true
- node.name=dev-core-elasticsearch
- node.master=true
- node.data=true
- network.publish_host=localhost
- discovery.type=zen
- discovery.seed_hosts=localhost
- discovery.zen.minimum_master_nodes=1
- path.repo=/usr/share/elasticsearch/snapshots

Relevant information
I get the following output in the Docker logs:

[2021-01-07T21:20:50,799][WARN ][o.e.c.c.ClusterFormationFailureHelper] [dev-core-elasticsearch] master not discovered yet, this node has not previously joined a bootstrapped (v7+) cluster, and [cluster.initial_master_nodes] is empty on this node: have discovered [{dev-core-elasticsearch}{NLRgylZtTHqUbCj4u7IM5A}{OSlOXlL_Sp-QiEcGoYFD-w}{localhost}{<IP_ADDRESS>:9300}{dimr}]; discovery will continue using [] from hosts providers and [{dev-core-elasticsearch}{NLRgylZtTHqUbCj4u7IM5A}{OSlOXlL_Sp-QiEcGoYFD-w}{localhost}{<IP_ADDRESS>:9300}{dimr}] from last-known cluster state; node term 0, last-accepted version 1 in term 0

Hi @daudich seems like this is an issue related to the opendistro security plugin migration.
https://opendistro.github.io/for-elasticsearch-docs/docs/upgrade/1-0-0/#security-index
I will transfer you to the corresponding security repo. Thanks.

Hi @daudich seems like this is an issue related to the opendistro security plugin migration.
https://opendistro.github.io/for-elasticsearch-docs/docs/upgrade/1-0-0/#security-index
I will transfer you to the corresponding security repo. Thanks.


support for BatchedInferencePipeline

faster whisper recently added BatchedInference support in version 1.1.0, allowing up to 4x faster transcription when transcribing large files.
Is there any plan to add this to this server?
Here are my test results, on my RTX 3090:

Model
Batch Size
Processing Time (s)
Total Time (s)
Speedup vs No Batch

Large v2
16
51.59
53.39
3.29x

Large v2
9
54.34
56.10
3.12x

Large v2
8
55.21
56.99
3.07x

Large v2
6
61.41
63.35
2.76x

Large v2
No Batch
169.75
171.50
1.00x

Turbo
16
33.37
34.65
2.48x

Turbo
8
33.39
34.52
2.48x

Turbo
No Batch
82.75
83.93
1.00x

(This was for a 1 hour audio file, Total Time includes the time it takes to load the model)
I did some testing using the faster whisper server, and its speed matches up with large v2 no batch.
When using smaller audio files (30 seconds) the speed up was around 24% on batch 8 and 16.
Additionally: can we treat the requests of multiple different users as one batch?

Yep, I'll definitely be bumping the faster-whisper version and adding support for batched inference
I like the idea. I'll take a look at how to implement this.


Address @casz review comments from EWM PR#68 (JCasC support)

This PR will attempt to address all feedback from @casz for https://github.com/jenkinsci/external-workspace-manager-plugin/pull/68. Once the work is finished and the changes approved, this PR can be merged by a user with write access on this repository to have the commits contribute directly to https://github.com/jenkinsci/external-workspace-manager-plugin/pull/68.
Review comments addressed:

Deprecated ConfigAsCodeTest class replaced with ConfiguredWithCode annotation to load .yaml configuration.
JenkinsConfiguredWithCodeRule replaces JenkinsRule.
Deleted unused "obsolete-configuration-as-code.yaml" file
Export configuration unit tests have been rewritten

In-progress - Determine cause of this configuration error: https://github.com/jenkinsci/external-workspace-manager-plugin/pull/68#discussion_r303040400
@martinda @imaffe

If you feel comfortable with the changes you can merge them to have the commits contribute to PR#68 (JCasC support for EWM).

This all wrong 😆 You basically removed the tests 😓

Haha! Thanks for the review, I’ll sort this out.

@casz I have implemented your suggestions. I think things are working properly now with the ConfiguredWithCode annotation. I am working on rewriting the unit test.

Export configuration unit tests have been fully rewritten @casz  @martinda

I have corrected some configuration-as-code.yaml indentations and removed the line for
jnlp:
from the commented out nodes configuration code (for https://github.com/jenkinsci/external-workspace-manager-plugin/pull/68#discussion_r295124968). It was these things that were causing the test failures. Does something different need to be passed through jnlp: or can this line stay omitted? @martinda
If the line can stay removed and the rest of the PR seems sensible, then all of @casz review comments from https://github.com/jenkinsci/external-workspace-manager-plugin/pull/68 have been addressed.


Add a from me banner and fix grunt watch

What does this PR do? (please provide any background)

As we are an open source project I think we should link to our open source repo on GitHub, and what better way then using the banners provided by them.
We have discovered the Watch task is broken from another PR, this is also fixed in this branch.

What tests does this PR have?
None
How can be this tested?
None
Screenshots / Screencast

What gif best describes how you feel about this work?

Developer Definition of Done/Quality Checklist (for PR author to complete BEFORE code review):

[x] I have checked the contributing document and I'm happy for this to be reviewed.

Software Engineer or Developer review:

[x] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[x] I’ve checked for coding anti-patterns.
[x] I’ve checked for appropriate test coverage.
[x] I’ve run all the tests locally and they pass.

Software Engineer or project guru review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve run all the tests locally and they pass.

:+1:

taken

still get the following ulimit related error on my machine:
http://grab.by/HJyg

if you change the docs/package.json to:
docsBuild: {
  command: 'cd docs; npm run build && cd -'
},
docsStart: {
  command: 'cd docs; npm start && cd -'
},

Then if the docs fail the error will bubble up to the main grunt task

this works for me now, i still get the initial server not found error, but this is due to the excessive separation of grunt tasks in the docs dir. This can be resolved another time.


Features/408 code generation refactor

…y model to its own Generator https://github.com/NoxOrg/Nox.Generator/issues/407
[Nox.Generators] [Odata] Refactor ODataModelGenerator extract EntitiyDto model to its own Generator https://github.com/NoxOrg/Nox.Generator/issues/408
Refactoring CodeGenerator.

Wrong source branch


METADATA.jl/kNN needs a version bump

The version of kNN pointed to by 0.0.0 in METADATA.jl/kNN is still using Stats rather than using StatsBase, which causes using kNN to fail with:
ERROR: Stats not found
 in require at loading.jl:47
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
 in reload_path at loading.jl:152
 in _require at loading.jl:67
 in require at loading.jl:51
while loading /home/keithc/.julia/v0.3/kNN/src/kNN.jl, in expression starting on line 6

Bumping METADATA.jl to the latest kNN update should resolve the 'Test Status: Package doesn't load'  reported by http://pkg.julialang.org/.

Afte new instaltion of kNN:
_
_       _ ()_     |  A fresh approach to technical computing
()     | () ()    |  Documentation: http://docs.julialang.org
_ _   | |  __ _   |  Type "?help" for help.
| | | | | | |/ ` |  |
| | || | | | (| |  |  Version 0.4.0 (2015-10-08 06:20 UTC)
/ |_'|||_'_|  |  Official http://julialang.org/ release
|__/                   |  x86_64-w64-mingw32
julia> Pkg.installed()
Dict{ASCIIString,VersionNumber} with 49 entries:
"ForwardDiff"        => v"0.1.2"
"Tk"                 => v"0.3.6"
"ArrayViews"         => v"0.6.4"
"Compat"             => v"0.7.8"
"Calculus"           => v"0.1.14"
"GZip"               => v"0.2.18"
"Clustering"         => v"0.5.0"
"StatsFuns"          => v"0.2.0"
"Cairo"              => v"0.2.31"
"HttpParser"         => v"0.1.1"
"DataFrames"         => v"0.6.10"
"Blosc"              => v"0.1.4"
"Distributions"      => v"0.8.7"
"LibExpat"           => v"0.1.0"
"FixedPointNumbers"  => v"0.1.1"
"SHA"                => v"0.1.2"
"LinearAlgebra"      => v"0.0.0-"
"kNN"                => v"0.0.0"
"HDF5"               => v"0.5.7"
"ColorTypes"         => v"0.2.0"
"Polynomials"        => v"0.0.4"
"MbedTLS"            => v"0.2.0"
"FileIO"             => v"0.0.3"
"Dates"              => v"0.4.4"
"HttpServer"         => v"0.1.5"
"PDMats"             => v"0.3.6"
"SortingAlgorithms"  => v"0.0.6"
"WinRPM"             => v"0.1.13"
"Docile"             => v"0.5.19"
"LinearLeastSquares" => v"0.0.10"
"JSON"               => v"0.5.0"
"StatsBase"          => v"0.7.4"
"Distances"          => v"0.2.1"
"BinDeps"            => v"0.3.19"
"Reexport"           => v"0.0.3"
"Graphics"           => v"0.1.3"
"NaNMath"            => v"0.1.1"
"URIParser"          => v"0.1.1"
"HypothesisTests"    => v"0.2.10"
?                    => ?
julia> using kNN
ERROR: LoadError: ArgumentError: Stats not found in path
in require at loading.jl:233
in include at boot.jl:261
in include_from_node1 at loading.jl:304
in require at loading.jl:243
while loading C:\Users\SAMSUNG2.julia\v0.4\kNN\src\kNN.jl, in expression starting on line 6
julia>


Less hackish solution for dynamic items

As suggested on #46, I'm making a pull request for my take on solving dynamic items in a better way. This ended up perhaps a bit more extensive than ideal, but I think it's solid. The pull request includes updated documentation and a couple of tests. I'm open to feedback and making improvements on this if required.
Some notes on the changes involved:

To maintain backwards compatibility, the new dynamic behaviour is turned on using a new dynamic prop on Sortable. For users not using that prop, everything should be the same as always (the tests pass, at least).

Because the sortable was clobbering the keys the user defined for the sortable items and replacing them with the sortable index, dynamically updating the internal lists on sort meant the original key data for my sortable items got lost when I updated the children, and they'd get their props swapped around instead of the actual elements being reordered. I figured clobbering the user's keys is kind of dodgy anyway, so I made it only set a key if the item does not already have one. As far as I can tell there's no realistic scenario in which this would impact a real-world user, but by all means correct me if I've missed something.

The dragging item now gets assigned the key '_dragging', since otherwise without the key-clobbering you'd get key conflicts if the user had defined a numerical key that happens to be equal to the length of the array.

The original dynamic demo had the remove button do its thing on mousedown; this led to a bizarre issue where the removed item would be rendered on top of the first item, because the sortable was rerendered before the draggingIndex was reset on mouseup and then doesn't get rerendered again until the user starts to move something. If there was some reason to use mousedown there that I'm missing, some better solution to that is probably required, but as far as I could tell making it into a click handler instead solved the problem just fine.

Coverage decreased (-0.2%) to 94.857% when pulling 9e85c766134c90f2f455db6a46d78faeae56a8eb on antialiasis:master into acdc46f2dc5331a8da04ff2b37996ae2f955b565 on jasonslyvia:master.

Great job, it looks much more flexible than before!

I must say this is such a brilliant PR, and since it might break the existing dynamic usage and I fail to figure out an appropriate way to add warning, maybe this should be released as 2.0.0?

The original dynamic demo had the remove button do its thing on mousedown

That's because when I was composing this demo, sortable had a bug that might tend to move when you  click inside a sortable item (such as the remove button),  so I just switched to onMouseDown to dodge this pitfall.
It should work with onClick now because I've added more solid check for that.

Hmm, will it break existing dynamic usage? Since the previous dynamic hack didn't use any special behaviour of the lib and just made use of React's handling of keys on Sortable, and the PR has the new behaviour apply only when the dynamic prop is set (and only changes the handling of keys on sortable items, which React will ignore if a parent's key has changed), I wouldn't think it'd change anything for someone using the old method... but I may have missed something.
(For backwards-compatibility purposes, it would also be possible to keep the old key-clobbering behaviour when the dynamic prop is falsey, although real-world use that somehow relies on user-defined keys being clobbered seems unlikely. Aside from that, the only thing the PR changes is conditioned on the new dynamic prop.)
But if you'd rather just release it as 2.0.0, that's fine by me too.

Hmm, you're right, the old hackish way would just work, but we can't inform them there is a new less hackish way to do dynamic sorting.

I guess we're good to go, thanks again for your brilliant contribution!


locationd: replace defaultdict with dict

The use of defaultdict  may not be optimal, given how frequently it is updated and read.
Replaced it with a standard dictionary to make key initialization and default values more explicit, reducing overhead and enhancing code clarity.

What metric does this improve?

I'm not entirely sure, as I haven't tested it. While defaultdict impacts opendbc performance, this change likely won't significantly affect locationd due to its complexity. The main benefit of replacing defaultdict with dict is improved code clarity, making key initialization and default values more explicit, which aligns with what we've done in SubMaster.


Fix LM state carryover for RNN-T

Added missing lines.

Codecov Report

Merging #271 (c1e726e) into master (26b9bcb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #271   +/-   ##
=======================================
  Coverage   64.44%   64.45%
=======================================
  Files          91       91
  Lines       10415    10417    +2
=======================================
+ Hits         6712     6714    +2
  Misses       3703     3703

Impacted Files
Coverage Δ

...eural_sp/models/seq2seq/decoders/rnn_transducer.py
85.90% <100.00%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Not Opening

Added all the things as mentioned but not working. It is not showing anything .

Please describe what you’ve already tried. Share some part of the code.
Are you getting any errors? Have you tried running the example or comparing your code with it?

return (

<TouchableOpacity style={styles.backButtonTouch} onPress={()=>this.props.start()}>

<Image style={styles.button} source={require('../assets/settingswheel.png')} />

)

Here above is the code where i am using it but it doesn't show anything

@Deep1210 I edited your comment to make it a code block, so that it renders correctly. For more on how GitHub uses Markdown, check out this guide.

@Deep1210 I wonder why you have wrapped the copilot step inside a TouchableOpacity. Do you want to use your first walkthrough step to tap to initiate the walkthrough?

Sorry to disturb but now it is running i have not started it in componentDidMount() function


tzlocal not listed as a dependency

I'm getting this error when running mailosaur Exception ModuleNotFoundError: No module named 'tzlocal'
I've noticed you add tzlocal to requirements.txt but not `setup.py so I think that's the problem.

Great catch, thank you! This has been fixed and released in version 6.0.5 now.


Add colorize & translateTime options

[x] Add second param to pass options to pino-pretty (closes #4 )
[x] Add colorize option (closes #3 )
[x] Add translateTime option that uses prettifyTime function directly from pino-pretty to have same formatting in both logs
[x] Update failing tests
[x] Add tests
[ ] Could set common options across both opts objects from first object itself (Thoughts?)

Would you mind adding some unit test for this?

Working on those, just wanted to get some feedback if everything looks alright at a glance.

The colorize test always fails, could be due to some string parsing in tap or through. Any ideas? I've added the test but marked it as todo

@mcollina Done from my side (except the colorize tests)

@rohit-smpx have you check how are we testing those in pino-pretty? The tests should pass.
https://github.com/pinojs/pino-pretty

@rohit-smpx have you check how are we testing those in pino-pretty? The tests should pass.
pinojs/pino-pretty

Fixed. Needed to use the unicode representation of \e

Would you mind to add some more docs for this feature?

Added!

Can this be merged (& released) or waiting for a review from someone else?

We are waiting for @davidmarkclements or @jsumners!

@mcollina Any update on this?

Released as v2.1.0.


analysis server fatal crash loop working on dartdoc

@scheglov
This started in the middle of editing/refactoring and the server seems to crash repeatedly now.  I will save the state of my tree if you want to take a look.  This crash report is from 6406f79fcb1bb931fb521690e7f7693c1d247eee but have verified it happens on the last two dev releases and 2.8.1 stable, too.
Analyzer Feedback from IntelliJ
Version information

IDEA IC-192.7142.36
2.9.0-edge.6406f79fcb1bb931fb521690e7f7693c1d247eee
IC-192.7142.36, JRE 11.0.4+10-b304.77x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.15.4, screens 3840x1080  Retina

Exception
Dart analysis server, SDK version 2.9.0-edge.6406f79fcb1bb931fb521690e7f7693c1d247eee, server version 1.27.4, FATAL error: NoSuchMethodError: The getter 'element' was called on null.

For additional log information, please append the contents of
file:///private/var/folders/b8/jg3_spkj02d51bcz3fsb5jjc000b33/T/report10.txt.
report10.txt

This could be related to #38067.

Closing, this is a duplicate of #38067.  I had added a super call in a mixin unintentionally referencing an interface member; when I stopped doing that, the analyzer stopped crashing.


New Game: Sonic Mania

Tried a few ideas and this one is actually the best IMO even though it's simple. You only have to collect rings, but it only starts progressing the quest when you have over 20. This means that you want to play well and keep your rings, because recollecting the ones you lost won't count for much (if anything) and will slow you down.

Tested, works like a charm on Sonic Mania without "Plus DLC".
The quest fills its function pretty well, can only be slitghly abused with "Blue rings" monitors which make all of your rings fall instead of 20 max.
In case you're looking for ideas, I have quickly studied retro Sonic games to add them and came up with those quests:

Monitors broken
Ring bonus (= rings carried up to the end of level)

Going to merge it anyway because it's already a very satisfying addition to the library as it is 👍


403 Forbidden

As of the last couple hours, I am getting 403'd on every channel.
2019/10/12 20:00:12 [xTeVe] Streaming Status:      Playlist: M3U - Tuner: 0 / 1
2019/10/12 20:00:21 [xTeVe] Buffer:                true [xteve]
2019/10/12 20:00:21 [xTeVe] Buffer Size:           8192 KB
2019/10/12 20:00:21 [xTeVe] Channel Name:          USA
2019/10/12 20:00:21 [xTeVe] Client User-Agent:     Lavf/58.9.100
2019/10/12 20:00:22 [xTeVe] Streaming Status:      Playlist: M3U - Tuner: 1 / 1
2019/10/12 20:00:22 [xTeVe] Streaming Type:        [HLS / M3U8]
2019/10/12 20:00:22 [xTeVe] Streaming URL:         http://peer2.savitar.tv/USANetwork/myStream/playlist.m3u8?wmsAuthSign=c2VydmVyX3RpbWU9MTAvMTEvMjAxOSA5OjA2OjAxIFBNJmhhc2hfdmFsdWU9cTFaTnQrYmpDTFgxcDhtdkNMVjNydz09JnZhbGlkbWludXRlcz0zNjAmaWQ9MA==
2019/10/12 20:00:22 [xTeVe] Content Type:          text/html
2019/10/12 20:00:22 [xTeVe] Streaming Status:      HTTP Response Status [403] Forbidden
2019/10/12 20:00:22 [xTeVe] Error with this URL:   http://peer2.savitar.tv/USANetwork/myStream/playlist.m3u8?wmsAuthSign=c2VydmVyX3RpbWU9MTAvMTEvMjAxOSA5OjA2OjAxIFBNJmhhc2hfdmFsdWU9cTFaTnQrYmpDTFgxcDhtdkNMVjNydz09JnZhbGlkbWludXRlcz0zNjAmaWQ9MA==
2019/10/12 20:00:22 [xTeVe] [ERROR] Forbidden (This error message comes from the provider) - EC: 4004

The script needs to be rerun every 6 hours to refresh the wmsAuth token, did you try rerunning it?
If you have rerun the script, what version of the script are you using?

Yes I have re-run the script. I'm not sure how to check which version I used, but I downloaded it within the last few days. I believe I spoke to you on reddit. I used this guide: https://sites.google.com/view/piptv-plex/home

You can try updating with
pip install -U piptv_pmg.pmg

Try rerunning the script and playing the m3u in a player like VLC. I am running all the streams without issue right now, so if it works in a different player for you, it's unfortunately probably a plex/xteve config issue, which I can't really help with :/

Thanks. I actually deleted the whole thing and just reinstalled and also made some changes in xTeVe. I'm pretty sure reinstalling this is what fixed it.

Unfortunately I'm back in the same boat with the 403 errors. Deleting the .m3u produced did not help this time.

If you're not running the latest version of the script, update it.
pip install -U piptv_pmg.pmg

If you are, try running the m3u generated by the script in VLC or any other media player and see if it works. I will leave this open to give you a chance to respond, but this isn't an issue with the script, it still works on every machine in my house and the site it scrapes hasn't changed anything in it's configuration that would break the script, which means it's likely an error in your configuration.
See if you can reach the site itself within a browser also, they may be blocking your ISP.

I have updated. The .m3u does not work in VLC or PotPlayer, I get a forbidden error in them. I am able to reach the site via browser.
This issue is sporadic at this point now, it works for an hour or so and then reverts back to this issue.

Well, the fact that it is sporadic is evidence that it's either your config or your ISP, unfortunately. With that being said I can't really help you. If the script is the latest version and it runs without error then the problem is somewhere else down the line :/
If the script starts throwing an exception, open another issue, but I am going to close this one. I hope you figure out what's going on!


Unset AWS_PROFILE env var in prod release script

We don't need to set the AWS_PROFILE env var since we're pushing to the same account.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/lgtm


📝 Clean up the documentation

Resolves bunch of unrelated documentation issues
Wrong capitalisation of headers and brands
Wrong indentation of code examples
Various typos, missing Oxford commas, bad style, or typography
Examples didn't follow Style Guidelines in many places
Tries to slim down on unneeded newlines
Bunch of files have been wrongly named or wrongly placed
Attempt to update some outdated details
Overall there are other things the would deserve some work but that felt too ambitious for this already huge changeset

[x] My pull request is made against the dev branch.
[x] I have ensured that the changed library can be built on your target system. I did not introduce any platform-specific code.
[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] I tested my commits, by adding a test case to the unit tests if needed
[x] I have ensured that I did not break any existing API calls.
[x] My code follows the coding style guide (if you are not sure, match the code style of existing files including indent style etc).
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight. Where I have generated this pull request using a tool, I have justified why this is needed.

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

Dialog here is more correct? It's the name for a pop-up window that the user can submit input to. This is how Qt, Windows, and pretty much every UI refers to them
Discord in fact only refers to them as "Modal"

we should strictly use the term modal and nothing else, as this is what discord api calls them. not dialog, or popup or window or any other term as this will confuse users.

this has so many conflicts now as to be unmergable and parts of it have made their way into the docs anyway, should this pr be closed? It could always be reopened in the future.

this has so many conflicts now as to be unmergable and parts of it have made their way into the docs anyway, should this pr be closed? It could always be reopened in the future.

I would like to try get some files changed to add parts from this PR, then I'll close it when I'm done :)

I am now closing this PR as #990 now adds anything left that hasn't been added from other PRs.


Add sticky header functionality

Problem statement
One hotly requested feature for our tables is to have the header row to be "sticky," i.e., scrolls with the user down the page for tables with a lot of rows.  We should implement this functionality for our consolidated Table component because ReportStream will always be dealing with a lot of data. Having the ability for the header row to stay "sticky" will enable a better user experience.
What you need to know

CodeSandbox example of position: sticky

Acceptance criteria

[ ] The Table component can take a prop to enable/disable sticky headers

Awaiting responses from USWDS

Associated PR: https://github.com/CDCgov/prime-reportstream/pull/9418


fix(driver vsphere): fix lint error in vsphere driver

Passage of the parameter "ctx context.Context" in first position
Fixes #3831
Signed-off-by: Martial Langry<EMAIL_ADDRESS>

Oh, excellent!  Thanks a bunch for this!!

LGTM


Distinct doesn't work on MV columns with a "where" clause

Step to reproduce :

create an offline table with some columns and at least a multi-valued one
load some data
try this request : select distinct myMultiValuedColumn from MyTable where otherColumn in ('MY_VALUE') limit 1000
an error occurs :

 "message": "QueryExecutionError:\njava.lang.UnsupportedOperationException
org.apache.pinot.segment.spi.index.reader.ForwardIndexReader.readDictIds(ForwardIndexReader.java:84)
org.apache.pinot.core.common.DataFetcher$ColumnValueReader.readDictIds(DataFetcher.java:418)
org.apache.pinot.core.common.DataFetcher.fetchDictIds(DataFetcher.java:89)
org.apache.pinot.core.common.DataBlockCache.getDictIdsForSVColumn(DataBlockCache.java:109)",
"errorCode": 200

The same request without the "distinct" or the "where clause" works fine.
Reproduced with Pinot 0.10.0

Pinot does not support distinct on MV column yet (tracked in #8850). The reason why it works without a filter is because Pinot will use dictionary to solve the query if there is no filter and only one single column within the distinct.

Fixed with #8850