Support Web Components v1

The dart:html library should deprecate the current v0 specifications of Custom Elements and Shadow DOM and move to using v1 of both specifications.

DDC related issue https://github.com/dart-lang/sdk/issues/27311

@floitschG anything to follow for this? Chrome 54 shipped and comes with the v1 specs enabled.

@donny-dont there are a LOT of things we need to fixup in dart:html – we're holding off until DDC is locked and loaded. It'll be much easier to get everything working when our dev story is 100% javascript.

when our dev story is 100% javascript

I'm not sure I understand this. Could you please add some more info. Is it JS compatibility you're referring to or the possible inclusion of javascript specific tools in the build step, along with DDC?

@kulshekhar the problem is the Dart VM in Dartium where during development some Dart code is directly executed by the Dart VM in Dartium  but other stuff runs in JS land.
With Bazel and DDC they are working on building a developer story with fast edit-reload cycles with standard Chrome (without a Dart VM being involved) to simplify interop between the Dart code (transpiled to JS) and the native JS world.

Thanks @zoechi
That makes sense now :sweat_smile:

Any updates on this, we're in 2017 now :-D

I imagine there won't be any progress on this until Dart 2.0 rolls out completely removing Dartium.
I'd check back in a few more months, or use JS interop to "build your own":
https://pub.dartlang.org/packages/html5

Who's still following this issue in 2018? Sorry, couldn't resist.

We're in the process of updating our dart:html APIs – https://dart-review.googlesource.com/c/sdk/+/24501
That roll – and getting rid of Dartium – should make it easier to support the latest Browser features.
Having said that, our primary framework – AngularDart – is not using WebComponent features, so it's not a big priority for us.
My suggestion: once we've updated dart:html and friends, let us know specific APIs that are missing/broken

That's really sad.
In my mind a serious "community centered" effort should kind of use and depend on different independent blocks. to make the idea more clear, AngularDart for sure needs some stuff handling DOM and it seems sane to me to use something like dart:html as a basis.
so, every work from google would be contributed back to the basis and others building on top of this basis would automatically benefit - somehow like the whole OpenSource-Idea, no? ;)
currently  Dart is more like: "You want to do web? Sure, use AngularDart." (Hint: AngularDart != "the web")

@elmcrest Hi Marius.
I realize and understand you are disappointed. I'm one of the TLs of AngularDart, and let me make it clear I think web components are awesome, and would love Dart to support them well. The primary take away here is well though, and that will take a little bit of effort, because the current HTML libraries and browser bindings aren't well suited for this.
We could probably get some hacky support quicker, but it won't be something useful to most customers. It's something we'd like to revisit after Dart2 launches, but until then - again to be very honest - it's just not a priority. We care, but we have limited time and resources, and have to use them efficiently, like I'm sure you do on your project(s) and company.
Thanks for your patience!

The entire point of Web Components is that it's a "framework" that is built-in to browsers. You can just use it, without any extra libs or 3rd party frameworks.
AngularDart emulates many features of Web Components, why does it do that instead of just using Web Components directly?

@eukreign:

The entire point of Web Components is that it's a "framework" that is built-in to browsers

Well, sort of. It has yet to be supported in Microsoft Edge, for example, which is quite a large market segment still, and the polyfill performs quite badly. So until it works everywhere, it's not free.

AngularDart emulates many features of Web Components, why does it do that instead of just using Web Components directly

AngularDart (and JS/TS, and React, and Vue, and Ember) all were created before web components were a thing. A lot of these framework authors have evaluated web components and found them severely lacking. Here is one of the leads of React, arguably the most popular framework today:

We’re not going to use it at all at Facebook. We’re not going to build React on it because there’s a strong model difference – imperative in Web Components to declarative in React. Web Components doesn’t have an idiomatic way to define things like where events go. How do you pass data when everything is a string? We see it more as an interop layer that lets various frameworks talk to each other.
In talking to the Atom team, this doesn’t solve different framework idioms as it doesn’t have an opinion on how they relate.

Again, it's not that support will never be offered, but it's unlikely to happen in any immediate time frame. There are other APIs that are more useful for our users, like web workers, service workers, and more right now.
Thanks for understanding!

@matanlurey would you also say that Dart Observatory is a simple script that would have been better written in JavaScript or TypeScript? Because that's built in Dart using Web Components V0 by Dart Team. It's in this very repo.
I've also built a fully featured spreadsheet and word-processor using Dart and Web Components V0.
I understand you have to shill for AngularDart since it's your project but people do build large, complex, web applications with Dart and without using AngulartDart.
The issue is that v0 is old and more browsers support v1, but Dart has not been keeping up.

I also disagree to prefer typescript over dart for little snippets. The undefined, NaN, 0, false, null, whatever hell is one point, imports and package management still another...
Dart can really shine even for small snippets and if your build system is already setup anyway, why wouldn‘t I want to use dart for everything?
One truth tends to get hidden/ignored more and more these days: Javascript sucks! ;)

I'm locking this conversation for now since it's getting non-productive. This is something that is interesting to us, but it's not on the immediate roadmap at this time.

We don't plan to support this API in the short term. None of our core users require it and would involve substantial effort to implement it correctly.


Need support for TypeScript React projects

It'd be really nice to have static types, at least for the core features of the library.

I'm currently very busy but I will have a look into it soon.

Hey again, just wanted to let you know that typescript is on its way. It‘s basically finished and just needs some testing.

That's great news.. thanks!

While testing goes on you can already install the release candidate if you want.
yarn add react-parallax@next

Typescript is in master now and also published with version 3.0.0

Thank you. I'll upgrade now.


[FLINK-21509][python] Add 'withProcessingTime()' method call when creating proctime slide window assigner in 'StreamExecPythonGroupWindowAggregate' class

What is the purpose of the change
This pull request adds 'withProcessingTime()' method call when creating proctime slide window assigner in 'StreamExecPythonGroupWindowAggregate' class
Brief change log

Add 'withProcessingTime()' method call when creating proctime slide window assigner in 'StreamExecPythonGroupWindowAggregate' class.

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

The e2e is timeout and is not related to this PR.
Will merge this one first.


exp/lighthorizon: Refactor codebase to avoid using HistoryArchiveBackend as a filesystem interface

Remove kludges around using historyarchive.ArchiveBackend as a filesystem interface.
Currently, we're using historyarchive.ArchiveBackend as a way to access raw ledgers from the filesystem. This is pretty hacky since these aren't raw LedgerCloseMetas but rather a custom format on top of them (see #4450), so it doesn't make sense as a LedgerBackend anyway, plus we want to be able to use the version and maybe read other config data from the directory.
There's a little bit more context here; the general point is that we shouldn't hack in the ledgerbackend interface for Horizon Lite.

@Shaptic I'm catching up after Starbridge so let me know if I understand this correctly: LedgerCloseMeta archives will be completely separate from existing archives generated by Stellar-Core. If it's true, maybe we should leave historyarchive.ArchiveBackend (and historyarchive package in general) as is and just create a new ledgermetaarchive package solely for new archive purposes.
Also, I think the design of ArchiveBackend as filesystem makes sense because it's used to access archives in different backends like http, s3 or local file system. I think this can be also true for new archives so in the end we could extract ArchiveBackend to a separate helper package shared by historyarchive and ledgermetaarchive.
WDYT?

LedgerCloseMeta archives will be completely separate from existing archives generated by Stellar-Core.

Yes, definitely separate and most likely storing add'l information or in a different format (i.e. not exclusively a serialized LedgerCloseMeta).
If it's true, maybe we should leave historyarchive.ArchiveBackend (and historyarchive package in general) as is and just create a new ledgermetaarchive package solely for new archive purposes.
That's a great idea! For context, though, the HistoryArchiveBackend and the FsCacheArchiveBackend were both written by @paulbellamy specifically for Light Horizon. That means we can remove them from the historyarchive package and refactor them in whatever form we'd like for the ledgermetaarchive package.
I do agree that we should (and must, in fact) keep it backend-agnostic. The main pain point is that ArchiveBackend.GetLedger() returns a LedgerCloseMeta, while we want it to return our format (for now called SerializedLedgerCloseMeta). So I'm not sure if extracting it entirely into a separate package is the right idea because of that, but I do like it in principle because we have this "backend-agnostic filesystem emulation" paradigm in quite a few places.

🤔 So I spend a little more checking the latest code in lighthorizon branch and I think the current structure is fine (I got confused by ArchiveBackend vs ArchiveInterface case above):

FsCacheBackend is really just getting files using embedded ArchiveBackend with caching built-in.
HistoryArchiveBackend is implementing LedgerBackend. It's using ArchiveBackend simply for getting raw bytes and actual decoding happens inside HistoryArchiveBackend which looks good. The only thing I'd change is maybe a name because now it's easy to mistake it with normal history archives (maybe LedgerMetaArchiveBackend?).

I wonder if we should touch this at all. Eventually we could create the new package (ledgermetaarchive as mentioned above) that encapsulates the structure of files and also contains the code responsible for pushing the files (using one of the existing ArchiveBackends). Then the HistoryArchiveBackend is just a super thin adapter for  ledgermetaarchive struct. But maybe such refactoring is not urgent now? We can have a quick chat if you want.

maybe LedgerMetaArchiveBackend

+1 :+1:

I got confused by ArchiveBackend vs ArchiveInterface case above

I think I did too, apologies for that :facepalm: I think everywhere I said historyarchive.ArchiveBackend, I actually meant ingest.LedgerBackend.
I think the main concern is that LedgerBackend.GetLedger() returns xdr.LedgerCloseMeta when we probably want it to return xdr.SerializedLedgerCloseMeta so that we can easily add fields there in the future and have them accessible from Light Horizon. Of course this isn't critical now since there's only the v0 right now that returns only the meta.
But already for example it's slightly problematic that a repository of unpacked ledgers like the one @2opremio just made is not directly associated with a network (pubnet or testnet). The network passphrase isn't stored anywhere and it probably should be. Maybe not per ledger but definitely per repository. And we'd need a way to access this from Light Horizon, but the LedgerBackend interface is too... restricting? Maybe we can keep it implemented, but add more functionality?
We also have another Archive interface in exp/lighthorizon/archive that's a weird amalgamation of a subset of LedgerBackend and TransactionReader... It describes the minimum necessary for Light Horizon.
Basically, getting to a single sane state would be great :joy: As a whole, I'm not entirely sure about the best way to refactor this all, or if it's even necessary as you noted, but I want to throw out that there will be more requirements on the backend interface soon, so we should handle that sooner rather than later if we can. I think I'll leave it up to you, since your picture of the codebase is roughly the same as mine now :+1:

I think that https://github.com/stellar/go/pull/4488 implemented the changes we discussed here.


Suggestion: temporarily disable desktop app popup for Linux

I had a go at implementing this myself but ran into trouble with dependencies.
david@kittycad:~/kc-repos/modeling-app/src-tauri$ cargo add tauri-plugin-os
    Updating crates.io index
      Adding tauri-plugin-os v2.0.0-beta.3 to dependencies.
    Updating crates.io index
error: failed to select a version for `webkit2gtk-sys`.
    ... required by package `webkit2gtk v2.0.1`
    ... which satisfies dependency `webkit2gtk = "=2.0.1"` of package `tauri v2.0.0-beta.12`
    ... which satisfies dependency `tauri = "^2.0.0-beta.12"` of package `tauri-plugin-os v2.0.0-beta.3`
    ... which satisfies dependency `tauri-plugin-os = "^2.0.0-beta.3"` of package `app v0.1.0 (/home/david/kc-repos/modeling-app/src-tauri)`
versions that meet the requirements `^2.0.1` are: 2.0.1

the package `webkit2gtk-sys` links to the native library `web_kit2`, but it conflicts with a previous package which links to `web_kit2` as well:
package `webkit2gtk-sys v0.18.0`
    ... which satisfies dependency `ffi = "^0.18"` of package `webkit2gtk v0.18.2`
    ... which satisfies dependency `webkit2gtk = "^0.18.2"` of package `tauri v1.6.1`
    ... which satisfies dependency `tauri = "^1.6.1"` of package `app v0.1.0 (/home/david/kc-repos/modeling-app/src-tauri)`
Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the `links = "web_kit2"` value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

failed to select a version for `webkit2gtk-sys` which could resolve this conflict

Frank fixed


You could add dependencies for Fedora/Suse/CentOS ... in the Wiki

I successfully build on Fedora 29 with the following packages. (I could try to make a RPM on COPR https://copr.fedorainfracloud.org/coprs/fulltext/?fulltext=zyn-fusion but it is easier and safer to compile from source.)
sudo dnf install @"C Development Tools and Libraries" git ruby libtool mxml-devel automake cmake fftw-devel jack-audio-connection-kit-devel liblo-devel zlib-devel alsa-lib-devel  libX11-devel  mesa-libGL-devel cairo-devel fontconfig-devel  python bison

Note: @"C Development Tools and Libraries"  includes bison, make, gcc and other stuff, maybe it could be optimized.  But at least it is a start.

It would be great to expand build instructions to make it simpler on
other distros.
From what I recall, the wiki should be publicly editable, so if you
have a package list then I'd recommend adding it with any caveats about
your personal setup.

I see no way to edit/create Wiki pages  and  GitHub features no Pull Request for them.
In fact, creating a BUILD.md next to the README.md might be easier to deal with :-)

There should be an edit button in the upper right of each wiki page. I don't think there even is an option for collaborator only wiki's. From the github repo setting page:

GitHub Wikis is a simple way to let others contribute content. Any GitHub user can create and edit pages to use for documentation, examples, support, or anything you wish.

I checked again, but there is no Edit button anywhere in the page.
The best I can do is to see the 6 revisions by Victoria, that's all. No Edit, No create, No delete.

As it turns out I'm in need of a new pair of glasses.

I have now fixed the setting.

OK, done. Hope there will be no SPAM and defacing ...


Unexpected token import

I downloaded this template and ran npm install as well as npm run dev.
When the browser starts, I see a blank page and this error message in console:

The error seems to occur here in main.js:

So, first I thought something was wrong with transpiling ES6 to ES5 in the bundled app.js but it looks fine: app.js.zip
What's going on?
Using Node v6.2.0

I fresh cloned template and i used npm i after that npm run dev and everything worked perfect.
I tested npm run build command too, it worked perfectly too.
I using node 7.2.1.

Very weird, could be the node version, I guess

Jep, it's definitely a Node version issue. Tested with v7.3.0 and now it works.


Invoice

The last one is working Invoice serialize the biggest one need to go step by step and see how to perform it on the end by XML file and validate it

I am working to serialize the Invoice class to create an endpoint that will be.

I need with DomDocment to create the XML file that will be passed to validation.

Directive 2014/55/EU led to the definition of the European standard on eInvoicing (EN 16931). The Directive promotes the uptake of eInvoicing in public procurement. I am working with this standard

https://peppol.helger.com/wsdvs?wsdl=1
This is for documentation service validation defines.

SoapClient can check that if our document will be passed or not.

https://peppol.helger.com/public/locale-en_US/menuitem-en16931
Technical specification and requirements

Invoice object I am just working for serializing them every object that we have for now most of them done, but the base class Invoice need to working

https://github.com/ConnectingEurope/eInvoicing-EN16931


qwen2.5-coder isn't utilizing the GPU

What is the issue?
When running a query with qwen2.5 (32b) it only uses the CPU for some reason. I can switch to another model (llama, phi, gemma) and they all utilize the GPU.
Reproduce:

Run docker in an Ubuntu container on an standalone server
Install Ollama and Open-Webui
Download models qwen2.5-coder:32b and another model like llama3.2
Run a query on llama3.2 and use nvtop, where you have ollama installed, to see GPU usage
Run a query on qwen2.5 and use nvtop to see GPU usage. It should show similar to step 4 but it doesn't.

OS
Linux, Docker
GPU
Nvidia
CPU
Intel
Ollama version
0.4.1

qwen2.5:32b is 20GB, llama3.2 is 2GB.
Server logs will show memory allocation, and ollama ps will show how much of the model is residing in GPU vs CPU.


Docker

this PR adds a Dockerfile.
this Dockerfile is identical to that of the Search-Portal repo, except for the webpack bundle location.

i learned after starting this PR that Gatsby provides a base [Docker image](https://github.com/gatsbyjs/gatsby-docker, so this became a bit more simple.
confirmed this works with the following commands:
docker build -t neurobridge/website .

and
docker run --rm -p 80:80 neurobridge/website


removed broken utf-8 BOM

Python's json module won't load the two response/status test cases, because of a unicode character at the start of the file. (github's diff doesn't show it, but git diff on the terminal will.)

Thanks.


what is the "testa.txt" in the file named pretrain.py?

I'm a noob in NLP. I want to run this project and understand it fully to learn more about NLP. But when I ran python pretrain.py, I found testa.txt was missing. Could u tell me what's that. Maybe dev dataset of the train.jsonl?

I'm a noob in NLP. I want to run this project and understand it fully to learn more about NLP. But when I ran python pretrain.py, I found testa.txt was missing. Could u tell me what's that. Maybe dev dataset of the train.jsonl?
add my wechat and i will give you a hand,(ID CreepUI)


Dropping support for Node.js 6 and npm 3

Drop support for:

Node.js 6
npm 3 and npm-shrinkwrap.json file

Remove scripts to sanitize npm-shrinkwrap.json
Update docs and NEWS

Should I merge or you want me to pass some acceptance?

GitHub says that the branch has conflicts :trollface:. Let's fix them and deploy to staging to test that everything works properly, just in case.

Let's fix them and deploy to staging to test that everything works properly, just in case.

✅ Done!


Extension issue - FairyGUI BMFont don't load

axmol version: release [40e7b2cee7]
devices test on: iOS Simulator, Samsung J5 Pro
developing environments

NDK version: r23c
Xcode version: 14.1
cmake version:

Steps to Reproduce:

Create a FairyGUI project
Import a bitmap font in the FairyGUI project
Use the font on a text field in a FairyGUI component
Load the component in a scene

The the text field does not display the selected font, and there is an error message in the console log:

No file found at ui://[fairygui_url]. Possible missing file.

Source snippet?

bool HelloWorld::init() {
    if (!Scene::init())
    {
        return false;
    }

    _groot = GRoot::create(this);
    _groot->retain();

    FileUtils::getInstance()->addSearchPath("fui");
    UIPackage::addPackage("Futura_White_Black_128");
    UIPackage::addPackage("HexGrid");
    GObject *grid = UIPackage::createObject("HexGrid", "HexGrid");
    grid->setPosition(_groot->getWidth() / 2.f, _groot->getHeight() / 2.f);
    _groot->addChild(grid);

    return true;
}

Futura_White_Black_128 is a bitmap font. There is a text field using this font in the HexGrid component. The component is displayed, but the text field appears with a wrong font and I get the following message:

axmol: fullPathForFilename: No file found at ui://6jswxvktmvpka. Possible missing file.

Where is this font (path correct)?

It is not the issue. This glitch is also present in the fairguy demo.

Wich fargui test?

axmol/tests/fairyguitest-tests

Which one please.

@halx99:
maybe a label like:
'extension issue' is also helpfully?

It seems to be a FairyGUI bug (see preview)
I checked it on the editor:

Here the preview on the editor (Bitmap Font is not correct)

I just ran it with Cocos2d-x 4.0 and it works perfectly fine. So I think the glitches are caused by some of the changes in FontAtlasCache or FileUtils.

@theSlyest
maybe the title is wrong?

@theSlyest Any chance you could try to re-add the code removed in this commit and see if it works correctly?
https://github.com/axmolengine/axmol/commit/6f4fe63208f768f1f297adb81dc23f8a0e1b72f7

@theSlyest Any chance you could try to re-add the code removed in this commit and see if it works correctly?
6f4fe63

Ok, I'll try

It requires more changes to be reverted. I can not test it for now.

@theSlyest Any chance you could try to re-add the code removed in this commit and see if it works correctly?
6f4fe63

Rebuild with the "latest" changes BEFORE 6f4fe63
BitmapFont issue is same

Test it on axmol-adxe-1.0-b3 also.  (VS2022 64bit)
Same result: BitmapFonts on FairyGui be not correct (using system font)
I have no longer VS2019, I cannot build older versions  of axmol.

adxe-1.0-a9 is the first one with the extension FairyGUI

adxe-1.0-a9 is the first one with the extension FairyGUI

I'll test out that version since I still have VS2019 installed.

No luck testing adxe-1.0-a9, since I can't download the extensions from anywhere:
==> Prepare to download external libraries!
==> version file doesn't exist
==> Ready to download 'v56.zip' from 'https://github.com/c4games/engine-x-3rd/archive/v56.zip'
==> Error: Could not find the file from url: 'https://github.com/c4games/engine-x-3rd/archive/v56.zip'
==> Http request failed, error code: 404, reason: Not Found

That link doesn't exist any longer, and I don't have copies of the external library zip files. @halx99 Would you by any chance have all the external zip files somewhere, and if possible, upload them somewhere, like the https://github.com/axmolengine/archive repo?

I prefer to check this issue in current version

I prefer to check this issue in current version

Fair enough, then no point wasting time testing the older versions.


为什么官方api文档还是js版的。。。<EMAIL_ADDRESS>

RxJS version:
Code to reproduce:
Expected behavior:
Actual behavior:
Additional information:
please（请） update (更新)  一下 api 文档 （ts版本的）。
api文档里面的大部分还是js方法。

Just a bit translation
Why is the official API documentation using JS code? I am using<EMAIL_ADDRESS>Please update the ts-version of API documentation, which is still mostly referring js code and method.

@haoliangyu thank you for the translation.
@lei1996 we are currently working on updated documentation that will derive from the TypeScript files directly. Currently we're using ESDoc to generate the documentation, and that requires us to build to ES2015, then generate docs from that. That is why you see ES2015 when you click the file links. My apologies for the inconvenience.

closing, for documentation progress, you can check https://github.com/ReactiveX/rxjs-docs .


KAFKA-5532: Making bootstrap.servers property a first citizen option for the ProducerPerformance

Added bootstrap-server as option for specifying Kafka cluster to connect to

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/5766/
Test PASSed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/5752/
Test PASSed (JDK 8 and Scala 2.12).

@ijuma what do you think about that ?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6558/
Test PASSed (JDK 8 and Scala 2.12).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6573/
Test PASSed (JDK 7 and Scala 2.11).

@ijuma do you think that it's ok now ?

Can we add a simple unit test?

@ijuma I agree but in this case I need to startup an "embedded" Kafka cluster before launching the main. I see that something like that is provided by the EmbeddedKafkaCluster class but it is in the streams module and of course there is no dependency with the tools module.
I see that the tests for the tools in the core module extends the KafkaServerTestHarness class so something like the embedded cluster as before.
Some other classes like the ConsoleConsumer are more testable because for example the ConsoleConsumer expose the ConsumerConfig method useful for verifying args and process for executing the action.
What about the ProducerPerformance case ?

We could have a method that returns the Properties used by the producer. The unit test would simply check that the property was populated correctly. No need for starting the Kafka broker.

+1

@ijuma this kind of simple test it's fine for me but it means that I have to split the ProducerPerformance logic because just calling the main method will start the producer (other than set the configuration) so the cluster is needed. I'm going to split the logic in a way that we can call a configuration method and run from the unit test but they will be called inside the main method so that nothing changes from a user and kafka-producer-perf-test.sh perspective.

Yes, sure. That kind of refactoring is good as it makes the code more testable.

@ijuma sorry for the delay on committing the refactored version for having code more testable but I thought about it and it turned our (for me) that this PR is a good way for starting to add the CommandOptions base class I have already introduced in the WIP #3514 (on TopicCommand work) which other developers are using/waiting for a trunk merged version in order to use it for other tools refactoring (as KAFKA-3268).

@ijuma related to the 1.0.0 release, this PR add a new flag but it's a refactoring as well. Do you think we can get it into 1.0.0 with a review ? It adds some classes useful for future use of Java for tools, so maybe it could be good having them from this release. Wdyt ?

@ijuma I got some feedback from @viktorsomogyi and applied in some way his suggestion. Maybe you could have a review on that for going ahead ? Thanks !

FAILURE
No test results found.
--none--

FAILURE
5891 tests run, 0 skipped, 23 failed.
--none--

FAILURE
5891 tests run, 0 skipped, 23 failed.
--none--

@ijuma just removed sorry. Maybe we could have **/out in the .gitignore (I'm going to open a MINOR PR for that).

FAILURE
No test results found.
--none--

FAILURE
5891 tests run, 0 skipped, 23 failed.
--none--

FAILURE
5892 tests run, 0 skipped, 1 failed.
--none--

FAILURE
4021 tests run, 1 skipped, 1 failed.
--none--

SUCCESS
8073 tests run, 5 skipped, 0 failed.
--none--

SUCCESS
8073 tests run, 5 skipped, 0 failed.
--none--

@ijuma no other updates/requests on this PR after 15 days ?


GeekDetails's modal is not configured to work in dark mode

https://github.com/tigera/docs/blob/45b547a91b6901fce2e6e266fe7683ed63d6dc8a/calico_versioned_docs/version-3.29/getting-started/kubernetes/quickstart.mdx?plain=1#L50-L53
This is how the popup is rendered in the website when using dark mode:

I have to select the text to be able to read it:

@ctauchen fix is live

@ronanc-tigera Thanks!


UniversalLink and ConditionalLink accepts also an item to link to.

if item is of @type Link (has the remoteUrl property), a direct link to remoteUrl is generated if user is not logged.
This avoids the passage from the Link detail view page even when users are not authenticated. Especially when Link-type objects appears in listing blocks.

Can we have some documentation for the UniversalLink? It starts to get big and handle all sort of cases.

where we could put this components'documentation?

One option would be a story for Storybook, something like this:
https://github.com/plone/volto/blob/7216d1db01ee391332014152af37ca1ef7cb883b/src/components/manage/Toolbar/Dropdown.stories.js
Or a new page under https://docs.voltocms.com/recipes/

One option would be a story for Storybook, something like this:
https://github.com/plone/volto/blob/7216d1db01ee391332014152af37ca1ef7cb883b/src/components/manage/Toolbar/Dropdown.stories.js
Or a new page under https://docs.voltocms.com/recipes/

i've added the storybook for UniversalLink


workspace.horizontalLayout broken

It always returns false now in the horizontal playground. I think it's because you switched to having the flyout on the left

I'm not sure I understand. When you switch the layout to top/bottom, it looks to me like workspace.horizontalLayout is true. That property refers to whether the toolbox/flyout/palette is displayed horizontally, not whether the blocks are rendered horizontally - is that what you mean?

Oh whoops, my mistake. Is there a way to check which grammar we're using?

Hmm, I don't think there's a good way to check that from the outside right now. One "hack" way you could do it for now is checking properties that are defined in core/block_render_svg_horizontal.js but not core/block_render_svg_vertical.js.
E.g.,
var isHorizontalRendering = typeof Blockly.BlockSvg.IMAGE_FIELD_WIDTH !== 'undefined';
I warn you that this is pretty fragile, and any of these properties are subject to change at any time! Hope that helps.


Adding information on Visa and how to handle LOI requests.

Based on the discussions that happened in the HHEU group chat.
Will probably need improvements.

lgtm

I would add a fairly strongly worded disclaimer in there about how this isn't official advice and that organisers should definitely seek advice from their university / SU in relation to this. Maybe something like this?

Note that this information does not constitute official advice, merely outlines experiences previous hackathon organisers have gone through when trying to handle LOIs. You should always reach out to your university or students' union for advice regarding visas. The gov.uk website also has some useful advice.

Ah! I knew I was forgetting something.


Open the error polymorphic variant

val of_string : string -> (t, error) result

to:
val of_string : string -> (t, [> error ]) result

It's done but I should do a release.


problem permission add

Laravel Version: #.#.#
Voyager Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
Solved: fix the error this way
$this->authorize('edit', app($dataType->model_name));
to
$this->authorize('add', app($dataType->model_name));
in TCG\Voyager\Http\Controllers\VoyagerBreadController.php line 286
Steps To Reproduce:

All fields are required (versions, a description of the issue, and steps to reproduce the issue).  It seems you've provided only a description of the solution, not the issue.
Please update the issue appropriately and we'll reopen this.


Beginning of Report - Title and Introduction

This PR contains the Title of our report along with the introduction and inspiration of the analysis we will be conducting. The reasoning behind the analysis, history, and application of the report in the future is explained here.

Looks good to me! It is a great research topic, thanks Alan!


Replacing normalize by use of StandardScaler

Normalize' will be removed in the version 1.2 release
of sklearn. Adding Pipeline with a StandardScaler as
a preprocessing stage.

Reference Issues/PRs
Fixes half of #2143
What does this implement/fix? Explain your changes.
Normalize' will be removed in the version 1.2 release
of sklearn. Adding Pipeline with a StandardScaler as
a preprocessing stage.
Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?
Clarity of git commit message
Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Excellent, @KishenSharma6!
There are a couple linting errors (whitespace, newlines etc), kindly check the "details" in the code-quality test.
I strongly recommend you set up linters and possibly pre-commit locally (see the dev docs for that).

@fkiraly That was incredibly helpful feedback, thank you.
I was able to set up pre-commit locally, I believe this is ready to be merged to the main branch. Please let me know if this requires further action, I will start on finishing the rest of https://github.com/alan-turing-institute/sktime/issues/2143

Normalize' will be removed in the version 1.2 release
of sklearn. Adding Pipeline with a StandardScaler as
a preprocessing stage.
Fixes half of #2143


Invalid vertex name raised in xgb.plot.deepness

Someone sent me this via email:
@pommedeterresautee Could you take a look? I am able to reproduce under CRAN version.
require(xgboost)
# load in the agaricus dataset
data(agaricus.train, package='xgboost')
data(agaricus.test, package='xgboost')
dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
dtest <- xgb.DMatrix(agaricus.test$data, label = agaricus.test$label)

param <- list(objective = "binary:logistic", booster = "gblinear",
              nthread = 2, alpha = 0.0001, lambda = 1)

watchlist <- list(eval = dtest, train = dtrain)
num_round <- 2
bst <- xgb.train(param, dtrain, num_round, watchlist)
ypred <- predict(bst, dtest)
labels <- getinfo(dtest, 'label')
cat('error of preds=', mean(as.numeric(ypred>0.5)!=labels),'\n')

xgb.plot.deepness(model=bst)

Errors:
Error in as.igraph.vs(graph, from) : Invalid vertex name(s)
In addition: Warning messages:
1: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 2 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
2: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 3 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
3: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 4 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
4: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 5 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
5: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 6 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
6: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 7 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
7: In data.table(ID = c(idBranch, idLeaf), Feature = c(featureBranch,  :
  Item 8 is of size 120 but maximum size is 121 (recycled leaving remainder of 1 items)
8: In as.igraph.vs(graph, from) : NAs introduced by coercion

I have not checked yet but  booster = "gblinear" looks suspicious when you want to display tree deepness...

Oops, I didn't pay much attention to that. Perhaps we can add a check there to give more informative error.

I don't know if it s a good idea. I mean R is not typed, we can add hundred of cases and check in the code to avoid these cases. But it is a lot of work and not sure it will be very useful. I mean, checking that the number if trees is a positive integers makes sense: easy to check + lots of possible errors + parameters used by all users.
But checking that the model provided match our expectation to plot it: hard to check + few people will have this kind of error + require to adapt the checks for each modification of the core code.
However, if other contributora think it is useful, I ll do it.

While the xgb.plot.deepness does check that the columns in a tree table
are as expected, the issue stems from xgb.model.dt.tree returning a
nonsensical tree table for a gblinear model. It would be fairly easy to add
an extra check to xgb.model.dt.tree to prevent it from trying to create a
tree table from a text dump that doesn't look like a tree booster dump.
On Wed, Aug 3, 2016 at 7:22 AM, Michaël Benesty<EMAIL_ADDRESS>wrote:

I don't know if it s a good idea. I mean R is not typed, we can add
hundred of cases and check in the code to avoid these cases. But it is a
lot of work and not sure it will be very useful. I mean, checking that the
number if trees is a positive integers makes sense: easy to check + lots of
possible errors + parameters used by all users.
But checking that the model provided match our expectation to plot it:
hard to check + few people will have this kind of error + require to adapt
the checks for each modification of the core code.
However, if other contributora think it is useful, I ll do it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/1434#issuecomment-237222028, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABie7x0Gnlm0_2kxyMUQObLQxhf8QKpIks5qcIgfgaJpZM4JaxYk
.


Prevent last column on Tree page from wraping

With localizations, there can be longer text then 13em so there is a line break.
This fix prevent this line breaking and thus every line has normal height again.

Thanks!


NO-ISSUE: Fix weekly Jenkins job build params

I think the optaplanner:main now should map to quickstarts: development, no other option valid from now on.


Adjust the composer.json and run-test.yml according to template

I adjusted the composer.json file to remove the requirements that were superfluous. I also copied the run-test.yaml from the escape-room package over to this package.

Need to get those .DS_STORE files removed from this PR, along w/an addition to .gitignore to keep them out.


merge from template-sync/template-arlac77-github

package.json

chore(deps): add ^15.6.2 remove ^15.6.1 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v1.0.44
GitHub release

Your semantic-release bot :package::rocket:


Incorrect widget class name in example

The class name in example at pub.dev page is InfinitePagerView, should be InfiniteViewPager

the documentation did not mention it. Please can you suggest example?

the documentation did not mention it. Please can you suggest example?


Release new minor releases of Humanitec Plugins

Motivation
I want to make sure that our automated publishing works
Approach
Bump all Humanitec packages by a minor release

📣 NOTIFICATION
You are receiving this message because we did not publish any packages.
Generated by @thefrontside/actions


Define what --clean does better

The help for collect is currently:
usage: cloudmapper.py [-h] [--config CONFIG] [--account ACCOUNT_NAME]
                      [--profile PROFILE_NAME] [--clean]

optional arguments:
  -h, --help            show this help message and exit
  --config CONFIG       Config file name
  --account ACCOUNT_NAME
                        Account to collect from
  --profile PROFILE_NAME
                        AWS profile name
  --clean               Remove any existing data for the account before
                        gathering

Explain that --clean only removes the local, previously collected data.

Done so in https://github.com/duo-labs/cloudmapper/pull/850


(nuclio) Failed to start yolo-v3-tf

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Hello, I deployed successfully yolo-v3-tf
nuctl deploy --project-name cvat \
  --path serverless/openvino/omz/public/yolo-v3-tf/nuclio \
  --volume `pwd`/serverless/common:/opt/nuclio/common \
  --platform local

Then I tried to test the deployed model
image=$(curl https://upload.wikimedia.org/wikipedia/en/7/7d/Lenna_%28test_image%29.png --output - | base64 | tr -d '\n')
cat << EOF > /tmp/input.json
{"image": "$image"}
EOF
cat /tmp/input.json | nuctl invoke openvino.omz.public.yolo-v3-tf -c 'application/json'

I got this error
Error - Function not found: openvino.omz.public.yolo-v3-tf @ nuclio
    /nuclio/pkg/nuctl/command/invoke.go:132

Call stack:
Failed to invoke function
    /nuclio/pkg/nuctl/command/invoke.go:132

How can I solve it?
Thank you.
Your Environment
Docker 23.0.6
KDE neon 5.27
Nuctl 1.8.14

@rongirl

Hello, I deployed successfully yolo-v3-tf

Nuclio literally says that the model does not exist. What is output of nuctl get functions?


build error

When I run build，I get an error:  triggerUncaughtException(err, true /* fromPromise */)

Please provide a minimum reproducible code repository, which can help me locate the problem.

@luxi-record same issue here, just wondering if you can share a fix?


Set device wallpaper to Unlanucher theme's background color

During normal interaction with a device, there are several places where the device's wallpaper is still visible (e.g. the lockscreen, #22).  It would make for a much cleaner, end-to-end experience if Unlauncher would set the device's wallpaper to match the background color from the Unlauncher theme.

https://stackoverflow.com/questions/20053919/programmatically-set-android-phones-background

@jkuester I wonder if the app should ask the user if the wallpaper should be changed on initial startup or when it is set as the device's main launcher?


chore: Improve Carbon ad style

Minor adjustment to the style to match the site overall theme.

Remove the text underline
Set the font-size smaller with dynamic line-height
Set the hover color to match the CSS --link-color

@nzakas kindly review and feel free to make adjusment as needed.

@sayzlim please be sure to sign the CLA

@nzakas signed!


Add signalfd and related to android

Looks like CI is failing?

@alexcrichton Yes, 64-bit android builds are failing because the 64-bit fields within signalfd_siginfo are the wrong size I guess. This doesn't jive with what I'm seeing in the source headers anywhere, however, so I'm trying to figure out what's going on here. I'll come back to this in the coming days and see if I can sort it out.

@ndusart You updated the Android NDK version recently, so I'm wondering if you have any familiarity with Android APIs here. As far as I can see when searching the same headers I find that signalfd_siginfo field is defined thesame for all architectures. I even checked the __u64 is the same on all platforms, as it is. Is there something I'm missing about 64-bit versus 32-bit Android platforms?

These fields are 64bits on each platforms but there are not the same type.
__u64 is a typedef to unsigned long long (defined in asm-generic/int-ll64.h, included from asm-generic/types.h)
and uint64_t is only unsigned long long on 32-bits architectures, not 64-bits where it is unsigned long (see stdint.h)
you should use ::c_ulonglong here

So uint64_t will be 64-bits on all platforms while __u64 will vary? Ugh that's some confusing naming. I figured that had to be the case but couldn't believe it given their naming.
@alexcrichton This brings up a good point though with typedefs, what should I be using for datatypes here? __u64 and family is defined on Linux, but not android. Should I be trying to use the exact same types with all the typedef facade (define __u64 and friends and use them), should I use equivalent types (c_ulonglong instead of __u64), or should I use explicit types (u32 and u64) depending on the platform?

BTW @ndusart Thanks for looking at this!

Explicit types will not work either as ctest translate them to the stdint types, so you'll have to same behavior as using uint64_t.
I didn't look if __u64 was defined in this crate, It's probably better to define __u64 in android as well then.

@Susurrus I'd only require a type with the same size/align/sign, other than that I don't think it's necessary to strive to have the same exact type as C, there's no way that we're "correct" in all other structs.

No, __u64 should be 64-bits on all platforms. It's just that one decided to use unsigned long long on all platforms while the other chose to use unsigned long on 64 bits. These are different C types.

Ahh, I think I got it! While the bit-width is the same the types are different and the compiler complains. Makes sense.

@Susurrus I'd only require a type with the same size/align/sign, other than that I don't think it's necessary to strive to have the same exact type as C, there's no way that we're "correct" in all other structs.

Alright, I did it the minimally-invasive way, using c_ulonglong directly instead of defining the __u64 type. Looks ugly, but it should be correct.

Looks like CI is still red?

@alexcrichton Fixed.

@bors: r+

:pushpin: Commit 83ab9a3 has been approved by alexcrichton

:hourglass: Testing commit 83ab9a341f4cfeab7f96e5e6dd9ae6ee49aa5468 with merge 516df7f77331a253b41cf743178392aa9ad66f50...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 516df7f77331a253b41cf743178392aa9ad66f50 to master...


TypeScript preserves 'override' modifier on properties in JavaScript esnext output

Bug Report
🔎 Search Terms
override, property, esnext
🕗 Version & Regression Information
Bug in new feature of version 4.3
💻 Code
class Base {
    state?: string;
}

export default class TestClass extends Base {
    override state = "string";
}

🙁 Actual behavior
Compiling this code for target esnext the override keyword on properties is preserved:
tsc -t esnext TestClass.ts

class Base {
    state;
}
export default class TestClass extends Base {
    override state = "string";
}

🙂 Expected behavior
The override keyword should be removed from the javascript output, as it is already done correctly for methods.
(Compiling for other targets (e.g. es2021) works correctly as the property is transformed to a constructor initialization.)
Maybe related to #43535

CC @Kingwl

We have tests for this, we just didn't read them correctly.

Thanks @a-tarasyuk!


Redesign network tab in VM template form

Description
Redistribute elements for a more defined and minimalist design
Use case
Make the UX more comfortable and attractive
Interface Changes

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

also consider changes to the attach nic dialog on virtual router view

to be redesign in #5422


RFC:

Proposal
Clear and detailed description of the proposal.
When there are implicit arguments to the goal or a hypothesis, it is sometimes possible to invoke the apply tactic and get back the message, apparently generated by [this line],(https://github.com/leanprover/lean4/blob/5f59d7f7b4b2f15a2af2296a3dbc5ff572686de6/src/Lean/Meta/Tactic/Apply.lean#L53):
failed to unify {indentExpr eType}
with {indentExpr targetType}

Where the displayed values for the two types are exactly the same. See, for example, this comment.
When this happens, I propose appending the following message to the end of that error message which says
"If these terms seem like they should unify, it is possible that implicit arguments do not match. You can try using set_option pp.explicit true to see the full type".
Consider the following questions:

User Experience: How does this feature improve the user experience?

It affects the user experience only for people who reach this exact state, with the pretty-printed types being the same. It improves that experience by suggesting something, rather than unhelpfully saying that two terms that look identical can't be unified.

Beneficiaries: Which Lean users and projects benefit most from this feature/change?

This will most help new users, or users unfamiliar with the strategy of using set_option pp.explicit true to solve this kind of issue.

Maintainability: Will this change streamline code maintenance or simplify its structure?

No, this will only add to the code.
Community Feedback
Ideas should be discussed on the Lean Zulip prior to submitting a proposal. Summarize all prior discussions and link them here.
I linked above one instance of this issue, and in that thread, it was mentioned that an error message that better explains what to do would be helpful. This is not the first time this issue has happened, though, see here, and here, and a similar issue here
Something that's also mentioned in these threads is trying convert so perhaps that could be part of the new error message.
Impact
Add :+1: to issues you consider important. If others benefit from the changes in this proposal being added, please ask them to add :+1: to it.

Closing in favor of #3232


Slack OAuth plugin

Hey, it would be really cool of Slack was added to the list of supported authentication systems. Is that something you've considered, or seen in a fork?

Hey @palewire
Have you seen this? https://github.com/tappleby/slack_auth_proxy

Wow. No. Thanks for passing it along. I'll check it out.

Did you try slack_auth_proxy yet? I wasnt too sure about this since the configuration is via file and not variables that could be parsed into a docker container.


RHAddressBook::people - Duplicate entries for linked contacts

While playing around with RHAddressBook for an app i'm working on, I noticed that the people method in RHAddressBook.m will return duplicates if the contact exists in multiple sources. This seems to be a common issue, related to the underlying API.
Googling around, I came across this stackoverflow thread which seems to offer up a decent algorithm for a solution.
I think this would make a good addition to RHAddressBook. Something like this, but probably with better method names..
-(NSArray*)peopleWithoutLinkedDuplicates; // This would return an array of all users with 'linked duplicates' removed, returning the person from the default source for any duplicates (convenience method for below)

-(NSArray*)peopleWithoutLinkedDuplicatesShowingSource:(RHSource *)source; // This would return an array of all users with 'linked duplicates' removed, returning the person from the supplied source for any duplicates

I'd be happy to look at implementing this and sending a pull request if you'd like. Just let me know how you'd want the methods named, etc.

Got the same issue, any resolution about this?.


fix depth-preserving left subtree insertion

I think there is  a typo, and that tv should be the root of the tree, not v

Any chance to get this merged ?


limit on max number of groups

Currently, it is not possible to part of more than 10 groups without causing trouble for yourself. I for one, am in
users wheel disk audio lp video networkmanager systemd-journal vboxusers nixbld
and that exhausts that limit already. The relevant bit of code is in
https://github.com/NixOS/nix/blob/master/src/libstore/build.cc#L551
As suggested by @the-kenny on IRC it should be upped to 256, and maybe conditionally bumped to NGROUPS_MAX if the previous invocation fails.

Sorry, wrong place for this issue. Closing.


🚀 Release  1.6.0

:robot: I have created a release beep boop
1.6.0 (2022-05-12)
👷 Maintenance

deps-dev: bump eslint from 8.14.0 to 8.15.0 (#163) (9ca9e69)
deps: bump core-js from 3.22.4 to 3.22.5 (#164) (15badcd)

✨ Features

404 layout (#167) (4f11e29)
header and sidebar layouts (#165) (a7e41c8)
maintenance layout (#168) (31fa50d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/carlosdevpereira/mr-components/releases/tag/v1.6.0 :sunflower:


On Documentation

Command maps are of class Dictionary where the key is the command and the value is a class with three members: callback, arguments, description.
This is an upgrade on the idea of using an array as value: [callback, arguments, description]
var coreCommands: Dictionary = {
    "/load/file": [loadCommandFile, "path:s", "Load a file blah blah..."]
    "/set": [setVar, "varName:s value:*", "Set a variable to a value..."]
    ...

Help files on any format can be generated from this.
The _init for CommandDescription would take those three arguments:
func _init(fn: Callable, args: String, desc: String):
    callable = fn
    argsDescription = args
    description = desc

Actually, if we make a class for these Dictionary entries (as I think we should), it would be more like this:
# Core commands:
var coreCommands: Dictionary = {
    "/load/file": CommandDescription.new(loadCommandFile, "path:s", "Load a file blah blah..."),
    "/set": CommandDescription.new(setVar, "varName:s value:*", "Set a variable to a value..."),
    ...
}

I think when a def is defined, it could use the CommandDescription as you describe, except the callable could be a lamda function that calls executeCommands (or whatever, not looking at the code right now) with the array of commands "bound" to it. Sorry, this doesn't directly answer your question about documentation, but I think what you suggest could be made to work for that.

I diverged from here when moving most of the core OSC commands to /defs. Most of the docs now are comments in the .ocl file. I decided unilaterally to use AsciiDoc syntax because it felt right. It might not be the best solution, as we were trying to keep everything in one place (code and documentation).
One advantage I see using AsciiDoc is that it's readable, a lot like Godots own BBCode -- or even better--, and it has it's own compiler to HTML which can be associated to a CSS, which is not necessarily used, because both GH and GitLab can render .adoc files out of the box. What we do need is a parser from .ocl to .adoc or HTML. I added a very small and simple script, but it's *nix shell-only and not finished; needs more work.

The use of AsciiDocs is also justified on most of the documentation being now out of GDScript, in OSC /def files.
I added a doc autogenerator from the (asciidoc) comments in the main OSC file in c00281981c201c8edf38d89e370b51b2c868716b. It can be called with a path to generate a new .adoc file in the ./docs/ directory. In the future it could be used to autogenerate help files from the comments in any file the user might load with /commands/load.


question: anyone was able to run the generate.command?

because cant get my json, so if anyone found a way to make it work, i'll be super happy to follow along.
question number 2: why not share the file ready to be imported?

Same issue

I encountered the same issue on my M3 MBP. I was able to recover my Dracula.json from a previous install on my Intel MBP.
If interested, I have the JSON in a Gist.
Hopefully, this will help someone until the issue is resolved.

@aftaylor2, thank you so much for this, you are awesome. I have encountered this issue with at least three different apps now. I spent several days delving into Ruby trying to learn how it worked, how the Homebrew file system and environment was set up, etc. just to solve the problem... I had no luck and eventually just gave up. @kappesante, I agree with you, I don't understand why we can't have direct access to the file. The developers don't seem to check these issues, and there is no contact info on the website to ask for help or report an issue.


ParseAndMarshalModelTestCase changes

http://lists.jboss.org/pipermail/wildfly-dev/2015-July/004187.html
I'll add the EAP 6.0/6.1/6.3/6.4 tests in a separate commit.

Linux with security manager Build 1792 is now running using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3

Windows Build 1970 is now running using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3

Linux Build 6895 is now running using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3

Linux with security manager Build 1792 outcome was SUCCESS using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3
Summary: Tests passed: 864, ignored: 311 Build time: 0:05:47

Linux Build 6895 outcome was SUCCESS using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3
Summary: Tests passed: 2907, ignored: 372 Build time: 0:47:14

Windows Build 1970 outcome was SUCCESS using a merge of 33ecf4aa2ed447538e813c1dafeccdb23bf928e3
Summary: Tests passed: 2907, ignored: 372 Build time: 0:48:05


chore modify the order of store_row_column and `is_dynamic…

…_schema` to be compatible with branch-1.2-lts
Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

Not merge it in these days because many performance env will re-import data, I will merge it in April.

run buildall

run buildall


Make changes in settings page

Instead of the dropdown edit button should be there.

can I do this @sarthology

Can I take this task


sa argocd-rbac-operator-controller-manager cannot list configmaps at the cluster scope

this MR will allow the operator to get confimaps clusterwide
:bulb: another way to solve the problem is changing the operator so that it works only with cm in the same namespace (personally I would like this option more)
when trying to run the operator I was getting:

W1212 18:17:35.941862       1 reflector.go:547] pkg/mod/k8s.io/client-go@v0.30.1/tools/cache/reflector.go:232: failed to list *v1.ConfigMap: configmaps is forbidden: User "system:serviceaccount:argocd:argocd-rbac-operator-controller-manager" cannot list resource "configmaps" in API group "" at the cluster scope
E1212 18:17:35.941892       1 reflector.go:150] pkg/mod/k8s.io/client-go@v0.30.1/tools/cache/reflector.go:232: Failed to watch *v1.ConfigMap: failed to list *v1.ConfigMap: configmaps is forbidden: User "system:serviceaccount:argocd:argocd-rbac-operator-controller-manager" cannot list resource "configmaps" in API group "" at the cluster scope

:point_right:  I was able to make it run with the proposed tweaks in the RBAC

:information_source: this also solves colliding configmap verbs between leader-election-role and manager-role

https://argo-cd.readthedocs.io/en/stable/getting_started/
since "argocd" is the standard namespace for Argo CD, I don't see the point in changing it to clusterwide. Argo CD also reads the CM from "argocd" to enforce the policies IIRC.
you can change your local manifests for deployment, if you wish to deploy into another Namespace.
We want to offer a way to customize the namespace and CM name in future iterations, but we have to come up with a good solutions first :)


fix: Warning: mkdir(): File exists in FileHelper.php on line 64

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

How to reproduce it?

Delete assets, then press f5 and get this error

OK. The change is not correct but that's good that you've raised the issue. What should be done:

We can't rely on try-catch. The code was assumed to work with Yii 2 where warnings/errors are always converted to exceptions. Instead we need to use

set_error_handler(static function (int $errorNumber, string $errorString, string $errorFile, int $errorLine) {
    throw new ErrorException($errorString, $errorNumber, 0, $errorFile, $errorLine);
});
// something
restore_error_handler();

We should avoid creating directory if it exists but with an extra check at the very beginning of the method.
The idea of original code that you're changing is "create a directory then check if it's there".

Why do is_dir after mkdir? If mkdir doesn't create folder it will return false.

Right. Maybe there's an edge case but out of my head, I don't remember why it was made like that.

Clsoing in favor of https://github.com/yiisoft/files/pull/41


Copy to a variable inside a Structure

What is the issue you have?
Hi,
I think there is an issue/bug with the library when we want to copy the content of a json variable to another that belongs to a Structure variable.

Please describe the steps to reproduce the issue.
Just need to compile the provided code.
Can you provide a small but working code example?
Simple code below:
#include
#include <json.hpp>
using namespace std;
using json = nlohmann::json;
int main(){
struct S
{
	json j;
};

S s;
json value = "test";
s.j = value;

return 0;

}
What is the expected behavior?
Compile without errors.
And what is the actual behavior instead?
The compiler gives me a bunch of errors, as can be seen in the attachment.
Which compiler and operating system are you using?
I have many other programs on the same machine, using this library (without using structures).

Compiler: GCC 7.1
Operating system: Raspberry OS

Which version of the library did you use?

[x ] latest release version 3.9.1
[ ] other release - please state the version: ___
[ ] the develop branch

If you experience a compilation error: can you compile and run the unit tests?

[x] yes
[ ] no - please copy/paste the error message below

Are you sure you are compiling with C++11? There is no standard given in the command line arguments of your GCC call.

And what is the error? I can only see warnings in the screenshot.

This seems to be a duplicate of #2850.


Have voltage on OSD but no RSSI, LQ or cronometer

Hello good people
I've just installed on DJI V1 goggles, WTF, the OSD and power packages. From the items I chose on OSD, only disarmed and voltage are displayed, no LQ, RSSI dbm, warnings or cronometer. I'm using ELRS Happymodel EP1 and sensors are working. Any ideas? Thank you.

@Helix-in-Wonderland Did you do the msp and serial port setup in betaflight?

Yes I did. And voltage is showing. Thanks

Well, can't get no one to answer here or any other place. I'll go back to the craft name hack. No wtf for me.

It'd be useful to get your BF configuration @Helix-in-Wonderland . Maybe show us what diff all shows?
It might be that you have feature -TELEMETRY or some of the telemetry_disabled stuff.

Thank you. I'll try to do that as soon as possible as I'm not with access to the quad at this moment. Cheers


Fixed issue with incorrect dns record set ttl parameter name

Hi @MikhailTryakhov, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@amarzavery : Node build in travis is failing. Any idea why? It is not obvious from the log. Does not seem related to this PR.


Use pnpm instead of npm

Unfortunately a Storybook issue currently prevents this from going through 😔 See https://github.com/storybookjs/builder-vite/issues/141 - upgrading to Storybook v7 should solve this, but first we're going to investigate moving to Histoire in #39, so let's wait on a conclusion on that before moving forward with this. Closing this PR for now.


add classes as dataclasses

I am changing this and will use ABS/abstractmethodclasses instead to create a superclass for Position, then TD_Position and Test_Position as subclasses


fix and rename real_len

Fixes #606 -- see added test cases. This logic matches what happens draw_target.rs as well. It'll now also correctly handle lines with only ANSI codes.
https://github.com/console-rs/indicatif/blob/81aa4c65d7202128d471b3a452a0236e03f3bc43/src/draw_target.rs#L505-L518

Can we share the code with draw_target somehow, so that it is more obviously using the same algorithm?
If we're going to move this I'd like to see two separate commits, one that moves/renames the code and one that changes the algorithm. Ideally tests that work with the old algorithm are introduced before this, and then the fixing commit adds tests that didn't work previously -- this will make it easy for me to review. (Please squash linting changes etc.)

Can we share the code with draw_target somehow, so that it is more obviously using the same algorithm?

Doesn't seem like a worthwhile seeing as that is counting lines while drawing, while this does it ahead of time.

If we're going to move this I'd like to see two separate commits, one that moves/renames the code and one that changes the algorithm. Ideally tests that work with the old algorithm are introduced before this, and then the fixing commit adds tests that didn't work previously -- this will make it easy for me to review. (Please squash linting changes etc.)

Please see new commits -- first commit moves + add working tests, second commit adds the failing tests, third commit fixes the failing tests.

CI seems like a flake, can't repro at all locally.


Beepbox extension support

I know this'll exist one day. This issue is a reminder that whenever out-of-program extensions are created for Beepbox or any other mod of it, to make them compatible with Midbox if possible.

Another "if" for this would be if I even need to make them compatible. From the few I've seen I think they are compatible with most Beepmods including Midbox already.


ROX-13001, ROX-12813: (Almost) Exclusively use unstructured objects in the updater

Description
The updater must not depend on the version of the K8s client libraries, as otherwise the K8s client library version becomes a minimum supported version.
Concretely, the issue we saw after upgrading to the 0.24.x libraries was that the NetworkPolicy kind in networking.k8s.io/v1 now has a status field (ROX-13001). Because the Go type defines this type as a value instead of a pointer, it always gets serialized in JSON, even when empty (as status: {}). This makes the schema validation fail, as the OpenAPI schema on older K8s versions does not contain a status field. Disabling schema validation is risky, too, because fields that we require to be set might end up not being set, and therefore was not considered as an acceptable option.
Instead, this PR changes the upgrader to use unstructured objects throughout (except for in the case of creation of some special resources like secrets and configmaps, which can be considered extremely stable). This ensures that the typed objects from the Go Kubernetes libraries do not affect the upgrader logic at all. Particularly, the use of a scheme is avoided, as a scheme is a client-side concept only, that defines the mapping between Go types and unstructured JSON.
While this does make the code more complex in some regards (resource preservation), it also simplifies it in many other ways, due to the omission of schemes and codecs. And while a simpler quickfix would have been possible (just delete the status field for network policies), this PR is doing the right thing as the upgrader should more or less just be dumb and pass through objects from YAML files in the bundle to the API server, without making modifications to them.
This also fixes the test flake ROX-12813 through the use of a higher timeout for re-activating the metrics server (this was not sent as an independent PR, because upgrade tests are currently deterministically broken, and thus it is not possible to de-flake them in that state, while at the same time the flake makes the fix by this PR harder to demonstrate).
Checklist

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

If any of these don't apply, please comment below.
Testing Performed

CI

/test gke-upgrade-tests

/test gke-upgrade-tests

/test gke-upgrade-tests

/test gke-upgrade-tests

/test gke-upgrade-tests


[FEATURE] Exoplayer is now deprecated

Need to move androidx.media3
https://github.com/androidx/media/releases/tag/1.1.0
Migration Guide
https://developer.android.com/guide/topics/media/media3/getting-started/migration-guide

If it can help i can share media3 migration code
I write for own project
but not sure about 100% works

Leave


Added Improved Support for Drake-ify + Added a Few More Features for The First "Real" Scene in Motion Planning

Codecov Report
Attention: Patch coverage is 88.01843% with 26 lines in your changes missing coverage. Please review.

Project coverage is 90.43%. Comparing base (28c7bbe) to head (f879833).

Files with missing lines
Patch %
Lines

src/brom_drake/utils/search.py
57.89%
8 Missing :warning:

src/brom_drake/scenes/roles/role.py
76.66%
7 Missing :warning:

src/brom_drake/urdf/mesh_file_converter.py
95.45%
4 Missing :warning:

...brom_drake/scenes/motion_planning/offline/shelf.py
91.42%
3 Missing :warning:

src/brom_drake/scenes/types/base.py
57.14%
3 Missing :warning:

...brom_drake/scenes/types/motion_planning/offline.py
88.88%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #12       +/-   ##
===========================================
+ Coverage   79.15%   90.43%   +11.27%
===========================================
  Files          41       51       +10
  Lines        1166     1307      +141
===========================================
+ Hits          923     1182      +259
+ Misses        243      125      -118

Flag
Coverage Δ

90.43% <88.01%> (+11.27%)
: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.


Interface MTU has to be equal or less then 32767 (2^15-1), but it should be 2^16-1

Environment

Python version:  3.5.2
NetBox version: 2.3.8-dev

Steps to Reproduce
Edit an interface of a device, and set an MTU of i.e. 65536.
Expected Behavior
Accept an 16-bit value, or at least 16-bit minus 1.
Observed Behavior
Netbox threw an error: "Ensure this value is less than or equal to 32767."
Additional notes
I'm somehow confused that Linux has an MTU for the loopback interface lo a value of 65536, seen by
# cat /sys/class/net/lo/mtu
65536
# uname -r
4.17.0-1-amd64

but regarding to https://tools.ietf.org/html/rfc791 the maximum value of MTU is just 65535 (2^16-1).
The value of 2^16 comes directly from https://github.com/torvalds/linux/blob/master/drivers/net/loopback.c#L176
	dev->mtu	= 64 * 1024;

So one thing is that the max. MTU in netbox is not as big as the RFC says it can be; and why David and Eric had chosen 2^16 instead of 2^16-1 is another :)

This is because we use a PositiveSmallIntegerField to store the MTU, which uses a signed 16-bit small integer for the database column but only allows positive values.
To accommodate values larger than 32767 we would need to migrate this to a PositiveIntegerField and add a validator to limit the maximum to 65535 (or 65536?).


[iOS] Hide TBW target audiences

Issue

#814

Overview (Required)

The title says it all (What does "TBW" mean?)

Links

Screenshot

Before
After

@woxtu
I heard TBW is an abbreviation for "To Be Written" 👍🏼

@ry-itto That makes sense, thanks!


Rejected_Bad_Status for working online

i am trying to crawl over this site http://kycon.com/gz/website/Index.html, but every time the collector returns this status - Rejected_Bad_Status - for some online links, i took a sample of rejected online and found it working, ex.
Need to know why, what is this status mean and what is the best solution for it ?
May also you suggest me the best configuration for this site, to learn from it.
Many thanks for your time.

Chances are the returned HTTP response code is not valid (e.g. other than 200).  Do you have a few URLs examples that have a bad status?
If you want to find out more about rejections, you can change the log level to DEBUG in the log4j.properties file found in the installation directory.
For instance, you can change this line:
log4j.logger.CrawlerEvent.REJECTED_BAD_STATUS=DEBUG

Done, here is what i get in log file:
Link Locator Crawler: 2016-10-07 22:24:09 INFO -       REJECTED_BAD_STATUS: http://anacapa.kycon.com/WebScript.nsf/partSearchDXLxsltNF?OpenAgent&xslF=WebPartSrchCerts3d1.xsl&Start=1&Count=10&Keyfield=part_id&Query=JSX-1035 (Subject: HttpFetchResponse [crawlState=BAD_STATUS, statusCode=400, reasonPhrase=Bad Request])
Why statusCode=400 although when open on browser no problem?
Thank you for time.

This is a bug within the norconex-commons-lang library.  URLs with query string having parameters with no equal sign saw those parameters removed ("OpenAgent" in your case).  That library has been fixed and you can download the latest version here.  Locate the norconex-commons-lang-1.12.2.jar in the lib folder in the zip and replace the older version found under the lib folder of your HTTP collector install with this new one.
Alternatively, a new snapshot release of HTTP Collector was just made, which includes that updated library dependency.
Please confirm.

Really fixed no rejected status found,
Many thanks for your time and for your great support.

You are welcome.


Tracking issue for replace github.com/pingcap/check with github.com/stretchr/testify

Background
as described in the thread https://internals.tidb.io/t/topic/141/6
I did ever use testify in pingcap/errors [[1]](https://github.com/pingcap/errors/pull/36) and it works well with richer matchers and default parallelism support.

I think the modification from gocheck to pingcap/check mainly focuses on parallelism. It is possibly we switch to testify to live with a stronger community instead of a lack of maintenance version of a smaller community.

* [testify ](https://github.com/stretchr/testify) has 188 contributors and latest updated at Apr. 27
* [go-check](https://github.com/go-check/check) has 13 contributors and latest updated at last year.

If you think this direction is good to go, maybe we can draft a pre-RFC to collect wider input.

This is somewhat a subtask of https://github.com/pingcap/tidb/issues/26022 that we make a decision that deprecating pingcap/check and adopting testify. Take a look at the tidb issue for more information.
After an offline discussion with @overvenus we think it is fine we make the same change on this repo and here is the tracking issue for the effort.
If you have any further concern, please comment below and reply will be in days.
We first write down a task list and since I have no time to continue the actual work, let's kick off when there is a contributor ask for starting the subtasks.
Join Forces
If you want to participant the movement, you can comment under this issue to acquire pkg or files to migrate. For example,
Hi, I'd like to migrate tests under `errno` pkg. Please create a subtask and assign to me.

React will be in time and do the favor for you.
Also you can directly comment under any open issues below without assignee to require assignment.
You can take a look at https://github.com/pingcap/tidb/pull/26375 as an example to do the migration.
Note

new unit-test

New uts should use testify and goleak directly. Ref to the Pr 2833
Pkg github.com/pingcap/tidb/util/testkit should be replaced by github.com/pingcap/tidb/testkit

parallel

Keep test cases as parallel as possible.
Don't use t.Parallel() if pkg has failpoint control. Need review Carefully.
When some test cases should be run in serial, take use of go testing subtests (t.Run(name string, f func(t *T))) and parallel the parent test only. In this way, test cases in the same subtests set run in serial.
Avoid using suite of testify as discussed errno: migrate test-infra to testify #26082 (comment)

migrate kits

Migrate Script or Migrate Tool  can do some replace work quickly, try and make pr.

Progress
Considering test-infra migration is a long-term process，we need to divide the whole process into several phases. After that,
each pkgs will be classified into specific phase.

Phase0: small pkgs, no kits dependent,  migrate script/migrate tool can do most work.
Phase1: small pkgs dependent on kits, need to deal after we migrate kits
Phase2: medium pkgs,  need to modify suite logic manually
Phase3: large pkgs,  need to modify suite logic manually

Subtasks

Phase0:

[x] #2919
[x] #2890
[x] #2928
[x] #2927
[x] #2925
[x] #2923
[x] #2921
[x] #2859
[x] #2869
[x] #2870
[x] #2871
[x] #2936
[x] #2935
[x] #2934
[x] #2933
[x] #2930
[x] #2825
[x] #2954

Kits

[x] #2874
[x] #2879

Phase1:

[x] #2932
[x] #2878
[x] #4200
[x] #4878

Phase2:

[x] #2912
[x] #4879
[x] #2826
[x] #2902
[x] #2903

Phase3:

[x] #2899
[x] #2900
[x] #2901
[x] #2904
[x] #2905
[x] #2906
[x] #2907
[x] #2908
[x] #2909
[x] #2910
[x] #2911

I have a question:
Why we need a test framework? Can we just use the golang's built-in test framework?

I have a question:
Why we need a test framework? Can we just use the golang's built-in test framework?

more functionality support i guess, ref https://github.com/stretchr/testify, also could discuss the option in https://internals.tidb.io/t/topic/141/6 with community, i think

Off-line discussion:
I support the use of testify, but the migration costs may be higher. Because there are a lot of tests.
From @overvenus , we can use testify, because pingcap/check requires writing redundant templates. But we need to make sure that leak test is supported.

But we need to make sure that leak test is supported.

Unnecessary. As done in pingcap/tidb and tikv/client-go, we can migrate testleak to goleak also, which requires less setups and is able to control (ignore) options per test (package). testleak is a toolkit build on pingcap/check and we don't have to maintain it if find a better replacement.

Also I don't think this is a feature request.

@ben1009 the next time you can comment this effort on the post or the issue of tidb so that we're aware of its happening.

I support the use of testify, but the migration costs may be higher. Because there are a lot of tests.

Agree. I think there are several potential contributors to work on this topic. Let's postpone this issue util there is actually someone want to work on it. We should avoid throwing ourselves into a situation that maintain two kinds of test cases for a long time.

Looks like I can't convert task into issue. I assume contributor can create task and reply to this issue to create link, am I right?

Looks like I can't convert task into issue. I assume contributor can create task and reply to this issue to create link, am I right?

Yes. See also https://github.com/pingcap/tidb/issues/27618#issuecomment-907790848 .

@tisonkun hello, tisonkun,  i am new to the ticdc group. We made offline discussion recently , and wanted to make the migration to testify and goleak step by step.  What should we pay attention to the migration?  Did you met something bad in the progress?

@maxshuang I don't remember with an offline discussion though. What I can say it just go ahead to do it. Maybe start from some utility package like pkg/version.
If you'd like to do it, I'll create a subtask and you can assign yourself by /assign.

@maxshuang you can take a look at https://github.com/pingcap/tidb/pull/26082 for best practices collected and previous discussions.

Thank you all for your excellent work!


Clean up some wording in README

Closing this PR, as it has gone unmerged for over three years now.


Browserstack iOS testing - Error converting element ID for using in WD atoms

Hello,
Sorry if this is the wrong place to post this, but it seems wdi5-related. If it is not, I would appreciate it if you could tell me where to ask instead.
I have been trying to run some wdi5 tests on Browserstack, and they work just fine across different browsers and operating systems, but I keep running into issues with iOS specifically. When I try to run wdi5 tests in iOS, I get the following error:
Error converting element ID for using in WD atoms: ':wdc:1678092332762'
It doesn't occur immediately (other controls are used before the error without any issues), so maybe there's some other underlying issue with the specific test it crashes on. It just doesn't make sense to me that it would work just fine in other browsers and operating systems if that were the case.
Has anyone else had this issue, or something similar? Any help is appreciated. Also, let me know if you need any additional information.
Thank you for reading.

Hi 🙋♂ first of all, I think it‘s great that wdi5 is working well for you cross-browser and -OS!
if you‘d share more details on the test(s) that cause above output, then we‘d be more likely to get a grasp on things ⚙

Hello again,
Thank you for the reply. I have written the tests using the mocha framework and here's the test where the error occurs:
    it("should open the settings popover when the settings button is clicked", async () => {
        const settingsBtn = await Dashboard.getSettingsButton();

        await settingsBtn.press();

        const selectedLangBtn = await Dashboard.getSelectedLanguageButton();
        expect(await selectedLangBtn.getVisible()).toBeTruthy();
    })

The methods used on the Dashboard page object are using the browser object and the asControl method with selectors to retrieve the controls. The mentioned error seems to occur specifically when the getSelectedLanguageButton() method is called. Here's a screenshot of the error from Browserstack:

Just now as I'm writing this I noticed that there's a difference between the id of the control which causes the error and the others:

This is from a different control, and here you can see 'xml-view' is part of the id (presumably because the view name is specified in the selector for that control). I have not specified it in the selector for the other control since it's in a fragment and I didn't seem to need it while testing locally, but I will try to include it and report back.

Hello,
I have now tried some different things out and I still get the same error when running the tests on iOS. In the tests I have been trying to run to fix the issue, the error first occurs when trying to use one of the getter methods on a button control (getVisible).
The tests continue to run (and work correctly, despite the errors in the terminal) after this, though, but eventually fail completely when trying to use the getContent() method on a dialog control. I'm not sure where to go from here, so if anyone has any suggestions, please let me know. Also, if

Tried now on some other testing platforms (Sauce Labs & LambdaTest), and the issue doesn't seem to occur there.


CompileOption render error with typescript

The following does not work.
import Twig from 'twig';
Twig.renderFile('./path/to/someFile.twig', {foo:'bar'}, (err, html) => {
  html; // compiled string
});

With the error:
Argument of type "foo": String is not assignable to parameter of type CompileOptions

I'm guessing this is due to a problem with the typings wherever you have fetched them from.
We don't directly support typescript, so you'll need to figure out where the typings are and submit a PR there to fix them.
I'm guessing https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/twig

Thanks. I'll look into that.


cc 1.0.42 with "parallel" feature hangs on Windows

To reproduce

cargo new --bin сс_jobserver
cd cc_jobserver
In Cargo.toml, put the following:[dependencies]
rusqlite = { version = "*", features = ["bundled"] }
zstd = { version = "*", default-features = false }

(my Cargo.lock if you want to reproduce perfectly)
cargo build

Expected result
Build completes in finite time.
Actual result
The build hangs in the following state:
> cargo build
    Updating crates.io index
   Compiling log v0.4.8
   Compiling getrandom v0.1.12
   Compiling jobserver v0.1.17
   Compiling cc v1.0.42
   Compiling zstd-sys v1.4.13+zstd.1.4.3
   Compiling libsqlite3-sys v0.16.0
    Building [============================================>            ] 32/40: zstd-sys(build), libsqlite3-sys(bu...

The process tree looks like this:

These two build-script-build.exe don't use CPU and don't make any progress.

I think this is related to cc 1.0.42 because first, if I pin cc to 1.0.41, it builds successfully, and second, because there was this change: https://github.com/alexcrichton/cc-rs/commit/4fda8db89e666f402392366501505a4fbd14b23f .

Version info:
> cargo --version --verbose
cargo 1.39.0-nightly (3f700ec43 2019-08-19)
release: 1.39.0
commit-hash: 3f700ec43ce72305eb5315cfc710681f3469d4b4
commit-date: 2019-08-19

rustc --version --verbose
rustc 1.39.0-nightly (521d78407 2019-08-25)
binary: rustc
commit-hash: 521d78407471cb78e9bbf47160f6aa23047ac499
commit-date: 2019-08-25
host: x86_64-pc-windows-msvc
release: 1.39.0-nightly
LLVM version: 9.0

I observe this problem on a machine with 2 cores.

Oops sorry about that! Should be fixed in https://github.com/alexcrichton/cc-rs/commit/238ee4f093c37084314711f05154c14eb6362711

Thank you. I confirm it appears to work fine with cc 1.0.45.


Pilot PopTo Doesn't Work with Parameters

I have a project that utilizes parameters within its routing. While using pilot.push(.Locations(status: Status.none)) works, doing pilot.pop(.Locations(status: Status.none)) doesn't work. When I used the debugger, I noticed that the internal popTo function navigated to the the log statement UIPilot - Route not found.
If I had to take a guess on what is wrong, I think exact matching might with Equatable might not be right. This is because when doing .Locations(status: Status.none), it's creating a new unique identifier for that route? Maybe if it was checking for a route name instead, it might work?
iOS Target: 14.1
Swift: 5.7
UIPilot: 1.3.1

Hi @Eli017 ,  UIPilot uses Equatable to see if any matching route is in the stack, so maybe you should check if you have implemented Equatable correctly on your routes enum.

facing same issue @jimmy0251 can you add one example of complex navigation in repo which uses popTo()  with parameters


Time for next generation Eureka?

This is an open discuss. I am curious whether there are plans for the next generation of eureka recently. Or we can contribute to the next generation of eureka with the help of the community and during the process of contribution, we can get some help from Netflix experts.
thanks for any reply : )

We can have a conversation in #1272.


Rework docker-compose to use segregated networks for security, other fixes

Put front and backend apps into their respective network groups
Add frontend service port mappings to UFW
Rework .env.example to contain latest from api and frontend repos, with API BASEURL preconfigured

@pitastrudl Could you take another look and explain what the expected port mappings in each env are?


we should separate CFLAGS, HOSTCFLAGS and TARGETCFLAGS

While building, if we provide a custom CFLAGS such as -march=i586 it will be used for both cross compiling and compilation of kbuild, that will make configure targets to fail.
Of course the same applies to LDFLAGS.

I checked that case and I understand this case doesn't apply. The kconfig uses the HOST_CFLAGS and HOSTEXTRA_CFLAGS var and the CFLAGS value is never appended to it, do you have an easy way to reproduce what you've noticed?

confirmed. It was a mess up in my setup.


Freeform vector tool

related: https://github.com/excalidraw/excalidraw/issues/24
I would like to suggest a vector "drawing" tool where you can create an arbitrary vector shape by connecting points similar to vector tools in apps such as Sketch. I believe that it would be valuable even without supporting curved lines. I really missed a vector drawing like tool when I wanted to create an app mockup with a notch on top.

We have curved lines so this I would say is done. Let's close it :)


fix(oas): xml model serialization

closes #119

@pmstss do not rejoice before time, it is still very draft)


1.0.0-preview3-final release notes?

1.0.0-preview3-final of Microsoft.AspNetCore.Server.IISIntegration.Tools was pushed to NuGet earlier this week. I am unable to find release notes on this.
Should we be updating our tooling?

This was part of the 1.1.0-preview1 release: https://github.com/aspnet/IISIntegration/releases/tag/rel/1.1.0-preview1
You only need to update your tools if you are updating your runtime to 1.1.0-preview1.
https://blogs.msdn.microsoft.com/webdev/2016/10/25/announcing-asp-net-core-1-1-preview-1/

Here are the release notes: https://github.com/aspnet/IISIntegration/releases/tag/rel/1.1.0-preview1. If your target 1.1.0-* I would recommend using preview3 version. Technically, even if your application targets 1.1.0 runtime and you have 1.0.0 runtime installed preview2 version will continue to work using 1.0.0 runtime (while the application will use 1.1.0 runtime) but I think it's best to keep both in sync.

@moozzyk @Rick-Anderson Currently, we have ...
"tools": {
  "Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-preview2-final"
},

... in the Publish to IIS doc.
Update it there (aspnet/Docs) to 1.0.0-preview3-final?
Is there any point to updating Microsoft/Docs-1, since that section will either change dramatically or be removed completely for the 1.1 release?

Is there a branch of the docs for 1.1-pre?

@GuardRex - I guess it would be nice to be more general and say that it is good practice to make sure that tools target the same runtime as the application. In case of 1.1 runtime preview3 is the version.

@moozzyk ok ... gotcha.
Do the old docs (aspnet/docs) need the update tho?
My instruction from @Rick-Anderson was to only submit PR's to Microsoft/Docs-1. Since that might not land for customers until 1.1 and that whole section will possibly get major changes later, I'm wondering if this should be held off for now?

Please wait. We don't want to confuse customers with preview bits in the 1.0 docs.

I just asked b/c we already have ...
"tools": {
  "Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-preview2-final"
},

... in the current doc. If that's ok, we're all good.

That's correct for 1.0.0 runtime packages.

@GuardRex  I hope we have this repo converted to .MD in the next couple of days.


ServiceBus Topics Api

The specs in this repo don't match the documentation: https://msdn.microsoft.com/en-us/library/azure/mt639397.aspx
Unfortunately that documentation also seems to be wrong (GetTopic returns a different schema back and CreateTopic doesn't accept the listed schema).

@sazeesha - Can you take a look at this?

@andyroyle: When you create a topic you are setting only the properties that can be set, and GET returns all the TopicProperties. You can consider swagger to be the source of truth. Can you point me to what is inconsistent.

The simplest case of CreateTopic; as per this spec: https://msdn.microsoft.com/en-us/library/azure/mt639409.aspx
The only required field should be location, but if you send such a request then you get a 400 (Bad Request) response.

@andyroyle - Are you still experiencing issues with the CreateTopic call?

Closing this issue as CreateTopic is succeeding with just the location parameter.

Hey, sorry; I stepped away from this for a while.
I just tried it again and it seems that it's now passing.
Not sure what changed!


Not used MRB_FLAG_IS_INHERITED

MRB_FLAG_IS_INHERITED is 22th bit from LSB, but flags field is allocated 21 bits in MRB_OBJECT_HEADER.

from include/mruby/object.h:10
#define MRB_OBJECT_HEADER \
  enum mrb_vtype tt:8;\
  uint32_t color:3;\
  uint32_t flags:21;\
  struct RClass *c;\
  struct RBasic *gcnext

from include/mruby/class.h:68
#define MRB_FLAG_IS_INHERITED (1 << 21)

Therefore MRB_FLAG_IS_INHERITED is ignored always (same as constant zero).

test code (test_inherited_bit.c)
#include <mruby.h>
#include <mruby/class.h>

int test(void)
{
  struct RClass c = { 0 };
  c.flags |= MRB_FLAG_IS_INHERITED;
  return c.flags;
}

assembled result by gcc7 -I include -S -O -o - test_inherited_bit.c
	.file	"test_inherited_bit.c"
	.text
	.globl	test
	.type	test, @function
test:
.LFB16:
	.cfi_startproc
	movl	$0, %eax
	ret
	.cfi_endproc
.LFE16:
	.size	test, .-test
	.ident	"GCC: (FreeBSD Ports Collection) 7.3.0"
	.section	.note.GNU-stack,"",@progbits

Sorry to report only.

Seems to be introduced in: https://github.com/mruby/mruby/commit/4c9a60487799b923ff905def46cb2497bcc1e4dd
It seems like this flag is used to clear method cache when inherited.
And out of range flag may cause method cache bug...
Bit 18 is used by frozen: https://github.com/mruby/mruby/blob/c3c882f681451c76ad6ffde984ec33432c5b579a/include/mruby/object.h#L26
so moving it with bit 17 should solve this.
Though this problem only occur on MRB_METHOD_CACHE enabled mruby.

I wish this was a compiler warning...
Anyway valuable report!


Azure File Sync over ExpressRoute

How is Azure File Sync configured to traverse / tunnel over an ExpressRoute connection. Is there some further detailed advice on how to do this?

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

ID: 02c9ebf2-ed06-9058-2efa-a6d87e775913
Version Independent ID: 5a868afb-d5ca-d478-24f5-88513c755429
Content: Azure Files networking considerations
Content Source: articles/storage/files/storage-files-networking-overview.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@challengelogic Thanks for your question. We are checking on this and will respond to you soon.

@The Azure File Sync agent has no requirements regarding special channels like ExpressRoute, etc. to Azure

@challengelogic I'd recommend posting your question in https://social.msdn.microsoft.com/forums/azure/en-US/home?forum=windowsazuredata since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


How to skip ssl when establish connection

Hi,
I had a database server with mysql5.6. When I try to connect to DB I have this issue.
ERROR 2026 (HY000): SSL connection error: error:00000001:lib(0):func(0):reason(1)
How can I skip ssl connection or is there a way to add ssl certification?
Thanks!

Fluent 3 now longer uses the libmysql library. SSL will be implemented soon and it will support using a plaintext connection 👍