Add required prop to Dropdown

As a Dropdown can easily be used in a form it makes sense to pass the required prop down to the <input> for HTML5 validation (browser will complain if its value is empty.)
I couldn't get the test working but I believe I wrote it correctly, any pointers? The test:watch doesn't seem to be doing what it's supposed to... :open_mouth:
This PR is still incomplete though as when we select an item the <input> value is emptied. Any ideas on working around that?

(I need to fix the commit message as well.)

Upon further inspection this would require an <input> element for a non-search Dropdown as well.

The Dropdown has no formal browser compatible form control (i.e. input element).  Because of this, there is no input for the browser to validate.
The input that this PR touches is the input that takes a search query, not the input that holds the value.  That is because the value is stored in state only.
This is not the case with other controls, such as an <Input /> which does have a valid browser form control that we can pass props such as required to.
The correct solution here is going to be implementing a form validation solution.  See #407 for several options.


Code cannot be saved in the webpage

I finished the question 2 (compiled successfully and got tests passed) and moved forward to question 3. When I went back to question 2 (click "2" in the process bar) to review the code, the code has been reset and my code was not saved.

Definitely an issue we've had -- how would you propose fixing this?

Can we use cookie to do this, since we don't have concept of server and session? @eyurko

Can we just let chrome or any other browser to cache the code?

Thank you for reply! I do not have specific details yet but the basic two ideas is to use backend database or web cookie. Thanks! @eyurko

@luofei11 we do keep some concept of a session -- check the TranscriptObjectFactory / SnapshotObjectFactory.
Also, this is a duplicate of #54.

Copying comments from #54 and closing it since this thread seems to have progressed farther. Noting @eyurko's comment from there, too:

Generally, we'd like to be able to preserve student code on a problem if they navigate between questions or if they refresh the page, but we preprocess the code before saving it in CodeEvalResult. This suggests that it would be worth saving the unprocessed, raw student code as part of the snapshot as well, otherwise we'd have to reverse the preprocess step, which would be needlessly expensive.

My suggestion would be to use localStorage/sessionStorage, since we don't have a backend database. I recommend coming up with a brief technical description of the proposed solution (what namespace are you going to use, for example) before trying to implement it.

@seanlip Can you please assign me this issue or add me to the collaborator since I am working with Kaikai @sniffsky on this issue together.


Use ClientLogger in testing output

Description
Exploratory PR where many System.out, System.err, and Throwable.printStackTrace calls are replaced with logging using ClientLogger. This should reduce noise seen in CI pipelines and push that information into log files which are more durable and more easily parsed when troubleshooting.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

A few formatting error in Analyze CI, otherwise LGTM on ARM libs.
Is there any easy practice when running local (where we may prefer seeing the log on stdout etc.)?

Right now, all libraries when running tests will log to target/test.log at a verbose level for troubleshooting


Lkas error remote start

Describe the bug
If I remote start my jeep I get a lkas error
Provide a route where the issue occurs
5c77528ab6bf019a|2023-03-01--17-34-37--0
openpilot version
0.8.15
Additional info
No response

Double post by accident please.close this.


Can't get example to work

I have been trying all morning to get the example 'server.py' to work. I followed the instructions per the README.md file as follows:
pip install fbmq
cd example
virtualenv env
source env/bin/activate
pip install -r requirements.txt
python server.py
I am on Ubuntu 14.04 using python2.7
I get the following error:
env) michaels@michaels-ThinkPad-W520:~/work/python/plugins/fbmq/example$ python server.py
Traceback (most recent call last):
File "server.py", line 8, in
import example.messenger
ImportError: No module named example.messenger

@xicore
You did it right. I don't know why it doesn't work.
Try below another way
cd example
touch __init__.py
python server.py

The source code imports from 'example.messenger' but the project root is already in 'example' I also see that in previous versions of this example the path 'example.' is not used.

ok. it works now with the touch init.py
it seems that this is necessary as when I do 'pip install fbmq' it also attempts to install the modules in the example directory. Without the init.py Python does not install and that was the problem.
This is a bug that must be fixed. Just add an empty init.py in examples and we're done.
Thanks

@xicore Thanks. fixed it

Hi Karl,
I am trying to work through the server example, which seems awesome. I am
new to Python and Facebook apps and am having difficulty getting the
various server examples to run. What do I need to do to see the various
menus, icons and buttons appear on my messenger page. I have my bot app
validated and I am getting and sending text messages.
BR
On Mon, Mar 13, 2017 at 2:02 AM, Karl Woojung Kim<EMAIL_ADDRESS>wrote:

Closed #20 https://github.com/conbus/fbmq/issues/20.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/conbus/fbmq/issues/20#event-996521636, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFdWpulSaFdWfucsQlwWGI0AeNv4r8p_ks5rlIengaJpZM4MZOl8
.


Add support for MHLO scatter op lowering to LinalgExt when the indices have an implicit trailing dim

Fix for issue 9669

What's the standard for adding tests for mhlo -> LinalgExt? I couldn't find a dedicated tests folder @rsuderman

What's the standard for adding tests for mhlo -> LinalgExt? I couldn't find a dedicated tests folder @rsuderman

mhlo -> LinalgExt lit tests are in https://github.com/iree-org/iree/blob/main/compiler/src/iree/compiler/InputConversion/MHLO/test/convert_mhlo_to_linalg_ext.mlir

Please remember to use "squash and merge" when the commit history on your branch is messy or not interesting. Things like "fix comment" and "Merge branch 'iree-org:main' into main" are now in the permanent git history :-/


Vendor in latest go-selinux

This should fix the issue with iptables being denied execution on
container cleanup
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

Hopefully this will finally fix https://github.com/projectatomic/libpod/issues/923

I like vendor changes, so easy to review. LGTM once tests green.

@rh-atomic-bot r=mheon


Better way to update document without ID selector

Hello,
When updating a document that you don't have it's ID and when it's simulated on the client it throw an error
var commentId = 1;
var post = { _id: 2};

Comment.update({
        _id: {$ne: commentId},
        postId: post._id,
        pinned: true,
      }, { $set: {pinned: false} }, {multi: true});

What would be the best way to update documents directly without having their ids.
Error in console
Exception while simulating the effect of invoking 'Comment.togglePin'
errorClass {error: 403, reason: "Not permitted. Untrusted code may only update documents by ID.", details: undefined, message: "Not permitted. Untrusted code may only update documents by ID. [403]", errorType: "Meteor.Error"}

Thanks :)

It's not a problem with Astronomy. It's how Meteor works. In principle it's not allowed to updated and remove documents on the client by anything but ID.

Oh Thank.


Historical Reference: DOS 5 upgrade marketing video

This video is worth a watch if you're into quirky corporate videos
https://www.youtube.com/watch?v=dmEvPZUdAVI

Thanks, but closing since this isn't an issue and requires no further action


TracerPreparedStatement.executeBatch() is failing.

First, let’s revisit the interface/class hierarchy:

java.sql.PreparedStatement extends java.sql.Statement
io.opentracing.contrib.jdbc.TracingStatement implements java.sql.Statement
io.opentracing.contrib.jdbc.TracingPreparedStatement implements java.sql.PreparedStatement
io.opentracing.contrib.jdbc.TracingPreparedStatement extends io.opentracing.contrib.jdbc.TracingStatement

Some fact:

executeBatch() method is defined in java.sql.Statement interface. And io.opentracing.contrib.jdbc.TracingStatement class has implemented this method.
Both TracingPreparedStatement and TracingStatement class have their independent Tracer variable (instance level).

Now, let’s look at the below piece of code. It adds 10 records to a batch and send all 10 records to the database at one go. (Read the inline comments for better understanding) :
// Get the Connection object, which already has the Tracer object in it.
// getConnection() object is an utility method to return a TracingConnection with a valid Tracer object in it. (I have not given the implementation here).
// This connection (conn) is of type io.opentracing.contrib.jdbc.TracingConnection.
Connection conn = getConnection();
// The same Tracer object will be passed internally to the newly created PreparedStatement (pstmt) object.
// This PreparedStatement is nothing but of type io.opentracing.contrib.jdbc.TracingPreparedStatement.
PreparedStatement pstmt = conn.prepareStatement(“”);
// Start the batch operation.
for (int i = 0; i < 10; i ++) {
pstmt.setXXX(…);
pstmt.setXXX(…);
pstmt.setXXX(…);
pstmt.addBatch();

}
// It calls io.opentracing.contrib.jdbc.TracingStatement.executeBatch() [note fact #1].
// Because TracingStatement has its own copy of Tracer object [note fact #2], which is
// still in uninitialized state, the call fails.
int[] count = pstmt.executeBatch();
The last line pstmt.executeBatch() is failing, because it's Tracer object is null.

Because TracingPreparedStatement is extending TracingStatement, I feel they should share the same Tracer object defined in the TracingStatement. Having a separate Tracer object in the child class (TracingPreparedStatement) is causing the issue.

TracingConnection passes owned instance of tracer to TracingPreparedStatement and TracingStatement.
So they all have the same tracer object.

Could you provide JUnit test for your case (like in https://github.com/opentracing-contrib/java-jdbc/blob/master/src/test/java/io/opentracing/contrib/jdbc/JdbcTest.java) or sample application?

I created PR #38 based on what you described, actually you are right, fields of TracingStatement should be reused. Although I still don't understand why executeBatch() is failing for you.

Note that, we are not using GlobalTracer to store our platform Tracer instance.
Before we go into the test case, let me try to explain it further. Consider the below 3 classes:

public class Base {
private Tracer tracer;
public Base(Tracer tracer) {
    this.tracer = tracer;
}

public void executeBatch() {
    System.out.println("Tracer Object: " + tracer);
}

}

public class Child extends Base {
private Tracer tracer;
public Child(Tracer tracer) {
    this.tracer = tracer;
}

}

public class Moderator {
private Tracer tracer;
public void setTracer(Tracer tracer) {
    this.tracer = tracer;
}

public Child prepareChild() {
    return new Child(tracer);
}

}

Here is the Main class:
public class Main {
public static void main(String[] args) {
Tracer tracer = new Tracer();
Moderator m = new Moderator();
m.setTracer(tracer);
    // The below call will pass the Tracer object to the Child object.
    // So the "child" now has valid non-null tracer object.
    Child child = m.prepareChild();

    // Below call goes to Base::executeBatch() method, and because Base class has it's "local"
    // tracer instance, which is still null, the call fails.
    child.executeBatch();
}

}
Proposal:
May be we should change the constructor of Child class to below:
[ It should NOT maintain it's own local copy of Tracer object ]
public Child(Tracer tracer) {
super(tracer);
}

Here is the unit test case:
public void testBatchInsert() {
Connection conn = null;
PreparedStatement pstmt = null;
Tracer tracer = null;
try {
    tracer = getTracer();    // Utility method to return platform tracer object.

    Class.forName("io.opentracing.contrib.jdbc.TracingDriver");

    // Now, try setting the tracer instance.
    for (Enumeration<Driver> enm = DriverManager.getDrivers(); enm.hasMoreElements(); ) {
        Driver driver = enm.nextElement();
        if (driver instanceof ELFTracingDriver) {
            ((TracingDriver)driver).setTracer(tracer);
        }
    }
    conn = DriverManager.getConnection("jdbc:postgresql://localhost:5432/postgres");

    pstmt = conn.prepareStatement("INSERT INTO employees (emp_id, emp_name, location, age) VALUES (?, ?, ?, ?)");

    for (int i = 1; i <= 10; i ++) {
        pstmt.setInt(1, i);
        pstmt.setString(2, "Name-" + i);
        pstmt.setString(3, "HV-" + i);
        pstmt.setInt(4, i * 15);

        pstmt.addBatch();
    }
    int[] count = pstmt.executeBatch();    // FAILS here.
    conn.commit();
    Assert.assertEquals("Must insert two employees", 10, count.length);
} catch (Exception e) {
   try {
        getConnection().rollback();
    } catch (SQLException ex) {
        // Do Nothing
    }
} finally {
    close(pstmt, conn);
}

}

thanks @sudiptasish , got it.
So my merged PR #38 should fix the problem now.
Let me just release a new version.

0.1.3 version released


Damage class

The player and enemies will each have a Damage instance, which encapsulates taking/dealing damage, and death (more precisely, being in a "dead" state which the owner will decide what to do on: player will trigger a game over, enemy will trigger destroy animation and award points).

#6


getting-started: add references to interesting repos in the org

This PR adds references to some interesting repos in the org, which might go unnoticed:

sphinxcontrib-hdl-diagrams
yosys-symbiflow-plugins
symbiflow-bitstream-viewer
fasm
fpga-tool-perf

Moved to f4pga/f4pga-website#2.


Renter screen crashes when trying to upload files without first buying storage

A 3gb upload was already in progress, nearly finished. I tried to upload another large file without first buying extra storage, to see if i would be prompted or whether it would be allowed. It did allow me to go through the process of selecting the file, but then the renter screen went completely black and the UI became unresponsive. I closed the UI and reopened with the same result. The wallet no longer unlocks.
I can still unlock the wallet and see the files/hosting in an earlier version, but the rented files all show as incomplete uploads.
I've tried a fresh install of 1.0.3rc but it still will not unlock, just sits for hours on the unlocking message.
I'll leave it like that all day to see if it has unlocked by tonight.

ok i've made a bit of progress with this that may help, just to summarize:
i had already uploaded some files and a folder to the renter, which worked ok.
i had started an upload of 3gb which was in progress, reducing my available storage to 10gb
i then tried to upload a 14gb folder without buying more storage, to see if i would be prompted.
After accepting the folder without prompting, the UI crashed to a black screen and became unresponsive and would not unlock.
I've  deleted the files and folder from the renter and the UI now unlocks and is usable again.
so there seems to be a hole that needs patching around a prompt for extra storage space needed if not enough has been purchased.

You mentioned that you uploaded the UI folder, which has lots and lots of files in it. I'm wondering if you ran into issues because either the UI or siad itself was unable to correctly process so many files.
That hunch is pretty easy to test, we should have more information soon.

I've tested it, i've tried to upload a very large folder while another is uploading. This time i have bought enough storage so that is not the issue. The same black screen has appeared, i'll leave it like that and see if it eventually clears.

The renter is still working and uploading the first folder, so although the screen is black it is still working. Hopefully when the first folder finishes uploading it will clear and start the next one.

@johnathanhowell perhaps the UI is blocking somewhere (an API call?) and never reaching the render code?

Update: when the first folder finished uploading, the screen still remains black. I restarted the UI and the wallet will not unlock, the renter/host do not show any files. The only way to get it working again is to delete all uploaded folders in the renter file. Single uploaded files do not affect it however, only folders. It may be a problem with folder uploads rather than files?

We have so far been unable to reproduce the problem.
Maybe there's a deadlock that's triggering somewhere in the renter? That would the UI to block. I agree with @lukechampine the next step is to figure out if there's something that can cause the UI to fail to hit the rendering part of the code.

This was fixed with https://github.com/NebulousLabs/Nodejs-Sia/pull/37. @Chehands, please re-open this issue if you encounter this problem again.


[Backport 1.x]Change JNA to upstream version from elastic JNA build

Signed-off-by: Xue Zhou<EMAIL_ADDRESS>Description
Backport #1003
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@xuezhou25 looks like this did not make it to 1.2. Was that by choice?
Was running into JarHell because of different JNA versions being pulled.
Ref: https://github.com/opensearch-project/OpenSearch/issues/1240

@xuezhou25 looks like this did not make it to 1.2. Was that by choice? Was running into JarHell because of different JNA versions being pulled. Ref: #1240

Hi Sarat, I think this was not intentionally, I merged this PR on that branch-cutting day.
I thought it was merged before the 1.2 branch being cut, because it was before the version bump from 1.2 to 1.3 on 1.x branch (https://github.com/opensearch-project/OpenSearch/commits/ce7b091da6ae50bbc3c015a78b7611923475386c).
But I might be wrong and not noticed the 1.2 branch has already been cut at that time. 🙁

@xuezhou25 looks like this did not make it to 1.2. Was that by choice? Was running into JarHell because of different JNA versions being pulled. Ref: #1240

Hi Sarat, I think this was not intentionally, I merged this PR on that branch-cutting day. I thought it was merged before the 1.2 branch being cut, because it was before the version bump from 1.2 to 1.3 on 1.x branch (https://github.com/opensearch-project/OpenSearch/commits/ce7b091da6ae50bbc3c015a78b7611923475386c). But I might be wrong and not noticed the 1.2 branch has already been cut at that time. 🙁

Thanks for that @tlfeng I did get past the problem. Its not a blocker.
Unless this is important i think its better to hold off to do another backport to 1.2.


Meal API doesnt work in singleplayer and crashes on server (?)

Mod Name
Meal API
Mod Homepage
https://modrinth.com/mod/mealapi
Minecraft version
1.20.1
Describe the bug
In singleplayer, the game loads but meal api does not seem to work. When attempted to load on a server, it crashes.
Steps to reproduce
Singleplayer;

Load into game with meal api and a mod which utilizes it (in this case i am using Meal Api Addon)
attempt to gain fullness (meal api's additional food meter). in this case with meal api addon i attempted this using the saturation effect, which in a fabric only environment quickly fills up the fullness meter
observe fullness meter does not fill at all

Server;
Saw in #1016 that it crashes on the server, but was unable to replicate as i couldnt get the forge server installed ,-, this issue was closed due to being invalid but was quickly responded to with a valid entry however has remained closed for 2 weeks hence why im posting this slightly different issue here instead of in that issue.
Logs
https://pastebin.com/jZ1f6gcP - singleplayer
Additional context
Singleplayer used the following mods;
Forge 47.1.3
Sinytra 1.0.0-Beta43
Forgified Fabric Api 0.92.1+1.11.7
Meal Api 0.3.2
Meal Api Addon 1.0.1

id like to know why it posted as i closed google chrome to restart my pc but ok

should be good now

As our LTS support target, 1.20.1 now only receives critical fixes related to Connector bugs. Please upgrade to the latest version for mod compatibility support.


Feature Request: Enable Prompt on the next line ala Spaceship

Great theme, the asynchronous git status is a godsend.
See title! Would really love this. I have a little workaround by adding \n to the symbol, but this breaks sometimes with error messages etc.

Thank you @boris-kuz. Is this a dupe of #21?

Oh yes it is! Sorry about that, I completely missed it.
Thanks for the quick response :)


Update benchmark.py

Change the output format to markdown

Change the output format to markdown so that you can directly copy it into a markdown environment.


add a new play

I have an idea to add a play which demonstrates the use of APIs to generate random data (it can be anything, quotes, images, memes, etc)
@atapas Let me know if this works!

Random meme will be awesome? Want to try that? Go for it. Please edit the issue title accordingly.

Yes, random meme generator would be cool!

Yes, random meme generator would be cool!

Super, I have updated the issue title.

Hey @atapas I am getting an error while creating a play!

That's Weird. Let me take a look.

Hey @atapas I am getting an error while creating a play!

I just tried it with the same value and it worked. Could you please try it once with Git Bash Terminal?

I'm suspecting some input it not going well with the VS Code terminal setting. PLease let me know @murtuzaalisurti

For me worked with VS Code too.. Please let me know if it went well with Git bash/command prompt for you?

For me worked with VS Code too.. Please let me know if it went well with Git bash/command prompt for you?

Getting the error in git bash too!

It says, replaceAll is not a function

What's the Node, Yarn version?

@atapas yarn 1.22.17

updated yarn but still the same issue!

updated yarn but still the same issue!

That's really strange! Want to get on a quick call now? Can check it out.

yeah sure!
@atapas

yeah sure! @atapas

Pnged you on Twitter @murtuzaalisurti

@atapas I used a recent version of node and it didn't work in administrator cmd! So, there's an issue with node.

It might be that the function or expression is deprecated, or something like that.

@atapas I used a recent version of node and it didn't work in administrator cmd! So, there's an issue with node.

So with which version of node it finally worked?

@atapas I used a recent version of node and it didn't work in administrator cmd! So, there's an issue with node.

So with which version of node it finally worked?

16.14.0

@atapas I used a recent version of node and it didn't work in administrator cmd! So, there's an issue with node.

So with which version of node it finally worked?

16.14.0

We need to document that. Please create an issue.

@atapas I used a recent version of node and it didn't work in administrator cmd! So, there's an issue with node.

So with which version of node it finally worked?

16.14.0

We need to document that. Please create an issue.

Sure, will create an issue!


Pitch tracker sometimes not continuous.

Pitch tracker sometimes not continuous.
常常會唱到一半斷斷續續

Add a property to allow developer to smooth the pitch?

See this demo :
https://www.speechandhearing.net/laboratory/ampitch/
Seems smooth and already good for use.

Seems using this package to deal with pitch tracking algorithm is the better way.
Also reduce unnecessary/hard-to-define code.

Fixed in #29


[7.x] Unable to prepare route [api/user] for serialization. Uses Closure. (LogicException)

Laravel Version: 7.0.6
PHP Version: 7.4.2
Database Driver & Version: MariaDB 10.4.11 (XAMPP)

Description
I have installed laravel 7 in my local server. When I run php artisan route:cache command then laravel return error:

LogicException
Unable to prepare route [api/user] for serialization. Uses Closure.

This problem was not in previous versions of the framework.

@devmansurov
When route has closures you can not cache route so you need to map with controller action

This problem was not in previous versions of the framework.

Closure routes were never supported so this definitely was.

I am facing this problem with laravel 7 from laravel5.3. Please help me.

This issue still exists, even using a controller method didn't help at all. Laravel still throws the same error

@msonowal
you said when a route has closures you can not cache route so you need to map with controller action I am a new Laravel learner I also met this issue on how to map with controller action?

route/api. Then comment this:
//Route::middleware('auth:api')->get('/user', function (Request $request) {
//    return $request->user();
//});

In Laravel 8 it'll be possible to cache routes using closures.

I have same issue ( laravel 7.23.2 ), after run php artisan optimize then laravel return error
Unable to prepare route [/] for serialization. Uses Closure.

My web root
Route::get('/', function () {
    return view('welcome');
});

Going to lock this to prevent more "I have this too" comments.


Parking: Set up CI/CD for RMP branches

This includes setting up CI/CD for the following branches:

parking
messaging
v6.1-rc

v6.1-rc has been configured for iOS/Android in App Center

parking and messaging branches have been set up for iOS/Android in App Center.


Choose cipher suite matching the certificate

Description
Reference issue
Fixes #398

Approve and run button is not appeared. Reopening PR

Seems causing panic in some test cases

cherry-picked to https://github.com/pion/dtls/pull/417
Will fix and add you as reviewer @at-wat


is it possible to mock no response?

I want to force the request to not return anything - simulating a server being non existent. Perhaps I want to simulate google.com being down.
fetchMock.mock('www.google.com', null)
This syntax doesn't actually work. I can return an empty response pretty easy but that isn't the same as a lack of response. Is this possible to do? I'd love to unit test different states of error.
I could just not mock the url at all, but this won't work if the url typically does exist, like google.com.

Would fetchMock.mock('www.google.com', {throws: 'Server not found'}) work? http://www.wheresrhys.co.uk/fetch-mock/api

This isn't working for me.
When I do the following:
try{
    fetch('http://thisSiteDoesntExist.com/foo')
        .then( res => console.log('It worked'))
        .catch( err => console.log('Promise error'));
}catch(e){
    console.log('Fetch threw!');
}

If I don't mock, then this will print:
Promise error

If I use the code that @wheresrhys  pasted above (or more specifically             fetchMock.mock(glob:*, {throws: 'Server not found'});) this causes the fetch function to throw, not the promise.  This will print:
Fetch threw!

I did some digging and this behavior seems to be enforced with a unit tests (responses.test.js):
			it('response that throws', async () => {
				fm.mock('http://it.at.there/', {
					throws: 'Oh no'
				});

				try {
					fm.fetchHandler('http://it.at.there/')
					expect(true).to.be.false
				} catch (err) {
					expect(err).to.equal('Oh no');
				}
			});

This seems odd to me.  Any input would be appreciated.

Logged a new Issue to discuss this issue see #295.


Katib samples need to be updated to v1alpha3

Katib samples use v1alpha2 APIs, need to be updated to v1alpha3

Katib examples have been updated. We can close this story.


avoid using unchecked casts or arithmetic

@bors r+

:pushpin: Commit 8b6af3eacbeb359c2041a3b5efc12042d3c33deb has been approved by RalfJung

:hourglass: Testing commit 8b6af3eacbeb359c2041a3b5efc12042d3c33deb with merge 12b7cf877b35b8309a9df8dbd6621edd5998fe95...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 12b7cf877b35b8309a9df8dbd6621edd5998fe95 to master...

There's a clippy lint for this if we want to make sure we don't add more of these in the future: https://rust-lang.github.io/rust-clippy/master/index.html#integer_arithmetic


Deployment preview

Before starting the deployment, ceph-bootstrap deploy should show a preview of what will change on each node.
Example:
=== The following changes will be applied ===

  -- node1:
     - Updates:         Some packages will be updated
     - SSH:             SSH Keys will be changed
     ...

Do you want to continue? [Y/n]:

ceph-salt apply is idempotent, there's no need for a preview ATM.


Corrected CLI script syntax on line 135

CLI script throws the following errors as the syntax is incorrect:
"admin user name cannot contain upper case character A-Z, special characters /"[]:|<>+=;,?*@#()! or start with $ or -"
"The value Standard_DS2_- provided for the VM size is not valid.
Hence corrected the syntax

@ralwani : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Thanks @ralwani! Signing off on this fix now.
#sign-off


Appstudio update hacbs-test-project-kzgt

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-kzgt-on-pull-request-
creating pipelinerun hacbs-test-project-kzgt-on-pull-request- in namespace rhtap-demo-koyu-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-kzgt-on-pull-request-
creating pipelinerun hacbs-test-project-kzgt-on-pull-request- in namespace rhtap-demo-vtak-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


feat(notation): avoid number below 0.001

Hey there! 👋
Making this change will avoid having < 0.001 displayed.

@RomainLanz <0.001 means you have a UST balance below 0.001 such as 0.00001. This cannot be changed because it is a display policy.
Thank you.

Alright!
I believe it is weird to display dust amount.
Having < 0.001 in earn will also display that kind of numbers for the Expected Interest, etc.
I do not believe this is valuable and useful but it seems that will not be changed. 😋


Error in data fromats

Help, an error occurs:
TypeError: SizeOf error for undefined : Cannot mix BigInt and other types, use explicit conversions
When executing the Move Player package
player.upstream.queue('move_player', {
    runtime_id: ptick,
    position: {x: 9.518291473388672, y: 67.62001037597656, z: -0.6351199150085449},
    pitch: 9.698684692382812,
    yaw: -14.1260986328125,
    head_yaw: -14.1260986328125,
    mode: 'normal',
    on_ground: true,
    ridden_runtime_id: 0,
    teleport: { cause: 'unknown', source_entity_type: 0 },
    tick: 0
});

Tell me, and then I've been suffering with this for half a day:(

This error is because you are supplying a 64bit number as a JS Number as opposed to a BigInt.


 I think it's been long enough now that we can yank them and update the metadata to point to the new versions.

Ok this is all done now. I've published an empty version with a deprecation notice and yanked all dependencies for the following crates:

elastic_macros
elastic_date_macros
elastic_types_macros
elastic_hyper


Support also other chains using an explorer compatible with etherscan

Hi,
on my projects I developed a wrapper for etherscan and I was curious to see if there was already an NPM package for this. I found this project.
I've seen you are developing a next gen = new version of this?
If so, please consider also to support other chains using etherscan (rebranded version) e.g.:

Fantom (ftmscan)
BSC (bscscan)
Polygon (polygonscan)
Avalanche (snowtrace.io)
etc.

Note: not all EVM chains cloned etherscan, so not all EVM chains have an API like those listed above, unluckily.
But the API works basically the same in all chains for what I know, maybe in some cases a specific call is not supported, else all same same, only change is the API URL.
I'd help if I had time, but as per usual I'm stressed in my project and will have to implement my wrapper on my own. I think supporting more chains doesn't require too much extra work, basically keep the explorerUrl a variable and replace it based on chain.
Also if using testnet, the suffix will be api-testnet.explorerUrl instead of api.explorerUrl.
Hope this helps, cheers and keep up the good work.

maybe a way to mod the chains would be  a way to help you? Can you point me to one documentation of an alternative 'ethrscan'. Maybe a time to resurrect the new branch, based on typescript and much more moddable.

@cryptotester would you be able to sponsor this feature request?

partly implemented.
Would you help to test the other ones if I add them?


Dragging speed reorderable_list

Hi guys,
increase or decrease the dragging speed, is it possible?

What do you mean by this? Your finger controls the dragging speed, no?

it does not work well, sometimes it gets jammed ...
another problem I found is the excessive space at the bottom of the list, what is it due to?

I think to investigate we would need a minimal runnable reproduction
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce.
Please add the output of flutter doctor -v.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


looks like the MJSXJ09CM camera isn't supported

Hi, first of all, great to see this module!
Currently trying to set up my Mi 360 Home Security Camera 2K with it, while getting the token and successfully added it via the integrations page. I'm not getting much info from it, it also says "unavailable" in the entities overview.
Could anyone confirm if the camera type is supported?

Have you enabled miot_cloud.

Configuration > Integrations > Xiaomi Miot Auto > Options > Enable miot cloud

yes I've enabled could, but that didn't make any change, after that, I've decided to manually put the xiaomi_miot section into my configuration with the correct server specified which helped a step further and now I'm also seeing 2 entities from that integration which both are available
so it looks like the camera is available, but I'm not seeing any image when clicking on it so no screens it shows a browser broken image icon any idea what that could be? I'm running home assistant on a raspberry pi btw :)

btw after rebooting home assistant the camera was gone again so it looks a little bit glitchy

Get the stream_address from the entity state attributes (Developer tools > State), and test whether it can play through VLC and other players.


Front End - Provide ability for Admin to set a test case to readonly for all other user types

Is your feature request related to a problem? Please describe.
Sometimes one test case needs to be read-only for all user types
except administrator. Only an administrator should be able to "lock"
a test case.
Describe the solution you'd like
Add a button to the test case dialog. When clicked, this button will
"lock" the test case, preventing any updates until an admin unlocks
the test case.

User with type=Admin can lock a test case so only any admin user can update it. Other user types will not be able to update it until an admin user unlocks the test case.


Alvin/test

Add style / unit test to CI

C++ and Python source files are style-checked following the ROS2 convention
A launch test example is created for testing pd_ctrl_node. Checkout the Python test launch and CMakeLists.txt change for future reference.

No logic changes, and most likely not interesting to other people, so merged without review.


Crashed once during 25 people call, please add length check

this is the stack trace for the error
panic: runtime error: index out of range [0] with length 0
goroutine 6038874 [running]:
livekit/pkg/sfu/buffer.(*VP8).MarshalTo(0xc00cbbfd40?, {0xc002d74600?, 0xc000140f90?, 0x11d7732?})
livekit@v0.0.0-20220829095212-a2ba53600200/pkg/sfu/buffer/helpers.go:165 +0x1b6
livekit/pkg/sfu.(*DownTrack).translateVP8PacketTo(0xc000140f90?, 0xc001c99214?, 0x366?, 0xc01178f8c0?, 0x17110c63c652913b?)
livekit@v0.0.0-20220829095212-a2ba53600200/pkg/sfu/downtrack.go:1379 +0xda
livekit/pkg/sfu.(*DownTrack).retransmitPackets(0xc010d19900, {0xc027f7f000?, 0xc027fc1cb0?, 0xc0024857b8?})
livekit@v0.0.0-20220829095212-a2ba53600200/pkg/sfu/downtrack.go:1271 +0x665
created by livekit/pkg/sfu.(*DownTrack).handleRTCP
livekit@v0.0.0-20220829095212-a2ba53600200/pkg/sfu/downtrack.go:1158 +0x715
We can add a length check in marshal to function. can add this with if condition len(buf) == 0 ||

Please fill out a bug report in its entirety, including version number & platform.


4.1.0-beta2: dynamically added classes get lost

Describe the bug
consider the following example code:
<script>
  export let bar = false;

  function toggle() {
    bar = !bar
  }
</script>

<div class:foo={bar} on:click={toggle}>something</div>

<style lang="postcss" windi:global>
  div {
    color: red;
  }

  .foo {
    color: green;
  }
</style>

This will always produce red text, even though Svelte does a correct job of appending "bar" to the HTML element's class list. However the class is not present in CSS.
The bug is only present in 4.1.0-beta2, not in 4.1.0-beta1.
System Info
System:
    OS: Linux 5.14 Arch Linux
    CPU: (24) x64 AMD Ryzen 9 5900X 12-Core Processor
    Memory: 49.44 GB / 62.72 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 16.9.1 - /usr/bin/node
    Yarn: 1.22.11 - /usr/bin/yarn
    npm: 7.23.0 - /usr/bin/npm
  IDEs:
    Emacs: 27.2 - /usr/bin/emacs
    VSCode: 1.60.0 - /usr/bin/code
    Vim: 8.2 - /usr/bin/vim
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Firefox: 92.0
  npmPackages:
    svelte: ^3.42.5 => 3.42.5
    svelte-windicss-preprocess: 4.1.0-beta.2 => 4.1.0-beta.2
    windicss: ^3.1.7 => 3.1.7

Used Package Manager
pnpm

Thanks for submitting the issue. Will work on a fix asap. Since there are not many changes in beta2 you can stick to beta1 for now.

@ptrxyz just tried this in a clean repo and can't reproduce it. Can you provide an repo?

/stale


My tutorial needs to be reviewed

Hey teams! Can you please review my "how to upload raw phenotypic data" tutorial when you have a chance? Thank you in advance :)

Here are my suggestions!
Main page

For title, I suggest "How to store raw phenotypic data in KnowPulse"
For Purpose, add an appostrophe + s to KnowPulse ("This lesson will show you how to use KnowPulse's raw phenotype importer...")

Setup tab

"Creat an account" is missing an e
The screenshot at the bottom of the page appears to be missing

Project setup

I would change "Once you have your data file ready to be uploaded," to "Before you start collecting phenotypic data in the field," since it would ideal to set up a project before they even start collecting! And then they will hopefully be backing up the data as its collected.
I'm a little confused by the details that are needed to set up the project. I would try to make this section more clear and provide examples. For example, what "Name" do you mean? ;) I would say something like "Trait name (eg. Days till 10% of Plants have Elongated Tendrils)"
Instead of "The trait description step can also be finished during data uploading." I would say "Alternatively, the trait definition can be described at data upload."
For the last paragraph, I would say something like "Once someone has helped create a project for you, the raw phenotype importer will generate a customized spreadsheet template for your data collection."

Fill your data in the data collection spreadsheet

A simpler title for this section could be "The data collection spreadsheet"
When you provide links in a tutorial that requires being logged in to access, I like to add in brackets: (needs login to access).
"A to E are default columns, follow by your traits." could be changed to "A to E are default columns, followed by one column per trait." In this paragraph I would also briefly describe the difference between green and grey coloured columns. Actually, is this information needed from the phenotyper in the previous step? It should be brought up then as well!
Instead of "if you intend to upload multiple files" perhaps "If you think you will need to upload multiple files"
Under Key Points, change to "Make sure all the names under the name column match existing germplasm in KnowPulse." and perhaps also provide a link to the germplasm search so they know where to check!

Backup and upload your data

The Objectives appears to be incomplete
The link for "Data backup function" should also mention that it needs account access
I would emphasize that the file needs to be XLSX format here!
"A text file for file comments, description and notes is provided for future reference." I think you're referring to the text box here. So I would word it as "A text box for describing data collection in its current state is provided to make it easier to reference back to if needed."
I think the "My Raw Phenotypes" panel only shows up on the KP homepage, not all pages. I would mention that the panel can be found at the top of the homepage.
Under "2. Describe New Trait" I would say "At this stage, your spreadsheet is examined for additional traits. Traits should have been described during project setup and you will see the descriptions in a greyed-out text box. If they haven't yet been described, please do this now (see Project Setup for details)". @reynoldtan can you double-check that they do appear greyed out and non-changeable?
Under "Key Points", I would change "even it is not clean." to "regularly. The data is expected to be raw, so don't worry about cleaning it up between backups or for upload."

View and download your data

The first sentence should say something like "like in the example below." before the link it provided (also needs access, I think!)
Congratulation is missing an "s" at the end
Instead of "Helps and more" I would title this "Final Thoughts"
Under Key Points, change to "Log into KnowPulse with your user account before viewing and to download your data."

Great work, please poke me if you need any clarification on anything I suggested!

Project setup

Once you have your data file ready to be uploaded, please get in touch with us via
I think we want to do all the setup prior to any phenotypic data collection work...
suggestion: To begin data collection in your trial, please contact<EMAIL_ADDRESS>to setup your project in Knowpulse.

Then emphasize what user will be getting or benefits if they have a properly setup project in KP namely

Access to information (text and photo) on how data should be collected for each trait
A page to download data collection spreadsheet file pre-filled with project specific traits and formatted in way expected by Knowpulse/module. This file is used to record data.
A space in Knowpulse that they can backup spreadsheet file and access backed up files anywhere.

Apart from trait description that will be asked, we also need information about the project and from the Add project page of Tripal it needs

Project Name
Project Description

Let's not suggest at this point that they can describe the traits later during the upload. We can add this message when describing the upload stage in another page. The real purpose of the describe trait during upload is for 1 or 2 maybe 3 traits that have been detected in the spreadsheet that is/aren't defined in the project and not the entire trait set of the project.

Here's the project page

Download Data Collection Spreadsheet

Here's a step by step guide to downloading a spreadsheet file.

How to download data collection spreadsheet file?

Click Change Project link, located next the project title of the currently active project to show Change Project Form.

Select a project when prompted. The instructions page will reload with a new active project. This will ensure that you are downloading the data collection spreadsheet file corresponding to a project that you are working on.

Click the Download Data Collection Spreadsheet link in the Standard Procedure Tab to begin file download. Click save when prompted.

To focus on the columns... If possible, can you take another screenshot of your sample spreadsheet file but hide the controls of ms excel? Try clicking the up arrow head (^) at the extreme right hand end of that control panel to hide.

see @carolyncaron comments on the paragraphs following this screen capture.

Data Backup

Overview/objective suggestion
Backup page allows active users appointed to a project to backup multiple versions of their data collection spreadsheet file into the server before submitting them to the module through the Upload Data Page.

Page anatomy

How to manage your backed up file
How to back up a data collection spreadsheet file?

Select the project that corresponds to data collection spreadsheet file.
Drag and Drop file to the File Drop Zone Area or click choose file to launch file browser window.

Type notes, comments or description.
Click Upload Button.

How to download a file?

In File Browser, click the name of the file link under the File column or the download icon (arrow pointing down) to start file download.
Click save file when prompted

How to archive, restore and delete a file?

In File Browser, click the Trash Bin icon to archive a file. Trash icon will indicate that there are archive files in a project.

Click Archive file indicator to disclose archive files table.

In archive files table, options to restore file and physically delete file will become available. You will be prompted to confirm any action to a file.

Can the data upload be in a separate page? This page is huge.. I have full details of each stages and instructions as well as troubleshooting upload. Let me know if when you're setting up this separate page and I will send the info to you.

View data and download data and Helps and more

Let's move the part where they can get help in the fist page - setting up projects page. In the help page lets include email, KP homepage, link to SOP page, Link to backup, Link to Demo page
I have more info about view data and download. Please let me know when you're doing these pages.


How to do color manipulation?

Hi guys,
Is it possible to do image manipulation like brightening, darkening, contrast... or applying some function to the whole image on pixel by pixel basis?
Thanks,
Russell

Hello, did you see #609?

I 2nd the request to expose libvips modulate functionality in sharp, definitely would make a useful addition.

@RussellRC Hopefully #609 will provide what you need, please feel free to re-open with more details if not.


Crash on boot: unable to load driver: nouveau_dri.so

Godot version:
Godot_v3.1-alpha3_x11.64
OS/device including version:
Arch linux on laptop with a NVidia 820m, nouveau drivers.
Issue description:
Godot 3.1 alpha 3 does not boot.
Both the ready to use, unzipped, Godot program and the manually built with scons.
When launching it it crashes immediately with the following output:
libGL error: unable to load driver: nouveau_dri.so
libGL error: driver pointer missing
libGL error: failed to load driver: nouveau
libGL error: unable to load driver: nouveau_dri.so
libGL error: driver pointer missing
libGL error: failed to load driver: nouveau
libGL error: unable to load driver: swrast_dri.so
libGL error: failed to load driver: swrast
ERROR: initialize: Condition ' ctxErrorOccurred || !p->glx_context ' is true. returned: ER
At: platform/x11/context_gl_x11.cpp:188.
handle_crash: Program crashed with signal 11
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /usr/lib/libc.so.6(+0x37e00) [0x7f2a009a0e00] (??:0)
[2] /lib64/ld-linux-x86-64.so.2(+0xd005) [0x7f2a00b61005] (??:0)
[3] /lib64/ld-linux-x86-64.so.2(+0x14406) [0x7f2a00b68406] (??:0)
[4] /usr/lib/libc.so.6(_dl_catch_exception+0x77) [0x7f2a00aa0f57] (??:0)
[5] /lib64/ld-linux-x86-64.so.2(+0x13dff) [0x7f2a00b67dff] (??:0)
[6] /usr/lib/libdl.so.2(+0x115a) [0x7f29ffe0515a] (??:0)
[7] /usr/lib/libc.so.6(_dl_catch_exception+0x77) [0x7f2a00aa0f57] (??:0)
[8] /usr/lib/libc.so.6(_dl_catch_error+0x33) [0x7f2a00aa0ff3] (??:0)
[9] /usr/lib/libdl.so.2(+0x18bf) [0x7f29ffe058bf] (??:0)
[10] /usr/lib/libdl.so.2(dlopen+0x4a) [0x7f29ffe051fa] (??:0)
[11] ./Godot_3.1_alpha() [0xd4ae25] (??:?)
[12] ./Godot_3.1_alpha() [0x10d94be] (:?)
[13] ./Godot_3.1_alpha() [0x7a1261] (??:?)
[14] /usr/lib/libc.so.6(__libc_start_main+0xf3) [0x7f2a0098d223] (??:0)
[15] ./Godot_3.1_alpha() [0x7aa83e] (??:?)
-- END OF BACKTRACE --
Aborted (core dumped)
Steps to reproduce:
Launch Godot on a simple arch linux installation with nouveou installed.

Does it work if you run Godot with DRI_PRIME=0 godot?

It worked!
I set DRI_PRIME=0 with
export DRI_PRIME=0
and then launched Godot via the same terminal and it opened up the project manager.
I went on and created a project and it opens the editor without problems, too.

Same cause as #24198 then, CC @hpvb.

Fixed on master

This just fixed 4.1.1 for me


My moto e 2014 bricked

My moto e 2014 bricked in edl mode
it all started when my motorola moto e 2014 was good and one day I installed the sbl1 partition by mistake and it bricked me please help me

@Vicc2008 Have you tried the steps that are already mentioned?

But i wanted to install Windows phone 8.1 to my moto e 2014

@Vicc2008 Have you tried the steps that are already mentioned?

Yes

@Vicc2008 Have you tried the steps that are already mentioned?

Yes but i get errors

@Vicc2008 Make sure you have met all prerequisites and requirements.

In my case is a qualcomm snapdragon msm8210 and Android 5.1 Lollipop

But i wanted to install Windows phone 8.1 to my moto e 2014

But i wanted to install Windows phone 8.1 to my moto e 2014

@Vicc2008 If your phone is bricked, you need to unbrick it first. Also, I no longer have a Moto E with me to experiment and this project is no longer maintained. You might want to check the following which is what worked years ago for many people:

https://github.com/aravindvnair99/Motorola-Moto-E-XT1022-condor-unbrick/blob/main/Prerequisite.md
https://github.com/aravindvnair99/Motorola-Moto-E-XT1022-condor-unbrick/blob/main/Unbrick methods.md
https://github.com/aravindvnair99/Motorola-Moto-E-XT1022-condor-unbrick/tree/main/Proofs

In my case my model is a XT1021

In my case my model is a XT1021

@Vicc2008 Works on XT1022, XT1021, XT1025 as mentioned here.

I need a blank-flash for the xt1021 but with the UEFI

I need a blank-flash for the xt1021 but with the UEFI

@Vicc2008 I'm sorry, I no longer have a Moto E with me nor am I maintaining or supporting this project any longer. The Moto E is an old phone now. This was like a hobby for me 4 years ago or so and not anymore. You are free to go through the stuff I have put up here and XDA, the links which I have given you already. But beyond that, I don't have anything to say.

Oh

or with thor2 from Windows device recovery tool

or with thor2 from Windows device recovery tool

@Vicc2008 The Windows Device Recovery Tool is not meant for Android devices.

Oh

but from cmd

but from cmd

@Vicc2008 CMD method is here: https://github.com/aravindvnair99/Motorola-Moto-E-XT1022-condor-unbrick/blob/main/Unbrick methods.md#method-5-fastboot

But in edl mode

With blank-flash

With blank-flash

@Vicc2008 https://github.com/aravindvnair99/Motorola-Moto-E-XT1022-condor-unbrick/blob/main/Unbrick methods.md#method-2-blank-flash

Oh


ZMQ: Can't connect to transaction host if it's mapped to a different port.

Background
I'm trying to connect a debug version of LND to a bitcoind instance running in a Polar docker container. The Polar bitcoind docker container maps the ZMQ transaction host port (28335) to another local port (29335). LND is unable to connect to the ZMQ transaction host.
Your environment

lnd debug build of the current master branch (41b91bd5beb2dcff3b5d7cddb74a319bba18b8e5)
OS: 21.1.0 Darwin Kernel Version 21.1.0: Wed Oct 13 17:33:23 PDT 2021; root:xnu-8019.41.5~1/RELEASE_X86_64 x86_64
bitcoind 0.21.1 regtest running in Polar 1.3.0 in default configuration
Polar connection details:

RPC Host | http://<IP_ADDRESS>:18443
P2P Host | tcp://<IP_ADDRESS>:19444
ZMQ Block Host | tcp://<IP_ADDRESS>:28334
ZMQ Transaction Host | tcp://<IP_ADDRESS>:29335

Docker port mapping details

18443/tcp | <IP_ADDRESS>:18443
18444/tcp | <IP_ADDRESS>:19444
28334/tcp | <IP_ADDRESS>:28334
28335/tcp | <IP_ADDRESS>:29335

Steps to reproduce

Run bitcoind in Polar, docker ports mapped like above config
Start LND with ./lnd-debug --bitcoin.active --bitcoin.regtest --bitcoin.node=bitcoind --bitcoind.rpchost=<IP_ADDRESS>:18443 --bitcoind.rpcuser=polaruser --bitcoind.rpcpass=polarpass --bitcoind.zmqpubrawblock=tcp://<IP_ADDRESS>:28334 --bitcoind.zmqpubrawtx=tcp://<IP_ADDRESS>:29335
lncli-debug unlock

Expected behaviour
LND should connect to the ZMQ transaction host over a mapped port in a docker container.
Actual behaviour
After unlocking the wallet I get this error:
2022-03-11 12:47:31.573 [INF] LNWL: Started listening for bitcoind block notifications via ZMQ on <IP_ADDRESS>:28334
2022-03-11 12:47:31.573 [INF] LNWL: Started listening for bitcoind transaction notifications via ZMQ on <IP_ADDRESS>:29335
2022-03-11 12:47:31.585 [ERR] LTND: unable to create partial chain control: unable to subscribe to zmq tx events on <IP_ADDRESS>:29335 (bitcoind is running on <IP_ADDRESS>:28335)
2022-03-11 12:47:31.586 [ERR] LTND: Shutting down because error in main method: error creating wallet config: unable to create partial chain control: unable to subscribe to zmq tx events on <IP_ADDRESS>:29335 (bitcoind is running on <IP_ADDRESS>:28335)

Remarks

The same error occurs when connecting to bitcoind.zmqpubrawtx=tcp://<IP_ADDRESS>:29335, so it must be the port mapping that is the problem.
Apparently in this case LND is able to connect to the ZMQ block host. Note that the port mapping for the block host in above configuration is 28334:28334. So same port apparently does work.

The relevant check in the code is here:
https://github.com/lightningnetwork/lnd/blob/41b91bd5beb2dcff3b5d7cddb74a319bba18b8e5/chainreg/chainregistry.go#L547
I suppose bitcoind doesn't know which port it is exposed under. It 'thinks' it's exposed under port 28335, but it cannot know that we're in fact mapping the port.
Removing the check fixes the problem for me.

I added this constrain because of a security concern we had. I can add a pr which will exclude localhost, think it should be enough.

I added this constrain because of a security concern we had. I can add a pr which will exclude <IP_ADDRESS>.

I don't think that will help. It's the port number in this case. If I'm hosting the bitcoind ZMQ events under <IP_ADDRESS>:5555, but it's proxied over <IP_ADDRESS>:1111, I should be able to subscribe to the proxy I think.

I think we just have to add a check for ZMQ events in the continuous health check. At this moment, I think it would be best to remove this check. This would be a less discrete way to solve the issue than adding yet another config parameter.
Will post a pr this weekend. Thanks for your feedback.

I guess the problem is that you cannot know you're actually subscribed to ZMQ events at all, right?
That's right. It's obvious how we best solve this.

Ran into this as well and would welcome enableZMQSanityCheck. Or perhaps allow users to omit the port in configuration and in that case use the port retrieved from bitcoind?

@joostjager @JssDWt I added a pr which removes the zmq tx/block port check. I am not a fan of adding yet another config parameter to address this minor check. Remember, this only occurs when the port numbers are mixed up. I rather see I or someone add an health check that gives a warning to the user in runtime.
If you rather like me to fix this with an config parameter I'll gladly do that.
Thanks for your feedback.


When reading ALIAS records, also read the value of EvaluateTargetHealth

When reading ALIAS records into our local store we assume a value for its field EvaluateTargetHealth based on a flag: https://github.com/kubernetes-incubator/external-dns/blob/v0.5.6/provider/aws.go#L343
This becomes problematic when we want to delete an ALIAS that has a different value than this flag. (When deleting a record we need to specify those values to successfully delete it).
For instance, assume you're running external-dns with --aws-evaluate-target-health=true and create some records. You then later change it to run with aws-evaluate-target-health=false. If at some point external-dns wants to delete a record from the first time it will fail because the value for that field in the DELETE request doesn't match the actual value.
This will cause the entire ChangeRequest batch to fail and leads to external-dns not creating/updating any records anymore.
This can be avoided by reading the actual value from the record in AWS instead of assuming (similar to TTL). We could use the new ProviderSpecific fields on Endpoint to store that value. 🎉
Another way to mitigate the effects of an accidentally introduced bug like this is to avoid failing the entire batch when a single record is permanently broken: https://github.com/kubernetes-incubator/external-dns/issues/104

This issue is fixed in the https://github.com/kubernetes-incubator/external-dns/releases/tag/v0.5.8


Missing line when splitting the second time

Testing #44446 in Mac
There are 3 lines rendered in the terminal. The first split captures all of them. The second split results in the second terminal losing one of the lines

Ok so I didn't think I could do anything about this when I saw it but I was wrong :smile:
What caused the weirdness with the second terminal is that it received 2 layouts/resize events, one when the last terminal pane was added the group and one when the sizes are "reset" afterwards. I changed it such that terminal layouts are disabled when the pane is being added and re-enabled immediately afterwards, so they get a single "reset" resize.

Checked from source. Can still repro, only now the problem occurs when you split the third terminal

To clarify, this will still happen because it's bash telling the terminal what to draw when the terminal is resized, it just won't happen as much as before because before it was resizing to some value (lower than correct) and then the correct value. Right now you're just seeing bash messing up when you resize the window which will also happen on external terminals.


Label for text_style_transfer

I can not read in non-integer label like 0.456 for text_style_transfer code:
iterator = tx.data.FeedableDataIterator({'train_g': train_data,'val': val_data, 'test': test_data})

To read in non-integer label, set the data_type here to 'float'.
Note that the example assumes binary label, e.g., the code here flips the label by 1-label


provide refresh option always on all tables when the data loading fails

Similar to the screenshot below, all tables in Xpanse UI must provide an option to refresh data even when the data loading fails.

[ ] MyServices, Polices, Service Policies - Refresh buttons to be always available.

I'll take this.


[Market] Add mortgage mechanism for provider

Resolved #136
With tests

Talked offline, like bond operation in staking, we divided pledge into 3 actions:

Using pledge to replace register;
pledge_extra to add more mortgage;
cut_pledge to allow provider do redemption;

User scenario

pledge
register
[Optional] pledge_extra/cut_pledge/register
[Deny] pledge again


Upgrade to more recent image of Docker

Thanks for publishing this package. It's just what we needed!
A recent change in Docker 20.10.3 adds support for the use of host.internal.docker on Linux machines (which are what's typically used in Github Actions). It would be valuable for our project - and probably others! - to upgrade to 20.10.x.
Relevant issue: https://github.com/docker/for-linux/issues/264#issuecomment-779490294

That's awesome! Thanks for the PR.


feat(system_monitor): support loopback network interface

Description
Why this PR is needed

Current System Monitor (Net Monitor) doesn't support lo (local loopback)
Since Autoware has a large amount of topic communications, watching the lo communication is important to analyze/improve system peformance

What this PR changes

lo network is monitored when "lo" or "*" is described in net_monitor.param.yaml
Note that:

network.capacity is inavailable for lo, therefore -1 is set
the value of network.tx and that of network.rx are identical
lo is not added for nethogs because it may increase CPU usage

Related links
None
How was this PR tested?

lo is monitored when devices in net_monitor.param.yaml is ["lo"] or ["*"]
The increase in CPU usage due to this change is minimal.

Original result

Result with this PR (lo is supported)

Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@ito-san @TetsuKawa
Could you review this PR?

@TetsuKawa
The cppcheck-differential workflow failed due to snap issue. All cppcheck-differential jobs failed this evening and it looks fixed now.
Could you re-run cppcheck-differential workflow?   (I don't have a permission to re-run the job, but you may be able to click re-run button in the following job result page.
https://github.com/autowarefoundation/autoware.universe/actions/runs/11717422551/job/32637185610?pr=9067


fix remove 'push_down_predicates' and fix BE crash with decimal predicate

Proposed changes
Previously, for JNI scanner, BE will assemble a push_down_predicates from colname_to_value_range
and pass it to JNI scanner. and JNI scanner will determine how to use it to do the predicate pushdown.
But actually, this push_down_predicates is never used.
This PR removes the push_down_predicates.
This will also avoid BE crash when encountering decimal type predicate(a bug, not fixed, but skip it)
Further comments
Actually the predicate passed to JNI scanner can be assembled and serialized on FE side, and pass to BE
then pass to JNI scanner directly, just like what PaimonJniScanner do(see paimon_predicate).

run buildall

run buildall


Add tests for Data Movement Controller

This adds two test cases:

Verify data movement
Verify data movement cancellation

Since data movement relies on mpifileutils and other dependencies, these tests rely on setting the NNF_TEST_ENVIRONMENT environment variable to use sleep instead. It also short circuits getStorageNodeNames and returns a list of just the localhost. This then using the existing short circuit in getWorkerHostnames.

What is the test coverage after running this? I think our goal is >90%. And I know my manager controller manager is missing some.

What is the test coverage after running this? I think our goal is >90%. And I know my manager controller manager is missing some.

It's only 51% so there is some work to do here. We're only doing these sunny day scenarios (and not using mpirun) and we're also short-circuiting getStorageNodeNames and getWorkerHostNames
I can continue to improve on these while working on the progress feature.

It's only 51% so there is some work to do here.

That's a solid 10 percentage point gain!

getStorageNodeNames & getWorkerHostNames need a bit more setup, but not terrible
handleInvalidError() scenarios... needs 1, but not that critical if you ask me
r.Update() / RetryOnConflict() errors are nearly impossible to test
cancel after complete - test should be easy, but not that critical... and it would only handle one if condition


[FIX] Remove attributes with a NULL label

This is not a great fix - but better than having the app crash under us

Closes #123

Changes proposed in this pull request:

filter any attribute with a label: null
display a warning
add a (basic) regression test

Checklist
This section is for the PR reviewer

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
[x] PR has a label for the release changelog or skip-release (to be applied by maintainers only)
[x] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass

For new features:

[x] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

:rocket: PR was released in v0.2.1 :rocket:


Scalability of Shred

Something which came to mind, but I don't know if it has been discussed, yet, or if my understanding of how the scheduling works is just wrong :)
Example (from SPECS):
impl<'a> System<'a> for SysA {
    type SystemData = (WriteStorage<'a, Pos>, ReadStorage<'a, Vel>);

    fn run(&mut self, (mut pos, vel): Self::SystemData) {
        // The `.join()` combines multiple components,
        // so we only access those entities which have
        // both of them.
        for (pos, vel) in (&mut pos, &vel).join() {
            pos.0 += vel.0;
        }
    }
}

Given we have several million entities with Pos and Vel components, how does Shred dispatch this system? Does it pass all entities to a single call of run() or does it call run() multiple times with viewer entities, but on its own thread each? If I understand Shred correctly, it passes all entities to a single call of run(), hence a single thread, which would mean that, if I only have one system, only one core of my CPU would be used for all data sequentially, and e.g. 31 other cores would idle around doing nothing (I know, this example is very constructed, but I want to demonstrate my train of thoughts and concern if Shred, hence Specs, hence Amethyst and any other high-level library and application depending on this crate is scheduled suboptimally).
That being said, wouldn't it be more optimal to ideally take the number of systems (which can run in parallel) and entities and CPU cores and feed smaller chunks to the system, so that one system runs multiple times in parallel?

If you want multithreading in the above code sample, take a look at par_join. That gives more control to the user as there might be things you want to do just once per dispatch.

@torkleyy Thanks, that's very interesting! I found an example after searching for that keyword.
fn run(&mut self, (comp_int, mut comp_float): Self::SystemData) {
        use rayon::prelude::*;
        (&comp_int, &mut comp_float)
            .par_join()
            .for_each(|(i, f)| f.0 += i.0 as f32);
    }

There should be more info about par_join() in the docs and the books, so people actually find this useful functionality. If I have time tonight, I will at least contribute a few sentences mentioning it (in Specs) :)

Though it still seems as if Shred doesn't support such a way of scaling systems...

Sorry for the late response, I just wanted to say:
shred does not have a scalability problem; the concept "Entity" does not even exist in shred; Systems just work on Resources. And running the same system on multiple views of entities is besides being horribly unsafe simply not practical. There are e.g. things you want to do only once per dispatch, there are loops you cannot parallelize. The current design gives control to the system implementor and also ensures that all safety requirements are fulfilled.


[dev-env] Save instance data state

Description
Save instance data string to a file on creation.
This will enable us to re-trigger the setup wizard in order to reconfigure the existing environment.
Steps to Test

create new environment.
In the location you can find - instance_data.json

Why are we adding a hardcoded version number? What does it mean?

Good question, the idea there is that if in future releases we would switch some format or structure of data stored we can still kinda detect that.
Say as an example right now we store { phpmyAdmin: true, xdebug:false} in the future we would store {extraServices: [ 'phpmyadmin']} with the version we will be able to know how to parse the file correctly.
I am however open to ideas ... maybe it is unnecessary.

@pschoffer That sounds reasonable and it makes sense. However, we don't have that versioning problem yet, so I think it's a bit overkill. Whenever we introduce a breaking change, we can start at version 2 and assume that if a config file doesn't have an explicit version number, it's version 1.

Sounds good I dropped it. Thanks @pauarge


Instructions

Instruction modal before starting out the quiz.

Different quiz file for different type of quizzes.

Deployed Link: https://nature-quiz.vercel.app/


Feature/point stac load

Add basic loading capabilities - interpolation isn't working out of the box yet.

I have no suggestions, looks good send it! (merge it)


Update to some outdated information

The first bullet point on the page mentions Teams support 5,000 users. This is out of date. Teams currently support 10,000 users per team. This is being increased to 25,000 users Dec 2020 to Jan 20201 see Roadmap id 68723.
Also the Teams respecting the Microsoft Group naming policy is in GA and not private preview.

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

ID: 8c310bab-f8d8-b173-0bff-9f93fe1f7abf
Version Independent ID: 1f983e68-65e3-aa7d-aaaa-d62912f37e4c
Content: Plan for Microsoft 365 Groups when creating teams - Microsoft Teams
Content Source: Teams/plan-Office-365-groups.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@nikki-c Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Thanks for your feedback @nikki-c


New package: jadquir.MRA version 1.10.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Resolves #182625

Microsoft Reviewers: Open in CodeFlow

Verify:

Hi @Trenly,
How should the version number discrepancy be handled? The application seems to be displaying 0.111 Beta. It shows release notes for version 0.110 Beta in the Updates tab - I'm guessing that dots and implied zeroes might be redistributed in that number to become the 1.10.0 shown by the Registry. And also guessing that the application silently updated without updating the Registry.

The only way to handle it would be to add DisplayVersion; I don’t think it silently updated since it is the 0.111 installer


The "build" script cannot be run on Windows platform

Steps to reproduce

on Windows platform
pnpm install
pnpm run build

What is expected?
build successfully
What is actually happening?
Terminal ：
><EMAIL_ADDRESS>build D:\Project\interface-ui
> pnpm -F './packages/*' -r --stream build

No projects matched the filters in "D:\Project\interface-ui"

additional comments
On Windows the build worked when I removed the single quotes

分模块，如果想运行build，请到具体的文件夹下运行

分模块，如果想运行build，请到具体的文件夹下运行

到具体的文件夹下，那和贡献指南，提交PR第三条说的不一致啊

分模块，如果想运行build，请到具体的文件夹下运行

到具体的文件夹下，那和贡献指南，提交PR第三条说的不一致啊

Hey @windowdotonload, the documentation for contributing needs to be updated. Sorry for the inconvenience.
Note: You can build all packages located under the root directory using pnpm run build. Alternatively, you can manually build specific packages located in their corresponding folders.
We have received your pull request and will test it on both Windows and Mac devices! Thank you for your contribution.


I want to be able to see the list of new bugs, the list of pending bugs, and the list of fixed bugs. (US 8.1)

Extension to #12.
I want to be able to see the list of new bugs, the list of pending bugs, and the list of fixed bugs.
Acceptance Criteria

[ ] When the count for any of the categories is clicked, the corresponding list of bugs should be shown

Original Acceptance Criteria

When the count for any of the categories is clicked, the corresponding list of bugs should be shown

Comments
Is redundant since the user can simply filter the available bug reports


HadoopTables vs HadoopCatalog

Today we have both these APIs to work with HDFS tables.  It seems all functionality of HadoopTables  is supported by HadoopCatalog, with one exception. HadoopCatalog expects all its tables under a common root dir.  @aokolnychyi , @rdblue , what do u think. Should should we replace usage of HadoopTables with HadoopCatalog and remove HadoopTables and its parent interface Tables?

I think these are two different scenarios, especially in Spark. Catalog (both hadoop and hive catalog) can be plugged into Spark's multi-catalog interface, which will benefit the SQL scenario, as well as DataFrame of course; whereas HadoopTables I think the main scenario is DataFrame API, where we don't require a catalog necessarily.

Agree that they are two different use cases.
In our use case, we have a decoupled Catalog that has minimal knowledge about Iceberg. Thus we use the HadoopTables implementation.

This makes sense to me. I'll close this ticket. Thks everyone!


Recent TOC item padding changes

Hi, after updating to 2.3.0, a change in the padding of the TOC items has become noticable. This is with FF118 on Linux Debian.

Adding the top-padding back would improve the overall look, IMHO.

The top/bottom space for the H4 (now .b) items also helped, but seems optional.

Your thoughts?

BTW, nice updates, the archive.html rename, and the posts.html content block simplifications, amongst others.

Thanks, Some very large pages can have very long index. The idea was to make better use of the space. I just added a bit of top padding to the blockdiv which I agree looks a bit better.


Disable create account functionality when 25 accounts exist

Well, after you unlock/buy the 25 extras you can still create new accounts tho 🤔  so a full disable is not user friendly.

@basst85 what does card limit have to do with this ? 😉 but i think it does.

@OGKevin Whoops, LOL! I mean IBAN activation limit

@OGKevin It'd be better to simply warn the user than? I'm not sure how getting more accounts is billed but I'm guessing it happens as soon as you create the account?

@Crecket AFAIK you can only do that via the app and not via the pub API. But you can always confirm in sandbox 😁


{ reduxMiddleware } utility

Provide support for Redux middleware

This is not the direction that Eigenstate is looking to go. Middleware is going to change/simplify, and the new structure won't allow for this.

Refactoring for this could be immensely helpful for multi-threading support, because we'll need serializable JSON actions anyway. Plus I think it's worth reconsidering our present action events, since they are more like change/stateChange events

Reconsidering (again)... This could lead to some bad habits for people overly used to Redux.


Deprecated selector in minimal-dark-zen-ui/index.less

In minimal-dark-zen-ui/index.less:
Starting from Atom v1.13.0, the contents of atom-text-editor elements are no longer encapsulated within a shadow DOM boundary. This means you should stop using :host and ::shadow pseudo-selectors, and prepend all your syntax selectors with syntax--. To prevent breakage with existing style sheets, Atom will automatically upgrade the following selectors:

atom-text-editor[mini] .placeholder-text, atom-text-editor[mini]::shadow .placeholder-text => atom-text-editor[mini] .placeholder-text, atom-text-editor[mini].editor .placeholder-text

atom-text-editor[mini] .selection .region, atom-text-editor[mini]::shadow .selection .region => atom-text-editor[mini] .selection .region, atom-text-editor[mini].editor .selection .region

Automatic translation of selectors will be removed in a few release cycles to minimize startup time. Please, make sure to upgrade the above selectors as soon as possible.

@canibal Thank you for notice. Will fix


Fix restore.sh

This script had some wrong arguments (bad copy-paste from a more
complex repo). The restore.cmd script already had this fixed.

Bypassing CI because this isn't used in the CI 😆


Connections are not displayed anymore.

Could you elaborate more? I can't reproduce the issue on my end.

I think I know your mistake, you probably forgot to issue a "bower update" as the new update relies on a new feature in pandora.


Make arrays optional when they should be in swift

This is a very good addition!

Thank you @pagr — can you please explain the change?

This fixes the issue where optional arrays from graphql are decoded as non optional arrays in swift


Extract Source based on the product name & Update Doc

So that future product names can be automatically supported could we instead of using a constant extract with regex the value of the field used in the filter replace space with _ and force lowercase?

@rfaircloth-splunk I'm not able to find any example to convert the macro value into the lowercase. Could you please provide some doc link for the reference.

In the Splunk search, the values are case insensitive. Do you think we should still convert it into the lowercase?

the reason I would normalize case is that search field=value is case insensitive, props/source:: license metrics, internal stats, and | stats | tstats are not. if there are cases where the vendor is inconsistent it would make use more difficult.

Gotcha. updated PR.  please review it.


Rename

TL;DR
Update org references
Type

[x] Bug Fix

Are all requirements met?

[x] Code completed

Complete description
update org references
Tracking Issue
We didn't make one for this...  I should create one for all.

Codecov Report

Merging #73 (fe73d87) into master (334748c) will increase coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   70.10%   70.46%   +0.35%
==========================================
  Files          57       57
  Lines        1967     1967
==========================================
+ Hits         1379     1386       +7
+ Misses        449      443       -6
+ Partials      139      138       -1

Flag
Coverage Δ

unittests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cache/auto_refresh.go
81.05% <ø> (+7.36%)
:arrow_up:

cli/pflags/api/generator.go
69.65% <ø> (ø)

cli/pflags/api/sample.go
0.00% <ø> (ø)

config/section.go
41.93% <ø> (ø)

config/utils.go
16.12% <ø> (ø)

logger/config.go
42.85% <ø> (ø)

logger/logger.go
62.85% <ø> (ø)

pbhash/pbhash.go
50.00% <ø> (ø)

profutils/server.go
53.19% <ø> (ø)

promutils/labeled/counter.go
34.78% <ø> (ø)

... and 16 more

Continue to review full report at Codecov.

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

@EngHabu -- interesting... passed the tests, but not lint (also, I'm used to seeing linters before the unit tests, not after).
It looks like you had something special with mockery and your own repo.  I'll keep thinking of approaches (and I'm probably making a mess of other things, lol ;-) )

@EngHabu -- also, I'm wondering why boilerplate is in the repo.  Shouldn't we make this its own repo/module (distinct from flytestdlib, flytectl, flyteidl, etc).  It looks like this is ~approx the same in each repo.

@EngHabu  -- OMG, it is GREEN.


2 errors - every level platformer - physics bugs

here are the 2 errors:
Every level is an platormer: Even I've been trolled, congregration, etc.
physics: The mutliplayer collision is extremely weird, getting me to the floor(most of the levels are up from the floor).
comment for More detailed information.

i will also provide some videos...


[Discussion] Re-Numbering / handling framework version changes so the major number matches the React / React-Native releases

Hi All,
I want to start a discussion were moving forward I'm thinking that we should change at least the major version number of these plugins to match the major version of React / React-Native framework that is targeted at, this may also mean moving out the react plugins in the same way we did for the Angular plugin.
I have basically this same discussion happening in that repo as well https://github.com/microsoft/applicationinsights-angularplugin-js/issues/57
Because of the size of our team, we don't have the capacity or the active working knowledge to keep all of the React / React-Native versions up to date with the current framework releases with the latest Application Insights version. And internally, we are generally not supporting updated frameworks until it has been released for at least 6 months to give it time to settle down (not withstanding / considering any beta releases it may have had for this timeframe)
This comes particularly to a head when the version of the frameworks update in a non-backward compatible manner. As the current React 18 update appears to be causing...
The basic thoughts on the approach will be to use branches to keep the difference between any release isolated so that we could apply any required security patches (if necessary), however, as a mono-repo using branches for different components becomes problematic as keeping the "Core" portion of those branches would just be practically possible. So this then means any older framework versions would become frozen to the version of Application Insights at the point of branching.
Branching Options

Keep in mono-repo and freeze older framework versions to older Application Insights (No chance new feature implementation, or new config support (for typing)) -- will have potential issues with multiple SDK "instances" being resolved for npm packages
Move out to new repo and target AI version "ranges" (like we have done for the Angular Plugin), so we will only need to "publish" an updated version of the React / React-Native plugins when TypeScript typing changes or new features are required and also potentially decoupled from AI releases.

For either of these
The master/main branch will target the minimum supported release to the "current" version, (note singular for mono repo -- no ranges)
And when we change the targeted verson (in master) of React / React-Native we then

Update the major version # and reset to 0.0 (so 14.0.0)
Create a branch for the previous version (for hotfixes) and active development will generally pause except for security related issues

The main driver of this is that we are already drifting from the previously Application Insights <-> React/Native Plugins connection and with the number of versions of breaking changed Angular versions it makes it difficult to understand what / which version should be used. And it less of a cognitive matching or requiring to always lookup some compatibility table.
So while Application Insights v2.8.1 will load and work with any older version of the React Plugin (when just injected via the "extensions" config, TypeScript is unhappy with the flexible signature changes that we have introduced to support older and new extension / features, if you try and "compile" within the same project. This does seem to depend on your tsconfig.json settings as I don't believe everyone is seeing these issues.
Numbering Options
So then the question is about numbering and targeting the framework versions, do we just target link the major number or the major and minor numbers
So do we number as
<major framework#>.<minor framework #>.

this WILL cause potential breaking changes leaking on updates for Application Insights, but not for the framework.

<major framework #>.<major Application Insights #>.

while this will avoid major breaking changes for Application Insights but will cause Typing signature compiling issues and not for the frameworks.

Examples
Using Targeting Application Insights major as minor (eg 2.8.1) with any version of react 18.x.x

18.2.8 (this could have TypeScript typing issue for cases where patch releases update the signatures (in a backward compatible way -- using a union type from AI perspective), but this causes an issue in that what version number do we use to represent an updated "patch" release to "fix" the typing issue?
18.2.0801 (We could do something like this? But it assumes we will never end up with a minor / patch number > 99)
And when Application Insights jumps to v3 (next major ES3 breaking change) it jumps to
-18.3.0000

Or using framework Major and minor)
React 18.0.x - 18.0.020801 (for React 18.0.x with Application Insights 2.8.1)
React 18.1.x - 18.1.020802 (for React 18.1.x with Application Insights 2.8.2)
React 18.2.x - 18.2.030000 (for React 18.2.x with Application Insights 3.0.0) <- This WILL likely have breaking changes from Application etc
Any other Suggestions?
Part of this is to also allow numbering "space" for releasing patches if required that would at least follow semver (mostly)

Move out to new repo and target AI version "ranges" (like we have done for the Angular Plugin), so we will only need to "publish" an updated version of the React / React-Native plugins when TypeScript typing changes or new features are required and also potentially decoupled from AI releases.

I would support this approach, because it seems to work for the angular one.
Is there a way to help?

@kirkone - Not at this point in time. But if/when we create the separate repos (which I think is likely) assistance will be more than welcome.

It's a complicate situation where it's difficult to have both world. Basically, the package is in the middle of a relation between IA and React. It should have it own independent numbering, but that would be even more confusing.
I would try to guess that more people are interested in developing React application and use AI than people would like to use AI and maybe use React. So I'd be more in favor of matching React numbering (for major).
But of course, the dilemma is how to then also keep track of AI and update it for multiple versions?
Anyway, I just saw light and I'm waiting for React 18 compatibility 😅
Good luck!

FYI - I've started the process of moving the react-js and react-native extensions to their own repo's.
So the next releases will be from those repos. For the initial releases they will most likely just unpin from the single specific versions of Application Insights SDK to allow upgrades.
Once these initial "unpinning" releases are done we can then look at re-versioning based on the major framework version.
As react-native is still 0.x.x we will probably keep its existing versioning until it goes prime time.

FYI - New repo's are now live and we are planning to perform an initial release from them tomorrow with the only difference will be that they will unpin from v2.8.5 of AI. And we will also validate the release pipeline for these new repo's.

If you need testers with the new solution, we'd be happy to help and provide feedback.

v3.3.6 (React 17) release from the new repo is successfully published.
I will probably be a few weeks before we can get some time to create a new React 18 release, unless someone wants to contribute by creating a PR to speed up the process 😸

Great job! It will make those libraries more maintainable.

Closing now that the react-js and react-native are in their own repo.
Plan is to wait until the next major release of the AI Sdk and then we will use start using the major number to match the framework (so React will become 18.x.x) react native will continue the current numbering (because it's 0.69.x)


Allow xmlattr to generate HTML boolean attributes

Expected Behavior
Passing a dict with boolean values to xmlattr should add them as HTML boolean attributes i.e. without a value or with its name as a value
Actual Behavior
The attribute gets inserted with True respectively False as its value
Template Code
{{ {'hidden': True}|xmlattr }}

Proposal
Add an option (e.g. html_style) to the xmlattr filter or create a new filter called htmlattr

For now I am happy with using the alternate syntax for HTML5 boolean attributes - i.e. hidden="hidden"


Latest commit causes index error on Windows

OS: Windows 10
PyQt version: 5.12.0

The latest commit (31463497c89928e7d2ea47d3e8bf5a953fdf8aeb) does not seem to be compatible with Windows file names:
Traceback (most recent call last):
  File "labelImg.py", line 1271, in openDirDialog
    self.importDirImages(targetDirPath)
  File "labelImg.py", line 1282, in importDirImages
    self.openNextImg()
  File "labelImg.py", line 1355, in openNextImg
    self.loadFile(filename)
  File "labelImg.py", line 1096, in loadFile
    self.showBoundingBoxFromAnnotationFile(filePath)
  File "labelImg.py", line 1114, in showBoundingBoxFromAnnotationFile
    filedir = filePath.split(basename)[0].split("/")[-2:-1][0]
IndexError: list index out of range

Specifically, ln 1114 in labelImg.py is: filedir = filePath.split(basename)[0].split("/")[-2:-1][0] which will always fail on Windows.

Open labelImg.py and change line 1114 to
filedir = filePath.split(basename)[0].split("\\")[-2:-1][0]

update the corresponding folder with following :
def show_bounding_box_from_annotation_file(self, file_path):
if self.default_save_dir is not None:
basename = os.path.basename(os.path.splitext(file_path)[0])
xml_path = os.path.join(self.default_save_dir, basename + XML_EXT)
txt_path = os.path.join(self.default_save_dir, basename + TXT_EXT)
json_path = os.path.join(self.default_save_dir, basename + JSON_EXT)

Isn’t it fixed? It should not exist

It was fixed by 62585531ca1a6e7450f1e108e137a3fe67365e68


remove omitempty specified in bool type value

Remove omitempty for bool type values in Create and Update.
This is because if omitempty is specified and false is specified, it will be omitted and the intended request will not be possible.

Codecov Report
Attention: Patch coverage is 50.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 39.60%. Comparing base (426f46a) to head (25bac77).

Files
Patch %
Lines

examples/create_event_webhook/main.go
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   39.56%   39.60%   +0.03%
==========================================
  Files          93       93
  Lines        2060     2063       +3
==========================================
+ Hits          815      817       +2
- Misses       1110     1111       +1
  Partials      135      135

Flag
Coverage Δ

unittests
39.60% <50.00%> (+0.03%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Fix attachments, tree structure and update logging

Fix attachments not uploading
Fix tree structure not uploading correctly
Add support for Confluence cloud authentication
Finish refactoring logging (mkdocs build -v will show debug logging)

@chalbersma - great work on your fork which leverages the Atlassian Python API client.
I could not get attachments to work and folder structure was not preserved. As I had to understand what was going on, I finished the logging refactoring you started and fixed my two issues.
Have a look, I will let you see how you want to contribute back to upstream.

Hey @ojacques Sorry I didn't notice this before.

But just a note, I just have a fork of this project, I'm not the upstream. I'm going to take your commit though.


strictest silently allows to pass {} or undefined as defs -> real dependencies are used

Looks like it's a normal behaviour for regular mode (and may be for strict), but my expectations is 'you MUST provide full defs object to get this work, tested module can't work with empty defs". Sure, it's only my expectations, but I propose to change docs and reflect that 'empty defs > no mocking', or throw runtime exception. Thx!
https://github.com/iambumblehead/esmock/pull/191

thanks for opening the issue. I'm very busy at this moment and do not know when I will be available to respond effectively.


nixos/keycloak: Use LoadCredential to load secrets + module formatting

Motivation for this change

Use LoadCredential to load secrets
Use systemd's LoadCredential mechanism to make the secret files available to the service.
This gets rid of the privileged part of the ExecPreStart script which only served to copy these files and assign the correct
permissions. There's been issues with this approach when used in combination with DynamicUser, where sometimes the user isn't created before the ExecPreStart script runs, causing the error

install: invalid user ‘keycloak’

This should fix that issue.
Unfortunately, all of the ExecPreStart script had to be moved to ExecStart, since credentials aren't provided to ExecPreStart. See https://github.com/systemd/systemd/issues/19604.

Reformat the module
Instead of referencing all library functions through lib. and builtins through builtins. at every invocation, inherit them into the appropriate scope. This is a follow-up to #154193.
Also, reformat the module with nixpkgs-fmt.

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Tested with my development servers, works well. Will merge tomorrow to give some time for other potential reviewers -- unless you want to merge it by yourself :)

Great! That plan sounds good to me :)

@talyz can you backport it manually?

Sure! Here it is: #155898


Update hdinsight-hadoop-create-linux-clusters-arm-templates.md

No Updated Needed

@ashishthaps1 : Thanks for your contribution! The author, @hrasheed-msft, has been notified to review your proposed change.

#sign-off


File deep link has a typo

The documentation for deep linking to a file has a typo in it. The deep link path should start with /l/file instead of /I/file. The character after the first slash should be a lower case "L" instead of an "I".

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

ID: 45541337-ecd7-5cf6-cbe4-3f0d2f39aa79
Version Independent ID: 6b4203f0-2844-06c9-eaa3-9cc03e6f76c0
Content: Create deep links - Teams
Content Source: msteams-platform/concepts/build-and-test/deep-links.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Raised PR for the fix: https://github.com/MicrosoftDocs/msteams-docs/pull/3187