Upgrade Airflow to 2.3.2

Requires changing the pip_url in https://github.com/meltano/hub/blob/main/_data/meltano/orchestrators/airflow/apache.yml, as well as core.sql_alchemy_conn which is now database.sql_alchemy_conn (with env AIRFLOW__DATABASE__SQL_ALCHEMY_CONN), plus changing postgres:// to postgresql:// in the value per https://github.com/apache/airflow/blob/01a5d8fe93a48883f4ae0d6ab65eeb82a30a09b9/airflow/configuration.py#L372.
We'll have to document how to upgrade from an older version to 2.3.x by changing their config key (and value if postgres) in meltano.yml and env var key in .env and the broader environment when they upgrade (by running meltano lock to pull the latest metadata from the Hub and then manually updating the pip_url).
Have we thought about package-version-specific metadata on the Hub? I recall having seen that somewhere, but can't find the issue now.

Airflow 2.4.0 is now available, as of 2 days ago.


Refactor Script::bytes_to_asm_fmt to use iterator

Because of this change being somewhat large I didn't do it together with #658.
I also don't mind if this is merged after Taproot (feel free to put off review).
This refactors `Script::bytes_to_asm_fmt`` function to use an iterator
instead of index. Such change makes it easier to reason about overflows
or out-of-bounds accesses. As a result this also fixes three unlikely
overflows and happens to improve formatting to not output space at the
beginning in some weird cases.
To improve robustness even better it also moves read_uint
implementation to internal function which returns a more specific error
type which can be exhaustively matched on to guarantee correct error
handling. Probably because of lack of this the code was previously
checking the same condition twice, the second time being unreachable and
attempting to behave differently than the first one.
Finally this uses macro to deduplicate code which differs only in single
number, ensuring the code stays in sync across all branches.
I also attempted to get this work with no_panic crate to prove absence of panic but looks
like the compiler can not do that.
If someone wants to give it a try I can publish that change.
Unresolved question
Since Error seems to be related to script execution maybe UintError should be public?
At least NumericOverflow variant is strange.

I have force-pushed all obvious style changes. Will address size checking when replied and fuzz test in a different commit, not sure if I will have time to do it today.

CI error: error: failed to run LLVM passes: unknown pass name 'sancov'

@Kixunil this is known issue: https://github.com/rust-bitcoin/rust-bitcoin/issues/667
#664 will fix that and after we will need a rebase for nearly all PRs

Rebased on top of master to include #664 and added the fuzz test. Disclaimer: didn't try to compile the fuzz test, (ab)using CI for that.

Needs rebase to fix failed CI with fuzzying

@dr-orlovsky that was not CI related but my mistake. :)

@sanket1729 ah, I got confused when reading your comment previously.  I hope I can look at them soon. I came here just for quick sync need to go AFK.

@TheBlueMatt comment was addressed, so merging now


Some types cannot be obtained via Type.GetType

There was similar feedback in #6 , but now a new unrecognized type has been discovered.
methods of the interface： List GetAll(DbTransaction trans = null);
unobtainable type：DbTransaction
ParameterTypeName：System.Data.Common.DbTransaction,System.Data
Type.GetType("System.Data.Common.DbTransaction,System.Data")
will eventually return null thus throwing an exception
Is there a good way to solve this problem?

I seem to have found the cause of the problem.
Type.GetType("System.Data.Common.DbTransaction,System.Data.Common")
This gets the type correctly.
After the comma is the namespace of the class.
So changing parameterInfo.ParameterType.Assembly.GetName().Name to parameterInfo.ParameterType.Namespace in your 732e686 change should do the trick.

I tested various things, only these 2 works:
Type.GetType("System.Data.Common.DbTransaction, System.Data.Common");
Type.GetType("System.Data.Common.DbTransaction, System.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089");
Possibly Microsoft has made some tricks with this type and moved it, "forwarded" it or something strange.

The assembly-qualified name you tried also worked for me, so wouldn't it be more generic if we changed the ParameterTypeName = parameterInfo.ParameterType.FullName + "," + parameterInfo.ParameterType.Assembly.GetName().Name, to ParameterTypeName = parameterInfo.ParameterType .AssemblyQualifiedName, in 732e686?

To pass database transaction over the wire, System.Transactions.Transaction (TransactionScope) should be used.


docs: add docs badge and fix some errors in the documents

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

Project coverage is 91.88%. Comparing base (2114a81) to head (4ff81d7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   97.76%   91.88%   -5.89%
==========================================
  Files          10        4       -6
  Lines        1119      271     -848
==========================================
- Hits         1094      249     -845
+ Misses         16       13       -3
  Partials        9        9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


IDE project sync has memory leak in ProtocolToModelAdapter

After investigating a real-life memory dump, it seems like that ProtocolToModelAdapter$ViewGraphDetails contains an ever-increasing map.

The views field contains almost a million entries taking up 2.3 GB retained heap size. Looking at the Gradle source code indicates that map is never cleaned up, and also ViewKey.equals() has a suspicious implementation.

Since I don't know much about how this works, I have a naive question. Is this specific to IDE sync, or would it affect any TAPI build? For example, I think I'm seeing memory usage increase on each build when using gradle-profiler, which uses TAPI by default.

This is a general TAPI problem, so your assumption might be valid.

For what it's worth, I searched in a non-sync heap dump (generated during a gradle profiler run) for ProtocolToModelAdapter, and didn't find anything. I did find other TAPI classes, but didn't dig in yet to see if anything was suspicious.

Is the heap dump from the Gradle daemon or the IDE? This type is used in both places.

It is from a Gradle daemon.

A large number of entries in a map does not necessarily mean this is a leak. The Studio model is enormous on this large build and could easily contain millions of objects in the graph. I think we'd need more data that this is, in fact, a leak.
The entries in the map are retained only by a decorated object, so will be discarded when the decorated object is discarded by the (Studio provided) build action. So, we'd also need to see where the decorated objects are retained from.

Let me know if I can get you more information. Please include precise instructions if so (for example, run multiple syncs on the same daemon and get a heap dump before/after each one, etc).

Did we decide this isn't a leak?

This is still relevant.

Has this been fully addressed by #25267?

Yes, I think so.


Client-side errors in Chrome: no response or bad input

Hi guys,
I'm running etherium fine:
› geth --rpc --rpccorsdomain 'http://localhost:8080' --networkid 1010101 --protocolversion 60 --shh --unlock primary --bootnodes "enode://035b7845dfa0c0980112abdfdf4dc11087f56b77e10d2831f186ca12bc00f5b9327c427d03d9cd8106db01488905fb2200b5706f9e41c5d75885057691d9997c@[::]:30303" console
I0617 13:01:26.374563   77040 backend.go:269] Protocol Version: 60, Network Id: 1010101
I0617 13:01:26.374627   77040 backend.go:279] Blockchain DB Version: 3
I0617 13:01:26.380330   77040 chain_manager.go:247] Last block (#213059) 87f5dc15e24797828a57b0e028475dd5776dacad62fa671f7ca3a466ae5c2b41 TD=2001929772616
I0617 13:01:28.157373   77040 cmd.go:148] Starting Geth/v0.9.30/darwin/go1.4.2
I0617 13:01:28.157565   77040 server.go:293] Starting Server
I0617 13:01:28.498805   77040 udp.go:189] Listening, enode://615d3ae6bb6702b46e14691b5252df9e3e3c884c29d0707ab03dd6db0b17344122186f7a2cc9c7b798cf3ff1159635ba988701eb880525b46102c963f31a7698@[::]:30303
I0617 13:01:28.498926   77040 whisper.go:144] Whisper started
I0617 13:01:28.498937   77040 backend.go:494] Server started
I0617 13:01:28.498927   77040 server.go:530] Listening on [::]:30303
Unlocking account 0x6f21c15f759b0c5a67d28c40d8f6a5c08a4f4068 | Attempt 1/3
Passphrase:
Account '0x6f21c15f759b0c5a67d28c40d8f6a5c08a4f4068' unlocked.
I0617 13:01:33.473205   77040 ipc_unix.go:74] IPC service started (/Users/aroche/Library/Ethereum/geth.ipc)
>

And augur-client built and ran fine.  I'm running https://github.com/AugurProject/augur-client/commit/480c1ff76bf660d5f17ed9a4e4af3f8cdacb500e.

--protocolversion 60 should be --protocolversion 59

I deleted the blockchain and tried that, and it still has the same problem.

What version of geth are you running (geth version)?  Also, are you connected to any peers that are also on version 59?  (To check, type admin.peers() at the geth console.)  It may be you just do not have our contracts downloaded yet!

› geth version
Geth
Version: 0.9.30
Protocol Version: 60
Network Id: 0
Go Version: go1.4.2
OS: darwin

admin.peers()
[{
  Caps: 'eth/59',
  ID: 'f5fc10dafe8c44702748c7ead4f30d7b3fe35450d2e66158231a9bf9b1838f93d06b25908b8447c85b2429bdaeff45709f17e67083791053e0bac6e282c969fe',
  LocalAddress: '<IP_ADDRESS>:62112',
  Name: 'Geth/v0.9.31-dfd18d24/linux/go1.4.2',
  RemoteAddress: '<IP_ADDRESS>:30303'
} ]

Still doing block synchronisation though. Is this the error message you're expecting if the blocks haven't finished synchronising?

Embarrassingly, yes, it is.  We need to have a more informative message/popup for "contracts not downloaded"!  Let me know if you still get this error once block synchronization is complete.

you will get odd errors if you're not current.  however, there should be a friendly modal that comes up if you're last block is older than maybe a minutes.

thanks guys! under https://github.com/AugurProject/augur-client#running-ethereum you don't mention the protocol version thing, even though the latest default version seems to be 60. might be worth adding a note?

oh, i'll change to the new instruction

can we close this?

Yep! Thanks for your help
On Thu, 18 Jun 2015 at 7:31 am Scott Leonard<EMAIL_ADDRESS>wrote:

can we close this?
—
Reply to this email directly or view it on GitHub
https://github.com/AugurProject/augur-client/issues/72#issuecomment-112955213
.


compile localized dashboard messages when installing from source/package

Leslie asked me to take a look at this and it was a simple enough fix.

PR message say it is for source but code change is for both source and package install. One of the 2 needs to be corrected.

@nirajdp76 the compiled localization files are missing from packages (but not RHOSP). If we can get this step into package building, then we can reduce the constraint to source instead of not distro.


Update README.md

Adding a great online course on deep learning by Andrew Ng

Thank you for the suggestion! It seems this link has changed and is now a company. Closing for now.


ReferenceError: $jscomp is not defined

Hello!
I've downloaded the most recent version of compiler and i've tried to optimize js code with these flags:
java -jar closure-compiler-v20211006.jar -W VERBOSE -O WHITESPACE_ONLY --language_out ECMASCRIPT5 $script_path_in --js_output_file $script_path_tmp
closure compiler has optimized this lines of code:
for(var extraProperty of extraProperties){
		option.setAttribute(extraProperty,initialOption[extraProperty]);
	}

into
 for (var $jscomp$iter$0 = $jscomp.makeIterator(extraProperties), $jscomp$key$extraProperty = $jscomp$iter$0.next(); !$jscomp$key$extraProperty.done; $jscomp$key$extraProperty = $jscomp$iter$0.next()) {
        var extraProperty = $jscomp$key$extraProperty.value;
        {
            option.setAttribute(extraProperty, initialOption[extraProperty])
        }
    }

And as a result i receive such error in browser:
all_compressed.js Uncaught ReferenceError: $jscomp is not defined

The problem described in the original post is that we really shouldn't allow both WHITESPACE_ONLY mode and transpilation.
That's a contradiction. If you're transpiling, then you're not changing only whitespace.
I believe @blickly is working on a change that will adapt WHITESPACE_ONLY to automatically disable transpilation.
The secondary problem is that the code given to the compiler is trying to conditionally redefine globals the compiler expects to already be there. You can probably get the compiler to be OK with this by suppressing as suggested earlier. Here's what I specifically suggest.
/**
 * If `Node` isn't present, supply my own implementation.
 *
 * Tell closure-compiler to ignore the fact that I'm defining something it thinks is already defined.
 * @suppress {duplicate}
 */
var Node = Node || MyNodeImplementation;

I believe @blickly is working on a change that will adapt WHITESPACE_ONLY to automatically disable transpilation.

It's actually very bad in this particular situation, because i've used transpilation to feed resulting code to YUICompressor, because it's out of date and doesn't support new features, although it obfuscates code with no errors out of the box.

The secondary problem is that the code given to the compiler is trying to conditionally redefine globals the compiler expects to already be there. You can probably get the compiler to be OK with this by suppressing as suggested earlier. Here's what I specifically suggest.

Thank you, @brad4d. It's a good option that solved problem with node, however a bunch of other errors arose.
I suppose there is no easy way to move from  YUICompressor to "closure-compiler" solution...
It would be great if closure compiler may have a "dumb" mode, just to do a code obfuscation without any fancy things like it does YUICompressor.
$ java -jar closure-compiler-v20211006.jar -W VERBOSE -O SIMPLE  ../WebContent/pages/scripts/all.js
 ../WebContent/pages/scripts/all.js:241:16: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: key
  241|             var key = keys[i], str = Str(key, value, stack);
                       ^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:3783:4: WARNING - [JSC_REFERENCE_BEFORE_DECLARE] Variable referenced before declaration: element
  3783|     element = $(element);
            ^^^^^^^
../WebContent/pages/scripts/all.js:3783:16: WARNING - [JSC_REFERENCE_BEFORE_DECLARE] Variable referenced before declaration: element
  3783|     element = $(element);
                        ^^^^^^^
../WebContent/pages/scripts/all.js:4154:34: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: match
  4154|         var type = Expr.order[i], match;
                                          ^^^^^
../WebContent/pages/scripts/all.js:4542:26: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: i
  4542|                 for ( var i = 0, l = not.length; i < l; i++ ) {
                                  ^^^^^
../WebContent/pages/scripts/all.js:5248:20: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: values
  5248|     for (var i = 0, values = []; i < length; i++) {
                            ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:6187:10: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: pos
  6187|       var pos = ((-Math.cos(pos*Math.PI)/4) + .75) + Math.random()/4;
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:10125:41: WARNING - [JSC_REFERENCE_BEFORE_DECLARE] Variable referenced before declaration: selected
  10125|        option = new Option(option, option, selected, selected);
                                                    ^^^^^^^^
../WebContent/pages/scripts/all.js:10125:51: WARNING - [JSC_REFERENCE_BEFORE_DECLARE] Variable referenced before declaration: selected
  10125|        option = new Option(option, option, selected, selected);
                                                              ^^^^^^^^
../WebContent/pages/scripts/all.js:11224:12: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: time
  11224|         var time = date.getTime();
                     ^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:13764:16: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: ret
  13764|             var ret = invokeAsMethod(this, this.sizeLimit.blurFunction, [event, this.sizeLimit]);
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:16289:8: WARNING - [JSC_TYPE_PARSE_ERROR] Bad type annotation. expecting a variable name in a @param tag. See https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler for more information.
  16289| * @param: defaultUrl - is not context aware, just last part of address, i.e. action name /searchSmth
                 ^
../WebContent/pages/scripts/all.js:17688:5: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: cal
  17688|        var cal = cal;
                    ^^^^^^^^^
../WebContent/pages/scripts/all.js:17719:5: WARNING - [JSC_REDECLARED_VARIABLE] Redeclared variable: cal
  17719|        var cal = cal;
                    ^^^^^^^^^
../WebContent/pages/scripts/all.js:4982:3: ERROR - [JSC_UNDEFINED_VARIABLE] variable Sizzle is undeclared
  4982| })(Sizzle);
           ^^^^^^
../WebContent/pages/scripts/all.js:6076:16: ERROR - [JSC_UNDEFINED_VARIABLE] variable Selector is undeclared
  6076|   Object.extend(Selector, {
                        ^^^^^^^^
../WebContent/pages/scripts/all.js:6408:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable property is undeclared
  6408|     for(property in this)
                ^^^^^^^^
../WebContent/pages/scripts/all.js:6627:17: ERROR - [JSC_UNDEFINED_VARIABLE] variable scrollTo is undeclared
  6627|     function(p){ scrollTo(scrollOffsets.left, p.round()); }
                         ^^^^^^^^
../WebContent/pages/scripts/all.js:7133:34: ERROR - [JSC_UNDEFINED_VARIABLE] variable $$ is undeclared
  7133|         var elements = [$(ids) || $$(ids)].flatten();
                                          ^^
../WebContent/pages/scripts/all.js:7274:4: ERROR - [JSC_UNDEFINED_VARIABLE] variable deepest is undeclared
  7274|     deepest = drops[0];
            ^^^^^^^
../WebContent/pages/scripts/all.js:7535:10: ERROR - [JSC_UNDEFINED_VARIABLE] variable tag_name is undeclared
  7535|       if((tag_name = src.tagName.toUpperCase()) && (
                  ^^^^^^^^
../WebContent/pages/scripts/all.js:7603:81: ERROR - [JSC_UNDEFINED_VARIABLE] variable width is undeclared
  7603|         with(this._getWindowScroll(this.options.scroll)) { p = [ left, top, left+width, top+height ]; }
                                                                                ^^^^^
../WebContent/pages/scripts/all.js:7603:92: ERROR - [JSC_UNDEFINED_VARIABLE] variable height is undeclared
  7603|         with(this._getWindowScroll(this.options.scroll)) { p = [ left, top, left+width, top+height ]; }
                                                                                ^^^^^^
../WebContent/pages/scripts/all.js:7777:40: ERROR - [JSC_UNDEFINED_VARIABLE] variable documentElement is undeclared
  7777|       if (w.document.documentElement && documentElement.scrollTop) {
                                                ^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:7781:12: ERROR - [JSC_UNDEFINED_VARIABLE] variable body is undeclared
  7781|         T = body.scrollTop;
                    ^^^^
../WebContent/pages/scripts/all.js:9232:15: ERROR - [JSC_UNDEFINED_VARIABLE] variable sheet is undeclared
  9232|                 for (var h=0;sheet=Behaviour.list[h];h++){
                                     ^^^^^
../WebContent/pages/scripts/all.js:9234:4: ERROR - [JSC_UNDEFINED_VARIABLE] variable list is undeclared
  9234|                                 list = document.getElementsBySelector(selector);
                                        ^^^^
../WebContent/pages/scripts/all.js:9240:13: ERROR - [JSC_UNDEFINED_VARIABLE] variable element is undeclared
  9240|                                 for (i=0;element=list[i];i++){
                                                 ^^^^^^^
../WebContent/pages/scripts/all.js:9350:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable sourceName is undeclared
  9350|         sourceName = source;
                ^^^^^^^^^^
../WebContent/pages/scripts/all.js:9352:37: ERROR - [JSC_UNDEFINED_VARIABLE] variable parent is undeclared
  9352|         if (source == null) source = parent.frames[sourceName];
                                             ^^^^^^
../WebContent/pages/scripts/all.js:9656:17: ERROR - [JSC_UNDEFINED_VARIABLE] variable j is undeclared
  9656|             for (j = 0; j < current.length; j++) {
                         ^
../WebContent/pages/scripts/all.js:11394:3: ERROR - [JSC_UNDEFINED_VARIABLE] variable sep is undeclared
  11394|                        sep = separator;
                                ^^^
../WebContent/pages/scripts/all.js:11647:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable ret is undeclared
  11647|         ret = new Object();
                 ^^^
../WebContent/pages/scripts/all.js:12711:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable numbers is undeclared
  12711| numbers = new Input(JST_CHARS_NUMBERS);
         ^^^^^^^
../WebContent/pages/scripts/all.js:12712:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable optionalNumbers is undeclared
  12712| optionalNumbers = new Input(JST_CHARS_NUMBERS);
         ^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:12714:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable oneToTwoNumbers is undeclared
  12714| oneToTwoNumbers = new Input(JST_CHARS_NUMBERS, 1, 2);
         ^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:12715:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable year is undeclared
  12715| year = new Input(JST_CHARS_NUMBERS, 1, 4, getFullYear);
         ^^^^
../WebContent/pages/scripts/all.js:12716:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable dateSep is undeclared
  12716| dateSep = new Literal("/");
         ^^^^^^^
../WebContent/pages/scripts/all.js:12717:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable dateTimeSep is undeclared
  12717| dateTimeSep = new Literal(" ");
         ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:12718:0: ERROR - [JSC_UNDEFINED_VARIABLE] variable timeSep is undeclared
  12718| timeSep = new Literal(":");
         ^^^^^^^
../WebContent/pages/scripts/all.js:12754:9: ERROR - [JSC_VAR_MULTIPLY_DECLARED_ERROR] Variable i declared more than once. First occurrence: ../WebContent/pages/scripts/all.js:9279:9
  12754| for (var i = 0; i < 32; i++) {
                  ^^^^^
../WebContent/pages/scripts/all.js:12758:9: ERROR - [JSC_VAR_MULTIPLY_DECLARED_ERROR] Variable i declared more than once. First occurrence: ../WebContent/pages/scripts/all.js:9279:9
  12758| for (var i = 112; i <= 123; i++) {
                  ^^^^^^^
../WebContent/pages/scripts/all.js:14939:373: ERROR - [JSC_UNDEFINED_VARIABLE] variable context is undeclared
  14939|                        this.header.innerHTML = "

" + text + "

";
                                                                                ^^^^^^^
../WebContent/pages/scripts/all.js:15136:20: ERROR - [JSC_UNDEFINED_VARIABLE] variable pathPrefix is undeclared
  15136|        var path = isEmpty(pathPrefix) ? pathPrefix_ : pathPrefix;
                                   ^^^^^^^^^^
../WebContent/pages/scripts/all.js:15190:22: ERROR - [JSC_UNDEFINED_VARIABLE] variable richEditorsToInitialize is undeclared
  15190|                for (var i = 0; i < richEditorsToInitialize.length; i++) {
                                            ^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:15256:1: ERROR - [JSC_UNDEFINED_VARIABLE] variable request is undeclared
  15256|        request = new Ajax.Request(action, {
                ^^^^^^^
../WebContent/pages/scripts/all.js:15268:6: ERROR - [JSC_UNDEFINED_VARIABLE] variable alertAjaxError is undeclared
  15268|                if (alertAjaxError){
                            ^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:15459:1: ERROR - [JSC_UNDEFINED_VARIABLE] variable Calendar is undeclared
  15459|        Calendar.setup({
                ^^^^^^^^
../WebContent/pages/scripts/all.js:15460:22: ERROR - [JSC_UNDEFINED_VARIABLE] variable calendarDateTimeFormat is undeclared
  15460|                ifFormat: useTime ? calendarDateTimeFormat : calendarDateFormat,
                                            ^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:15460:47: ERROR - [JSC_UNDEFINED_VARIABLE] variable calendarDateFormat is undeclared
  15460|                ifFormat: useTime ? calendarDateTimeFormat : calendarDateFormat,
                                                                     ^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:15468:14: ERROR - [JSC_UNDEFINED_VARIABLE] variable dateTimeParser is undeclared
  15468|                timeFormat: dateTimeParser.mask.indexOf("h") >= 0 ? "12" : "24",
                                    ^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:15516:1: ERROR - [JSC_UNDEFINED_VARIABLE] variable allMenus is undeclared
  15516|        allMenus.each(function(menu) {
                ^^^^^^^^
../WebContent/pages/scripts/all.js:15572:1: ERROR - [JSC_UNDEFINED_VARIABLE] variable container is undeclared
  15572|        container = getSubMenuContainer(menu);
                ^^^^^^^^^
../WebContent/pages/scripts/all.js:16055:17: ERROR - [JSC_UNDEFINED_VARIABLE] variable event is undeclared
  16055|                modifyButton = event.target.type == "button"? event.target : this.down(".modifyButton");
                                       ^^^^^
../WebContent/pages/scripts/all.js:16085:23: ERROR - [JSC_UNDEFINED_VARIABLE] variable cancelLabel is undeclared
  16085|                modifyButton.value = cancelLabel;
                                             ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:16112:24: ERROR - [JSC_UNDEFINED_VARIABLE] variable modifyLabel is undeclared
  16112|                        modifyButton.value = modifyLabel;
                                                     ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:16127:1: ERROR - [JSC_UNDEFINED_VARIABLE] variable callback is undeclared
  16127|        callback = function(field) {
                ^^^^^^^^
../WebContent/pages/scripts/all.js:16217:41: ERROR - [JSC_UNDEFINED_VARIABLE] variable defaultMessageText is undeclared
  16217|        div.appendChild(document.createTextNode(defaultMessageText || " "));
                                                        ^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:16524:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable params is undeclared
  16524|                params = $H();
                        ^^^^^^
../WebContent/pages/scripts/all.js:16749:6: ERROR - [JSC_UNDEFINED_VARIABLE] variable accountNumberLength is undeclared
  16749|                if (accountNumberLength > 0 && !usernameControl.mask) {
                            ^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17097:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable render is undeclared
  17097|                render(container, stringValueElId, json, stringValue);
                        ^^^^^^
../WebContent/pages/scripts/all.js:17098:25: ERROR - [JSC_UNDEFINED_VARIABLE] variable JsonSchemeHandlerInjected is undeclared
  17098|                console.debug("inj: ", JsonSchemeHandlerInjected);
                                               ^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17119:52: ERROR - [JSC_UNDEFINED_VARIABLE] variable noPictureCaption is undeclared
  17119| var noPictureDescriptor = new ImageDescriptor(null, noPictureCaption, context + "/systemImage?image=noPicture&thumbnail=true");
                                                             ^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17199:14: ERROR - [JSC_UNDEFINED_VARIABLE] variable imageRemoveMessage is undeclared
  17199|                if (confirm(imageRemoveMessage)) {
                                    ^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17212:12: ERROR - [JSC_UNDEFINED_VARIABLE] variable errorRemovingImageMessage is undeclared
  17212|                                                alert(errorRemovingImageMessage);
                                                              ^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17246:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable imageDetailsSuccess is undeclared
  17246|                alert(imageDetailsSuccess);
                              ^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17250:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable imageDetailsError is undeclared
  17250|                alert(imageDetailsError);
                              ^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17277:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable imageRemovedMessage is undeclared
  17277|                alert(imageRemovedMessage);
                              ^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17326:46: ERROR - [JSC_UNDEFINED_VARIABLE] variable headBehaviour is undeclared
  17326|                $A(cell.getElementsByTagName("input")).each(headBehaviour.input);
                                                                    ^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17357:22: ERROR - [JSC_UNDEFINED_VARIABLE] variable receiptPrinterNoConfigurationError is undeclared
  17357|                 alert(receiptPrinterNoConfigurationError);
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17392:29: ERROR - [JSC_UNDEFINED_VARIABLE] variable receiptPrinterNotFoundError is undeclared
  17392|             alert(replaceAll(receiptPrinterNotFoundError, "#printer#", printerName));
                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17413:18: ERROR - [JSC_UNDEFINED_VARIABLE] variable receiptPrinterAppletError is undeclared
  17413|             alert(receiptPrinterAppletError + "\n\n" + debug(e));
                           ^^^^^^^^^^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17428:22: ERROR - [JSC_UNDEFINED_VARIABLE] variable printError is undeclared
  17428|                 alert(printError + "\n\n" + e.getLocalizedMessage());
                               ^^^^^^^^^^
../WebContent/pages/scripts/all.js:17655:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable addEvent is undeclared
  17655|                addEvent(el, "mouseover", dayMouseOver);
                        ^^^^^^^^
../WebContent/pages/scripts/all.js:17655:28: ERROR - [JSC_UNDEFINED_VARIABLE] variable dayMouseOver is undeclared
  17655|                addEvent(el, "mouseover", dayMouseOver);
                                                  ^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17656:28: ERROR - [JSC_UNDEFINED_VARIABLE] variable dayMouseDown is undeclared
  17656|                addEvent(el, "mousedown", dayMouseDown);
                                                  ^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17657:27: ERROR - [JSC_UNDEFINED_VARIABLE] variable dayMouseOut is undeclared
  17657|                addEvent(el, "mouseout", dayMouseOut);
                                                 ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17658:6: ERROR - [JSC_UNDEFINED_VARIABLE] variable is_ie is undeclared
  17658|                if (is_ie) {
                            ^^^^^
../WebContent/pages/scripts/all.js:17659:28: ERROR - [JSC_UNDEFINED_VARIABLE] variable dayMouseDblClick is undeclared
  17659|                        addEvent(el, "dblclick", dayMouseDblClick);
                                                         ^^^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17803:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable removeEvent is undeclared
  17803|                removeEvent(document, "mouseup", tableMouseUp);
                        ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17803:35: ERROR - [JSC_UNDEFINED_VARIABLE] variable tableMouseUp is undeclared
  17803|                removeEvent(document, "mouseup", tableMouseUp);
                                                         ^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17804:37: ERROR - [JSC_UNDEFINED_VARIABLE] variable tableMouseOver is undeclared
  17804|                removeEvent(document, "mouseover", tableMouseOver);
                                                           ^^^^^^^^^^^^^^
../WebContent/pages/scripts/all.js:17807:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable _C is undeclared
  17807|                _C = null;
                        ^^
../WebContent/pages/scripts/all.js:17808:9: ERROR - [JSC_UNDEFINED_VARIABLE] variable stopEvent is undeclared
  17808|                return stopEvent(ev);
                               ^^^^^^^^^
../WebContent/pages/scripts/all.js:17926:37: ERROR - [JSC_UNDEFINED_VARIABLE] variable calDragIt is undeclared
  17926|                removeEvent(document, "mousemove", calDragIt);
                                                           ^^^^^^^^^
../WebContent/pages/scripts/all.js:17927:35: ERROR - [JSC_UNDEFINED_VARIABLE] variable calDragEnd is undeclared
  17927|                removeEvent(document, "mouseup", calDragEnd);
                                                         ^^^^^^^^^^
../WebContent/pages/scripts/all.js:17947:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable addClass is undeclared
  17947|                addClass(el, "hilite active");
                        ^^^^^^^^
../WebContent/pages/scripts/all.js:17993:11: ERROR - [JSC_UNDEFINED_VARIABLE] variable getElement is undeclared
  17993|                var el = getElement(ev);
                                 ^^^^^^^^^^
../WebContent/pages/scripts/all.js:17994:6: ERROR - [JSC_UNDEFINED_VARIABLE] variable isRelated is undeclared
  17994|                if (isRelated(el, ev) || _C || el.disabled)
                            ^^^^^^^^^
../WebContent/pages/scripts/all.js:17996:2: ERROR - [JSC_UNDEFINED_VARIABLE] variable removeClass is undeclared
  17996|                removeClass(el, "hilite");
                        ^^^^^^^^^^^
../WebContent/pages/scripts/all.js:18000:36: ERROR - [JSC_UNDEFINED_VARIABLE] variable _TT is undeclared
  18000|                        el.calendar.tooltips.innerHTML = _TT["SEL_DATE"];
                                                                 ^^^
77 error(s), 14 warning(s)

Thank you @all for your support!

It is a sad fact that closure-compiler often chokes on JS code that was not written with closure-compiler in mind.
Our compiler is intentionally very restrictive and opinionated about what its input should look like, because that allows us to both enforce certain coding practices we consider more readable and less bug-prone and ensure some assumptions the compiler needs to make to do an efficient job are correct.
If you have a relatively small code base, it can make sense to fix the JS code to meet closure-compiler's restrictions.
Most of them exist for good reason and fixing the errors will likely make your code less bug-prone.
However, this isn't a reasonable solution for a lot of large projects.
If you aren't looking to get the high level of code checking and code size optimization closure-compiler is designed for, you probably want to use some other tool.
Sorry.

I totally agree with you @brad4d, but in this case the project is quite large, and we are searching a tool that will work out of the box and wouldn't force us rewriting all code we have.

Our compiler is intentionally very restrictive and opinionated about what its input should look like, because that allows us to both enforce certain coding practices we consider more readable and less bug-prone and ensure some assumptions the compiler needs to make to do an efficient job are correct.

And that is great, however, why you haven't considered adding a support for “JS code that was not written with closure-compiler in mind”? We have done nothing with our js code to be able to use YUICompressor, and as i can imagine that means that closure-compiler can do that, too, there just should be an option to control how restrictive closure-compiler must be. Of course, only for code obfuscation.
Anyway, thank you, @all, for your support and for creating such a great tool!
Best wishes!

The problem described in the original post is that we really shouldn't allow both WHITESPACE_ONLY mode and transpilation.
That's a contradiction. If you're transpiling, then you're not changing only whitespace.

I believe @blickly is working on a change that will adapt WHITESPACE_ONLY to automatically disable transpilation.

@brad4d was that implemented already? We are hitting the same issue in ClojureScript (which relies on Closure) where $jscomp is undefined when language out is set to es5. Is there a workaround that doesn't involve setting optimizations to simple? We've started seeing it after upgrade of Closure from v20210505 to v20220601

@brad4d Here's a repository demoing a tiny reproduction case for the issue @roman01la mentioned. In our case the issue seems to be rooted in ES6 classes that extend from other classes and $jscomp.inherits being undefined: https://github.com/stigi/clojurescript-es5-issue


Add bloat-check workflow

This will (hopefully) run a binary size comparison against master
when you comment /bloat on a PR.
(this might not work until it gets into master? not totally sure, I tested on a fork and it was working? 🤷♂)

/bloat
(I suspect this has to be merged to be runnable, which is annoying.)

I promise to fix any messes I make. 😁


C/C++ extension's intellisense doesn't work. First I couldn't update the extension, then when I manually downloaded it and installed the VSIX file it says that intellisense will be disabled

[

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

https://imgur.com/a/59rlrUv

I'm using VS code(32 bit) v1.70.2

/extC++

It looks like this is caused by the C++ extension. Please file the issue to the C++ extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Allow multiple evaluations of the same field in a rule

When writing rules, gui or CLI, if I try to evaluate {{ trigger.body }} with a contains and an additional ncontains, only one of the criteria is saved.  It would be VERY handy to be able to do 2 or more criteria evals on the same field.

@lmarty You can achieve by writing another rule. This is intentionally designed that way so rules are simple to grok. Nevertheless, good feedback.

@lmarty do you mean something like OR operator (by default, if you have multiple conditions, it behaves as AND).
If you indeed mean OR, you can achieve that, as @lakshmi-kannan has pointed out, by using multiple rules.
If that behavior is not documented yet, or documented poorly, we should make it more explicit in the documentation.

So I need to create a workflow if I want to evaluate message bodies for multiple criteria?  That seems off to me, with the way the GUI is laid out.  We can click to add AND criteria, no mention of limits there or in the docs.
@Kami, I am not looking for an OR, definitely an AND. For example, an Amazon SQS message triggers an event.  I first want to evaluate trigger.body of that event and make sure it is the type of event I am looking for, then, I want to check to make sure it did not originate from an expected source.  ie: Message contains text X AND ncontains text Y.  Both of those data points being in trigger.body.  If this is truly by design, it should definitely be called out in the docs/tooltips/help/somewhere.

@lmarty Looks like I misunderstood you then.
If you are looking for AND, this should work out of the box just fine - you simple need to add multiple criteria expressions.
In your case, it would look something like that:
criteria:
    trigger.body:
        type: "contains"
        pattern : "X"
    trigger.body:
        type: "ncontains"
        pattern : "Y"
...

You can also add more criteria expressions if needed.

Thanks Kami!  Unfortunately, when I do as described above, via the GUI or the CLI, it only takes and displays the last criteria element.  So in the above case, the "contains" criteria would be dropped and the "ncontains" would be the only active element.  I'm on st2 1.4, and I've rebuilt the box fresh to validate the behavior.

@lmarty You are welcome.
This actually sounds like a WebUI bug. It should work just fine if you create rule using the CLI (st2 rule create path/to/rule/file.yaml and then st2 rule get  to verify the criteria).
/cc @enykeev

@lmarty Actually, sorry, I was wrong (I just noticed it now).
Now I see why the example I posted above won't work - criteria is a dictionary which means you can't reference the same key / field (in this case trigger.body) twice.
That's an unfortunate limitation and one of the workarounds you can probably use is a regex operator and regular expression which fits your criteria.
/cc @lakshmi-kannan @manasdk et. al. this seems a pretty unfortunate limitation, we should discuss how to fix it.
One option would be to allow body of a single criteria expression to be a list of dictionaries, but for backward comparability we would also allow plain dictionaries, or something along those lines.

Now I see why the example I posted above won't work - criteria is a dictionary which means you can't reference the same key / field (in this case trigger.body) twice.

This is indeed a limitation.

I knew I wasn't crazy.   Bummer guys! This would be a very handy feature to have for us.  Keep me in the loop as you move forward.  I'll try and find a way around this with other sources of unique in my event stream in the meantime.

My guess is that this would need some sort of schema change to do this the right way.

Took a closer look at this today and the only reasonable approach would require a schema change of the criteria field in rules; move it from dictionary to list. This ofcourse has right upto what we save in the DB so would require migration scripts etc.

Any update on this? I would like to do something like the following:
"trigger.body.data.latitude": {
  "type": "lessthan",
  "pattern": "30"
},
"trigger.body.data.latitude": {
  "type": "greaterthan",
  "pattern": "50"
},
"trigger.body.data.longitude": {
  "type": "greaterthan",
  "pattern": "-90"
},
"trigger.body.data.longitude": {
  "type": "lessthan",
  "pattern": "-100"
},

Sort of a geo-fence. I get the same problem described above, only the latest key is kept.


Note the fork for current development

Add link to https://github.com/Morchana/morchana-app

This can be close, as the development will be moved back to this repo, according to PR #52.

This can be close, as the development will be moved back to this repo, according to PR #52.


Update 1.4.0

Update to version 1.4.0.

src/rp2_common/hardware_pll/pll.c changed from 1.3.0 to 1.4.0
added code to refer to MHZ. Modify it additionally.

irq_is_enabled() that defined in irq.h of PICO-SDK conflicts with zephyr's API.
Rename it to avoid compile errors.

@soburi Can you fix the merge conflict?

@yonsch
I've already fixed the conflict caused by running git merge 1.4.0.
Is this answering the intention of your question?

little commit message updating.

@yonsch

@soburi Can you fix the merge conflict?

I got this message. It seems cannot.

1.5.0 merged with #4.


Add subsections to components and libraries

Goal: more clarity.
Propositions:
Charts
Testing
UI elements

What if the library can be in multiple categories? Like UI elements and Charts?

It's a very good point, I asked myself the same question when I wrote these propositions. We have to think a little more: subsections trees, or we put the library in the more restrictive subsection which fits.
Or maybe "UI elements" is too large.
We have to think a little more about that but I think it could be great to order the libraries in the same ways we order the samples, it should be easier to find a library you are interested in.

Proposition:

Charts
Testing
CSS Frameworks
JavaScript ports
Others

I think a differentiation between component bundles (multiple different components in one package/repo, e.g. MatBlazor, BlazorMdc, Syncfusion, Telerik, etc.) and individual components (e.g. just a map) would be useful.
A fantastic thing would be a table for the bundles that shows which component types are available in each bundle. But maybe on a separate page?
What do you mean by "JavaScript ports"? Are you referring to components that simply wrap a JS component library, or to JS API wrappers (e.g. geolocation), or both?

I like the idea from @stefanloerwald about having a table with mapped features of the library. And one thing with table is that it could be like a badge-of-honnor in a way. So only top 5 for example would be on the list.

I created a draft for more categories in the component section here. Is this too many categories? Or do you think some are missing?
I haven't yet started work on a feature comparison table.

I've now also added a draft table for the feature comparison. Due to the high number of libraries, the table is not easy to digest, so we need to split it up somehow. Ideas?
feature comparison table for component bundles

The idea is to have the number of components implemented by the library in the "Components" row, then a simple yes/no (maybe also "preview") for all the specific components, and a list of any other notable components in "Other".

Hi stefanloerwald,
Thank's for the work you've done, it's a very good start!
Could you push you changes without the table at start?
About the feature comparison table, I think it's a good idea too but maybe we can create a separate page for this and put a link to this page in the subsection introduction? What do you think? It will let us the time to find the best formula to present data then to reintroduce it in the main page later. What do you think?

Regarding the table maybe it would be better to switch columns and rows, so that columns will represent the features. I think that way ti would be easier to read.
Example

Description
CSS framework
Components
DataGrid
Charts

MatBlazor
...
?
?
?
?

AntDesign Blazor
...
?
?
?
?

Blazorise
...
?
?
?
?

etc.

I agree!

@AdrienTorris will push asap.
@stsrki I'm not sure which way round is easier. Maybe splitting the table up into two is more sensible? One for metadata (license, stars, last commit, demo etc), one for feature comparison.

And then probably transposed, you're right.


Davids dev

@Duane321

maybe I'll create another branch to implement the vectorization idea we have discussed and have another PR when finished.
let's don't merge it on main for this one.

Sure, that's fine. Thanks David!


Dev guide (Glossary and Product Survey)

Pull Request Test Coverage Report for Build 12

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

Totals

Change from base Build 6:
0.0%

Covered Lines:
1470

Relevant Lines:
1601

💛  - Coveralls


Unfixable broken bank

Reporting on behalf of zecture#4266 /<PHONE_NUMBER>04937728
Bso Bank is broken, =rp checkbank --fix does nothing, see below
any =b command returns "That item doesn't exist."

Update: its his whole minion not just bank

Its because you have a yellow graceful item (that was deleted, its a fake interface item and not the real one...) equipped, try to unequipall from the setup its in, then do the rp checkbank fix. that should fix it.


Feature/migration logic

Main change introduced in this PR:
Capturing webhook items migrated by Hecate from SNS to Argus.
Minor changes:

Remove group loggers.
Log errors when converting items to webhooks
Don't return partial data: when an item (from among many) fails to convert into a webhook, the whole getAll operation should fail unless we advertise GetAll to be a best effort operation.

Closing this in favor of #31


[C&U] fix bug in Targets#capture_vm_targets

introduced in eef58e1b8f7d0de430
specs in metric_capture exposed this
This will potentially not capture vms for C&U

Marked fine/yes because the PR that introduced this https://github.com/ManageIQ/manageiq/pull/16004, is also fine/yes

Gaprindashvili backport details:
$ git log -1
commit 03579dd4ca13781838871a2665871d502649873f
Author: Adam Grare<EMAIL_ADDRESS>Date:   Wed Nov 1 15:48:12 2017 -0400

    Merge pull request #16373 from kbrock/fetch_collector_timer_ems_fix

    [C&U] fix bug in Targets#capture_vm_targets
    (cherry picked from commit 7bee9ce4f3f01ecd8fb994a176e694b19dbe9422)


[DO-NOT-MERGE] invalid group entries

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #3835   👈

This comment was auto-generated by Graphite.

/test all


Functions missing in production slot after deploying to staging slot

I have a function app with 2 slots - staging and production. Both staging and production slots have some functions.
I have the following YAML task to deploy to staging deployment slot:
- task: AzureFunctionApp@1
  displayName: 'deploy ${{ parameters.name }} function app (staging)'
  inputs:
    appType: 'functionapp'
    azureSubscription: sub
    appName: 'appname'
    Package: 'name.zip'
    deploymentMethod: 'runFromPackage'
    deployToSlotOrASE: true
    slotName: 'staging'
    resourceGroupName: 'rg'

After deployment I see functions inside staging slot but I see NO functions (or ONLY one function with the same name as function app) inside production slot. What am I missing?

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

ID: d955bb83-fa46-22e0-de73-7700ae214af8
Version Independent ID: 1f55672f-711e-24fa-e6bc-edc0ca723eba
Content: Azure Web App task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-rm-web-app.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @azooinmyluggage
Microsoft Alias: atulmal

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

If there is a documentation issue here, we can re-open this issue.


Configure btp-operator priorityClass

Description
BTP-Operator will allow configuring priorityClass in https://github.com/kyma-incubator/sap-btp-service-operator/pull/23, KEB should set it properly to kyma-system for the deployment.
Related issue(s)

/retest


Please add test files to the release archive

Hello
I'm working on a Debian package for rstcheck and I would like to run the test suite at build time. Could you please ship the two test files in the code archive using the MANIFEST.in system?

I updated the build process entirely.
The test_rstcheck.py and test.bash files are added to the tarball but not the wheel.

I changed a lot with the testing stuff.
Tests are all now included in sdist including the examples needed for the system tests to run.
The system tests were rewritten from the test.bash script to pytest tests like the unit tests too.


Questions about Topk REINFORCE

Hello, thanks for sharing!
I have some questions about pi_beta_sample in models.py, you use this function in _select_action_with_TopK_correction, but it seems only sample one item each time?
I am also confused by Equation 6 in the original paper,

as we want to sample a set of top k item, shouldn't it be
? a_{t, i} represent the ith item at time t.
I appreciate any comments for my question since it's been bothering me for a long time

I honestly don't know, I tried to make it like the paper's authors suggested (with one little tweak discussed here https://github.com/awarebayes/RecNN/issues/7). Although, this seems to be more logical. Have you tried to implement this? I can also test it to make sure it's doing the thing. And recently I've found that he algorithm really lacks a TopK normalizing term in prediction, thus it often gets stuck at recommending  only one / several items. Thus, adding some diversity penalty or taking top k recommendations could be an improvement.
If you happen to implement this (optionally in another function), you can submit a commit here.

I don't implement this paper but the definition of action in my research is similar to this paper. More specifically, I also need to get a set of items according to the item score produced by the neural network.
When selecting items, I use
        if deterministic:   # if testing             w_p, w_idx = torch.topk(scores, K)         else:    # training             w_idx = torch.multinomial(scores, K)
then I use torch.gather to get the corresponding log probabilities of each selected item. After that, I add these log probabilities up at each step and multiply the R_t when updating the net.
I find this paper because the example code of policy gradient are all only choose 1 item each step, so I wonder whether my method is correct or not. I don't add any correction factor in my code so my gradient is

a_{t, i} represents a selected item at time step t.
I try to email the authors about my question, but until now, there has been no response.

Yes, they also didnt respond to my letter where I have notified them about my repo. Thanks for sharing, I will work on making the algo more stable and make sure to try this approach

Ok, I will keep watching this repository, please let me know if you have any new thought, and thanks for your sharing too.

In case someone else might come back to this at some point:
I was wondering the same thing and I implemented it in the scenario where per slate only one action can/will be clicked anyway, hence when receiving feedback we know which item that feedback responds to.
I guess the authors did the same thing because this sounds like it:

(2) While the main policy head π θ is trained using only items on
the trajectory with non-zero reward^3 , the behavior policy β θ ′ is
trained using all of the items on the trajectory to avoid introducing
bias in the β estimate.

with footnote 3 saying:

We ignore them in the user state update as users are unlikely to notice them and as a result,
we assume the user state are not influenced by these actions


Expose generateBaseType as part of generator interface

There are cases (like in Robin) where the user does not need the full type with the export piece, exposing this important bit will be VERY useful, this was a major oversight.

Proposed name is "GenerateItemType"


Breaks dashdot / dash. selfhosted app

Domain
https://getdashdot.com/
Link
https://getdashdot.com/docs/install/docker-compose
Version
0.0.0
What browsers are you seeing the problem on?
Microsoft Edge
What OS are you seeing the problem on?
Windows
Anything else?
In my case is a selfhosted private instance of dashdot, that show all white page and the next error in console:
`runtime.37bfc31cd171f5de.esm.js:1
   Uncaught Error: Automatic publicPath is not supported in this browser
at runtime.37bfc31cd171f5de.esm.js:1:2129
at runtime.37bfc31cd171f5de.esm.js:1:2271
at runtime.37bfc31cd171f5de.esm.js:1:3064`

Disabling the extension works

are you sure FastForward is causing this?

Pretty sure, I tested recently only with that extension and happened too.
With that said, might be a incompatibility between different things as Edge, my reverse proxy and more.


ble.sh: init at 0.3.3

Description of changes
Adds bre.shble.sh.
Use this package by executing source ble.sh on bash.
Users can modify their ~/.bashrc to have it loaded on login:
# bashrc

# Add this lines at the top of .bashrc:
[[ $- == *i* ]] && source ble.sh --noattach

# your bashrc settings come here...

# Add this line at the end of .bashrc:
[[ ${BLE_VERSION-} ]] && ble-attach

See https://github.com/akinomyoga/ble.sh#13-set-up-bashrc for more detail.
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@akinomyoga Pinging the creator of bre.sh for his reference. Please let me know if you find something that I missed!

Maybe @SuperSandro2000 also have comments as @SuperSandro2000 seemed to have once attempted to use ble.sh with NixOS.

Oh sorry for typo😅
I didn't noticed of #160857. I wish I had found it; it might sped up my work.
I wonder @axelf4 can take a look at my work? He reviewed the previous work, so he might interested.
The existence of bin/ble.sh  will be the main target of discussion. And also the explicit gawk dependency?

This is a quick note for reviewers:
I made this derivation first for the latest version of ble.sh not v0.3.3, and downgraded it afterwards. v0.3.3 does not require git and .git dir, but I left it on purpose to make it easier to upgrade. Those who want to use the latest version can override it with ease.

Resolved all the discussion above🎉
Thanks all of you guys for your review! Please let me know if I still missing something

Added one commit from discussion at https://github.com/akinomyoga/ble.sh/discussions/212#discussioncomment-3196490.
This PR is still ready to merge!

I don't know what I should do to fix the EditorConfig error...
It is right that I mixed up tabs and spaces, but it is intended. I used here-document and it requires tabs.
Can I just ignore the failing test?

Maybe @SuperSandro2000 also have comments as @SuperSandro2000 seemed to have once attempted to use ble.sh with NixOS.

I am using it right now but I just cloned the repository.
I would recommend to not package the latest stable but either 0.4.0-devel2 or unstable.

great to see work done on ble.sh on nix!
i stopped using nixos for a while, hence couldn't really work on the PR

It does not. It also works with spaces.
Are you sure? I made a quick test for this, and the spaces does not work as intended (EOS with the end caused the error).

Then fetchFromGitHub must be used with leaveDotGit.

I didn't noticed fetchFromGitHub has leaveDotGit! My bad, I should have read the docs more carefully. Thanks, I'll update this

I don't think latest master should be packaged. ble.sh is a frequently updated project. I don't think it's a good idea that I keep deciding which commit is valuable enough to update the nix package as a package maintainer.
Do you think it's a good idea to distribute the "nightly" version, @akinomyoga? Do you have a plan to update the nightly tag?

Do you think it's a good idea to distribute the "nightly" version, @akinomyoga? Do you have a plan to update the nightly tag?

There is no single nightly version, but the nightly version is updated for every push to GitHub. If you mean the commit tagged as nightly by "the nightly version", I wouldn't think that that makes sense because the tag is actually just a dummy for the GitHub release and there is nothing special about that commit. The tag just points to the commit that we have introduced the continuous integration for the nightly builds. Currently, I don't have a plan to recreate the tag at a different commit soon, but I'm not sure for the future.
I am currently thinking about tagging 0.4.0-devel3 in near future, but I currently don't have sufficient time to complete stuff that I would like to finish before the new devel release.

Thank you for clarification! I misunderstood that the "nightly" version just references the commit on 16 Jun 2022. (And thus I was wondering how it is nightly😅)
Then, for now, I'll just pin the current unnamed a450775 version, and I'll update it after you released 0.4.0-devel3.

Oh well, I remembered my current derivation cannot pass make check.
I'll add a new commit soon, and it should be broken because it does not pass a check phase. This seems to be caused by make check being not processed on interactive bash. I tried to force bash to run make check interactive mode but failed.
I hope any of you have some good ideas to fix it.

What!? Ofborg reports no error! On my environment (darwin x86-64) the check phase is broken😭
It's okay if it is just a problem on my environment. It helps if anyone can clone this PR and check it locally.

I have never run the check on macOS. Could you share the error messages for the make check?

This is a result when I execute `make check` on bash prompt:
$ make check
bash make_command.sh check
MACHTYPE: x86_64-apple-darwin21
BLE_VERSION: 0.4.0-devel3+1c7f7a1
BASH_VERSION: 3.2.57(1)-release
/Users/aiotter/repo/github.com/akinomyoga/ble.sh/out/lib/test-main.sh:54 ():
    ble/util/readlink f.txt
    [[ $ret != /* ]] && ret=${PWD%/}/$ret
--- 81028.ret.expect    2022-07-23 22:13:13.000000000 +0900
+++ 81028.ret.result    2022-07-23 22:13:13.000000000 +0900
@@ -1 +1 @@
-/tmp/blesh/501/80939.test/81028.d/ab/cd/ef/file.txt
+/private/tmp/blesh/501/80939.test/81028.d/ab/cd/link1/ef/file.txt

 94.7% [section] ble/main: 18/19 (1 fail, 0 crash, 0 skip)
/Users/aiotter/repo/github.com/akinomyoga/ble.sh/out/lib/test-util.sh:1445 (): ret=$BASHPID
--- 81194.ret.expect    2022-07-23 22:13:16.000000000 +0900
+++ 81194.ret.result    2022-07-23 22:13:16.000000000 +0900
@@ -1,2 +1 @@
--n 81194
-
+81194

/Users/aiotter/repo/github.com/akinomyoga/ble.sh/out/lib/test-util.sh:1452 (): [[ $BASHPID == $ppid ]]
--- 81202.exit.expect   2022-07-23 22:13:16.000000000 +0900
+++ 81202.exit.result   2022-07-23 22:13:16.000000000 +0900
@@ -1 +1 @@
-0
+1

 99.8% [section] ble/util: 1183/1185 (2 fail, 0 crash, 41 skip)
100.0% [section] ble/canvas/trace (relative:confine:measure-bbox): 5/5 (0 fail, 0 crash, 12 skip)
100.0% [section] ble/canvas/trace (cfuncs): 17/17 (0 fail, 0 crash, 1 skip)
100.0% [section] ble/canvas/trace (justify): 2/2 (0 fail, 0 crash, 22 skip)
100.0% [section] ble/canvas/trace-text: 11/11 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/textmap#update: 5/5 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/unicode/GraphemeCluster/c2break: 38/38 (0 fail, 0 crash, 34 skip)
---.-% [section] ble/unicode/GraphemeCluster/c2break (GraphemeBreakTest.txt): 0/0 (0 fail, 0 crash, 3251 skip)
100.0% [section] ble/decode: 33/33 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/edit: 2/2 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/syntax: 22/22 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/complete: 7/7 (0 fail, 0 crash, 0 skip)
ble: The test log was saved to '/Users/aiotter/repo/github.com/akinomyoga/ble.sh/out/cache.d/501/test.20220723.221312.log'.
make: *** [check] Error 1

And this is a log created on building a derivation of this PR (quite long):
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking sources
unpacking source archive /nix/store/fw6nappbiz2p6h6d24aa8hx43bp94sg3-source
source root is source
@nix { "action": "setPhase", "phase": "patchPhase" }
patching sources
@nix { "action": "setPhase", "phase": "configurePhase" }
configuring
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "buildPhase" }
building
build flags: SHELL=/nix/store/8269022kjb67pdc375mjcmnx0a5xhz3r-bash-5.1-p16/bin/bash
mkdir -p out
DEPENDENCIES_PHONY=1 DEPENDENCIES_OUTPUT=out/ble.dep DEPENDENCIES_TARGET=out/ble.sh FULLVER=0.4.0-devel3 \
  /nix/store/lmcw4d36hax9zshc54gv1pqkyvjhak08-gawk-5.1.1/bin/gawk -f make/mwg_pp.awk ble.pp >/dev/null
mkdir -p out/keymap
cp -p keymap/emacs.sh out/keymap/emacs.sh
cp -p keymap/vi.sh out/keymap/vi.sh
cp -p keymap/vi_digraph.sh out/keymap/vi_digraph.sh
cp -p keymap/vi_digraph.txt out/keymap/vi_digraph.txt
cp -p keymap/vi_test.sh out/keymap/vi_test.sh
mkdir -p out/lib
cp -p lib/init-term.sh out/lib/init-term.sh
cp -p lib/init-bind.sh out/lib/init-bind.sh
cp -p lib/init-cmap.sh out/lib/init-cmap.sh
/nix/store/lmcw4d36hax9zshc54gv1pqkyvjhak08-gawk-5.1.1/bin/gawk -f make/mwg_pp.awk lib/init-msys1.sh > out/lib/init-msys1.sh
cp -p lib/core-complete.sh out/lib/core-complete.sh
/nix/store/lmcw4d36hax9zshc54gv1pqkyvjhak08-gawk-5.1.1/bin/gawk -f make/mwg_pp.awk lib/core-syntax.sh > out/lib/core-syntax.sh
cp -p lib/core-test.sh out/lib/core-test.sh
cp -p lib/core-cmdspec.sh out/lib/core-cmdspec.sh
cp -p lib/core-debug.sh out/lib/core-debug.sh
cp -p lib/core-edit.ignoreeof-messages.txt out/lib/core-edit.ignoreeof-messages.txt
cp -p lib/core-decode.emacs-rlfunc.txt out/lib/core-decode.emacs-rlfunc.txt
cp -p lib/core-decode.vi_imap-rlfunc.txt out/lib/core-decode.vi_imap-rlfunc.txt
cp -p lib/core-decode.vi_nmap-rlfunc.txt out/lib/core-decode.vi_nmap-rlfunc.txt
cp -p lib/vim-surround.sh out/lib/vim-surround.sh
cp -p lib/vim-arpeggio.sh out/lib/vim-arpeggio.sh
cp -p lib/vim-airline.sh out/lib/vim-airline.sh
cp -p lib/test-main.sh out/lib/test-main.sh
cp -p lib/test-util.sh out/lib/test-util.sh
/nix/store/lmcw4d36hax9zshc54gv1pqkyvjhak08-gawk-5.1.1/bin/gawk -f make/mwg_pp.awk lib/test-canvas.sh > out/lib/test-canvas.sh
cp -p lib/test-decode.sh out/lib/test-decode.sh
cp -p lib/test-edit.sh out/lib/test-edit.sh
cp -p lib/test-syntax.sh out/lib/test-syntax.sh
cp -p lib/test-complete.sh out/lib/test-complete.sh
mkdir -p out/contrib
mkdir -p out/contrib/airline
mkdir -p out/contrib/config
mkdir -p out/contrib/integration
cp -p contrib/bash-preexec.bash out/contrib/bash-preexec.bash
cp -p contrib/fzf-completion.bash out/contrib/fzf-completion.bash
cp -p contrib/fzf-git.bash out/contrib/fzf-git.bash
cp -p contrib/fzf-initialize.bash out/contrib/fzf-initialize.bash
cp -p contrib/fzf-key-bindings.bash out/contrib/fzf-key-bindings.bash
cp -p contrib/prompt-defer.bash out/contrib/prompt-defer.bash
cp -p contrib/prompt-elapsed.bash out/contrib/prompt-elapsed.bash
cp -p contrib/prompt-git.bash out/contrib/prompt-git.bash
cp -p contrib/prompt-vim-mode.bash out/contrib/prompt-vim-mode.bash
cp -p contrib/airline/atomic.bash out/contrib/airline/atomic.bash
cp -p contrib/airline/dark.bash out/contrib/airline/dark.bash
cp -p contrib/airline/dark_minimal.bash out/contrib/airline/dark_minimal.bash
cp -p contrib/airline/google_dark.bash out/contrib/airline/google_dark.bash
cp -p contrib/airline/google_light.bash out/contrib/airline/google_light.bash
cp -p contrib/airline/landscape.bash out/contrib/airline/landscape.bash
cp -p contrib/airline/light.bash out/contrib/airline/light.bash
cp -p contrib/airline/minimalist.bash out/contrib/airline/minimalist.bash
cp -p contrib/airline/monochrome.bash out/contrib/airline/monochrome.bash
cp -p contrib/airline/solarized.bash out/contrib/airline/solarized.bash
cp -p contrib/airline/solarized_flood.bash out/contrib/airline/solarized_flood.bash
cp -p contrib/airline/term.bash out/contrib/airline/term.bash
cp -p contrib/airline/term_light.bash out/contrib/airline/term_light.bash
cp -p contrib/airline/transparent.bash out/contrib/airline/transparent.bash
cp -p contrib/airline/xtermlight.bash out/contrib/airline/xtermlight.bash
cp -p contrib/config/execmark.bash out/contrib/config/execmark.bash
cp -p contrib/integration/zoxide.bash out/contrib/integration/zoxide.bash
mkdir -p out/doc
cp -p README.md out/doc/README.md
cp -p README-ja_JP.md out/doc/README-ja_JP.md
cp -p LICENSE.md out/doc/LICENSE.md
cp -p docs/CONTRIBUTING.md out/doc/CONTRIBUTING.md
cp -p docs/ChangeLog.md out/doc/ChangeLog.md
cp -p docs/Release.md out/doc/Release.md
mkdir -p out/doc/contrib
cp -p contrib/LICENSE out/doc/contrib/LICENSE
cp -p contrib/README-ja.md out/doc/contrib/README-ja.md
cp -p contrib/README.md out/doc/contrib/README.md
@nix { "action": "setPhase", "phase": "checkPhase" }
running tests
check flags: SHELL=/nix/store/8269022kjb67pdc375mjcmnx0a5xhz3r-bash-5.1-p16/bin/bash VERBOSE=y check
bash make_command.sh check
ble.sh: insane environment: The command(s), `ps', not found.  I have fixed some of the problems found in the log (https://github.com/akinomyoga/ble.sh/commit/fa955c1ac21dab48ea8104819dea0d5d35434a28).
But some of the problems found in building the derivation is not yet solved. They seem to be related to the available locales in the building environment of nix. In particular, A UTF-8 locale needs to be set for make check, but even en_US.utf8 doesn't seem to be available in the building environment as far as I see the log. Could you tell which locales are available in the building environment of nix, or can you add UTF-8 locales as build-time dependencies?

It's weird that the shell invoked by nix develop has en_US.utf8 whereas it's absent while building derivation. I guess it is related to the isolated environment while building. I'll share it if I find out something related.

ble.sh is a frequently updated project.

Not that frequently to be honest. Sometimes it does not receive a commit for 3 weeks which is totally fine.

I don't think it's a good idea that I keep deciding which commit is valuable enough to update the nix package as a package maintainer.

We don't need to decide that. Whenever something more than a documentation change happens we just update it or when we feel like it. Packaging a two year old version and then requiring everyone to maintain their own overlay is just much more work for everyone.

It's weird that the shell invoked by nix develop has en_US.utf8

Not really. nix develop build on top of your current shell which of course has a locales available. If you want to use them in the checkphase add glibcLocales to checkInputs and then add export LC_ALL=en_US.UTF-8 to preCheck.

@akinomyoga Thanks for your fix! I have updated the locale (thanks to @SuperSandro2000), almost all the errors have gone while building the derivation, but there still remains one regarding to readlink.
running tests
check flags: SHELL=/nix/store/8269022kjb67pdc375mjcmnx0a5xhz3r-bash-5.1-p16/bin/bash VERBOSE=y check
bash make_command.sh check
ble.sh: insane environment: The command(s), `ps', not found. Check your environment variable PATH.
ble.sh: modified PATH=/usr/bin:/bin:/usr/sbin:/sbin:$PATH
ble.sh: insane environment: $USER is empty.
ble.sh: modified USER=_nixbld1
ble/term.sh: updating tput cache for TERM=xterm-256color... done
MACHTYPE: x86_64-apple-darwin17.7.0
BLE_VERSION: 0.4.0-devel3+0b95d5d
BASH_VERSION: 5.1.16(1)-release
locale: LANG=en_US.UTF-8 LC_ALL= LC_COLLATE=C
/private/tmp/nix-build-ble.sh.drv-0/source/out/lib/test-main.sh:54:
    ble/util/readlink f.txt
    [[ $ret != /* ]] && ret=${PWD%/}/$ret
--- 39858.ret.expect    2022-07-24 15:32:44.000000000 +0000
+++ 39858.ret.result    2022-07-24 15:32:44.000000000 +0000
@@ -1 +1 @@
-/tmp/blesh/301/39761.test/39858.d/ab/cd/ef/file.txt
+/private/tmp/blesh/301/39761.test/39858.d/ab/cd/ef/file.txt

 94.7% [section] ble/main: 18/19 (1 fail, 0 crash, 0 skip)
/private/tmp/nix-build-ble.sh.drv-0/source/out/lib/test-util.sh: line 685: warning: setlocale: LC_ALL: cannot change locale (en_US.utf8): No such file or directory
/private/tmp/nix-build-ble.sh.drv-0/source/out/lib/test-util.sh: line 827: warning: setlocale: LC_ALL: cannot change locale (en_US.utf8): No such file or directory
100.0% [section] ble/util: 1214/1214 (0 fail, 0 crash, 12 skip)
100.0% [section] ble/canvas/trace (relative:confine:measure-bbox): 5/5 (0 fail, 0 crash, 12 skip)
100.0% [section] ble/canvas/trace (cfuncs): 17/17 (0 fail, 0 crash, 1 skip)
100.0% [section] ble/canvas/trace (justify): 2/2 (0 fail, 0 crash, 22 skip)
100.0% [section] ble/canvas/trace-text: 11/11 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/textmap#update: 5/5 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/unicode/GraphemeCluster/c2break: 72/72 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/unicode/GraphemeCluster/c2break (GraphemeBreakTest.txt): 3251/3251 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/decode: 33/33 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/edit: 2/2 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/syntax: 22/22 (0 fail, 0 crash, 0 skip)
100.0% [section] ble/complete: 7/7 (0 fail, 0 crash, 0 skip)
ble: The test log was saved to '/private/tmp/nix-build-ble.sh.drv-0/source/out/cache.d/301/test.20220724.153244.log(B'.
make: *** [GNUmakefile:206: check] Error 1

This build is on https://github.com/akinomyoga/ble.sh/commit/0b95d5d900b79a63e7f0834da5aa7276b8332a44, and I made it sure that the change introduced https://github.com/akinomyoga/ble.sh/commit/fa955c1ac21dab48ea8104819dea0d5d35434a28 is actually in ble.pp.

@ofborg build blesh

@SuperSandro2000 Can I ask one thing about commit message? You renamed this PR from ble.sh to blesh. The name of the derivation is still ble.sh, but the attribute name is now changed from nixpkgs.ble-sh from nixpkgs.blesh. Should I reword all the prefixes of commit messages from ble.sh:  to blesh:  as well as that of this PR?

I would have squash merged everything anyway.

@ofborg build blesh

can we just open an issue in the upstream nixpkgs (this repository)?

Yes or write on matrix or if you have a fix just open a PR.

@akinomyoga Thank you for your patch and review. I have confirmed that make check exits with no errors🎉
I'll check your review and open another PR for a patch. See you soon!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nix-print-dev-env-command-shows-some-assinments-to-readonly-variables/20916/7


Exposing arangojs in browser bundle.

Fixing #245, by exposing a global arangojs var.

While this would work I think using the library and libraryTarget options is the more conventional solution for Webpack: http://webpack.github.io/docs/configuration.html#output-library
The expose-loader seems to be mostly for exposing libraries a bundled application depends on to the global scope so other code can access it (e.g. jquery plugins or developer tools that aren't part of the bundle).

Yes, that looks better. I can try to make that change, but that will
probably be the end of my webpack endeavors...! I'm sure there's a method
to it, but I don't have the energy to make sense of it right now.
Den sön 19 juni 2016 17:31Alan Plum<EMAIL_ADDRESS>skrev:

While this would work I think using the library and libraryTarget options
is the more conventional solution for Webpack:
http://webpack.github.io/docs/configuration.html#output-library
The expose-loader seems to be mostly for exposing libraries a bundled
application depends on to the global scope so other code can access it
(e.g. jquery plugins or developer tools that aren't part of the bundle).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arangodb/arangojs/pull/247#issuecomment-227003406,
or mute the thread
https://github.com/notifications/unsubscribe/AAH9khria33BVb95jjehbV51uFsCPznFks5qNWC-gaJpZM4I4-2U
.

It's done, using the default setting which just creates an arangojs var. If you'd like something else, let me know.


Add calico-system to ignore list

During a upgrade to AKS 1.20 calcio-system starts to getting used also in AKS.

Ignore the downgrade of the OPA helm chart, it's just for my test. I will update this later


Automated Package Upgrade

:tada: This PR is included in version 25.3.227 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cannot copy keys to list of known hosts

After adding a key (via the dokku commands and manually), the key successfully loads:
-----> Checking Hostkeys Plugin sanity ...
-----> Installing Hostkeys in build environment ...
       No app keys available.
       Adding shared keys.
-----> Done Installing Hostkeys in build environment ...
       Added the following keys:
       github.com ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ==

When I attempt to connect to github, I get the following error:
remote: Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/etc/ssh/ssh_known_hosts).

Did you ever figure this out? I'm having related problems...

You may want to look into #12 for a solution.
Sorry that the response took so long.


SqlFileStream in System.Data.SqlClient 4.6.1

I'm trying to use SqlFileStream class, but unable to do so.
CS0246	The type or namespace name 'SqlFileStream' could not be found (are you missing a using directive or an assembly reference?)
Running on Windows. Project compiled with netcore 2.1. System.Data.SqlClient version is 4.6.1.

Hi @reflash
SqlFileStream was added in .NET Core 3.0 as mentioned in #15 (comment). It's also available in Microsoft.Data.SqlClient previews, you could test with either of the two and that should solve your problem.

Hi @cheenamalhotra
Thank you! I was not aware SqlFileStream wasn't included in earlier versions of netcore, so using Microsoft.Data.SqlClient preview works for me.


refactor: home component refactoring

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description
The Home.jsx component in src/common/home is being refactored to enhance code readability and maintainability. The primary motivation is to improve developer experience by breaking down the component into smaller, more focused components. Folder Structure:
home/
┣ Contributors.jsx
┣ home.css
┣ Home.jsx
┣ HomeBanner.jsx
┣ HomeContent.jsx
┣ HomeContributors.jsx
┣ HomeFeatures.jsx
┣ HomeIdeas.jsx
┣ HomePlays.jsx
┣ HomeSponsors.jsx
┣ HomeTestimonials.jsx
┣ index.js
┗ Sponsors.jsx
Fixes #1384
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
The refactored code has been thoroughly tested using the npm run build script to ensure its compatibility and reliability.
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots or example output

@priyankarpal @Sachin-chaurasiya could you please review this?

@priyankarpal @Sachin-chaurasiya could you please review this?
yeah let me check

contributions

No problem. Looking forward to contribute more! :)

Great work @v4xsh  🙌🏻,
Thanks for the contributions ✨

No problem. Looking forward to contribute more @Sachin-chaurasiya :)

@v4xsh please look into it

Before
After

@atapas please deploy & check


Error trying to use the "initial-work"

Git repo used: https://github.com/redhat-helloworld-msa/aloha
Error in the builder image:
[INFO] ------------------------------------------------------------------------
850	[INFO] BUILD SUCCESS
851	[INFO] ------------------------------------------------------------------------
852	[INFO] Total time: 33.408 s
853	[INFO] Finished at: 2016-03-21T15:59:52+00:00
854	[INFO] Final Memory: 30M/154M
855	[INFO] ------------------------------------------------------------------------
856	[WARNING] The requested profile "openshift" could not be activated because it does not exist.
857	Copying all (fat) JAR artifacts from /opt/app-root/src/source/target directory into /opt/openshift for later deployment...
858	F0321 11:59:53.226834       1 builder.go:204] Error: build error: non-zero (13) exit code from <IP_ADDRESS>:5000/helloworld-msa/vertx-s2i@sha256:2bfb06279a973a3bb63590e46f4ce07ea1ab093560e69d64b4f2708060055f1c

Check that the fat jar file name ends with -fat.jar.

Yes, as far as I see in your pom.xml file you need to add:
 <outputFile>${project.build.directory}/${project.artifactId}-${project.version}-fat.jar</outputFile>

in the maven shade plugin configuration.

Thanks. That solved this issue


Ensure labels creation

Repo labels could be deleted any time.
So could we attempt to create them at startup (like description) ?
(If a label name already exist, the api call will be silently ignored)
edit : just seen I shipped 27b8b65 from #435 is this a problem with our squash enabled strategy ?

@Swizz What happens if label already exists? it's just ignored?

Github API return 422 HTTP Code with
{
  "message": "Validation Failed",
  "errors": [
    {
      "resource": "Label",
      "code": "already_exists",
      "field": "name"
    }
  ],
  "documentation_url": "https://developer.github.com/v3/issues/labels/#create-a-label"
}

I previously experimented, label deletion, but I think this isn't a good compatibility idea.
We want here to create new label and ensure they are still here, deleting labels will remove label on all PR/Issue they were.
Notes :

This way disallows us to change label name or color.
Once a label is put in settings, it will be (re)created at start time, if a label is no longuer needed, we can drop it from settings but it will be still here on the repository without human needs (excepted behavior)

Erf, 422 seem to not fail silently with request.
06-01 11:46 urllib3.connectionpool DEBUG    https://api.github.com:443 "POST /repos/chaosbot/Chaos/labels HTTP/1.1" 422 186
Traceback (most recent call last):
  File "chaos.py", line 120, in <module>
    main()
  File "chaos.py", line 66, in main
    github_api.repos.create_label(api, settings.URN, label, color)
  File "/root/workspace/Chaos/github_api/repos.py", line 46, in create_label
    return api("post", "/repos/{urn}/labels".format(urn=urn), json=data)
  File "/root/workspace/Chaos/github_api/__init__.py", line 93, in __call__
    resp.raise_for_status()
  File "/root/.virtualenvs/chaos/lib/python3.6/site-packages/requests/models.py", line 929, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 422 Client Error: Unprocessable Entity for url: https://api.github.com/repos/chaosbot/Chaos/labels

Yikes, crashed the thing. Where's the automatic rollback script when you need it?
Encountered the same in #453, workaround is simple

Yeah, like I said in my Public Apology #458, I didnt noticed the resp.raise_for_status(), so will need to catch any HTTP code which said "error or something"

@hongaar This could do the trick or I need to move at github_api level ?
log.info("Ensure creation of issue/PR labels")
for label, color in settings.REPO_LABELS.items():
    try:
        github_api.repos.create_label(api, settings.URN, label, color)
    except HTTPError:
        pass

@Swizz It'd place try/except in create_label() function:
try:
    return api("post", "/repos/{urn}/labels".format(urn=urn), json=data)
except HTTPError:
    # Label might already exist
    pass

@Swizz in the function is cleaner  IMHO
P.S. Ive trusted you... an you haven't even tried it out ;(

@phil-r Without resp.raise_for_status() it would been silent x)

maybe change the implementation to replace_label (analogue to SQL replace), which checks if a label exists first, updates it if it does, and creates it if it doesn't?

At start, I didn't want too much logic, but, I guess it could be great.
Only deletion and renaming are against the backward compatibility specs.
So only label creation and color change will be allowed.

Ok hotfix out, chaos back up.


Nyan cat?

First if all, your app is great, but it's missing nyan cat. Please add it.

Thanks :)
I would accept a pull request adding Nyan Cat, but I don't have plans or time to add it myself.

If you are willing to review it I might try, but I have no experience with Android/Java. I jave holidays now though, so I probably will try.

Sure, I actually don't write a lot of Android code myself these days but I'd be happy to look over it!


fix lerna / migrate away from lerna

Feature Request
As you can see in our ci workflow we're having multiple issues with lerna right now.
https://github.com/lerna/lerna/issues/2788
https://github.com/lerna/lerna/issues/1893
Lerna seems to be no longer properly maintained: https://github.com/lerna/lerna/issues/1893
Might make sense to put some effort into fixing them/maintaining a fork as prs at lerna seem to be no longer to be accepted.
Alternatives to consider

We only have two packages right now and might want to consider just publishing things manually(via ci, but without lerna tooling).
https://www.npmjs.com/package/bolt
https://github.com/nrwl/nx

https://github.com/atlassian/changesets seems like another alternative people are migrating to


Refactor Footer component

Title: Enhanced the UI of the footer section
Issue No: #509

Description:

Redesigned the entire footer for improved aesthetics and user experience.
Organized footer content into distinct sections for better readability.
Enhanced styling using Tailwind CSS to ensure consistency with the overall site design.
Updated link styles and hover effects for improved interactivity.

Screenshots/Video:
https://github.com/user-attachments/assets/4f5207c4-5e3d-4518-9aa5-7d10bc562211

Notes:

Please review and merge my pr under gssoc-ext and hacktoberfest.

Conflicts..

Conflicts..

Resolved sir.
Please check now.

@T-Fathima  checks failed, i guess you made some mistake when fixing the conflicts. Kindly recheck


pgvector without Prisma

Hello, I'm trying to set up a project that uses LangChain to (among other things) store document embeddings in a vector column in our postgreSQL database. I'm not in love with the way Prisma handles things; is there a more direct way to get a VectorDataStore that connects to postgres?

Hi @maxfurman!
Understandable, #1436 done by @MaximeThoonsen will introduce an alternative vector store based on TypeORM, which could be more interesting for you..

Hey @maxfurman, FYI the PR has been merged


Cleaned up code

Cleaned up code in lib/phin.js. Updated the documentation in README.md.
Code has been changed to work and look cleaner.

Please fix library callback params to include body as center parameter. :) cb(err, body, res)


Create Wallets Best Practices

File to add notes and links to Industry best Practice for wallet management.

Pull to incorporate Ivan's and my source materials.


Houdini: Publish of Redshift ROP is broken

Running version
3.15.1-nightly.5
Describe the bug
Implementation of Redshift ROP publishing is currently broken in Houdini not only due to the changes in new publisher api.
To Reproduce
Steps to reproduce the behavior:

Create Redshift ROP in Publisher
Try to publish

There are various issues from creator itself to collector and other publish plugins.
Expected behavior
Redshift ROP to publish correctly
Desktop (please complete the following information):

OS: Windows 10
Host: Houdini 19.5

[cuID:863g2pd6q]

Similar issue addressed by this PR #4193


Recovering run that failed while executing LTR_retriever

Used version: TETools git commit bf94dc1
Hi, I ran RepeatModeler on my assembly through the Docker container and it failed while executing LTR_retriever. The LTR_retriever.log file in the LRET_49037.WedFeb260356552020 contained the following error: RepeatMasker is not running properly!. As I have managed to complete a run of RepeatMasker through the Docker container in the past, I believe that a simple rerun of RepeatModeler from that point may solve the issue.
However, I am unable to rerun RepeatModeler using the --recoverDir option, as it tells me that the working directory appears to contain a successful run. Do you have any suggestions on how I could finish this run without starting from scratch?
Below you will find the LTR_retriever.log file, please let me know if you need any additional details from my side.
LTR_retriever.log

I see this quite a few times in the log:

sh: 1: Cannot fork

This error suggests resource exhaustion, in particular running low on memory or hitting the maximum number of processes/threads. This can cause all kinds of issues and false reporting of error conditions. Can you share the output of ulimit -u from inside the container, and do you know how much memory was in use at the time / the memory available? The main log / screen output of the overall RepeatModeler run may also be helpful if you still have it. And finally, what was the full command line you used to run RepeatModeler?

Do you have any suggestions on how I could finish this run without starting from scratch?

You can run the LTRPipeline separately: LTRPipeline -pa 4 genome.fa (where 4 is the number of parallel threads to use). However, it is not easy to combine the results between RECON+RepeatScout and LTRPipeline if you do them separately. I will open a GitHub issue for RepeatModeler to properly support -recoverDir when only the LTR pipeline failed.
Note that the fork error could have impacted the previous rounds, so it may be worth re-doing the run anyway if that turns out to be a serious problem.

I ran the container on a shared server, so the fork error could indeed be caused by a lack of available threads during runtime. Memory was likely not the issue (I assume that there was at least 100G available at the time), nor were any limits placed on the container itself (ulimit -u and ulimit -m both gave unlimited as output when ran inside the container).
The full command with which I ran RepeatModeler was: docker run -it --mount type=bind,source="/mnt/local_scratch/wijfj001/repeatmodeler_dir",target=/work --mount type=bind,source="/lustre/BIF/nobackup/wijfj001/Software/bin/TRF",target=/opt/trf,ro --user "$(id -u):$(id -g)" --workdir "/work" --env "HOME=/work" "rwijfjes/tetools:1.1" RepeatModeler -database Hinc_ctgs -pa 5 -LTRStruct >& repeat_modeler_run.log
Besides the message that no results were obtained after running LTR_retriever, the main log file does not contain any red flags as far as I can tell:
repeat_modeler_run.log
It seems that restarting the run on a system on which I do not have to share threads is the most practical solution for now. Thanks for the help up to this point!

Alright, I managed to complete a run through the container on a different server and the main log file does not seem to point out any issues. However, we had to make the increase the pid_max value of the server to 200,000 to make it happen, as RepeatModeler had generated 57,795 zombie processes by the time it reached the final RepeatClassifier stage. Any idea what could have caused this?
The full command that I used was: docker run -it --mount type=bind,source="/mnt/local_scratch/wijfj001/repeatmodeler_dir",target=/work --mount type=bind,source="/mnt/scratch/wijfj001/Software/bin/TRF",target=/opt/trf,ro --user "$(id -u):$(id -g)" --workdir "/work" --env "HOME=/work" "rwijfjes/tetools:1.1" RepeatModeler -database Hinc_scfs -pa 6 -LTRStruct

as RepeatModeler had generated 57,795 zombie processes by the time it reached the final RepeatClassifier stage. Any idea what could have caused this?

That is definitely wrong, and it's possible it is the cause of or otherwise related to to the cannot fork error you experienced earlier. However, RepeatModeler does wait on its children so I am surprised that you would accumulate any zombies let alone such an absurd number of them. Do you know if the zombies were of RepeatModeler itself or of another script/program that it runs?

The zombie processes were generated while RepeatModeler was running LTR_retriever and seemed to stick around until the complete run had finished. Nevertheless, the main log file reported that LTR_retriever had finished (see attachment). There was unfortunately no log file (besides makeblastdb.log) present in the working directory of the LTRpipeline part of the run, so I could not check what the stderr output was during that time.
repeat_modeler_run.log

That helps a bit. Does running LTRPipeline -pa X genome.fa by itself also create the zombie processes? I have not been able to reproduce it yet but this does help narrow it down.

I tried out the following command: docker run -it --mount type=bind,source="/mnt/local_scratch/wijfj001/repeatmodeler_dir",target=/work --mount type=bind,source="/mnt/scratch/wijfj001/Software/bin/TRF",target=/opt/trf,ro --user "$(id -u):$(id -g)" --workdir "/work" --env "HOME=/work" "rwijfjes/tetools:1.1" LTRPipeline -pa 6 scaffold_sequences_short_ids.fa >& LTRPipeline_scaffold_rerun.out
This command had spawned ~57,000 [bash] <defunct> processes before I killed it. They were all generated between 14:17 and 14:19, which corresponded to the end of the modules 2-5 part of the pipeline (see log):
LTR_retriever_20200316.log
Hope this helps!

Thanks. And I realize I should have asked these right away and forgot to, in case there is a known bug in those versions:

Host operating system and version (from uname -a, lsb_release -a, etc.)
Version of docker and where you got it from (OS package manager or from source)
Whether and how you modified the Dockerfile to build rwijfjes/tetools:1.1

You can disregard the previous message - I was able to reproduce this after all.
When you run a command directly as in docker run ... LTRPipeline ..., child processes whose parents die are reparented to LTRPipeline which does not reap adopted orphans automatically (and it does not usually need to).
One simple change that should work for you is to use bash -c as an intermediate, which does reap adopted children:
docker run -it --mount type=bind,source="/mnt/local_scratch/wijfj001/repeatmodeler_dir",target=/work --mount type=bind,source="/lustre/BIF/nobackup/wijfj001/Software/bin/TRF",target=/opt/trf,ro --user "$(id -u):$(id -g)" --workdir "/work" --env "HOME=/work" "rwijfjes/tetools:1.1" bash -c 'RepeatModeler -database Hinc_ctgs -pa 5 -LTRStruct' >& repeat_modeler_run.log
This is often not a huge problem, but LTR_retriever indirectly spawns a large number of processes so this issue became noticeable.

The --init option did the trick! I ran a docker run ... LTRPipeline ...  command with this flag and it finished without starting a zombie apocalypse. I expect that the full RepeatModeler pipeline will now run properly as well.
Thanks for getting to the bottom of this, Jeb!

I added --init to the docker command in dfam-tetoolsh.sh, so this should not negatively impact other users in the future.
I have also opened Dfam-consortium/RepeatModeler#65 to track the issue where one cannot resume the run if only the LTRPipeline step fails.
Thanks for reporting both issues!


DataVolumeTemplates API regression

/kind bug
When I made the recent change to the DataVolumeTemplate section of the VirtualMachine API in this pr #4261 it resulted in an unexpected regression.
Previously, the DataVolumeTemplate referenced a list of DataVolume objects, which contained a Status.  The new revision removed the Status struct. Even though the status struct is marked as "omitempty", it is still persisted in etcd for each DataVolumeTemplate entry that was created before my changes in #4261
This means upgrades are broken. Existing VMs with the DataVolumeTemplate can not be processed if they contain the Status: {} field.
Example. While the status is never used by the template and the user has no control over those values, it is still persisted in etcd on the template.
  spec:
    dataVolumeTemplates:
    - metadata:
        creationTimestamp: null
        name: test-dv
      spec:
        pvc:
          accessModes:
          - ReadWriteOnce
          resources:
            requests:
              storage: 1Gi
        source:
          blank: {}
      status: {}

Now that Status no longer exists for the DataVolumeTemplate, it causes the VM objects to be rejected by our validation and mutation webhooks.

/release-blocker master

/cc @xpivarc @rmohr
I see two solutions.

Add an empty "no-op" status field back to the DataVolumeTemplateSpec.  It woudl just exist for backwards compatibility

Add logic in the mutation webhook to strip the status from the DataVolumeTemplateSpec when requests come in.

I'm most confident in item 1 resolving the issue.


Fix the inaccurate expected attestation metric

PR Description
Fix the inaccurate expected attestation metric by using ActiveValidatorTracker to get the number of expected attestations.
Fixed Issue(s)

Fixes #3261
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Something's still not right here.  Running with 1024 validators in an interop config I get:
Attestation performance: expected 537, produced 1024, included 1024 (100%), distance 1 / 1.00 / 1, correct target 1024 (100%), correct head 1024 (100%)

I've seen 529, 509 and 537 as the expected validator count which is quite weird...


(447) List bookings

This lists the bookings for a premises on the premises view page. The manage link then goes through to the page where we can mark the booking as arrived:
Screenshot

 List bookings

No drama - my fault for missing it! 🤦♂