PascalCase has leading capitalization, camelCase does not 🐪

Because it looks like a camel with a single hump in the middle: 🐪

We should probably get @killercup's permission

There are no emoji for higher-order-camels

My favorite out-of-context quote for the month.

FWIW, I personally prefer to call these "lower camel case" and "upper camel case"

I've seen them called lowerCamelCase and UpperCamelCase for many years in both C++ and JS projects before a single PascalCase reference (which confused me at first). YMMV of course.

Personally, my vote is for :camel: case, which has been what I have always heard it called...

@alexcrichton @BurntSushi This is a pretty low effort RFC to nominate, either of you care to bring it up at the next libs team meeting?

I don't think we should call this "PascalCase"; informally, I'd say that I don't think that term has nearly the name recognition. And even "UpperCamelCase" doesn't seem clear except insofar as being self-documenting. We should just explain and/or define the terms we use in detail the first time we use them, which will make it clear even to people who haven't heard any of the terms before. We give examples in a table, but I don't think we have an explanation anywhere that actually says "capitalize the first letter of each word".

I may have forgotten about this PR for a while. I've removed all PascalCase and replaced with UpperCamelCase. When this is merged, I'll fix the api-guidelines repo to match.

Rust is the first time I've heard of UpperCamelCase in 500 years of coding. I'm sad it's not called PascalCase here too.

@nodingneu that's wonderful! I'm glad that we got to be a small part of your learning today. I hope that when I am 500 years old I am still able to learn something new every day as well. Have an enriching learning journey!


Feature/get categories

📢 Type of change

[ ]  Bugfix
[x]  New feature
[ ]  Enhancement
[ ]  Refactoring

📜 Description
First rest endpoint created to retrieve categories from DB, with MVC structure,
Dto mapped with mapstructs and JpaRepositories
💚 How did you test it?
CategoryRepository tested with @DataJpaTest

Postman requests

📝 Checklist

[x]  I compiled before submitting the PR
[ ]  I reviewed the submitted code

🔮 Next steps
Create addProduct rest endpoint

Refused because of Ci doesn't build


Lancedb as  retriver

I have added support for
langchain : https://python.langchain.com/docs/integrations/vectorstores/lancedb/
lancedb:https://github.com/lancedb/lancedb
#note i haven't added poetry.toml with langchain-community
@vbarda  can you please check this pr. thank you

@vbarda any chance to check it please


[Bug]: punycode deprecation warning on Node 22 LTS

Version
29.7.0
Steps to reproduce
Run any test suite with jest environment jsdom using Node 22.
Expected behavior
No deprecation warning is printed.
Actual behavior
The following warning is printed before tests run.
(node:57226) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)

Additional context
This problem is also mentioned in https://github.com/jestjs/jest/issues/15148, but while upgrading jest-environment-jsdom is a possible solution, I'd like to specifically track the deprecation warning happening on the stable release (#15148 seems to be about the alpha release).
Environment
System:
OS: macOS 15.0.1
CPU: (12) arm64 Apple M2 Pro
Binaries:
Node: 22.11.0 - ~/.local/state/fnm_multishells/98505_1730127764871/bin/node
npm: 10.9.0 - ~/.local/state/fnm_multishells/98505_1730127764871/bin/npm
npmPackages:
jest: ^29.7.0 => 29.7.0

relevant https://github.com/jestjs/jest/pull/15276

(no activity indeed)

Jest feels abandoned ):

@ginosanterre https://github.com/vitest-dev/vitest is currently the most popular option

Anyone successfully patch this?

This clumsy npm overrides works for our app. It also fixes punycode problem in other our deps. packages.
Because punycode problem is warning only and will be same for a long time, I hope it will be fixed in jest@next at least. 🙏
 "overrides": {
    "psl": "npm<EMAIL_ADDRESS><EMAIL_ADDRESS>{
      "whatwg-url": "12.0.1"
    },
    "jest-environment-jsdom": {
      "jsdom": {
        "whatwg-url": "12.0.1"
      }
    }
  }


Use -an -sn in ffmpeg to discard audio and subtitles streams

https://github.com/nicknsy/jellyscrub/commit/146545672b30bb641e06707773d1fef4910525b9#commitcomment-100693737

Fixed by https://github.com/nicknsy/jellyscrub/commit/debbcd2c304e2426bb09a997192f0ed1f9906fba


updating the olm config file with proper information

Issue #, if available:

Fixes: aws-controllers-k8s/community#1556

Description of changes:
Updating the olm config with proper information. operator-sdk is expecting a URL value in specs.links.url this is why the build was failing.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Signed-off-by: Adam D. Cornett<EMAIL_ADDRESS>

/ok-to-test

/hold

/unhold


Fix release build, update dependencies

While trying to use this driver in the PXFLOW board support crate, I found that read_registers worked fine in dev build but not in release.  In release the driver would wait forever blocking on SPI read.
Rewriting that function to accept a provided buffer and remove the unsafe mem::uninitialized call  seems to fix the problem. I went ahead and updated the crate with newer dependencies as well.
Tested dev and release builds on PX4FLOW hardware.

The symptoms I saw sound similar to #6  -- it appears any call to read_registers would fail

bors r+


New version: Tencent.TencentMeetingRooms version <IP_ADDRESS>9

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115210&drop=dogfoodAlpha

@wingetbot run


macOS version is Catalina,the location path use external disk, the aerial can't work.

macOS version: Catalina 10.15 (19A602)
Aerial version: 1.6.3
When I change cache location to external disk the aerial is not work. The path like '/Volumes/ColdiceSD/Aerial'. When I change cache location to Mac Internal disk, the aerial screensaver can work.
2019-10-27 00:03:22 : Error while enumerating files /Volumes/ColdiceSD/Aerial: The file “Aerial” couldn’t be opened because you don’t have permission to view it.
When I change the path to my Mac disk, the Screensaver was worked. But the video tab bar still can't preview the video. Like the pic.

Hi @coldice945
This is no longer possible sadly, this is a restriction imposed by Catalina for the moment and there's no workaround (besides disabling SIP which is not a good idea for many reasons). You can only set a cache on your primary disk right now.
You can read more about Catalina limitations in this issue : https://github.com/JohnCoates/Aerial/issues/801

Hi @coldice945
This is no longer possible sadly, this is a restriction imposed by Catalina for the moment and there's no workaround (besides disabling SIP which is not a good idea for many reasons). You can only set a cache on your primary disk right now.
You can read more about Catalina limitations in this issue : #801
Edit: About the preview, please System preferences (close and reopen), this will fix it.

SIP is disabled on my end. How do I enabled cache folder on a different partition?

Hi @NewsGuyTor !
Go to Caches tab, then click "Change cache location". You'll have to type the path to your volume (so something like "/Volumes/Some volume/Some path"). Then quit syspref and restart it. You need to create your folder before you set it.

That didn't work for me. I got it working like this though:

Create a new cache folder whereever you want
You move the contents of "/Users/username/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application\ Support/Aerial" to your new cache folder.
Then you delete the old Arial folder
Then you run "ln -s /Volumes/Newlocation/Aerial /Users/username/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application\ Support/Aerial" in iTerm

Right, my apologies. I gave you the instructions in case you had previously downloaded all the videos and wanted to move the cache folder. By default it only downloads in the container so your symbolic link solution is the correct one. Sorry again for sending you on the wrong path!


Shopify Integration

It would be great if this repo could support shopify for order fulfilment as an example.

I totally agree it would be awesome a Shopify integration

Something like this project? https://github.com/zeon-studio/commerceplate

@surjithctly I tried to use the Shopify Storefront API, but it's been difficult to add it to the current backend code. I previously worked on a project with the Shopify Storefront API in Next.js, which you can check out here: https://github.com/zeon-studio/commerceplate.


create scanpy.object by "CreateSeuratObject(pbmc.matrix, meta.data = pbmc.MetaData)"

sorry to start this issue, but i failed in all the methods i found to transport a seurat.object into scanpy, including loom file, SingleCellExperiment ...
hence, i wonder if scanpy have the function like seurat that can create an object by "CreateSeuratObject(pbmc.matrix, meta.data = pbmc.MetaData)".
because i can successfully export the matrix and meta.data from seurat.object, and the meta data is the key information i want to import into scanpy for further analysis.

Hey!
Check out anndata2ri and how to use it to convert Seurat objects into scanpy on the fly in a Juptyer notebook here. This is the easiest way to convert between the two objects. However, going via loom files should also work. If you search the scanpy issues (also closed issues) you should find some examples.

anndata.AnnData()

Thank you for you suggestion! I failed all other methods, including anndata2ri. God know why I encounter so much error. And I've posted issue in the specific place.

Great, so I assume this is fixed?


[Feature] Add setting for cursor_max_speed

Cursor acceleration is wonderful for moving large distances but it also makes 2 things very difficult

fine/precise mouse movements in a small area
ie. the timing for each press of a movement key [h,j,k,l] has very small margin for error

using the numeric prefix buffer has inconsistent results
ie. typing 20j should move the cursor the precise same amount each time, but acceleration yields mixed results

@qaptoR
About cursor_max_spped
cursor_max_speed was removed when the version was upgraded from 4 to 5. This is because it is difficult to distinguish between close and long distance moves in an adaptive way, and its presence experimentally had little effect on moves.
20j does not return stable results, but this is probably a bug and can be fixed.
For comfortable and accurate movement.
Lately, I have been feeling that discrete movement is better than the continuous mouse movement that we currently have.
For example, #115 and #92 are EasyMotion-like. In addition, it might be better to move a fixed amount of width, like text.
Please give me your opinion.

I concur that discrete movement would be better that any movement at a particular speed. Realistically, we know where we want the mouse to go, so if we can get it there in one or two motions rather than a time-while-pressed sensitive input as it is currently then I think it will feel more vim-like and thus more in line with what feels good to our muscle memory.
To solve this, I think there needs to be 3 layers:
Layer 1: #115 - making large jumps around a monitor, or between monitors should feel impressive. But the gradations between cells ought to be maybe 1/12th the screen width / 1/8th the screen height. This leaves room for layer 2.
Layer 2: #92 - just as shown in the example, there should be roughly 25 cells which makeup this layer. That way users will inevitably become familiar with this motion as a commonly reached for tool. This layer should be serviceable as a motion, eg. <92>4h should move the cursor in that 'left and slightly up' direction 4 times.
Layer 3: updating the current implementation of hjkl movement. - since layer 2 will still be somewhat sizeable portions of the screen, very fine and precise movement will be necessary. I believe this should be pixel perfect. press h once, move left 1 pixel. There should be NO press and hold to move (or at the very least it should be default 'off' with an option to turn on). This will encourage the use of 5h, 10h, etc, and users will learn and be able to trust their judgement when jumping around in a small area.
This way, any motion should be completeable in 3-4 motions:
eg: <115>ar <92>h 5h5j
** <115> and <92> are representations of whatever the command name will be to invoke those motion overlays **

@qaptoR
Thanks for your detailed consideration.
This is a very reasonable suggestion. Looks very good!
Then we should add this feature as a priority.
I'm a little busy this week, but will aim to release this feature by the end of this month.
About these layers
As you said, the number of cells should be fixed, e.g., set l1gridsize=12x8. This makes the deterministic assignment of hints and allows us to jump to any position with macros. Here in layer 1, the first hint should be assigned to each monitor uniquely. For example, the hint for monitor 1 starts with A, the hint for monitor 2 starts with B, etc.
What should be discussed is layer 2 and layer 3. Layer 3 said per pixel, but since the size of the pixel varies depending on the screen size and DPI, layer 3 will converge to layer 2, which cell size is only different. Therefore, I think Layers 1 and 2 are enough.
For precise movement, the dynamic grid size, which could be gradually small as in Grid Mode of warpd, can be applicable. However, instead of starting from the entire screen like warpd, we start from a local small scale, assuming a combination with layer 1.
Thanks.

I see your point about screen resolutions being a potential issue, and I think you're suggestion of something like warpd could work well as an alternative.
So really it's just layer 1 followed by a warpd style layer 2? I think that could really work.

@qaptoR
That is correct.
For now, we should make its prototype in the gridmove branch and then discuss the need for layer3.
Of couese your pull request is welcome. I'll also use pull request for its implementation.

This issue's title does not fit discussions of its implementation, so I created a new discussion https://github.com/pit-ray/win-vind/discussions/155.

@qaptoR
Today, I released v5.1.0-beta including layer-1.
Please try it.
https://github.com/pit-ray/win-vind/discussions/155#discussioncomment-5766623

@pit-ray
I have tried the beta version, I am very impressed how it works. There are a few bugs I'm noticing on my system though.

the grid only shows up on two of my monitors (1 and 4). I think this is because all 4 of my monitors are 1080p, 1 and 4 are 32 inch that I run at 1440p resolution so that the relative visual pixel density is that same as 2 and 3 which are 22 inch monitors.
So, it could be the reason that 1 and 4 are showing the grid only is because they both share the same resolution and win-vind is basing the grid display on the main monitor 1.

on monitor 4 the grid text is very choppy and broken. I can't get a screenshot, but it's very hard to read. I tried increasing the text size but that did not help. it might also be related to the next thing I noticed

the text on monitor 1 was jittery and some of the letters were swapping very very fast. for example the letter for one grid space DAG was switching to BAG very fast over and over again. It made me wonder if that was related to issue number 1. because I think each monitor gets it's own letter correct? monitor 4 starts with A, and monitor 1 starts with D, but maybe because the two other monitors don't show anything it's trying to reassess and make monitor 1 B, since ABCD for 4 monitors but only AB for 2 monitors.

Let me know if I can provide any more information. I can try and get a photo with my phone of the text on monitor 4 to show the glitching and post it if you'd like.


name change of repo to pact-cypress-adapter

Just to keep it in line with the published package
https://www.npmjs.com/package/@pactflow/pact-cypress-adapter
and
https://www.npmjs.com/package/@pactflow/pact-msw-adapter
👍🏾

Thanks @B3nnyL 👏🏾


Issue using an external js library.

Hi, I have the following issue while trying to use an external js library in a script:
Based on the wordcount example:
function runSimple(sparkContext) {

    var Tuple2 = require('eclairjs/Tuple2');

    var file = ((typeof arguments !== "undefined") && (arguments.length > 1)) ? arguments[1] : "samplefile.txt";

   var rdd = sparkContext.textFile(file).cache();

   var turf = require("./lib/turf.js");

    var rdd2 = rdd.flatMap(function (sentence) {
        var split = sentence.split(" ");
        return split;
    });

    print("rdd2 count:" + rdd2.count());
    print("rdd2 collect:" + rdd2.collect());

    var rdd3 = rdd2.filter(function (word) {
        print((typeof  word) + ':' + word);
        return word.trim().length > 0;
    });

    var rdd4 = rdd3.mapToPair(function (word, Tuple2) {
        return new Tuple2(word, 1);
    }, [Tuple2]);

    var rdd5 = rdd4.reduceByKey(function (a, b) {
        return a + b;
    });

    var rdd6 = rdd5.mapToPair(function (tuple, Tuple2) {
        return new Tuple2(tuple._2() + 0.0, tuple._1());
    }, [Tuple2])

    var rdd7 = rdd6.sortByKey(false);
    return rdd7.take(10);

}

The sample file used in this case only contains (ignore the begin file and end file):
--begin file--
a b
--end file, no new line after b, just a, b--
The printed values of evaluating the function runSimple():
rdd2 count:3
rdd2 collect:a,b,[object Object]
string:a
string:b
object:[object Object]
The expected count would be in this case 2, and only a and b should be printed:
If the library turf.min.js is not loaded (or even moved down to the end of the script), the problem goes away, which indicates it has some kind of interaction.  The extra object aparently corresponds to new lines and end of file, as adding new lines increases the counter in the same amount.
The library I am including is Turf.js.
Any advice for solving this issue?
Thanks in advance.

@jtbarbetta module loading question

Turf.js is for node.js or browsers, Eclairjs-nashorn runs on the Spark driver and workers and is meant for batch type operation. If you are attempting to write a node spark interactive  application  that uses Turf.js you need to use EclairJS-node. EcairJS-node will allow you to use Turf.js

Thank you for answering this Bill!

Thank you very much for your answers, I was hoping to be able to use Turf for using the  geo-spatial analysis capabilities. Indeed I have tested a simple example running on top of eclairjs-nashorn and seems to work properly. It is now clear for me that it is introducing some other issues due to the fact that is for running on the browser o on node.js.  Thanks again!


Fix code scanning alert - Pinned-Dependencies

Tracking issue for:

[x] https://github.com/gilbertalgordo/workbox/security/code-scanning/87

https://gist.github.com/gilbertalgordo


chore: Migrate iot synth.py to bazel

The changes in grpc stubs are caused by the gRPC upgrade from 1.10 (more than a year old) to 1.27 (same version which is used as runtime dependency)

@chingor13 PTAL


[WIP] mean_variance with exclude_zeros

WIP alert: this only works for mean. Variance isn't implemented yet.
Notes: implementing the same approach in MeanVarianceAccumulator would require significant work since the method supports batching and the division is implemented down in the numba methods. Since batching isn't required by the mean_variance method, we could just rewrite a MeanVarianceAccumulatorUnbatched that also supports exclude_zeros. Otherwise, this can be implemented even for batching but it will require some refactoring of the code and will take more time. Thoughts?

Notes: implementing the same approach in MeanVarianceAccumulator would require significant work since the method supports batching and the division is implemented down in the numba methods. Since batching isn't required by the mean_variance method, we could just rewrite a MeanVarianceAccumulatorUnbatched that also supports exclude_zeros. Otherwise, this can be implemented even for batching but it will require some refactoring of the code and will take more time. Thoughts?

Actually, I think it is pretty easy, as the current code already counts the number of values per variable (equivalent to what you call denominator in this PR -- called n in the existing code), and it already corrects for the difference between the non-zero values and the total samples (see the code in _mbomv_sparse_correct_batches)
I believe that it may be as simple as not calling _mbomv_sparse_correct_batches in the new mode.  _mbomv_sparse_correct_batches is simply updating all the batch accumulators for the implicit zero-valued (i.e., sparse or missing) array elements.

One pervasive confusion I have with this PR is if we are adding a flag to:

ignore zeros by value
or ignore nnz (i.e., ignore implicit zeros, but include explicit zeros)

The latter is coded. But the argument flag (exclude_zeros) is ambiguous.
SOMA defines nnz as explicitly written values, even if that value is zero. The Census never intentionally writes a zero, but given that this API operates on SOMA data, the flag should make it clear which is implemented, and make the code / arg names match.
My personal opinion is that the latter (ignore implicit zeros) is what you want, for both API consistency and performance reasons.

spoke with @pablo-gar and we can go with ignoring nnz (which is what the code currently implements), and we'll add a note to the docstrings saying that the census doesn't add explicit zeros. This should hopefully clarify doubts that users may have with respect to this issue

go with ignoring nnz

great. The additional change required, besides docstrings, is unit tests.  They are a bit muddled on which they test for - there should be an explicit test with and without explicitly stored zeros.


Idea: ?_group_count=country to return counts by a specific column

Imagine if this:
https://stateless-datasets-jykibytogk.now.sh/flights-07d1283/airports.jsono?country__contains=gu&_group_count=country
Turned into this:
https://stateless-datasets-jykibytogk.now.sh/flights-07d1283?sql=select country, count(*) as group_count_country from airports where country like '%gu%' group by country order by group_count_country desc
This would involve introducing a new precedent of query string arguments that start with an _ having special meanings. While we're at it, could try adding _fields=x,y,z

This should support multiple columns, e.g. ?_group_count=precinct,candidate

Should this support sum/avg/etc as well?

It would be great if this could support foreign key references and automatically resolve and hyperlink them if they are detected.

This would enable faceted search - moving it to the search milestone.

Some demos:
Single column: https://sf-trees-flat.now.sh/sf-trees-flat-ba738ce/Street_Tree_List?_group_count=qSpecies
Multi column: https://sf-trees-flat.now.sh/sf-trees-flat-ba738ce/Street_Tree_List?_group_count=qLegalStatus&_group_count=qSpecies

Remaining work for this is tracked in #150


Post editing doesn't work when OldLander is enabled

Describe the bug
The edit button just does not function at all.
To Reproduce
Steps to reproduce the behavior:

Create a Reddit post
Click on the edit button (pen icon)
Nothing happens
With the MobiDevTools addon installed, the console shows this error: Script error at https://www.redditstatic.com/reddit-init.en.53pfE9v1ZNc.js:0:0

Expected behavior
The button should work.
Screenshots

Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):

Device: Nokia G10
OS: Android 11
Browser: Firefox
Version: 131.0.2

Reddit page where bug can be easily reproduced
N/A
Additional context
When I disable OldLander, the edit button on regular old.reddit.com works fine.

Console errors from Firefox on desktop

@OctoNezd too busy?

Afraid so, if you can do the fix I would be grateful.

Sorry, I tried looking into it today, but I couldn't figure it out.
The error is coming from a JS file that included in Reddit and not the extension.
It happens when the jQuery method .get() is called.
I tried checking whether or not OldLander changes the node structure under the .usertext-edit element, it doesn't look like that's happening.


⚠ Bofrost.at has degraded performance

In 51c155d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1048 ms

Resolved: Bofrost.at performance has improved in 264f6d8 after 24 minutes.


[stable/kiam] Added 3.0 rc1 version support

What

Support kiam v3.0-rc1

Why

tls problem fixed in that release

@mattfarina @mgoodness Can you check this PR?

/ok-to-test

/ok-to-test

/ok-to-test

@goruha I have been working on kiam v3 changes over here: https://github.com/ewbankkit/charts/tree/kiam-v3.
There are a few more changes than the name of the binaries.

@goruha My branch has been rebased.

@ewbankkit Done.

@mattfarina I addressed your comments.
Can we merge it?

@goruha I just copied from here.

@goruha We typically don't want release candidates as the installed version for a stable chart. I've added the topic to the next charts chat meeting so we can discuss.

@goruha Can you incorporate the change to server.yaml made in https://github.com/uswitch/kiam/pull/146? Thanks.

Just tried using this chart, the rebase with the branch from @ewbankkit has created a broken Helm chart, because his branch is referring to command line flags of kiam (specifically --whitelist-route-regexp) that don't exist in kiam v3.0-rc1; @ewbankkit's changes in kiam's repo were merged into a side branch api-v3, not the master where v3.0-rc1 was tagged from.

Hey all, would it be possible to get https://github.com/helm/charts/pull/7762 merged before this chart? Mine is simple: add the Prometheus metrics endpoints for the agent and server Services to be compatible with Prometheus operator ServiceMonitor resources.

Kick @mbarrien @scottrigby @ewbankkit @linki @sameersbn @mattfarina See above

@ewbankkit I splash all commits included from your repo.
Because I had to signoff all commits, and I do not know to to do that for commits that are 1 month deep in the git history.
Are there any concerns about this from your side?

@mattfarina comments addressed

/lgtm


Accessing claims from IS4 from ASP.Net MVC application and not ASPNet core application

To do a POC I created an API, MVC client, and IS4 on Asp.Net core and things work just fine.
OpenIdConnectOptions, as shown in the screen shot(https://puu.sh/C85fs/b60d0f0e37.png), is a class from Microsoft.AspNetCore.Builder and has properties like GetClaimsFromUserInfoEndPoint.
Problem is my IS4 is built on Asp.Net core but my MVC client is on ASP.Net MVC 4 framework and there we have OpenIdConnectAuthenticationOptions(https://puu.sh/C85qj/86676a62ee.png) which is class from Microsoft.Owin.Security.OpenIdConnect namespace and does not have the same property and to call UserInfoEndpoint, I think I will have to call it manually.
Am I missing something or Is this the way to do it?
Similarly to read the claim defined here in my user, it is very straightforward in ASP.Net core MVC client like below using HttpContext class(https://puu.sh/C85yf/6098ff9e9f.png)
https://puu.sh/C85vR/6304fb836d.png
while in order to do the same thing in a Non-core application, it is really difficult and in fact, I am not getting all claims defined in the user on the IS4 level(https://puu.sh/C85Ba/09c479ef18.png) in identity variable shown here https://puu.sh/C85DL/d8e87f711c.png
What is the best way to consume IS4 from an ASP.Net MVC 4 application?  MVC client created on Asp.Net core is working completely fine but I have an old application which does not use ASP.Net core
Please suggest.

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use one of the our free or commercial support options
See here for more details.
Thanks!


fix(openamt): fix IsFeatureFlagEnabled, rename MPS Url to MPS Server [INT-6]

Backport fixes from latest commit in EE

yup LGTM


Add ESLint

Address the issues cited in review of #232 , rebased the branch on master, removed the merge commit and squashed 644b36d.

Right on, thanks!


Add request listeners

This PR adds listener logic to GoogleR object. A listener is called
before every request. No data is passed to listener.

Elegancki PR!!!!


[FeatureReq - Partner Center] Programmatically Renew API Keys in Publish API v1.1

Is your feature request related to a problem? Please describe.
I understand that a recent change was made to move from access tokens to rotating API keys to enhance the security of the Microsoft Edge Publish API. These API keys now expire every 72 days to reduce the risk of compromised credentials.
As it stands, there is no way to renew these keys programmatically.  This means that every 72 days, a developer must manually log into the Partner Center, regenerate the API keys, and then manually update them in their CI/CD pipelines.
This manual process presents several challenges:

Increased Risk of Human Error: Manual key rotation increases the likelihood of mistakes, such as misconfiguration, forgetting to update the keys in all necessary locations, or forgetting to renew the keys entirely.
Scalability Issues: For small teams or projects that are maintained infrequently, this requirement adds an unnecessary operational burden.
Potential Security Risks: Ironically, the manual process may introduce security risks if keys are shared insecurely during the update process.

Describe the solution you'd like
I would like to have a way to programmatically renew API keys through an API endpoint or automation script.
Implementing this feature would:

Enhance security by allowing for more frequent key rotations without additional manual overhead.
Reduce the risk of deployment failures due to expired keys.

Hey @gabriel-stackhouse, thank you for your suggestion. We are looking into this and will let you know as soon as we have an update from our team.
Thank you.

Hi @gabriel-stackhouse, we got an update from our team and would like to inform you that we have added this feature request to our upcoming feature list. However, at this time, we cannot commit to an exact timeline for its implementation. Rest assured, we recognize the importance of this feature and will prioritize it accordingly.
Thank you for your understanding and patience.

@ManikanthMSFT Thank you for the update - I look forward to this feature being implemented!


2.0 Update

Update to USWDS 2.0!
Things that still need to be fixed...

[x] Buttons
[x] Form Controls
[x] Navigation
[x] Accordions
[x] Alert paragraph guidance
[x] Footer logo size and positioning
[x] Sample sites on the layout page

Looks like there might be a minor error on the tooltip hover css:

noticing that accordion headers that are closed look the same as ones that are open. I'll look into that. too along with the tooltip


User Can Logout

User will be able to logout
Logging out will redirect to login page
User should see a message that they’ve successfully logged out
Session should be destroyed (so the back button doesn’t allow them back in)

https://github.com/ReyMArias/IT202009/commit/b9ac696f83a3f4aa2588316a47fb69bdc6699d33#diff-f44dac6748c8e40807f31190d6a6db8b99011f2196be94bf97fe029d172170a2

https://github.com/ReyMArias/IT202009/pull/48


Why targets=actions?

In https://github.com/crazydonkey200/neural-symbolic-machines/blob/master/nsm/agent_factory.py

In https://github.com/crazydonkey200/neural-symbolic-machines/blob/master/nsm/model_factory.py
I guess targets should at least be related to rewards.
Thank you!!

The reward is provided through the weights. The targets are the actions. You can think of REINFORCE as a weighted log-likelihood training where the reward or advantage times probability is used as the weight and the action is used as the target. You can see this

thank you very much


react-popper and react 15.6.1 and react-dom 15.6.1

warning.js:36 Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: boolean.
react-dom.min.js:15 Uncaught Error: Minified React error #130; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=130&args[]=boolean&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
at r (react-dom.min.js:15)
at p.i [as _instantiateReactComponent] (react-dom.min.js:15)
at performInitialMount (react-dom.min.js:13)
at p.mountComponent (react-dom.min.js:13)
at Object.mountComponent (react-dom.min.js:14)
at mountChildren (react-dom.min.js:14)
at m._createInitialChildren (react-dom.min.js:13)
at mountComponent (react-dom.min.js:13)
at Object.mountComponent (react-dom.min.js:14)
at performInitialMount (react-dom.min.js:13)

without a reproducible example I hardly believe we'll be able to find your problem

Hey FezVrasta,
I have updated react, react-dom and react-addons-test-utils from version 15.3.0 to 15.6.1. In course of that I have replaced the deprecated React.PropTypes with the explicit prop-types package. Afterward I've gotten this error. In Hope to fix it I've upgrade popper.js from 1.8.1 to 1.10.8 aner react-popper from 0.4.3 to 0.7.0.
resulting package.json:
{
"repository": { ... },
"keywords": [
"gulp",
"browserify",
"sass",
"javascript"
],
"scripts": { ... },
"engines": {
"node": "~5.1.0"
},
"licence": "private",
"dependencies": {
"axios": "^0.15.3",
"babel-polyfill": "^6.23.0",
"classnames": "^2.2.5",
"conduitjs": "^0.3.3",
"cookies-js": "^1.2.3",
"immutable": "^3.8.1",
"jquery": "^1.12.2",
"lodash": "^4.17.4",
"lodash.noop": "^3.0.1",
"lodash.throttle": "^4.1.1",
"parser-front-matter": "^1.6.2",
"parsley": "^0.1.0",
"parsleyjs": "^2.6.2",
"photoswipe": "^4.1.1",
"popper.js": "^1.10.8",
"postal": "^2.0.5",
"postal.preserve": "^0.2.0",
"react-click-outside": "^2.3.0",
"react-google-maps": "^7.0.0",
"react-input-range": "^0.9.1",
"react-popper": "^0.7.0",
"redux": "^3.6.0",
"redux-thunk": "^2.1.0",
"selectric": "git+https://github.com/lcdsantos/jQuery-Selectric.git#1.10.1",
"slick-browserify": "^1.5.9"
},
"devDependencies": {
"ajv": "^4.11.3",
"async": "^2.0.0-rc.3",
"autoprefixer": "^6.3.6",
"axios": "^0.15.3",
"babel-cli": "^6.18.0",
"babel-core": "^6.21.0",
"babel-jest": "^18.0.0",
"babel-plugin-add-module-exports": "^0.2.1",
"babel-polyfill": "^6.23.0",
"babel-preset-es2015": "^6.22.0",
"babel-preset-es2015-loose": "^8.0.0",
"babel-preset-react": "^6.16.0",
"babel-preset-stage-1": "^6.16.0",
"babel-register": "^6.18.0",
"babelify": "^7.2.0",
"backstopjs": "^2.6.6",
"body-parser": "^1.16.0",
"browser-sync": "^2.18.8",
"browserify": "^14.1.0",
"browserify-global-shim": "^1.0.3",
"browserify-shim": "^3.8.12",
"cors": "^2.8.1",
"css-mqpacker": "^5.0.1",
"deep-equal": "^1.0.1",
"deep-extend": "^0.5.0",
"del": "^2.2.0",
"enzyme": "^2.9.1",
"enzyme-to-json": "^1.5.1",
"eslint": "^3.18.0",
"eslint-config-airbnb": "^14.1.0",
"eslint-plugin-immutable": "^1.0.0",
"eslint-plugin-import": "^2.2.0",
"eslint-plugin-jsdoc": "^2.3.1",
"eslint-plugin-jsx-a11y": "^4.0.0",
"eslint-plugin-react": "^6.0.0",
"express": "^4.14.0",
"fs": "0.0.1-security",
"fs-utils": "^0.7.0",
"get-value": "^2.0.6",
"git-guppy": "^2.0.0",
"glob": "^7.0.3",
"gulp": "^3.9.1",
"gulp-changed": "^2.0.0",
"gulp-chug": "^0.5.1",
"gulp-data": "^1.2.1",
"gulp-debug": "^3.1.0",
"gulp-eslint": "^3.0.1",
"gulp-extname": "^0.2.2",
"gulp-favicons": "^2.2.7",
"gulp-file": "^0.3.0",
"gulp-front-matter": "^1.3.0",
"gulp-hb": "^5.1.4",
"gulp-htmlmin": "^3.0.0",
"gulp-iconfont": "^8.0.0",
"gulp-if": "^2.0.0",
"gulp-imagemin": "^3.1.1",
"gulp-json-to-sass": "0.0.4",
"gulp-layouts": "^0.1.0",
"gulp-newer": "^1.3.0",
"gulp-notify": "^3.0.0",
"gulp-nunjucks-render": "https://github.com/carlosl/gulp-nunjucks-render#2.0.0",
"gulp-plumber": "^1.1.0",
"gulp-postcss": "^6.1.0",
"gulp-real-favicon": "^0.2.2",
"gulp-rename": "^1.2.2",
"gulp-rev": "^7.0.0",
"gulp-rev-napkin": "^0.1.0",
"gulp-rev-replace": "^0.4.3",
"gulp-sass": "^3.1.0",
"gulp-sequence": "^0.4.5",
"gulp-sizereport": "^1.1.3",
"gulp-sourcemaps": "^2.4.1",
"gulp-streamify": "^1.0.2",
"gulp-uglify": "^2.0.1",
"gulp-util": "^3.0.7",
"gulp-watch": "^4.3.5",
"guppy-pre-push": "^0.2.1",
"handlebars": "4.0.6",
"handlebars-helpers": "^0.8.0",
"icu-loader": "^2.0.3",
"imagemin-pngquant": "^5.0.0",
"is-number": "^3.0.0",
"jest": "^18.1.0",
"layouts": "^0.13.2",
"merge-stream": "^1.0.0",
"normalize-path": "2.0.1",
"object.entries": "^1.0.4",
"open": "0.0.5",
"postcss-pxtorem": "^4.0.0",
"postcss-reporter": "^3.0.0",
"postcss-scss": "^0.4.1",
"pretty-hrtime": "^1.0.2",
"prop-types": "^15.5.10",
"qs": "^6.4.0",
"react": "15.6.1",
"react-addons-test-utils": "15.6.0",
"react-dom": "15.6.1",
"react-redux": "^5.0.2",
"react-router": "^4.0.0",
"react-test-renderer": "^15.6.1",
"redux-responsive": "^4.1.1",
"require-dir": "^0.3.0",
"retire": "^1.2.12",
"sass-mq": "^3.2.9",
"streamify": "^0.2.5",
"striptags": "^2.2.1",
"stylelint": "^7.9.0",
"swag": "^0.7.0",
"through2": "^2.0.1",
"vinyl-buffer": "^1.0.0",
"vinyl-source-stream": "^1.1.0",
"watchify": "^3.7.0"
},
"browserify-shim": {
"jquery": "global:$"
},
"browserify-global-shim": {
"immutable": "Immutable",
"react": "React",
"react-dom": "ReactDOM",
"react-redux": "ReactRedux",
"redux": "Redux",
"axios": "axios",
"postal": "postal",
"parsleyjs": "Parsley",
"prop-types": "PropTypes",
"handlebars": "Handlebars"
},
"browserify": {
"transform": [
"browserify-shim",
"browserify-global-shim"
]
},
"jest": {
"setupFiles": [
"./test/jest-setup.js"
],
"collectCoverageFrom": [
"src/components//*.{js,jsx}",
"!/node_modules/",
"!/vendor/",
"!src/components/organisms/styleguide-nav/styleguide-nav.js",
"!src/components/organisms/search/partials/action-types.js",
"!src/components/organisms/basket/partials/action-types.js",
"!src/components//styleguide.js",
"./test/jest.js"
],
"snapshotSerializers": [
"/node_modules/enzyme-to-json/serializer"
]
}
}
resulting issue:

Hope you can help me to find the issue,
BR Hanne

Unfortunately, there is not enough provided information to help diagnose the problem. Please feel free to reopen this issue if you can provide a Codepen example, or an isolated repo with a reproducible issue. Thank you.


Revert back BYOK and remove BYOC

Summary
Return back the Bring Your Own Key and remove the Bring Your Own Cloud part.

@wiersgallak can you approve pls?


No user name

@gniezen see https://trello.com/c/aCrFMbXm for details.
Turns out if VCA had data storage attached then we would see it in the searchable accounts and it would be preselected at the top of the list

 VCA Uploader should recognize when an account has setup a DSA and ignore it [Uploader] <0:ux 1:dev 1:tst>


State does not enums datatype

TypeError: State vars must be primitive Python types, Plotly figures, Pandas dataframes, or subclasses of rx.Base. Found var "clicked" with type <enum 'ButtonOption'>.
The issue can be potentially fix by adding a serializer in serializer.py
enums.py
from enum import Enum

class ButtonOption(Enum):

    PROFILE = "Profile"

    ACCOUNT = "Account"

    APPEARANCE = "Appearance"

    NOTIFICATIONS = "Notifications"

    DISPLAY = "Display"

forms_states.py
import reflex as rx

from .enums import ButtonOption

class State(rx.State):

    clicked: ButtonOption = ButtonOption.PROFILE

    def set_clicked(self, option: ButtonOption):

        self.clicked = option

    def get_clicked(self):

        return self.clicked

REF-654

Please assign this to me


test-bitnami-helm-chart-for-metrics-server-3411

why testing of Use bitnami helm chart for metrics_server#3411

New branch


Add getitem to Stimulus for indexing

It would be nice to have a way for indexing into/slicing the data container of a Stimulus object. Make it so that __getitem__ takes a time point and returns the data at that time point, including interpolating as needed.

Closed via #127


Coverage doesn't have "conditional assignment" into consideration

🐛 Bug Report
When an object is set like this:
const myObj = {
  myProp: myVariable === 'given expected value',
};

Jest (or maybe nyc?) is not considering it as a branch.
However if I do:
const myObj = {
  myProp: myVariable === 'given expected value' ? true : false,
};

This is considered as a branch...
To Reproduce
Steps to reproduce the behavior:

yarn create react-app
yarn add classnames
Change the src/App.js to:

import classnames from 'classnames';
import React from 'react';
import logo from './logo.svg';
import './App.css';

function App({ test }) {
  const myClassName = {
    [test]: test === 'yeah'
  };

  return (

          Edit src/App.js and save to reload.

          Learn React

  );
}

export default App;

Change the src/App.test.js to:

import React from 'react';
import { render } from '@testing-library/react';
import App from './App';

test('renders learn react link', () => {
  const { container, getByText } = render(<App />);
  const linkElement = getByText(/learn react/i);
  expect(linkElement).toBeInTheDocument();
  expect(container).toMatchSnapshot();
});

Run yarn test --coverage
Verify that the coverage is still 100%:

Expected behavior

Link to repl or repo (highly encouraged)
https://github.com/mAiNiNfEcTiOn/jest-conditional-assignment-repro
envinfo

  System:
    OS: macOS 10.15.2
    CPU: (8) x64 Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
    Memory: 2.19 GB / 32.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.13.1 - /usr/local/bin/node
    Yarn: 1.19.1 - /usr/local/bin/yarn
    npm: 6.12.1 - /usr/local/bin/npm
  Managers:
    Homebrew: 2.2.2 - /usr/local/bin/brew
    pip2: 19.3.1 - /usr/local/bin/pip2
    pip3: 19.3.1 - /usr/local/bin/pip3
    RubyGems: 3.0.3 - /usr/bin/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.21.0 - /usr/bin/git
    Clang: 1<IP_ADDRESS> - /usr/bin/clang
    Subversion: 1.10.4 - /usr/bin/svn
  Servers:
    Apache: 2.4.41 - /usr/sbin/apachectl
  Virtualization:
    Docker: 19.03.5 - /usr/local/bin/docker
    VirtualBox: 6.0.14 - /usr/local/bin/vboxmanage
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
  IDEs:
    Nano: 2.0.6 - /usr/bin/nano
    VSCode: 1.41.1 - /usr/local/bin/code
    Vim: 8.1 - /usr/bin/vim
    Xcode: 11.3/11C29 - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Perl: 5.18.4 - /usr/bin/perl
    PHP: 7.3.9 - /usr/bin/php
    Python: 2.7.16 - /usr/bin/python
    Python3: 3.7.5 - /usr/local/bin/python3
    Ruby: 2.6.3 - /usr/bin/ruby
  Databases:
    SQLite: 3.28.0 - /usr/bin/sqlite3
  Browsers:
    Chrome: 79.0.3945.88
    Chrome Canary: 81.0.4020.0
    Firefox: 71.0
    Safari: 13.0.4

Thanks for the report. This is correct behavior, there is no branch, it is just setting the property to the boolean value that the strict equality comparison results in.

Hi @jeysal, thank you for your time to review it.
I agree that the property always get the result of the condition, but a condition is evaluated to a boolean which can be either true or false, right?
In the example, you end up with 2 possible cases that the only way to "reproduce" the same behaviour without using the non-detected method, would be to do myVariable === 'given expected value' ? true : false which is the exact same effect, just has a ternary operator.
Specially with packages similar to classnames, where they receive flags in an object and such, this leads to different markup being generated. When one is truthy the class is added, when it's falsy it's not added...
Still, I understand the reasoning behind it and in order to make this detectable I'll ask my team members to change cases like this - where a flag might affect the output of a React component - to change it to a ternary... 👍

What you are referring to with the classnames is that the boolean is at some later point used in a conditional - for example inside the classnames package. In that case, the coverage will be tracked where that branching happens, but not where the boolean value is first created.

👍 Thank you for taking the time to reply. I really appreciate it.
Yes, my team as already moved on with using ternary in such situations in order to state different branches when analyzing coverage 😉
Although I understand your point and do not disagree with it, it forces us to create a ternary operation when we're receiving a property on a React component.
The creation of the boolean is in fact done outside of the component but is passed to it as a flag that is then given to the classnames package which indeed will be the one enabling or disabling a className's string...
But again, thank you for your patience and for explaining, it was insightful too.

I also think you may be overusing the coverage metric here, it can never tell you if you're actually testing all relevant cases on its own.
Code reviews are still required so that a human can spot at the usage of classnames that this piece of code requires a test case for "has the class" and one for "does not have the class" :)

Agree. We're not avoiding code review in any case, though. We're just trying to get this type of feedback before the code review process (via git hooks and such). I compare that process to the same as linting code... It is to ensure that standards are met where it can analyze 😉


Add support for Razer Goliathus Extended Chroma

https://www.razer.com/gaming-mouse-mats/razer-goliathus-chroma
Would be great if it had the same type of support the RGB Corsair mouse pad has. It only has 1 lighting zone so only simple animations would work but that's fine imo.

It should follow the mouse color in the current version. More device types are planned for the next versions.


Drop the test for finding a b2g milestone for the current date since there are no such milestones anymore

This test started failing when we stopped adding new b2g milestones to bugzilla (and in turn to bugherder). I say we just drop it.

@edmorley any objections?

Go ahead :-)


UNBLOCKED GAMES basketball stars unblocked games | Play Free Games Online - 26tj2

UNBLOCKED GAMES basketball stars unblocked games | Play Free Games Online - 26tj2
basketball stars unblocked games is a popular website that offers a wide variety of games that can be played in environments where access to gaming sites is typically restricted, such as schools and workplaces.
Enjoy playing all your favorite casual basketball stars unblocked games, office, or home. Dive into the world of online free games without downloads!
👉🔴 SITE Games Unblocked >> basketball stars unblocked games
👉🔴 PREMIUM Games Unblocked >> basketball stars unblocked games
🙋♀ A short introduction - what is your organization all about?
🌈 Contribution guidelines - how can the community get involved?
👩💻 Useful resources - where can the community find your docs? Is there anything else the community should know?
🍿 Fun facts - what does your team eat for breakfast?
🧙 Remember, you can do mighty things with the power of Markdown
basketball stars unblocked games: Unlocking Premium Fun in Educational Settings
In recent years, the phenomenon of basketball stars unblocked games has taken the online gaming community by storm. These games, designed to bypass restrictions often placed by schools and workplaces, provide a unique blend of entertainment and engagement, especially for students. Among the vast array of options, unblocked premium games stand out as a popular choice, offering enhanced features, better graphics, and a more immersive gaming experience.
What Are basketball stars unblocked games?
basketball stars unblocked games are online games that can be accessed and played without restrictions, typically in environments like schools or offices where gaming sites might be blocked. The appeal of these games lies in their accessibility and the diverse range of genres available, from action and adventure to puzzles and strategy. Students often turn to these games during breaks or downtime, providing a quick escape from their academic routines.
The Rise of Premium basketball stars unblocked games
As the unblocked games market has evolved, so has the demand for premium content. Premium basketball stars unblocked games are those that offer superior gameplay experiences, often including advanced graphics, engaging storylines, and multiplayer functionalities. Many of these games are adaptations of popular titles, enabling players to enjoy high-quality gaming even in restricted environments.
The allure of premium unblocked games lies in their ability to combine fun with an element of strategy and critical thinking. Many of these games challenge players to solve complex problems, collaborate with peers, and develop skills that are beneficial beyond the gaming world.
Benefits of Playing basketball stars unblocked games

Stress Relief: For students juggling a demanding academic schedule, unblocked games provide a much-needed break. Engaging in play can reduce stress and enhance overall well-being.
Social Interaction: Many unblocked games allow for multiplayer modes, enabling students to connect with friends or even make new ones. This social aspect is particularly valuable in school settings, fostering teamwork and collaboration.
Skill Development: Numerous unblocked games require strategic thinking, quick reflexes, and problem-solving abilities. These skills can translate to academic success, making gaming a potentially productive pastime.
Creativity Boost: Games that allow for customization and creative expression can inspire students to think outside the box, enhancing their creative skills.

Popular Genres of basketball stars unblocked games Premium Games

Action and Adventure: Games that provide thrilling narratives and exciting challenges, often featuring iconic characters and rich worlds. Titles like "Among Us" or "Minecraft" have gained immense popularity in unblocked formats.
Puzzle and Strategy: These games challenge players to think critically and plan ahead, with examples including "2048" and "Bloons Tower Defense."
Simulation: Games like "The Sims" or "Animal Crossing" let players create and manage their virtual lives, promoting creativity and organization.
Sports and Racing: Titles such as "Basketball Stars" and "Slope" offer fast-paced action and competitive gameplay, appealing to sports enthusiasts.

Conclusion
basketball stars unblocked games premium games represent a dynamic intersection of fun and education, providing students with a unique platform to unwind while also honing valuable skills. As educational institutions continue to grapple with balancing entertainment and learning, these games offer a solution that can enhance the school experience. By integrating gaming into their downtime, students can enjoy the thrill of play while developing critical skills, making unblocked premium games a worthwhile addition to any educational setting.

vnvnvn


codebeat badge

Is it fine to add codebeat badge to README?
codebeat is automated code review tool for Swift, Ruby, Go & Python that helps get instant feedback on code quality.
"Quick wins" suggested by codebeat could be a nice candidate for a pull request and help other developers become contributors.
FYI. To be fully open and honest. I'm co-founder of codebeat.

Very nice! We will investigate using it. If we proceed, we may want to address the first pass of quick wins before adding the badge. As you mentioned, we're open to the community taking on these refactoring improvements as well.
Ideally, this tool would be integrated with Travis, so that we can see how the code quality changes with each build, including builds that include pull requests.

@kweinmeister thank you!

BTW... currently we are working on support for pull requests in codebeat.
Integration with CI tools is also in our backlog :) If there are any other things you would like to see in codebeat please don't hesitate share it with us. Thank you!

We will consider adding this badge in the future. Closing the PR for now.


Preparing next release

Codecov Report

Merging #70 (0bad078) into main (3086cd8) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 0bad078 differs from pull request most recent head 6fb8566. Consider uploading reports for the commit 6fb8566 to get more accurate results

@@           Coverage Diff           @@
##             main      #70   +/-   ##
=======================================
  Coverage   97.77%   97.77%
=======================================
  Files           2        2
  Lines          45       45
=======================================
  Hits           44       44
  Misses          1        1

Flag
Coverage Δ

pytest
97.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Packet-Forwarder does not start on startup (Multitech gateway)

Hello !
I have followed the tutorial of this page to install the packet forwarder on my Multitech gateway :
https://www.loraserver.io/lora-gateway-bridge/gateway/multitech/
But, while it worked perfectly on older gateways (same model, just older mlinux versions), it happens that on mlinux 5.0.0, after adding the packet forwarder to defaults with update-rc.d and rebooting, an error message appears on startup :
Found MTAC-LORA-H-868 with MTAC-LORA-1.5 hardware at ap1
Starting lora-packet-forwarder: renice: getpriority: No such process
OK

Is there a way to fix this ?
By advance, thank you very much !
Kind regards,
Etienne

Are you sure the logged error is from starting the packet-forwarder (I've seen sometimes that when processes are logging async, that lines can get mixed).
On my Conduit I have installed according to the instructions you mentioned. After a reboot, I do see that the forwarder is started.

Thank you for your answer.
I will be a little more precise :
This morning (Friday August 2nd, ~09h00 AM GMT+1), I have flashed once again the 5.0.0 firmware on my mlinux Gateway (model : MTCDT-LEU1-246L-EU-GB), so I could go all over again from the beginning with a "clean" gateway
Here are the steps I have followed from here :

PPP configuration (my gateway functions by GSM)
Installing the packet forwarder through the instructions of your tutorial here : https://www.loraserver.io/lora-gateway-bridge/gateway/multitech/
(Gateway Bridge not installed yet)
reboot with USB serial plugged so I could see booting steps.
I had this result, which seemed fine

https://image.noelshack.com/fichiers/2019/31/5/1564730561-capt1.png

Logged in on the gateway
Tried "ps aux"
No packet forwarder in the list (see below)

https://image.noelshack.com/fichiers/2019/31/5/1564730561-capt2.png

Tried to configure ppp on boot (with update-rc.d /etc/ppp defaults)
reboot once again the gateway
I had this result, with the renice problem

https://www.noelshack.com/2019-31-5-1564730561-capt3.png

I have removed from launch the Multitech Packet Forwarder (update-rc.d -f lora-packet-forwarder remove), then rebooted, but same result.
I have removed from launch your packet forwarder (update-rc.d -f lora-packet-forwarder-ap1 remove), then re-added it while gaving it the same priority the Multitech Packet Forwarder had (update-rc.d lora-packet-forwarder-ap1 defaults 80 30)/
Now, I am back to the 1st problem : The renice priority problem seems solved, but I can't see the packet forwarder with ps aux, and it is not detected by the LoRaServer I installed, so it really seem not to run.

Thank you very much for reading this and for caring about this problem,
Kind regards,
Etienne

I have received a similar report:

The semtech packet forwarder does not start up if the MT Conduit's cellular interface is enabled and it does not get a signal from the carrier.

One option would be to look into adding Monit as a service monitoring daemon. With the init script it either starts or fails but it will not re-retry. Using Monit (which I'm also using for the LoRa Gateway OS), it will retry if it detects that the process has failed.


[Question] Multi-Speaker Training

Hi rishikksh20!! Thank you for your excellent code.
I have a question about multi-speaker training.
Can I train VocGAN in a multi-speaker environment?
I used WaveRNN, trained with five speakers. I want to know if this VocGAN is also possible, and if it is possible, will it be a little lower in sound quality?
Thank You!

Hello, @sooftware .
For satisfying my curiosity, I trained with VCTK, LJ-Speech, KSS, and Korean Emotional Speech dataset  at once.
Unfortunately, in this case, modified-VocGAN produces noisy samples.
What about using Hifi-GAN?
In the paper, the authors provide experimental results regarding unseen speakers.

Thank you!  @Jackson-Kang.
I used WaveRNN (five speakers).
WaveRNN's result is good.
So I was wondering what VocGAN would be like.


icon proposal

I would like to propose this icon that I created for the application.

Looks very nice! Thanks for suggestion it. I think I'm gonna stick with the current one though, as it follows the Twitter blue color and has ties to the name (Anatine means duck).

Now I understand why because a duck on the icon and thanks for answering :-)


Add option to configure TLS/mTLS endpoint for Tornjak

This PR adds TLS and mTLS connection to Tornjak backend. It is assumed users create secret tornjak-certs prior to deploying the helm charts when TLS and/or mTLS are enabled.
Sample secret (to be documented):
kubectl -n spire-server create secret generic tornjak-certs \
--from-file=key.pem="client.key"  \
--from-file=cert.pem="client.crt" \
--from-file=rootCA.pem="CA/rootCA.crt"

# testing TLS:
curl --cacert CA/rootCA.crt https://localhost:20000/

# testing mTLS:
curl  --cacert CA/rootCA.crt --key client.key --cert client.crt https://localhost:30000

@marcofranssen, I asked him to do it as two secrets before.
There are multiple reasons not exhaustive:

tls certificates in k8s (secret type=tls) has a particular format that includes private key and cert. Other systems such as cert-manager can automatically create certs in this form. So its important that we support it.

Oh right! Very good point. Then others are just not using this type of secret. I like to use the k8s native types.

the user ca is needed for mtls. Its what the client certificate is verified against. if you use cert-manager to setup your user ca as well, its likely a completely separate ca then your service ca. This should come in through an alternate configmap/secret/the new ClusterTrustBundles type (ClusterTrustBundles (previously Trust Anchor Sets) kubernetes/enhancements#3257)

Oh then it is probably the naming that confused me. However why would the CA be different?
I would do it like this

Cluster CA

server cert
intermediate CA

client cert
client cert
client cert

Isn't the root CA sufficient to build the trustchain? Now you made me doubt though :)

Having a create option though where the user can specify the secrets in values would be ok. That could be in this pr or in another followon feature pr?

User CA's sometimes need to be on the internet so that users can properly get certificates for themselves. Host CA's dont need to be accessible on the internet. So keeping them separate has security benefits.You don't necessarily want a User CA to be able to issue valid Host certs from the User CA as then there is more attack surface.
There are other reasons why you may want separate chains of trust there. Hosts often need to be trusted by browsers or other services not under the sysadmin's control, so having a more traditional chain of trust makes hosts validation easier. For example, with letsencrypt. While user ca validation only happens on your own hosts, so easier to distribute to the set of machines needing that CA for client validation.
So its best to allow treating host chain of trust completely separate from user chain of trust. Some users may combine the two if they like but that should be up to them and their situation.

User CA's sometimes need to be on the internet so that users can properly get certificates for themselves. Host CA's dont need to be accessible on the internet. So keeping them separate has security benefits.You don't necessarily want a User CA to be able to issue valid Host certs from the User CA as then there is more attack surface.
There are other reasons why you may want separate chains of trust there. Hosts often need to be trusted by browsers or other services not under the sysadmin's control, so having a more traditional chain of trust makes hosts validation easier. For example, with letsencrypt. While user ca validation only happens on your own hosts, so easier to distribute to the set of machines needing that CA for client validation.
So its best to allow treating host chain of trust completely separate from user chain of trust. Some users may combine the two if they like but that should be up to them and their situation.

So isn't that intermediate as I shown above exactly what you need to just make that available on the internet, but still have it trusted by the upstream cluster CA?

No. If you have your own private CA for the user ca, and a public chain of trust such as letsencrypt for the hosts, it is not possible for the intermediates to be rooted at just one CA. Nor is it desirable as I explained, for security reasons. You don't want machines to trust host certs signed by the user ca so yiou can't trust the a root where there is a shared root between a user and host ca.
Also, an intermediate CA is just a way to add an additional layer of security so you don't have to expose the root ca. The intermediate CA's can have expiry times much less then the root ca's expiration date. But can issue certs much the same as the root.
The user ca must be configurable separately then any ca used for the hosts.

There are a few different options we may want to consider for the tls certs and for the user ca.....
For the host cert for each of the frontend and backend, we could support an enum with:
existing, cert-manager, values
where existing would just use the secret that exists, cert-manager would create a Certificate object to generate the tls secret, and values could create the secret from a key/cert thats specified in values
For now, this pr only supports an existing secret. Or the user can use cert-manager to create it manually. This is probably a good first step and the other options could be added in a follow up pr.
For the userCA, we have some similar options.... some combination of
existing-secret, existing-configmap, cert-manager, values
where existing-secret/configmap is preexisting objects, cert-manager would create a self signed user ca and then reference it, and values would have the ca.pem specified directly in the values and would get put in a secret.
Similar to the previous, I'd recommend the additional options be added in a follow up pr.

https://github.com/spiffe/helm-charts/pull/338#discussion_r1234512531
Thank you for the comment @kfox1111
I think the most recent version already has two mount-points: https://github.com/spiffe/helm-charts/blob/tornjak-tls/charts/spire/charts/spire-server/templates/statefulset.yaml#L213-L218

@marcofranssen per your suggestion, Tornjak Helm charts currently determine automatically the Tornjak connection type, based on provided information (secrets or ConfigMaps). We will update the Tornjak code (https://github.com/spiffe/tornjak/issues/268) to do this determination on the level of the Tornjak image. Once completed there, we will update these helm charts here with a new configuration format, as a follow-up PR.

@marcofranssen You blocked this review with a request for changes, but subsequent changes have moved your requests into the place that deleted files go.  Would you re-review this commit and if you decide it is sufficient, approve it?

@marcofranssen Can we come to an agreement on the secrets issue? Basically I can see some scenarios where the secrets are not to be generated through the Helm interface, such as when they are provided by Vault and accessed through a secrets claim that has vault populating the secret.
I see this as a very good addition that is nearly ready to ship, if we can come to a resolution on the secrets item soon.

This PR isn't blocked on a secrets discussion neither are we ruling out any options to manage secrets. It is blocked because of implementation bugs. Working with Marius to get those resolved, no need to chase us.

Minor naming convention updates to be handled by follow-up PR (issue #389 )


support upload of objects other than File

currently I must pass a File object, which is fine if I have an input type=file. What if I try to manipulate an image before uploading? Or try to have the browser generate derivatives for an image and upload those as well? So what if all I have to upload is the dataUrl of a canvas?

@dsdjung sorry for the delay. Please check #39.

Excellent.. Will check it out today. Thanks a bunch...
DJ
2015-01-04 17:52 GMT-08:00 Garik Suess<EMAIL_ADDRESS>
Closed #22 https://github.com/CulturalMe/meteor-slingshot/issues/22 via
#39 https://github.com/CulturalMe/meteor-slingshot/pull/39.
—
Reply to this email directly or view it on GitHub
https://github.com/CulturalMe/meteor-slingshot/issues/22#event-213866594
.

This has now been released in 0.3

Hi @gsuess , an example of how to use this would be nice. I am currently doing:
uploader.send(imageBlob, function (error, downloadUrl) {
console.log(error);
console.log(downloadUrl);
});
imageBlob does not have file name, and it doesn't work. How do I provide a file name? Thanks :)

This looks correct to me. There is nothing special about blobs, other than them not having a filename.
Slingshot does not require the file to have a file.name, but if it doesn't then your directive needs to support it to. So in your directive, don't expect file.name to always be set.

I think it depends on your key function. This is what I use and it works
        var upload_id = Meteor.uuid(); // or whatever

        template.cropCanvas.toBlob(function(blob){
          var ext = blob.type.split(/\//).pop();
          // we can totally overwrite the name of the blob which will be used in the key method
          blob.name = upload_id+'.thumbnail.'+ext;
          // we can not overwrite the file.name, it is read-only
          template.uploader.send(blob, function (error, url) {
          });
        });

and my key function for the uploader
    key: function (file) {
      var dir = 'slingshot/';
      // this only works if file has name of course
      var fn = file.name;
      return dir + fn ;
    }

Thank you. I figured out what gave me the Error: Match failed [400],
This is what I have for allowedFileTypes:
allowedFileTypes: ["image/png", "image/jpeg", "image/gif"]
And the blob I generate has the type of "png", so the problem goes away once I change the blob type to "image/png"
Thanks!


Adjust default SMT settings for RPC based proofs

Blocked on: https://github.com/runtimeverification/evm-semantics/pull/1646
This attempts to up the SMT timeout to find the smallest viable one for getting things merged faster.
This PR:

Adds a new argument parser for smt_args, which contains:

--smt-timeout: control how long smt queries take.
--smt-retry-limit: control how many times to retry the SMT solver.

Threads through these CLI arguments to all places that invoke the backend via KCFGExplore.
Removes find_free_port() and relies on upstream port finding, which does not have the race condition present here.

We should still get https://github.com/runtimeverification/evm-semantics/pull/1635 sorted, because @jberthold is reporting that every invocation of the new booster backend we are accumulating those Ceil constraints, nested on each other, when we really only need them the first call to generate the new conditions. The booster backend doesn't have a ceil simplifier, or a simplifier at all, so it results in the term it sees growing indefinitely every time the execute endpoint is called.

Here: https://github.com/runtimeverification/evm-semantics/pull/1640 We actually end up updating K to the point where we get proper "SMT failure" messages, so now we'll try and up the timeout to see if those messages go away.
Another effort is here: https://github.com/runtimeverification/evm-semantics/pull/1635 to make it so we don't need the solver as much anyway.
We'll re-run CI here a few times to make sure it's actually consistently passing, then reduce the increased timeout. "Successful" means "did not produce any SMT errors".
500ms

Successful 3d1c27d GH13 46m: https://github.com/runtimeverification/evm-semantics/actions/runs/4453881500/jobs/7822775725
Successful 31e7ea0 GH10 64m: https://github.com/runtimeverification/evm-semantics/actions/runs/4455757069/jobs/7825866023
Successful 31e7ea0 GH15 75m: https://github.com/runtimeverification/evm-semantics/actions/runs/4455757069/jobs/7826356983 (race condition on opening socket)
Successful 2e4e6e4 GH15 74m: https://github.com/runtimeverification/evm-semantics/actions/runs/4456726625/jobs/7827443074
Successful 2e4e6e4 GH13 46m: https://github.com/runtimeverification/evm-semantics/actions/runs/4456726625/jobs/7827996896

250ms

Successful 9b96224 GH13 46m: https://github.com/runtimeverification/evm-semantics/actions/runs/4457362759/jobs/7828463559
Successful 9b9224 GH10 64m: https://github.com/runtimeverification/evm-semantics/actions/runs/4457362759/jobs/7828923280 (some docker error, tests passed)
Successful 9b96224 GH13 46m: https://github.com/runtimeverification/evm-semantics/actions/runs/4457362759/jobs/7829658784
Successful 9b96224 GH10 64m: https://github.com/runtimeverification/evm-semantics/actions/runs/4457362759/jobs/7830151570
Successful 9b96224 GH10 64m: https://github.com/runtimeverification/evm-semantics/actions/runs/4457362759/jobs/7830652493

125ms

Successful 5cb09d6 GH15 75m: https://github.com/runtimeverification/evm-semantics/actions/runs/4459074665/jobs/7831383287
Successful 5cb09d6 GH10 65m: https://github.com/runtimeverification/evm-semantics/actions/runs/4459074665/jobs/7831853323
Successful bdd1ff1 GH16 64m: https://github.com/runtimeverification/evm-semantics/actions/runs/4459703797/jobs/7832406077
Successful bdd1ff1 GH10 65m: https://github.com/runtimeverification/evm-semantics/actions/runs/4459703797/jobs/7832785869
Mostly successful 8be4355 GH10 66m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461460010/jobs/7835381088 (Haskell backend proofs failed due to resource exhaustion)

Default: --smt-timeout 125 --smt-retry-limit 4

Successful 6031a0e GH16 50m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461777091/jobs/7836009228 (failure on a Nix stage)
Successful 6031a0e GH16 48m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461777091/jobs/7836962660
Successful 6031a0e GH10 67m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461777091/jobs/7837466151
Successful 6031a0e GH10 66m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461777091/jobs/7837920365
Successful 6031a0e GH10 67m: https://github.com/runtimeverification/evm-semantics/actions/runs/4461777091/jobs/7839495404


Error: protected method `declared_runtime_dependencies' with Swiftformat

Hello!
On of our CI jobs over at fastlane (https://github.com/fastlane/fastlane/pull/18173) started failing with this error below when trying to install SwiftFormat
The output told me to create an issue over here so that is what I’m doing 😇
I was hoping that somebody would be able to help me determine if this is an error on my end or not! I appreciate your time helping me with this ❤
brew bundle
➡  Brew
==> Tapping homebrew/bundle
Cloning into '/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle'...
Warning: Permanently added the RSA host key for IP address 'ip.address' to the list of known hosts.
Tapped 1 command (109 files, 1.7MB).
Error: protected method `declared_runtime_dependencies' called for #<Formulary::FormulaNamespace8dc1512549d8fbfb463670fa67223481::Swiftformat:0x00007f8b49297d28>
Please report this bug:
  https://github.com/Homebrew/homebrew-bundle/issues
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:147:in `formula_to_hash'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:114:in `add_formula'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:46:in `formulae_by_full_name'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:188:in `conflicts_with'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:198:in `resolve_conflicts!'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:34:in `install_change_state!'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:27:in `run'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:12:in `install'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/installer.rb:35:in `block in install'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/installer.rb:11:in `each'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/installer.rb:11:in `install'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/lib/bundle/commands/install.rb:10:in `run'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/cmd/bundle.rb:94:in `bundle'
/usr/local/Homebrew/Library/Homebrew/brew.rb:122:in `<main>'
rake aborted!
Command failed with status (1): [brew bundle...]
/Users/distiller/project/fastlane/swift/formatting/Rakefile:11:in `block in <top (required)>'
/Users/distiller/project/.bundle/ruby/2.5.0/gems/rake-13.0.3/exe/rake:27:in `<top (required)>'

I think https://github.com/Homebrew/homebrew-bundle/pull/907 fixed this. brew update and see if the problem persists.

OMG... I totally searched for this and can't believe I didn't see that issue 🤦♂
Thank you for the help!


Cannot compute output tensor of tf.keras.layers.Lambda + tf.unstack

System information

Have I written custom code: Yes
OS Platform and Distribution: Ubuntu 16.04
TensorFlow installed from: binary
TensorFlow version: tf-nightly 1.5.0-dev20171224
Python version: 3.5.4

Source code
This is a tiny code that creates a model which stacks and unpacks input tensors.
import tensorflow as tf
from tensorflow.python import keras

x = [keras.layers.Input([2]) for _ in range(3)]
h = keras.layers.Lambda(lambda inputs: tf.stack(inputs, axis=1))(x)
y = keras.layers.Lambda(lambda inputs: tf.unstack(inputs, axis=1))(h)

model = keras.models.Model(x, y)
model.compile('adam', 'mse')

This code raises a AssertionError.
Traceback (most recent call last):
  File "example.py", line 9, in <module>
    model.compile('adam', 'mse')
  File ".../tf-nightly/lib/python3.5/site-packages/tensorflow/python/keras/_impl/keras/engine/training.py", line 681, in compile
    masks = self.compute_mask(self.inputs, mask=None)
  File ".../tf-nightly/lib/python3.5/site-packages/tensorflow/python/keras/_impl/keras/engine/topology.py", line 787, in compute_mask
    _, output_masks = self._run_internal_graph(inputs, masks)
  File ".../tf-nightly/lib/python3.5/site-packages/tensorflow/python/layers/network.py", line 927, in _run_internal_graph
    assert str(id(x)) in tensor_map, 'Could not compute output ' + str(x)
AssertionError: Could not compute output Tensor("lambda_1/unstack:1", shape=(?, 2), dtype=float32)

I found that this is an error that occurs when the number of outputs of Layer.output_masks() does not match the number of outputs of Layer.call().

Can you give some hint about how to debug with this error message: "assert str(id(x)) in tensor_map, 'Could not compute output ' + str(x)"? It troubles me too, and I can't find what's wrong.

_run_internal_graph() forget to put all outputs in tensor_map. I encounter this in TF1.4, but looks like it is fixed in TF1.8.


Implement SassDoc

Jira
http://vjira2:8080/browse/BDS-518
Summary
Implements all comment documentation for SassDoc
Details
Previously before this PR, we had undocumented variables, mixins, and functions. The work contained within this branch completes the first round documentation of those missing items.
How to test

Pull down the branch and run npm run sassdoc:build at the root of the project
Change directories into /apps/pattern-lab and run npm run start
When PL opens, navigate to Styleguide-->Sassdoc

@rockymountainhigh1943
I could not get npm run sassdoc:build to work. Is there something else I need to do? Sassdoc command is not found.

@remydenton – would you be willing to review this and merge if it's all set?

@remydenton @rockymountainhigh1943 @mikemai2awesome this is good to merge down to master as soon as I get the OK from the Travis build currently running. I've also added in a few things so this compiles out consistently whenever our Webpack build runs, automatically -- no more npm script required.
Nice job guys!!

@sghoweri, I just added this to our 1.x release notes for you #840.  Please make sure to do that step whenever you merge a PR into master.

Thanks @remydenton! Gah — sorry. I didn’t forget about this one, I swear! 😅


Use "pdm" as the license instead of "publicdomain"

Description

Some files incorrect specify the license as "publicdomain". Our standard slug for public domain content is "pdm". Instead of making a change in the openverse-attribution library to handle non-standard slugs, this PR adds the replacement code to the API. (In my view, it is the API's fault for sending bad data to the library.)
The endgoal solution for this should be for the catalog to correct the data to have "pdm" as the slug.
Testing Instructions

Change some rows in the DB to have "publicdomain" as the slug.
Try to serialize these files by visiting their detail page.
They should render as "pdm".

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

We also do not handle "sa". The closest to that is "by-sa".

Thanks for catching and explaining this @stacimc because mapping "sa" to "by-sa" would not have been right! I have updated the code in 179182f to treat "sa" accurately as a valid, but deprecated license.

license shadows built-in name.


ValueError because reward is turning up to be ndarray after several correct runs

I am trying out FinRL for the first time by preparing my own notebook based on this example notebook.
When I run this line:
trained_a2c = agent.train_model(model=model_a2c,
                                tb_log_name='a2c',
                                total_timesteps=50000)

it was able to log to tensorboard correctly only at the first attempt, but then it failed with ValueError:
Logging to tensorboard_log/a2c\a2c_1
------------------------------------
| time/                 |          |
|    fps                | 633      |
|    iterations         | 100      |
|    time_elapsed       | 0        |
|    total_timesteps    | 500      |
| train/                |          |
|    entropy_loss       | -1.41    |
|    explained_variance | -0.087   |
|    learning_rate      | 0.0002   |
|    n_updates          | 99       |
|    policy_loss        | 0.003    |
|    std                | 0.994    |
|    value_loss         | 0.0812   |
------------------------------------
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
TypeError: only size-1 arrays can be converted to Python scalars

The above exception was the direct cause of the following exception:

ValueError                                Traceback (most recent call last)
<ipython-input-10-9b19b057dbd1> in <module>
----> 1 trained_a2c = agent.train_model(model=model_a2c,
      2                                 tb_log_name='a2c',
      3                                 total_timesteps=50000)

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\finrl\model\models.py in train_model(self, model, tb_log_name, total_timesteps)
    122
    123     def train_model(self, model, tb_log_name, total_timesteps=5000):
--> 124         model = model.learn(total_timesteps=total_timesteps, tb_log_name=tb_log_name)
    125         return model
    126

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\stable_baselines3\a2c\a2c.py in learn(self, total_timesteps, callback, log_interval, eval_env, eval_freq, n_eval_episodes, tb_log_name, eval_log_path, reset_num_timesteps)
    188     ) -> "A2C":
    189
--> 190         return super(A2C, self).learn(
    191             total_timesteps=total_timesteps,
    192             callback=callback,

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\stable_baselines3\common\on_policy_algorithm.py in learn(self, total_timesteps, callback, log_interval, eval_env, eval_freq, n_eval_episodes, tb_log_name, eval_log_path, reset_num_timesteps)
    225         while self.num_timesteps < total_timesteps:
    226
--> 227             continue_training = self.collect_rollouts(self.env, callback, self.rollout_buffer, n_rollout_steps=self.n_steps)
    228
    229             if continue_training is False:

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\stable_baselines3\common\on_policy_algorithm.py in collect_rollouts(self, env, callback, rollout_buffer, n_rollout_steps)
    166                 clipped_actions = np.clip(actions, self.action_space.low, self.action_space.high)
    167
--> 168             new_obs, rewards, dones, infos = env.step(clipped_actions)
    169
    170             self.num_timesteps += env.num_envs

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\stable_baselines3\common\vec_env\base_vec_env.py in step(self, actions)
    161         """
    162         self.step_async(actions)
--> 163         return self.step_wait()
    164
    165     def get_images(self) -> Sequence[np.ndarray]:

c:\users\crrma\.virtualenvs\stocks-pv8ke_ig\lib\site-packages\stable_baselines3\common\vec_env\dummy_vec_env.py in step_wait(self)
     41     def step_wait(self) -> VecEnvStepReturn:
     42         for env_idx in range(self.num_envs):
---> 43             obs, self.buf_rews[env_idx], self.buf_dones[env_idx], self.buf_infos[env_idx] = self.envs[env_idx].step(
     44                 self.actions[env_idx]
     45             )

ValueError: setting an array element with a sequence.

I tried debugging it a bit. When env_stocktrading.step() is called inside dummy_vec_env.step_wait(), it mostly returns self.reward as numpy.float64. But once it returns it as numpy.ndarray and ends up in above error. In my case, it returned [0.261415 0.261415] and then threw exception while assigning it to self.buf_rews[env_idx].  I added print statement before this line:
print(str(type(self.reward)) + " - " + str(self.reward)  + " - " + str(type(self.reward_scaling)) + " - " + str(self.reward_scaling))

The last four printouts before error was:
<class 'numpy.float64'> - -973.2887000000046 - <class 'float'> - 0.0001
<class 'numpy.float64'> - -63.092250000016065 - <class 'float'> - 0.0001
<class 'numpy.float64'> - -1112.7334499999997 - <class 'float'> - 0.0001
<class 'numpy.ndarray'> - [-55.75 -55.75] - <class 'float'> - 0.0001

Seems that reward is somehow turning into ndarray. I am unable to guess why so  (just starting up with this 😶). Can someone please give me some quick pointer?
Is there any version incompatibility issue? But then, how it can work well for first tensorboard log and then fail. (This is my current setup.)
PS: Here is my notebook.
Update
I was referring to single stock example notebook which is now moved to old folder. The newer notebooks like for multistock (1 and 2) are using class finrl.env.env_stocktrading_v2.StockTradingEnvV2 which I don't see anywhere in the repository. Whats going wrong here?

I wasnt aware that that SingleStockTrading notebook is merged with MultiStockTrading one.


Default Python actions to use Python 3

The documentation says the following about Python actions:

can also create an action that runs with Python 3.6 by explicitly specifying the parameter --kind python:3.

By this point, Python 3 is established as the recommended Python version. The OpenWhisk platform can align with Python 3 by using as default.
Where projects need to use Python 2.7, they can specify --kind python:2

It is the default already now.


nut: tries to start upsd and upsdrv in netclient mode (should only start upsmon)

Describe the bug
A clear and concise description of what the bug is.
Steps To Reproduce
Steps to reproduce the behavior:

set power.ups = { enable = true; mode = "netclient"; }
rebuild switch
observe in syslog/systemd that upsd and upsdrv tried to start but failed

Expected behavior
Only the service upsmon is configured, enabled and started when in netclient mode.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
> nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.0, NixOS, 22.05 (Quokka), 22.05.20220606.f05e7d3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.9.0`
 - channels(root): `""`
 - nixpkgs: `/nix/store/c9p9vibs97r2n06gpzvbkg87dl643nm4-source`

(Not using channels but a flake, I'm on 22.05).

Duplicate of https://github.com/NixOS/nixpkgs/issues/113735.


release: 0.3.1

0.3.1

【fix】Fix the issue of incompatible use of the plugin with eslint-loader in webpack
【fix】Fix the issue of being able to recognizing the code-insiders IDE in macOS system

close #36


Streamline CSS

Context
With all of the recent changes with SRI and SCP, I moved a lot of stuff into my main.css file. Need to clean that up
Ideal behavior
No unused CSS is in main.css
Things to consider
Are there tools that can help me find unused CSS?
Should I split main.css into multiple files?

Done in https://github.com/jsolly/blogthedata/commit/f83ab63f58bd58dd35aa80c6d3622923b9fb3958


[BUG] Null reference exception in isolated Function application

Library name and version
Azure.Messaging.ServiceBus, v. <IP_ADDRESS>/omewhere it uses <IP_ADDRESS>
Describe the bug
I've noticed that my out-of-process Azure Function app fail sometime with a LOT of NullReferenceException that are thrown from Azure.Mesaging.ServiceBus.ReceiverManager.RenewMessageLockAsync.
Checking the logs in log analytics I get a long list of exceptions:

For some reason the logs also come from the same operationid, this error is logged around 7600 times.
My function app use <IP_ADDRESS> but it seems the version <IP_ADDRESS> is used somewhere (since that is in the stack trace).
Call stack:
System.NullReferenceException:
   at Azure.Messaging.ServiceBus.ReceiverManager+<RenewMessageLockAsync>d__18.MoveNext (Azure.Messaging.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=92742159e12e44c8)

FormattedMessage:
Message processing error (Action=RenewLock, EntityPath=<subscription>, Endpoint=<servicebus-endpoint>)

At the top of the execution a TaskCanceledException is thrown. I wonder if this has to do with when a instance is being terminated?
Expected behavior
No NullReferenceException is thrown.
Actual behavior
NullReferenceException is thrown.
Reproduction Steps

Create a out of process function app with a service bus trigger to a topic with subscription.
Trigger the function app 50 times per second (could vary from 30-60 requests per second).
See the error (hopefully) occurs, I see it around after 20 minutes.

Environment
Windows
dotnet6

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@torepaulsson,
This NullReferenceException was an issue in 7.8.0 that was fixed in 7.8.1. Can you include your projects dependencies so we can try to understand where this version is getting pulled in?

I think this may be due to how dependencies work when using isolated functions. I've reached out to the Functions team for more info.

It looks like we will need a new version of the isolated worker to be shipped. I'm transferring this issue over to repo that has the worker project.

Issue moved to Azure/azure-functions-dotnet-worker #1031 via Zenhub


Is it possible to display legend in another place ?

For example
https://www.iheartblazor.com/PieChart
there are 4 labels only. They are located in the center of page.
I have 24 labels. Is it possible to place the legend to the right bottom ?
Or somehow to change the existing layout?

From https://www.chartjs.org/docs/latest/configuration/legend.html#position might help you there. So, basically, you should be able to set the following positions:

top
left
bottom
right

In the library, you should be able to set Legend.Position (Check https://github.com/mariusmuntean/ChartJs.Blazor/blob/master/src/ChartJs.Blazor/ChartJS/Common/Properties/Legend.cs).
_config = new PieConfig
        {
            Options = new PieOptions
            {
                Legend = new Legend
                {
                    Position = Position.Bottom
                }
            }
        };

The example from https://www.iheartblazor.com/PieChart is located under https://github.com/mariusmuntean/ChartJs.Blazor/blob/master/samples/Shared/ChartJs.Blazor.Sample.Shared/Components/PieChartComponent.razor.
It's not possible to place the legend on the right bottom as far as I know. (Well, not easily at least).

Thank you @SeppPenner.
Nothing to add to his great comment except that we're sorry for the lack of documentation currently available.


what is the precision or performance of your model?

Sorry for the late answer.
with im2latex-100k dataset, Perplexity is about 1.1 and Exact match accuracy is about 0.8~ with 120 max token.


Add kill command

shitpost🤡
I wanted to implement the even better source engine kill command explode as well but I didn't feel like figuring it out
This command also sux, you can land in water and not die lol, maybe it should just make the player unconditionally get sucked. This is of extreme importance, this command is vital for the functionality of FEZUG.

If you're in water and do this, it should do the lava/sewer kill animation

sure buddy


Rename spec.sync to spec.quickSync

What this PR does / why we need it:
sync field has been changed to a quickSync field

https://pipecd.dev/docs/user-guide/configuration-reference/#kubernetesquicksync

Which issue(s) this PR fixes:

https://github.com/pipe-cd/pipe/pull/572

Does this PR introduce a user-facing change?:

NONE

/cla sign

/cla sign

Thank you.
/approve

Thank you.
/approve

Thank you for your contribution!

Thank you for your contribution!


Fix treatment of long file paths in Windows

3e5d171 (from November 2011) introduced code to fix treatment
of long file paths in Windows. Then, one week later, this commit
fixed long file paths directly in Node's fs library:
joyent/node@1f16a7b
So, the code currently in fstream actually /breaks/ long file
paths in Windows.
This fixes it, but keeps an additional mysterious line introduced
by that original commit that I don't understand.
I also added an automated test for this, and also guarded the
existing symlink test to not run on Windows... It would be nice
if the test suite were set up to automatically run on Windows as
well.
Fixes #30. Thanks to @sdarnell for identifying the problem.

I have some comments on this, and I'd like to get the eyes of somebody who has a better grasp of Node's history with UNC paths to take a look at this (@piscisaureus, is this something you could take a look at, or do you know who we could ask to take a look at it?). This is a more or less invisible piece of npm's infrastructure, so touching it is a little nervewracking for me, but if there's code in here that complicates filesystem handling on Windows, it would be good to get rid of it.


Create a Contact Us component for the fifth theme

As per provided designs, Create a Contact Us component for the fifth theme e

Reference for the design: https://www.figma.com/file/0ISIz6XklEToDd4KXfIWRB/SCoRe-Lab-Theme-designs?node-id=3%3A4
Cc: @charithccmc, @dinithminura

Hi! Can I be assigned for this task?


Fix setNativeProps for all elements that inherit from Shape.

The native props being set were not passing through extractProps, leading to various mismatches in the values.
More discussion here:
https://github.com/react-native-community/react-native-svg/issues/180
And here:
https://github.com/react-native-community/react-native-svg/issues/326

This does not appear to be needed. Closing this PR.


Bug: strange suggestion in my code base

Related to #12
Here is a list of header that were reported with your tool (v1.0.5)

non-canonical header "X-Real-IP", instead use: "X-Real-Ip"
non-canonical header "WWW-Authenticate", instead use: "Www-Authenticate"

These sounds strange, but I think it would be the same for ETag and others

@ccoVeille, Thank you for issue
I will close this issue, and created PR for https://github.com/lasiar/canonicalheader/issues/12.
See https://github.com/lasiar/canonicalheader/issues/12#issuecomment-2074217092.

Sure, thanks.


Create code_of_conduct.md

Hi ! I would like to add a code of conduct developed for your repository. Please let me know if it is okay to proceed. Please assign to me and add GSSoC24 Ext, hacktoberfest-accepted Labels.
Thanks

Hey @KirtiPratihar, any updates? :shipit:


Add search resource by uniqueName

The commit include:

Change findResourceByFriendlyName to findResourceByKeyAttribute to reflect the implementation
Use uniqueName (if existing) as priority key for searching resource by attribute

Fix #9

@kaiquelupo I tested the changes with the following script and a project with TaskRouter Taskchannel called sms (e.g. Flex):
'use strict';
const pulumi = require('@pulumi/pulumi');

const { Resource } = require('twilio-pulumi-provider');

const TaskChannel = new Resource('new-taskchannel', {
  resource: [
    'taskrouter',
    { workspaces: 'WS541570c274a259b51c95c9061d1ccadd' },
    'taskChannels',
  ],
  attributes: {
    friendlyName: 'SMS2',
    uniqueName: 'sms',
  },
});

Before the change, attempting to do pulumi up resulted in the following error:
     Type                               Name             Status                  Info
 +   pulumi:pulumi:Stack                unique-name-dev  **creating failed**     1 error
 +   └─ pulumi-nodejs:dynamic:Resource  new-taskchannel  **creating failed**     1 error

Diagnostics:
  pulumi:pulumi:Stack (unique-name-dev):

    error: update failed

  pulumi-nodejs:dynamic:Resource (new-taskchannel):
    error: Cannot create TaskChannel sms since a TaskChannel with the same UniqueName already exists

With the chnages, this is the output:
     Type                               Name             Status      Info
 +   pulumi:pulumi:Stack                unique-name-dev  created     2 messages
 +   └─ pulumi-nodejs:dynamic:Resource  new-taskchannel  created

Diagnostics:
  pulumi:pulumi:Stack (unique-name-dev):

    info: [runtime] Resource "sms" will be imported and attributes are different
    info: [runtime] Resource "sms" was imported

Resources:
    + 2 created

Let me know what you think.


asciinema is not reflecting in README file

Based on analysis, it seems to work in normal internet but certain corporates network is blocking its visibility.
Eg.  Wipro network
@panickervinod


Would a simple class to support declaration of resources via hiera be accepted into this mod?

Hi
There is already such a wrapper in the [augeasproviders] (https://github.com/hercules-team/augeasproviders/blob/master/manifests/instances.pp) module

:facepalm:


Pause download of some files within the torrent, But still downloads

WebTorrent version: 0.8.1 (Windows)

Even after pausing the some file within the torrent. The paused file still continues to download.
I tried multiple torrents, but same issue pops up

You need to first run:
// Remove default selection (whole torrent)
torrent.deselect(0, torrent.pieces.length - 1, false)

Then you can selective run:
torrent.files[0].select() // download the 1st file
torrent.files[5].select() // download the 6th file

// ... and do not download any of the other files

@feross What is the earliest time torrent.deselect() can be called? (to avoid downloading pieces and creating the files on disk). (After metadata or ready event?)

Right now, you have to call it after the ready event. But I'm going to fix that so it can be called earlier, after the metadata event.

Released as 0.95.6.