Unexpected behaviour of methods buckets.pullPath and buckets.pullIpfsPath.

Hi, I have been working on a react-native project in which I make use of textile buckets to push files to IPFS and retrieve it using the buckets.pulIIpfsPath but the file content seems to be corrupted. I have also tried the buckets.pullPath but it doesn't seem to work either.
I make use of react-native-fs to read and write binary files
pushPath method:
const content = await RNFS.readFile(uri, 'base64');
const _file = {
	path: '/' + file.name,
	content: Buffer.from(content),
};

const pushResult = await this.state.buckets.pushPath(this.state.bucketKey, file.name, _file);

the file uri is of format: content://path/to/file
Then I do this which is similar to the example provided here in bucket-photo-gallery line 182 and line 214:
const ipfsFilePath = pushResult.root;
const pullResult = await this.state.buckets.pullIpfsPath(ipfsFilePath);
const { value } = await pullResult.next();
let buffer = '';
for (var i = 0; i < value.length; i++) {
	buffer += String.fromCharCode(value[i]);
}

Is this the correct way of getting the value back? Should I not iterate over the pullResult as it is an AsyncIterableIterator<Uint8Array>? If so, how do I do that?
The buffer here seems to be corrupted. I don't know what I'm doing wrong. Any help would be greatly appreciated.
Thanks.
Update: also note that I am not able to iterate over the pullResult like so:
for await (const chunk of pullResult) {
	console.log('chunk: ', chunk);
}

it throws this error: TypeError: Invalid attempt to iterate non-iterable instance. In order to be iterable, non-array objects must have a [Symbol.iterator]() method.

Hi @theabdulmateen , you update is the correct way to consume the AsyncIterableIterator that get's returned from buckets.pullIIpfsPath... but you don't need to await it. I think your error above stems from only having part of the expected buffer, you'll want the whole thing.
const ipfsFilePath = pushResult.root;
const pullResult = this.state.buckets.pullIpfsPath(ipfsFilePath);
const chunks = []
for await (const chunk of pullResult) {
	console.log('chunk: ', chunk);
        chunks.push(chunk)
}

If you want to combine the whole thing into a single Uint8Array, something like this would work:
const  combined = Int8Array.from(Array.prototype.concat(...chunks.map(a => Array.from(a))));

Now you can do things like Buffer.from(), or work with your file bytes.

Hi @carsonfarmer thank you for your reply.
So I tried to consume the pullResult like you suggested:
const pullResult = this.state.buckets.pullIpfsPath(ipfsFilePath);
console.log('pullResult: ', pullResult);
let chunks = [];
for await (const chunk of pullResult) {
	console.log('chunk: ', chunk);
	chunk.push(chunk);
}
console.log('chunks: ', chunks);

but unfortunately, I get the same error:
ExceptionsManager.js:179 TypeError: Invalid attempt to iterate non-iterable instance.
 It looks like your build env doesn't support async iterators? Are you using Metro for this app?
Here's one possible work around:
const pullResult = this.state.buckets.pullIpfsPath(ipfsFilePath);
console.log('pullResult: ', pullResult);
const chunks = [];
const iterator = pullResult[Symbol.asyncIterator]();
while (chunks.length < Infinity) {
    const { value, done } = await iterator.next();
    if (done) break;
    chunks.push(value);
    console.log(value);
}

If that doesn't work, the returned pullResult itself is an async iterator, so you can probably just call next on it directly:
const pullResult = this.state.buckets.pullIpfsPath(ipfsFilePath);
console.log('pullResult: ', pullResult);
const chunks = [];
while (chunks.length < Infinity) {
    const { value, done } = await pullResult.next();
    if (done) break;
    chunks.push(value);
    console.log(value);
}

Thank you for your reply, yes I am using metro bundler,
So I tried both methods of retrieving the complete value but it seems that the loop gets broken after getting value once and it looks like the final value chunks gives the same buffer string as what was retrieved from my attempt in the initial post

Ok, so if you aren't getting any errors from the loop, then it seems your buffer is correct? Are you using an encrypted bucket by chance? What is the content of the buffer value, and what was originally pushed? Are you able to create a small reproducible example we can test?

Looks like a base64 encoded string? So you have a Uint8Array of base64 encoded bytes. Perhaps this might be useful to decode it from UInt8Array? https://gist.github.com/enepomnyaschih/72c423f727d395eeaa09697058238727

You could also use Buffer.from on the Uint8Array and then use the Buffer to encode/decode the content.

Ok so yes, the bucket was encrypted and this is definitely my fault and to give you an idea of what I am trying to achieve is:

user opens a bucket and stores a file on the IPFS using the buckets and the bucketKey generated.
share the pushResult.root to another user who then creates an instance of his own buckets and bucketKey and I was naive to think that the IPFS path itself would be enough to retrieve the file.

since the buckets were encrypted the receiver creates a new bucket altogether and that is probably where the problem stems.
so to solve this I assume I would need access to the threadID of the bucket that contains the files. This isn't the same as the bucketKey is it? If it isn't how do I retrieve it. Do I need to access it from the links generated from buckets.links() or is there a better way?
I apologize for the confusion.

Also to answer your questions, I read a .jpg file using RNFS with base64 encoding and passed it to pushPath with content: Buffer.from(content) as mentioned in my initial post.
After retrieving the value from pullIpfsPath the file that was written back was only of around 0B - 788B which could be the .textileseed stored on the new bucket the receiver generated.


A Rewrite

Hello author here!
I wrote an article about the state of tilemaps in bevy. Mostly around my experience with creating bevy_ecs_tilemap and its pitfalls. You can find it here:
https://startoaster.netlify.app/blog/tilemaps/
My main takeaway is that bevy_ecs_tilemap needs a re-write in order to facilitate a better API design. Feel free to leave any questions you have here!
Be on the lookout for a branch called rewrite. I'll comment here as well when I push it up. This branch will track the changes I make and will be a from the ground up re-write of the entire library. Feature wise I'll be focusing on the basics and moving on to add more features as I go.
What does this mean for the code base currently? I will no longer be accepting any PR's that add new features. This is to reduce the amount of work I'll end up needing to do during this re-write period. I will still accept bug fixes.
If you would like to see any new features, or suggest changes to the new API design, feel free to leave them here!
Thanks!

Are you planning on updating the plugin in its current state for bevy 0.7?

I'm not sure when bevy 0.7 will come out. Regardless I think an update will happen.

We are being told perhaps tomorrow.

This sounds great, thanks for all the hard work you're putting into this. The most important feature to me is performant large multi-layered isometric tile maps. The only thing that comes to mind from an API perspective is keeping anything chunk related out of the public API as much as possible, it feels like an implementation leak, but I think you;re already trying to do this.

The most important feature to me is performant large multi-layered isometric tile maps.

I have a newer isometric tile map renderer I'm using for a game I'm making. It can handle upwards of 1m tiles at 30fps. There are some downsides I've discovered:

As bevy entity counts increase rendering performance decreases. At 40m tiles for example I was rendering at 5fps most of which was being consumed on the CPU side by bevy's internal logic. I filed a bug report here:
https://github.com/bevyengine/bevy/issues/3953 I think there's been a little bit of progress on it.
Mapping screen coordinates to 3D tile coordinates is a nightmare. I have a somewhat working solution.
Biggest issue is that bevy's sprite renderer does not have a depth buffer. Which means that there is no way to quickly sort tiles. Because of this you have to either create your own sprite renderer(not very user friendly), or slow down the tile renderer by splitting each row of tiles into its own mesh/chunks so that they can have the correct sorting applied. Sorting is typically the Y value of the tile after projection if I remember right.

Yep, generally most AAA games shy away from these days. A few other reasons not to use them:

No WebGPU/WebGL support.
Geometry shaders are being replaced by mesh shaders (no wide support for them either yet really though).

I haven't had to do any low level graphics work in a long long time, but if there is anything more general that would be good for someone else to put PRs in for, I'd be willing to try and help out.

Right now I need some ideas on how I can make tile removal more pleasant. I had planned on doing it when you despawn an entity, however I can't really get any data from a despawn event which means we have a few different API choices we could make here:

Add a RemoveTile component that allows me to know when a Tile is being removed and to remove it from the renderer and later from the world.
Keep a cache of data for each tile that I can use when a tile is despawned. More memory would be used but users could just despawn tile entities like normal.
Some other method I haven't thought of yet?

@StarArawn I'd like to give a warning about rewrites: they can easily turn into a daunting and demoralizing task and cause burn-out. They can literally kill your project.
You have already spend many months developing the current version of the crate. Are you really convinced that you are going to have sustained long term mental energy (and time and dedication) to drive a rewrite, even to feature parity with the current version? It's probably going to be a lot of work. If you really are ripping everything out and starting from scratch, because you want to change the project architecture that much, it's not likely much of the existing code would be reusable. Recreating something big (that is already quite impressive), that you have already done before, can feel pretty tedious and even demoralizing, as there is little novelty (you gotta reimplement all the stuff that was already working, before you can get to making cool new stuff).
I am talking from experience. Both from my own projects and from observing others. Complete rewrites rarely go well.
It's easy to imagine in your head how your project could probably be so much better, if only you could somehow not have to deal with all the technical debt and a bunch of legacy poor architectural decisions getting in the way ... but does that really have to call for a complete rewrite?
Especially with Rust, where "compiler-driven development" is a thing, and you can relatively confidently do major re-architecturing and then largely just fix errors until it works again ...
Taking something working, that you can hack on, and breaking and fixing it over and over again as needed, etc ... is often so much more productive than starting from scratch. You get the mental satisfaction of improving the thing ... rather than throwing out all your previous hard work.
I strongly encourage you to reconsider.
Look at Bevy: it managed to literally replace its heart: the whole ECS, and is now in the process of replacing its core scheduling ... and that was largely possible because there was all the other stuff – there was something working to come back to, after you fix the major breakage you caused. If Bevy had decided to abandon its course and do a complete from-scratch rewrite in order to fix its core foundations ... it probably would have died as a project.

I am talking from experience. Both from my own projects and from observing others. Complete rewrites rarely go well.

I should be a bit more careful with my words I guess.  I'm also reusing as much logic as I can like this:
https://github.com/StarArawn/bevy_ecs_tilemap/blob/main/src/neighbors.rs#L68
Which will be largely kept just moved into a new location and tweaked to work with the new API.
The one spot where I can't really facilitate reuse is the 3d iso stuff and I've mentioned some of the hardships I had there. I'm close to having square tile maps at a point where I can release it into the wild and get feedback on the new API. At which point adding the other 2D tile types should be straight forward as they don't really change logic that much outside of the shaders(which are already written).
Eventually I would like to re-write the renderer for 2d tile maps as well to swap over to vertex pulling, but that's not on the top of my agenda at the moment. As for 3D tilemaps, I.E. stacked isometric, stacked hexagon, I've largely already written the code necessary for that and then some(Including some basic 3d isometric tile picking). Example:
https://i.imgur.com/wdi78vd.mp4

Especially with Rust, where "compiler-driven development" is a thing, and you can relatively confidently do major re-architecturing and then largely just fix errors until it works again ...

My approach so far has been to keep existing code intact. It gives me a clear baseline and a place to work off of.

OK. Sorry for being over-dramatic.
Sounds like you are confident in the process and it's going well. :)
Wish you all the best! Looking forward to the improvements. :)

OK. Sorry for being over-dramatic.

I don't think you were overly dramatic. It's important to highlight issues with any development. I'm certainly not the best developer and I'm always looking for opportunities to learn new things!

I have a very early WIP branch which can be found here:
https://github.com/StarArawn/bevy_ecs_tilemap/tree/rewrite

Are you planning on updating the plugin in its current state for bevy 0.7?

I'm not sure when bevy 0.7 will come out. Regardless I think an update will happen.

Is the new version 0.6 for bevy_ecs_tilemap ready for release or there are any problem with it? Is there a way to link to the trunk version of a dependency in the cargo.toml? Sorry, I'm just a beginner to the rust ecosystem and sorry for the out of topic comment.

@StarArawn please don't leave your existing users/community behind, while you work on your rewrite. Many people are already using and counting on your crate and it does not feel good to be abandoned with no maintenance, and having to wait for your rewrite.
Bevy 0.7 has been released now. Could you at least please release a new version of the existing codebase with support for Bevy 0.7?

@sggt, hopefully we'll get 0.6 as an official release soon, but if you need this crate in the meantime you can specify that cargo pulls directly from the main branch of the repo. See this section of the cargo docs where this is explained.
My project is not complex but I confirmed that after updating my Cargo.toml appropriately my project now builds against bevy 0.7. Relevant Cargo.toml update below:
# this should change to `bevy_ecs_tilemap = "0.6.0"` whenever that is released
bevy_ecs_tilemap = { git = "https://github.com/StarArawn/bevy_ecs_tilemap.git", branch = "main" }

@sggt, hopefully we'll get 0.6 as an official release soon, but if you need this crate in the meantime you can specify that cargo pulls directly from the main branch of the repo. See this section of the cargo docs where this is explained.
My project is not complex but I confirmed that after updating my Cargo.toml appropriately my project now builds against bevy 0.7. Relevant Cargo.toml update below:
# this should change to `bevy_ecs_tilemap = "0.6.0"` whenever that is released
bevy_ecs_tilemap = { git = "https://github.com/StarArawn/bevy_ecs_tilemap.git", branch = "main" }

The 0.6 release is live. 👍

The 0.6 release is live. 👍

FYI it's not on Github releases, which was a tiny bit confusing for a moment.

The 0.6 release is live. 👍

FYI it's not on Github releases, which was a tiny bit confusing for a moment.

Ah yes I'll add one shortly.

Hi! I really like the new API, I'd like to readd support for animated tiles and color materials, do you reckon it's okay if I try to contribute those parts in?

Hi! I really like the new API, I'd like to readd support for animated tiles and color materials, do you reckon it's okay if I try to contribute those parts in? In that case, did you plan to have similar API for them (component to enable animation and passing color to tiles for color material).

Yeah that would be a great help! I haven't had as much time as I'd like lately, but I'm planning on finishing up the rewrite soon. :)

Hey did bevy ever fix the performance problems with having large number of entities

Hello! Will the rewrite have a more flexible animation feature?

Hey did bevy ever fix the performance problems with having large number of entities

Sadly not yet as far as I am aware.

Hello! Will the rewrite have a more advanced/flexible animation feature?

Not initially but it's something I would like to support! It just requires sending a bit more information to the GPU.

How's the rewrite progressing? Are you planning on updating the crate for Bevy 0.8 with the main branch code?
I updated the main branch code for bevy main here: https://github.com/rparrett/bevy_ecs_tilemap/tree/bevy-0.8 and would be happy to PR if that's the plan.
Bevy 0.8 is likely to land this week.


Concatenation of contiguous vector registers into 1 array varnode

Is your feature request related to a problem? Please describe.
It is currently impossible to create an array of arrays if the contained arrays are in a vector register. Trying to create an array of size 4 out of 4 float vector registers (float[4][4]) yields the following error. Failed to retype variable 'm1': Storage cannot be expanded to 64 bytes: ...
Describe the solution you'd like
To allow the storage to be expanded for registers which have been declared as a vector register within the pspec as long as the registers are contiguous.
Describe alternatives you've considered
Naming each part of the matrix array 'm[index]' as seen below.

I think this was implemented. At least I'm not having a problem trying to make any float[4][4] types anymore.


Cannot require a customized js file

A js file which exports some functions is required by a http script, however the requiring result is only an empty object.
JS file:   test.js
const TestFunc = () => {
  console.log("test");
};

exports.TestFunc = TestFunc

test.http file
### Test Common Functions
{{
    const test = require('./test.js')
    console.log(test);
}}

Guess what is printed...
It turns out to be just an empty object.

So how can I write a js file which can be required by the http file?

### Test Common Functions {{ const test = require('./test.js') console.log(test.TestFunc()); }}

### Test Common Functions
{{
const test = require('./test.js')

console.log(test.TestFunc());
}}

You need to call your method

I called the method with following codes:
### Test Common Functions
{{
    const test = require('./test.js')
    test.TestFunc()
}}

Unfortunately I got nothing output...

It works, but not the way you expect it to. It works directly on the command line. In vscode it would also work, but the console output is not the output channel but really stdout.

For scripts that I execute directly, I overwrite the console output so that it appears in the OutputChannel. For require scripts I have not yet found a trick for this, so it writes directly to stdout. Pass the function the reference to the console then it would work.
### Test Common Functions
{{
const test = require('./test.js')

console.log(test.TestFunc(console));
}}

const TestFunc = (c) => {
  c.log("test");
};

exports.TestFunc = TestFunc

It works, but not the way you expect it to. It works directly on the command line. In vscode it would also work, but the console output is not the output channel but really stdout.

For scripts that I execute directly, I overwrite the console output so that it appears in the OutputChannel. For require scripts I have not yet found a trick for this, so it writes directly to stdout. Pass the function the reference to the console then it would work.
### Test Common Functions
{{
const test = require('./test.js')

console.log(test.TestFunc(console));
}}

const TestFunc = (c) => {
  c.log("test");
};

exports.TestFunc = TestFunc

Yeah ,passing the outer console object to the required function works, it print to OutputChannel and the log can be read in the httpyac Console.
And if there is a way to overwrite the global console.log method or overwrite the origin required script to override the console.log method in the script ? Or maybe it's a problem that if it's worthy to do such stuff   ......

And if there is a way to overwrite the global console.log method or overwrite the origin required script to override the console.log method in the script

PR is welcome. I've been looking into NodeJS vm for a while, but haven't found it. If you have an idea, please feel free.

@AnWeber Quote of the day: "It works, but not the way you expect it to." :-)


The Log Noise plugin isn't compatible with the bosh release

This should be noted in the release documentation and the nozzle documentation.

@MasslessParticle this could also be resolved by adding the route-registrar from the routing release to register the nn-accumulator route.

We are planning to deprecate the noisy neighbor bosh release - https://www.pivotaltracker.com/story/show/160111476


deployment to heroku is not working because i cant serve json over https - need this SSL resolved STAT!

https://drupal.stackexchange.com/questions/24072/how-to-simply-make-the-whole-site-https
https://young-island-27652.herokuapp.com/test

finished today, issue was with .htaccess file in /public_html/amman


Integrate zenify/doctrine-fixtures within API

Codecov Report

Merging #378 (7add543) into master (343aa13) will increase coverage by 0.04%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
+ Coverage   71.83%   71.87%   +0.04%
==========================================
  Files         444      448       +4
  Lines       14598    14668      +70
==========================================
+ Hits        10486    10543      +57
- Misses       4112     4125      +13

Impacted Files
Coverage Δ

app/helpers/ZenifyFixtures/Alice/AliceLoader.php
68.00% <68.00%> (ø)

...tures/Alice/Fixtures/Parser/Methods/NeonParser.php
73.91% <73.91%> (ø)

...fyFixtures/Contract/Alice/AliceLoaderInterface.php
100.00% <100.00%> (ø)

...pp/helpers/ZenifyFixtures/DI/FixturesExtension.php
100.00% <100.00%> (ø)

app/model/repository/AssignmentSolutions.php
90.36% <0.00%> (+1.20%)
:arrow_up:

Continue to review full report at Codecov.

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


[Event Change] T83 "Item Journal Line" - procedure OnBeforePickDimension()

------------------ Change IntegrationEvent Attribute ----------------------
IncludeSender : true, GlobalVarAccess : true
-------------------------------------------------------------------------------

[IntegrationEvent(TRUE, TRUE)]
local procedure OnBeforePickDimension(var ItemJournalLine: Record "Item Journal Line"; var IsHandled: Boolean,
CurrFieldNo: Integer, TableArray: array[10] of Integer, CodeArray: array[10] of Code[20], InheritFromDimSetID: Integer, InheritFromTableNo: Integer)
begin
end;

What is you version? Property GlobalVarAcces have been obsoleted long time ago? And why you need IncludeSender property, you already have ItemJournalLine parameter with var?

GlobalVarAccess was deprecated in 16.x, there is concept of protected var instead. You can list here the global vars that have to be protected and accessible from extensions.
Please put the code surrounding the place, where you expect a new event call, as a text not as a screenshot.
It helps us finding the right place quickly if you put enough code to identify the unique place.


[Feature Request]any plan about desktop web test?

It is so amazing!if we can use it in desktop web test, it can be great!

Hi @chenshuichao! There is experimental support for web - docs can be found here. Let us know how it works for you!


Enable running of additional ceph commands before cephadm deployment

Why before? cephadm shell won't work before deploy...

Why before? cephadm shell won't work before deploy...

Yeah, should go just after cephadm-deploy.yml

Well spotted, thanks guys. My fault for trying to cut corners...


[Bug Report] Component select won't auto collapse when click outside of this component

Element UI version
1.4.4
OS/Browsers version
chrome 60
Vue version
2.4.0
Reproduction Link
https://jsfiddle.net/793n0uLe/1/
Steps to reproduce
Steps to reproduce

Click the third select component: this component is expanded
Click outside of this component:  this component doesn't collapse.

This issue is caused by nodeList(in clickoutside.js) deleting incorrect item, item which to be deleted shouldn't determined by id.

What is Expected?
This select component will auto collapse.
What is actually happening?
This select component doesn't collapse.

Fixed in #6973


Can't resolve 'styled-components'

Having this error occur trying to use your package. Otherwise,

it looks great!

we have styled-components as peer dep, it should work
https://github.com/lucasbesen/react-kanban-dnd/blob/baf0c3c91be03ef209700a6ecd7f94f35d82ac49/package.json#L58-L60
do you have any reproducible bug?

Same problem here..
this is my package.json
{   "name": "milestonekanban",   "version": "0.1.0",   "private": true,   "dependencies": {     "@material-ui/core": "^3.9.3",     "@material-ui/icons": "^3.0.2",     "react": "^16.8.6",     "react-dom": "^16.8.6",     "react-kanban-dnd": "^0.3.0",     "react-router-dom": "^5.0.0",     "react-scripts": "2.1.8"   },   "scripts": {     "start": "react-scripts start",     "build": "react-scripts build",     "test": "react-scripts test",     "eject": "react-scripts eject"   },   "eslintConfig": {     "extends": "react-app"   },   "browserslist": [     ">0.2%",     "not dead",     "not ie <= 11",     "not op_mini all"   ] }

Same problem here
My package.json is:
{
  "name": "milestonekanban",
  "version": "0.1.0",
  "private": true,
  "dependencies": {
    "@material-ui/core": "^3.9.3",
    "@material-ui/icons": "^3.0.2",
    "react": "^16.8.6",
    "react-dom": "^16.8.6",
    "react-kanban-dnd": "^0.3.0",
    "react-router-dom": "^5.0.0",
    "react-scripts": "2.1.8"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject"
  },
  "eslintConfig": {
    "extends": "react-app"
  },
  "browserslist": [
    ">0.2%",
    "not dead",
    "not ie <= 11",
    "not op_mini all"
  ]
}

add styled-components as dependencies

thanks you @jgcmarins . Add styled-components as dependencies does not completely solve the problem, because add styled-components install the lastest version and your library is not full compatible with it.
My current styled-components version is ^4.2.0 and when I run reactJs I get 18 errors where the first is:

Error: Invariant failed: Cannot get draggable ref from drag handle

you can reproduce these errors changing the styled-components from  3.4.9 to ^4.2.0 on live demo.
If I change styled-components version to ^4.2.0 to 3.4.9 it works.

@Blackleones Does this work with styled-components 4.2.0?


Setting custom bracket color when using built-in "Bracket Pair Colorization"

With the latest stable vscode v 1.6.0, feature "Bracket Pair Colorization" is built-in, but the custom color setting for this feature is confusing. It hints that default high light color could be overrided by  setting in  "workbench.colorCustomizations".
Q1: what property could user config in settings.json  to change bracket color (I did googled, but not found )?
Q2:For instance, I just want to change the most inner-layer bracket color, how to do it, (what if the most 2, 3 and so on... inner-layer color)?
After clarify the answers on questions above, I hope there coulde be some ui settings to config for this new feature.

/assign @hediet

As for Q1, I found the editorBracketHighlight.foreground[1-6] properties.
I'm using them like the following, which works fine:
    "workbench.colorCustomizations": {
        "editorBracketHighlight.foreground1": "#ffd700",
        "editorBracketHighlight.foreground2": "#90ee90",
        "editorBracketHighlight.foreground3": "#87cefA",
        "editorBracketHighlight.foreground4": "#da70d6",
        "editorBracketHighlight.foreground5": "#ff7f50"
    },

@athos, thanks. Now it is worked.


Do a full optimization pass on Link Generation

From @rynowak on Monday, November 23, 2015 8:12:59 AM
We haven't put any work yet into optimizing the link generation process, but we need to give it a look - primarily for allocations of 'scratch' data structures like dictionaries.
Copied from original issue: aspnet/Routing#236

Closing as Link Generation has now shipped!


support scp-style shorthand urls with users other than git@

See discussion: https://discuss.dvc.org/t/dvc-import-fails-with-git-failed-to-fetch-ref-from/1997
scp shorthand URL format is as follows:
[user@]host.domain.com:path/to/resource

So, user can be anything, and is optional.

[!NOTE]
Does not work with absolute paths in scp-style URLs, eg: git@github.com:/path/to/resource.

thanks @skshetry !

The current LFS SSH URL doesn't support a leading / in the URL path when the host and path are separated by a :. I'll need to check whether GitLab and GitHub allow for the leading : in their SSH URLs.

I am not sure how lfs uses the user, but it would be nice to remove the assumption about the user from scp-style shorthand URLs.

@skshetry LFS uses the user to run
ssh <user>@<host> git-lfs-authenticate <project_path>.git download|upload

to retrieve credentials for making requests to the batch API. I don't see any reason why the user would have to be git, I've never seen another one though, but it seems it does happen. I think we can relax the LFS SSH URL regex to support arbitrary users.

We convert it to http url, right? So, what is user going to be used for?

The current LFS SSH URL regex doesn't support a leading / in the URL path when the host and path are separated by a :. I'll need to check whether GitLab and GitHub allow for the leading / in their SSH URLs.

The resource path could be anything. If it's a leading slash, it's an absolute path, otherwise it's a relative path to the user's directory.

We convert it to http url, right? So, what is user going to be used for?

Well, we infer the LFS API URL from the Git URL, so there the user is irrelevant. But we need the user for the
ssh <user>@<host> git-lfs-authenticate <project_path>.git download|upload

command. But there what matters is the right user is used, and it should be same user used for cloning via SSH. So I think it's fine to use any user and not only git.
I've tested absolute and relative paths on GitHub and GitLab, both work for cloning via SSH URLs and for running git-lfs-authenticate. Would you like (me?) to refactor the _SSHLFSClient implementation to use the new SCP regex?

Thank you. I think I understand it a bit more.
I think the check like follows are enough then, right?
if url.startswith("ssh://") or is_scp_style_url(url):
  ...

From what I understood, we can get rid of _URL_PATTERN, right? If that's enough, let me know and I can do the changes. Otherwise, feel free to push in the pull request.

The check and removal of _URL_PATTERN look good. But there's more – roughly this:

Extracting information from the SSH URL must be adapted to use SCP_REGEX:
https://github.com/iterative/scmrepo/blob/1096899158e9d195b4f6fcae9d320ebdbaf0912e/src/scmrepo/git/lfs/client.py#L236-L239
SCP_REGEX doesn't capture a port, so port is not matched anymore; user is matched now; SCP_REGEX doesn't use named capture groups, so capture group matches must be extracted differently (watch out for the "user" capture group including a trailing @, which needs to be stripped – or we adapt SCP_REGEX).

_SSHLFSClient's constructor must accept an optional user argument (should default to None), and port must be removed (class instance attributes must be adapted accordingly as well as their uses in _get_auth_header and _git_lfs_authenticate).

_git_lfs_authenticate must accept an user argument, and port must be removed. The user keyword argument in the self._ssh.run_command(...) call must receive the user argument.

If you'd prefer I refactor the LFS code, feel free to assign the PR to me, @skshetry.

If you'd prefer I refactor the LFS code, feel free to assign the PR to me, @skshetry.

Hi, I pushed aa1ce61. Let me know what you think. :)

Thank you @sisp for the reviews. 🙂

No problem, @skshetry. 🙂


[dev/1.26] repo: Dev v1.26.6

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

quic...but I think it is not responsibility of this patch.

the coverage seems to be indeterminate - its failing ~1/5 ci runs - it failed on the reopen - so i think lets fix it here


Add source version to javadoc config

Test if this will fix these warnings

Warning: [WARNING] javadoc: warning - The code being documented uses modules but the packages defined in https://docs.oracle.com/javase/8/docs/api/ are in the unnamed module.

Close. Will merge against main-test


Chapter 12 missing optimistic updates

The summary of Chapter 12 ends with "How to use the React’s useFormStatus hook for optimistic updates", but that is not actually covered.

Thanks! Will fix.


Update dependency eslint-config-prettier to v8.6.0

This PR contains the following updates:

Package
Type
Update
Change

eslint-config-prettier
devDependencies
minor
8.5.0 -> 8.6.0

Release Notes

prettier/eslint-config-prettier
v8.6.0
Compare Source

Added: [vue/multiline-ternary]. Thanks to @xcatliu!

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (8.6.0). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


domain controller: run a driver sync after updating

closes: #340
What
Ensures that the Status.LoadBalancer.Ingress[].Hostname field is set after the target CNAME is resolved on the Domain.
How
Add a call to Driver.updateIngressStatuses in the UpdateStoreHandler.Update method.
Breaking Changes
None

Is the driver Ingress-specific? I already saw it being shared by the ingress controller and moduleset controller, so I figured why not?

The ingress controller is already watching Domains among other objects:
I think the issue is that when it watches these, it is just updating the store with the updates, but that isn't actually triggering an resync on any resources. I think this may actually be a more generic class of problem that we should handle.

Yeah, this was the first path I tried to go down and didn't really get anywhere with. That Watches for all of the other resources didn't seem to do anything that I could tell. I expected that the ingress controller would get notifications when any of those resources was updated, but that didn't seem to be the case.

Oops, did that let me merge before checks passed?


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for parsimony-dev processing.
🔨 Explore the source changes: cf31a1001a6443c6e6bfb318f04cb6e663690af6
🔍 Inspect the deploy log: https://app.netlify.com/sites/parsimony-dev/deploys/617fa5b1f2c9690007359093


Update print statements from WMComponent/ directory

test this please

Please merge

Sorry Eric,  could you fix the conflict?

@ticoann updated. Conflict was code you removed, so all good!

Thanks Eric


psi paper add

Also, you may follow the example PR (for formatting etc.): https://github.com/secretflow/secretflow/pull/541

Since this PR has been inactivate for more than two weeks, I'll close this PR. Please do re-open this or propose another one if necessary, thanks.


Adding ROKA URL to the 'Places using Card' section

Hi @jessepollak,
Could you please accept this pull request to add (https://www.rokahub.com) as part of the 'Places using Card'.
Thanks,
Vikram

Thanks @BallisticPain :)


Dota 2 effects not working Aurora 0.6.4b

Expected Behaviour
After patching aurora gsi for dota 2, the app should get info for Dota 2 Health, mana, and some heroes specific effects should work
Actual Behaviour
It just detect the game and lit the keyboard but nothing else happens
Reproduction steps
The issue can be reproduced by doing following steps:
Select Dota 2 profile in Aurora RGB
patch GSI
Start the game
Demo a hero like Axe to test effects
-> no effects
Frequency
Every time
Aurora Version:
v0.6.4b
Previously an Issue?
IDK
2019-07-27 23.17.06.log

#1638 fixes this issue. Will leave open until it's merged

#1638 fixes this issue. Will leave open until it's merged

ok thanks but could you tell me what i have to do exactly, that would be very kind ^^


I would like to add some free and paid courses in the readme

please assign

Sure, go ahead :)

Can u also add a "hacktoberfest" label tag in your project
So my contribution will be counted in hacktoberfest

This is technically a collection repo and as they are not in line with Hacktoberfest that's why I haven't added the hacktoberfest label. I don't want to open the repo up for people adding low quality links just for a hacktoberfest t-shirt. I hope you understand.
I have a couple of other repos that have the hacktoberfest label if you wish to contribute to those. I'll be listing more issues over the coming day.

No issue I will contribute in this💜

Can i work on this issue?
The assignee doesn't seems making any progress

Sure, if you have some useful links then go ahead and add them


[bug] ADB Reverse Command Fails on Android Development with Next Js

Describe the bug
Bug Report: ADB Reverse Command Fails on Android Development
Description
When running the command pnpm tauri android dev, I encounter an error where the adb reverse command fails with exit code 1. This prevents the app from being served correctly to the Android emulator.
Expected Behavior
The app should compile successfully, and the adb reverse command should allow access to the local server running on http://localhost:3000 from the emulator.
Actual Behavior
The following error is displayed in the terminal output:
Error command ["/home/sgeist/Android/Sdk/platform-tools/adb", "reverse", "tcp:3000", "tcp:3000"] exited with code 1

Additional Information

Tauri Version: 2.0.0-rc.15
Next.js Version: 14.2.13
Operating System: [Your OS, e.g., Ubuntu 20.04]
Android Emulator: Pixel 8a API 35
Log Output: The adb logcat output shows various messages, but no specific errors related to adb reverse.

Troubleshooting Steps Taken

Verified that ADB is functioning correctly with adb devices.
Attempted to run the adb reverse tcp:3000 tcp:3000 command manually, which also fails.
Restarted the Android emulator multiple times.
Ensured that the necessary permissions and configurations for ADB are correctly set.

Possible Solution

It may be beneficial to implement better error handling for the adb reverse command within the Tauri CLI or to provide clearer messages if the command fails, and why.

Reproduction
No response
Expected behavior
When executing the command pnpm tauri android dev, the following should occur:

Successful Compilation: The Tauri and Next.js applications should compile without errors, indicating that the build process has completed successfully.

Server Start-Up: A local development server should start at http://localhost:3000, allowing access to the application in a web browser or from the Android emulator.

ADB Reverse Command Success: The command adb reverse tcp:3000 tcp:3000 should execute successfully, establishing a reverse proxy from the emulator to the local development server. This would enable the Android application to access resources hosted on the local machine.

App Functionality: Once the above steps are completed successfully, the Android emulator should launch the application, allowing users to interact with it without any connectivity issues. The application should display the UI correctly, reflecting any updates made in the development environment.

No Error Messages: There should be no error messages or warnings in the terminal output regarding the ADB connection or reverse command, ensuring that the setup is working as intended.

Full tauri info output

[✔] Environment
    - OS: Fedora 40 X64
    ✔ webkit2gtk-4.1: 2.44.3
    ✔ rsvg2: 2.57.1
    ✔ rustc: 1.78.0 (9b00956e5 2024-04-29)
    ✔ cargo: 1.78.0 (54d8815d0 2024-03-26)
    ✔ rustup: 1.26.0
    ✔ Rust toolchain: stable-x86_64-unknown-linux-gnu (default)
    - node: 20.11.1
    - pnpm: 9.4.0
    - yarn: 1.22.22
    - npm: 10.8.1

[-] Packages
    - tauri [RUST]: 2.0.0-rc.15
    - tauri-build [RUST]: 2.0.0-rc.12
    - wry [RUST]: 0.43.1
    - tao [RUST]: 0.30.2
    - @tauri-apps/api [NPM]: 1.6.0
    - @tauri-apps/cli [NPM]: 2.0.0-rc.0

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../out
    - devUrl: http://localhost:3000/
    - framework: React (Next.js)
    - bundler: Webpack

Stack trace
No response
Additional context
No response

Here the ERROR output
`pnpm tauri android dev                                                                         !10303

<EMAIL_ADDRESS>tauri /home/sgeist/qr-pong-native-next
tauri "android" "dev"

Info Detected connected device: Medium_Phone_API_35 (sdk_gphone64_x86_64) with target "x86_64-linux-android"
Running BeforeDevCommand (`pnpm dev`)

<EMAIL_ADDRESS>dev /home/sgeist/qr-pong-native-next
next dev

▲ Next.js 14.2.3

Local:        http://localhost:3000
Environments: .env

✓ Starting...
Info detected host target triple "x86_64-unknown-linux-gnu"
Finished dev profile [unoptimized + debuginfo] target(s) in 0.33s
Info symlinking lib "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" in jniLibs dir "/home/sgeist/qr-pong-native-next/src-tauri/gen/android/app/src/main/jniLibs/x86_64"
Info "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" requires shared lib "libandroid.so"
Info "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" requires shared lib "libdl.so"
Info "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" requires shared lib "liblog.so"
Info "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" requires shared lib "libm.so"
Info "/home/sgeist/qr-pong-native-next/src-tauri/target/x86_64-linux-android/debug/libapp_lib.so" requires shared lib "libc.so"
Error command ["/home/sgeist/Android/Sdk/platform-tools/adb", "reverse", "tcp:3000", "tcp:3000"] exited with code 1
 ELIFECYCLE  Command failed with exit code 1.
`

Is Fixed on the newest Version:
2.0.0-rc.16


Output mime type action feedback

Refs: #93742
Some feedback regarding the Output mime type action

Use a hover message
Make it accessible, so users can navigate to it via keyboard - or have it in command palette
Do not trigger on right click, only trigger on left click
"bulitin"  description of each quick pick option not that useful. Also not readable. I suggest to not use it, or if used consider changing to "built-in"

Now users can navigate through tabbing.

@rebornix nice, thanks! Should I open new issues for my concerns 1., 3. and 4?

1 and 4 are fixed as well. I'm a bit not clear about 2, doesn't that mean "Tab"? If not feel free to create a new issue with details.

Yeah 2 means accessible via Tab and that works. That is why I think that one is fine and I did not mention it in my comment. I was only conerned for 1 3 and 4


Flaky fail on final test invokes pytest session fixture

I have a pytest fixture which runs at the "session" scope. This fixture starts a local web server, and initiates a sauce labs connection. Upon teardown, it stops the server and closes the sauce connection.
When the final test of my test run is a "flaky" test and fails, it acts as if the session is over and calls my fixture tear down.
Then, the flaky test is rerun, which invokes the session fixture once again.
I would like the session fixture to only be called once obviously due to the overhead of having to spin up the server and start the sauce connection again.
This could be a pytest issue, but I figured I would open an issue here as well to see if there is any suggestions or solutions in the works?

I have a similar issue with the following example:
`import pytest
from flaky import flaky
class TestFlaky():
@classmethod
def setup_class(cls):
    print("In setup class")

@flaky(max_runs=5, min_passes=1)
@pytest.mark.parametrize('param', [1, 2, 3])
def test1(self, param):
    print("In test1, param={}".format(param))
    assert param == 1

@classmethod
def teardown_class(cls):
    print("In teardown class")`

When the last test fails, the teardown is triggered, but then the whole sequence of setup->test->teardown is triggered again for the remaining attempts.

Same issue here. A dumb workaround for now is to create a non-flaky dummy test at the end of the module/session scope:
@flaky
def my_last_real_test():
    ...

def test_nonflaky_dummy():  # Exists as a workaround for https://github.com/box/flaky/issues/128
    pass


[Feature Request]: Add OrderedListState test in StateInternalsTest

What would you like to happen?
As the number of state operators in Apache Beam grows, we need to update the existing StateInternalsTest.
Currently, tests exist for ValueState, BagState, SetState, MapState, CombiningValueState, and WatermarkHoldState, but not for MultiMapState and OrderdListState, which were added later.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


not support viewport units: vh and vw

I wanna use this polyfill to fix vh/vw not working in old safari, here is a demo: https://codepen.io/mherchel/pen/mIueA. But it doesn't work. Do you have any plan to support it?

As far as I can see there is no Safari Version that does support viewport units and not calc.
http://caniuse.com/#feat=calc
http://caniuse.com/#feat=viewport-units
Both CSS Features are supported since Safari-Version 6 and 6.1 for iOS.
Additionally it is not the intend of this polyfill to add support for viewport units for browsers lacking this.


Kerberos OAuth2 Grant Documentation Error

Description:
While configuring the Kerberos Grant type in the APIM 3.1.0 we were able to observe that in the documentation it has mentioned enabling "[oauth.grant_type.kerberos_grant]" in the IS-as-KM "deployment.toml" but when referring to the "identity.xml.j2" file we can observe that the if condition is set as below.

{% if oauth.grant_type.kerberos.enable is sameas true %}

Hence even though we have enabled Kerberos Grant we were not able to see it in the UIs. The value we should use in the "deployment.toml" file in order to enable Kerberos should be "[oauth.grant_type.kerberos]".
Content Positioning in Documentation:

Link:https://apim.docs.wso2.com/en/latest/learn/api-security/oauth2/grant-types/kerberos-oauth2-grant/

Heading (& Step): Configuring Kerberos Grant using IS as KM

Suggested Labels:
APIM 3.1.0

Fixed 3.1.0 space with https://github.com/wso2/docs-apim/pull/2744
Already fixed in 3.2.0 and master


Critical dependency warning

Webpack gives the following warning
WARNING in<EMAIL_ADDRESS>3:24-31
    Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Fixed by v1.0.3


Add an HTML Helper-to-Partial Tag Helper migration section

Fixes https://github.com/aspnet/Docs/issues/7464
Fixes https://github.com/aspnet/Docs/issues/7476
Internal Review Page

@scottaddie  I'm on VAC /cc @guardrex


CMPXCHG not emulated

For fun, I tried building a recent Linux 6.9 kernel using Osamu Kurati's FM TOWNS patchset (http://www.nurs.or.jp/~kurati/towns/#kernel) When trying to boot it in Tsugaru, the VM halts on an unhandled instruction:
VM Aborted!
Device:486DX
Reason:Clocks-Passed is not set.  Opcode=0FB1H
Debugger Enabled.
Towns TIME (Nano-Seconds):<PHONE_NUMBER>5
CS:EIP=0060:C1422CB3  LINEAR:C1422CB3  EFLAGS=00000046  CPL=00
EAX=00000000  EBX=C16A8DB0  ECX=FFFFFFFB  EDX=C15C27C0
ESI=00000000  EDI=00000001  EBP=C15B7F5C  ESP=C15B7F58
CS=0060(LIN:00000000)  DS=007B(LIN:00000000)  ES=007B(LIN:00000000)
FS=0000(LIN:00000000)  GS=0000(LIN:00000000)  SS=0068(LIN:00000000)
CR0=80050033  CR1=00000000  CR2=00000000  CR3=0175B000
CF0  PF1  AF0  ZF1  SF0  TF0  IF0  DF0  OF0  IOPL00  NT0  RF0  VM0  AC0
Default Operand Size=32  Default Address Size=32  Stack Address Size=32
SS+00000000:A0 43 5D C1 70 7F 5B C1 B4 91 72 C1 A0 43 5D C1
SS+00000010:00 00 00 00 63 32 54 C1 84 7F 5B C1 7D 93 72 C1
0060:C1422CB3 0FB1

0FB1 is the encoding for CMPXCHG according to https://www.felixcloutier.com/x86/cmpxchg .

Interesting!  Thank you for testing!  I've added support for CMPXCHG in the latest source.  But, I cannot test in my environment if it is correct.  Let me know if it makes your Linux kernel run!

Nice, thanks! I'll give it a try when the next binary gets tagged.


mdt-column with ng-repeat

Hi guys,
Great library to use for table and interesting features. But I'm having small issue with mdt-column thing. I wan't create the column via ng-repeat but every time I render it, it doesn't show any column at all. Any solution for this.
Cheers guys.

delock,
I am using something like this:
            <mdt-header-row>
                <mdt-column align-rule="left"  column-sort="true" ng-repeat="col in $ctrl.headerTitle">{{col | fieldToHeader:true}}</mdt-column>
            </mdt-header-row>

And it works fine for me, is this what you want?


Fixed MSOLEDBSQL encryption related errors

Adding -Encrypt Optional allows fallback to insecure channel, the default on WID

also fixed in issue #19 but added some other optimizations there as well.


Refactoring Linx duplicated types

Working to mitigate unnecessary duplicated types on Linx related types.

Once you are happy with the progress here you can tag me @Samuel-Brito19 !
Btw: I don't think you need to update the deco version, seems out of scope for this PR

Hi @viktormarinho! I did some modifications trying to solve the problem, but I have some doubts about some cases.
For example, this Response interface is present in 3 files but have no use in none of them. Should I take it off?
export interface Response {
  Errors: LinxError[];
  IsValid: boolean;
  RefreshBasket: boolean;
  ResponseCallBack: ResponseCallBack;
  SuccessMessage: null;
  Url: null;
  Warnings: unknown[];
}

There's other cases similar to that one. Check my changes and please give me a feedback!

Hi @viktormarinho! I did some modifications trying to solve the problem, but I have some doubts about some cases. For example, this Response interface is present in 3 files but have no use in none of them. Should I take it off?
export interface Response {
  Errors: LinxError[];
  IsValid: boolean;
  RefreshBasket: boolean;
  ResponseCallBack: ResponseCallBack;
  SuccessMessage: null;
  Url: null;
  Warnings: unknown[];
}

There's other cases similar to that one. Check my changes and please give me a feedback!

@Samuel-Brito19 If there are types without i believe you can delete them. Can you share the other cases? For the Response case it's ok.


Add the global and local prompts/models support

This pull request introduces support for managing global and local prompts/models. This implementation is based on the requested feature outlined in #96.
The key features include:

Definition of Global and Local Prompts/Models:

Global Prompts/Models:

Created using CREATE GLOBAL PROMPT or CREATE GLOBAL MODEL.
Shared across different databases.

Local Prompts/Models:

Created using CREATE LOCAL PROMPT, CREATE PROMPT, or equivalent for models.
Default to local if no type is specified.
Limited to a single database.

State Transition Support:

Users can toggle a prompt/model's state between global and local using:

UPDATE PROMPT 'prompt_name' TO GLOBAL;
UPDATE PROMPT 'prompt_name' TO LOCAL;

Naming Constraints:

Prompt and model names must be unique across both global and local types, ensuring no conflicts.

Simplified Update and Delete Operations:

Operations such as UPDATE and DELETE do not require specifying the type of prompt/model (global or local).
If the type is specified, the system will handle it appropriately.

Thank you very much


Issue configuring service end points for prometheus.yml

I am having hard time figuring out on how to structure code for service end points as I completely new to coding.
My goal is to get pod and container metrics from my kubernetes-cluster.
I have deployed kube-state-metrics agent post which I am trying to configure the service endpoints in the Prometheus configuration file and I am getting error. Please need you guys help. Thank you so much in advance.
I have attached my configuration file.
prometheus.txt
Also find the error when executing the yaml file.
prometheus.service - Prometheus
Loaded: loaded (/etc/systemd/system/prometheus.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Mon 2017-10-30 10:22:37 UTC; 27min ago
Process: 26346 ExecStart=/usr/local/bin/prometheus -config.file /etc/prometheus/prometheus.yml -storage.local.path /var/lib/prometheus/ (code=exited, status=1/FAILURE
Main PID: 26346 (code=exited, status=1/FAILURE)
Oct 30 10:22:37 ip-172-31-21-51 systemd[1]: Stopped Prometheus.
Oct 30 10:22:37 ip-172-31-21-51 systemd[1]: Started Prometheus.
Oct 30 10:22:37 ip-172-31-21-51 prometheus[26346]: time="2017-10-30T10:22:37Z" level=info msg="Starting prometheus (version=1.7.1, branch=master, revision=3afb3fffa3a29
Oct 30 10:22:37 ip-172-31-21-51 prometheus[26346]: time="2017-10-30T10:22:37Z" level=info msg="Build context (go=go1.8.3, user=root@0aa1b7fc430d, date=20170612-11:44:05
Oct 30 10:22:37 ip-172-31-21-51 prometheus[26346]: time="2017-10-30T10:22:37Z" level=info msg="Host details (Linux 4.4.0-1038-aws #47-Ubuntu SMP Thu Sep 28 20:05:35 UTC
Oct 30 10:22:37 ip-172-31-21-51 prometheus[26346]: time="2017-10-30T10:22:37Z" level=info msg="Loading configuration file /etc/prometheus/prometheus.yml" source="main.g
Oct 30 10:22:37 ip-172-31-21-51 systemd[1]: prometheus.service: Main process exited, code=exited, status=1/FAILURE
Oct 30 10:22:37 ip-172-31-21-51 systemd[1]: prometheus.service: Unit entered failed state.
Oct 30 10:22:37 ip-172-31-21-51 systemd[1]: prometheus.service: Failed with result 'exit-code'.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


TriCopter Mixers

additions to allow for control of arbitrary y-configuration tricopters

tricopter multirotor table for pitch and roll
tricopter mixer for yaw
autostart files
autostart index range 14000-14999

includes two yaw servo configurations

yaw+ for when positive servo rotation induces a positive yaw rotation
yaw- for when positive servo rotation induces a negative yaw rotation

this has been flown, works great

Awesome!

Merged, thanks!


Align prometheus operator job labels with grafana

we want to use this for release qualification. Without these changes 99% of the metrics don't show up on the dashboards


Telegram- sending 2 notifications per event and repetitive line in logger

Few things I'm noting with Telegram:
I receive 2 notifications per event.
Get repetitive line in logger: "Connection pool is full, discarding connection: api.telegram.org"
"alert_catch": {
"all": {"operator": "and", "cp": 9, "iv": 0.9}
}
but I'm still getting notifications for every single catch. That should only be sending messages for pokemon with at least CP 9 and IV 0.9, correct?

I'm getting the same "connection pool is full" error... I believe the fault is on their side, not ours.

From what I can tell, we should be able to safely ignore these warnings (see https://github.com/python-telegram-bot/python-telegram-bot/issues/341)
I don't know enough about telegram to say for sure.

which bot commit?

This still happens as of now; commit 4cce54823e4209ad0000b5166be7443e5c775194

check on master

Issue with "connection pool is full" fixed #5307


Fix readthedocs build

https://django-appsettings.readthedocs.io/ currently reports 404 Not Found. I don't think we changed anything, but we should take a look at it and fix it.

It seems that I don't have enough permissions to set up readthedocs webhook. Maybe @pawamoy will have to set up the readthedocs integration under his account.

I've recreated the project on RTD and added you as a maintainer. I think we'll need this in the repo:
# .readthedocs.yaml
# Read the Docs configuration file
# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details

# Required
version: 2

# Set the OS, Python version and other tools you might need
build:
  os: ubuntu-22.04
  tools:
    python: "3.11"
    # You can also specify other tool versions:
    # nodejs: "19"
    # rust: "1.64"
    # golang: "1.19"

# Build documentation in the "docs/" directory with Sphinx
sphinx:
   configuration: docs/conf.py

# Optionally build your docs in additional formats such as PDF and ePub
# formats:
#    - pdf
#    - epub

# Optional but recommended, declare the Python requirements required
# to build your documentation
# See https://docs.readthedocs.io/en/stable/guides/reproducible-builds.html
# python:
#    install:
#    - requirements: docs/requirements.txt

Also, build failed: https://readthedocs.org/projects/django-appsettings/builds/21718297/
Probably changes required by new Sphinx version.

I created #101. The tox -e build-docs now passes for me, locally and generates HTML docs.
I'm not sure how to test it with RTD though. I can now see the project, but it's till telling me that GitHub integration is not active: https://readthedocs.org/dashboard/django-app-settings/integrations/
@pawamoy Perhaps you could try to resync webhook (or set it manually)?

I've enabled builds on pull requests, could you try to amend and force-push or something, to see if it triggers a build correctly?


saveAsNTriplesFile creates folder not file

When I call saveAsNTriplesFile as described in examples or as in Sansa stack faq, I get a folder with the described path but inside are a success file and a list of partial nt formatted  files. they have a name: part-XXXXX (X for some Number). those files contain partial data for the desired file. A can of method: ".coalesce(1, shuffle = true)" leads to a folder with two files. one success and one merged part-00000 file. this is closer to what I want but is not what I or others would expect, right?

You're describing the expected behavior of normal spark operation. The coalesce(1) method moves all data to a single partition, and writes all data out to a single partition file. This approach works for smaller data sets but as your data grows larger your job will run slower.

Nothing to fix here as this is the expected behaviour of Spark.
As already mentioned, using coalesce(1) or repartition(1) may work for small data-sets, but large data-sets would all be thrown into one partition on one node. This is likely to throw OOM errors, or at best, to process slowly.
For Hadoop 2.x I would highly suggest that you use the FileUtil.copyMerge() function from the Hadoop API. This will merge the outputs into a single file.
With Hadoop 3.x this method has been removed though:
https://issues.apache.org/jira/browse/HADOOP-12967
https://issues.apache.org/jira/browse/HADOOP-11392
The best practice will be to use getmerge:
Usage: hadoop fs -getmerge [-nl]
Takes a source directory and a destination file as input and concatenates files in src into the destination local file. Optionally -nl can be set to enable adding a newline character (LF) at the end of each file. -skip-empty-file can be used to avoid unwanted newline characters in case of empty files.
Example:
hadoop fs -getmerge -nl /data.nt /opt/data.nt


[RHDM-894][ Update RHDM images to use RHEL8 (UBI) base layer

[RHDM-894][ Update RHDM images to use RHEL8 (UBI) base layer
https://issues.jboss.org/browse/RHDM-894
Signed-off-by: David Ward<EMAIL_ADDRESS>Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

Closing since UBI 8 will supported in 7.4.0, not 7.3.1.


Add tests for full text search query

What
Add unit tests for work included in PR https://github.com/cloudant/sync-android/pull/115 and enable full text search functionality.
Why
Tests are needed to validate and exercise the full text search query code.
How

Add validation tests to QueryValidatorTest
Add translation tests to SQLTranslatorTest
Create a QueryTextSearchTest and add text search query execution tests to the class
Enable full text search functionality via the validator.

reviewer @mikerhodes
reviewer @rhyshort
BugId: 46310

Looks good to me +1

Tests look good now, +1.


nflog: write as func and fix locker usage.

Closures are better written as functions.
Defer breaks the use of lock-unlock

Signed-off-by: johncming<EMAIL_ADDRESS>

@simonpasquier thoughts?

I don't feel strongly about closures vs. methods.
Regarding locking/unlocking, I don't see a problem right now. But the code you've submitted could have a deadlock:
	l.mtx.RLock()
	if le, ok := l.st[stateKey(q.groupKey, q.recv)]; ok {
                // No call to RUnlock().
		return []*pb.Entry{le.Entry}, nil
	}
	l.mtx.RUnlock()

I guess I don't really see the benefit of this

I don't feel strongly about closures vs. methods.
Regarding locking/unlocking, I don't see a problem right now. But the code you've submitted could have a deadlock:
	l.mtx.RLock()
	if le, ok := l.st[stateKey(q.groupKey, q.recv)]; ok {
                // No call to RUnlock().
		return []*pb.Entry{le.Entry}, nil
	}
	l.mtx.RUnlock()

yes, deadlock. I will fix it.
RUnlock undoes a single RLock call; it does not affect other simultaneous readers. It is a run-time error if rw is not locked for reading on entry to RUnlock.

Changed to a function, because this is not a typical scenario where closures are used, only params. See if this change is necessary, and if not, keep only the changes to lock.


Broken models in 1.18.10 and up

Yeah, I'm aware about this bug. It's due to new changes with the add-on API. Will put this on the task board for v1.2.0

I also noticed when I toggled on the global pack for Furnideco, all Marketplace skins, skin packs and skin parts are gone, except for uploaded skins.

That's because FurniDeco uses custom player animations for sitting, which disables the Character Creator skins


Unable to open markdown files.

Issue Type: Bug
Open any book that has a markdown readme file
Try opening the markdown file.
Fails with the below error:

Azure Data Studio version: azuredatastudio 1.37.0 (d904740d93d7df76a0ba361f20e4351813b57645, 2022-06-14T19:53:12.357Z)
OS version: Darwin x64 21.5.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
16.00GB (1.17GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

kusto
Mic
0.5.4

Known issue, this has to do with running multiple versions of ADS at the same time (insiders/RC1/stable). Nothing we can realistically do about it - fix/workaround is just to close all other instances of ADS and then  open up the version you're testing with.

Same as #18048. Closing as known issue with multiple instances open.


Option to bundle-split pega-component-map

Is your feature request related to a problem? Please describe.
I am using react-sdk-components to embed Pega views inside a non-pega Portal, but when importing from either react_pconnect.jsx or sdk_component_map.js, my bundle-size balloons to over 1.5 MB (Minified). From what I can tell, this is because sdk-pega-component-map.js and all components on that map are loaded unconditionally- whether a given component is or is not available on the sdk-content-server, or on the local component map.
Am I doing something wrong here? If not, can either alternate options be made available?
Describe the solution you'd like
This is not strictly what I want to happen, but merely one way to solve this issue.
If in sdk-pega-component-map.js, the following:
import SomeComponent from './SomePath/SomeComponent.jsx';
const pegaSdkComponentMap = {
  'SomeComponent': SomeComponent,
  ...,
}

Were changed to something like:
import { lazy } from 'react';

const pegaSdkComponentMap = {
  'SomeComponent': lazy(() => import('./SomePath/SomeComponent.jsx')),
  ...,
}

Only components that are not available from the SDK content server and the local component map would be loaded.
Describe alternatives you've considered
Outside of not using the sdk-component-map and react_pconnect, I dont think there are any alternatives(?).
A colleague of mine did notice that the (not embedded) Portal does not have this same issue, and instead appears to load components as needed.
Additional context
Add any other context or screenshots about the feature request here.

I just got done updating to all the latest versions, and the issue has gotten worse.
When I npx @pega/dx-component-builder-sdk create a Text-Input component, the custom-sdk version of that component uses import { getComponentFromMap } from '@pega/react-sdk-components/lib/bridge/helpers/sdk_component_map';, which in turn imports all other components through the component map, causing the bundles generated for this, and other custom components to be >= 1.5 MiB in filesize.

Thanks for using our react sdk components.  We're reviewing your feedback within the team and we will revert back as soon as we've completed this.


Add support for diffirrent Godot project paths

The extension currently assumes that the project path for your Godot project is in the workspace folder root. This PR detects projects which may be nested in sub-folders and lets you choose which one to use.
Dropdown for selecting the Godot project

Statusbar for showing current project

WIP - proof of concept of the project selector. Need to add directory scanning for project.godot files, skip prompt when there's only one Godot project, and skip extension activation if no projects are detected.

Any idea about when this might get merged? Thanks

neikeq has said in https://github.com/godotengine/godot-csharp-vscode/issues/25#issuecomment-926953926 that he'll take a look in October, since he's quite busy at the moment.

Thanks!

Thanks! (and sorry for the long wait)

No worries, thank you!


vscode: Fix on unstable

When switching from stable to unstable nixos, I was getting errors due to this vscode plugin not having some of the vscodeExt* attributes set:
error: attribute 'vscodeExtUniqueId' missing

       at /nix/store/y34pw71w25ay4j5bm5239m8gw0pp4w63-source/pkgs/applications/editors/vscode/extensions/vscode-utils.nix:106:12:

          105|     identifier = {
          106|       id = ext.vscodeExtUniqueId;
             |            ^
          107|       uuid = "";
(use '--show-trace' to show detailed location information)

Disabling the stylix vscode target prevents this from happening, but then vscode isn't themed.
I have a quick dirty fix in the PR. I want to clean it up before merging.

Fixed this in 936d97a0578f7e1a6e8ba6273654f5147b6a5886 as it's been broken for a while now. If you have more changes to add feel free to open another PR


Added convenience methods to ConstraintMakerRelatable for superview access

Added two new convenience methods to ConstraintMakerRelatable for superview access
greaterThanOrEqualToSuperview
lessThanOrEqualToSuperview
As mentioned in the issue #294

@txaiwieser thanks!


Mistake in CycleGAN tutorial about identity loss

In CycleGAN Tutorial.ipynb, the description and code about identity loss are wrong.
Current code is :
fake_a = gen_b2a(image_b)
fake_b = gen_a2b(image_a)
loss_identity = 0.5 * (F.l1_loss(fake_a, image_a) + F.l1_loss(fake_b, image_b))
The correct code should be:
loss_identity = 0.5 * (F.l1_loss(gen_b2a(image_a), image_a) + F.l1_loss(gen_a2b(image_b), image_b))

gen_a2b means it converts image_a to image_b. The same holds true for gen_b2a. The current definition seems to be consistent across all the losses used in the tutorial.

Dear avik-pal, I'm pretty sure it's a mistake about identity loss, you can refer to the source code delivered by the author of Cyclegan:
https://github.com/junyanz/pytorch-CycleGAN-and-pix2pix/blob/master/models/cycle_gan_model.py
and it's consistent with the explanation  of 'identity' in the Cyclegan paper.
Anyway, thanks for your excellent work and the torchgan framework really help me a lot:)


audits: Use new audit table for audit filter

The audit filter uses a subquery to the sv_audits table instead of the audits field of the interviews table.
The list of validation errors come from the sv_audits table as well.
Sequential tests are updated accordingly.

This PR adds a subquery to the validation list query. I expect performances will be degraded a bit in our production surveys, but by how much, I am not sure. @kaligrafy, if you have the opportunity, can you try this PR on the complete database and confirm that performances are not so bad, or au contraire, it's impossible to run?

I'll merge and when/if performance becomes a problem, we can revisit the queries.


bug: concept self-references

@benstear
Statement of problem
The OWLNETS-UMLS-GRAPH script introduces self-references to concepts, in which a concept has a reference to itself. This results in "rosettes" in the knowledge graph in response to queries such as
match (p1:Concept)-[]-(p2:Concept) where p1.CUI=p2.CUI return *--e.g.,

Cursory examination of the self-referential relationships indicate that they have SABs from ingested ontologies--i.e., they were created by the script.
According to @computationdoc and existing in-line documentation, the script had accounted for self-references in two places:

Immediately after reading in the edges file
When assigning a preferred CUI for each ingested node from the nodes file.

The logic in the second part of the script involved:

building for each node a list of CUIs (cuis) ordered by preference

nodeCUIs - the UMLS CUI for a cross-reference that existed previously in the ontology CSVs
CUI_CODEs - the UMLS CUI for the node that existed previously in the ontology CSVs
XrefCUIs - the non-UMLS CUIs for a cross-reference that existed previously in the ontology CSVs
base64cui - the base64 CUI (base64 encoding of the node id) for a node that does not currently exist in the ontology CSVs

Looping through the list as follows:
node_idCUIs = []
nmCUI = []
for index, rows in node_metadata.iterrows():
    addedone = False
    for x in rows.cuis:
        if ((x in node_idCUIs) | (addedone == True)):
            dummy = 0
        else:
            nmCUI.append(x)
            node_idCUIs.append(x)
            addedone = True
    # JAS 17 oct 2022
    # For the edge case in which multiple nodes have the same cross-reference AND the
    # cross-reference maps to a single CUI (the case for around 20 nodes in MP--e.g., all
    # those that share CL 0000959 as a cross-reference), only one of the nodes will be
    # assigned the CUI of the cross-reference. The rest of the nodes will lose the
    # cross-reference.
    # The following block keeps the nmCUI list the same size as node_metadata['CUI'].
    if addedone == False:
        nmCUI.append('')
        node_idCUIs.append('')

node_metadata['CUI'] = nmCUI

I added the code at the bottom of the block to account for errors that occurred when ingesting MP.
Because this loop can consume a prohibitively amount of resources for large sets of nodes, I attempted to replace the loop with a simpler call:
# Assign the CUI in order of precedence. This should be the first element in the list of cuis.
#node_metadata['CUI']=node_metadata['cuis'].str[0]

During discussion of this change, we discovered that self-references had been re-introduced. However, when I reverted to the original loop, the self-references were still in the graph. This means that either the original code did not address the issue of self-references or I re-introduced self-references as part of other changes to the script.
Possible solution options to explore
I still think that the simpler code is equivalent to the original loop because the calculated cuis field orders potential CUIs such that the first element in a node's list is the preferred CUI. However, I will verify this by comparing the intermediate DataFrames generated by each code.
I also think that it may be possible to eliminate self-references at the point of appending to CUI-CUIs.CSV.

Code to remove self-references
I added a line of code that removed the self-referenced edges before writing to CUI-CUIs.csv:
start of script
...
code to assign preferred CUI to node_metadata
code that merges edgelist to node_metadata
# -------------------------------------------------
# JAS 27 MAR 2023
# Remove "self-references"--i.e., assertions in which the CUIs for subject and object nodes are identical.

print('Removing self-reference edges...')
edgelist = edgelist[edgelist['CUI1']!=edgelist['CUI2']]

code that appends data from edgelist to CUI-CUIs.csv
...
end of script
I compared CUI-CUIs.csv before and after for the new code for the ingestion of UBERON. Self-refererences (where both of the CUIs were identical) were removed in the "after" version.

Analysis: Comparison of the CUI assignment algorithms
The script assigns a preferred CUI for each node in the node list.
Prior to the selection, the script derives a list of CUIs in order of preference:

nodeCUIs - the UMLS CUI for a cross-reference that existed previously in the ontology CSVs
CUI_CODEs - the UMLS CUIs for the node that existed previously in the ontology CSVs
XrefCUIs - the non-UMLS CUIs for a cross-reference that existed previously in the ontology CSVs
base64cui - the base64 CUI (base64 encoding of the node id) for a node that does not currently exist in the ontology CSVs

CUI_CODEs and XrefCUIs can be lists.
The important factor here is that more than one node can link to a CUI, especially as cross-references.
Original algorithm
The original algorithm matches each node to a unique CUI by keeping track of assignments:
# iterate to select one CUI from cuis in row order - we ensure each node_id has its own distinct CUI
# each node_id is assigned one CUI distinct from all others' CUIs to ensure no self-reference in edgelist

node_idCUIs = []
nmCUI = []
itest = 0
for index, rows in node_metadata.iterrows():
    addedone = False
    for x in rows.cuis:
        if ((x in node_idCUIs) | (addedone == True)):
            dummy = 0
        else:
            nmCUI.append(x)
            node_idCUIs.append(x)
            addedone = True
    if addedone == False:
        print('node:',rows.node_id,'has no available CUIs; assigning blank')
        nmCUI.append('')
        node_idCUIs.append('')

node_metadata['CUI'] = nmCUI

New algorithm
The new algorithm simply assigns the first CUI in the ordered list.
node_metadata['CUI']=node_metadata['cuis'].str[0]

Example of different results
When ingesting UBERON, the script calculates the following ordered list of CUIs:
['C0162412', 'C0024955', 'VUJFUk9OIDAwMDAzNzc=']
The list has two cross-referenced CUIs and the base64 CUI.
The first algorithm selects C0024955 because a node earlier in the list was assigned C0162412.
The second algorithm selects C0162412
First algorithm pros and cons
Pro: every node is assigned a unique CUI, including cross-references
Cons:

The algorithm is slow.
The algorithm requires considerable resources, especially for large sets of nodes. Ben has proposed an optimization that replaces the list with a set.
It is possible that a node will not have a CUI available. Cases were first identified in the ingestion of MP.

There are at least two edge cases in which a node will not be assigned a CUI:

Node 1 defines a cross-reference to Node 2 before Node2 defines its CUI AND Node2 only has a base64 CUI. Node2's CUI is used for Node1 and Node2 does not have a CUI
Node3 is defined with only a base64 CUI and Node 1 and Node 2 define a cross-reference to Node 3. Node 3 gets its base64 CUI and the two other nodes do not have a CUI available.

MP currently has around 20 CL nodes that are not assigned CUIs.
Originally, ingesting MP would cause the script to fail, because the node_metadata and nmCUI lists would be of different sizes. I added the code that assigned a blank CUI to nmCUI; however, this results in lines like this in CUI-CODEs.csv:
:START_ID,:END_ID
,CL 0002357
AZ presents another challenge: multiple nodes that have the same cross-reference. In this case, a node can lose its cross-reference.
For example, AZ:0000004, AZ:0000025, and AZ:0000080 share a cross-reference with CL:0000236, and CL:0000236 cross-references to 2 CUIs. The script calculates the following CUI lists and assignments:

AZ code
CUI list
CUI

AZ:0000004
['C1267837', 'C0004561', 'QVogMDAwMDAwNA==']
C1267837

AZ:0000025
['C0004561', 'C1267837', 'QVogMDAwMDAyNQ==']
C0004561

AZ:0000080
['QVogMDAwMDA4MA==', 'C1267837', 'C0004561']
QVogMDAwMDA4MA==

In other words, the third AZ node loses its cross-reference.

Verification of new functionality
I regenerated a knowledge graph using the script that had the new changes:

Code that removed self-references before writing to CUI-CUIs.csv.
The simpler algorithm for preferred CUI assignment.

As the images in the prior post show, the simpler algorithm corrects some issues related to CUI assignment.
The knowledge graph also no longer contains self-references.
I will refresh the HuBMAP and Data Distillery knowledge graphs with CSVs that are generated with the revisions to the script.

Completed additional regression testing, checking that nodes and edges established as expected.
Testing tracker


refactor: change sponsors styling

The different categories were not clear enough with the previous distribution
@resinas Does this look now good to you?

Yes, it looks good


[Enhancement] (nereids)implement DropWorkloadGroupCommand in nereids

What problem does this PR solve?
Issue Number: close #42621
Related PR: #xxx
Problem Summary:
implement DropWorkloadGroupCommand in nereids
Release note
None
Check List (For Author)

Test

[X] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run p0


[INLONG-6996][Dashboard][Manager] Add JSON format for InlongStream

Prepare a Pull Request
[INLONG-6996][Dashboard][Manager] Support JSON format

Fixes #6996

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[x] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@ZuoFengZhang I think we have to make sure the Sort could know this data type.
Otherwise, do you miss the TubeMQ Operator?

@featzhang please rebase from the master and fix the conflict.

@featzhang I think we have to make sure the Sort could know this data type. Otherwise, do you miss the TubeMQ Operator?

I have fixed it in another pr #6998


How to Turn Off Swagger-ui in Production?

https://www.baeldung.com/swagger-ui-turn-off-in-production
This is an example of spring. And I want to know how to turn off/hide swagger UI in production.
I'm using it this way, but it doesn't seem to be the proper way.
install(SwaggerUI) {
        ignoredRouteSelectors = ignoredRouteSelectors + RateLimitRouteSelector::class
        swagger {
            swaggerUrl = "swagger-ui"
            forwardRoot = true
        }
        info {
            title = "API"
            version = "latest"
            description = "API"
        }
        server {
            // Some features are disabled, but the UI is still visible.
            url = "a"
            description = "Development Server"
        }

Hi,
you can completly distable swagger by not installing it in "production mode":
if(!prodMode) {
    install(SwaggerUI) { /*...*/ }
}


cypress plugins: use cypress-log-to-output instead of custom script, remove task log

replacing local console-logger with https://github.com/flotwig/cypress-log-to-output
cy.task is unused, removing the log

oh this outputs a LOT
we may want to just remove the plugin if cypress 10 has a problem with our custom one


Portworx needs special characters for volume name (corresponds to meta-data)

Rancher versions:
rancher/server: 1.5.5
rancher/agent: 1.2.2
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
[root@my-centos-3 ~]# docker info
Containers: 9
Running: 9
Paused: 0
Stopped: 0
Images: 9
Server Version: 17.03.1-ce
Storage Driver: overlay
Backing Filesystem: extfs
Supports d_type: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 4ab9917febca54791c5f071a9d1f404867857fcc
runc version: 54296cf40ad8143b62dbcaa1d90e520a2136ddfe
init version: 949e6fa
Security Options:
seccomp
Profile: default
Kernel Version: 3.10.0-514.16.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.702 GiB
Name: my-centos-3
ID: AUID:D7C2:KA65:HEHD:WB5X:FWSR:YF5Z:O2PK:GFRI:AOFB:FCW4:SR2C
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: wilkins
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-ip6tables is disabled
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
3.10.0-514.16.1.el7.x86_64
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
DO
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Make sure to install Portworx from the Rancher Catalog
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
N/A
Steps to Reproduce:
"Add Container".   Use default ubuntu:14.04.3
"Volumes".   "Add Volume"
"Volume Driver" :  "pxd"
Portworx provides "storage on demand".   You don't have to create volumes in advance,  you can create them at run-time by providing all the appropriate volume meta-data  (see https://docs.portworx.com/scheduler/mesosphere-dcos/inline.html)
For example, the volume name can be :   "name=jvol,size=10,repl=3"
Results:
This should work fine.   But currently throws an error:
"Volume names can only contain letters, numbers, hyphen, underscore, and dot."
Please fix this --- atleast when "volume driver = pxd".
Thanks.

I think this should be fixed by #8646

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


Fix regress test line no failures

Change logs
These two tests(exttableext and task) will print the line no of functions in regress test. But it may get different results in different OS, like Ubuntu and Kylin.
So just remove the line-no output in tests. It has no effect on the correctness of the results.
Why are the changes needed?
Describe why the changes are necessary.
Does this PR introduce any user-facing change?
If yes, please clarify the previous behavior and the change this PR proposes.
How was this patch tested?
Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.
Contributor's Checklist
Here are some reminders and checklists before/when submitting your pull request, please check them:

[ ] Make sure your Pull Request has a clear title and commit message. You can take git-commit template as a reference.
[ ] Sign the Contributor License Agreement as prompted for your first-time contribution.
[ ] List your communication in the GitHub Issues or Discussions (if has or needed).
[ ] Document changes.
[ ] Add tests for the change
[ ] Pass make installcheck
[ ] Pass make -C src/test installcheck-cbdb-parallel
[ ] Feel free to @cloudberrydb/dev team for review and approval when your PR is ready🥳

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.roseduan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed type in documentation related to custom filters

Simple typo fix.

@envious Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@envious Thank you for signing the Contributor License Agreement!

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4f34127). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #737   +/-   ##
========================================
  Coverage          ?   69.2%
  Complexity        ?     968
========================================
  Files             ?     134
  Lines             ?    3923
  Branches          ?     276
========================================
  Hits              ?    2715
  Misses            ?    1061
  Partials          ?     147

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4f34127). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #737   +/-   ##
========================================
  Coverage          ?   69.2%
  Complexity        ?     968
========================================
  Files             ?     134
  Lines             ?    3923
  Branches          ?     276
========================================
  Hits              ?    2715
  Misses            ?    1061
  Partials          ?     147

Continue to review full report at Codecov.

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


Add docker and podman as known commands

Allow MiqEnvironment::Command.supports_command? to check for docker and podman binaries
Required for:

https://github.com/ManageIQ/manageiq-providers-workflows/pull/28

taken from https://github.com/ManageIQ/manageiq-providers-workflows/pull/28
Does it make sense to introduce a method over here that runs the logic in the right order and returns :podman or :docker or :kubectl or what ever?

huh. wonder why we are running AwesomeSpawn.run!(which, :params => [cmd]) instead of using run and looking at the return code.

I'm guessing so exceptions other than AwesomeSpawn::CommandResultError would still be raised but I don't know

@bdunne you good with this?

Backported to petrosian in commit ee84b8a9be2ebb99de3c563dec7d661515492329.
commit ee84b8a9be2ebb99de3c563dec7d661515492329
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Wed Jul 12 11:12:01 2023 -0400

    Merge pull request #22603 from agrare/add_podman_miq_environment_command

    Add docker and podman as known commands

    (cherry picked from commit 466314ac8161f79ec5d47adeebd3e77db3c70ca5)


improved docs

This is a bad-faith PR in order to get a free t-shirt for hacktoberfest. https://twitter.com/eed3si9n/status/1311617216748679168?s=19


Enabling preserveModules leads to empty sourcemaps with vue plugin

Describe the bug
I've set up a project in librarymode with rollupOptions.output.preserveModules set to true. After running vite build, the sourcemaps output to dist have an empty array for their sources property instead of the path to the source vue or ts file
Reproduction
https://stackblitz.com/edit/vitejs-vite-qspxpy?file=vite.config.ts
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^3.0.3 => 3.0.3
    vite: ^3.0.9 => 3.0.9

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

vite-vue-typescript-starter.js, vite-vue-typescript-starter3.js does not use any codes from the source code so I think it's correct to have a empty sources array.
vite-vue-typescript-starter2.js has ["../../../src/components/HelloWorld.vue"] because it uses code from that file.
Closing as the behavior is correct.

Ok it looks like there was a couple things throwing me:

The dist file names not matching the source. This is somewhat improved by specifying fileName: '[name]'
Even with that, src/components/HelloWorld.js isn't actually what maps to HelloWorld.vue - src/components/HelloWorld.vue_vue_type_script_setup_true_lang is (the script block is what actually gets mapped, not the component "root" - in practice, I guess this seems reasonable since sourcemaps are only useful for the code itself)
If src/index.ts only contains export { default as HelloWorld } from './components/HelloWorld.vue', it has no sourcemap content. However if I add something like export const x: number = 5;orconsole.log('x');, index.js.map` does have sourcemap content. @sapphi-red is that intentional? I can imagine this could be intended as there is ostensibly nothing to debug in the file, but want to verify.

Thanks for pointing me in the right direction!

If src/index.ts only contains export { default as HelloWorld } from './components/HelloWorld.vue', it has no sourcemap content. However if I add something like export const x: number = 5; or console.log('x');, index.js.map does have sourcemap content. @sapphi-red is that intentional? I can imagine this could be intended as there is ostensibly nothing to debug in the file, but want to verify.

I think yes.


[MINOR]: Refactor to increase readability

Which issue does this PR close?

N.A
Rationale for this change

This is a refactoring PR to increase code readability.
What changes are included in this PR?

can_skip_sort and check_alignment now return Result<Option<bool>> instead of Result<(bool, bool)>. Also since arrow now supports ! on SortOptions we remove reverse_sort_options method.
Are these changes tested?
Existing tests should work.

Are there any user-facing changes?
No.

Thanks @mustafasrepo  and @comphead


uart: native_posix: Check fcntl() return value

fcntl() may (very  unlikely) fail when setting the new pseudoterminal to non-blocking.
Let's check for this condition, handle it, and in the process silence a coverity issue.
Fixes #14396
Coverity CID: 195872
Signed-off-by: Alberto Escolar Piedras<EMAIL_ADDRESS>

Codecov Report

Merging #14478 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14478   +/-   ##
=======================================
  Coverage      52%      52%
=======================================
  Files         308      308
  Lines       45554    45554
  Branches    10545    10545
=======================================
  Hits        23691    23691
  Misses      17063    17063
  Partials     4800     4800

Continue to review full report at Codecov.

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


Local RAG from website with DBRX

@PrashantDixit0

Okay merging. Make readme changes if you want to.

Okay merging. Make readme changes if you want to.

I'll add it in examples section


duckplayer typed messages

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#941  👈
#846
main

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

Join @shakyShane and the rest of your teammates on  Graphite


Style not applied even if classname correctly created

I'm getting started with react-jss on a project and got stuck because there is no change on the UI even though the class is correctly added to the DOM.
import React from 'react';
import classNames from 'classnames';
import uuid from 'uuid/v4';
import PropTypes from 'prop-types';
import withStyles from 'react-jss'
import compose from 'recompose/compose';

import './CheckboxSwitch.css';

const styles = (theme) => ({
  checkboxSwitch: {
    label: {
      color: 'red',
      '&:active': {
        borderColor: theme.primaryColor,
        boxShadow: `0 0 5px ${theme.primaryColor}`,
      }
    }
  }
});

const CheckboxSwitch = (
  {
    // Only use default when the input is disabled, because we don't need it in that particular case
    disabled,
    name = disabled ? uuid() : null,
    id = name,
    value = false,
    labels,
    onChange = (a) => a,
    classes,
    ...rest,
  }
) => {
  return (
    <div className={classNames("checkbox-switch", classes.checkboxSwitch, {disabled: disabled})}>
      <input
        type={'checkbox'}
        id={id}
        name={name}
        checked={value}
        onChange={onChange}
        {...rest}
      />
      {
        labels.map((label, i) => {
          return (
            <label
              key={i}
              htmlFor={id}
              className={classNames(`label-${i}`, { active: label.value === value })}
              style={{...getLabelStyle(i, labels), ...label.style || {}}}
            >
              {label.text}
            </label>
          );
        })
      }
    </div>
  );
};
// ...

export default compose(
  withStyles(styles),
)(CheckboxSwitch);

Generated HTML:

The class CheckboxSwitch-checkboxSwitch-0-1-1  is correctly generated and added to the DOM. But it doesn't seem to be linked to any style. I think the generated CSS isn't injected in the page at all, only the class.
In the console, I have this Warning: Rule is not linked. Missing sheet option "link: true". for some odd reason. react-jss is supposed to work out of the box without any kind of configuration, so I wonder what I'm missing here.

Nevermind... I forgot to use the & for nested elements.