ERROR in 1.client-bundle.60090d500a0bfc6d85e5.js from UglifyJs Unexpected token: punc (() [1.client-bundle.60090d500a0bfc6d85e5.js:1022,10]

ERROR in 1.client-bundle.60090d500a0bfc6d85e5.js from UglifyJs
Unexpected token: punc (() [1.client-bundle.60090d500a0bfc6d85e5.js:1022,10]

I'm going to do some testing tomorrow and update the dependencies, okay?
Thanks @wsdo

@wsdo
vue-disqus have no dependencies.
This problem is with Uglify.js, please update the package or remove the node_modules/ directory and install again.

Did removing the node_modules directory work? I tried that still get the same error and updating Uglify.js doesn't help

@BenjaminWaye
Import the component to see if it works
import VueDisqus from 'vue-disqus/vue-disqus.vue'

This dependency was not found:

vue-disqus/vue-disqus.vue in ./src/main.js
To install it, you can run: npm install --save vue-disqus/vue-disqus.vue

I have run npm install vue-disqus

@BenjaminWaye
Sorrry, try it import VueDisqus from 'vue-disqus/dist/vue-disqus.vue'

Now build goes fine however get this error instead [Vue warn]: Unknown custom element: , so the vue-disqus is not loading.
Npm run dev runs fine, it is odd that only npm run build doesn't work


[feat] add cairo-wasm support for Shinigami frontend

Cairo wasm allows users to run Cairo program directly on the client. Starklings app leverages this to run starklings in browser which improves the application performance. Currently working on creating a similar solution for Shinigami
References
wasm-cairo
starklings-app

Hi @okhaimie-dev please can I work on this issue

@okhaimie-dev  can i work in this

OK I am here now.
Cause Shinigami using cairo 2.8.2 now, I need to update WASM-Cairo to this version first.

A few things contributors need to know before attempting this project

cairo-wasm only compiles single file cairo code
cairo-wasm library is being updated to match shinigami's current version
writing a script to merge all shinigami cairo code into one file and manually fixing this is possible... but is challenging when you factor in that shinigami uses external packages such as ripemd160, sha1 and cairo_test.
compiling opcodes on the frontend using a library such as bitcoinjs-lib is not adviced as opcodes such as OP_DATA_X (x being the number of the opcode) is not supported
running shinigami on the backend is super slow and requires lots of server resources per request

Thank you for posting your findings! I'll mark this as blocked for now since cairo-wasm only compiles for single file cairo code.

Thank you for posting your findings! I'll mark this as blocked for now since cairo-wasm only compiles for single file cairo code.
For contributors:
WASM-Cairo needs this feature of WASM to support multi files. https://github.com/WebAssembly/wasi-filesystem?tab=readme-ov-file

I am also finding another way for some large Cairo project like Shinigami for this.
@b-j-roberts Do we need change Shinigami's Cairo code frequently on the frontend?

@b-j-roberts @okhaimie-dev @supreme2580
Hey guys! WASM-Cairo now support Cairo v2.8.2.
https://github.com/cryptonerdcn/wasm-cairo/releases/tag/v0.10.2


cannot use find method if id is a string

I have a table with an id that is a string (I know it's bad for performance, but I prefer id to be url friendly).
However, using the Model's find method, throws an error if the id is a string.
So this doesn't work:
Model.find(strId)

But this does:
Database.from('table').where('id', strId).first()

Nvm, it's working now, not sure why it wasn't before


Add to OpenVSX registry

Hi, I'm unable to install the vscode-rails extension in the Onivim2 editor because this extension is a dependency but is not present on the OpenVSX registry. It would be great if it were to be added!

I re run workflow action , it says it's already published , I checked the https://open-vsx.org/extension/bung87/vscode-gemfile, it shows "Published 11 months ago", Can you confirm if there's problem?

Sorry, it looks like there is not a problem on your side. Must be some problem with the editor I'm using not properly fetching the dependencies of the Rails vscode extension. Sorry to waste your time.


#5260 Sum of LL added

Have you read the Contributing Guidelines on Pull Requests?
Yes
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#5260

@ankitaggarwal23 created another PR for the same.
Kindly merge this PR.


Implement a way for student contacts to join a group

This block is open for debate, but something like this will need to be implemented

this is similar work as completed in,

https://github.com/albert118/Studio-Management-System/issues/80

something like this will need to be implemented on the UI,


add serviceaccount annotations to allow IRSA in the helm chart

Description
This allows us to add annotations to the serviceaccount in the helm chart, which will allow people to enable IRSA for EKS, which will let things like cosign be able to access ECR or use KMS keys or other activities that require IAM permissions.
Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

closing in favor of #1126


Trying to page.open, but no joy

Using phridge to run Phantom to login into a site and retrieve an attachment. This is done inside the SailsJS framework.
Running
sails.phridge
  .spawn()
  .then(function(phantom) {

    return phantom.run(function() {
      console.log('run');
      page = webpage.create();
      page.open('https://commercialservices.freshservice.com/support/login', function(status) {
        console.log('hi');
        page.render('test.png');
      });
      console.log('run');
    });

  })
  .catch(function(e) {
    sails.log.info('catch');
    sails.log.error(e);
  })
  .finally(function(f) {
    sails.log.info('finally');
    sails.log.info(f);
  });

does not result in a console.log() or a page.render() inside the page.open(). However, I do see the two console.log('run') lines output to the screen. Running the same block of code inside page.open() directly in PhantomJS does produce a PNG. Am I doing something wrong?
PS: FWIW, the sails.phridge... line is equivalent to a require('phridge')

Close?

Yes. I have moved on to another approach, and so if I somehow circle back to this, I'll reopen or create a new ticket. Sorry for the inconvenience.

No worries. :+1:


added steps for granting read permission to space

This PR addresses two issues:

In order for Gaia Hub to work properly it must be able to request a file from its datastore. If the file doesn't exist, the hub expects the data store to return a 404 error. With digital ocean spaces, if File Listing isn't enabled, it will return a 403 error.

The data store must have a public read policy enabled.

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

@reedrosenbluth I need to make changes to add this file to the navigation and reveal it.  So, I'm going to carry and close your PR with another PR. When my PR merges, it will auto close yours.


Fix additional troublesome Windows characters

I found some additional files that are troublesome for Windows and have made the fixes.

Excellent, thanks!


include android

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#993  👈
#992
#991
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @shakyShane and the rest of your teammates on  Graphite


Fix satellite orbit plot on miller projection map

Overview
plot_satellite_orbit_on_miller,py might have bug
Issue

#405

Details
N/A
Validation results

Scope of influence
eg. The behavior of XX will be change.
Supplement
Write additional comments if you need.
Note

If there are related Projects, tie them together.
Assignees should be set if possible.
Reviewers should be set if possible.
Set priority label if possible.

修正しました。よろしくお願いします。


Add option for the optimistic locking in nexrender-server when using Redis

Is your feature request related to a problem? Please describe.
We are trying to run multiple instances of the nexrender-server in order to have more performance and stability overall. We are splitting those instances in different availability zones, to avoid failure in case a zone is down. However, we have noticed that in such setup, some jobs can be picked up by two workers. In general, this is not a huge issue at the moment as both would actually render the same job and give the same output, but it would be nice to overcome this and always have one worker picking up.
Describe the solution you'd like
In order to do this, we should implement a optimistic locking when Redis is used. This way only one worker would be able to pick up a job. More info: https://redis.io/docs/manual/transactions/#optimistic-locking-using-check-and-set
Describe alternatives you've considered
None.
Additional context
I would be up in contributing this. I would also enable it on the environment variable only, and use current behavior as the default one. In case when the check and set fails, I would suggest that the pickup responds in the same way as if there is no available job, or re-try.

Hey @ivansenic 👋
That sounds amazing. I did not see a possibility of using multiple nexrender-servers in combination.
Any contribution is highly welcomed, especially something complex like that!

Great.. I'll try to post the PR as soon as I find some free time..


Don't mutex on windows and windows-sys

While the doc comments in lib.rs describe the issue already, a brief version of it might help get someone here to suggest ideas.
Currently when two crates define the same type with the same content and even the same repr, they're considered different types by the compiler. This means that a lot of stuff that's present in both windows-rs and windows-sys would either need to be implemented twice, or we'll have to make them mutually exclusive so it's possible to import either based on the selected dep and then implement it for that one type. Perhaps a macro isn't a bad idea, but I don't feel comfortable writing complex arcane metacode at the level of the stdlib (they do that for integer operations, among others).

A macro is unlikely to work here, as some types are just fundamentally different (for instance PWSTR has a wrapping struct in windows-rs, while it's a simple typedef in windows-sys-rs). If we want to support both, we'd need to reimplement things twice. And at that point, we may as well just make it two separate crates.
IMO, this crate should just focus on windows-rs, and ignore windows-sys. I suspect the users of windows-sys won't want to import an additional crate anyways, as the goal of that crate is to have minimal dependencies/build time, at the cost of ergonomics.

It should also be noted that cargo-features are additive. Thus, if one crate enables winrs and another crate enables winsys, you'll end up with both of them enabled (docs).

Oh god, I know.this but didn't realize one could end up in a situation where different crates would depend on this...
I might also just look into feature gating impls in a way that allows windows-sys users to grab utility functions that don't have direct deps - that would also allow more minimal access for windows-rs consumers

Closed as part of #6 when the compile error and dep were removed. windows-sys support might be a future goal, but for now it's not.
Thanks for the input roblabla and Nerixyz!


chore: configure codesandbox to buidl storybook

SUMMARY:
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[ ] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c36836b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #293   +/-   ##
=======================================
  Coverage        ?   88.38%
=======================================
  Files           ?      385
  Lines           ?    10421
  Branches        ?     3523
=======================================
  Hits            ?     9211
  Misses          ?     1190
  Partials        ?       20

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix this single/lone read-only string. And „The project website URL points to a non-existing location:“

I already had mentioned this, but seems like no one paid any attention to it, this single string is read-only.

Other languages do have it translated before it was set like that, others like Russian and Lithuanian don't.

„Weblate“ counts it as another category.

Such strings should not appear in a translation project if they are not supposed to be translated, but in this case, it should be translated, as in Lithuanian (žr. 2019-11-07 Valstybinės lietuvių kalbos komisijos nutarimą Nr. N-8 (178) „Dėl Lietuvių kalbos skyrybos taisyklių“) it would be:
„Webroot“
And Russian:
«Webroot»

P.S. Red !

Only a couple of components seem to be affected.

I don't know about weblate, but a manual PR here would work :)

I don't know about weblate, but a manual PR here would work :)

A simple „flag“ removal/edit to editable of the string would work.

This can only be done by maintainers.

I removed the read-only flag in weblate from that string in Lithuanian, but no idea how that flag has surfaced there.


Release: v0.9.0

This issue template is used to track the rollout of a new KEDA HTTP add-on version.
For the full release process, we recommend reading this document.
Required items

[ ] https://github.com/kedacore/http-add-on/issues/1213
[ ] bump KEDA v2.16.1

Timeline
We aim to release this release in the week of December, 23. - 29, right after KEDA v2.16.1 is released.
Progress

[x] Add the new version to GitHub Bug report template
[x] Create KEDA release
[x] Prepare & ship Helm chart
[x] Publish on Artifact Hub (repo)
[x] Provide update in Slack
[x] Tweet about new release

Slack -> https://kubernetes.slack.com/archives/CKZJ36A5D/p1735322501056629
Twitter -> https://x.com/kedaorg/status/1872705514817372410


[TASK] Establish SMTP connection once and use the same connection for all email notifications

Description
Currently email notification for new contributions and new reviewers establishes new SMTP connection each time in adminutils.send_email() method. The SMTP connection is established during POST request which increases the time to POST. Change this to have one SMTP connection that could then be utilized for all email notification functionalities.
Make sure that the connection is active before sendmail() method is called.
Acceptance Criteria
SMTP connection (with login) established once and used for all notifications.

Closing as PR was closed.


parse Kindle Touch metadata correctly

My Kindle Touch appears to generate slightly different clipping metadata strings:

e.g. "Your Highlight" instead of "Highlight" (whether there's some feature that would put something other than "Your" there, I don't know; haven't encountered anything else yet)

"Location" not "Loc."

(As an aside, it appears that location numbers previously written '865-6' are written as '865-866' by the Touch.)

I've updated the gem so that version 0.2.1 includes your changes. Thanks!


Decide if db.GetTasks is needed

We currently have a method:
// GetTasks gets tasks based on query
func (db *GormDB) GetTasks(query *pb.Task) ([]*pb.Task, error) {
	var tasks []*pb.Task
	err := db.conn.Find(&tasks, query).Error
	if err != nil {
		return nil, err
	}
	if len(tasks) == 0 {
		return tasks, gorm.ErrRecordNotFound
	}
	return tasks, err
}

This is currently used only in tests; consider whether or not to keep this.

When handeling late enrolling groups this method will become necessary, as we need it to get all the tasks of a course, and create issues based on them.


fix(Notification): replace const enum

Description:
This PR replaces the NotiticationKind const enum that was introduced in 6.4.0 with a string-literal union type. The latter is type-safe and compatible with single-module transpilation. Const enums are not, as the transpiler cannot know what should be used as a replacement for a reference to a const enum without using type information.
The NotiticationKind const enum is a problem for any projects using the --isolatedModules flag or for projects using the Babel TypeScript transform - which means any TypeScript projects generated with create-react-app.
Related issue (if exists): #4538

BTW, I attempted to add a transpile script to reproduce the error - and guard against it in the future - but I was unable to get it to work, as TypeScript seems to only perform the check that effects that error for packages consumed from node_modules. Consuming them from dist effects a truckload of other errors about re-exports also not being compatible with --isolatedModules.

Pull Request Test Coverage Report for Build 8086

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 96.803%

Totals

Change from base Build 8066:
-0.002%

Covered Lines:
5238

Relevant Lines:
5411

💛  - Coveralls

This is great, and will allow us to move to v6.4. Are there any concerns about merging this in?

Unfortunately, this is a breaking change.  Anyone using the NotificationKind enum like NotificationKind.NEXT will be broken with this change.

I think if we just leave the enum where it is, and convert everything else to just use 'N'|'E'|'C' we should be able to resolve the bug.
We can then @deprecate NotificationKind.

@benlesh Unfortunately, this has to be a breaking change. It's the export that is the problem - regardless of whether or not NotificationKind is used:
node_modules/rxjs/internal/Notification.d.ts:3:27 - error TS1209: Ambient const enums are not allowed when the '--isolatedModules' flag is provided.
3 export declare const enum NotificationKind {

The introduction of the const enum was a breaking change for many people - myself included - so I see no reason the fix cannot also be a breaking change.

Maybe just const enum to enum?

@benlesh I've updated the PR to use a literal union in the Notification signature with NotificationKind being kept as a non-const enum. A const enum cannot be exported without effecting an error in situations in which isolated modules are used.
I've added a deprecation - with a message - to the enum. In the next major version, the enum could be ditched and NotificationKind could be declared as a type alias for the literal union.

It looks like this is actually passing, the Travis error seems to be unrelated. Any chance we can get this in now?


Android: Fix sensor controlled app orientation

Make our flag landscape and portrait same than in android api.
We should think why user controlled mode is not working
because that is maybe better in user perspective but
this is fine for now.
BUG=XWALK-5199

one nitpick: s/censor/sensor/

Can you make the first line of the commit msg go something like:
Android: Fix sensor controlled app orientation

Done


Change angle brackets to braces

Fixes #8013

services.Add{ServiceName} vs. services.Add{Service-Name} ... I think I'd be more confused by the latter. I don't usually separate words in placeholders with a hyphen. I use underscores much more frequently ...

services.Add{Service_Name}


Add missing dependencies + add include clause for galaxy integration

Hi,
Here's a little PR to include jbrowse conf from a "data/datasets.conf" that can be created by a galaxy tool I will add to IUC today hopefully.
While doing it, I had problems of missing dependencies, so I added them.

Interesting. Does this fail if that file is missing?

No it won't fail, it just ignores the missing file
This whole stuff looks a lot like what we do for apollo, I think you can get the same result with public data in apollo. But apollo takes much more resources, so if you only need a ro browser, jbrowse can be enough.


Windows 7 cannot find ssh command.

This issue is fairly simple to resolve.
We just need to have the user install Git for Windows from https://gitforwindows.org/
Then, we can add bash.exe as a shell option for Windows users when it opens the PTY.

Needs more robust detection of best possible shell automatically.
Code to do this is here:
https://github.com/openpilot-community/workbench/blob/14acd121af4bb8bf104c81fefb91ad4cde607d4a/app/background/server.js#L54


fix serialport.list missing ports when ports have same id

@rwaldron ping

fixed in #487 which allows the operator to select how to handle this.


Fixing the mvn based image build

Also switched to JDK 11 to match our current practice.

(btw I'm planning to automate the maven releasse/image build in the same way we have for kas-fleetshard etc)

(btw I'm planning to automate the maven releasse/image build in the same way we have for kas-fleetshard etc)

Sounds good.  Do you plan to do the same for the java build artifacts?

(btw I'm planning to automate the maven releasse/image build in the same way we have for kas-fleetshard etc)

Sounds good. Do you plan to do the same for the java build artifacts?

Yes..


KDE Plasma Style (aka KStyle)

Hi oclero!
It's possible to build qlementine as a KStyle for KDE Plasma 6? If yes, how can I achieve this?

No it's not possible currently, and I don't plan to do this.


Suppress x-cadl-name output in OpenAPI emitters by default, turn on via option

I received some feedback from the API Stewardship Board in August that the x-cadl-name metadata we attach to schemas in our OpenAPI emitter output is noisy and should not be added by default.  They see the value in having it for debugging purposes, though, so we should allow it to be enabled via an option.

pri: 2
est: 2

Describe the breaking change
The x-cadl-name extension, currently emitted by typespec-autorest for inline schema, will be omitted by default
Back-compat design to avoid immediate breaking
The feature can be turned on through TypeSpec emitter configuration
Link to detailed breaking design
Impact
This change affecting:

[x ] ARM
[x] Data-plane
[x] Existing specs: approx. # of service/spec impacted ~16: https://github.com/search?q=repo%3AAzure%2Fazure-rest-api-specs x-typespec-name&type=code
[ ] New specs
[ ] Emitters
[ ] SDK

Targeted Sprint
Sprint: July 2023
Additional information
Add any other information about the change including fixes for here.

@timotheeguerin can you please update the doc with a better example how to turn it on back? which emitter with which option?

I see, so it must be include-x-typespec-name.

yes indeed include-x-typespec-name: inline, the release notes mention the new entry
https://azure.github.io/typespec-azure/docs/release-notes/release-2023-07-11#azure-toolstypespec-autorest-emitter

Sorry, my bad, somehow incorrectly copy & pasted or overlooked the right name. Thanks!


Not possible to set state of checkbox

Adding/removing the attribute "checked" has no effect on the state of the checkbox? Is it possible to reset a checkbox?

Why not set 'ng-model' property true or false? "checked" attribute don't watched by defaulty in angular.js. Please look at https://docs.angularjs.org/api/ng/input/input[checkbox] page for checkbox.

Changing the ng-model property does not change the state of the checkbox in my interface.
<input type="checkbox" ng-model="buy_order.create_open_order" data-label-text="Create order" data-true-label="Yes" data-false-label="No" data-nq-switch-button="" data-theme="primary">

Setting buy_order.create_open_order to true or false, doesnt change the checkbox.

OK, As I see, there var a scope problem.
Currently you can solve by initializing ng-model before or parent element.
Something like this.
<div class="ng-model" ng-init="buy_order.create_open_order = buy_order.create_open_order || false">
<input type="checkbox" ng-model="buy_order.create_open_order" data-label-text="Create order" data-true-label="Yes" data-false-label="No" data-nq-switch-button="" data-theme="primary">
</div>

Bu this way you ng-model propery would be available in parent scope.

Unfortunately this has no effect for me neither. No matter if I do
ng-init="buy_order.create_open_order = false"
or
ng-init="buy_order.create_open_order = true"

The checkbox is always not checked.

Ok. Sorry, Now I find to reason.
If you can edit the code, search for  nqSwitchButton directive,
then find scope.$watch(attr.ngModel
then replace it with scope.$parent.$watch(attr.ngModel
no other thing required to do.

Cool, thanks, it works now with the parent div. However I still cannot set the checkbox state in the angular controller. As changing the scope variable $scope.buy_order.create_open_order doen't have any effect on the state.

How do you change $scope.buy_order.create_open_order
if state is a dom event you shoul change it in
$scope.$apply(function(){
    $scope.buy_order.create_open_order=...
})

thank you. all works now!


[Bug]: Configured patterns do not follow symlinks

What happened?
I am using GNU Stow to manage the pattern files for fabric. Stow creates a symlink in the config dir to the location of my patterns repo.
It can be reproduced by:

Create $HOME/.config/fabric/patterns/test/system.md
Symlink from other dir to $HOME/.config/fabric/patterns/test-2/system.md

Now running fabric -l will give test as a pattern, but not test-2.
Expected behavior: Symlinks should be followed when collecting the available patterns
Version check

[X] Yes I was.

Relevant log output
No response
Relevant screenshots (optional)
No response

@eugeis, I'm happy to look at this.
I think we might need a common utils go, to handle file, path, absolute path, relative path...

@mattjoyce great, thank you.

Fixed in PR
https://github.com/danielmiessler/fabric/pull/1181


Add changelog for 2.12.0 release

What does this PR do?
2.12.0 release was missing CHANGELOG file update, fixing that.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.03%. Comparing base (82871ea) to head (25e4f1f).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2179      +/-   ##
===========================================
+ Coverage    70.01%   70.03%   +0.03%
===========================================
  Files          726      726
  Lines        26997    26998       +1
  Branches      4525     4524       -1
===========================================
+ Hits         18900    18908       +8
+ Misses        6829     6814      -15
- Partials      1268     1276       +8

see 29 files with indirect coverage changes


@bentley/imodeljs-native 3.1.5

No seriously for real this time.

@kabentley macOS still crashing.

The most recent change in the addon fixed all but one of the macOS crashes; core-full-stack-tests still crashes in chrome because process.exit() bypasses our shutdown logic.
@wgoehrig is investigating a fix to ChromeTestRunner.

@pmconne I just pushed a fix.  All full-stack tests that start HTTP servers now need to supply certa with a "cleanup" callback, otherwise the test process will hang.
I've updated the core and rpc full-stack tests, but does anyone know of any other certa users that might be affected? @calebmshafer @aruniverse

@wgoehrig AFAIK not too many people use the backendInitModule outside of rpc tests in other repos. We'll need to follow up and fix them there. I'll make a note to do that.
The Civil team is the only one that I could think would be doing something similar.

does anyone know of any other certa users that might be affected?

@wgoehrig AFAIK not too many people use the backendInitModule outside of rpc tests in other repos. We'll need to follow up and fix them there. I'll make a note to do that.

I think we're good for all code in GH, need to check ADO
https://cs.github.com/?scope=org%3AiTwin&scopeName=iTwin&q=backendInitModule

full-stack-tests/ecschema-rpc-interface, @wgoehrig could you check this?

@aruniverse Bill and I discussed and we should be fine here given the way it's returned.

@wgoehrig all builds have failed with core-full-stack-tests due either to handle leak or timeout.


Change the "Master" nomenclature

Is your feature request related to a problem? Please describe.
OpenSearch repository is going to replace the terminology "master"with "cluster manager".
issue: https://github.com/opensearch-project/OpenSearch/issues/472#issuecomment-960298995, with the plan for its terminology replacement.
Although the existing usages with "master" will be supported in OpenSearch version 2.x to keep the backwards compatibility, please prepare for the nomenclature change in advance, and replace all the usages with "master" terminology in the code base.
All the OpenSearch REST APIs and settings that contain "master" terminology will be deprecated in 2.0, and alternative usages will be added.
Describe the solution you'd like
Replace the terminology "master" with "cluster manager".
When being compatible with OpenSearch 2.0:

Replace "master" in code comment and internal variable / class name.
Specifically, internal class name https://github.com/opensearch-project/anomaly-detection/blob/<IP_ADDRESS>/src/main/java/org/opensearch/ad/cluster/MasterEventListener.java

When being compatible with OpenSearch 3.0:

Replace "master" in the reference to OpenSearch Java API

Describe alternatives you've considered
None.
Additional context
None.

I began this process by replacing any variable, logs or comments with the use of "master" to "clusterManager" (https://github.com/opensearch-project/anomaly-detection/pull/504). This process will continue when Opensearch Core makes further changes to classes and methods that AD is using with this terminology.

For 2.0, at a minimum please ensure that the plugin is not calling any deprecated APIs in core or another plugin, and confirm below, then remove the 2.0.0 label. If you have time, do the complete deprecation as described in this issue.

@dblock Any leftover uses of "master" are from classes in Opensearch that haven't been changed yet. To my understanding this isn't something that will be done until 3.0. Anything that could have been removed for 2.0 has already been removed with the mentioned PR above. For example (https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/cluster/LocalNodeMasterListener.java) and other classes, folder and etc. Any comments or logs have been changed in AD.

I think any leftovers will be done in 2.x, so I labeled this for v2.1.0.

Hi @dblock, I confirmed that @amitgalitz is correct that usages of "master" in OpenSearch Java APIs (public field, method, class, and package names) will be replaced in OpenSearch version 3.0, which is tracked in issue https://github.com/opensearch-project/OpenSearch/issues/1684, and I haven't started yet. The plugins can only replace these usages when the public Java APIs are renamed in OpenSearch repository, and I will keep update on this progress.

@amitgalitz is there any changes left on the AD side?

Java APIs completed and released on 2.2 except for AcknowledgedResponse.
@amitgalitz Could you please complete the remaining part from your end? let's target 2.3 for all usages except for AcknowledgedResponse (there will be separate issue to track it).

All are resolved in #644 besides the AcknowledgedResponse change which won't be available until 3.0.
@anasalkouz if there will be a separate issue for that, should we close this issue now?

Please reopen if necessary.


In november 2019, i created  wdca policy

in the month of November 2019, i utilized the benefits of WDAC policy by  created new policy with fallback hash parameter and then i converted in to sipolicy.bin and then applied to efi partition, afterward i run below commands successfully.
mountvol X: /s
copy %WINDIR%\System32\SecConfig.efi X:\EFI\Microsoft\Boot\SecConfig.efi /Y
bcdedit /create {0cb3b571-2f2e-4343-a879-d86a476d7215} /d "DebugTool" /application osloader
bcdedit /set {0cb3b571-2f2e-4343-a879-d86a476d7215} path "\EFI\Microsoft\Boot\SecConfig.efi"
bcdedit /set {bootmgr} bootsequence {0cb3b571-2f2e-4343-a879-d86a476d7215}
bcdedit /set {0cb3b571-2f2e-4343-a879-d86a476d7215} loadoptions ENABLE-LSA-ISO,ENABLE-VBS
bcdedit /set {0cb3b571-2f2e-4343-a879-d86a476d7215} device partition=X:
bcdedit /set hypervisorlaunchtype off
mountvol X: /d

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

ID: 41c2cb44-269d-3334-019a-726eb5432f1c
Version Independent ID: 251024cc-b818-18c6-a112-4ac0276cb4b1
Content: Manage Windows Defender Credential Guard (Windows 10) - Microsoft 365 Security
Content Source: windows/security/identity-protection/credential-guard/credential-guard-manage.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

Closing via #6575.


Table header overlaps table content when page breaks

<EMAIL_ADDRESS>commented:

Which version of PhantomJS are you using? Tip: run 'phantomjs --version'.
1.8.0 development
What steps will reproduce the problem?

execute phantomjs rasterize.js multi_headers.html table_test.pdf "a4"
2.
3.

What is the expected output? What do you see instead?
Table headers without pagebreaks and headers not overlapping content.
Which operating system are you using?
OSX Mountain Lion & CentOS 6.3
Did you use binary PhantomJS or did you compile it from source?
Compiled from source
Please provide any additional information below.
See attachments
related to issue 880
http://code.google.com/p/phantomjs/issues/detail?id=880

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #927.
:star2:   7 people had starred this issue at the time of migration.

Yikes, this is a bear. My solution is far from ideal, but workarounds tend to be that way. Sometimes you just need to make it works.  The only reliable fix I found was to put the table header in the header for the page itself; using pageSize. So basically I:

created an HTML table that has no  because anything in thead was repeating on each page.
I did, however, put the headings in the HTML, I just did it right in
I return a string in the pageSize header that repeated the table header on each page

With a little styling, it looked pretty good.
 You can still style it differently than other tabular data by hooking into tr:first-of-type and th's. May not work for everyone who stumbles onto this page, but it worked for me.

an additional element at the top of the page - before the table - did solve my issue.

I solved the issue by adding the following class to my table.
.table-fix {
    margin-left: 1px;
}

I "fixed" this by removing the font size css attribute for my 's.

In case it helps anyone, I just "fixed" an occurrence of this by specifying a width and min-height on the first column. We had one table where this never happened and one where it always happened, was able to narrow it down to that. We're using bootstrap, so in our case it was adding class col-sm-1 to the first th in the header.

@chrispbailey thanks.
It works for me too.
But tfoot has the same problem so I do
thead, tfoot {display: table-row-group;}

Thanks @brycepj -- that fix of switching out the table-header worked well for me, too.

@giaosudau's thead, tfoot {display: table-row-group;} worked for me, too. Thanks!

I had the same problem and solved by replacing the blanks by & nbsp;

I had the same problem until I added
 into thead after my actual headings:

            Heading 1	Heading 2	Etc.

For me, removing line-height from body fixed it.


updating to next 13.5 app directory

Current dependencies on/for this PR:

main

PR #2   👈

This comment was auto-generated by Graphite.


Module about PowerToys Run, there is a bug

ℹ Computer information

PowerToys version: v0.25.0
PowerToy Utility:PowerToys Run
Running PowerToys as Admin:Yes
Windows build number: [run "winver"]18363.1198

📝 Provide detailed reproduction steps (if any)
1.Open PowerToys
2.Click the PowerToys Run Module
3. set the Appearance to Dark
4. and when you Use Alt+Space to call the Search Box next time,the Appearance has not changed.
✔ Expected result
the Appearance can be changed automatically.
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Duplicate of #7754


How to add RO volume

Hello,
I just started using UI for docker but i can't find how to add bindings to volumes on local storage when i am creating container. After container was created i set up Bindings and they work. Can someone tell me how i can set up bindings when i am creating container?
Thanks,
Tadas

@bithost have a look at portainer: https://github.com/portainer/portainer
It can be used as a replacement for UIFD and it allows to declare read-only volumes when creating containers.

@deviantony, installed portainer and it looks very nice. Thanks!


fix: alacritty config import -> general.import

This change will silence the following warning for users:
[alacritty_config_derive] Config warning: import has been deprecated; use general.import instead

Thank you!


ILogger for Controller is not correctly resolved

When I add ILogger parameter to regular class Worker it is resolved as instance with context Worker and errors are correctly logged with SourceContext set to Worker.
However, when ILogger is parameter of UserController it is resolved as instance without context. So error messages don't contain SourceContext.
API is .NET Core 2.0.

I've encountered the same problem when trying to use the Autofac serilog integration within an ASP.NET Core 2.1 application - when injecting the ILogger into a Controller, {SourceContext} is blank, but when injecting it into any other class, {SourceContext} works as expected.
2018-10-27 08:56:17.919 INF [] - This is a log line from a controller.
2018-10-27 08:56:17.927 INF [SerilogSample.Test] - This is a log line from a class.

I've put together a basic project that illustrates the problem: https://github.com/alvaromarithompson/serilog-bug-sample

https://www.strathweb.com/2016/03/the-subtle-perils-of-controller-dependency-injection-in-asp-net-core-mvc/#comment-2702995712

The reason for this is something that we already explained. While controller’s constructor dependencies would be resolved by MVC from the IServiceProvider (so in our case an Autofac adapter), the instance of the controller itself (and its disposal too) is created and owned by the framework, not by the container.

The quickest workaround is to AddControllersAsServices in the configure services method


internal/ui: updateFrame is more often called than expected on macOS

Though FPS says 60, updateFrame is called about 120 times per second. Why?

This was just my misunderstanding!


fix for annotation variant set

Creates the default ensembl version variant set needed by variant annotations

Thanks Sarah

Could you remove the "undef" in the return please? You should never return undef, just "return" if there's no value. Thanks.

Thanks @lairdm - that's updated.


write Tutorial document

As #257 I have an idea of providing a Gradle plugin which enables user to create his/her own Visual Inspection Project with template.
Also I want the tutorial documentations of materialstore should have ZIP files which is downloaded from GitHub Releases page. Once a zip is downloaded and unzipped, user would find a complete runnable sample project.
Of course the code in the distributables (zip) must be 100% tested runnable.

at v0.13.0-SNAPSHOT, I started with the :turorial subproject.

I will initiate a new project "materialstore-tutorial" outside the "materialstore".
I will transfer the code developed for this #214 into the new project.

I have created a new project "materialstore-tutorial", have migrated the :tutorial project there.
https://kazurayam.github.io/materialstore-tutorial/
Closing this issue.


⚠ Bofrost.es has degraded performance

In e210871, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1527 ms

Resolved: Bofrost.es performance has improved in 44a1bb5 after 6 minutes.


[BUG] Manual downloading shows Internal error

On the setting page, click the download button to manually download campus events. It shows the Internal error message box.

I tested the manual download of the Illinois Mobile App Master Calendar. I run the events manager form pycharm and docker container. Both cases work in my laptop.


Update Chrome and Firefox extension store descriptions

Note the new use of https for the links and instructions for building from source in particular:

Access privacy-friendly apps that keep you in control of your data and identity.
The Blockstack extension is a full client-side authenticator for accessing apps built with Blockstack Connect without having to create or enter your Secret Key with Blockstack PBC’s hosted authenticator at https://app.blockstack.org/.
The extension will automatically handle all attempts to authenticate such apps instead of redirecting you to the hosted authenticator. There is no need for configuration.
Instructions for building from source are also available at https://www.blockstack.org/install.

cc @zone117x

All set!


Integrate BB docs into main EOEPCA docs

The Data Access BB docs should be integrated with the main EOEPCA docs
Acceptance criteria

[x] Data Access BB discoverable from main EOEPCA Read the Docs (devel branch?)

Done in

https://github.com/EOEPCA/docroot/pull/5


fix noqa issues

Ruff was throwing errors that many noqa directives were unnecessary.
Where applicable, I removed them and fixed the underlying issue.
In other cases, I created file-level ignores

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#194  👈
#193
#192
#191
#190
#189
#188
#187
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @chrisaddy and the rest of your teammates on  Graphite

Merge activity

May 19, 10:11 PM EDT: @chrisaddy started a stack merge that includes this pull request via Graphite.


Statusbar distracts

It would be nice if i could turn off notifications about saving files. Or make it less notable. There is nohing important happens every time i press Ctrl+S

I see 2 notifications every time; one of them is about the CSS file being created... I would prefer just 1 that says everything was successful.
Also, it should be more discrete... I used a similar plugin for Brackets editor, and there was just a little check mark icon in the bottom status bar that let you know whether the compiling was successful or not.

Agreed. Just installed this and saw the big green popup the first time, and assumed it would simply happen once when you create the file for the first time. But now every moment I click save which is non stop as I continue to preview changes the big green box pops up. It should only show it the first time it exports a file and then should only appear if something goes wrong such as a syntax error and can't compile. Since there is no value in seeing it every moment you click save.


Fix for Debian Package

Fixing error for Debian GNU/Linux 7
During install launching: $ debuild binary
Select your operating system below:
1: Linux
2: FreeBSD
3: Solaris
4: Mac OS X
5: OpenBSD
Which is your operating system (1 - 5) ? :
Wrong number.
make[1]: *** [configure_config] Error 1
make[1]: Leaving directory `/usr/local/src/SoftEtherVPN'
make: *** [binary] Error 2
debuild: fatal error at line 1333:
couldn't exec debian/rules:

Thank you so much for your contribution to enrich the SoftEther VPN source code.
Your patch has been merged on the main source-tree of SoftEther VPN.
As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT
Thanks again for your contribution.


Allowing string for comparison operator >,<,>=,<=

DataFilter doesn't allow string for comparison operator >,<,>=,<= by default.
However mysql, mssql and php all allows string comparisons.
ex : 'ball' < 'water' is true
Shouldn't DataFilter allow it too?
class DataFilter extends Model
{

    public $operatorTypes = [
        '<' => [self::TYPE_INTEGER, self::TYPE_FLOAT, self::TYPE_DATETIME, self::TYPE_DATE, self::TYPE_TIME,],
        '>' => [self::TYPE_INTEGER, self::TYPE_FLOAT, self::TYPE_DATETIME, self::TYPE_DATE, self::TYPE_TIME],
        '<=' => [self::TYPE_INTEGER, self::TYPE_FLOAT, self::TYPE_DATETIME, self::TYPE_DATE, self::TYPE_TIME],
        '>=' => [self::TYPE_INTEGER, self::TYPE_FLOAT, self::TYPE_DATETIME, self::TYPE_DATE, self::TYPE_TIME],
        '=' => '*',
        '!=' => '*',
        'IN' => '*',
        'NOT IN' => '*',
        'LIKE' => [self::TYPE_STRING],
    ];

    public function init()
    {
        // Manually adding string to comparison operators
        $this->operatorTypes['<'][] = self::TYPE_STRING;
        $this->operatorTypes['>'][] = self::TYPE_STRING;
        $this->operatorTypes['<='][] = self::TYPE_STRING;
        $this->operatorTypes['>='][] = self::TYPE_STRING;

        parent::init();
    }

What steps will reproduce the problem?
What is the expected result?
What do you get instead?
[{"field":"filter","message":"\"someString\" does not support operator \"lt\"."}]
Additional info

Q
A

Yii version
2.0.14

PHP version

Operating system

Indeed, I was assuming it might be a db dependent feature. But to my knowledge, most modern db and programming languages allows string comparison, >,<,<= and >=.

It is like that because mostly these operands do not make sense with strings if we're talking about data filtering. Property is public so if you need it you can always configure it.

It is like that because mostly these operands do not make sense with strings if we're talking about data filtering.

It makes perfect sense. If I want to find records starting from o to z all I need is to use >=o. I've used this feature multiple times in CRUDs.

OK. Let's fix it then.


Add startup probes to deployments

The startupProbes will help prevent a slow API discovery from sending
the containers into CrashLoopBackOff. On clusters where startupProbes
might not be available (pre k8s 1.20), we have to use an equivalent
initialDelaySeconds parameter.
Refs:

https://github.com/stolostron/backlog/issues/23874

Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/hold wait for https://github.com/open-cluster-management-io/governance-policy-template-sync/pull/117

/unhold

@mprahl actually ready again now, sorry for the delay


Avoid window.print in a microtask

Fixes #7547.
The mozPrintCallback polyfill is not used in Firefox, so if Firefox follows in deprecating window.print and other sync APIs in a microtask, a similar work-around needs to be applied to other window.print calls..

/botio-linux preview

Looks good, thanks!


sensitive_vol_mount Too Permissive

Describe the bug
While looking at the definition of this in k8s_audit_rules.yaml, I noticed it only covers:

/proc
/var/run/docker.sock
/
/etc
/root
/var/run/crio/crio.sock
/home/admin
/var/lib/kubelet
/var/lib/kubelet/pki
/etc/kubernetes
/etc/kubernetes/manifests

One of the ways to trivially bypass this is to mount either of these:

/var
/var/run

and then use these to access the Docker / CRIO daemon.
Wouldn't it better if we added both of these directories to this list?
How to reproduce it
See above
Expected behaviour

Screenshots
See above

Environment
This is applicable to the latest version as of the time fo this writing - https://github.com/falcosecurity/falco/blob/master/rules/k8s_audit_rules.yaml
Additional context

Wdyt @Kaizhe ?

Wdyt @Kaizhe ?

thanks @ossie-git , you're right, and definitely there are more sensitive host paths should be added in the rule. Would you like to create a PR to fix it?

thanks @ossie-git , you're right, and definitely there are more sensitive host paths should be added in the rule. Would you like to create a PR to fix it?

Ok. I'll also add these paths to sensitive_mount in falco_rules.yaml which also lacks these paths and currently looks like this or you would prefer it as a separate PR?
- macro: sensitive_mount
  condition: (container.mount.dest[/proc*] != "N/A" or
              container.mount.dest[/var/run/docker.sock] != "N/A" or
              container.mount.dest[/var/run/crio/crio.sock] != "N/A" or
              container.mount.dest[/var/lib/kubelet] != "N/A" or
              container.mount.dest[/var/lib/kubelet/pki] != "N/A" or
              container.mount.dest[/] != "N/A" or
              container.mount.dest[/home/admin] != "N/A" or
              container.mount.dest[/etc] != "N/A" or
              container.mount.dest[/etc/kubernetes] != "N/A" or
              container.mount.dest[/etc/kubernetes/manifests] != "N/A" or
              container.mount.dest[/root*] != "N/A")

Ok. I'll also add these paths to sensitive_mount in falco_rules.yaml which also lacks these paths and currently looks like this or you would prefer it as a separate PR?
- macro: sensitive_mount
  condition: (container.mount.dest[/proc*] != "N/A" or
              container.mount.dest[/var/run/docker.sock] != "N/A" or
              container.mount.dest[/var/run/crio/crio.sock] != "N/A" or
              container.mount.dest[/var/lib/kubelet] != "N/A" or
              container.mount.dest[/var/lib/kubelet/pki] != "N/A" or
              container.mount.dest[/] != "N/A" or
              container.mount.dest[/home/admin] != "N/A" or
              container.mount.dest[/etc] != "N/A" or
              container.mount.dest[/etc/kubernetes] != "N/A" or
              container.mount.dest[/etc/kubernetes/manifests] != "N/A" or
              container.mount.dest[/root*] != "N/A")

I added a PR. It only touches on sensitive_vol_mount in k8s. I also added the following two directories, in addition to the ones mentioned above:

/etc/kubernetes/pki
/etc/kubernetes/pki/etcd

I added a PR. It only touches on sensitive_vol_mount in k8s. I also added the following two directories, in addition to the ones mentioned above:

/etc/kubernetes/pki
/etc/kubernetes/pki/etcd

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Inconsistency with indention.

Pom files have the wrong indention on the project attributes. Here: https://prnt.sc/uk4ox6
Java files have inconsistency indentions, for example, https://prnt.sc/uk4pr4

I suggest using .editorconfig file from Intellij IDEA.


how to use the input masks??

As the title says,  when I followed the instructions, I didn't meet my expectations。
instructions：
mask_input (np.ndarray): A low resolution mask input to the model, typically  coming from a previous prediction iteration. Has form 1xHxW, where  for SAM, H=W=256.
Can you provide an example？ thanks

我也遇到同样的问题

yes, we need code to create custom mask

Update, I guess they preprocess mask by logits:
def logit(x):
    a = torch.tensor(x)
    return torch.special.logit(a, eps=1e-6).numpy()
def inv_logit(x):
    return 0.5*(1. + np.sign(x)*(2./(1. + np.exp(-np.abs(x))) - 1.))

#load mask
mask = cv2.imread("images/4.png", cv2.IMREAD_GRAYSCALE)
mask = cv2.resize(mask, (256,256))
mask =  mask[None, :, :][None, :, :]
mask_logits = logit(mask)

Update, I guess they preprocess mask by logits:
def logit(x):
    a = torch.tensor(x)
    return torch.special.logit(a, eps=1e-6).numpy()
def inv_logit(x):
    return 0.5*(1. + np.sign(x)*(2./(1. + np.exp(-np.abs(x))) - 1.))

#load mask
mask = cv2.imread("images/4.png", cv2.IMREAD_GRAYSCALE)
mask = cv2.resize(mask, (256,256))
mask =  mask[None, :, :][None, :, :]
mask_logits = logit(mask)

this is my code, I'm not sure if it correct 100%

I tried but failed again. The output mask is nothing.

hello,
I am getting the same outcome, nothing if I input a binary mask insted points or bbox.

hello, I am getting the same outcome, nothing if I input a binary mask insted points or bbox.

Hi,
I guess, it cannot support that the prompt is only mask, but with points.
Maybe the mask is just a secondary role

Any updates on this issue? Did you manage to make it work with a custom mask?
I tried converting my grayscale mask to SAM with this function:
        # Normalize the mask to [0, 1]
        ref_mask_norm = ref_mask / 255.0

        # Add a small constant to avoid extreme values
        epsilon = 1e-7
        ref_mask_norm = np.clip(ref_mask_norm, epsilon, 1 - epsilon)

        # Resize the mask to (256, 256)
        ref_mask_resized = resize(
            ref_mask_norm, (256, 256), mode="reflect", anti_aliasing=True
        )

        # Convert probabilities to logits
        ref_mask_logits = logit(ref_mask_resized)

        # Add an extra dimension to make the shape (1, 256, 256)
        sam_mask = ref_mask_logits[np.newaxis, :, :]

        return sam_mask
        ```

The output is not the one I expected, though - any ideas why?

Thanks a ton!! 💙🖖

Similar thread: https://github.com/facebookresearch/segment-anything/issues/169

This kinda works: https://github.com/facebookresearch/segment-anything/issues/169#issuecomment-1550222757

this might be useful: Specifying a specific object with a box

I think that if it is the output MASK of a certain network, it may be taken out in the layer before the network output as low_mask


cmd/asm: riscv64 assembler claims that<PHONE_NUMBER> does not fit in 32 bits

After some compiler tweaks, I got this complaint out of the RISCV64 assembler:
# cmd/compile/internal/ssa
cmd/compile/internal/ssa/compile.go:429:45: 00038 (/Users/josh/go/tip/src/cmd/compile/internal/ssa/regalloc.go:2555)	MOV	$4294967295, X3: constant<PHONE_NUMBER> too large: immediate does not fit in 32-bits:<PHONE_NUMBER>
cmd/compile/internal/ssa/compile.go:429:45: 00068 (/Users/josh/go/tip/src/cmd/compile/internal/ssa/regalloc.go:2636)	MOV	$4294967295, X3: constant<PHONE_NUMBER> too large: immediate does not fit in 32-bits:<PHONE_NUMBER>

But<PHONE_NUMBER> does fit (barely!) in 32 bits. So either the error message is wrong, or the instruction should get encoded.
(My compiler changes aren't ready to share, but you should be able to reproduce just with assembly.)
cc @4a6f656c

Not only that, the message has bad grammar: s/32-bits/32 bits/
src/cmd/internal/obj/riscv/obj.go:884
The test seems to think you have a signed immediate, in which case the error is correct, if poorly worded.

This issue fixed by https://go-review.googlesource.com/c/go/+/345050, closed


Rename whitelist to allowlist

This PR renames the Whitelist class to Allowlist to make the language more welcoming and inclusive. This follows the IETF RFC on "Terminology, Power and Offensive Language".

Consider added a deprecated Whitelist back because otherwise semVer will demand that this becomes version 2.0.x.
Something like the following should suffice:
package org.jsoup.safety;

/**
 * This class is deprecated and will be removed in version 2. Please change your code to use the class {@link Allowlist} instead.
 */
@Deprecated
public class Whitelist extends Allowlist {
}

@jhy Do you agree that a deprecated Whitelist class should be present still? I'm happy to add it back if that's what's required to get this into a release.

Yes, unfortunately you are correct.  The only way forward I can see is to
add a full-fledged deprecated Whitelist.  We might be able to use
delegation to Allowlist so that the code is not completely duplicated
(e.g. which tags are part of "basic") . Something like the following

@Deprecated
public class Whitelist extends Allowlist {

    private static Whitelist white(Allowlist allowlist) {
        Whitelist cloned = new Whitelist();
        cloned.tagNames = allowlist.tagNames;
        cloned.attributes = allowlist.attributes;
        cloned.enforcedAttributes = allowlist.enforcedAttributes;
        cloned.preserveRelativeLinks = allowlist.preserveRelativeLinks;
        return cloned;
    }

    private Allowlist allow() {
        Allowlist cloned = new Allowlist();
        cloned.tagNames = tagNames;
        cloned.attributes = attributes;
        cloned.enforcedAttributes = enforcedAttributes;
        cloned.preserveRelativeLinks = preserveRelativeLinks;
        return cloned;
    }

    public static Whitelist none() {
        return white(Allowlist.none());
    }

    public static Whitelist simpleText() {
        return white(Allowlist.simpleText());
    }

    public static Whitelist basic() {
        return white(Allowlist.basic());
    }

    public Whitelist addTags(String... tags) {
        return white(allow().addTags(tags));
    }
}

On Tue, Aug 18, 2020 at 10:56 AM Filip Procházka<EMAIL_ADDRESS>wrote:

@fprochazka commented on this pull request.
In src/main/java/org/jsoup/safety/Allowlist.java
https://github.com/jhy/jsoup/pull/1408#discussion_r472379995:

XSS attack vectors. URL attributes are particularly vulnerable and require careful validation. See
http://ha.ckers.org/xss.html for some XSS attack examples.

@author Jonathan Hedley
*/
-public class Whitelist {
+public class Allowlist {
IMHO this is not enough, you also have to have overloaded variants of
methods where you've changed the signature
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jhy/jsoup/pull/1408#discussion_r472379995, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIU5AOHB4DJHQIH7OXHZTDSBK6GLANCNFSM4OICOU3Q
.

I will push an alternative pull request that adds full backwards compatibility (with added test cases).

Pull request #1423 now contains the same but with full backward compatibility

Closing in favour of #1423

Thanks - this is implemented now with #1464


Deal with issue of overtyping of keywords,

I find myself frequently overtyping the keyword, with the result that it goes into the first field.
I think the best fix might be to add a short delay after selection is made and new frame added, and then to clear the keyboard buffer. before continuing.
Add this into AbstractSelector where indicated:

(I will then experiment with the length of delay in milliseconds for effect.)

Superseded by #187


Improving the UI of About Us page

changing background
adding animation and hover effects
changing layout of boxes

add link as well


Add new parameters to support use_annotations

I think this is a breaking change; it refers to parameters that don't exist in the current getModelManifest function, so will require users to update to a version of schematic with the changes proposed in PR 456: https://github.com/Sage-Bionetworks/schematic/pull/456
These changes surface the sheet_url and use_annotations parameters from some of the lower level functions for the shiny app. I found through testing that I needed to surface the sheet_url parameter, otherwise getModelManifest returned a manifest_df and not a manifest_url to the shiny app. I am not 100% sure why that started happening after adding the use_annotations parameter.

I think it would be really swell if use_annotations T/F was surfaced to the end user with a prettyCheckboxInput, but I don't have time at the moment to add and test that.

Ok, so after some further testing, I found that we do need to explicitly define the datasetId parameter if use_annotations = T (without it, the annotations don't actually populate), so suggesting this change.

@sujaypatil96 , done in #173!


fix: Remove dead plugin Pocco81/AbbrevMan.nvim

This plugin no longer exists: https://github.com/Pocco81/AbbrevMan.nvim

I think a better way would be to add a warning (which is currently not possible in a clean way).
Kind of related to #75
Not sure, but until that is implemented, I think its better to leave it, so the user at least knows why its not working, and is not surprised by the fact that the plugin, which used to work before, is just gone.


New Systems are not added in Swarm

Hey,
This playbook is working only for new Setups.
But if i try to add one more system in an existing setup through this script,the system is not added in the swarm.
Check this thing.
Thanks

Which playbook are you using? If it is the swarm.yml playbook and not the swarm-facts.yml playbook, then I did see an issue in the loop which was fixed with this commit: 27ddeab09a0446eac38cf1e8d8a824e6e37b382e.

Ok..I will Check this out.
Thanx


Fix build

looks like moving the pysam(stats) install to conda from pip broke something. which is weird, because doing it the other way broke things on the cluster?
gotta figure out what dependencies are being messed up
update: as a hack, maybe try installing pysam(stats) before the rest of the conda dependencies, then filling in other stuff afterwards?

Looks like pinning Python to 3.6 or 3.7 fixes this. The conda install fails with Python 3.8 and 3.9, but works with 3.6 and 3.7. (3.6 is the lowest we can go with Python because my code uses f-strings, which were introduced in 3.6.)
How does the install fail, specifically? It's hard to tell, but it looks like there are certain C dependencies that later versions of Python require -- but that pysam < 0.16 doesn't work with. I'm attaching the log, for reference.
log.txt
Maybe it's possible to figure out a way around this (if nothing else, by figuring out what to update for pysamstats to get it up to date with newer pysam versions), but for now it's easiest to just pin the Python versions and move on.


installation issue

I installed sql-cli as the tutorial indication at the websitehttps://www.npmjs.com/package/sql-cli. After finished install, I run the command  "mssql -h", the terminal shows information as bellow. Could you give me some suggestions on how to solve this problem thx!

I installed the sql-cli just run command "npm install -g sql-cli"

Please install latest version of node
On Dec 27, 2016 7:59 AM, "Albert_Chen"<EMAIL_ADDRESS>wrote:

I installed sql-cli as the tutorial indication at the website
https://www.npmjs.com/package/sql-cli. After finished install, I run the
command "mssql -h", the terminal shows information as bellow. Could you
give me some suggestions on how to solve this problem thx!
[image: screen shot 2016-12-27 at 11 57 37 pm]
https://cloud.githubusercontent.com/assets/11963227/21503177/4e4d5312-cc90-11e6-904e-00db46f831b1.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hasankhan/sql-cli/issues/29, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRm8EmolOcyVuETl_DCLx5ALfJuMiwsks5rMTXKgaJpZM4LWWy_
.


Subshell executions have wrong number of lines

Running ls-interactive in a subshell causes it to default to a certain number of lines.
Repro

Run cd $(ls-interactive)
Observe behavior

Expected
ls-interactive fits to the terminal even when invoked from within a subshell.

What do you mean “default to a certain number of lines” ?
I'm on Windows and can't reproduce this, neither on NuShell / PowerShell / Batch

If my terminal is 10 lines tall, ls-interactive will be 10 lines tall, but cd $(ls-interactive) will be taller than the terminal, so the cursor can go off screen and the header doesn't show.

Thats very weird since we just write lines to terminal then clear the last lines on refresh
Without more info it would be really hard for me to fix this (since I can't reproduce this on my NuShell terminal)
https://github.com/Araxeus/ls-interactive/blob/f82e0d00c6b9738d7b6c53623f71d01c5e1b052b/src/structs/prompt_renderer.rs#L253-L263
https://github.com/Araxeus/ls-interactive/blob/f82e0d00c6b9738d7b6c53623f71d01c5e1b052b/src/structs/prompt_renderer.rs#L303-L308

I have yet to learn Rust but I have good programming experience, I'll try to clone and setup a dev environment and see if I can whip a PR up.
Disclaimer, I'm not fully taking responsibility for this bugfix and cannot guarantee success :sweat_smile:

Sure, FYI I just set up WSL2 and ran the program in bash. it ran fine

Interesting. I tried investigating issues in my setup as well but couldn't find anything. I'll keep searching in that space as well.

Thank you for being patient!
Unfortunately I couldn't find the time I would need to get around in Rust.
However, I tested an interactive command substitution in Python and can confirm that the issue doesn't occur there. I'm able to get the correct layout with no difference in my environment other than just running ls-interactive.

I've personally tested Bash + Batch + Powershell + Fish + NuShell (Gnome terminal on Ubuntu + Windows Terminal + WSL)
So my guess is that there is some sort of weird conflict in your bash settings

I'm not going to chase this any further I think.
I'll let you figure out if you want to keep this open or close it as could not reproduce.
Thank you for your time!


Button focus while tabbing

There is no visual cue for when one is focused on a button while tabbing through a document view. You can see this on http://twitter.github.com/bootstrap/examples/fluid.html I'm using v2.0.0 in my development and am experiencing the same issue.
This is an accessibility concern for persons who cannot use a mouse.

I've fixed my issue by changing 'bootstrap.js' line 644 from
input:focus, textarea:focus {
to:
input:focus, textarea:focus, button:focus {
Probably not the "correct" solution, but it works for my purposes so far.

You might want to also consider tabindex if accessibility is a concern.

Resolved in 2.0.1-wip with an update to the .btn on focus to specify the color of the outline.


Unexpected results from Circuit computation

Using Yao circuits and translating them to tensor networks in Tenet, in principle we should be able to obtain statevectors performing the contraction after merging the tensor networks related to the circuit and the state ansatz, written as a tensor too. However, for the GHZ and W circuits, involving entangling gates, results are very different:
using Yao
using Tenet

n_qubits = 3
# Create the W state circuit
phase = 2*acos(1/sqrt(3))
circuit_W = chain(
    n_qubits,
    put(1=>Yao.Ry(phase)),
    Yao.control(1, 2=>Yao.H),
    Yao.control(2, 3=>Yao.X),
    Yao.control(1, 2=>Yao.X),
    put(1=>Yao.X)
)

# Create the GHZ state circuit
circuit_GHZ = chain(
    n_qubits,
    put(1=>Yao.H),
    Yao.control(1, 2=>Yao.X),
    Yao.control(2, 3=>Yao.X)
)

# Choose circuit
circuit = circuit_W     #circuit_GHZ

# Statevector from Yao circuit
SV_Yao = zero_state(n_qubits)
apply!(SV_Yao, circuit)

# Statevector from Tenet
psi000 = Tenet.Quantum(Tenet.Product(fill([1, 0], n_qubits))) #|000>
SV_Tenet = reshape(Tenet.contract(merge(Tenet.Quantum(circuit), psi000')), 2^n_qubits)

println(SV_Tenet)
println(statevec(SV_Yao))

Statevectors from Yao are correct. The GHZ statevector from Tenet seems to have the proper amplitudes but on different basis states (the overlap with the |111> state is zero). The W statevector does not have proper amplitudes. This may affect expectation values too (when computing overlaps, for example).

You're calling merge incorrectly: you're doing $\bra{000 \mid H}$ instead of $\ket{H \mid 000}$. Unlike the math/physics notation (i.e. from-right-to-left), merge goes from-left-to-right due to the semantics of Base.merge so you should do:
merge(psi000, Tenet.Quantum(circuit))

Check out that this way both Yao and Tenet agree with the result:
julia> statevec(SV_Yao)
8-element Vector{ComplexF64}:
               -0.0 + 0.0im
 0.5773502691896258 + 0.0im
 0.5773502691896257 + 0.0im
                0.0 + 0.0im
 0.5773502691896257 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im

julia> SV_Tenet = reshape(Tenet.contract(merge(psi000, Quantum(circuit))), 2^n_qubits)
8-element reshape(::Tensor{ComplexF64, 3, Array{ComplexF64, 3}}, 8) with eltype ComplexF64:
                 0.0 + 0.0im
 -0.5773502691896257 + 0.0im
  0.5773502691896258 + 0.0im
                 0.0 + 0.0im
 -0.5773502691896257 + 0.0im
                 0.0 + 0.0im
                 0.0 + 0.0im
                 0.0 + 0.0im

@mofeing In the Tenet result there are negative signs, and in the Yao one there aren't...

@mofeing In the Tenet result there are negative signs, and in the Yao one there aren't...

Ah right.

As a side note, this is not a problem of Yao, since the same problem happens with Quac:
julia> using Yao

julia> n_qubits = 3

julia> circuit_GHZ = chain(n_qubits, put(1=>Yao.H), Yao.control(1, 2=>Yao.X), Yao.control(2, 3=>Yao.X))

julia> SV_Yao = zero_state(n_qubits); apply!(SV_Yao, circuit)

julia> statevec(SV_Yao)
8-element Vector{ComplexF64}:
 0.7071067811865475 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
 0.7071067811865475 + 0.0im

 julia> julia> println(statevec(SV_Yao)' * statevec(ArrayReg(bit"111"))) # expected result = 1/√2
0.7071067811865475 + 0.0im

With Tenet:
julia> using Quac; using Tenet

julia> n_qubits = 3

julia> circuit_GHZ = Circuit(n_qubits); push!(circuit_GHZ, Quac.H(1))

julia> push!(circuit_GHZ, Quac.Control(1, Quac.X(2))); push!(circuit_GHZ, Quac.Control(2, Quac.X(3)))

julia> psi000 = Tenet.Quantum(Tenet.Product(fill([1, 0], n_qubits))) #|000>

julia> psi111 = Tenet.Quantum(Tenet.Product(fill([0, 1], n_qubits))) #|111>

julia> julia> Tenet.contract(merge(psi000, Tenet.Quantum(circuit))) |> r -> reshape(r, 2^n_qubits) # GHZ |000>
8-element reshape(::Tensor{ComplexF64, 3, Array{ComplexF64, 3}}, 8) with eltype ComplexF64:
 0.7071067811865475 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
 0.7071067811865475 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im
                0.0 + 0.0im

julia> SV_Tenet # expected result = 1/√2
0-dimensional Tensor{ComplexF64, 0, Array{ComplexF64, 0}}:
0.0 + 0.0im

Sorry, yes, I thought that was implicit. I started using Yao instead of Quac because of the deprecation you told me about :)

mmm @jofrevalles in your example there seems to be an issue with the ordering of the values, unlike with @amrFRE's example whose issue is the signs.

mmm @jofrevalles in your example there seems to be an issue with the ordering of the values, unlike with @amrFRE's example whose issue is the signs.

Yes, these are different circuits.

Okay, the problem should be solved with PR #212. Maybe I will add some tests there to cover for this potential problem.
By the way, while trying to solve this problem I came across this weird thing, why did we define the Y gate like this?  😂
julia> mat(Yao.Y)
2×2 LuxurySparse.SDPermMatrix{ComplexF64, Int64, Vector{ComplexF64}, Vector{Int64}}:
 0.0+0.0im  0.0-1.0im
 0.0+1.0im  0.0+0.0im

julia> Matrix(Quac.Y())
2×2 Matrix{ComplexF64}:
 0.0+0.0im  -1.0+0.0im
 1.0+0.0im   0.0+0.0im

I am going to fix that in Quac. That is why we had some different results on my previous comment @mofeing

Yes, it seems that it works now. I have tested it for the GHZ and W circuits without issues.
Only a couple of things:

inds_order = find_permutation(merged_circ.sites) should instead be inds_order = find_permutation(merged_circ)
Changes are still on the branch fix/Yao-extension. I mean that something else had to change because running your example on master gives this minus sign weird result again.

Thank you!

inds_order = find_permutation(merged_circ.sites) should instead be inds_order = find_permutation(merged_circ)

Oops, true! I messed up the argument here, thanks!

2. Changes are still on the branch fix/Yao-extension. I mean that something else had to change because running your example on master gives this minus sign weird result again.

Yes, I will wait until @mofeing wakes up and approves this PR. Maybe I will add a small test set but I think it is ready to merge.
Thanks, @amrFRE!


Release 3.0.16

@vijaykiran, thanks for aligning the dependencies in https://github.com/sodadata/soda-core/commit/f36ea6bf154c58102d61ee6d1be785c981476b96.
Would it be okay to cut a new release containing it?

SODA-1350

Release was scheduled for today and is available now: https://github.com/sodadata/soda-core/releases/tag/v3.0.16

Thank you!


[FLINK-12187][Examples] Bug fixes, Use BufferedWriter in a loop instead of FileWriter

What is the purpose of the change
This pull request changes FileWriter to BufferedWriter when the write() method is intensively invoked in loops. This can improve the performance by reducing frequent IO operations.
Brief change log
Wrap the FileWriter with BufferedWriter.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: ( no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: ( no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi @rmetzger , could you please kindly help to review the code change? Thanks.

@flinkbot approve description

@flinkbot approve all


Anomaly score check not evaluating after >4 scans

The expectation is that an anomaly score check begins evaluating after a minimum of four, regular-frequency scans that gathered measurements. However, after multiple regularly-scheduled hourly scans of a dataset, the anomaly score check is still not evaluation.

Why does it not show the "Last scan ended" value? The duration value in the other view attests to the fact that the scan did complete.
The Scan success results are puzzling-- what does the value 11.54 for the Anomaly test indicate?

Ultimately, just can't get the anomaly score to yield expected output and we're not sure why.

@janet-can  is it still an issue ? If so can you please open in JIRA?

I'm not sure-- this one came from Zefr as an issue. I'll check back with Jose and if it's still an issue, I'll create a Jira.


Specialty genes missing ability to filter on e-value, percent identity, etc-beta

I realize that you can do it by clicking on the different column heads, but if one wants to do several different filters based on BLAST hits, this will be hard to accomplish with the new site.

Also, we used to be able to filter on the source...like in the above instance I want to see only those hits coming from Salmonella, my target organisms.  How do I do that with the new site?

This is not implemented yet. will work on this.


[Indexer] Remove unused method from the address indexer

This is for the next release.

Please dont remove getaddressbalances.
Why?

Please dont remove getaddressbalances.
Why?

Isn't it obvious from the heading of the PR?

Isn't it obvious from the heading of the PR?

It was more of a rhetorical question
This method will be 100% needed for exchanges integration. It is exposed on API. No need to delete it just because none of our own code uses it.

Isn't it obvious from the heading of the PR?

It was more of a rhetorical question
This method will be 100% needed for exchanges integration. It is exposed on API. No need to delete it just because none of our own code uses it.

Why didn't you just explain that in the first place?
I will close this PR and add a comment that explains this.

Why didn't you just explain that in the first place?

Sorry, I thought I've mentioned it before at some point.

I will close this PR and add a comment that explains this.

Thanks, mate!


Showing wrong result

Hi,
I am using the hashes functionality to get the output but the thing is its showing me wrong result.
In the following link it contain 15 Detections but when I am analyzing this thing by saving the md5 in the text file and executing the munin command it shows me Unknown 0 Result
https://www.virustotal.com/en/file/f789f276c02aad873a7811447827a5f854b295fa18e2fe11e22cb35756c926e7/analysis/
Waiting for your reply.
.
Thanks & Regards
Seantree

Ok Thanks for this I will check it again :)

I cannot run python3 munin.py -i my.ini -f munin-demo.txt this problem. I use API virustotal of my profile
Traceback (most recent call last):
File "munin.py", line 1395, in
VT_PUBLIC_API_KEY = config['DEFAULT']['VT_PUBLIC_API_KEY']
File "/usr/lib/python3.6/configparser.py", line 1233, in getitem
raise KeyError(key)
KeyError: 'VT_PUBLIC_API_KEY'
[E] Config file 'my.ini' not found
[E] No Virustotal API Key set or wrong format
Include your API key in a custom config file and use munin.ini as a template
More info:
https://github.com/Neo23x0/munin#get-the-api-keys-used-by-munin


path intersection speedups and misc fixes

To avoid a division by zero in is_path_simple when the path has duplicate points, the test:
       if (/*approx(normv1,0) || approx(normv2,0) ||*/ approx(v1*v2/normv1/normv2,-1)) 1]  == []
should be read as:
        if ( (normv1>eps) && (normv2>eps) && approx(v1*v2,-normv1*normv2,eps=eps))  1]  == []

Shouldn't the calls to _path_region_intersections in is_region_simple include a parameter closed=true ?

I am not sure what exactly OpenSCAD does in the iteration of s2 but to be sure I would compute the list of segments s2 of the region before the loop of si to avoid recomputing close_region and pair for each path segment.

I think division by zero is harmless.  You get inf which is not -1.  The question is whether polygons with repeated vertices should be counted as simple or not.  Deduplicate gets run on a lot of inputs...so duplicate vertices are maybe harmless?
Yes, closed=true is needed.
That code where s2 is computed in the loop was based on your code---originally just one for loop.  I was wondering a bit about whether that gets recomputed.   I wrote a version that didn't use pair() at all, or close_region() and it was slower, so it probably isn't doing something inefficient.


We need to have different types of roles in the system

Guest: Someone who joins the meetup to spectate.
Presenter: Person who has the ability to present (share screen, be the focused mic and video).
Host: Host of the meetup

As host, I should be able to do everything a presenter can do

Administrator: Administrator of the platform.

I was thinking it could be useful to have a specific role for the camera(s) that capture the in-person audience. It could have the same powers as Guest, but we could make sure all participants always subscribe to it, i.e. it's always visible to everyone, even when other guests are hidden because there are too many (this is related, for example, to running Mux Spaces in manual mode -- see #14).
@montevideo-tech/summer-camp-2023 what do you think?

I like the idea, also  I imagine that a guest can be in person or remote.
If the guest is remote he/she will need to have watch the cameras, othter guests etc... its main focus is the presenter/host
If the guest is inperson, he/she will probabley use a smarthpone or a tablet, and she/he will have more focus on things like the chat, questions

In this case I mean the camera/webcam/stream pointing at the crowd at the in-person event. I think it should actually be a different role, as in how these streams will be handled by the media server and how other participants will see them. For example, we might want to make the video of the in-person crowd always visible in the room, to let everyone get a better feeling of the event.
We might also make it so that the client that's streaming the crows is a stream-only one. In this case, we could say, for example, if LocalParticipant has a role of in-person crowd, don't subscribe to any remote streams. (I'm not sure if this is actually needed, but you get the idea: We use the role to determine how to interact with the room in the media server).
In my opinion, this is not the same as what Front-End experience we want to give to participants depending on how they'll interact with the event. Although there will likely be some overlap, like if you have a role of host you'll be presented with a different view, etc...

After giving a thought on how to implement the structure of tables in order to implement the roles privileges we came up to the solution of having Admin role globally and Host & Presenter roles on rooms level (see #79)


GitLab badges are not working

Are you experiencing an issue with...
shields.io
🐞 Description
It seems that several GitLab badges are not working properly. At least I have detected License and Release.
🔗 Link to the badge
For example, for this GitLab project, the project id is 42779682. The shields shows project not found:

The same applies if you use the project path ogarcia/lesma.
💡 Possible Solution
No response

I do, although to the best of my recollection, any token should do. GitLab declined to give us a token with a higher rate limit, and in fairness I don't rate limiting has been an issue.
Additionally, I don't believe GitLab supports renewing/modifying the expiration date on an existing token so a new one will indeed be needed. I'll try to dig up the credentials (I thought I'd shared them privately in Discord), though anyone of us with a GitLab account should be able to create and provide an alternative token if I'm unable to do this expeditiously.

Thanks.
Just set the new key. GitLab badges should be working now (or at least coming back as things drop out of cache).
I had not focused that we just have a standard PAT for GitLab. I'll make a note of this in the docs when I get a sec.

Glad to hear it. Let me know if we run into an issues with the token. I think I selected all the relevant scopes we'd need but can't say I'm particularly familiar with them


Cypress does not test sitemap links

Where to find the issue
cypress/integration/check_links.js
Describe the issue
The cypress/integration/check_links.js test does not check the sitemap pages:

https://www.coronawarn.app/en/sitemap/
https://www.coronawarn.app/de/sitemap/

Even though there was a link error in the sitemaps, the Cypress test suite showed no error.

See https://github.com/corona-warn-app/cwa-website/issues/2765.

Steps to reproduce
npm run test:open
select check_links.js
note that the sitemap directories /de/sitemap/ and /en/sitemap/ are not listed in the Cypress log
Suggested change
Add the following to pages in cypress/integration/check_links.js

'/de/sitemap/'
'/en/sitemap/'

PR #2768 submitted to resolve this issue

This issue is confirmed resolved.
Tested with npm run test:open
select check_links.js to run this test.
The sitemap directories /de/sitemap/ and /en/sitemap/ are listed in the Cypress log


Aboud badge in the document

I have created a new badge for Jest with the project badges, and I hope it can be accepted to be an official one. Thanks.

Nice! I wonder if lighter (e.g. white) background under the logo wouldn't be better?

cc @cpojer @ericnakagawa

The badge is generated with the official designed logo with SVG formatted, and if you want a lighter white border, I have to say I do need abother source file.

Sorry for the confusion, I mean the background, not the logo color 😅

This is cool!

@thymikee greate idea, and I will make it true soon.

 A badge used for projects using Jest for testing JavaScript
 A badge used for projects using Jest for testing JavaScript

Looks cool! The second one fits nicer in GitHub's light theme imho. Feel free to send a PR adding these to README and maybe under "Join the community" section in https://facebook.github.io/jest/en/help.html? :)

👌


Command "python setup.py egg_info" failed with error code 1

Expected Behavior
Install -r requirements.
Actual Behavior
Error is:
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/vp/048s_4x51jqfq4f73yq253yc0000gn/T/pip-build-uP7IxE/protobuf-to-dict/
Full Output:
`Collecting numpy==1.11.0 (from -r requirements.txt (line 1))
Using cached numpy-1.11.0-cp27-cp27m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
Collecting networkx==1.11 (from -r requirements.txt (line 2))
Using cached networkx-1.11-py2.py3-none-any.whl
Obtaining pgoapi from git+https://github.com/tejado/pgoapi.git@81e786cabf027a1c8fbd1e9a07e1c11aa3d8ee8b#egg=pgoapi (from -r requirements.txt (line 3))
Skipping because already up-to-date.
Collecting geopy==1.11.0 (from -r requirements.txt (line 4))
Using cached geopy-1.11.0-py2.py3-none-any.whl
Collecting protobuf==3.0.0b4 (from -r requirements.txt (line 5))
Using cached protobuf-3.0.0b4-py2.py3-none-any.whl
Requirement already satisfied (use --upgrade to upgrade): requests==2.10.0 in ./lib/python2.7/site-packages (from -r requirements.txt (line 6))
Collecting s2sphere==0.2.4 (from -r requirements.txt (line 7))
Using cached s2sphere-0.2.4-py2.py3-none-any.whl
Collecting gpsoauth==0.3.0 (from -r requirements.txt (line 8))
Using cached gpsoauth-0.3.0.tar.gz
Collecting protobuf-to-dict==0.1.0 (from -r requirements.txt (line 9))
Using cached protobuf-to-dict-0.1.0.tar.gz
Complete output from command python setup.py egg_info:
Download error on https://pypi.python.org/simple/nosexcover/: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590) -- Some packages may not be found!
Couldn't find index page for 'nosexcover' (maybe misspelled?)
Download error on https://pypi.python.org/simple/: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590) -- Some packages may not be found!
No local packages or working download links found for nosexcover
Traceback (most recent call last):
File "", line 1, in
File "/private/var/folders/vp/048s_4x51jqfq4f73yq253yc0000gn/T/pip-build-uP7IxE/protobuf-to-dict/setup.py", line 25, in
'Topic :: Software Development :: Libraries :: Python Modules',
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/core.py", line 111, in setup
_setup_distribution = dist = klass(attrs)
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/setuptools/dist.py", line 348, in init
self.fetch_build_eggs(attrs['setup_requires'])
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/setuptools/dist.py", line 394, in fetch_build_eggs
replace_conflicting=True,
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/pkg_resources/init.py", line 826, in resolve
dist = best[req.key] = env.best_match(req, ws, installer)
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/pkg_resources/init.py", line 1098, in best_match
return self.obtain(req, installer)
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/pkg_resources/init.py", line 1110, in obtain
return installer(requirement)
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/setuptools/dist.py", line 461, in fetch_build_egg
return cmd.easy_install(req)
File "/Users/cameronroland/PokemonGo-Bot/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 658, in easy_install
raise DistutilsError(msg)
distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('nosexcover')
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/vp/048s_4x51jqfq4f73yq253yc0000gn/T/pip-build-uP7IxE/protobuf-to-dict/`
Steps to Reproduce
In dev try to $ pip install -r requirements.txt
Other Information
OS: mac
Git Commit: d42e32491b675ac75619e9e4cacc6b988049e5c2
Python Version: 2.7.10

This ticket solve this for me;
https://github.com/AHAAAAAAA/PokemonGo-Map/issues/1475#issuecomment-234864434

I can't see ticket solve

This ticket solve this for me;

https://github.com/AHAAAAAAA/PokemonGo-Map/issues/1475#issuecomment-234864434

See here*
https://github.com/PokemonGoMap/PokemonGo-Map/issues/200


Add support for passing through flags

Particularly in large Terraform installations (such as if you manage large, multi-thousand-records sized DNS zones), a plain tftarget apply can take many minutes to load.
It would be very appreciated if one could pass through arbitrary flags (e.g. -refresh=false or, per #11, -target) to Terraform to reduce the amount of time needed to run.

@msmuenchen I've just seen #28, which might solve your issue.
Not tested myself, not sure if documented, just seen the linked issue and thought it might help you.
Let us know if it does.


Advanced Item Filter not filtering as it should

Issue Description:
Advanced Item Filter's not working as it should. There are 2 issues in here

[x] damage filter on blacklists allows damaged items
[ ] empty nbt versus null nbt

damage filter "any" on blacklist allowing damaged items

it still inserts damaged items:

I think the setting "blacklist" should block anything not in the list even if any damage is allowed, shouldnt it?
whitelist "match NBT" does not work on stock items:

thats the 3rd chest in my setup here which should receive all non enchanted items. But it can only insert something as soon as I disable "match NBT". (then it inserts unenchanted + enchanted items here). it seems those items seem to have two different nbts: kind of "empty" nbt, and "null"?
Steps to reproduce:

build 3 chests
wire them up with item conduits
add above filters to the 2 output chests
put random items in there (damage, enchantments ...)

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-3.1.161
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: forge-1.10.2-<IP_ADDRESS>2-universal

I now found out: the 2nd issue is also the reason for the first one

let's see.

And yes, for Minecraft "no NBT" and "empty NBT" are two different things.
Also, you seem to have a fundamental misunderstanding here. "I think the setting "blacklist" should block anything not in the list"---that is the definition of a whitelist...

oh there was a "not" to much in my sentence.
Then there's still strange why "empty nbt" and "no nbt" is not the same?

https://coderanch.com/t/501509/java/difference-null-isEmpty#2260859

@Flow86 it's the same reason why items with empty NBT do not stack with items which don't have NBT.

or the same reason why hitting your head on an empty cupboard hurts more than if that cupboard isn't there.
But to be honest, there's no reason why we cannot have special code that treats those two cases the same. I'm on it, but I want to redo the fuzzy filter at the same time. No player understands how that one works---it took me half an hour reading the code to understand it...

Related: Ore dictionary filter stopped working in 1.10.2-3.1.169

Expansion on this issue:
When on white list mode, ignoring Metadata, ignoring nbt, ignoring damage, oredict on: it seems to pull the exact items used to create the filter but nothing else that should be pulled
But you keep all the sethings and switch it to blacklist and it seems to work as it should

Another update: It looks like the behavior of Ore Dictionary mode changed. It still works, but you now have to ignore metadata for it to work correctly. This ends up causing problems when combined with mods like Thermal Expansion and Advanced Rocketry, which use the same item ID for multiple dusts or ingots. Before the change, Ore Dictionary mod used to ignore metadata.


Not visible unicode characters

Over time, I noticed that three folders were created for the game "THE FINALS", which all had the same name at first glance.

I suspected invisible Unicode characters there and I was right. I don't know why "THE FINALS" keeps changing the invisible Unicode characters over time, but that's what causes the different folders.
A possible solution would be either to allow only ASCII characters or at least to filter invisible Unicode characters.

Unicode inspector for folder 1
Unicode inspector for folder 2
Unicode inspector for folder 3

If it helps I also added TM with - [™<>:\\/\"|?*]
However I prefer a cleaner result with simply using - path = string.gsub(path, "%W", "") -- Remove illegal path characters for Windows

This is what I have tested to work great for my use.
path = string.gsub(path, "[^%w -()'+.]", "") -- Remove illegal path characters for Windows
Should only allow clean spaces and remove non unicode weird ones. May play funky with other language game titles however.


Popup close button adding #close hash to URL

Starting with version v1.8.0, clicking the popup close 'x' button adds #close to the URL
Behavior is reproducible on the leaflet.js website in examples with popups such as:
https://leafletjs.com/SlavaUkraini/examples/quick-start/example.html
Previous to v1.8.0 clicking the popup close button prevented #close hash from being added to the URL
Not ideal for Single Page Applications that rely on hash routing.

Can confirm (firefox, linux). Doesn't happen on 1.7.0 (see https://plnkr.co/edit/TFdFTrIxdQXXOccK , make sure to open up the preview in a separate tab).
Doing a git blame of the relevant file points at https://github.com/Leaflet/Leaflet/pull/7794 and https://github.com/Leaflet/Leaflet/pull/6639 like the possible culprits of this regression. Not sure if this is something triggered by the ARIA attributes, or if something slipped by in the tooltip refactor, but doing a preventDefault of the click event seems to work.

While refactoring, DomEvent.stop(e) was removed:

Ah, so the source of the regression is https://github.com/Leaflet/Leaflet/pull/7439 , I see.

Added this to the 1.8.1 milestone, looks like a regression we should fix in the next patch release.

yes i faced this issue yesterday:
Cause : The X button to close the pop up is placed as "<a class="leaflet-popup-close-button" href="#close">×"
so it is redirecting the app
Solutions:
1- remove the Close buttton
const popup = L.popup({
closeButton: true
})
2- catch the event onclose and change href (but it will refresh the page) not recommended
marker.getPopup()?.on('remove', (e) => {
e.target._closeButton.href = this.router.url;
});
3- Change Leaflet version from 1.7.1 to striclty 1.7.0  (saved me)

Change Leaflet version from 1.7.1 to striclty 1.7.0 (saved me)

The problem occurs since Leaflet 1.8.0 so it makes no difference if you use 1.7.1 or 1.7.0

Change Leaflet version from 1.7.1 to striclty 1.7.0 (saved me)

The problem occurs since Leaflet 1.8.0 so it makes no difference if you use 1.7.1 or 1.7.0

I work on an app exhibiting the same behavior on leaflet 1.7.1 and changing to 1.7.0 stopped the behavior. So it does make a difference

Can confirm the error with 1.8.0, is solved when going back to 1.7.0
Closing the popup shouldn't route anywhere, right?


Prerequisites

Please help us with the Prerequisites because for its not compatible with PHP 7.4.9

This library is designed to require at least PHP 8.0, I am not planning to support any older version.

I think we should have support to older version because globally PHP 8.0 is used by only 1%

PHP 8 is used about 40% [0] and PHP 7.4 will not be supported anymore in 7 months and will not receive any security anymore in 9 months [1]. So PHP 7.4 is almost dead, and I will not make any effort to support a PHP version which will be EOL soon. You can just upgrade your version. You are also free to fork the package and change everything to support PHP 7.4.
[0] https://packagist.org/php-statistics
[1] https://www.php.net/supported-versions.php