libbson:x86-uwp & x64-uwp. Build failed

I want to build mongo-cxx-driver for uwp. When I try this, I get this error:
-- CURRENT_INSTALLED_DIR=D:/Alex/Documents/SourceTree/vcpkg/installed/x86-uwp
-- DOWNLOADS=D:/Alex/Documents/SourceTree/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/Alex/Documents/SourceTree/vcpkg/packages/libbson_x86-uwp
-- CURRENT_BUILDTREES_DIR=D:/Alex/Documents/SourceTree/vcpkg/buildtrees/libbson
-- CURRENT_PORT_DIR=D:/Alex/Documents/SourceTree/vcpkg/ports/libbson/.
-- Using cached D:/Alex/Documents/SourceTree/vcpkg/downloads/libbson-1.5.1.tar.gz
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Configuring x86-uwp-rel
-- Configuring x86-uwp-rel done
-- Configuring x86-uwp-dbg
-- Configuring x86-uwp-dbg done
-- Package x86-uwp-rel
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:14 (message):
    Command failed: D:/Program Files/CMake/bin/cmake.exe;--build;.;--config;Release;--target;install;--;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No;/m
    Working Directory: D:/Alex/Documents/SourceTree/vcpkg/buildtrees/libbson/x86-uwp-rel
    See logs for more information:
    D:\Alex\Documents\SourceTree\vcpkg\buildtrees\libbson\package-x86-uwp-rel-out.log
    D:\Alex\Documents\SourceTree\vcpkg\buildtrees\libbson\package-x86-uwp-rel-err.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_install_cmake.cmake:27 (vcpkg_execute_required_process)
ports/libbson/portfile.cmake:17 (vcpkg_install_cmake)
scripts/ports.cmake:73 (include)

Error: Building package libbson:x86-uwp failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: libbson:x86-uwp
Vcpkg version: 0.0.78-db73e73c562a00228dd8c01ee2a03f03012023fc

Additionally, attach any relevant sections from the log files above.

I have cmake 3.8.0 rc4 x64 version installed. I have VS 2015 Update 3, Windows 10 Pro x64.
P.S. Visual Studio was installed on D:\ drive (D:\Program Files (x86)\Microsoft Visual Studio 14.0)
P.P.S. I have the similar error trying to build x64-uwp version or when I build this lib with integrated cmake 3.8.0 rc2

Thanks for reporting this! It should be fixed in 4765c58, though I haven't tested your original objective (mongo-cxx-driver).


Allow only one scan [iOS]

Awesome library, I'm playing with it a little and got wondering if there is a way to limit the scans to only one for iOS?
Thanks in advance.

Right now it's not possible since VisionKit doesn't have that option.

@dmarcs is this still not possible?

@dmarcs is this still not possible?

According to the documentation of VNDocumentCameraViewController its still not possible. https://developer.apple.com/documentation/visionkit/vndocumentcameraviewcontroller
I was also interested and did a small research 🙂

@sanduluca Thank you.


Multiple Author

Please add features to make a post have more than one author.

I will add this feature soon.

now can set more than one author, see doc


rule update(Non sudo setuid): check user id as well in case user name…

… info is not available
Signed-off-by: Kaizhe Huang<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Fix false positive
Which issue(s) this PR fixes:
#1664

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

rule update(Non sudo setuid): check user id as well in case user name info is not available

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from kaizhe after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: f44e2675f4b1f0afd8e6b6b9b6a37b4c06d1a1b0

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Kaizhe, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [Kaizhe,leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Texts for markings can not be positioned individually.

It would be nice, if texts for markings could be positioned individually as it can be done within Flot diagrams. Furthermore, the font styles should be changeable. Currently, both ist not possible.

In addition, there are doubled text entries on the y-axis according to the shown screenshot.

Currently, both ist not possible.

Meanwhile, I found out that this is not true. Both, font styles and horizontal position of marking texts can be changed. However, I would like to see an additional option to also change the vertical position by a respective y-offset. Furthermore, it would be nice to horizontally center and right-align the text.


Does NNfusoin support tensor core?

I didn't see any NNfusion document talking about tensor core and/or FP16 support.

Sorry, this is not a but. getting mislabled.

Hi, thank you for your attention. The FP16 support is under development (#329).

Thanks for the timely response. Closing issue.


Fixed #18549 -- Fixed heading for inlines with a OneToOneField.

Currently, the admin app uses verbose_name_plural for all inlines. It makes more sense to use verbose_name for OneToOneFields:
https://code.djangoproject.com/ticket/18549

@tim-mccurrach Thanks for this patch :+1:  Welcome aboard :boat:
I simplified tests and moved extra tests to a separate commit.

@tim-mccurrach Thanks for this patch :+1:  Welcome aboard :boat:
I simplified tests and moved extra tests to a separate commit.


add update instructions

preview: http://docs.junolab.org/previews/PR52/man/update/#Update-Instructions-1 :)

@pfitzseb I wrote up the detailed update instructions -- now we can just point to the page when ppl face update issues :)
I'm going to merge this and make a 0.12 release of the documentation.


XML Structural error after upgrade to <IP_ADDRESS>54

Hello Markus,
the script working very good until the previously version, after upgrade to the last one build <IP_ADDRESS>54 I get this error:
XML: Structural error in xml file, 3 open items. -- JSON: The returned JSON does not match the expected structure (Invalid JSON.). (code: PE231)
I use the script with PS64, these are the PRTG parameters: .\PSx64.exe -f="PRTG-VeeamBRStats.ps1" -p="veeamhostname"
Thank you for this great script!

Hi @cipo80 ,
have you upgraded the Veeam Console on the PRTG Probe after the Veeam Backup & Replication Upgrade?

Hi @cipo80 ,
have you upgraded the Veeam Console on the PRTG Probe after the Veeam Backup & Replication Upgrade?

Hi @vMarkusK , I've upgrade the console and now it's working, thank you for the suggestion!


Use display-color-cells to enable 24-bit color terminal support

This change enables the use of nord-theme with emacs' new 24-bit color terminal support.

I see this PR has been here for a while. Are there someone with write access here that could do a review? It would be nice to see if this PR solves #64 as well.

@arcticicestudio

Sorry for the enormous delays, but "real life" keeps me busy almost ~18h a day 😟
Like described in https://github.com/arcticicestudio/nord-emacs/issues/59#issuecomment-447625096, I heavily rely on the community for this port project since I'm not a Emacs user and I resist to learn the language of syntactically arranged brace characters (Lisp) 😝
Anyway, I promise to review, reproduce and test this PR no later than Saturday afternoon. I've seen a lot of forks of this repository only for this fix and many users approved the problems was resolved using the changes from this PR so I guess it'll work we way it should without breaking any other users setup.

@visigoth I can also go ahead and rebase your PR on develop and remove both unrelated code blocks if you like, just let me know so we can get this finally merged :smile:

Since the source branch of this PR is not a feature branch but the default branch visigoth/nord-emacs@develop there's no way for me to update the required PR changes. I'll create a new branch based on this PR and open a new PR to superseed this one.

Superseded by #88.


Minor wording changes , based on OMG Architecture Board review.

Comments received from Nick Stavros.

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


Add authClient

See https://marmelab.com/admin-on-rest/Tutorial.html#adding-a-login-page

Great work!
ASAP I will check this PR on a proof of work i have done: https://github.com/Cambalab/test-feathers-admin


UNKNOWN error installing express

Ubuntu 11.10 running in a vm.
 Error: UNKNOWN, unknown error '../express/bin/express'
npm ERR! Report this entire log at:
npm ERR!     http://github.com/isaacs/npm/issues
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>npm ERR!
npm ERR! System Linux 3.0.0-15-generic
npm ERR! command "node" "/usr/local/bin/npm" "install" "express"
npm ERR! cwd /mnt/hgfs/projects/CFHello
npm ERR! node -v v0.6.7
npm ERR! npm -v 1.1.0-beta-10
npm ERR! path ../express/bin/express
npm ERR! code UNKNOWN
npm ERR! message UNKNOWN, unknown error '../express/bin/express'
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /mnt/hgfs/projects/CFHello/npm-debug.log
npm not ok

I just tried it and got the same result.    Here is the tail end of the log file:
verbose linkStuff   false,
verbose linkStuff   false,
verbose linkStuff   '/mnt/hgfs/projects/CFHello/node_modules/express/node_modules' ]
info linkStuff<EMAIL_ADDRESS>verbose linkBins<EMAIL_ADDRESS>verbose linkMans<EMAIL_ADDRESS>verbose rebuildBundles<EMAIL_ADDRESS>verbose rebuildBundles [ 'formidable' ]
info install<EMAIL_ADDRESS>info postinstall<EMAIL_ADDRESS>verbose installOne cb<EMAIL_ADDRESS>verbose about to build /mnt/hgfs/projects/CFHello/node_modules/express
info build /mnt/hgfs/projects/CFHello/node_modules/express
verbose from cache /mnt/hgfs/projects/CFHello/node_modules/express/package.json
verbose linkStuff [ false, false, false, '/mnt/hgfs/projects/CFHello/node_modules' ]
info linkStuff<EMAIL_ADDRESS>verbose linkBins<EMAIL_ADDRESS>verbose bins linking [ { express: './bin/express' },
verbose bins linking   '/mnt/hgfs/projects/CFHello/node_modules/.bin',
verbose bins linking   false ]
verbose linkMans<EMAIL_ADDRESS>verbose rebuildBundles<EMAIL_ADDRESS>verbose rebuildBundles [ 'connect', 'mime', 'mkdirp', 'qs' ]
ERR! error installing<EMAIL_ADDRESS>info unbuild /mnt/hgfs/projects/CFHello/node_modules/express
verbose from cache /mnt/hgfs/projects/CFHello/node_modules/express/package.json
info preuninstall<EMAIL_ADDRESS>info uninstall<EMAIL_ADDRESS>verbose unbuild<EMAIL_ADDRESS>[ true,
verbose unbuild<EMAIL_ADDRESS> '/mnt/hgfs/projects/CFHello/node_modules',
verbose unbuild<EMAIL_ADDRESS> '/mnt/hgfs/projects/CFHello/node_modules' ]
verbose binRoot [ '/mnt/hgfs/projects/CFHello/node_modules/.bin',
verbose binRoot   { express: './bin/express' } ]
info postuninstall<EMAIL_ADDRESS>verbose installOne cb<EMAIL_ADDRESS>ERR! Error: UNKNOWN, unknown error '../express/bin/express'
ERR! Report this entire log at:
ERR!     http://github.com/isaacs/npm/issues
ERR! or email it to:
ERR<EMAIL_ADDRESS>ERR!
ERR! System Linux 3.0.0-15-generic
ERR! command "node" "/usr/local/bin/npm" "install" "express"
ERR! cwd /mnt/hgfs/projects/CFHello
ERR! node -v v0.6.7
ERR! npm -v 1.1.0-beta-10
ERR! path ../express/bin/express
ERR! code UNKNOWN
ERR! message UNKNOWN, unknown error '../express/bin/express'
verbose exit [ 1, true ]


How can we change datatype of column in mleap dataframe?

There are a few operations in FrameBuilder that you can use to do this.
Suppose you have this leapframe
  val fields = Seq(StructField("test_string", ScalarType.String),
    StructField("test_double", ScalarType.Double))
  val schema: StructType = StructType(fields).get
  val dataset = Seq(
    Row("hello", 42.13),
    Row("there", 13.42)
  )
  val frame = DefaultLeapFrame(schema, dataset)

To change the field type of test_double from Double to String (as an example), you can do
val updatedFrame = frame.withColumn("double_string", "test_double") {
          (r: Double) => r.toString
        }.get
          .drop("test_double").get
updatedFrame.printSchema()

Just to make you aware, mleap can do quite a bit of casting, if that's what you're after
https://github.com/combust/mleap/blob/master/mleap-core/src/main/scala/ml/combust/mleap/core/types/Casting.scala
https://github.com/combust/mleap/blob/master/mleap-core/src/test/scala/ml/combust/mleap/core/types/CastingSpec.scala
Hope this helps!

@mtsol I believe the solution above will achieve what you were looking for, to change the datatype of a column, please re-open the issue if it's not the case. Thank you!


Exact Change Bonfire - Test Cases Impossible

Challenge Exact Change has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2486.0 Safari/537.36 Edge/13.10586.
Please describe how to reproduce this issue, and include links to screenshots if possible.

The fourth and fifth test cases are not possible to obtain. The problem is that if you treat these results as numbers, the trailing zeroes are dropped off, so they fail the test, since the test asserts trailing zeroes. If you use the .toFixed(2) function in order to get the trailing zeroes to show, the result is a string, and since the asserted test cases use numbers, there is no match.

My code:
function checkCashRegister(price, cash, cid) {
  var returnChange = [];
  var denoms = [
    ["Penny", 1],
    ["Nickel", 5],
    ["Dime", 10],
    ["Quarter", 25],
    ["One", 100],
    ["Five", 500],
    ["Ten", 1000],
    ["Twenty", 2000],
    ["Hundred", 10000]
  ];
  var changeNeeded = (cash * 100 - price * 100);

  //multiply CID by 100 for accurate calculations
  for(var i = 0; i < cid.length; i++) {
    cid[i][1] *= 100;
  }

  var totalCID = getTotalCID();
  console.log(totalCID);

  function getTotalCID() {
    var tempCID = 0;
    for(var i = 0; i < cid.length; i++){
      tempCID += cid[i][1];
    }
    return tempCID;
  }

  var CalculateBillsAndCoins = function(amtAvail, amtIncrement, amtNeeded){
    var returnAmt = 0;
    while(amtAvail > 0 && amtNeeded >= amtIncrement) {
      returnAmt += amtIncrement;
      amtAvail -= amtIncrement;
      amtNeeded -= amtIncrement;
    }
    //console.log(returnAmt);
    return returnAmt;
  };

  var thisCurrencyAmount = 0;

  for(var j = denoms.length -1; j >= 0; j--) {
    thisCurrencyAmount = CalculateBillsAndCoins(cid[j][1], denoms[j][1], changeNeeded);
    //console.log(thisCurrencyAmount);
    changeNeeded = (changeNeeded - thisCurrencyAmount);
    if(thisCurrencyAmount > 0) {
      returnChange.push([denoms[j][0], (thisCurrencyAmount / 100)]);
      cid[j][1] = (cid[j][1] - thisCurrencyAmount);
    }
  }
  console.log(cid);

  if(changeNeeded > 0) {
    return "Insufficient Funds";
  }

  if(getTotalCID() <= 0) {
    return "Closed";
  }
  //console.log(cashRemaining);
  return returnChange;
}

//checkCashRegister(19.50, 20.00, [["PENNY", 1.01], ["NICKEL", 2.05], ["DIME", 3.10], ["QUARTER", 4.25], ["ONE", 90.00], ["FIVE", 55.00], ["TEN", 20.00], ["TWENTY", 60.00], ["ONE HUNDRED", 100.00]]);
//checkCashRegister(19.50, 20.00, [["PENNY", 0.01], ["NICKEL", 0], ["DIME", 0], ["QUARTER", 0], ["ONE", 0], ["FIVE", 0], ["TEN", 0], ["TWENTY", 0], ["ONE HUNDRED", 0]]);
checkCashRegister(3.26,

The reason your code does not pass is not the lack of trailing zeroes, it's the capitalization of your denominations. "Quarter" != "QUARTER".

Sorry about that. Thanks.


Contributions showing "unknown" for keybindings

VS Code Insiders 1.7
macOS
I have created this extension. The contributions is a list of keybindings from Atom.
The following keybindings are not resolving. In the contributions list it shows the keybinding as unknown.
{
  "mac": "cmd+shift+{",
  "win": "ctrl+pagedown",
  "linux": "ctrl+pagedown",
  "key": "ctrl+pagedown",
  "command": "workbench.action.nextEditor"
},
{
  "mac": "cmd+shift+}",
  "win": "ctrl+pageup",
  "linux": "ctrl+pageup",
  "key": "ctrl+pageup",
  "command": "workbench.action.previousEditor"
},
{
  "mac": "cmd++",
  "win": "ctrl++",
  "linux": "ctrl++",
  "key": "ctrl++",
  "command": "workbench.action.zoomIn"
}

macOS:

Windows:

My first thought was that the characters {, }, and + needed to be escaped. However, doing so causes an error.

This is also happening with Sublime Text Keymap extension.
Windows:
 {
  "mac": "alt+cmd+[",
  "key": "ctr+shift+[",
  "command": "editor.fold",
   "when": "editorFocus"
},

{
  "mac": "cmd++",
  "key": "ctrl++",
  "command": "workbench.action.zoomIn"
}

{
  "mac": "alt+f12",
  "key": "context_menu",
  "command": "editor.action.showContextMenu",
  "when": "editorTextFocus"
},

+ and { are not valid key codes. Please see https://code.visualstudio.com/docs/customization/keybindings#_accepted-keys

Thanks @alexandrudima. To clarify my understanding - there is no way to set a keybinding for { or +? Sounds like I need to make a feature request for this :)

Created this issue - https://github.com/Microsoft/vscode/issues/14478


Unable to add authentication using web gui

I'm using the latest web gui via MacOS and I can successfully add server but not the authentication details.  They are greyed out and won't allow input.

This is mainly for security reason, as it's not recommanded to use browsers to store plaintext credentials. But also because you need to be authenticated to load the Web UI, so you will still need to enter your credentials before loading the Web UI.
But if you still need this feature, you can inspect the inputs with the Dev Tools and remove the "disabled" attribute.

Ok so I tried to add this to the linuxserver docker version of Transmission (https://github.com/linuxserver/docker-transmission) and placed the files in the /config folder (which is mapped to the container) and set the TRANSMISSION_WEB_HOME to /config/transmissionic/.  The UI loads correctly but it won't allow me to add the server.  Is Transmissionic similar to kettu and combustion-release or am I missing something?

Yes it should work like any other Web UI, if it's not the case I will try to fix this. I've never tested with a Docker installation so maybe i'm missing something.
What's the problem when you add a server? And are the Web UI and RPC accessible from the same host and port? Otherwide there may be CORS problems.

The webui is accessible from the port I have set (in this case 8889) but if I try and access /transmission/rpc/ I get the following:
409: Conflict

Your request had an invalid session-id header.

To fix this, follow these steps:

When reading a response, get its X-Transmission-Session-Id header and remember it
Add the updated header to your outgoing requests
When you get this 409 error message, resend your request with the updated header
This requirement has been added to help prevent CSRF attacks.

X-Transmission-Session-Id: VinWJxkpPMwa8nEAIqC8uUdzWMHtHXDYAGVMG2fvjT1xEB3p

Ok perfect, this normal. Did you get the same error if you access /transmission/rpc without the last slash?
If no, set your RPC path to /transmission/rpc/ in the server details, it should work.

I get the same response with or without the /.

So the problem doesn't come from there, did you get any error in the Web UI?
I don't understand where exactly your problem is, at which step it doesn't work?

Ok so I think I have this working.  It was totally user error, I entered the port in the wrong part of the UI when adding the server :(.  Sorry for the confusion.
For those looking to setup transmissionic with llinuxserver/docker-transmission place the tranmissionic webui folder in the config folder that you mapped to the container.  Name the folder anything you want (I called it transmissionic) and then add - e TRANSMISSION_WEB_HOME = /config/transmissionic/ to your docker command.
Unlike other webui's don't forget to add the server details of your transmission container (correctly) for it to work.

Ok no problem, I'm glad you solved your problem. Thank you for sharing the solution.
So it doesn't work if you leave all inputs empty in the server details? The port and address should be automatically detected.

Looks like I was wrong and you were right (again), just removed my server details and it automatically set the Default to the correct server settings.  I have amended my post above.

Well, I have to admit that this part is not well documented, you couldn't have guessed that. I will try to make this more clear.
Thank you for your feedback!


[contractcourt] Republish close tx on startup

This PR makes us store the closing TX we propose to the database, such that the ChainArbitrator can re-publish them at startup for channels in state waiting close.
This is meant to handle the case were we attempt to close the channel, but go down before the tx has propagated to the network. Now we'll instead store this tx the moment it is ready, such that we can re-publish it.

Comments addressed. PTAL @Roasbeef

Is it possible to fix this issue without adding more persistent state? Maybe by doing a bit more republishing, possibly redundant, like in the sweeper?

Yes, but it needs to store this transaction somewhere. For channels that are closing or closed, it currently isn't stored anywhere. We can't reconstruct this data by ourselves since we need the signature of the remote node along side it.

Yes, the idea behind storing the tx was to handle all possible closes uniformly. Our own force close tx we can re-derive, but not coop closes. It would also allow us in the future to store the remote force close tx if we retrieve it.

So with that said, I think we can move forward on this? Or are we still unclear w.r.t the goals here?

My question about why db is answered.

Is it possible to fix this issue without adding more persistent state? Maybe by doing a bit more republishing, possibly redundant, like in the sweeper?

We can reconstruct our commitment transaction for force closes, but we can't reconstruct the co-op close transaction since we need a new signature from the remote party. So then the question is should we have a unified location to check for closing transaction, or special case the possible closing paths. At the very least, we need a new key in the database to know what type of close was attempted and where we are in that process it seems.

Another open question is if we deem the level of fix implement in this PR acceptable to move forward with, or if we'd like to step back and resolve the other issues as well I outlined in an earlier comment.

OP bump.

Pushed a new version, where the main difference is that we will resend channel reestablish messages for all borked channels with a peer on connect.

The peer that lost data should still re-send their chan reest message upon each connection until the channel has been force closed. Otherwise, it's possible to recognize we lost data, but the remote peer (say it's unupgraded) crashes before it processes our message. In this case, we've marked it as borked locally, so we won't load it back into the link

With the above mentioned change, any channel having state LocalDataLoss we'll resend the chan sync message for when connecting to the peer.

We should write the chan reest message as soon as we initiate a DLP for an eroded client (lost data). This way we can send the message upon their next reconnection, rather than until the channel is fully confirmed.

Exactly as above, for channels where we have recognized remote data loss, we'll resend the chan sync message on reconnect.
Main question left os how to handle the atomic transition when force closing a channel (in chain_arbitrator). See discussion there.

Travis fails with:
        panic: runtime error: invalid memory address or nil pointer dereference
        [signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xac4d07]

        goroutine 1 [running]:
        github.com/lightningnetwork/lnd/contractcourt.(*ChannelArbitrator).stateStep(0xc0000c41e0, 0x5db, 0x0, 0xc0002d1530, 0x3, 0x3, 0x14f7ec0)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/contractcourt/channel_arbitrator.go:806 +0x287
        github.com/lightningnetwork/lnd/contractcourt.(*ChannelArbitrator).advanceState(0xc0000c41e0, 0x5db, 0x0, 0xc0000a7c01, 0x3, 0x3, 0x700000000000000)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/contractcourt/channel_arbitrator.go:1042 +0x19d
        github.com/lightningnetwork/lnd/contractcourt.(*ChannelArbitrator).Start(0xc0000c41e0, 0xc0000181e0, 0xc0009373d8)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/contractcourt/channel_arbitrator.go:387 +0x4a5
        github.com/lightningnetwork/lnd/contractcourt.(*ChainArbitrator).Start(0xc000144360, 0x0, 0x0)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/contractcourt/chain_arbitrator.go:543 +0x12e6
        github.com/lightningnetwork/lnd.(*server).Start.func1()
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/server.go:1215 +0x430
        sync.(*Once).Do(0xc000146488, 0xc000937880)
        	/home/travis/.gimme/versions/go1.12.9.linux.amd64/src/sync/once.go:44 +0xb3
        github.com/lightningnetwork/lnd.(*server).Start(0xc000146480, 0x1396a30, 0xc0000cd040)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/server.go:1151 +0x6c
        github.com/lightningnetwork/lnd.Main(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/lnd.go:616 +0x1c92
        main.main()
        	/home/travis/gopath/src/github.com/lightningnetwork/lnd/cmd/lnd/main.go:14 +0x32

Looks like we'll need to set the ForceCloseChan field of a ChannelArbitrator for pending close channels.

However, neutrino itest fails with:
    --- FAIL: TestLightningNetworkDaemon/data_loss_protection (13.37s)
        lnd_test.go:100: Failed: (data loss protection): exited with error:
            *errors.errorString unable to connect Dave to carol: rpc error: code = Unknown desc = dial tcp <IP_ADDRESS>:19799: connect: connection refused
            /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:8213 (0xd4c4a3)

Is this unrelated, do we have any idea what is causing the failure? It seems distinct from the prior lack of force close related errors due to the inconsistency/race-condition.

Yeah, I saw this flake pretty regularly when working on the initial neutrino itests. It was caused by some connection replacement in the peer/connmgr logic. It went away mostly with the connection manager changes, but maybe it is still not totally robust yet.
I gave it a squash, but still needs #3510 before final swuash an merge

However, neutrino itest fails with:
    --- FAIL: TestLightningNetworkDaemon/data_loss_protection (13.37s)
        lnd_test.go:100: Failed: (data loss protection): exited with error:
            *errors.errorString unable to connect Dave to carol: rpc error: code = Unknown desc = dial tcp <IP_ADDRESS>:19799: connect: connection refused
            /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:8213 (0xd4c4a3)

Is this unrelated, do we have any idea what is causing the failure? It seems distinct from the prior lack of force close related errors due to the inconsistency/race-condition.

Yeah, I saw this flake pretty regularly when working on the initial neutrino itests. It was caused by some connection replacement in the peer/connmgr logic. It went away mostly with the connection manager changes, but maybe it is still not totally robust yet.

Just hit this also on the travis run for #3510  https://travis-ci.org/lightningnetwork/lnd/jobs/589294404, so it is existing.


fix for #562 procedures missing sockets in typeblocks

The previous code would return a list of all procedures and always pick the first one (lexicographically sorted). Now it returns the right procedure by using its original name.
You can play with it at: http://ai-procs-typeblock.appspot.com/
#562 has a good step by step guide on how to reproduce the error.
Please review @jisqyv @halatmit @fturbak

@josmas LGTM -- On Gerrit

@josmas MERGED (commit message edited for Gerrit, so manual closing needed)

thanks, @jisqyv


add regex options

There is no way to use case insensitive regex.
I add a new processingOptions - "regexOptions"

For some very strange reason I was never notified of this PR. My apologies, and many thanks for the submission.
I'm just looking into case sensitivity (#22) and I will fix this problem, but I don't think it's the best approach to allow random regex options. I'll close this PR, but I'll be interested to hear whether you have other regex option applications that don't have to do with case sensitivity?


Refactor os checks

Commenting so that some other PRs are the least recently updated. They are used as test fixtures.


[ZEPPELIN-5638] Add flink job interval check

What is this PR for?
zeppelin-interpreter-flink-yarn produce a lot of logs, caused disk space to run out
What type of PR is it?
Improvement
Todos

[x] - Add validation the value of zeppelin.flink.job.check_interval

Need Help
I think negative value is invalid input, so I make it just return and not to add job
But, the issuer suggest that set default value 1000. Is it better?
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5638

How should this be tested?

CI

Questions:

Does the licenses files need to update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

BTW, if it returns without a progress checker, doesn't it occur infinitive loop?


change the order of the type parameters of core.Parser…

… to work better with -Ypartial-unification.
This fixes #176. I don't intend this to be merged in its current form as there are a bunch of other type constructors whose parameter order should also be changed, i. e. all subtypes of Parser, Parsed and its subtypes and more. But before I make those changes I'd like to find out if there is any support for the idea, hence the PR.

Out of date due to 2.0.0


chore: fix broken import paths in nested docs plugin README

Description
Closes #4365
Nested docs plugin README has outdated import paths.

[X] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[X] Chore (non-breaking change which does not add functionality)

@jacobsfletch updated to the correct paths.

Superseded by #4462


Updated python-hyperlink to 21.0.0

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Updated python-hyperlink to 21.0.0 for 2.0
Change Log

Updated python-hyperlink to 21.0.0 for 2.0

Does this affect the toolchain?

NO
Test Methodology

local build

Waiting to merge until main is unfrozen


OSError: [Errno 98] error while attempting to bind on address: address already in use

Hello,
after FHEM update and PIP update i got same error every 30 seconds. In fhem-log there are the following lines:
2021.08.28 19:15:28 3: Pythonbinding_15733: starting
2021.08.28 19:15:28 3: Pythonbinding_15733: using logfile: ./log/fhempy-2021-08-28.log
2021.08.28 19:15:29 3: Pythonbinding_15733: read: end of file reached while sysread
2021.08.28 19:15:29 3: Pythonbinding_15733: stopped
in fhempy-log:
Traceback (most recent call last):
File "FHEM/bindings/python/bin/fhempy", line 113, in
fpb.run()
File "/opt/fhem/.local/lib/python3.7/site-packages/fhempy/lib/fhem_pythonbinding.py", line 468, in run
pybinding, "<IP_ADDRESS>", port, ping_timeout=None, ping_interval=None
File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
return future.result()
File "/usr/lib/python3.7/asyncio/tasks.py", line 603, in _wrap_awaitable
return (yield from awaitable.await())
File "/opt/fhem/.local/lib/python3.7/site-packages/websockets/legacy/server.py", line 1071, in await_impl
server = await self._create_server()
File "/usr/lib/python3.7/asyncio/base_events.py", line 1378, in create_server
% (sa, err.strerror.lower())) from None
OSError: [Errno 98] error while attempting to bind on address ('<IP_ADDRESS>', 15733): address already in use

Hi,
it seems that another fhempy instance is running already.
Please execute
sudo -u fhem pip3 install fhempy==0.1.103

Restart FHEM afterwards.

Command and restart done but error still occurs

few minutes ago i saw that there was again a new FHEM update for pythonbinding. i did it and after fhem restart it seems that the log-error is gone now.


Also install bc in the installation script

Did you check if the eurosys one liners run without error with this update?

Yes I did run a subset of them on my machine and the execution times are properly reported.

@nvasilakis Update image served by img.pash.ndr.md


Bump lodash to v4.6.1

Hey @natecavanaugh
This just bumps the lodash version, so that my tools don't have to install multiple versions of lodash when using npm 3.

Coverage remained the same at 100.0% when pulling 67d989f120f49a6cf6b32f072d4ac33e11a2fa2d on Robert-Frampton:dev into 04b50cb2dce1b79c2c44fb8d291c02bc70568108 on natecavanaugh:master.

Merged and pushed to Brian. Thanks :)
View total diff: https://github.com/natecavanaugh/lodash-bindright/compare/04b50cb2dc...67d989f120


AppArmor configuration for snap install

I am on Ubuntu 16.04 running Hugo installed with snap.
I have reported an issue on the forum, and I think I have narrowed it down to the AppArmor configuration of the snap application, which might be missing certain rules to allow the user to create sites everywhere.
More about the configuration of my system:
$$ cat /proc/version
Linux version 4.10.0-32-generic (buildd@lcy01-01) (gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4) ) #36~16.04.1-Ubuntu SMP Wed Aug 9 09:19:02 UTC 2017

$$ hugo version
Hugo Static Site Generator v0.26 linux/amd64 BuildDate: 2017-08-07T08:54:34+01:00

$$ which hugo
/snap/bin/hugo

The AppArmor configuration is in file /var/lib/snapd/apparmor/profiles/snap.hugo.hugo and I have pasted it here. I think the relevant lines in that file are 392-404.
As I reported in the forum, the app does not seem to allow me to create new sites anywhere else than in subfolders of the home directory, but I would like to create sites on a different partition (mounted from the fstab).

/cc @anthonyfok

Any news on this side?

@anthonyfok I got hit by this as well. Could you plz give us an update on this?

Please read https://gohugo.io/getting-started/installing/#snap-package and #3143.
For better or for worse, that is Snap's confinement security model by design.
There are ways around it using the so-called "classical confinement" model, but I'm not sure if we want to go there.

I'm wondering why Hugo's Snap doesn't implement Snap's removable-media interface as a slot.
I can't see any severe security related concerns in granting access to this interface.


CLONE - Test Issue Paypal

PHP Version
Test Issue Paypal

Shopware Version
Test Issue Paypal

Expected behaviour
Test Issue Paypal

Actual behaviour
Test Issue Paypal

How to reproduce
Test Issue Paypal

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/PPI-969.
[created from PPI-969, comment 483993]

[public] This is just a test issue. We will close it
[created from PPI-969, comment 484015]


Wdt automate tokenize credentials

discoverDomain changed to automatically replace password attribute values with a property token, and the property written a specified variable file IF the -variable_file argument is included on the command line.
This will work in conjunction with any variable injector directives the user specifies.

the default variable file, model.properties, is zeroed out even if I do not put the -variable_file argument on the command line

These properties for the datasources are really long. Is it hard to shorten them?
Currently:
JDBCSystemResource.Derek-DataSource.JdbcResource.JDBCDriverParams.PasswordEncrypted=
Suggested alternatives:
Derek-DataSource.PasswordEncrypted=
Datasource.Derek-DataSource.PasswordEncrypted=
JDBCSystemResource.Derek-DataSource.PasswordEncrypted=

One algorithm I can think of is to replace every MBean/Name pair, with Name, and remove each singleton MBean, except for the last one. So that the long name above becomes
Derek-DataSource.JDBCDriverParams.PasswordEncrypted - Richard, Derek any more ideas on how to get it shorter with an automated algorithm?


Add pipeline support for nsh commandline

Summary
Add pipeline support for nsh commandline through pipe2() and posix_spawn_file_actions_adddup2()

Pack parameters of nsh_execute() as struct nsh_exec_param_s

Input redirect flags currently is hardcode, passing by arguments maybe better.
Only support redirect to path_name, redirect to fd is needed for pipeline.

Add pipeline support for commandline

And nesting pipeline supported.
Impact
nsh: nsh_parse_command()
Testing
General
# Redirectin
cat < /etc/init.d/rc.sysinit

# Pipe
ls | cat

# Pipe (nested)
ls | cat | cat | cat

# FIFO
mkfifo /dev/testfifo
cat /dev/testfifo &
ls > /dev/testfifo

# CMDPARAMS
set name `uname`
echo $name

# binfmt/sotest
sotest

With dd
Depends: https://github.com/apache/nuttx-apps/pull/2751
# Env
# sim:nsh with `PIPES`, `NSH_PIPELINE_NEST_DEPTH=5`, `FS_HOSTFS` and `SIM_HOSTFS` enabled.

# Read from stdin and write to "of"
mount -t hostfs -o fs=. /data
ls | cat | dd | cat | dd of=/data/test_dd_stdin

#Read from stdin and write to stdout
ls | cat | dd | cat | dd

patch

$ git diff
diff --git a/nshlib/nsh_ddcmd.c b/nshlib/nsh_ddcmd.c
index 601e64408d..812d51c103 100644
--- a/nshlib/nsh_ddcmd.c
+++ b/nshlib/nsh_ddcmd.c
@@ -107,7 +107,9 @@ static int dd_write(FAR struct dd_s *dd)
   written = 0;
   do
     {
+      write(dd->outfd, "<dd ", 4);
       nbytes = write(dd->outfd, buffer, dd->nbytes - written);
+      write(dd->outfd, " dd>", 4);
       if (nbytes < 0)
         {
           FAR struct nsh_vtbl_s *vtbl = dd->vtbl;
diff --git a/nshlib/nsh_fscmds.c b/nshlib/nsh_fscmds.c
index 413ed24634..e6a453ab5b 100644
--- a/nshlib/nsh_fscmds.c
+++ b/nshlib/nsh_fscmds.c
@@ -806,7 +806,9 @@ int cmd_cat(FAR struct nsh_vtbl_s *vtbl, int argc, FAR char **argv)
           if (n == 0)
             break;

+          nsh_write(vtbl, "<cat ", 5);
           nsh_write(vtbl, buf, n);
+          nsh_write(vtbl, " cat>", 5);
         }

       free(buf);

runtime

nsh> ls | cat | cat | dd | dd
<dd <dd <cat <cat /:
 bin/
 data/
 dev/
 etc/
 proc/
 tmp/
 var/
 cat> cat> dd> dd>nsh>
nsh> ls | dd | cat | cat | dd
<dd <cat <cat <dd /:
 bin/
 data/
 dev/
 etc/
 proc/
 tmp/
 var/
 dd> cat cat><cat > cat> dd>nsh>
nsh>

Apache:NuttX CI

[Experimental Bot, please feedback here]
The PR partially meets the NuttX requirements.
Strengths:

Summary provides a clear explanation of the changes and their purpose.
Testing section includes specific examples and mentions CI, which is good.

Areas for improvement:

Impact:

The impact section is incomplete. It only mentions nsh: nsh_parse_command() without explaining the impact of the change on users, builds, hardware, documentation, security, or compatibility.
You need to thoroughly address all impact points, stating "NO" or "YES" with a clear and concise description if applicable. For instance:

Impact on the user:  Will users need to change how they use the nsh command line due to pipeline support?
Impact on build: Will any build system changes be needed for this feature?
Impact on documentation:  Does this change require documentation updates? If yes, have you updated the documentation?

Testing:

While you mention "Selftest" and "Apache:NuttX CI," providing snippets of the actual logs (before and after the change) would significantly strengthen this section.
Detailing the build host and target environments used for testing would also be beneficial. For example:

Build Host: Ubuntu 20.04, GCC 9.4
Target:  SIM,  ARM Cortex-M4

Recommendations:

Expand the Impact section.  Provide a "NO" or "YES" answer and a brief explanation for each impact point.
Include detailed testing logs. Add snippets of the actual testing logs before and after your changes for both the selftest and CI environments.
Specify testing environments. Clearly state the build host operating system, compiler, and target architecture/board used for testing.
Consider future work: Briefly mention your plans to upstream nested pipeline support. This demonstrates ongoing commitment and a roadmap for future improvements.

By addressing these points, your PR will be more comprehensive and easier for the NuttX maintainers to review.

Link https://github.com/apache/nuttx-apps/pull/2469

The commit 11008cbc3021f15b304a7b4ad67b4bc379f28b4f is only for CI testing, will be removed later.

Reverted to the latest reviewed version.

@JianyuWang0623 please fix the conflict.

@JianyuWang0623 please fix the conflict.

@xiaoxiang781216  done.

ping :-) i have restarted CI for verification, is this PR ready for review? :-)

ping :-) i have restarted CI for verification, is this PR ready for review? :-)

Thank you, but this PR is not ready yet.
e.g. NSH will block if pipe buffer full filled.

Link https://github.com/apache/nuttx-apps/pull/2831

Error of CI may be not related
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.
make[1]: *** [uClibc++/Make.defs:28: uClibc++-0.2.5.tar.bz2] Error 60
make[1]: Target 'context' not remade because of errors.
make: *** [tools/Unix.mk:457: libs/libxx/.context] Error 2
make: Target 'all' not remade because of errors.

@lupyuen please take a look, seems like uClibc web page certificate expired. I think we need to move these files to our mirrot at github.com/nuttx

@JianyuWang0623 I have cloned your branch to my repo and enabled all builds. Please check this for the build results: https://github.com/nuttxpr2/nuttx-apps/actions/runs/11744174826

@simbit18 Any idea why we're not using https://cxx.uclibc.org/src/uClibc++-0.2.5.tar.xz? The SSL Cert here is OK, unlike https://git.busybox.net/

@simbit18 Any idea why we're not using https://cxx.uclibc.org/src/uClibc++-0.2.5.tar.xz? The SSL Cert here is OK, unlike https://git.busybox.net/
@lupyuen I have no idea !

Thanks @simbit18 I'm preparing the patch, switching to https://cxx.uclibc.org. The busybox cert just expired today sigh.

@acassis Could you please review the PR? I tested OK on Ubuntu. Thanks!

https://github.com/apache/nuttx/pull/14705

please rebase to the last master which fix the ci error.

please rebase to the last master which fix the ci error.

Rebased to master;
"sh -c" related updated : restore from split args to concat args(copy from the existing code in nsh_execute()）；

Hi @JianyuWang0623 !
This PR broke esp32s3-devkit:smp's ostest just like reported at https://github.com/apache/nuttx/pull/14611#issuecomment-2457181253 (it is the same behavior). Could you please take a look?

Perhaps @hujun260 may have a clue here too...

Hi @JianyuWang0623 !
This PR broke esp32s3-devkit:smp's ostest just like reported at apache/nuttx#14611 (comment) (it is the same behavior). Could you please take a look?

OK, I'll prepare the environment and try to reproduce.

Hi @JianyuWang0623 !
This PR broke esp32s3-devkit:smp's ostest just like reported at apache/nuttx#14611 (comment) (it is the same behavior). Could you please take a look?

@tmedicci OK, I'll prepare the environment and try to reproduce.

@tmedicci
1. A simple test with qemu-armv8a:nsh_smp was passed:
user_main: Exiting
ostest_main: Exiting with status 0
nsh>

2. But having problems when executing esp32s3-devkit:smp.
Ref to @masayuki2009 `s comment in  https://github.com/apache/nuttx/pull/14656 ：
/home/ishikawa/opensource/QEMU/qemu-esp-develop-8.2.0-20240122/build/qemu-system-xtensa -nographic -M esp32 -smp 2 -drive file=/home/ishikawa/qemu_efuse.bin,if=none,format=raw,id=efuse -global driver=nvram.esp32.efuse,property=drive,value=efuse -drive file=./nuttx/nuttx.merged.bin,if=mtd,format=raw

Pulling ghcr.io/apache/nuttx/apache-nuttx-ci-linux for build
Download "qemu-system-xtensa" from https://github.com/espressif/qemu/releases

But don`t know how to generate the "qemu_efuse.bin". @masayuki2009 @tmedicci Do you know where has the quickstart doc? thank you;-)

@tmedicci  Still has errors with esp32s3-devkit:smp

nuttx.merged.bin: no such file - Enable ESP32S3_MERGE_BINS to gen
only 2, 4, 8, 16 MB flash images are supported
machine type does not support if=mtd,bus=0,unit=0

$ ../../qemu-xtensa-softmmu-esp_develop_8.2.0_20240122-x86_64-linux-gnu/bin/qemu-system-xtensa -nographic -M esp32 -smp 2 -drive file=../../qemu_efuse.bin,if=none,format=raw,id=efuse -global driver=nvram.esp32.efuse,property=drive,value=efuse -drive file=./nuttx.merged.bin,if=mtd,format=raw
Adding SPI flash device
qemu-system-xtensa: Error: only 2, 4, 8, 16 MB flash images are supported
qemu-system-xtensa: -drive file=./nuttx.merged.bin,if=mtd,format=raw: machine type does not support if=mtd,bus=0,unit=0

@tmedicci esp32s3-devkit:smp(QEMU) ostest passed
./tools/configure.sh -l esp32s3-devkit:smp
# Enable ESP32S3_MERGE_BINS

qemu-system-xtensa -nographic -machine esp32s3 -drive file=nuttx.merged.bin,if=mtd,format=raw

nsh> ostest
... ...
user_main: Exiting
ostest_main: Exiting with status 0
nsh> uname -a
NuttX 12.7.0 f41b0d00fdc-dirty Nov 12 2024 12:58:43 xtensa esp32s3-devkit
nsh>

@tmedicci esp32s3-devkit:smp(QEMU) ostest passed
./tools/configure.sh -l esp32s3-devkit:smp
# Enable ESP32S3_MERGE_BINS
# Enable ESP32S3_QEMU_IMAGE

qemu-system-xtensa -nographic -machine esp32s3 -drive file=nuttx.merged.bin,if=mtd,format=raw

nsh> ostest
... ...
user_main: Exiting
ostest_main: Exiting with status 0
nsh> uname -a
NuttX 12.7.0 f41b0d00fdc-dirty Nov 12 2024 12:58:43 xtensa esp32s3-devkit
nsh>

I tested with real hardware. Don't you have one at your disposal? I can try on QEMU too later.

I tested with real hardware. Don't you have one at your disposal? I can try on QEMU too later.

ok, will try on hardware later

Just to add some details: I forgot to mention that our internal CI enables CONFIG_DEBUG_ASSERTIONS. This makes total difference to the test. I tried with and without applying the following patch to the esp32s3-devkit:smp.
The patch:
diff --git a/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig b/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
index a72faf9b2d..bf6fbad093 100644
--- a/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
+++ b/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
@@ -21,8 +21,12 @@ CONFIG_ARCH_STACKDUMP=y
 CONFIG_ARCH_XTENSA=y
 CONFIG_BOARD_LOOPSPERMSEC=16717
 CONFIG_BUILTIN=y
+CONFIG_DEBUG_ASSERTIONS=y
+CONFIG_DEBUG_ASSERTIONS_EXPRESSION=y
+CONFIG_DEBUG_FEATURES=y
 CONFIG_DEBUG_FULLOPT=y
 CONFIG_DEBUG_SYMBOLS=y
+CONFIG_ESP32S3_MERGE_BINS=y
 CONFIG_ESP32S3_UART0=y
 CONFIG_FS_PROCFS=y
 CONFIG_HAVE_CXX=y

Applying this patch, the ostest hangs:
user_main: pthread_rwlock test
pthread_rwlock: Initializing rwlock
pthread_exit_thread 30: Exiting

I attached the firmware generated by applying this patch with the following revisions:

nuttx: f41b0d00
nuttx-apps: c052bd83 (the commit from this PR)
nuttx.zip

Just to add some details: I forgot to mention that our internal CI enables CONFIG_DEBUG_ASSERTIONS. This makes total difference to the test. I tried with and without applying the following patch to the esp32s3-devkit:smp.
The patch:
diff --git a/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig b/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
index a72faf9b2d..bf6fbad093 100644
--- a/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
+++ b/boards/xtensa/esp32s3/esp32s3-devkit/configs/smp/defconfig
@@ -21,8 +21,12 @@ CONFIG_ARCH_STACKDUMP=y
 CONFIG_ARCH_XTENSA=y
 CONFIG_BOARD_LOOPSPERMSEC=16717
 CONFIG_BUILTIN=y
+CONFIG_DEBUG_ASSERTIONS=y
+CONFIG_DEBUG_ASSERTIONS_EXPRESSION=y
+CONFIG_DEBUG_FEATURES=y
 CONFIG_DEBUG_FULLOPT=y
 CONFIG_DEBUG_SYMBOLS=y
+CONFIG_ESP32S3_MERGE_BINS=y
 CONFIG_ESP32S3_UART0=y
 CONFIG_FS_PROCFS=y
 CONFIG_HAVE_CXX=y

Applying this patch, the ostest hangs:
user_main: pthread_rwlock test
pthread_rwlock: Initializing rwlock
pthread_exit_thread 30: Exiting

I attached the firmware generated by applying this patch with the following revisions:

nuttx: f41b0d00
nuttx-apps: c052bd8 (the commit from this PR)
nuttx.zip

Hi @JianyuWang0623 , although the issue happens on those references, I've just on the latest master and it seems to be fixed:
Just for the records, just tried with:

nuttx: 3d90079f50eda04821b48a0a401fbffb5ea8a63f
nuttx-apps: 1c7a7f7529475b0d535e2088a9c4e1532c487156

So, don't mind trying to find a solution for outdated code. We will keep testing it in our internal CI. Thanks!


EXCEPTION: ele.addEventListener is not a function

Hi, i have this issue
error_handler.js:47 EXCEPTION: ele.addEventListener is not a function
ErrorHandler.handleError @ error_handler.js:47
IonicErrorHandler.handleError @ ionic-error-handler.js:56
next @ application_ref.js:272
schedulerFn @ async.js:82
SafeSubscriber.__tryOrUnsub @ Subscriber.js:223
SafeSubscriber.next @ Subscriber.js:172
Subscriber._next @ Subscriber.js:125
Subscriber.next @ Subscriber.js:89
Subject.next @ Subject.js:55
EventEmitter.emit @ async.js:74
NgZone.triggerError @ ng_zone.js:278
onHandleError @ ng_zone.js:257
t.handleError @ polyfills.js:3
e.runTask @ polyfills.js:3
invoke @ polyfills.js:3
error_handler.js:52 ORIGINAL STACKTRACE:
ErrorHandler.handleError @ error_handler.js:52
IonicErrorHandler.handleError @ ionic-error-handler.js:56
next @ application_ref.js:272
schedulerFn @ async.js:82
SafeSubscriber.__tryOrUnsub @ Subscriber.js:223
SafeSubscriber.next @ Subscriber.js:172
Subscriber._next @ Subscriber.js:125
Subscriber.next @ Subscriber.js:89
Subject.next @ Subject.js:55
EventEmitter.emit @ async.js:74
NgZone.triggerError @ ng_zone.js:278
onHandleError @ ng_zone.js:257
t.handleError @ polyfills.js:3
e.runTask @ polyfills.js:3
invoke @ polyfills.js:3
error_handler.js:53 TypeError: ele.addEventListener is not a function
at listenEvent (ui-event-manager.js:196)
at new PointerEvents (ui-event-manager.js:48)
at UIEventManager.pointerEvents (ui-event-manager.js:163)
at ImageViewerGesture.PanGesture.listen (drag-gesture.js:46)
at new ImageViewerGesture (image-viewer-gesture.js:27)
at image-viewer.component.js:32
at t.invoke (polyfills.js:3)
at e.run (polyfills.js:3)
at NgZone.runOutsideAngular (ng_zone.js:144)
at ImageViewerComponent.ngOnInit (image-viewer.component.js:32)

same issue here

Could you please tell me what versions of Ionic / Angular you are running so I can have a better look at  the problem?
Thanks

Yeah sorry, ionic 2.2.1, same angular

I guess you mean Ionic 2.0.1 with Angular 2.2.1 ?
I just tried it with a fresh app install and it works well...
Have you tried to update you npm packages ? Not sure what it could be.
I'll set up an updated Plunkr with the last Ionic version, hoping it will help if you have a problem somewhere.

@Riron i still have the same issu
see bellow my package.json
{
"name": "ionic-hello-world",
"author": "Ionic Framework",
"homepage": "http://ionicframework.com/",
"private": true,
"scripts": {
"clean": "ionic-app-scripts clean",
"build": "ionic-app-scripts build",
"ionic:build": "ionic-app-scripts build",
"ionic:serve": "ionic-app-scripts serve"
},
"dependencies": {
"@angular/common": "2.2.1",
"@angular/compiler": "2.2.1",
"@angular/compiler-cli": "2.2.1",
"@angular/core": "2.2.1",
"@angular/forms": "2.2.1",
"@angular/http": "2.2.1",
"@angular/platform-browser": "2.2.1",
"@angular/platform-browser-dynamic": "2.2.1",
"@angular/platform-server": "2.2.1",
"@ionic/storage": "^1.1.7",
"@types/whatwg-fetch": "0.0.33",
"ionic-angular": "2.0.0-rc.4",
"ionic-img-viewer": "^1.2.1",
"ionic-native": "2.2.11",
"ionicons": "3.0.0",
"rxjs": "5.0.0-beta.12",
"whatwg-fetch": "^2.0.2",
"zone.js": "0.6.26"
},
"devDependencies": {
"@ionic/app-scripts": "0.0.47",
"typescript": "2.0.9"
},
"cordovaPlugins": [
"cordova-plugin-whitelist",
"cordova-plugin-console",
"cordova-plugin-statusbar",
"cordova-plugin-device",
"cordova-plugin-splashscreen",
"ionic-plugin-keyboard"
],
"cordovaPlatforms": [],
"description": "ISDzudzar2: An Ionic project"
}

Oh i see that you are using ionic-angular 2.0.0-rc.4. Support for this version of Ionic goes with ionic-img-viewer v1.1.8.
So either use ``ionic-img-viewer 1.1.8, or better, upgrade your Ionic to the latest 2.1.0` 😄

Oh I see, you're right!
But being really dumb (it's my first project in ionic) I'am still not able to make it work... now I have no error messages but it simply don't work... I've imported in the appmodule and it's on the @ngmodule imports.
Then I've, on a template, a  but when I tap it nothing happens, what am I missing?

Thanks Riron, its working perfectly

@josuper cool 👍
@albertodeago your image is broken. You followed the README ? Installation and then Usage ?


Bug with groupBy and database strict mode

Laravel version: 5.4.28
MySQL version: 5.7.16

Description
Have this simple structure:
class Report extends Model
{
    public function invoice()
    {
        return $this->hasOne('App\Invoice');
    }
}

class Invoice extends Model
{
    const STATUS_WAIT_PAYMENT = 42;

    public function scopeUnpaid($Q)
    {
        $Q->where('status_id', self::STATUS_WAIT_PAYMENT);
    }
}

class Bar extends Model
{
    public function reports()
    {
        return $this->hasMany('App\Report');
    }
}

Report::where('sum', '>', 0)->whereHas('invoice', function($Q) {
    $Q->unpaid();
})->select('bar_id')->groupBy('bar_id')->havingRaw('count(bar_id) >= 1')->paginate(20);

This code generates two queries:
select count(*) as aggregate from `reports`
    where `sum` > '0' and exists (select * from `invoices` where `reports`.`id` = `invoices`.`report_id` and `status_id` = '42')
    group by `bar_id` having count(bar_id) >= 1

select `bar_id` from `reports`
    where `sum` > '23' and exists (select * from `invoices` where `reports`.`id` = `invoices`.`report_id` and `status_id` = '25')
    group by `bar_id` having count(bar_id) >= 1 limit 20 offset 0

Why sum > 0 replaced to sum > 23 and status_id = 42 to status_id = 25?
If disable strict_mode in config/database.php and remove ->select('bar_id') all works fine.
P.S. If replace status_id = 42 to unexisting status_id = 9999, then second query will not execute.

A quick comment, your issue does not mention the actual problem. I presume you get an error message. You also mention that you can solve it by "disable strict_mode in config/database.php and remove ->select('bar_id')", which implies that both any of those two actions will cause the error.
Is the problem about pagination with GROUP BY or HAVING? If so it is a known issue; Pagination does not work when use having in query builder

@sisve MySQL version is 5.7.16.
I have no errors. I just look in Debugbar on result of queries and dont understand this:

Why sum > 0 replaced to sum > 23 and status_id = 42 to status_id = 25?

Another query with groupBy and having works fine even with ->paginate(20)
select count(*) as aggregate from `reports`
    where exists (select * from `bars` where `reports`.`bar_id` = `bars`.`id`) and `sum` > '0'
               and exists (select * from `invoices` where `reports`.`id` = `invoices`.`report_id` and `status_id` = '42') group by `id` having sum(`sum`) <= 1500

@sisve

You also mention that you can solve it by "disable strict_mode in config/database.php and remove ->select('bar_id')", which implies that any of those two actions will cause the error.

I have not error if i switch strict mode to false and remove ->select('bar_id'), because in this mode set select to only one field, which will be using in groupBy. So, MySQL just ignore this error.

New values for sum and status_id gets from previous binding.
Before query above i use many withCount, one of which this:
$builder->withCount([
    'revises as revise_success' => function($Q) {
        $Q->where('status_id', 42);
    }
]);

And after reset selects (->select('bar_id')) previous bindings replacing to sum, etc...


Add a row indexer object

So that:
df.row[0] === [df.cols[0][0], df.cols[1][0], ...]

This should be done by returning a Proxy which will allow to overload the indexing operator.

Ensure that you can also write to it so that when I assign to df.row[0] = [1, 2, 3, ...] the 0th value in each column will be replaced with 1 in the case of 0th column, 2 in the case of 1st column etc.


Correct the xp to liquid xp ratio

Describe the your request
the xp to liquid xp ratio is currently 1:1
the standard ratio is actually 1 xp to 20 mb liquid xp
if this is unfixed, it could lead to xp dupes

Could you please explain what is "standard ratio" and where it from?

The ratio is from CoFH, the developers of the thermal suite. in line 176 of  github.com/CoFH/CoFHCore-1.12-Legacy/blob/1.12/src/main/java/cofh/core/init/CoreProps.java it defines the xp to liquid xp ratio as 1 xp to 20 mb.

1.A ratio from another mod is definitely not a "standard" which we should follow.
2.The example you gave is from a 1.12 mod.
3.Most importantly, It doesn't lead to dupe problem. Different implementation of liquid experience.


fix: resolve plotly rendering issue by using ipython html for job pro…

…gress messages
Fixes bug that was preventing plotly rendering to show after the progress bar. Original ipywidgets implementation isn't necessary for basic opening of urls
Screen recording: https://github.com/googleapis/python-bigquery-dataframes/assets/15842009/5225ce05-117a-4808-9ff0-cb2c3aaf3a40

…gress messages
Fixes bug that was preventing plotly rendering to show after the progress bar. Original ipywidgets implementation isn't necessary for basic opening of urls
Screen recording: https://github.com/googleapis/python-bigquery-dataframes/assets/15842009/5225ce05-117a-4808-9ff0-cb2c3aaf3a40

Is this recording after your change? If yes, wondering what was the issue before?

…gress messages
Fixes bug that was preventing plotly rendering to show after the progress bar. Original ipywidgets implementation isn't necessary for basic opening of urls
Screen recording: https://github.com/googleapis/python-bigquery-dataframes/assets/15842009/5225ce05-117a-4808-9ff0-cb2c3aaf3a40

Is this recording after your change? If yes, wondering what was the issue before?

The plotly graph didn't render at all. Running theory is that this was due to ipywidgets interactive nature. There's more explanation in the internal bug report b/297062404.


How to retrieve configured data source username on gateway

It looks like Microsoft changed the Manage connections and gateway web interface so that not even admins can see the existing configured username on an underlying gateway data source. It is masked similarly to the password field. Why in the world this was done I have no clue. There's no option to "unmask".
What is the expected way to retrieve the configured username on an underlying gateway data source? Is there a hidden setting/option in the gateway web interface I'm missing? If I have to decrypt it on the gateway on-prem server are there guidelines/instructions?
I’ve looked at APIs (REST/Powershell), downloaded and installed the gateway PowerShell cmdlets (Microsoft.PowerBI.ServiceContracts.Api.GatewayClusterDatasource), looked through the logs on the actual physical gateway server, scanned github forums, and I still have no idea what Microsoft is thinking. I don’t even know how we would debug login issues if we, for instance, can't tell the DBAs what username is being used.
Thanks.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 92e0c038-1195-d758-ccf4-903b689139ca
Version Independent ID: 181904c0-1ec5-0a1d-4cbf-a5a639bbdfeb
Content: Add or remove a gateway data source - Power BI
Content Source: powerbi-docs/connect-data/service-gateway-data-sources.md
Service: powerbi
Sub-service: powerbi-gateways
GitHub Login: @arthiriyer
Microsoft Alias: arthii

Thanks for your comment, @bparkin1283. This issue has been assigned to the doc author for review.

@bishalgoswami why are you assigning this to someone who has had no activity in this repo in over a year? Is it an administrative function? Thank you.

Thanks for posting – note that we can only answer feedback related to the documentation, and can’t provide direct support or respond to feature requests.
Your best venue for getting an answer or feedback on your question is from the community (https://community.powerbi.com) or from support at https://support.powerbi.com. You can also provide feedback using the Give Product Feedback button at the bottom of the article page.


Cow: Watch bdwebit.com

Cow requests the watch of the watch_keyword bdwebit\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bdwebit\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Update GitHub Actions Workflow to specify a name for every step

I don't recall whether I have used a name for every step in the job. If not, the step name defaults to the text in the run command.

Already done.


hydra-eval-faliures: Print Dependency failures as well as direct failures and update to Python3

Motivation for this change
some packages fail due to non-exposed dependencies and would thus not appear in the list, for example gcj
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[X] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This needs to be rebased. There shouldn't be any merge commit.

Other than that it looks good to me.


[BUG] MySQL - Chat flow JSON data is truncated if more than 65,535 characters

Describe the bug
When using MySQL database, the flowData column in the chat_flow table only supports up to a maximum of 65,535 characters. Values greater than this length are truncated. The column flowData column stores JSON data and when it's truncated, an error in the browser Uncaught SyntaxError: Unterminated string in JSON at position 65535 is thrown because the data is incomplete.
This issue only happens when using MySQL database, SQLite and Postgres works just fine.
To Reproduce

Use MySQL as a database (DATABASE_TYPE=mysql)
Log in to Flowise and create a new Chatflow
Add nodes as much as you can until the size of the Chatflow JSON reaches more than 65,535 characters
Save Chatflow
Go back to the main page
Open the Chatflow you created in step 3

Expected behavior
It should load the Chatflow/canvas.
Screenshots

Setup

Installed with Docker
Flowise Version 1.6.4 (docker.io/flowiseai/flowise:1.6.4)
OS: Linux
Browser: Google Chrome

Additional context

Maximum length of the TEXT data type in MySQL is documented here https://dev.mysql.com/doc/refman/8.3/en/string-type-syntax.html

A variable-length string. M represents the maximum column length in characters. The range of M is 0 to 65,535. The effective maximum length of a VARCHAR is subject to the maximum row size (65,535 bytes, which is shared among all columns) and the character set used

I'm using Postgres database as an alternative solution because Postgres' TEXT data type can store much larger amount of data https://www.postgresql.org/docs/9.6/datatype-character.html.

closing as https://github.com/FlowiseAI/Flowise/pull/2908 merged


Implementation of fetching AE max columns

Coverage increased (+22.5%) to 73.759% when pulling 22b4817f658a16edb8d24eba7eba2b2e44e2b01c on v-kaywon:fetch_maxtypes into 3ab6df91051e6b498ecebbcae38450340db41a05 on Microsoft:AlwaysEncrypted.

Performance measurement for fetching varchar(max) columns with and without AE:
Performance test information:
Driver: PDO_SQLSRV
Table columns: char(5), char(5, deterministic encrypted), char(5, randomize encrypted), varchar(max), varchar(max, deterministic encrypted), varchar(max, randomize encrypted)
Number of rows in table: 2
Number of iterations: 1000
Number of measurements: 10 (each measurement is the sum of all 1000 iterations)
Mean(with AE): 98.71267
Mean(without AE): 85.88191

@meet-bhagdev we have the performance measurement on AE retrieval, can you review?

Coverage increased (+22.5%) to 73.759% when pulling 7a5384650b1972651661a1ee810cd5322d2da2c0 on v-kaywon:fetch_maxtypes into 3ab6df91051e6b498ecebbcae38450340db41a05 on Microsoft:AlwaysEncrypted.

@Hadis-Fard Sure! Let me know where the report is and i will take a look.

@meet-bhagdev It's mixed in the discussion thread, pasting it here:
Performance measurement for fetching varchar(max) columns with and without AE:
Performance test information:
Driver: PDO_SQLSRV
Table columns: char(5), char(5, deterministic encrypted), char(5, randomize encrypted), varchar(max), varchar(max, deterministic encrypted), varchar(max, randomize encrypted)
Number of rows in table: 2
Number of iterations: 1000
Number of measurements: 10 (each measurement is the sum of all 1000 iterations)
Mean(with AE): 98.71267s (SD = 19.78386)
Mean(without AE): 85.88191s (SD = 9.94258)

Coverage decreased (-17.5%) to 33.781% when pulling 0100910be1d4db4cdd7ab49df7dfade3811d771f on v-kaywon:fetch_maxtypes into 3ab6df91051e6b498ecebbcae38450340db41a05 on Microsoft:AlwaysEncrypted.

Coverage increased (+22.6%) to 73.945% when pulling f8816b16de9eea3083ea57d942ec997b0b602381 on v-kaywon:fetch_maxtypes into 3ab6df91051e6b498ecebbcae38450340db41a05 on Microsoft:AlwaysEncrypted.

Coverage increased (+22.6%) to 73.945% when pulling ff671fcde34b402752b9dfe0635bc7e6ad905409 on v-kaywon:fetch_maxtypes into 3ab6df91051e6b498ecebbcae38450340db41a05 on Microsoft:AlwaysEncrypted.


⚠ Bofrost.be has degraded performance

In c60b85c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1840 ms

Resolved: Bofrost.be performance has improved in e68f14e after 46 minutes.


stable-baselines3 v2.0.0 with additional fixes

Closes: https://github.com/conda-forge/stable-baselines3-feedstock/pull/10
Closes: https://github.com/conda-forge/stable-baselines3-feedstock/pull/12
Closes: https://github.com/conda-forge/stable-baselines3-feedstock/pull/13
Closes: https://github.com/conda-forge/stable-baselines3-feedstock/pull/14

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

After https://github.com/conda-forge/stable-baselines3-feedstock/pull/11 is merged, also this PR is ready for review, thanks!

After #11 is merged, also this PR is ready for review, thanks!

fyi @YYYasin19 given your interest in the repo.

@conda-forge-admin, please rerender


Make debugger tooltip customisable

Issue Type
Feature Request
Description
Make debugger tooltip (which appears on hover over variables):

able to be pinned
able to be floating
resizable

@isidorn @weinand Also take a look at #43325 because it may be also included as a part of the current issue.

@Deilan thanks for creating seperate issues (which is the correct thing to do). However let's keep this in one issue and see if it attracts traction of users.

no traction -> no plans to work on this.

I know this one is not very popular. But I think it's important it terms of debugging user experience.

Other missing features from VS are adding/removing/editing the expressions in the DataTip:
https://docs.microsoft.com/en-us/visualstudio/debugger/view-data-values-in-data-tips-in-the-code-editor

I'd say the resizable feature is the most important in terms of debugging UX, even compared to the other 2.

I agree that making the debug hover resizable would be great, for that assigning this to September just to investigate what is needed to be done here.

Actually decided to go down this path https://github.com/microsoft/vscode/issues/106865
So please try it out with VS Code insiders and provide feedback.
For now we will not make resizing the deubg hover a manual process since imho that would not be a nice experience, ideally this automatic resizing which we do know should improve the expreience.

Since we did improvement in #106865 there are no more plans for change here.


fix bugs:issue32

fix bug https://github.com/dotnetcore/NPOI/issues/32
When Create xlsx workbook,delete temp file logic has some bugs.

Merged, thank you.


introduced WCAGLinkDataIndexer and wcag-taxonomy-provider

Description of changes
Note: Still a work in progress for feature 1515731

Added WCAGLinkDataIndexer to create a sorted (ascending) array of WCAG tags (wcag-link-data.ts) and a dictionary to map WCAG tags to their indices in the sorted array, for later use when processing rules and results in a future PR
Added wcag-taxonomy-provider to return the properties of the WCAG taxonomy and all taxa associated with it. Taxa properties are based on WCAG link data (wcag-link-data.ts)
Added corresponding unit tests (wcag-link-data-indexer.test.ts and wcag-taxonomy-provider.test.ts)
Modified sarif-converter-21 to use WCAGLinkDataIndexer and getWcagTaxonomy to populate taxonomies in sarif output

Pull request checklist

[ ] PR title respects Conventional Commits (starts with fix:, feat:, etc, and is suitable for user-facing release notes) N/A this is part of a feature
[x] PR contains no breaking changes, OR description of both PR and final merge commit starts with BREAKING CHANGE:
[x] (if applicable) Implements WI: 1548215
[x] Added relevant unit tests for your changes
[x] Ran yarn precheckin
[x] Verified code coverage for the changes made

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Patcher not working

Sir I'm on Android and when I try to patch a pokemon game moon black 2 whenever I try to patch game it shows it is trying to patch but it didn't patch the game I let it sit for on hour to patch but it didn't pls help me with this issue

What patch exactly you're trying to apply? What browser you're using and what version? Have you tried different browsers?
Unfortunately unless you can provide errors logs we're unlikely to figure out what's wrong.

Closing, not enough details to reproduce.


added asset overwrite, enabled entries, removed module dependency wit…

CS-42122

Enabled entries overwrite
Fixed asset duplication (Phase 1)
Remove module dependency of overwrite

Hi @shafeeqd959,
It seems some snyk issue is failing in GH Workflow of this PR.

Hi @shafeeqd959, It seems some snyk issue is failing in GH Workflow of this PR.

Fixed now, Thanks


Chore ember upgrade

Coverage remained the same at 100.0% when pulling 88cf89e4d9df145c23c55cad7dfc50fe56f58cc5 on chore-ember-upgrade into 48f63a9cbae4beb38dd622dd2e2af8d7e86b27ed on develop.


quick reply with content type email or phone returns error

hi im trying to use the quick reply for email or phone in the way below
convo.addQuestion({
"text": "Here is a quick reply!",
quick_replies: [
{
content_type: 'user_email',
},
{
content_type: 'user_phone_number',
},
],
}
and getting this error
nfo: An error occurred while sending a message:  { message: '(#100) The parameter message[quick_replies][0][content_type] is required',
type: 'OAuthException',
code: 100,
fbtrace_id: 'DqOcSLT3A8I' }
if i use content types like text and location they are working without any issue

anyone ? its more a bug then help wanted (:

@peterswimm  i looked deeper into it and found this piece of code
if (item.content_type === 'text' || !item.content_type) {
quick_reply = {
content_type: 'text',
title: titleLimit(item.title),
payload: item.payload,
image_url: item.image_url,
};
} else if (item.content_type === 'location') {
quick_reply = {
content_type: 'location'
};
} else {
// Future quick replies types
}
a really easy fix would be to add the 2 types of quick replies under the else clause
content_type: 'user_email',
content_type: 'user_phone_number',

@gilLionKing feel free to submit a PR request if you feel that the current code could be refined.

Here's the fix #1301


Feature gatekeeper lib 0.7.5

Close this and use: https://github.com/XenitAB/terraform-modules/pull/536 instead.


Slow documentation window

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config

Description
First of all. Great job on the plugin; it works well and fast.
One thing though that is slow is the documentation window. It works slowly both on my config and on configs like LunarVim. Sometimes I have to wait more than 3 seconds for it to show. And sometimes it works instantaneously. And it doesn't seem to matter how much data the documentation window is going to load
(The example below is not that bad, but sometimes it gets much slower)

Steps to reproduce
Use it with a large typescript project
Expected behavior
Show it in 1 second max, not 3
Actual behavior
It is slow for large typescript projects. Even though shift+k works instantaneously
Additional context
No response

I have a similar issue with a Rust project.
If I cargo add env_logger any completion beginning with e will lag for a couple of seconds as it takes a very long time to load the documentation window for env_logger. It completely freezes nvim while it loads the documentation.
I'd hope to see this load hiccup removed/the result cached so it only happens once but I'd also expect to be able to smoothly type the next letter without any hiccups. Seems odd that the documentation load is blocking the rest of the UI thread.

Are you using Noice by any chance?
I noticed a similar problem and turned out it was caused by displaying snippets. A snippet typically has dollar signs in them, that are incorrectly identified as latex by the markdown parser.
There was an issue with the noice highlighter where I didn't cache the highlight query, which means that after every garbage collect (every 2 seconds roughly), the full latex query had to be reloaded. Which is terribly slow.
I pushed a fix for this to noice yesterday, so only the very first invocation would be slower.
You could also just uninstall the latex treesitter parser.
The same issue might have causes slow-downs for other treesitter grammars as well.

Are you using Noice by any chance? [...] caused by displaying snippets [...]
...
You could also just uninstall the latex treesitter parser.

I have the same issue and use neither noice nor treesitter. I have not tested this with snippets, but it happens with clangd and zls too. The problem here is not that the docs take a while to load. The problem is that nvim is completely frozen for the few seconds the docs are loading.
For those who have found this discussion and want a workaround (from the docs):
local cmp = require('cmp')
cmp.setup{
  window = { documentation = cmp.config.disable },
  ... your other stuff here ...


Cleanup of sample scripts and operator extension

After playing with this for a bit, I decided to not "extend" call nodes via operator nodes for now. Meanwhile, @mpollmeier has fixed the ODB bug, which lead to us using implicit conversions for this purpose.
Cleaned up the remainder of the operator classes I brought in yesterday to work like ArrayAccess
Adapted sample queries

Concerning empty classes: they are not entirely empty, they do have that call field. The reason I introduce them now already is that it is inconsistent to have some of the classes wrap a type from nodes, while others bring in their own node. Also, methods in these classes will follow very soon. I can hold off on merging this one though until these methods are present.


Fix: Update Version Constraints

what

Provider bump

why

Prevent stale .terraform directories from preventing planning this module

references

https://github.com/cloudposse/terraform-aws-kms-key/pull/30

/test all


Bug fix/display exit survey

Could not push to branch protected by review so made a new pull request.

Fixed incorrect syntax on imports for SurveyWrapper.jsx and ExampleExitSurvey.jsx
Added survey-react to client dependencies
Tested survey on simple experiment batch
Fixed Merge Conflicts with Docker Update

#13

@Alan-Qiao go ahead and merge!

The best way to solve this conflict is probably to merge main into your branch, and then you should be able to merge.

Did not pass UI test because page displays request for microphone/camera permissions instead of jitsu screen. This merge does not deal with any of that.


Error when using the precompiled 'deps.zip' from this repo

Hello and thanks for this amazing piece of work,
When trying to use your deps.zip in an AWS lambda deployment (with the same model you suggest in your tutorial in an S3 bucket), Lambda execution fails with the following error:
{
    WARNING: "This caffe2 python run does not have GPU support. Will run in CPU only mode."
    WARNING: "Debug message: No module named caffe2_pybind11_state_gpu"
    ERROR: "The graph does not have an ir_version set properly"
}

I assume this may be an issue with the model?
Any idea?
Thanks!

I've managed to pass this by essentially using v1.4 (latest, instead of 0.1 as in your write-up) of ONNX, but in this case the results of the pretrained network are empty:
Outputs()
when it should be:
Outputs(x,y,z)
Once again, any help would be greatly appreciated :)
Thanks!


Throw validation errors in custom(){}

I have a couple of functions that determine if field is required, ideally one of them would just throw an error, instead of having to determine if anyone of those triggered.

This doesn't seem worth the extra code since it's just as easy to return the error details.

Like validate the validators and concatenate them 0.o.


[ListField] Updating state on values array length change.

ListField doesn't rerender when incomming values array length is changed.

Coverage increased (+0.0004%) to 99.89% when pulling 74b05a5abae8462266f4eb18355d3f85853ae844 on ksobolewski-netins:v2 into 431e42abbf973523f1f44568af8eed31353281d7 on mprzodala:v2.


Incompatible with Django 3.2 and python-binary-memcached<=0.29

has_key() returns True for all keys.

Presumably something to do with https://github.com/django/django/commit/bb64b99b78a579cb2f6178011a4cf9366e634438 -- adding _missing_key = None makes it work correctly.
Was testing with python-binary-memcached==0.29, will try newer version and see if that supports default values in get()

0.30 changelog says "Add ability to return default value on get but breaking get’s API" so maybe that will fix it with https://github.com/jaysonsantos/python-binary-memcached/commit/470b5dabf507e6c7a9d5485e46648840c303cd97

Confirmed it works with python-binary-memcached>=0.30 -- not sure if something can be added to the dependency info to ensure people don't get bitten by the django>=3.2 + python-binary-memcached<=0.29 combo?

Hey there @DrMeers that is a hard one to tackle as people install this package and freeze the whole venv and do not bump the required library when needed.
I'd suggest using something like poetry that would know how to deal properly with this locking.
So, if you get a the current version of django-bmemcached and python-binary-memcached, does it work?

So, if you get a the current version of django-bmemcached and python-binary-memcached, does it work?

Yep, confirmed above it works fine for python-binary-memcached>=0.30


Refactor for cards

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

58 of 58 (100.0%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
888

Relevant Lines:
888

💛  - Coveralls


Remove tests that fail with Swift 3 expression type checking support …

…removed.
Most of these are tested with 4.x as well, and remain. I've wholesale
deleted the ones that are only tested with 3.x, but could revert those
parts of XFAIL them instead if there is a reason to do so. My thoughts
are that it's easy to replace these when we get hashes to text 4.x with.

@swift-ci Please test

@clackary I can certainly XFAIL these instead, but they are definitely not going to build with 4.2 with the current hashes.
Note that SwifterSwift has two complete entries here, and I've deleted just the one that was testing 3.0.

@clackary Let me know if you think it's worth XFAILing. If I do that I will need one or more JIRAs to reference for the tests that are XFAILed.

@rudkx I think it is worth XFAILing. I think there's an expectation to at least notify maintainers and give them a chance to update it before a project removed.

@clackary Okay, updated to XFAIL instead (except for the project that had two entries, where I've now removed the 3.0 one).

@swift-ci Please test

The failures are all UPASSes for the tests that have been XFAILed but which do not fail without my compiler PR (https://github.com/apple/swift/pull/17691).

Please test with following pull request:
https://github.com/apple/swift/pull/17691
@swift-ci Please test

Most of these were covered by https://github.com/apple/swift-source-compat-suite/pull/210 and other PRs. I'll open a new PR for anything else that is still blocking my compiler PR.


Update stable version in repo from 1.0.0 to 1.0.2

Think about update stable version to 1.0.2
Version: stable=1.0.0 (60 days old) => latest=1.0.2 (30 days old)
Installs: stable=1602 (43.04%), latest=561 (15.07%), total=3722
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.0.2 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

Is there any reason to delay adding this adapter to stable any longer?
Please drop a commit if you know any reason to refrain from adding.
If there is no response until 27.9.2024 we could decide to update anyway

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.0.2 but the current stable version is already 1.0.2.
So this issue should be closed.
@mcm1957 for evidence


HOLD and NEXT piece png images currently have white borders + slightly off centered

To add to this, it seems the next piece is not loading anymore.

Likely what you can do is set the size of the image view smaller or make the images of the pieces smaller when being loaded in.

Hold and Preview images are now inaccurate to current tetrimino visuals.


convert to python3

Need convert input.py to py3k

I've written a similar request to node and have actually generated the python3 compatibility for them.  They indicated to me that it isn't worth it because gyp currently doesn't support python3.  I believe they would be willing to support python3 if  gyp did.  That would mean tools like nexe would be able to work on arch.
If you want I can do the conversion and give you a merge request.  I have useful vim tools ready to go.

it would be nice if this happened soon since python2.7 EOL is soon.


Update BasePage and BaseAction templates to use simpler inheritance

Made possible by https://github.com/luckyframework/web/issues/56

Also closed by https://github.com/luckyframework/cli/commit/c44b6e48602752cf0fa39eab4d6e2e0e0fe8fa60


add remote app deploy demo

love to see a live demonstration of this using docker.
I'd only question the title. There is already a 'ivy-deploy-app' sample ... which is also showing de fact 'remote' deployment isn't it?
So what I'd expect is a title that puts emphasize on the 'encrypted-http' deployment approach shown here. Can we show that in the 'directory-name' and README.md?

No the other demo is not remote deploying. In this demo the app is directly mounted into the engine deploy directory.


Include user guide under Help menu

Include user guide under Help menu so users know how to get started (e.g. step 1 Upload, step 2 wait for workflow to finish, step 3 goto collection view and click analyze, etc)

Currently working on a 'getting started with sample app guide'. This could go to the Help menu as well as on the project page as well.

I verified the Help link is set to the IMPLEMENTATION GUIDE in the latest version.


#42 - enums fixed with SwitchCaseSemicolonToColonFixer update

Okay, so I updated ECS and PHP-CS_Fixer underneath. Enums are working properly right now.
The problem to discuss is versioning. I bumped development environment Docker to 8.1 and everything is okay. But if anyone would like to run tests with PHP 8.0, then it'd break because it'd not understand enum syntax. Generally Codestyle as a package should work with PHP 8.0 projects (the problem is for internal testing only).
Should we exclude this one particular test from test cases for PHP < 8.1? Or maybe we should just bump required PHP version to 8.1? I'm open to discussion.
It should close #42.

First of all, we should use @require annotations, and "tag" tests.
https://phpunit.readthedocs.io/en/9.5/incomplete-and-skipped-tests.html#skipping-tests-using-requires
Next, I think we should use matrix strategy and run test on several PHP versions/systems in GitHub Actions:
https://github.com/marketplace/actions/setup-php-action#matrix-setup
The last thing might be refactor local test environment and run test on several PHP versions also.
We can achieve this with makefile and docker - This is my first thought.


Finalize querying mechanism with all service tests passing

This pull request fixes the querying mechanism and gets all service tests passing which should finalize the first version of this adapter. Closes #1

Booya! Thanks @daffl for kicking that over the finish line.


Memory leak?

What is the correct way to cleanup an object? I've pasted what I've tried below but memory usage slowly creeps up over a session of opening and closing windows.
window = Window(parent=self.region.root, width=320, height=240)
window.remove_all_children()
window.clear_parent()

Is this issue related to your PR?

Yeah, my PR fixed this issue. For anyone else who may be wondering how to appropriately clean up LUI objects, call clear_parent(), remove_all_children(), and unbind_all() on the object. You may also have to call unbind_all() on any child that has an event bound to a method belonging to the object.


[BUG] Load or chunk "results" return an array of array insted of array

Package version: "maatwebsite/excel": "~2.1.0",
Laravel version: 5.2
Hi,
I have this code:
Excel::filter('chunk')->load($file)->chunk(100, function($results) use (&$count) {
    foreach ($results as $row) {
    }
});

I saw that the $row variable is still an array of all rows, instead of being an excel $row.
I get the same behaviour If I do not use chunk function but only load function.

I'm not sure what the problem is? This is how the package works.

$row variable should be the excel row, not ALL the rows of the file.

I think your having a wrong assumption then. The first loop is always the sheets, the second the rows.


Fail to make -j4 on Mac, all dependencies are installed already

You may see the output below when i make -j4.

build/common/tblob_op_registry.d
In file included from src/common/tblob_op_registry.cc:6:
In file included from include/mxnet/ndarray.h:17:
In file included from include/mxnet/./base.h:13:
In file included from ./mshadow/mshadow/tensor.h:16:
./mshadow/mshadow/./base.h:120:14: fatal error: 'cblas.h' file not found
In file included from src/c_api/c_predict_api.cc:9:
In file included from include/mxnet/symbolic.h:17:
In file included from include/mxnet/./base.h:13:
In file included from ./mshadow/mshadow/tensor.h:16:
./mshadow/mshadow/./base.h:120:14: fatal error: 'cblas.h' file not found
In file included from src/c_api/c_api.cc:11:
In file included from include/mxnet/base.h:13:
In file included from ./mshadow/mshadow/tensor.h:16:
./mshadow/mshadow/./base.h:120:14: fatal error: 'cblas.h' file not found
#include <cblas.h>
^
#include <cblas.h>
^
#include <cblas.h>
^
In file included from src/resource.cc:8:
In file included from include/mxnet/base.h:13:
In file included from ./mshadow/mshadow/tensor.h:16:
./mshadow/mshadow/./base.h:120:14: fatal error: 'cblas.h' file not found
#include <cblas.h>
^
1 error generated.
1 error generated.
1 error generated.
1 error generated.
make: *** [build/common/tblob_op_registry.o] Error 1
make: *** Waiting for unfinished jobs....
make: *** [build/c_api/c_predict_api.o] Error 1
make: *** [build/resource.o] Error 1
make: *** [build/c_api/c_api.o] Error 1

./mshadow/mshadow/./base.h:120:14: fatal error: 'cblas.h' file not found
You didn't have blas or didn't set the path to it in config.mk

Closing for now, please feel free to re-open, seems to be problem of not installing BLAS

Met exactly the same issue in EI Capitan.
I installed openblas thru brew, and changed USE_BLAS = openblas. Also tried the following, but no luck. Any way to troubleshoot?
export LIBRARY_PATH=/usr/local/Cellar/openblas/0.2.15/include/


Deploy app to both radix clusters

A lot of code repetition going on here, created #2079 to improve not only this, but the whole pipeline

Ask radix-team to external review (change clusters on dev)
Ask radix team details on how the redirect is done (prevent issues with CNAME, SSL etcs)

Radix would talk with HCL on the CName and SSL sertificates regarding different clusters


Can't use SetSDKContext() more than once

I have a program that needs to interact with multiple chains. I've given each chain-specific sub-module its own ChainClient & ChainClientConfig to work with, but any time I want to interact with two chains or more (either via query or tx), the second one to get executed uses the Bech32 prefix of the module that was executed first.
The calls to the SDK config did not panic, so it can't be sealed. AFAICS the calls are made each time AccAddress.String() or ValAddress.String() are called, so I tried the following:
done := ChainClient.SetSDKContext()
// Get string representation of the account in the keyring and cache it.
accAddr := key.String()
done()

// Use the 'accAddr' variable from here on instead of calling the String() method each time.

But that didn't work either. I'm not sure how much further down this rabbit hole goes, so maybe someone can enlighten me here.
How To Reproduce
Create two accounts in your local keyring, e.g. one for Cosmos and one for Osmosis.
$ lens keys add cosmos_acc --chain cosmoshub
$ lens keys add osmo_acc --chain osmosis

Then, run this program.
package main

import (
	"fmt"
	"log"
	"os"
	"os/signal"
	"path"
	"sync"
	"syscall"
	"time"

	lens "github.com/strangelove-ventures/lens/client"
	"go.uber.org/zap"
	"go.uber.org/zap/zapcore"
)

func main() {
	encConfig := zapcore.EncoderConfig{
		MessageKey:	"msg",
		LevelKey:	"level",
		TimeKey:	"time",
		NameKey:	"logger",
		CallerKey:	"caller",
		FunctionKey:	"func",
		StacktraceKey:	"trace",
		LineEnding:	zapcore.DefaultLineEnding,
		EncodeLevel:	zapcore.LowercaseLevelEncoder,
		EncodeTime:	zapcore.ISO8601TimeEncoder,
		EncodeDuration:	zapcore.SecondsDurationEncoder,
		EncodeCaller:	zapcore.ShortCallerEncoder,
	}
	logger := zap.New(
		zapcore.NewCore(zapcore.NewJSONEncoder(encConfig), zapcore.AddSync(os.Stdout), zapcore.InfoLevel),
		zap.AddStacktrace(zapcore.PanicLevel),
	)

	homeDir := path.Join(os.Getenv("HOME"), ".lens")
	keyDir := path.Join(homeDir, "keys")

	cosmosCCC := &lens.ChainClientConfig{
		Key:		"cosmos_acc",
		ChainID:	"cosmoshub-4",
		RPCAddr:	"https://cosmos-rpc.polkachu.com:443",
		AccountPrefix:	"cosmos",
		KeyringBackend:	"test",
		GasAdjustment:	1.2,
		GasPrices:	"0.025uatom",
		KeyDirectory:	keyDir,
		Debug:		false,
		Timeout:	"20s",
		OutputFormat:	"plain",
		SignModeStr:	"direct",
		Modules:	lens.ModuleBasics,
	}
	cosmosClient, err := lens.NewChainClient(logger, cosmosCCC, homeDir, os.Stdin, os.Stdout)
	if err != nil {
		log.Fatal(err)
	}

	osmoCCC := &lens.ChainClientConfig{
		Key:		"osmo_acc",
		ChainID:	"osmosis-1",
		RPCAddr:	"https://osmosis-rpc.polkachu.com:443",
		AccountPrefix:	"osmo",
		KeyringBackend:	"test",
		GasAdjustment:	1.2,
		GasPrices:	"0uosmo",
		KeyDirectory:	keyDir,
		Debug:		false,
		Timeout:	"20s",
		OutputFormat:	"plain",
		SignModeStr:	"direct",
		Modules:	lens.ModuleBasics,
	}
	osmoClient, err := lens.NewChainClient(logger, osmoCCC, homeDir, os.Stdin, os.Stdout)
	if err != nil {
		log.Fatal(err)
	}

	cosmosAddr, err := cosmosClient.GetKeyAddress()
	if err != nil {
		log.Fatal(err)
	}
	osmoAddr, err := osmoClient.GetKeyAddress()
	if err != nil {
		log.Fatal(err)
	}

	var wg sync.WaitGroup

	wg.Add(1)
	go func() {
		defer wg.Done()
		time.Sleep(time.Second)
		done := cosmosClient.SetSDKContext()
		fmt.Println("Cosmos: ", cosmosAddr.String())
		done()
	}()

	wg.Add(1)
	go func() {
		defer wg.Done()
		time.Sleep(time.Second)
		done := osmoClient.SetSDKContext()
		fmt.Println("Osmosis:", osmoAddr.String())
		done()
	}()

	wg.Wait()
}

This code example is intentionally racy as to show that the results are different each time you run the thing.
# Cosmos client runs first.
Cosmos:  cosmos13x77yexvf6qexfjg9czp6jhpv7vpjdwwnsrvej
Osmosis: cosmos13x77yexvf6qexfjg9czp6jhpv7vpjdwwnsrvej

# Osmosis client runs first.
Osmosis: osmo13x77yexvf6qexfjg9czp6jhpv7vpjdwwmtsu0q
Cosmos:  osmo13x77yexvf6qexfjg9czp6jhpv7vpjdwwmtsu0q

Was just talking about this today with some folks and happened to see this issue.
This behavior is a known issue with the Cosmos SDK. When you call SetSDKContext, that setting is global within the SDK. I recommend that you avoid the AccAddress's .String() function and call EncodeBech32AccAddr in Lens client/address.go instead.
As a second option, you could call a function that holds the lock on the SDK context until its finished generating the address. For example, your function could be something like this:
sdkConfigMutex.Lock()
defer sdkConfigMutex.Unlock()
sdkConf := sdk.GetConfig()
sdkConf.SetBech32PrefixForAccount(cc.Config.AccountPrefix, cc.Config.AccountPrefix+"pub")
sdkConf.SetBech32PrefixForValidator(cc.Config.AccountPrefix+"valoper", cc.Config.AccountPrefix+"valoperpub")
sdkConf.SetBech32PrefixForConsensusNode(cc.Config.AccountPrefix+"valcons", cc.Config.AccountPrefix+"valconspub")
osmoAddr.String()

Thanks @KyleMoser , the first solution seems to work 🙏🏼


Allow to use stream as source

Allow to use a stream on .src() so it could allow to use less memory instead of having all the file in memory on a Buffer object.

+1

Any news about this option?

A good pull request with docs and tests would be welcomed.

Unfortunately, it's going to be a bit tricky since https://github.com/kevva/decompress-unzip doesn't support streams, and probably never will because of https://github.com/thejoshwolfe/yauzl#no-streaming-unzip-api.


Longest Common Prefix

C++ code to find longest common prefix using Trie data structure

@hrithikpurwar Can you provide a bit explanation for this implementation.
Thanks!