Update cherry-picking-a-commit-in-github-desktop.md

Why:
Closes:
Only the readme file has been updated. After dropping the commit to the respective branch, there is another step to complete the cherry pickup process. You will need to push those changes to the target branch.

@PElanchezhiyan Thanks so much for submitting a PR! I'll get this triaged for review :zap:


Lifecycle Events and Task Scheduling not working

Hi @sandeepsuvit ,
I changed MongooseModule to TenancyModule then i got warning like this.
Task Scheduling and onModuleInit not working. Please check

Project to test proplem
https://github.com/canhhungit/NestJS-tenancy

@canhhungit thanks for writing in. I checked your code. Seems like you are using a cron trigger inside your service class. The way the library is designed and the tenant is resolved is via a Request scoped object that comes from the client applications. In case of cron triggers they are originated from the backend application itself and not via a client side request. Hence there is no way the module can understand and resolve the correct tenant schema model. I am guessing for this cron cases the connections needs to be manually resolved and used.

Thank you @sandeepsuvit. While waiting from your  response, i understand proplem as you said.
I resolve it menual with connection string with monggose


Implement AUTO-PEERING

UPDATE- looks like upon further investigation, Testnet side doesn't have to do anything, it all comes down to Rafiki
Implement Auto-peering with testnet, so that anyone who has rafiki instance or has the necesary graphqls, can peer with Testnet.
Notes from Roadmap - Rafiki changes also included in the notes:

Notify the peer being peered with
New mutation to create a peer without an assetId, the asset input instead (assetCode & scale)
create asset & add asset liquidity within single call
expose HTTP route for peering details discoverability
Automatically accept the connection (in testnet) when a peer goes online
Make local playground peer-able with testnet

Auto-Peering is done, you can now peer with rafiki.money if you have an instance of rafiki running with your application.


enables cheat window during gameplay

@ChrisPJohn , not sure if best location for this toggle is under House Rules. We might need to consider a new tab for these...


The oauth state was missing or invalid.

I know there's been some other questions on this, but I'm trying to use an OAuth2 client that uses a slightly older version of OAuth2 and I was wondering if there's a way to disable state and still be able to use OAuth2 middleware or am I going to have to make the calls on my own.
The other question is this seems also an older version as stated above and after I made the initial call to the /authorize it immediately returns the refresh_token in a POST. Which seems to be the Implicit Flow, which I know is not as secure, but I don't have many other options.
What is the best way to use the middleware with Implicit Flow and ignoring the state?
Thanks,
Jamie

The state is required in ASP.NET Core's auth system to help prevent certain types of attacks. Closing this issue because we have no plans to change this behavior.


Split KMS functionality into separate optional crates?

Hi! I'm working on a PR to bump the version of *ring* used in roughenough, but I'm having difficulties building with the versions of the KMS crates which are optional dependencies. The KmsProvider trait seems like it could be a good basis for an interface between separate crates, and might even be useful in its own separate crate.
Would you be interested in contributions to split things up a little?

Hello @anp! The KMS dependencies do complicate things and I'd welcome work to split things up.

Hi @anp. I'm closing the issue as it's stale but continue to welcome a PR or further discussion.

OK! I’d still like to do this but it’ll be a bit before I can circle back to it again :).


[Site Request] LRCLib

https://lrclib.net
https://github.com/tranxuanthang/lrcget
A privately-run open source synced (and plaintext) lyrics server.  It also has an API.

Oh cool, I didn't know that existed. I've actually been kinda tempted to build something like that myself a few times. Good to know I don't have to, nice!
API availability is super useful and it even has a submission API that I think it would be neat to try support sometime.

are you adding lrclib.net as source or the database functionality or the server @jacquesh

Being able to submit lyrics would be useful. the LRCGET app is kinda janky.

If you add this source, please add a toggle for synced/unsynced. I only ever need synced lyrics, and so far this source has proven invaluable for explicit tracks, which frequently have no results on Musixmatch, and the Chinese sites are often terribly formated and censored.

ESLyric has the source included in the meantime if anyone wants it.

I've added the source, so that'll be in the next release.
The issue of preferring synced or unsynced though, is in some ways a bit more complicated. The musixmatch source implicitly prefers synced. Already in this thread different people seem to have different preferences in this regard so I can see why you might want it to be configurable. The question then though, is does it really need to be configurable per-source or could it just be a single global "prefer this sort of lyric from sources that appropriately support both" (which would be simpler both to build/maintain and for users to understand/use".

Personally, I only care about synced lyrics, and would gladly see a global toggle.

Is there a way to report the lyrics -

Not that I'm aware of, but you're welcome to ask for it, since LRCLib is open-source

Does LRCLib tell the source ? If yes then add a option to hide duplicates from MusixMatch.


Check when SerialPort onData ends to write

Summary of Problem
Hello, im using serialport, and it's amazing. But i have a little problem, because actually I get a data from a port, but i don't know when onData function ends to write. I'm using Express to create a new instance of serialport and get the returned value from onData function and this is the code.
This is my code:

var SerialPort = require('serialport');
var express = require('express');
var NodeCache = require("node-cache");
var fs = require('fs');

var app = express();

//Variables Globales
var _serialPort = null;
var _dataSP = "";
var _dataSP2 = "";
var _isError = false;
var _errorMessage = "";

//Creates a new serialport instance
app.get('/open_serial_connection', function (req, res) {
    var lMessage = "";
    var lError = 0;
    try {
        if (_serialPort == null) {
            _serialPort = new SerialPort("COM1", {
                baudRate: 9600,
                //parity:    ,
                //dataBits:  ,
                //stopBits:  ,
                autoOpen: false
            });
            _serialPort.open();
            _serialPort.on("error", onError);
            _serialPort.on("open", onOpen);
            _serialPort.on("data", onData);
            _serialPort.on("close", onClose);
            lMessage = "Success.";
        } else {
            //Checks if serialport is actually opened
            if (_serialPort.isOpen) {

                lError = 1;
                lMessage = "Connection is open.";
            } else {
                lMessage = "No connections available.";
            }
        }

    } catch (e) {
        lMessage = "Error trying to create a serialport instance";
        lError = 1;
    }
    res.send({ Error: lError, Message: lMessage});
});

function onOpen(data) {
...
}

function onClose(err) {
...
}
function onData(data) {
    _dataSP = data.toString('utf-8');
    _dataSP = _dataSP.replace(/(\r\n\t|\n|\r\t)/gm, " ");
    _dataSP2 += _dataSP.replace(/(\r?\n|\r)/gm, '') + " ";
    console.log(_dataSP2);
}

function onError(data) {
    _errorMessage = data.toString();
    _isError = true;
    console.log(data);
}

app.get('/result_connection_serial', function (req, res) {
    res.send({ Result : _dataSP2 });
});

With this code, I can get the data from the serialport connection but my actual problem is check when all the data returned from any serialport equip ends to send the data.
Any idea?
Thanks.
Versions, Operating System and Hardware

<EMAIL_ADDRESS>Node.js v8.12.0
Windows 7

@jnsanchezg Because Serialport is based on a Node.js duplex steam, (docs reference), there should be an end event is "emitted when there is no more data to be consumed from the stream". It is probably not documented by Serialport because Node.js already documents it in their stream docs. /cc: @reconbot for confirmation

@HipsterBrown ,thanks for the answer, so adding something like:
app.get('/open_serial_connection', function (req, res) {
...
_serialPort.on("end", onEnd);
...
});
....
function onEnd(){
  console.log("Ends")
}

This code should work?  Actually I'm making tests using a Codebar Lector but when i get the data, it doesn't display the console.log from onEnd.

@reconbot this is my result in console:
**
$ node index.js
App in port 3000.
Open:
75.5455
75.5455 75.5455
75.5455 75.5455 100
75.5455 75.5455 100 13.25
75.5455 75.5455 100 13.25 13.25
Closed**

@jnsanchezg what type of device are you reading data from? Is there any sort of ending delimiter? See: https://serialport.io/docs/api-parser-delimiter

@HipsterBrown I handle with 3 types of devices:
A bar reader.
A digital refractometer.(This one returns something like 001 Result = 45.40, if i get this data, first it gets 001 Result, and then = 45.40, so I'm concatenate the result in a global variable)
A digital weighing scale(This one returns always a value, it's continuous, and I use the same idea in the digital refractometer).

You might be able to use the Readline parser to get the complete read .on('data'). Then you don't need to do the String.replace manually in your onData callback.
https://serialport.io/docs/api-parser-readline

Got it, I'm going to use in those devices Readline and them I'll share the results.

@HipsterBrown using Readline works very well in the three devices, but I forgot to mention another device(a digital titrator).
This one throws this data:
$S PC/LIMS V1\r\n$S Devices V1\r\n$S device 848 Titrino plus V1\r\nP 5.848.0025\tS 32650\r\n$S ADC1\r\nADC 3.848.1210\tS 94849\r\n$E\r\n$S MSB1\r\n$S 848\r\nS 85896\tS 30017659\r\n$E\r\n$S 801\r\nS 82544\t\r\n$E\r\n$S 848\r\n\t\r\n$E\r\n$E\r\n$E\r\n$E\r\n$S Sample data V1\r\n\t\t1.0\tg\r\n$E\r\n$S MPL V2\r\n$S Mode 1\t04\tSET pH\tV1.0\r\n1\t0.0\t4.019\t0.0000\t0.0\t25.0\r\n2\t2.0\t4.057\t0.1220\t3108.8\t25.0\r\n3\t4.0\t4.065\t0.1580\t875.3\t25.0\r\n4\t6.0\t4.085\t0.2160\t1494.6\t25.0\r\n5\t8.0\t4.118\t0.2960\t2428.6\t25.0\r\n6\t10.0\t4.158\t0.3660\t2172.2\t25.0\r\n7\t12.0\t4.189\t0.4400\t2103.8\t25.0\r\n8\t14.0\t4.225\t0.5120\t2334.6\t25.0\r\n9\t16.0\t4.264\t0.5840\t2042.2\t25.0\r\n10\t18.0\t4.300\t0.6560\t2144.6\t25.0\r\n11\t20.0\t4.336\t0.7280\t2156.8\t25.0\r\n12\t22.0\t4.371\t0.8020\t2171.6\t25.0\r\n13\t24.0\t4.408\t0.8760\t2314.8\t25.0\r\n14\t26.0\t4.438\t0.9420\t1958.4\t25.0\r\n15\t28.0\t4.474\t1.0240\t2418.9\t25.0\r\n16\t30.0\t4.510\t1.0920\t2120.0\t25.0\r\n17\t32.0\t4.544\t1.1600\t2091.2\t25.0\r\n18\t34.0\t4.583\t1.2320\t2112.3\t25.0\r\n19\t36.0\t4.615\t1.2980\t1950.3\t25.0\r\n20\t38.0\t4.656\t1.3800\t2463.6\t25.0\r\n21\t40.0\t4.684\t1.4360\t1782.1\t25.0\r\n22\t42.0\t4.719\t1.5140\t2192.2\t25.0\r\n23\t44.0\t4.765\t1.5780\t2131.9\t25.0\r\n24\t46.0\t4.792\t1.6380\t1643.2\t25.0\r\n25\t48.0\t4.838\t1.7180\t2468.4\t25.0\r\n26\t50.0\t4.870\t1.7700\t1609.0\t25.0\r\n27\t52.0\t4.904\t1.8440\t2107.8\t25.0\r\n28\t54.0\t4.952\t1.9000\t1896.6\t25.0\r\n29\t56.0\t4.983\t1.9620\t1636.5\t25.0\r\n30\t57.9\t5.035\t2.0360\t2361.4\t25.0\r\n31\t60.0\t5.064\t2.0800\t1397.3\t25.0\r\n32\t61.9\t5.102\t2.1460\t1776.3\t25.0\r\n33\t63.9\t5.150\t2.2040\t1896.1\t25.0\r\n34\t65.9\t5.186\t2.2600\t1559.4\t25.0\r\n35\t67.9\t5.236\t2.3160\t1831.1\t25.0\r\n36\t69.9\t5.280\t2.3720\t1550.6\t25.0\r\n37\t71.9\t5.316\t2.4260\t1701.9\t25.0\r\n38\t73.9\t5.360\t2.4860\t1714.5\t25.0\r\n39\t75.9\t5.406\t2.5380\t1660.1\t25.0\r\n40\t77.9\t5.448\t2.5940\t1596.1\t25.0\r\n41\t79.9\t5.498\t2.6440\t1581.4\t25.0\r\n42\t81.9\t5.538\t2.6960\t1494.5\t25.0\r\n43\t83.9\t5.592\t2.7500\t1694.5\t25.0\r\n44\t85.9\t5.632\t2.7940\t1326.4\t25.0\r\n45\t87.9\t5.681\t2.8500\t1635.0\t25.0\r\n46\t89.9\t5.724\t2.8940\t1447.0\t25.0\r\n47\t91.9\t5.772\t2.9500\t1519.9\t25.0\r\n48\t93.9\t5.816\t2.9960\t1518.9\t25.0\r\n49\t95.9\t5.871\t3.0480\t1504.2\t25.0\r\n50\t97.9\t5.908\t3.0960\t1385.4\t25.0\r\n51\t99.9\t5.978\t3.1460\t1608.9\t25.0\r\n52\t101.9\t6.018\t3.1820\t1045.8\t25.0\r\n53\t103.9\t6.062\t3.2380\t1603.0\t25.0\r\n54\t105.9\t6.118\t3.2820\t1448.8\t25.0\r\n55\t107.9\t6.172\t3.3240\t1212.2\t25.0\r\n56\t109.9\t6.221\t3.3720\t1363.2\t25.0\r\n57\t111.9\t6.270\t3.4160\t1373.3\t25.0\r\n58\t113.9\t6.332\t3.4580\t1278.4\t25.0\r\n59\t115.9\t6.380\t3.4960\t1118.7\t25.0\r\n60\t117.9\t6.440\t3.5440\t1437.3\t25.0\r\n61\t119.9\t6.500\t3.5760\t1121.1\t25.0\r\n62\t121.9\t6.557\t3.6180\t1102.8\t25.0\r\n63\t123.9\t6.629\t3.6500\t1163.5\t25.0\r\n64\t125.9\t6.695\t3.6860\t942.8\t25.0\r\n65\t127.9\t6.758\t3.7220\t1119.0\t25.0\r\n66\t129.9\t6.834\t3.7520\t984.5\t25.0\r\n67\t131.9\t6.899\t3.7860\t927.0\t25.0\r\n68\t133.9\t6.984\t3.8140\t1005.6\t25.0\r\n69\t135.9\t7.040\t3.8420\t776.0\t25.0\r\n70\t137.9\t7.129\t3.8760\t984.0\t25.0\r\n71\t139.9\t7.208\t3.8980\t882.9\t25.0\r\n72\t141.9\t7.280\t3.9280\t741.6\t25.0\r\n73\t143.9\t7.390\t3.9560\t912.6\t25.0\r\n74\t145.9\t7.445\t3.9760\t740.2\t25.0\r\n75\t147.9\t7.511\t4.0100\t785.3\t25.0\r\n76\t149.9\t7.633\t4.0340\t1018.2\t25.0\r\n77\t151.9\t7.706\t4.0540\t699.0\t25.0\r\n78\t153.9\t7.783\t4.0840\t691.0\t25.0\r\n79\t155.9\t7.863\t4.1080\t841.1\t25.0\r\n80\t157.9\t7.944\t4.1380\t781.7\t25.0\r\n81\t159.9\t8.027\t4.1620\t862.2\t25.0\r\n82\t161.9\t8.097\t4.1740\t774.4\t25.0\r\n83\t163.9\t8.104\t4.1740\t690.9\t25.0\r\n84\t165.9\t8.105\t4.1740\t623.6\t25.0\r\n85\t167.9\t8.104\t4.1740\t559.5\t25.0\r\n86\t169.8\t8.101\t4.1740\t267.6\t25.0\r\n87\t171.9\t8.130\t4.1940\t146.0\t25.0\r\n88\t173.9\t8.143\t4.1940\t140.2\t25.0\r\n89\t175.8\t8.145\t4.1940\t134.9\t25.0\r\n90\t177.8\t8.145\t4.1940\t130.0\t25.0\r\n91\t179.8\t8.144\t4.1940\t143.9\t25.0\r\n92\t181.8\t8.143\t4.1940\t19.1\t25.0\r\n93\t182.8\t8.143\t4.1940\t17.5\t25.0\r\n$E\r\n$E\r\n$S DETERM V1\r\n$S Props V2\r\nAcidez\tsaved\t20181003-103746\t326501538563066000\t2018-10-03 10:37:46\toriginal\tRegular without errors\tTicofrut\t61\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\r\n$E\r\n$S Results V1\r\n$S Result 1C.R1 V1\r\nAcidez mL\t4.194\tmL\t0\r\n$E\r\n$S Result 1C.R2 V1\r\n\t\tmeq/L\t-1\r\n$E\r\n$S Result 1C.R3 V1\r\n\t\tg/L\t-1\r\n$E\r\n$S Result 1C.R4 V1\r\n\t\t%\t-1\r\n$E\r\n$S Result 1C.R5 V1\r\n\t\t%\t-1\r\n$E\r\n$E\r\n$S Mode 1\t04\tSET pH\tV2.0\r\n$S EP V1\r\n4.1940\t8.143\t\t182.8\t25.0\t1\r\n$E\r\n$S Cal V1\r\n$E\r\n$S StdAdd V1\r\n$E\r\n$E\r\n$S Other Variables V1\r\n$S System V1\r\n195.0\t61\t1426\t0\t\t\t0\t\t0\r\n$E\r\n$S Mode 1\t04\tSET pH\tV2.0\r\n1.000\t1.000\t4.1940\t183.9\t0.0000\t4.024\t25.0\t4.019\t25.0\t1.1\t8.143\t25.0\t\t\t\t6.963\t99.2\t\t93\t\t\tRegular stop\tmanual\t\t\tNaOH.\r\n$E\r\n$S Statistic V2\r\n$E\r\n$S ComVar local V1\r\n$E\r\n$S Titrant V1\r\n$S Titrant1 V1\r\nNaOH.\t1.000\tmol/L\t1.000\t\t\t2018-06-11 15:56:11\t30017659\t20\t495484\r\n$E\r\n$E\r\n$S Sensor V1\r\n$S Sensor1 V2\r\npH electrode\tpH\t\t\t99.2\t6.963\t25.0\t(manual)\t2018-10-03 07:00:45\tCAL pH\tTicofrut\r\n$E\r\n$E\r\n$E\r\n$S Calc Vars V1\r\n$S Calc1 V1\r\n$S R1 V1\r\nEP1\t4.1940\t\r\n$E\r\n$S R2 V1\r\n\r\n$E\r\n$S R3 V1\r\n\r\n$E\r\n$S R4 V1\r\n\r\n$E\r\n$S R5 V1\r\n\r\n$E\r\n$E\r\n$E\r\n$S Messages V1\r\n$E\r\n$E\r\n$S Methodproperties V1\r\nAcidez\tsaved\tTicofrut\t2008-01-10 12:59:06\t\t2018-06-12 16:20:58\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\r\n$E\r\n$S Method V1\r\n$S Methodoptions V1.1\r\noff\t2\toff\tID 1\tID 2\ton\ton\toff\t1.0\tg\toff\t0\t9999999999\toff\t30 s\ton\t\toff\toff\tCard 1\t\tID1\ton\ton\toff\ton\toff\tno\t**********00**\toff\toff\r\n$E\r\n$S 01\tT+\tSET pH\r\noff\t120\r\n$E\r\n$S 02\tCTRL\tSend remote signal\r\nTitrando 1\ton\toff\t200 ms\r\n$E\r\n$S 03\tREQUEST\tData request\r\noff\toff\toff\toff\ton\r\n$E\r\n$S 04\tSET pH\tSet endpoint titration pH\r\nTitrando 1\toff\t0 s\t1 s\t0.00000 mL\tmax.\t0 s\t0 s\t8.1\tuser\toff\t10.00 mL/min\t250 ?L/min\tdrift\t20 ?L/min\t10 s\toff\toptimal\t2.000\t10.00 mL/min\t25.00 ?L/min\tdrift\t20 ?L/min\t10 s\tauto\t0 s\t25.0 ?C\t2.0 s\t20 mL\toff\tmax.\ton\tpH electrode\tautomatic\ton\tNaOH.\ton\t5\ton\toff\t20 ?L/min\toff\t0.0 ?L/min\toff\t20.0000 mL\toff\toff\toff\toff\r\n$E\r\n$S 05\tBEEP\tAcoustic signal\r\n1\r\n$E\r\n$S 06\tCALC\tCalculation\r\n$S R1\r\n1\tAcidez mL\tEP1\t3\tmL\t0\toff\toff\t0\ton\toff\tRound\toff\tinvalid\tinvalid\tDisplay message\t\t\r\n$E\r\n$S R2\r\n2\t\t\t2\tmeq/L\t1\toff\toff\t1\ton\toff\tRound\toff\tinvalid\tinvalid\tDisplay message\t\t\r\n$E\r\n$S R3\r\n3\t\t\t2\tg/L\t2\toff\toff\t2\ton\toff\tRound\toff\tinvalid\tinvalid\tDisplay message\t\t\r\n$E\r\n$S R4\r\n4\t\t\t2\t%\t3\toff\toff\t3\ton\toff\tRound\toff\tinvalid\tinvalid\tDisplay message\t\t\r\n$E\r\n$S R5\r\n5\t\t\t2\t%\t4\toff\toff\t4\ton\toff\tRound\toff\tinvalid\tinvalid\tDisplay message\t\t\r\n$E\r\n$E\r\n$S 07\tT+ Report\t\r\noff\toff\toff\toff\toff\r\n$E\r\n$E\r\n$E\r\n
From this result, I can't use Readline, because it throws the result in parts.
If I use a [(https://serialport.io/docs/api-parser-regex)] regex to get the last one \r\n, it should work?
Thanks!

Good to hear the Readline parser works for those three devices.
I think the Regex parser is worth a shot for that other device given the odd output. :+1:
Can we close out this issue?

@HipsterBrown but now I have another question? Using a regex, this one applies under a specific buffer? It is because all the previous data is a lot of Buffers.
Thanks.

In wich way?


Problems with systemd inside docker container

Issue Type

Bug report

Molecule and Ansible details
ansible <IP_ADDRESS>
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides
  python version = 2.7.13 (default, Jan 19 2017, 14:48:08) [GCC 6.3.0 20170118]
molecule, version <IP_ADDRESS>rc16

Molecule installation method: One of source, pip
Ansible installation method: One of source, pip, OS package

Desired Behaviour
Testing for systemd services inside containers should work
Actual Behaviour (Bug report only)
In molecule v1.25, it was possible to get round the systemd error:
stderr=u'Failed to connect to bus: No such file or directory'

by adding:
docker:
  containers:
      privileged: True

However, this doesn't seem to work in v2 (and there's no corresponding "docker" key. I tried adding:
platforms:
  - name: instance
    image: solita/ubuntu-systemd:16.04
    privileged: True

...but this doesn't seem to work. As you can see, I'm using a systemd-enabled container. I can confirm that this was working in v1.25 with the "privileged: True" option.

Are you modifying the molecule.yml inside the scenario directory?  As you see I am able to test these types of containers in my etcd role.

Thanks, it was the command: /sbin/init that did the trick:
platforms:
  - name: instance
    image: solita/ubuntu-systemd:16.04
    privileged: True
    command: /sbin/init

Many Thanks

Many Thanks

👍


refactor handling profiles from CLI

Better implementation of handling profiles from CLI

Added profile property in IOpenConfiguration - profile to use while opening windows with this configuration
Consumers (CLI / Ext host Debug) parse CLI args and set the profile to use while opening the windows

Please note that most of the changes in windowMainService are to undo and go back to the state before https://github.com/microsoft/vscode/pull/160163

Overall seems nice, something "fishy" is that we only want the "new window" behaviour when there is a profile passed form the CLI I suppose? But I am not sure how to tell that apart if profile is just on the open configuration.

something "fishy" is that we only want the "new window" behaviour when there is a profile passed from the CLI I suppose? But I am not sure how to tell that apart if profile is just on the open configuration.

I think it's not fishy but cleaner. This behavior is controlled from outside by those who open windows Eg: 'app.ts' by using the flag 'forceNewWindow'. It's good that we are reusing existing flags instead of adding new flags like 'forceNewWindowIfProfileIsPassed' or something.

Yeah using the flags is very good, what I meant is that its not clear how the profile comes in in some cases. For example here, you can see the profile is resolved from CLI:
https://github.com/microsoft/vscode/blob/1c2eb0be756d092caadb0d6d8c601fc284df8c02/src/vs/platform/launch/electron-main/launchMainService.ts#L119-L120
But here its less obvious:
https://github.com/microsoft/vscode/blob/1c2eb0be756d092caadb0d6d8c601fc284df8c02/src/vs/code/electron-main/app.ts#L541-L543
Because the profile is passed in as argument.
Anyway, not a big deal...

I see. Actually I thought of moving profile resolving from CLI to openFirstWindow method in app.ts where we process all CLI args. Since it is sync and resolving profile is async, I left it where it is now and passed the profile to openFirstWindow method.


Modify path DaprRoot add the SystemDrive environment variable

Description
DaprRoot defaults to C:/Ddapr, but it does not rule out the possibility of SystemDrive that is not C:.

ping

@artursouza @yaron2 Was there any particular reason to select C: to install Dapr ? And is it ok to change to SystemDrive from C: ?
@hueifeng Based on this reply, this might also need some document changes ... Will ping you on that once we get a reply for this.

@mukundansundar Okay, thank you for your reply.

@artursouza @yaron2 Was there any particular reason to select C: to install Dapr ? And is it ok to change to SystemDrive from C: ?
@hueifeng Based on this reply, this might also need some document changes ... Will ping you on that once we get a reply for this.

I think it should be fine

@mukundansundar look  Files changed 2 is normal?

@mukundansundar look  Files changed 2 is normal?

But why are there 25 commits? And also commits that actually are not part of this PR?

@mukundansundar I suspect it was because the PR was committed a while ago, so when I changed it again, I merged the master content.

@mukundansundar I suspect it was because the PR was committed a while ago, so when I changed it again, I merged the master content.

Need to solve this problem?

@hueifeng There are other commits that are already there in master ... Can you fix it ? Probably rebase ?

@mukundansundar I think of it ,this happened when I signed the DOC after I made the change.

@hueifeng There are other commits that are already there in master ... Can you fix it ? Probably rebase ?

@mukundansundar I think of it ,this happened when I signed the DOC after I made the change.

This needs to be fixed please.

@mukundansundar Please help to see whether it is right, I feel when performing Update branch after there will be other commit.

@hueifeng Thanks for making the suggested changes. For this documentation also has to be updated here and the [README] (https://github.com/dapr/cli#installing-dapr-cli) also needs an update.


Invalid Discord link

When you go to https://python.useinstructor.com/ and try to join the discord the above is what you get.

Hey @wasauce , I don't seem to get the same error. But I regenerated a new link for you to the discord here - https://discord.gg/Z5F4g8TA, can you see if it works for you?

@ivanleomk thank you! Yes this worked.


--- title: Prebuilding your codespaces intro: 'To speed up codespace creation, you can configure your project to prebuild codespaces for specific branches in specific regions.' product: '{% data reusables.gated-features.codespaces %}' versions:   fpt: ''   ghec: '' topics:   - Codespaces redirect_from:   - /codespaces/customizing-your-codespace/prebuilding-codespaces-for-your-project children:   - /about-codespaces-prebuilds   - /configuring-prebuilds   - /managing-prebuilds   - /testing-dev-container-changes ---   {% data reusables.codespaces.prebuilds-beta-note %}[Issue]:

Describe the Issue

title: Prebuilding your codespaces
intro: 'To speed up codespace creation, you can configure your project to prebuild codespaces for specific branches in specific regions.'
product: '{% data reusables.gated-features.codespaces %}'
versions:
fpt: ''
ghec: ''
topics:

Codespaces
redirect_from:
/codespaces/customizing-your-codespace/prebuilding-codespaces-for-your-project
children:
/about-codespaces-prebuilds
/configuring-prebuilds
/managing-prebuilds
/testing-dev-container-changes

{% data reusables.codespaces.prebuilds-beta-note %}
Environment
Live/Production
Version
No response
Expected Behavior
No response
Actual Behavior
No response
Steps to Reproduce
No response
Pre-conditions
No response
Relevant log output
No response

Spam, not planned.


google auth is working with full app connection

only works locally. once you guys test I will setup paths to production app


Undefined offset: 2

I am getting an error 'Undefined offset: 2' at the second loop of getRoles() function on ZendDB.php...
I was trying to debug, and when I make a print_r of roleObj, this is what I get
BjyAuthorize\Acl\Role Object
(
[roleId:protected] => guest
[parent:protected] =>
)
BjyAuthorize\Acl\Role Object
(
[roleId:protected] => user
[parent:protected] =>
)
BjyAuthorize\Acl\Role Object
(
[roleId:protected] => admin
[parent:protected] => BjyAuthorize\Acl\Role Object
(
[roleId:protected] => 2
[parent:protected] =>
)
)
I am not sure but I think the 2 should not be there. This is how my user_role table looked like
+----+---------+-----------------+-------------------+
| id  | role_id | is_default | parent_id |
+----+---------+-----------------+-------------------+
|  1 | guest   |               1 |       NULL |
|  2 | user     |               0 |       NULL |
|  3 | admin   |              0 |        2       |
+----+----------+----------------+---------------+
I changed it to
+----+---------+-----------------+-------------------+
| id  | role_id | is_default | parent_id |
+----+---------+-----------------+-------------------+
|  1 | guest   |               1 |       NULL |
|  2 | user     |               0 |       NULL |
|  3 | admin   |              0 |        user  |
+----+----------+----------------+---------------+
The error disappeared but am not sure that's how I should do this.
Any help will be appreciated

I had the same issue. It was same as @lawrence615 pointed out. When I changed to "bjyoungblood/bjy-authorize": "dev-master" the error gone.


Add preview function

Add function preview() to get a string that can be use to debug without using the printer.
Example response preview():
    ┌───────────────────────────────────┐
    │▓▓▓▓▓▓▓▓▓▓▓ PosPrinter ▓▓▓▓▓▓▓▓▓▓▓▓│
    ├───────────────────────────────────┤
    │Date: 08.07.2015                   │
    │Time: 19:46                        │
    │Ticket No: 1                       │
    ├───────────────────────────────────┤
    │          - Hot dishes -           │
    │— 3 Kazan kabob               60,00│
    │— 2 Full-Rack Ribs            32,00│
    │                       Total: 92,00│
    │                                   │
    │            - Salads -             │
    │— 1 Turkey & Swiss             4,50│
    │— 1 Classic Cheese             3,30│
    │— 1 Chicken Caesar Salad       7,00│
    │                       Total: 14,80│
    │                                   │
    │           - Desserts -            │
    │— 1 Blondie                    5,00│
    │— 2 Chocolate Cake             7,00│
    │                       Total: 12,00│
    │                                   │
    │          - Drinkables -           │
    │   50% sale for Coke on mondays!   │
    │— 3 Coca-Cola                  6,00│
    │— 7 Tea                        3,50│
    │— 2 Coffee                     3,00│
    │                       Total: 12,50│
    ╞═══════════════════════════════════╡
    │Total gift                     3,00│
    │Total                        128,30│
    ╞═══════════════════════════════════╡
    │        *** THANK YOU ***          │
    └───────────────────────────────────┘

That sounds like a lot of fun, but I think its real world applicability is quite limited as it would only properly work for single-width/height text, and it would not display decorations such as underline or bold.
Did you have a use case with only single-size text when you created this issue? Would you have used this if it existed?


Added twoFA feature

generates a random 6 digit session id
uses SMS service to send sms to the registered PHONENo
added config file for SMS service
added Middleware to check if secured endpoints have sessionId
added tests for two factor
modified auth.py to include middleware tests
modified README to include instructions for 2fa flow

@MohammedAman23, could you please use the auto-format tools on all the files? I recommend running JSHint and Prettier.

@mahidharc I have used the pretttier tool on main.js.


[RFC][Frontend] Add a PaddlePaddle Frontend

Add a PaddlePaddle frontend for TVM, also I have created a RFC topic in forum where we can discuss at same time.
The related pull request is https://github.com/apache/tvm/pull/8645, still lack of UT, which will be finished in a week.
@tqchen @FrozenGene @junrushao1994 @jcf94

@jiangjiajun Thanks for bringing this rfc, which will help TVM to serve more conveniently for paddlepaddle users. This rfc is good generally speaking. However, there are many spell grammar error so that I suggest you could do one round of proof-reading

@jiangjiajun Thanks for bringing this rfc, which will help TVM to serve more conveniently for paddlepaddle users. This rfc is good generally speaking. However, there are many spell grammar error so that I suggest you could do one round of proof-reading

Thanks for your comments, I'll review the RFC this week and update this pull request~

cc @mbrookhart who works with paddle before

Thanks @will-jl944 for revising and polishing this document, it really helps me a lot!
@FrozenGene  And an example code is provided in Guide-level explanation, could you review this RFC again, thanks~

@jiangjiajun Please go ahead to handle pr.

@FrozenGene @jiangjiajun please update the file to reflect the RFC number as in other RFCs, please also create a tracking issue per the RFC procedure

Two followup actionable items per https://github.com/apache/tvm-rfcs#rfc-workflow

Please rename the rfc to the original PR number (19)
Please create a tracking issue per guideline

Two followup actionable items per https://github.com/apache/tvm-rfcs#rfc-workflow

Please rename the rfc to the original PR number (19)
Please create a tracking issue per guideline

Okay


Add nominee: MET Norway Weather API

Automatic addition of a new nominee submitted through the online form available at https://digitalpublicgoods.net/submission

@nathanbaleeta: this is a duplicate of https://github.com/unicef/publicgoods-candidates/blob/master/nominees/met-norway-weather.json. Both files should be merged into one 🙏

@lacabra resolved in commits c20e2c9 & ced4776.


chore: add Cloudflare Workers to Serverless Platforms

Updated Serverless deployment guides list

@ruheni I have zero excuses for the delay, double-jeez.


Start using the new RefCounter datatype

Description
This replaces the reference counting functionality on Runs. Other uses, such as adding reference counting to merging runs, will be added later
Checklist

[x] Read our contribution guidelines at CONTRIBUTING.md, and make sure that this PR complies with the guidelines.

Whoops, I messed up a force push so the PR was accidentally closed

Whoops, again


Being able to configure the color palette easily.

Hello, it would be awesome to have access to the color for the Tokyo Night theme, I would love to be able to change the html color to red, the same as the vscode tokyo night theme.

You can use tokyonight.colors module, there is an example under extras section and issue https://github.com/folke/tokyonight.nvim/issues/75

Hi! I just added a bunch of documentation on how to override colors & highlight groups.


[Doc][GitHub Readme] Unit test badge failing

The unit test badge on the main readme has been failing for a while. Seems like a configuration issue. Not a super high-priority issue, but would be nice if someone can fix this when convenient.

I check the Badge Link and the related Github Action, they seem alright to me:
Badge: https://github.com/apache/incubator-devlake/actions/workflows/test.yml/badge.svg
Action Page: https://github.com/apache/incubator-devlake/actions/workflows/test.yml
I had posted a thread on https://github.community/ , and it is waiting for Approval from Moderator, interesting...  Moderated Community...

Alright, it was approved, https://github.community/t/github-action-badge-always-show-failing/259290

The reason for failing is ci haven't run in branch main.
Echarts will run ci when merge master to release.
we don't have a release branch so I will try to run ci in the main branch every night.


[GSSoc'23] Chrome Dinosaur Game

Aim

This project creates a Dinosaur game (similar to the Chrome Dinosaur game )

Details

It uses the Python Pygame Library.
It will have functionalities similar to the Chrome dinosaur game.

@avinashkranjan Can you assign this issue to me under GSSoC'23 label?

@avinashkranjan Can I work on this issue?

Go Ahead @Atharvashirsh

Hey @avinashkranjan, I've added the PR which contains the game files.


Cannot use on custom model

Tried to use my custom model, But when i try to hit the Scan button the app crashes. Do you have a detailed instructions on how to use custom model ?

Do you have any specific error messages? Try these steps:

Download your custom model (I've tried "inception v4_1") and place it in app/src/main/res/assets.
Uncomment lines in MLKitObjectDetector to specify the model.

@hallowcard13 @devbridie
hi any solution resolved?

Hello @devbridie
Thank you for follow ups.
I have trained a custom model. But since very long tried to integrate it. Unfortunately it's failing.
Error, it says -
E/AndroidRuntime: FATAL EXCEPTION: DefaultDispatcher-worker-1     Process: com.google.ar.core.examples.java.ml, PID: 2794     com.google.mlkit.common.MlKitException: Failed to initialize detector. Unexpected number of dimensions for output index 1: got 3D, expected either 2D (BxN with B=1) or 4D (BxHxWxN with B=1, W=1, H=1).         at com.google.mlkit.vision.vkp.PipelineManager.start(com.google.mlkit:vision-internal-vkp@@18.0.0:63)         at com.google.mlkit.vision.objects.custom.internal.zzg.tryLoad(com.google.mlkit:object-detection-custom@@16.3.1:3)         at com.google.mlkit.common.sdkinternal.model.CustomModelLoader.load(com.google.mlkit:common@@17.1.1:3)         at com.google.mlkit.vision.objects.custom.internal.zzh.load(com.google.mlkit:object-detection-custom@@16.3.1:2)         at com.google.mlkit.common.sdkinternal.ModelResource.zza(Unknown Source:18)         at com.google.mlkit.common.sdkinternal.zzn.run(Unknown Source:10)         at com.google.mlkit.common.sdkinternal.zzp.run(com.google.mlkit:common@@17.1.1:2)         at com.google.mlkit.common.sdkinternal.MlKitThreadPool.zze(com.google.mlkit:common@@17.1.1:4)         at com.google.mlkit.common.sdkinternal.MlKitThreadPool.zzc(Unknown Source:8)         at com.google.mlkit.common.sdkinternal.zzj.run(Unknown Source:2)         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)         at com.google.mlkit.common.sdkinternal.MlKitThreadPool.zzd(Unknown Source:10)         at com.google.mlkit.common.sdkinternal.zzk.run(Unknown Source:2)         at java.lang.Thread.run(Thread.java:923) D/TransportRuntime.SQLiteEventStore: Storing event with priority=VERY_LOW, name=FIREBASE_ML_SDK for destination cct D/TransportRuntime.JobInfoScheduler: Upload for context TransportContext(cct, VERY_LOW, MSRodHRwczovL2ZpcmViYXNlbG9nZ2luZy5nb29nbGVhcGlzLmNvbS92MGNjL2xvZy9iYXRjaD9mb3JtYXQ9anNvbl9wcm90bzNc) is already scheduled. Returning... I/Process: Sending signal. PID: 2794 SIG: 9
Custom model trained using -
https://www.tensorflow.org/lite/api_docs/python/tflite_model_maker/object_detector/EfficientDetSpec
I understand, ML kit wants to run a model with batch size one. But also if I have batch size one too, it throws this error.
So to clear up things, I tried tensorflow debugging using few commands to get the tensors.
Still it's not clear to me what needs to be changed.
I'm attaching a tflite file for your testing purpose.
It will be great if you can test it out. Also please let me know, if you need any further detailed information. I'm happy to let you know in the debugging process.
model.zip

@hallowcard13 @devbridie @tbikash62
The error is still present. Any news?
I've tried other .tflite models but the error is the same.
In the next few days I'll try to mix up the actual sample code with https://github.com/googlesamples/mlkit/blob/2f80c301bae79136efa9096881b6e44bee7866a3/android/android-snippets/app/src/main/java/com/google/example/mlkit/kotlin/ObjectDetectionActivity.kt and https://github.com/googlesamples/mlkit/blob/4be2fe856e0b3fd95af370fe5413a33494698067/android/android-snippets/app/src/main/java/com/google/example/mlkit/kotlin/MLKitVisionImage.kt#L93-L118
Hope to update this comment with a solution :)
Thank you


Issue with student enrollment being null or no data

See Crown King Elementary School

Fixed issue for schools with no data but still have an issue with schools that have missing data.


Update Dockerfile to install ruby 2.2.1 from rvm

The version being installed from yum is old (1.8) and is incompatible with a gemfile in fpm

No no no please. We can get the right version of ruby without RVM.

Does it matter for this repo? It is a container that runs as a one off job to build an rpm.
Though if we were to change it are you thinking of something like this? https://gist.github.com/chiraggude/11128087

This is the Dockerfile for ruby:2.2
https://github.com/docker-library/ruby/blob/90b6b506a4b495cd337115d9a2b4ba6f80c72004/2.2/Dockerfile
It's more or less like that gist.

Fair enough I'll do it sans rvm.

Just made the updates to do this without rvm... turns out centos:7 has a version of ruby that works with fpm so using that

I will raise a new PR as the title of this one does't really make sense now


Optimize deleting last property on an object

Today if we are in PathTypeHandler and we delete a property we go to SimpleDictionaryTypeHandler. Having a type in SimpleDictionaryTypeHandler is expensive both performance and memory wise when it comes to adding properties. Most of the time, it evolves new type when properties are added and it is less likely to get shared. Optimize so that if we are deleting last property then transition to previous type of PathTypeHandler.

Closing since this is complete and merged

Fixes https://github.com/Microsoft/ChakraCore/pull/1897


Merge back Windows fixes from 6.0 branch

Closed in favour of https://github.com/ROCmSoftwarePlatform/hipRAND/pull/86


Transferlearning only detects new classes

Hi Moses. Thank you for making AI easier.
My question is related to custom model detection and transferlearning. I use yolov3 and wanted to the model to recognize owls as birds. It's understandable it doesn't as owls look funny and non-birdy. I followed the guide here, https://github.com/OlafenwaMoses/ImageAI/blob/master/imageai/Detection/Custom/CUSTOMDETECTION.md. It now detects owls as birds, but nothing else.  I assumed my new model would continue to detect other classes as well. Is this the correct behavior? If so, can models be merged into single model?

ImageAI does custom object detection using transfer learning. Transfer learning work by training a model on an already trained model (a pretrained model). That is the pretrained model has weights that it has already learned from the previous dataset it was originally trained on. When we used the model to train our own data with transfer learning the pretrained model become fined tuned to our own custom dataset. You might ask why do we even do transfer learning in the first place we do so because it makes our model converge faster at the global minima.


autocomplete not working

Recently installed atom-typescript with Atom 1.10.0 on Ubuntu 14.04.
Syntax highlighting appears to be working as is the lint at the bottom, however autocomplete does not appear to be working.
Does not recognise ane previously created variables or any language words.

Hey, I also had an issue with atom-typescript autocomplete. It was working but it was extremely slow.
You need to have a tsconfig.json in the project root (anyone can correct me if I'm wrong).
atom-typescript can create this file for you by having your cursor in any open .ts file and hitting ctrl+shift+p (atom shortcut) and entering create config. The top result create the file (tsconfig.json).
You can check if it works by opening some .ts file with maybe these contents:
export class Test {
  constructor (
    public name: number,
    p
  ) { }
}

Put your cursor right after standalone p and hit ctrl+space. Give it some time, it was taking maybe 5-8 seconds before anything showed up (that was before letting the addon generate the config, after using the genrated one I am getting auto complete suggestions in less than half a second).
If auto complete is then still slow for you, you can add folders/files to the exclude list in tsconfig.json (it adds some by itself, like node modules).


Add pwsh and cmd as pixi shells to the test

closes #78

Or we just don't mention cmd.exe at all for this feature 😅

Lets do that @pavelzw, you made my day a little better with that comment.
The jobs that still fail aren't an issue for this pr right?

The jobs that still fail aren't an issue for this pr right?

the auth-token-install job is failing because the secret is not leaked to forks.
The Reference latest Pixi version in README job failing can be fixed in this PR :)


Tutorial

@memelopez, you did a great job implementing the tutorial.
I don't have any comments to say, just merge your project and keep learning!!!


[BUG] Pagination inconsistent on BBs and TPS accounts endpoints

Describe the bug
At the moment we are using the limit and skip functions on MongoDB to handle pagination on the POST /bbs/accounts and POST /tps/accounts APIs. As mentioned here: https://www.mongodb.com/docs/manual/reference/method/cursor.skip/#using-range-queries, this is inefficient and not guaranteed to be consistent if the underlying data changes. This may result in duplicated or missing data across pages.
To Reproduce
Steps to reproduce the behavior:

Begin paging through results
Add or remove document from collection
Continue paging to see duplicates/missing data

Expected behavior
We should use the "range query" method described here: https://www.mongodb.com/docs/manual/reference/method/cursor.skip/#using-range-queries instead of the skip function to handle pagination. For this API, we do not care what field is used for ordering, so feel free to sort by the _id field which is guaranteed to be unique and indexed. We should deprecate but still support skip for backwards compatibility.

We should look for similar problems in all of our BBs. This resource has additional information that will be useful for more complex cases, including ones where we want to sort by non-unique fields: https://medium.com/swlh/mongodb-pagination-fast-consistent-ece2a97070f3.


Workbench still pointing to version 39.0 instead of 40.0.

Workbench still pointing to version 39.0 instead of 40.0. Can you fix it?

Hi Team,
Workbench still pointing to version 39.0 instead of 40.0. Can you please fix it as soon as possible.
I am not able to Test Post and Get methods. If version will be updated to 40.0, it will help me a lot.

v40 is available now.


Magritte: merge with the upstream project

Thanks for opening this. Note that the only incompatible change I made in the PR where I addressed your requested changes was that I renamed MADescriptionMethodSource to MADescriptionContext. I don't remember my rationale and I'm happy to change back to the GT terminology if you would like. Let me know what you think.

Hi @seandenigris, thank you for the comment. I removed the MADescriptionMethodSource and use MADescriptionContext. I do not mind using this name.
I will push the changes in the morning.

@seandenigris: do you prefer a pull request or will you cherry-pick our recent changes? If I understand it properly, you preferred cherry-picking during the last sync attempt.

@JurajKubelka A pull request would be great! I only did the cherry pick because there were conflicts and I changed some package names during the merge


[Accessibility] Tri-State Checkbox

Description
Related #1492
Description
I was considering the accessibility of the Checkbox component according to the ARIA Authoring Practices Guide (APG). For the Dual-State Checkbox, the current Checkbox is sufficient to ensure accessibility. On the other hand, I thought it was necessary to create a new component for the Tri-State Checkbox.

WAI-ARIA Roles, States, and Properties

[x] The checkbox has role checkbox.
[ ] The checkbox has an accessible label provided by one of the following:

[x] Visible text content contained within the element with role checkbox.
[ ] A visible label referenced by the value of aria-labelledby set on the element with role checkbox.
[ ] aria-label set on the element with role checkbox.

[ ] When checked, the checkbox element has state aria-checked set to true.
[ ] When not checked, it has state aria-checked set to false.
[ ] When partially checked, it has state aria-checked set to mixed.
[ ] If a set of checkboxes is presented as a logical group with a visible label, the checkboxes are included in an element with role group that has the property aria-labelledby set to the ID of the element containing the label.
[ ] If the presentation includes additional descriptive static text relevant to a checkbox or checkbox group, the checkbox or checkbox group has the property aria-describedby set to the ID of the element containing the description.

Problem Statement/Justification
We should prioritize web accessibility to ensure our UI components are usable by everyone, including people with disabilities. This inclusivity enhances user experience and complies with legal standards, broadening Yamada UI's user base.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
None
Additional Information
No response

@koralle
Hi, is this a feature that already exists?Is this issue closed?


Update service-fabric-resource-governance.md

fix typo

Learn Build status updates of commit 9836142:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/service-fabric/service-fabric-resource-governance.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Update pytest instructions in documentation

This PR updates the pytest instructions in the documentation.

@anutosh491 I've updated the instructions.Are you happy with them now?

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

Project coverage is 79.31%. Comparing base (3ef699d) to head (1511181).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   75.08%   79.31%   +4.22%
==========================================
  Files          17       17
  Lines         602      609       +7
  Branches       59       59
==========================================
+ Hits          452      483      +31
+ Misses        150      126      -24

see 2 files with indirect coverage changes
see 2 files with indirect coverage changes


Fix of grammatical issues and weird other stuff, that doesnt make sen…

…se in German.
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Hi @vividhacker thanks for making this PR and your interest in contributing to translations. I recommend that you visit https://translate.freecodecamp.org to update these instead.


domination: init at 1.2.3

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 104499 run on x86_64-darwin 1

1 package built:

domination

Ping

Ping

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104499 run on x86_64-darwin 1

1 package built:

domination

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104499 run on x86_64-darwin 1

1 package built:

domination

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104499 run on x86_64-linux 1

1 package built:

domination

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104499 run on x86_64-linux 1

1 package built:

domination


singularity cannot mount container rw for copy over /dev/loop2

$ sudo singularity copy src/test_binary images/test-1.2.img /usr/bin/
ERROR  : Failed to mount (rw) '/dev/loop2' at '/usr/local/var/singularity/mnt': Invalid argument

That error typically means the container is corrupt. Can you try another image to see if another one works?
Thank you!

It looks like a HDD was going bad. Sorry for trouble.

No trouble, thanks for letting me know! Whew!! Lol


My input contained 3 files with format .jpg, .mtl, .obj.
The log file is as follows:
Input directory provided - scanning for importable meshes
Found 1 input meshes under: input
Outputting results to /home/jingtao/lsfm/output

****** 1. DENSE CORRESPONDENCE **********

Correspondence: [==========] 100% (1/1) - done.

****** 1. INITIAL PCA + PRUNE **********

Allocated data matrix of size 1.22 MB (1 samples)
Building data matrix: [==========] 100% (1/1) - done.                           /home/jingtao/anaconda3/envs/lsfm/lib/python3.5/site-packages/menpo/math/decomposition.py:136: RuntimeWarning: invalid value encountered in true_divide
  C = np.dot(X, X.conj().T) / (n - 1)

Retaining 98.50% of eigenvalues keeps 0 components
/home/jingtao/anaconda3/envs/lsfm/lib/python3.5/site-packages/menpo/model/pca.py:322: RuntimeWarning: invalid value encountered in double_scalars
  return self._total_variance() / self.original_variance()
ance: 213.860

Input Error: Incorrect objective type.
 nbrpool statistics
        nbrpoolsize:            0   nbrpoolcpos:            0
    nbrpoolreallocs:            0

 Runtime parameters:
   Objective type: Unknown!
   Coarsening type: METIS_CTYPE_SHEM
   Initial partitioning type: METIS_IPTYPE_EDGE
   Refinement type: METIS_RTYPE_FM
   Perform a 2-hop matching: No
   Number of balancing constraints: 1
   Number of refinement iterations: 0
   Random number seed: 212860
   Number of separators: 21860
   Compress graph prior to ordering: Yes
   Detect & order connected components separately: No
   Prunning factor for high degree vertices: -126523184.000000
   Allowed maximum load imbalance: 213.860

Input Error: Incorrect objective type.
 nbrpool statistics
        nbrpoolsize:            0   nbrpoolcpos:            0
    nbrpoolreallocs:            0

 Runtime parameters:
   Objective type: Unknown!
   Coarsening type: METIS_CTYPE_SHEM
   Initial partitioning type: METIS_IPTYPE_EDGE
   Refinement type: METIS_RTYPE_FM
   Perform a 2-hop matching: No
   Number of balancing constraints: 1
   Number of refinement iterations: 0
   Random number seed: 212860
   Number of separators: 21860
   Compress graph prior to ordering: Yes
   Detect & order connected components separately: No
   Prunning factor for high degree vertices: -129997568.000000
   Allowed maximum load imbalance: 213.860

Input Error: Incorrect objective type.
 nbrpool statistics
        nbrpoolsize:            0   nbrpoolcpos:            0
    nbrpoolreallocs:            0

 Runtime parameters:
   Objective type: Unknown!
   Coarsening type: METIS_CTYPE_SHEM
   Initial partitioning type: METIS_IPTYPE_EDGE
   Refinement type: METIS_RTYPE_FM
   Perform a 2-hop matching: No
   Number of balancing constraints: 1
   Number of refinement iterations: 0
   Random number seed: 212860
   Number of separators: 21860
   Compress graph prior to ordering: Yes
   Detect & order connected components separately: No
   Prunning factor for high degree vertices: -129742136.000000
   Allowed maximum load imbalance: 213.860

Input Error: Incorrect objective type.
 nbrpool statistics
        nbrpoolsize:            0   nbrpoolcpos:            0
    nbrpoolreallocs:            0

Traceback (most recent call last):
  File "/home/jingtao/anaconda3/envs/lsfm/bin/lsfm", line 285, in <module>
    main(docopt(__doc__))
  File "/home/jingtao/anaconda3/envs/lsfm/bin/lsfm", line 281, in main
    pca_and_prune(r, verbose=verbose)
  File "/home/jingtao/anaconda3/envs/lsfm/bin/lsfm", line 100, in pca_and_prune
    verbose=True)
  File "/home/jingtao/anaconda3/envs/lsfm/lib/python3.5/site-packages/lsfm/model.py", line 20, in pca_and_weights
    model.trim_components(retain_eig_cum_val)
  File "/home/jingtao/anaconda3/envs/lsfm/lib/python3.5/site-packages/menpo/model/pca.py", line 568, in trim_components
    self.n_active_components = n_components
  File "/home/jingtao/anaconda3/envs/lsfm/lib/python3.5/site-packages/menpo/model/pca.py", line 209, in n_active_components
    raise ValueError(err_str)
ValueError: Tried setting n_active_components to 0.985 - value needs to be a float 0.0 < n_components < self._total_kept_variance_ratio (nan) or an integer 1 < n_components < self.n_components (0)

Hi, I solved the above error. However, I get into another error. For bin/lsfm, line 78, c = landmark_and_correspond_mesh(mesh, verbose=verbose). Inside this function, mesh object is okay until running line 40: mesh_in_img = camera.apply(aligned_mesh). It seems that I should use specific img_shape for different images. Could you please shed some light on what img_shape is? Since it is not the pixel size of the input image, but a fixed number (320, 240). Thanks

@sjt-moon, hi sorry to to bother you. I notice that you have  passed the step  "orrespondence: [==========] 100% (1/1) - done" where I'm stuck in. maybe you have the key file "balanced_frontal_face_aam_v0_py3.pkl".Can you share,Thank you 😊.

Hi,
I'm sorry I haven't been focus on this topic for a while and I'm afraid I
forget the details. Maybe you can check to see if the variance is NaN. Good
luck.
sjt-moon
fcervon<EMAIL_ADDRESS>于2019年3月12日周二 下午8:04写道：

@sjt-moon https://github.com/sjt-moon, hi sorry to to bother you. I
notice that you have passed the step "orrespondence: [==========] 100%
(1/1) - done" where I'm stuck in. maybe you have the key file
"balanced_frontal_face_aam_v0_py3.pkl".Can you share,Thank you 😊.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/menpo/lsfm/issues/24#issuecomment-472261186, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaxyTA_FRI1a3ASEUPXjT0pr7Lf_48fuks5vWGqlgaJpZM4X4lRz
.

@sjt-moon, hi sorry to to bother you. I notice that you have passed the step "orrespondence: [==========] 100% (1/1) - done" where I'm stuck in. maybe you have the key file "balanced_frontal_face_aam_v0_py3.pkl".Can you share,Thank you .

hello, do you have the key file "balanced_frontal_face_aam_v0_py3.pkl" now?I face the same problem

I've just attempted to fix the server configuration for the static file - can you try the download again @fcervon/@baymin182?

Just one sample to capture PCs?


Fix read() on posix

This is one step toward fixing https://issues.dlang.org/show_bug.cgi?id=14125

Whoa. A 50-line @trusted function?!  There's no way this can possibly be maintainable. Unless we setup github to issue big fat warnings in large red font whenever a PR touches one of those 50 lines, or otherwise refuse to merge anything that touches anything in the function without a thorough review of the code. This is exactly the kind of test case that proves the need for a syntactic construct to mark up unsafe portions of the function body that need to be reviewed and verified by hand. Otherwise, you better believe that after a year or so, there will be nasty memory safety bugs in this code and nobody would know any better.

I agree there is a problem with these tiny nested functions (and I have argued against them before, on various grounds), but I don't think brazenly applying @trusted in a big swath like this is a good solution.
Surely we should be searching for a solution that preserves readability and compiler-enforced safety guarantees?

Yes, a 50 lines @trusted functions that (a) interfaces with the OS, (b) optimizes the bejesus out of memory allocation and system calls, (c) replaces a much larger function that was in fact more difficult to maintain because it introduced unsafety at every step.

Surely we should be searching for a solution that preserves readability and compiler-enforced safety guarantees?

This is it.

I'm pulling this, but I believe that this is a work in progress and still merits another look in regards to how it uses trusted.


please implement inbound get method

There is an endpoint in the api that allows to get an inbound by its id http://localhost:2053/panel/api/inbounds/get/{inboundId}
However it is not present in the library AsyncInboundApi. Please implement that

@beliboba, hello!
Thanks for the suggestion, this method will be added in the next release.
Source docs for ref: https://www.postman.com/hsanaei/3x-ui/request/uu7wm1k/inbound
Sincerely

@beliboba, the PR is attached to the issue and after the merge, it will be automatically closed.
Regards


[Proposal] 5.3 - collections (7.0 Branch)

Updating first function to fix breaking change in 5.3 collections
#322

Also reference #320


fix: unique output check

Fixes the bug of checking the output_velocity boolean twice instead of the output_velocity and output_torque.

Thanks for the fix and sorry for the silly mistake!


Exception: Latex error converting to dvi

I have recently come across an issue with TexMobjects that I cannot seem to fix. I have tried looking through the many other issues created by others, and none of the solutions seemed to help. To briefly describe my goal, I am trying to create a simple animation, briefly going over implicit differentiation, so that I can get used to working with manim.
I have already created the LaTeX equation , which seems to work when put together as one long string, but it fails when I attempt to separate it, which has confused me for quite some time now, as I used an identical method in an earlier animation, and there were no problems:
text = TexMobject(r"\frac{\text{d}", r"\text{y}}", r"{\text{d}", r"\text{x}}")
I am separating the \text{x} and \text{y}, so that I can eventually recolor them, using set_color_by_tex_to_color_map; however, I am confident that the recoloring is not my issue, as I have successfully used it before, and the animation still refuses to render, even when I remove the command to color the equation and just try to get it to appear on the screen. In an attempt to troubleshoot my problems, I created a separate class intended only to get this specific part of my animation working, which I will paste below. The issue remains the same.
class test (Scene):
    def construct(self):
        text = TexMobject(r"\frac{\text{d}", r"\text{y}}", r"{\text{d}", r"\text{x}}")

        self.play(Write(text))

After trying to play this animation, with python -m manim anim.py test -pl, I get the following error:
Media will be stored in ./media\. You can change this behavior by writing a different directory to media_dir.txt.

Traceback (most recent call last):
  File "C:\Users\tulsk\Documents\manim\manimlib\extract_scene.py", line 150, in main
    scene = SceneClass(**scene_kwargs)
  File "C:\Users\tulsk\Documents\manim\manimlib\scene\scene.py", line 52, in __init__
    self.construct()
  File "anim.py", line 405, in construct
    text = TexMobject(r"\frac{\text{d}", r"\text{y}}", r"{\text{d}", r"\text{x}}")
  File "C:\Users\tulsk\Documents\manim\manimlib\mobject\svg\tex_mobject.py", line 146, in __init__
    self.break_up_by_substrings()
  File "C:\Users\tulsk\Documents\manim\manimlib\mobject\svg\tex_mobject.py", line 174, in break_up_by_substrings
    sub_tex_mob = SingleStringTexMobject(tex_string, **self.CONFIG)
  File "C:\Users\tulsk\Documents\manim\manimlib\mobject\svg\tex_mobject.py", line 44, in __init__
    self.template_tex_file_body
  File "C:\Users\tulsk\Documents\manim\manimlib\utils\tex_file_writing.py", line 19, in tex_to_svg_file
    dvi_file = tex_to_dvi(tex_file)
  File "C:\Users\tulsk\Documents\manim\manimlib\utils\tex_file_writing.py", line 67, in tex_to_dvi
    "See log output above or the log file: %s" % log_file)
Exception: Latex error converting to dvi. See log output above or the log file: .\files\Tex\280909e253fde66a.log

I would think that my method of separation should work perfectly fine, as replacing the code with the following works without any errors. I am using the exact same method in this attempt, as I am in the previous one, yet this is the only one that successfully produces an animation:
class test1 (Scene):
    def construct(self):
        text2 = TexMobject(r"\frac{\partial}{\partial", r"\text{x}", r"}")

        self.play(Write(text2))

Command Used: python -m manim anim.py test1 -pl
Other Information:

Python 3.7
Latest version of manim
Windows 10
I am using PyCharm as my editor
I am using MiKTeX, as suggested by the installation tutorial

I am comparatively new to Python; however, I would consider myself knowledgeable enough to understand the basics, but am not quite sure how to fix this issue, or what else I should look for, as I have already tried some of the suggested fixes found in other resolved issues, but nothing seemed to work. It is also entirely possible that I am missing something quite obvious; however, I have tried everything that I can think of and have not yet found a solution.
If anyone could help me solve my issue, and figure out why the first scene is giving me an error, it would be greatly appreciated. Please feel free to let me know if I need to provide any additional information.

Manim works better with "{a","\over","b}"

Thank you so much! After switching to using \over instead of \frac it seems that I am not longer getting any issues, and it is working fine.


Format of blob in column ZDATA in table ZICNOTEDATA

what is the format of this blob column in the notes table structure?
How to convert it to readable text?

Apologies for the delay. Apple's changed the formats for notes recently, so this tool may not be up to date yet.
I'm not sure exactly what the format of this column is yet, but here's some encodings you might want to look into:

binary plist (apple is really fond of these)
utf8 text

I have a few write ups of how to turn this column into the note data here in these articles. It ends up essentially being a protobuf and the bits I have figured out are documented in this protoc. You should be able to compile that to javascript and run with it pretty quickly.
I hope this helps!


ENV.02 deprecated

Installing Datastorm tap on MacMini (M1) using
brew install zeroc-ice/tap/datastorm
getting warning as below - most probably the case to the other formulae too.
==> Installing datastorm from zeroc-ice/tap
==> Downloading https://github.com/zeroc-ice/datastorm/archive/v0.2.0.tar.gz
==> Downloading from https://codeload.github.com/zeroc-ice/datastorm/tar.gz/v0.2
##O#- #
**Warning: Calling ENV.O2 is deprecated! There is no replacement.**
Please report this issue to the zeroc-ice/tap tap (not Homebrew/brew or Homebrew/core):
  /opt/homebrew/Library/Taps/zeroc-ice/homebrew-tap/Formula/datastorm.rb:16

==> make install prefix=/opt/homebrew/Cellar/datastorm/0.2.0 V=1 ICE_BIN_DIST=al
🍺  /opt/homebrew/Cellar/datastorm/0.2.0: 16 files, 1.5MB, built in 19 seconds

Thanks @mreinart

Thanks @mreinart


Models not showing on Ask Question command

I created some custom models, but I can't see them in Ask Question command. While in Conversations, I can use them. This seems to be a bug. Hope you could fix soon.

This bug seems to only happen when there is no selected text, otherwise it works fine.


Add a 64bit version of CDB?

Similar to https://github.com/pcarrier/cdb64?

I just added a 64bit version.
https://github.com/chrislusf/cdb64


[stable/elastalert] How to configure email credentials?

How to configure email credentials (SMTP server) in ElastAlert? The values.yaml file contains a rules section, but nothing on how to specify the email credentials. Apparently, the email credentials have to be set in the email_authentication.yaml . How to do this in the ElastAlert chart?

No response so far @stale bot.


Portfolio growth over time

User Story
As a user, I would like to see my portfolio's growth over time.
Detailed Description
Both backend and frontend are involved:

backend to create a 'snapshotting' feature that keeps track of portfolio values at the end of each time interval (e.g. hourly) for users.
frontend to create a graph over time of the user's portfolio value.

Subtasks (Please also link task PRs here)

[]
[]
[]

There is 2 parts to the backend for this:

create the portfolio growth table, and a cron job that runs at every interval to calculate every user's total portfolio value, and save to this table.
create HTTP route to

For testing, I'm not sure if there's anything to test since you'll be using functions that have been tested alr.


Added blog  "Setting Up Prometheus Alerts with Webex Integration"

Added new blog for "Prometheus Alerts with Webex Integration".

@RichiH would you review this PR ?


Document how to configure shell completion

It would be helpful for users who aren't familiar with shell configuration.

try rye self completion <shell name. eg: bash>
https://github.com/mitsuhiko/rye/blob/952581aa18a2a8947f3b6e847261d4b452806108/rye/src/cli/rye.rs#L52-L58
Hope this can be fix

Or per user (no sudo)
rye self completion > ~/.local/share/bash-completion/completions/rye.bash

try rye self completion bash > /usr/share/bash-completion/completions/rye.bash

This may be slightly incorrect. rye self completion -s bash > (your_shell_completion_path) may be better.
In any case, we would appreciate a clearer indication how to configure shell completion.


yolov8 on RP 3B+

did any one tired yolo v8 on rp 3b+ ??

@dipu0,
If you have an 64-OS, you can use the same code.
Of course, the inference times are longer.


is there any attribute like autoPlay?

In react-slick there is an attribute to automatically play the slideshow like autoplay and autoplayspeed. So is there any attribute like this?

@darrylsepeda there currently isn't but that seems like it could be a pretty nice feature to implement. I can try and add that on but feel free to submit a PR if you'd like.


Add Rahvaalgatus' API.

Hey,
Here's an entry for the new Rahvaalgatus' API. Does that look formatted correctly?
Thanks in advance!

I've updated the files and copied the dataset verbatim to the English directory, too. I also added the Estonian name to the English directory's organization file as I understand it's used for matching with datasets.


Re-enable unexpected_cfgs

This is an attempt to re-enable unexpected_cfgs using the new unexpected_cfgs config in Cargo.toml. This avoids the need for a build script while still checking for typos in global cfgs.

Hm, looks like it isn't enough to just add it to the workspace lint table.

Well that's less bad. 🥲 Thanks Chris!

Oh. Hm. Looks like this is expected on stable. A fix to silence the warnings was backported to beta but not current stable. So it will be an issue until the next release.


Declaring those as private breaks the compilation

[EDIT by guardrex to add the metadata]
https://github.com/dotnet/AspNetCore.Docs/blob/87d7bf6a039e6b6e950e64bb63c7c9817895c5e9/aspnetcore/blazor/file-uploads/samples/5.x/Components/FileUpload.razor#L62

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

ID: 5ea474d5-234a-f393-8609-892a2deb5d2e
Version Independent ID: c11d981c-05af-c19d-a333-feedd5978639
Content: ASP.NET Core Blazor file uploads
Content Source: aspnetcore/blazor/file-uploads.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @moonolgerd ... Best to open issues on the repo from the topic (This page feedback form). No worries ... I'll add the metadata.
I'll get back to you tomorrow on this.

Very strange. IIRC, this code was cut-'n-paste from ...
https://github.com/pranavkm/FileUpload/blob/master/Pages/Index.razor#L95
No worries ... I'll patch that right now.


perf: avoid allocations for enum expressions

LayerPropertyEnum values are all constants, so we can instantiate their expressions ahead of time and avoid allocations when they're used. Also moved these to the expression package
relates to #42

#59  👈 (View in Graphite)
main

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

Good idea! LGTM


[feat] Add ability to specify the compilation database

Add ability to specify the compilation database through clang-format -p xxx.json machinery, so that the head files can be included properly.

closes #41

I've been testing this. It works locally, but I get an error on the test repo using a generated database from CMake on the GitHub runner.
  INFO:CPP Linter:Running "clang-tidy-10 -checks=boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-* --export-fixes=clang_tidy_output.yml -p build src/demo.cpp"
  DEBUG:CPP Linter:Output from clang-tidy:

  WARNING:CPP Linter:clang-tidy-10 raised the following error(s):
  LLVM ERROR: Cannot chdir into "/home/runner/work/test-cpp-linter-action/test-cpp-linter-action/build"!

I'm still investigating if its just a workflow problem. The path seems correct though (duplicated folder names are from git checkout).
It would seem different version's of clang-tidy output something different:
  WARNING:CPP Linter:clang-tidy-12 raised the following error(s):
  LLVM ERROR: Cannot chdir into "/home/runner/work/test-cpp-linter-action/test-cpp-linter-action/build"!
  PLEASE submit a bug report to https://bugs.llvm.org/ and include the crash backtrace.
  Stack dump:
  0.	Program arguments: clang-tidy-12 -checks=boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-* --export-fixes=clang_tidy_output.yml -p=build src/demo.hpp
  Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):
  /lib/x86_64-linux-gnu/libLLVM-12.so.1(_ZN4llvm3sys15PrintStackTraceERNS_11raw_ostreamEi+0x23)[0x7f54b9db8ef3]
  /lib/x86_64-linux-gnu/libLLVM-12.so.1(_ZN4llvm3sys17RunSignalHandlersEv+0x50)[0x7f54b9db7210]
  /lib/x86_64-linux-gnu/libLLVM-12.so.1(+0xbd955f)[0x7f54b9db955f]
  /lib/x86_64-linux-gnu/libpthread.so.0(+0x153c0)[0x7f54c15ef3c0]
  /lib/x86_64-linux-gnu/libc.so.6(gsignal+0xcb)[0x7f54b8ce618b]
  /lib/x86_64-linux-gnu/libc.so.6(abort+0x12b)[0x7f54b8cc5859]
  /lib/x86_64-linux-gnu/libLLVM-12.so.1(+0xb34209)[0x7f54b9d14209]
  /lib/x86_64-linux-gnu/libclang-cpp.so.12(_ZN5clang7tooling9ClangTool3runEPNS0_10ToolActionE+0x10cb)[0x7f54c09d704b]
  clang-tidy-12[0x87c377]
  clang-tidy-12[0x45833d]
  /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf3)[0x7f54b8cc70b3]
  clang-tidy-12[0x45612e]

I also can't seem to get clang-tidy-13 to work, but again, that's not a problem with these changes.

I have a feeling it may have something to do with the docker image's access to runner's File System.

I think its ok to merge this, but I'd really like to get the database option to work when specified. Its causing clang-tidy to crash which we have to ignore because of multiple files. To be clear, it does work as a python package executable:
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-python@v1
      - name: Install clang-tools
        run : sudo apt-get update && sudo apt-get install clang-tidy-${{ matrix.clang-version }} clang-format-${{ matrix.clang-version }}
      - name: Install linter package
        run: python -m pip install git+https://github.com/shenxianpeng/cpp-linter-action/@refs/pull/42/head
      - name: Generate  compiler database
        run: mkdir build && cmake -Bbuild src
      - name: run linter as package
        id: linter
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        run: cpp_linter -s=file -i=build -p=build -V=${{ matrix.clang-version }}

But it doesn't work using:
    steps:
      - uses: actions/checkout@v2
      - name: Generate compilation database
        run: mkdir build && cmake -Bbuild src
      - name: run linter as action
        uses: shenxianpeng/cpp-linter-action@refs/pull/42/head
        id: linter
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          style: file
          ignore: build
          database: build
          version: ${{ matrix.clang-version }}

I think its a File system access thing.

I cannot reproduce the bug you mentioned. See a github run action result.
If you believe it's a bug, you can open another issue and provide a minimal reproducible example.

@DingXuefeng Your example workflow doesn't actually use a generated build/compile_commads.json.
      - name: Generate compilation database
        run: mkdir build && cmake -Bbuild . -DCMAKE_EXPORT_COMPILE_COMMANDS=ON

What happens when there is a database that exists?

Hi, I can reproduce

Hi, I can reproduce what you see now.
The problem is due to the fact that compile_commands.json has absolute command built-in, while in the docker they are mapped to /github/workspace folder. I have no good solution now. Maybe ln -sf $GITHUB_WORKSPACE /github/workspace.
Wondering how other github actions handled this.

Wondering how other github actions handled this.

We can use the built-in env var: GITHUB_WORKSPACE. I've already push that change with consideration that it isn't needed locally (executed when not on a runner).

Well done. thank you both!


[BUG] rich.progress.track not respecting arguments

Describe the bug
rich.progress.track does not appear to respect the refresh_per_second or update_period parameters.
To Reproduce
from rich.progress import track
import time

for step in track(range(1000), refresh_per_second=1):
    time.sleep(0.01)

or
from rich.progress import track
import time

for step in track(range(1000), update_period=1):
    time.sleep(0.01)

both produce output much more frequently than once/second.
Platform
This is on Python 3.7.4 on MacOS. First noticed in a Jupyter notebook, and verified in an interactive Python session.

I can't reproduce any problem with progress.track with Rich 7.1.0
Auto refresh doesn't work with Jupyter, and it would default to refreshing once per loop.
In the terminal it is refresh_per_second argument that determines how often the progress bar is drawn, and AFAICT that works as expected.
The update_period doesn't affect refresh rate, so in your second example it would refresh at the default 10 times a second, although the tasks will only be updated once a second. You can probably leave update_period as the default for most purposes.
Let me know if you can identify anything that doesn't fit with the above.

Thanks for looking into this and explaining the distinction between refresh rate, and task update rate! I think I had an incorrect understanding of the parameters.
For my Jupyter usage, I'll just switch to explicitly using a Progress() object instead of using track() when I can't afford to refresh every loop.


POLYGON omits isolated purely-horizontal or purely-vertical line segments

Since Ferret v7.13 dated 7/6/2017, the POLYGON command no longer renders isolated purely-horizontal or purely-vertical line segments:
 	NOAA/PMEL TMAP
 	FERRET v7.44 (optimized)
 	Linux 2.6.32-696.30.1.el6.x86_64 64-bit - 12/10/18
 	20-Mar-19 15:40

yes? let xpts = {2,8,,2,8,,2,2.1,,8,8}
yes? let ypts = {2,2.1,,8,8,,2,8,,2,8}
yes? poly/hl=0:10/vl=0:10/line=14 xpts, ypts
yes? plot/ov/vs/line=16 xpts+1, ypts+1

The bug is in both the latest Ferret and PyFerret.  It was not in v7.12 dated 5/16/2017, which produced this:

This bug is messing with our longstanding utility scripts that use POLYGON to produce lines and/or shapes.

This behavior changes between Ferret v7.1 and v7.2

Between 7.12 and 7.13.

This comes from the fix for PyFerret bug #1814, where a filled polygon with zero extent causes the graphics call to hang.  It turns out that that check needs to be applied only for FILLED polygons, so it's simple enough to change the test and let the lines be drawn, skipping over the degenerate filled polygons.


arch/x86_64:Configure arch delay using CONFIG_ALARM_ARCH

Summary
Configure arch delay using CONFIG_ALARM_ARCH
Impact
no impact
Testing
ostest

[Experimental Bot, please feedback here]
No, this PR does not meet the NuttX requirements.
Missing Information:

Detailed explanation of the change:

Why is this change necessary? Is there a bug, performance issue, or a need for a new feature?
What exactly does CONFIG_ALARM_ARCH do?
How does configuring it impact the system?

Specific testing details:

Build Host:  Provide details about your build environment (OS, CPU, compiler version).
Target(s): Specify the architectures and boards you tested on.
Testing logs:  "ostest" is not sufficient. Include relevant log snippets demonstrating the issue before the change and the improvement after the change.

Recommendations:

Expand the Summary: Provide a clear and concise explanation of the problem this PR solves or the enhancement it introduces.
Elaborate on the Impact: Even if there is no user-facing impact, explain why. For example, "This change is internal to the alarm driver and does not affect the public API."
Provide Specific Testing Details:  Include the required details about your build and target environments, along with relevant log excerpts.

Example for Improved Testing Section:
## Testing

I confirm that changes are verified on my local setup and work as intended:

* **Build Host:** Ubuntu 20.04, Intel Core i7, GCC 9.4.0
* **Target(s):**
    * `sim:nsh`
    * `stm32f4discovery:nsh`

**Testing logs before change:**

(Logs showing the incorrect or inefficient behavior before the change)

**Testing logs after change:**

(Logs demonstrating the corrected or improved behavior)


Fix build on ROS2 Humble

Fix for missing header file.
botanbot_sim/botanbot_gui/include/botanbot_gui/control_plugin.hpp:151:30: error: ‘Buffer’ is not a member of ‘tf2_ros’

can you please target the PR to humble branch that I just created


Upgrade opencore and all kexts

Do you have an update for the opencore and kexts?  I did update my machine but when updating WhateverGreen the brightness stops working, you know what it can be?

It's on my todo list, I got some time sensitive jobs this year and couldn't afford to spend time without my machine in case something happens during the update. I think I'll update by the end of the month. Newer versions of OpenCore changed the boot process and need some tweaks. You can try updating other kexts alongside whatevergreen. At least Lilu (whatevergreen is a Lilu plugin) and VirtualSMC. Let me know if it works!
When Acidanthera release OpenCore 0.6.9 (probably first monday of may) I'm going to catch up with the releases and update this repo.

Great news, I will follow this update.

Repo updated with latest OpenCore and kexts, enjoy!


Add new class to handle snapshot for colocated tables

This ticket requires addition of a new class which will handle snapshot for colocated tables. The general change is that instead of looking up checkpoints or sourceInfo based on tabletId use a lookup key of the format tableId.tabletId

Upon discussion, it was found that this approach is a little change heavy and prone to certain errors.
We are moving ahead with another approach which will change the way all the OpIds are stored. Currently they are stored and can be keyed by the tabletId but with the new change discussed, they will be keyed by the combination tableId.tabletId. The new changes are tracked with the issue #194


Add format tasks (api and app) to the gruntfile

Coming from #1423.
This PR adds a new task named fmt with an optional argument. This argument is interpreted as follows:

===api, format content of api/ with gofmt -w.
===app. format content of app/ with eslint --fix.
Any other string or undefined value executes both, app first and api after that.

As a result, any user can run grunt fmt (or grunt fmt:api and/or grunt fmt:app) to have the codebase automatically formated. Note that, on top of fixing some syntax errors, eslint will still throw around 20 "fixed" errors. A note should be added to docs for new contributors to be aware of it.

Adding the --fix option makes eslint overwrite the files which are analyzed, in order to fix issues. At now, gruntfile is included in the list of files to check. It is not convenient to make grunt trigger a script which might be indirectly overwritten on runtime. To avoid this, the existing eslint configuration in the gruntfile is renamed to eslint:nofix. A second one is added, named eslint:fix, which enables overwritting and removes the gruntfile from the list.
Then, existing tasks, such as default and lint, now call eslint:nofix. The new configuration, eslint:fix is only called from fmt.

Another commit adds Code Climate CLI to the gruntfile. It is a single task which takes any argument and passes it to a codemirror container:

grunt shell:codeclimate:help: see available commands
grunt shell:codeclimate:validate-config: validate .codeclimate.yml
grunt shell:codeclimate:engines\:install: install (pull) all the required complementary images to run the tests listed in .codeclimate.yml
grunt shell:codeclimate:analyze: run all the engines/plugins listed in .codeclimate.yml

This allows any developer to run locally exactly the same tests which are executed when commits are pushed to a PR: gofmt, golint, govet, csslint, structure, duplication, eslint and fixme.

A third commit renames .eslintrc.yml to .eslintrc_default.yml. Both, gruntfile.yml and .codeclimate.yml are adapted to use it.
This allows contributors to use their preferred configuration, as long as they run grunt fmt prior to committing.

The two remaining commits update eslint to version 4.11.0, gruntfity-eslint to 4.0.0 and grunt to 1.0.1.

A minor modification included in this PR is to move the inner block in the "vendor" task to a separate function named checkVendorList. This was done to get rid of a complexity issue.

The .codeclimate.yml is updated to version 2, which implies to:

Rename engines to plugins.
Rename exclude_paths to exclude_patterns.
Comment ratings because it is deprecated.

All these changes were suggested by the codeclimate cli.

node_modules/ and bower_components/ are added to "exclude_patterns" because they will exist in a local development repo.

Hmm, I'm ok with the added gofmt for the backend, but eslint is more related to linting than formatting the codebase ? We already have grunt lint for that.

There is no "lint" executable/application/binary in grunt which is diferent from eslint. It is just an alias: https://github.com/portainer/portainer/blob/develop/gruntfile.js#L54


Upgrade to the latest cryptography

Automated message from Dropbox CLA bot
@alex, it looks like you've already signed the Dropbox CLA. Thanks!

There is a failing frontend test, but I can't imagine it's relevant.

Yeah we have a frontend test flake issue right new.  Will review tomorrow.
-Tim Abbott (mobile)
On Jul 14, 2016 4:24 PM, "Alex Gaynor"<EMAIL_ADDRESS>wrote:

There is a failing frontend test, but I can't imagine it's relevant.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/1285#issuecomment-232820904, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACnm2spGWFkXfVD0nHtYZgmbTcrA7zxPks5qVsU0gaJpZM4JM79t
.

Looks great, merged!  A few notes:

Just a bit of history: we've got a good number of Python dependencies that are currently pinned to the versions in Ubuntu Trusty (since we were getting Python deps from apt in production).  We just did a transition to using a virtualenv in production (it used to be used only in development), so now we're free to upgrade the versions of ~everything.
We use a rebase-based workflow where we merge individual commits, so you should amend/squash fixup commits (I took care of this for you here).

Noted re:rebasing!


chore: Update CLI dependencies

Closes #
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Update CLI dependencies, extracted from https://github.com/t3-oss/create-t3-app/pull/1226#pullrequestreview-1306042302.

I removed the changeset since there's no point making a release just for version bumps.


[Office add-ins march 2022] Remove consuming graph services demo

Please don't merge until all reviews are in!
Category

[x ] Content fix
[ ] New article
[] Example checked item (delete this line)

Contents of the Pull Request
Adds note that video of the "Consuming Graph Services using SSO and CORS from event–based Outlook add–ins" demo is removed due to a security risk we discovered in the demo.

@davidchesnut So sorry it took me until 6 PM today. I assume you want this to go live, so I am going to go ahead and publish it for you. Hope that's OK!


Several NuGet curations

I've integrated the changes into https://github.com/oss-review-toolkit/ort-config/pull/225.


Ng new command throwing error

Bug Report or Feature Request (mark with an x)
- [ x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [x ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

node --version
v10.13.0
npm --version
6.4.1
ng --version
Angular CLI: 7.0.5
Node: 10.13.0
OS: win32 x64
Angular:
...
Package                      Version
@angular-devkit/architect    0.10.5
@angular-devkit/core         7.0.5
@angular-devkit/schematics   7.0.5
@schematics/angular          7.0.5
@schematics/update           0.10.5
rxjs                         6.3.3
typescript                   3.1.6
OS- Windows 7
Repro steps

I ran the "ng new myworkspace" command
The log given by the failure

I got the following log in command prompt terminal-
CREATE myworkspace/e2e/src/app.po.ts (208 bytes)
events.js:167
throw er; // Unhandled 'error' event
^
Error: spawn C:\Windows\system32\cmd.exe ENOENT
at Process.ChildProcess._handle.onexit (internal/child_process.js:232:19)
at onErrorNT (internal/child_process.js:407:16)
at process._tickCallback (internal/process/next_tick.js:63:19)
Emitted 'error' event at:
at Process.ChildProcess._handle.onexit (internal/child_process.js:238:12)
at onErrorNT (internal/child_process.js:407:16)
at process._tickCallback (internal/process/next_tick.js:63:19)
Desired functionality

I would like to create a new workspace/app
Mention any other details that might be useful

None in mind for now

Duplicate of #12670

Same Error as Above.
Kindly help to fix this issue.


D3v5 compatibility

I am trying to use d3v5 dagre-d3 and d3-context-menu .
"d3": "^5.16.0",
"d3-context-menu": "^1.1.2",
"dagre-d3": "^0.6.4",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-scripts": "3.4.1"

d3-context-menu.dew.js:122 Uncaught TypeError: d3.selectAll is not a function
at SVGGElement. (VM990 d3-context-menu.dew.js:122)
at SVGGElement. (VM1310 d3-selection.dew.js:722)
(anonymous) @ d3-context-menu.dew.js:122
(anonymous) @ d3-selection.dew.js:722
Here is the plunker to reproduce the problem.
https://plnkr.co/edit/t4CKN9IaBvO35uKd
Please let me know if you require other info.

I haven't used D3 version 5 yet. Poking around a bit, it looks like the library would need to be changed to export the anonymous function in https://github.com/patorjk/d3-context-menu/blob/master/js/d3-context-menu.js rather auto-executing it with the this object (which for most cases was the window object). Then when imported the function would simply need to be invoked with the d3 object.
If you're interested in doing this I'd accept a PR. If not I can tackle this, but I'm not sure on the time frame.

I will give it a try and create PR .

I can use the latest version (1.1.2) with no issues with D3 5.16.0.
However D3 6.0 is out now, which has breaking changes in the event handling logics, and this plugin no longer works. I'll open a separate issue for that.

Fixed in latest release.

Fixed in latest release.

I know this is marked as closed, but I'm now getting a different error related to the same stuff:
"d3": "^5.16.0",
"d3-context-menu": "^2.1.0",
"dagre-d3": "^0.6.4",

I can run
import * as d3 from 'd3';
import * as d3cm from 'd3-context-menu';
...
.on("contextmenu", d3cm.contextMenu(node_events["contextmenu"]));

And i get the error d3_context_menu__WEBPACK_IMPORTED_MODULE_7__.contextMenu is not a function

TypeError: Cannot read property 'pageX' of null is still an issue
"d3": "^5.16.0",
"d3-context-menu": "^2.1.0",


chore: add sonarjs eslint plugin

Overview

Added the SonarJS eslint plugin and disabled failing rules. Tickets will be opened to re-enable the rules and resolve the findings.
Added an ignore for JIL code and tests and removed rule overrides that only failed on JIL code.
Related Issue(s)

https://issues.newrelic.com/browse/NR-99995
Testing

Codecov Report

Merging #463 (dafc4ad) into main (04d8de3) will increase coverage by 3.40%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##            main    #463      +/-   ##
========================================
+ Coverage   3.54%   6.94%   +3.40%
========================================
  Files        119     119
  Lines       3979    3974       -5
  Branches     989     988       -1
========================================
+ Hits         141     276     +135
+ Misses      3026    2920     -106
+ Partials     812     778      -34

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


Sync: CharacterTraits Fields

Description

Sync the values in the Synced list below.
Create an end to end (E2E) tests for all fields (1 for server changing the value and 1 for client changing the value if applicable)
Create a command to test changing each value in game see similar command

Also see additional information at the bottom for more information and templates

Key
Server side only (only allow the running of the function on the server side)
Client side only (only allow the running of the function on the client side)
Client side sync (request the server to change the value, server then changes the value and replies to the client allowing the change)
Server side sync (only server allows running of the function and send to all clients the value changed and clients update the value/call original function)
Synced

Method/Field Name
Sync Type
Additional Information

TODO
TODO
TODO

Deferred

None
Externally Deferred

None
Non-Synced

None
Intended Design

For the patch itself please refer to https://github.com/Bannerlord-Coop-Team/BannerlordCoop/wiki/Patching-Basics#field-patching
Field Sync Dataflow

Integration Testing

Location

Create a branch based from development
Related Issues

Blocked by:

#955

Requirements

N/A
Additional information

Using the GameInterface Service Tempale create commands to test the synced values.
Definition of Done

[ ] Class level comments exist for all new classes.
[ ] XUnit E2E tests exist for every property on the server (and client if applicable)
[ ] Each field has a command getter and setter that used the correct intercept

Only a const value


Add Contract Pack: Defense Combat Advisors from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Tazius, to add Contract Pack: Defense Combat Advisors to CKAN.
Mod details:
name = /mod/1090/Contract%20Pack:%20Defense%20Combat%20Advisors
author = Tazius
abstract = DCA is a BDArmory military contract pack for Contract Configurator
license = CC-BY-SA 2.0 https://creativecommons.org/licenses/by-sa/2.0/
Homepage = http://forum.kerbalspaceprogram.com/index.php?/topic/152496-121-contract-pack-defense-combat-advisors-dca/
description = Defense Combat Advisors [DCA]
DCA is a BDArmory military contract pack for Contract Configurator
DCA contains 17 missions of enemies from the Air, Land & Sea.
Requirements:
**Contract Configurator: **
https://github.com/jrossignol/ContractConfigurator/releases
**BDArmory: **
https://github.com/PapaJoesSoup/BDArmory/releases
Possible Mod Dependencies:
MechJeb 2
Mechjeb and Engineer for all!
Download on SpaceDock & CKAN:
http://spacedock.info/mod/1088/Defense Combat Advisors
Installation:
Extract Zip File
Drop GameData folder from either Legacy Aim Mode or Default Aim Mod folder into the KSP installation directory.
May need to launch game and open contract configurator and select DCA contract, may need to turn off other contracts to make DCA contracts visible.
Please direct questions about this pull request to Tazius.

Broke it into two mods since it ships two different playstyles split into seperate folders.
I did what I could to recombobulate the mess I did with the initial installationstanzas and I think I got it but someone should probably look through it... The .zip has a GameData root which splits into 2 separate modes which both have their own GameData roots in them.

LGTM


npm start not working on angular seed !

/AngularJSSTB/node_modules/.bin/bower: line 1: ../bower/bin/bower: No such file or directory
npm ERR! Darwin 14.1.1
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.12.3
npm ERR! npm  v2.9.1
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>postinstall: bower install
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'bower install'.
npm ERR! This is most likely a problem with the angular-seed package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     bower install
npm ERR! You can get their info via:
npm ERR!     npm owner ls angular-seed
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/harsha/projects/workspace/myprojects/AngularJSSTB/npm-debug.log

Getting the same error. Did you ever fix it?

This fixed it for me
$ git config --global url."https://".insteadOf git://

@harshattray  did that fix the error for you?

Yes it did .. thank you

No problem! Want to follow me back on Github? :)


fix: log dialog testids, css selectors fixes, and package creator modal

Not sure what is going with the other PR, trying to isolate by splitting up further.
Will update the other depending PR's if this succeeds and is merged.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rdar://problem/26380562 SILGen: Properly lower types of ObjC generic methods dispatched off AnyObject.

When an ObjC generic method is found by AnyObject dispatch, we don't have any type information to bind generic parameter dependencies. Sema expands these generic parameters to their upper bounds in an AnyObject dispatch. However, SILGen was still lowering the type of a dynamic method invocation from the method's formal type, expecting its generic parameters to be bound by substitutions provided from a call. Lower dynamic method calls using the substituted type from the AST instead to avoid this. Fixes rdar://problem/26380562.

@swift-ci test

@DougGregor Do you mind reviewing this?

@swift-ci test linux

Linux failure looks unrelated.  Merging.