Discrete vs Continuous quantities

@bherd-rb Some tradable resources have quantities that are naturally discrete whilst other tradable resources have quantities that are natural continuous.  I have been trying to come up with a solution to get this information into the type system.  I think that the Wurman et al (2001) paper provides a solution.
Suppose that we have a trait Quantity that extends AnyVal and is parameterized on the type of value it contains.  Then we can define two case classes for DiscreteQuantity and ContinuousQuantity as follows.
sealed trait Quantity[+T <: AnyVal] extends AnyVal {
  def value: T
}

case class DiscreteQuantity(value: Long) extends Quantity[Long]

case class ContinuousQuantity(value: Double) extends Quantity[Double]

Thoughts?

@bherd-rb There are some issues with the above implementation. First, one can not extend from a trait/class that extends from AnyVal nor can one mixin a trait with a class that extends from AnyVal.  More details on this issue can be found on SO.  This means that if we move forward with this implementation we must accept that additional memory overhead associated with allocating instances of the Quantity objects which was saved by the use of value classes.  Second I would prefer to restrict the type of T to be any numeric value rather than the more generic AnyVal but at present this it is not possible to do this.  I did learn about the specialized annotation that should help improve run-time performance of calculations using quantities.
Here is the correct implementation...
  sealed trait Quantity[@specialized(Long, Double) +T <: AnyVal] {

    def value: T

  }

  final case class DiscreteQuantity(value: Long) extends Quantity[Long]

  final case class ContinuousQuantity(value: Double) extends Quantity[Double]

@bherd-rb I think I managed to find a way to restrict the type T to be any type that can be converted to a Double using implicit conversions...
sealed abstract class Quantity[@specialized (Long, Double) +T](implicit ev: T => Double) {

  def value: T

}

final case class DiscreteQuantity(value: Long) extends Quantity[Long]

final case class ContinuousQuantity(value: Double) extends Quantity[Double]

However, this is having this change in type impacts the order types in non-trivial ways that I am trying to sort out.

@bherd There is now a PR #15 that implements some of the ideas discussed above.


go: Add whether a device is the current device to DeviceHistoryListRPC

r? @patrickxb
(Also some spacing cleanups and a typo.)

LGTM w/ one change.


Not Updated for New Commerce Experience

[Enter feedback here]
This page and the instructions do not seem to apply to the "new commerce experience". The options to manage "add-ons" in a parent subscription is no longer available.

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

ID: b31aa550-3d84-95b8-6a74-b3e7ad1cbfe9
Version Independent ID: 795391d3-aced-e2f3-f52d-034e14fe1aee
Content: Add more SharePoint storage to your subscription
Content Source: microsoft-365/commerce/add-storage-space.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

[Enter feedback here]
This page and the instructions do not seem to apply to the "new commerce experience". The options to manage "add-ons" in a parent subscription is no longer available.

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

ID: b31aa550-3d84-95b8-6a74-b3e7ad1cbfe9
Version Independent ID: 795391d3-aced-e2f3-f52d-034e14fe1aee
Content: Add more SharePoint storage to your subscription
Content Source: microsoft-365/commerce/add-storage-space.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@WhiskeyTFox Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10185
Once the author approves, the changes will be live.
Thanks
Sri

@WhiskeyTFox Closing as per author's Input. Thanks


Update bot.invalid_photo_extension to Hindi

Type: Feat
Subject: Translate bot.invalid_photo_extension to Hindi

@Salz0 FYI I have no idea whether this translation is OK, so I would assume it it, unless someone speaking Hindi proves otherwise.


WV-106: Add map point filters for source_code and WV-107: Add way to show/remove map points based on their source_code

Took code from https://github.com/wevote/WeVoteServer/pull/1925/files, modified it so it uses GOOGLE_MAPS_API_KEY from environment_variables.json.

👍


Prod Data Migration - add 2 new regulated products

Describe the task
IRC has requested that we add 2 new regulated products to our database because an external user in Reg 1 needs to select them from the dropdown of regulated products.
Acceptance Criteria

[ ] create a new data migration to add the following to our table of Regulated Products:
Forged steel balls: less than 3.5 inches diameter
Forged steel balls: greater than 4 inches diameter

Additional context

once the migration has been created, will need to cut a release to prod, and then notify IRC (in particular TL)
Or here

Closing - already completed, just not released to prod yet


Update visual-studio-code to 1.14.0,b8cd5b9556d8b70ea560d35b903422363f6c5c40

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Thank you for your contribution.


Setting auto mode turns device off

First off, thanks for putting this together, this is super useful.
I tried it with my Classic 300S and everything seems to be working great except auto mode. When I set the device to auto via HomeKit the device turns off. It's not just that the mist turns off, the device shows up as off in the VeSync app. This happens regardless of what the target humidity is set to. Also, if I set the device to auto mode in the VeSync app, it shows up as being in manual mode in homekit.
Let me know if there's any other information I can provide that'd be helpful. I might try to dig into the code a bit if I have some time this weekend.
Thanks!

Thanks for the bug! I have also been seeing this but haven't had a lot of time to dig into it.
I should be able to take a look this weekend.

Can confirm it’s doing the same thing for me as well on the 300s

So far as I can tell and based on my limited familiarity with the Vesync API (I sniffed out the request from their app using Charles on my phone), the API call to the Vesync API looks good and right, and mode should be set to "auto" (and it is!) — gonna check and see if debug mode gives me anything interesting.

Thanks @kreeger. Still haven't taken a look, but that's very helpful. From your logs, it seems like switching to Auto is not calling changeMode, but setSwitch. I'll try to get to this in an hour or so

I've published this fix in<EMAIL_ADDRESS>Here's the PR: https://github.com/pschroeder89/homebridge-levoit-humidifiers/pull/4
The auto mode still can't be adjusted currently FWIW.

Works great, thanks!

seems to be working on my end as well. or at least it doesnt just shut itself off.


Fix design of engine_traits for mutual associated types / type bound equality

Part of https://github.com/tikv/tikv/issues/4184, abstracting the storage engine.
My most recent work in https://github.com/tikv/tikv/pull/6122 ran into a problem with the way associated types are used in engine_traits.
The basic problem is that multiple traits have different associated types which must resolve to the same type in generic functions, but I don't know how to make that happen. Here is a simplified example:
trait KvEngine {
    type Snapshot: Snapshot;

    fn get_snapshot(&self) -> Self::Snapshot;
}

trait Snapshot {
    type KvEngine: KvEngine;

    fn snap(&self, e: &Self::KvEngine);
}

fn do_thing<E>(e: &E) where E: KvEngine {
    let snapshot = e.get_snapshot();
    snapshot.snap(&e);
}

This doesn't compile because:
   |
15 |     snapshot.snap(e);
   |                   ^ expected associated type, found type parameter
   |
   = note: expected type `&<<E as KvEngine>::Snapshot as Snapshot>::KvEngine`
              found type `&E`

I don't know the idiomatic solution to this today.

I think Snapshot should not have the engine assoc type - assoc types should be a DAG, IMO.
You can then do something like:
trait KvEngine {
    type Snapshot: Snapshot;

    fn get_snapshot(&self) -> Self::Snapshot;
}

trait Snapshot: Sized {
    fn snap<K: KvEngine<Snapshot = Self>>(&self, e: &K);
}

fn do_thing<E>(e: &E) where E: KvEngine {
    let snapshot = e.get_snapshot();
    snapshot.snap(e);
}

You might need some intermediate bounds too - like a trait which is a sub-trait of KvEngine and abstracts over the things Snapshot requires from a KvEngine.

How about
fn do_thing<E>(e: &E)
where
    E: KvEngine,
    E::Snapshot: Snapshot<KvEngine=E>
{}

I'm not sure what we want to with fn snap(maybe create a new snapshot?), though I think we can change KvEngine to a genic parameter.
trait KvEngine {
    type Snapshot: Snapshot;

    fn get_snapshot(&self) -> Self::Snapshot;
}

trait Snapshot {
    type SyncSnap;

    fn into_sync(&self) -> Self::SyncSnap;

    fn snap<E: KvEngine>(&self, e: &E)
    where
        Self: Sized;
}

fn do_thing<E>(e: &E) -> Box<dyn Snapshot<SyncSnap = ()>>
where
    E: KvEngine,
    E::Snapshot: Snapshot<SyncSnap = ()> + 'static,
{
    let snapshot = e.get_snapshot();
    snapshot.snap(e);

    Box::new(snapshot)
}

Thanks for the suggestions. I'll do some experiments with these ideas and report back.

I've addressed this in https://github.com/tikv/tikv/pull/6179 by turning any "backward"-referencing associated types into type parameters on the trait.


Test your code

Please reach a higher level of code coverage through unit tests

I have added some text.... still a lot to add....


How to get Single Table Inheritance (STI) models completely grouped under parent endpoint?

Hello,
It might sound like a regression (I found issues that seem to describe the behaviour I'm looking for as a "bug", and "fixed" it), but I'll ask about it anyway. I have an ActiveRecord::Model with STI (and a type attribute). Say it's an Animal model, with a complete zoo of types (Cat, Dog, what have you).
I would rather not have to declare a JR resource and controller for each type, and would like only one /animals endpoint in my API, to handle all operations (index + CRUD). I'd also prefer the resource type in the API's responses to always be animals, and have a simple type attribute that will expose what kind of animal we have.
Is there a way to do this?

I'll answer to myself, although I have no idea if this is good practice or not (kinda looks gross to me).
class Api::AnimalResource < JSONAPI::Resource
  include Api::CentralizedSti
end

module Api::CentralizedSti
  extend ActiveSupport::Concern

  included do
    attribute :type
  end

  module ClassMethods

    def resource_type_for(model)
      model = model.becomes(_model_class) if model.is_a?(_model_class)
      super(model)
    end

    private

    def check_reserved_attribute_name(name)
      # Do not warn about NAME COLLISION on the type attribute.
    end

  end

end

I'm also interested to know that solution.

The model_hint method you can call seems like a nice way to have one resource for STI.
So for your example something like:
class Api::AnimalResource < JSONAPI::Resource
  model_hint model: Cat, resource: :animal
  model_hint model: Dog, resource: :animal
end

I tried that, does not work. Causes the gem to look for AnimalResource::CatResource


Update AzureKeyVault arguments

Update argument names to reflect the ones given in the YAML snippet.
Also, mark runAsPreJob as optional.

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

@50Wliu Based on the task json here, I think the opposite it true. It looks like the descriptions don't match the updated task.
Closing the PR and will update the task descriptions.

Just to follow-up. The issue was that we had documentation for V1 and V2 of the task in the same article. I'm adding an update now. Thanks for bringing this to our attention!


Remove unused graphql code

Fixes: https://github.com/Granola-Team/mina-indexer/issues/262

Closing for now. Will retry later.


Add vl53l0x driver - finish work by Afp316 #291

This just finishes off the fine work by @afp316 in #291
Rebased off of latest main, amended last commit to remove .pio/.vscode changes, ran clang-format.

Build failing due to https://github.com/adafruit/Adafruit_MQTT_Library/issues/217

Not sure how to retrigger workflow, but just read about hash sign-off so here goes:
#sign-off

I'm not familiar with #sign-off...
On the workflow, https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/3451873666 you should be able to re-run jobs?

Sadly not available, I think there's a repository or Org setting for allowing workflow runs from pull requests / forks. This was related https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#enabling-workflows-for-forks-of-private-repositories

-Wmissing-field-initializers is a warning being treated (correctly) as an error with the ESP32 compiler. Please open an issue over on the Adafruit vl53l0x repo and tag me and I'll (or an engineer from adafruit) will fix it.

It did pass, but the build warnings were still thrown: https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/3473545620/jobs/5805679630#step:8:73
I'm not sure why at the moment..

Also how should it be fixed, does the .h file that defines the struct get defaults specified, or does on define only on each initialisation

When a struct is initialized, it must contain all members within the brace-enclosed list of initializers

https://github.com/adafruit/Adafruit_VL53L0X/blob/master/src/vl53l0x_def.h#L220

Is it ever okay to just disable that warning for a line like that initialisation?

Not if we can fix it :)

So there was a missing bit in wippersnapper related to the setting of the time of last poll for the proximity sensor type.
Now just another initialisation warning that I'll stick up an issue for or fix eventually.
Not really happy with the reliability of the sensor in default config so I've selected high accuracy, but may experiment with other libraries too. The Dev flag in components with the banners is great :)

@tyeth  - OK, addressed in https://github.com/adafruit/Adafruit_VL53L0X/pull/59 and released in https://github.com/adafruit/Adafruit_VL53L0X/releases/tag/1.2.1. Arduino Lib manager will pick this up (probably later today) and I will re-kick the CI.
Thanks for being patient.

@tyeth This is failing clang/doxy (https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/3517128928/jobs/6038336339). Should be ready to go after. Thanks for your patience. I've also fixed the flaky behavior for building the ESP32-Sx platforms via the CI.

Wow green lights across the board, a more rare and pleasant sight that we might like to admit 📉
Congratulations on the CI work🥇 Just done the doxygen fix so that should be it!

Nice job and thank you both for your hard work and patience.
I couldn't figure out the Clang and doxygen stuff.
On Thu, Dec 1, 2022 at 10:21 AM Tyeth Gundry @.***>
wrote:

Wow green lights across the board, a more rare and pleasant sight that we
might like to admit 📉
Congratulations on the CI work🥇 Just done the doxygen fix so that should
be it!
—
Reply to this email directly, view it on GitHub
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/pull/363#issuecomment-1333930137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXAAGVQNQKJ2EICXDJYOQF3WLC7AVANCNFSM6AAAAAAR6NF2JQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great work - thank you both. Will be included in next release.


python3.pkgs.class-doc: patch out poetry dependency

Description of changes
The original postPatch to replace poetry with poetry-core left out one place where it needs to be patched so that we can turn on stronger build dependency validation using build. I decided to create a patch upstream (maybe it will be merged) and pull it in.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 248850 run on x86_64-linux 1

6 packages built:

octoprint
octoprint.dist
python310Packages.class-doc
python310Packages.class-doc.dist
python311Packages.class-doc
python311Packages.class-doc.dist


Composer problem

Run composer require encore/laravel-admin "1.3.x-dev" for Laravel 5.3, got this line:

Failed to download encore/laravel-admin from source: Failed to clone https://github.com/z-song/laravel-admin.git, git was not found, check that it is installed and in your PATH env.
'git' is not recognized as an internal or external command,
operable program or batch file.

composer use git to pull the package from github when install development branches under, so please make sure git client was installed.

@z-song thanks!
By the way, what version of laravel that you use on demo? Laravel 5.3? How can I manage route in Laravel 5.3? You put it on routes.php or routes/web.php?
Because I got problem with routes in Laravel 5.3, everytime I access Role, and click to Users page, apps redirect into login page. This happen everytime I switch pages. I'm pretty sure I logged on.

my demo is under laravel 5.3.
you can find directory app/Admin, all admin routes are puts on app/Admin/routes.php , and controllers under directory app/Admin/Controllers.

@z-song
I think you should update the docs, and add repo for the demo version. :)
Thank you!

ok， I will write english documentation lately 😝.

I installed but when i try login it's giving an error object not found. Can you include full documentaion. Thanks

@gudyosh Please show me more error information?

@z-song
Object not found!
The requested URL was not found on this server. The link on the referring page seems to be wrong or outdated. Please inform the author of that page about the error.
If you think this is a server error, please contact the webmaster.
Error 404
test.uz
Apache/2.4.16 (Win32)

@z-song
login page not working  He did not even checked properly I entered login and password

@z-song
it's giving me this error if i even leave this fields empty

@gudyosh You mean you see this error after you submit the login form?

@z-song
i'm using laravel 5.3 please help

The error was thrown by Apache but not Laravel, I think maybe your Laravel not installed correctly with Apache.

yes after submit

can you include documentation of installation process step by step Thanks

and when i type php artisan admin:make UserController --model=App\User
i'm getting error model does not exists!

@gudyosh try:
php artisan admin:make UserController --model=App\\User

So this are steps  witch i done:
1 composer require encore/laravel-admin "1.3.x-dev" then
2 Inconfig/app.phpaddServiceProvider:
i added Encore\Admin\Providers\AdminServiceProvider::class
3 php artisan vendor:publish --tag=laravel-admin
4 php artisan admin:install
then i'm opening http://test.uz/public/admin and i redirected to http://test.uz/public/admin/auth/login

@gudyosh Your server root should be http://test.uz/, not http://test.uz/public
And the admin root is http://test.uz/admin
see https://www.codementor.io/magarrent/tutorials/how-to-install-laravel-5-xampp-windows-du107u9ji and https://laravel.com/docs/4.2#pretty-urls

@z-song
thank you problem solved


Multiuser and simultaneous requests

Congrats on great project! I started playing with it and have two questions so far:

Does it support different sessions and several users?
Does it support simultaneous requests for inference? Like two users on different browsers or computers opened web ui from the same server and prompted at the same time - what happens? Will there requests be threaded and inference happens simultaneously for both of them or do the one who was a bit faster just waits for server to respond to the other one first?

I also saw a binding which name assumes that I can connect this web ui to other lollm server - does it allow multiple inferences happening on the same server then?

Hi, lollms is a mono-user project.
You can use multiple users if you install lollms (not lollms-webui), which creates a server for any client who wants to generate images. if you install it on a pc, you can use it from a remote PC or mac using lollms remote nodes.

Thank you for your fast answer! To make lollms multi-user and server concurrent requests should I use some separate web server like Apache / Nginx or is Flask default web server is enough for this? Also, can different users request for generation with different models or all users need use only one particular model which is selected server-side only?

Hi and sorry for being this late. I'm very very buzzy and lollms is taking me a huge chunk of my night life.
To use lollms in remote mode, you need to install lollms on a server, and run lollms-server --host <IP_ADDRESS> which will start a lollms service on the PC at you ip address:9601.
Then you install lollms-webui on one or multiple PCS, each PC has its own local database, the server don't store your requets. in the bindings you select lollms remote nodes and you go to its settings and add http://ipaddress:9600 to the hosts list (you can have multiple hosts by the way).
For now, the server should be configured via lollms-settings command that will allow you to select the binding, model and mount as many persnoas as you want.
The users of the webui use the service with the selected model. technically you can run multiple services with different models but that may require you to have big ressources in terms of GPUs.
you can also do this remotely from another place or share servers with friends as the lollms remote nodes supports multiple servers. You can create a network of generators/clients and there is a queuing mechanism in case more queries than servers.
I hope this answers your question.


fix: link to create notion api key

Description

Fix the link that allows creating an integration to generate the Notion API key.
Remove unnecessary spaces in README example files

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

You have signed the CLA already but the status is still pending? Let us recheck it.

thanks for this, fixed in https://github.com/makenotion/notion-sdk-js/commit/8ae90b034652dd3b637a3f0879961ac6fa5ed02a so closing for now.


Notebook doesn't restore correctly after discard from Git

Testing #122033

change note book
save
go to git
discard the change

Observe: the notebook doesn't update to the previous state

I see the notebook flicker after the discard, so I suspect the UI is just not updating properly.

same cause as https://github.com/microsoft/vscode/issues/122328

/duplicate of https://github.com/microsoft/vscode/issues/122328


Verify product dependencies using the build env platform, vs. hardcoded macOS

Product dependencies are now verified using the buildEnvironment's platform, instead of the hardcoded macOS platform.
This builds on the work of #6732, now allowing cross-compilation of product dependencies 🎉
Motivation:
Let's say we're cross-compiling macOS -> iOS simulator, with a host system of macOS v13.
let package = Package(
    name: "SwiftFlashlight",
    products: [.library(...)],
    platforms: [.iOS(.v16)],
    dependencies: [
        // internally, for iOS(.v16), macOS(.v14)
        .package(url: "probably/alamofire", exact: "1.2.3")
    ],
    // ...
)

As long as we set our triple & SDK, we should be good... (and, with #6828, only the triple needed)
> swift build --triple arm64-apple-ios-simulator \
              --sdk "$(xcrun --sdk iphonesimulator --show-sdk-path)"
...
error: the library 'SwiftFlashlight' requires macos 10.13,
but depends on the product 'probably/alamofire' which requires macos 10.14;  [...]

Even though SwiftFlashlight only supports iOS(.v16), SupportedPlatform "[by] default, [...] assigns a predefined minimum deployment version for each supported platforms unless you configure supported platforms [...]".  As a side-effect, macOS is inferred as a valid platform. (This is why no "target/platform mismatch" error was thrown.)
Previously, product dependencies were verified against macOS, rather than the target platform. Since macOS is always inferred, this surfaced as a "woah! your dependencies are out of sync".

let swiftFlashlight = Package(
    platforms: [
        .iOS(.v16),
        .macOS(.v13) // <-- inferred-|
    ],                               |
)                                    |-- // since --triple is .iOS,
                                     |   // shouldn't be comparing
let probably/alamofire = Package (   |   // macOS.
    platforms: [                     |
        iOS(.v16),                   |
        macOS(.v14)] // <-- inferred-|
    ]
)

Now, the intention of --triple is respected 🐱
Modifications:

validateDeploymentVersionOfProductDependency accepts a buildEnvironment containing the target platform

Result:
Product dependencies can now be compiled for darwin other than macOS -- i.e, iOS :)
Future work

Target-level or product-level platform support is still up in the air
This only applies to triple.isDarwin() -- perhaps a next step of, verifying dependencies against non-darwin?

This was a fun one to chase down 😅
(note to self: Apple Swift version 5.9 (swift-5.9-RELEASE))
Some questions:

I tried running swiftformat but it changed 600+ files -- just BuildPlan was ~300 lines. Thoughts?
Happy to add some tests around this; wanted to get some thoughts before diving in there :)

I tried running swiftformat but it changed 600+ files -- just BuildPlan was ~300 lines 😵, thoughts?

Please run Utilities/soundness.sh script, which will only format files that were changed.

Happy to add some tests around this; wanted to get some thoughts before diving in there :)

Yes, please add the tests, that would be great. Thanks!

Thanks! I'll take a crack at it soon :)

@MaxDesiatov Apologies for the delay on this! One test added, one test modified 🧪
Since utilities/soundness.sh works from the git diff, I added a newline in each file I modified + ran the script as a separate commit for readability. As you can see, those files haven't had a run in a while, heh.

Thanks! Just as heads up, this will conflict with https://github.com/apple/swift-package-manager/pull/7161

Resolved conflicts 👍
Looks like soundness.sh disagrees with how Swift requires raw string literals to be formatted tho. One example:
// BuildPlanTests L#5749
func testArtifactsArchiveBinaryTargets(...) {
    // ...
                        "path": "all-platforms/mytool",
                        "supportedTriples": ["\(
                            artifactTriples.map(\.tripleString)
                                .joined(separator: "\", \"")
    )"]
                }
            ]
        }
    }
}
                    """
                //   ^ swiftformat throws this here,
                //     but is a swift build error :(

I've manually reverted the script's raw string changes for now, but it seems like the way swiftformat expands inline closures (among other things) could be improved -- that's my guess 🚀
Also the soundness.sh script didn't print any output unless I tweaked L60. Assuming it's a me-issue, but can push up that change if useful.
swiftformat $changed_file > dev/null 2>&1 # old
swiftformat $changed_file 2>&1            # new

@swift-ci test

@swift-ci test windows

@swift-ci test

@swift-ci test windows


[New game]: Car Race Trial

🎮 Game Request
The player will drive their car as long as they can without running into obstacles to try and get a high score.
Point down the features
Points will be determined by how far the player makes it without hitting any obstacles.
Select program in which you are contributing
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Deegee13244 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#3044

Hey @Deegee13244 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Deegee13244 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


Support for a min-height

I was wondering if you'd be interested in supporting a min-height setting? Instead of completely closed and completely open, it would not collapse further than the min-height if that property's been set.
So for example:
{{#cp-panel is-open=true min-height=50px}}

I think you could achieve this by overriding the template for cp-panel-body. Check the template out here, and just write your own, using {{else}} to {{yield}} in a new element with class, and set your min-height on that element.
Let me know if that works, and if not we'll reopen. Thanks!

Ok, but wouldn't the body content only be rendered when the panel is open? A min-height would require it to be rendered when it's closed as well.

That's correct, so the current template is
{{#if isOpen}}

    {{yield}}

{{/if}}

You'd want your template to look like
{{#if isOpen}}

    {{yield}}

{{else}}

    {{yield}}

{{/if}}

Something like this, anyway. Does that make sense?

I understand how I'd override the template if this were one of my own components. But I don't see how I can override one of your addons templates? I might be misunderstanding, but after installing the addon, I wouldn't find its templates in my app folder, they be somewhere in node_modules right?

Nope if you just create a template for a component named cp-panel-body it will prefer that. That's how ember cli's resolver works, when you render {{cp-panel-body}} Ember CLI will first look for that template in your application dir, then up the addon chain.
It makes extending addon components super easy. Try it out!

Dude, I had no idea. Thanks, I'll try that!

Closing for now, let me know if you need more help!


PocketFi issue

Hi today I got an issue when claim. It's show
Unable to click. CPU may not be fast enough. Let's come back in 1 hour.
But my CPU is very strong with 4 CPU/24GB ram
When restart the service it's working for a time

Hi,
The original issue was that PocketFi requires a very fast click cycle, it needs to carry out 4 actions in a fraction of a second or it will fail - hence the original error message.
However, they have now introduced some extra steps that need navigating before a claim can be made, and their site seems to be down at the minute. We'll have to wait until it comes back up before we can do something further.
Thanks.


Python 3.11 and new emscripten-wasm32 platform

For some reason, the RTD mamba environment creation fails on this PR, although it does works on the main branch and the build-environment.yml file has not changed in this PR.
It fails with a memory issue (turns out I also had an issue with micromamba locally with memory piling up)

Let's merge (probably make an alpha release) and test this more properly before the final

meeseeksdev backport to 0.x


Optionally ignore methods

Personally I have never named a spec file after the name of the method it’s testing. Perhaps I’m a minority?
With this PR, I can configure RSpec/FilePath with 'IgnoreMethods' => true and it will only look at the described class/module name when recommending a path for this spec file.

This looks great @bquorning thanks for doing this. Can you add documentation for this config option in the class documentation? Sort of like this cop's docs: https://github.com/nevir/rubocop-rspec/blob/master/lib/rubocop/cop/rspec/hook_argument.rb#L13-L59

Can you add documentation for this config option in the class documentation?

Sure, I’ll do that.

Personally I have never named a spec file after the name of the method it’s testing.

By the way, while I don't do this all of the time, I would recommend it. When you limit yourself to one describe per file (and you try to only test through one public method per describe) I think having per-method spec files helps reenforce some pain you should feel when your public API for an object is getting fat

When you limit yourself to one describe per file (and you try to only test through one public method per describe) I think having per-method spec files helps reenforce some pain you should feel when your public API for an object is getting fat

Valid point. But the pain of managing a single, 300-line spec file also tells me that my public API is too large :-) If I have a class with a small public API – 3 methods, say – wouldn’t it be preferable to test them in one file instead of in three different files?

By the way, I have fixed the documentation in the class.
One thing I forgot to mention: While 'IgnoreMethods' => true allows naming the spec file only after the class, it doesn’t enforce it. This is because of the * in the generated regex.

If I have a class with a small public API – 3 methods, say – wouldn’t it be preferable to test them in one file instead of in three different files?

Yeah this is why I don't require myself to follow a hard rule for this. Basically, if I'm writing more than two or three tests for a method, I'll consider moving it to a file. Otherwise small class can stay in one file

Thanks @bquorning. I've rebased this branch on top of master and merged them

Rebase and merge was in #202. I did the rebase myself since we're in different time zones and it is easier this way to manage the this way. Let me know if you're interested in joining as a collaborator in the future. That way you could just push to a branch on this repo and I could rebase directly

Rebase and merge was in #202. I did the rebase myself

No problem – and thank you.

Let me know if you're interested in joining as a collaborator in the future.

Sure.

I did the rebase myself since we're in different time zones and it is easier this way to manage the this way.

Just Almost in time: https://github.com/blog/2247-improving-collaboration-with-forks

haha


Fix Issue with multiple calls to close() in progress_window.py

As it is now if close() is called more than once per show (or called
before show) some of the class attributes will not exist during the
following calls to close. Added check to see if self.window has been
setup to detect a previous call to close().  This seems to be happening
in: hsoft/dupeguru#449.

Hum, ok, why not? It would be nice to find the root cause of the double-call though.

Yeah I'll spend some more time trying to reproduce it, does not happen all the time.


Add a more advanced host example

It took me some time to get this to work. (And I'm not sure if that's the intended usage.) Adding a host example which utilizes AudioBuffers and (Midi)Events would make this aspect of the project more approachable.

Seems to be a great start at an example! It will be important to find a way to make it platform agnostic though. The most obvious approach in that case might be to have it load an example plugin and use that. The hard part would be the bundling the plugins on OS X. But if we could use the bundler script from a build.rs it could be done.

Thank you for your example, I have tested it on OS X, but it segfaults.
Does anyone on OS X has the same experience?

It's quite possible that my example isn't doing stuff the right way(tm) but I did test it on Windows and Linux. Which VST did you try? I've tested Pianoteq 5 and sine_synth from the examples.

After tracing what https://github.com/teragonaudio/MrsWatson is doing, I think I know the problem:
OpCode::GetTime isn't handled here and thus 0 is returned. (MrsWatson handles the opcode like this.)
But Spire expected a pointer to a VstTimeInfo (from the VST2 SDK) and tries to read a field from the struct causing the segfault.
(Dr. Memory reports UNADDRESSABLE ACCESS: reading 0x0000000000000019-0x0000000000000021 8 byte(s))
Somebody would need to clone the struct definition to Rust but I'm not sure if handling timing should be in scope of this library. (I tend to side with no but we'd then need a way to delegate that request to the library user.)
In case anybody else wants to further investigate:
I used mrswatson64 --midi-file c-scale.mid --output out.wav --plugin $PATH_TO_DLL --verbose.
MIDI file: https://my.mixtape.moe/deikbd.mid (MrsWatson only supports Type 0 MIDI files; Type 1 support is a PR)
Partial output (OpCode::GetTime is 7):
Processing audio with plugin 'Spire-1.1.dll'
Plugin 'Spr2' called host dispatcher with 7, 0, 512
Current PPQ position is 4.85451
Plugin 'Spr2' called host dispatcher with 7, 0, 1024
Plugin 'Spr2' called host dispatcher with 7, 0, 1024
Plugin 'Spire-1.1.dll' spent 3ms processing (0% effective CPU usage)
Wrote 1024 samples to PCM file

@phonkee Perhaps #32 will fix your problem.

Thank you, this is awesome. I have tried other vst and it works. My actual target vst is Virus TI. I will try it in next days.
It's multi output vst that controlls Virus hardware synthesizer, I will share my experience then..
Lastly I want to thanks all people involved in this awesome project and I hope that I will be able to contribute in the future.
Thank you and Merry christmas to you all!

Merry xmas and happy holidays to you as well!  Hope it works out.  Feel free to join our telegram group if you'd like to get more involved.

Host example that shows how to output to speakers real-time would be super helpful, since you can't just send PluginInstance to audio thread.


chore: revert arange test

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant and/or add your own.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

Seems like theres still issues
FAILED conversion/test_arange_aten.py::TestArangeConverter::test_arange_dynamic - RuntimeError: aten::arange() Expected a value of type 'number' for argument 'end' but instead found type 'Tensor'.
Position: 1
Value: tensor(7, device='cuda:0')


Use a dagger core command as readiness probe

Since https://github.com/dagger/dagger/pull/6100#issuecomment-1809271407 that using buildctl as the healthcheck for the engine is essentially a no-op. There is an on-going discussion here of what the ideal healthcheck for the engine would be, we haven't yet reached a conclusion but now that we have the dagger CLI in the engine image and the core subcommand we can use something more suitable on our helm chart. I don't think this should be considered as "the way" to check for engine readiness, but its the best I could come up with and what we were doing was already broken.
Internally we were doing something similar to this already, but taking care of downloading the CLI and using dagger query. Now with the CLI in the engine image and the core subcommand we can simplify it and officially release it here.
There were many possible dagger core commands I could use. I chose one that would force the engine to do work but not depend on external registry or networks.

Potentially worth a changelog entry? Up to ya :tada:


voting UI design

Most of the basic functions are finished by now, so we should be able to start planning the voting functions.
The first question is how are we gonna design the UI for the votings? It should be as easy as possible to vote and see the results.
If you got any ideas, please post them as a comment.

I think, it should work something like this:
In the group

As an admin or moderator, use the command voting to make a voting. Add "<Voting description>" behind the command to add a description to the voting.

In the private chat with the bot

Use the command currentVotings to see which votings are currently running. The bot will give you a list with the IDs and the descriptions of the votings.
Use the command vote <id> <yes/no/abstention> to vote. (Maybe you could also say which answer options are available for the voting when making it)
Use the command voteInfo <id> to see the current status of the results of the vote you're choosing in a graph.

I think, that it is much more pleasant for the group members if you let the bot do as few as possible important things inside the group and let it communicate directly with them. Maybe it would also be good to let the bot remind you to vote like one day before the voting ends.
After the voting has ended, it sends a message into the group chat and shows the results in a graph.

That seems like a got idea

Do you mean a „good“ Idea? <(˶ᵔᵕᵔ˶)>


Fix use internal vars in async operation

Fixes #148

Codecov Report

Merging #162 into master will increase coverage by 0.44%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   89.07%   89.52%   +0.44%
==========================================
  Files          35       35
  Lines        2233     2233
==========================================
+ Hits         1989     1999      +10
+ Misses        244      234      -10

Impacted Files
Coverage Δ

Sources/RequestOperation.swift
83.65% <88.88%> (+9.61%)
: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 8431e75...ac2acbc. Read the comment docs.


Restrict RC path to a definite 31 chars

Fixes DataDog/dd-trace-php#2892.

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

Please upload report for BASE (main@92272e9). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #676   +/-   ##
=======================================
  Coverage        ?   71.78%
=======================================
  Files           ?      271
  Lines           ?    40979
  Branches        ?        0
=======================================
  Hits            ?    29415
  Misses          ?    11564
  Partials        ?        0

Components
Coverage Δ

crashtracker
20.41% <ø> (?)

crashtracker-ffi
9.07% <ø> (?)

datadog-alloc
98.73% <ø> (?)

data-pipeline
91.92% <ø> (?)

data-pipeline-ffi
0.00% <ø> (?)

ddcommon
82.73% <ø> (?)

ddcommon-ffi
69.12% <ø> (?)

ddtelemetry
59.10% <ø> (?)

ddtelemetry-ffi
22.13% <ø> (?)

dogstatsd
88.86% <ø> (?)

dogstatsd-client
79.77% <ø> (?)

ipc
82.65% <ø> (?)

profiling
84.26% <ø> (?)

profiling-ffi
77.42% <ø> (?)

serverless
0.00% <ø> (?)

sidecar
36.09% <100.00%> (?)

sidecar-ffi
0.00% <ø> (?)

spawn-worker
50.36% <ø> (?)

tinybytes
94.77% <ø> (?)

trace-mini-agent
72.45% <ø> (?)

trace-normalization
98.25% <ø> (?)

trace-obfuscation
95.73% <ø> (?)

trace-protobuf
77.67% <ø> (?)

trace-utils
93.07% <ø> (?)


Larger Map Size - Gas Doesn't Dissipate

TL;DR: With just Harmony, SCGF, Knockout Gas loaded, gas lingers forever on maps where both axis are >= 400
Was having an issue with gas not disappearing, and after narrowing 400+ mods down to nothing, I eventually discovered it was an issue between SCGF and my map size.
Tested with just Harmony, SCGF, and Knockout Gas enabled.
To replicate:
Settings > Dev > "Enable test map sizes"
New Game
Planet Screen > Advanced > 400x400 Map Size
Start game, spawn launcher, launch gas at colonist
4x and wait
You'll see that some gas disappears, but not all of it.  The gas lingers forever. While on a vanilla 400x400 with just base + DLCs loaded, the rot stink disappears within seconds.
Oddly enough, the other debug map size (350x350) works fine. So I did some further testing with Better (custom) Map Sizes. 399x399 worked fine. 400x399 worked fine. 1000x399 even worked, so it's not a cell count issue. Seemed to take longer to dissipate, probably because of TPS. But as soon as both axis are >=400, the gas just won't disappear.
Apologies if this is an issue on my end, but I went as far as temporarily clearing Rimworld's AppData for a clean boot.
 The current area of effect (which, also seems smaller on larger maps but maybe I've gone crazy trying to figure this all out lol) makes them kind of ineffective, and requires you to spam them around if, for example, you needed to fumigate a cave base for roaches. Also ideas: floor vents, tank + pipe network for filling up 1 location to distribute where needed.
All that being said, I do like the mod for what it does and prefer it over its alternatives. Appreciate the work you've done on it and the effort to keep it performance-friendly. Thanks for your time.

Thanks for making this issue, as well as providing step-by-step instructions to reproduce it! I'll get around to hopefully fixing this over the weekend sometimes, seems very strange how different map sizes are affecting gas behaviours.
Pipe networks aren't on the todo list atm since that would be quite complicated, but there have been talks about merging this framework with a certain other one that has pipe networks as well ;)
Vents being underpowered can be addressed however, I'll look into doings some polls later down the line.

Yeah this one is really weird, I've done some testing and it seems some of the gas tiles dissipate normally (although I think slower than they would on a normal map size, again super weird stuff going on here), while other tiles don't, becoming stuck at either 100% or 20% density.

My immediate thought is that this could be due to floating point rounding errors, as the ExtendedGasGrid.Tick() method gets cells in a random order to tick, however this behaviour is 1:1 with what vanilla does, plus you said it works fine on larger map sizes, so this is unlikely to be the cause.
https://github.com/NachoToast/SimpleCustomGasFramework/blob/e3241c31a86cf7bf519d2c6618c251758c7eb03d/Source/SCGF1.4/ExtendedGasGrid.cs#L215-L217
I will keep working on solving this issue, but it is quite low down on the priority list due to its obscure and relatively less-impactful nature.


Selecting multiple folders to add to workspace produces "The folder name is not valid"

VSCode Version:  1.18.0
OS Version:  Windows 10 14393.1770

Steps to Reproduce:

Open a New VSCode Window.
Select Add Folder to Workspace
Select multiple folders and click Add

Reproduces without extensions: Yes

@TaylorBarrick I cannot reproduce, it must happen with a specific folder you pick. Does it happen always? Which folders do you pick?

I have tried on a different machine and did not experience this issue,  However, on a work laptop, I am getting this regardless of the folder selection.  I have tested with deep and small paths and paths without spaces on my laptop and it is still the same.  I have also tried running code as administrator, but no luck.

I also tried to reproduce on Windows now to see if that makes any difference and could not. I will close this issue until I get steps to reproduce.


cherry-pick：Rename operator for 0.3.0

What type of PR is this?
/kind cleanup
What does this PR do?

Which issue(s) does this PR fix?

Fixes #407
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/close


Insert / Copy / Delete buttons

These buttons should be left aligned above the treeview, and not move if the panel is resized.

Fixed in new design


python311Packages.glean-parser: 13.0.1 -> 14.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.glean-parser is: Tools for parsing the metadata for Mozilla's glean telemetry SDK
meta.homepage for python311Packages.glean-parser is: https://github.com/mozilla/glean_parser
meta.changelog for python311Packages.glean-parser is: https://github.com/mozilla/glean_parser/blob/v14.0.1/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 14.0.1 with grep in /nix/store/fkw25a9vcdha77hbgclfahsjyrzknks0-python3.11-glean-parser-14.0.1
found 14.0.1 in filename of file in /nix/store/fkw25a9vcdha77hbgclfahsjyrzknks0-python3.11-glean-parser-14.0.1

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath

mozillavpn
mozphab
python311Packages.glean-parser
python311Packages.glean-sdk
python312Packages.glean-parser
python312Packages.glean-sdk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fkw25a9vcdha77hbgclfahsjyrzknks0-python3.11-glean-parser-14.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.glean-parser https://github.com/r-ryantm/nixpkgs/archive/34046dc7d86241596e92b2e89f3e480069af13c5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/34046dc7d86241596e92b2e89f3e480069af13c5#python311Packages.glean-parser

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fkw25a9vcdha77hbgclfahsjyrzknks0-python3.11-glean-parser-14.0.1
ls -la /nix/store/fkw25a9vcdha77hbgclfahsjyrzknks0-python3.11-glean-parser-14.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

python312Packages.glean-sdk
python312Packages.glean-sdk.dist

9 packages built:

mozillavpn
mozphab
mozphab.dist
python311Packages.glean-parser
python311Packages.glean-parser.dist
python311Packages.glean-sdk
python311Packages.glean-sdk.dist
python312Packages.glean-parser
python312Packages.glean-parser.dist

Maintainer pings

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.glean-parser is: Tools for parsing the metadata for Mozilla's glean telemetry SDK
meta.homepage for python311Packages.glean-parser is: https://github.com/mozilla/glean_parser
meta.changelog for python311Packages.glean-parser is: https://github.com/mozilla/glean_parser/blob/v14.1.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 14.1.0 with grep in /nix/store/r05qkiplh3fx8mh6xqavp6hw9mw8z94q-python3.11-glean-parser-14.1.0
found 14.1.0 in filename of file in /nix/store/r05qkiplh3fx8mh6xqavp6hw9mw8z94q-python3.11-glean-parser-14.1.0

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath

mozillavpn
mozphab
python311Packages.glean-parser
python311Packages.glean-sdk
python312Packages.glean-parser
python312Packages.glean-sdk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r05qkiplh3fx8mh6xqavp6hw9mw8z94q-python3.11-glean-parser-14.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.glean-parser https://github.com/r-ryantm/nixpkgs/archive/757561538c3c994901d64fb734013f6ec47d0ad5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/757561538c3c994901d64fb734013f6ec47d0ad5#python311Packages.glean-parser

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r05qkiplh3fx8mh6xqavp6hw9mw8z94q-python3.11-glean-parser-14.1.0
ls -la /nix/store/r05qkiplh3fx8mh6xqavp6hw9mw8z94q-python3.11-glean-parser-14.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

python312Packages.glean-sdk
python312Packages.glean-sdk.dist

9 packages built:

mozillavpn
mozphab
mozphab.dist
python311Packages.glean-parser
python311Packages.glean-parser.dist
python311Packages.glean-sdk
python311Packages.glean-sdk.dist
python312Packages.glean-parser
python312Packages.glean-parser.dist

Maintainer pings


Basic example with appendix

Can this plugin/extension help me add an appendix after the references for my paper?
I tried installing this extension but it didn't seem to do anything...
I'm using the IEEE format, if that matters

I tried installing this extension but it didn't seem to do anything...

I tested it with the official charged-ieee template and reproduced the issue you mentioned.
After diving into it, I find that it used a custom show rule on headings, which will overwrite the #set heading(numbering: xxx) rule.
I think the easiest way is to just write it manually.

#heading(numbering: none)[Appendix A\
Title of the 1st appendix]

Ahh that worked, thanks @flaribbit. Makes sense the template would interfere here.
Do you by chance know how to move the appendix past the references?

Ahh, nvm, I found out I just have to supply none to ieee.with function and just manually add the appendix after.
Thanks for the help!


Fix links in 0.5 announcement post

Part of fixing #450.

I think it makes sense to link to http://package.elm-lang.org/packages/elm-lang/core/1.0.0/Http for the HTTP stuff, otherwise the text says the wrong thing.

I think it makes sense to link to http://package.elm-lang.org/packages/elm-lang/core/1.0.0/Http for the HTTP stuff, otherwise the text says the wrong thing.

Okay, fixed that.

Okay, fixed that.

Cool, thanks!

Cool, thanks!


fix(background): probe even with same currency

Context

Part of https://github.com/interledger/web-monetization-extension/issues/394
Changes proposed in this pull request

probe even with same currency
ensure not to skip previous probing amount if temporarily failed to probe

Extension builds preview

Name
Link

Latest commit
5d5b8aff17a26941418911b0f773fa3356f53f67

Latest job logs
Run #10215248917

DownloadDownload


Remove GOV.UK VM stuff from the README.

Stagecraft was removed from govuk-puppet, so it no longer works with the GOV.UK VM.
Ref:

https://github.com/alphagov/govuk-puppet/pull/6720
https://github.com/alphagov/govuk-puppet/pull/6732
https://github.com/alphagov/govuk-puppet/pull/6742

Rebased to fix conflicts. Merging this as it's really old.


Update introduction.md

🚀 Deployed on https://6580cc65b771992fa04c7125--cozy-fox-0defec.netlify.app


docs: update nuxt config example

The current example does not work with the nuxt plugin, because the unstyled prop should be inside options: https://primevue.org/nuxt/#unstyled

Hi @atakantepe, is this repository open for contributions?


Unable to compile log 1.1.0-beta.2 module in beta1 due to API change in ballerina/observe in beta2

Description:
When compiling the log module released with beta 2 following error occurs
ERROR [ballerina/log/1.1.0-beta.2::natives.bal:(146:35,146:59)] undefined function 'getSpanContext'

This is due a new function getSpanContext added to observe module in beta2 release.
This will impact building a http service in beta1 once beta2 packages are released to central.
Steps to reproduce:
Create a http service and build with beta1
Affected Versions:
slbeta1 and slbeta2
[1] https://github.com/ballerina-platform/ballerina-lang/blob/master/observelib/observe/src/main/ballerina/natives.bal#L78

The dependency resolution of Ballerina should support adding a function (which is ideally a backward compatible change). Hence we cannot add back the logic (reverted in https://github.com/ballerina-platform/module-ballerina-log/pull/223) until it is supported.


GUACAMOLE-773: Update all Java dependencies to latest available stable versions.

This change updates all immediate Java dependencies to their latest stable versions. The only exceptions to this are:

"mybatis-guice" - this has a more recent stable release (3.12), but that stable release depends on an unstable and incompatible alpha release of SLF4J.
Libraries that are not bundled, but provided by the servlet container or JVM (Tomcat/Jetty versions of the legacy WebSocket API, various standard JSR annotations, the non-legacy WebSocket API, the servlet API, etc.)
Webjars (I'll handle updates to all JavaScript dependencies in a separate PR)

This change also adds the "maven-enforcer-plugin" to the build to help ensure that transitive dependencies are sane.
I have avoided overriding dependencies except where needed to:

Ensure differing versions of transitive dependencies do not clash
Ensure that we don't end up bundling a large variety of versions of the same utility libraries when versions are backwards compatible

Forgot to mention - these changes also add a new build property, ignoreLicenseErrors, which allows errors that occur during the RAT check and LICENSE generation to be bypassed for the sake of testing. For example:
mvn -DignoreLicenseErrors=true

This ended up being useful for testing things before having to commit to the upgrade, since committing to the upgrade means re-researching the license that applies to the dependency, verifying the copyright information we have is up-to-date, etc.

How extensively have you tested the changes, particularly for the various extensions?

So far, I have only tested against MySQL, MariaDB, PostgreSQL, and SQL Server.

Is that something you need help with? I can probably resurrect some of my old test environments that I used to develop things like SAML and CAS if that would help.

Yes, please. I have no current means to test RADIUS and CAS.
I can readily test LDAP, JSON, and TOTP, and should be able to set things up for SAML, OpenID, and Duo.

Well, off the bat seeing this in the RADIUS module as soon as I try to access the Guacamole logon page:
19:34:30.103 [http-nio-8080-exec-191] WARN  o.a.g.e.AuthenticationProviderFacade - The "radius" authentication provider has encountered an internal error which will halt the authentication process. If this is unexpected or you are the developer of this authentication provider, you may wish to enable debug-level logging. If this is expected and you wish to ignore such failures in the future, please set "skip-if-unavailable: radius" within your guacamole.properties.
19:34:30.144 [http-nio-8080-exec-191] ERROR o.a.g.rest.RESTExceptionMapper - Unexpected internal error: Unable to provision, see the following errors:

1) [Guice/ErrorInjectingConstructor]: LinkageError: loader constraint violation: when resolving method "org.slf4j.impl.StaticLoggerBinder.getLoggerFactory()Lorg/slf4j/ILoggerFactory;" the class loader (instance of org/apache/guacamole/extension/ExtensionClassLoader) of the current class, org/slf4j/LoggerFactory, and the class loader (instance of org/apache/catalina/loader/ParallelWebappClassLoader) for the method's defining class, org/slf4j/impl/StaticLoggerBinder, have different Class objects for the type org/slf4j/ILoggerFactory used in the signature
  at AuthenticationProviderService.<init>(AuthenticationProviderService.java:51)
  while locating AuthenticationProviderService

Learn more:
  https://github.com/google/guice/wiki/ERROR_INJECTING_CONSTRUCTOR
1 error

======================
Full classname legend:
======================
AuthenticationProviderService: "org.apache.guacamole.auth.radius.AuthenticationProviderService"
========================
End of classname legend:
========================

19:34:30.146 [http-nio-8080-exec-191] DEBUG o.a.g.rest.RESTExceptionMapper - Unexpected error in REST endpoint.
com.google.inject.ProvisionException: Unable to provision, see the following errors:

1) [Guice/ErrorInjectingConstructor]: LinkageError: loader constraint violation: when resolving method "org.slf4j.impl.StaticLoggerBinder.getLoggerFactory()Lorg/slf4j/ILoggerFactory;" the class loader (instance of org/apache/guacamole/extension/ExtensionClassLoader) of the current class, org/slf4j/LoggerFactory, and the class loader (instance of org/apache/catalina/loader/ParallelWebappClassLoader) for the method's defining class, org/slf4j/impl/StaticLoggerBinder, have different Class objects for the type org/slf4j/ILoggerFactory used in the signature
  at AuthenticationProviderService.<init>(AuthenticationProviderService.java:51)
  while locating AuthenticationProviderService

Learn more:
  https://github.com/google/guice/wiki/ERROR_INJECTING_CONSTRUCTOR

1 error

======================
Full classname legend:
======================
AuthenticationProviderService: "org.apache.guacamole.auth.radius.AuthenticationProviderService"
========================
End of classname legend:
========================

	at com.google.inject.internal.InternalProvisionException.toProvisionException(InternalProvisionException.java:251)
	at com.google.inject.internal.InjectorImpl$1.get(InjectorImpl.java:1103)
	at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1138)
	at org.apache.guacamole.auth.radius.RadiusAuthenticationProvider.authenticateUser(RadiusAuthenticationProvider.java:68)
	at org.apache.guacamole.extension.AuthenticationProviderFacade.authenticateUser(AuthenticationProviderFacade.java:190)
	at org.apache.guacamole.rest.auth.AuthenticationService.authenticateUser(AuthenticationService.java:174)
	at org.apache.guacamole.rest.auth.AuthenticationService.getAuthenticatedUser(AuthenticationService.java:300)
	at org.apache.guacamole.rest.auth.AuthenticationService.authenticate(AuthenticationService.java:453)
	at org.apache.guacamole.rest.auth.TokenRESTService.createToken(TokenRESTService.java:174)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.glassfish.jersey.server.model.internal.ResourceMethodInvocationHandlerFactory.lambda$static$0(ResourceMethodInvocationHandlerFactory.java:52)
	at org.glassfish.jersey.server.model.internal.AbstractJavaResourceMethodDispatcher$1.run(AbstractJavaResourceMethodDispatcher.java:124)
	at org.glassfish.jersey.server.model.internal.AbstractJavaResourceMethodDispatcher.invoke(AbstractJavaResourceMethodDispatcher.java:167)
	at org.glassfish.jersey.server.model.internal.JavaResourceMethodDispatcherProvider$TypeOutInvoker.doDispatch(JavaResourceMethodDispatcherProvider.java:219)
	at org.glassfish.jersey.server.model.internal.AbstractJavaResourceMethodDispatcher.dispatch(AbstractJavaResourceMethodDispatcher.java:79)
...
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method "org.slf4j.impl.StaticLoggerBinder.getLoggerFactory()Lorg/slf4j/ILoggerFactory;" the class loader (instance of org/apache/guacamole/extension/ExtensionClassLoader) of the current class, org/slf4j/LoggerFactory, and the class loader (instance of org/apache/catalina/loader/ParallelWebappClassLoader) for the method's defining class, org/slf4j/impl/StaticLoggerBinder, have different Class objects for the type org/slf4j/ILoggerFactory used in the signature
	at org.slf4j.LoggerFactory.getILoggerFactory(LoggerFactory.java:423)
	at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:362)
	at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:388)
	at org.apache.guacamole.auth.radius.AuthenticationProviderService.<init>(AuthenticationProviderService.java:56)
	at org.apache.guacamole.auth.radius.AuthenticationProviderService$$FastClassByGuice$$3893118.GUICE$TRAMPOLINE(<generated>)
	at org.apache.guacamole.auth.radius.AuthenticationProviderService$$FastClassByGuice$$3893118.apply(<generated>)
	at com.google.inject.internal.DefaultConstructionProxyFactory$FastClassProxy.newInstance(DefaultConstructionProxyFactory.java:82)
	at com.google.inject.internal.ConstructorInjector.provision(ConstructorInjector.java:114)
	at com.google.inject.internal.ConstructorInjector.construct(ConstructorInjector.java:91)
	at com.google.inject.internal.ConstructorBindingImpl$Factory.get(ConstructorBindingImpl.java:296)
	at com.google.inject.internal.InjectorImpl$1.get(InjectorImpl.java:1100)
	... 60 common frames omitted

I seem to recall having this issue before with the RADIUS module - seems like one of the jradius dependencies pulls in a different version of SLF4J if you don't specify the provided tag for that dependency in the pom.xml file.

Also, seeing a bunch of these type errors at startup (Tomcat 8):
26-Apr-2021 19:43:07.073 SEVERE [localhost-startStop-26] org.apache.catalina.startup.ContextConfig.processAnnotationsJar Unable to process Jar entry [module-info.class] from Jar [file:/opt/tomcat8/webapps/guacamole/WEB-INF/lib/jackson-databind-2.12.2.jar] for annotations
 org.apache.tomcat.util.bcel.classfile.ClassFormatException: Invalid byte tag in constant pool: 19
	at org.apache.tomcat.util.bcel.classfile.Constant.readConstant(Constant.java:97)
	at org.apache.tomcat.util.bcel.classfile.ConstantPool.<init>(ConstantPool.java:54)
	at org.apache.tomcat.util.bcel.classfile.ClassParser.readConstantPool(ClassParser.java:174)
	at org.apache.tomcat.util.bcel.classfile.ClassParser.parse(ClassParser.java:83)
	at org.apache.catalina.startup.ContextConfig.processAnnotationsStream(ContextConfig.java:2053)
	at org.apache.catalina.startup.ContextConfig.processAnnotationsJar(ContextConfig.java:2000)
	at org.apache.catalina.startup.ContextConfig.processAnnotationsUrl(ContextConfig.java:1970)
	at org.apache.catalina.startup.ContextConfig.processAnnotations(ContextConfig.java:1923)
	at org.apache.catalina.startup.ContextConfig.webConfig(ContextConfig.java:1163)
	at org.apache.catalina.startup.ContextConfig.configureStart(ContextConfig.java:775)
	at org.apache.catalina.startup.ContextConfig.lifecycleEvent(ContextConfig.java:299)
	at org.apache.catalina.util.LifecycleBase.fireLifecycleEvent(LifecycleBase.java:94)
	at org.apache.catalina.core.StandardContext.startInternal(StandardContext.java:5087)
	at org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:150)
	at org.apache.catalina.core.ContainerBase.addChildInternal(ContainerBase.java:752)
	at org.apache.catalina.core.ContainerBase.addChild(ContainerBase.java:728)
	at org.apache.catalina.core.StandardHost.addChild(StandardHost.java:734)
	at org.apache.catalina.startup.HostConfig.deployWAR(HostConfig.java:952)
	at org.apache.catalina.startup.HostConfig$DeployWar.run(HostConfig.java:1823)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Gah.
Well, the "Invalid byte tag in constant pool: 19" error looks like an issue with that particular Tomcat and Java 8, not necessarily these changes (though if this was working before, I'm not sure what's up there):
https://stackoverflow.com/questions/52867430/invalid-byte-tag-in-constant-pool-19-error-message
I'll double-check CAS and RADIUS for SLF4J - I remember seeing something like that when I tried mybatis-guice 3.12 and reverted to 3.10 when I realized the former was using 2.0.0-alpha1 and was thus incompatible with SLF4J 1.x.

OK - I've excluded SLF4J from inclusion in all extensions by declaring it as a provided dependency within the parent extensions project, and added an enforcer rule to check that no SLF4J dependencies are ever pulled into a scope that would result in them being bundled with an extension.
Now to figure out that "byte tag" error...

@necouchman - which specific version of Tomcat 8 are you using, and which JVM?

CAS and RADIUS modules check out fine, now - errors are gone and logins work.
Regarding Java/Tomcat, here is the version information:
openjdk version "1.8.0_275"
OpenJDK Runtime Environment (build 1.8.0_275-b01)
OpenJDK 64-Bit Server VM (build 25.275-b01, mixed mode)
Apache Tomcat Version 8.5.9

So, the "invalid byte" error is due to an incompatibility between the BCEL library (used by Tomcat for annotation scanning) and the Java version of the class being scanned. From https://stackoverflow.com/questions/52867430/invalid-byte-tag-in-constant-pool-19-error-message:

The BCEL library here is used to parse class files when Tomcat scans for annotations. The version that you are using does not support some features in class file format of Java 8, and thus fails when trying to parse the file.

This isn't entirely correct in our case. Our error deals with module-info.class, which is apparently a Java 9 thing:
https://openjdk.java.net/jeps/261
As far as I can tell, the jackson-databind library is built for Java 7, regardless of the fact it includes a module-info.class.
It may be possible to strip module-info.class from the bundled .jar files, though I don't know whether this would hurt anything:
https://stackoverflow.com/questions/56063566/maven-how-to-remove-module-info-class-warning-for-shaded-jar
It's also possible to stop the error by disabling annotation scanning, which involves:

Bumping the required version of the servlet API from 2.5 to 3.0
Adding metadata-complete="true" to the <web-app> element of web.xml
Adding an empty <absolute-ordering/>' element to web.xml`

...though I don't know whether that would hurt anything either.

Aha! The issue with module-info.class and BCEL is a known bug in older versions of Tomcat. From https://bz.apache.org/bugzilla/show_bug.cgi?id=60688:

Fixed in:

trunk for 9.0.0.M18 onwards
8.5.x for 8.5.12 onwards
8.0.x for 8.0.42 onwards
7.0.x for 7.0.76 onwards

@necouchman, assuming the BCEL error is non-fatal and the application otherwise deploys successfully, how would you feel about not attempting to avoid the error given that it's a known bug that has since been addressed by Tomcat? I'm apprehensive about updating the expected Java Servlet API from 2.5 to 3.0 and verifying that there are no side effects to disabling annotation processing.

@mike-jumper: Yeah, I'm fine with that - I can try to update Tomcat and see what happens, there, but no need for that to block this.
I assume this pull request is good, otherwise?

I believe so. I'll do a quick regression test of LDAP, etc. and make extra sure.

OK - I have now tested the following:

guacamole-auth-duo
guacamole-auth-header
guacamole-auth-jdbc-*
guacamole-auth-json
guacamole-auth-ldap
guacamole-auth-openid
guacamole-auth-quickconnnect
guacamole-auth-saml
guacamole-auth-totp

and all seems well.


widget not saved

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.23475.1000
Other software
OS Build Version: 10.0.23475.1000.amd64fre.ni_prerelease.230602-1331
.NET Version: .NET 6.0.16
Steps to reproduce the bug
The dashboard does not maintain the widgets added after the system restarts.
Expected result
that the widgets already assigned to the main dashboard are maintained
Actual result
Assign the GitHub widgets to the dashboard and restart the computer and the widget is deleted
Included System Information
CPU: 12th Gen Intel(R) Core(TM) i7-12650H
Physical Memory: 15.63GB (7.52GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe

Duplicate of #742


Scenario fails with Error Message "Expected to find internal spec properties with reference = 2 "

Current behavior
Cypress - Cucumber Framework is included with  "experimentalRunAllSpecs" as true. If we execute the all specs at a time,  one of the scenario get fails showing the error message as below

If we execute each feature file, script is working fine. Issues happen only, when we use "Run All Specs"
Desired behavior
Expected to execute all the scenarios to without any issues with "Run All Specs"
Test code to reproduce
Unfortunately. I couldn't share the test code here
Versions

Cypress version:  12.9.0
@badeball/cypress-cucumber-preprocessorversion:  16.0.3
@bahmutov/cypress-esbuild-preprocessor version:  2.2.0
Node version:  18.13.0

Checklist

[X] I've read the FAQ.
[X] I've read instructions for logging issues.
[X] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Hey, and thanks for reporting this issue. You need to provide a minimal, reproducible example issues that illustrates the issue.

Updated everything related to cypres to latest version as of today.

cypress: 12.11.0
cypress-localstorage-commands: 2.2.3
@badeball/cypress-cucumber-preprocessor: 16.0.3
@bahmutov/cypress-esbuild-preprocessor: 2.2.0
@cypress/webpack-preprocessor: 5.17.0
node: 18.14.2

Chrome: 112
MacOS: Ventura 13.3.1 m1
Also get the same error.

Expected to find internal spec properties with reference = 10 (this might be a bug, please report at https://github.com/badeball/cypress-cucumber-preprocessor)
Because this error occurred during a after each hook we are skipping all of the remaining tests.
[node_modules/@badeball/cypress-cucumber-preprocessor/lib/assertions.js:7:1](http://localhost:4200/__/#)
   5 | const package_json_1 = require("../package.json");
   6 | function createError(message) {
>  7 |     return new Error(`${message} (this might be a bug, please report at ${package_json_1.homepage})`);
     | ^
   8 | }
   9 | exports.createError = createError;
  10 | function fail(message) {
View stack trace
 Print to console
at createError ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/assertions.js:7](http://localhost:4200/__/#))
at fail ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/assertions.js:11](http://localhost:4200/__/#))
at assert ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/assertions.js:18](http://localhost:4200/__/#))
at assertAndReturn ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/assertions.js:22](http://localhost:4200/__/#))
at retrieveInternalSpecProperties ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/create-tests.js:36](http://localhost:4200/__/#))
at Context.eval ([webpack://my-webapp/./node_modules/@badeball/cypress-cucumber-preprocessor/lib/create-tests.js:452](http://localhost:4200/__/#))

Unfortunately as most of the people I cannot upload my company's code.
The error occurs only when running All Spec (15 in my case) and it fails somewhere semi-randomly in the middle of ~50 tests. Running features separately work perfectly fine.
@badeball maybe you can give some clues or pointers what we can try?
Unfortuntely nobody will write a fresh frontend app (and maybe backend) with a lot of features in hopes to maybe reproduce a bug in regards to minimal, reproducible example issues

I think I've already made myself clear.


Implement SAX analysis

Sub-chunking of a time series in fragments of a chronix chunk size
Linear regression on chunks
Mapping of regression values to symbolic representations

see http://www.cs.ucr.edu/~eamonn/iSAX.pdf


Feature/add extensions pagination tests

Overview
This pull request introduces a comprehensive suite of unit tests for the PaginatorExtensions class, enhancing the reliability and maintainability of our pagination logic.
Changes

Added unit tests for SimplePaginate to verify its behavior with null arguments, out-of-range values, and correct pagination functionality.
Introduced unit tests for SimplePaginateAsync focusing on asynchronous execution and expected outcomes.
Implemented unit tests for CursorPaginate and CursorPaginateAsync to ensure proper cursor-based pagination, including handling of null parameters and pagination logic.
Addressed the proper handling of ArgumentOutOfRangeException and ArgumentNullException across pagination methods.

Impact

Reliability: These tests fortify the pagination logic against regressions and unintended behavior changes.
Maintainability: With specified tests in place, future modifications to pagination logic can be made confidently, ensuring that existing functionalities remain intact.
Documentation: The tests also serve as a form of live documentation, illustrating expected behaviors and usage of the pagination methods.

Testing

All new tests have been executed against the current codebase, ensuring that they pass and align with expected behaviors.
Manual testing has been conducted where necessary to verify the integration of the pagination methods within larger workflows.

Missing test


Add Vito Labalestra to Contributors list

@xit4, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


trigger resize event on window when hiding structureboard

comment in the code explains it

LGTM

Coverage remained the same at 89.227% when pulling a2ab6eee9440c6977d1901fc4cff24148bf7d21d on FinalAngel:feature/hide-structureboard-resize into ee7d1cb4aeb6c7e6b0272db2c5356dae65ea7b18 on divio:release/3.2.x.


Error while installing Qiskit 0.13.0

Hello,
While running "pip install qiskit" within the conda environment for qiskit, I get the error detailed below. Could you please suggest what should I do? (latest version of anaconda=2019.10)
Collecting qiskit
  Using cached https://files.pythonhosted.org/packages/9e/2d/4c09bc1292a3be92f8c003b8491215ac9a0d81b014384e95a659a58ddf57/qiskit-0.13.0.tar.gz
Collecting qiskit-terra==0.10.0
  Using cached https://files.pythonhosted.org/packages/4e/86/180961c6ae2fecf521c5ea4246a3a3f52e9cbfed022c537733cc5203c228/qiskit-terra-0.10.0.tar.gz
Collecting qiskit-aer==0.3.2
  Using cached https://files.pythonhosted.org/packages/a8/46/11edbd99d7b732e63a542623109f394f17d9ad4051ddeb1e2dd9f36d1aac/qiskit-aer-0.3.2.tar.gz
    ERROR: Command errored out with exit status 1:
     command: /home/sam/softwares/anaconda3/envs/qiskitenv/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-8153ylk0/qiskit-aer/setup.py'"'"'; __file__='"'"'/tmp/pip-install-8153ylk0/qiskit-aer/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-8153ylk0/qiskit-aer/pip-egg-info
         cwd: /tmp/pip-install-8153ylk0/qiskit-aer/
    Complete output (113 lines):
    Collecting scikit-build
      Using cached https://files.pythonhosted.org/packages/8a/b5/c6ca60421991c22e69b9a950b0d046e06d714f79f7071946ab885c7115fb/scikit_build-0.10.0-py2.py3-none-any.whl
    Requirement already satisfied: setuptools>=28.0.0 in /home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages (from scikit-build) (41.6.0.post20191030)
    Requirement already satisfied: wheel>=0.29.0 in /home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages (from scikit-build) (0.33.6)
    Collecting packaging
      Using cached https://files.pythonhosted.org/packages/cf/94/9672c2d4b126e74c4496c6b3c58a8b51d6419267be9e70660ba23374c875/packaging-19.2-py2.py3-none-any.whl
    Collecting six
      Using cached https://files.pythonhosted.org/packages/65/26/32b8464df2a97e6dd1b656ed26b2c194606c16fe163c695a992b36c11cdf/six-1.13.0-py2.py3-none-any.whl
    Collecting pyparsing>=2.0.2
      Using cached https://files.pythonhosted.org/packages/c0/0c/fc2e007d9a992d997f04a80125b0f183da7fb554f1de701bbb70a8e7d479/pyparsing-2.4.5-py2.py3-none-any.whl
    Installing collected packages: six, pyparsing, packaging, scikit-build
    Successfully installed packaging-19.2 pyparsing-2.4.5 scikit-build-0.10.0 six-1.13.0
    Searching for cmake
    Reading https://pypi.org/simple/cmake/
    Downloading https://files.pythonhosted.org/packages/0a/f5/3212616a15b4112d7ad075a407f007eac2cac59292a9d973f2ee7e3c4068/cmake-3.13.2.post1.tar.gz#sha256=b45cc0f56f3f5705f4b4678ecf1f82e09c852c01cb3ffff43effdf8ffa323b1d
    Best match: cmake 3.13.2.post1
    Processing cmake-3.13.2.post1.tar.gz
    Writing /tmp/easy_install-js4pj4s3/cmake-3.13.2.post1/setup.cfg
    Running cmake-3.13.2.post1/setup.py -q bdist_egg --dist-dir /tmp/easy_install-js4pj4s3/cmake-3.13.2.post1/egg-dist-tmp-kch16ylg
    Traceback (most recent call last):
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 154, in save_modules
        yield saved
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 195, in setup_context
        yield
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 250, in run_setup
        _execfile(setup_script, ns)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 45, in _execfile
        exec(code, globals, locals)
      File "/tmp/easy_install-js4pj4s3/cmake-3.13.2.post1/setup.py", line 32, in <module>
        name='qiskit-aer',
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 425, in setup
        _parse_setuptools_arguments(kw)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 212, in _parse_setuptools_arguments
        for cmd in [dist.get_command_obj(command) for command in dist.commands]:
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 212, in <listcomp>
        for cmd in [dist.get_command_obj(command) for command in dist.commands]:
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/distutils/dist.py", line 858, in get_command_obj
        cmd_obj = self.command_obj[command] = klass(self)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/__init__.py", line 163, in __init__
        _Command.__init__(self, dist)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/distutils/cmd.py", line 57, in __init__
        raise TypeError("dist must be a Distribution instance")
    TypeError: dist must be a Distribution instance

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-8153ylk0/qiskit-aer/setup.py", line 31, in <module>
        setup(
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 456, in setup
        return upstream_setup(*args, **kw)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/__init__.py", line 144, in setup
        _install_setup_requires(attrs)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/__init__.py", line 139, in _install_setup_requires
        dist.fetch_build_eggs(dist.setup_requires)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/dist.py", line 717, in fetch_build_eggs
        resolved_dists = pkg_resources.working_set.resolve(
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/pkg_resources/__init__.py", line 780, in resolve
        dist = best[req.key] = env.best_match(
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1065, in best_match
        return self.obtain(req, installer)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1077, in obtain
        return installer(requirement)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/dist.py", line 787, in fetch_build_egg
        return cmd.easy_install(req)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 679, in easy_install
        return self.install_item(spec, dist.location, tmpdir, deps)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 705, in install_item
        dists = self.install_eggs(spec, download, tmpdir)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 890, in install_eggs
        return self.build_and_install(setup_script, setup_base)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 1158, in build_and_install
        self.run_setup(setup_script, setup_base, args)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 1144, in run_setup
        run_setup(setup_script, args)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 253, in run_setup
        raise
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/contextlib.py", line 131, in __exit__
        self.gen.throw(type, value, traceback)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 195, in setup_context
        yield
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/contextlib.py", line 131, in __exit__
        self.gen.throw(type, value, traceback)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 166, in save_modules
        saved_exc.resume()
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 141, in resume
        six.reraise(type, exc, self._tb)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/_vendor/six.py", line 685, in reraise
        raise value.with_traceback(tb)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 154, in save_modules
        yield saved
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 195, in setup_context
        yield
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 250, in run_setup
        _execfile(setup_script, ns)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/sandbox.py", line 45, in _execfile
        exec(code, globals, locals)
      File "/tmp/easy_install-js4pj4s3/cmake-3.13.2.post1/setup.py", line 32, in <module>
        name='qiskit-aer',
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 425, in setup
        _parse_setuptools_arguments(kw)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 212, in _parse_setuptools_arguments
        for cmd in [dist.get_command_obj(command) for command in dist.commands]:
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/skbuild/setuptools_wrap.py", line 212, in <listcomp>
        for cmd in [dist.get_command_obj(command) for command in dist.commands]:
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/distutils/dist.py", line 858, in get_command_obj
        cmd_obj = self.command_obj[command] = klass(self)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/site-packages/setuptools/__init__.py", line 163, in __init__
        _Command.__init__(self, dist)
      File "/home/sam/softwares/anaconda3/envs/qiskitenv/lib/python3.8/distutils/cmd.py", line 57, in __init__
        raise TypeError("dist must be a Distribution instance")
    TypeError: dist must be a Distribution instance
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

The exact same thing is happening to me; I tried in fresh environments on two different machines (one running CentOs 7.3.1611, the other running Ubuntu 18.04).
I'm using:

Python 3.8.0
conda 4.7.12

I was able to make some progress by running pip install cmake, but then it got stuck (on both machines) trying to install pyscf and linking to BLAS libraries. Not sure if it is an issue with qiskit at that point though.

Update: it seems that it is an issue when using Python 3.8. The process works just fine in 3.7.5.

I've documented (using the bug-issue format) the same problem in #705.  The issue will be addressed by documentation changes (see #660).

Took me a while to get back. Thank you for the suggestions, qiskit is now properly installed with "python=3.7.5". Thanks again!

Qiskit 0.15.0 is out now and includes support for python 3.8. I'm going to close this issue now, please feel free to reopen it if I'm missing something.

I have the same problem on Windows 10 x64.
Python 3.7.4 on virtualenv.
Any solution?


Cosmos model package

This PR addresses API - review comments for creating a model package - and moving model classes to the new model package.
Moves all these classes to model:
** Response classes
** Most of the enums
** Some of the policies
** Sql param / query / spec classes
** Resource
** JsonSerializable
To help the change, and reduce the scope, ModelBridgeInternal has been added to model package
It will be easier to review this PR comment by comment.

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests


Version changing is not working

Hello again!
Yesterday I decided to test the gametest build of Studio, so I used the mod manager and launched it in gametest. After downloading everything that it needed, it launched studio but it wasn't running that version (given the changelogs in your other repo, there was supposed to be a property in skyboxes named "MoonTextureId", and it errored when changed from the command bar).
I tried rebuilding studio, but it also didn't work: I had to manually extract every .zip file in the downloads directory to the root folder to make it work in that version, when it worked.
After testing, I wanted to change back to the live build, but the same happened again. So, again, I went and extracted every .zip file in the downloads folder.
And sometimes, when changing from a newer version to the live version, Team Create becomes inaccessible, throwing a lot of errors of the kind "Type failed to learn x" and crashing studio (probably due to having a newer version).
I tried deleting every possible file left by the mod manager, including registry keys and logs, yet it still happens to me: I don't know if this only happens on my machine, couldn't test on other computers yet.
The default Roblox Studio works fine.
Thanks!

Yeah I've noticed there's something wrong with the file delta stuff. I'm going to investigate this further ASAP.

This issue should be fixed with the latest commit, let me know if there are any other issues!
https://github.com/CloneTrooper1019/Roblox-Studio-Mod-Manager/commit/6678da6c543684e433942c9a51f18e10cdf39de9

This issue should be fixed in the latest commit.


[1.11.2] Dupe in crafting grid (refinedstorage-1.4.20)

Issue description:
Shift clicking on an item that is in the crafting grid should insert it, instead it is duped in the RS system.
I just updated from 1.4.18 and noticed this when inserting items into the crafting grid. Woops.
What happens:
Item is duped.
What you expected to happen:
Item should be deposited into the RS system
Steps to reproduce:

Open crafting grid
Shift click item that is already inside the RS system
Item is duped.

Version (make sure you are on the latest version before reporting):

Minecraft: 1.11.2
Forge: forge-1.11.2-<IP_ADDRESS>1-installer.jar
Refined Storage: refinedstorage-1.4.20.jar

Does this issue occur on a server? [yes/no]
Yes!
Animated gif
http://i.imgur.com/SjKmUVs.gif

List of installed mods:
https://paste.ee/p/JlLHx

More info:
Downgrading to refinedstorage-1.4.18.jar again fixed it!
So if you're hosting a server avoid running 1.4.20 for now. I have not tried if 1.4.19 is safe.
Love this mod, keep up the great work smashing bugz!
Cheers :)

That doesn't make sense, 1.4.18 <-> ,1.4.20 didn't change anything related to grids... I'm suspecting this is a serverside mod interaction issue.

Any clues what I can test? Anything I can check to try to figure out the cause?
I suspected JEI but toggled cheat mode without any change.

I have no idea.

Maybe some mod like InvTweaks or MouseTweaks? Anything that could manipulate the inventory.

I did not change any other mod aside from Refined Storage, still using the same version of InvTweaks and MouseTweaks.

Im getting something very similar on a server using Refined storage but in 1.12 .. If i have 1 of an item in the RS system and shift click it out i end up with 2 stacks of that item which you can repeat and use on any item that stacks more then 1 of itself.
Forge *.2420
RS 1.5.13
Although with your prev replies raoul, i fully suspect it has something to do with mousetweaks perhaps since it was only recently added an this started to occur.

Sorry, I can't reproduce this.


fix(logging): Prevent IO ex concurrent writes

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)
Bug fix.
:arrow_heading_down: What is the current behavior?
File.AppendText takes an exclusive write lock on a file. This means
that when multiple writes are occurring at the same time, an
IOException is thrown saying: "The process cannot access the file X
because it is being used by another process".
:boom: Does this PR introduce a breaking change?
No.
This fix will only work as long as the logger is used as a "singleton" throughout the application as the lock is not a static property. I decided against making it static because the constructor has file path as a parameter.
:bug: Recommendations for testing
Use --logfile. This caused issues especially in the package update lookup process as it's parallelized.

Looks good. Thank you for your contribution.


When instance deployment fails, TARGETS and DEPLOYED fields both becom "failed" instead of showing numbers.

Not sure the exact criteria, but it seems to have something to do with timeouts.
NAME           STATUS   TARGETS   DEPLOYED
box-instance   Failed   failed        failed

Queue deployment job failed

NAME STATUS TARGETS DEPLOYED
box-instance Failed unknown unknown

Query deployment succeeded, no summary returned

NAME STATUS TARGETS DEPLOYED
box-instance Pending unknown unknown

Query deployment summary timed out

NAME STATUS TARGETS DEPLOYED
box-instance Failed "last queried data" "last queried data"