New debugging APIs

This PR adds debugging APIs to ChakraCore and test debugger to ch.
Sample tests are under test\Debugger. Once this PR is merged next PR will port existing PDM based debugger tests to run with ch/ChakraCore. I have already run those tests and don’t see any issue.
New APIs are prefixed with JsDiag to group them under a pseudo namespace, in future if we plan to have other diagnostic APIs this will help.
 Currently the APIs on only exposed in ChakraCore once we have windows review this will be exposed in Chakra as well.
With this change the binary size of ChakraCore/Chakra increase by about a max of 19KB across all architectures which is acceptable.
To demonstrate the use, capabilities and completeness of these APIs Node-ChakraCore have been enhanced to provide debugging support with VSCode (no change to VSCode required). ChakraCore code in Node-ChakraCore is synced with this PR commit. The code is available at https://github.com/agarwal-sandeep/node-chakracore/tree/debugging
There is some more investigation required to make Node-Inspector work with Node-ChakraCore. I am working on it.

@akroshg @Yongqu @liminzhu @aruneshchandra please take a look

Looking good overall :). Left a few nitpicks here and there.

Is this Visual Studio compatible? I can not try to step into Javascript code in Visual Studio with codes mix with C++ and Javascript

@MidoriYakumo VisualStudio support debugging using Chakra and PDM and is currently not supported for ChakraCore. If your scenario runs using node the you can use Node-ChakraCore to debug using VSCode


feat: add default version option to asdf plugin versions

Adds 'GetVersionOrDefaultto return a default version of the tool if it's missing in.tool_versions`

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 11 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 7.602%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

catalog/asdf/asdf.go
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
13

Relevant Lines:
171

💛  - Coveralls


Allow instanciating multiple globals of the same type & protocols refresh

[client] Add Client.get_globals
[protocol] Refresh wayland.xml and xdg-shell.xml to their latest version

Hope this helps!

Merged, Thank you!


Hub: use relative path in repos.hub to support ticat data-dir relocating

Now hub path in repos.hub is absolute path. If we transfer ticat/ to another machine/dir, it may cause errors.

Supported at #254


[CI] Added typechecking

[x] I have followed (at least) the PR section of the contributing guide.

Disabled automatic inclusion of all visible @types packages see https://github.com/DefinitelyTyped/DefinitelyTyped/issues/33015#issuecomment-463062037 and https://www.typescriptlang.org/docs/handbook/tsconfig-json.html#types-typeroots-and-types

Added typechecking to the entire repo, except

docs/src/pages/demos/pickers/MaterialUIPickers.tsx due to a bug in the definitions for material-ui-pickers
examples folder and *spec.ts(x) files

The errors reported in the build are fixed in https://github.com/mui-org/material-ui/pull/15402

What does this accomplish? What files didn't we typecheck before?

It checks all files, compared to what happens now.
I don't know exactly what was not getting tested but it's clear the *.d.ts files in material-ui-styles wasn't getting checked properly, as it contains invalid imports.

I don't know exactly what was not getting tested but it's clear the *.d.ts files in material-ui-styles wasn't getting checked properly, as it contains invalid imports.

yarn workspace @material-ui/styles typescript did run on those files. I don't think we need another task.
Could you explain why these are invalid imports? Our demos are relying on them so it seems like they're fine.

Could you explain why these are invalid imports? Our demos are relying on them so it seems like they're fine.

If you check out this PR before I changed the imports and go to any of the demos and hover over the makeStyles function this is the result:

And classes:

From there go to the definition of makeStyles and scroll up to the imports, this is the result:

If you change the imports to be relative or add this to the root tsconfig.json it finds everything as it should
"@material-ui/styles": ["packages/material-ui-styles/src"],
"@material-ui/styles/*": ["packages/material-ui-styles/src/*"]

However, then you run into issues where createStyles isn't used, this issue is fixed in newer versions of TypeScript.
So the fix for these is either use createStyles, update typescript, or allow all properties to be strings

We can't verify if that isn't an issue with the editor. Those can e.g use different typescript versions.
Can you provide a clean repository that fails on a build step?

We can't verify if that isn't an issue with the editor. Those can e.g use different typescript versions.

VSCode is set to use the same version as the repo uses (3.2.2)

Can you provide a clean repository that fails on a build step?

No, it works perfectly fine other places. It's only from the docs folder it's having issues

@eps1lon Turns out tslint just ignored the import errors, I simplified this to just set the alias in docs/tsconfig.json

Well I have no idea what happened but it seems like our docs never type-checked properly anything related to @material-ui/styles. I don't know where typescript pulled the declarations for @material-ui/styles from or why it didn't report implicit any.

That's what I've been trying to get across xD

@merceyz Yeah I'm super grateful for this. I had a lot of assumptions about our demos that were basically not true. I'm still not sure why this happened but at least we know what's wrong now.
I think we should rather use
 tsconfig.json | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tsconfig.json b/tsconfig.json
index cfbb2026495..6a66cfbc698 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -17,6 +17,8 @@
       "@material-ui/core/*": ["./material-ui/src/*"],
       "@material-ui/lab": ["./material-ui-lab/src"],
       "@material-ui/lab/*": ["./material-ui-lab/src/*"],
+      "@material-ui/styles": ["./material-ui-styles/src"],
+      "@material-ui/styles/*": ["./material-ui-styles/src/*"],
       "@material-ui/system": ["./material-ui-system/src"]
     }
   },

and add the noUnusedLocals later.

@eps1lon Agreed. Now the question is, what to do about the type inference issues, it's fixed in newer versions of typescript.

The error at material-ui/docs/src/pages/system/basics/JSS.tsx:14:6 is fixed by https://github.com/mui-org/material-ui/pull/15548


.Net: Remove support for passing multiple functions to Kernel.RunAsync

Motivation and Context

Kernel.RunAsync will only support running a single function
Plans can be use dto run multiple functions if needed or
Developers can create their own strategy to do this

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Kernel.RunAsync will only support running a single function

Are you thinking this is temporary until the hooks support is moved into KernelFunction.InvokeAsync?
Or are you thinking we'd ship with this? Once the hooks support is removed, and like this would just be a one-liner whose sole purpose was to swap the order of arguments.

Kernel.RunAsync will only support running a single function

Are you thinking this is temporary until the hooks support is moved into KernelFunction.InvokeAsync?
Or are you thinking we'd ship with this? Once the hooks support is removed, seems like this would just be a one-liner whose sole purpose was to swap the order of arguments?

The intent is to ship without this specific functionality.
We will provide a sample demonstrating how to implement a custom pipeline.

Ok, so the single function support is just temporary?
(If we were to keep it, I'd want to rename it to InvokeAsync, to match both InvokeAsync on a function and InvokePromptAsync.)

Ok, so the single function support is just temporary?
(If we were to keep it, I'd want to rename it to InvokeAsync, to match both InvokeAsync on a function and InvokePromptAsync.)

No single function support will be the permanent. My understanding is that we wanted to simplify things so that the Kernel extension methods could be used as a convenience to run a single kernel function.
Renaming so we only have one name for this makes sense. That avoids the question as to what the difference between the two methods is.

Renaming so we only have one name for this makes sense. That avoids the question as to what the difference between the two methods is.

Yeah, and functionally we'll want to keep them identical, with the kernel method just delegating to the function method.
It also doesn't need to be an extension anymore; it can just be an instance method on kernel.

Renaming so we only have one name for this makes sense. That avoids the question as to what the difference between the two methods is.

Yeah, and functionally we'll want to keep them identical, with the kernel method just delegating to the function method.
It also doesn't need to be an extension anymore; it can just be an instance method on kernel.

I'm going to make this change in a seperate PR as there are a lot of files to be changed and it will be easier to code review on it's own.


Avoid player leaving the scene

At the moment its possible for the player to leave the "office"-scene, that should be avoided.

What do you mean? Of course, a player should be able to leave the game. Or do you mean that an avatar should not be able to walk through the walls of the office? Your issue should be clearer about that.

What do you mean? Of course, a player should be able to leave the game. Or do you mean that an avatar should not be able to walk through the walls of the office? Your issue should be clearer about that.

Sorry! I meant going through walls and the floor. I edited the description and title, hopefully it’s more clear now.


[update] New syscall 'svc_alarm'

The new svc_alarm syscall triggers a user handler after some user defined amount of time (in milliseconds)

Great ! Thanks @arnauldm !


fix directive sort compare

fix #3768
Object.keys(groupedMap) returns an array whose elements are strings, so it breaks the sort function.

Thanks! LGTM
I'm not sure about the test though

Ok, current test case seems not to fully cover the code. I will try to update the test case soon

forgot it. I will fix it later today

Thanks! 👍

Thanks, fixed the bug 👍


Deployment questions

Hi, Awesome work!  I was exploring to do inference using TensorRT and was wondering if the layers are supported or if you guys had any plugins if you have tested it already!
Thanks.

Hi, we haven't tried TensorRT yet, as we are not experts on this field. SparseBEV can be implemented by native pytorch (the CUDA operator is optional and only 15% faster), so there should be no problem using TensorRT.
We welcome your contributions, feel free to post a pull request if you have any progress!


ISSUE: shortcuts are not working using Windows

ISSUE
Hi,
I just realized that shortcuts don't work with Windows :o cmd key does not exist.
And it's probably the same with Linux environment. You should try to adapt shortcuts according to environments?
Like:
'.platform-darwin':
  'cmd-alt-v' : 'project-viewer:toggle-select-view'

'.platform-win32, .platform-linux':
  'ctrl-alt-v' : 'project-viewer:toggle-select-view'

Thx :)
Environment

OS: Windows 10;
Atom: 1.12.0;

hey @lneveu thanks! my bad :+1:
let me see if i can make if available tomorrow, fixing some anoying bugs from 3rd party deps...

@lneveu o.O what version of the package you are using?
latest version uses the following keymaps
{
  ".platform-darwin, .platform-win32, .platform-linux": {
      "ctrl-alt-cmd-v": "project-viewer:toggle-select-view",
      "ctrl-alt-cmd-b": "project-viewer:toggle-display",
      "ctrl-alt-cmd-n": "project-viewer:toggle-focus"
  }
}

Can you validate this for me?

(I'm using the last version of the package at home, but at work I'm still with the 0.2.34 because of this issue #76 ^^)
Sorry, I misspoke :/ It was just an example of how you can (maybe?) fix this issue.
Here, the full keymaps that I have changed to match with my usage:
{
  ".platform-darwin": {
    "cmd-alt-v": "project-viewer:toggle-select-view",
    "cmd-alt-b": "project-viewer:toggle-display",
    "cmd-alt-n": "project-viewer:toggle-focus"
  },
  ".platform-win32, .platform-linux": {
    "ctrl-alt-v": "project-viewer:toggle-select-view",
    "ctrl-alt-b": "project-viewer:toggle-display",
    "ctrl-alt-n": "project-viewer:toggle-focus"
  }
}

I let you try if it's ok for you? Or if you want to change keys or something.

:tophat: sir

i have mapped for the 1.0.0 release the following combinations:
shift-ctrl-alt-<x> do you feel comfortable with this?

Thx!
Maybe those combinations are not "very handy" to be easily used but, as you mentioned it, there are so much packages and the number of keys is limited ^^
Anyway, people can re-map the commands with their own shortcuts using the keymap.cson file if they want 😄
(in my case, I'll certainly re-map them with shift-ctrl-<x> 😛 )

Just release 0.3.26. Can you check this out please?

Thank you! All seems good! (and don't forget to update README.md file with updated shortcuts 👍 )

Good catch, just updated. Will close this then!


Question regarding tomlkit pin

Hello,
Is the tomlkit pin still required? It's causing a bit of headache on a project of mine and was wondering if the pin could be removed?
I saw #336 and #337 but it does not seem like a proper long term fix as noted in 337.
Cheers

The question is not resolved, don't know how to keep the issue alive

up


Amazon SQS message retrieval optimization

Message retrieval & processing pipe could be optimised. Currently PrefetchCount messages are retrieved from transport and all of them are processed (Task.WhenAll). Assume 10 messages have been retrieved, 9 of them completed in 15 ms but one damn bastard took 1500 ms to complete. This means entire bunch took 1500 ms to complete.
Some sort of buffered stream of retrieved messages could be implemented where n * PrefetchCount messages could wait for processing. Concurrency could be controlled by .UseConcurrencyLimit() perhaps? Whenever number of available messages drops below some point a new set of PrefetchCount messages could be retrieved from transport.
This isn't a simple thing to write and is even more difficult to control and configure well, but could potentially improve performance a lot. Like really a lot. Considering my initial example extra 891 messages could be processed (assuming 15 ms per message, 10 messages at once limit, all of them being fast ones). That's 10 messages (9 fast + 1 slow) vs. 901 (9 + 891 fast + 1 slow). Two orders of magnitude.
Things to consider:

should be optional most likely as it will be difficult to tune,
should be considered breaking change (PrefetchCount no longer meaning same thing),
messages in buffer could expire prematurely,
classical producer vs consumers race: ensure buffer is filled at pace, but not to fast; seems like System.Threading.Channels API could be useful

PS. I believe you already mentioned something like that in some SQS-related issue. It's also a driving factor for me personally. I'll work on some prototype soon.
Originally posted by @Crozin in https://github.com/MassTransit/MassTransit/issues/1821#issuecomment-617631249

I believe this is not AWS SQS specific as all transports work like that, am I right?

It's very SQS specific, all the other transports are push-based. Only SQS has to pull messages.

okay, so I chatted with @NooNameR and we have a plan for this, along with pseudocode. Will likely take a whack at it in the next day or two.

Okay, I've got this optimized really well - it is so much faster. I need to dry it up and test it some more, but with localstack or SQS is is significantly faster. ORders of magnitude faster.

I'm amazed you guys managed to implement that so quickly, wow! Just a question, as I haven't got time to review that in detail yet.
Main loop contains:
catch (Exception exception)
{
  LogContext.Error?.Log(exception, "BasicConsume Faulted in Polling Loop");
}

Shouldn't we somehow inform user-land about that? In case of some fatal crash where we cannot recover (say... invalid SQS permissions) this will silently* loop forever. Maybe some policy would be beneficial to crash app in such a case?

simple log entry seems to be not enough?


draft: Disabled redis cache

Hey there and thank you for opening this pull request! 👋🏼
We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Details:
Unknown release type "draft" found in pull request title "draft: Disabled redis cache".

Available types:
 - fix
 - hotfix
 - feat
 - feature
 - chore

/deploy-review

/deploy-review

/deploy-review

/deploy-review


neural network hidden layers

Hi,
I read your eBook on Machine Learning which very well explains everything about linear regression, neural networks etc. It's really helpful.
I however had a question on the neural network implementation in Python (https://dafriedman97.github.io/mlbook/content/c7/construction.html).
For simplicity, the model has only 1 layer between input and output as stated in the beginning of the text.
What about the following line in the code then? -> ffnn.fit(X_boston_train, y_boston_train, n_hidden = 8)
n_hidden is set at 8. Can you exaggerate on this? What does n_hidden mean exactly and why is it set at 8 in this example?
Kind regards,
Matthias

Hi Matthias,
I'm glad you're finding the book helpful!
I think the confusion is between the number of hidden layers and the number
of nodes per hidden layer. I assumed the model only had one hidden layer,
however this hidden layer has 8 nodes. I think the name "n_hidden" might
have been a bit confusing.
Consider the diagram below. It has two hidden layers, but each hidden layer
has 3 nodes. Does that clarify things? Please let me know if not.
Best,
Daniel
[image: Screen Shot 2020-10-05 at 7.42.12 PM.png]
On Mon, Oct 5, 2020 at 5:55 PM Matthias Van Hove<EMAIL_ADDRESS>wrote:

Hi,
I read your eBook on Machine Learning which very well explains everything
about linear regression, neural networks etc. It's really helpful.
I however had a question on the neural network implementation in Python (
https://dafriedman97.github.io/mlbook/content/c7/construction.html).
For simplicity, the model has only 1 layer between input and output as
stated in the beginning of the text.
What about the following line in the code then? ->
ffnn.fit(X_boston_train, y_boston_train, n_hidden = 8)
n_hidden is set at 8. Can you exaggerate on this? What does n_hidden mean
exactly and why is it set at 8 in this example?
Kind regards,
Matthias
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dafriedman97/mlbook/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ4JTZR7JUWM5FRVKMZ2WITSJI6FLANCNFSM4SFHJKBQ
.

Hi Matthias,
I'm glad you're finding the book helpful!
I think the confusion is between the number of hidden layers and the number
of nodes per hidden layer. I assumed the model only had one hidden layer,
however this hidden layer has 8 nodes. I think the name "n_hidden" might
have been a bit confusing.
Consider the diagram below. It has two hidden layers, but each hidden layer
has 3 nodes. Does that clarify things? Please let me know if not.

Hi,
Oh I see, so it's the number of nodes in the one hidden layer.

Ok, but then again, why is the number of nodes for that hidden layer set at 8?
Is it trial and error which number of nodes get the best result?

Can the number of nodes be higher than the number of input parameters? Or is always nNodes ≤ nParameters

Kind regards,
Matthias

Hi,

Exactly, number of nodes in the one hidden layer.

I pretty arbitrarily chose 8—maybe I should make that clearer. There is no one-size-fits-all rule for determining the optimal number of nodes, and it is more or less trial and error. In practice, I would suggest using cross validation to determine the number of nodes. There are some general rules-of-thumb, though. A common (but broad) one is that the number of nodes in the hidden layer should be between the number of nodes in the input layer (number of predictors) and the number in the output layer (number of targets). I would say it's more or less trial and error within that range.
There is no reason why the number of nodes can't be higher than the dimension of the input. However, increasing the number of nodes increases the potential for overfitting, so be careful!


Reports

2 reports are attached:

demonstration of Trello tasks (analysed Lighthouse reports and extracted tasks and added to Trello board)
demonstration of upskilling done to contribute to the frontend of the project (improve Lighthouse performance score)

Fixed Prettier errors in all files.

 Upskill: Learn CSS structures.

 Upskill: Learn vue.js, debugging JavaScript code & how to use docker

 Report on Lighthouse review and associated improvements added in backlog

 Upskill: JavaScript basics with HTML and CSS

 Write report demonstrating upskilling

 Upskill: Learnt TreeShaking, Ant Design, Code Splitting, Webpack usage and Lazy Routing.

Closing this as I have submitted PR on their behalf.


small fixes pointed by golint

The purpose of this PR is to do small fixes pointed by the linter.
Most of them are about documentation.
see https://go-lint.appspot.com/gorgonia.org/gorgonia

Coverage increased (+0.02%) to 62.913% when pulling 781a21e00664bc15a098bc9a67da477a2c66ea9d on linter into 93e238839b7603ce80c10bb7cff058a77869ea33 on master.

Once the work is done, the only messages that should remain are those related to Public functions returning private objects.
But this goes beyond linting and the fixes should be postponed to a release (it will probably have an impact on the API)

can I merge this?

Good to merge. Thank you.


Omero python deps

Move python requirements that are common to OMERO.server/OMERO.py and OMERO.web into a separate role. Adds configuration options for:

recommended vs minimal python deps: in theory the minimal deps are enough for web, in practice most people should just install all the deps (default)
options for choosing mostly distribution packages (default) vs mostly pypi packages. This currently only affects pillow but could be extended to cover most python requirements

This means the existing omero-runtime role (which omero-server depends on) is now just a "meta" role which only installs dependent roles, so could arguably be completely removed.
This is mostly aimed at runtime requirements, this doesn't touch the python-devel role.
omero-web-runtime now only installs web-specific requirements (gunicorn, django)

This fixes the IDR user photo:

just for the future. ansible/roles/omero-python-deps/tasks/pypi.yml  could be fully configurable from vars. Ideally if pip/yum packages could go to {{ omero_py_pip_dependencies }} like:
- name: omero.py pip dependencies
  become: yes
  yum:
    name: "{{ item }}"
    state: present
  with_items: "{{ omero_py_pip_dependencies }}"
  when: omero_py_pip_dependencies

- name: omero.py pip dependencies
  become: yes
  pip:
    name: "{{ item }}"
    state: present
  with_items: "{{ omero_py_pip_packages }}"
  when: omero_py_pip_packages

sorry for being picky, is zeroc-icepy not OMERO.py requirements?. How do we want to handle that? Ice role installs much more then just icepy.

Yes, that's why I added "This role does not install the Ice Python module." so we can leave that for later.

@aleksandra-tarkowska : ok to handle ice in a follow-on?
@manics : I assume this should result in no change for most servers?

I assume this should result in no change for most servers?

Recently there was a fight between two roles, one installing the python-pillow rpm, the other removing the rpm and using pip install pillow, so some servers may end up with both installed, or may already have both.

Merging for deployment at EBI. Follow ups to ... follow.


Any plans to public the training code?

Hi Stability-AI team,
Thank you for your outstanding work!
I looked at the code base and only found the test code, are there any plans to publish the training code?
Thanks again!

Hello ! Any update on this ? It would be great for the community to learn a little more about the training procedure to foster future research :)


travis: update go versions

As discussed at https://github.com/github/hub/pull/1573

@e-beach @mislav done.

Thank you!


Asus Aura Implementation 02

This pull request proposes the following changes:

Removes old Asus implementation as it does not quite work
Introduce AuraSDK2 support, using Asus's guidelines on how to implement it
Asus ROG Strix Flare Support (this time with extra LEDs and a new layout)

Known issues/To do:

AuraSDK2 Is known to be buggy, so this needs a lot of testing before it is merged in

https://cdn.discordapp.com/attachments/542047451201404943/680956713905553453/2020-02-23_12-58-18.mp4
An example of how to set up custom keys for peripherals

Could you add support for asus scope(sdk name: Charm), the sdk keycodes are not working for some the ctrl and windows key, and the asus sdk not supporting the uk layout just the us one,
I already done it in this pr: https://github.com/diogotr7/Aurora/pull/4

I'll look into it, I have Strix Flare. The internal name for it is also charm :/
Perhaps you have a different LED count to differentiate them? What's yours? I believe if you check the log when running my build should tell you

Then it is look like there is no difference for the 2 keyboard from the sdk perspective. :( Than I think the only way to differentiate them is to put multiple layout in aurora and check which one is selected.
I sending the my device information:

Hi, can't seem to get my Claymore to work using the build generated by AppVeyor...
Is there anyway I can help?

@xQwexx I added the RIGHT_WINDOWS key (on my keyboard it's FN), but the ctrl codes seem to match so im now sure what's going on there

@jdcmarques That might be because of your version of Aura that is installed. There is a guide to get that correctly setup on the Discord channel #asus-aura on the Aurora Discord Server.

So there is two thing first sdk hashcodes not support the uk layout so DeviceKeys.HASHTAG and DeviceKeys.BACKSLASH_UK not working so you need to set them manually, like this.
            //UK keyboard Layout
            if (keyCode == 0x56)
            {//Hashtag Key
                light = keyboard.Lights[(int)(3 * keyboard.Width + 13)];
            }
            else if (keyCode == 0x59)
            {//BackSlash Key
                light = keyboard.Lights[(int)(4 * keyboard.Width + 1)];
            }
            else
            {
                light = keyboard.Key[keyCode];
            }

Secondly asus strix scope support, because of the bigger left shift the left window and alt are not in the right place, which are not corrected by the sdk.
            // Asus Strix Scope
            if (keyboard.Name == "Charm")
            {
                if (keyCode == 0XDB)
                {//left window
                    light = keyboard.Lights[(int)(5 * keyboard.Width + 2)];
                }
                else if (keyCode == 0x38)
                {//left alt
                    light = keyboard.Lights[(int)(5 * keyboard.Width + 3)];
                }
            }

Added a way to test lights for custom peripheral devices. To make life easier

i just get cant find aura sdk and i have to idea zhere to get it i can only find aura creator on my pc


Adjust the project to be able to collaborate with the development

First of all, thanks for exposing this awesome project to the open source community.
Being honest I would like to collaborate with this project but certain APIs turn that task into a nightmare, so, I forked this project and saw that some envs like "CEDULA_API", "JCE_PHOTO_API", "ORY_SDK_URL" can be mocked just creating an external server that would act like an external development server for those that are not part of the core team but would like to collaborate.
Already speaking of the other dependencies to which the project obeys as AWS and ORY, I think that you must provide an step by step for any developer interested in collaborating should look for their own credentials ( Not recommended 😅) or explain some ways to mock them up.
If this issue stays open during a long time, maybe I will create that solution for the community, I am just open this issue for anyone interested as I don't have the time right now to focus on that task, I just open the invitation. :)
On tune!

Totally agree with @Devrax 👍🏽

do we have any update for this? @gustavovalverde


fix: publish release failed

Summary

Modify the classifiers in setup.py.
Solution Description

'License :: OSI Approved :: Apache Software License 2.0' -> 'License :: OSI Approved :: Apache Software License'

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your<EMAIL_ADDRESS>seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[5.7] Improve warning for deprecated collections config

5.7 cherry-pick of #5745

@swift-ci please smoke test


Source code for orbital convention figure

@iancze: do you have the source code that you used to generate the orbit3D figure?

Turns out it was an SVG made in inkscape.


search results do not show upon visiting /search/user_query/

If one visits https://sapir.artsrn.ualberta.ca/cree-dictionary/search/niskak/, the query "niskak" will appear on the search bar, but search results won't show untill you make changes on the search bar.

actually this works for me locally -- I can't reproduce the issue. Can anybody confirm? @eddieantonio @kobexamoh

I can still reproduce :(
Try this: https://sapir.artsrn.ualberta.ca/cree-dictionary/search/wapamew/

Lol nm, I just didn't wait long enough for the app to redeploy >.<
Please close this again!
El mar., 18 de feb. de 2020 15:57, Matt<EMAIL_ADDRESS>escribió:

Closed #210
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/issues/210
via #276
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/pull/276.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/issues/210?email_source=notifications&email_token=AARQE7NY44EHETVRIYKA5IDRDRRUDA5CNFSM4KHK6SB2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWXADHYQ#event-3049272290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARQE7L3QQXKCEZXXRH55E3RDRRUDANCNFSM4KHK6SBQ
.


Not working in all Desktops

KeyCastr is working only in only one Desktop.
assume! it was launched in Desktop1, then it is working only in Desktop1, Now if I want it to work in Desktop2 then I have to first open the preference window than move it to Desktop2,then it will work only in Desktop2.
please fix this.
Thanks in advance

I fixed it by!
I just displayed it's icon in dashboard from preferences. right click -> Options ->Assign To All Desktops
now it's working everywhere.
Awesome Software..


Fix hack 2

Looks prettier this way.

One of the motivations here is to actually not recreate the fi at all. I'm trying to refactor towards that.

Coverage increased (+0.008%) to 82.601% when pulling e49575ad82a906aedfa30d9176a193f4728e1acc on tdaede:fix_hack_2 into ef7a66a2ff710352d8fd0048c67048158c2e28ff on xiph:master.

can i has a better title for this PR? 🙏


[BUG] - shared mount files not showing up on recent upgrade for 2024.9.1rc3

Describe the bug
After the changes  in #2593, a small bug was introduced concerning how the shared volume mount was being handled. Instead of mounting to /shared/<group_name>, an extra slash delimiter was unintentionally added due to a bug in one of the Keycloak to JupyterHub sections of the code.
This led to new subpaths being included in the pod specifications as /shared//<group_name>. For newer deployments, this issue was not apparent and was difficult to catch unless inspecting the pod's YAML specification directly, since the subpaths and symlinks were generated correctly—albeit pointing to a double-slash folder location.
However, the problem arises in existing deployments because the symlinks would have been broken. The volume mounts were not correctly mounted since those subpaths did not exist, which would most likely generate conflicting data.
Bellow you can see a comparassion of a upsgraded cluster and a fresly created one:

Here due to an issue with the setings in the volumeMoutns, the vlume was not created and thus the expected shared link was not correcly binding in the user jupyterlab container as seen below:

Here you can see the fresly instanciated EFSs from a new cluster do accept the double slashing, albeit this is wrong.

Expected behavior
shared folder should exist and contain the required user groups on upgraded nebari clusters
OS and architecture in which you are running Nebari
Linux
How to Reproduce the problem?
upgrade an existing nebari deployment to 2024.9.1rc* release
Command output
No response
Versions and dependencies used.
No response
Compute environment
None
Integrations
No response
Anything else?
No response

The associated fixing PR addresses the broken link and incorrect subPath in the pod volumes specification. Still, it does not address the user facing issue after spinning up the cluster (after the upgrade), as seen in the image below:

After the upgrade, the user with a logged-in session will have an issue with his /their shared folder, as seen in the image below. This is because the user_data is "cached" internally, resulting in the required metadata for the group mounting not showing up in the lab's spawning process.
There's nothing we can do except log in and log in again to update the data.... every user will need to do that. I wonder if this could be added as part of the upgrade command as an option to the user as well #2739


Angular module load errors are suppressed when they occur as part of UI-Router resolution

I've spent about 2 hours trying to track this down -- the first hour mainly scratching my head, trying to figure out what "suddenly changed" in my app to break it.
The short version is that I inadvertently named a service differently from the name I used to attempt to inject said service. There is a bit of a dependency chain, but at the top is a login service which I'm pulling into the (default) abstract state:
  $stateProvider.state('app', {
    abstract: true,
    resolve: {
      auth: ['login', function () {
         console.log('start..?');
      }]
    },
    views: {
      header: {
        template: ''
      },
      footer: {
        template: ''
      }
    }
  });

Angular fails to resolve the login service (because of a misnamed dependency) but the error is suppressed (does not appear in the console). In addition, because this service is not resolved, UI Router fails to initialize and I just end up staring at a blank page -- no console output, no error.
A work-around (found from StackOverflow) is to listen for $stateChangeError events:
  $rootScope.$on('$stateChangeError', function (ev, toState, toParams, fromState, fromParams, error) {
    if (error) {
      console.error(error);
    }
  });

Obviously this has to be done early on, otherwise the error still doesn't show. The workaround I'm using at the moment is a service which does this and then I request resolution of that service before resolution of the login service:
...
resolve: {
   auth: ['logRouteErrors', 'login', function() {
   }
}
...

But this is fragile as any error which potentially happens before this is still suppressed.
Not having a full knowledge of the internals of UI Router at this point, I'd like to suggest that, at the very least, the above handler be added by default -- suppressing errors just leads to hours of frustration on the part of the hapless developer (read: me), trying to find the source of a problem which was actually trivial to fix. Probably better would be finding the cause of the suppression and eliminating it.

Eh, wait, I think I may need more info here. The problem is real, but my solution may not be (the subscription). Will re-submit when I can confirm a work-around.

above handler be added by default

1.0 has a defaultErrorHandler

That's great -- except a developer has to activate this just to get back errors that they would have gotten from angular core. If they don't know that ui-router can suppress errors, then they waste time trying to figure this out -- in my case, around 2 hours. And the SO thread I got the idea from was answered by someone with exactly the same frustrations. Please consider this for a fix in ui-router: libraries should not suppress each others' messages.

1.0 has defaultErrorHandler which is enabled by default

That's great -- how do I get it? The angular-ui-router package at npmjs.com is at version 0.3.1. So anyone using
```npm install angular-ui-router``
(such as most people packing with webpack, for example) will get this apparently quite outdated version.
I also don't know (perhaps you could elucidate) if 1.0 supports Angular 1? Angular 2 is great and all, but the RCs are still churning a little too much for production work, imo.
Side-note: if the default was working for me, surely I wouldn't have bothered with opening a ticket? Did it occur to perhaps ask why, when this has been fixed in 1.0, I would raise a ticket about it?  The point of raising a ticket is not just to whine about it -- for someone not already embedded in the code, reporting bugs is the next best thing to creating a PR.

[0.3.1] apparently quite outdated version

0.3.1 is still the latest stable release.  1.0 is in beta, rapidly approaching final release. You can get 1.0 from npm like so:
npm install<EMAIL_ADDRESS>or even npm install angular-ui-router@next

you should expect reports of errors already fixed upstream when you haven't updated the package, no?

Yes definitely... there's no problem with bug reports like this.  I closed this issue because the fix is complete, and already merged to master.

if 1.0 supports Angular 1?

1.0 supports both angular 1 (as angular-ui-router) and angular 2 (as ui-router-ng2).  This migration guide has details about 1.0 (and the default error handler in particular.

Thanks kindly for the response. It really is appreciated.
At the risk of sounding like a spoilt, pushy user, is there an eta (no matter how rough) on a final release for 1.0? Knowing a timescale would help me decide whether to use vnext or leave my kludgy workaround in place. I'm just trying to make the best decision for a project barely started, but which should run for a while henceforth, bearing other (less experienced) team members in mind. In other words, I'd like to leave the code with the least "wat", should I mysteriously disappear.
I'd like to reiterate appreciation for your time spent on this ticket and apologize if I came off abrasively. It's been one of those days :/

BTW, "it will be ready when it's ready" is also an acceptable answer (:

If it takes longer than a month to get 1.0 final out the door, I'll be very disappointed.

Excellent! The kludge remains, with a TODO to remove it in preference of updating to 1.0, when released. Thanks again for the prompt response.


Delete adjustment probabilities

Closes #582
Should be merged after #579

After removing the adjusment probabilities - all the blends are fixed! Except for one that I should investigate where perhaps its close to the border of a tile.
The precision/recall/f1 is the best I've seen as well.


Some manpages are incomplete / cut off

Like here: https://manpages.debian.org/stretch/dash/sh.1.en.html
The page only half covers until the section "Shell Patterns"and rest is missing.

We’ve had a similar report recently: https://github.com/Debian/debiman/issues/100
I still cannot reproduce the issue.
Which browser are you using? Can you enable the developer tools and tell me which IP address you’re hitting?

Now the dash manpage works. But wget doesn't. My browser is firefox 59.0.1. But I seem to be getting the complete page content and it may just be a firefox bug.
The IP address I'm hitting is <IP_ADDRESS>.

I can observe the following in Firefox 58: pressing C-l to get the page from cache makes it show up completely, and disabling the cache makes it not show up competely.
Seems like a Firefox bug to me. Can you report it in the Firefox bug tracker please?

The issue is reported to mozilla.

@dosimple Can you provide a link to their Bugzilla?

Sorry, I can not find the link. I might have used their bug reporting tool outside of their Bugzilla. I followed "Report an issue with a web site that I use" at https://bugzilla.mozilla.org/enter_bug.cgi

I have recently been seeing this with Chrome as well.
The symptom is that the page is cut off in the browser, but interestingly enough the full page shows up in the developer tools network tab:

Refreshing the page always seems to help.
This happens on two separate instances of debiman that use different static file serving stacks (one using Apache, the other one using nginx behind cloudflare), so I think the problem is client-side, not server-side.
Perhaps we’re triggering some sort of CSS bug? Not really sure how to go about this, or how to reliably reproduce it even :-/

Let’s see if perhaps https://salsa.debian.org/manpages-team/debian-assets/-/commit/6fd93402ad7b9acbb5a649be5b4570d2339d6b62 helps (removing JavaScript related to font loading). It’ll be a while before that will be live on manpages.debian.org.


Replace all instances of rawTransaction: Uint8Array with an actual RawTransaction

Description
After working with the wallet adapter more, I realized my initial motivation for suggesting @0xmaayan use a Uint8Array was misled and premature, because we can perform this conversion at the wallet adapter layer if necessary.
Until it's necessary, it is ideally more strongly typed as an actual RawTransaction instance.

This saves us an unnecessary round of serialization/deserialization in deriveTransaction
Increases clarity, because it's confusing knowing what the Uint8Array bytes really are
We can very easily just perform this conversion at the wallet adapter layer when it's necessary

After trying to build transactions/come up with a more intuitive flow, I realized how premature converting to bcs bytes was, and that's the motivation for this PR.
Test Plan
pnpm jest

I remember another reason to use bcs and not json is when servers need/want to send the transaction to another server, json is pretty sucks. Is that not a concern anymore?

@0xmaayan correct, it's not a concern because you can always do the conversion right before transport across some network layer for yourself.
I suppose we could come up with a different type for it, but I still kind of think it was premature, since most cases where you're handling this, the wallet adapter can/will just do it for you

@0xmaayan correct, it's not a concern because you can always do the conversion right before transport across some network layer for yourself.
I suppose we could come up with a different type for it, but I still kind of think it was premature, since most cases where you're handling this, the wallet adapter can/will just do it for you

so the idea it to rely on the adapter to do it for you? what if you use the sdk in a node project, and therefore dont use the adapter? what if the dapp doesnt use the adapter?

@0xmaayan hmm I suppose they could run into issues- but they can ultimately just serialize it themselves if they need to use a custom transmission function to a network. My initial thoughts for why we should do it were that the wallet adapter would run into issues if the user sends a RawTransaction, but we can always just serialize it before that. Tbh I'll put this in draft and we should discuss it/the flow again- sorry for prematurely putting this out there

Please update CHANGELOG


The following dependencies do not satisfy the required version: project ':google_api_availability'

🔙 Regression

After recent update of Flutter plugin
The following dependencies do not satisfy the required version:
project ':google_api_availability' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71
Know issue
https://stackoverflow.com/questions/54693595/flutter-the-android-gradle-plugin-supports-only-kotlin-gradle-plugin-version-1
Old (and correct) behaviour
Android builds
Current behaviour
Android build breaks -
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.3.0 and higher.
The following dependencies do not satisfy the required version:
project ':google_api_availability' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71
Reproduction steps
Android build
Configuration
Version: 3.0.0
Platform:

[ ] :iphone: iOS
[ x] :robot: Android

A couple of temporary work rounds have been described
Fixing the versions until google_api_availability is updated.
geolocator: 3.0.0               # AndroidX - Breaking!
google_api_availability: 1.0.6  # Geolocator Dependency.
meta: 1.1.6                     # Geolocator Dependency.
permission_handler: 2.2.0       # Geolocator & Meta Dependency.
Is what I did, and that worked with the latest 3.3.1 of the Android Gradle build plugin
or using and older version of Android Gradle build plugin - But for me at least this just broke other things!
Info here may be duplicated with other issues, please use revise as is helpful.

Thank you, this worked for me.

A couple of temporary work rounds have been described
Fixing the versions until google_api_availability is updated.
geolocator: 3.0.0 # AndroidX - Breaking!
google_api_availability: 1.0.6 # Geolocator Dependency.
meta: 1.1.6 # Geolocator Dependency.
permission_handler: 2.2.0 # Geolocator & Meta Dependency.
Is what I did, and that worked with the latest 3.3.1 of the Android Gradle build plugin
or using and older version of Android Gradle build plugin - But for me at least this just broke other things!
Info here may be duplicated with other issues, please use revise as is helpful.

jump in this file . ../flutter/.pub-cache/hosted/pub.dartlang.org/google_api_availability-1.0.6+1/android/build.gradle
replace ext.kotlin_version = '1.3.21'

@nicolasjon answer didnt work for me :(

@nicolasjon this should be solved in version 3.0.1 where we removed all dependecies on Kotlin (using version 2.0.0 of the google_api_availability plugin and version 3.0.0 of the permission_handler)

Works. Thanks! Nicolas


resolve the crash dependency conflict

if installed now, the API will not serve but crash.
This pull resolves the crash due to dependency conflict between hug, waitress and falcon.

thanks!


feat: Added SEO referencing

As per instruction on Google search Engine, this PR adds the google analytics tracker for the referencing to be working properly.

Hey there,
good idea, because why having a site if it is not referenced (at least on google :) )
Juste to be sure, are you using pyronear gmail as the "owner" of  the website ? Then did the ownership has been validated/verified ?

Hey there,
Thanks, this PR is a good idea, because why having a site if it is not referenced (at least on google :) )
Just to be sure, are you using pyronear gmail as the "owner" of the website ? Then did the ownership has been validated/verified ?

Not entirely sure I understood the question, but the Google tag was generated from the Google Analytics account of<EMAIL_ADDRESS>Once the PR is deployed, we'll be able to reference it on Google Search Console (one of the method available)!


How to reset data?

Why was choose Singlton pattern?
Its absolutly not useable in queue jobs!
There is an accumulation of data, how to reset data in gamp object?

I agree with unlike777

I've just released v1.6, switched to binding. Should resolve this issue.

Facades are singleton by design, this won't fix the "reset data" problem.
I think you should add a note in documentation regarding this issue.
If you need to reset data, you should resolve another instance of Analytics class.


Unable to run the unit tests of Jolokia

I might be missing something but Jolokia's unit tests are kinda hard to run successfully. Even on travis-ci, they seem to be failing for JDK8 configuration.
But locally using jdk8, the tests fail in all weird places depending on where I run them.

I think presence of multiple network interfaces causes some problems for tests. On my laptop I have 5 network interfaces. Apart from lo, eth0, wlan0  I also have docker0 and tun0.
I could get tests to pass in a vagrant image though.

Unit test on master should run and do locally.
On Travis there is indeed an issue with JDK8 where I don't know why the Java JVM itself heap dumps. I also don't have a chance to reproduce this. So I'm fine as long as it runs with OracleJDK 8
For branch 2.0 there as unit test failures on Travis which are about an already bound socket (and which I don't have locally). I will fix that.
@gnufied could you send me the error log you got ? The Jolokia unit tests are, well, not pure unit trests as you already remarked but a bit of a mixture of unit and system/integration tests. However, as long as they still run quick enough I typically don't care much about the difference ;-). However, the danger is higher that the tests fail in different environments. Especially the multicast discovery feature, which is part of the unit tests currently, can fail in certain circumstances, however I would be curious what happened (becasue thats also good to know what can happen somewhere else).

Here is how they are failing for me:
simple(org.jolokia.discovery.DiscoveryMulticastResponderTest)  Time elapsed: 2.022 sec  <<< FAILURE!
java.lang.AssertionError: expected [true] but found [false]
        at org.jolokia.discovery.DiscoveryMulticastResponderTest.simple(DiscoveryMulticastResponderTest.java:36)

simple(org.jolokia.discovery.MulticastSocketListenerThreadTest)  Time elapsed: 1.008 sec  <<< FAILURE!
java.lang.AssertionError: Exactly one in message with the send id should have been received expected [1] but found [0]
        at org.jolokia.discovery.MulticastSocketListenerThreadTest.simple(MulticastSocketListenerThreadTest.java:89)

initWithAgentDiscoveryAndGivenUrl(org.jolokia.http.AgentServletTest)  Time elapsed: 1.505 sec  <<< FAILURE!
java.lang.AssertionError: No agent found
        at org.jolokia.http.AgentServletTest.initWithAgentDiscoveryAndGivenUrl(AgentServletTest.java:154)

initWithAgentDiscoveryAndUrlCreationAfterGet(org.jolokia.http.AgentServletTest)  Time elapsed: 1.016 sec  <<< FAILURE!
java.lang.AssertionError: expected [true] but found [false]
        at org.jolokia.http.AgentServletTest.initWithAgentDiscoveryAndUrlCreationAfterGet(AgentServletTest.java:204)

initWithAgentDiscoveryAndUrlLookup(org.jolokia.http.AgentServletTest)  Time elapsed: 1.011 sec  <<< FAILURE!
java.lang.AssertionError: expected [true] but found [false]
        at org.jolokia.http.AgentServletTest.initWithAgentDiscoveryAndUrlLookup(AgentServletTest.java:167)

Thanks !
Form the log output could you paste the content between the two "=====================================" lines + the output after "Network interfaces" (if its not confidential, of course). For me interesting especially are the interfaces listed with 'mc: true' which are supposed to support multicasts. You must have at least one interface indicating that it supports multicast, otherwise the test wouldn't have run.

@gnufied I just tuned the test so that it should work even when no multicast request can be send out over any interface.
Does it work now for you ?

I just pulled and it still doesn't. I have updated the gist (btw - github doesn't seem to be create notifications on secret gists )

damned, it seems that although you could send udp packets to a multicast group address, it never receive a packet on this interface.
As a last resort, in the test I start a second try with a much increased timeout (30s). If this doesn't help, I'm going to make this test optional as its too fragile and I have no idea how to fix it otherwise.
Could you please give it a last try ?

I did try again by pulling latest code and it fails for me. I have updated the gist I shared with you.
I am going to try and look into this more closely later this week as well and see if I can find something unique about my local setup.

All tests work now both in main and 2.0 branches.
The problems with DiscoveryMulticastResponderTest are related to UDP multicast firewall settings.
For example on Fedora I had to run:
# nft add rule inet firewalld filter_IN_FedoraServer_allow ip daddr <IP_ADDRESS> accept


Linux Development resources Added

PR Title samples:

Linux development - Tools & Techniques

Closes #346
Description:

I have added the "Linux Development" resources . There are some contents about the Roadmaps, Tools, Techniques , Books, Youtube Channels & GitHub repository at the Resource page.

Resource URL:

Here is some resources

https://roadmap.sh/linux
https://www.gnu.org/software/gcc/
https://github.com/
https://www.kernel.org/
https://www.codecademy.com/learn/learn-the-command-line
https://github.com/dastergon/awesome-sre

Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[x] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[x] I followed the prescribed PR title template. (Check this if you're adding a resource)

@jfmartinz  Sir please help me on this problem

@jfmartinz i have created new pr , check this time my pr

@jfmartinz Sir Please review once

@jfmartinz i cannot understand what the problem is... Can you please explain this to me..... And please give a reply about this pr if i have to send a new one or not... But give a reply it is over than 2 days since I have sent this pr

@jfmartinz Sir What's going wrong with my pr


KOGITO-1320: Prevent editors from opening with an unsupported file

Task
https://issues.redhat.com/browse/KOGITO-1320
Description
Upload a file with the  Choose a local file button

Only the supported files will be showed by default (.dmn, .bpmn and .bpmn2).
If an unsupported file is selected, an error message will be displayed for 2 seconds and will be hidden in 0.3 seconds.

Upload a file with the Drag and Drop zone

Only the supported files will be accepted (.dmn, .bpmn and .bpmn2).
If an unsupported file is dropped on the drag and drop zone, the color will be changed to red for 2 seconds and changed back to the original color in 0.3 seconds.

Demo

Considerations

It was observed that on the Chrome browser sometimes a bug occurs on the React.DragEvent when the File is undefined.

Thanks, @tiagobento - yes, the Patternfly color --pf-global--danger-color--100 should be used here for the red. Patternfly doesn't recommend the pinkish background color, for one it reduces constrast below the guideline for a11y. I think the red border and message is enough, but if you disagree then I'd go with an Empty State component with the warning-triangle icon inside the upload box (the bullseye layout would center it).
I also see a padding in the new CSS; we should use either the utility classes for spacing https://www.patternfly.org/v4/documentation/core/utilities/spacing or the global spacer variables --pf-global--spacer--xs etc.
We also have a couple of global variables for animation - --pf-global--TimingFunction and --pf-global--Transition which might be useful here but I have not looked specifically at those to see.
Nice feature, @ljmotta thanks for adding it!

Thanks for the reply! @srambach
I changed the colors and the padding as you asked but the animation is a little more complicated. It would be necessary to change a couple of things on the code and it would make it more complex. Hope that is ok :smile:
Could you please review it again? @tiagobento @srambach

@tiagobento @ljmotta  Thank you for switching to use PF variables. When I had a chance to look at this a little more, I am concerned about the way the A11y (doesn't) work for this method of reporting the error. There is no indication to a screen reader that a problem has occurred. In addition, the error message is transient, which can also be an a11y problem. The way it is done here looks nice, but I think that the right thing to do would be to use an Alert.
https://www.patternfly.org/v4/design-guidelines/usage-and-behavior/alerts-and-notifications#main-content
If time is of the essence, I am fine with approving this so that there is error handling in place, but I believe there should be a Jira created to work with design to replace this with an appropriate alert.
@kywalker-rh

@srambach Ok, that makes sense. So are we good to merge that as is and create a separate JIRA to improve a11y later?

I tested in Firefox and Chrome. Works well.
One small issue, in chrome, I wasn't able to select only supported files (.dmn,.bpmn,*bpmn2) like in firefox when I used 'Choose local file'

@srambach I did't thought about the a11y at all, sorry about that, I'm going to pay more attention to that. Already create the JIRA (https://issues.redhat.com/browse/KOGITO-1517). Thanks for the review.
@domhanak That's a little odd, here's what I did (tested with Chrome 80.0.3987.132):

Please tell me if I'm missing something.

@ljmotta Yea I wrote in in a confusing way. Disregard. :) All works well.


DCOS_OSS-4579: [1.12] accept floating points in FrameworkConfigurationForm

Testing
Use a browser other than Chrome.. With your browser and OS in locale that uses "," as a decimal separator  (de-DE, ru, fr-FR, etc...).

Go to Catalog and add a new service
Go to the "Nodes" tab
Change CPUs to another floating point

The field should not be considered invalid
Trade-offs

Setting novalidate on this form makes us miss out on browser validation (e.g.: people can put letters into an input[type="number"])
We're setting novalidate on this form, but there may be other forms where we have this issue that we just haven't caught yet

Dependencies
None
Screenshots
Before:

After:

Probably not related to this PR, but the validation looks a bit weird.

@GeorgiSTodorov,  we don't - feel free to create one :)

https://jira.mesosphere.com/browse/DCOS-46311

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


Add a mechanism for discovering popular streams other people are using

For people joining existing teams, it'd be great to have an easy mechanism for users to discover what the popular streams in a realm are (either by number of subscibers or by traffic) to help inform their decision on which streams to join.  This seems like something that if done well could be an important new part of the onboarding flow for new users.
This could be done as adding new sorting controls on the /#subscriptions page, or via some other mechanism (e.g. a new /discover page that is focused on this purpose).

This ties into something that I've always kind of wanted, which is a "Recent History" view that shows recent streams/topics and some kind of count.  Basically, I want a bird's-eye view of what's happened lately that is intermingled like the home view, but with no messages and more tabular.
I'm also up for the other extreme--let the system figure out what popular streams you seem to be ignoring, and have something like the Welcome Bot send you a "psst...you're missing out" message.

This is mostly addressed by us now having subscriber counts, and #3458 proposes making "Manage Streams" have the ability to sort by subscriber count.  I do think there are more advanced things we can do here, but I'm confident they will come up in future discussions, so I'm closing this for now.

I'll open a new ticket for the "/discover" idea.


Wild Illager Summon appears misnamed

Describe the bug
Wild Illager ritual still listed as Possessed Evoker
Expected behavior
Ritual is now part of the Wild series of summons, so it would make more sense for the ritual to be renamed Summon Wild Illager Raid.
Screenshots

Spawn egg probably needs a rename too

The Dictionary of Spirits refers to it by yet a third name altogether.

System (please complete the following information):

Occultism Version:

OS: [e.g. Windows] Windows

Minecraft Version: 1.21.1

Modpack Link and Version, or list of mods: Enigmatica 10

Found a fourth name. Should just be consistent, whatever it's named.

May be worth having a second look at all of these rituals actually. I'm seeing a lot of inconsistencies in names between the book and recipe viewer.
Considering these are thematically no longer Possession rituals, it may also make sense to redo the possession chapter in the book to move these into their own category.

"Wild" possessions is just a group possession
(and horde if all mobs are the same)

With the way categories are laid out in the book, these appear to have been fully divorced from 'possession' as those all use the yellow chalk line of pentacles and the book no longer refers to them as possession, but contact with wild spirits.
They're wild. That's fine. It's a new category and effectively quite different from Possession. The names should consistently reflect that. It will more accurately signal to the player that these are Wild summons and different from possession.

Closed in #1237 unless I am much mistaken


Make SSL_CTX_set_cert_store increase the reference counter

I'm not sure if we covered something like this in our discussion about the style guide. But I think since it's freeing the old one, it should increase reference counter for the new one.
This came up in https://github.com/nodejs/node/pull/8491#pullrequestreview-5976947

This has been the behavior since SSLeay 0.8.1b, so it's probably not reasonable to change at this point. Calling free while not taking a reference is consistent with all the other set0 functions in the library. If you want one that takes a reference, probably call it  SSL_CTX_set1_cert_store.
Though given that this function already was split into SSL_CTX_set1_verify_cert_store and SSL_CTX_set1_chain_cert_store which two have both set0 and set1 variants, perhaps leaving it alone and deprecating the combined one in favor of the split ones makes more sense?

I assume this was a new function, but you're right it's not. And the new functions already do the right thing it seems, so maybe I should just use those instead.


Exchange providers for US West 3?

Are there any exchange providers for US West 3? This document doesn't list it. It does list Phoenix, but only US Gov Arizona, and not US West 3.

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

ID: a0acd798-2e5a-2a7e-bb14-7424576a8e38
Version Independent ID: 841f2ce1-b101-0fbf-e33d-4e56a6244ed9
Content: Locations and connectivity providers for Azure ExpressRoute
Content Source: articles/expressroute/expressroute-locations-providers.md
Service: expressroute
GitHub Login: @duongau
Microsoft Alias: duau

@liutang
Thanks for your feedback! We will investigate and update as appropriate.

@liutang
I'd recommend working closer with our support team via an Azure support request. Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.Teams Q&A forum for technical questions about the configuration and administration of Microsoft Teams on Windows.Microsoft Teams Community forum


Added pod labels to RevisionTemplate specs and retooled configuration…

… creation logic

This change is

/assign @yuzisun

@ellis-bigelow would you update https://github.com/kubeflow/kfserving/blob/master/docs/DEVELOPER_GUIDE.md#smoke-test-after-deployment here to use your new label ?

/lgtm

/approve


Create setting to ignore certain locations in Visits map

There should be a setting to add locations with the following attributes:

Name
Lat
Lon

These locations will then be ignored in the Visits plot.
Potentially, create a filter to dynamically select which locations should be ignored within the visits plot filters.

Why do you want to ignore some visits?
I don't mind knowing that I went to work on a particular day, for example.

Good question @sinizterguy, over time, specially during some periods, you will see that your home location trumps all other locations just because of the sheer amount you spend there.
This makes viewing other locations difficult in the map as they plot relative to the time you spent there compared to all other locations.
I.e.: if you spend 3 days at home because you were sick and then have other visits like 1 hour somewhere, that visit will barely show up in the map.
I want to add an option for users to add a set of coordinates and radius to ignore. I.e.: your home or work location with a radius of 200 meters.
This way you can, temporarily or permanently, clearly see all the places you visited outside of your home, work or other locations you spend the majority of time at.


Versioning conflicts

Good evening!
I tried to install your tool, but ansible-galaxy threw the following error:
 [WARNING]: - mimacom.bamboo-agent was NOT installed successfully: Unable to compare role versions (v0.0.1, 0.0.1) to determine the most recent version due to incompatible version formats. Please contact the role author to resolve versioning conflicts, or specify an explicit role version to install.
Maybe you can fix the version tag conflict :)

ansible-galaxy 2.5.2
Best regards,
Mtze

Hi mtze. Thank you, it's resolved now.


dva-core v1.2.0 introduces a Breaking Change

Hello,
Mutating the passed action by injecting __dva_reject and __dva_resolve functions is a big deal.. It breaks code that relies on the action object not being mutated
In this particular case, it specially breaks plain object serialization (for ex: it breaks code that used WebWorker postMessage calls) since functions are not serializable. Also breaks any tests that make assertions on the action object by ways of value equality.

Isn't better to have an actions map that keeps state outside the action descriptor object itself?
(we could use a ES6 Map() and use the object references as key I suppose)

I really feel the action obj should be treated as immutable by dva... Feel free to chat here if want any help on this.
best regards

Is there any example to reproduce the problem?

const action = {type: "MyAction", payload: {}}
const before = JSON.stringify(action)

dispatch(action)
const after = JSON.stringify(action)

expect(before).toEqual(after)

my current workaround was to copy the action object with only the keys type and action so I do not get errors from the new keys that were injected.


Very slow JSON serialization and deserialization and blocking event loop

What is the current behavior?
PrerecordedResponse.from_json(result) (link to code) is very slow, especially for larger inputs. This is due to the Dataclasses JSON library, where they are already aware of that performance issue but haven’t addressed it since 2020. In addition to .from_json(), the .to_dict() operation is also very slow, which one would use if they want to parse the output from the Deepgram SDK into their own Pydantic model.
In our case, for recordings lasting around 1 hour:
source = {"url": signed_url}
options = rerecordedOptions(
        model="nova-2-general",
        diarize=True,
        utterances=True,
        paragraphs=True)
deepgram.listen.rest.v("1").transcribe_url(source, options=options)

The .from_json() takes over 10 seconds. Pydantic parsing takes ~30ms.
For a 7-minute recording, the .from_json() operation took ~1.7 seconds, while Pydantic parsing took ~5ms.
This issue also affects the asynchronous version, where the problem is even more significant as it blocks the event loop for a long time.
Expected behavior
JSON serialization and deserialization shouldn't take that long, and CPU-heavy operations should definitely not block the event loop. Please consider using Pydantic or raw dataclasses.

Adding __slots__ to the dataclasses may help -- this is worth a quick try. I have not tested, and I don't know if dataclasses actually support __slots__, but adding the class variable can result in dramatic speed improvements.
Overall, my opinion is that dataclasses begin to break down once the scope of their usage extends past the immediate value proposition of dataclasses, and a different implementation tends to work better. Pydantic tends to be used for input validation, which isn't a critically important feature within this SDK because responses do not need to be validated. That said, I'm a big fan of pydantic in general. But choosing a different class implementation may give us the speed and flexibility wins we're looking for. That said, moving away from dataclasses will be a major breaking change.


fix dot z ref (auth/dependencies section corrupt in prev update)

There was an alteration here https://github.com/KxSystems/docs/pull/285#discussion_r1624526717 for a pull request, which is maybe some sort of copy/paste problem ,but the change has removed various content from the original proposed change.
Its also caused the definition of .z.b in https://code.kx.com/q/ref/dotz/ to disappear.
Fixed in provided change.

changes made, thanks


Added more customization options

I've added several new configuration options for the plugin (basically the useful ones that are supported by bean-format). Also, tiny bit of clean-up of tabs/spaces in package.json

Thanks very much, I have publish a new version(1.2.0) to marketplace.


Istioctl manifest falsely parsing k8s resource separtor

Running istioctl from master head. When running istioctl manifest diff against the following resource, istioctl complains about missing object Kind:
---
apiVersion: v1
data:
  custom-resources.yaml: |-
    # These policy and destination rules effectively enable mTLS for all services in the mesh. For now,
    # they are added to Istio installation yaml for backward compatible. In future, they should be in
    # a separated yaml file so that customer can enable mTLS independent from installation.

    # Authentication policy to enable mutual TLS for all services (that have sidecar) in the mesh.
    apiVersion: "authentication.istio.io/v1alpha1"
    kind: "MeshPolicy"
    metadata:
      name: "default"
      labels:
        app: security
        chart: security
        heritage: Tiller
        release: istio
    spec:
      peers:
      - mtls: {}
    ---
    # Corresponding destination rule to configure client side to use mutual TLS when talking to
    # any service (host) in the mesh.
    apiVersion: networking.istio.io/v1alpha3
    kind: DestinationRule
    metadata:
      name: "default"
      namespace: istio-system
      labels:
        app: security
        chart: security
        heritage: Tiller
        release: istio
    spec:
      host: "*.local"
      trafficPolicy:
        tls:
          mode: ISTIO_MUTUAL
    ---
    # Destination rule to disable (m)TLS when talking to API server, as API server doesn't have sidecar.
    # Customer should add similar destination rules for other services that don't have sidecar.
    apiVersion: networking.istio.io/v1alpha3
    kind: DestinationRule
    metadata:
      name: "api-server"
      namespace: istio-system
      labels:
        app: security
        chart: security
        heritage: Tiller
        release: istio
    spec:
      host: "kubernetes.default.svc.cluster.local"
      trafficPolicy:
        tls:
          mode: DISABLE
  run.sh: |-
    #!/bin/sh

    set -x

    if [ "$#" -ne "1" ]; then
        echo "first argument should be path to custom resource yaml"
        exit 1
    fi

    pathToResourceYAML=${1}

    kubectl get validatingwebhookconfiguration istio-galley 2>/dev/null
    if [ "$?" -eq 0 ]; then
        echo "istio-galley validatingwebhookconfiguration found - waiting for istio-galley deployment to be ready"
        while true; do
            kubectl -n istio-system get deployment istio-galley 2>/dev/null
            if [ "$?" -eq 0 ]; then
                break
            fi
            sleep 1
        done
        kubectl -n istio-system rollout status deployment istio-galley
        if [ "$?" -ne 0 ]; then
            echo "istio-galley deployment rollout status check failed"
            exit 1
        fi
        echo "istio-galley deployment ready for configuration validation"
    fi
    sleep 5
    kubectl apply -f ${pathToResourceYAML}
kind: ConfigMap
metadata:
  labels:
    addonmanager.kubernetes.io/mode: Reconcile
    app: security
    chart: security
    heritage: Tiller
    istio: citadel
    k8s-app: istio
    release: istio
  name: istio-security-custom-resources
  namespace: istio-system
---

error logs from istioctl:
2020-02-29T00:19:13.045871Z	error	Failed to parse YAML to a k8s object: error decoding object: error unmarshaling JSON: while decoding JSON: Object 'Kind' is missing in '{"apiVersion":"v1","data":{"custom-resources.yaml":"# These policy and destination rules effectively enable mTLS for all services in the mesh. For now,\n# they are added to Istio installation yaml for backward compatible. In future, they should be in\n# a separated yaml file so that customer can enable mTLS independent from installation.\n\n# Authentication policy to enable mutual TLS for all services (that have sidecar) in the mesh.\napiVersion: \"authentication.istio.io/v1alpha1\"\nkind: \"MeshPolicy\"\nmetadata:\n  name: \"default\"\n  labels:\n    app: security\n    chart: security\n    heritage: Tiller\n    release: istio\nspec:\n  peers:\n  - mtls: {}"}}'

looks like istioctl falsely separted resource with the separator in the embeded configmap data.
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

The reason is that we have multiple yaml separator --- embedded inside the data section, then when it parse to k8s objects(during which we separate manifests by the yaml separator), it would split the one object to multiple objects mistakenly.

it should check --- is at the start of the line I think?

Actually that’s what we did previously, but later we found out some templates were malformed with separators have leading spaces, we updated the logic to trim first and check. I remember you updated the temples for that, so we can change it back


Add Shupapan

Add a new Cask. :)

Shupapan v.5.9.7

Seeing as url doesn’t point to a specific version, this cask should use version :latest and sha256 :no_check. See an example.

I changed my codes with your example :)

I added a comment "# todo: change license and remove this comment; ':unknown' is a machine-generated placeholder comment "


RTEMS Serial Transport support

Following #283 and #287, made a rapid patch for serial transport support.
Tested with PingPong app as usual.
Details about how to test this transport using QEMU will be added in https://github.com/micro-ROS/micro_ros_setup/issues/397 tracker issue.
Note: as for TCP/UDP, this patch implements a variant of POSIX transports using select() call instead of poll() call, which is not officially supported by RTEMS 5 core. Even if RTEMS-libBSD adds some poll() support, many boards (BSPs) would be potentially excluded from microROS usage due to inability to build libBSD for them.

Build status:

Linux
Windows

Build status:

Linux
Windows

Can you rebase this onto develop and make this PR target develop?

@pablogs9 done. Poll me if I have messed up something in the process.

Build status:

Linux
Windows


Add LICENSE

To update that repo, I wipe out the previous files, and replace them by the result of the make build-demo in react-admin.
So next time I'll update the source, the license will go... I suggest you add the LICENSE to the demo example instead, so that it ends here.

My bad, I update the GH pages branch, you PRed against the master branch


Track description

After start or after stop allow to add track description and show it in track list

Description replaced with "activity type"


fix: Kotlin linter errors

This fixes the following Kotlin linter errors in the Android build:

Current master branch Android build fails due to the linter errors described above inside PiwikProSdkModule.kt


with angular2 project it throws a (SystemJS) expected expression, got '<'

I have an issue when trying to use the library in my code.
Basically when I access the first page in the browser I get a (SystemJS) expected expression, got '<'
I think it mast be an incorrect way I declare things in the systemjs.conf.js.
If you could help me in the right direction it would be great.
Here goes part of the code.
thanks
systemjs.conf.js
    map['typedjson/js'] = 'node_modules/typedjson/js';
    packages['typeddjson/js'] = { main: 'index.js', defaultExtension: 'js' };

person.ts
import {JsonObject, JsonMember} from 'typedjson/js';

@JsonObject
export class Person {
    @JsonMember
    name:string;
    @JsonMember
    surname:string;
    @JsonMember
    age:Number;

    public getFullDescription() {
        return this.name + " " + this.surname + " "  + this.age;
    }
}

MapUtilsTest
import {TypedJSON} from 'typedjson/js';
import {Person} from "./person";

export class MapUtilsTest {
    public static testPerson(): Person {
        let json = PersonDataFixtures.getPerson();
        let jsonStr = JSON.stringify(json);
        const person = TypedJSON.parse(jsonStr, Person);

        this.print('person');
        this.print(jsonStr);
        this.print(person.getFullDescription());

        return person;
    }

Hm. Let's see. The JS distribution of TypedJSON is compiled with RequireJS/CommonJS module syntax. Can you please try compiling typed-json.ts along with your project?

hi Jhon,
thanks for your quick response.
yes your suggestion did work. :)
I followed your suggestion but I guess the way I did it is not the correct one and maybe you could tell me.
I copied the typed-json.ts out of the node_modules/typed-json and I copied manually into my project as you suggested to transpile it along with my project and I referenced that typed-json.ts file and not the node_modules/typed-json.
I have two questions:

is there a better way instead of me having to copy the  typed-json.ts manually into my app.
Maybe via the systemjs.conf.js.
But in that case how should my systemjs.conf.js map and packages look like and how should I import it in my code?
why was it failing ? because you had it compiled using RequireJS/CommonJS syntax ? to make it work should it be transpiled to angular2 ?

thanks for your help

Initially, TypedJSON was meant to be compiled with other source files, and initially only the source was uploaded -- which is pretty much just a proof of concept before I'll go through it in depth, sorting out issues and streamlining the API a bit. The npm module was added later by a generous contributor, and I didn't extensively test its workings yet.
The idea is that the compiled .js of TypedJSON is virtually useless without TypeScript, hence why it was meant as to be included with other files in a project.

Closing  - the current version package should support most modern build systems.


Screen protected from any Screenshot and record screen tools

hello dear @neckaros  thank for this good package
do you have any plan to add support protect application from screenshot and screen recorder ?
thanks you so much

You are welcome! On iOS? I think it already blocks on Android but it's been long since I did this!
Need to check how it can be done on iOS

thank you so much
Currently, this package does this
but this not support windows and macos platform
That's why I left a message for you 🥲

@neckaros


Elevation data

Any plans to in cooperate more elevation data sources?
It look like you only have the SRTM from the way it cuts of at 60N.
There are dem data available for Sweden under ccby4, see  dataset "GSD-Höjddata, grid 50+"

Thank you for the report, we will check this out!

Topographical data (from which I think you can generate the elevation information?) for Finland is also available as open data. See Topographic data and how to acquire it from the National Land Survey of Finland, if you are interested.

We are closing this feature as we currently don't have the resources to fulfill it.