change TRS V1 response objects to correctly link to v1 response url

When given a GA4GHV1 api request, the tool would first be converted to a V2 object to work with the GA4GH (v2) api. This would also convert the url link to V2 in the response, but otherwise the code worked. Therefore this fix includes adding a change back to the v1 api response to correctly reflect v1 in urls of a ga4gh/v1 request. Also added unit tests.
A v1 api request now correctly returns a v1 url response.
A v2 api request still correctly returns a v2 url response.
Can test by running curls locally:
v1:
curl http://localhost:8080/api/ga4gh/v1/tools/quay.io%2Fpancancer%2Fpcawg-bwa-mem-workflow
v2:
curl http://localhost:8080/api/ga4gh/v2/tools/quay.io%2Fpancancer%2Fpcawg-bwa-mem-workflow

#1849

Restarted stalled build

When given a GA4GHV1 api request, the tool would first be converted to a V2 object to work with the GA4GH (v2) api. This would also convert the url link to V2 in the response, but otherwise the code worked. Therefore this fix includes adding a change back to the v1 api response to correctly reflect v1 in urls of a ga4gh/v1 request. Also added unit tests.<<

I would not say the tool would first be converted to a V2 object -- the word convert implies that it starts out as something else. The way the code works is it "creates" a V2 object, then converts it to V1. And the conversion from V2 to V1 was not handling the urls.

Coverage decreased (-0.0006%) to 77.158% when pulling 3f190b16514feb27bdcf93a60b02a2bedf1953b4 on feature/1849/TRS-version-response into d4e776fab05edd812d469219977e05ed72c6b29f on develop.


Missing commands.create.fail.no-ItemName when using the command

Getting this error in the chatbox when using the command to create a new shop

You can add this manually. Will fix in next release.

Like how? I don't know what this error is telling me. If I could fix this for my self I would.

Add this in locale.conf, if you are clever enough you can guess how.


i cant view the Remnux ethO Ip address and cant see /dev in remnux

Good day Admin,
After installing Remnux on my virtualbox, i cant see the Ip address of the ethO. Also, the remnux is not pinging google.com.
Also, i cant see /dev folder through the cli.
Kindly assist as i am setting up lab.
Thanks

Good day @koleademola, can you possibly provide your saltstack.log file, and let us know what version of Ubuntu you used to install REMnux, including any commands you provided with the CLI?

@koleademola, also, please take a look at the instructions on the REMnux documentation page that start with "If your REMnux virtual machine is unable to communicate over the network..." Let us know if those steps end up working for you.


This is to fix issue 27. Renamed the readme image file.

xqueuejob-controller.png renamed to multi-cluster-app-dispatcher-controller.png and also renamed the link to multi-cluster-app-dispatcher-controller.

@jenellemillison Could you resolve the conflicts (see above).  My guess is that the latest master has not been merged with your local repo.  (Please email me if you need some further assistance with fetching the upstream commits and merging them into your local repo.)  Thanks!

Closing this PR due to inactivity.


editor view's web view doesn't retrieve image file from Documents folder

Hi,
this might not be directly related to this framework, but maybe someone can give me a hint to the right direction to take for solving this problem.
When I try to insert an image like so:
toolbar.editor!.insertImage(fileURL.absoluteString, alt: "new image")
all works fine.
The image data is written to a file in the documents folder and then the refreshed html contains the image tag with the image path.
However, after I stop the simulator and restart it, the image can't be loaded into the html of the editor's web view properly.
See the screenshots below. The image tag in the html looks like this
<img src="file:///Users/Dominik/Library/Developer/CoreSimulator/Devices/BC19750C-FE45-4C7A-853E-1933DE0B5515/data/Containers/Data/Application/6F3EFBA9-E1CA-45F7-9567-D5E653972DB6/Documents/3E9BCE98-033D-4754-9125-683F704E2CEA.jpg" alt="image">
I have checked in the finder that the file is there... it has been saved indeed. doesn't matter if I convert the ui image to jpg or to png.
The weird thing is that when creating the the file, the web view has to update as well and retrieve the file from the documents folder. Otherwise it wouldn't show....  the html string is properly retrieved from core data, otherwise there would not be any text with html text attributes.

Thanks for looking into this!  The code that saves the image to the Documents folder of the app is very similar to yours (see below). I also tried using safari in the simulator to save some random image to the simulator camera roll rather than using one of the default images.  The result is the same...
In the logic of this app however, the user cannot just insert an image by adding a web URL. It all works through the camera or the image library.
After dismissing the edit view, the image and reopening the same item ( a note object in this case), the image appears as part of the Html. but I guess it is kept in cache so the simulator does not actually reload it from the documents folder...
When I stop the simulator and rebuild however, the web view in the editor apparently can't reload the image...
text information in the html-string can be retrieved without any problem though - it is just the image that is missing.
func imagePickerController(picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : AnyObject]) {

        if let pickedImage = info[UIImagePickerControllerOriginalImage] as? UIImage {

            if let imageData = pickedImage.resizeImage(200.0 , maxWidth:300.0, compression:0.7) {

                 let filename = NSUUID().UUIDString + ".jpg"

                let path = getDocumentsURL()

                let fileURL = path.URLByAppendingPathComponent(filename)

                print("file url: \(fileURL)")

                do {
                    try imageData.writeToURL(fileURL, options: NSDataWritingOptions.DataWritingAtomic)
                }
                catch let error as NSError  {

                    print(error)
                }

                self.keyboardToolbar.editor?.insertImage(fileURL.absoluteString, alt: "new image")

                dismissViewControllerAnimated(true, completion: {

                    print("file url abs string \(fileURL.absoluteString)")
                })

            }

    }

        else {
            dismissViewControllerAnimated(true, completion: nil)
    }

}

func getDocumentsURL() -> NSURL {
    let documentsURL = NSFileManager.defaultManager().URLsForDirectory(.DocumentDirectory, inDomains: .UserDomainMask)[0]
    return documentsURL
}

Ah, I see. You're actually saving the local image URL in the HTML, and then displaying it again the next time with the same HTML.
I played around with it again today. It seems like the Documents directory URL is changing each time you launch the app. For example, for one run, I got
file:/// ... /Application/6D820E65-30F6-480A-90FE-C31B0CDB6F65/Documents/pasta.jpg

But then the next time I got
file:/// ... /Application/6399E27A-E729-46D6-A523-45FD2D65CFBA/Documents/pasta.jpg

So it worked for my sample code, because I am always generating the documents directory code before inserting it into the web view.
For you, it's a little bit more difficult. I might suggest, when you save the HTML, you take the documents directory, and change it to some token, like [DOCUMENTS_DIRECTORY]/pasta.jpg . Then, when you load the content the next time, you pre-process the HTML, changing [DOCUMENTS_DIRECTORY] with whatever the real new documents directory is.
How does that sound, does that fit your situation?

I think I'm going to close this issue, because I don't think it's really a problem with the editor itself. I don't think local URLs like that were supported in the first place (or else why would they change the directory? lol)
If you think differentl or something, feel free to reopen the issue 👍

Thank you for figuring this out!  I have opened the sub folder in the core simulator path and indeed the folder name changes every time I do a new build.
This is actually an issue that only occurs in the simulator. I have just tested my app on a real device for the first time. When I add an image to a note, save it and then simply remove the app from memory and re-start it, the image is displayed as expected.  Thanks anyway and yes, you can close they issue!

@DominikButz  Can you get the resizeImage()  method  in there ?  I try to pick image from Photo Library, and then set image to "toolbar.editor", there I found you have worded that . So I copied your code ,then showed error in  resizeImage()  method . Need your help, Thx

You won't be able to just copy/paste it, but I'm pretty sure he's using an extension on UIImage that has a resizeImage method that looks something like what is in the SO answer

yes, that is correct, I used an extension to UIImage. there are several ways of resizing images. you can find a lot of solutions on numerous sites.
here is the one I used:
public extension UIImage {
public func resizeImage(maxHeight:CGFloat, maxWidth:CGFloat, compression:CGFloat)->NSData? {

    var actualHeight = self.size.height
    var actualWidth = self.size.width
    var imgRatio = actualWidth/actualHeight
    let maxRatio = maxWidth/maxHeight

    if actualHeight > maxHeight || actualWidth > maxWidth
    {
        if imgRatio < maxRatio
        {
            //adjust width according to maxHeight
            imgRatio = maxHeight / actualHeight
            actualWidth = imgRatio * actualWidth
            actualHeight = maxHeight
        }
        else if(imgRatio > maxRatio)
        {
            //adjust height according to maxWidth
            imgRatio = maxWidth / actualWidth;
            actualHeight = imgRatio * actualHeight;
            actualWidth = maxWidth;
        }
        else
        {
            actualHeight = maxHeight;
            actualWidth = maxWidth;
        }
    }

    let rect = CGRectMake(0, 0, actualWidth, actualHeight)
    UIGraphicsBeginImageContext(rect.size)
    self.drawInRect(rect)
    let image = UIGraphicsGetImageFromCurrentImageContext()
    let imageData = UIImageJPEGRepresentation(image, compression)
    UIGraphicsEndImageContext()

    return imageData! as NSData

}

}

So kind of you, I just wonder why I insert image does not work, I made the class  achieve UIImagePickerControllerDelegate , there in
func imagePickerController(picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : AnyObject])   like the DominikButz's code , then there no image insert, I try to print some information  , the console has no hint. I cannot find the result

Did you set you class to be the image picker's delegate? Is the delegate method actually being called?

yes, you need to create an instance variable like   let imagePicker = UIImagePickerController() first and in viewDidLoad :    imagePicker.delegate  = self
use print(...) and check in the console if the function  didFinishPickingMediaWithInfo   is called.


Sync with aosp/main

Synchronized to packages/modules/Bluetooth
commit e61bf0f15c413fb82376d6ff168c55598ee281ca
Changelist:

Return accurate phy types in extended advertising reports
Prevent abort on unknown APCF opcode

Fixes #24


highcharts map not loading

Fiddle: https://jsfiddle.net/v57ryz96/15/
I am trying to load map in my application. This is the sample version of it. Am i missing anything in the configuration ?
Expected Behaviour
Map should show up in fiddle
Current Behaviour
Map not visible
Your Environment

Version of highcharts-ng used:  1.0.1
Browser Name and version: Chrome
Operating System and version (desktop or mobile): Desktop

More suited for stack overflow.


Update download.rs Change BASE_URL to new address

Change BASE_URL to new address

Thank you for contributing!


Modify JavaFX UI according to initial prototype specifications

Good progress. Keep up the good work.

Good work.

New task card should look like this

Done.


Set up Moralis integration

Initially just adding pages that integrate with Moralis before adding the deep integration, minting & displaying NFTs in place of pins.
SanityCMS Config
It's likely that we may force an API to mint to Sanity and we display from Sanity for the time being -> before moving to another CMS more web3 friendly
Pins
The pins schemas will be updated to reflect the statistics that our in-game NFTs will have (similar to the Opensea stats)
Other relevant issues:
https://github.com/Signal-K/marketplace/issues/15
https://github.com/Signal-K/marketplace/issues/8

Notion: https://skinetics.notion.site/ShowM3-b4482c3a54d2497dbd0d4a1ff292ef68


Major history revision to the Halotools master branch

At the time of this Issue, the astropy/master branch has had a major history revision that impacts all open Halotools branches. At some point along ago, I merged in a couple of ~20Mb files and deleted them. This was back before Duncan and I wrote the pair-counter, and we were experimenting with the one written by Manodeep. This inadvertently resulted in an unwanted expansion of the git history file size. I rewrote history this morning with the following command:
git filter-branch --force --index-filter  'git rm --cached --ignore-unmatch halotools/mock_observables/sinha_pairs/*.ff' --prune-empty --tag-name-filter cat -- --all
This fix was found by @eteq on https://help.github.com/articles/remove-sensitive-data
I then pushed directly to astropy/master. This decreased the size of the Halotools repo from 40Mb down to 7Mb.
Thus all open branches need to be rebased on the new master. Though the simplest thing by far is for anyone with a copy of Halotools to simply re-clone master from scratch. There is no change to functionality whatsoever with this patch, so re-cloning is only necessary if you want your working copy to submit a PR to astropy/master. This is unfortunate, but needed to be done and was better addressed now than after the beta-release (or worse, after the v1 release).
CC @duncandc , @surhudm , @yyao , @BenediktDiemer

I'm not sure whether this would be overkill, but actually the safest thing would be to obliterate your copy on github entirely, and re-fork from the recently-updated master.

For any looking here, another approach to deal with your github forks (not your local copy, which is distinct from the github version) would be to just not have your fork be tied to master at all.  I generally prefer this approach anyway, because then you won't get confused between your master and the main repo's master.
The easiest way to do this is:

make a new branch on your github repo called whatever you want (I use "staging").  You can do this either by pushing from a local branch to github, or via the github "new branch" web site interface.
change the "default branch" to your new one ("staging") on the settings page of your github fork
delete the master branch from your github fork (make sure you do this on your fork, not the main repo, if you have access to it...)


"simple" example appears to not work: always "empty data"

Create a Trino database with some data in it, and try to select it out with get_all.


Non async version

Description
My app supports back to iOS 11 so I am not using async-await yet. Is it possible to use this package without async await?

I have the exact same question

Hey @paulsUsername, @DominiCanes  👋🏽 at the moment, typesense-swift only supports iOS 13 and above with swift modern concurrency, so there is technically no way to use this package without async-await 😢 We might bring callback oriented architecture to the framework but not in the near-future.
You can checkout the Typesense Documentation for more information on how Typesense generally works.


Write some docs about stubbing

How to work with sinon and @skpm/test-runner?

@FRSgit (can't seem to be able to assign you to the issue 🤔)

@mathieudutour  Hm, thought it would be possible. Nevertheless, mentioning was well enough 😄

Now it works ¯_(ツ)_/¯


Not Mobile Site Friendly

Describe the bug
When prompting the AI to make the site "Mobile-Friendly" its begins the prompt. Afterwards the preview shows only code on the screen and not the actual site.

Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-aktozqvw
Steps to reproduce

Prompt Ai to make already established website, "Mobile-Friendly".

Expected behavior
I expected the site to finetune and fix the broken code.
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 55827300,
  "usedJSHeapSize": 50165592,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Goldsilk17
Chat ID = ecb41d9d7d46

Additional context
No response

Could you make https://bolt.new/~/sb1-aktozqvw public so others can access it, in an attempt to debug the error.


clientv3/balancer: update eps if pinAddr is not included in updateAddrs

FIXES #7392

All fixed. PTAL

All fixed. PTAL

All fixed. PTAL

All fixed. PTAL

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a2a6b69). Click here to learn what that means.
The diff coverage is 89.18%.

@@            Coverage Diff            @@
##             master    #7397   +/-   ##
=========================================
  Coverage          ?   70.26%
=========================================
  Files             ?      320
  Lines             ?    26206
  Branches          ?        0
=========================================
  Hits              ?    18414
  Misses            ?     6340
  Partials          ?     1452

Impacted Files
Coverage Δ

clientv3/balancer.go
94.18% <89.18%> (ø)

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 a2a6b69...a23609e. Read the comment docs.


Add meson build file

Dear,
In order to be able to build with meson. I added a meson.build file
$ meson build
$ ninja -C build
$ ninja install

Also, we need to resolve this issue with the compiler and cas. Not sure what needs to happen there, it seems like a compiler bug.

Also, we need to resolve this issue with the compiler and cas. Not sure what needs to happen there, it seems like a compiler bug.

I think too is a compiler bus as I built with meson the latest release with success

One final request, is there a way to ensure the meson build is tested during CI?

One final request, is there a way to ensure the meson build is tested during CI?
I will take a look, I agree with you is better with test.

So I updated CI in order to add meson build part of test.
Moreover I extended both compiler compiler and os use to ldc and osx
Only meson with ldc build actually.
It is strange as I have do any change on dub and io source code.

I think too page deployement is broken froma log time as travis documentation tell to set the secret variable from the settings page of io repository and use this code:
deploy:
  provider: pages
  skip_cleanup: true
  github_token: $GITHUB_TOKEN  # Set in the settings page of your repository, as a secure variable
  keep_history: true
  on:
    branch: master

To meet the travis documentation we have to

set the secrete variable
remove before_deploy section
update the deploy section accordingly to the documentation

Looking at the meson build output, it doesn't look like it's running unittests. Is this expected? If it can run the unit tests, that would be ideal.

I think the meson test is failing because it's running the test application in the build directory, not in the root directory. Is there a way to fix it?

I think the meson test is failing because it's running the test application in the build directory, not in the root directory. Is there a way to fix it?

Thanks @schveiguy I will take a look

OK, I don't know how long it will be for LDC/DMD to get these bugs fixed. If you want to change the meson with ldc to not required, that's probably good enough for now, we can squash-merge.

@bioinfornatics I think we fixed the issue for LDC in #30. Can you add back in the ldc test with 1.18.0? I want to see if it works.

Good job  that works 😉

Alright excellent! I think when ldc 1.19.0 comes out, we may have to revisit, but it's green right now. Thanks @kinke and @bioinfornatics.


fix: skip annotations if not in fields

Porting https://github.com/docarray/docarray/pull/1777
@JoanFM

Hey @NarekA here we need you to signoff ur commits. Could you do this please?

@JoanFM I can't figure out signing github commits, why don't you just merge your branch. I'm giving up on this one.


Add deployment annotations

[x] Chart version not bumped (the versions are all bumped and released at the same time)
[x] README.md updated with any new values or changes
[x] Updated template tests in ${CHART}/tests/*.py
[ ] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

I've signed the agreement.

Jenkins test this please

@clyfish Tests all look good.
However it looks like there's now a merge conflict. Would you mind merging master and updating the PR?

@fatmcgav done

Thanks @clyfish
Jenkins test this please


Changed sf::String::InvalidPos from 'inline const' to constexpr

Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[ ] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[ ] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
While looking through all of the header files for an unrelated issue I noticed this and wanted to update it for both modernization reasons as well as clarity. (It's a very trivial change)
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Describe how to best test these changes. Please provide a minimal, complete and verifiable example if possible, you can use the follow template as a start:
#include <SFML/System.hpp>
#include <iostream>
int main() try
{
    sf::String str{};
    if(str.find("empty") == sf::String::InvalidPos) throw std::runtime_error("this should not throw");
    return 0;

}
catch(std::runtime_error const & e){
   std::cerr << e.what();
}

SFMLBuildMaster: Build this please.


Installation

There will be 5 ways to install and deploy the Verto trading post.

With Node.js

[x] NPM
[x] Bundled verto.js
[x] Git

Standalone

[x] Pre-built binaries
[x] Installer

Just waiting for release :wink:


ENH: Refactor Rocket.addFins to avoid breaking changes

Pull request type
Please check the type of change your PR introduces:

[ ] Code base additions (bugfix, features)
[x] Code maintenance (refactoring, formatting, renaming, tests)
[ ] ReadMe, Docs and GitHub maintenance
[ ] Other (please describe):

Pull request checklist
Please check if your PR fulfills the following requirements, depending on the type of PR:

Code base maintenance (refactoring, formatting, renaming):

[x] Docs have been reviewed and added / updated if needed
[x] Lint (black rocketpy) has passed locally and any fixes were made
[ ] All tests (pytest --runslow) have passed locally

What is the current behavior?

The proposed implementation of Rocket.addFins (#172 ) will cause breaking changes.
What is the new behavior?

The new proposed implementation creates two methods: Rocket.addTrapezoidalFins and Rocket.addEllipticalFins to avoid breaking changes. Furthermore, it marks Rocket.addFins as a future deprecation to be removed in version 2.0.0 (to be discussed).
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information
Work to fix notebooks and docs that mention addFins must be worked on to reflect theses changes.

Hey! This seems to fix the breaking change, I just fixed a few small bugs regarding the radius parameter
Just wondering about some repeated code on both new functions, like the functions def beta(mach) and def finNumCorrection(n) or the cl calculations. Wouldn't it be better split up de addFins functions into even more functions? Or create a class maybe?

@MateusStano, I completely agree that it is about time to create a class to manage fins and perhaps even the remaining aerodynamic surface.
How about something like this in the future?
from rocketpy import Rocket
from rocketpy.aerodynamics import BoatTail, TrapezoidalFins, ElipticalFins, NoseCone

TestRocket = Rocket(...)
TestFins = TrapezoidalFins(...)

TestRocket.addAerodynamicSurface(TestFins)

With something like this, both TrapezoidalFins and ElipticalFins could be a class inherited from the same Fins class, which could contain the repeated code. Furthermore, the base Fins class could isnheret from a even more basic and abstract AerodynamicSurface class.
What do you think?
The only issue is that this causes breaking changes. So I would advise releasing the ElipticalFins implementation first and then create theses changes.

I really like the approach for the classes. I think it would make the code simpler to understand and easier to expand. Maybe instead of using inheriting abstract calss would fit better for teh Aero Surfaces class for example
I guess the best way then is to merge this PR and leave the Aerodynamic Surface and Fin class to some other time


Create a scale method

could be inplace or as a new array
I think we should not throw an error
function scale(array, min, max) {

    var minMax=ML.Stat.array.minMax(array);
if (minMax.max===minMax.min) throw new Error('normalize: max must be different from in');
    var slope=(max-min)/(minMax.max-minMax.min);
    console.log(slope);
    for (var i=0; i<array.length; i++) {
        array[i]=(array[i]-minMax.min)*slope+min;
    }
    console.log(array);
    return array;
}

i zlthink it should be scale(array, options). Min and max can have default values

min:0, max:1 ?

No default values. If only min is defined the array will be multiplied simply by min/Stat.min(input) and if only max is specified then  max/Stat.max(input). If both are specified then:
var minMax = Stat.minMax(input);
var factor = (max - min)/(minMax.max-minMax.min);
for(var i=0;i< y.length;i++){
    y[i]=(input[i]-minMax.min)*factor+min;
}

The method also receive the inplace option, by default false

Don't know which behaviour is the more logical. I was expecting:
You specify nothing : min = Stat.min, max=Stat.max -> we do nothing
You specify min : min = newMin, max=Stat.max
You specify max : min = Stat.min, max=newMax
You specify min and max : min = newMin, max=newMax
But I don't know what is the "usual" behaviour in some other software

about the option, should we use camelCase ?
=> inPlace instead of inplace

inPlace change was made. About the other problem, I would prefer that if you don't specify min or max the 0 be maintained. So, it is about what you prefer to keep, your baseline or your unspecified value.

OK I guess both are useful ... What about other programs ? They allow both ? what could be the option name ?


Implemented a mtg search function as well, together with fixing concerns

A lot of duplicate code though, but I don't see a way to do it better.

To remove the duplicate code, alias all four commands to a single function with extra arguments like this: function (target, room, user, connection, cmd)
cmd will then let you check the exact command used (typing /ygo Kuriboh has cmd equal 'ygo'), and direct the HTTP Request to the appropriate Wikia page.

You don't actually fix a lot of the problems. :|

I'm probably going to make it all neat and nice whenever I can, just don't have the time nor knowledge to this this correctly.

If you don't have the time nor knowledge to do it correctly, the correct answer seems to be to delete your plugin.

Says the person that merged it.

Yeah sorry I shouldn't attempt sarcasm. Well anyway I see it can be done better, and would have been fine if you didn't merge it until it was better. I would probably have time this weekend to do that, if that's still wanted.

It's not good, but it's better than what was there before, which is why I merged it.

Yeah I'll make another PR this weekend then. Do notify me of all the concerns you have with it and I'll try my best to fix everything.


I can see the Title and Github Link on Detail page

Description
As a Concourse User
I can see the title of the resource and the link to the resource depository
So That I can see clearly see what resource the details page is associated with and easily get to the resource depository if I want
Acceptance Criteria
As OSS Concourse User
When  I go to the detail page on duty free
I can see

the resource name as title
github icon linking to github resource repository

TITLE = Resource Source on Github

Accepted by @Pivotal-Kevin-Curley. Behaviour is as per acceptance criteria.


[CB-15333] Add SSB-SSE-API

See detailed description in the commit message.

please resolve conflicts

@zsedert could you resolve these conflicts?


one line output is taller than one line input

i am not 100% sure this is a problem but i don't understand why a one-line output cell is taller than the one-line input side of the same cell.  above is what i get and if you copy and paste, i see a six pixel difference:

btw those cells were JS.  looks like HTML cells are behaving a bit differently I have to look into that, prob an unrelated issue.

I agree for a single line "text" output they should be the same height, obviously as soon as there is any formatting it is going to be quite hard to ensure that they are the same height, but for "raw text" I think we can make a single line code output line up.


Translate.tla counterexample using modelator tla tla-trace-to-json-trace <filename> results in parsing error.

Bug:
Running ./modelator tla tla-trace-to-json-trace examples/counterexample1.tla does not work. Results in parsing error.
Content of counterexample.tla is at the bottom of this issue.
Expected:
JSON output for translated tla.

It will parse following successfully,
f = (p :> "-" @@ n :> "+")

but not this
f = (p :> "-" @@ (n :> "+"))

also not this
f = p :> "-" @@ n :> "+"

@danwt, it fails on the above counterexample.tla because bank variable is set with a nested function.

Sorry for the weekend work. I just had to finish the nested parser.
Anyway, this does not resolve the issue. Now it complains, it didn't parse the whole state string.
thread 'main' panicked at '[modelator] full TLA state should have been parsed', modelator/src/module/tla/json/mod.rs:10:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Looks like the states in TlaTrace are not correctly generated. Specifically impl FromStr for TlaTrace is buggy.
https://github.com/informalsystems/modelator/blob/b45faeb866a01e7ff714bb4f32904d8389b69f2b/modelator/src/artifact/tla_trace.rs#L26-L30
It doesn't handle comments, it seems.

After a quick look, it seems,

It doesn't handle comments.
Relies on next State{} == keyword to stop adding more lines to stop for current state.
This allows the violated invariant to be added to the last state, causing Error.

@danwt I pushed a new TLA file parser. This should fix this issue :)

Looks like, the mentioned TLA file is not a proper TLA file. It should be the trace generated from modelator [tlc|apalache] test.
@andrey-kuprianov what should I do here? rework the parser (very little work) to support both types of TLA files (proper syntax and trace) or generate the TLA trace with proper TLA syntax?

I went with option (2). TLA trace from modelator [checker] test will output a valid TLA file, which can be used for modelator tla tla-trace-to-json-trace.


Add a page with information about the project

This issue was more relevant when the gremlint library and the gremlint.com website were in the same repo. I moved it over to the gremlint.com repo: https://github.com/OyvindSabo/gremlint.com/issues/3


Fix qubit filtering in BackendV2Converter with add_delay

Summary
This commit fixes an issue with the filter_faulty and add_delay flag were both set on the BackendV2Converter and convert_to_target flag. In these cases the function would incorrectly add delay operations to the faulty qubits even though they should have no operations preset on them. This commit fixes the oversight to ensure that no operations are present on a qubit if it's listed as non-operational in an input BackendProperties.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
25 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.04%) to 85.9%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/vf2_layout.rs
1
86.44%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
1
95.14%

crates/accelerate/src/sabre_swap/mod.rs
2
99.53%

crates/qasm2/src/lex.rs
3
91.39%

crates/qasm2/src/parse.rs
18
96.65%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
70990

Relevant Lines:
82643

💛  - Coveralls


Log desired LRP validation errors

[x] Read the Contributing document.

Summary
Improves the app logging when the internal routes limitation per LRP is reached.
Issue: https://github.com/cloudfoundry/diego-release/issues/962
Backward Compatibility
Breaking Change? No

Pinging @ameowlia for approval since she has the same amount of context as I do regarding the changes.

@ameowlia I think the changes are ready for final review and merge.

Hi @ameowlia,
Can we proceed with the merge as feedback requested is already applied?


new stat: # of data: URIs

This would be hard because we'd also have to look in stylesheets. Closing.


Lack of Documentation for Translators onboarding

We need to ease onboarding of new translators:

Why translation is important?
How to get started (account creation)

Create an account and then get approved by a Wagtail team member? Does this happen seamlessly or do we need to ping someone?
What are the steps required to be a reviewer and not just a translator?

How to navigate around Transifex

A link to this Wagtail Space Lightning Talk would be a good start, but we probably want a text version of that and/or a longer format video

Invite translator to join the corresponding Slack channel
How to test the translations locally

There is the Managing Wagtail Translations wiki page, but it’s aimed at contributors.
It ties back to the lack of context (see #7116), but it can be hard to know if the translation is correct without being able to check it. Otherwise, we end up waiting for the next version to be released and then wait again for the following version to fix any issues.

Can each language have its own page to clarify some common decisions?

Transifex' Glossary is a good start, but:

it isn't really the place to document why a specific decision has been made
it has to be tied to a specific term, but some decisions might concern style or grammar in general

Would we require such pages to be written in English?

Being able to use the target would be simpler for translators, but it would make moderation harder.

Should it live with the rest of the documentation? Doing so would mean any change has to go through a PR.

Hi @thibaudcolas I will like to work on this

@banjo-tech have you used Transifex? It might be good to pick this up if you have had some, even minimal, experience with writing translations.
If so, I recommend the first step would be to write a bit of a draft docs section as a comment, including where you think this should go in the docs.
Then after a bit of feedback you can prepare a PR
Also - it would be good to have a read of the Related discussion about how to make sure Wagtail works better across the board for non-english speakers.
https://github.com/wagtail/wagtail/discussions/8017

Bit of confusion about the 'structure' of Transifex - could be good to pull out Matt's response to help with this onboarding page.
https://github.com/wagtail/wagtail/issues/9378

@lb- sorry for the late reply. I have not used transifex. I have however created an account .What I intend to do is make a translation using transifex and then document the process if that is a good idea. I would also pull out Matt's response and go through it. Thank you for the suggestion. When I create a draft I will update this comment before making a pull request

That's a great way to go about it. It's also ok to solve 'some' of this PR with a start in the docs.

Alright I will thank you

wagtal onbording translators.docx
@lb-  I have attach a draft in this comment, please go through it and let me know what you think.

@banjo-tech I'm not sure on the approval stuff, when I joined (a while ago) I didn't see that requirement.
Maybe join the #translators chat in Slack and ask there. https://wagtailcms.slack.com/archives/CTKN6UXKN
Also, any chance you can upload a txt file / markdown file instead?

translator onboarding .md
Thank you @ lb the slack channel was a useful suggestion.
Please review this draft

A few points if feedback on the draft doc

we use en-GB spelling so organisation not organization etc (basically remove the z's and replace with s)
I get the idea of the intro - we appreciate your interest - but I think let's focus this intro on a high level overview. Something like 'Wagtail uses transifex to translate the content for the admin interface. Our goal is ensure that Wagtail can be used by those that speak many different languages. Translation of admin content is a great way to contribute without needing to know how to write code' .
it would be good to add an overview of the translation workflow. Basically each new release we have a two week ish release candidate period. At the start of that period the translations are put into transifex, we notify the channel in slack that new translations are needed. Those are then updated each RC until the final release. If translations reach a threshold of X% (I think it's 90) we communicate that this is an officially supported translation in the docs.

To help with your PR - it would be good for you to have a look at how we write docs.

Read our documentation on writing Documentation - for example try to avoid links with the label 'here'. https://docs.wagtail.org/en/stable/contributing/documentation_guidelines.html
https://docs.wagtail.org/en/stable/contributing/documentation_modes.html
https://docs.wagtail.org/en/stable/contributing/general_guidelines.html

As for docs. We should ensure there is a link to this ne page in this section https://docs.wagtail.org/en/stable/contributing/index.html#translations
And make this new page a sub-page if the contributing section.
Also it might be good to take some inspiration from Django's docs for this kind of thing. https://docs.djangoproject.com/en/dev/internals/contributing/localizing/
Thanks for putting your ideas out there - this is an awesome start. Looking forward to the Pull Request.

Thank you for the thorough review. I will make the necessary corrections and go through the documents

@lb-  does the RC in the test above mean required contribution?

@banjo-tech RC means release candidate.

@lb can you please explain this sentence "Those are then updated each RC until the final release" .
I don't quiet get the message you are trying to pass across

@banjo-tech have a read of how the Django docs explains it - our process is essentially the same.
But basically when we have a release candidate - we will pull in the new translations from transifex into the code. Then we do that again for the final official release.
Also have a look at the migration of translations instructions.
https://github.com/wagtail/wagtail/wiki/Managing-Wagtail-translations
Note. This is for developers but this is the process that happens each RC

Thank you so much @lb- I think I am done with the document now.
internal link to another document - I want to be sure if this is how I'd link the new document to the contributing  to wagtail page

@banjo-tech I reckon the next step now is to open a PR with your proposed new page.

@lb- I will like to open a pull request but I'm unable to link the new file to the index page.

Can you reach out on Slack in the Outreachy channel. We can help more there. Just explain what you're doing and the error you have encountered

Okay thank you

@loicteixeira I will close this as fixed once https://github.com/wagtail/wagtail/pull/9431 is merged as I think it covers most of the original intent of this issue.
Some outstanding items though;

How to test the translations locally
I have added this to the https://github.com/wagtail/wagtail/issues/7116 issue.

Can each language have its own page to clarify some common decisions?

Not sure if we should split this to its own issue, I think it needs a bit of a discussion though, maybe it is best to be added to https://github.com/wagtail/wagtail/discussions/8017 ?

Hi @lb- Thank you for the review on the previous PR. I will like to work on the remaining part of this issue but I guess I will have to wait on a response from @loicteixeira or could I go ahead without creating an issue first?

Thank you both for moving this along 🙏

How to test the translations locally
I have added this to the https://github.com/wagtail/wagtail/issues/7116 issue.

Yeah, it makes sense to move it to the other issue. I'll also comment there though.

I will like to work on the remaining part of this issue but I guess I will have to wait on a response from @loicteixeira or could I go ahead without creating an issue first?

Sorry I haven't been more available, but don't let me block you.
That being said, I'm not sure what needs a new issue? @lb- added the remaining points of this issue to https://github.com/wagtail/wagtail/issues/7116, but if this is something else, please create an issue (or comment on https://github.com/wagtail/wagtail/issues/7116 if that's related and we will update the top comment accordingly).

Can each language have its own page to clarify some common decisions?
Not sure if we should split this to its own issue, I think it needs a bit of a discussion though, maybe it is best to be added to https://github.com/wagtail/wagtail/discussions/8017 ?

I'm torn on this one. On one hand, it should be with the documentation (separate page though) so anybody can update it, but I'm not sure all translators would want to/be able to go through the whole clone/edit/commit/pull-request process. On the other hand, GitHub discussions would be a nice free form thing, but it's hard to "pin" decisions, and would probably require to update the top level comment, which can only be done by the author of that comment. So yeah, it probably makes sense to discuss this futher on https://github.com/wagtail/wagtail/discussions/8017.

Sounds good @loicteixeira  - yeah the 'glossary' thing is good but agree it needs to be translated, it may also provide a clear 'home' for naming conflicts like https://github.com/wagtail/wagtail/issues/9201
But there is also a balance between this and expecting translators to have some knowledge of how Wagtail works.
@banjo-tech based on this discussion so far, I do not think there is anything else to be done under this issue. But I will comment on https://github.com/wagtail/wagtail/issues/7116

Thank you @lb- and @loicteixeira I will read through #7116 and see if the issue is something I can work on


Install Bootstrap 4

To facilitate our move to a responsive design, we will be using Bootstrap 4. We need to install Bootstrap 4 and its Angular implementation ng-bootstrap.

Fixed as of #43


When restore password using mnemonic words, presss on "PASTE", the app crashes.

When restore password using mnemonic words, presss on "PASTE", the app crashes.

solove at here


Add Josh as an approver

Description
Add Josh as an approver
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

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

Project coverage is 94.50%. Comparing base (b6cc49b) to head (536198d).
Report is 65 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1773      +/-   ##
==========================================
- Coverage   94.90%   94.50%   -0.40%
==========================================
  Files          60       61       +1
  Lines        2689     2714      +25
==========================================
+ Hits         2552     2565      +13
- Misses        137      149      +12

/lgtm
/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"


[BUG] Hauler fileserve fails on large files due to hardcoded 15 second timeout.

Environmental Info:

Linux REDACTED 4.4.302+ #69057 SMP Fri Jan 12 17:02:27 CST 2024 x86_64 GNU/Linux synology_v1000_1821+

Hauler Version:

GitVersion:    1.0.3
GitCommit:     f7c24f6
GitTreeState:  clean
BuildDate:     2024-04-24T16:50:33Z
GoVersion:     go1.21.9
Compiler:      gc
Platform:      linux/amd64

Describe the Bug:

When trying to pull a large file from the fileserver, the hardcoded timeout can affect the ability to pull that artifact.

See: https://github.com/rancherfederal/hauler/blob/main/internal/server/file.go#L36-L37
Steps to Reproduce:

For me, I was utilizing hauler store serve fileserver to serve the Harvester ISO for PXE booting (5.6Gi in size).
When pointing Harvester PXE boot to Hauler, the download of the ISO continuously fails at 15 seconds due to the timeout.

Expected Behavior:

One of two options:

The timeout is configurable.
The timeout is set to a higher amount than 15 seconds. Perhaps 60 or 120.

Actual Behavior:

See above.

Additional Context:

N/A

Just wanted to chime in here, I was really curious about this issue as I also hit it somewhat recently so just wanted to provide a second round of proof.
Environmental Info:
Linux e851a7a7b747 6.4.16-linuxkit #1 SMP PREEMPT Thu Nov 16 10:49:20 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

Hauler Version:
GitVersion:    1.0.3
GitCommit:     f7c24f6
GitTreeState:  clean
BuildDate:     2024-04-24T16:50:33Z
GoVersion:     go1.21.9
Compiler:      gc
Platform:      linux/amd64
Describe the Bug:
When trying to pull a large file from the fileserver, the hardcoded timeout can affect the ability to pull that artifact.
See: https://github.com/rancherfederal/hauler/blob/main/internal/server/file.go#L36-L37
Steps to Reproduce:

Generated a 10Gb file:

dd if=/dev/urandom of=random.txt count=1024 bs=10M
tar -cf archive.tar random.txt

Added it to the store:

root@e851a7a7b747:/# ./hauler store add file archive.tar
2024-05-26 23:04:09 INF adding 'file' [archive.tar] to the store as [hauler/archive.tar:latest]

root@e851a7a7b747:/# ./hauler store info
+-----------------------------------------------+------+----------+----------+---------+
| REFERENCE                                     | TYPE | PLATFORM | # LAYERS | SIZE    |
+-----------------------------------------------+------+----------+----------+---------+
| hauler/archive.tar:latest                     | file | -        |        1 | 10.7 GB |
| hauler/hauler_1.0.3_linux_amd64.tar.gz:latest | file | -        |        1 | 63.3 MB |
+-----------------------------------------------+------+----------+----------+---------+
|                                                                    TOTAL   | 10.8 GB |
+-----------------------------------------------+------+----------+----------+---------+

Waited....
Attempted to curl and would you look and I also saw the connection close at 15 seconds.

PS C:\Users\user> curl.exe --verbose *.*.*.*:8080/archive.tar -o archive.tar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying *.*.*.*:8080...
* Connected to *.*.*.* (*.*.*.*) port 8080
> GET /archive.tar HTTP/1.1
> Host: *.*.*.*:8080
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Accept-Ranges: bytes
< Content-Length:<PHONE_NUMBER>0
< Content-Type: application/octet-stream
< Last-Modified: Sun, 26 May 2024 23:15:10 GMT
< Date: Sun, 26 May 2024 23:15:52 GMT
<
{ [512 bytes data]
  4 10.0G    4  431M    0     0  29.3M      0  0:05:48  0:00:14  0:05:34 30.1M* transfer closed with<PHONE_NUMBER>8 bytes remaining to read
  4 10.0G    4  445M    0     0  29.1M      0  0:05:50  0:00:15  0:05:35 29.3M
* Closing connection
curl: (18) transfer closed with<PHONE_NUMBER>8 bytes remaining to read

Expected Behavior:
A configurable timeout flag for fileservers.
Additional Context:

I did attempt to try this with small and large files using curls --limit-rate flag but that keeps the connection alive and refreshed.

PS C:\Users\user> curl.exe *.*.*.*:8080/archive.tar -o archive.tar --limit-rate 1k
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying *.*.*.*:8080...
  0 10.0G    0 36352    0     0   1022      0 121d 14h  0:00:35 121d 14h  1019

Opened a PR to adjust this: #247
As well as a documentation PR: 19

Closing as resolved per PR https://github.com/rancherfederal/hauler/pull/247!


Added files helper function.

Be aware that on lines 802 and 805, the $path can still being null

Thanks for you @JuanDMeGon , I fixed it now.

You know what? I think that I get confused on my first comment. Basically, if the path is null it will always return app('files').
It seems like your original code was correct and the latest commit does not need the additional condition, because ! is_null($path) is going to be true always after the first if statement.
Very sorry for the confusion here.

@JuanDMeGon No problem, I reverted it now.

@taylorotwell why?  😥


feat: introduce small object disk cache

What's changed and what's your intention?

Please explain IN DETAIL what the changes are in this PR and why they are needed. :D

Checklist

[ ] I have written the necessary rustdoc comments
[ ] I have added the necessary unit tests and integration tests
[ ] I have passed make all (or make fast instead if the old tests are not modified) in my local environment.

Related issues or PRs (optional)

Duplicated with #658


Use 24 hour format for displaying dates

Instead of Jul 24, 2023, 4:30:06 PM it should be Jul 24, 2023, 16:30:06
24hour format should also be used in date picker on market creation

Instead of Jul 24, 2023, 4:30:06 PM it should be Jul 24, 2023, 16:30:06
24hour format should also be used in date picker on market creation

Is there anywhere that's not doing this currently?


Time keeps on slipping

The perfect excuse to troll both craig and alex.

:+1: trolls be :trollface:


Loosen test thresholds

Fixes https://github.com/JuliaAI/MLJXGBoostInterface.jl/issues/43.
@ExpandingMan was there any reason for choosing exactly these thresholds that I now have changed?
Thanks @tylerjthomas9 for your comment (https://github.com/JuliaAI/MLJXGBoostInterface.jl/issues/43#issuecomment-1751513611). It's very useful to know that

I think the change to hist as the default tree method in libxgboost v2 is causing the test failures. Changing the default tree_method="exact" causes all the tests to pass.

So as long as @ExpandingMan does not have any objections, I think we can move forward with just loosening the thresholds. I find it always a bit strange with statistical models that non-breaking changes can affect model performance a lot, but I also wouldn't want to force model authors to call every little model change breaking.

I find it a little bit worrying that these had to be loosened, I wouldn't have put them in if I did not feel they were somewhat generous, however I very much doubt that I had any rigorous justification for the bounds I placed, so I'm perfectly willing to go along with others judgement on this.

In some sense, we should not feel it is our responsibility to even carry out these tests, since we are only providing an interface. On those grounds I have not objection to loosing the bounds of what is ostensibly just a smoke test for us.
Unless we are also needing to bump the XGBoost.jl compat, I don't see we need to tag a new release of MLJXGBoostInterface.jl as breaking. (However, I'm not following the policy vis a vie XGBoost.jl and it's dependency xgboost.)

Okay, I see that XGBoost 2.0 has just been released.(I just fixed CompatHelper which was broken.) So, in that case, assuming we bump the compat, we should also tag our new release as breaking.
In line with that, I support the current PR.

Okay, I see that XGBoost 2.0 has just been released.(I just fixed CompatHelper which was broken.) So, in that case, assuming we bump the compat, we should also tag our new release as breaking.

Uhm are we talking about XGBoost or XGBoost.jl here? XGBoost has released a breaking version (1.7.6 -> 2.0) which XGBoost.jl incorporated in a non-breaking version (2.3.2 -> 2.4). Therefore, the XGBoost 2.0 version has and will now automatically be installed for most systems. This is not a critique or anything, by the way. As said above, I understand the decision making process here.

That's right, the libxgboost changes were breaking but the XGBoost.jl changes were not.


Add the space-frame-work to the KS setup scripts

The goal is to add the space framework to the existing KS setup scripts. This doesn't require any changes in the KS controller code.
Tasks:
phase 1: PR-1

Setup and bringup the SF container (start the SM controller)
Install KubeFlex
Use the existing KCP container (will be used as a KCP provider)
Create and configure the KubeFlex space provider,
Create and configure KCP provider

The space framework is not being used, only setup and configured.
Phase 2: PR-2
Change the KS scripts to use the space-manager + KCP space provider
All direct KCP calls to create workspaces should be replaced with using the space manager
No KS controllers code is changed

https://github.com/kubestellar/kubestellar/pull/1218 submitted for PR1.


I want to contribute to this open source

I want you to grant me the opportunity to contribute to this open source repo.

Hey man, go right ahead


Add changelog reminder action

Context: Now that we are working on v0.2.0, having a changelog reminder will help us avoid additional work during the release, by ensuring all changes are documented in PRs :)
Description of the Change: Adds an action to remind us to edit the changelog if we haven't already.
Benefits: As above!
Possible Drawbacks: n/a
Related GitHub Issues: n/a

Note: I'm not going to add a changelog reminder in this PR since I am adding a repo action, and not touching the actual catalyst package :)

However, you can't have a changelog entry, with reference to the correct PR, before opening said PR. Is there any way to resolve this?

Yep it's a chicken-or-egg situation. On PL typically we add the changelog entry to our branch first, and once the PR is open edit in the PR number.

Yep it's a chicken-or-egg situation. On PL typically we add the changelog entry to our branch first, and once the PR is open edit in the PR number.

That's unfortunate, easy to forget adding the PR number for both the author and reviewer 😞

Yep. We haven't come up with a better solution (there might be one?) but the reminder has been incredibly helpful. It is very easy to forget to update the changelog at all


Fix FITS file close handling

The AsdfInFits class should not be closing the FITS file, since it's passed in and it doesn't "own" it.
Ideally HDUList.close() would close and flush all of it's mmap handles, but it doesn't.  Fixing that is sort of a astropy.io.fits issue that is sort of non-trivial to fix (see astropy/astropy#3827), and pyasdf shouldn't be trying to patch around that with magic.
Here, the magic is moved to the testing functions only just to get setup.py test --open-files to not complain.

Would it help if I were to get on that?  (I see you already have a solution anyways so maybe it's not urgent.)  I'd still like to fix this though.

Yeah -- it's not urgent, but it would help.  My realization in this PR was essentially that I was papering over something rather than fixing it in the right place...


supports latest sprockets

Currently compass-rails is locked to
gem.add_dependency 'sprockets', '< 2.13'
any idea what it takes to make it support latest sprockets?

+1

Yes, it takes analysis, software updates and testing. I'll happily take a pull request if you want to attempt an update.

Closing this unless someone wants to take a stab at it.


Task :r2dbc-bom:checkVersionCatalogCompatibility FAILED

Issue description
% ./gradlew build
...

> Task :r2dbc-bom:generatePomFileForMavenPublication
Projects included into BOM:
    - :r2dbc-core
Skipped projects which do not apply the publishing plugin:
    - :test-graalvm:common

> Task :r2dbc-bom:checkVersionCatalogCompatibility FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':r2dbc-bom:checkVersionCatalogCompatibility'.
> Failed to calculate the value of task ':r2dbc-bom:checkVersionCatalogCompatibility' property 'baseline'.
   > Could not resolve all dependencies for configuration ':r2dbc-bom:detachedConfiguration1'.
     The project declares repositories, effectively ignoring the repositories you have declared in the settings.
     You can figure out how project repositories are declared by configuring your build to fail on project repositories.
     See https://docs.gradle.org/7.5.1/userguide/declaring_repositories.html#sub:fail_build_on_project_repositories for details.
      > Could not find io.micronaut.r2dbc:micronaut-r2dbc-bom:3.1.0.
        Searched in the following locations:
          - https://repo.maven.apache.org/maven2/io/micronaut/r2dbc/micronaut-r2dbc-bom/3.1.0/micronaut-r2dbc-bom-3.1.0.pom
        If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
        Required by:
            project :r2dbc-bom

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

Think this was caused simply by the task being run before 3.1.0 had hit maven central
It passes now


Add missing dialect items.

can_jump is among them.
spr_dir ?

Theyre missing because they are only valid in certain files or contexts.


Config for 10.0.19041.962 required

Hi,
Does anyone have a working config for 10.0.19041.962?
I've found this in github - but it doesn't appear to work - I get the "number of connections is limited" error.
[10.0.19041.962]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=89F31
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=0CB22
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=18A15
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=1E29C
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.962-SLInit]
bInitialized.x64      =107108
bServerSku.x64        =10710C
lMaxUserSessions.x64  =107110
bAppServerAllowed.x64 =107118
bRemoteConnAllowed.x64=107120
bMultimonAllowed.x64  =107124
ulMaxDebugSessions.x64=107128
bFUSEnabled.x64       =10712C
Any help, greatly appreciated.
Thanks,
--JS

I copied the entire file (rather than just the 10.0.19041.962 sections) and rebooted - then it all worked properly.
Closing this issue.
Thanks,
--JS


Change app logo to vector graphic

This reduces the size of the apk a bit and should make changes in the future easier

Here's the logo

Feature graphic added to Play Store.


Major: Refactor parameters and combine registry and type constants

Fixes #366
Fixes #147
See: #424

Created https://jira.diamond.ac.uk/browse/I03-866 to do the GDA side work


ES5 set accessors: incorrect warning expected typedef for call-signature

class Foo {
private _bar: boolean = false;
get bar(): boolean {
	return this._bar;
}
set bar(theBar: boolean) {
	this._bar = theBar;
}

}
gives error: expected call-signature: 'bar' to have a typedef
However, the typescript compiler throws an error if you try to give a typedef to the return value of a set accessor.

fix here: #334

ty


DecimalFormat format problem edge case

DecimalFormat works fine for me 99.9% of the time. But every once in a while I hit something like this:
String str = new DecimalFormat("#.##").format(0.9977993000000007d);

For me this produces the string ".:".

I wonder if this is one of the classes pulled in from Apache Harmony?  Interesting case.
The source is here: https://github.com/konsoletyper/teavm/blob/master/classlib/src/main/java/org/teavm/classlib/java/text/TDecimalFormat.java
Looks very different from the Apache Harmony code here: http://svn.apache.org/viewvc/harmony/enhanced/java/branches/java6/classlib/modules/text/src/main/java/java/text/DecimalFormat.java?view=markup
So perhaps this is one that was hand-coded.  We should contribute a test case.

I wrote a quick test and ran it on a desktop JVM.
DecimalFormat decimalFormat = new DecimalFormat("#.##");
org.junit.jupiter.api.Assertions.assertEquals("1.00", decimalFormat.format(0.9977993000000007d));  // Expected should be 1, really

Output:
org.opentest4j.AssertionFailedError: expected: <1.00> but was: <1>
Now '1' is the right answer, if I'm reading the javadocs right.  It rounds up if more than halfway, and it isn't supposed to show fractional zeroes by default.
So if this isn't working in the browser it is something deeper with how doubles get translated into JS.


Enable uv tool uninstall uv on Windows

Summary
Extending self-delete and self-replace functionality to uv itself on Windows.
Closes https://github.com/astral-sh/uv/issues/6400.

I figure this pull request is loosely on hold until we figure out what we want to do with #9143 and https://github.com/astral-sh/uv/issues/9144

@charliermarsh security flagging looks fixed, let's merge this?

Sg.


Tenants API

Resolves https://trello.com/c/QdPWdhWN
@abellotti @gtanzillo please review

@miq-bot add-label core api

@miq-bot add-label enhancement

@miq-bot add-label core, api

looks good @imtayadeway  minor comments, and the rubocop warn. Thanks.

@abellotti addressed your comments and the rubocop offense (and some new rubocop offenses) in subsequent commits. Anything to add?
One thing I meant to bring up yesterday - do the GETs need to have RBAC applied to them? I see that there are roles for lists and shows for tenants, but could not get this to work so left it out.
I'll squash the smaller stuff before this gets merged :)

@h-kataria can you advise on the proper roles for list/show of tenants in the API?

@imtayadeway all the tenant features are listed in miq_product_features.yml, https://github.com/ManageIQ/manageiq/blob/master/db/fixtures/miq_product_features.yml#L2290 for list & show of Tenants you might need rbac_tenant_view feature that allows both rbac_tenant_show_list & rbac_tenant_show features. Let me know if you need more information.

Thanks @h-kataria, that helps.
Added some infrastructure here https://github.com/ManageIQ/manageiq/pull/4754 to support adding RBAC validation for the show/list actions

@abellotti just rebased to incorporate RBAC for GET requests, with added tests

@abellotti just applied your patch and waiting for green. Thanks!

:cake:


[question] ghidra + python : createData() and other issues

I find it very, very difficult to try and use what I find in the builtin API docs, from a python script or the console.
Here's a typical transcript of my attempts to use createData() to create a simple pointer :
>>> createData(toAddr(0x10), Pointer)
Traceback (most recent call last):
  File "python", line 1, in <module>
NameError: name 'Pointer' is not defined

>>> createData(toAddr(0x10), PointerDataType)
Traceback (most recent call last):
  File "python", line 1, in <module>
NameError: name 'PointerDataType' is not defined

>>> getDataTypes("pointer")
array(ghidra.program.model.data.DataType, [pointer])

#### maybe no caps ?

>>> createData(toAddr(0x10), pointer)
Traceback (most recent call last):
  File "python", line 1, in <module>
NameError: name 'pointer' is not defined

Ok, I marked location 0 as a pointer via the Codebrowser. I should be able to get the type from there...
>>> getDataAt(toAddr(0x0)).dataType
pointer

### thanks

>>> createData(toAddr(0x10), ghidra.program.model.data.DataType.pointer)
Traceback (most recent call last):
  File "python", line 1, in <module>
AttributeError: type object 'ghidra.program.model.data.DataType' has no attribute 'pointer'

This is absolutely horrible
>>> createData(toAddr(0x10), getDataAt(toAddr(0x0)).dataType)
addr 00000104

### what if I try to import stuff ?

>>> from ghidra.program.model.data import DataType
>>> createData(toAddr(0x14), pointer)
Traceback (most recent call last):
  File "python", line 1, in <module>
NameError: name 'pointer' is not defined

>>> from ghidra.program.model.data import PointerDataType
>>> createData(toAddr(0x14), PointerDataType)
Traceback (most recent call last):
  File "python", line 1, in <module>
TypeError: createData(): 2nd arg can't be coerced to ghidra.program.model.data.DataType

### Come on, seriously ?

>>> from ghidra.program.model.data import Pointer
>>> createData(toAddr(0x14), Pointer)
Traceback (most recent call last):
  File "python", line 1, in <module>
TypeError: createData(): 2nd arg can't be coerced to ghidra.program.model.data.DataType

>>> createData(toAddr(0x14), Pointer.getDataType())
Traceback (most recent call last):
  File "python", line 1, in <module>
TypeError: getDataType(): expected 1 args; got 0

I don't know what else to try, and I don't understand how I'm supposed to be mapping stuff from java to python in cases like this. Can't find anything equivalent in the example scripts either.

from ghidra.program.model.data import PointerDataType
createData(toAddr(0x14), PointerDataType)

What happens if you put "()" at the end of PointerDataType?
createData(toAddr(0x14), PointerDataType())

this seems like a basic programming/REPL issue and not really ghidra/API.
Its returning the type, you're not assigning it
>>> ptr_type = getDataTypes("pointer")
>>> data = createData(toAddr(0x10), ptr_type[0])

What happens if you put "()" at the end of PointerDataType?

Of course, that works...

ptr_type = getDataTypes("pointer")
data = createData(toAddr(0x10), ptr_type[0])

And that too.
Thanks.


Add case to build upstream kernel from git source

@ader1990 it's not duplicated. We need the VM that contains upstream source as well as compilation history, for developers to use. With the VM, they can apply patch and quickly build the kernel. The -customKernel parameter only supports install kernel from package.

@lizzha So why not just modify customKernelInstall.sh for your needs and use BVT-VERIFY-DEPLOYMENT-PROVISION? This way you'll have a VM with a custom kernel installed

@adriansuhov that's a good suggestion, let me update customKernelInstall.sh

@lizzha please check again the functionality here: https://github.com/LIS/LISAv2/blob/master/Testscripts/Linux/customKernelInstall.sh#L289
You can reuse that functionality or change it if need be, but we wouldn't want to end up with duplicate kernel build code in lisav2.
and please do not remove the labels without a proper discussion on why they have been added.


error fetching ISO via minio client

There have been some reports that the latest minio client is causing the ISO to fail to download correctly.
It has been suggested that a fix is to replace https://github.com/equinix/terraform-metal-vsphere/blob/b96face0faa50cf565281e8be5aac6c1e9fc1920/templates/download_vcenter.sh#L32 with a tagged release, for example:
curl -Lo mc https://dl.min.io/client/mc/release/linux-amd64/archive/mc.RELEASE.2022-01-07T06-01-38Z

Currently awaiting confirmation that this is a real bug or working solution.
Regardless, pinning to a fixed version of assets will provide greater reliability - verifying that asset against a known checksum would be even better.

Change is good.  I just verified with a fresh pull + change as above

The upstream issue is https://github.com/minio/mc/issues/3938


Never register non-production deployments in global endpoint

or @oyving

I did the cleanup after deploying the patch.


ui - main menu

[x] play
[ ] settings
[ ] settings > key bindings
[ ] settings > osu!python songs folder
[ ] settings > songs folder sharing
[ ] settings > exporting single beatmap from lazer or stable client
[ ] exit
[ ] random music
[ ] music controls

moved to 1st major release


[FEATURE] AtomReference members are private

Is your feature request related to a problem? Please describe.
register an event to an AtomReference is not possible via code, since the members are private and there are no getters.
https://github.com/AdamRamberg/unity-atoms/blob/master/Packages/Core/Runtime/References/AtomReference.cs
Describe the solution you'd like
Add public getters, which throw errors if the usage type does not match.
Additional context
Asked via discord:

Is there any way to register an action to a Reference within a script? I'm able to register to an Instancer with myInstancer.Variable.Changed.Register(myAction);, but not a Reference. Am I missing something?

Sounds good to me! Let's add it in the next version of Unity Atoms! :)

Fixed in #158


Reactify issue.

Not exactly sure why, maybe it's browserify bug, but for some reason when I try to browserify and this module is included it will attempt to run the "reactify" transform. Since "reactify" is a devdependency and I use babel instead it doesn't find it and errors out.
Cannot find module 'reactify' from '/.../node_modules/react-dragula'

A quick fix would just be making reactify a regular dependency or not using the package.json browserify fields.

Ah sorry, this is my fault. You can pin to 1.1.0 for the time being.

Released #9 as 1.1.13


Adds Mesh implementation

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🔖 Release
[ ] 🚩 Other

Description
Adds several classes to the Geometry namespace to implement a half-edge mesh data-structure.
It also adds some initial basic unit tests to ensure everything works as expected.
Related Tickets & Documents
Fixes #225
Added tests?

[x] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📓 docs
[ ] 🙅 no documentation needed

This is still a draft, but I thought I'd open this already so that you guys can have a peek too and let me know what you think.
I'll do a review in a bit to highlight the most "controversial" changes ;)


Jsonapi-ts should export the utils used in the serializer

While the approach used as default in the serializer is really cool, it would be really helpful to export the string utilities used in the project.

I just realized, that this is blocking dynamic resources for @chameleon

Done!


3.10.x: nuxt build creates vue files in .nuxt/dist/server/_nuxt folder

Environment
- Operating System: Darwin
- Node Version:     v21.2.0
- Nuxt Version:     3.10.0
- CLI Version:      3.10.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, css, modules, runtimeConfig, image, build, i18n, ignore, nitro, routeRules, security, vite
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

Reproduction

Describe the bug
With 3.10.0 and 3.10.1 I get an error at the end of nuxt build: ERROR  RollupError: At least one <template> or <script> is required in a single file component, with 3.9.3 everything is fine.
Unfortunately I cant reproduce this in the sandbox but I already tried to debug this for hours. I nailed it down to one of my ~10 components: If it contains a <style> tag with ANY content (even just a comment). Weirdly when I delete the content of the <script> tag I can have styles:
Throws the error:
<script setup lang="ts">
// hello
</script>

<template>
  hello
</template>

<style lang="scss">
// hello
</style>

Doesnt throw an error:
<script setup lang="ts">
// hello
</script>

<template>
  hello
</template>

<style lang="scss"></style>

Doesnt throw an error:
<script setup lang="ts"></script>

<template>
  hello
</template>

<style lang="scss">
// hello
</style>

It also only happens with this one component named overlay.vue. I feel like I am getting crazy or something but this seems to make absolutely no sense.
Full error:
 ERROR  RollupError: At least one <template> or <script> is required in a single file component.         nitro 9:00:59 PM

undefined

 ERROR  At least one <template> or <script> is required in a single file component.                            9:00:59 PM

  at error (node_modules/rollup/dist/es/shared/parseAst.js:337:30)
  at Object.error (node_modules/rollup/dist/es/shared/node-entry.js:18507:20)
  at Object.error (node_modules/rollup/dist/es/shared/node-entry.js:17616:42)
  at node_modules/rollup-plugin-vue/dist/sfc.js:22:49
  at Array.forEach (<anonymous>)
  at Object.transformSFCEntry (node_modules/rollup-plugin-vue/dist/sfc.js:22:16)
  at Object.transform (node_modules/rollup-plugin-vue/dist/index.js:99:38)
  at node_modules/rollup/dist/es/shared/node-entry.js:18692:40

 ERROR  At least one <template> or <script> is required in a single file component.                            9:00:59 PM

error Command failed with exit code 1.

Additional context
No response
Logs
No response

This is unlikely to be an issue with Nuxt - rollup-plugin-vue is not used by Nuxt or Nitro and is what is throwing your error.

I am not 100% sure about that. Yes this plugin throws the error BUT I disabled the plugin and realized there is exactly one .vue file in the folder .nuxt/dist/server/_nuxt which is exactly the file I debugged to be the one creates the error. I guess there shouldnt be any vue file in this folder which one mean that there actually is a bug within Nuxt.

Interesting. Have you customised assetFileNames by any chance?

I did not

I probably can't look into this without a reproduction or more info.

I can try to dig more into this and make this reproducable.

@danielroe So it took me a few hours of removing more and more stuff from my app until I ended in an empty app and I realized:
This happens on a blank empty app created with nuxi init and just a single index page and a single component in it! If the component has a script with lang=ts and a style tag (both with ANY content) a vue file is in the .nuxt/dist/server folder!
Since StackBlitz doesnt seem to work, here is the repo: https://github.com/MickL/nuxt-bug

When you say that StackBlitz doesn't work, what do you mean?

I can't reproduce with that repo.
Here's a StackBlitz created from it - seemingly working fine: https://stackblitz.com/edit/github-iqamf7.
Could you share any more info about your setup?

I used bun install if that changed anything. My specs are at the very top. I think #25690 could be the same issue

When you say that StackBlitz doesn't work, what do you mean?

I meant that the links on the Nuxt website (the two I posted above) both dont start and produce errors.
So I double checked on another machine. I downloaded my repo, run npm install and then npm run buld. If I check the folder `` I see (as I described) there is a vue file which is probably supposed to be a js file:

You're quite right; it is reproducible there - I was checking .output, not .nuxt.

I am not sure if this is causing me problems now - I updated my packages to latest (all of them), and somehow I get an error:
cannot find module './stringify' (module in question is qs, which is only used by Strapi module as far as I can tell)
This only happens in dev mode, build is running fine. When I don't include it in the configuration everything works.
If I keep use the Nuxt ^3.10.1 version I have no problems. Does any of this ring a bell?


undefinedn

SCSS code I'm using:
@for $i from 1 through $grid-columns {

		.col-#{$i*5} {    //for more col size options for tables
			lost-column: $i / $grid-columns;
		}

	}

where $grid-columns = 20
which is compiling to:
...  (only showing one section for brevity)
.col-10 {
    width: calc(99.99% * 0.1 - (30px - 30px * 0.1));
  }

  .col-10:nth-child(n){
    float: left;
    margin-right: 30px;
    clear: none;
  }

  .col-10:last-child{
    margin-right: 0;
  }

  .col-10:nth-child(undefinedn){
    margin-right: 0;
  }

  .col-10:nth-child(undefinedn + 1){
    clear: left;
  }

...
what's with the "undefinedn" ?
any help is greatly appreciated.

sigh, I think it's because I'm compiling the scss first and it's convertin $i/$grid-columns to a decimal... so like it gives .col-10 { lost-column: 0.1; } going to see about getting it to leave it as a fraction

What happens when you compile Lost first and then Sass?

same result if I compile Lost first then sass

This is a problem, but moreover, what you're trying to do won't work like I think you're expecting it to. I don't think there's a way to generate functional classes like that with Lost yet.
I'll look into this though and get back to you in a few days.

ah, figured it out. had to do this:
.col-#{$i*5} {    //for more col size options for tables
			lost-column: unquote($i + '/' + $grid-columns);
		}

which now gives:

  .col-10 {
    width: calc(99.99% * 2/20 - (30px - 30px * 2/20));
  }

  .col-10:nth-child(n){
    float: left;
    margin-right: 30px;
    clear: none;
  }

  .col-10:last-child{
    margin-right: 0;
  }

  .col-10:nth-child(20n){
    margin-right: 0;
  }

  .col-10:nth-child(20n + 1){
    clear: left;
  }

sorry, not a lost grid thing but instead a lack of sass knowledge on my part.

Keep me posted on whether you get functional classes out of this. I'm betting all the nth-child stuff will mess it up.
I really need to add a function in there that generates classes like this.


Add support for heap sampling profiler in the standalone VM

The Dart VM provides a mechanism to sample Dart heap allocations through a C API, but does not provide any support for sampling allocations out of the box. We should hook up the sampling profiler in the VM and provide support for accessing it through the VM service protocol.

Can this be closed as done?

Can this be closed as done?

This is still an open issue since the standalone VM doesn't support heap allocation profiling. We'd have to configure it and make it available through the service protocol.


[CORE-1741] use forked pytesseract to reduce calls to tesseract

This PR resolves CORE-1741 by using a new function pytesseract.run_and_get_multiple_output, see forked repo for more details: https://github.com/Unstructured-IO/unstructured.pytesseract/releases/tag/0.3.11-dev1
This reduces the call to tesseract by half per page of PDF/image during partition, roughly reducing the runtime by 48%.
The new function is in forked unstructured.pytesseract. A PR has been made to the upstream repo and once that is merged we should switch to the up stream version. For now we add a new dependency: unstructured.pytesseract.
testing
Existing unit tests should serve as tests to the new function.
To demonstrate the changes in performance:

checkout main
run ./scripts/performance/profile.sh and select ocr_only strategy, using the 10th document (16 page layout paper in pdf format)
examine the speedscope profile or time profile in flamegraph -> should see two dominant time spenders are pytesseract.image_to_text and pytesseract.image_to_boxes, with both about the same total time (see attached first image)
checkout this branch
run the same profile.sh with the same options
examine the profile again and this time should notice 1) total runtime is reduced by more than 40%; 2) only unstructured_pytesseract.run_and_get_multiple_output is the top time spender and its total time is about the same as either the pytesseract.image_to_text or pytesseract.image_to_boxes time (see second image below)

~related to other comment, we also want to be sure that pip install "unstructured[all-docs]" pulls in unstructured.pytesseract not pytesseract.~
nevermind, per @badGarnet 's prior comment, this is now available in upstream pytesseract

except I forgot to bump version for upstream .... so gonna do that and wait for that happen...

gave up on waiting for upstream: here is local test on @cragwolfe 's comment
pyenv activate unstructured
python setup.py bdist_wheel
pyenv activate test
pip install "dist/unstructured-0.10.13.dev3-py3-none-any.whl[all-docs]"

confirm the list of output installed unstructured.pytesseract. Unfortunately we can't avoid pytesseract from being installed because there are packages depend on it and we can't change those packages dependencies (or am I missing some trick?)
Successfully installed MarkupSafe-2.1.3 Pillow-10.0.0 XlsxWriter-3.1.3 antlr4-python3-runtime-4.9.3 beautifulsoup4-4.12.2 certifi-2023.7.22 cffi-1.15.1 chardet-5.2.0 charset-normalizer-3.2.0 click-8.1.7 coloredlogs-15.0.1 contourpy-1.1.0 cryptography-41.0.3 cycler-0.11.0 dataclasses-json-0.6.0 ebooklib-0.18 effdet-0.4.1 emoji-2.8.0 et-xmlfile-1.1.0 filelock-3.12.3 filetype-1.2.0 flatbuffers-23.5.26 fonttools-4.42.1 fsspec-2023.9.0 huggingface-hub-0.17.1 humanfriendly-10.0 idna-3.4 importlib-metadata-6.8.0 importlib-resources-6.0.1 iopath-0.1.10 jinja2-3.1.2 joblib-1.3.2 kiwisolver-1.4.5 layoutparser-0.3.4 markdown-3.4.4 marshmallow-3.20.1 matplotlib-3.7.3 mpmath-1.3.0 msg-parser-1.2.0 mypy-extensions-1.0.0 networkx-3.1 nltk-3.8.1 numpy-1.24.4 olefile-0.46 omegaconf-2.3.0 onnx-1.14.1 onnxruntime-1.15.1 opencv-python-<IP_ADDRESS> openpyxl-3.1.2 packaging-23.1 pandas-2.0.3 pdf2image-1.16.3 pdfminer.six-20221105 pdfplumber-0.10.2 portalocker-2.7.0 protobuf-4.24.3 pycocotools-2.0.7 pycparser-2.21 pypandoc-1.11 pyparsing-3.1.1 pypdfium2-4.20.0 pytesseract-0.3.10 python-dateutil-2.8.2 python-docx-0.8.11 python-magic-0.4.27 python-multipart-0.0.6 python-pptx-0.6.21 pytz-2023.3.post1 pyyaml-6.0.1 regex-2023.8.8 requests-2.31.0 safetensors-0.3.3 scipy-1.10.1 soupsieve-2.5 sympy-1.12 tabulate-0.9.0 timm-0.9.7 tokenizers-0.13.3 torch-2.0.1 torchvision-0.15.2 transformers-4.33.1 typing-inspect-0.9.0 tzdata-2023.3 unstructured-0.10.13.dev3 unstructured-inference-0.5.25 unstructured.pytesseract-0.3.12 urllib3-2.0.4 xlrd-2.0.1 zipp-3.16.2


update-fonts

I prefer not to use external resources (styles, fonts, scripts), so everything is stored locally.
SVG is actually a good idea, although it fits wrong in 540px example. But for 1080px it's great.

Local fonts. SVG size fix.

Thanks, merged.
Also accidentally pushed it to your branch :)


Azure IoTHubSource stale dependency

The latest @dependabot update of Azure EventHubs module revealed the issue with Azure IoTHubSource - one of third-party dependency that we use in the source is outdated and hence incompatible with the latest eventhubs client (details).
I'd wait a bit and see if the third-party client is going to be updated, and if not, we should either fix it ourselves or retire IoTHubSource entirely.

@FranBarrera lets get the Azure library update you're working on done first, maybe it will trickle down and impact this issue.


Skip AdditionalGID's for LCOW oci spec

Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

This commit fixes the ability to run LCOW via CTR.
C:\> ctr image pull --snapshotter windows-lcow --platform linux/amd64 docker.io/library/alpine:latest
C:\> ctr run --runtime io.containerd.runhcs.v1 --snapshotter windows-lcow --rm docker.io/library/alpine:latest lcow-test sh -c "echo Hello World!"
Hello World!

Codecov Report

Merging #2698 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2698      +/-   ##
==========================================
- Coverage    43.6%   43.59%   -0.02%
==========================================
  Files         101      101
  Lines       10718    10722       +4
==========================================
  Hits         4674     4674
- Misses       5310     5314       +4
  Partials      734      734

Flag
Coverage Δ

#linux
47.25% <0%> (-0.04%)
:arrow_down:

#windows
40.72% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

oci/spec_opts.go
20.77% <0%> (-0.12%)
:arrow_down:

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 de4bb2d...223acbc. Read the comment docs.

LGTM

LGTM


@if formatting is not working

@if formatting is not working
actual formatting
        @if ($params->allow_google_in_country)

            aaa

        @endIf

expected formatting
        @if ($params->allow_google_in_country)

                aaa

        @endIf

The formatting and indentation for blade code is a feature which is in work in progress.
It is not working yet.

Hi,
With the release of https://github.com/haidubogdan/netbeans-php-blade-plugin/releases/tag/nb180-v2.1.21
I've added the formatting action. But you will have to enable it manually for the moment.

v 2.1.21 still has some fornatting issues, do I recommend waiting for a later release .

I've fixed the relevant issues so https://github.com/haidubogdan/netbeans-php-blade-plugin/releases/tag/nb180-v.2.1.3
Can be tested for the formatting flows.

I just downloaded the latest release and tested with a simple blade
<x-base::layouts.app>

        @include('base::components.breadcrumbs')

                    {{ __('invoices::collection-invoices.index.Collection invoices') }}

            <livewire:invoices::collection-invoice-table />

</x-base::layouts.app>

After formatting CTRL + SHIFT + F
<x-base::layouts.app>

        @include('base::components.breadcrumbs')

                    {{ __('invoices::collection-invoices.index.Collection invoices') }}

                                                        <livewire:invoices::collection-invoice-table />

                                                </x-base::layouts.app>

Test number 2
Openend somewhat bigger blade
Without formatting the blade looks like this

When removing the @foreach

So many new stuff : )
I will prepare a fix for the formatting issue.
And it seems I have I to take care about components include in the future
@include('base::components.breadcrumbs')

You know you are the life saver of netbeans and laravel! Great work.

I will put it on in progress as it's a bit "wonky".
Some existing issues:

Ctrl-Z is resetting each last format step by step
some tags are still not indented well

Closing this issue, some progress have been done, and new separate issues have been opened for other formatting errors.


CFE reports incompatible implementations error for a valid mixin

abstract class A<Result, Input> {
  Result f(Input v);
}

abstract class A0<Result, Input> extends A<Result, Input> {

}

abstract class M extends A<num, num> {
  @override
  num f(num v) => null;
}

class _C extends A0<num, num> with M {
}

void main() {

}

causes
file:///tmp/xxx.dart:15:7: Error: '_C' can't implement both '#lib1::A<dynamic, dynamic>' and '#lib1::A<dart.core::num, dart.core::num>'
class _C extends A0<num, num> with M {
      ^

which does not seem correct to me.

It's really weird that we don't seem to have any tests covering this case.

Proposed fix: https://dart-review.googlesource.com/c/sdk/+/42106

Here's a slightly different version of Kevin's approach: https://dart-review.googlesource.com/c/sdk/+/42260


WARNING - The model and loaded state dict do not match exactly.                                                             When I used the yolox training model, the terminal trained twice. When I used the last saved results of the second training for the third time, I found that epoch began at the end of the first training?

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

When I used the yolox training model, the terminal trained twice. When I used the last saved results of the second training for the third time, I found that epoch began at the end of the first training?

Sorry for not getting your points. Could you provide the config and error report following the issue template?

This issue will be closed due to inactivity. Feel free to reopen it if you have any questions.


[Docs] Add auto-compact docs

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[X] Other (Docs)

Description
Adding docs for auto-compact. Rendered:

How was this patch tested?
Generated the docs

Does this PR introduce any user-facing changes?

Jenkins Trigger: Delta Connectors Tests / Run tests (2.12.17) (pull_request)


Modifications:

Move cmd package under dogma/cmd
Revamp CLI to use Go client library
Add --message or -m flag to add commit message
Remove searchCommand
Modify getDiff in the REST API to distinguish fetching one diff or more than one diff

Result:

Better code

Codecov Report

Merging #172 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #172      +/-   ##
========================================
+ Coverage   63.96%    64%   +0.04%
========================================
  Files         275    275
  Lines        9532   9533       +1
  Branches     1065   1065
========================================
+ Hits         6097   6102       +5
+ Misses       2810   2805       -5
- Partials      625    626       +1

Impacted Files
Coverage Δ

...raldogma/server/internal/api/ContentServiceV1.java
91.17% <100%> (+0.08%)
:arrow_up:

...ogma/client/armeria/legacy/LegacyCentralDogma.java
72.34% <0%> (-1.07%)
:arrow_down:

...internal/storage/repository/git/GitRepository.java
79.14% <0%> (+0.3%)
:arrow_up:

...ernal/storage/repository/git/CommitIdDatabase.java
74.56% <0%> (+1.75%)
:arrow_up:

...ogma/server/internal/storage/StorageException.java
44.44% <0%> (+22.22%)
: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 b5abb0a...c3ba80b. Read the comment docs.


sharding-proxy mysql  varchar can not complete display

mysql varchar(30)
with shrading proxy ,this waybill_no varchar(30)   field  can not complete display

Please provide a version of sharding-proxy ,It would be nice if you could provide configuration and table structure statements for the test

@wangzhengxi Any new for this issue? do you test it again?

shardingRule:
tables:
ops_pick_task:
actualDataNodes: yl-ops-0${1..9}.ops_pick_task_${0..31},yl-ops-1${0..2}.ops_pick_task_${0..31}
databaseStrategy:
standard:
shardingColumn: order_id
preciseAlgorithmClassName: com.yl.sharding.spi.MyPreciseShardingAlgorithm
tableStrategy:
inline:
shardingColumn: order_id
algorithmExpression: ops_pick_task_$->{Math.abs(order_id.hashCode()) % 32}

sharding-proxy4.0.0-RC1

i find the answer。when  use navicat 11.1.13 Enterprise Edition，some field can not complete display。but when i use navicat 12.0.29 Enterprise Edition, it complete display .

the existing application's compatibility also needs to be handled

i find the answer。when use navicat 11.1.13 Enterprise Edition，some field can not complete display。but when i use navicat 12.0.29 Enterprise Edition, it complete display .

Is it a navicat's issue?

Closed for no response any more, please create a new issue if it still existed.


Fastlanez-sys is always rebuilt

Happens because header file is always moved and cargo only checks modification times

I think this is no longer an issue. Seems that zig got smarter or something else changed but fastlanez-sys seems to be correctly up to date now


Request ran for longer than 1.8 seconds

Hi!
I'm using rack-timeout 0.2.0 on Heroku with Rails.I get this error from time to time:
2015-03-12T08:30:11.110453+00:00 app[web.2]: E, [2015-03-12T08:30:11.110420 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/gems/unicorn-4.8.3/lib/unicorn/http_server.rb:670:in `worker_loop'
2015-03-12T08:30:11.110484+00:00 app[web.2]: E, [2015-03-12T08:30:11.110450 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/gems/unicorn-4.8.3/lib/unicorn/http_server.rb:525:in `spawn_missing_workers'
2015-03-12T08:30:11.110514+00:00 app[web.2]: E, [2015-03-12T08:30:11.110481 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/gems/unicorn-4.8.3/lib/unicorn/http_server.rb:140:in `start'
2015-03-12T08:30:11.110577+00:00 app[web.2]: E, [2015-03-12T08:30:11.110512 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/gems/unicorn-4.8.3/bin/unicorn:126:in `<top (required)>'
2015-03-12T08:30:11.110611+00:00 app[web.2]: E, [2015-03-12T08:30:11.110576 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/bin/unicorn:23:in `load'
2015-03-12T08:30:11.110640+00:00 app[web.2]: E, [2015-03-12T08:30:11.110608 #12] ERROR -- : /app/vendor/bundle/ruby/2.2.0/bin/unicorn:23:in `<main>'
2015-03-12T08:30:11.111314+00:00 app[web.2]: source=rack-timeout id=84784c36-ceb1-41cb-a6c4-d0a8fbfb7761 wait=30696ms timeout=30000ms state=expired
2015-03-12T08:30:11.160040+00:00 app[web.2]:
2015-03-12T08:30:11.160044+00:00 app[web.2]: ActionView::Template::Error (Request ran for longer than 1.8885111324057604 seconds.):

So the request ran for 30696ms when the timeout is 30000ms, but the error says it ran for longer than 1.8 seconds, which is wrong.
Also, a weird part is that I have this in an initializer:
Rack::Timeout.timeout = 22 if Rails.env.production?

Am I doing something wrong, or did I catch a bug?
Thanks!

That error message is for a RequestTimeoutError (but rails is lame and wraps errors raised during view code in an ActionView error)… the line above says the state is expired, so that would have been a RequestExpiryError. And these are dropped before they ever reach rails so ActionView would have never been in play.
So basically the two lines are talking about different requests. You should have a log line somewhere else with state=timed_out and service=18???ms that correlates with the raised error you see.
As for timing out at 1.8 instead of 22, that's related to the wait time. You can read more about it in the readme.

Also I don't think you're running 0.2.0, otherwise the "ran for longer" message would have been different, secs in ms, wait listed.

Yes, I double checked the version and I'm using 0.2.0:
$ cat Gemfile.lock | grep rack-timeout
    rack-timeout (0.2.0)
  rack-timeout

Also, I've searched the logs and found some lines with state=timed_out:
Mar 12 01:18:33 empresaula app/web.1:  source=rack-timeout id=bec2ca50-682d-4c8e-8314-ae59ec7aa534 wait=28790ms timeout=1210ms service=1215ms state=timed_out
Mar 12 01:18:33 empresaula app/web.1:  source=rack-timeout id=3870d089-3e71-4c47-ae6c-95c3599fb4b1 wait=29391ms timeout=609ms service=611ms state=timed_out
Mar 12 01:18:34 empresaula app/web.1:  source=rack-timeout id=583ba945-f715-4797-8f4c-8d3023dfeba6 wait=29857ms timeout=143ms service=151ms state=timed_out
Mar 12 01:18:47 empresaula app/web.1:  source=rack-timeout id=0ddcf58b-ae0b-49a6-8cb3-4ab793ce76bf wait=29776ms timeout=224ms service=224ms state=timed_out

Taking, for example, the last line, I guess this is what happens: request runs for 29776ms before it gets any data (wait time). Then the server starts processing for 224ms, but as 29776 + 224 = 30000ms, request is timed-out. So the fault of these "Request ran for longer than 1.8 seconds" is the wait time, which is too high. Is that correct?

Correct. Also this wait time situation seems terrible. You should contact heroku support and see if we can help you with that.
As for you being on 2.0, it's really weird that you're getting that message in that particular form (it doesn't matter though, it's just a string). I wonder if I botched things when I released 2.0. Will check. I put a 2.1 out anyhow.


Content PR for auto-componentstate sourcegen

About the PR

requires space-wizards/RobustToolbox#3684
tried to remove as much as i saw. theres def still more that can be removed but isnt yet
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

redoing this with a diff content PR


irmin: fix structure of top level documentation

Fixes a stub heading in the top-level docs and improves the ordering somewhat.

Thanks!