Refactor API jar generation

This series of commits refines and restructures the implementation of
API member selection and extraction and the subsequent assembly of an
ApiJar based on that content.
These changes are being submitted as a pull request for several reasons,
but primary among them is to prompt a discussion about the future
organization and structure of the Gradle codebase. To this end, please
read the first five commit messages in this PR (particularly the fifth). For
convenience, here they are in order:

f216da1dde85e265d11c899f1ecf3914e5efc78a
17663638b7be27d68731f09cb344fe59b3332ea2
81837af6325f06bcc0e746fa358508a683824a6b
417b36d24174f049304c40a5d0ddb87d93e99915
a37a3ef90bc4b56355c48df56d9aee9e3ac9bf95

Together, these commits make a case for careful package design and
placement, and in particular make a case for the use of package-private
visibility as a mechanism for reducing the surface area of our own
public API. In conjunction with our @Incubating annotation, better
packaging and visibility management can perhaps help us move away from
over-reliance on "internal" package techniques.
Package-private visibility is not a panacea and these commits don't
intend to make that case. Rather they suggest that we treat it as an
important tool in helping us evolve our public API, manage compatibility
concerns, and so on.
The remainder of the commits in this PR are also intended to be read in
order, but in the somewhat more focused context of a code review of the
API jar generation feature itself.
Barring any particular objections, this PR will be merged tomorrow
(Friday) after discussion with the original authors of the code that's
been refactored here. Again, the PR itself exists primarily as a means
of raising awareness about these ideas and a place to discuss them in
context.

I have no strong objection on the renames (in fact, there is a difference between allowed and exported: allowed packages are not necessarily the same as exported ones. Take the example of the java-base module. It is actually surprising that no test fails after removing JavaBaseModule, because it is used in validation).
As for package private classes, I am -1 on this.  For me package privacy is broken. Should have it been inherited by sub-packages, or through class hierarchy (A in package pkg1, B extends A in pkg2, C extends A in pkg1), I would have maybe a different thinking.
But I really like the separation of internal package. It gives a lot of flexibility, and it is actually what is going to be publicized with Jigsaw modules. I am not sure we're not moving in the right direction doing this. In general, I consider that almost all types can be public, and it should be documented when to use or not to use one class. In particular, users might want to use internal classes. I did it in the past, and the fact that it's in a package named "internal" makes you aware that it might be broken in the future, but you are aware. On the other hand, if it had been package private, I wouldn't be able to do it, or I would just have worked around it using a class in the same package. Which is dirty.
Last but not least, @Incubating in Gradle has a tradition of saying "it's an experimental API". Yet, they belong to the public API. I don't remember any class marked as @Incubating which has been removed. On the other hand, I feel ok to remove, rename or move a class in an internal package. And as soon as it moves into a package that doesn't contain internal, to me, it becomes public API.

+1 for moving the task to a "public" package.  We probably want to support users being able to say tasks.withType(ApiJar) { /* configure */ }, which means ApiJar needs to be public.
-1 for moving the supporting classes to a non-internal package.  It makes determining what we consider public a lot harder, even if the classes are package private.  We've been trying to move internal-but-not-in-an-internal-package into internal packages over time.

Thanks, @melix and @big-guy; I'll hold off on a response to see if others would like to weigh in too. In the meantime, I just realized that the original PR description called out the first four commits in the series, but it's actually the fifth that is the most important with regard to the visibility and public API surface area conversation. I've updated the description to reflect, but for convenience, here is the fifth commit: a37a3ef90bc4b56355c48df56d9aee9e3ac9bf95

I'm also -1 on using package privacy for internal classes. I think we give
up more than we gain. I'd be +1 for more consistency with how we structure
our internal packages.
I strongly recommend not making this change without explicit go ahead from
Adam.
On Thu, 5 Nov 2015, 3:51 p.m. Cédric Champeau<EMAIL_ADDRESS>wrote:

I have no strong objection on the renames (in fact, there is a difference
between allowed and exported: allowed packages are not necessarily the same
as exported ones. Take the example of the java-base module. It is actually
surprising that no test fails after removing JavaBaseModule, because it
is used in validation).
As for package private classes, I am -1 on this.  For me package privacy is broken. Should have it been inherited
by sub-packages, or through class hierarchy (A in package pkg1, B extends A
in pkg2, C extends A in pkg1), I would have maybe a different thinking.
But I really like the separation of internal package. It gives a lot of
flexibility, and it is actually what is going to be publicized with Jigsaw
modules. I am not sure we're not moving in the right direction doing this.
In general, I consider that almost all types can be public, and it should
be documented when to use or not to use one class. In particular, users
might want to use internal classes. I did it in the past, and the fact that
it's in a package named "internal" makes you aware that it might be broken
in the future, but you are aware. On the other hand, if it had been package
private, I wouldn't be able to do it, or I would just have worked around it
using a class in the same package. Which is dirty.
Last but not least, @Incubating in Gradle has a tradition of saying "it's
an experimental API". Yet, they belong to the public API. I don't remember
any class marked as @Incubating which has been removed. On the other
hand, I feel ok to remove, rename or move a class in an internal package.
And as soon as it moves into a package that doesn't contain internal, to
me, it becomes public API.
—
Reply to this email directly or view it on GitHub
https://github.com/gradle/gradle/pull/541#issuecomment-154221212.

Based on feedback thus far, I am holding off on merging this today. Will discuss with @adammurdoch as @bigdaz suggests. Further comments are welcome.

In the interest of not leaving this PR hanging out too long, I have added commit 7eca9ab6822f0a0542072c511fff88e06dfa87aa, which introduces an internal package, moves all supporting types for ApiJar into it, and increases their visibility to public.
I'll continue the conversation about structure and visibility with @adammurdoch and others, but in the meantime, it's important that this otherwise substantive refactoring doesn't diverge from the rest of the codebase; I'll merge it shortly.


APT key for official OpenVPN repo has expired

err: W: GPG error: http://build.openvpn.net/debian/openvpn/stable xenial InRelease: The following signatures were invalid: KEYEXPIRED<PHONE_NUMBER>  KEYEXPIRED<PHONE_NUMBER>  KEYEXPIRED<PHONE_NUMBER>
err: W: The repository 'http://build.openvpn.net/debian/openvpn/stable xenial InRelease' is not signed.

See https://github.com/angristan/openvpn-install/runs/910542905

/etc/apt/trusted.gpg
--------------------
pub   rsa2048 2011-08-03 [SC] [expired: 2020-07-25]
      30EB F4E7 3CCE 63EE E124  DD27 8E6D A8B4 E158 C569
uid           [ expired] Samuli Seppänen (OpenVPN Technologies, Inc<EMAIL_ADDRESS>

If this is not resolved, we'll drop distros that rely on this repo (Debian 8 and Ubuntu 16.04). Debian 8 is not even in the CI anyways because the image is not available on DO.

I removed Debian 8 support https://github.com/angristan/openvpn-install/pull/654#issuecomment-626793126
Ubuntu 16.04 should fix it as it's still supported.

Great thanks, Debian 8 is indeed EOL anyway

Seems to be fixed!


Test env. Jenkins build error

react-player 2.11.0 works successfuly on local and dev environment but it can not built test environment on jenkins and gives this errors.
Module not found: Error: Can't resolve './ReactPlayer' in '/var/jenkins_home/....
Module not found: Error: Can't resolve './YouTube' in '/var/jenkins_home
Module not found: Error: Can't resolve './SoundCloud' in '/var/jenkins_home
Module not found: Error: Can't resolve './Vimeo' in '/var/jenkins_home
....
and for other video apps

I think, it is about env. server security policy so im closing the issue.


[type:fix][ISSUE #5004] Fix build error apiPath.

Fixes #5004

Make sure that:

[x] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #5006 (db00e8a) into master (9b5b844) will increase coverage by 0.18%.
Report is 5 commits behind head on master.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #5006      +/-   ##
============================================
+ Coverage     64.04%   64.22%   +0.18%
- Complexity     8561     8578      +17
============================================
  Files          1208     1208
  Lines         35899    35899
  Branches       3217     3217
============================================
+ Hits          22992    23057      +65
+ Misses        11059    10993      -66
- Partials       1848     1849       +1

Files Changed
Coverage Δ

...t/springmvc/init/SpringMvcClientEventListener.java
82.56% <100.00%> (+5.50%)
:arrow_up:

... and 11 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

pending discuss


nameko.exceptions.RemoteError: UnicodeEncodeError 'ascii' codec can't encode character u'\u6bb5' in position 0: ordinal not in range(128)

nameko support non ascii character?

n.rpc.greeting_service.hello("段")
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python2.7/site-packages/nameko/rpc.py", line 352, in call
return reply.result()
File "/usr/lib/python2.7/site-packages/nameko/rpc.py", line 338, in result
raise deserialize(error)
RemoteError: UnicodeEncodeError 'ascii' codec can't encode character u'\u6bb5' in position 0: ordinal not in range(128)

Hi,
This is not a problem with nameko, but a bug in the example GreetingService:
https://github.com/onefinestay/nameko/blob/master/docs/examples/helloworld.py#L8 calls  "Hello, {}!".format(name), which fails with non-ascii names. You can fix it by prefixing the string with u, (so "uHello, {}!".format(name)) or by calling from __future__ import unicode_literals at the top of the helloworld.py
Leaving this open, as we should get our examples fixed

Fixed by #350


can't catch parse error Maximum call stack size exceeded on safari

Hi, I'm using mermaid, but I find a problem in https://github.com/knsv/mermaid/blob/master/src/mermaidAPI.js#L291. When flowchart contains a Chinese symbol like '；'  or '《' ,  Chrome will catch Maximum call stack size exceeded, but Safari will call recursively and comsume memory endlessly,  and I can't do anything on Safari.

please paste code like
graph LR
A--》B

in http://knsv.github.io/mermaid/live_editor/

also for #405

Issue diappeared: https://mermaidjs.github.io/mermaid-live-editor/#/edit/Z3JhcGggTFIKQS0t44CLQg/error/TGV4aWNhbCBlcnJvciBvbiBsaW5lIDIuIFVucmVjb2duaXplZCB0ZXh0LgpncmFwaCBMUkEtLeOAi0IKLS0tLS0tLS0tLS1e


Add optional initContainers and volumes

Signed-off-by: David Young<EMAIL_ADDRESS>

Following up from #16, this PR adds the option to include additional initContainers, volumes, and volumeMounts (on the primary container). The use-case here for large tests which required external data. We have about 300MB of files which we upload/download as part of a test, and the easiest way to get these into a loadtest is via an initContainer and an emptyDir mount. This would also satisfy the requests for running tests from configMaps, per #7

Ugh. I messed up both PRs. Creating 2 clean ones...


feat: follow fully qualified service injections for Ember apps

In Ember apps you can have fully qualified service injections like so:
class SomeComponent extends Component {
  @service('authentication@authenticated-user') authenticatedUser;
}

'authentication@authenticated-user'
 ~~~~~~~~~~~~~~ ^^^^^^^^^^^^^^^^^^
 ~ addon name   ^service name

In these cases we likely want to treat this code as if it was consuming the service via an import. For example, this is what the actual TS code would look like after conversion:
import AuthenticatedUser from 'authentication/services/authenticated-user';

class SomeComponent extends Component {
  @service('authentication@authenticated-user')
  declare authenticatedUser: AuthenticatedUser;
}

This would mean that we would add the authenticated-user service to the graph order (instead of implying on heuristics) and it should come before SomeComponent.
If we see a service injection with a non qualified injection path
class SomeComponent extends Component {
  @service('tracking') tracking;
  @service('my-service') myService;
}

We have to follow this resolution logic:

checking to see if it's locally defined e.g. /app/services/*.js
If it is there than we need to add it to the graph as we did above.
If it's not there its coming from some addon.

In the case where the service is unknown likely going to want to ask where to resolve the service in interactive mode and/or provide a way to give the users a "unresolvable tokens" manifest that they can file out and re-run the migration with to resolve those identifiers.

I chatted with @chriskrycho about this today.
I hacked out some simple heuristics based off of file path, and that definitely won't cut it given that services can be used in multiple places components, adapters, routes, services.
The current work I've done from that hack I can carry forward into addressing this feature.

In addition to migration ordering, lets keep this issue open as we will also need to handle type inference to handle the import and declaration of the service eg:
>>>>> import AuthenticatedUser from 'authentication/services/authenticated-user';

class SomeComponent extends Component {
       @service('authentication@authenticated-user')
>>>>>  declare authenticatedUser: AuthenticatedUser;
}

@lynchbomb
Just to clarify the additional feature request:
rehearsal should use the meta data from the migration graph to transform the resolver based calls for service decorators, from:
import Component from '@glimmer/component';
import { inject as service } from '@ember/service';
import { action } from '@ember/object';

export default class CartContentsComponent extends Component {
  @service('ShoppingCart') shoppingCart;

  @action
  remove(item) {
    this.shoppingCart.remove(item);
  }
}

to the following:
import Component from '@glimmer/component';
import { inject as service } from '@ember/service';
import { action } from '@ember/object';

import ShoppingCartService from 'my-app/services/shopping-cart';

export default class CartContentsComponent extends Component {
  // https://docs.ember-cli-typescript.com/ember/services
  @service declare shoppingCart: ShoppingCartService;

  // OR this...
  @service('ShoppingCart') declare shoppingCart: ShoppingCartService;

  @action
  remove(item) {
    this.shoppingCart.remove(item);
  }
}

Reference for output.
You'd like for us to add the import and declare authenticatedUser: AuthenticatedUser;
Want to clarify this, before we decompose this to tasks.


Update DeveloperGuide.md

Closes #58 and closes #63.
Added details for Target User profile and Glossary.
Reformatted user stories table based on intelij recommendation.

Should consider #68 before merging this

Should consider #68 before merging this

Done


Force STAccount to 160-bit size (RIPD-994)

Things to consider in this pull request:

Do you agree with my assessment that it is okay to throw in the STAccount constructor?  If not, what is the alternative?

Do you agree that these changes correctly enforce that an STAccount is always 160 bits?

Did I correctly leave the serialization format of STAccount unchanged?

One serialization change I just realized is that a default constructed STAccount will be serialized as 20 8-bit zeros, not as an empty blob.  Will that be a problem?

Previously the default state of an STAccount was empty (a zero-length STBlob).  Now the default state of an STAccount is all zeros.  Is that an acceptable change?

Two places used to check that a constructed STAccount was actually 160 bits.  Those checks no longer makes sense.  However, does it make sense to check for a default constructed STAccount (all zeros) in those same locations?

I have enough uncertainty about the serialization consequences that I'd like really knowledgeable reviewers: @nbougalis, @JoelKatz.  There are not a lot of changes.  Hopefully the review will be quick.  Thanks.

Current coverage is 49.12%

Merging #1387 into develop will not affect coverage as of e8df4f8

@@            develop   #1387   diff @@
=======================================
  Files           960     960
  Stmts         66698   66664    -34
  Branches          0       0
  Methods           0       0
=======================================
- Hit           32763   32748    -15
  Partial           0       0
+ Missed        33935   33916    -19

Review entire Coverage Diff as of e8df4f8

Powered by Codecov. Updated on successful CI builds.

I'm closing this pull request and adding some unit tests so I can be more confident that I'm not changing the external behavior of an STAccount.  I shall return!


Restore autosave ?

The fact the jupyterlab does have the ability to prevent closing phosphor tabs and save notebook is great, though other things can happen like network connection down... etc, and a few time I've close JupyterLab withhout saving, with habit of normal notebook.
Can we re-enable, or speedup the delay between autosave?

Spit-balling:  We should create checkpoints for all documents from the context manager whenever we save, and have it create checkpoints for all open files periodically.  When we load a file, check for a checkpoint before getting the disk contents.

Okay, I had misinterpreted what a checkpoint is.  They were created only when created explicitly saving.  The file itself is auto-saved (but only notebooks previously).  I am going to bring this up at the meeting today.
I propose that we emulate what a desktop editor would do: aggressively save to a temporary file that can be recovered, and have a setting for an auto save interval to disk.  "Checkpointing" should be done using version control.  We should also periodically check for a file change on disk and ask to revert.

The current behavior, modeled on modern applications and the norm for web applications, was picked specifically as opposed to the behavior of apps like word and emacs, which necessitates unpleasant things like recovery dialogs, timestamp comparisons etc., the triggers for which are all much more likely to happen in a browser app than a regular desktop application.
With server-side document state, I don't think we would need necessarily need autosave anymore, because the browser is no longer the sole home for unsaved state.

Message received :).  I'll follow the IPEP.

Note that the IPEP apply to the notebook only, but I think consistency is important. I thought agree that is is less than optimal, and agree that a server-side state would be much better.

Bumping that please, I again lost 30 min of work because of a new "go to previous page" chrome shortcut [Ctrl+LeftArrow]

Yep, this is first on my list when I get back.  Also note you can disable that behavior on OSX at least.


Pin Traefik Chart to v24.0.0

Traefik Chart v25.0.0 was released with a breaking change (https://github.com/traefik/traefik-helm-chart/pull/934). This is a quick fix to pin to the old behaviour.

The PR is merged, so is the pinning still needed?

Without pinning: "Waiting for load-balancer to get an IP..."
Thanks for the quick fix!

The PR is merged, so is the pinning still needed?

I had to be a bit more clear. The pinning (to v24) is needed because https://github.com/traefik/traefik-helm-chart/pull/934 was merged (and then released as v25, which now needs a different values syntax).

The PR is merged, so is the pinning still needed?

I had to be a bit more clear. The pinning (to v24) is needed because https://github.com/traefik/traefik-helm-chart/pull/934 was merged (and then released as v25, which now needs a different values syntax).

Ok, that good to know, we shall move to the new traefik values syntax, that would be more future proof, then you folks will be able to upgrade.

Without pinning: "Waiting for load-balancer to get an IP..."

I don't know if this is really related. The error i get is:
Error: UPGRADE FAILED: execution error at (traefik/templates/_podtemplate.tpl:567:19): ERROR: Syntax of ports.web.redirectTohas changed toports.web.redirectTo.port. Details in PR #934.
The correct solution is to use the new values format of course. I will have a look into that now. The pinning was just a quickfix.

It seems that does not apply to the default values! Do you see something that needs changing?

I will have a look now

Ok, I will try on my end.

I now used v25.0.0 and the proposed syntax, that also worked fine

@mxmxchere Thanks for confirming. I will double check in the code if we make use of traefik redirects or not and do the necessary changes if needed. Closing this for now, but appreciate the initiative 🙏

That was fixed in v2.8.3. Thanks again for the heads up folks! 🚀


Gameshark Codes Not applying on SCPH-101 (But same exact Card & File applys on SCPH-7501)

SCPH-101
Verbatim CD-R (Game's start but issue is with Gameshark Codes not being detected after pressing [0] with lid up)
I Used PS2 to copy file to a official PS1 memory card just like release notes suggested

Tonyhax Internaional v1.3.0 FreePSXBoot slot2 BIOS4.4
**Fireman2 Jap Import / Star Trek Invasion / Hercs Adventure **
All 3 of these games are booting perfect on the SCPH-101 after using FreePSXBoot Slot2 International v1.3.0 BIOS 4.4 exploit
BUT the gameshark codes are not applying before the game starts, Im getting error after pressing [o] the TONYHAXINTGS file I loaded onto the offical PS1 memory card using wlaunchelf via the PS2 dose not apply codes im getting error see photos below.
That exact same memory card & file is working just fine when I press [0] button on my SCPH-7501 model, the TONYHAXINTGS file is detected & gameshark codes work using tonyhax international v1.3.0 rom cheatcart on BIOS4.1
video showing that the TONYHAXINTGS file is not working on SCPH-101/BIOS4.4 but is working on 7501/BIOS4.1 using same physical card     https://www.youtube.com/watch?v=wjeGPuU44k4&ab_channel=JayTime

Also there is zip file in Discussions
Five Codes Detected on 7501 BIOS4.1

D0 Error on BIOS4.4 SCPH101

F9 Error SCPH101

Fixed in v1.3.1 huge thanks for pointing this out and helping debug the issue

Confirmed fireman2 just worked great im going test the other 2 games great work thanks again 4.4 SCPH101 is supported !


Fixed method inlining bug to do with category 2 arg types.

Method inlining didn't correctly account for category 2 argument types in the descriptor of the method being inlined.
An example that could cause the problem:
public class Test {
    public void test(double x, int y, float z) {
        System.out.println(x);
        System.out.println(y);
        System.out.println(z);
    }
}

The transformer:
@CTransformer(Test.class)
public class TestTransformer {
    @CInline
    @CInject(method="test(DIF)V", target=@CTarget("HEAD"), cancellable=true)
    public void test(double x, int y, float z, InjectionCallback callback) {
        if (x == 1.0) {
            System.out.println("It's 1.0.");
            callback.setCancelled(true);
        }
    }
}

The InjectionCallback callback parameter, in this case, is incorrectly referenced as local 20 at callback.setCancelled(true), which in turn will cause a verification error.
Also made gradlew executable on *nix systems.

Thanks, I really missed that


Configure headers

Codecov Report

Merging #339 into master will increase coverage by 1.25%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #339      +/-   ##
========================================
+ Coverage    1.94%   3.2%   +1.25%
========================================
  Files          73     52      -21
  Lines        1795   1092     -703
  Branches      298    182     -116
========================================
  Hits           35     35
+ Misses       1759   1056     -703
  Partials        1      1

Impacted Files
Coverage Δ

atcoder-problems-backend/src/sql/streak.rs

atcoder-problems-backend/src/server/utils.rs

atcoder-problems-backend/src/sql/models.rs

...tcoder-problems-backend/src/sql/rated_point_sum.rs

atcoder-problems-backend/src/crawler.rs

...tcoder-problems-backend/src/sql/contest_problem.rs

atcoder-problems-backend/src/sql.rs

atcoder-problems-backend/src/sql/simple_client.rs

atcoder-problems-backend/src/s3.rs

atcoder-problems-backend/src/sql/accepted_count.rs

... and 7 more

Continue to review full report at Codecov.

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


rubocop linter issue

Don't know who or where to ask so forgive me if I'm not following protocol here.
Any chance this can get looked at and merged?  https://github.com/SublimeLinter/SublimeLinter-rubocop/pull/47
Been sitting around for a while. Thanks!

I don’t have any way to test this but if you’re confident it’s a good change that doesn’t break too many people’s setup, I’ll merge and release it. So confirm that and I’ll push the button.

I've been using this locally since i first created the PR with no issues.

others are welcome to download from my fork and test for themselves if you need more 3rd party approvals.

Good enough for me. Want to have access to the repo and do the release yourself?

@braver if you don't mind dealing with it, I'd appreciate it.


Test CODEOWNERS and docker scanner PR

What does this PR do?
Check if:

adequate code owner will be added
docker image scanner runs when list of allowed is changed images

Works as expected. Codeowner is added correctly and adequate action is triggered


multi-tenancy security

Is your feature request related to a problem? Please describe.
current volume setting only seperate different tenant by volume.
if a unexperienced admininstor name the volumes by some pattern, for example:
001,002....
or
order/customer/account/
or
blabla
then a malicious tenant will easily mount other tenant's volumn and steal other tenant's data.
it will be a huge secruity vulnerablity.
Describe the solution you'd like
A clear and concise description of what you want to happen.
volume setting should invlove some security mechnism such as appsecret to prevent hacker to guess.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
as a workround, in maintaince manual, we can suggest administrators should use a passcode generator or something a like to generate a strong  volume name.
Additional context
Add any other context or screenshots about the feature request here.
{
"mountPoint": "/mnt/fuse",
"volName": "test",
"owner": "cfs",
"masterAddr": "<IP_ADDRESS>:80,<IP_ADDRESS>:80,<IP_ADDRESS>:80",
"logDir": "/export/Logs/client",
"warnLogDir":"/export/home/tomcat/UMP-Monitor/logs/",
"profPort": "10094",
"logLevel": "info"
}

@skybabydad  it is a good Suggest，
We will support it in the next release

Hi @skybabydad ,
Thanks for your feedback.
Development of multi-tenant security is ongoing and is nearing completion.
ChubaoFS is a distributed fusion storage system. We also provide a POSIX-compatible file system interface and an Amazon S3-compatible object storage interface.
In order to provide more flexible security policies in the multi-tenant environment, we are developing highly granular user security authentication functions.

@skybabydad
We have a solution based on user security and authorization to improve access control of multi-tenant resources. This solution is described in issue #423 .
Under this solution, users could have multiple volumes, and could authorize other users to read-only or writable permissions on some volumes of their own, or revoke authorization at any time.


Fix CI settings

Codecov Report

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

@@            Coverage Diff            @@
##           master       #6     +/-   ##
=========================================
- Coverage   99.79%   99.69%   -0.1%
=========================================
  Files           8        1      -7
  Lines         979      330    -649
  Branches        0       56     +56
=========================================
- Hits          977      329    -648
+ Misses          2        0      -2
- Partials        0        1      +1

Impacted Files
Coverage Δ

declxml.py
99.69% <ø> (-0.31%)
:arrow_down:

tests/run_tests.py

tests/test_errors.py

tests/test_parsing.py

tests/helpers.py

tests/test_serilization.py

Continue to review full report at Codecov.

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


Feat/graph utils

Adds some graph utilities.
(Renamed findCycles.nix into graphUtils.nix)

SanitizeGraph returns a nix buildable dependency tree. (Even if the previous graph has cycles)
Compat functions that allow backwards compatibility for potential use cases in V1 (node granularBuilder)

Still TODO: Fix the old cycle remover, so it supports
Multiple connected cycles in the graph
e.g.
a->c
b->c
c->d
d->a
d->b

@mergify rebase

@mergify rebase


Submission example in readme missing necessary argument

observation = client.env_create(crowdai_token)
should be changed to
observation = client.env_create(crowdai_token, env_id='ProstheticsEnv')

Thanks!


Prisma throws error for fulltext search query containing @

Discussed in https://github.com/prisma/prisma/discussions/12905
Bug description
I've written a simple text search query with the Prisma client to search Usernames or emails in my users table. While it correctly works with usernames, it errors out saying, Unexpected<EMAIL_ADDRESS>believe that's due to @ is a special symbol.Is there anyway around this, because I need to be able to search full email addresses. (Emails are correctly saved in the database). And query return results if I omit the part from @.Ex: in<EMAIL_ADDRESS>query returns match if I just search abc.But If I use<EMAIL_ADDRESS>it fails with this error.
Error occurred during query execution:
 ConnectorError(
  ConnectorError {
   user_facing_error: None,
   kind: QueryError(Server(ServerError {
                           code: 1064, message: "syntax error,
                           unexpected '@',expecting $end", state: "42000" }
)) })

How to reproduce
let result = await prisma.user.findMany({
  where: {
    email: {
      search: '"foo@bar"'  // outer single quotes, search term wrapped in double quotes
    }
  }
})

Expected behavior
Client does not throw an error
Prisma Information
model User {
  id              Int     @id @default(autoincrement())
  username        String  @db.VarChar(255)
  email           String  @db.VarChar(255)
  avatar          String? @db.VarChar(255)
  password        String  @default("1") @db.VarChar(255)
  friends         User[]  @relation("friends")
  friendsRelation User[]  @relation("friends")
  hashRT          String? @db.VarChar(255)

  @@fulltext([username])
  @@fulltext([email])
  @@fulltext([username, email])
  @@map("users")
}

Prisma version
prisma                  : 3.12.0
@prisma/client          : 3.12.0
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine 22b822189f46ef0dc5c5b503368d1bee01213980 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 22b822189f46ef0dc5c5b503368d1bee01213980 (at node_modules/@prisma/engines/migration-engine-darwin-arm64)
Introspection Engine    : introspection-core 22b822189f46ef0dc5c5b503368d1bee01213980 (at node_modules/@prisma/engines/introspection-engine-darwin-arm64)
Format Binary           : prisma-fmt 22b822189f46ef0dc5c5b503368d1bee01213980 (at node_modules/@prisma/engines/prisma-fmt-darwin-arm64)
Default Engines Hash    : 22b822189f46ef0dc5c5b503368d1bee01213980
Studio                  : 0.459.0

I notice that this version is 3.12.0, I have version 4.10.1 and I cannot reproduce this issue.
Is it possible to upgrade and try again on the latest version? Thanks

@mcbethio That's odd that you're unable to reproduce. I've just reproduced it in my own environment with versions 4.5.0 and 4.10.1. Both times I get the error the original author posted.
If it helps, here's my environment info:
prisma                  : 4.10.1
@prisma/client          : 4.10.1
Current platform        : darwin
Query Engine (Node-API) : libquery-engine aead147aa326ccb985dcfed5b065b4fdabd44b19 (at ../../../common/temp/node_modules/.pnpm/@prisma+engines<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli aead147aa326ccb985dcfed5b065b4fdabd44b19 (at ../../../common/temp/node_modules/.pnpm/@prisma+engines@4.10.1/node_modules/@prisma/engines/migration-engine-darwin)
Format Wasm             : @prisma/prisma-fmt-wasm 4.10.1-1.80b351cc7c06d352abe81be19b8a89e9c6b7c110
Default Engines Hash    : aead147aa326ccb985dcfed5b065b4fdabd44b19
Studio                  : 0.481.0
Preview Features        : referentialIntegrity, interactiveTransactions, fullTextSearch, fullTextIndex

No worries at all, we will continue investigating...

I just hit this also.   Please advise.

@mcbethio could prisma privide NATURAL LANGUAGE MODE for fulltext search, it is useful for non-English content

+1 for the ability to explicitly specify which mode we want to use in MySQL (NATURAL LANGUAGE MODE or BOOLEAN MODE) - seems like a table-stakes feature to have.

@Ali1Ammar @pantharshit00 @timmeade
I recommend taking a look at this StackOverflow post: https://stackoverflow.com/a/26537463/6271468
TLDR: the fulltext indexes in MySQL don't actually index non-word characters, so the optimal approach is to remove those operator strings via regex by replacing them with spaces or just removing them entirely before running the query.
Note: I have not independently verified this solution yet, so I suggest doing your own experimenting.

Hey folks!
I tried this with the following matrix of details:
prisma_version: [3.12.0, 4.10.1, 5.14.0]
mysql_version: [v5.6, v5.7, v8, MariaDB]
When I wrapped the search term with two pairs of string quotes I consistently retrieved data with no errors
[
  {
    id: 1,
    username: 'username',
    email: 'email@bar',
    avatar: null,
    password: '1',
    hashRT: null
  }
]

And when I removed the second set of quotes for search:  i.e.
- search: '"foo@bar"'
+ search: "foo@bar"

I was able to confirm the same syntax error
14 }
  15
  16 const test = async () => {
→ 17   const res = await prisma.user.findMany(
Error occurred during query execution:
ConnectorError(ConnectorError { user_facing_error: None, kind: QueryError(Server(MysqlError { code: 1064, message: "syntax error, unexpected '@', expecting $end", state: "42000" })), transient: false })
 ELIFECYCLE  Command failed with exit code 1.


Remove AppVeyor when VSTS is turned on for Windows

[x] Disable the webhook
[x] Remove AppVeyor's permissions from the repo
[ ] Delete the AppVeyor file
[ ] Delete VS Code Python project registration on AppVeyor

@d3r3kk

@DonJayamanne @d3r3kk are we happy enough with VSTS to continue on with dropping AppVeyor?

I vote yes. But I defer to @DonJayamanne as I am not 100% certain he trusts the VSTS output as yet.  I would still be ok turning off AppVeyor at this point though, as I believe we are running the tests across many more versions of Python than we had before (and probably faster, anecdotally speaking).

During our stand-up Don said he has no objections, so I will start to remove the remnants of AppVeyor in the repo.
@DonJayamanne do you want to close your AppVeyor account for this repo?


python3Packages.sev-snp-measure: init at 0.0.9

Description of changes

Add the sev-snp-measure Python package to create measurements for SEV-SNP CVMs.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 303158 run on x86_64-linux 1

4 packages built:

python311Packages.sev-snp-measure
python311Packages.sev-snp-measure.dist
python312Packages.sev-snp-measure
python312Packages.sev-snp-measure.dist

You'll want to rebase on the latest master -- there was an issue that broke ofborg's ability to evaluate Nixpkgs that has since been resolved. Sorry for the inconvenience!

@ofborg build python3Packages.sev-snp-measure


how to understand the cost ?

Hi,
As a new learner in deep learning,
I'm a little puzzled about the cost definition,
def my_crossentropy(y_true, y_pred):     return K.mean(2*K.abs(y_true-0.5) * K.binary_crossentropy(y_pred, y_true), axis=-1)
Could you please explain the function ?
Why 2*K.abs(y_true-0.5) * ?
and then the mymask
def mymask(y_true):
return K.minimum(y_true+1., 1.)
Why y_true+1. and then minimum ?
and
what's the mycost ?
Should have the mymask(y_true) * (10*K.square(K.square(K.sqrt(y_pred) - K.sqrt(y_true)))  ?
and
what's in the my_accuracy ?
why 2*K.abs(y_true-0.5) ?
Thx

Hi,
As a new learner in deep learning,
I'm a little puzzled about the cost definition,
def my_crossentropy(y_true, y_pred): return K.mean(2*K.abs(y_true-0.5) * K.binary_crossentropy(y_pred, y_true), axis=-1)
Could you please explain the function ?
Why 2*K.abs(y_true-0.5) * ?
and then the mymask
def mymask(y_true):
return K.minimum(y_true+1., 1.)
Why y_true+1. and then minimum ?
and
what's the mycost ?
Should have the mymask(y_true) * (10*K.square(K.square(K.sqrt(y_pred) - K.sqrt(y_true)))  ?
and
what's in the my_accuracy ?
why 2*K.abs(y_true-0.5) ?
Thx

The value of y_true for vad is 0,0.5,1. With y_true -0.5 operation, the value range can be tuned to -0.5~0.5.
And the mask , it is used to mask some value which is -1.

HI,
Please can anyone tell :-
1] Why the VAD values range is tuned between  -0.5~0.5?


can't define padding, button jumps

Can we define the padding, so that the button width doesn't keep jumping around?

Hi

Hi
I'm not sure what the problem is, but have you tried wrapping with the Padding widget?

@mpcreza Simply wrapping with padding will not work.
I attached the examples to explain this.
This button's width will keep changing, its txt will keep jumping.
I'm wondering if there is a way to make it fixed
Right now what I can do is to wrap it with a theme widget, to override its OutlinedButtonThemeData
Theme(               data: Theme.of(context).copyWith(                 outlinedButtonTheme: OutlinedButtonThemeData(                   style: OutlinedButton.styleFrom(                     minimumSize: const Size(160, 40),                   ),                 ),               ),               child: OtpTimerButton(                 onPressed: () {                       setState(() {                         _code = "1234";                       });                 },                 text: const Text('Resend Code'),                 buttonType: ButtonType.outlined_button,                 duration: 10,               ),             ),

I will fix this as soon as I can, so that the widget width does not change when the timer appears.
But for now, instead of using the fixed size, can you use this way?
Padding(
  padding: const EdgeInsets.symmetric(horizontal: 10.0),
  child: Row(
    children: [
      Flexible(
        fit: FlexFit.tight,
        child: OtpTimerButton(
          onPressed: () {},
          text: const Text('Resend Code'),
          buttonType: ButtonType.outlined_button,
          duration: 10,
        ),
      ),
      const SizedBox(
        width: 20,
      ),
      Flexible(
        fit: FlexFit.tight,
        child: OutlinedButton(
          onPressed: () {},
          child: const Text(
            'Edit Number',
          ),
        ),
      ),
    ],
  ),
),


Introduces the possibility for a summary in the table of contents

Allows to specify a summary in the front matter so that it will appear in the table of contents when present.

Reopening. Closed due to assumed mistake in opening it.

@pmarsceill why was this never merged or at least declined?
I'd like to think that making a pull request here is not a complete waste of time.

Not a waste of time, this should be prioritized.


Text search is not able to identify text, but working with other pdf viewers

Attach (recommended)
Issue_Document.pdf
or Link to PDF file here:
Configuration:

Web browser and its version: Chrome and Version 98.0.4758.102
Operating system and its version: Windows 10
PDF.js version: Using  "ngx-extended-pdf-viewer": "^12.0.0-alpha.2" which a wrapper around Mozilla's PDF viewer
Is a browser extension: No

Steps to reproduce the problem:

Open the attached pdf document in 'https://mozilla.github.io/pdf.js/web/viewer.html?'
Search for text 'message persists' and 0 results are found
Open the attached document with Chrome
Search for text 'message persists' and '1' result is found
Open the attached document in Adobe Acrobat Reader
Search for text 'message persists' and '1' result is found

What is the expected behavior? (add screenshot)
Search result for 'message persists' should be 1

CHROME:

ADOBE ACROBAT READER

What went wrong? (add screenshot)
Pdf.js not able to find the text 'message persists'.
Pdf.js

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

We are not detecting the space between "message" and "persists". If you look for "messagepersists", a result is found.

We are not detecting the space between "message" and "persists". If you look for "messagepersists", a result is found.

Yes, but why it is different with other viewers

Yes, but why it is different with other viewers

It is a bug in pdf.js that we have to fix :)

There is a final whitespace at the end of the Tj for message, but it's skipped thanks to:
https://github.com/mozilla/pdf.js/blob/234aa9a50e33599c28877ac3b5fff3542c55cd61/src/core/evaluator.js#L2601
and so it isn't correctly detected when we begin with persists.
The text is likely not what it's displayed: the selection in Acrobat looks weird:

The text is likely not what it's displayed: the selection in Acrobat looks weird:

The "text" is actually an image, with invisible text (using a zero fill-alpha rather than the invisible text-rendering mode) placed on top for text-selection and searching purposes.
Furthermore, there's a bunch of what appears to be completely unnecessary font-size changes in this "invisible" text, that I assume contribute to the problems we see with guessing where spaces should should be inserted.

There is a final whitespace at the end of the Tj for message, but it's skipped thanks to:
https://github.com/mozilla/pdf.js/blob/234aa9a50e33599c28877ac3b5fff3542c55cd61/src/core/evaluator.js#L2601
and so it isn't correctly detected when we begin with persists.
The text is likely not what it's displayed: the selection in Acrobat looks weird:
![image](https://user-images.githubusercontent.com/5641725/156549752-03b6eb27-82f4-4e51-a09f-87ce25b217f5.p

There is a final whitespace at the end of the Tj for message, but it's skipped thanks to:
https://github.com/mozilla/pdf.js/blob/234aa9a50e33599c28877ac3b5fff3542c55cd61/src/core/evaluator.js#L2601
and so it isn't correctly detected when we begin with persists.
The text is likely not what it's displayed: the selection in Acrobat looks weird:

Page content on this pdf is written using PDFTRON. The original pdf had image in it to which content is written using PDFTRON

We already added an exception to keep whitespaces when they're between two non-whitespaces whatever the extra spacing (negative or not) can be, so I guess we can extend that.


Image classification fails on M1 chip Macs

Thanks for a great work. I just tried to add this package to my test C# API and failed to run it on MacOS Monterey.
After successfully adding to our project, when the code hit NSFW Spy, it exits immediately without any error. Any tips would be appreciated.

Installed SciSharp.TensorFlow.Redist
Injected dependency
Environment:
.NET SDK
Version:   6.0.105
Commit:    1c35735293

Runtime Environment:
OS Name:     Mac OS X
OS Version:  12.4
OS Platform: Darwin
RID:         osx.12-x64
Base Path:   /usr/local/share/dotnet/x64/sdk/6.0.105/
Host (useful for support):
Version: 6.0.5
Commit:  70ae3df4a6
.NET SDKs installed:
3.1.419 [/usr/local/share/dotnet/x64/sdk]
5.0.408 [/usr/local/share/dotnet/x64/sdk]
6.0.105 [/usr/local/share/dotnet/x64/sdk]

Hi @justinphamnz,
Thanks for the issue report.
Is it possible for you to post the code extract of how you're using NsfwSpy please? I've just tried using NsfwSpy on a Mac as part of a console application just now and it worked OK classifying a web image.
Can you also confirm you are using version 2.3.1 of SciSharp.TensorFlow.Redist?
Cheers 😊

Hi @d00ML0rDz,
Thanks for looking into the issue. Here is the basic Console App that I tried to implement https://github.com/justinphamnz/Console-NSFW
I think the issue could be with Apple M1 chip, not so sure if you or anyone has experienced this before.
I can confirm that I installed SciSharp.TensorFlow.Redist 2.3.1.
Have a great day.

Hi @justinphamnz,
That code extract all looks OK and I would expect that to work. The Apple M1 chip may be the issue here although as far as I'm aware ML.NET does support Apple's silicon and has done since 1.6.0. It's possible one of the dependencies we are using isn't playing ball.
Would you be happy to wrap the code extract you sent me in a try catch block like this and see if it writes out an error message please?
try
{
    Console.WriteLine("Started");
    var uri = new Uri("https://raw.githubusercontent.com/d00ML0rDz/NsfwSpy/main/NsfwSpy.Test/Assets/flower.jpg");
    var nsfwSpy = new NsfwSpy();
    var result = nsfwSpy.ClassifyImage(uri);
    Console.WriteLine(result.Pornography);
    Console.WriteLine("Completed. Press any key to exit");
    Console.ReadKey();
}
catch (Exception ex)
{
    Console.Write($"{ex.Message} | {ex.InnerException}");
}

Thanks @d00ML0rDz,
I did wrapped the try catch and seems there is no catch hit using Visual Studio Code. I then installed Visual Studio 2022 for Mac, was able to catch the Inner Exception
Message	"Unable to load shared library 'tensorflow' or one of its dependencies. In order to help diagnose loading problems, consider setting the DYLD_PRINT_LIBRARIES environment variable: dlopen(libtensorflow, 0x0001): tried: 'libtensorflow' (no such file), '/usr/local/lib/libtensorflow' (no such file), '/usr/lib/libtensorflow' (no such file), '/Users/justinpham/MacMini/ConsoleNSFW/ConsoleNSFW/ConsoleNSFW/bin/Debug/net6.0/libtensorflow' (no such file)"

I've just found this StackOverflow thread that seems to describe your issue, however it sounds like the solution was to use .NET 5 and I can see you're already using .NET 6 so suspect that won't work.
I also found this GitHub comment which makes me wonder if including the SciSharp.TensorFlow.Redist GPU packages for Windows and Linux by default is interfering in some way.
You could try cloning the NsfwSpy repo and in the NsfwSpy project removing the following package references:
<PackageReference Include="SciSharp.TensorFlow.Redist-Linux-GPU" Version="2.3.1" />
<PackageReference Include="SciSharp.TensorFlow.Redist-Windows-GPU" Version="2.3.1" />

and replacing them with:
<PackageReference Include="SciSharp.TensorFlow.Redist" Version="2.3.1" />

You should then be able to add a new console application to the solution that references that modified version of NsfwSpy and see if your code extract now runs correctly. If that does work, I can work out a solution, maybe two separate Nuget packages, one for GPU and one for CPU.
Sorry this isn't a quick fix!

Thanks @d00ML0rDz. That's super helpful.
Yeah, I found the thread on StackOverflow but as you said, should resolved by .NET 6. Let me try clone and see if it works then let you know.
Really appreciate your prompt response.
Looking forward to contributing to this wonderful project.

@justinphamnz just done some more research and found this Reddit post where the author said they had some luck using the Microsoft.ML.TensorFlow.Redist nuget package instead of SciSharp.TensorFlow.Redist with their M1 Mac. From my tests that didn't work on my Windows or Intel based Mac though.
In the comments of that post someone also commented this document saying that TensorFlow isn't supported on M1 Macs yet when using ML.NET, so that might be why it's currently not working 😕
Would love for you to help on the project if you're able to find a way to get it working though!

Hi
I also have issues on arm64 (RPI 4B 4GB), does the lib work with ASP.NET 6.0 inside of Docker on an arm64 machine?
Keep up the amazing work!
Adam Koprek

@Halfooda We have a page on our wiki that mentions CPUs must support AVX for this library, which from a quick google, sounds like the Raspberry Pi doesn't 😕
Don't think it will be possible to get this running unfortunately.

Ok, thank you for your fast response.


adding hexsticker

Added:

script using the hexSticker package
jar image
hexsticker image

I see @isteves is doing the crucial work here! I need to step it up, again!


elm-native-ui/core repository doesn't have option to create issues

@mk @ohanhi @eeue56
elm-native-ui/core repository doesn't have option to create issues.
There are a few blockers which are stopping me from using the core as is. I had to make changes in the src/Native/Runtime.js to make it work.
Can you please enable issues for that repository?

Enabled


Add white variant of Circular Progress Indicator

Ready for [review](https://www.figma.com/file/0bGXR2sCwMVSDNyyzu5BXrO5/?node-id=18442%3A38075 @lucasveil )

Ready for [review](https://www.figma.com/file/0bGXR2sCwMVSDNyyzu5BXrO5/?node-id=18442%3A38075 @lucasveil )

@BirteThornquist When using the moss-green  color we have mostly refered to it as "primary" color other places, for example buttons and textfield.
When we implement these color variants in code we need a set of color values for our users to choose from. In this use-case, should that be primary and white (+ danger for dots?)

@BirteThornquist When using the moss-green  color we have mostly refered to it as "primary" color other places, for example buttons and textfield.
When we implement these color variants in code we need a set of color values for our users to choose from. In this use-case, should that be primary and white (+ danger for dots?)

Primary and white sounds good to me @mimarz 👍

Primary and white sounds good to me @mimarz 👍

Using colour names in properties does not take theming into account 🎨 Why not use the neutral name as suggested above? Primary, neutral and danger for example?

Using colour names in properties does not take theming into account 🎨 Why not use the neutral name as suggested above? Primary, neutral and danger for example?

Using colour names in properties does not take theming into account 🎨 Why not use the neutral name as suggested above? Primary, neutral and danger for example?

@vnys @BirteThornquist
Hmm, good point, but is white really a color? What school of belief do we run with here? Do we believe in the light? huehuehue... 🤪
Maybe primary, neutral and tertiary?
danger feels so connected to validation in a way...
I see we are already using green and white in DotsProgress which needs to be fixed... but all this relates back to #801  🙄

Using colour names in properties does not take theming into account 🎨 Why not use the neutral name as suggested above? Primary, neutral and danger for example?

@vnys @BirteThornquist
Hmm, good point, but is white really a color? What school of belief do we run with here? Do we believe in the light? huehuehue... 🤪
Maybe primary, neutral and tertiary?
danger feels so connected to validation in a way...
I see we are already using green and white in DotsProgress which needs to be fixed... but all this relates back to #801  🙄

…is white really a color?

No, it’s just a figment of imagination like everything else in this world 😜
My point is that ⚪ will be ⚫ in dark mode 😎 So then it doesn’t make sense to call it white…

…is white really a color?

No, it’s just a figment of imagination like everything else in this world 😜
My point is that ⚪ will be ⚫ in dark mode 😎 So then it doesn’t make sense to call it white…

Absolutely, my bad. We should not reference color by name only. I like the suggestion of primary, neutral and tertiary. I will fix this in the dots as well. When it comes to the transparent white used for the circle, we have a 20% transparent white token which we can use; Interactive/Pressed Overlay/Light.

Absolutely, my bad. We should not reference color by name only. I like the suggestion of primary, neutral and tertiary. I will fix this in the dots as well. When it comes to the transparent white used for the circle, we have a 20% transparent white token which we can use; Interactive/Pressed Overlay/Light.

Righto, will update color names.
Is it safe to use the "Interactive/Pressed Overlay/Light" color token? Is is semantically correct? In case we change pressed overlay light color in future to something else, this might result in an unwanted side-effect 🤔

Good to go @BirteThornquist


Distinguish number between int and float

Hi, I have the case where I define javascript code which defines a number and on the PHP side I have to differentiate whether the value is an int or float. At the moment all defined numbers are float. I have looked a bit in the source and the check whether to return an integer or float from a number is already implemented (php_v8_value.cc:268) but commented. Do you have plans to activate this detection? Otherwise I have to perform this check on the PHP side with something like strpos('.', $value) === false ? (int) $value : $value which I think is not the best solution.

Hi, thanks for noticing!
I was not sure how good it would be when you pass NumberValue object with value, say,  withing uint32 range and get back Uint32Value object.
Though later I also faces a need to somehow differ float from int. As I'm currently doing some wild changes on php-v8 so I will cover this issue in one of commits.

FTR, psx-v8/composer.json: if you do mean to require php-v8, it requires PHP >= 7.0, so I'd suggest to specify "php": ">=5.5", like in php-v8-stubs/composer.json.

Thx for the info and the fix. Regarding the composer.json I have another project which requires this package and where the v8 engine is only a "nice" feature but not required for project to work. So users can still install it under 5.6. without v8. If they have PHP >= 7.0 they can also install the extension and use the v8 engine. This is only the reason for this "wrong" version constraint. But I will probably drop soon the 5.6 support thus I can set the correct constraint.

Hi, thx for the fix! Do you have roughly any time plan for a first stable version or are there plans for any huge BC breaks. I think the current API looks really great.

0.1.2 was just released and it has numerous BC breaks, though it's less likely you'll be affected, see release notice for details.
There will be more upcoming BC-breaking changes in forseable future, not specific details at this time, but it will be API simplification and performance tuning.


Sealed interfaces

Right now this is tightly coupled to sealed classes, but is there space to generally support the other polymorphic type cases and make sealed classes just a specific subtype?
Kinds:
abstract/open class + subclasses
abstract class Message {
}

class Success : Message()

interface + implementations
interface Message {
}

class Success : Message

interface + subinterfaces
interface Message {
}

interface Success : Message

Would need some sort of annotation to denote the subtypes.
annotation class JsonSubtypes<T : Any>(vararg val subtypes: Subtype<out T>)
annotation class Subtype<T : Any>(val type: KClass<out T>, val labelKey: String)

@JsonSubtypes<Message>(
  Subtype(Success::class, "success"),
  Subtype(Error::class, "error")
)
interface Message

Or just keep it simpler and reuse the label annotation. I'm leaning toward this
annotation class JsonSubtypes<T : Any>(vararg val subtypes: KClass<out T>)

@JsonSubtypes<Message>(Success::class, Error::class)
interface Message

@TypeLabel("success")
class Success : Message

For default objects, in Kotlin it can just be an object type.

With both java and kotlin formally planning to add support for these, I'm retargeting this issue towards them

Took an early stab at this, but will need to wait for upstream support in kotlin-reflect and kotlinx-metadata. Filed here: https://youtrack.jetbrains.com/issue/KT-44489

Took an early stab at this, but will need to wait for upstream support in kotlin-reflect and kotlinx-metadata. Filed here: https://youtrack.jetbrains.com/issue/KT-44489

After some discussion, I've tested this and it does appear to Just Work™ for Kotlin cases 🥳. I'll look at adding some test cases in the future (hard to do right now because KSP breaks on 1.4.30-RC and having to enable languageVersion 1.5). I've opened a separate issue for Java support here: #77

After some discussion, I've tested this and it does appear to Just Work™ for Kotlin cases 🥳. I'll look at adding some test cases in the future (hard to do right now because KSP breaks on 1.4.30-RC and having to enable languageVersion 1.5). I've opened a separate issue for Java support here: #77


Submitting to themes directory

Are the themes built using timber acceptable at wordpress.org themes directory.

I don't think WordPress does allow for that in their theme directory @mcdevlog


Create a platformer examples

Hi !
A platformer game example would be awesome to begin with the engine.
As I said in issue #335 , it can a bit hard to developp a platformer with the current findable ressouces.
The only one I found is : https://codepen.io/tanker837/pen/qaeQKZ
If somebody got something else, i'll be thankful.

@hilde2 - I think this is a great idea, and actually from my experience with MatterJS (making a simple slingshot game), I don't expect it would be too difficult. What would you be specifically looking for in terms of how it would be broken down? If you can help me plan out such a tutorial I would be more than happy to write it.

There is a bit of work to make something good for a platformer.
A documented tutorial like you explain in #373 could be awesome.
Something that I like and I found really usefull is what you can found in P2.js examples : https://github.com/schteppe/p2.js/blob/master/examples/canvas/platformer.html
I make a quick js fiddle if you want to try it : https://jsfiddle.net/2h4ah4k1/
This help a lot to start a platformer project on the engine :

Learn how to make the body stop rotating
Collissions detector
Slope
etc ...

And there is still this tunneling bug #335 . In this is issue there is this example : https://jsfiddle.net/hsassewa/
This just show the basics, like collissions sensors, and non rotating bodies.

I'm a bit of a novice programmer, but I wrote the platformer that you linked on codepen.  If anyone has any questions about putting the game together I'd be happy to help.
https://landgreen.github.io/sidescroller/
Press T to enter testing,
0-+  to zoom

I found your codepen really usefull ! But a bit too much complex for the one who just want to try MatterJS.

Agreed, I hope to use @landgreen's code to create a simpler example with just the core mechanics.

i did some coding with pixi js and matter js working together here, https://davidclews.com/post/11.html


Add Vega 4 extension

I just resolved a merge conflict 👍

Thanks!

I fixed this build issue and merged to master but I can't seem to force push to your branch. Can you git rebase -i upstream/master?

Can you just squash and merge with github?

That won't help CI pass, will it?

Nah, that's a bug in a dependency. I don't know why you see that coming up in my PR. It's not related as far as I can see.

It's because your branch is based on the previous master (before the fix). If you rebase to the current master, CI should pass.

Done.

CI is passing, thanks @domoritz!


bug : click sort column 3 times and the sort parameters become null

I am using the server sorting module configuration in a simple vue 3 page app.
If you click the column header to sort on that, it updates the v-model serverOptions correctly but click it a 3rd time, it sets the sort attributes to null.  If you click it a 4th time it works again.
I have tried the module and non-module builds with the same result.
indexSuccess.zip

@DustinAPI please refer to the issue here: #66


1.20 😤

I don't like that tone of voice
I'm working on 1.20 versions of all my mods, but obviously, I can't work on them all at the same time.

quick update: 1.20 release is now closer than ever

I released 1.20 since the last update


VSCode Insider edition setup has low resolution icon in header

Steps to Reproduce:

Wait for new Insider release
When new release found, VS Code Insider popups a notification, click "Update Now" button
You can see the low resolution VS Code Insider logo in the header of setup window

VSCode Version: Code - Insiders 1.11.0-insider (c995ecf2e94a00aa7eda81ed842e787b999e83ae, 2017-03-27T06:11:09.027Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

python
donjayamanne
0.6.0

EditorConfig
EditorConfig
0.6.2

Angular2
johnpapa
2.1.2

csharp
ms-vscode
1.8.0

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
Thanks for your understanding. Happy coding!


config(ticdc): remove some required_status_checks in release-4.0

Keep the same required_status_checks with release-5.0/release-5.1 and master branch in release-4.0.

/ok-to-test

PTAL @hi-rustin , thanks

@amyangfei, Lint required context has been removed in #388, could you mind resolve the conflict.

I'm sorry, I didn't notice your PR, and then I removed the Lint by raising the PR myself.
ref: https://github.com/ti-community-infra/configs/pull/388

/merge


Question: Is 1.0.63 (progressive linking?) a breaking change?

I've experienced recent problems building rust-rocksdb after the upstream (cc-rs) dependency started resolving to version 1.0.63.

When it consumes 1.0.63 (default behavior / semver thinks it is compatible), the linking complains about undefined symbols
rust-rocksdb does build fine, if I update my Cargo.toml file to consume a fixed version of 1.0.62
If you're interested in more details, please see issues/479

I don't know enough about the progressive linking change (commit id=e4ff87239325e7f9fe1f2b21a979b947f6f7503e), to tell if this is expected behavior (e.g. rust-rocksdb should be using cc-rs differently), or if there is an actual bug/regression in cc-rs itself.
Thanks in advance, for any information you can provide.

Thanks for the report and sorry for the breakage! This seems highly likely to be due to #564 indeed, although that was not supposed to be a breaking change. The intention is that we add 100 objects at a time to the final archive instead of all-at-once which can blow command line length limits for lots of object files.
Do you know if there's anything special about the targets you're compiling for? Or perhaps a unique toolchain? I think the ar invocations are right but I'm not 100% sure.

cc @maxded as well as the author of the original PR

Ah I see the issue, rocksdb has two objects called format.o and two objects called merge_operator.o. With the previous all-in-one command line they were all showing up in the output but now with incrementally building the *.a file they're replacing each other and only one is showing up (and the second object file is replacing the first).
I've got a fix locally I'll post soon.

Ah sorry! An oversight on my behalf, apologies for the inconvenience. Thanks, @alexcrichton for the quick investigation.

No worries, this was one that's pretty surprising if you can predict it ahead of time :)

Thanks @alexcrichton & @maxded, for the fast responses.
I did want to follow-up and say that I am still getting undefined symbols, even when consuming v1.0.64.
I'm not sure if the cargo build output (posted in the link) is going to be sufficient (though the reproduction steps are fairly straightforward).  Is there anything I can do to help confirm that ar is getting called with the proper flags?

Oh dear, sorry about that! I tested the OP of your issue and 1.0.64 works for me, but I'm running on macOS, not Linux. I'll test on Linux later today and see if ar does the same thing with the q flag that it does on mac.

Ok turns out Linux is indeed different. I believe https://github.com/alexcrichton/cc-rs/pull/570 should work for Linux.

er, gonna let the PR close this issue.

Ok turns out Linux is indeed different. I believe #570 should work for Linux.

I tried again with v1.0.65 on Linux, and it now works as expected.
Thanks again for all the help!


librispeech_decode : split save_results() -> save_asr_output() + save_wer_results()

THE IDEA

the idea is to support --skip-scoring argument passed to a decoding script
created for Transducer decoding (non-streaming, streaming) and CTC decoding (non-streaming)

OTHER CHANGES

also added --label for extra label in streaming_decode.py
and also added set_caching_enabled(True)

this has no effect on librispeech, but it leads to faster runtime on DBs with long recordings (assuming librispeech/zipformer scripts are the example scripts for other setups)

replaced some '-' with '_' and vice versa, to get more readable filenames with results

TESTING

still need to run the code to make sure it works properly (it is meld-ed from other recipe)
at this point, interested in getting early feedback from you...

Thank you.

Hi @csukuangfj ,
would it be ok to merge somethig like this PR ?
(adding --skip-scoring to the example decode scripts, it's handy for processing evaluation data)
What should be the next step ?
Best regards,
Karel

Looks good to me. Thank you for your contribution!


xrootd: 5.4.0 -> 5.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/xrootd/xrootd/releases.
meta.description for xrootd is: "High performance, scalable fault tolerant data access"
meta.homepage for xrootd is: "https://xrootd.slac.stanford.edu"
meta.changelog for xrootd is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/frm_xfragent had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdpfc_print had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdfs had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdcrc32c had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdadler32 had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/mpxstats had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdsssadmin had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/frm_xfrd had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/cmsd had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/frm_admin had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdshmap had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdacctest had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/test-runner had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdpwdadmin had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/frm_purged had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdmapc had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdcp had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrootd had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/cconfig had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdgsiproxy had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/wait41 had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdpinls had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrdgsitest had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin/xrootdfs had a zero exit code or showed the expected version
0 of 24 passed binary check by having a zero exit code.
0 of 24 passed binary check by having the new version present in output.
found 5.4.1 with grep in /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin
found 5.4.1 in filename of file in /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
xrootd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xrootd https://github.com/r-ryantm/nixpkgs/archive/0837d98624a5346e37029af0b602bd70812cc3b6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin
ls -la /nix/store/14vcprssc65bb6yal28ngvpn19031hs1-xrootd-5.4.1-bin/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xrootd

Maintainer pings
cc @ShamrockLee for testing.

Looks great.
Should I send a PR to remove the sed-based substitution to partially apply https://github.com/xrootd/xrootd/pull/1619, which is now included in the release.
https://github.com/NixOS/nixpkgs/pull/161743/files#diff-d06705002beb4906e0665665a63585ed1383184d34cfe657e7532390acfd8eb3R62-R66

Pushed the change, can we merge this when the build finishes?
P.s.: Looks like xrootd.passthru.tests fails on darwin but that is not regression... :eyes:
@ofborg build xrootd

@bobby285271 The test failure already happen on 5.4.0. The package itself is built on Darwin.
Not sure if there would be a problem for Hydra people. If not, it can be merged IMO.


Error URLs

Inspiration: https://rauchg.com/2016/addressable-errors
Suggested implementation:

Change log lines to accept an errCode param
Change console.warn to console.warn(message + ". Find help at https://github.com/pusher/pusher-platform-js/wiki/errors/" + errCode)
Change log calls to e.g. Logger.warn("AUTH_RESPONSE_NOT_JSON", ...);
Then the error lives at https://github.com/pusher/pusher-platform-js/wiki/errors/AUTH_RESPONSE_NOT_JSON
We may wish to put a proxy in the middle for nicer handling of non-existent error pages, e.g. https://errors.pusher.com/pusher-platform-js/AUTH_RESPONSE_NOT_JSON redirects to the above page

Done in #80


ocm: move commands to script file

Move all commands to script file, then copy this file into orchestration host and then run it.
Description
Fixes

Murali, Harshith, Russ
is it possible to merge this PR  now, so that CI can run it today. OCM CI is already broken, so it won't cause any regression :slightly_smiling_face:
These code changes are  running successfully in my playground http://venkataanil-test-nocp-airflow.apps.sailplane.perf.lab.eng.rdu2.redhat.com/dags/ocm/grid?dag_run_id=scheduled__2022-11-17T01%3A00%3A00%2B00%3A00&task_id=api-load


[enhancement]  HMR app.scss

The sass files in assets/sass are not live reloaded when a changes occured.
I'll try to fix soon this when I have time available.
My PR might be in angular-webpack-config repo.

@theomathieubhvr thank you very much for finding out this, seems like this feature need some enhancements.
Meanwhile I'll invite you as a contibutor to this project as well as angular-webpack-config project. And I will do the same if you're interested in others (almost all dependencies with ngx- prefix).

Ok, I'll do it ASAP. I've succeeded in doing this when you didn't use angular-webpack-config. I should be able to make it working with angular-webpack-config.

I know that angular-webpack-config seems to make things a little more complicated, but I'm depending on the same config on 10+ projects. Every time this config needs an update I had to make the changes on all those projects (which leads to mistakes, issues, etc.).
With angular-webpack-config changes are more or less centralized, increasing the version number works like a charm in most cases :smile:

Done with https://github.com/fulls1z3/angular-webpack-config/pull/1


[BUG] Deploy to cluster with windows nodes always fails

Describe the bug (🐛 if you encounter this issue)
longhorn will attempt to deploy code on windows nodes even if everything is set to run on linux in helm config
To Reproduce
Steps to reproduce the behavior:

make config with longhornManager.nodeSelector, longhornUI.nodeSelector, longhornUI.nodeSelector, longhornConversionWebhook.nodeSelector, longhornAdmissionWebhook.nodeSelector, longhornRecoveryBackend.nodeSelector, longhornDriver.nodeSelector all set to kubernetes\.io/os=linux
helm install ...
watch deployment schedule stuff on windows nodes

it will always attempt to schedule pod/discover-proc-kubelet-cmdline and pod/engine-image-ei-xxxxxxx-xxxx on windows
e.g.
longhorn-system    pod/engine-image-ei-f9e7c473-5g4gx                        0/1     ImagePullBackOff   0              20m    <IP_ADDRESS>   windows
Expected behavior
longhorn ignores windows nodes
Environment

Longhorn version: 1.4.2
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): helm

also tried setting defaultSettings.taintToleration=kubernetes.io/os:linux:NoSchedule and defaultSettings.systemManagedComponentsNodeSelector=kubernetes.io/os:linux

it is possible to fix the engine-image by manually editing kubectl -n longhorn-system edit DaemonSet/engine-image-ei-xxxx and adding nodeSelector section, but no clue about discover-proc-kubelet-cmdline

if I drain all windows nodes, it will work... not a very useful solution though

and every couple of minutes it will then reset the manual changes, which tries to start the pod/engine-image-ei and pod/longhorn-csi-plugin, and kubernetes somehow doesnt like that so it crashes the pod/kube-scheduler, which then crashes the pod/kube-controller-manager

Steps to reproduce the behavior:

make config with longhornManager.nodeSelector, longhornUI.nodeSelector, longhornUI.nodeSelector, longhornConversionWebhook.nodeSelector, longhornAdmissionWebhook.nodeSelector, longhornRecoveryBackend.nodeSelector, longhornDriver.nodeSelector all set to kubernetes\.io/os=linux
helm install ...
watch deployment schedule stuff on windows nodes

also tried setting defaultSettings.taintToleration=kubernetes.io/os:linux:NoSchedule and defaultSettings.systemManagedComponentsNodeSelector=kubernetes.io/os:linux

I think you for got to set the taintToleration for Longhorn UI, Longhorn manager, longhornConversionWebhook, longhornAdmissionWebhook,  longhornDriver
Brw, is your Windows cluster created by Rancher? If yes, we implemented a single setting that automatically apply all of these nodeSelector and taintToleration https://longhorn.io/docs/1.4.2/advanced-resources/deploy/rancher_windows_cluster/#deploy-longhorn-with-supported-helm-chart

not rancher, and I dont actually want longhorn on the windows machines, they are there for a legacy app that requires windows.
This is my complete config file for helm:
defaultSettings:
  defaultReplicaCount: 1
  systemManagedComponentsNodeSelector: "kubernetes.io/os:linux"
  taintToleration:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"

longhornManager:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux
longhornUI:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux
longhornDriver:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux
longhornConversionWebhook:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux
longhornAdmissionWebhook:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux
longhornRecoveryBackend:
  tolerations:
  - key: "kubernetes.io/os"
    value: "linux"
    effect: "NoSchedule"
    operator: "Equal"
  nodeSelector:
    kubernetes.io/os: linux

and it applies those just fine, but the discover-proc-kubelet-cmdline and engine-image are deployed from the manager.
ok, a few hours later while writing this message:
It seems the defaultSettings gets totally ignored. I can see that the configmap is created with the correct values, but when opening the settings page in the UI, it did not find them (blank values)
After I edit them, everything works just fine

It seems the defaultSettings gets totally ignored.

This is weird. Was it a fresh installation or an helm upgrade?

Fresh install, where I deleted the namespace multiple times to get fresh install (because uninstall doesnt work, just hangs forever)

(because uninstall doesnt work, just hangs forever)

Did you set this flag before doing Helm uninstallation?
I am trying to do a test to see if the defaultSettings gets totally ignored

For the  the defaultSettings gets totally ignored, I am trying to do a fresh Helm installation to confirm

For the defaultSettings get totally ignored on a fresh Helm installation. I am doing a test to confirm

Update: the setting gets applied. I couldn't see the mentioned issue. Closing the ticket. Feel free to reopen it if you need to discuss more


Koding now supports 2FA.

@mxxcon how often do you guys merge the changes?

@gniting look at https://github.com/jdavis/twofactorauth/wiki/FAQ_Merge for info about PRs to this repo.

:+1:


ERROR: Signature extraction failed

Here is my paste of this error:
youtube-dl https://www.youtube.com/watch?v=LaTbXphLhFk
[youtube] LaTbXphLhFk: Downloading webpage
[youtube] LaTbXphLhFk: Downloading video info webpage
[youtube] LaTbXphLhFk: Extracting video information
[youtube] LaTbXphLhFk: Downloading js player en_US-vflVlxFvV
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1001, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 915, in _extract_signature_function
errnote='Download of %s failed' % player_url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 503, in _download_webpage
res = self._download_webpage_handle(url_or_request, video_id, note, errnote, fatal, encoding=encoding, data=data, headers=headers, query=query)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 410, in _download_webpage_handle
urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 390, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1950, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 421, in open
protocol = req.get_type()
File "/usr/lib/python2.7/urllib2.py", line 283, in get_type
raise ValueError, "unknown url type: %s" % self.__original
ValueError: unknown url type: /yts/jsbin/player-en_US-vflVlxFvV/base.js
(caused by ValueError(u'unknown url type: /yts/jsbin/player-en_US-vflVlxFvV/base.js',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

You are using an outdated version of youtube-dl. Read this FAQ entry if you have troubles updating.


Fixed "Output" (#2202) - Some Remaining files

Replace "Output:" for "=>" in some remaining files
Related issue: #2202

Sure, here the changes


Show notification banner when role is upgraded to editor

What problem does this pull request solve?
This commit adds a banner which is shown to users when they first go to the index after their role is upgraded to editor. This uses the PaperTrail version history to establish if the role has changed to editor, and will create a new version the first time the role changes to editor, to prevent the banner being shown subsequently. I'm not sure this is the best way to achieve this, but it does avoid having to persist anything else (or anything feature specific) to determine if the the banner should be shown. It does also sound like this is a valid use case for PaperTrail.

Screenshot

Trello card: https://trello.com/c/IbfXhuYq

Things to consider when reviewing

Ensure that you consider the wider context.
Does it work when run on your machine?
Is it clear what the code is doing?
Do the commit messages explain why the changes were made?
Are there all the unit tests needed?
Has all relevant documentation been updated?

In the screenshot, it looks like the spacing between the title and body in the banner is not quite right. I think it should be a bit of a wider gap - like the trail account banner. (Sorry if this is already in hand!)

In the screenshot, it looks like the spacing between the title and body in the banner is not quite right. I think it should be a bit of a wider gap - like the trail account banner. (Sorry if this is already in hand!)

@hannahkc Good spot, I've resolved this now.


[stable/postgresql] default postgresUser should be in values.yaml

Is this a request for help?:
NO

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
Version of Helm and Kubernetes:
$ helm version
Client: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:55:54Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.10-gke.0", GitCommit:"16ebd0de8e0ab2d1ef86d5b16ab1899b624a77cd", GitTreeState:"clean", BuildDate:"2018-03-20T20:21:01Z", GoVersion:"go1.8.3b4", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/postgresql:0.9.2
The feature request:
The postgresql is hard to use as a dependency chart: some default values are not defined in the values.yaml, but instead in the templates using default. This means that in order to get the value from a parent chart, we must first redefine its value in the parent values.yaml.
https://github.com/kubernetes/charts/blob/4fb79536dfe6febb4dbfd73ff95d48c6bd2f24c5/stable/postgresql/values.yaml#L19-L23
https://github.com/kubernetes/charts/blob/4fb79536dfe6febb4dbfd73ff95d48c6bd2f24c5/stable/postgresql/templates/deployment.yaml#L41-L45
Take gitlab-ce as an example: they force a value for postgresql.postgresUser and postgresql.postgresPassword:
https://github.com/kubernetes/charts/blob/b71d5033a2e0dedd1c234be637ac5eda29807024/stable/gitlab-ce/values.yaml#L100-L101
https://github.com/kubernetes/charts/blob/b71d5033a2e0dedd1c234be637ac5eda29807024/stable/gitlab-ce/templates/secrets.yaml#L17-L18
Ideally we should be able to directly use .Values.postgresql.postgresUser without first redefine it.

/remove-lifecycle stale

:(


Documentation missing important information about the Active Directory Schema

Windows Hello for Business requires the 2016 schema.  This is accomplished two ways.  In key-trust, the schema is upgraded when you install the first Windows Server 2016 domain controller in the domain.  In Certificate-trust, the schema must be updated manually using the adprep.exe utility from the Windows Server 2016 media,

@MikeStephens-MS Thanks for this feedback Mike.
To document this, I think we will need more information. If you have any links to anything pertinent, that would be really helpful. In particular, do you have any pointers to information about exactly which schema must be updated? We already have a concept of extending the schema in Configuration Manager (for a different reason), and I don't want to confuse customers. Thanks!

Given we haven't seen an update, I'm going to close this issue. As needed, we can reopen it.


error  on debian(buster)

Installer: Error 2 (Binary folder not found)!
how to fix it

Can you show running in your terminal?


mobile/android: Remove rules_android_ndk patch

and ensure all ci is using the updated ndk

im a bit confused by this
we are definitely using the newer ndk:
ANDROID_NDK_HOME=/.android/sdk/ndk/26.3.11579264
ANDROID_SDK_INSTALL_TARGET=/.android
ANDROID_HOME=/.android/sdk
ANDROID_NDK_VERSION=26.3.11579264

but then when removing this patch it says:
-- The C compiler identification is Clang 9.0.9

and still fails with
    ld.lld: error: unknown -z value: separate-code

there is a rules_android_ndk api level setting which is still set to 21 - i had come to the conclusion that it was ignored but ill try changing it

changing api level had no effect
checking locally and we seem to have the correct clang bin
phlax@c39539a581db:/src/workspace/envoy/mobile$ $(./bazelw info output_base)/external/androidndk/toolchains/llvm/prebuilt/linux-x86_64/bin/clang --version
Android (11349228, +pgo, +bolt, +lto, -mlgo, based on r487747e) clang version 17.0.2 (https://android.googlesource.com/toolchain/llvm-project d9f89f4d16663d5012e5c09495f3b30ece3d2362)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /home/phlax/.cache/bazel/_bazel_phlax/b33947240e660e5d2c6450ec6397a5d1/external/androidndk/toolchains/llvm/prebuilt/linux-x86_64/bin

really struggling to understand where clang 9.0.9 is coming from

argh - so the problem is that the build is non-hermetic
running locally (with same rbe) and this passes
checking the rbe cache that ci is running with, and downloading the clang bin i see
$ ./clang --version
Android (7019983 based on r365631c3) clang version 9.0.9 (https://android.googlesource.com/toolchain/llvm-project a2a1e703c0edb03ba29944e529ccbf457742737b) (based on LLVM 9.0.9svn)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /home/phlax/dev/envoydev/envoy/.

not sure how to resolve that

argh - so the problem is that the build is non-hermetic

the problem was actually that ndk gets installed another way - we need to ensure these get updated together in future

so there were two ndks being installed ?


How can we integrate Google Adsense?

Dear Saltcorn Creator,
I have started using your product for personal projects, and let me say this, I am loving it.
You have done an amazing job.
I wanted to know is there a possibility to include ads from Google adsense.
Let's say, i wanted to create an application free for use for the internet users, but I would like to register the site on Google Adsense do to display ads so that i can generate some ad revenue if possible to keep the app alive.
Is there a way in which achieve this?

Hi @TheEngrNest to start with, does it work to use the Code element in the page / show view builder for the individual ads (i.e. the code you would drop would be the <ins...></ins> (https://support.google.com/adsense/answer/9190028?hl=en-GB&ref_topic=28893) and script tag at the top (https://support.google.com/adsense/answer/9274516?hl=en-GB) you can put under Settings ->About application -> Custom HTML.
If this works (or doesnt work...) I could write a plugin module to make this simpler


Specify tensorflow version

Hi, I'm using tensorflow 2.3.0, could you share the exact libraries version in requirements.txt please?

Hi, it’s on the readme on GitHub page
From: Steven Parra @.>
Sent: Monday, November 28, 2022 1:00 AM
To: @.>
Cc: @.***>
Subject: [kochlisGit/ProphitBet-Soccer-Bets-Predictor] Specify tensorflow version (Issue #2)
Hi, I'm using tensorflow 2.3.0, could you share the exact libraries version in requirements.txt please?
—
Reply to this email directly, view it on GitHubhttps://github.com/kochlisGit/ProphitBet-Soccer-Bets-Predictor/issues/2, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALUJURGB2M7LWRKOBHWUGXLWKPRZTANCNFSM6AAAAAASMX4B2M.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@kochlisGit Which Python version should we use?

Hello, Python version between 3.6 and 3.10 is recommended.
From: Muhammed @.>
Sent: Saturday, December 3, 2022 2:54 AM
To: @.>
Cc: Vasileios @.>; @.>
Subject: Re: [kochlisGit/ProphitBet-Soccer-Bets-Predictor] Specify tensorflow version (Issue #2)
@kochlisGithttps://github.com/kochlisGit Which Python version should we use?
—
Reply to this email directly, view it on GitHubhttps://github.com/kochlisGit/ProphitBet-Soccer-Bets-Predictor/issues/2#issuecomment-1335992933, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALUJURFOAIMUJXAG7FMOUS3WLKK55ANCNFSM6AAAAAASMX4B2M.
You are receiving this because you were mentioned.Message ID: @.***>


Duplicate date 27th October 2024

There is a duplicate of October 27. 2025.
I've verified it at https://werner-scholtz.github.io/kalender/ in Month and Multi Week view.
What i have seen so far:

double; once with 00:00 and once with 23:00

days after 27. have time 23:00 instead of 00:00
events added to a day with 23:00 will cause an exception in my project
on this day daylight saving time changes from CEST/MESZ to CET/MEZ
haven't seen any other day duplicates so far
the exception comes from here: https://github.com/werner-scholtz/kalender/blob/50043322a49ade38194ae9c477c5691bcb697294/lib/src/components/layout_delegates/multi_day_event_group_layout.dart#L62

flutter --version
Flutter 3.24.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 2663184aa7 (5 weeks ago) • 2024-09-11 16:27:48 -0500
Engine • revision 36335019a8
Tools • Dart 3.5.3 • DevTools 2.37.3

pubspec.yaml
name: name
description: "description"
publish_to: 'none' # Remove this line if you wish to publish to pub.dev
version: 1.0.0+1

environment:
  sdk: ^3.5.1

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.8
  provider: ^6.1.2
  path_provider: ^2.1.4
  uuid: ^4.5.1
  datetime_picker_formfield_new: ^2.1.0

  intl: any
  sqflite: ^2.4.0
  path: ^1.9.0
  share_plus: ^10.0.2
  flutter_archive: ^6.0.3
  file_picker: ^8.1.2
  open_file: ^3.5.8
  awesome_notifications: ^0.9.3+1
  flutter_localizations:
    sdk: flutter
  #flutter_carousel_widget: ^3.0.1
  url_launcher: ^6.3.1
  mime: ^1.0.6
  io: ^1.0.4
  image_picker: ^1.1.2
  flutter_colorpicker: ^1.1.0
  flutter_oss_licenses: ^3.0.2
  photo_view: ^0.15.0
  package_info: ^2.0.2
  kalender: ^0.4.1

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^5.0.0

flutter:
  generate: true
  uses-material-design: true

The cause seems to be in
https://github.com/werner-scholtz/kalender/blob/50043322a49ade38194ae9c477c5691bcb697294/lib/src/extensions.dart#L23-L51
I assume there is a problem with DateTime.add over daylight saving time.
Example:

I think i found the cause and a possible solution.
The cause seems to be that DateTime.add(Duration(days: i)) adds i*secondsPerDay seconds instead just a day.
Using DateTime(year, month, day+i)  incremets the day directly. It also works across months and years.
For example DateTime(2024,12,31+1) will result in 2025.01.01.
https://dartpad.dev/?id=2aa8b0fa0e3dda67af02fca9000f2344
void main() {
  DateTime start = DateTime(2024,10,27);
  DateTime end = DateTime(2024,11,2);
  List<DateTime> dates = [];
  int diff = start.difference(end).inDays.abs();
  for(int i=0;i<diff;i++){
    dates.add(DateTime(start.year,start.month,start.day+i));
  }

  print(dates.join('\n'));
}

created a pull request to solve this.
https://github.com/werner-scholtz/kalender/pull/94


Move to Smyte org action items

[x] Injectable autoprefixer (@petehunt)
[x] Version → 1.0 (@petehunt)
[x] Make clear that the webpack loader is experimental (@petehunt / @meyer)
[x] New, simplified docs (@petehunt / @meyer)
[x] LICENSE (Apache 2) (@petehunt)
[ ] Roadmap (@petehunt / @meyer)

I would like to be a contributor.

@hnordt we’re working on the roadmap at the moment. Once it’s done it’ll be a good place to see areas where you can contribute. In the mean time, feel free to open issue or submit a PR if there’s a feature that you’d like to see.

Any update on roadmap? Mostly curious: will a lot change API-wise in next version?

@natew the API will remain the same; updates to jsxstyle will build on the existing API.
Honestly, I’ve had a hard time coming up with a roadmap because jsxstyle is pretty close to what we need both at Smyte and in my personal React projects. As far as what’s next: I’d like to put some effort into performance and reliability. We do a few sensible things, like garbage collecting unused styles, but I’d like to make sure we’re doing everything we can to ensure that jsxstyle does what it needs to do as quickly as possible.
I’ve created a Github project for jsxstyle 2.0 features. If you’ve got any ideas you’d like to share, feel free to add them here or in a new issue.


Count number of host cores and suggest more cpu's for guest

On Windows: wmic cpu get NumberOfCores  or $NUMBER_OF_PROCESSORS
On OSX: sysctl -n hw.ncpu
On Linuc: nproc
Git, make and several other steps can really use a lot of cores.  Even including hyper threads helps with compile (e.g. 1 Proc, 4 Cores with 8 Hyper Threads, v.cpus=8 works well)

Windows ram:
 wmic memorychip get capacity|awk 'NR > 1 {x = x + $1 } END { print x / 1024 / 1024 }'

$ wmic memorychip get capacity|awk 'NR > 1 {x = x + $1 } END { print x / 1024 / 1024 }'
16384

Ram on OSX:
$ sysctl -n hw.memsize|awk '{ print $1 / 1024 / 1024 }'
12288

Cores on OSX:
$ sysctl -n hw.ncpu
16