Fix cloud provider / server side xml sync problems

There are two problems which combined to cause the "uuid" as title problem:

The cloud provider xml is only sent to the server when the "validate" global jquery event is called - the save button doesn't trigger this when opening the "publish" dialog, so the form fields need to be setup in the "presubmit" handler
The cloud provider xml is being used when it shouldn't be - it should only be used inside the
SectionEvent.PRIORITY_AFTER_EVENTS queued event, as this event get cancelled by the publish buttons.

There are two problems which combined to cause the "uuid" as title problem:

The cloud provider xml is only sent to the server when the "validate" global jquery event is called - the save button doesn't trigger this when opening the "publish" dialog, so the form fields need to be setup in the "presubmit" handler
The cloud provider xml is being used when it shouldn't be - it should only be used inside the
SectionEvent.PRIORITY_AFTER_EVENTS queued event, as this event get cancelled by the publish buttons.

I think this PR works well.  I can't reproduce errors we already know.
My understanding is "validating cloud control in "presubmit" event, which gets fired when clicking Save or changing pages. This event firstly set xmlDoc item's xml, and then validating other fields and adding their latest values to item's xml.


Data Retrieval from external monitoring systems

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

Environment

Python version:  3.5.3
NetBox version:  2.2.9

Description
Netbox provides an excellent overview of Layer 1-3 of a network, however the ability to see real-time status information alongside the plans would be very useful. Providing a full monitoring system is out-of-scope for Netbox, however it seems simple to load an external status (simply OK or fault) from another system. Ideally, this could be implemented similar to the reports section, as this would enable support for new monitoring systems to be easily implemented as necessary.

Started working on this here. Very rough at the moment, but putting something like the below into the monitoring folder makes devices look like this:

from dcim.monitoring import Monitoring

class TestMonitoring(Monitoring):
    def get(self, object):
        self.log_default("Unknown")
        self.log_ok("OK")
        self.log_info("Hello, World")
        self.log_warning("Might break soon")
        self.log_failure("Just exploded!")

I have to reject this feature request as out of scope for NetBox. #969 will allow for predictive linking to external NMS and other systems, but that's as far as we'll go. Implementing the ability for NetBox to directly pull information from other systems would open a Pandora's box of development, performance, and security issues.


W0182 Loan term warning

My values are well under 1200 but I keep getting the warning.

Make sure you have the latest version, this was fixed with https://github.com/cfpb/regtech-data-validator/issues/146


SDA-5813: Updated SDK version and ran go mod vendor

SDA-5813
This is an attempt to update the SDK in rosa as well as fix the pipeline failing: View here

@yasun1  we have removed the display name from ocm SDK and prow build is not failing. can you please check if you are using it in your code somewhere ?

@pvasant  I will check the issue and try to fix the image issue in Prow CI.

Prow CI is crashed. Need to wait for the service is back.

The rosa-aws-cli image build issue should be fixed by https://github.com/openshift/release/pull/30879.

/retest

/retest

@oriAdler @zgalor  Passing now, PTAL :pray:

/lgtm


fix(#1607): remove max height from container with thick heading

1. What this code change do:
This change allows a container with a thick heading to accommodate multi-line titles. It includes the following changes:

Removes max-height from the thick heading
Only renders title and actions slots if the slots are available. This avoids extra padding.
Uses flexbox with a column gap for the actions slot to match the Figma component.

Uses --goa-typography-heading-s for the heading title (see comment).
Replaces rem values with design tokens.

2. Make sure that you've checked the boxes below before you submit MR:

[x] I have read and followed the contribution guidelines

[x] I have run a build locally.

[x] I have ran unit tests locally.

[x] I have tested the functionality.

3. Which issue and JIRA item does this PR fix (optional)
4. Additional notes for the reviewer
Reviewer Checklist

[ ] Coding Standards followed.

When review is satisfactory reviewer will merge the changes.

I tested on Edge...

...and on Firefox...

...and on Chrome...

...and on Safari...

Issues is verified on Chrome, Firefox, and edge for react and angular, for mobile and desktop site

:tada: This PR is included in version 1.17.0-alpha.38 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.17.0-alpha.22 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.20.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.20.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix #156 - openExternalInsert() causing android app crash

Fix #156

@joachim-quis can we get a review please?

Thanks for the fix; approved.


Getting double flattened notes in scale.notes

Hello @danigb
First up this is a brilliant library and it s one of the main dependencies of a module called Scribbletune that I ve put together in order to provide a bunch of music generation functionality.
I wanted to know if there is a way to avoid getting double flattened notes? For instance, scale.notes('Db4 minor') will return  [ 'Db4', 'Eb4', 'Fb4', 'Gb4', 'Ab4', 'Bbb4', 'Cb5' ].
Here is there some way to not get A4 instead of Bbb4?
Similarly if I do scale.notes('d#4 major') I get [ 'D#4', 'E#4', 'F##4', 'G#4', 'A#4', 'B#4', 'C##5' ]
Here is there some way to not return double sharps? So that instead of F##4, I d get G4 instead.
Many thanks in advance!

Yes, there's a reason: to preserve intervals. It used to be aenharmonic module that allow to "simplify" a note (get the same pitch with simpler name), but I think I've lost it in the last refactor 😱
You can convert to midi and back to note names, meanwhile...

Thanks for such a quick response @danigb In the interim, I ll use something on lines of:
Tonal.Scale.notes(tonicOctScale).map(el => Tonal.Note.fromMidi(Tonal.Note.midi(el)))

I found the function. I don't know why is not in the API documentation (I will keep this open until I found the cause).
The solution is:
[ 'D#4', 'E#4', 'F##4', 'G#4', 'A#4', 'B#4', 'C##5' ].map(Note.simplify) // => [ 'D#4', 'F4', 'G4', 'G#4', 'A#4', 'C5', 'D5' ]

Thanks @danigb I ll switch to this in my code for now.

The documentation is here: http://danigb.github.io/tonal/api/module-Note.html#.simplify
It doesn't appear on the left bar... 🤔 http://danigb.github.io/tonal/api/
Closing the issue for now...


Add low-resolution wrappers for timing functions

We should not provide high-resolution timers to plugins, but some plugins do require a sense of time. For those, we should provide a low-resolution access to time, and probably make it a permission, too.
Added timers for beta purposes (ethers uses timers to monitor transaction completion) in https://github.com/MetaMask/metamask-snaps-beta/pull/134

i assume this is related to timing attacks, including spectre style vulns


fix embedded refs encoding

Fixes issue #522

----- Testing code formatting -----
black -l 110 -t py37 --check --exclude "reclass|helm_binding.py|version.py" .
would reformat /home/travis/build/deepmind/kapitan/tests/test_refs.py
Oh no! 💥 💔 💥
1 file would be reformatted, 58 files would be left unchanged.


[proposal] Omit empty TiledImage from TiledMap.tiledImages

What could be improved

Note: I am not sure if this is a bug or a proposal.

Change the TiledMap.tiledImages to return only tilesets that have an actual source.
https://github.com/flame-engine/tiled.dart/blob/1d3043f75dc59449e98c9f2f637141b8ac127508/packages/tiled/lib/src/tiled_map.dart#L205-L215
As-is
if (image != null) {
  imageSet.add(image);
}

imageSet.addAll(layers.whereType<ImageLayer>().map((e) => e.image));

To-be
if (image?.source != null) {
  imageSet.add(image!);
}

imageSet.addAll(layers.whereType<ImageLayer>().map((e) => e.image)).where((e) => e.source != null);

Why should this be improved
At present, TiledMap.tiledImages collects all images in the map including those without the source.
I think the TiledImages without source is not actually the "image".
For example, the following is a bare image layer without the source:
 <imagelayer id="4" name="Image Layer 2"/>

I think it is a layer but doesn't include any image.
The following is another example:
 <tileset firstgid="1810" name="tileset 1" tilewidth="1" tileheight="1" tilecount="0" columns="0"/>

Also, this is a tileset but not includes any "image".
Any risks?

This cannot be true but there might be some users that use the empty Tiledimage.
More information

It would be great :)


When a bad query is made (Example: Tom Cryz), appropriate error is not displayed

When a bad query is made like below, error should be something like "This seems to be an invalid topic."

If we're able to print the error message displayed on the wikipedia page, even that would be great!
the errro page according to the query is: https://en.wikipedia.org/wiki/Tom_cryz
in this page, the error text is not parsed properly.

Hi
i fixed this problem and created a PR.

Simple, yet effective!
Thanks @amirbigg


It is recommended to leave Microsoft.AspNetCore.Mvc.ApiExplorer

It is recommended to leave Microsoft.AspNetCore.Mvc.ApiExplorer, which can flexibly expand the generated RPC interface document. Swashbuckle defines the required ApiDescription.

To be honest, the text above doesn’t make much sense to me. Could you please elaborate with an example?

@domaindrivendev
Orleans.SwaggerGen
This is my Orleans based on Swashbuckle.AspNetCore
  The implementation of the Swagger document generation. You can understand it if you can look it up.
Because Orleans cannot get all the interfaces from IApiDescriptionGroupCollectionProvider. So I redefine ApiDescriptionCollectionProvider and ApiDescription
ApiDescriptionCollectionProvider
ApiDescription

Orleans based on Swashbuckle survival Swagger document component Swashbuckle.Orleans.SwaggerGen


How to resize the plot when using "%matplotlib widget"

When I used the "%matplotlib notebook," there is a small triangle in the right bottom corner of the plot that your can drag to resize. But it seems that this feature disappeared in the "%matplotlib widget" mode. I wonder how can I resize the plot by dragging.

Looks like issue ( https://github.com/matplotlib/jupyter-matplotlib/issues/44 ) is asking for the same feature. Should one of these be closed?

+1 for this feature request.

Closing as duplicate https://github.com/matplotlib/jupyter-matplotlib/issues/44, thanks for reporting! It would be indeed a nice feature


Escape quotation marks inside strings

I noticed on the https://html-to-elm.com/ website, " inside strings are not escaped in the generated Elm code. For example "Test" will generate Html.text ""Test""

Thanks for reporting the issue! This definitely looks like a bug.
I don't have bandwidth at the moment, if somebody wants to tackle this this would be a pretty good first contribution to this project.

Had the same problem today and I think I managed to fix it in #29

Thank you for the fix @mateusfpleite! Merged.


Message Tags Support

Add support for:
https://developers.facebook.com/docs/messenger-platform/send-messages/message-tags

Today released as part of v1.0.0-M3


Source Control view spawns new Changes entry each time changes are saved

Describe what happened:
A new Changes entry is added under the Source Control view for the guest every time file changes are saved.
What was your system configuration?
Product and Version: VSCode 1.51.1
OS Version: macOS 10.15.7 (also happens on Windows)
Live Share Extension Version: v1.0.3206
Target Platform or Language: Browser/Javascript
Steps to Reproduce / Scenario:

Join live share session as guest.
Either guest or host edits one or more files and saves changes.

Please attach logs to this issue:
You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue
live-share-logs.zip
Screenshots
If applicable, add screenshots to help explain your problem.

Looks like this is a duplicate of #3922


packit: Use alias for Fedora targets

Use the 'fedora-stable' alias instead of specifying targets by Fedora
version. This solves Packit errors due to Fedora 30 reaching EOL.
Signed-off-by: Hunor Csomortáni<EMAIL_ADDRESS>

whoops, I missed this PR and already changed packit.yaml to do fedora-all in another PR
thanks for the contribution though!


Added DEP-8 compliant test for autopkgtest...

debian/tests/: Added DEP-8 compliant test for autopkgtest...
debian/control: Corrected a missing libssl-dev dependency...
README.md: Updated Debian RFP and Ubuntu PPA notes...
{version.txt,debian/changelog}: Bumped version...

Codecov Report

Merging #605 into master will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #605      +/-   ##
==========================================
- Coverage   70.77%   70.61%   -0.16%
==========================================
  Files          49       49
  Lines        6970     6970
==========================================
- Hits         4933     4922      -11
- Misses       2037     2048      +11

Impacted Files
Coverage Δ

src/client/client.cc
78.81% <0%> (-2.1%)
: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 a84bb6d...fa89613. Read the comment docs.


about model checkpoint

Hi,
Thanks for your excellent job.
I have a question about the given model checkpoint(Model checkpoint: https://drive.google.com/file/d/1WQJcgWI5NRNQz8aNrWFx72SqSaM2XskH/view?usp=sharing).
When I loaded the checkpoint, there was an error that some keys missed and some were unexpected.
Do you know how to solve it?

Hi, thanks for the question.
The shared checkpoint was obtained using an old commit (https://github.com/DI4IR/SIGIR2021/tree/4e5d8414b074b411b30fcf389466bdce57112b56), then I have changed a bit the structure of the model (no differences in final results though).
Please try with this checkpoint and let me know: https://drive.google.com/file/d/17I2TWCB2hBSQ-E0Yt2sBEDH2z_rV0BN0/view?usp=sharing

Hi, there is only one unexpected key(bert.embeddings.position_ids) with the new checkpoint( https://drive.google.com/file/d/17I2TWCB2hBSQ-E0Yt2sBEDH2z_rV0BN0/view?usp=sharing)

Oh, I found a solution. Just update the transformers to 3.4.0 !
Thanks for your reply！


Move AWS endpoint configuration to services

aws-sdk-go-v2 has deprecated setting endpoint configuration at SDK-level, it now much be configured for each service.
#patch

Should unblock #211


Migrations: SnapshotModelProcessor not used when generating SQL

This could lead to errors or incorrect SQL on older migrations.

Mitigated by https://github.com/aspnet/EntityFrameworkCore/issues/18557
Might be superseded by https://github.com/aspnet/EntityFrameworkCore/issues/18620


(*Runner).doMetricsRequestIfEnabled code between Lock() and Unlock() should be a separate method

Extracted from #27.

The relevant locking was removed as part of #506.


anta get from-cvp does not build inventory with Management IP addresses

Issue is not related to ANTA getting information but more about CVP/EOS setup:

Cloudvision provision section is using 1 IP address as Management Address
EOS configuration is using another IP Address on Management interface

API is exposing IP Address provisioned for management in CVP

Output of anta get from-cvp is created with correct IP address from CVP standpoint
anta_inventory:
  hosts:
  - host: <IP_ADDRESS>
    name: s1-core2
    tags:
    - s1-core
  - host: <IP_ADDRESS>
    name: s1-core1
    tags:
    - s1-core


Item Sprite menu deletes wrong entry

Looks like when you use a filter to look for specific item sprites and then try to delete one, it deletes the wrong item sprite instead. My guess is that it used whatever should normally be in that slot order wise if you hadn't used a filter, but I'm not really sure about that. It's definitely doing something wrong though.

Don't think it has to do with the filter, but it seems sometimes you have to go one above or something to get it to delete what you want (I want to delete chesto berry here as I used the wrong naming convention)

Fix in https://github.com/RogueCollab/RogueEssence/commit/3b9fbc88b8e88470a4743885cb1c5d786b937861


When installing dev package, target a specific index

@praveenkuttappan @tjprescott I am pretty certain this will be fine for our runs.
Does this same concern affect how we currently install the api-stub-generator during APIView deploy?

@scbedd I think targeting a specific index makes sense, but shouldn't it be here, not the SDK? Didn't you move the pylint-guidelines-checker to this repo?

I see. We want to make this a dev install. Done.

@tjprescott No I think this is right.
This api-stub-generator is what gets used to create APIReviews right? And it uses the pylint-guidelines-checker to generate part of it's failures.
In the pipeline checking this api stub generator, do we want to use the version being developed or the same one as what APIView uses?

@scbedd we should what APIView uses so this looks like the right approach. Thank you!

This PR will fail until this other PR is invoked for the dependent packages.

/azp run tools - api-stub-generator - ci

/azp run tools - api-stub-generator - ci

/azp run tools - api-stub-generator - ci


Imagine adding instructions about how to install it

Simply Imagine

ReadMe.txt has installation instructions, although they are an one liner because there is nothing complicated about installing that patch.
On top of that, every release page (Far Cry included) from myself includes a link to Setup Instructions. This should be more than enough to figure out how to install.


fix: Fix osFileProcClause

Fixes osFileProcClause being unused in inFile.

Released on https://github.com/yajra/laravel-sql-loader/releases/tag/v1.7.1, thanks!


nodeos i can see producer_plugin only

when start the nodeos i can see only producer_plugin, is it correct?
command i input is
./nodeos -e -p eosio --plugin eosio::chain_api_plugin --plugin eosio::history_api_plugin
Please tell me, thanks

Yes, that looks fine.


feat: refresh timeout on replace

Hi,
Frist, thanks for the amazing job!
Then, onto business: this is a proposal to refresh the timeout when replacing an existing notification.
The use case I faced was that I wanted to create a mapping to cycle through some colorschemes and apply them. When applying a colorscheme I want to add a notification with its name.
The issue is better explained with a timeline, the current behavior is:

elapsed time (s)
action

0
id = vim.notify('test 1', vim.log.levels.INFO, { timeout = 4000})

1
Notification "test 1" visible

2
id = vim.notify('test 2', nil, { replace = id })

3
Notification "test 2" visible

4
Notification "test 2" closing

While I expected:

elapsed time (s)
action

0
id = vim.notify('test 1', vim.log.levels.INFO, { timeout = 4000})

1
Notification "test 1" visible

2
id = vim.notify('test 2', nil, { replace = id })

3
Notification "test 2" visible

4
Notification "test 2" visible

5
Notification "test 2" visible

6
Notification "test 2" closing

The proposed solution works, at least in my case, I don't know how to write a reliable test though...
I'm not sure about the approach neither.

I've been meaning to clean up the timeout handling a bit anyway so used this as an excuse for the first step. You should now be getting the behaviour you wanted :smile:


Add verify flag

I implemented --verify flag. Can you please review and merge this?
With this flag, ruboclean acts like below.

Do not write result to file.
exit with 1, when .rubocop.yml has changes.
exit with 0, when .rubocop.yml has no changes.

For backward compatibility, it exit with 0 when --verify flag is not passed.
closes #38.

Hi @kazzix14 👋
Thanks a lot for your proposal and your PR. I like the idea, let me review it in the next days, I'll get back to you.

Hi @lxxxvi!
Thank you for your quick response!
I'm glad to hear you like the idea. If there's anything you're wondering about or need me to clarify, just give me a shout. Looking forward to your feedback!]


Unexpected Output Shape

Running the example code from the readme, model(img) returns a tensor with shape (1, 1, 1000) rather than the expected (1, 1000):

I've been having trouble getting the model to work with 1D input (audio) and before I spend too much time playing with it I wanted to see if this is the expected output shape or if there is something going wrong.

@johnowhitaker :bug: , should be fixed!

Running the example code from the readme, model(img) returns a tensor with shape (1, 1, 1000) rather than the expected (1, 1000):

I've been having trouble getting the model to work with 1D input (audio) and before I spend too much time playing with it I wanted to see if this is the expected output shape or if there is something going wrong.

Hey John, were you able to get the code up for audio data?


fuels dev does not watch after finishing execution

Run the following: echo | pnpm create<EMAIL_ADDRESS>--pnpm && cd my-fuel-project && pnpm fuels dev and you'll see that it'll exit after writing the messages:
Contract IDs saved at: ~/my-fuel-project/src/sway-api/contract-ids.json
🎉  Dev completed successfully!

It shouldn't exit, the process should continue on living and waiting for changes as described in the docs:

Watches your Forc workspace and repeats the previous step on every change

Are you sure the process exited?
Does it work without the echo |?
Why are you echo | a long-lived process anyway?

Full output for your:
$ echo | pnpm create<EMAIL_ADDRESS>--pnpm && cd my-fuel-project && pnpm fuels dev

✔ What is the name of your project? … my-fuel-project
✔ fuelup is already installed.
✔ Copied template files!
✔ Installed dependencies!

⚡ Success! Created a fullstack Fuel dapp at my-fuel-project

To get started:

- cd into the project directory: cd my-fuel-project
- Start a local Fuel dev server: pnpm fuels:dev
- Run the frontend: pnpm dev

-> TS SDK docs: https://docs.fuel.network/docs/fuels-ts/
-> Sway docs: https://docs.fuel.network/docs/sway/
-> If you have any questions, check the Fuel forum: https://forum.fuel.network/

(node:30643) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
The following binaries on the filesystem are outdated:
 -> 'forc' is currently 'v0.61.1', but the current 'fuels' version requires 'v0.63.1'.
 -> 'fuel-core' is currently 'v0.30.0', but the current 'fuels' version requires 'v0.33.0'.
Starting node using: 'fuel-core'
Error: an error occurred while loading the chain state file: Some("/Users/anderson/Desktop/my-fuel-project/.fuels/snapshotDir/chainConfig.json")
Error: an error occurred while loading the chain state file: Some("/Users/anderson/Desktop/my-fuel-project/.fuels/snapshotDir/chainConfig.json")
Error: an error occurred while loading the chain state file: Some("/Users/anderson/Desktop/my-fuel-project/.fuels/snapshotDir/chainConfig.json")

Caused by:
    unknown variant `V4`, expected one of `V1`, `V2`, `V3` at line 1 column 627

fuel-core node under pid 30683 does not exist. The node might have been killed before cleanup was called. Exiting cleanly.

echo is for the  pnpm create<EMAIL_ADDRESS>--pnpm to skip the "entering name" part; pnpm fuels dev is independent from it.
You got an error because your fuelup toolchain isn't up to date so can't tell much from the reproduction.


Vue.js visual intro guide

Hi, I made a visual version of the doc. (PDF) But github does not recognize it as a pdf!
It contains many ovale bubbles showing lacks in actual doc, that should be addressed.
Not shure how we could work together for improving it. Of course github. But I do not know how much I can use it and it is not exactly a textbased doc.
The original is in Open Office Draw.
Is there a way to submit it? It could be a good way to accelerate Vue.js adoption.
It helps to follow along reading the official guide.
A reply would be appreciated before I move on another way of communication, etc.
Marc
<EMAIL_ADDRESS>

Are you generating the pdf from the docs source?

I try to stay as close to original as possible, that is why I consider it a schematic version of the original doc, today I got the idea to explain that even more, in the intro. I do it manually. It think it can be understood faster, especially if read along the original doc, but adding a visual grouping, and splitting code and comments, and highlighting what to focus in the code, with comments.
Ultimately, it would help - IMPOV - if it inducts a different format for any doc. I already have a version for Vue-router, and working on one about Vuex. I do it first for my own understanding, but if I can share it for the good of faster acceptance of Vue.js...
If you have any concern, or comments, let me know.
Marc
  De : Evan You<EMAIL_ADDRESS>
À : vuejs/vuejs.org<EMAIL_ADDRESS>Cc : Marc Tetreault<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Envoyé le : Dimanche 14 août 2016 17h06
Objet : Re: [vuejs/vuejs.org] Vue.js visual intro guide (#366)
Are you generating the pdf from the docs source?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@MarcTe Can you post the PDF somewhere and link here so we can take a look?

https://drive.google.com/file/d/0B_kQr3l7tmlWS2hMU21JaENiUFU/view?usp=sharing
Still not perfect and, beginning to experiment, I see lacks in the doc (things we should explain, especially things that are in the API side of the doc).Marc
  De : Chris Fritz<EMAIL_ADDRESS>
À : vuejs/vuejs.org<EMAIL_ADDRESS>Cc : Marc Tetreault<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Envoyé le : samedi 27 août 2016 15h26
Objet : Re: [vuejs/vuejs.org] Vue.js visual intro guide (#366)
@MarcTe Can you post the PDF somewhere and link here so we can take a look? —
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Interesting! Thanks for sharing. 😄  I'll go through the PDF and will be making updates in the next week using your oval comments. If there are larger changes to make, I'll open separate issues for them. There are sometimes things intentionally left out of the guide though, then only covered in the API page - for a variety of reasons, but especially if they're more niche features.
Again, thanks for sharing! When you're done with these, I think posting links on the forum and Gitter chat would be really helpful for others. In the meantime, I'll close this for now.

Not shure the best way to email to you (I suppose this will re-open the issue)
One idea would be to define several guidelines in order to help people grasp things ASAP. Some of them:
-Group topics in one page with a title (or a title in code side bar comments)-Group ideas in rectangles
-Clean code, getting comments out, in a side bar comment-Rectangled words showing the focus elements in code text,
-Arrows (I have several guidelines about which direction the arrow should be) One of them:
from existing assets (such as a function) to the one using it.-Useful comments in small boxes with arrows pointing to the related code.

In fact, my 1st vue.js project is an app that can be used for writing such document (viewable in both web and PDF), easily updatable)
I suppose it would be better to clean this doc before posting it.I could put the Open Office Draw file somewhere, so you can edit it directly.But only one can do it at once. (I still do not know well how to manage a github, and it cannot be edited just with text editor)

Marc
  De : Chris Fritz<EMAIL_ADDRESS>
À : vuejs/vuejs.org<EMAIL_ADDRESS>Cc : Marc Tetreault<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Envoyé le : samedi 27 août 2016 21h50
Objet : Re: [vuejs/vuejs.org] Vue.js visual intro guide (#366)
Interesting! Thanks for sharing. 😄 I'll go through the PDF and will be making updates in the next week using your oval comments. If there are larger changes to make, I'll open separate issues for them. There are sometimes things intentionally left out of the guide though, then only covered in the API page - for a variety of reasons, but especially if they're more niche features.Again, thanks for sharing! When you're done with these, I think posting links on the forum and Gitter chat would be really helpful for others. In the meantime, I'll close this for now.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix space

Change Description
Fix space in shell command example documentation
Issue reference
None
Checklist

[x] I have reviewed the contribution guidelines
[ ] I have signed the CLA (if required)
[ ] My code includes unit tests
[ ] All unit tests and lint checks pass locally
[x] My PR contains documentation updates / additions if required

/azp run


[Documentation] HA Configuration and Examples

We need to clearly define in our documentation how to set up HA and note what is currently supported.

We need to provide a minimum of one good example that would suggest how to set this up for production.
We need to provide a chart that provides a map explaining the different elements so it's easy to visualize.

This issue will be partially complete based on incoming changes to the docs. https://github.com/rancher/docs/pull/1886
Later we'd like to expand upon this and provide at least an image showing basically a map of a proper HA configuration and discuss in more detail what a production set up may look like and involve.

actually the doc update missed: https://github.com/rancher/k3s/issues/710
In the docs it says:

You should create a TCP Load Balancer for the master nodes. This will allow worker nodes to address any master node should one of them fail. The Load Balancer should point to each of the master nodes on port 6443 (Default port for the server).

however with k3s this is not strictly necessary.

Will fix this. Thanks!


SignalRRequestBody attribute disappeared.

SignalRRequestBody attribute disappeared.
Now Swagger UI  has no request schemas.

@KlestovAlexej , yes, in the new versions of SignalRSwaggerGen in order to specify that a parameter is mapped to the request body, you need to decorate the paratemer with [FromBody], [FromForm].

Thank you! Works.

@KlestovAlexej , please toss a start for this repo in order to support my work. Thanks!


error in get_unread, after this day code modification

for contact in  driver.get_unread():
File "webwhatsapi/init.py", line 328, in get_unread
raw_message_groups = self.wapi_functions.getUnreadMessages(include_me, include_notifications, use_unread_count)
File "webwhatsapi/wapi_js_wrapper.py", line 43, in getattr
wapi_functions = dir(self)
File "webwhatsapi/wapi_js_wrapper.py", line 61, in dir
self.driver.execute_script(script.read())
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/webdriver.py", line 546, in execute_script
'args': converted_args})['value']
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/webdriver.py", line 309, in execute
response = self.command_executor.execute(driver_command, params)
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/remote_connection.py", line 460, in execute
return self._request(command_info[0], url, body=data)
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/remote_connection.py", line 483, in _request
self._conn.request(method, parsed_url.path, body, headers)
File "/usr/lib/python3.6/http/client.py", line 1239, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1250, in _send_request
self.putrequest(method, url, **skips)
File "/usr/lib/python3.6/http/client.py", line 1108, in putrequest
raise CannotSendRequest(self.__state)
http.client.CannotSendRequest: Request-sent

and on send to id too :
driver.send_message_to_id(id, message)
File "webwhatsapi/init.py", line 532, in send_message_to_id
return self.wapi_functions.sendMessageToID(recipient, message)
File "webwhatsapi/wapi_js_wrapper.py", line 43, in getattr
wapi_functions = dir(self)
File "webwhatsapi/wapi_js_wrapper.py", line 61, in dir
self.driver.execute_script(script.read())
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/webdriver.py", line 546, in execute_script
'args': converted_args})['value']
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/webdriver.py", line 309, in execute
response = self.command_executor.execute(driver_command, params)
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/remote_connection.py", line 460, in execute
return self._request(command_info[0], url, body=data)
File "/usr/local/lib/python3.6/dist-packages/selenium/webdriver/remote/remote_connection.py", line 483, in _request
self._conn.request(method, parsed_url.path, body, headers)
File "/usr/lib/python3.6/http/client.py", line 1239, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1250, in _send_request
self.putrequest(method, url, **skips)
File "/usr/lib/python3.6/http/client.py", line 1108, in putrequest
raise CannotSendRequest(self.__state)
http.client.CannotSendRequest: Request-sent

@mhndm i think that error is not in the library, if you look to the error is in the communication with selenium and your navigator. If you go back to the last version works? I found a lot of this issues in a fast Google search, maybe is the case to see if any think can solve your problem.

yes the latest version work

Have you tried to clean up the selenium configuration and/or update your Firefox version? I have tested now and no problem :/. I’m using python 3.5.2

I use :
Mozilla Firefox 60.0.1
python: 3.6
selenium: 3.8.0
how i can  clean up the selenium configuration ?

same here

Im running the navigator in remote mode, in a docker container without problem, maybe that can help you.

Same here.

issue fixed when I update to firefox_61.0.1

i use the latest code using git
selenium 3.13.0 latest
Mozilla Firefox 61.0.1 latest
geckodriver 0.21.0 latest


Fixes #1818. Adds Title change events to Window

Fixes #1818. Adds Title change events to Window.
Adds:

OnTitleChanging and TItleChanging (cancelable)
OnTitleChanged and TitleChanged
Unit tests (the first Window unit tests!)

Oops. I accidentally already merged this into main as part of my other Wizard PR.
I'll leave it here for now so you guys can provide feedback. If I need to change anything I'll do an additional PR.

Dis you see my review?

Dis you see my review?

Sorry, no. Where is it?


[BUG] bug with GET /your_index/_search and POST _reindex in OS 2.11

What is the bug?
When you configure an index with a search pipeline as below  the
GET /your_index/_search  yields a 500 error - Null Pointer Exception.
It seems like by default the _reindex API uses the  GET /your_index/_search  as such if you attempt to perform re-index with a source_index that has a index.search.default_pipeline configured you will get a Null Pointer Exception.
POST _reindex
{
   "source":{
      "index":"source-index"
   },
   "dest":{
      "index":"destination-index",
      "pipeline": "your-pipeline"
   }
}`
### How can one reproduce the bug?

1- Create Pipeline:
`PUT /_search/pipeline/test-pipeline
{
  "request_processors": [
    {
      "neural_query_enricher" : {
        "tag": "tag1",
        "description": "your description",
        "default_model_id": "your_model_id"
      }
    }
  ]
}

2- Create index with pipeline and setup default search pipeline
PUT /source-index
{
  "settings": {
    "index": {
      "knn": true,
      "knn.algo_param.ef_search": 100
    },
    "default_pipeline": "test-pipeline",
    "index.search.default_pipeline" : "test-pipeline"
  },
  "mappings": {
    "properties": {
      "text": {
        "type": "text"
      },
      "passage_embedding": {
        "type": "knn_vector",
        "dimension": 384,
        "method": {
          "name": "hnsw",
          "space_type": "l2",
          "engine": "nmslib",
          "parameters": {
            "ef_construction": 128,
            "m": 24
          }
        }
      }
    }
  }
}

3- add documents
PUT /source-index/_doc/1
{
    "text": "The emergence of resistance of bacteria to antibiotics is a common phenomenon. Emergence of resistance often reflects evolutionary processes that take place during antibiotic therapy."

}

PUT /source-index/_doc/2
{
  "text": "The successful outcome of antimicrobial therapy with antibacterial compounds depends on several factors. These include host defense mechanisms, the location of infection, and the pharmacokinetic and pharmacodynamic properties of the antibacterial."
}

4- search (This will yield a NPE-500 error)
GET /source-index/_search
This will give NPE due to "index.search.default_pipeline" : "test-pipeline". If you remove this setting in the index setting, the query works just fine.
5- Proceed to Create a destination index
PUT /destination-index
{
  "settings": {
    "index": {
      "knn": true,
      "knn.algo_param.ef_search": 100
    },
    "default_pipeline": "test-pipeline",
    "index.search.default_pipeline" : "test-pipeline"
  },
  "mappings": {
    "properties": {
      "text": {
        "type": "text"
      },
      "passage_embedding": {
        "type": "knn_vector",
        "dimension": 384,
        "method": {
          "name": "hnsw",
          "space_type": "cosinesimil",
          "engine": "nmslib",
          "parameters": {
            "ef_construction": 128,
            "m": 24
          }
        }
      }
    }
  }
}

6- Attempt to perform re-indexing (This will give you a NPE - 500 error)
POST _reindex
{
   "source":{
      "index":"source-index"
   },
   "dest":{
      "index":"destination-index",
      "pipeline": "test-pipeline"
   }
}

The above will not work because GET /source-index/_search gives a NPE error due to the "index.search.default_pipeline" : "test-pipeline" as discussed in Step 4.
However if you remove the "index.search.default_pipeline" : "test-pipeline" the index settings, the query works.
7- if you try re-index with below it works
POST _reindex
{
   "source":{
      "index":"source-index",
      "query": {
          "match_all": {}
      }
   },
   "dest":{
      "index":"destination-index",
      "pipeline": "test-pipeline"
   }
}

Bugs:
1- Seems like by default the _reindex API is using GET /source-index/_search  instead of GET /your_index/_search{"query":{"match_all":{}}} and since the former throws a Null Pointer Error, the _reindex also throws the same as it is unable to get the list of documents in source index.
2- GET /source-index/_search does not work if the "index.search.default_pipeline" : "test-pipeline" is present in index setting
OpenSearch Version
OS 2.11

@vibrantvarun can you take a look into this issue. Seems like some issue with NeuralQueryEnricher processor

@vamshin @vibrantvarun Any updates on this?
Thanks

I tested it out. There's no null check below
https://github.com/opensearch-project/neural-search/blob/7368edcfd36c4a7d703d092fc2f73fa6c82f1d92/src/main/java/org/opensearch/neuralsearch/processor/NeuralQueryEnricherProcessor.java#L71
causing the NPE. It was handled in https://github.com/opensearch-project/neural-search/pull/615. Should resolved the issue. @martin-gaievski @vibrantvarun can you verify and we can close this one out?

Good catch, that PR should fix the issue. I'll run the test and update here whether or not we're good to close the issue.

@martin-gaievski did you get a chance to verify this?

I think info about ingest pipeline is missing in provided steps. Document ingestion will not work without that pipeline because it's needed to create embeddings.
My understanding is that we need to add following details to provided steps:
create ingest pipeline step:
PUT {{base_url}}/_ingest/pipeline/ingest-pipeline
{
    "description": "An NLP ingest pipeline",
    "processors": [
        {
            "text_embedding": {
                "model_id": "<model_id>",
                "field_map": {
                    "text": "passage_embedding"
                }
            }
        }
    ]
}

then put it as default pipeline for both indexes, e.g. This is needed because default_pipeline is of type ingest, not search.
PUT {{base_url}}/source-index
{
    "settings": {
        "index": {
            "knn": true,
            "knn.algo_param.ef_search": 100
        },
        "default_pipeline": "ingest-pipeline",
        "index.search.default_pipeline": "test-pipeline"
    },
    "mappings": {
        "properties": {
            "text": {
                "type": "text"
            },
            "passage_embedding": {
                "type": "knn_vector",
                "dimension": 384,
                "method": {
                    "engine": "lucene",
                    "space_type": "l2",
                    "name": "hnsw"
                }
            }
        }
    }
}

similarly it should ingest pipeline in the reindex request:
POST {{base_url}}/_reindex
{
   "source":{
      "index":"source-index"
   },
   "dest":{
      "index":"destination-index",
      "pipeline": "ingest-pipeline"
   }
}

If I do all this, then reindex operation works for me. Tested on latest 2.x (2.18)
@LandryK, please review my assessment and correct me if I'm wrong. Based on @owaiskazi19's finding, I believe this issue stems from either an incomplete configuration or a fixed scenario.

Seems the issue is fixed on the latest version. Please reopen it if the issue still exist.


Git application

Resolved #126

User for commit should be changed (now it is ExampleOrgz)
Sync status should be more visible in dashboard. I propose colored background for task item (red - unsync, yellow - sync but not merged, green - merged)

diff file was committed into my repository. Is it expected?

git url and path isn't dumped into the xml file itself.

Need to provide good default value for git path (e.g. <dump file name>.xml)

@nmanovic
"Error handling if ssh key isn't added should be better. It will be a usual mistake. Need to ask the user to add ssh key into git server explicitly"
Interaction with an user during building is actually good idea?

@nmanovic
"diff file was committed into my repository. Is it expected?"
changelog_*.diff commits to repos. It is expected. Other diffs files stores outside repository directory.

@nmanovic
"Sync status should be more visible in dashboard. I propose colored background for task item (red - unsync, yellow - sync but not merged, green - merged)"
It is very long for loading because each "get_remote_status" request does request to git server (1-3 sec). I have no idea how implement it effectively yet.

@nmanovic
Other comments have been implemented

Interaction with an user during building is actually good idea?

It isn't during building. It is when a user try to push something into a repository for which the user doesn't add generated ssh key. It will get some error that the repository cannot be cloned with a lot of extra stuff. I remember that it isn't so easy to understand why a repository cannot be cloned. Let's say that "Probably SSH key was not added to your profile for the cloned repository. Please add the key below and try again".

@nmanovic
"diff file was committed into my repository. Is it expected?"
changelog_*.diff is commited to repos. It is expected. Other diffs files stores outside repository directory.

Why do we need it?

It is very long for loading because each "get_remote_status" request does request to git server (1-3 sec). I have no idea how implement it effectively yet.

Indeed the task is very important. It is really good feature. I need it to be sure that latest tasks are synchronized and it will be inconvenient to click each time on a task to understand its status.
There are several ways to implement that:

Just cache the latest status always and return it together with task info for dashboard. But it will necessary to click each time the task push as well. But one clicked it will be "save" state.
Ask users to add a webhook. I don't like the approach but probably it will work best with the approach above.
Implement a background task which sync the status time to time for non-merged repos and cache it. It will work best from my point of view. Also when a user press "push" button the status is cached as well.

@nmanovic
"Why do we need it inside repository? Is it a temporary file for internal purpose?"
No, this contains information about changed objects and dates.

@bsekachev ,
Is it possible to incorporate the information into git commit message? Why do we need a separate file for that?

@bsekachev ,
The main problem here that our users don't need the file.  You can keep it internally. All relevant information should be added into git log message. Thus it is easy to see history of the repository using git log command.

@bsekachev,
To be clear I don't propose to delete the file. I propose don't commit it and add its content into commit message.

@nmanovic
Are that all requirements?

@bsekachev , Please notify me when it is ready. I have the following error now.

@nmanovic
You can review it.

Just one more comment. It seems that after some time it became "red". But after "save" method with non-zero number of bounding boxes it should be "red" immediately.

@nmanovic
I have implemented approach with regular status updating how it specified above.
If you check a status manually after changing or merge an annotation, it will be updated.
Otherwise it will updated after 10 minutes.
Also we have an ability to update status to "!sync" on save event, but we can't do it after merge on a remote repository, because we aren't notified about it.
I'll fix issue with an extension.

@nmanovic
Please, review it again.


QG2101B  firmware   <IP_ADDRESS>_202109231032 use b091qp cannot connect to wifi and rtsp

Hi, thank you very much for your contribution.
My cam chip is QG2101B and the firmware version is <IP_ADDRESS>_202109231032.
I tried #249 (reply in thread),
But wifi can't find the device, plug in the network cable directly, you can see the device ip, but rtsp still has no image.
Below are some pictures of my device, I hope it can help you. This is my factory firmware backup.

This is my factory firmware backup.
LINK REMOVED

The cam is bq091qp.
But there are different versions of b091qp.
I have to check.

thank you

The cam is bq091qp. But there are different versions of b091qp. I have to check.

Thank you and I look forward to your good news.
If you need help with anything, just let me know.

Hi, I have the same camera.
@pengkaiwei did you make it and whether the last release supports this version. Thanks

awesome . Good work I confirm that everything is fine. works perfect. many thanks


Send update every second

In order to be able to display the UI immediately please send a message for each bus once every second even if it has not moved. Simply send the last know position and match.

The use case sounds a lot like it should be solved in the message transport layer, not in the business logic.
Can we use MQTT retained messages to fulfil it?

If a broker receives a message on a topic for which there are no current subscribers, the broker discards the message unless the publisher of the message designated the message as a retained message. A retained message is a normal MQTT message with the retained flag set to true. The broker stores the last retained message and the corresponding QoS for the selected topic. Each client that subscribes to a topic pattern that matches the topic of the retained message receives the retained message immediately after they subscribe. The broker stores only one retained message per topic. This allows new subscribers to a topic to receive the most current value rather than waiting for the next update from a publisher.

https://en.wikipedia.org/wiki/MQTT#Overview

Unfortunately, mosquitto (the MQTT broker we use) seems to keep retained messages around forever.

Retained Messages

All messages may be set to be retained. This means that the broker will keep the message even after sending it to all current subscribers. If a new subscription is made that matches the topic of the retained message, then the message will be sent to the client. This is useful as a "last known good" mechanism. If a topic is only updated infrequently, then without a retained message, a newly subscribed client may have to wait a long time to receive an update. With a retained message, the client will receive an instant update.
https://mosquitto.org/man/mqtt-7.html
It seems to only have a config to enable/disable retained messages, but not configure for how long it retains them.
https://mosquitto.org/man/mosquitto-conf-5.html

What about https://github.com/eclipse/mosquitto/blob/master/mosquitto.conf#L104-L109 ?


Function calling using tools not working

Expected Behavior
The SDK shouldnt throw any error while setting tools config in startChat method
Actual Behavior
Getting error
 Failed to fetch output from model gemini-1.5-pro-latest with message [GoogleGenerativeAI Error]: Error fetching from https://generativelanguage.googleapis.com/v1beta/models/gemini-1.5-pro-latest:generateContent: [400 Bad Request] Invalid JSON payload received. Unknown name "tools" at 'generation_config': Cannot find field. [{"@type":"type.googleapis.com/google.rpc.BadRequest","fieldViolations":[{"field":"generation_config","description":"Invalid JSON payload received. Unknown name \"tools\" at 'generation_config': Cannot find field."}]}]

Steps to Reproduce the Problem

get model using `googleAI.getGenerativeModel({ model: "gemini-1.5-pro-latest"})
start chat session using .startChat
ensure sending tools to the startChat method. e.g.

.startChat({
            generationConfig,
            history,
            tools: [
              {
                functionDeclarations: [
                  {
                    name: "extract_data",
                    description: "extract name age and location from message",
                    parameters: {
                      type: FunctionDeclarationSchemaType.OBJECT,
                      properties: {
                        location: {
                          type: FunctionDeclarationSchemaType.STRING,
                          description:
                            "If not available this should be empty string",
                        },
                        age: {
                          type: FunctionDeclarationSchemaType.NUMBER,
                          description: "age of the person, e.g. 34, 52",
                        },
                        name: {
                          type: FunctionDeclarationSchemaType.STRING,
                          description: "the name of the person, e.g. Alexa",
                        },
                      },
                      required: ["location", "name", "age"],
                    },
                  },
                ],
              },
            ],
          })

Got error
Failed to fetch output from model gemini-1.5-pro-latest with message [GoogleGenerativeAI Error]: Error fetching from https://generativelanguage.googleapis.com/v1beta/models/gemini-1.5-pro-latest:generateContent: [400 Bad Request] Invalid JSON payload received. Unknown name "tools" at 'generation_config': Cannot find field. [{"@type":"type.googleapis.com/google.rpc.BadRequest","fieldViolations":[{"field":"generation_config","description":"Invalid JSON payload received. Unknown name \"tools\" at 'generation_config': Cannot find field."}]}]

Specifications

Version:
Platform:

Got the bug in the request. was able to resolve this


How to upgrade Terraform provider plugins and modules

Hi,
there is any way to make the terragrunt update the plugins and modules like i can do with ?
terraform init -upgrade
terraform get -update

but I would like to do like a get-all instead remove /home/user/.terragrunt to get the latest aws provider to all my modules

You can certainly run terragrunt get -update or terragrunt init -upgrade. There is no init-all or get-all command currently though.

In addition to the workarounds mentioned above, we also now have run-all so you can run terraform run-all init -upgrade.
Closing as the question has been solved and stale. Please open a new issue if this is still relevant to you!

Hi all
I get this Error: Failed to load state: unsupported checkable object kind "var" when i run terragrunt-run-all apply, this is stack specific as it was fine with other stacks, am using (terraform version  1.3.7 & terragrunt version 0.53.1) please help


Unable to get psn access code

I have the same problem as issue #105 with a freshly generated npsso code and it is impossible to exchange it for an access code.
my application uses ionic and I'm on angular. For my tests I did it from my browser in which I am well connected to Playstation and I get the NSSO via the link specified in your documentation via the same browser. Here is the exact error that exchangeNpssoForCode returns:
GET https://ca.account.sony.com/api/authz/v3/oauth/authorize?access_type=offline&client_id=09515159-7237-4370-9b40-3806e67c0891&redirect_uri=com.scee.psxandroid.scecompcall%3A%2F%2Fredirect&response_type=code&scope=psn%3Amobile.v2.core+psn%3Aclientapp 403 (Forbidden)
Error: Uncaught (in promise): Error:
      There was a problem retrieving your PSN access code. Is your NPSSO code valid?
      To get a new NPSSO code, visit https://ca.account.sony.com/api/v1/ssocookie.

Thank you for all

Hi @landscht ,
Thanks for using psn-api. I am actively using the latest version of psn-api on a daily basis and am currently unable to reproduce this issue. To be sure, I exchanged an NPSSO for an access code seconds before making this post.
A 403 error from PSN leads me to believe the NPSSO being used is somehow invalid. There are a few things you can try:

The obvious: be extra sure you copied, pasted, and are using the correct NPSSO. Hardcode it as an arg to the function before retrieving it from environment variables.
Try an NPSSO from a completely brand new PSN account (this is a good idea honestly so your personal account doesn't get rate-limited)

Hi @wescopeland,
I created a new account, retrieved the NPSSO code, I hard-coded it to the exchangeNpssoForCode method and I have exactly the same problem. Is it necessary to have playstation connection cookies to call the exchangeNpssoForCode method?

Hi @landscht ,
It is not necessary to have the PlayStation connection cookies to call this or any other method.
Can you please do me a favor? Try using this NPSSO and see what happens:
PIo0tWGq8cQIVYA8soDjG6Kz9bf1YKpMzKE77Th5K6GTKNvXHvZwFmcGuhkjnxYI

Another thought -- I think this is not NPSSO related. Can you check your network tab while the Angular app is open on localhost and see the response coming back from Sony's servers. This could actually be a CORS error that psn-api is blanket-reporting as an NPSSO problem. Most of us are using psn-api server-side and not from a SPA.

I went through a node server to use your api and it just happens to work. It must be a cors error but it's weird that it's not specified in the http error or in the request logs. Thank you for all your help.

No problem & sorry for the confusion. Yes, it must be a CORS error. The library should do a better job of informing the user if a CORS error happens when trying to exchange the NPSSO; I will add this functionality.


Indexes property with name constraint

Similar concept to #6579 but more restricted.
I want to able to define type where it’s keys are open set but restricted.
For instance define type where all properties starts with prefix.
interface A{
 [prop: /^prefix/]: string;
}

I don’t really need regex syntax, but if possible it will be great.

Seems like this would be subsumed by https://github.com/Microsoft/TypeScript/issues/6579.


[Snyk] Upgrade jest from 29.4.1 to 29.4.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade jest from 29.4.1 to 29.4.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2023-02-07.

Release notes

Package name: jest

29.4.2 - 2023-02-07Features

[@ jest/core] Instrument significant lifecycle events with performance.mark() (#13859)

Fixes

[expect, @ jest/expect] Provide type of actual as a generic argument to Matchers to allow better-typed extensions (#13848)
[jest-circus] Added explicit mention of test failing because done() is not being called in error message (#13847)
[jest-runtime] Handle CJS re-exports of node core modules from ESM (#13856)
[jest-transform] Downgrade write-file-atomic to v4 (#13853)
[jest-worker] Ignore IPC messages not intended for Jest (#13543)

Chore & Maintenance

[*] make sure to exclude .eslintcache from published module (#13832)
[docs] Cleanup incorrect links in CHANGELOG.md (#13857)

New Contributors

@ benjaminjkraft made their first contribution in #13848
@ LinusU made their first contribution in #13543
@ SuperSodaSea made their first contribution in #13857
@ kowalski made their first contribution in #13856

Full Changelog: v29.4.1...v29.4.2

29.4.1 - 2023-01-26Features

[expect, jest-circus, @ jest/types] Implement numPassingAsserts of testResults to track the number of passing asserts in a test (#13795)
[jest-core] Add newlines to JSON output (#13817)
[@ jest/reporters] Automatic log folding in GitHub Actions Reporter (#13626)

Fixes

[@ jest/expect-utils] toMatchObject diffs should include Symbol properties (#13810)
[jest-runtime] Handle missing replaceProperty (#13823)
[@ jest/types] Add partial support for done callbacks in typings of each (#13756)

New Contributors

@ jessevanassen made their first contribution in #13817
@ ymqy made their first contribution in #13795
@ MatteoH2O1999 made their first contribution in #13626

Full Changelog: v29.4.0...v29.4.1

    from jest GitHub release notes

Commit messages

Package name: jest

f0fc92e v29.4.2
900fd48 chore: update changelog for release
5a9870b fix: correctly handle CJS re-exports in ESM modules (#13856)
8313d00 Downgrade write-file-atomic to v4 (#13853)
be501e1 docs: clean up formatting of blog posts (#13862)
1a2ffe4 docs: bring back missing images (#13863)
c8e7577 feat(jest-core): Add performance markers around significant lifecycle events (#13859)
423142c chore: Cleanup incorrect links in CHANGELOG.md (#13857)
cfa2d39 docs: fix link formatting in Expect API page (#13860)
f93ddb6 docs: remove extraneous tab symbol (#13861)
ffe2352 chore: move `@ jest/test-utils` reference (#13468)
48ddbf4 fix: ignore all unrelated messages from child process (#13543)
485d8fb chore: use `glob` to select files in `buildTs` script (#13850)
2b313fa feat: explicitly mention that test is failing because `done()` is not being called (#13847)
427fe2b feat(@ jest/expect): Expose type of actual to Matchers (#13848)
836157f Remove unused dependency 'progress' (#13843)
fc8562d chore: add some missing dev dependencies (#13837)
742302f chore: use `link:` protocol instead of `eslint-plugin-local` (#13836)
596c886 chore: bump docusaurus (#13835)
bc7b20c chore: make sure to exclude .eslintcache from published module (#13832)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #135 (48cb046) into master (9753169) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##            master      #135   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           91        91
=========================================
  Hits            91        91

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Device Freezes and resprings

That problem being happening since the first version, my device freezes up when im using WhatsApp, Spotify or playing PUBGM even on Facetime, suddenly it starts to freeze, then i cant exit from the app or close it, just being stuck until it resprings then everything back to normal.. Im on the last version 3.0.0 b4 with the last substrate iPhone 8 11.3.1
Edit #1: After updating to v3.0.0 b8 it keeps doing the same, now even worst, it freezes and reboot..

Latest version is v3.0.0 b8

Okay, i’ll update and keep in touch in case of anything

@BryanM014 post diagnostic.

How i can post it ?

Make sure your on Unc0ver 3.0.0 Beta 13 & If the problem persists reply here. @BryanM014

@Cryptiiiic please help I cannot as this is out of by abilities. @BryanM014 go to the bottom of the settings page & hit share or copy diagnostics & post it here. & make sure you are on the latest unc0ver version which at this time is 3.0.0 Beta 15.

Im gonna update now, i couldn’t reply because i was busy ..

Im gonna update now, i couldn’t reply because i was busy ..

Okay, let me know & tag me in it when you reply. If it doesn’t work post the diagnostics log & I mentioned how to get it above 👆🏻 @BryanM014

@BryanM014 no one has permission to that google drive link.

@BryanM014 please post the log on something like Dropbox or grant me access. Which I’ve already requested, or take the log & copy everything out of the ZIP & post it on something like Pastebin or post it here.

@king4q @Cryptiiiic verify now and my bad tho

@Cryptiiiic here’s a link to the code in case you can’t see it.
https://pastebin.com/wexct5mH

@BryanM014 remove everything from both xarold repos, it's piracy and known to have broken tweaks that freeze your device.

@Cryptiiiic Done, something else besides that? i'll let you know if anything happens

@BryanM014 closing this tomorrow if no response.

@Cryptiiiic so far after i disabled few of my tweaks like CrashReport and Flex 3, i havent had any of the problems i mentioned up there, but before that i was even having night reboots with the newer versions, im on beta 27 and nothing so far..

@BryanM014 any vsd crashes before?

@Cryptiiiic yep, you need the diagnostics again?

@BryanM014 sure, and can this be closed? vsd crashes are caused by youtube tweaks.

 Just let me know what to do ..
https://drive.google.com/file/d/1OjUuGnj5U1_nr28NCY1FtvuWPRISbBFB/view?usp=drivesdk

@BryanM014 closing this, install cappd if you have any more freezing problems.


Please sir, can I have some docs?

What would I need to change/update to leverage this in a new GHEC Enterprise against a new LDAP environment?

[x] Development requirements
[x] Running app locally
[ ] Debugging tips and tricks
[ ] Running app in production (this should cover the question above)

Icons for docs? 🤔


MultiBench result is way too much different from what's reported on the paper

Local run(best score)
accuracy | MOSEI | MOSI | sarcasm | humor
FactorCL-SUP | 71.05 | 51.75 | 55.07 | 49.53
SupCon | 74.61 | 51.17 | 52.17 | 50.85
Reported
accuracy | MOSEI | MOSI | sarcasm | humor
FactorCL-SUP | 77.34 | 70.69 | 59.42 | 54.44
SupCon | 72.71 | 47.23 | 46.38 | 52.36
Hi authors,
I ran the notebook file(multibench_example.ipynb) without any modification and found significant gap compared to what's reported on the paper.
The numbers I wrote in the local run result are actually the best scores I could get from 5 to 10 independent runs, and the deviations were quite large ranging from 3 to 5% (compared to std of 0.2~ 2.8% from the paper)
Any idea what could went wrong with it?
Other experiments including Synthetic, IRFL were found to be reproducible.
Thanks in advance.

Hi!
I would like to apologize for the issues you found, as the code was actually outdated and buggy. We updated the code and notebooks recently, so you can try the new versions for experiments.
Please let us know if you still find any issues and thanks for your feedback!


Make landing page responsive

Please assign me this issue under gssoc21

I would like to work on this issue.

Please assign me this.


fix: add MasterKey key generation parameter spec into CryptoModule

Why is this necessary?

Why is this necessary?

To unify MasterKey generation on Crypto and Storage modules and eliminate possibly faulty creation on some of the Samsung devices

Was there an issue reported that led to this? Or did you read something somewhere?


Better DX for contributors

Configs for Babel, Flow, ESLint were missing so it was a little bit complicated to contribute to this repository.

Thanks Michal! I landed this PR but only partially because I had to comply with our internal Eslint setup. I might tune it a little bit more later.

@michalsanger Eslint and Flow are now working correctly for OSS. I did it a bit differently to comply with our internal monorepo. Thanks for your contribution! :)


Update Emoji Dictionary

CI is failing because
ok_woman
guardsman
man_with_turban
egg
baseball

are duplicates

Thank you @MihaZupan ! 😄 Just removed those.

Thanks!

@xoofx Do you plan on updating the nuget any time soon?

Pretty busy on a business trip, ping me back next week

@xoofx shazam! ⚡


Add proper markup on modal's page tooltips and popovers

closes #36110

Is this ready for review and merging @GeoSot?

By me yes. As I checked, tooltips are still working, tooltips & popovers on
modal's page, too. As we do not have any dynamic created tooltips, I think
is fine to go
On Sat, May 7, 2022, 03:42 Mark Otto @.***> wrote:

Is this ready for review and merging @GeoSot https://github.com/GeoSot?
—
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/36276#issuecomment-1120092775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFK6HL534R2NR4FLBIHPA33VIW36TANCNFSM5VBFIK7A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Split s out from s in instruction lists

While wrapping the
s in instruction lists in

tags is a valid use of the tags, in verbose mode the

 tag doesn't seem to add much and the CSS
supports just having the
 on it's own so I'd like
to split it out.
HTML parsing peculiarity
I've also noted that having a wrapping paragraph can
cause problems with the HTML parser.
It seems that when parsing the content in the
 tag
it splits it into flow content (the
) and phrasing
content (the rest) and splits the phrasing content
into
 tags.
I noticed this in chrome devtools:

I think it's parsing it using this algorithm in the spec:
https://www.w3.org/TR/html5/dom.html#paragraph

:+1:


Add datetime to CSV

Summary:
Adds the timestamp to the resulting CSV to close #1006
Example:
# stan_version_major = 2
# stan_version_minor = 26
# stan_version_patch = 1
# model = bernoulli_model
# start_time = 2021-05-07 20:09:04
# method = sample (Default)

Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Rok Češnovar
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Last call for reviews.

Can you give me today?

No one is racing to review this so take your time :)

I see no tests. Do we say that comments like these are not required to be tested? What would happen downstream if the date-time is not present or messed up?

And would it make sense to add the end time as well?

And what about time-zone?

I see no tests

Yeah, I am not sure what can we test in Cmdstan without doing some sort of parsing in Python. There will be tests in cmdstanr if that helps. We also dont tests the num_threads comments etc.

What would happen downstream if the date-time is not present or messed up?

This does not touch or change anything in downstream (Math/Stan services) sonothing. And CSV readers in upstream (rstan, cmdstanr/py) do not care about the number of comments or position nor should they.

And would it make sense to add the end time as well?

Hm, maybe, though that can also be obtained from start time + running time. The problem is that the way CSV is printed that would add another comment after the CSV and we are trying to avoid that as the format
# comments
values
# more comments

or even
# comments
values
# adaptiation results
more value
# comments

is causing problems for almost all fast CSV readers in R and Python and they need to be handheld a bit.

And what about time-zone?

Hm, good call. Will add a "UTC+3", "UTC-2".

What about stansummary printing out the date time... and that we test?

That means touching stansummary and I really dont want to do exapand stansummary for no reason except to test. Will try to add a test to parse in C++.

I fixed it to the UTC timezone and added a test. It checks that the start_datetime was found, that the year matches and that the start_datetime ends with "UTC".

If this PR is merged I would just proceed with the release.

Looking at it now.

Thanks for the quick review.


New Feature: implement http patch method in OSF session

I have searched for a way to update an existing OSF project (e.g., description, tags, category, ...) from within a script and couldn't find a way with the existing source code (I may have missed it, so please let me know if there is another, already implemented way to do this). This PR adds support for patching a resource, and thus lays the ground work for updating nodes via PATCH request.

Thanks, @adswa!  I've merged this into develop, and it will be included in the next release.

This has been merged into master and released in osfclient==0.0.5. Thank you, @adswa!


Update index.md

modify " ' " of the sentence

This change is

@hanxiaoshuai :wave: Please check the box to allow edits from maintainers.


upgrade to apptainer and ubuntu22.04(jammy)

It also brings a fully rootless installer thanks to apptainer v1.1

timescaledb tuner is moved to build stage
new command postgres admin for the initial admin access

Hey, thanks for contributing, love it. 🥃

Couple of things to consider:
I wrote this for use on high-performance computing cluster (specifically at Purdue University) but is also usable at other sites and likely DoE labs.

We don't as of now have Apptainer (but I'm interested in this I suppose). We only have Singularity.
The reason I have the tune step as a second stage thing and not as part of the build is because it can change depending on what node you land on. You could have 128-cores and a TB of memory, or 20 cores and very little. So within your batch job script, you would write tune after initdb to let it make good choices.

If you would mind, let's split this up into separate PRs.
First PR to update to PG 14. Second PR to clean up, add admin workflow, etc.
Then I would be interested in letting Singularity and Apptainer co-exist simultaneously here. We could facilitate that by leaving both build files, and use a variable in the script and maybe a bit of logic.

Got it.
I don't mind if you prefer to do some or all steps (not seeking for credits).
I'll try to find time next week for this, in your order.

Then I would be interested in letting Singularity and Apptainer co-exist simultaneously here

There's no need for that. Apptainer has internal transition helpers which allow to use singularity command as before, also SINGULARITY* env variables. So your version would currently works with Apptainer (but ubuntu:latest would have to switch to focal / 20.04 for the timescaledb part to work).
I am also running an HPC cluster (a small one with Slurm), including using GPUS.
I found Apptainer very efficient for using remote GPU and/or EGL.
What's great with Apptainer is that sudo becomes completely unnecessary.

closing per #2


How do I save and load this PU_bagging model?

Now I have completed the training of the PU_bagging model,but I  can't find how to save and load my model.And if  I save each child model in sklearn's method,how can I load the bagging model?

Did pickle-ing the whole object failed?

Yeah,I have used joblib to save and load my model successfully.Thank you!!

That's good to hear. Thank you for updating me. :)


Black screen (Modules not appearing)

Hello, I have installed my Magic Mirror and it has been working for a while without any issues
But now, the screen is just black and the modules are not appearing.
The MagicMirror tab used to have the name MagicMirror on the top of the screen, but now it is called electron(i don't know why)
Can anyone please tell me what to do? I am still a beginner, so please tell me what to do step by step.
Thanks in advance.

this usually means there is an erorr..
how do you start MM?  thru npm start, or pm2?
if pm2, then the pm2 logs --lines=100 command should show the output

I am having issues as well. first install works (calendar and news don't load) reboot only get black screen.
npm Start shows below
28.06.2021 22:02.25.572] [LOG]   Starting MagicMirror: v2.15.0
[28.06.2021 22:02.25.581] [LOG]   Loading config ...
[28.06.2021 22:02.25.598] [LOG]   Loading module helpers ...
[28.06.2021 22:02.25.603] [LOG]   No helper found for module: alert.
[28.06.2021 22:02.26.072] [LOG]   Initializing new module helper ...
[28.06.2021 22:02.26.074] [LOG]   Module helper loaded: updatenotification
[28.06.2021 22:02.26.078] [LOG]   No helper found for module: clock.
[28.06.2021 22:02.27.430] [LOG]   Initializing new module helper ...
[28.06.2021 22:02.27.432] [LOG]   Module helper loaded: calendar
[28.06.2021 22:02.27.435] [LOG]   No helper found for module: compliments.
[28.06.2021 22:02.27.438] [LOG]   No helper found for module: weather.
[28.06.2021 22:02.27.448] [ERROR] WARNING! Could not load config file. Starting with default configuration. Error found: Error: Cannot find module 'valid-url'
Require stack:

/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js
/home/pi/MagicMirror/js/app.js
/home/pi/MagicMirror/js/electron.js
/home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js

[28.06.2021 22:02.27.450] [LOG]   Loading module helpers ...
[28.06.2021 22:02.27.452] [LOG]   No helper found for module: alert.
[28.06.2021 22:02.27.453] [LOG]   Initializing new module helper ...
[28.06.2021 22:02.27.455] [LOG]   Module helper loaded: updatenotification
[28.06.2021 22:02.27.456] [LOG]   No helper found for module: clock.
[28.06.2021 22:02.27.458] [LOG]   Initializing new module helper ...
[28.06.2021 22:02.27.459] [LOG]   Module helper loaded: calendar
[28.06.2021 22:02.27.461] [LOG]   No helper found for module: compliments.
[28.06.2021 22:02.27.464] [LOG]   No helper found for module: weather.
[28.06.2021 22:02.27.468] [ERROR] App threw an error during load
[28.06.2021 22:02.27.471] [ERROR] Error: Cannot find module 'valid-url'
Require stack:

/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js
/home/pi/MagicMirror/js/app.js
/home/pi/MagicMirror/js/electron.js
/home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
at Module._resolveFilename (internal/modules/cjs/loader.js:972:15)
at Function.n._resolveFilename (electron/js2c/browser_init.js:249:921)
at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/module-alias/index.js:49:29)
at Module._load (internal/modules/cjs/loader.js:848:27)
at Function.f._load (electron/js2c/asar_bundle.js:5:12633)
at Module.require (internal/modules/cjs/loader.js:1032:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js:9:16)
at Module._compile (internal/modules/cjs/loader.js:1152:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1173:10)
[28.06.2021 22:02.27.474] [ERROR] Whoops! There was an uncaught exception...
[28.06.2021 22:02.27.490] [ERROR] Error: Cannot find module 'valid-url'
Require stack:
/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js
/home/pi/MagicMirror/js/app.js
/home/pi/MagicMirror/js/electron.js
/home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
at Module._resolveFilename (internal/modules/cjs/loader.js:972:15)
at Function.n._resolveFilename (electron/js2c/browser_init.js:249:921)
at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/module-alias/index.js:49:29)
at Module._load (internal/modules/cjs/loader.js:848:27)
at Function.f._load (electron/js2c/asar_bundle.js:5:12633)
at Module.require (internal/modules/cjs/loader.js:1032:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js:9:16)
at Module._compile (internal/modules/cjs/loader.js:1152:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1173:10) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/home/pi/MagicMirror/modules/default/newsfeed/node_helper.js',
'/home/pi/MagicMirror/js/app.js',
'/home/pi/MagicMirror/js/electron.js',
'/home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js',
undefined
]
}
[28.06.2021 22:02.27.497] [ERROR] MagicMirror will not quit, but it might be a good idea to check why this happened. Maybe no internet connection?
[28.06.2021 22:02.27.499] [ERROR] If you think this really is an issue, please open an issue on GitHub: https://github.com/MichMich/MagicMirror/issues
[28.06.2021 22:02.28.265] [LOG]   Launching application.

@minggaofu
Cannot find module 'valid-url'
yes, this was removed in 2.15, and some module uses it, but didn't document it as a dependency
workaround
cd ~/MagicMirror
npm install valid-url

@ahmed-asaleh Does this solve your problem? If so I can close this issue. Thanks.

Yes Thanks for your help

Thanks. Closed. @sdetweil Thanks again for your great support!

Indeed Thank you!!
Sent from my iPhone

On Jun 29, 2021, at 1:08 AM, Michael Teeuw @.***> wrote:

@ahmed-asaleh Does this solve your problem? If so I can close this issue. Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Record shard allocation

Relates to https://github.com/elastic/rally/issues/1242

@gingerwizard I started with _cat/shards API for now, it gives you  storage and docs count. do you think it's sufficient or would you prefer segments? I could use node stats api with ?level-shards, which will have segement info as well, but then it gives node id instead of of node name.. not sure if that's important.

For _cat/shards reposne "geonames         4     p      STARTED    211237     63mb <IP_ADDRESS> rally-node-0"
The sample doc would like like this:
{'name': 'shard-stats',
'shard-id': '4',
'index': 'geonames',
'prirep': 'p',
'docs': '211237',
'store': '63mb',
'node': 'rally-node-0'}

metadata:
{'cluster': 'default'}

@gingerwizard I wonder if we want to have store always in bytes? And for primary/replica instead of p vs r could be True/False

So i think we need:

A count of segments if possible
Yes re primary: true vs p/r. Easier to filter.
And yes we need to convert all units to bytes so we can perform math at agg time + chart.

@gingerwizard Example of a doc pushed to the metric store:
            {
                "name": "shard-stats",
                "shard-id": "0",
                "index": "geonames",
                "primary": True,
                "docs": 1000,
                "store": 212027,
                "node": "rally0"
            }

The structure LGTM @ebadyano and fullfils the requirements thanks. I will now review the code.

that's one stubborn print!


mutlipart/form-data and Chunked transfer encoding

Hi,
I am currently doing stuff with file uploads as multipart/form-data and i guess the ascii encode blows up because you can not be sure that you dont have binary data from the request inside the $first-chunk.
https://github.com/supernovus/perl6-http-easy/blob/master/lib/HTTP/Easy.pm6#L103
Chunked transfer encoding is also not supported, right?

Yeah, I'm pretty sure chunked transfer encoding was on the TODO list. I haven't actively worked on my Perl 6 libraries in quite some time. Most of the work on them in the last couple of years has been contributions from others.


Improve safety rules observability

Log the config at startup -- it is a boring config, so only print it once
Log internal state each time consensus_state is queried
Log internal state at startup
Log backtrace on panic

/land

@metajack, it seems this doesn't ping @mimoo for a review if I am the author?


Add a device tree reader for ip-devices

Create ip-device objects  from an os devices directory.

@stv0g can you add your review please?


Fix NSData tests on reference platform

Fix NSData description so it returns the same values as reference platform's version, and change initWithBase64Encoding to return an empty NSData object when given invalid input rather than nil as documentaiton suggests
Fixes #748 and #749 NSData.Base64DecodeWithOptions test failure

NSData.LongDebugDescription seems to fail because the reference platform uses a different class (NSObject) in the NSData class cluster when the size of the data is >= 2^16 - 9 bytes (a little under 64kb) on my test machine, possibly different sizes depending on the test platform. To fix this test we can either A: create the OS_dispatch_data protocol for NSObject, which will probably require quite a lot of work or B: lower the size of the test data so we do not encounter this issue. Because this seems to be more an implementation difference that should only come up during logging than an actual functionality one, I went with option B.

:shipit:

:shipit:

:clock1:

:shipit:

:shipit:


Improve bug reporting template

Describe the bug
The current template uses sentence:
A version of CMS you are using (you will find it in the footer of CMS Admin Panel).
The version cannot be checked in the footer
The template should contain new section:
Actual behavior
showing the actual problem, different from Expected behavior

Solved. The new bug template points out the version on the login page.


Test against rails 6

https://weblog.rubyonrails.org/2019/4/24/Rails-6-0-rc1-released/

@chanks just a question in general, why is the Gemfile.lock ignored?

@chanks Is the ci configuration broken? Or why is there no build to this?

Not sure how/if CircleCI works with pull requests, sorry.

This'd be awesome to get in! And, to get this tested I think you'd need to turn this setting on for circle https://circleci.com/docs/2.0/oss/#build-pull-requests-from-forked-repositories which will test PRs from forked repos like this one. It looks like its safe to do that because there's no secrets or sensitive data in the circle environment for the main repo but that page has more information on the risks associated with turning that option on.

-> https://github.com/que-rb/que/pull/2


Add docker and network pod functions to e2e framework

This PR is to add docker and network pod functions to e2e framework. These functions are originally needed for https://github.com/submariner-io/submariner/pull/1202 , however, as a result of discussion, we came to conclusion that it would be nice to add them to this framework to make room to leverage from other tests.

@mangelajo @sridhargaddam @tpantelis
Thank you for your reviews. I've fixed and squashed the commit.
I tested locally and confirmed that it works well with the submariner side's PR with "Fix for shipyard framework changes" commit.

Great work thanks @mkimuram


xkcdpass: py2 -> py3, add manpage and run tests

Motivation for this change
Drive-by cleanup
Things done

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

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

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

Using python3Packages in python-modules is not allowed because it breaks multiple python versions. Use disabled = isPyXX; instead.

Using python3Packages in python-modules is not allowed because it breaks multiple python versions. Use disabled = isPyXX; instead.

Thanks for the input guys.

Thanks for the input guys.

@GrahamcOfBorg build xkcdpass

@GrahamcOfBorg build xkcdpass


Overridden grades should be labelled

Overridden grades should be labelled as such on the student home page.

#49 addresses issue


PowerToys disabling some diacritic marks in Windows

Microsoft PowerToys version
0.70.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Fresh install, run as admin, only changed shortcut for Paste as a plain text to Ctrl+Shift+V.
I am using polish (programmer) keyboard input layout and diacritic marks are visible, e.g. when pressing AltGr in Windows On-Screen Keyboard, but not all of them render any input, while pressed both in OSK and on physical keyboard.
This was reproduced on two separate machines,
✔ Expected Behavior
All diacritic marks are properly input with AltGr key pressed. e.g. ą ć ż ź ć ś € ó ł ń
❌ Actual Behavior
Only some of diacritic marks are working input with AltGr key pressed, while PowerToys are in use.
Other Software
Windows 11 Pro 22H2 build 22621.1848  + Windows Feature Experience Pack 1000.22642.1000.0
Windows 10 Enterprise 21H2 build 19044.3086 + Windows Feature Experience Pack 1000.19041.1000.0

Keyboard layout affected (checked) - Polish (Programmers), especially diacritic marks like Ł (AltGr + L) and Ń (AltGr + N)

I have a moment to dig and it looks like enabling Mouse Without Borders causes this bug to keep happening

This issue also breaks AltGr+E that should yield € on a French keyboard.
Disabling both Keyboard Manager and Mouse Without Borders allows the shortcut to work.

My very first quick test of 0.73 with Keyboard Manager and Mouse Without Borders both enabled seem to indicate the AltGr+E problem is no longer there... yet this issue is not marked as closed....

Thanks for comme

This issue also breaks AltGr+E that should yield € on a French keyboard.
Disabling both Keyboard Manager and Mouse Without Borders allows the shortcut to work.

My very first quick test of 0.73 with Keyboard Manager and Mouse Without Borders both enabled seem to indicate the AltGr+E problem is no longer there... yet this issue is not marked as closed....

Thanks for commenting however after update to 0.73 issue is still present on my side.
AltGr+L results in logout instead of ł

This issue also breaks AltGr+E that should yield € on a French keyboard.
Disabling both Keyboard Manager and Mouse Without Borders allows the shortcut to work.

My very first quick test of 0.73 with Keyboard Manager and Mouse Without Borders both enabled seem to indicate the AltGr+E problem is no longer there... yet this issue is not marked as closed....

Problem's back, so I had to disable Mouse Without Borders...

Issue still present with 0.76.2

This issue is present on 0.83.0 (with Mouse Without Borders disabled).
As long as Keyboard manager is enabled, I cannot use Polish diactric marks.


Update plugin to new provider service

I think it's nice to let plugin developers know that there's a new provider service available so we can remove the legacy code as soon as possible. Besides that, the previously documented provider semver won't support the new version we're releasing.
There are two lists of plugins:

Plugins that consume the service and the public API.
Plugins that consume the service and use the return type.

List one:

[x] Menu Manager :arrow_right: https://github.com/jerone/menu-manager/pull/4
[x] Tool Bar Main :arrow_right: https://github.com/suda/toolbar-main/pull/15
[x] Toolbar Almighty :arrow_right: https://github.com/varemenos/atom-toolbar-almighty/pull/15
[x] Particle Dev :arrow_right: https://github.com/spark/particle-dev/pull/150
[x] PlatformIO IDE :arrow_right: https://github.com/platformio/platformio-atom-ide/pull/130
[x] Toolbar Shortcuts :arrow_right: https://github.com/JostCrow/atom-toolbar-shortcuts/pull/5
[x] Tool Bar Atom :arrow_right: https://github.com/jeselxe/tool-bar-atom/pull/7
[x] Toolbar for Markdown-Writer
[x] toolbar-basic :arrow_right: https://github.com/mattlovaglio/toolbar-basic/pull/1
[x] atom-toolbar-config :arrow_right: https://github.com/MunGell/atom-toolbar-config/pull/1
[x] atom-vital-tool-bar :arrow_right: https://github.com/taronpa/atom-vital-tool-bar/pull/1
[x] bloginatom :arrow_right: https://github.com/hevnly/bloginatom/pull/2
[ ] Toolbar Ioton :arrow_right: https://github.com/iotontech/ioton-atom/pull/1
[x] awesome-tool-bar :arrow_right: https://github.com/radjivC/awesome-tool-bar/pull/1
[ ] toolbar-iconshortcuts :arrow_right: https://github.com/thiagolucio/toolbar-iconshortcuts/pull/2
[ ] atom-toolbar-besser :arrow_right: https://github.com/besser/atom-toolbar-besser/pull/1
[x] bgc tool bar :arrow_right: https://github.com/bgc/tool-bar-bgc/pull/1
[ ] hayatan-tool-bar :arrow_right: https://github.com/hayatan/atom-hayatan-tool-bar/pull/1
[x] Atom Electric Imp Tools :arrow_right: https://github.com/deldrid1/atom-imp-tools/pull/1
[x] love-ide :arrow_right: https://github.com/rameshvarun/love-ide/pull/17

List two:

[ ] Flex Tool Bar :arrow_right: https://github.com/cakecatz/flex-toolbar/commits/coffee-to-es6
[x] Nuclide :arrow_right: https://github.com/facebook/nuclide/commit/173f26ea93c956b465f517eb89d962e75071b7f2
[ ] Atom-macros (in an invalid way btw) :arrow_right: https://github.com/pbihler/atom-macros/issues/8

This is a work in progress as there are way more plugins available. List seems complete for the moment.
Ref https://github.com/suda/tool-bar/issues/135, https://github.com/suda/tool-bar/pull/136, https://github.com/suda/tool-bar/pull/140.

Nuclide does tuesday releases (internally and open-source), so if you publish the new tool-bar on Monday morning, then that gives me a day to update. This would be the ideal case.

I'm going to hold of publishing for a little bit. This issue was opened to get a list of plugins using the tool-bar API and informing them of the changes before releasing the package. The list was summarized out of the top of my head, but requires some more research. The idea right now is to update the list, at least informing the package authors and maybe sending an PR.

Can someone confirm that using ^0 || ^1 is the correct semver range for using in the new service provider...
/cc @suda @cakecatz @zertosh

Yeah...
require('semver').toComparators('^0 || ^1')
// [ [ '>=0.0.0', '<1.0.0' ], [ '>=1.0.0', '<2.0.0' ] ]

Thanks for the confirmation on the ranges. Some explanation. Previously we had ^0.1.0 but I learned that that would only select >=0.1.0 <0.2.0. For 1.0.0 I want to make sure we get all new minor updates too. Seems valid I think.

Yup. See https://github.com/npm/node-semver#caret-ranges-123-025-004

[Caret Ranges allow] changes that do not modify the left-most non-zero digit in the [major, minor, patch] tuple. In other words, this allows patch and minor updates for versions 1.0.0 and above, patch updates for versions 0.X >=0.1.0, and no updates for versions 0.0.X.

Because of the very hard work and if you agree, I like you to add to the list of contributors on the readme.
In https://github.com/suda/tool-bar/pull/140#issue-154883976 you mentioned that there is room for improvements and performance. I'm especially interested in your thoughts about performance improvements.

Thanks @jerone! 😄

@jerone as far as perf:

Dynamically load the stylesheets for the icon sets you use. This will cut the startup time in half.
Debounce with requestAnimationFrame the resize and scroll events.
Insert the initial buttons in a single batch. Right now you're forcing a relayout for every button at startup. Really expensive.

This is an important one! The iconsets are "huge", so if we can somehow lazy load those files the startup time would indeed decrease. Besides that point, I can also see conflicts with other packages, when they load those iconsets too.
Both scroll on the tool-bar and resize on the window won't happen that much, but still it's an good idea to debounce those events.
Aldo I see the benefit of this (forcing redraw events), I don't see how we can improve this. At anytime it's possible to addItem to the tool-bar, not only at startup (or when the service is called).

Both scroll on the tool-bar and resize on the window won't happen that much, but still it's an good idea to debounce those events

I resize my window a lot - probably because I'm always opening/closing the devtools. This is easy to fix though.

Aldo I see the benefit of this (forcing redraw events), I don't see how we can improve this. At anytime it's possible to addItem to the tool-bar, not only at startup (or when the service is called).

I'm only concerted with batching the DOM updates during the Atom package activation cycle. We put 13 buttons on the tool-bar - that's a lot of redrawing. I'm on a mission to reduce start up time :)

Any word on publishing the space-view-less update?

@zertosh commented on 8 jun. 2016 23:47 CEST:

Any word on publishing the space-view-less update?

Before publishing the package with the awesome new changes we need to contact all plugins (send PR) so they have time to react on the breaking changes. I've already started to research some plugins above, but the list is not yet finished.
Currently I'm to busy to finish the list and send PR's. But in 1,5 week I have some vacation time to spend on this issue.
I would be great if others could help. /cc @suda @cakecatz

@cakecatz Do you have time to update Flex Tool Bar with the latest API changes ... so it's compatible when we release then new Tool Bar package.

Half of above mentioned packages are ready for a new Tool Bar release, with remaining package on very low install counts.
I plan on releasing a new version this Sunday.
@zertosh @cakecatz Any objections?

@jerone We're good on my side! Thanks!

Do you have time to update Flex Tool Bar with the latest API changes ... so it's compatible when we release then new Tool Bar package.

@jerone Sorry, I had no time to do that... 😫😫😫
But I will working for it from today 💪

Closing, as all plugins have been notified and/or updated.

@cakecatz Almost 5 months after removing space-pen dependency and switching to ES6, it's time to merge #153 and remove all legacy code. The only thing blocking this is the Flex Toolbar package still running on the old API (most noticeable packages have already switched). Once merged, Flex Toolbar will completely stop working. I noticed you already made an attempt to upgrade on a different branch. Is there any change to speed up this process?
/cc @zertosh

@jerone, we're all good on my end

@jerone Hi! I pushed commit for support new tool-bar provider just now.

@cakecatz Great! Do you want me to test? I probably have some spare time tonight.

@jerone Yes please 🙏

@cakecatz commented on 7 sep. 2016 12:15 CEST:

@jerone Yes please 🙏

Just tested your latest commit with #153 and it worked with my configuration. Let me know when you released Flex Tool Bar, so I can merge #153.

@jerone Thanks!
I'll release now 🚢

Hi Jerone.
So..i correct the package last Saturday Night. Thanks to warning. I think ok now. The package is:
"toolbar-iconshortcuts"
Thanks Again.