Rewhitelisting Projects mid 2017

@gridresearchcorp
My calendar tells me we will have to rewhitelist all (production) projects soon.

when lol do u have a specific date @Quezacoatl1

Ping @gridcoin @GridResearchCorp - please re-advertise BOINC project registrations...

@grctest are you sure they have to be re-advertised? I see 25 projects whitelisted in the NN

@Quezacoatl1 we dont know. but i do know that some of the NN hashs that pop'd up in explainmagnitude2 earlier had the projects stats but said most were Not_whitelisted  so mag wasnt counted in my stats anymore thats why i thought maybe it was related as well thus the comment i made on ur post about it upcoming.

Well now with the new superblock counting all projects we at least know they are still white listed.
@gridresearchcorp can we close this issue? If I may ask, when did you whitelist the projects, and did you include all projects or just the ones running? I'd then add a note to my calendar 2 weeks before they run off again.


refined to compile to dotty and publish dotty packages

Lets refined to be compiled with dotty and I removed the restriction to publish on dotty as we don't have dependencies requiring withDottyCompat

Thanks Carlos!


Drop build-backend from pyproject.toml

Per @gaborbernat's suggestion, dropping the build-backend. I don't know if it makes sense to keep build-system at all, but this is the minimal change required to allow us to use --no-use-pep517 (which is required on the current version of pip when building from the repo).
Bernat discovered this when building the first infrastructure for the integration testing repo.

Also, is even pip building/installing from the repo supported? Isn't there an issue due to the need to run bootstrap.py first?

Also, is even pip building/installing from the repo supported? Isn't there an issue due to the need to run bootstrap.py first?

Well, regardless of whether it is or isn't, at the moment the build-backend field is not accurate, because we aren't declaring a dependency on setuptools.


Combination methods are not available for OrderedSet

OrderedSet has a guaranteed iteration order, and therefore methods like zip(), interleave(), interpose() and other combination methods make perfect sense for it. But they are only available for IndexedIterable which OrderedSet does not inherit.

Great point! I'll try to figure out a way to include these kinds of methods on OrderedSet and OrderedMap

Not sure if .indexOf makes sense to include on OrderedMap, but I noticed recently that was missing there as well

I don't think so... as it's not indexed, it just has a fixed and specified iteration order vs non-ordered Map's unspecified iteration order.

@leebyron I don't think so... as it's not indexed, it just has a fixed and specified iteration order vs non-ordered Map's unspecified iteration order.

Is that why there is no OrderedSet::get(index)?

@leebyron indexOf would be useful to be able to take advantage of slice (since slice deals with indexes). While OrderedMap has the ability to sort, it lacks the ability to insert somewhere in the middle. Typically I wan't to say "Insert key foo before key bar". Right now, the best I can come up with is something like:
omap.takeUntil((val, key) =>key === 'foo').concat(OrderedMap({ bar: 'barValue' }).concat(omap.skipUntil((val, key) => key === 'foo'))

Closing this aging issue - note that you can always convert OrderedSet and OrderedMap to various kinds of Seq to access different kinds of lazy indexed APIs at O(N) cost.


DUnit compatible

Let's rename all AssertXxx methods to CheckXxx in all FPCUnit test classes to make compatible with DUnit.

@nunopicado please, check if it is compatible now.

It's one good step towards it, yes.
You can close #53.

Ok, thanks.


Migrate  jcenter to mavenCentral

JCenter is being sunset and is readonly now https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter. Migrate plugins build.gradle configs from jcenter() to mavenCentral() maven repository.
What does this change?

Especially important now when jcenter is down.
https://status.gradle.com/incidents/9740r6bzx438

@lyf571321556 @spydon Thanks!

@lyf571321556 @spydon
Thank you for always supporting me. I just released the v1.1.3.


Build status WIP

This should be good... except that the Node4 based build is being a little (^&#%*&@!%#@ and periodically failing with indeterminate errors...

@heckj still WIP or no?

@benbp sorry, not WIP any longer

:+1:
@tldavies

:+1:


python3Packages.pydeconz: 79 -> 80

Motivation for this change
Update to latest upstream release 80
Change log: https://github.com/Kane610/deconz/releases/tag/v80
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

I have cherry-picked this change into my staging tree, thank you!


Duplicate entry bug and missing dependencies in preinstaller

I noticed that the issues:

https://github.com/phanan/koel/issues/752
https://github.com/phanan/koel/issues/705
https://github.com/phanan/koel/issues/403
https://github.com/phanan/koel/issues/255

Have similar problems regarding the duplicate key. After struggling a long time I've finally managed to install Koel en wanted to start indexing. Then only one file was indexed and basically everything stopped working. I've tried it with and without docker, with PostgreSQL and MySQL, nothing seems to work. I wish it cost me an hour to get everything up and running but it costed me 10 so far and it's still not working. I'm thinking about writing the basic functionality myself now which would cost me the same time. But fact is Koel is looking nice and I would like to give it a try. It just doesn't work. How do I fix this?
The few rules at the site:
git clone https://github.com/phanan/koel
cd koel
composer install
php artisan koel:init
php artisan serve
Server started on http://localhost:8000/

Are misleading and too good to be true. I had to find and preinstall dependencies manually and edit PHP config and ENV files in order to work around the errors. Just to get a up-and-running interface. Any help? The duplicate key problem is now what I'm stuck on, just like the other issues are describing without a solution. I've tried truncating tables and removing one entry or file without luck too.

First thing first, the commands ("rules") are not "too good to be true." Granted, it's for development purpose (using PHP server) but below is my personal Koel installation/update script:
$ cat ./build-koel
#!/bin/bash

cd ~/domains/my.koel.domain
git pull
git submodule update --recursive --remote
composer install
php artisan koel:init

You see, all it takes is five statements (actually 4, as git submodule update --recursive --remote is only necessary because I'm on master branch).
Now to your struggling – I'm sorry it doesn't work right out of the box for you, but honestly I don't know where to start (to help you).

The duplicate key issue sounds like some weird encoding in your mp3 files, which I can't tell unless I have a copy of those files
The HTTPS issue (if you're having it) – I honestly believe this is a server config issue. I personally run Apache under a reverse-proxying nginx with SSL without any problem.
That you have to find and preinstall dependencies – if your environment is missing prerequisites or having them misconfigured, there's very little I can do. Btw you mentioned using docker – how come you still need to preinstall stuff?

Thanks for your reply.

I've tried it with and without docker

The dependency problems where with the attempt without docker. If there is an encoding problem can we please fix that so it's rock-solid for all possible encoding?

If there is an encoding problem can we please fix that so it's rock-solid for all possible encoding?

Again, I can not really fix a problem without knowing what the problem is i.e. without having the songs with funny encoding(s). And tbh, yours is a very edge case.

I expect it has something to do with encoding stuff in the filenames/paths. Can that be? Letters like "äëöïüáéóíú".

No, it’s the md3 tags for artist.
Aat2ya9873<EMAIL_ADDRESS>schrieb am So. 30. Sep. 2018 um 00:15:

I expect it has something to do with encoding stuff in the
filenames/paths. Can that be? Letters like "äëöïüáéóíú".
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/phanan/koel/issues/809#issuecomment-425678942, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHrt0jk6TaGk3GLzI90wrCigk0DuRaCYks5uf_DlgaJpZM4W6FFG
.

So, can we make a fix that handles all possible characters in filenames/paths and md3 tags properly?

Also, you said "The dependency problems where[sic] with the attempt without docker." Doesn't that mean it's NOT a problem with Koel, and instead with your own server/environment, which in turn renders a good part of this issue report an unconstructive rant?

I'm closing this issue because it's pretty misleading. If you want to report the actual issue with encoding, please open another one and provide more information i.e. the files with errors.


Issue 47509: Resolve sample names that are integers during assay data import

Rationale
Issue 47509
Sometimes sample names are composed of only integers. These are difficult to distinguish from rowIds when resolving the lookup to a sample during assay import.  In related issues for sample import using integer-name samples, we updated the logic for resolving samples to first look for the sample assuming the integer value is its name instead of rowId.  We adopt similar logic here when importing via a file since it is rare that users will provide rowId values for the samples being referenced. If they do, they will surely also have the name of the sample available to use during assay data import in cases where there might be ambiguities based on numeric naming patterns.
Related Pull Requests

https://github.com/LabKey/sampleManagement/pull/1757

Changes

Update checkData method in AbstractAssayTsvDataHandler to go through exp.findExpMaterial even for integer values
Update AbstractAssayTsvDataHandler.importFile to make sure the sampleId data being imported matches the resolved MaterialInputs from checkData.

Need to target 23.3


operators [N] [CI] clever-operator (0.3.5)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

@J0zi @mvalarh I will be glad, if you could give me some insights about the failing tests ci/prow/*. I think that I miss something, but I do not know what

/merge possible

/merge possible

@FlorentinDUBOIS please fill metadata.annotations.description in your CSV. Operator was not installed after subscribing. You can debug locally on k8s.

Thanks @J0zi, I will check out this documentation :)

/merge possible


Support more lock providers

digger currently supports  DynamoDB for locks. However, in Azure DynamoDB service doesn't exist (there's a CosmosDB, but not the same schema).
A more native way in Azure would be to use a Storage Account. Actually the same storage account where the terraform state is kept. We would rely on Storage Account table to store our lock.
But before that, we need to adjust the codebase to start using multiple cloud providers.

Start using a LockService struct
Add CloudLock interface as property

Then instead of using DynamoDB directly, we'll call LockService which will rely on the underlying CloudLock provider.
The right lock provider will be injected when creating a LockService.
Example of implementation, check below
// Add the right param and return type
type CloudLock interface {
	Lock()
        Unlock()
        ForceUnlock()
}

The Lock Service:
type LockService struct {
	lockProvider CloudLock
}

func (ls *LockService) Lock() {
    return ls.lockProvider.Lock()

StorageAccount in Azure, implementation example:
type StorageAccountLock struct {
}

func (sal *StorageAccountLock) Lock()  {
	// add table with data to storage account
}

Full workflow (kept easy to be readable):
//Detect cloud vendor (or will be defined in digger.yaml)
lockVendor := getLockVendor(&config)
var cloudLock CloudLock

// Use switch or if/else
if lockVendor == "azure" {
    cloudLock := cloudLock.(StorageAccountLock)
}

if lockVendor == "aws" {
     cloudLock := cloudLock.(DynamoDBLock)
}

lockService := LockService {
    lockProvider: cloudLock
}
lockService.Lock()

There is already a Lock interface pkg/utils/locking

Thank you for this detailed issue and outlining refactoring steps @ynden. All looks good here, looking forward to reading your PR

performed in a separate branch and will be merged gradually into mainline as we restructure bits and peices


🔧 chore(tooling): support cross-platform development

Summary
This PR adds support for developing on Windows environments.
Details

Switch to mypy for windows support
Fix lint issues from mypy
Fix a few invalid cast bugs

Looks great! let's just fix the result.value handling and this is good to go!


Preview pane doesn't close

Using the most recent version of phpcd with neoplete.vim on vim8. The preview pane shows up and updates correctly, but doesn't close when I select a completion. I'm willing to debug this further, just not sure what'd be the most helpful info to provide.

Adding autocmd InsertLeave,CompleteDone * if pumvisible() == 0 | pclose | endif fixes this, but I wouldn't have thought that this would be necessary given the gif and docs.

fixed at https://github.com/lvht/phpcd.vim/commit/89bdf90fef41980d9c279e17cc91a75bc1a47a79


App for custom bot

Is it possible to connect the app to a host rather than openai? I want to try the app on custom bot running on a certain server.

Hey @amit4aws, my use case is the same. So these were my observations while digging around the code:

Add an environment variable called OPENAI_API_HOST pointing to the domain where you have hosted the endpoints. http://localhost:8000 works as well.

Add an environment variable called OPENAI_API_KEY containing the OpenAI key (or set it to an empty string in case your backend is handling the OpenAI API calls)

To make minimum changes in the codebase, make sure you create the endpoints:

v1/models: Mimics the same endpoint in api.openai.com, sans the API Key in the headers.
v1/chat/completions: Same as above, with similar input and output. Note that the response has to be a Streaming HTTP Response, else stuff will break.

Hey @amit4aws, my use case is the same. So these were my observations while digging around the code:

Add an environment variable called OPENAI_API_HOST pointing to the domain where you have hosted the endpoints. http://localhost:8000 works as well.
Add an environment variable called OPENAI_API_KEY containing the OpenAI key (or set it to an empty string in case your backend is handling the OpenAI API calls)
To make minimum changes in the codebase, make sure you create the endpoints:

v1/models: Mimics the same endpoint in api.openai.com, sans the API Key in the headers.
v1/chat/completions: Same as above, with similar input and output. Note that the response has to be a Streaming HTTP Response, else stuff will break.

Hi @anuran-roy, Do you have an example to implement point 3? Thanks.


OSX: Command+W should close current Window

Command+W closes the entire app right now, but it should just close a window.

Dupe with #992 ?

Ah thanks @yunti ! Sure is.


Query raw byte positions of the header and file of an entry

Implement methods on Entry for querying the raw byte positions of the entry's header and file within the archive.
The main motivation is that there are cases when the Read object passed to Archive implements Seek or is simply byte-sliceable, and we want to perform random access on individual entries of the archive. A concrete example is a large memory mapped tar file, for which we want to avoid unnecessary copies through Read.
This commit does what seems to be the least invasive thing to achieve the above: (1) adding two helper methods to Entry for querying the raw byte positions of the header and file, and (2) putting the responsibility to the user for figuring out whether it is valid to use the raw byte representations (e.g. when the entry type is not a sparse file).

Looks great to me, thanks!


[Bug]: Issue using Filament reactive fields

What happened?
Hi,
When using reactive fields to set a currency masked field (Field X). Field X display nothing when logging in the 'afterStateUpdated' the value of the state is NULL and the value of the old state is the desired state for some reason.
\Filament\Forms\Components\TextInput::make('full_commission_amount')
->numeric()
->currencyMask()
->afterStateUpdated(function (?string $state, ?string $old) {
      info([$state, $old]);
// returns
// [  0 => NULL,
// 1 => '1000',]
  })

How to reproduce the bug
// Field on updated currency field (full_commission_amount) on afterStateUpdate
   \Filament\Forms\Components\TextInput::make('commission_percentage')
      ->afterStateUpdated(fn(Forms\Get $get, Forms\Set $set) => self::updateProjectCalculations($set, $get))

// here the $set value can't be seen
\Filament\Forms\Components\TextInput::make('full_commission_amount')
 ->numeric()
 ->currencyMask()

Package Version
1.7.3
PHP Version
8.2
Laravel Version
11.4
Which operating systems does with happen with?
No response
Notes
No response

test 1.7.4


Fix #12944, Fix Sphinx python docstring formatting error.

Description
Use relative class name to avoid reference target conflict.
Fixes #12944
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot [pr-awaiting-review]


Camera probe retrieval when in spaceship doesn't always work

Left hand grip + B doesn't always retrieve a launched camera probe when flying in the spaceship.  Using Oculus Quest + Link. All other controls appear to work fine, including camera probe retrieval when outside of ship.

It's supposed to be right hand grips not left. Or did you change the defaults?

Sorry I may have mistyped that.  I basically follow the pop up instructions that tell you what to press for the action you desire.  It works sometimes...but often doesn't.  Only happens when in the spaceship.  When using the handheld camera probe, it works 100%.

Hm, don't know. It's hard for me to diagnose it with such little info on how to reproduce. If you can eventually figure out what actually triggers the bug, that would be very helpful.

Hi ! I have the exact same problem, once the scout cam sent with the space ship, it becomes impossible to retreive it, b button still working but not the right hand grip button. Tried to leave the ship and retreive it with the tool but no response, only taking photos after that.
Another problem is in the game menu, whenever i try to acces it after a few minutes (audio settings), the selected items go far left, as if a button is triggered but nothing happening irl. Maybe theses two are related i dont know ?
Setup for vr is the oculus quest recognised in steam as rift, with touch controllers

I seem to be getting the same bug, using a Rift S with Touch controllers. haven't touched the default bindings.
I'm able to consistently produce the bug by firing the scout into the Quantum Moon. hopefully that helps.
I actually discovered it because I was trying to snap a photo of the moon but lost it in there, which is unfortunate because it basically renders the ships scout useless for the remainder of that loop.
Also want to say awesome work on the mod!

I also had this issue with Quest and AMD ReviveVR while on Giant's Deep. Maybe the scout got sucked into a tornado or fell beneath the current?

Fixed in 1.0.3


Switch service refs and module requires style

It would make more sense to have the service refs come before module requires so they can use any service refs that were initialized before.

Hi, thanks for the PR but requires should come first. It's common practice in programming languages to have all imports/requires at the top before anything else.


huynd_#247

The title is the name of the variable.

@huynd2904
I have merged it, but this appears to be a PR to the master branch.
Can you please re-create the PR to the develop branch?


Cannot drag splitter

Hi @hexerrus,
I'm having issue with splitter. Cannot drag it.
The setup is pretty simple:
<Rows>         <Cell>1</Cell>         <Splitter />         <Cell>2</Cell>       </Rows>
I've set styles to splitter from readme.
I can see in the dev tools that splitter dragging property is being updated on mouse down/up
Could you help me with it?

Same issue here

same

The issue is caused in this file when assigning the new width/height of the Cell style without a unit. The style of the Cell element is assigned a Number (ie 200) when is should be a unit String (ie '200px'). This can be resolved by making sure your page has <!DOCTYPE html> defined. This indicates to the browser which default units to use ('px' in this case).


error: could not compile polars-arrow (lib) due to 6 previous errors

Version
List the versions of all flood packages you are using.
Platform
The output of uname -a (UNIX), or version and 32 or 64-bit (Windows)
Linux ubuntu 5.4.0-162-generic #179-Ubuntu SMP Mon Aug 14 08:51:31 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Description
Enter your issue details here.
One way to structure the description:
error when install using :
git clone https://github.com/paradigmxyz/cryo
cd cryo
cargo install --path ./crates/cli
[short summary of the bug]
Compiling polars-utils v0.32.1
|            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: disambiguate the method for candidate #2

I tried this code:
[code sample that causes the bug]
I expected to see this happen: [explanation]
Instead, this happened: [explanation]

Is there any way to get around this error ?

Is there any way to get around this error ?

Build locally!

Is there any way to get around this error?

Build locally!

That doesn't seem to work either, cloning and running cargo install --path ./crates/cli gives the following error:
Caused by:
  error inheriting `version` from workspace root manifest's `workspace.package.version`


min/max size constraints for layouts

Currently SizeLayout allows layouts to have a specific length along one or both dimensions.
It would be useful to be able to specify maximum width/height and minimum width/height for a nested layout.
One option would be to create a MinSizeLayout and a MaxSizeLayout. Unsure if it would make sense to combine these into one layout (e.,g. SizeRangeLayout), or even combine them with SizeLayout.
If we stick with separate layouts, then we might also want to rename SizeLayout to ExactSizeLayout

Thanks for opening this, probably I prefer the solution of SizeRangeLayout with convience init methods to just specify just MaxSize or MaxWidth or MaxHeight or MinSize or MinWidth or MinHeight.
I need this for my layouts so happy to work on it.

I think it might make sense to put this functionality in SizeLayout. Exact size is just a special case of a size range where max size == min size.
Something like this (I omitted all the other layout parameters for clarity)
public class SizeLayout {

    public let minWidth: CGFloat?
    public let maxWidth: CGFloat?
    public let minHeight: CGFloat?
    public let maxHeight: CGFloat?

    public init(minWidth: CGFloat? = nil,
                maxWidth: CGFloat? = nil,
                minHeight: CGFloat? = nil,
                maxHeight: CGFloat? = nil) {

        self.minWidth = minWidth
        self.maxWidth = maxWidth
        self.minHeight = minHeight
        self.maxHeight = maxHeight
    }

    public convenience init(width: CGFloat? = nil, height: CGFloat? = nil) {
        self.init(minWidth: width, maxWidth: width, minHeight: height, maxHeight: height)
    }

    public convenience init(width: CGFloat, minHeight: CGFloat? = nil, maxHeight: CGFloat? = nil) {
        self.init(minWidth: width, maxWidth: width, minHeight: minHeight, maxHeight: maxHeight)
    }

    public convenience init(height: CGFloat, minWidth: CGFloat? = nil, maxWidth: CGFloat? = nil) {
        self.init(minWidth: minWidth, maxWidth: maxWidth, minHeight: height, maxHeight: height)
    }

    public convenience init(size: CGSize) {
        self.init(width: size.width, height: size.height)
    }

    public convenience init(maxSize: CGSize) {
        self.init(maxWidth: maxSize.width, maxHeight: maxSize.height)
    }

    public convenience init(minSize: CGSize) {
        self.init(minWidth: minSize.width, minHeight: minSize.height)
    }
}

SizeLayout(height: 100)
SizeLayout(height: 100, width: 75)
SizeLayout(height: 100, minWidth: 50)
SizeLayout(height: 100, maxWidth: 100)
SizeLayout(height: 100, minWidth: 50, maxWidth: 100)

@lucabartoletti Have you had a chance to work on this? Have any thoughts on my proposal?


Question : Licence menu with vRP.TryPayment

Hi, i have a question, i try to insert vrp try payment to my menu but i have a error
code:
menu_licence["Licence"] = {function(player, choice)
local user_id = vRP.getUserId(player)
  if user_id ~= nil and vRP.tryPayment(user_id,"5000") then -- It is the line with error
    vRPclient.startLicence(player,{})
  else
    vRPclient.notify(player,{"No enoughts money."})
  end
end,""}

Issues are for bug reports or features, not for help. Wait for the release.

Okey :(


Introduce multi-node training setup

Enable multi-node GPU training with SLURM
This PR adds support for multi-node GPU training using the SLURM job scheduler. The changes allow the code to detect if it is running within a SLURM job and automatically configure the number of devices and nodes based on the SLURM environment variables.
Key changes

Set use_distributed_sampler to True when not in evaluation mode to enable distributed training
Detect if running within a SLURM job by checking for the SLURM_JOB_ID environment variable
If running with SLURM:

Set the number of devices per node (devices) based on the SLURM_GPUS_PER_NODE environment variable, falling back to torch.cuda.device_count() if not set
Set the total number of nodes (num_nodes) based on the SLURM_JOB_NUM_NODES environment variable, defaulting to 1 if not set

Rationale for using SLURM
SLURM (Simple Linux Utility for Resource Management) is a widely used job scheduler and resource manager for high-performance computing (HPC) clusters. It provides a convenient way to allocate and manage resources, including GPUs, across multiple nodes in a cluster.
By leveraging SLURM, we can easily scale our training to utilize multiple GPUs across multiple nodes without the need for manual configuration.

I have implemented the latest feedback, updated the CHANGELOG and added a SLURM-example submission script to /docs/examples (is that a good location?) as discussed with @leifdenby.  A new small section was added to the README.md.
@joeloskarsson yes, every cluster is different and I also have to adapt my submission scripts after major changes. Do you have ticket-support with your HPC-provider? They usually know what to do...

As discussed at the dev-meeting just now, here is a slurm submission script example. I realized I put everything + some documentation already in this here PR: docs/examples/submit_slurm_job.sh


preselect issue

why preselected items not remain selected in the dropdown?

i mean
const [selected, setSelected] = useState([{value:"mango", label:"mango"]);
Mango is not remained ticked in dropdown menu


Hono does not seem to work correctly with Vue

What version of Hono are you using?
4.5.3
What runtime/platform is your app running on?
Bun
What steps can reproduce the bug?
Pull this repo: https://github.com/buzzy/hono-vue
Look in the file /src/api/server.ts
It's a very simple example that uses hono and zod-validator

Look in the file /src/api/client.ts

As you can see in the picture, all the types get imported correctly.
Now open file /src/views/HomeView.vue
This is a Vue component file.

Using the exact same imports as in the server.ts, all types are working except for the input type. It's blank.
What is the expected behavior?
That all the types are imported, including the "input" attribute.
What do you see instead?
When using Hono RPC in a Vue component, the "input" is always blank.
Additional information
No response

Is there any possibility that an extension (.ts, .js) is needed?
There is a possibility that it is necessary depending on the configuration.

Hi @buzzy
Add a strict: true in your tsconfig.json:
diff --git a/tsconfig.json b/tsconfig.json
index 66b5e57..5c6a83b 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -1,5 +1,8 @@
 {
   "files": [],
+  "compilerOptions": {
+    "strict": true
+  },
   "references": [
     {
       "path": "./tsconfig.node.json"
@@ -8,4 +11,4 @@
       "path": "./tsconfig.app.json"
     }
   ]
-}
+}

I'll close this issue. If you need any help related to Hono, please reopen. Anyway, this issue is caused by TypeScript settings.


Typo

typo at https://github.com/VcampSoldiers/Swin-Transformer-Tensorflow/blob/main/models/swin_transformer.py#L72

My bad


Using arcgis-webpack-plugin with Angular 8

I was able to get a prototype running using arcgis-webpack-plugin and Angular 8 CLI.
There's still a little bit of testing that needs to be done before converting this repo over from esri-loader. And, please send me your feedback 😄.  The steps I took to build the prototype are outlined below. I "think" I captured everything from my notes, but let me know if I missed something.

Do a fresh install of an app using the Angular 8 CLI (latest). You can find more info here: https://cli.angular.io/.

Install this handy helper for a custom webpack builder:
npm i -D @angular-builders/custom-webpack.
Here's the doc and code: https://github.com/just-jeb/angular-builders/tree/master/packages/custom-webpack

Install arcgis-webpack-plugin:
npm install --save-dev @arcgis/webpack-plugin --verbose.
Here's a reference to doc and code: https://github.com/Esri/arcgis-webpack-plugin

Install ArcGIS JS API types:
npm install --save @types/arcgis-js-api

In angular.json under projects.architect I replaced build and serve with the following

        "build": {
          "builder": "@angular-builders/custom-webpack:browser",
          "options": {
            "customWebpackConfig": {
              "path": "./extra-webpack.config.js"
            },
            "outputPath": "dist/angular-builder-esri",
            "index": "src/index.html",
            "main": "src/main.ts",
            "polyfills": "src/polyfills.ts",
            "tsConfig": "tsconfig.app.json",
            "aot": false,
            "assets": [
              "src/favicon.ico",
              "src/assets"
            ],
            "styles": [
              "src/styles.scss"
            ],
            "scripts": []
          }
        },
        "serve": {
          "builder": "@angular-builders/custom-webpack:dev-server",
          "options": {
            "browserTarget": "angular-builder-esri:build"
          },
          "configurations": {
            "production": {
              "browserTarget": "angular-builder-esri:build:production"
            }
          }
        },

I made a few minor tweaks to tsconfig.json. I'm not sure what was 100% necessary since I was just trying to get things going, it will take more testing to determine which libs are needed for example.

{
  "compileOnSave": false,
  "compilerOptions": {
    "baseUrl": "./",
    "outDir": "./dist/out-tsc",
    "sourceMap": true,
    "declaration": false,
    "downlevelIteration": true,
    "experimentalDecorators": true,
    "module": "esnext",
    "moduleResolution": "node",
    "importHelpers": true,
    "target": "es5",
    "typeRoots": [
      "node_modules/@types",
      "./types"
    ],
    "lib": [
      "dom",
      "es2015.promise",
      "es5",
      "es6"
    ],
  }
}

Create a new file in root called extra-webpack.config.js:

const ArcGISPlugin = require('@arcgis/webpack-plugin');
/**
 * This is where you define your additional webpack configuration items to be appended to
 * the end of the webpack config.
 */
module.exports = {
  plugins: [new ArcGISPlugin()],
  node: {
    process: false,
    global: false,
    fs: "empty"
  }
};

In tsconfig.app.json add "types": ["arcgis-js-api"]

At this point I'd recommend doing a test serve by running ng serve and seeing if there are any errors. If there are errors then fix them now before moving on.

Generate the scaffolding for your mapping component

  ng g component esri-map

Copy the app.components contents from angular-cli-esri-map/src/app into your new esri-app/src/app/ directory. Yep, I know, the app.component.html file still has some text referring to esri-loader, I'll fix that later.

Copy the contents from the angular-cli-esri-map/src/app/esri-map folder into your new esri-app/src/app/esri-map directory.

Replace the contents of esri-map.components.ts with the code from this gist: https://gist.github.com/andygup/b4736f8a7cc97a007a196dcb42af74cf

14 Give it a go: ng serve. This is definitely not optimized. It should compile without errors. Here's a partial screenshot of my build console messages. I had a few extra goodies in my test app so you can ignore the esri-geometry message:

And here's the app running

Successful but took over 60s to compile the 2700+ modules.

Thanks @sbuscher! Yep, the initial load will take a while, plus I didn't do any optimization via module exclusion which can decrease the number of generated bundles => https://github.com/Esri/arcgis-webpack-plugin#excluding-modules.  Note-to-self to doc that in the README.

Glad to see that you are making good progress with removing esri-loader. Keep up the good work!

Closing. The webpack project is now available here: https://github.com/Esri/angular-cli-esri-map/tree/arcgis-webpack-angular

ArcGIS.txt
This is my HTML page that is save txt file and in this i created WebMap with featurelayer in ArcGIS Javascript but i want to that convert in angular so pls how can i do this.


Gauss homolumo

Draft not to be merged

Based on these comments, this branch could be almost entirely scrapped. I think the only thing possibly worth leaving a pull request for is the lines adding the LUMO (so literally a couple of lines that say LUMO = HOMO + 1).


Limit of characters in the Log.d

Hi there,
First of all, I have to say that I'm a big fan of this plugin :smile: :+1: but I found a quite annoying issue, this plugin takes as TAG the name of the class but it's unable to detect that a class name is bigger than 23 characters which creates errors and you have to delete manually a part of the TAG in order to have a working TAG.
I think this issue could be easily fixed by removing all characters after the 23th one. Unhapilly, I do not have that much time to understand your code and modify it in order to get it to work :unamused: .
Best,

Thank you for using the plugin. And Thank you for the report.
I has released the 1.1.6. Its features have been included in the release.

Pretty cool and fast ! Great plugin!
Just tested it and it works! Thanks a lot (you save me ~ 5 minutes per day with this update!)


[DCOS-51158] Improved Task ID assignment for Executor tasks

What changes were proposed in this pull request?

Bugfix for Spark Executor ID being different from Mesos Task ID
Bugfix: thread-safe sequential task IDs for Executors

How was this patch tested?

updated unit test
Integration tests from mesosphere/spark-build

Release notes

[Bugfix] Fixed bug for Spark Executor ID being different from Mesos Task ID

Closing this PR due to incompatibility with DCOS CLI. Whenever we try to use scheduler task ID as a part of child task IDs we’re going to hit this CLI issue:
dcos task log --completed --lines=1000 driver-20190404003245-0001
found more than one task with the same name: [
'ae94906c-af9c-4cb2-8976-8be248c401cf-0009-driver-20190404003245-0001-1',
'ae94906c-af9c-4cb2-8976-8be248c401cf-0009-driver-20190404003245-0001-2',
'ae94906c-af9c-4cb2-8976-8be248c401cf-0009-driver-20190404003245-0001-0',
'driver-20190404003245-0001'].

Please provide a unique task name.

Reopened the PR due to Executor ID and thread safety bugfixes


Check Load-Balance Down (digitalocean)

I try install, everything running but Load-Balance Down.
How to fix it,
Thanks for support.

Can you post the error here

Did you update the params << update this >> in all places?

Yes, I update all,
data:
TZ: Asia/Ho_Chi_Minh
XMPP_SERVER: meetup.saza.vn # << update this >>  update your main prosody XMPP domain
XMPP_BOSH_URL_BASE: http://meetup.saza.vn:5280  # << update this >>
DOCKER_HOST_ADDRESS: meetup.saza.vn  # << update this >>
JICOFO_COMPONENT_SECRET: namtel  # << update this >>
JICOFO_AUTH_PASSWORD: namtel # << update this >>
JVB_AUTH_PASSWORD: namtel  # << update this >>
config: |
# Jitsi Videobridge settings
# sets the XMPP domain (default: none)
JVB_HOSTNAME=meetup.saza.vn
## << update this >>
# sets the hostname of the XMPP server (default: domain if set, localhost otherwise)
JVB_HOST=

How to:?
// ## << update this >>
deploymentInfo: {
shard: "shard0",
region: "sfo-2",
userRegion: "sfo-2"
},
     - name: OCTO_REGION
          value: tor-1 # << update this >>
  volumes:

Can upload your logs here for web, jicofo and jvb (use kubernetes dashboad on DO).
Also make sure you pointing your domain to the load balancer IP and firewall UDP ports opened

Thanks  so much, It worked, How to apply SSL for my domain?

Good to hear that! upload your certificate to the load balancer settings (choose https instead of TCP at port 443).
Or if you know the certificate id, add this to web service:
apiVersion: v1
kind: Service
metadata:
  labels:
    service: web
  name: web
  namespace: jitsi
  **annotations:
    service.beta.kubernetes.io/do-loadbalancer-certificate-id: your_certificate_id**
...


can you comment the "Use local jar file" in the reader.py?

This feature may not support recently. I will consider in future release.


Fixed typo in release_notes.md

Description of the change: Fixed typo in unsub module release note.
Reason for the change:
Link to original source:

Closes #

Hello @Whatthefoxsays,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Not working without a fixed height on div

I am using a container div with flexbox to have a dynamic height.
However the scrollbar only shows with a fixed height style is added to the inner simplebar div.
The inner div that contains the simplebar init has no styling, unless adding the height to test.
I am adding html to the simplebar div dynamically using the recommended method
simpleBar.getContentElement().insertAdjacentHTML
The code is compiled from a Svelte component.
Example of it not working below.
Join room button, then spam the enter key to fill up the div.
Adding fixed height to #chatbox2 simplebar div fixes it but I need a dynamic height using flexbox.
http://duelreality.com/simplebar/
Thanks

Solved with a css hack by setting a height of any minimal pixels to the container div and then 100% to the simplebar div.


@Reference annotation register in spring too late

Reference annotation register in spring BeanPostProcessor
when @Reference annotation use param like timeout or check, i need copy reference again and again,  i can't just use @Resource to visit the reference.
like this:
public class Demo1 {
    @Reference(url = "dubbo://<IP_ADDRESS>:7100", timeout = 5000, check = false)
    private UserService userService;
}

public class Demo2 {
    @Reference(url = "dubbo://<IP_ADDRESS>:7100", timeout = 5000, check = false)
    private UserService userService;
}

can we scan @Reference annotation in beanDefinition stage?
public class Demo1 {
    @Reference(url = "dubbo://<IP_ADDRESS>:7100", timeout = 5000, check = false)
    private UserService userService;
}

public class Demo2 {
    @Resource
    private UserService userService;
}

pls. try this:
public class Demo1 {
    @Reference(id="user1", url = "dubbo://<IP_ADDRESS>:7100", timeout = 5000, check = false)
    private UserService userService;
}

public class Demo2 {
    @Autowired
    @Qualifier("user1")
    private UserService userService;
}

pls. try this:
public class Demo1 {
    @Reference(id="user1", url = "dubbo://<IP_ADDRESS>:7100", timeout = 5000, check = false)
    private UserService userService;
}

public class Demo2 {
    @Autowired
    @Qualifier("user1")
    private UserService userService;
}

it not work
@Autowired effect in AnnotationInjectedBeanPostProcessor
@Reference register reference to spring in ReferenceAnnotationBeanPostProcessor
when Demo2 init first, AnnotationInjectedBeanPostProcessor can't find reference in spring


Add compatibility matrix for supported VMClarity features

Create a matrix that contains compatibility data about:

Execution mode (standalone/go module/VMClarity stack)
Execution environment (cloud providers, docker, localhost, windows/unix)
Capabilities (what is supported: e.g. discovery logic, filesystems, scanners)

Done in https://github.com/openclarity/vmclarity/pull/1956


Fix hash function "hash" in "ckarrayindex.h" to include the first element in the loop

Original author: Vipul Harsh
Original issue: https://charm.cs.illinois.edu/redmine/issues/1199

This was suggested by Alexander Frolov. For 1d arrays, this makes for a continuous sequencing. Needs performance testing before merging

Original date: 2016-10-05 22:36:34

~~https://charm.cs.illinois.edu/gerrit/1827~~ https://github.com/UIUC-PPL/charm/commit/85d452b86cb48338fdd7373041dfaefe19b7c8d9
Performance results showed benefits for tested cases.


How do I exclude layers from being printed on every image?

Is there a way to exclude certain layers a percentage of the time?  i.e. there are 5 attribute layers, but I don't want 5 attributes included in every dna.
Thank you in advance for the help!

Just put a blank transparent image called none in the folder


invalid handshake version: 22, only 0 is valid

Seeing many entries like this in the LND log.
[ERR] BTCN: Can't accept connection: unable to accept connection from <IP_ADDRESS>:60368: act one: invalid handshake version: 22, only 0 is valid
like every 5seconds on average.
Pretty sure its related to some of my channel partners disconnecting from me and remain unreachable. *shrug
Tried:

Recreating certs
Installing Tor from source
IP2Tor for clearnet IP
Flashing a new SD

but nada.
Debug: https://termbin.com/ysr1
How can I fix that?

@openoms any idea? Or maybe something to ask at the LND dev slack? https://dev.lightning.community/

Must be some erroneous setting in the configuration: https://github.com/lightningnetwork/lnd/issues/1173
I would check the
/mnt/hdd/lnd/lnd.conf
first.
See the raspiblitz defaults: https://github.com/rootzoll/raspiblitz/blob/v1.7/home.admin/assets/lnd.bitcoin.conf

Unfortunately nothing has changed with the replacement of the conf. I'll close this anyway and see what can be found over at LND


{gfortran{,48,49},gfortran{6..14}}: move to by-name

Motivation for changes
None, really.
Description of changes
Amused myself by moving these tiny derivations to pkgs/by-name/.
Why are there so many?
Inquiring minds want to know also.

#61130

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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.

I told @reckenrode on Matrix I would do some campsite cleaning to make this PR less shuffling deck chairs around. I've added that to the notebook of TODO.

I discovered today that while this PR is equivalent in terms of rebuilds and content, it's not equivalent in terms of overrides.
With the old version of the code, gfortran.override accepts these parameters:
bintools
cc
coreutils
expand-response-params
extraBuildCommands
extraPackages
extraTools
fortify-headers
gccForLibs
gnugrep
includeFortifyHeaders
isCcache
isClang
isGNU
lib
libc
libcxx
name
nativeLibc
nativePrefix
nativeTools
nixSupport
noLibc
propagateDoc
runtimeShell
stdenvNoCC
useCcForLibs
zlib

With the new version, it only accepts wrapCC and gcc.
This makes sense, as pkgs/by-name implicitly uses callPackage:
https://github.com/NixOS/nixpkgs/blob/087f33007fd846309a30cf805bf2cfae55eec442/pkgs/top-level/by-name-overlay.nix#L55
... and the only two names used in the forwarding package are wrapCC and gcc.
I don't know of a way to make callPackage drop the layer of dereference, and I don't think it should. I'm planning on effectively reverting this patch in order to accomplish the campsite cleaning requested.


Version 5 causes build conflicts due to build files being shipped

Looking at https://www.npmjs.com/package/@dusty-phillips/rescript-zora?activeTab=code it seems things such as .merlin and lib/bs got shipped due to the .npmignore file existing and being blank. That causes build failures because the compiler used might not align with what the user of the library has, e.g. https://github.com/brettcannon/check-for-changed-files/actions/runs/9210746204/job/25338461041?pr=99#step:5:503 .
And FYI, https://rescript-lang.org/docs/manual/latest/project-structure#publishing suggests not shipping the JS files when the library wraps an existing JS library (although maybe you included them for the blog post series?).

Unfortunately fixing this will require a new release (and maybe yanking the 5.0.0 release?).

I also opened https://forum.rescript-lang.org/t/what-would-belong-in-a-gitignore-template-for-rescript/5263 to find out exactly what should be ignored.

And in case you're wondering why this is happening, https://docs.npmjs.com/cli/v10/using-npm/developers#keeping-files-out-of-your-package suggests that having a .npmignore overrides your .gitignore. Probably another solution is to just copy the .gitignore over to .npmignore (or maybe symlink it?).

I'm pretty sure the library predates that publishing advice. Looking at the git history, I think npx np was complaining about the lack of a .npmignore and I just added it to shut it up without researching what it should contain.

Released 5.0.1 and unpublished 5.0.0. 🤞🏼


UCR Unblocker exception handling

Make UCR unblocker automatically use the current directory, and be a little bit more verbose.

Changes should be made on develop and targeted as such. Please see https://github.com/Snoothy/UCR/blob/master/CONTRIBUTING.md


Bump react-redux peerDependency version

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2631   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          66      66
  Lines        1371    1371
======================================
  Hits         1371    1371

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 fc2b82f...4fc1a90. Read the comment docs.


Migrate to new scope constraint format

Wikidata is changing the way they are representing scope constraints. This PR adapts the code to the new format.
Discussion: https://www.wikidata.org/wiki/Wikidata:Property_proposal/property_scope

merging this in a few days unless anybody disagrees


Fix wasm builds with file_watcher enabled

Objective
Currently, in 0.12 there is an issue that it is not possible to build bevy for Wasm with feature "file_watcher" enabled. I think it would still compile but have no effect on that platform.
Solution

Remove notify-debouncer-full dependency on WASM platform entirely.
Code that requires "embedded_watcher" or "file_watcher" feature now does require that target arch is not wasm32.

Changelog
Fixed

Fixes compilation error on wasm with "file_watcher" feature enabled.

Repro steps:

Add "Fixes #10507" to the PR description please.
Now the difficult question. Is this what we want? What's the problem with failing to compile invalid code?
The proposed solution of having hot-reloading silently become broken on WASM targets seems far from ideal. When a user compiles their game with the file_watcher feature, they expect hot-reloading to work. Nothing else. With this change, it sure compiles, but it won't work, breaking user expectation.
Here is a suggestion:
#[cfg(all(feature = "file_watcher", target_arch = "wasm32"))]
compile_error!(
    "The \"file_watcher\" feature for hot reloading does not work \
    on WASM.\nDisable \"file_watcher\" (or \"embedded_watcher\") \
    when compiling to WASM"
);

Question: does this print the error message when trying to compile with the setup you describe? If so I think it might be preferable.

What's the issue with the failing compilation? That it is unclear how to fix it? If so, the compile_error! trick should fix the problem.

@nicopap I did not about this macro, it's great! So I did remove most of the changes and added the macro. One thing that I left is I still disabled notify-debouncer-full dependency, so it is not compiled on wasm. I did it because without that change, it does not even get to the macro compilation and error is not obvious:
   Compiling notify-debouncer-full v0.3.1
error[E0432]: unresolved import `file_id::get_file_id`
   --> C:\Users\rybop\.cargo\registry\src\index.crates.io-6f17d22bba15001f\notify-debouncer-full-0.3.1\src\cache.rs:6:15
    |
6   | use file_id::{get_file_id, FileId};
    |               ^^^^^^^^^^^ no `get_file_id` in the root
    |
note: found an item that was configured out
   --> C:\Users\rybop\.cargo\registry\src\index.crates.io-6f17d22bba15001f\file-id-0.2.1\src\lib.rs:111:8
    |
111 | pub fn get_file_id(path: impl AsRef<Path>) -> io::Result<FileId> {
    |        ^^^^^^^^^^^
    = note: the item is gated behind the `unix` feature
note: found an item that was configured out
   --> C:\Users\rybop\.cargo\registry\src\index.crates.io-6f17d22bba15001f\file-id-0.2.1\src\lib.rs:121:8
    |
121 | pub fn get_file_id(path: impl AsRef<Path>) -> io::Result<FileId> {
    |        ^^^^^^^^^^^
    = note: the item is gated behind the `windows` feature

For more information about this error, try `rustc --explain E0432`.
error: could not compile `notify-debouncer-full` (lib) due to previous error

And now on my branch it fails nicely:
   Compiling bevy_asset v0.12.0 (C:\PROJECTS\bevy\crates\bevy_asset)
error: The "file_watcher" feature for hot reloading does not work on WASM.
       Disable "file_watcher" when compiling to WASM
 --> C:\PROJECTS\bevy\crates\bevy_asset\src\io\mod.rs:2:1
  |
2 | / compile_error!(
3 | |     "The \"file_watcher\" feature for hot reloading does not work \
4 | |     on WASM.\nDisable \"file_watcher\" \
5 | |     when compiling to WASM"
6 | | );
  | |_^

error: could not compile `bevy_asset` (lib) due to previous error

You guessed the point of the change, right- I just want to make clear what build is failing if it has to fail.

Add "Fixes #10507" to the PR description please. So that the issue gets automatically closed.
You can click on the … on the top left of your PR description, and there should be an "Edit" option.

This seems completely non-breaking, as we still have the same behavior, just a different compilation error message. So I think it's a candidate for 0.12.1

Add "Fixes #10507" to the PR description please. So that the issue gets automatically closed.
You can click on the … on the top left of your PR description, and there should be an "Edit" option.

Description updated


retry cypress tests on failure

fixes: #8627
The cypress retry logic does not seem to actually be working for some reason and I cannot seem to get it to work. I'm not sure why exactly. I have reverted this to doing the retry purely in bash by trapping the exit code from the failed tests and triggering the retry.

should we close out https://github.com/pulumi/docs/pull/8247?

should we close out #8247?

not sure. @zchase?


Error when ConcatDataset: TypeError: super(type, obj): obj must be an instance or subtype of type

Hi,
when I try to torch.utils.data.ConcatDataset([dataset, dataset]) after from torchdrug import layers,  I have got the following error trace:
TypeError                                 Traceback (most recent call last)
Input In [4], in <cell line: 1>()
----> 1 torch.utils.data.ConcatDataset([dataset, dataset])

File /gxr/anaconda3/envs/yuto/lib/python3.8/site-packages/torch/utils/data/dataset.py:288, in ConcatDataset.__init__(self, datasets)
    287 def __init__(self, datasets: Iterable[Dataset]) -> None:
--> 288     super(ConcatDataset, self).__init__()
    289     self.datasets = list(datasets)
    290     assert len(self.datasets) > 0, 'datasets should not be an empty iterable'  # type: ignore[arg-type]

TypeError: super(type, obj): obj must be an instance or subtype of type

I am observing the same behavior, it is caused by these lines of code: https://github.com/DeepGraphLearning/torchdrug/blob/master/torchdrug/patch.py#L153
patch.py is a file that makes me really despair. It is causing huge, significant side effects in all your downstream code. Whenever you import torchdrug and afterwards work with any of:

layers and activation functions (torch.nn)
dataset (torch.utils.data.Dataset)
optimizers and LR schedulers (torch.optim.Optimizer)

you simply can not expect normal behaviour. Effectively, this makes torchdrug the worst possible dependency since it will break your downstream code in very unexpected situations. Therefore, I recommend against using it.

For some of these situations, ugly fixes can be made.
E.g., to get back the normal behaviour of torch.nn you can do:
nn.Module = nn._Module

This is because when torchdrug overwrites the default nn.Module it at least saves the original one in nn._Module.
However, the situation for this issue is different. It's caused by overwriting all Dataset classes that are child classes of torch.utils.data.Dataset (e.g., ConcatDataset) with a configurable class that mimics its behavior. The configurable class is obtained here and then it is assigned in lines 158 respectively 132. The problem in this overwriting process is that python is smart enough to recognize that this configurable class is not the real class but only a dummy that is mimicking it. Therefore some sanity checks that are executed when calling the constructor of ConcatDataset are failing.
In particular:
issubclass(torch.utils.data.ConcatDataset, torch.utils.data.dataset.Dataset)

normally returns True (you can see from the native torch code that ConcatDataset is a subclass of Dataset).
but after executing patch.py (or equivalently after running import torchdrug) this issubclass test will return False.
Subsequently, you observe this bug in the constructor of ConcatDataset @ChifengWoo because the datasets you provide are no longer instances/subtypes of the real ConcatDataset.

I dont see an obvious way to fix this (I tried with importlib.reload) but I was not successful.
It would be really good to hear the opinion from @KiddoZhu @Oxer11 and all the other developers.
Why did you not just use class inheritance to define your own datasets, modules etc with customized behavior?
And what are your plans to improve torchdrug and stop it from silently, invasively breaking downstream code?


Refactor recommender signature

This PR implements a breaking change by refactoring the recommender signature such that:

it accepts an optional Objective
it expects training data as a single dataframe in experimental representation

Apart from fixing certain responsibilities (e.g. Campaign now only acts as a meta-data handler, just like it is supposed to) this enables/prepares several new features:

Users can directly use Recommenders as entry point instead of being forced to go via Campaign
Because recommenders now operate on experimental representations, they can offer the same interface back to users, e.g. when exposing model-internal things such as acquisition functions and surrogate models
Additional pre-processing steps that require access to the experimental representation (such as data-augmentation) can now happen inside recommenders

Hi @AVHopp, this PR might come somewhat surprising to you because it is contrary to what we have agreed on, but there have been new developments that require us to prioritize this. Can give you an update in a call 🙃


a problem in colab

when I run it in colab, I faced a problem like this

Can you tell me how to solve it?

Sorry for the inconvenience you're facing.
I have just checked the Colab, and you're right. Thank you for notify me.
There must be someone who forgot to create a Copy of the Notebook BEFORE running the code. So my original Colab folder has been modified, which caused the issue.
I'll reply to this issue as soon as the Colab is fixed.
Please stay tuned and check back later.
In the meanwhile, you can run the code on your local machine. Thanks for your patience!

We do not have plan to maintain Google Colab anymore. Sorry for the inconvenience.


Added threshold to joystick axis

Please test this with your controller before merging!
When i tried playing nothing with my X-Box controller, the player just moved to left until i pushed the stick to the right.
Mainstream game engines like unity have a threshold, until a joystick movement is considered to be a real movement, so i added this here aswell. The value is kinda arbitrary though.
If you have a better solution or you don't care, feel free to ignore this, but for me the game was hard to play

@Funkschy looks good to me! Thank you for the contribution!


NestedFrame.query() should handle mixed base and nested columns without erroring

Today, NestedFrame.query checks the expression it's given and raises ValueError if it mixes nested and base columns.  This is because in order to handle such expressions correctly, it would need to tease out the sub-expressions that are strictly against the nested columns (by traversing the abstract syntax tree of the input expression), apply and re-pack into an intermediate result, and then apply the base column expressions to this intermediate result.
In an expression like a > 2 & nested.flux > 50, for example, the user would expect the resulting NestedFrame to have no a values which were <= 2 and no nested.flux values which were <= 50.  And in an expression like a > 2 | nested.flux > 50, the user would still expect to retain rows where a <= 2 so long as it had some nested.flux > 50, but within those rows, they wouldn't expect to see any nested.flux <= 50.  For those rows where a > 2, though, they'd expect to see all the nested.flux rows.  In other words, as soon as there is mixed-level expression, the nested rows sometimes need to be queried and repacked before continuing, or at least that should be the final effect.
Logically, if there was a method to unpack all nests and broadcast all base columns across them, then we would take the result of self.eval(expr) and do something like self.flatten_all().loc[result].repack_all(), but this would likely not be performant.

I’m against the broadcasting approach: it may cause memory usage to explode, while one of the core ideas of nested-pandas is to never have the “joined” version of the base and nested columns. We either need to find another way to do it or not implement this feature.


Fix search results on ecosystem page

PR Summary
Searching for “okx” in the search input on the ecosystem page causes the site to crash. The root cause of this is that there is no desc in the fetched “OKX NFT Marketplace” data. To fix this error, when we add a condition for the desc data by changing text={desc} on line 188 in the ProtocolCard.tsx file, this error disappears.

Checklist

[ ] There are breaking changes
[ ] I've added/changed/removed ENV variable(s)
[ ] I checked whether I should update the docs and did so by updating /docs

Description
When we type the word “okx” in the search input on the ecosystem page and "OKX NFT Marketplace" is encountered on page 26 of the data on the ecosystem page, an error occurs that causes the site to crash. This error is the absence of the desc field in the “OKX NFT Marketplace” data retrieved. To fix this error, we need to check for the presence of desc data by adding a condition to text={desc} on line 188 in the ProtocolCard.tsx file.
One of the main causes of this error is the absence of desc data. In this case, the existing code generates an error by trying to process data that doesn't exist, causing the site to crash.
We check for the presence of desc data by updating this part of the code to text={desc || ""}. If the desc data is not present, we can avoid the error by skipping this part of the code. This makes the application more robust and fault tolerant.

Thank you very much for pointing out this issue. We’ll use another method to solve it.


Windows CI: test-unit on pkg\archive part 2

Signed-off-by: John Howard<EMAIL_ADDRESS>Part 2 of porting the unit-tests for pkg\archive to Windows CI (part 1 @ https://github.com/docker/docker/pull/20284). There's a lot of tests that now pass, and several annotations in there as for further work to do.

I think we should stop using /temp and Getenv("TEMT") altogether. Use ioutil.TempDir should do the right thing in either OS:
https://golang.org/pkg/io/ioutil/#TempDir

@calavera Generally, yes, I agree. However, in the particular case you're citing here, it's actually going to complicate the porting of the tests. The reason being that as some things are being through APIs which require "real" (c:\foo\bar) Windows paths, and others through bash commands which requires "fake" (/c/foo/bar) Windows paths. Golang has no such capability built in. Sure, it's an achievable goal, but in the context of the actual changes being made here, IMO the extra work right now doesn't seem worth it yet. Certainly as a potential follow-up PR though.

You're going to have hundreds! Grep the code for 'TODO Windows'... Honestly think it would be defeatist to have issues for all of them.

@jhowardmsft Not an issue for each one, a tracking issue for all of them (a checklist of things we have to look at).

Sure. Feel free to open an issue. But I really don't see the need when it could (will) take many, many months and multiple releases to get resolved. cf., it's not like there's an issue open for every matching "FIXME:" and "TODO" on the Linux side....

@jhowardmsft don't read too much in my comment, I just noticed there's a lot of TODO's added, and was wondering if there's a desire to have a tracker issue (using the "checkbox" notation) to track progress.

LGTM

LGTM


remove superjson dependency

closes #209
This is a big bundle size reduction for the library itself (see below, -44.93% on what we ship to the browser with the Next.js package), but beyond that it also cuts down a lot on data transported by the library, as SuperJSON adds a lot of extra bulk in their serialization format that we just don't need.
That said, some of our users might have non-serializable values in the cache and also want that transported over, so I'm leaving an escape hatch: stringify and revive can be configured.

I tried to explain this better. I'm really sorry for the 🤯  - I had a ton of difficulties wrapping my mind around what needs to be done here and what's already implicitly happening, too, but I didn't really know how to explain it better (without half a page of text, at least).


Working branch

Added test.md in .gitignore
Created this pull request. Will be posting this without merging.
Made Changes to the readme file according to the Microverse Template.

STATUS: CHANGES REQUIRED ♻ :
Hi @misalazeem ,
Good job so far! There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights 🎩 👍

The markup has been created 👍

Required Changes ♻

[REQUIRED] Kindly make sure your linter test is passing 👍 as per the project requirements
[REQUIRED] I suggest adding a professional README flle to your project as provided in this template  to make your project professional.
Check the comments under the review.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

So lighthouse linter error wont go away. The problem is it keeps saying this: "ChromeLauncher Waiting for browser"
Then  returns with the following error
"[0321/174605.416109:WARNING:sandbox_linux.cc(393)] InitializeSandbox() called with multiple threads in process gpu-process.
[0321/174605.855580:WARNING:bluez_dbus_manager.cc(247)] Floss manager not present, cannot set Floss enable/disable.
Unable to connect to Chrome
Error: Process completed with exit code 1.
##[debug]Finishing: Lighthouse Report"
Kindly guide.


Statsd server consumes more memory than it should

And consumption grows over time so this could be some memory leak here

Fixed in 732cac49e0b7821890a0b2be9c09918448df33fc


WIP: feat: Add charge prices table

Description

Add charge prices table in drawer with data.
References

#1688
https://app.zenhub.com/workspaces/team-volt-60a61059028fd7000e49956e/issues/energinet-datahub/geh-charges/1688

Codecov Report

Merging #1061 (c860d44) into main (e0a1916) will decrease coverage by 0.78%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1061      +/-   ##
==========================================
- Coverage   69.96%   69.17%   -0.79%
==========================================
  Files          18       18
  Lines         263      266       +3
  Branches       25       26       +1
==========================================
  Hits          184      184
- Misses         69       72       +3
  Partials       10       10

Impacted Files
Coverage Δ

...ataHub.WebApi/Controllers/ChargeLinksController.cs
100.00% <ø> (ø)

...ce/DataHub.WebApi/Controllers/ChargesController.cs
33.33% <0.00%> (-8.34%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


tests/doc: Fixed group names and docs required of some modules

Contribution description
This new PR organize all test group for the documentaton. Was fixed renaming all the test doxygen groups. this adds some file that didn't exist in some test files.

Testing procedure
run make docs:
make docs

Issues/PRs references

Looks good, only do a squash and ready to go ahead

Ok, squash is already :+1:


Updated 'Getting Started Managed' to fix command issue

Flashing command for ESP32 targets was missing the --update option

 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.

@DogFive pelase sign the CLA! 😉


add support for output of certs so kubectl can be configured

This would close issue #257. After certs are generated, they are copied from the remote master[0] to a directory in the kargo root called output/certs/. This will allow the kubectl config commands to be run easily.

Following our chat on slack, i think we should add it as an option configure_kubectl: false (default false).
Then we can :

use kubectl config to configure it
or create the file ~/.kubectl/config

yeah I like the default to false.. but have a way to spit it on disk.. with all those slurp calls in gen_certs.yml, isn't there a way we could construct a fully fledged kubeconfig with a template ?

just a thought...
Do you think its good idea to add template for config.yaml that can be used with i.e
kubectl --kubeconfig=./config.yaml get pods
with content of certificates using:
certificate-authority-data: REDACTED
client-certificate-data: REDACTED
client-key-data: REDACTED
As @idcrook mentioned in #908  - relative paths may cause some problems.

cc @bogdando @mattymo

@pskrzyns @flyingcougar Let's discuss with @rsmitty. I'm ok with updating this patch, but this PR doesn't allow other owners to update it.

I'd love to see this PR happen. I'd say the best plan is probably to open another PR with the updated code and I'll close this one out.

@rsmitty Ok, thx for info

@rsmitty Going way back for this one but this can be closed as of #1647, right?


Add new Methods for Button Navigation. Shift, Reset, Center and ZoomBy

I added a few new Methods to Navigate the image with Button Clicks.
    resetImage: function () {
        _resetImage.call(this);
    },
    centerImage: function () {
        _centerImageHorzVert.call(this);
    },
    shift: function (direction) {
        _shift.call(this, direction);
    },
    zoomBy: function (delta) {
        _zoomBy.call(this, delta);
    }

resetImage: Fit the Image in the Wrapper and center horizontal and vertical
centerImage: center the Image horizontal and vertical
shift: param direction (up,right,down,left) to shift the image percentage
zoomBy: zoomBy a delta value (1,-1)

Very cool - thanks for the PR.  I'll merge this in when I get time to pull and review.

@luebke By using  centerImage() function, It fixed an error on ios where rotating an image would result in a black image. When calling centerImage() after rotate(), it seems to have fixed whatever centering error was being caused by rotate. @thedustinsmith  I definitely recommend adding that function and documenting it as a method on https://foliotek.github.io/Croppie/.

Any news on this please? Can we see the changes?

Hi,
This functions look great
Are them already implemented?
How to create the new buttons and call the functions?
Regards


Add an option for consul http check

Currently there are only options to enable TCP/TTL service level checks in consul plugin. Need to extend it to support HTTP health checks as well

Would you be willing to implement this and open a PR ?

Can you please review https://github.com/go-micro/plugins/pull/79 ?


Window Maximization Behavior Consistency Issues

Describe the bug
Objective:
Lock the window size and prohibit the user from modifying the window size and maximizing it.
This can be achieved with the following code.
var presenter = (OverlappedPresenter)AppWindow.Presenter;
presenter.IsResizable = false;
presenter.IsMaximizable = false;
But double clicking on the title bar still maximizes it. While I suspect that customizing the title bar would solve this issue, I think that when the developer sets (presenter.IsMaximizable = false;), the ability to maximize the title bar on double click should be disabled.
I don't know if there is a way to disable "double-click titlebar maximization" other than "custom titlebar".
Steps to reproduce the bug
        var presenter = (OverlappedPresenter)AppWindow.Presenter;
        presenter.IsResizable = false;
        presenter.IsMaximizable = false;

Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.5.0: 1.5.240227000
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Is anyone working on fixing this issue or at least provide a workaround?

An usual workaround in Win32 is to intercept WM_SYSCOMMAND and SC_MAXIMIZE
(I tested in Windows App SDK 1.6.240829007, with SetWindowSubclass to intercept SC_MAXIMIZE (return 0))

An usual workaround in Win32 is to intercept WM_SYSCOMMAND and SC_MAXIMIZE (I tested in Windows App SDK 1.6.240829007, with SetWindowSubclass to intercept SC_MAXIMIZE (return 0))

Thanks, I had seen that elsewhere but preferred to stay away from hooking into WndProc and so on. This bug affects such a basic functionality that I simply don't understand why it hasn't been given the highest priority. This is the kind of issue that fuels the "WinUI is dead" kind of comment.

This is the workaround that work for me. Already cleared out the critical System.ExecutionEngineException
at this line
return CallWindowProc(originalWndProc, wndHwnd, msg, wParam, lParam);
Happened when I aggressively drag the titlebar across the screen lol.
Took me almost 2 days to figured this out. Now double clicking the title bar no longer maximize the window.  Idk why the basic things like this haven't implemented yet in WinUI3.

Create a class name NativeWindowHelper.cs

using System;
using System.Runtime.InteropServices;
using Microsoft.UI.Xaml;
using WinRT.Interop;

namespace YourNamespace
{
    public static class NativeWindowHelper
    {
        private const int WM_NCLBUTTONDBLCLK = 0x00A3; // Non-client left button double-click
        private const int GWLP_WNDPROC = -4;

        // Static field to hold the delegate, preventing it from being garbage-collected
        private static WndProcDelegate _currentWndProcDelegate;

        // Delegate for the new window procedure
        private delegate IntPtr WndProcDelegate(IntPtr hwnd, uint msg, IntPtr wParam, IntPtr lParam);

        public static void DisableDoubleClickMaximize(Window window)
        {
            var hwnd = WindowNative.GetWindowHandle(window);

            if (hwnd == IntPtr.Zero)
            {
                System.Diagnostics.Debug.WriteLine("Invalid window handle. Cannot hook window procedure.");
                return;
            }

            // Store the original WndProc and assign the new one
            IntPtr originalWndProc = GetWindowLongPtr(hwnd, GWLP_WNDPROC);
            if (originalWndProc == IntPtr.Zero)
            {
                System.Diagnostics.Debug.WriteLine("Failed to retrieve the original WndProc.");
                return;
            }

            _currentWndProcDelegate = (wndHwnd, msg, wParam, lParam) =>
            {
                if (msg == WM_NCLBUTTONDBLCLK)
                {
                    // Suppress double-click maximize
                    System.Diagnostics.Debug.WriteLine("Double-click maximize suppressed.");
                    return IntPtr.Zero;
                }

                try
                {
                    // Ensure parameters are valid before calling originalWndProc
                    if (wndHwnd != IntPtr.Zero && originalWndProc != IntPtr.Zero)
                    {
                        return CallWindowProc(originalWndProc, wndHwnd, msg, wParam, lParam);
                    }
                    else
                    {
                        System.Diagnostics.Debug.WriteLine("Invalid parameters in WndProc call.");
                        return IntPtr.Zero;
                    }
                }
                catch (Exception ex)
                {
                    // Handle exceptions to avoid crashing
                    System.Diagnostics.Debug.WriteLine($"Error in WndProc: {ex.Message}");
                    return IntPtr.Zero;
                }
            };

            try
            {
                // Hook the new WndProc
                IntPtr result = SetWindowLongPtr(hwnd, GWLP_WNDPROC, Marshal.GetFunctionPointerForDelegate(_currentWndProcDelegate));
                if (result == IntPtr.Zero)
                {
                    System.Diagnostics.Debug.WriteLine($"Failed to set new WndProc. Error: {Marshal.GetLastWin32Error()}");
                }
            }
            catch (Exception ex)
            {
                System.Diagnostics.Debug.WriteLine($"Error hooking window procedure: {ex.Message}");
                return;
            }

            // Prevent garbage collection of the delegate (redundant but safe)
            GC.KeepAlive(_currentWndProcDelegate);
        }

        // Win32 API declarations
        [DllImport("user32.dll", SetLastError = true)]
        private static extern IntPtr CallWindowProc(IntPtr lpPrevWndFunc, IntPtr hWnd, uint Msg, IntPtr wParam, IntPtr lParam);

        [DllImport("user32.dll", EntryPoint = "GetWindowLongPtr", SetLastError = true)]
        private static extern IntPtr GetWindowLongPtr64(IntPtr hWnd, int nIndex);

        [DllImport("user32.dll", EntryPoint = "GetWindowLong", SetLastError = true)]
        private static extern IntPtr GetWindowLong32(IntPtr hWnd, int nIndex);

        [DllImport("user32.dll", EntryPoint = "SetWindowLongPtr", SetLastError = true)]
        private static extern IntPtr SetWindowLongPtr64(IntPtr hWnd, int nIndex, IntPtr dwNewLong);

        [DllImport("user32.dll", EntryPoint = "SetWindowLong", SetLastError = true)]
        private static extern IntPtr SetWindowLong32(IntPtr hWnd, int nIndex, IntPtr dwNewLong);

        private static IntPtr GetWindowLongPtr(IntPtr hWnd, int nIndex)
        {
            return IntPtr.Size == 8 ? GetWindowLongPtr64(hWnd, nIndex) : GetWindowLong32(hWnd, nIndex);
        }

        private static IntPtr SetWindowLongPtr(IntPtr hWnd, int nIndex, IntPtr dwNewLong)
        {
            return IntPtr.Size == 8 ? SetWindowLongPtr64(hWnd, nIndex, dwNewLong) : SetWindowLong32(hWnd, nIndex, dwNewLong);
        }
    }
}

In your MainWindow or any other window you don't want the double click maximize.

You can call the
NativeWindowHelper.DisableDoubleClickMaximize(this);
Example
using Microsoft.UI.Xaml;
using WinUIEx;

namespace YourNamespace
{
    public sealed partial class MainWindow : WinUIEx.WindowEx
    {
        public AddButtonWindow()
        {
            this.InitializeComponent();
            this.CenterOnScreen();
            this.Height = 800;
            this.Width = 800;
            this.IsResizable = true;
            this.IsTitleBarVisible = false;

            ExtendsContentIntoTitleBar = true;
            SetTitleBar(YourCustomTitleBar); // Use Custom TitleBar

            // Disable double-click maximize behavior
            NativeWindowHelper.DisableDoubleClickMaximize(this);
        }
    }
}

Tested with x86 and x64 and Window App SDK 1.6.240923002
Tips
Notice how I can easily set the window size with just  this.Height = 800; ?
I use WinUIEx extension. It is really helpful when it comes to customizing the window.
Pardon for my bad English. it is not my native but I will try my best to spoon-fed everyone with the little effort and knowledge that I have.


Is it possible to completely disable color output?

I've been messing with it and it seems like it all depends on the tools you use. For example, in order to remove color output from ipa, I had to make a custom patch here: (https://github.com/keatongreve/fastlane/commit/1ac86703c24a1f56c9b6a017e902d308e6fc8394)
It changed my build output slightly, the > symbols indicate that the output of ipa is no longer styled, but the fastlane output still has color codes at the beginning. Here's a sample of output from TeamCity:
[20:17:57] [37m[15:17:57]:  [0m [32m----------------- [0m
[20:17:57] [37m[15:17:57]:  [0m [32m---  [0m [32mStep: ipa [0m [32m --- [0m
[20:17:57] [37m[15:17:57]:  [0m [32m----------------- [0m
[20:17:57] [37m[15:17:57]:  [0m[SHELL COMMAND]:  [33mset -o pipefail && krausefx-ipa build -w "MyApp.xcworkspace" -s "Scheme" -c "Adhoc" --verbose | xcpretty --no-utf [0m
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-CocoaAsyncSocket [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-Appsee [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-CocoaLumberjack [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-AFNetworking [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-FLKAutoLayout [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-DDURLParser [Test]
[20:18:24] [37m[15:18:10]:  [0m[SHELL]: > Cleaning Pods/Pods-Block-KVO [Test]

I love the color feature otherwise, unfortunately I'm stuck with TeamCity which doesn't support it.

Isn't there a plugin for TeamCity that enables the standard terminal colours? I don't think it's easily possible to get rid of colours in fastlane. How do you solve that in CocoaPods?

I did a bit of searching and I couldn't find anything. If you've found something I would appreciate a link!
I agree, the only way I could think of is writing wrapper methods around red/yellow/etc. Or piping all final output through sed.
Good point, I didn't think about one like CocoaPods since I don't run it in my lanes. We can't control the output of every action. I got lucky that the xcpretty call in the ipa action was added in fastlane instead of in shenzhen itself. I guess you could starting forking everything :stuck_out_tongue_closed_eyes:. sed is starting to sound like a better option.

You can close this if you want, I just wanted to share my thoughts. Thanks for the response!

To be honest, I don't think it's a huge problem to have a [0m in front of every line. But you're right, monkey patching the colour methods would probably be the way to go.

CocoaPods has --no-ansi option.
In that cases they indeed just monkey patch String.colorize.
See https://github.com/CocoaPods/CocoaPods/blob/84e84ad5159a618af4e9f253d56f99b1469300a3/lib/cocoapods/command.rb#L82-L84

@KrauseFx how would you like this feature to work?
A --no-ansi option to the fastlane command? Or a directive in the Fastfile?
I'd be happy to send a PR with what you prefer.

@keatongreve I'm also using TeamCity and was having the same issue.
Until there's a builtin way to disable colors, you can simply add this String.send(:define_method, :colorize) { |string, _| string } to the top of your Fastfile.

@jeanregisser Perfect. :+1:

@jeanregisser I'd have to further look into it, but probably adding an option to /bin/fastlane should do the job.

FYI, TeamCity 9.1 now supports ansi colors! https://confluence.jetbrains.com/display/TCD9/Build+Log#BuildLog-ANSIstyleColoringinBuildLog

Just for the record, using Xcode & Xcode Server, colorized output seems to be a problem.  Viewing the log files from the build server through Xcode causes Xcode to hang.  Also, downloading the log files and viewing them in Finder, the QuickLook preview hangs Finder as well.  In this instance, I'm calling sigh from some python scripts.  (They python scripts handle the integration with Xcode Server.)  If I strip all the colorizing characters from sigh's output in python and then send the text to stdout, all of these problems go away.
So, looks like Xcode, Xcode Server, and Finder have weird issues.  I've got a workaround for now, but a flag to disable that would be great.
Xcode 7.0.1, Xcode Server (OSX Server 5.0.4), El Capitan (OSX 10.11.0)

:+1:
I'd also like to disable xcpretty altogether (maybe that's already possible?).

@armadsen right now that is not possible. But there is MR #626 that can do that. If it is merged, that will be possible to do.

@KrauseFx  I think has just added both of these features.

Yeah, this will be possible with the new UI abstraction layer, which will be rolled out over the next few weeks to all tools. Right now it's already used in match
All you have to do is set the FASTLANE_DISABLE_COLORS environment variable :rocket:

This now works across all tools. Make sure to install the latest version of fastlane and set the environment variable using
export FASTLANE_DISABLE_COLORS=1

@KrauseFx The environment variable does not work. I have Fastlane version 1.48.0 and output is still colorized. Where exactly should I put this variable please?

@tomassliz This will be updated to all tools over the next weeks :+1:

@KrauseFx Amazing 👍 It will be nice for feature release notes mention this behaviour, but anyway thank you for amazing commitment!

@tomassliz You mean something like here? https://github.com/fastlane/fastlane/releases/tag/1.48.0

Yes.

For people coming from Google, use the FASTLANE_DISABLE_COLORS ENV variable 👍


Web project separation

Separate Web project into web (angular) client & web service.
This would allow us to better fine tune the libraries for each case.
But first solve #9

Was solved in 0.42.0


Transparent frameless windows no longer work

It appears that the latest release v0.26.0 broke transparent frameless windows, a small grey bar of chrome now appears across the top in OSX. I'm not sure if this is also the case on other platforms.

Almost certainly caused by https://github.com/atom/electron/pull/1585.
If you use 'standard-window': false in your window options, does it work?

@joshaber that fixes it, thanks! I'd say this issue should remain open as the default behaviour here is now quite odd - you need to get the combination of window options just right

Yup, you're right. We should infer that from transparent.


Ui animation system

Internal PR
Ping @viveret

So is refactoring a good or bad sign of my code? Otherwise it looks a lot like the structure I had so it should work good

Ping @msteiger

This is what it could look like: https://youtu.be/lNQTJu6GADw

@msteiger Looks great!

Good starting point from your side, @viveret - let me know what you think of the code if you find the time. A few things still need to be adjusted (ConsoleScreen, integration of all menu buttons, ESC to close, etc) before this is ready.

@msteiger Awesome. I think the code looks great, and renaming some of the classes was a very good choice. I also think the andThen method is fantastic and easy to understand. Perhaps there should be some way to create animations in xml or text files and then import them whenever needed so that similar animations can be easily re-used and edited w/o needing to rebuild.


wow.js visible only in the viewport and not on scrolling

Good morning,
I'm on Codeigniter and I don't know why but on scrolling down all wow items don't change their visibility as you can see:

this is my head:
<!DOCTYPE html>

  <link rel="stylesheet" href="css/animate.css">

this is my code:

    <?php
    foreach ($top->feed->entry as $key => $value)
    {
        $value->title->label = str_ireplace( "- ".$value->artist->label, "",  $value->title->label);
        if($key >= $this->config->item("items_top"))
            continue;
        $image = $value->image[2]->label;
        if($image == '')
            $image = base_url()."assets/images/no-cover.png";
        $image = str_ireplace("170x170", "200x200", $image);
    ?>
    <div class="wow bounceInUp col-xs-6 col-sm-4 col-md-3 col-lg-2" data-wow-delay="<?php echo $delay; ?>s">

                    <img class="r r-2x img-full " src="<?php echo $image; ?>" style="no-repeat top center; -webkit-background-size: cover; -moz-background-size: cover; -o-background-size: cover;  background-size: cover;"/>

    <?php $delay+=0.1; }?>

This is the link to my website:
http://dreammy.org/music

This is because you're not scrolling the entire page, you're scrolling a
, namely the one with the scrollable class. This is currently not supported by WOW.js, see #65. For now, I'd suggest you don't use WOW.js with scrollable divs, simply scroll the entire page.

Thank you Attilah, however it's very bad for me


Anonymous guests can be made read-write, is this intended?

I've noticed two inconsistencies with regard to anonymous (unauthenticated) guests:

The docs say unauthenticated guests may only have read-only access, but I'm able to grant them write access after initially giving them read-only access.
Initially selecting the option to grant read-write access when they request to join: this appears to only grant read-only access.

Version: 1.0.5864

It is possible to give write access after initially giving read-only access.
For 2, can you confirm the clients used to reproduce this? So far we've been unable to reproduce.

Host:
OS: macOS
Version: 1.80.0-insider
Commit: 6545c4c2671127c323182225963fcd732e1fbcc5
Date: 2023-06-15T05:54:42.613Z
Guest:
OS: Ubuntu
Version: 1.78.2
Selecting read-write access via the initial join notification results in "Joined (Read-Only)" in the guest's status bar, and all shared files are read-only. Right-clicking on the guest and selecting "Make Read/Write" actually grants read/write access.

bump

This is still occurring.


Replace repo with other redis image

I've spent several days building a complete new redis image.
I would to request an evaluation from you, to see if we can use this as the basic redis image instead of sameersbn/docker-redis.
Source repository:

https://github.com/GJRTimmer/docker-alpine-redis
https://gitlab.timmertech.nl/docker/alpine-redis

Both source repositories are the same.
Overview:

alpine:3.4
S6-Overlay
redis:3.2.5
master / slave support
full sentinel support (optional, can be activated from docker-compose)
full docker support for master / slave / sentinel
USERMAP_UID/GID support which was requested with an issue on your repo.
ability to run either redis-server or redis-sentinel or both within the same image.

For more information about sentinel: https://redis.io/topics/sentinel
I've thoroughly tested the master with multiple slaves with sentinel, automatic failover etc..., everything is working.
Also everything is documented within the README and I've provided 7 docker-compose examples, see the readme for more information.
I really hope you like it, and are willing to evaluate it and look at the possiblity to either reference it within docker-gitlab or fork my github repo.

I wonder, why there's a gitlab specific redis image at all. Why not simply use the official redis image? https://hub.docker.com/_/redis/
Is there any gitlab specific modification in here?

Hey folks,
why not using official images?


Ability to invoke all std benchmarks via jmh

Closes #7489
Pull Request Description
Important Notes

All the benchmarks have configured phases (number of iterations/seconds for warmup or measurement) such that the total number of benchmark invocations per warmup and measurement is somewhere around 100 at least and their score is not less than 1 ms.
There is yet no API for providing setup or teardown methods.

An example of how lazy setup can be done for Text/compare is in https://github.com/enso-org/enso/pull/7519/commits/3ca6cd79c30655790fde40b9fb682d258edfaae8
Without the lazy setup, the collection of benchmarks in Text/compare takes 7 seconds, with lazy setup, the collection takes 160 ms.

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

As far as I can say we don't need teardown right now. As such, please let's keep this pull request simple and focused. Let's use existing Enso language lazy atom fields concept to get proper setup behavior without changing the BenchProcessor, without introducing set_setup and set_teardown and without having problems to invoke these methods from JMH.

Sounds like a good idea. I reverted all the commits that introduced the special API for setup and teardown methods. For future reference, it is kept in a new issue https://github.com/enso-org/enso/issues/7566.

All the benchmarks are now successfully collected with project std-benchmarks; clean; Bench/compile; Bench/run -l, in 9 seconds. All the benchmarks are also refactored to use lazy data loading so that the data is not evaluated during the benchmark collection, but rather during the first startup.
Let's create a new GH Action now and try to run it on the CI.

The GH Workflow is added in https://github.com/enso-org/enso/pull/7597. Since we need to first merge #7597 to try the new action, let's already merge this PR. I will also revert the commits that contain the CI-related changes.


docs: add example for websocket with bindings

adding a missing websocket example that many people ask for
preview here: https://playground.asyncapi.io/?url=https://raw.githubusercontent.com/derberg/asyncapi/add-websocket-example/examples/2.0.0/websocket-gemini.yml

@fmvilas please approve again. I fixed server name I noticed during my life stream, left over from my asyncapi file about Shrek app :)

It is actually not included in the release. This was an unintentional release caused by the release automation initial run. 1.0 version of the spec was released a long time ago. The release mentioned by the bot was removed.

Not included in release 2.1 yet, 2.1 was mistakenly triggered by our automation.


Feature Request: Tabular results UI for EdgeQL queries

As I've been learning EdgeDB and EdgeQL and working with the EdgeDB UI, I've found that a number of my struggles boil down to data explorability.
To this end, I've come to realize that most of this could be solved by having the Query Editor able to display results in a tabular form, much like the current Data Explorer.
I'm aware of the Grafana plugin, however, having this support natively in the EdgeDB UI would be a great addition for many users, and the value-add could extend to cloud users as well.
Initial Implementation:

Read-only table results (even having this support would provide ~80% of the value right now)
Columns auto-populate from top-level properties in the query results
Text-only results (link support could come later)

Later implementations:

Edit individual table cells to update data - similar to current Data Explorer
Link-support (like the Data Explorer - #51 also becomes relevant)
Tagging individual properties, regardless of depth, to appear as columns in the query results
More advanced display features (colouring cells, formula support, sorting and filtering, re-ordering columns, customizable footer (mean / sum calculations, formulas, etc...), bar and other special field display types)
Dashboard support: save queries, and display multiple tables on the screen at once (like a Grafana dashboard)
Auto-updating tables (useful for dashboard view)

We currently use metabase for many of the things cited here. Would be a plus to get the data explorer view (even in read only mode).


Delete Venafi TPP test certs created by e2e tests

Currently our TPP test instance has a bit over 7000 unexpired certs, large part of them created by cert-manager e2e tests.
We might want to look into cleaning up Venafi TPP certs as part of e2e test cleanup.
It looks like the vcert library currently does not have an option to delete certificates - see this issue.
/kind cleanup

We would welcome the ability to optionally delete and/or revoke certificates when the certificate object is deleted in k8s. We have had to implement this functionality ourselves as a webhook.