New function - resize database files

I'm looking for a function that would allow to manage file size. Something like Set-DbaDatabaseFile or Set-DbaDatabaseFileSize
As I see it it would be a wrapper around the alter database [...] modify file
In my opinion it would be useful to manually grow files and prevent autogrowth and it might be useful for VLF  management too. It should work with Get-DbaDatabaseFile.
Shrinking file doesn't have to be supported, but if it was to be supported I think ti should require something like a -Force parameter to prevent accidental shrinking
examples
1 Set-DbaDatabaseFile -Database mydb -File mydbfilename -Size 1024
this would resize the mydbfilename in mydb database on the localhost to 1GB only if the current size is less than 1GB. If it was already bigger a warning would be displayed explaining how bad shrinking files is and with details about how much of the space is actually used and suggesting use of -Force parameter if there is enough of free space in the file to shrink the file.
2 Set-DbaDatabaseFile -Database mydb -File mydbfilename -Size 1024 -Force
when the database file size is already greater than 1024MB it would attempt to shrink it to 1024
3 Set-DbaDatabaseFile -Database mydb -FileType ROW -IncreaseSizeBy 128
this would add 128 MB to all data files in mydb
4 Get-DbaDatabaseFile -SqlInstance myinstance | Where { ... } | Set-DbaDatabaseFile -IncreaseSizeBy 128
That's something I would use to grow files which I think should be grown by 128MB

I'm up for this command being added. A few notes/points:

We already have a command that handles shrinking files Invoke-DbaDatabaseShrink, so shrinking and the Force command should not be included in this command.
The verb Set sounds more like you are setting a property(ies) on a given file and not an operation of resizing. I'd say more appropriate command name would be Invoke-DbaDbFileSize or maybe just Invoke-DbaDbResizeFile.
In cases where the size passed in is less than the current size of the file we can just make note of the shrink command in the warning message.
An issue to consider is the value you pass in for size, we will have to contend with what unit size is (possibly a parameter for that). Especially if you want to support piping from Get-DbaDatabaseFile as it the size property is a DbaSize type, so we'd have to ensure we account for the raw value that is output (but not sure what that does in piping on that type).

I missed the Invoke-DbaDatabaseShrink and I agree with the Invoke- rather than Set- name. But since we have already *Shrink I think it would make sense to call this new command Invoke-DbaDatabaseGrow. With that change there is really no need nor room for the -Force parameter.
As for the units, I think the MB would be good standard and a good default if we need it to be configurable. I'll check what DbaSize type look like. If we really need to have it configurable we could follow the way the ALTER DATABAE MODIFY FILE works, where you can specify units in the value itself.
I'll have a look at it tomorrow, create a first draft implementation perhaps.

Fred and Klaas need to chime in on the command name.
I can see another option of having Set-DbaDbFileGrowth which would handle the auto growth properties of a given file. This would be more inline or match up to what classes in SMO would be used. A resize of a file is changing the growth settings, and if you want to shrink a file with high amount of free space anyway you can't shrink it below the initial file size....so that property has to he changed before you can shrink it.
To grow it you change or set the property as well. So set is likely the good verb to use...just matter coming up with good noun to use.

I think Set-DbaDbFile with various options (including growth settings or manual size manipulation) would be the way to go, unless that has too many options and needs to be split from a functionality perspective.
Note on the size input:
I'd go with byte (could use DbaSize as input type for that without issues). Reason: PowerShell has a real comfortable way of writing those numbers. Instead of this:
Set-DbaDbFile -Database mydb -File mydbfilename -Size 1024

you'd write this:
Set-DbaDbFile -Database mydb -File mydbfilename -Size 1024MB

I can update the class to also accept this input:
Set-DbaDbFile -Database mydb -File mydbfilename -Size "1024MB"

In case people get confused about strings

Wouldn't it make sense to keep the names consistent? The get function is Get-DbaDatabaseFile so I think it would make sense to keep the full Database word in any Set or Invoke function, or change the Get to be Get-DbaDbFile
Thinking about it I like both the ideas with Set-DbaDatabaseFile and Invoke-DbaFileGrowth. Set command could be used to modify the settings and if the setting is size, then it could trigger growth, that's how SQL Server behaves. But having an Invoke-DbaFileGrow would make sense to me too, I would use it to 'manually' grow the file in the maintenance window. That's my original use case
Get-DbaDatabaseFile | Where { ... } | Invoke-DbaFileGrow
And that would take the default growth settings and simply grow the file according to the configuration.  It could have an optional parameter to allow calls like
Invoke-DbaFileGrow -By 1024MB or Invoke-DbaFileGRow -By 10%

After thinking about it the Invoke-DbaDbGrow to complement the existing Invoke-DbaDatabaseShrink might be closer to what I need, so that is where I will start. I still think the Set-DbaDbFile might be useful, and I will implement it later too.
Unless there are any objections I will follow the Invoke-DbaDatabaseShrink implementation as much as practical but I think my primary use case will look something like this
Get-DbaDatabaseFile | Where { ... } | Invoke-DbaDbGrow where I will expect the files I pipe to Invoke-DbaDbGrow to be grown by whatever their configuration is, one increment only.
Invoke-DbaDatabaseShrink is more autonomic. If called on its own with database names it will go and shrink them all. Should I implement it similarly, where one could do something like
Invoke-DbaDbGrow -SqlInstance mysqlinstance -Threshold 5% and all database files which have less than 5% free space will be grown by one increment?
I'll start with the help comments and parameter with examples of usage I plan to implement it to take the discussion further. Than, if you don't mind, I'll open a PR just to help facilitate the discussion with code examples. I find it easier than constantly sending links to branches

@FriedrichWeinmann I like your idea to use DbSize so that we can do
Invoke-DbaDbGrow -By 128MB

but I would like it to easily work with Get-DbaDatabaseSpace which returns objects with properties like UsedSpaceMB, FreeSpaceMB. Do you guys have any plans to change those parameters to UsedSpace and FreeSpace respectively and make them use DbSize instead of an int type? If that's the direction in which 1.0 goes I can do the change, but I'd like your advise on how to limit the scope of breaking changes.

I see us going towards using DbaSize wherever possible, especially with the upcoming changes in https://github.com/sqlcollaborative/dbatools/pull/3383 as those significantly improve the user's ability to have control over display style.
Whether to remove existing properties, I'm not so sure - I'd mark that up for 1.0.0 as a switch-over task, rather than do it right now. That way we can administer all the changes in one bitter pill, rather than keep it as a constant issue.
Thus what I'd do right now for code already producing objects with those properties:

Add dbasize properties
Hide the old *MB properties from the default display
Create a changes-to-do-for-1.0.0.md file into the project root folder and list it there

    .PARAMETER DatabaseFileSpace
        A list of files as produced by Get-DbaDatabaseFreeSpace which are to be grown

    .PARAMETER Database
        The database(s) to process - this list is auto-populated from the server. If unspecified, all databases will be processed.

    .PARAMETER ExcludeDatabase
        The database(s) to exclude - by default this list is empty

    .PARAMETER AllUserDatabases
        Run command against all user databases only.

    .PARAMETER FileType
        Specifies the files types that will be grown

            All
                All Data and Log files are grown
            Data
                Just the Data files are grown
            Log
                Just the Log files are grown

    .PARAMETER When
        Specifies then condition that needs to be met for the file to grow. By default it is set to 10% which means
        when there is exactly or less free space in the file than 10% of the growth increment then the file will grow.
        Always the grwoth setting is used to calculate whether the condition is met and the -By parameter is ignored in that regard.
        It can be set to an absolute value like 128MB and then the file will grow if the free space is at or below 128MB.

        "Always" can be passed as value to ensure each file is grown regardless of conditions. Useful when piping files into the function.

    .PARAMETER By
        Specifies by how much the file should grow. Valid values include 5%, 128MB. By default the By value is not set and the file
        growth configuration will be used to determin by how much it should grow. If no unit is specified bytes are assumed.

        "Default" can be specified to explicitly use database file configuration.

How do you plan on defining what falls under AllUserDatabases?
By and When sound more like date parameters or ones used for pulling information based on a date range. Are there better names we can come up with for this functionality? Do you really need a When parameter (e.g. if I'm piping data from the space command I can filter it down from there "when" I want the operation, and would likely be the common use: Get-DbaDatabaseSpace -SqlInstance MyServer | Where-Object PercentUsed -gt 90 | Invoke-DbaDbGrowth...)
By in the description, you note it will use the increment for growth based on the file auto growth configuration if not provided. Do we really want that ability? If someone has not configured best practices on file growth and has it set to 10% growth, are we going to let that be done? I would require a size/value be provided regardless and not let this command handle any auto growing. It can lead to many mistypes and unecessary headaches.
DatabaseFileSpace this one I would disagree with because it should not be locked down to only allowing piping from Get-DbaDatabaseSpace command. I believe @potatoqualitee has been using InputObject and based on the type handle getting the needed information/properties from it.
FileType - I would not allow the use of All because best practice around file growth you can grow a data file from 50MB to 100GB in one operation. However, doing that with a log file will cause problems in other areas such as VLF count. The operation should be done either with a data file or a log file, but not both.

I believe Set-DbaDbFile needs to be written first and not jump the gun on writing command(s) that will end up just having duplicate code and/or end up needing a complete rewrite later down the road. Overall it should be a pretty basic command to write, even if right now it just sets the file size...we can expand that one down the road without breaking changes.
I'm still up in the air on invoking a growth event. Use of Set command to change the file size will ultimately invoke a growth event, I mean that is how you grow a file in increments anyway via T-SQL (e.g. grow a log file from 10MB to 20GB, I'd do that in a set size each time to lessen VLF count).

Thank you for the detailed critique @wsmelton.

How do I define user databases? Those are databases which are not system databases. And system databases are defined here https://docs.microsoft.com/en-us/sql/relational-databases/databases/system-databases
The above might not be that important. The only reason I have added AllUserDatabases parameter was to follow the example of already implemented function, the Invoke-DbaDatabaseShrink which has such parameter. To see how it does determine what is a user database I have read the code and it turns out the parameter is not really used internally. I'm happy to remove it from my implementation and list the one in the reference implementation to be removed for 1.0
By and When have time related meaning, but that is not the only meaning. Out of the two perhaps https://en.oxforddictionaries.com/definition/when has a stronger time connotation but still it is quite natural to say something like 'I will increase the size of the file when there is little free space left'. Perhaps If would be a better word to indicate condition?
Do I really need this When or If? I probably don't. SImilarily to the AllUSerDatabases I have added it to make the grow command more in line with what there is already for shrink. My prefered option was piping filtered list of files to be grown, but now I see how perhaps that percentage option could be useful to quickly do things like

Invoke-DbaDbGrow -When 10% -WhatIf

which would display the files that need to be grown and by how much and if I don't see any problems with it I could then run the command without -WhatIf But if that is against the design principles of dbatools, then I'm happy for it not to be there. I'd just suggest to change the Invoke-DbaDatabaseShrink

As for https://en.oxforddictionaries.com/definition/by I must disagree. It is perfectly natural. When there is not enough space I will _grow_ my file _by_ 100MB. I cannot think of a better word.
The use of auto growth configuration. For me it is very important, but that is because I'm thinking about using this command not on a single database, but on hundreds or thousands on multiple instances. Those are my databases, and I have configured the auto growth the way I want it, the way they need to be for various reasons so I'm not too concern that suddenly my files will start growing by 10%. This option allows me to grow all the files that need growing, by the pre-configured increment specific to the file in my maintenance window.
If you really want to discriminate against clue-less or accidental DBAs I suggest we remove the % option and change the behaviour so it skips (with a warning) any file that is configured to a percentage growth.
Do we need the ALL option in FileType? That's again just a copy from the existing method which I see as analogous. The log files shouldn't need to grow as frequently as data files. If they do, you probably need to think why it is, perhaps the backups are not working? Was it a one of maintenance type of action and it will be better than do a log backup, or wait those few minutes until the next scheduled one instead of growing the file? Perhaps when the files are piped into the grow function it should take any file that somebody piped in? Or should it display warnings when log files are piped to it unless there is -FileType ANY (I would probably use any instead of all in that case). But I think there should be a way to use Invoke-DbaDbGrow to grow the log file too.
I agree the function should take input from not only the Get-DbaDatabaseSpace command even though this one seems most natural in my use case and that is why I listed it there.
The FileType and disallowing ALL. What is the harm in this (or using ANY instead in my previous example) if we don't make it the default value (it should be DATA) and disallow percentage growth altogether?

I believe in code refactoring which allows me not to believe in the need of having Set-DbaDbFile necessarily first. Especially that the scope of Set-DbaDbFile is not well defined at the time, and most likely is out of the scope of my original interest that prompted me to work on this. You are right, to start with it might be a pretty basic command and perhaps even it could do it all without the need for Invoke-DbaDbGrow, but it was you who argued that the growth should not be implied in a set command (that was your first comment, second point you made). I do agree with you from 6 days ago, and disagree with you yesterday. The Invoke verb is better for the file growth. If we implement Set-DbaDbFile that does few things like configuring auto growth and then allows to set the file size... well, then perhaps that's a side effect we have to take, but I probably would explain in the documentation that if you want to grow or shrink a file you should use the 'Invoke' commands explicitly and they will change that 'setting' for you. Creating a simply Set-DbaDbFile that only grows a file goes completely against it.

Is there a code branch for this I might be able to take over and work on this, this will help me out with a presentation I'm doing and replace a SQL script with dbatools?

@tboggiano I'm not aware of any activity, so you are more than welcome to start working on something if you would like.
You can create a draft PR if you would like to have it reviewed as you work on it.

Hi,
I just wondered if the Invoke-DbaDbGrow was coming soon?
Looks like a good feature.


Test against PHP5.6, PHP7, & HHVM

Extensions don't compile. :-(

Same as with the event loop extensions :(. Looking into those first then moving on to ZMQ

The php-zmq extension is now compiling and passing a majority of tests in PHP7, so it might be worth revisiting this.

There you go, tests just passed on PHP 7, thanks for the heads up :+1:

For event loop pecl ev-beta is compiling under php7

Is there any news? Does this bundle support php7 yet?

This landed with #42


fyhj

to make it clear it's the same
https://github.com/alexeyneu/libbase58/blob/1ff61c979ce6cb8642ca3c723e919cf77d67f56e/base58_c.cpp#L109-L110


--group_by_length flag train/loss anomoly

What is the "odd train/loss" that is referenced to be caused by this flag? I have it on (and havent tested with it off) and my train/loss looks like this:

Is this consistent with the specific "oddness" the flag creates, or am I doing something else terribly wrong?

Sorry I meant to post this in alpaca-lora repo


[i18nIgnore] Remove Vue TypeScript troubleshooting section

What kind of changes does this PR include?

New or updated content

Description
Reflects the upcoming breaking change in https://github.com/withastro/astro/pull/5724. The troubleshooting section isn't an issue anymore as it has been fixed by Volar.
I also removed it for the translated docs.

@delucis I think this can be merged now? Upstream PR is merged and released.

Hmm. Interesting question. At the moment most users will still be on a 1.x version, so I wonder in this case if it makes sense to only remove once we actually release 2.0 to stable not just beta.
General principle I’m thinking:

For new features available in v2 beta: merge early and flag they require v2
For general housekeeping like this where our advice is currently valid, I’d lean towards merging late once v2 proper is out.

What do you think?

Makes sense! And, if a problem simply isn't going to come up in v2, then there's less danger having "advice for when this happens" there, because no one will read it!  ;)
We could update (rather than remove the thing entirely) with a note saying that upgrading to v2 (even beta) is a solution, as this problem is fixed in v2?

I think the only thing tying this change is that Astro 1.0 would have a hardcoded link to this section of the docs. In Astro 2.0, the link is removed so the docs can be removed as well. The fix for the section is to upgrade Volar (VSCode extension), and given most extension auto-updates, it could also be safe to remove this section early if needed 🤔

So currently if a user runs into this, they would be shown a link to this section of the docs? That would make me lean towards remove later rather than earlier.
Unless you’re saying that most people will already have upgraded Volar so this error is probably already rare?

Ah I miss this notification. Yeah they would be shown the old link, but only if they try to integrate Vue and and a JSX framework. Plus yes that Volar should already be upgraded to latest for most users so they shouldn't have an issue with it today. So I think this is a pretty rare case.

Cool, sounds reasonably safe then to merge when you want.


hexens audit #4: voting power defaults to 0

VOTING POWER DEFAULTS TO ZERO
SEVERITY: Low PATH: HATToken.sol
REMEDIATION: self-delegate destination on mint and transfer if delegates[dst] is zero, alternately this could be done directly into _moveDelegates()
STATUS:
DESCRIPTION:
The delegatee is zero by default, until an explicit call to delegate() is made. As seen in getPriorVotes() and other related methods, the delegated votes are the only value that counts for voting power (not the actual balance). So by default, a tokenholder's voting power is zero, until he delegates to self. This is unintuitive and forces token holders to explicitly self delegate via an additional transaction before they can start voting.

(a) this is not really wrong behavior and makes sense for some use cases and (b) we have a code freeze on the hattoken en will only change it if there are Big Problems


LocalScanner: Fix-up the creation of missing file archives

Previously, mapNotNullTo() added the mapped elements to the wrong set,
and thus missingArchives was guaranteed to always remain empty.
Trivially fix this by assigning the variable properly. This is a fix-up
of 90de765.

Trivially fix this by assigning the variable properly.

Thanks for the fix, BTW!


TRON RxSwift Error Handling

I've been trying to handle RXSwift Request Errors
ErrorHandling:  APIError<FIAppError>(request: Optional(https://url), response: Optional(<NSHTTPURLResponse: 0x1c0238ee0> { URL: https://url } { status code: 404, headers {
    "Content-Length" = 36;
    "Content-Type" = "application/json; charset=utf-8";
    Date = "Fri, 30 Jun 2017 13:15:12 GMT";
    Etag = "W/\"24-nuP261L8lpDkCVKMtmlHgw\"";
    Server = "nginx";
    "Set-Cookie" = "sails.sid=djskdjks; Path=/; HttpOnly";
    Vary = "X-HTTP-Method-Override";
    "cf-ray" = "hshdjsd-LHR";
    "x-powered-by" = "Sails <sailsjs.org>";
} }), data: Optional(36 bytes), error: Optional(Alamofire.AFError.responseValidationFailed(Alamofire.AFError.ResponseValidationFailureReason.unacceptableStatusCode(404))), errorModel: Optional(company.FIAppError))

This is my FIAppError class.
class FIAppError: JSONDecodable {

    var errors: [String:[String]] = [:]

    required init(json: JSON) {
        if let dictionary = json["errors"].dictionary { <- This block isn't entered because it isn't a dictionary
            for (key,value) in dictionary {
                errors[key] = value.arrayValue.map( { return $0.stringValue } )
            }
            print("FinalError: ", errors)
        }
    }
}

And this is my RequestHandler
func provisionDevice(key: String) -> Observable<FIUser> {
        var params = [String: Any]()
        param["key"] = key
        let request: APIRequest<FIUser, FIAppError> = tron.request("".subUrl())
        request.method = .post
        request.parameters = params
        return request.rxResult().flatMap { Observable.just($0) }
    }

Subscribing to the response
fileprivate func fetchOrganizationInfo() {
        repository.provisionDevice(key: organization.value)
            .flatMap{ Observable.just($0.userMainResponse) }
            .subscribeOn(ConcurrentDispatchQueueScheduler(qos: .background))
            .observeOn(MainScheduler.instance)
            .subscribe { (event) in
                switch event {
                case .next(let response):
                    print("ResponseCheck: ", response?.data ?? "No data available")
                    break
                case .error(let error):
                    print("ErrorHandling: ", error)
                    self.isLoading.value = !self.isLoading.value
                    break
                case .completed:
                    self.isLoading.value = !self.isLoading.value
                    break
                }
            }.addDisposableTo(disposeBag)
    }

Any help handling the error? Thanks.

Please post a JSON you are parsing into error object.

@DenHeadless I am not parsing any JSON I am testing the httpError handling and looking at how to deserialize request errors. So, the fetchOrganizationInfo () function is the one utilizing. It's not a stubbed JSON.

I understand, but you are saying that you are having problems deserializing request errors, i presume here?
required init(json: JSON) {
        if let dictionary = json["errors"].dictionary { <- This block isn't entered because it isn't a dictionary
            for (key,value) in dictionary {
                errors[key] = value.arrayValue.map( { return $0.stringValue } )
            }
            print("FinalError: ", errors)
        }
    }

If that is the issue you are dealing with, i'm asking you to share a JSON you are receiving in this method.
If your question is about Rx and how to handle already parsed error using Rx, then please be more specific, it's not clear what you are trying to achieve.

Sorry for not being clear enough.
I logged the error in onError in the Observable and it logged this
ErrorHandling:  APIError<FIAppError>(request: Optional(https://url), response: Optional(<NSHTTPURLResponse: 0x1c0238ee0> { URL: https://url } { status code: 404, headers {
    "Content-Length" = 36;
    "Content-Type" = "application/json; charset=utf-8";
    Date = "Fri, 30 Jun 2017 13:15:12 GMT";
    Etag = "W/\"24-nuP261L8lpDkCVKMtmlHgw\"";
    Server = "nginx";
    "Set-Cookie" = "sails.sid=djskdjks; Path=/; HttpOnly";
    Vary = "X-HTTP-Method-Override";
    "cf-ray" = "hshdjsd-LHR";
    "x-powered-by" = "Sails <sailsjs.org>";
} }), data: Optional(36 bytes), error: Optional(Alamofire.AFError.responseValidationFailed(Alamofire.AFError.ResponseValidationFailureReason.unacceptableStatusCode(404))), errorModel: Optional(company.FIAppError))

So, I passed the error object to the APIError Object and it returned nil


STRATO-3107 Adding parameters validation [WIP]

These marketplace changes will go in with marketplace branch changes within starto-platform repo.

Closing in favor of the revision 2 PR: https://github.com/blockapps/strato-getting-started/pull/90


https://kubernetes-charts.banzaicloud.com/charts not available

Describe the bug
Not possible to download a helm chart or to add the helm repo. The dns entry seems to be missing.
curl https://kubernetes-charts.banzaicloud.com/charts/thanos-operator-0.3.7.tgz
curl: (6) Could not resolve host: kubernetes-charts.banzaicloud.com
Steps to reproduce the issue:
Try helm repo add banzaicloud-stable https://kubernetes-charts.banzaicloud.com or the above curl command
Expected behavior
Downloading the helm chart

It wasn't on Friday but seems to be working now...strange 🤔

Is it working now?
Getting Error: looks like "https://kubernetes-charts.banzaicloud.com" is not a valid chart repository or cannot be reached: Get "https://kubernetes-charts.banzaicloud.com/index.yaml"

It worked for a while, but we have copied the helm file as .tar.gz inside our repository already.

Same here ... still having issues downloading the charts
Error: looks like "https://kubernetes-charts.banzaicloud.com" is not a valid chart repository or cannot be reached: Get "https://kubernetes-charts.banzaicloud.com/index.yaml": dial tcp: lookup kubernetes-charts.banzaicloud.com on <IP_ADDRESS>:53: no such host.


AVS to one skill only

Not sure if this is the right place, but one of the skills I am making using this I want to integrate into an existing iOS\Android app.
The problem is I just want that skill to work nothing else. Is that possible?

What do you mean by "that skill to work"?

It is probably not possible. Say your app is a trivia app.
Clicking the microphone icon would allow you to speak to Alexa and replicate the trivia game using an Alexa skill.
I believe this is not possible as you would need the skill to be enabled for that user. But curious to know if anyone had thought of this use case.
If you don't understand I can try to explain it further, if not and you feel it's not possible just close this issue.

I haven't looked too closely at AVS, but I don't believe there's an API that exists at a layer between "here's an Alexa Skill that runs on an Alexa device" and "here are low-level primitives to train your own language model".
You might want to poke at the Microsoft Bot Framework. The tooling they provide to create Cortana Skills are fairly similar to Alexa, but they do provide enough access across all levels of abstraction that you might be able to deploy a voice-enabled iOS app in that way.

Interesting, thanks. So that would do iOS and Android.
I am not a real coder just looking at commericalising Alexa and thought this might be a way


[breaking] drop Node 14 support

closes https://github.com/sveltejs/kit/issues/4916

AFK or I'd do this myself right now - we should get rid of the node 14 crypto stuff as well

Actually, scratch that. We should change install-fetch to be install-polyfills or something and assign require('node: crypto').webcrypto to globalThis.crypto, and put any other things we need in there (like AbortController and ReadableStream etc). Will do that in a follow-up

Your not done yet should of read the whole lesson plan when you had the chance.


Stuck in an elevator

Minecraft Version:
-1.11.2
Forge Version:
-<IP_ADDRESS>5
Advanced Rocketry Version:
-1.2.5a
LibVulpes Version:
-1.11.2  0.2.4
is this the latest curseforge build?
No
Explain issue in detail:
The space elevator works just fine for the most part until a single capsule gets stuck in the at the beginning of the decent, making other attempts to use the elevators useless. There is no apparent reason for one to get stuck like that, no opticals, entities, any form of collusion, nor any specific condition is there. It happens across planets, moons, and stations in every world I create.

This should have been fixed in the most recent alpha build for 1.11.2, can you try that and report back?

It works! But I ran into a crash while rocketing from the moon back to earth, But I don’t think Advanced rocketry was the culprit as it crashed 5 hours into gameplay.(sent a sat and went afk) But yea, Problem fixed!
From: zmaster587
Sent: Sunday, February 4, 2018 16:53
To: zmaster587/AdvancedRocketry
Cc: FaolanKusibo; Author
Subject: Re: [zmaster587/AdvancedRocketry] Stuck in an elevator (#1122)
This should have been fixed in the most recent alpha build for 1.11.2, can you try that and report back?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Expose statsd

Moved statsd client init and statsd import to root index.js and added statsd to export object in the same file.  The statsd client is available anywhere by require(monument).statsd
However, I could not figure out a way to test, and though the client is showing I am uncertain if it will fire data to an actual statsd server.
This PR relates to this issue:
https://github.com/ansble/monument/issues/529

Looks like  ✖ routes › router.statsd › when not configured does not send anything to statsd for each standard request is failing now
And also need to merge latest changes on master into your branch.
Thanks for the work, and sorry it has taken me a few days to respond.

My trouble seems to be the config that is used to initialize the statsdClient is passed into the router, and I can't seem to find a good way to get it out of there. Is that config passed in on server start up?

@cursiv3 yeah it should be passed in at server startup

Ok the other random failing tests are fixed on master. If you merge master into your branch they should go away...
If I have some time over the weekend I'll pull what you have in and see if I can think of some ways to get around the issue you are running into


autodq parsing proposal

@thinhha this is how I'd handle the cameCase/snake_case fields in the spec files. Basically first try cameCase and if it fails then you try snake_case. What do you think?
@ludoo wdyt?

This is simpler for the user. I'm fine with this approach.
Some cons of the approach is that it feels a bit magical and also allow end users to mix casing in their spec.
Perhaps I'd still keep the input type as an object. This is because we may want to expand this in the future to allow users to supply an additional file containing default values for variables like ignore_null, row_filters, threshold, etc... as well as a collection of rule templates that they can use in the spec.


[ModelZoo] Support Co_Action Network

 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.

这是来自QQ邮箱的假期自动回复邮件。你的邮件我已经收到，我会尽快回复你的。


Don't log GossipTimeout if target node left the cluster

in the case where a node has left the cluster. and there are inflight gossip requests.
these timeouts should not be logged. as the result is expected.

done


Revisit %ll  (long long)  printing for 64bit values

%ll  printf string seems to be no good for printing 64bit values on windows.
We should consider introducing GPR_PRIi64, GPR_PRIu64 and GPR_PRIx64 macros to address this in a really portable way.
The 32bit versions of the macros might also be useful at times.
cc @ctiller  @nicolasnoble

I might have unknowingly done some printf evil in #4397.

We can just use PRIi64 etc... now.


Terminal: Right click on list should show some actions

Like:

New terminal
Close all terminals
Hide list (to mitigate https://github.com/microsoft/vscode/issues/121602)

Right now it 🐛 pastes the clipboard onto the terminal.

Thanks, WIP. Should be in soon 👍🏼  https://github.com/microsoft/vscode/issues/121236


Do not send drain events if the target service state is not considered active

https://github.com/rancher/rancher/issues/10011
@cjellick pls. review

LGTM


Any way to set adjustment

Hi,
Is there any method to set adjustments for start of the Hijri months?

Welcome brother you can adjust with own
use Alkoumi\LaravelHijriDate\Hijri;

   Hijri::Date('m/d/Y', $date);


is there n instruction on how to use this?

is there n instruction on how to use this?

Hi @WizardKid , I was planning to create a gem but after a bit of research I found out that stripe is not supported in my country(Nepal). So I moved on. It would be nice if you can take over this :)

I would be willing to do that. I will look ok over your repo. Can you look at my issues I have raised on somethings I am experiencing with the latest catarse? I managed to get it up and running but there is some issues (maybe the database or the code)


Order history

Exchanges endpoints

binance
coinbase

OpenLimits
Function:
get_order_history

Data format: Array of the existing order structure.

I'm on it.


Version conflict

In the release (branch master), we are at v1.17.0
But when I compile, we are at v1.16.5
Sending build context to Docker daemon  7.147MB
Step 1/2 : FROM python:3.7-alpine3.8
 ---> aadc3feb2b19
Step 2/2 : RUN set -ex &&     pip3 install xkcdpass
 ---> Running in 2f511c93ddaa
+ pip3 install xkcdpass
Collecting xkcdpass
  Downloading https://files.pythonhosted.org/packages/8c/c5/ce1a71fd4a9d13275baf4ba58418887d006d802788c64d4afcf5c97902ed/xkcdpass-1.16.5.tar.gz (917kB)
Building wheels for collected packages: xkcdpass
  Running setup.py bdist_wheel for xkcdpass: started
  Running setup.py bdist_wheel for xkcdpass: finished with status 'done'
  Stored in directory: /root/.cache/pip/wheels/7e/a1/ab/bb3c0e7763f1aa0951a51bcc844834af78956215d62b2f5651
Successfully built xkcdpass
Installing collected packages: xkcdpass
Successfully installed xkcdpass-1.16.5

Just curious about the way you manage your branch :)
Thank you!

Hey - thanks for flagging this. It looks like the pypi version wasn't properly uploaded. It is fixed now!


Add delete button to the data upload table

Add a button to remove the data added by a given file in the upload data table

@ericboucher i am thinking of adding a new foreign key to table time_series, referencing data_uploads, to be able to identify where any row came from. Are there any data uploads at the production database?


Make sure key javascript files are not retrieved from browser cache after update

I've had several instances where we had a sudden increase of support calls after installing an update of XOT (i.e. from 3.0 to 3.1)
Most of those were due to caching issues. i.e. new modelfiles were used against a previous cached version of xenith.js
There are several solutions to this:

At a timestamp in the url that loads xenth.js , ie src="xenith.js?_=" or similar

The drawback of course is that xenith.js (and similar files) are not cached anymore, increasing the time needed to load an LO

Add a version to the filename of xenith.js, i.e. everytime something changes in xenith.js, change the version.

This is the approach used by a lot of javascript libraries, cf. jQuery. But it it's painfull, and errorprone (or we have to have a build process).

An interesting variation could be the combination of 1. and 2.

We have a version string maintained in the file version.txt in the root folder of the installation. This file is automatically updated when publishing zips for the community websites. We could add ?version='' to all the files we want to reload after a new release, such as xenith.js

Implement option 3


Update TE-3.7

Changes:

Implement TODOs
Add implicit in-place add scenario.
Add implicit in-place remove scenario.
Update implicit in-place replace scenario.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

https://github.com/openconfig/featureprofiles/pull/1255 is implementing this.


Rename hackEDU to Hack Club

This'll be broken until we secure the hackclub GitHub org name. Going to wait on that.

It looks like hackclub has just been secured (just reloaded the page to suddenly find the org renamed!)

Indeed! I got us the name.


update modules with fixes, improvements

beefy PR

Ah thanks for spotting that! Updated!

Something I just came across - in the documentation (eg. this page) there is use of mapie models for which the tagging format has changed, so these docs probably need an update to avoid an error when referencing with the old tagging style


Added Finnish translation

@jdufresne Can we land all translations PRs #548, #522, #521?

Thanks!


Create /users/verify endpoint

What to do:

[ ] Create a route /users/verify.
[ ] Route should be authenticated before calling the controller so that we know which user called the API. (Hint: See other endpoints)
[ ] Controller should call the identity service /verify endpoint with the username in the body.
[ ] We just have to call the identity service endpoint and don't wait for the response if it's active. We are kind of queueing a request by calling that identity service endpoint.

Hi
I would like to work on this

Hi Lakshay,
I am not clear with what do we mean by Identity Service Endpoint ?
Can you help me with this ?

Hi @Deepansharora27
It is basically https://someurl.com/verify

@lakshayman
May I also ask you that
What purpose does this API Serve ?
Is is responsible for checking whether the current user is authenticated or not ?

@lakshayman Can you provide me the link to the Identity Service API ?


loadChildren doesn't work with named router-outlet

I add the outlet property to the app.routing.ts:
    {
      path: 'flights',
      outlet: 'test',
      loadChildren: () => loadRemoteModule({
          remoteEntry: URL,
          remoteName: 'mfe1',
          exposedModule: './Module'
        })
        .then(m => m.FlightsModule)
    },

In the template, add the name attribute for the router-outlet tag:
<router-outlet name="test"></router-outlet>

The microapplication stops loading inside the named router-outlet tag.
This error does not allow me to connect multiple micro-applications on the same routing path.

Hi Jeka, do you have solved this problem? thanks

This error does not allow me to connect multiple micro-applications on the same routing path.
This doesn't work because they all compete for the same URL on load. You need to queue the loading of multiple MFEs and then trigger the initial route one by one....

Thanks @smasala, I agree.
This is how the Angular Router works and not connected to this package or Module Federation in general.


Update instructions for generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent.md

On macOS Monterey the command ssh-add -K ~/.ssh/id_ed25519 results in a warning message:
WARNING: The -K and -A flags are deprecated and have been replaced
         by the --apple-use-keychain and --apple-load-keychain
         flags, respectively.  To suppress this warning, set the
         environment variable APPLE_SSH_ADD_BEHAVIOR as described in
         the ssh-add(1) manual page.

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Thanks @stevecat 🖖


fix: improve error messages when farm or queue ID not defined

What was the problem/requirement? (What/Why)
When first submitting a job to AWS Deadline Cloud using its render node, I received unexpected error messages with an Access Denied error. These were related to missing default farms and queues.

What was the solution? (How)
Added a check whether the farm and queue are defined before calling GetQueue in the submitter.

What is the impact of this change?
Easier onboarding for new customers
How was this change tested?

Created a simple scene with geometry, light, camera
Connected a Karma ROP to the Deadline Cloud ROP
Clicked submit (without adding settings, or after clearing the default farm and queue with deadline config set defaults.farm_id "" and deadline config set defaults.queue_id "")

Was this change documented?
No, not required
Is this a breaking change?
No

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for fixing this bug! The changes here look good, but it looks like we could add the same logic to

settings_callback
login_callback
update_queue_parameters_callback

in src/deadline/houdini_submitter/python/deadline_cloud_for_houdini/submitter.py
Which would allow us to resolve #117
Do you mind adding the checks for unset farm/queue there too?

After syncing offline, we've agreed to tackle these other places in a future PR
Ship it :shipit:


Cannot set property 'eventBus' of undefined (GridLayout.vue)

I get this error Cannot set property 'eventBus' of undefined when using the below code. Do you have any hints on how to `import this component?
<template lang="html">
    <grid-layout
            :layout="layout"
            :col-num="12"
            :row-height="30"
            :is-draggable="true"
            :is-resizable="true"
            :vertical-compact="true"
            :margin="[10, 10]"
            :use-css-transforms="true"
    >

        <grid-item v-for="item in layout"
                   :x="item.x"
                   :y="item.y"
                   :w="item.w"
                   :h="item.h"
                   :i="item.i">
            {{item.i}}
        </grid-item>
    </grid-layout>
</template>

<script>

import VueGridLayout from 'vue-grid-layout';
var GridLayout = VueGridLayout.GridLayout;
var GridItem = VueGridLayout.GridItem;

export default {
    components: {
	    GridLayout,
	    GridItem
    },
    mounted(){
    },
    data(){
        return {
	        layout: [
		        {"x":0,"y":0,"w":2,"h":2,"i":"0"},
		        {"x":2,"y":0,"w":2,"h":4,"i":"1"}
            ]
        }
    }
}
</script>

With what version of vue-grid-layout and vue?

I've just update Vue to 2.2.1 and all is working well now, thanks for the hint :)
It was previously:

vue: 2.1.0
vue-grid-layout: 2.1.5


Updating the properties too quickly

It seems that short poll has no impact on the interval for updating the nodes/controls.

That is correct.  That's the nice part about wireless tags :) There is no polling, the updates are pushed from the hub as it sees the changes.
Are you asking if the node server should queue up those changes and only send on short poll's?

@mkohanim was there something you wanted changed here?

No?
With kind regards,

Michel Kohanim
(p)<PHONE_NUMBER>
http://www.universal-devices.com

From: JimBoCA @.>
Sent: Friday, February 18, 2022 2:36:46 PM
To: UniversalDevicesInc-PG3/udi-wirelesstag-poly @.>
Cc: Michel Kohanim @.>; Mention @.>
Subject: Re: [UniversalDevicesInc-PG3/udi-wirelesstag-poly] Updating the properties too quickly (#34)
@mkohanimhttps://github.com/mkohanim was there something you wanted changed here?
—
Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc-PG3/udi-wirelesstag-poly/issues/34#issuecomment-1045286458, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPPHMNRQAMDHMI6HEF3U33CX5ANCNFSM4KPNU6UQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


IAI_Kinect2 optimization

Dear Thiemo,
I hope you're doing great. Last semester I developed a vision system with the help of your great bridge between Kinect and ROS with Matlab. Final results were acceptable but some issues were sighted during the implementation. Although your code is optimized with OpenCL, still CPU usage is exorbitant.   Running the bridge at 5HZ on Intel i7 Haswell with 4 cores, leads to 58 percent CPU usage. I know hardware is an issue but do you have any solution about further optimization that I can do a research on? I read the threads about Jetson and Cuda and they were interesting. Is there any other solutions like optimizing buffers or more on OpenCL? And also can I ask why you chose OpenCL instead of OpenMP? Did you have any special criteria?

Most demanding is compression. Try to use the uncompressed topics if possible. Also you can update libfreenect2. They recently implemented a VA-API based JPEG decoding which uses the Intel GPU to decode the JPEGs, on my system (i7 Haswell) the CPU load is around 2 to 3 percent, when libfreenect2 is processing the frames with VA-API and CUDA/OpenCL.

Thanks a lot Thiemo. I will look into this.


[WIP] Action runner output streaming single collection approach

This pull request works on top off #3657 and changes two collection approach with single "generic" collection approach.
It addresses the feedback people had and makes the whole approach a bit more generic so it can work with other / future runners which produce just a single output stream.
The main difference is that this approach utilizes single "action_output_d_b" collection with the following fields:

execution_id
action_ref
runner_ref
timestamp
type
data

Most fields are self explanatory, but two fields of interest are "data" and "type".
"data" contains actual partial execution output. Depending on the runner, this could either be chunk, line or similar. For all the existing runners for which we implement streaming right now this is a line. I was thinking about a different name (e.g. chunk), but in the end I settled with data.
type contains type of the output. Right now that is either stdout or stderr, but for runners with just a single output stream, this could simply be output or similar. I'm open to suggestions for a better name for this field.
I was thinking about "stream" / "stream_type" / "output_type", but in the end I settled with "type". If we can't come up with anything better, I will probably change it to "output_type" so it doesn't clash with Python built-in type function.
As for now, I only implemented model changes so I won't cause unnecessary additional work for myself in case people don't agree with this approach.
Once we agree that's the model we all like, I will go ahead and implement rest of the changes.
If we agree on this approach, actual API endpoint and CLI command will look like this:

/v1/executions/<execution id>/output[?type=stdout/stderr/etc]
st2 execution tail [--type=stdout/stderr/etc]

TODO

[ ] Update affected API endpoints
[ ] Update affected event stream code
[ ] Update affected cli docs
[ ] Update affected tests
[ ] Update affected docs

@lakshmi-kannan @m4dcoder @bigmstone Let's please agree on this data model today if possible so I can finish rest of the changes and get this feature out.

Do you need rebase this branch? It's including some unrelated commits/fixes from master (unicode, travis, etc.)

LGTM except for a few minor questions regarding unrelated commits in this branch and adding an auto increment sequence field in the ActionExecutionOutput collection.

Do you need rebase this branch? It's including some unrelated commits/fixes from master (unicode, travis, etc.)

It was already synced with master and the other branch is based on so no idea why Github is showing this diff the way it is.

Will merge those changes into existing PR and work from there.


Modify velero AWS plugin name in deployment

Signed-off-by: Xun Jiang<EMAIL_ADDRESS>What this PR does / why we need it
Change AWS plugin name from "velero-velero-plugin-for-aws" to "velero-plugin-for-aws".
Details for the Release Notes (PLEASE PROVIDE)

NONE

Which issue(s) this PR fixes

Fixes: #
Describe testing done for PR

Special notes for your reviewer

Thanks, Nick. I forgot to update the image in project file.


Fix heap-buffer overflow from FPRDecodeString

This is to fix #8628 and #8577
The issue was not reproducible.
The fix is to remove the usage of FPRDecodeString  in the code.

@jeremyjiang-dev Since FPRClient is the only place where we are using FPRDecodeString, we might be safe here. But, I think we need to get to the root cause if the code exists in our ecosystem. As an alternate, do you think we can FPRDecodeString as an API? This is not used anywhere else except for unit tests.
Heap overflow occurs when there is a byte array conversion to NSData. If the byte array does not contain a 'null' or '\0' at the end, there is no way for the system to undertsand where the character array ends.
One way to attempt to reproduce this issue would be to force a really long character array to see if FPRDecodeString cannot decode it.

Discussed offline. I moved the decoding methods to testUtil and note the methods may cause heap-buffer overflow.


Send an anonymized hash with each contribution non-logged in device

Send an anonymized hash for each non-logged in device so that we can block any spammer or bad-meaning person server side.
It has to remain the same across contributions, but shouldn't contain any information that could let anyone trace back to the user.
Strongly hashing a uuid might be an option. We need to add a secret string to further protect against any rainbow tables of hashed uuids that might exist on the internet.
Perhaps Android has a native mechanism for that.

https://android-developers.googleblog.com/2011/03/identifying-app-installations.html

Interested, working on this issue.

@teolemon Review this commit. The hashed code needs to be parsed at the server side. I have sent it with the key 'identity_code'. Also, the code will be same for a single Installation.

https://world.openfoodfacts.org/product/99999971/test
Can you send it in the comment field instead ?


replace:true and ignore:true options for insert API

Equivalent to https://sqlite-utils.datasette.io/en/stable/cli.html#insert-replacing-data

Looks like I did implement these already:
https://github.com/simonw/datasette/blob/9a1536b52a07e32da5900652da1bd7894c58fa9f/tests/test_api_write.py#L270-L273
But forgot to document them!

Updated docs at the bottom of this section: https://docs.datasette.io/en/latest/json_api.html#inserting-rows


How to add bots to channel

I tried addChatMember and setChatMemberStatus methods,
but it didn't work
{
    "@type": "setChatMemberStatus",
    "chat_id": -10012131313,
    "member_id": {
        "@type": "messageSenderUser",
        "user_id":<PHONE_NUMBER>4141,
    },
    "status": {
        "@type": "chatMemberStatusAdministrator",
        "can_promote_members": true,
        "can_delete_messages": true,
        "can_post_messages": true,
        "can_edit_messages": true,
        "can_change_info": true,
        "can_manage_chat": true,
        "can_be_edited": true,
    }
}

error: result
{@type: error, code: 400, message: USER_BOT }

See method scheme for the up-to-date TDLib version: https://github.com/tdlib/td/blob/master/td/generate/scheme/td_api.tl. Class chatMemberStatusAdministrator was changed.

thanks
 {
                                  "@type": "setChatMemberStatus",
                                  "chat_id":  217291291,
                                  "member_id": {
                                    "@type": "messageSenderUser",
                                    "user_id": 12131414,
                                  },
                                  "status": {
                                    "@type": "chatMemberStatusAdministrator",
                                    "rights": {
                                      "@type": "chatAdministratorRights",
                                      "can_promote_members": true,
                                      "can_delete_messages": true,
                                      "can_post_messages": true,
                                      "can_edit_messages": true,
                                      "can_change_info": true,
                                      "can_manage_chat": true,
                                      "can_be_edited": true,
                                    }
                                  }
                                },


Bad debugger suggestions for pwa-node

I tried debugging the node sample app in Dev Containers without having js-debug activated and the following message showed up:

However, when I click on the leftmost button to install extensions, I get the following query (tag:debuggers @sort:installs) and suggestions, none of them having to do with js or node debugging:

Searching just 'pwa' helped me realize that I forgot to activate js-debug, and searching 'pwa-node' yielded no results.

This suggestion should not generally be relevant since js-debug is built-in, and built-in extensions do not show in results. @rzhao271 do you use the js-debug nightly?
I've seen this sometimes where js-debug is not installed in remotes, but I assumed this was because I use the nightly extension and have the built-in one disabled.

@connor4312 I don't remember the exact state I was in, but I had the nightly version installed by that point. It was likely that I had both the nightly and stable versions disabled by accident, though I still felt confused by the results under tag: debuggers sort:@installs. I'm wondering if there's any way to add an OR into that query so that it can also list relevant disabled installed extensions?

It will usually list installed extensions (e.g. it shows the Debugger for Chrome in your screenshot), but not built-in extensions.
Giving to Sandeep who owns this but not sure it's worth fixing just for people using js-debug nightly.

Query to market place shows all result extensions in Marketplace - we do not filter built in extensions if they are also available in Marketplace. For eg., searching for pwa shows both stable (which is builtin) and nightly js debugger extensions

It seems the query tag:debuggers @sort:installs is also showing both those extensions but they are not shown on the top but somewhere in the middle due to sorting criteria

@connor4312 Moving this back to you if you would like to change the filter to get expected results

Ideally we could have metadata for packages in the marketplace to show the run with the right debug type... but this is not currently possible.


Add blog.vinfall.com

Website: https://blog.vinfall.com
This PR is:

[x] Adding a new domain

[x] The domain is in the correct alphabetical order
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ page, particularly the red section about inappropriate content, and I attest that this site is appropriate based on those criteria.

[x] Check to confirm

- domain: blog.vinfall.com
  url: https://blog.vinfall.com/
  method: darkonly
  last_checked: 2023-10-14

Thanks!


[clj] Implement cycle

(defn cycle
  "Returns a lazy (infinite!) sequence of repetitions of the items in coll."
  {:added "1.0"
   :static true}
  [coll] (lazy-seq
          (when-let [s (seq coll)]
              (concat s (cycle s)))))

https://github.com/clojure/clojure/blob/clojure-1.6.0/src/clj/clojure/core.clj#L2683
https://github.com/lfex/clj/issues/1

Hi @yurrriq
I'm newbie in the LFE community. Nevertheless I'd like to implement some features in the LFE for self-education purposes. I figured out that README at https://github.com/lfex/clj is a bit outdated (as it points out to a branch of @oubiwann ) and current development process on implementing Clojure in LFE lives at your branch https://github.com/yurrriq/lfe/tree/feature/add-clojure-lib - I'd like to join it by submitting PRs there and I'm willing to start from features tagged with [clj] like this one (or may be from something a bit simplier).
As for the feature itself I am interested in how should it be implemented - I had not found any mentions of lazy-seqs in LFE except clj:next in the way it returns function yielding sequence. Is there any documentation or thoughts about this issue? Should clj:cycle also return a function yielding an infinite sequence?
Thank you.

Hi @profitware! Thanks for interest. That sounds awesome. My branch you found has actually been merged into rvirding/lfe@develop and is included in lfe-1.2. Rather that submit PRs there, I'd suggest you submit them here, to rvirding/lfe.
 As far as the cycle/1 implementation, I think the resulting data structure should look and feel like those from clj:next as you suspected. There is some documentation here, but it could definitely use some polishing.
I think there general idea for cycle/1 is that it should return a "lazy list" (as in next/{1,2,3}) that behaves something like this:
> (set xs (clj:cycle '(1 2 3)))
#Fun<clj.3.141592653>
> (clj:take 7 xs)
(1 2 3 1 2 3 1)

Hi @yurrriq
Should we close this issue? Or anything else should be done on it?

I think it's fine for now, but at some point we'll want to rethink the lazy lists in general and update clj:cycle/1 accordingly. Thanks for your all your work on this.


vulkan

Is there any plans on supporting vulkan?

What kind of support are we talking about here?

equivalent of "OpenGL" I suppose

Still don't understand what you are looking for here. I know what Vulkan is. What do you want pyglet do do?

I think he suggests rendering with Vulkan API if supported by a hardware. Great idea for pyglet 3.0 by the way :) right now we are focused on OpenGL 3 with pyglet 2.0

Yes, what @t-o-n-y-p said.
Also if there is any plays to suppose async event managers? Say if you are loading an image right now it would block your event till its done loading or open a new thread? You could img = await loadimage(path)
I am not sure all these falls under pyglet, maybe it too much changes? maybe i am better off create my own package, not sure... figured i ask first since i like pyglet.

maybe i am better off create my own project, not sure...
@YoSTEALTH Was thinking the same lol.

This should be considered somewhat important. Apple announced that it'll drop OpenGL in future versions of macOS. So, there will be a time, in which pyglet will not be useable on macOS anymore.

It does sounds like a pyglet 3.x thing like already stated.
I highly doubt the OpenGL api will disapper completely from MacOS any time soon, but it will surely be stuck on version 4.1 and won't be optimized or improved. Their drivers was never great in the first place 😄
Let's at least hope it has support in the new ARM based macs. If this is not the case I think pyglet on macs are doomed for a long time.
Moving to vulkan means a complete rewrite of all the rendering. I also means all GPUs below DX11/GL4.1 are no longer supported. pyglet 2.x are just moving from 2.1 to 3.3.

@einarf Another solution would be to use a graphics API wrapper like sokol-gfx,
which wraps OpenGL (X11), DirectX (Win32) and Metal (macOS) under a thin layer.
This would, however, require pyglet to either build some native binaries or have them shipped right with. The other solution would be, of course, to write a custom wrapper.

aren't new opengl versions implemented on top of vulkan?

No, not necessarily. There might be implementations around, that use Vulkan under the hood. I think the best bet could also to expose an OpenGL ES interface and use MoltenGL (given the case, that pyglet receives a free license).

Hmm... looks like OpenGL is being run on top of Vulkan using Mesa

@iperov SPIR-V is an open standard and there are several compilers targeting this bytecode format. One could, in theory, come up with a pure python compiler, that either compiles a DSL, a subset of python or even just (a subset) of python bytecode to SPIR-V.
This would be an extensive process, but it'd be doable.


Added Missing Converter Tests

Description of Change
Added missing converter tests.
Added:

IntToBoolConverter_Tests
InvertedBoolConverter_Tests
IsNotNullOrEmptyConverter_Tests
ListIsNotNullOrEmptyConverter_Tests
ListIsNullOrEmptyConverter_Tests
MultiConverter_Tests
NotEqualConverter_Tests
TextCaseConverter_Tests

NOTE: With this, we have all the converters tests ready, we need to identify what else we want to test.

Great work @Char0394 ! Thank you so much!

NOTE: With this, we have all the converters tests ready, we need to identify what else we want to test.

I think we already have some other things in our codebase already. And I noticed this one lost converter here: https://github.com/xamarin/XamarinCommunityToolkit/blob/main/XamarinCommunityToolkit/Helpers/SafeAreaTypeConverter.shared.cs
Not sure if it should be moved into the converters namespace

Not sure if it should be moved into the converters namespace

I believe, it shouldn't. SafeAreaTypeConverter is subclass of TypeConverter (not implementing IValueConverter). So this one is used for xaml purposes. (Determines how a string should be parsed to SafeArea)


Clarify timezones between UI and server

📑 I have found these related issues/pull requests
Is this a question?
I searched time zone. I clicked on a few. I didn't click on all of them. So I don't know but probably not.
🏷 Feature Request Type
Dashboard
🔖 Feature description
Set custom Time Zone per each Monitor.
If one Monitor is in Australia and the other in California it doesn't make sense that UI is on CET.
Even having side by side TimeZones would be helpful.
✔ Solution
To know what time server went down and up in the timezone where the servers is located at.
❓ Alternatives
No response
📝 Additional Context
No response

I want to work on this plz assign me this

We don't assign people because the burocrscy does not add value for us.
Feel free to drop us a PR, but make sure that you include both the case where timezones are equal and where they are not.


VS2015 support

Would it be possible to add VS2015 support for the SDK? Our product currently supports VS 2015 and in order to have seamless integration and to pull regular updates it would be important for the SDK to be built in VS 2015.
I have compiled/tested the sdk and there does not seem to be any issue.

We're still under internal discussion about this question, will keep you posted on any updates.

@JinmingHu-MSFT  Is there any update on this?
Thank you

@katmsft is currently in charge of this question. He'll contact with you.

Thank you

Can you send an email to<EMAIL_ADDRESS>so this can be more properly discussed? Thanks!


Fastlane ruby error

I am new with fastlane, trying to build lane run next issue:
$ fastlane dev-deploy
[13:40:54]: -------------------------------------------------
[13:40:54]: --- Step: Verifying required fastlane version ---
[13:40:54]: -------------------------------------------------
[13:40:54]: fastlane version valid
[13:40:54]: ------------------------------
[13:40:54]: --- Step: default_platform ---
[13:40:54]: ------------------------------
[13:40:54]: /Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:141:in `rescue in method_missing'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:136:in `method_missing'
Fastfile:38:in `block in parsing_binding'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:108:in `call'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:108:in `platform'
Fastfile:24:in `parsing_binding'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:48:in `eval'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:48:in `block in parse'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:37:in `chdir'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:37:in `parse'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/fast_file.rb:27:in `initialize'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/lane_manager.rb:12:in `new'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/lane_manager.rb:12:in `cruise_lane'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/lib/fastlane/command_line_handler.rb:31:in `handle'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/bin/fastlane:35:in `block (2 levels) in run'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:153:in `run'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:428:in `run_active_command'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:68:in `run!'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/commander-4.3.5/lib/commander/delegates.rb:15:in `run!'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/bin/fastlane:152:in `run'
/Users/Opium/.rbenv/versions/2.0.0-p645/lib/ruby/gems/2.0.0/gems/fastlane-1.39.0/bin/fastlane:158:in `<top (required)>'
/Users/Opium/.rbenv/versions/2.0.0-p645/bin/fastlane:23:in `load'
/Users/Opium/.rbenv/versions/2.0.0-p645/bin/fastlane:23:in `<main>'
Successfully submitted a crash report. If this is a problem with one of the tools specifically,
please submit an issue on GitHub and attach the following number to it: '0fadef224136f540d9deff9061f770a7'
The crash report has been stored locally '/tmp/sentry_0fadef224136f540d9deff9061f770a7.json'

$ ruby --version
ruby 2.0.0p645 (2015-04-13 revision 50299) [x86_64-darwin14.3.0]

# Customise this file, documentation can be found here:
# https://github.com/KrauseFx/fastlane/tree/master/docs
# All available actions: https://github.com/KrauseFx/fastlane/blob/master/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# By default, fastlane will send which actions are used
# No personal data is shared, more information on https://github.com/fastlane/enhancer
# Uncomment the following line to opt out
# opt_out_usage

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.39.0"

default_platform :ios

platform :ios do
  before_all do
	ensure_git_status_clean
    cocoapods

    # xctool # run the tests of your app
  end

  desc "Runs all the tests"
  lane :test do
    snapshot
  end

  desc "Deploy development server target build"
  lane :dev-deploy do

  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    sigh
    gym # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here
  end

  desc "Deploy a new version to the App Store"
  lane :deploy do
    snapshot
    sigh
    gym # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/KrauseFx/fastlane/blob/master/docs/Platforms.md
# All available actions: https://github.com/KrauseFx/fastlane/blob/master/docs/Actions.md

I think you're not allowed to use a - in your lane name. Could you try renaming the lane?

@KrauseFx Thanks a lot. Indeed - was a problem.


Minimum Cost Tree From Leaf.cpp

Related Issue
Solved issue number: #341
Describe the changes you've made
Updated the code of Minimum Cost Tree From Leaf Values
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
Checklist:

[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[x ] I have commented my code, particularly in hard-to-understand areas.
[x ] I have documented the code.
[x ] New and existing unit tests pass locally with my changes.

Your folder name should be "Minimum Cost Tree From Leaf Values"

@Nivedita967 Is it ok now?


Create package on PyPI

Is it ok to publish this repo as a package on PyPI?  I can do so if necessary.

I'm working on it, but I wanted to finish up the documentation first. Give
me a week or two, and if there still isn't a package then ping me again
then.
On Fri, Jan 5, 2024 at 1:41 PM Buckeyes2019 @.***>
wrote:

Is it ok to publish this repo as a package on PyPI? I can do so if
necessary.
—
Reply to this email directly, view it on GitHub
https://github.com/TutteInstitute/datamapplot/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3IUBPT3XCTK2Y364EDK3TYNBCPJAVCNFSM6AAAAABBO2B6CGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DOOBWG44DIOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@Buckeyes2019 It's now on PyPI


Code workflow

Configure Azure Pipelines
This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: AnatolyPalyony sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


uncacheable_asset helper function

I'd like to suggest a helper function that reloads (the passed file in parameter) on the browser whenever its content has been changed.
    /**
     *  forces the browser to load the asset file if it was modified
     */
    public function uncacheable_asset($path, $secure = null)
    {
        $asset = URL::asset($path, $secure);
        $file = public_path($path);
        $version = filemtime($file);
        // prevents the browser to use the cached version
        // by using last_modif timestamps as an index version
        return $asset.'?v='.$version;
    }

@netfox01 can you open a PR?

done #18263


tangent.grad_dot fails with (3,) (3,) arguments

Once I produce my gradient function with tangent.grad, calling the function fails with the following error
/shared/sdoerr/Software/miniconda3/lib/python3.6/site-packages/tangent/utils.py in grad_dot(dy, x1, x2)
    773       numpy.sum(x2, axis=tuple(numpy.arange(numpy.ndim(x2) - 2)))))
    774   dy_x2 = numpy.sum(dy, axis=tuple(-numpy.arange(numpy.ndim(x2) - 2) - 2))
--> 775   return numpy.reshape(numpy.dot(dy_x2, x2_t), numpy.shape(x1))
    776
    777

ValueError: shapes (1,1) and (3,1) not aligned: 1 (dim 1) != 3 (dim 0)

ipdb> x1
array([ 0.63199997, -0.01399994,  1.66399956])
ipdb> x2
array([1.32600021, 1.09599972, 0.45800018])
ipdb> dy
array([[0.00041678]])

Had to do: np.dot(x[jj], np.reshape(x[kk], (-1, 1))) to fix it. Not a huge issue but it could confuse users.

Do you have the original code and arguments you used for gradients?
A possible cause might be that tangent.grad assumes scalar output - can you try to see if using  tangent.autodiff works instead (that will require you to supply an initial gradient value, if it's not scalar).

import tangent
import numpy as np

def test(x, y):
    return np.dot(x, y)

xxx = tangent.grad(test)
xxx(np.random.rand(1, 3), np.random.rand(1, 3))

I see - the error originates from np.dot, the matrix sizes don't align properly. For correct matrix multiplication, the call should be xxx(np.random.rand(1, 3), np.random.rand(1, 3)), as you mentioned.
For example, the following code fails with the same error:
import numpy as np

def test(x, y):
  return np.dot(x, y)

test(np.random.rand(1, 3), np.random.rand(1, 3))

That said, I think it would be useful to wrap errors and more clearly indicate when an error originates in the forward code.

Oh right, sorry. Well you can still make it work with numpy but fail with tangent even though it becomes a different error now
import tangent
import numpy as np

def test(x, y):
    return np.dot(x[0, :], y[0, :])

test(np.random.rand(1, 3), np.random.rand(1, 3)) #runs
xxx = tangent.grad(test) # doesnt
xxx(np.random.rand(1, 3), np.random.rand(1, 3))

Yes, it seems that we have a bug in the handling of slice operators. This might be insufficient for your immediate needs, but it should run:
def test(x, y):
    return np.dot(x, y)[0, 0]  # Use matrix multiply instead of inner product

test(np.random.rand(1, 3), np.random.rand(3, 1))
xxx = tangent.grad(test)
xxx(np.random.rand(1, 3), np.random.rand(3, 1))

Alternatively, you could use tangent.autodiff which does not assume the result is a scalar. Implementation-wise they are very similar, but tangent.autodiff is more technically correct in that case:
def test(x, y):
    return np.dot(x, y)  # Result is a 1 x 1 matrix

test(np.random.rand(1, 3), np.random.rand(3, 1))
xxx = tangent.autodiff(test)
# Add a third parameter for the gradient seed, which matches the shape of f's result.
xxx(np.random.rand(1, 3), np.random.rand(3, 1), np.ones((1, 1)))


tests are simplified

This PR simplifies the tests of stbln and stgcn methods (as much as possible) to address issue #167 by lowering the memory load.

I'm not against keeping those tests even if the same functionality is executed within other methods, especially in case of problems we would more easily be able to narrow it down. I've made a few tests and it seems the only change we really need is lowering the batch_size as you did. Doing so drops the ram load by around a factor of 2, which is plenty enough.
Also, when we merge this PR we should remove this line

I'll run it 2 or 3 times and see if there are issues, but I suspect this might be enough

Looks like the 2nd round the tests failed, but not because of the change made, but rather heart anomaly detection (gru):
 ./opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/sklearn/metrics/classification.py:1436: UndefinedMetricWarning: Precision is ill-defined and being set to 0.0 in labels with no predicted samples.
  precision = _prf_divide(tp_sum, pred_sum,
/opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/sklearn/metrics/classification.py:1438: UndefinedMetricWarning: Recall is ill-defined and being set to 0.0 in labels with no true samples.
  recall = _prf_divide(tp_sum, true_sum,
/opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/sklearn/metrics/classification.py:1436: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples.
  precision = _prf_divide(tp_sum, pred_sum,
/opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/sklearn/metrics/classification.py:1438: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no true samples.
  recall = _prf_divide(tp_sum, true_sum,
./opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/torch/autograd/__init__.py:130: UserWarning: CUDA initialization: Found no NVIDIA driver on your system. Please check that you have an NVIDIA GPU and installed a driver from http://www.nvidia.com/Download/index.aspx (Triggered internally at  /pytorch/c10/cuda/CUDAFunctions.cpp:100.)
  Variable._execution_engine.run_backward(
.....F...

======================================================================

FAIL: test_fit (gated_recurrent_unit.test_gated_recurrent_unit_learner.TestGatedRecurrentUnitLearner)
**********************************
----------------------------------------------------------------------
TEST Attention Neural Bag-of-Feature Learner
Traceback (most recent call last):
**********************************
  File "/home/runner/work/opendr_internal/opendr_internal/tests/sources/tools/perception/heart_anomaly_detection/gated_recurrent_unit/test_gated_recurrent_unit_learner.py", line 82, in test_fit
Pretrained model downloaded to the following directory
    self.assertFalse(torch.equal(old_weight, new_weight),
/tmp/tmp4eby43my
AssertionError: True is not false : Model parameters did not change after running fit.
Pretrained model is loaded successfully

----------------------------------------------------------------------

Ran 11 tests in 73.154s
**********************************

TEST Gated Recurrent Unit Learner
FAILED (failures=1)

I've re-run it like 5 times locally but I can't reproduce. Could be a fluke.

@passalis Would you please approve this again, so that I can merge?


Crash asyncio loop

The current version seem to crash with python 3.11 (using this in docker):
sml-exporter-sml-exporter-1  | Traceback (most recent call last):
sml-exporter-sml-exporter-1  |   File "/usr/local/bin/sml-exporter", line 8, in <module>
sml-exporter-sml-exporter-1  |     sys.exit(main())
sml-exporter-sml-exporter-1  |              ^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1130, in __call__
sml-exporter-sml-exporter-1  |     return self.main(*args, **kwargs)
sml-exporter-sml-exporter-1  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1055, in main
sml-exporter-sml-exporter-1  |     rv = self.invoke(ctx)
sml-exporter-sml-exporter-1  |          ^^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1404, in invoke
sml-exporter-sml-exporter-1  |     return ctx.invoke(self.callback, **ctx.params)
sml-exporter-sml-exporter-1  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/click/core.py", line 760, in invoke
sml-exporter-sml-exporter-1  |     return __callback(*args, **kwargs)
sml-exporter-sml-exporter-1  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/sml_exporter/__main__.py", line 26, in main
sml-exporter-sml-exporter-1  |     loop.run_until_complete(proto.connect(loop))
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
sml-exporter-sml-exporter-1  |     return future.result()
sml-exporter-sml-exporter-1  |            ^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  |   File "/usr/local/lib/python3.11/site-packages/sml/asyncio.py", line 132, in connect
sml-exporter-sml-exporter-1  |     self._lock = asyncio.Lock(loop=loop)
sml-exporter-sml-exporter-1  |                  ^^^^^^^^^^^^^^^^^^^^^^^
sml-exporter-sml-exporter-1  | TypeError: Lock.__init__() got an unexpected keyword argument 'loop'

This seem to be because of:
https://bugs.python.org/issue46796

Looks like you are using pysml<0.0.8. This was resolved in https://github.com/mtdcr/pysml/commit/cc4bfd150bd50d7f227c13999d1ea9c26579528b. Let me update the dependencies and push out a new release.

Can you check whether the 0.1.5 release fixes your problem?


Slow changes in UI and messy code

When requesting a GUI item to be updated (e.g. summon about box, change ticker data ...) the GUI responds very slowly. Presumably because the http requests are blocking. I've tried threading but I obviously don't really understand how this is to work together with GTK.
Secondly, the code is in need of urgent cleanup. It needs to be way more flexible and extensible than now. I don't really have the time and my eyes are already growing calices. Would appreciate any help and/or pointers.

First part is dealt with in v0.5.0 (commit c20aa0f) ; 2nd part is partly addressed by version 0.6.0

partially alleviated in 089c243c9cd810c4989c8375312d73915a495b99


Fix #249: Use p-limit for throttling requests more elegantly.

Switch to using the "p-limit" npm module for parallelizing requests to the CAN API, which I think is a bit more elegant and slightly more efficient (we don't wait for all 50 to complete before starting the next batch of 50).
Tested via running yarn generate-data-snapshot in hackathon repo.
Fixes #249.

Thanks!  That's part of my next PR to rework caching and fix #241 . 😁


swipeLayout.close() does nothing

Hi,
When I am calling the method close() in a SwipeListener() nothing happen.
Here is a sample code :
            mSwipeLayout.addDrag(SwipeLayout.DragEdge.Right, mHideSwipeView);
            mSwipeLayout.setClickToClose(true);
            mSwipeLayout.addSwipeListener(new SwipeLayout.SwipeListener() {
                @Override
                public void onStartOpen(SwipeLayout swipeLayout) {

                }

                @Override
                public void onOpen(SwipeLayout swipeLayout) {
                }

                @Override
                public void onStartClose(SwipeLayout swipeLayout) {

                }

                @Override
                public void onClose(SwipeLayout swipeLayout) {

                }

                @Override
                public void onUpdate(SwipeLayout swipeLayout, int leftOffset, int rightOffset) {

                }

                @Override
                public void onHandRelease(SwipeLayout swipeLayout, float xvel, float yvel) {
                    if(swipeLayout.getOpenStatus() == SwipeLayout.Status.Open) {
                        onPictureHide();
                        swipeLayout.close();
                    }
                }
            });

This code is in a ViewHolder which is in Adapter which extend RecyclerSwipeAdapter
The goal is to trig an action once the swipe is visible and totally open. Then it closes itself automatically.
Unfortunatly  the bottom view does not close... With a swipeLayout.close(false); it works but there is the issue #125 ! Finally with a call on adapter.closeItem(position) it works, but there is a mRecyclerAdapter.notifyDataSetChanged(); which destroy my UX.
Feel free to ask more information ! :)

Try close with a delay:
swipeLayout.postDelay(new Runnable() {
    @Override
    public void run() {
        swipeLayout.close();
    }
}, 50);

It works fine :) thank you
Just a little edit on the method name which is postDelayed()
I will let the issue open because this solution is a kind of hack, but very good for the moment !

@darzul  Awesome!
Thanks @linfaxin 's help.

close(true,false) closes correctly but still calls onClose

I don't believe you @matpatten..Joking! The issue is with the smooth+notify with the close method.
if smooth:
https://github.com/daimajia/AndroidSwipeLayout/blob/f9ac392ef6cd2588cd78458ab2c972e696e1a7dc/library/src/main/java/com/daimajia/swipe/SwipeLayout.java#L1321
then our dragHelper will issue this method:
https://github.com/daimajia/AndroidSwipeLayout/blob/f9ac392ef6cd2588cd78458ab2c972e696e1a7dc/library/src/main/java/com/daimajia/swipe/SwipeLayout.java#L385
And the dispatch does issue a notify regardless.
Correct me if I'm wrong @daimajia. Otherwise feel free to send a pull request.

And @darzul in any adapter try to make use of the ItemManager class:
mItemManger.closeItem(int pos)
Since you're RecyclerViewAdapter should be extended from RecyclerSwipeAdapter that should be inherited.

@matpatten Make sure that you are using the latest version. It shouldn't call the onClose listener, if you make sure that you are all right, please paste your code here.

@daimajia See my comment. I think there may be a bug. I'll try to replicate it tomorrow. (My Test Suite I'm building should help with things like this as well.)

@jpshelley Yeah, I got your point! the dispatch method should handle the notify flag.
Looking forward your test suite, it will make this library much more professional. ^^

@daimajia close(true, false) and open(true, false) still call onClose and onOpen. I do not think it is fixed, could you please take a look at it agian?

When using the SwipeLayout in recycler view, close(false) method does not really work and close() ends up in an inconsistent state when view is recycled ( sometimes remains half open or in best case, use can see the swipe is being closed on recycled views)
To your point @jpshelley on using RecyclerSwipAdapter that also fails because the closeItem method eventually calls notifyDataSetChanged which if you call it in your onBindViewHolder it endsup in and error from RecyclerView that notifyDataSetChanged is not allowed at that time.
Anyone has any idea how to deal with this?


Handling UI for Portrait and Landscape orientations

FYR, Check the Workaholic from Portrait to Landscape or Vice-Versa.

Fixed in example. Please check, Main.storyboard that how I am using Workaholic view in the example app.
Things to consider:

It's important to manage constraints/frames of Workaholic as per your need.
Default height requirements in Landscape (160.0)
Default height requirements in Portrait (80.0)

Note: It's not recommended to use Workaholic in a portrait mode.
Current example is looks like this:


add a field to create update api getter

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify - adding a field to an existing api in preview version

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
Asap, since feature already in public preview and soon to GA.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
Asap, since feature already in public preview and soon to GA.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[x] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

/azp run

/azp run

I am running checks in this pr: https://github.com/Azure/azure-rest-api-specs/pull/21621
Unsure why the checks here are not running.

/azp run

I am running checks in this pr: #21621
Unsure why the checks here are not running.

it needs permission :D

/azp run

@xiaoxuqi-ms it seems the pipeline was stuck. can you help to take a look? thanks~

@xiaoxuqi-ms it seems the pipeline was stuck. can you help to take a look? thanks~

@ericshape / @xiaoxuqi-ms  since pipeline is stuck, I have created a duplicate pr and run the checks there: https://github.com/Azure/azure-rest-api-specs/pull/21621
Also, this is not a breaking change since I have only added a field to existing API. The breaking change tag is showing up due to existing swagger changes which havent been merged yet. https://github.com/Azure/azure-rest-api-specs/pull/21415

/azp run

/azp run

/azp run

/azp run

/azp run unifiedPipeline

Since the pipeline is stuck with github issue, please use the new created pr for review and this pr can be closed.


don't create "simple" docker image- skip container/ unit tests as con…

…tainer is covered by others

Signed-off-by<EMAIL_ADDRESS>

@muralisrini hard-coding the variable's value o_O? why not pass an argument?

@christo4ferris : added a command line option "-run-controller-tests" to run the tests in controller_test.go. Default is to not run them.

@muralisrini :-)

LGTM


[FORMATTING] Why are tabularLeft parentheses so ugly? Can you add hierarchical styles or align backspace

That's one of the reasons why the tabular formatting styles have been deprecated and will be removed in the next major release.

You mean just keep the standard model？

yes.


Fresh installation under 1.4.0 not working?

Hi there,
after fresh Sylius 1.4.0 installation (Symfony 4.2.3) I can not edit the channel anymore:

Neither the property "billingData" nor one of the methods "getBillingData()", "billingData()", "isBillingData()", "hasBillingData()", "__get()" exist and have public access in class "App\Entity\Channel\Channel".

In my channel entity I miss the:
implements ShopBillingDataAwareInterface {     use ShopBillingDataTrait; }

Also not possible to edit order status to complete.
InvalidArgumentException
HTTP 500 Internal Server Error
Sylius\InvoicingPlugin\Entity\InvoiceInterface

Or something missing in your installation docs?
Mike

Hello Mike!
Try installing the newest version of InvoicingPlugin (which is 0.8.1) on your fresh Sylius 1.4.0 project and BillingData-related issues should be gone. Let me know if your problems are solved.

Hi,
channel works now. But to complete a order still brings

InvalidArgumentException
HTTP 500 Internal Server Error
Sylius\InvoicingPlugin\Entity\InvoiceInterface

Can you pass the whole stack trace? I've just checked it on a fresh install and it works 😕

It was just happen with the dummy orders. Created before plugin. Now i created own order, it works fine.


list.index-of

Let's create a new method index-of, which will find the position of an object in the list, comparing objects using .eq:
eq.
  index-of.
    list
      * 1 2 "hello" 4 5
      "hello"
  2

If the object is not in the list, -1 must be returned.

@Graur oops, I see that we have it already, but its name is found, which is confusing. Let's rename it to index-of.


ALStackTraceNotFound

When trying to upgrade our app from an older version (<IP_ADDRESS>) to <IP_ADDRESS> we get this error message:
Deployment failed. Errors: App ID : 26fb6f9e-04f2-47c1-863e-9725fcdc9784 Message :
{ A request to the Data Plane Admin Service failed with an internal server error.
Http status code: InternalServerError
Error:
{
"code": "Unknown",
"message": "The formatter threw an exception while trying to deserialize the message:
There was an error while trying to deserialize parameter http://www.microsoft.com/Dynamics/NAV/51:StartNavAppDataUpgradeResult.
The InnerException message was 'Member 'ALStackTrace' was not found.'.
Please see InnerException for more details."
}
} - Job Id : 10f37cf4-30bd-4382-a359-
Did anyone get this error and has a suggestion what might go wrong?
Background information:
There is a known bug in our upgrade codeunit raising an error when the update environment is not english.
The installation works in an english environment.
We can avoid this error by selecting "english" during installation.
But then we get the above mentioned internal server error.
Because no stack trace is available, we cannot  determine if there is another error in our upgrade code or if this is a BC issue.

Which version of the AL compiler and Business Central are you using?
Is this issue happening on a production tenant?

Hi,
I am using this version:
Version: 1.44.2 (user setup)
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Datum: 2020-04-16T16:36:23.138Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
Betriebssystem: Windows_NT x64 10.0.17134
the version of the AL extension is:
Name: AL Language
ID: ms-dynamics-smb.al
Beschreibung: AL development tools for Dynamics 365 Business Central
Version: 5.0.288712
Herausgeber: Microsoft
Link zum Visual Studio Marketplace:
https://marketplace.visualstudio.com/items?itemName=ms-dynamics-smb.al
We tried this in a sandbox version (I think in production environment
deploy with VSCode is not possible, right?).
The BC installation is:
Version: DE Business Central 16.2 (Platform 16.0.13440.14506 +
Application 16.2.13509.13619)
Azure AD tenant: accd54d0-fa87-4633-974c-6e3fff0c368c, Environment:
SandboxVAPSTest (Sandbox)
Best regards,
Lutz
Am 16.07.2020 um 13:54 schrieb qutreson:

Which version of the AL compiler and Business Central are you using?
Is this issue happening on a production tenant?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/6037#issuecomment-659361861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ2GZO4LLMYABQWNNTWLXQDR33S75ANCNFSM4O35SKMA.

--
Mit freundlichen Grüßen,
*Lutz Nitzsche
Projektleiter MES
Telefon +49 2408 141-50 |  Fax +49 2408 141-33 |  Email
<EMAIL_ADDRESS>NETRONIC Software GmbH
*Pascalstr. 15
52076 Aachen
www.netronic.de http://www.netronic.de/
Geschäftsführer: Dr. Elmar Karlowitsch & Dr. Martin Karlowitsch,
Handelsregister Amtsgericht Aachen HRB 1827

This GitHub repository is dedicated to handling issues and feedback for the latest Developer Preview releases of the AL development tools for Dynamics 365 Business Central.
If you can reproduce the problem on the latest Business Central Developer preview version we will be happy to look further into it. Otherwise please contact the regular support channel.
You can open Support Request to CSS through PartnerSource portal or contact your Service Account Manager (SAM) in the local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also step by step direct you how to open a support request or how to get credentials if this is the first time for you / your company.
For more details please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#i-found-an-issue-on-nav-2018-what-do-i-do