feat: luarocks support

nvim-web-devicons is a common dependency across Neovim plugins. Using luarocks may alleviate the need for users to specify their plugins' dependencies in their plugin manager (e.g., vim-plug or packer). See also this blog post for details.
This PR adds a release workflow that publishes this plugin to LuaRocks whenever a tag is pushed,
as well as a rockspec that can be used to release to the dev channel.
For the release workflow to work, someone with a LuaRocks account will have to add their API key to this repo's secrets.
Note that I have added a shield to the readme that assumes the existence of an nvim-tree LuaRocks account.

@alex-courtis @gegoune I've bumped the version of the action (it now auto-detects plugin directories) and removed the rockspec, since it's a redundant second source of truth.
What's the status on this from your side?

I guess missing bit still is

For the release workflow to work, someone with a LuaRocks account will have to add their API key to this repo's secrets.

@alex-courtis Have you got an account there?

I guess missing bit still is

For the release workflow to work, someone with a LuaRocks account will have to add their API key to this repo's secrets.

@alex-courtis Have you got an account there?

This rock is managed by an organisation of some sort: https://luarocks.org/modules/openresty/lua-cjsone
It's the same (except case) name of the github organisation.
We can't log in to github as an organisation and I can't register github at luarocks as anything but myself.
Email is required for non-github registration.
Possibilities:

create an nvim-tree github user
create a luarocks user
use a personal github account

1 seems manageable, although we'd need a few email addresses for password resets etc.

@alex-courtis I don't think it's possible to create a github user of the same name as an existing organisation.
@teto how did you do this with the neovim and nvim-treesitter accounts on LuaRocks? I guess it was a non-github registration?

so IIRC I've created a "neovim" user and then I've generated several LUAROCKS_API_KEY, one for neovim and one for nvim-treesitter

so IIRC I've created a "neovim" user and then I've generated several LUAROCKS_API_KEY, one for neovim and one for nvim-treesitter

That sounds reasonable. Is this the neovim user? https://luarocks.org/modules/neovim
How do you manage the user and its API keys? How are that user's credentials maintained? Considering a  "5 years time" situation when, say, a new key is needed.

the neovim user is meant to provide API keys for "official" packages, ie., maintained by the neovim core team (or close, like nvim-treesitter's team). In your case you would need to register yourself on luarocks.org (github account works) and create the LUAROCKS_API_KEY yourself.

This is not practical until we get a deployment strategy that is not tied to an individual.
Please reopen if you have any further ideas.

@alex-courtis So creating a nvim-tree luarocks.org account that is not linked to a GitHub account would not be an option for you?
Asking, because we have been working on rocks.nvim, which facilitates using luarocks to manage plugins.

@alex-courtis So creating a nvim-tree luarocks.org account that is not linked to a GitHub account would not be an option for you?
Asking, because we have been working on rocks.nvim, which facilitates using luarocks to manage plugins.

rocks.nvim looks incredible! Thank you for all your work.

Unfortunately we still have the problem of authentication in the far future: if the password is lost or the API key expires we cannot recover it via, say, email.
Is there prior art other nvim itself and nvim-treesitter? Is there another plugin we could emulate?

Pragmatic but fragile solution:

create a luarocks account for nvim-tree organisation with a username/password
an email must be provided<EMAIL_ADDRESS>works
record the credentials in a secure location inside the organisation
retrieve / generate API keys as needed via luarocks.org

In the event of credential loss or expiration luarocks.org would need to manually recreate or reactivate the account based on the claim of an organisation member.
@gegoune @Akmadan23 what are your thoughts?

Pragmatic but fragile solution:

create a luarocks account for nvim-tree organisation with a username/password
an email must be provided<EMAIL_ADDRESS>works
record the credentials in a secure location inside the organisation
retrieve / generate API keys as needed via luarocks.org

Seems a good idea... We could create an email account for the nvim-tree organization so that we can all access any important email from luarocks and even recover/recreate the password if we lost it.

An email and luarocks.org account nvim-tree has been created.
All details recorded in the private nvim-tree repository, including recoveries etc.

Done: manual run for v0.99 tag
https://luarocks.org/modules/nvim-tree/nvim-web-devicons
Unfortunately the label was not set; I've done it manually.

Done: pushed changes to master and manually ran for v0.99 tag

🚀🌘
Thanks for all the effort!

Unfortunately the label was not set; I've done it manually.

Yes, that's a shortcoming that needs to be fixed.

Is there anything else we need to do here

Looks all good to me.
I'll close this PR 😃


i18n(es): Updated contribute.mdx

What kind of changes does this PR include?

New or updated content
Translated content

Description

What does this PR change? Give us a brief description.
Updated contribute.mdx based on these commits 99fe714562b35720768a9e8d93185f2672b8bb32, 762eceb4e8b6d8f5382f0df788d28fa2f778718b & 38f70780398b3a59697d0be8a4eff013a234e8df

Listo, gracias @Waxer59  🚀


Color contrast

This PR increases color contrast in the Wagtail admin.
Chrome Lighthouse is used to check for accessibility errors on contrast. All contrast tests pass.
I used the design by @benenright

The individual commits might give additional insight in my work process.

Increase text on white contrast
Detach nav colors from color-grey vars gives fine grained color control
Simplify color functions and round numbers.
Remove color from privacy icon to make hover state more notable and design less complex
Stop style bleeding on listing. Make Privacy text label white
Increase breadcrumb contrast
Drop redundant color color-grey-1-1
Use color variable
Use color functions where there is a obvious relation between them

Current:

New:

Also note the changes to search input and privacy toggle and the body text.

Checks:

Do the tests still pass? YES
Does the code comply with the style guide? YES
For Python changes: N/A
For front-end changes. Tested browsers:

Mobile Safari | iOS Phone => iPhone 6s IOS 9 Safari, iPhone XS Max 12 Safari
Mobile Safari | iOS Tablet  => iPad Air2 IOS 8 Safari, iPad Air 12 Safari
Chrome | Android => Google Nexus 6 Chrome 63 && Samsung Galaxy S5 Android 4.4 Chrome
IE | Desktop | 11 => Win7 IE11
Chrome | Desktop | Last 2
MS Edge | Desktop => Windows 10 Edge 17
Firefox | Desktop => Windows 10 Firefox 61 and 62
Firefox ESR | Desktop | OSX 10.13.6 Firfox ESR 60
Safari | macOS => OSX 10.13.6 Safari 12

For new features: N/A

LGTM 👍
What I did to check:

Read all the comments in the PR and checked their related commits
Did a checkout of the PR and ran it locally, clicked around to see if nothing breaks
Was amazed by how much better I like the new contrast!

Great work!

@nmorduch Will you bee so kind to review the last three commits and approve? I think this PR is ready to be merged. I'd like your approval.

For the purpose of the release candidate I'll take @nmorduch's comment as permission to go ahead and merge :-) No problem to roll back for the final release if need be.


[MAP] Design - view

details in https://github.com/reach4help/reach4help/issues/416

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.
Belonging to Epics:

[EPIC] [@isma-osy] #413 - Prepare MAP for Go-To-Market / Marketing strategy

Moved to Icebox to be able to use zenhub as our clean unique project management tool

This was done a while ago too. The landing page is live on https://map.reach4help.org/!


Fix for issue #276

Changed the published_datetime field of
stix.exploit_target.vulnerability.Vulnerability
to be wrapped in a property descriptor, analogously to
discovered_datetime.  I hope this fixes python-stix issue #276.

Thanks @chisholm, looks good! Merging.


Update FirebaseMessaging Dependency in v2 Branch to Ensure Compatibility with Newer Firebase Versions

Description:
Hello Customer.io Team,
We are currently using the Customer.io Flutter plugin, which relies on the v2 branch of the Customer.io iOS SDK. However, the existing FirebaseMessaging dependency in v2 is outdated and causing compatibility issues when trying to build with the latest Firebase versions that include bug fixes for building themselves with Xcode 16
Since the Customer.io Flutter plugin has not yet been updated to use the v3 iOS SDK, we are still dependent on v2. In order to resolve these compatibility issues without waiting for the Flutter plugin to update to support the v3 iOS SDK, we need to update the Firebase dependency in the v2 branch of the iOS SDK.
Proposed Change:

Update the FirebaseMessaging dependency in the v2 branch to allow versions >= 8.7.0 and < 12. This will ensure compatibility with Xcode 16 and the latest Firebase versions, while maintaining support for projects using the v2.x SDK.

Impact:

This change will allow apps using the Customer.io Flutter plugin (which depends on the iOS SDK v2) to build and work with the latest Firebase versions, including compatibility with Xcode 16.

I have created a branch with the required urgent updates and will be submitting a pull request for your review. Thank you for your consideration!

Hi @AristideVB
Thank you again for sharing great detail and opening the PR. I want to let you know that we've released the fix on iOS SDK version 2.14.2 and is now available to be used in your flutter app.
I'll go ahead and close this issue but feel free to let us know if you face any issues.


Improve gcs driver

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "feature/improve-gcs-driver" git@github.com:whoshuu/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354131200
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

ptal @dmp42 @dmcgowan @caervs

Vetted by @davidswu
@caervs @cquon @robbkistler let's take a hard look at this

@dmp42 @davidswu rebased

LGTM


Fix bug of not outputting error message for new teammate

Other bugs fixed:

Output box
Losing focus

Codecov Report

Merging #277 (789d7ee) into master (af2a1b6) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #277      +/-   ##
============================================
- Coverage     69.60%   69.57%   -0.03%
- Complexity      929      930       +1
============================================
  Files           136      136
  Lines          3086     3090       +4
  Branches        416      417       +1
============================================
+ Hits           2148     2150       +2
- Misses          791      792       +1
- Partials        147      148       +1

Impacted Files
Coverage Δ
Complexity Δ

...ess/logic/commands/project/AddTeammateCommand.java
88.88% <0.00%> (-11.12%)
7.00 <0.00> (ø)

...ain/java/seedu/address/model/project/Deadline.java
73.33% <ø> (ø)
19.00 <0.00> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
86.50% <0.00%> (+1.58%)
44.00% <0.00%> (+1.00%)

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 af2a1b6...789d7ee. Read the comment docs.


Changes made by AJAX not reflected in page object

With driver wait set to false the changes in the HTML made by an AJAX call will never be seen by the Capybara object page, even though Capybara.default_wait_time is set. Make sure the driver waits that amount of time to see those changes.

This a patch for tag 0.4.0. I think this problem has been addressed in more recent versions.

Hmm, there are a number of problems with this pull request.
I'm not sure what the issue is you're trying to address, but whatever it is, you'll need to prove that it exists through specs.
 Note that Selenium overrides this method, here and that RackTest doesn't execute JavaScript.
In general I am very, very wary of pulling in bug fixes to old releases of Capybara. You should really upgrade to the latest version.
Capybara has a "no tests = no merge" policy, if you want you pull requests to be merged, they need to have tests.

Well, I am using Capybara with Selenium to do some integration test. Here is the code that was not working:
page.find("#edit_#{to_do.id}").click        # Does an AJAX request that changes the HTML
assert_equal("01/10/2010",page.find("#todo_due_date_#{to_do.id}")["value"])  # Finds an element introduced after the AJAX
In order to make it work I could either do
page.find("#edit_#{to_do.id}").click
sleep(1)
assert_equal("01/10/2010",page.find("#todo_due_date_#{to_do.id}")["value"])
or
class Capybara::Driver::Base
def wait?
true
end
end
This happens, as I said with version 0.4.0 (which I need to use because of legacy code), with the following configs:
Capybara.default_driver = :selenium
Capybara.default_selector = :css
Capybara.default_wait_time = 30
This patch solves my problem, but still I am going to try and do a test.

Ok, so I have been probing the code again, and that patch doesn't actually solve the problem, which brings me back to the begining, to using the sleep(). Don't understand why, though...


No request body requests break the following blocks

As a follow on to #74 I noticed that if you have a request with no body i.e

You end up with the following written to the file system:
/*
## Test
*/
/*
#### Test 1
*/
GET https://example.com/api/auth/logout HTTP/1.1
X-Apitoken: {{api_token}}
/*
#### Test 2
*/
###
{{@response
    global.api_token = JSON.parse(response.body).token;
}}
POST https://example.com/api/auth/login HTTP/1.1
Content-Type: application/json
{
    "email": "{{httpbook_email}}",
    "pwd": "{{httpbook_password}}"
}

Which after re-opening results in:

a workaround would be to add an empty line after X-Apitoken. ```
GET https://example.com/api/auth/logout HTTP/1.1
X-Apitoken: {{api_token}}
/*
Test 2
*/

Looks like an error in the vscode-httpyac parser. I'm trying to fix it. I'm a fan of blank lines between blocks, so I've had more errors that way.

Ya that fixes it but the minute it's saved by the editor it gets removed

The cause was the way I built the cells. The idea was to use the symbols of the Outline View and create the cells with them. Unfortunately, blank lines are not assigned to an outline and are therefore swallowed. I now iterate directly over the source lines and determine the associated outline to then be able to determine what type of cell is needed. Additionally, there was an error that the initial comment was interpreted as an extension of the url line. For this I have published an update of vscode-httpyac that fixes this (https://github.com/AnWeber/httpyac/commit/e25c04ba191eb64f8288b3b091ccf05e94a60919).


Update feedback popup

Why: Edwin asked to add a note to let people know they can use the feedback form to request adding an organization to the website.

Coverage remained the same at 87.228% when pulling 7f4e8aad9427968d03b9770f98bd6462e0c7cae2 on update-feedback into d843cd2f5cefcaae7963a22388c3ec1a11c7565c on master.


Tell user they have reached max projects

When user clicks to add project,
Display Informational modal tell user they have reached project limit of 10.
Questions:
The user can create new projects a few ways,

duplicate existing.
forking someone someone else project.
creating a new project.
saving default project (doesn't hit hard limit)

Currently we have a hard limit of 10 projects. Do we block all these creation methods if they already have 10 projects?
Where do we enforce this hard limit? Seems like it's a limit per web browser since we don't have accounts per username.

@sideninja @DylanTinianov the enforcement would happen client side correct?

Yes, but still have to have a backend validation.


feat(examples): bump turbo for svelte

Description
Update turbo in svelte example to pickup latest passthrough

Allow CI Workflow Run

[ ] approve CI run for commit: 644e6ab317c79fd99a7770c0b490716bf70e34e1

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer


Outdated Installation Instructions for Ubuntu 20.04 - 22.04 and Debian

Step 2 of the installation tab says to run the following commands (or similar) for Ubuntu/Debian:
sudo apt install apt-transport-https ca-certificates
wget -O /tmp/ondemand-release-web_3.1.0-focal_all.deb https://apt.osc.edu/ondemand/3.1/ondemand-release-web_3.1.0-focal_all.deb
sudo apt install /tmp/ondemand-release-web_3.1.0-focal_all.deb
sudo apt update

sudo apt install ondemand

However, the 3.1.0 versions of OOD are no longer present at that link, and these instructions should be update to refer to 3.1.1.

thanks, I'll take a look into it.


TextInput Ctrl+A shortcut

OS: Windows 8.1 64bit, Kivy: master
When I write something, everything is ok, but when I want to select all with Ctrl+A shortcut, TextInput somehow blocks itself and doesn't respond to anything but Backspace(<--) or Delete. If it's not a bug, how to fix my issue?
There's no specific code, only added TextInput as a widget in kv

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Can you try just running the showcase to see if you have the issue? Because I cannot reproduce it on Windows.

Can you check if you're on latest master by doing a pull? Because supposedly this has been fixed recently.

reopen if you still have the issue on latest master


Notebook execution and caching through cli

How would be go about incorporating these instructions to _config.yml ?

How's this? https://github.com/ExecutableBookProject/cli/pull/58


Option to put TabContainer tabs at bottom

part of: https://github.com/godotengine/godot-proposals/issues/1986
salvaged: https://github.com/godotengine/godot/pull/44420
Adds tabs_at_bottom to TabContainer.
Enabling will move the tab bar to the bottom.

and especially for the future, this should be an enum. Right now it would only contain TOP and BOTTOM.  TOP would be the default and this way we do not break compatibility (also when adding the last two directions LEFT and RIGHT at one point)

There is also a PR making it possible to add tabs to the left: #63873

Changed to use an enum TabPosition.
I used a theme variation for the bottom style TabBarBottom that is set when the position is set. TabContainer now sets its internal TabBar to use the same theme variation.
Editor theme variation (for testing):

Default theme variation:

fixed docs and don't override type variation if set

changed to use theme item for _bottom styles in tab container. I found a way to not have to modify TabBar at all using the theme overrides! That was a big concern I had with this method originally.
I am now looking into flipping the StyleBox automatically

Now automatically flips the tab's stylebox instead of using separate theme items.

Rebased and fixed merge conflicts.
Removed unnecessary line: tab_bar->set_theme_type_variation(get_theme_type_variation());

Probably not important, but there is a small difference between popup position in top and bottom tabs:

I think the bottom one looks better and the top one looks wrongly implemented (it uses icon height, not the actual header size).

Resolved issues.
I made the icon popup appear at the bottom or top of the menu icon. I don't just use the tab height in case it is large, the popup would look disconnected.

Thanks!


HWP-426 Create Splash Page

Description
This PR includes the following proposed change(s):

Overhaul of LoginPage.jsx
Login route now leads to keycloak login route

Requires
Add 'Requires Attention' label if appropriate.

[ ] Requires an update to the .env file
[ ] Requires adding a file listed in the .gitignore
[ ] Requires an opinion or answer to a question, see comments.
[ ] Requires other attention, see below.

If there are any requirements for the developer to make after this PR is merged please list them here.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
For pull requests that are not just a simple fix, please check that the branch works on your local machine.

[x] Tested by Zach.
[ ] Tested by Brandon.
[ ] Tested by Brady.

Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes
[ ] If attention is required by the developer such as updating the .env file, these requirements have been listed.

@ZachBourque What about when TOGGLE_KEYCLOAK_AUTH isn't set or is false?

@ZachBourque What about when TOGGLE_KEYCLOAK_AUTH isn't set or is false?

Because of the difficulty of getting env variables into the frontend, for the time being I put a DO NOT MERGE label on this PR and we will not merge it until it is ready.

Looks good but my 100% browser zoom messes up the display. We need to tweak the responsiveness.


V4 release is not compatible with jest tests

Description
V4 release changes bundling
Running jest unit tests - without transformations - with components importing this package now break with following error:
 ● Test suite failed to run

    Jest encountered an unexpected token

    Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.

    Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.

    By default "node_modules" folder is ignored by transformers.

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.
     • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/configuration
    For information about custom transformations, see:
    https://jestjs.io/docs/code-transformation

    Details:

    node_modules/@traefiklabs/faency/dist/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,jest){export { AccordionContent, AccordionItem, AccordionRoot, AccordionTrigger } from './components/Accordion/Accordion.js';
                                                                                ^^^^^^

    SyntaxError: Unexpected token 'export'

    > 1 | import { Button } from '@traefiklabs/faency'

Investigation
Jest is not able to handle es modules properly:

https://jestjs.io/docs/ecmascript-modules

Previously, we exported a cjs version of the package.
In an attempt to improve bundling and tree-shaking, it was removed.
Could we put back cjs packaging and choose the esm module by default and only the cjs one for unit testing?

This has been fixed in:

https://github.com/traefik/hub-ui/pull/586


In the SQL rewriter, the implementation of VARIANCE breaks privacy guarantees

This is a consequence of this other bug but should probably be tracked separately, in case the fix is applied earlier in the rewriting process.
The code to calculate VARIANCE in the SQL rewriter measures the square of each value, but nothing is done to indicate to that aggregation that the clamping bounds should also be squared. So if your initial clamping is [0,50], you'll (I assume) scale the noise by 50 instead of by 50*50=2500.

The noise is added before the value is squared, so I believe this is working as expected.  We will review and confirm.
Note that there is a sample of stochastic evaluator over variance at: https://github.com/opendifferentialprivacy/whitenoise-system/blob/master/evaluation/Differential Privacy Verification.ipynb

How can the noise be added before the value is squared? The variance formula is a sum of squares; so either noise is added to each individual element before squaring (which would be valid, but extremely noisy, similarly to local DP), either noise is added at the end (in which case the above bug happens).

Oh, right, I was thinking of avg_squared rather than avg_of_square.  For the sum of squares we indeed square the noise.
query = 'SELECT age, VAR(age) AS var_age FROM PUMS.PUMS'
private = PrivateReader(meta, reader)
sq, q = private.rewrite(query)
syms = sq.all_symbols()
assert syms[1][1].sensitivity() == 100 # age
assert syms[2][1].sensitivity() == 100 * 100 # var_age

The details of the VAR computation are documented here: https://github.com/opendifferentialprivacy/smartnoise-sdk/blob/master/papers/DP_SQL_budget.pdf


can't find referenced method 'void onUnhandledInputEvent in library class android.webkit.WebViewClient Proguard

I'm getting this error when i try to use proguard. I have put the code in proguard:
-keep class * extends android.webkit.WebChromeClient { *; }
-dontwarn im.delight.android.webview.**

But still i'm getting this warning.
Warning: im.delight.android.webview.AdvancedWebView$1: can't find referenced method 'void onUnhandledInputEvent(android.webkit.WebView,android.view.InputEvent)' in library class android.webkit.WebViewClient

Thanks!
Could that be related to https://github.com/delight-im/Android-AdvancedWebView/issues/83?
What Android version are you running this on?

I'm compiling with API level 24. But still i haven't solved this. Yes is related to https://github.com/delight-im/Android-AdvancedWebView/issues/83

Sorry for the problems!
Let's track it in the other (related) issue then, though.


go: Add input mapping option when importing Graph

I added a test, but I'm not able to run the test because it says
cannot find package "github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto"

Can you run go generate github.com/tensorflow/tensorflow/tensorflow/go/op and then run the test? (As mentioned in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/go/README.md)

It should be working now. PTAL.

@jhseu Can you please review this PR ? Thanks!

Thanks! I'm new to this so let me know if there is anything else required from me. Will this be included in v2.4?


what happens if we have a number of occurences that can not be represented with 3 digits ?

What happens if we have a number of divs or other elements > 999 ? The program goes from 0 to 999 as you can see 03d. Should we not start at 001 instead of 000 ?

public class TypeBasedVariableNameStrategy implements VariableNameStrategy {
    private final Map<String, AtomicLong> counters = new HashMap<>();

    @Override
    public String nextName(@NonNull String type) {
        // NOTE: issue#145 careful with custom element about casing and dash, not to translate in JavaScript identifiers
        var identifier = type;
        final var HAS_DASH = type.contains("-");
        final var IS_ROOT = "targetElement".equals(type);
        final var HAS_UPPER_CASE = !type.chars()
                .allMatch(character -> Character.toLowerCase(character) == character);

        if (!IS_ROOT) {
            identifier = HAS_UPPER_CASE ? type.toLowerCase() : identifier;
            identifier = HAS_DASH
                    ? type.replaceAll("-", "_").replaceAll("_+", "_") : identifier;

            if (HAS_DASH || HAS_UPPER_CASE) {
                identifier = "custom_%s".formatted(identifier);
            }
        }

        return format("%s_%03d", identifier, counters.computeIfAbsent(type, __ -> new AtomicLong()).getAndIncrement());
    }
}

https://github.com/osscameroon/js-generator/issues/240

Finally, nothing to worry about. If we have more than 3 digits, the number is represented as it is. This %03d formatting doesnt break the code.


Manually added Authorization header does funny things

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
So, when adding an 'Authorization' header, I need to fill it with a custom value. So not a 'Bearer'-token' but an 'AfasToken'-token. As you can see in the screenshot, Bruno assumes a Bearer token anyway. This way I cannot use Bruno to make requests to AFAS.
I assume the Authorization header is overwritten from the Collection. But I expect that when I set the same header on the request itself, it should overwrite the one from the collection.
Screenshots/Live demo link

You can try using a pre-request script to req.setHeader instead of using the form.
req.setHeader("transaction-id", nanoid());

You can try using a pre-request script to req.setHeader instead of using the form.
req.setHeader("transaction-id", nanoid());

That seems to be a bit cumbersome for such a basic feature.
Imho request-set headers should always overwrite the Collection-set headers.

@bbredewold when that happens - what is the authorization mode used for your request? Is it inherit, None, or Bearer token? It looks to me that the problem is your request Auth overwriting your manually set header, but that should not happen when using auth mode = 'none' .


Add namespace awareness

Thanks for this adapter module, great work and really helped us out a lot. We did run into a problem with a namespace in the payload causing the xpath evaluation to fail. A similar problem is described here: https://stackoverflow.com/questions/36905688/xpath-expression-unable-to-match. This PR contains a fix for that. I have not tested it on non-namespaced payloads yet. Please check if you are willing to merge otherwise just ignore

Hi Johan,
Thank you for your interest in this development - and special thanks for your contribution. Before I progress with the merge, can you please advise scenario where you faced this issue? In particular, when developing the module, I thought it to be primarily used in communication channels for adapters, where dynamic queue name / sequence ID can not be determined - such as JDBC - but in those adapters, the XML document (of message payload) is normally constructed with the message type namespace or namespace defined in the communication channel configuration (hence, all elements of the XML document will inherit the same namespace). I checked reference link and the code, it all makes sense - awareness about the scenario for me is to be able to recreate similar conditions and test the enhanced module version, as I will need to run regression and new functionality tests before publishing it here.
Regards,
Vadim

Closed due to inactivity.


Added a point to 3.1 on automating the "ETL" pipeline

This is often forgotten in the heat of the "build an ML project/case", and not picked up if the ML case falls through. This is damaging, because work that could be used by others (an automated pipeline for collection, aggregation etc) is then lost.

Hi, Good input. I added a following up question in the same line 'If not, why?'


Admin - Security AU-02 - Audit Events

Description
Investigate if applying to a job should be an auditable event
(A)	The organization determines that the information system is capable of auditing the following events: [Assignment: organization-defined auditable events].
(B)	The organization coordinates the security audit function with other organizational entities requiring audit-related information to enhance mutual support and to help guide the selection of auditable events.
(C)	The organization provides a rationale for why the auditable events are deemed to be adequate to support after-the-fact investigations of security incidents.
(D)	The organization determines that the following events are to be audited within the information system: [Assignment: organization-defined audited events (the subset of the auditable events defined in AU-2 a.) along with the frequency of (or situation requiring) auditing for each identified event].
Required for Completion

[ ] Documentation

For now we do not think any events in Talent Cloud need special handling for audit records.

Nevermind, this is actually just a decision of what needs to be auditable based on the staffing process. We should start a conversation with departmental HR to see if there's a buisiness need for us to audit anything at this time. Because HR is still accountable for this, we think there will be minimal requirements on Talent Cloud for auditing. Will confirm.

Check to see if the database can do audit triggers.

Preliminary audit logs are there. Need to tell TBS infrastructure to start collecting them and review what is being logged.


Update android arch lifecycle version

This change makes com.trello.lifecycle2.android.lifecycle.AndroidLifecycle_LifecycleProvider  implements android.arch.lifecycle.GeneratedAdapter.
Fixes #245

It looks like this fails due to lint issues. Could you look into that?
Also, I thought fixing #245 would require some source code changes (inheritance-wise).

According to this lifecycle component issue, I think there is no code changes needed.
And also I reported about the RestrictedApi lint issue. It can be solved by applying a new version of buildTools and android build gradle version.
I will update my PR to fix build failure.

Thanks so much for the work!


JAVA_OPTS / JDK_JAVA_OPTIONS not respected

Version(s)
1.0.4
Describe the bug
I have my Java proxies set in JAVA_OPTS and JDK_JAVA_OPTIONS, but scala-cli seems to ignore or reset them.  I had to put the arguments on the scala-cli command line for them to work.
I first used scala-cli through the metals project, and it (metals) failed because scala-cli couldn't download any files.  In cases like this, where the user is not even aware that they are using scala-cli, it would be nice if scala-cli would use the standard environment variables for proxies.
I'm not really sure if this is a bug or just a feature request... I think it depends on whether the current behavior is intentional.
To Reproduce
env JAVA_OPTS='-Dhttp.proxy=<IP_ADDRESS> ...' JDK_JAVA_OPTIONS='-Dhttp.proxy=<IP_ADDRESS> ...' scala-cli
Expected behaviour
I expected the proxies in JAVA_OPTS or JDK_JAVA_OPTIONS to be used.

Scala CLI indeed does not analyze those environment variables, so this mechanism could be added.
However, as of now, Scala CLI offers some other ways to specify Java properties:

a global properties file residing in /java-properties/scala-cli-properties
a local properties file with a name .scala-jvmopts residing in the directory where scala-cli is called
using config to set properties globally: scala-cli --power config java.properties '-Dhttp.proxy=<IP_ADDRESS> ...'

@MaciejG604 Once I realized the problem was in scala-cli, I didn't have much problem figuring out how to set the Java properties, thanks to scala-cli's very clear documentation.  But because I was using Metals, which under the hood uses scala-cli, I was not even aware that I was using scala-cli until I asked the Metals devs for help.  I think all the current mechanisms that you list are not generic, and would require the user to know they are using scala-cli and react accordingly.
Also, if you have never been forced to use a proxy, it's a huge pain. Needing to configure individual applications to get them to work makes it even worse!


MPEG-SD anchoring extension

This pull request brings the implementation of the MPEG_Anchor extension (link) along with a few bugfixes and reworks on the interactivity extension.
The extension implement:

The support for trackables

Including these types:

Floor
Viewer
Plane
2D Marker
Marker geo
Application

The support for anchors

For this pull request to be  working properly and to be able to test it, a version of Unity-player is needed, as well as the new version of rt-xr-content.
This extensions offers a lot of core fonctionnalities for AR, has been tested on a Android mobile phone and comes with the following dependencies versions: For Unity version 2022.3, most of the following packages are installed via the global AR Package from Unity. Here is the complete list:

ARCore Extensions - 1.40.0 - com.google.ar.core.arfoundation.extensions
Apple ARKit XR Plugin - 5.1.1 - com.unity.xr.arkit
AR Foundation - 5.1.1 - com.unity.xr.arfoundation
Google ARCore XR Plugin - 5.1.1 - com.unity.xr.arcore
OpenXR Plugin - 1.9.1 - com.unity.xr.openxr
XRCore Utilities - 5.3.0 - com.unity.xr.core-utils
XR Interaction Subsystems - 2.3.0 - com.unity.xr.interactionsubsystems
XR Legacy Input Helpers - 2.1.10 - com.unity.xr.legacyinputhelpers
XR Plugin Management - 4.4.0 - com.unity.xr.management

@Etiennefaivredarcier regarding the iOsResolver dll error.
The steps to resolve the issue must be documented in the player build steps.
I ended up installing the iOS build kit.
However the project now only compiles for Android because of the dependency on ARCore.
Windows build no longer works.

@Etiennefaivredarcier I can now build for Android but the Windows build is still broken


Converting to adata.X

If I'm wanting to pass the 'X_harmony" array downstream to other scanpy external packages like scanpy.external.pp.magic (https://scanpy.readthedocs.io/en/stable/generated/scanpy.external.pp.magic.html) or scanpy.external.tl.phate (https://scanpy.readthedocs.io/en/stable/generated/scanpy.external.tl.phate.html) how would I convert it to take the place of adata.X? sort of like how correct_scanpy() works in Scanorama (https://github.com/brianhie/scanorama) ? Thank you!!

@dmiyagi Sorry that I just saw your issue.
The normal use case of Harmony algorithm is to apply it to some embedding, such as PCA matrix. Therefore, The "X_harmony" array is usually of shape (n_cells, n_comps), where n_comps is the number of components. This is different from adata.X which is of shape (n_cells, n_genes).
So I don't think the direct conversion could work. Otherwise, you may think of creating another AnnData object of shape (n_cells, n_comps) with X_harmony matrix being its adata.X.

Hi @dmiyagi ,
Since I didn't hear from you for one year, I'll close this thread for the time being. Please feel free to reopen it if you still have the issue.
Sincerely,
Yiming


Plugin integration

How do I integrate your plugin in my project?. Can I compile it in pubspec.yaml instead of downloading ?. I want to use this library to show a video slider explaining usage instructions. It doesn't require any play and pause controllers as it should loop infinitely. I recently tried Flutter video player plugin for the purpose. But it worked except one minor issue which i couldn't fix. Video was having different aspect ratio I guess, as a result the frame dint fill the actual space of the video player and a dark line was visible on the right side.And interestingly not all phones had the issue except high quality screen phones like Samsung S8. Any work-around for the issue will be welcomed. I just want to see whether your plugin can help me serve the purpose .Thanks in advance

Use https://pub.dev/packages/video_player


[CssSelectorBridge] Improvements (#3537)

Continuation from #3457 including suggestions from #3537
Improve parameter documentation / add tooltips

Allow extracting content from home page instead of article page

Using site https://yohane.net/news/ as suggested by @antermin in #3537:
Home page:

Bridge parameters:
?action=display&bridge=CssSelectorBridge&home_page=https%3A%2F%2Fyohane.net%2Fnews%2F&url_selector=article&content_cleanup=div.p-article__body&format=Html

Resulting feed:

Feed is generated from home page using only one HTTP request to the website.
Keep titles from home page when every page  is the same
Now let's assume we want expanded content but each article page <title> is the same:

(First tab is home page, others are 3 first articles. Title is the same).
In that case, the bridge detects that each article <title> is useless and keeps titles taken from links in home page:
Bridge parameters:
?action=display&bridge=CssSelectorBridge&home_page=https%3A%2F%2Fyohane.net%2Fnews%2F&url_selector=article&content_selector=div.c-detail__article__bloc&content_cleanup=.c-detail__article_ttl,%20.c-detail__article_time,%20.c-detail__line,%20.c-detail__article_share&format=Html

Resulting feed:

Compatibility notes
Changes contained in this pull request should not affect compatibility of existing feeds. The only side effect I think of is that existing feeds targeting link parent element may now show some content while they previously did not.

@dvikan Thanks for reviewing my PR 🙂

getting this
Trying to get property 'href' of non-object at bridges/CssSelectorBridge.php line 202
Trying to get property 'plaintext' of non-object at bridges/CssSelectorBridge.php line 203
Trying to get property 'href' of non-object at bridges/CssSelectorBridge.php line 209

getting this
Trying to get property 'href' of non-object at bridges/CssSelectorBridge.php line 202
Trying to get property 'plaintext' of non-object at bridges/CssSelectorBridge.php line 203
Trying to get property 'href' of non-object at bridges/CssSelectorBridge.php line 209

@dvikan Managed to reproduce your issue. This happens when providing a selector to a DOM element that is not a link and does not contain any link. Fixing in #3585.


Language Search

Brief Description: Search for some languages that don't use the Roman alphabet do not show up in serach
Operating System (OS and version): Osx el capitan
OpenBazaar version (shown on About OpenBazaar page in menu): 1.1.4
Hardware: Macbook Pro Retina 13"
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce): Always
Steps to reproduce:

Go to settings, after initialization in English
Select the Languages drop down
Start typing. Doesn't find Chinese, Korean or Ukrainian, but can find Japanese

Observed Behavior: Doesn't find any language when one types in Ko, Ch, Uk,
Expected Behavior: It finds the language.
Additional info (links, images, etc go here):

this is fixed.


Content Copied from a Microsoft Teams Code Snippet is Pasted in Reverse in Windows 10

Steps to reproduce

Create and send a multi-line code snippet in Microsoft Teams
Copy the code snippet from Teams (can be from either Teams desktop app or https://teams.microsoft.com)
Paste the code snippet into either Pwsh.exe or PowerShell.exe

$date = Get-Date
$date

Expected behavior

Code is pasted in the same order it was copied

$date = Get-Date
$date

Actual behavior

Code lines are reversed

$date
$date = Get-Date

Environment data

I could not replicate this behavior on macOS and was not able to test on Linux. I also tested pasting into many other applications and did not observe the text order being reversed, so I believe this is a PowerShell issue and not a Teams issue (Notepad, Notepad++, Word, Command Prompt, Teams).

Windows 10 1909 10.0.18363.778
Microsoft Teams <IP_ADDRESS>63

Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I just tried this out on my machine:

I copied the contents of the snippet with Ctrl+V and pasted into the ordinary pwsh.exe conhost with Ctrl+C. Right-click to paste also got the order right, but omitted the newline (which I believe is a known limitation in that form of paste).
Some followup questions:

How are you copying?
How are you pasting?
Are you running normal pwsh.exe in the conhost (as in my screenshot), or are you running it in another terminal emulator (like Windows terminal or Hyper)?
Does this also occur with powershell.exe?
Does this also occur with the current PowerShell preview (7.1.0-preview.2)?
Does this occur for you on any other machines?
Does this occur with any other Electron applications, like VSCode or Slack, or possibly Chrome?

I've noticed this seems to be getting a little better as the months pass, so I'm not sure if this is something that the app is doing incorrectly, or if PS needs to handle it better.
I notice it most with apps like floobits that we use commonly in the PS slack/discord, pasting snippets from those into PS is very odd unless you go via some other text editor.

@rjmholt - It looks like you just formatted your text as monospaced as opposed to using a code snippet. Should look something like this:

To answer your questions though:

I am copying with Ctrl+C
I am pasting with right-click (I just discovered that the issue doesn't occur with Ctrl+V)
Yes, pwsh.exe is being run in the console host and is not using an emulator
Yes, issue exists in 7.1.0-preview.2 (see screenshot below)
I have observed this on other computers as well that have the same version of Windows as me
I also tested with VSCode and didn't observe the behavior

I can confirm that creating a full code snippet doesn't change the behaviour I see either.
One last question: if you disable PSReadLine with rmo PSReadLine, does this issue still occur?

I have observed this on other computers as well that have the same version of Windows as me

Given that PowerShell's version is independent of Windows, that is more suggestive of this bug living beyond PowerShell's own code.
/cc @SteveL-MSFT and @daxian-dbw in case they have any insights into any manipulation we might do with the clipboard

It is probably https://github.com/PowerShell/PSReadLine/issues/579
As @rjmholt said, can you try without PSReadLine?

I would agree that this is a PSReadLine issue after looking at those two issues. When I paste after disabling PSReadLine, it's in the correct order, but line returns are not understood:

I also tried the workaround recommended in PowerShell/PSReadLine#496 and it appeared to offer the same functionality:

I think the best work around until this is fixed is just to use Ctrl+V which doesn't have these issues. I also think this can be closed now that I see it has an existing open issue within PSReadLine.


KeyValue ->reorderable() doesn't work

Package
filament/filament
Package Version
v3.2.16
Laravel Version
v10.41.0
Livewire Version
No response
PHP Version
PHP8.2.15
Problem description
KeyValue::make('meta')
    ->reorderable()

This should reorder the key/values and save them in the db as such. However when you change the order and reload the page or go back to the edit page. The ordering has not changed
Expected behavior
The Ordering set by the user should be respected
Steps to reproduce

go to http://filament-issue.test/admin/products/1/edit
Save an order.
Go back to http://filament-issue.test/admin/products/1/edit
Change the order and save.
Reload and the order has not changed

Reproduction repository
https://github.com/sandersjj/filament-issue
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

Duplicate of #9030? Please check your Livewire version

I have checked it and I am running livewire v3.4.2.
This Pr has been merged in v3.4.0 so it didn't solve the issue yet.

@sandersjj, have you found the fix yet?

@sandersjj, have you found the fix yet?

No I ended up not using the KeyValue field.

I think the problem itself is in trying to add the ordering to one-dimensional array.
json.org states that:

An object is an unordered set of name/value pairs.

That not only means that order of name/value pairs is (could be) not preserved, but also that two objects which differ only by order of name/value pairs are (should be) considered the same.
On the other hand, php.net states that:

An array in PHP is actually an ordered map.

The order is preserved for string keys, but numeric keys could not be ordered inherently.
Also, as we can see, Livewire tracks changes in JS objects with no regards to name/value pairs order, in order to minimize payload (and imho it's a good choice).
Thus, the KV component itself should either not have "ordering" feature, or change the storage mechanism to "array of objects".
For example, instead of storing as:
{ one: "One", two: "Two" }
it could be:
[ { key: "one", value: "One" }, { key: "two", value: "Two" } ]
This way, the ordering will be preserved both in frontend and backend.

change the storage mechanism to "array of objects".
For example, instead of storing as: { one: "One", two: "Two" } it could be: [ { key: "one", value: "One" }, { key: "two", value: "Two" } ]

@danharrin what do you think about this? do you think this can be implemented in v3 with backward compatible or only in v4?

This has been handled in v4 by updating the internal data structure as per #12943, and then masking that internal state with a new "cast" to avoid breaking changes to $get() and $set() etc.


High security vulnerability: markdown-spellcheck

We use it in open-aea repo as well
https://github.com/valory-xyz/consensus-algorithms/blob/39aca973e1083f8c9049a832afd2be90d26eefda/.github/workflows/main_workflow.yml#L98
zarathustra@zarathustra-XPS-13-9305:~/Repositories/valory/consensus-algorithms$ sudo npm install -g markdown-spellcheck
npm WARN deprecated<EMAIL_ADDRESS>This package is unmaintained. Use @sinonjs/formatio instead
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated in favour of @sinonjs/samsam

added 127 packages, and audited 128 packages in 8s

37 packages are looking for funding
  run `npm fund` for details

1 high severity vulnerability

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

Not a prod dependency


Question

Hello, I want to know whether this tool can generate SBOM by analyzing Java project files (binary or source code) or other files, if it could generate SBOM, what formats does it support？
Maybe, the tool is made for consume SPDX SBOM?
I will appreciate it very much if you could answer my questions.

Hi @Donkey-Hao - This project is designed to be a library used by any tool which generates or consumes SPDX document - it isn't really a standalone application.  It supports SPDX version 2.0 and later formats (2.0, 2.1, 2.2 and 2.3).
There is a general utility tools-java which is a command line tool to help manage SPDX documents - but it doesn't do any analysis or scanning itself.
If you're looking for a scanning solution, check out the SPDX tools page for a list of open source and commercial tools which support the SPDX format.


Meta does not parse paths with leading double-colon

Minimal reproduction:
use syn;

fn main() {
    // correctly parses
    let attr: syn::Attribute = syn::parse_quote!(#[derive(serde::Serialize)]);
    let _ = dbg!(attr.parse_meta());

    // error, expected identifier or literal
    let attr: syn::Attribute = syn::parse_quote!(#[derive(::serde::Serialize)]);
    let _ = dbg!(attr.parse_meta());
}

I would expect the second example to parse since it's a valid path. I think the meta parsing code does a lookahead expecting only a literal or an identifier, but paths can start with the non-identifier ::.
I read the previous discussion about not wanting Meta to have to cover all possible attribute syntaxes, but I think this fits solidly within the current expectations of what Meta should parse. Given that it stores Paths, it should be able to parse any kind of Path.

Fixed in 1.0.46.


Hotfix/class rolls

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Pie Chart with Json data

Hi, I have an issue issue, my data as follows.
[{"LC":"xyz","count":1000},{"LC":"abc","count":2000},{"LC":"BCD","count":1500},{"LC":"Others","count":500}]';
Is it possible to generate pie chart ? i am able to generate line & bar but when using the chart type as pie. It is just showing 100 %.
Regards
Harish

Hi,
I have similar need, json data with two columns datalabel and value.
I think pie chart should work as other charts and not to exception.


Merge branch 'master' into barnhark/finish_base_tests

Addresses #46
Now all BC handlers should have 100% coverage

Pull Request Test Coverage Report for Build 201

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

Totals

Change from base Build 196:
0.0%

Covered Lines:
931

Relevant Lines:
2336

💛  - Coveralls

Pull Request Test Coverage Report for Build 201

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

Totals

Change from base Build 196:
0.0%

Covered Lines:
931

Relevant Lines:
2336

💛  - Coveralls

Pull Request Test Coverage Report for Build 201

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

Totals

Change from base Build 196:
0.0%

Covered Lines:
931

Relevant Lines:
2336

💛  - Coveralls

This PR was made in error


(Scoping) Chain abstraction SDK not working with 1inch API anymore

Describe the bug
Integrators are reporting that the chain abstraction SDK is no longer working.
To Reproduce
Steps to reproduce the behavior:

Try to use chain abstraction SDK to get quotes from 1inch.

Additional context
Add any other context about the problem here.

Need to scope out what it takes to fix and analyze cost

1inch public api has been deprecated.
If we want to continue with 1inch we will need to use via 1inch dev portal(fyi: it's 1rps for basic account, so for sure pricy to use it.)
mean finance is using the latest dev portal, https://github.com/Mean-Finance/sdk/blob/854f94f3f14273689c35357e21c1a6f83d57d57c/src/services/quotes/quote-sources/1inch-quote-source.ts#L6.
we can put config updates, so dapps can add there own 1inch API.

Yes we should allow them to add their own 1inch API as the quick fix.

Closing this to open a new task.


ColumnEncodingUtility should preserve foreign keys

The Column Encoding Utility drops all of the table's foreign keys.  Would be nice if it preserved them.  (Will have to recreate them after recreating the table).

Agreed - I'll look into this

So I've dug into this, and we do recreate FK's. Can you tell me what you are seeing that suggests this isn't working?

Ah I see my mistake.   I thought I had some foreign keys defined where I did not.  Sorry for the red herring.


Guidance on QoS for SRS

Where can I find more information on "It is advised to enable Quality of Service (QoS) for Skype Room System."?

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

ID: 9d24bf2b-244d-0c6a-3984-f65def69bbb1
Version Independent ID: 33b21755-7b6f-2093-8983-34694327bb8f
Content: Skype Room System domain joining considerations
Content Source: Skype/SfbServer/deploy/deploy-clients/domain-joining-considerations.md
Service: unspecified
Product: skype-for-business-itpro
GitHub Login: @jambirk
Microsoft Alias: jambirk

I've updated the topic with some links to QoS information. Update should be visible later today.


Violet aura is OP.

Placing few (5-10) nodes up and filling them with violet aura gives you power for decades.

The violet aura mechanics have already been significantly reworked in the latest builds


Create a tag?

Hello,
We work on project that use this package (namely NataliaKor/lisaflow - currently private).
I have noticed that the main branch hasn't moved for some time now (although development branches seem active). Do you think it would be wise to tag a version? This would help when installing packages that depend on this one.

Hi, lisatools I going through its release update. So this is now on the main branch.

Hi,
Sorry for the delay. I'm not sure I understood your message but from the commits, I understand that you're in the process of merging branches and fixing bugs. So maybe we can expect a stable version soon. Right ?

Yes. I would say it is stable now. But there may be a quick flurry of bug activity with a large increase in users.


Add in text rendering for class lists, with EducatorLabel authorization

Who is this PR for?
WH educators
What problem does this PR fix?
Class lists have different authorization rules that mean access doesn't stay static over time, particularly after the Aspen rollover and new homeroom and grade assignments for educators and students.  Some folks at WH want to access these lists, but we've turned off the feature.
What does this PR do?
Uses the EducatorLabel in https://github.com/studentinsights/studentinsights/pull/2028 to grant access to an endpoint that renders class lists as text.  This is simpler than reworking the authorization rules, and reworking the UI code to work for historical class lists as well as for prospective class lists.  Those changes would be quite involved, and folks just want to see their class lists to do their work now, so this gives them the data in a more minimal format, without requiring moving student data through other systems.
This also provides a good way for developers to compare the result of the existing endpoints and UI if we want to extend those to work with historical lists as well, which I assume we will want to do in October or later this fall.
Screenshot (if adding a client-side feature)
Checklists

[x] Author included specs for new code

selfie


fix mimir sort when using order by desc and asc

Fix mimir ordering in different directions.
Fixes #3183

:tada:

Going to reassign this "feature" status since it's kind of a big bugfix.  :-P


Give a slight shimmer to every cell along path during build mode

To highlight where the mobs will pass. Since the path will update every time a building is being attempted to be placed, it should always show updated.

Shimmer implemented, will need to activate it during build mode.


Adding 'Previous Role' to the reconfiguration text

Currently, if a partition is undergoing reconfiguration, there is no means to determine the primary replica, as the only information available to customers via SFX is the target role of the replica. The introduction of 'Previous Role' will significantly reduce the risk of accidentally terminating the primary replica, a scenario that could potentially lead to data loss.
While a partition is undergoing reconfiguration, in addition to the 'Target Role', 'Previous Role' will also be displayed.

ADO work item

could we also add a new column to the replica list for partition which shows previous replica role? this could be useful in investigation

could we also add a new column to the replica list for partition which shows previous replica role? this could be useful in investigation

We can't do that, since the API used for getting previous replica role Get-ServiceFabricReplica
 only returns previous role if the partition is undergoing a reconfiguration.
But we can add Previous Role in partition view if the partition is reconfiguring.
Something like this:

Latest changes to the UI:

Also, I need help with adjusting the CSS for replica-tile component. Tile's height doesn't adjust to the added field... It looks like the relevant styles are in src\SfxWeb\src\app\modules\partition-replication\replica-status-container\replica-status-container.component.scss

Latest changes to the UI:
Also, I need help with adjusting the CSS for replica-tile component. Tile's height doesn't automatically adjust to the added field... It looks like the relevant styles are in src\SfxWeb\src\app\modules\partition-replication\replica-status-container*replica-status-container.component.scss*

I only meant the table at the bottom, like an extra column added next to replica role

Latest changes to the UI:
Also, I need help with adjusting the CSS for replica-tile component. Tile's height doesn't automatically adjust to the added field... It looks like the relevant styles are in src\SfxWeb\src\app\modules\partition-replication\replica-status-container*replica-status-container.component.scss*

I only meant the table at the bottom, like an extra column added next to replica role

Oh, we can do that, but like I already mentioned, we can only show previous role if a partition is undergoing a reconfig.

my mistake I didnt see that in the previous message.

@jeffj6123 @chensation After a considerable break, this PR has been revived! 😊 Please take a moment to review the most recent updates.

@vpavlovicMSFT , I think my requested changes still stands. The same role logic is inside DeployedReplica.ts and we should change it as well unless there's a reason to not change it.
In addition, although you added a PreviousReplicaRole field to the cypress fixtures, replica.cy.ts and  deployedReplica.cy.ts don't have any tests to check on if roles are displaying properly. I think it's good practice to always update and add tests with every feature we add so we can maintain and increase our test coverage.
@jeffj6123 , I know your original comment said you only wanted a "Previous Role" column at the bottom table, but would you say that adding a "Previous Role" row to the Replicator Status would still make it easier to read?

this particular change has been reverted, and now looks like this

Hi @jeffj6123 and @chensation, we addressed the requested change and adjusted the logic in DeployedReplica.ts. We would like to complete this change soon. Can you please review the latest version?

@mssusnjar can you add tests into replica.cy.ts and deployedReplica.cy.ts to test for this change?


Remove tracking parameters from copied URLs

Before Submitting Your Feature Request

[X] Check that there isn't already a similar feature request to avoid creating a duplicate.
[X] I have seen the FAQ.

Problem
Inspired by this tweet.
Many links contain UTM / other tracking parameters that may be unwanted.
Solution
A simple toggle that removes tracking parameters from a URL when you copy it to your Maccy clipboard.

There is a wonderful and free app https://apps.apple.com/us/app/pure-paste/id1611378436?mt=12 which already does it.


Update artifact to DG 8.1.1. (CR2)

Change the config generator to 2.0.3.Final
Bump the version to GA.3 to avoid tag collision

Thanks @osnipso

Thanks @osnipso


Add blocking collections in place of slims

Switch to blocking collection instead of slims

Thanks. This open my eyes for the good BlockingCollection feature.


(yaml) Add support for inline sequences and mappings

Describe the issue
Current highlighting does not support inline sequences and mappings.
Which language seems to have the issue?
YAML
Sample Code to Reproduce
https://jsfiddle.net/sda50gr8/
Expected behavior

Can these be recursive also?

Yes, these can be nested. I've started working on this, but the simple implementation (just add a rule for all brackets and commas) auto-detected a "clean" file as yaml.

Wait. What is the goal here again?  I’m not certain it’s clear from the snap.

I think top priority should be given to making sure numbers are not highlighted as strings (e.g. {1 or 2.0, or 3] would all be highlighted as strings). A secondary concern might be highlighting braces, brackets, and commas as a separate type than strings.

A secondary concern might be highlighting braces, brackets, and commas as a separate type than strings.

Traditionally we don't highlight symbols.
You might look into the JSON grammar for inspiration.


Data send to the server unencrypted - question

Hi!
In here https://atmospherejs.com/typ/accounts-ldap under "Issues + Notes" you have mentioned that "Because the package binds/authenticates with LDAP server-side, the user/password are sent to the server unencrypted. I still need to figure out a solution for this."
What does that mean and what is the impact on the security part?

All this means is...  if someone/something is sniffing the network between the client (browser) and server (meteor/node) then they could see the clear-text credentials [username & password] on the wire. Difficult (relatively) but still possible. Probably less of an issue in an internal network (intranet), but major issue over the internet.
Simplest fix is to have an SSL connection to the server! But involves HA proxy, stud and/or cloudflare (if server is public).


Change build steps for a smoother publish

This PR makes improvements in the way we build the dist folder which now we include the type definitions in the package.
How to test:

npm run dist
Note the dist folder contain the js and d.ts files from each raw ts/tsx.

chore(dependencies): update and add dependencies
Update dependencies to latest versions.

@types/react-dom

Add new dev dependency

rimraf
This is a OS independent alternative to rm -rf

Move dependencies

node to dev
npm to dev

The move was necessary because they also got installed when the user installed the package.

Commit:
b4555aa31f3270d1e4c5273e5989eb9a59349319
chore(export): change export syntax style
This changes the syntax of the exports to be less verbose.

Commit:
340802eaef7f45e4ecdb9e293846ae770649f76a
chore(namespace): add `@dcos` as a namespace to the package name

Commit:
ab69a1f9103e66d9e80b1fb0340f32fe4cc29eca
chore(tsconfig): change tsconfig to improve build
This adds declarations to the build which unfortunately also means that we need to disallow JS.
Also this changes the libs which are included and removes some redundancy.
The build also only includes components which are linked in the index.js file.
The build will exclude test and story files as they are not needed as a dependency.

Commit:
2f591e82dec163ff0f4c2e5d6e5320576bd9a0a0
chore(git-hook): change git hook setup script
The git hook is now installed by running
npm run setup:git-hook
this was necessary as by installing the package npm was trying to setup the git hooks as well in the
other project.

Commit:
f76bb7c362a7cbd614baab01653bfb2c23a216a3
chore(dist): change dist npm script to use tsc instead of webpack
The dist script now uses the typescript compiler instead of webpack by that we are now supporting a
handful of options for the user to include a component

import Badge from "@dcos/ui-kit/dist/components/badge";
Direct importing will only include the code necessary for that component into the build of the
application.
import { Badge } from "@dcos/ui-kit";
Destruct import will include the complete ui-kit library in the build of the application this is
only recommended if most of the components are actually used in the application to prevent build
bloat as tree shaking will also not properly work.

2.5. import { Badge } from "@dcos/ui-kit/dist/components";
This is basically the same import as in 2. but more verbose.
close DCOS-21892 DCOS-21891

Commit:
ad08a923b4783a9ee0ef333b625aca7073bd5452
chore(npm-scripts): clean up npm scripts
Change npm scripts to be alphabetical sorted, also spreading some scripts into pre scripts.

Commit:
4892c3bb0483c3d8b60dcfee4073ae5c1f45cc66
chore(jenkinsfile): change artifact check
This changes the glob jenkins is checking after the build step from dist/* to dist/**/*

Commit:
fdb46f696b4082764462ee20193799f7753a871b
chore(tsconfig): add tsconfig.dist.json
This adds a tsconfig for the dist creation and also readds allowJs option to tsconfig.json so
tsconfig.json is used for storybook and tsconfig.dist.json is used to create the bundle.

Commit:
5cda098b31221b0cf46ee5c0045ee689af143369

@Poltergeist and I took offline.

Works as seen on TV


Issue with unprotect_branch_new_api

Hi,
In version 2.0.0post1,

Procedure _make_requests_to_api, core.py line 113: exception when response.status_code is 404, exception message "json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)".

Procedure _make_request_to_api, core.py line 208,: exception when response.status_code is 404, exception message "json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)".

Seems expected_codes contains code 404, not quite understand why the judgement in line 192 of core.py is presented.
        if response.status_code == 404:
            raise NotFoundException("Resource path='%s' not found!" % url)

In branch_protector.py line 53, exception when response has no "unprotect_access_levels".

Thanks.

Hi @bethmal !
Thanks for reporting the issues.
It would help if you could provide some more context though. Preferably configuration snippets that have caused those errors.
Ad 3: can you take a loot a this please, @YuraBeznos ? Maybe it will be obvious to you what is the cause.

@gdubicki
Please find below configuration snippets. Basically the case is like:
If unprotect_access_level is not provided in configuration yml file, and meanwhile master branch is not protected (the reason could be either it was not protected on purpose or is was unprotected by gitlabform ), then procedure protect_branch does nothing. In this scenario, if a file need to be updated as well, the re-protect operation fails after the file is pushed.
If master branch is not protected (the reason could be either it was not protected on purpose or is was unprotected by gitlabform ), and then unprotect_branch_new_api is invoked somehow, exceptions mentioned above would be raised. The response code 404 is not json dump-able or loadable.
The response of get_branch_access_levels contains no unprotect_access_levels in our case.
projects_and_groups:
  "gitlabform_test/*":
    branches:
      # see comment above - this branch config will be ADDED to the branches configured on a group level
      master:
        protected: true
        push_access_level: 40
        merge_access_level: 40
        unprotect_access_level: 40

    files:
      ".gitlab/merge_request_templates/MR.md":
        overwrite: true
        branches:
          - master
        skip_ci: true
        file: MR.md

If unprotect_access_level is not provided in configuration yml file, and meanwhile master branch is not protected (the reason could be either it was not protected on purpose or is was unprotected by gitlabform ), then procedure protect_branch does nothing. In this scenario, if a file need to be updated as well, the re-protect operation fails after the file is pushed.

This is fixed in main branch. Now to the second case..

Hi @gdubicki ,
Could 404 exception issues (protect unprotected branch) be solved as early as possible? This issue is blocking gitlabform critical feature.
Thanks.

Sorry @bethmal, but we need more contributors to get issues fixed faster. I am already doing the best I can.
Please check out v2.0.2 - it should have all your issues fixed.

Please check out v2.0.5rc1, @bethmal .

@gdubicki Thanks for the update. v2.0.5rc2 works perfect for this scenario.

Great! I just released v2.0.5 final with this fix then so we can close this issue. Of course please reopen if you see any issues with this again.


Minor documentation nitpick: Python development mode

Issue description
The documentation for Python development mode says:

If we create a shell.nix file which calls buildPythonPackage, and if src is a local source, and if the local source has a setup.py, then development mode is activated.

This sounds like you need a shell.nix file and you need to use buildPythonPackage. But is that really true? I have a project that only has a default.nix and uses buildPythonApplication and it still seems to call setup.py develop.

Right, the name of the Nix file is not important – you can use any name and then run nix-shell yourfile.nix. I would guess the important thing is that you ran the shell (that is $IN_NIX_SHELL environment variable is set) but I do not see any relevant mentions of develop in nixpkgs.
buildPythonApplication does basically the same thing as buildPythonPackage so it makes sense that they work similarly:
https://github.com/NixOS/nixpkgs/blob/f86e309a9415049d6b0e4ad67fcf16f80e8b087d/pkgs/top-level/python-packages.nix#L54-L67

Correct, any .nix file works, it's just that by convention we call it shell.nix and nix-shell also chooses shell.nix over default.nix.

Okay, thanks for the clarification guys!


Cannot deserialize json that contains comment

Describe the bug
If the json has a comment in it, SchemaSerializer.DeserializeObject throws the exception '/' is an invalid start of a property name. Expected a '"'
It looks like this can be resolved by adding the following to the JsonSerializerOptions: ReadCommentHandling = JsonCommentHandling.Skip
Steps to reproduce
Try deserialize json with a comment in it. This can be demonstrated by the unit test below:
[Fact]
    public void Deserializing_ProductJsonLd_HasComment()
    {
        var jsonWithComment =
            "{" +
                "\"@context\":\"https://schema.org\"," +
                "\"@type\":\"Product\"," +
                "\"name\":\"Executive Anvil\", // some comment" +
                "\"description\":\"Sleeker than ACME's Classic Anvil, the Executive Anvil is perfect for the business traveller looking for something to drop from a height.\"," +
                "\"image\":\"https://www.example.com/anvil_executive.jpg\"," +
            "}";

        var exceptionMessage = string.Empty;
        try
        {
            SchemaSerializer.DeserializeObject<Product>(jsonWithComment);
        }
        catch (System.Text.Json.JsonException ex)
        {
            exceptionMessage = ex.Message;
        }

        Assert.Equal(string.Empty, exceptionMessage);
    }

Expected behaviour
Deserializes the json without throwing an exception
Schema objects
https://schema.org/Product (but affecting all objects)

I could add a pull request for this if you are happy to use the JsonCommentHandling.Skip option?

We'd be happy to take a PR. Thank you @dermotblairca!

Thanks @RehanSaeed. I'm a bit stuck on this one. I have changed the DefaultSerializationSettings/HtmlEscapedSerializationSettings in SchemaSerializer to add the option ReadCommentHandling = JsonCommentHandling.Skip.
However it then crashes on the following line: https://github.com/RehanSaeed/Schema.NET/blob/f64670fafa0eb14916ddda181a3a9a0bfa250a03/Source/Common/ValuesJsonConverter.cs#L192
with exception Expected start of a property name or value, but instead reached end of data”
This can be reproduced by updating the following unit test to add a comment as I did below:
[Fact]
    public void ReadJson_Values_SingleValue_ThingInterface()
    {
        var json = "{\"Property\":" +
            "{" +
                "\"@context\":\"https://schema.org\"," +
                "\"@type\":\"Book\"," +
                "\"@id\":\"https://example.com/book/1\"," +
                "\"name\":\"The Catcher in the Rye\", // some comment" +
                "\"url\":\"https://www.barnesandnoble.com/store/info/offer/JDSalinger\"," +
                "\"image\":\"book1.jpg\"," +
                "\"author\":{" +
                    "\"@type\":\"Person\"," +
                    "\"name\":\"J.D. Salinger\"" +
                "}" +
            "}" +
        "}";
        var result = DeserializeObject<Values<string, IBook>>(json);
        var actual = result.Value2.First();

        Assert.Equal(new Uri("https://example.com/book/1"), ((Book)actual).Id);
        Assert.Equal("The Catcher in the Rye", actual.Name);
        Assert.Equal(new Uri("https://www.barnesandnoble.com/store/info/offer/JDSalinger"), (Uri)actual.Url!);
        Assert.Equal(new Uri("book1.jpg", UriKind.Relative), (Uri)actual.Image!);
        var author = Assert.Single(actual.Author.Value2);
        Assert.Equal("J.D. Salinger", author.Name);
    }

There doesn't seem to be an overload (or similar method) to JsonDocument.ParseValue that allows you to pass in the options.

I think asking on the .NET repo about System.Text.Json would help here.

Thanks @RehanSaeed I have added an issue to the .NET repo: https://github.com/dotnet/runtime/issues/67004

This is actually just related to the comment not having a newline after it, so the ParseValue function doesn't know where the comment ends: https://github.com/dotnet/runtime/issues/67004#issuecomment-1076161522

Cool, does this mean we can close?

Yes fine to close this, thanks @RehanSaeed

This looks like it is still an issue actually. For example, Schema.Net package crashes trying to deserialize json with a comment but System.Text.Json works fine for the same json.
To reproduce, I created this unit test which deserializes a Product with both Schema.Net and System.Text.Json:
[Test]
public void GetSchemaOrgOffers_GetOffer_DeserializeComment()
{
    var pageContent = File.ReadAllText("C:/SchemaOrgTestPages/SchemaNetProductExample.json");

    string schemaNetException = "", systemTextJsonException = "";
    try
    {
        var product = SchemaSerializer.DeserializeObject<Product>(pageContent);
    }
    catch(Exception ex)
    {
        schemaNetException = ex.Message;
    }

    try
    {
        var options = new JsonSerializerOptions
        {
            ReadCommentHandling = JsonCommentHandling.Skip
        };
        var product = JsonSerializer.Deserialize<Product>(pageContent, options);
    }
    catch(Exception ex)
    {
        systemTextJsonException = ex.Message;
    }

    Assert.AreEqual("", schemaNetException);
    Assert.AreEqual("", systemTextJsonException);
}

The result of this is that SchemaSerializer.DeserializeObject throws the exception /' is an invalid start of a property name. Expected a '"'..
JsonSerializer.Deserialize does not throw an exception and deserializes the json successfully.
The contents of the SchemaNetProductExample json files are:
{
	"@context": "http://schema.org",
	"@type": "Product",
	"@id": "https://shopping.coop.dk/vare/braun-stavblender-multiquick-5-mq5235wh-hvid-1000-watt-inkl-piskeris%2c-minihakker-og-baeger/8021098774903",
	"description": "Stavblender fra Braun Et fuldendt stavblendersæt til den flittige kok. Her kan du både blende, piske og hakke, og med hele 21 hastigheder kan du tilpasse styrken til ethvert formål. 21 hastigheder Skridsikkert greb Inkl. minihakker (500 ml), piskeris og blenderbæger Stavblenderen er designet med ergonomisk udformet og skridsikkert håndtag, så du har et sikkert greb. Blenderhovedet, piskeriset og minihakkeren udskiftes nemt med EasyClick, og alle dele tåler maskinopvask, så rengøringen er hurtig klaret. Minihakkeren rummer 500 ml og har sugekopper i bunden, så den altid står stabilt under brug.",
	"name": "Braun stavblender - Multiquick 5 MQ5235WH - Hvid",
	"url": "https://shopping.coop.dk/vare/braun-stavblender-multiquick-5-mq5235wh-hvid-1000-watt-inkl-piskeris%2c-minihakker-og-baeger/8021098774903",
	"category": "/koekken/koekkenapparater/stavblendere",
	"image": "https://coopcdn-res.cloudinary.com/image/upload/c_lfill,q_auto,f_auto,g_auto,w_500,h_500/produktbilleder/braun-stavblender-multiquick-5-mq5235wh-hvid.jpg%3fid=6709882",
	"gtin13": "8021098774903",
	"sku": "1000204181",//Todo: Måske?
	"offers": {
	"@type": "Offer",
	"priceValidUntil": "01-01-2099 00:00:00",
	"availability": "http://schema.org/SoldOut",
	"price": "400.00",
	"priceCurrency": "DKK",
	"url": "https://shopping.coop.dk/vare/braun-stavblender-multiquick-5-mq5235wh-hvid-1000-watt-inkl-piskeris%2c-minihakker-og-baeger/8021098774903"
	}
	,"brand":"Braun","model":"5235WH"
	,"additionalProperty": [{"@type":"PropertyValue","name":"Serie","value":"Multiquick 5"},{"@type":"PropertyValue","name":"Type","value":"Stavblender"},{"@type":"PropertyValue","name":"Antal hastigheder","value":"21"},{"@type":"PropertyValue","name":"Motoreffekt i watt","value":"1000"}]
}

See the ToDo comment after the sku field.


Documentation for remote layout management server

Background
#57 introduced the remote server for layout management. We need to write documentation on how to run the server and publish this to the main Vuu docusaurus site. #27 also integrated remote layout management into the UI using the server, and provides a means of choosing either the local or remote implementation with envars. We should document how to change from local/remote.

Where does documention get published from?
What format? (MD?)


New IO for NSDF format

This is to add a new IO class for NSDF. It supports reading and writing Blocks, Segments and AnalogSignals with all relationships and meta data. Support for another neo objects will be added in the near future.

Coverage decreased (-0.4%) to 49.945% when pulling b54f61fe920eb86436a5d47473d7abdcc6d1ebde on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

Coverage decreased (-0.4%) to 49.94% when pulling e2e463cc8405e8ea0482db961b63024634834a1e on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

Coverage decreased (-0.4%) to 49.945% when pulling 6305c3d500994424d83df23c0cdd10694cbd9608 on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

Coverage decreased (-0.4%) to 49.945% when pulling 6305c3d500994424d83df23c0cdd10694cbd9608 on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

Coverage decreased (-0.4%) to 49.937% when pulling 65efac7b4092551e0e618808e1a7dc2ed39fa5d3 on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

I'm sorry, there were few things that I haven't notices before creating this PR, but now it seems that all is right, so can you have a look at it and merge if all is OK please?

Thank-you for contributing this! Please could you add your name and affiliation (if appropriate) to doc/source/authors.rst before I merge this?

Thank you for the reply. I updated authors.rst file as you said.

Coverage decreased (-0.4%) to 49.923% when pulling 2d0b0d29b61c324bfda43e52d58db647f421413c on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.

Coverage decreased (-0.4%) to 49.923% when pulling 2d0b0d29b61c324bfda43e52d58db647f421413c on mgr0dzicki:master into 9bd5f01a3b3dc311a89c0be3b0008e516fd3a098 on NeuralEnsemble:master.


[User Profile] Update top nav to link to profile

@julielorch heads up this is a visualization for what we mentioned in the kick off meeting.

Not sure what needs to happen here, but this is one of the last issues to close out this milestone.

This mock is still accurate, first name + pipe character.

Ok i've turned this on on staging to give more clarity.  There is no pipe after the username, so we need that.  The pipe spacing is also funky when the user is logged out.
What should the text read when the user is logged out?  "Create an account"? "Register" ? @lkpttn @mikefantini

@aaronschachter @deadlybutter I'd vote for "Create an account" as long as the longer text doesn't mess with responsiveness

We have a new solution for the crowded header:

And mobile


Submission for week 2 assignment

I've added my file (aws-utils.rb) at the specified location ($BOOTCAMP/source/week2/ggrpmnn/). I attempted several of the bonus activities as well, but ran out of time for all of them.

That's awesome! Although, we might change the location of where assignments are submitted for Bootcamp.


Array: Unify Chunks

There's a pretty major TODO in the array implementation, to implement unify_chunks, the code that takes two arrays of possibly different chunking / dimensions, and aligns them so their chunking corresponds.  There are several TODO notes in blockwise.py around this as the align_arrays= keyword.
My recollection is that this was the biggest missing piece (among several missing pieces) that stopped the implementation from being somewhat useful.
No urgency of course, just highlighting where I think the next big step might be.

that's done now


chore: remove config.yml

Description
As per https://github.com/zeta-chain/node/issues/543
Would it be also good to remove the starport placeholder comments in the code? It could be confusing for someone looking at the code (to be handled in a separate PR to keep this one as chore)
Closes:
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Checklist:

[ ] I have added unit tests that prove my fix feature works

🥲

fwiw we also still have handlers, which are no longer required by Cosmos SDK, but used by Ignite CLI. If we're no longer using Ignite, might as well remove them (in a different PR perhaps).


Underestimate the number of faces

In the image in the face_detect.py file, the number of faces is 26. But the program counts 19.

That's not a bug, actually. The number of faces you get will vary on the image and classifier parameters you've chosen. You can't place a hard-and-fast bet on the number of faces you get because it depends on the image (how noisy, grainy etc) and how accurate you've asked the classifier to be (changing the parameters)


i18n filter doesn't work in v1.1.2

I have a fairly large app in development that uses the i18n filter in angular-localization quite a bit. I was using 1.0.1 and just upgraded to 1.1.2 and the filter stopped working. The attribute directive and service still work as expected. I confirmed this behavior in the demo app included in this repository.
I stepped into the code, and it looks like the filter fires when expected and causes the language files to load from the server, but the strings are never replaced after the language file is loaded.
I also upgraded Angular to 1.3.0 at the same time, though I don't know if that matters.

Thanks @doshprompt
Im using 'ui.router' and now it is working. My code is:
(function() {
  angular.module("ppaUi", ['ionic', 'ngLocalize', 'restangular', 'ui.router']).config([
    '$stateProvider', '$urlRouterProvider', function($stateProvider, $urlRouterProvider) {
      $stateProvider.state("home", {
        url: "/",
        templateUrl: "app/main/main.tmpl.html",
        controller: "MainCtrl"
      }).state("login", {
        url: "/login",
        templateUrl: "app/auth/auth.tmpl.html",
        controller: "LoginCtrl",
        resolve: {
          langs: function(locale) {
            return locale.ready('auth');
          }
        }
      });
      return $urlRouterProvider.otherwise('/login');
    }
  ]);

}).call(this

@doshprompt
When I use 'locale.setLocale(loc);' occurs same problem when I dont have the resolve on stateProvider.
When I call 'locale.setLocale(loc);' only directives receive translations, in filter i18n nothing happens.

@doshprompt
How can i change locale at run time by clicking on button??
Here is my code example but it's not changing locale.
app.controller('exampleCtrl', [ '$scope', 'locale', 'localeEvents',
function($scope, locale, localeEvents) {
$scope.changeLocale = function() {
console.log("::::I:::::" + locale.getLocale())
locale.setLocale("fr-FR");
console.log("::::II:::::" + locale.getLocale())
}
} ]);

@abdheshkumar what version of angular are you running? Also, can you create a plunker/jsfiddle to reproduce the problem, it will be easier for me to debug there.

Has this been fixed yet?  When i use your router fix i get the following error
Error: [$rootScope:inprog]


Give move context in Violation struct

Currently the violation struct looks like this:
pub struct Violation {
    constraint: String,  // represents the constraint that created this violation
    code: ViolationCode, // represents an error code that indicates the type of the violation
    message: String,     // represents the detailed error message for this violation
    violations: Vec<Violation>,
}

It would be great if it also contained line numbers and a snippet of the content of the line that caused the error. Right now it's very difficult to figure out what caused the violation with the current context.

Thanks for reaching out about this @backesj!
As per the current implementation, ion-schema-rust doesn't have Display implementation for Violation. I am working on a Display for Violation which could help make it more readable. It might not include line numbers at the moment but maybe a nested Struct with all the violations found during the validation process along with the type name that caused the violation would helpful enough to point at the root cause.
I have created this issue for reference: https://github.com/amzn/ion-schema-rust/issues/105
Let me know if there was some specific use case you were looking for to add enough context to violation Display implementation.

Thanks for the response! Mostly I think it would be valuable to provide line numbers and the content of the line so that users can more quickly identify the source of validation errors. This isn't in the formatted display currently, and it can be challenging to quickly locate the problem for violations that occur deep in an ION structure.


Update configure-local-policy-overrides-microsoft-defender-antivirus.md

Adding instructions for disabling local admin merge via MEM.

Docs Build status updates of commit 986f8f3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:warning:Warning

Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md

Line 99, Column 11: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/mem/endpoint-manager-overview' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 99, Column 11: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/mem/endpoint-manager-overview' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bca527b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit dadae8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 36956a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0b617cd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0b617cd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/configure-local-policy-overrides-microsoft-defender-antivirus.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


RBAC Example

hi man,
beego-reg is basicly, power full, awesome, of web application with golang, I think it will more prefectly if it include rbac modules.
can you give me please, simple example using RBAC like beego-admin in beego-reg?
sorry im newbe with this framework,
thanks
sciefylab

Hey sir!
Glad to hear you found the code somewhat useful. Unfortunately I'm really
swamped with personal stuff at the moment.
Cheers,
Tal
On Mar 14, 2018 11:55 PM, "SciefyLab"<EMAIL_ADDRESS>wrote:

hi man,
beego-reg is basicly, power full, awesome, of web application with golang,
I think it will more prefectly if it include rbac modules.
can you give me please, simple example using RBAC like beego-admin in
beego-reg?
sorry im newbe with this framework,
thanks
sciefylab
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TalLannder/beego-ureg/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADzXvEJr5jYxTA51z9iPQOA3dtA-OoMlks5teeY1gaJpZM4SrhQP
.


Dropdown updates

Description

Add icons to Theme and Language dropdowns
Remove BG label from BackgroundSelect
Make border-widths 2px when BG & Settings are open

@raboid should the DropdownIcon's open and close the dropdown? I'm fine either way, but if they aren't supposed to, then let's remove the cursor: pointer from them