Rewrite everything with async/await (transpile using Babel) and modularized

This is a huge pull request which rewrites classy-lambda-common.  It does several things:

Switches most internal functionality to be async/await, implemented with a babel transpile
Exposes new async/await APIs but preserves the old callback-based APIs
Modularizes the loading of keys into the configuration
Adds a few useful small pieces of functionality
Moves to classes where possible
Brings credstash and config stuff back into classy-lambda-common, as they were never really big enough to merit the complexity of being their own projects

This is a cursory PR.  I need to add some tests, but I wanted to get an initial version out for review as soon as possible.

In my opinion we shouldn't use Babel anymore now that node 8 is supported. I know you mentioned we may not want to early adopt, but I think since node 8 runtimes have been around for a while it prob isn't a brand-new docker image they are using.

Since there are non-pay clients of classy-lambda-common I didn't want to force everyone to update to node 8 before taking this change.  If I were confident that other lambdas weren't using "latest" in their package.jsons for classy-lambda-common I would just bump the major version and not use Babel.
I'm sorta open to different approaches here.  If there's some way we can safely bump the version so that this isn't automatically picked up by stuff that's still stuck on node 6, I'm fine with removing Babel.

Agree with what @jdeon-classy said in spirit, however Jeremy and I did talk about making this backwards compatible with existing lambda functions for now since they are running on 6.10 until we can get them upgraded.


Return more than 2000 records

Hi,
I am using your toolkit, but I need to do some data comparison and I need to extract more than 2000 records.
I don't know if is implemented but in your documentation it doesn't said anything about it.
Thanks

I think the problem is on the salesforce side:
https://developer.salesforce.com/docs/atlas.en-us.salesforce_app_limits_cheatsheet.meta/salesforce_app_limits_cheatsheet/salesforce_app_limits_platform_apicalls.htm

WSC clients can set the batch size by calling setQueryOptions() on the connection object. C# client applications can change this setting by specifying the batch size in the call QueryOptions portion of the SOAP header before invoking the query() call. The maximum batch size is 2,000 records. However this setting is only a suggestion. There is no guarantee that the requested batch size will be the actual batch size. This is done to maximize performance.

The batch size will be no more than 200 if the SOQL statement selects two or more custom fields of type long text. This is to prevent large SOAP messages from being returned.

You should use the QueryContinuationAsync method to get more records if you need to get all.
This is a code fragment in my library that does this:
            var init = await client.QueryAsync<T>(tmpquery);
            var finished = false;
            do
            {
                finished = init.Done;

                // add any records
                if (init.Records.Any())
                    result.AddRange(init.Records);

                // if more
                if (!finished)
                {
                    // get next batch
                    init = await client.QueryContinuationAsync<T>(init.NextRecordsUrl);
                }
            } while (!finished);

Is this still an issue? Please re-open if so.

Hello, I can help you

Problem is long solved with the QueryAllAsync Merge. :)


API: Worker statistics question

Hello,
i was designing my front-end and i have seen a strange behavior (i think) in the results of the api. When i query the pool performance stats (/api/pools/id/performance) i am getting 25 samples for the last 24 hours (that's fine and normal behavior),but when query the worker stats (/api/id/worker/adddress/) i am getting 24 samples for the last 24 hours.And this makes a huge gap in the statistics i am showing (hour, hour and a half), i start digging in the source code and i found this:
https://github.com/coinfoundry/miningcore/blob/807490022c9de7b70321e70d69a282c66b523b78/src/MiningCore/Api/ApiServer.cs#L194-L195
and this:
https://github.com/coinfoundry/miningcore/blob/807490022c9de7b70321e70d69a282c66b523b78/src/MiningCore/Api/ApiServer.cs#L197-L198
this makes so the api is return only 23 hours of samples(because the hour and minutes are removed), what is the point?

Please use the following format to file an issue. Thank you for your cooperation.
Description
[Description of the issue]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior
[What you expect to happen]
Actual behavior
[What actually happens]
Reproduces how often
[What percentage of the time does it reproduce?]
Version
If you are using released version, please state your version.
If you are using clone from dev branch, please state commit hash of your copy.
Also, please include the OS name and version you're currently running.
Additional Information
Any additional information, log, configuration or data that might be necessary to reproduce the issue.


Need An npm package for infracost

Hello Infracost Team,
I need An Npm Package to estimate the price for the terraform without using the file passing the terraform file as a string to a function and then the function will return the price for the resources like that

@kalyanaramansmart can you possibly share more about what you're trying to achieve? what's the use-case? If you're trying to create an Infracost integration, please see this doc.

I need a Package like the following code;
import {getCost} from infracost;
function getCostForTerraform(){
let terraformCode = `terraform {
required_providers {
aws = {
source  = "hashicorp/aws"
version = "~> 4.0"
}
}
}
Configure the AWS Provider
provider "aws" {
region = "us-east-1"
}
Create a VPC
resource "aws_vpc" "example" {
cidr_block = "<IP_ADDRESS>/16"
}`
let costResponse = await getCost(terraformCode);
}
//like this if i pass the terraform code as a string then the package will return the json cost  as response  i can use infracost without running a terminal / powershell

Thanks for adding more details, you can subscribe to this issue for updates.


suggestion

Ionic version:  (check one with "x")
[x] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[x] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
suggestion:
Hello, Thanks a lot for your hard great work!
I have a suggestion to improve the work and reduce the time of answering all fail issue which considers the difference of environment between ionic Team and developers.
I think if it's possible to create a virtual machine which copies ionic team development enviroment and include all correct versions of requirement on it.
eg. Laravel framework they have Homestead which include a copy of Ubuntu + PHP +ngnix etc..
for example, why not have ionic 1 -- a virtual machine environment
ionic 2 -- a virtual machine with its own environment etc ...
which will include android SDK + suitable node version + any free editor + NPM global requirement
I understand it has a lot of difficulty's especially for IOS developers but it will reduce a lot of question and most of the errors will be code errors, not environments.
with regard's

That is a cool idea.
However, that machine will have to be a mac, which is not legal, I think.
It can't be windows for the same reason, however less so (can create VM, gotta have license key)
So it can only be Linux, with GUI (for the emulator), but if it is Linux, you can't emulate to ios.
So I don't really see a full solution here, just one for android, that adds a virtual machine.
Am I missing something?

@AmitMY It took me a bit to decipher what was being said.. but I think you are correct in that it seems more for Android. However, the VM could be hooked up to ionics cloud platform and use the cloud ios emulator.
I think this is a cool concept, and while I don't see the Ionic team setting this up necessarily.. having a Virtual Machine that you could RDP into and have a stock setup for ionic ready to go.. it could definitely help out with reducing the number of different configurations out there.


Bump to 0.2.12

@alexcrichton: no appropriate reviewer found, use r? to override

:pushpin: Commit 91c5dbf has been approved by alexcrichton

@bors: r+

:hourglass: Testing commit 91c5dbf with merge 70dce05...

:sunny: Test successful - status-appveyor, travis


tf: disable slow negative cases

Having a test that checks a pod doesn't come up is nice in theory but
adds extremely minimal coverage and extends test runtime by >1min
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
5d2a71627e9c1eec9bdcc7e0c6d5d069755e84ea
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Regex replace mangling ID in edge case

Hey @nathan-sain,
We had an interesting case with our world where references started to fail for the following ID string: a7y5webpSBRhPGx4L. This was after we refactored all references from .png to .webp using the following command:
fwt dedup --ext=".png" --ext=".webp" --byname --preferred=".*webp"

The likelihood of jpg or png appearing as a substring in an ID is very low but seems like it is possible.
Any chance the regex can be altered to avoid this?

Scratch this. Turns out the offending change was caused by a prior, naive regex replacement using sed.


Include generate two queries

Hi,
I have this linq expression:
                var query =
                    from permissionPerfil in _context.PermissionPerfil
                        .Include(x => x.PermissionItemActions)
                    where
                        permissionPerfil.Id == #myID#
                    select permissionPerfil;

                PermissionPerfil userPermissionPerfil =
                    query.FirstOrDefault();

This linq expression produces two queries:
SELECT "x"."Id", "x"."DateCreated", "x"."DateUpdated", "x"."Description", "x"."IdProject", "x"."IdUserCreated", "x"."IdUserUpdated", "x"."IsDefault", "x"."IsDeleted", "x"."Name", "x"."RowVersion"

FROM "PermissionPerfil" AS "x"

WHERE "x"."Id" = $1

ORDER BY "x"."Id"

LIMIT 1

and
SELECT "p"."Id", "p"."DateCreated", "p"."IdPermissionAction", "p"."IdPermissionItem", "p"."IdPermissionPerfil", "p"."IdUserCreated"

FROM "PermissionItemAction" AS "p"

INNER JOIN (

    SELECT DISTINCT "x"."Id"

    FROM "PermissionPerfil" AS "x"

    WHERE "x"."Id" = $1

    ORDER BY "x"."Id"

    LIMIT 1

) AS "x0" ON "p"."IdPermissionPerfil" = "x0"."Id"

ORDER BY "x0"."Id"

I am doing this workaround:
                var query =
                    from permissionPerfil in _context.PermissionPerfil
                    join permissionItemAction in _context.PermissionItemAction
                        on permissionPerfil.Id equals permissionItemAction.IdPermissionPerfil into PermissionItemActions
                    from permissionItemAction in PermissionItemActions.DefaultIfEmpty()
                    where
                        permissionPerfil.Id == #myID#
                    select new { permissionPerfil, permissionItemAction };

                PermissionPerfil userPermissionPerfil =
                    query.ToList().FirstOrDefault()?.permissionPerfil;

This produces the right sql:
SELECT "permissionPerfil"."Id", "permissionPerfil"."DateCreated", "permissionPerfil"."DateUpdated", "permissionPerfil"."Description", "permissionPerfil"."IdProject", "permissionPerfil"."IdUserCreated", "permissionPerfil"."IdUserUpdated", "permissionPerfil"."IsDefault", "permissionPerfil"."IsDeleted", "permissionPerfil"."Name", "permissionPerfil"."RowVersion", "permissionItemAction"."Id", "permissionItemAction"."DateCreated", "permissionItemAction"."IdPermissionAction", "permissionItemAction"."IdPermissionItem", "permissionItemAction"."IdPermissionPerfil", "permissionItemAction"."IdUserCreated"

FROM "PermissionPerfil" AS "permissionPerfil"

LEFT JOIN "PermissionItemAction" AS "permissionItemAction" ON "permissionPerfil"."Id" = "permissionItemAction"."IdPermissionPerfil"

WHERE "permissionPerfil"."Id" = $1

ORDER BY "permissionPerfil"."Id"

Any one got a better solution?
PS: I am using Npgsql but roji told me this is EFcore related. link
Thanks.

It is true that sometimes the query can be split in two queries if it is very complex. I read it time ago and it is normal. Anyway, sure other people can be more details about it if you need.

This is by design, we produce additional query for every collection navigation that is included. Note that in the second example you will potentially send much more data over the wire - every row that gets returned has values for both PermissionPerfil (duplicated) and PermissionItemAction.
The first query will only fetch PermissionPerfil data once and then stitch it together with PermissionItemAction data on the client. We decided this was in general more efficient than sending one big query.


image/gif: EncodeAll panics when a palette is too large to encode

What version of Go are you using (go version)?
1.8.1
What operating system and processor architecture are you using (go env)?
darwin / amd64
What did you do?
Created a gif image with a palette ( either global or local color table for a frame ) containing > 256 colors.
https://play.golang.org/p/xfDZCcSIFF
What did you expect to see?
The EncodeAll function returns errors when image blocks are too large, on a zero-length gif, etc. As such, in this case since we know the gif can't be encoded due to the palette length being > 256, I would expect a similar error, along the lines of errors.New("gif: image palette is too large to encode") to match the style here
What did you see instead?
It panics here trying to read log2Lookup[-1] defined here
I have a change-set ready with the proposed functionality and accompanying tests.

Go 1.9 is fine too if trivial/safe.
/cc @nigeltao

@gopherbot Ah, okay. Welp, that looks a lot like what I was going to submit, so LGTM :D

Ah, I didn't read your final line that said you had a change-set ready. Sorry.

@nigeltao No worries 😄


remove moved pages

I assume that those pages come from a time in which we didn't had a .htaccess redirect file?
Is there any reason to keep them?

Yes, and actually we should also have dmd.html etc.
Apache is an implementation detail of dlang.org, most people don't test their changes on a live Apache instance - so these pages are still useful for testing.

Yes, and actually we should also have dmd.html etc.

My PR initially had that :/
Ok I will submit this file soon.

Apache is an implementation detail of dlang.org, most people don't test their changes on a live Apache instance - so these pages are still useful for testing.

Yes, but as far as I understood are those pages moved and the .htaccess links are only for compatibility with old links with dlang.org out there somewhere in the wild. So I don't understand why those empty pages are useful for testing.

If there are no internal links to these pages (incl. from Phobos), then I guess they can be removed.

If there are no internal links to these pages (incl. from Phobos), then I guess they can be removed.

I could only find three links to memory.html which I replaced ;-)

Um. What happened here?

No idea. Fixed.


Tuple-modifying differentiates when imported from existing package, but not when written locally.

To replicate this bug, create a dev version of Oceananigans.jl and add this export to the file Fields.jl:
export flattened_unique_values
Then run the attached code (in tuple_modifying.txt), with Enzyme.jl and Oceananigans.jl in your environment.
The function flattened_unique_values works with autodiff when imported from Oceananigans, but not when written locally (including its helpers). Both versions of the function run normally without Enzyme.
I've attached an error message from my run of this issue in tuple_modifying_error.txt @wsmoses
tuple_modifying.txt
tuple_modifying_error.txt

using Enzyme

Enzyme.API.printunnecessary!(true)

@inline function local_flattened_unique_values(a)
    a = a[]
    flattened = flatten_tuple(a)
    return flattened
end

# Outer-inner form
# @inline flatten_tuple(a::Tuple) = tuple((a[2:end])...)
@inline flatten_tuple(a::Tuple) = tuple(a[2:end])

# The function we wish to differentiate, now using the locally-defined helper:
function local_has_flattened_tuple!(maybe_nested_tuple)

    flattened_tuple = local_flattened_unique_values(maybe_nested_tuple)
    return nothing
end
nested_tuple = ((1,2), (2,(4,5)), 3, 2, 1)
d_nested_tuple = ((0,0), (0,(0,0)), 0, 0, 0)
autodiff(Reverse, local_has_flattened_tuple!, Duplicated(Ref{Any}(nested_tuple), Ref{Any}(d_nested_tuple)))

using Enzyme

Enzyme.API.printunnecessary!(true)

@inline function local_flattened_unique_values(a)
    a = a[]
    flattened = flatten_tuple(a)
    return flattened
end

# Outer-inner form
# @inline flatten_tuple(a::Tuple) = tuple((a[2:end])...)
@inline flatten_tuple(a::Tuple) = tuple(a[2:end])

# The function we wish to differentiate, now using the locally-defined helper:
function local_has_flattened_tuple!(maybe_nested_tuple)

    flattened_tuple = local_flattened_unique_values(maybe_nested_tuple)
    return nothing
end
nested_tuple = ((1,2), (2,(4,5)), 3, 2, 1)
d_nested_tuple = ((0,0), (0,(0,0)), 0, 0, 0)
autodiff(Reverse, local_has_flattened_tuple!, Duplicated(Ref{Any}(nested_tuple), Ref{Any}(d_nested_tuple)))


Support multiple methods for a single ApiListener

closes #5443 once implemented.
first draft, new test cases proof that ApiListener is succesfully registered with multiple methods.
might want to add more test cases to proof the added functionality.
open to suggestions for improvement

Looking good. A few minor remarks. I was also wondering if it is a good idea to change an existing Larva test to support 2 methods, so we also test it with integration tests.

@Laurens-makel, you've resolved all my remarks, except for this one. Is this feasible and a good idea, do you think?

Looking good. A few minor remarks. I was also wondering if it is a good idea to change an existing Larva test to support 2 methods, so we also test it with integration tests.

@Laurens-makel, you've resolved all my remarks, except for this one. Is this feasible and a good idea, do you think?

I've indeed missed that remark. We could test indeed that when an adapter exposes >1 methods, that all methods are accepted & processed by adapter.
Could you reference me towards the existing tests you mentioned?

We could test indeed that when an adapter exposes >1 methods, that all methods are accepted & processed by adapter.
Could you reference me towards the existing tests you mentioned?

@Laurens-makel, I didn't search for a specific Larva test actually. If you search for one using the adapter with an existing GET/POST, you could just add a HTTP method, so we have it at least running with 2 supported methods. Testing both would be even better, but Larva is not my favorite hobby 😉


Green colors not matching across pages

The "Download" and "Request a Change" buttons are different shades of green. We might want to use a contrasting color here, like blue or match the other greens on the site.

Related: We have variables to have matching colors: https://github.com/AASHE/hub/blob/master/hub/static/scss/base.scss#L1-L3

Another related question (lower priority) - is it possible for color theme under topics to match the color of the sustainability topic button? For example, Curriculum and Research are both light blue, so instead of green, the contrast color for those toolkits and resource lists would be light blue as well.

I actually tried that and it didn't look good. The unity theme is pretty high key by default, and adding big blocks of color look off. You should try it out, but try very light high key colors.

The overall theme that you are using looks great with a minimal amount of green contrasting drk grey and white. My question is about just substituting the green we're using now with other colors.


Workspace order wrong if all-outputs == true

Hey,
first of all, thanks for this cool status bar!
So as you might have noticed I just spent some time fixing some issues with the workspace module but this appears to be one I don't know how to fix.
The bug: if you open workspace 1 and 3 on output 1 and then workspace 2 on output 2, they are ordered 1 - 3 - 2.
To me this actually seems like an issue with GTK rather than your code. The calls to box_.reorder_child seem to be correct to me. I added some debugging and they always call the function with the right number for the corresponding workspace. Only GTK seems to ignore this number as the workspaces are always shown in the order they are received from the IPC command and that orders them by output. Sadly I couldn't find much information as to how reorder_child is to be used.
A solution would most likely be to create an ordered list from the workspaces_ Json Value but that seems suboptimal to me.

Fixed in add9e925f4f242e37f6a5e50aaabcf9acbcb4eae
But I think the workspace module needs to be rewritten to be more optimized


ProgbarLogger no longer prints out custom metric implemented in callbacks

Hi guys. I notice one change made in the latest version of keras. The ProgbarLogger callback is now created right after BaseLogger, as shown here https://github.com/keras-team/keras/blob/master/keras/engine/training_arrays.py#L96-L107. I am wondering what is the reason why the implementation was changed from ProgbarLogger being the last callback appended to the callback list.
The previous implementation seems a bit more diverse to me, because you can manipulate the logs before ProgbarLogger prints it out, so that you can implement your own metric in a callback. But with the current implementation, this is not possible, simply due to the fact that ProgbarLogger would print the logs out before hitting custom callbacks.
Any thoughts?

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

Can you describe your use case? Metrics should be defined and put in the compile call. Why do you use a callback for that?

Hi @gabrieldemarmiesse. Thanks for your reply. The reason why I implemented the metric in callback is due to complicated metrics from business units. As a simple example, we might be interested in rank correlation within some kind of groups. To compute that, one has to pass a group index along  with y_true and y_predict to the metric function. To the best of my knowledge, it is challenging to do with compile call. However it is pretty straightforward with on_epoch_end in callbacks. I would be happy to learn if it can be done with compile. Please let me know if this makes sense.

So you use numpy to compute your metric, right?

I think numpy can definitely do it. But I have been mainly using pandas as the easiest solution.


Syntax highlighting fails with double quotes inside triple quotes

JetBrains Just plugin version: 0.4.1
When using double quotes inside triple backticks, the syntax is incorrect. A picture is worth a thousand words.

Removing the double quotes in the first block makes the syntax highlighter happy!

Here's the text for testing.
set positional-arguments := true
set shell := ['nu', '-c']
set export := true

export SOME_DIR := "some/dir"

export NU_VAR := ```
		$nu
		print $"Hello World"
		let $something = "nothing"
		$nu
	```

# Require a command to be available
[private]
require-command command:
	#!/usr/bin/env nu
	print $" '{{ command }}' "

Some of the other issues mention the need for rewriting the lexer and/or token analyzer to fix parsing issues. I understand if that's the case for this issue as well.
Thanks for an awesome plugin!

Fixed with version 0.4.3

TYVM! Awesome turnaround time.


bundle gem xxx creates wrong xxx.gemspec under Windows

Under Windows, when using bundle gem xxx to create new gem a gemspec is wrong regarding gem binaries.
This is what it looks like now:
spec.bindir        = "exe"
spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }

There is no directory exe, but it is bin like usually.
This is how it should be instead regardless of OS used:
spec.bindir        = "bin"
# Add only spec.name binary by default, otherwise console and setup will be also added, which we do not want
spec.executables   = [spec.name] if File.exists?(File.join(spec.bindir, spec.name))

Bundler version is 1.10.5.

No, when you bundle gem a gem with executables now, it places those in exe. bin is now used for commands used while developing the gem.

see http://bundler.io/blog/2015/03/20/moving-bins-to-exe.html for more info

I see. Didn't know about this fundamental change in gem development.
Closing this issue now if that's the way of the future. Thanks for the information!


App crashes and auto triggers OpenApp

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1721743515656639
Action Performed:

Open HybridApp
Tap Google sign in button

Expected Result:
Should be logged in
Actual Result:
App crashes and triggers OpenApp without needing sign in
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/bea58d2f-65a5-483c-a2fe-261ac10c0709

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I believe @staszekscp is looking into this.

@staszekscp are you looking into this?

@bfitzexpensify I am currently focusing on a mysterious crash that sometimes appears on HybridApp, and I won't be able to have a closer look at it this week. Depending on how the HybridApp rollout will go, I may have more time to focus on this issue next week

Sure thing, thanks for the update


http2: change stream state to closed from open if END_STREAM is sent and received

This corrects the HTTP/2 Stream State transition of origin server side HTTP/2 connection. I'm not sure this makes any impact or not, but this is consistent with client side HTTP/2 connection.
https://github.com/apache/trafficserver/blob/29872c258edd21fb05bec51e5667df2deae2e969/src/proxy/http2/Http2Stream.cc#L402-L408
https://datatracker.ietf.org/doc/html/rfc9113#name-stream-states

[approve ci autest]
[approve ci fedora]

[approve ci fedora]

[approve ci autest]


Running from source, missing dependency 'run-in-terminal'

It seems like vscode is depending on npm module 'run-in-terminal' which does not get installed by running scripts\npm install. There is nothing mentioned in the package.json file in the root dir (it is, however, listed as a dependency in package.json in extensions\csharp-o). gulp watch shows one error in extensions\csharp-o\src\features, line 13 reads import {runInTerminal} from 'run-in-terminal';
Running npm install run-in-terminal manually fixes this.
Edit: Might somehow be connected with #321? I rarely use npm/gulp and I am far from an expert when it comes to dependencies etc.

@jrieken
I cannot reproduce this. Please try the following steps:
git clean -xfd .
.\scripts\npm install

This should populate extensions\csharp-o\node_modules including the run-in-terminal dependency. Please reopen if you don't see that behavior.


Node.js 6.x LTS Boron Deprecation Warning

Bug description
With a clean install of the DefectDojo setup, I get a deprecation notice from NodeJS. This is because of the following nodesource url in config-vars
https://github.com/DefectDojo/django-DefectDojo/blob/d411791a84ad5b84ee4d5706d855d83c07afaa32/setup/scripts/common/config-vars.sh#L115
Steps to reproduce
$ sudo mkdir /opt/dojo
$ cd /opt/dojo
$ sudo git clone https://github.com/DefectDojo/django-DefectDojo --depth 1
$ cd django-DefectDojo
$ sudo ./setup/setup.bash

...
Select install type: (1) Single-Server, (2) Dev Install or (3) Stand-alone: 1
...
Select database type: (1) SQLite, (2) MySQL or (3) PostgreSQL: 2
...
Select (1) Remote, (2) Local: 2
...
Select (1) Install of MySQL required, (2) MySQL server already exists/is running: 1
...
Select (1) Drop existing database, (2) Keep existing database & exit: 1
...
Current MySQL database install config includes:
Do you want to change any of these values? Select (1) Yes, (2) No: 2
...
Current OS install config includes:
Do you want to change any of these values? Select (1) Yes, (2) No: 2
...
Current admin user config for DefectDojo:
Do you want to change any of these values? Select (1) Yes, (2) No: 2
...
...
Deprecation Warning

Expected behavior
Non-deprecated version of NodeJS should be used by default.
Deployment method (select with an X)

[ ] Kubernetes
[ ] Docker
[x] setup.bash

Environment information

Operating System: Ubuntu 18.04
DefectDojo Commit Message: 027d879: Fixes to plantuml action [2020-05-09 06:50:52 +0200]

The fix is merged to dev branch in #1902
However in master and other branches like <IP_ADDRESS> its still 6.x
https://github.com/DefectDojo/django-DefectDojo/blob/e73f033ca7d582acb69ac30c337c13ffe21848ab/setup/scripts/common/config-vars.sh#L115
Is dev branch stable and production ready ?

No, but release 1.6.0 is around the corner.

Yes. release/1.6.0 branch worked perfectly.
Is it intentional to not update the master branch and then create a release branch from it ?
The release/1.6.0 branch says 348 commits ahead & 61 commits behind master. Thats a lot of difference IMO.

Should be all good now, dev will become in sync with master after we merge back 1.6.5 into dev.


use tracked data and source code under project directory in episode 03?

In episode 3, the learner is instructed to clone the bast/make-pipeline repo somewhere, and then later copy the datafiles and source files to a project directory. Would it be better (more realistic and a better practice) to create a separate repo which already has the correct directory structure, so that the example project directory is already a git repo?

@Vathasav also we need to decide where we do that make part. If this is moved to a different session, please inform me about it. Until further notice I assume that this is still part of the CMake session. Any solution is fine but we need to communicate about it.

In episode 3 I have introduced make and have covered most of the points you have mentioned in the  make introduction. I will try to cover other points you mentioned. @bast we can remove the make introduction in CMake session, as you are not using it any other episodes.

OK - I will keep it in the CMake material for the moment but probably then skip it next week when teaching. Later, we can possibly completely remove it but it may make sense to have it there if CMake is treated independently since it provides (IMO) a nice introduction to it. It is not important for me that you cover everything that I have mentioned in the make part - cover the things that you find important to cover.

But sorry for distracting from the actual question (first post).


Some CI tests involving compression seem unusually slow on s390x

After enabling the CI worker for s390x noticed a few flaky failures in couchdb_file_compression_tests test module.
[2023-04-10T18:14:47.241Z] module 'couchdb_file_compression_tests'
[2023-04-10T18:14:47.241Z]   CouchDB file compression tests
[2023-04-10T18:14:53.824Z]     couchdb_file_compression_tests: fun.should_use_none...[4.950 s] ok
[2023-04-10T18:14:56.558Z]     couchdb_file_compression_tests: fun.should_use_deflate_1...[3.456 s] ok
[2023-04-10T18:15:02.110Z]     couchdb_file_compression_tests: fun.should_use_deflate_9...*timed out*
[2023-04-10T18:15:02.110Z] in function gen:do_call/4 (gen.erl, line 214)
[2023-04-10T18:15:02.110Z] in call from gen_server:call/3 (gen_server.erl, line 243)
[2023-04-10T18:15:02.110Z] in call from ioq:queued_call/3 (src/ioq.erl, line 115)
[2023-04-10T18:15:02.110Z] in call from couch_file:pread_iolist/2 (src/couch_file.erl, line 170)
[2023-04-10T18:15:02.110Z] in call from couch_file:pread_binary/2 (src/couch_file.erl, line 166)
[2023-04-10T18:15:02.110Z] in call from couch_file:pread_term/2 (src/couch_file.erl, line 155)
[2023-04-10T18:15:02.110Z] in call from couch_btree:get_node/2 (src/couch_btree.erl, line 474)
[2023-04-10T18:15:02.110Z] in call from couch_btree:stream_node/7 (src/couch_btree.erl, line 1079)
[2023-04-10T18:15:02.110Z]     undefined
[2023-04-10T18:15:02.110Z] erl_child_setup: failed with error 32 on line 265
[2023-04-10T18:15:02.110Z]   [done in 13.418 s]
[2023-04-10T18:15:02.110Z] [done in 14.401 s]

The times there seem unusually high. On PPC64LE they look like:
[2023-04-10T15:47:13.523Z]   [done in 0.375 s]
[2023-04-10T15:47:13.523Z] module 'couchdb_file_compression_tests'
[2023-04-10T15:47:13.523Z]   CouchDB file compression tests
[2023-04-10T15:47:14.448Z]     couchdb_file_compression_tests: fun.should_use_none...[0.436 s] ok
[2023-04-10T15:47:14.911Z]     couchdb_file_compression_tests: fun.should_use_deflate_1...[0.490 s] ok
[2023-04-10T15:47:15.374Z]     couchdb_file_compression_tests: fun.should_use_deflate_9...[0.500 s] ok
[2023-04-10T15:47:15.835Z]     couchdb_file_compression_tests: fun.should_use_snappy...[0.432 s] ok
[2023-04-10T15:47:16.297Z]     couchdb_file_compression_tests: fun.should_compare_compression_methods...[0.425 s] ok
[2023-04-10T15:47:16.297Z]     [done in 2.299 s]
[2023-04-10T15:47:16.297Z]   [done in 3.027 s]

On x86_64:
[2023-04-10T15:42:36.676Z] module 'couchdb_file_compression_tests'
[2023-04-10T15:42:36.676Z]   CouchDB file compression tests
[2023-04-10T15:42:37.378Z]     couchdb_file_compression_tests: fun.should_use_none...[0.389 s] ok
[2023-04-10T15:42:37.728Z]     couchdb_file_compression_tests: fun.should_use_deflate_1...[0.411 s] ok
[2023-04-10T15:42:38.078Z]     couchdb_file_compression_tests: fun.should_use_deflate_9...[0.404 s] ok
[2023-04-10T15:42:38.833Z]     couchdb_file_compression_tests: fun.should_use_snappy...[0.391 s] ok
[2023-04-10T15:42:38.833Z]     couchdb_file_compression_tests: fun.should_compare_compression_methods...[0.385 s] ok
[2023-04-10T15:42:39.183Z]     [done in 1.995 s]
[2023-04-10T15:42:39.183Z]   [done in 2.431 s]

The test in question is https://github.com/apache/couchdb/blob/main/src/couch/test/eunit/couchdb_file_compression_tests.erl

Another timeout on s390x:
[2023-04-12T17:31:45.455Z]   [done in 2.908 s]
[2023-04-12T17:31:45.455Z] module 'couch_mrview_compact_tests'
[2023-04-12T17:31:45.455Z]   Compaction tests
[2023-04-12T17:31:53.058Z]     couch_mrview_compact_tests:51: should_swap...*timed out*
[2023-04-12T17:31:53.058Z] in function gen:do_call/4 (gen.erl, line 214)
[2023-04-12T17:31:53.058Z] in call from gen_server:call/3 (gen_server.erl, line 243)
[2023-04-12T17:31:53.058Z] in call from couch_mrview_util:get_view_index_state/5 (src/couch_mrview_util.erl, line 194)
[2023-04-12T17:31:53.058Z] in call from couch_mrview_util:get_view/4 (src/couch_mrview_util.erl, line 153)
[2023-04-12T17:31:53.058Z] in call from couch_mrview:query_view/6 (src/couch_mrview.erl, line 286)
[2023-04-12T17:31:53.058Z] in call from couch_mrview_compact_tests:'-should_swap/1-fun-1-'/1 (test/eunit/couch_mrview_compact_tests.erl, line 52)
[2023-04-12T17:31:53.058Z] in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
[2023-04-12T17:31:53.058Z] in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
[2023-04-12T17:31:53.058Z]     undefined
[2023-04-12T17:31:54.335Z]   couch_mrview_compact_tests:82: should_remove...[0.632 s] ok

This ones also IO related doing compaction and swaps.

Another failure in couch_btree tests:
[2023-04-12T23:03:39.072Z]     couch_btree_tests:84: btree_open_test_ (Ensure that created btree is really a btree record)...ok
[2023-04-12T23:03:39.072Z]     BTree with sorted keys
[2023-04-12T23:03:39.072Z]       couch_btree_tests:192: should_set_fd_correctly...ok
[2023-04-12T23:03:39.072Z]       couch_btree_tests:195: should_set_root_correctly...ok
[2023-04-12T23:03:39.072Z]       couch_btree_tests:198: should_create_zero_sized_btree...ok
[2023-04-12T23:03:39.072Z]       couch_btree_tests:203: should_set_reduce_option...ok
[2023-04-12T23:03:39.072Z]       couch_btree_tests:207: should_fold_over_empty_btree...ok
[2023-04-12T23:03:41.181Z]       couch_btree_tests:220: should_return_complete_btree_on_adding_all_keys...*timed out*
[2023-04-12T23:03:41.181Z] in function gen:do_call/4 (gen.erl, line 214)
[2023-04-12T23:03:41.181Z] in call from gen_server:call/3 (gen_server.erl, line 243)
[2023-04-12T23:03:41.181Z] in call from ioq:queued_call/3 (src/ioq.erl, line 115)
[2023-04-12T23:03:41.181Z] in call from couch_file:pread_iolist/2 (src/couch_file.erl, line 170)
[2023-04-12T23:03:41.181Z] in call from couch_file:pread_binary/2 (src/couch_file.erl, line 166)
[2023-04-12T23:03:41.181Z] in call from couch_file:pread_term/2 (src/couch_file.erl, line 155)
[2023-04-12T23:03:41.181Z] in call from couch_btree:get_node/2 (src/couch_btree.erl, line 474)
[2023-04-12T23:03:41.181Z] in call from couch_btree:lookup/3 (src/couch_btree.erl, line 314)
[2023-04-12T23:03:41.181Z]       undefined

Another failure noticed on s390x only. This one doesn't seem to be related to disk IO:
[2023-04-13T16:08:57.568Z]     couch_js_tests: fun.should_exit_on_internal_error...*failed*
[2023-04-13T16:08:57.568Z] in function couch_js_tests:should_exit_on_internal_error/0 (test/eunit/couch_js_tests.erl, line 273)
[2023-04-13T16:08:57.568Z] in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
[2023-04-13T16:08:57.568Z] in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
[2023-04-13T16:08:57.568Z] **error:{assertException,
[2023-04-13T16:08:57.568Z]     [{module,couch_js_tests},
[2023-04-13T16:08:57.568Z]      {line,273},
[2023-04-13T16:08:57.568Z]      {expression,
[2023-04-13T16:08:57.568Z]          "couch_query_servers : proc_prompt ( Proc , [ << \"reset\" >> ] )"},
[2023-04-13T16:08:57.568Z]      {pattern,"{ throw , { os_process_error , _ } , [...] }"},
[2023-04-13T16:08:57.568Z]      {unexpected_exception,
[2023-04-13T16:08:57.568Z]          {exit,
[2023-04-13T16:08:57.568Z]              {noproc,{gen_server,call,[<0.22316.0>,{prompt,...},infinity]}},
[2023-04-13T16:08:57.568Z]              [{gen_server,call,3,[{file,...},{...}]},
[2023-04-13T16:08:57.568Z]               {couch_os_process,prompt,2,[{...}|...]},
[2023-04-13T16:08:57.568Z]               {couch_query_servers,proc_prompt,2,[...]},
[2023-04-13T16:08:57.568Z]               {couch_js_tests,should_exit_on_internal_error,0,...},
[2023-04-13T16:08:57.568Z]               {eunit_test,run_testfun,...},
[2023-04-13T16:08:57.568Z]               {eunit_proc,...},
[2023-04-13T16:08:57.568Z]               {...}|...]}}]}
[2023-04-13T16:08:57.568Z]   output:<<"">>
[2023-04-13T16:08:57.568Z]

This test exercises the case where it generates large JS terms to trigger the Spidermonkey InternalError on OOM.

Another failure:
[2023-04-13T20:57:28.075Z] in function couchdb_db_tests:'-should_kill_caller_from_load_validation_funs_for_deleted_db/1-fun-1-'/1 (test/eunit/couchdb_db_tests.erl, line 87)
[2023-04-13T20:57:28.075Z] in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
[2023-04-13T20:57:28.075Z] in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
[2023-04-13T20:57:28.075Z] **error:{assertException,
[2023-04-13T20:57:28.075Z]     [{module,couchdb_db_tests},
[2023-04-13T20:57:28.075Z]      {line,87},
[2023-04-13T20:57:28.075Z]      {expression,"couch_db : load_validation_funs ( Db )"},
[2023-04-13T20:57:28.075Z]      {pattern,"{ error , database_does_not_exist , [...] }"},
[2023-04-13T20:57:28.075Z]      {unexpected_exception,
[2023-04-13T20:57:28.075Z]          {throw,internal_server_error,
[2023-04-13T20:57:28.075Z]              [{couch_db,load_validation_funs,1,[{file,...},{...}]},
[2023-04-13T20:57:28.075Z]               {couchdb_db_tests,
[2023-04-13T20:57:28.075Z]                   '-should_kill_caller_from_load_validation_funs_for_deleted_db/1-fun-1-',
[2023-04-13T20:57:28.075Z]                   1,
[2023-04-13T20:57:28.075Z]                   [{...}|...]},
[2023-04-13T20:57:28.075Z]               {eunit_test,run_testfun,1,[...]},
[2023-04-13T20:57:28.075Z]               {eunit_proc,run_test,1,...},
[2023-04-13T20:57:28.075Z]               {eunit_proc,with_timeout,...},
[2023-04-13T20:57:28.075Z]               {eunit_proc,...},
[2023-04-13T20:57:28.075Z]               {...}|...]}}]}
[2023-04-13T20:57:28.075Z]   output:<<"">>
[2023-04-13T20:57:28.075Z]

We got a new KVM-based s390x CI runner and I haven't see the flaky disk failure on it any longer.
I do keep seeing this error multiple times on it:
[2023-04-13T20:57:28.075Z]      {expression,"couch_db : load_validation_funs ( Db )"},
[2023-04-13T20:57:28.075Z]      {pattern,"{ error , database_does_not_exist , [...] }"},
[2023-04-13T20:57:28.075Z]      {unexpected_exception,
[2023-04-13T20:57:28.075Z]          {throw,internal_server_error,
[2023-04-13T20:57:28.075Z]              [{couch_db,load_validation_funs,1,[{file,...},{...}]},

But not sure if that's related to disk issues, I saw it on the newer instance too now.

I haven't recently seen these failure on s390x. After the KVM switchover, s390x is usually the 2nd fastest CI worker we have now, too. Closing for now. I am sure race conditions in that tests are real but may not be related to s390x specifically.


New Document Type: Circle (M)

Summary:
Circles are structural elements of a DAO and can signify teams, groups, units, departments or any other grouping schema. Circles assist in the budgeting process, as all activities are assigned to a primary circle.
Description:
A new organizational assets (document type) called Circle can be proposed to the members. All activities in the DAO must be assigned to exactly one circle. Circles can be nested (via selection of a parent circle).
Acceptance Criteria:
The following functions are available:

[ ] Propose circle
[ ] Edit Document (update budget)  - @josejulio can proposer can edit the document type later?
[ ] Withdraw circle
[ ] Suspend circle

Create New Document Type with the following attributes / fields:

[ ] Title
[ ] Description
[ ] Parent Circle (dropdown)
[ ] Budget Number (Quarterly available Budget)

Use Cases:

[ ] Assign Assignments & Contributions to circles (Dropdown in proposal creation wizard)
[ ] Members can one or more circles to themselves in their profile (dropdown)
[ ] You can filter member & proposal list by circles
[ ] Add labels for circle names on cards

Designs:
https://xd.adobe.com/view/27bc7fff-1845-449c-913a-4ecfb3c9461c-72ef/screen/92577aa4-2fc8-478b-aec4-3f1eeba94493
https://xd.adobe.com/view/27bc7fff-1845-449c-913a-4ecfb3c9461c-72ef/screen/9e4d3c8c-ea69-46b9-a065-e1dcf85fd9a6
Step 2 - tbd

[ ] Projected Budget: when user proposes assignment / contribution - compensation token need to be removed from budget and show e.g. bar chart with budget available minus compensation for proposal + a warning if compensation is higher than budget (Design to be done)
[ ] Actual Budget Number: Show actual budget number after claims (not redemption) on /organisation page
[ ] Add "Lamp Lighter" Badge to make circle budgets editable
[ ] Add Dropdown of circles to profile

@sceenius @n13 to discuss with @markflowfarm:
We are implementing Circle Budgets and are planning to have two numbers showcased (e.g. on the / organization page - or circle subpage):

Projected Budget: when user proposes assignment / contribution - compensation token need to be removed from budget and show e.g. bar chart with budget available minus compensation for proposal + a warning if compensation is higher than budget (Design to be done)
Actual Budget Number: Show actual budget number after claims (not redemption) on /organisation page

Does this make sense to you?
Also how can we integrate this with @AugustoLara s accounting tool?

All activities in the DAO must be assigned to exactly one circle.
What does this mean ? A proposals will be part of a circle now?

[ ]  Edit Document (update budget)  - @josejulio can proposer can edit the document type later?

The update is going to create a proposal ? i.e. "propose to update the budget of circle XYZ" ? It's possible to edit a document, yes.

Only activities are assigned to circles (=assignments and contributions). Updates to circles (=budget figure) would create a new proposal, the difference is that updates can happen anytime.

Only activities are assigned to circles (=assignments and contributions). Updates to circles (=budget figure) would ? create a new proposal, the difference is that updates can happen anytime.

The purpose of circles is to make budgeting easier.
So therefore:

The budget of a circle must be editable. How do we solve this? LL? Vote? I think it should come with a badge. We could use the treasurer badge, but not sure that's not a different one - it could be the finflows badge - then finflows, can change the budgets. We could have a fin flows badge for simplicity, not tied to a specific circle - it would be more like a role in the organization. Like a discord role has certain rights. Of course the holder of the finflows badge would only change the budget on a council decision.

Voting on the DAO for each circle budget change, I think is too cumbersome. Budgeting is a finflows activity.

Circle members need to be able to be partial, example, I can be 25% on the light wallet circle, and 15% in the ReFi circle. I thought that that's necessary in order to match roles with circles.

Alternatively, when joining a circle as a member, we could join the role, not the member - then the % commitment and so on - and the total cost would be known.
Either way it doesn't make sense to me to assign people to circles when we can't figure out what the expense is for having these people in the circles.
"Activities are assigned to circles" - how to do this? Right now the implementation is to assign members to circles.
Perhaps we can simply do it as stated - assignments are assigned to circles, end of stories.
But that's not what is implemented in the PR right now.


Flux e2e tests delete repo

Description of changes:
As part of enabling Flux e2e tests we wanted to delete the repos that we auto create.

Added DELETE_REPO api support to git provider.
Modified the flux e2e tests to call delete repo as a final step

I have not enabled running these tests in this PR. Will do in a follow up.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing PR and recreating on new fork.


Control KV-cache size for StaticLLMPipeline

Overview
Introduce MAX_PROMPT_LEN and MIN_RESPONSE_LEN;

MAX_PROMPT_LEN - The maximum number of tokens that StaticLLMPipeline may process for input prompt
MIN_RESPONSE_LEN - The minimum number of tokens that will be returned as the result of generation

ov::AnyMap pipeline_config;
pipeline_config["MAX_PROMPT_LEN"] = 1024u;
pipeline_config["MIN_RESPONSE_LEN"] = 100u;
ov::LLMPipeline pipe(model_path, "NPU", pipeline_config);

The KV-cache sizes for models will be calculated the following way:

Prefill KV-cache: MAX_PROMPT_LEN
Generate KV-cache: MAX_PROMPT_LEN + MIN_RESPONSE_LEN

By default MAX_PROMPT_LEN and MIN_RESPONSE_LEN are assigned to 1024 and 150 respectively.

You are very verbose in PR description and completely ignore the actual docstrings and readmes which are the actual stuff that is read. Please, update them.

I don't think we have any readme as part of this repo. I believe we may have one but will it block the PR to get into release for now?
For now our main doc is located there: https://docs.openvino.ai/nightly/learn-openvino/llm_inference_guide/genai-guide-npu.html


proposal: builtin: iszero

Proposal: Implementing a Built-in iszero Type in Go
Abstract
This proposal aims to introduce a built-in iszero type in the Go programming language. The iszero type will provide a simple and efficient way to check if a variable is zero, enhancing the readability and maintainability of Go code.
Background
Currently, Go developers need to write custom logic to check if a variable is zero. This process can be cumbersome and prone to errors, especially for complex data types. The iszero type will simplify this process and make the code more concise and readable.
Proposal
We propose to add a new built-in type, iszero, to the Go language. This type will have a single method, IsZero(), that returns a boolean value indicating whether the variable is zero.
Here is a potential usage example:
var x int
if iszero(x) {
    fmt.Println("x is zero")
}

Rationale
The iszero type will provide several benefits:

Simplify the code: Developers will no longer need to write custom logic to check if a variable is zero.
Improve readability: The iszero type will make the code more readable and easier to understand.
Reduce errors: By providing a built-in way to check if a variable is zero, we can reduce the potential for errors in this common operation.

Compatibility
The iszero type will be a new addition to the language and will not affect existing code. It will be optional to use, so developers can choose to use it or not based on their needs.
Implementation
var zeromap = map[any]bool{}

func zero[T any](val... T) T {
	var v T
	return v
}

func iszero[T any](v T) bool {
	z:=zero(v)
	zeromap[z] = true

	return zeromap[v]
	// return reflect.ValueOf(v).IsZero()
}

Open issues

How should the iszero type handle complex data types?
Should the iszero type be available for all data types, or only for specific ones?

We welcome feedback and suggestions on these and other issues.
References

#60695

Note that it is already possible to write reflect.ValueOf(v).IsZero().  Are there any cases where that would differ from the proposed iszero(v) ?

Sorry, I see that you mentioned reflect.Value.IsZero in your code.  Does this really come up often enough to justify adding another way to do this operation?

@ianlancetaylor
Yeah it comes in the form of err=nil , *ptr==nil , s=="", b==0.

We do not want to encourage people to change from the very ordinary err == nil to iszero(err).  That would be enormous churn for zero benefit.

@ianlancetaylor
but the issue with err==nil is that  I have accidentally used  err!=nil in places of err==nil .
Its easy to misplace one with other
Same applies to the basic types like int,str etc .. as Go implicitly assigns default values; its easier for newcomers to have zero and iszero.

This would be useful in generics (and generated) code. For example, it would let cmp.Or loosen its type constraint to any so it could be used with functions.

@Nasfame I want to be very clear that we are not going to encourage people to rewrite the millions of line Go code that say if err != nil into if !iszero(err).  To the extent that that is a consequence of this proposal, I think that would be a reason to not adopt it.
(I do see other reasons for this proposal, but that is not one of them.)
@jimmyfrasche We could decide to write cmp.Or using reflect.ValueOf().IsZero().  I'm not sure that adding an iszero built-in function changes that decision.

An isZero predicate is one way to resolve #26842 (the other being introduce a universal zero value and allow any value to be compared against it).
Reflect is an option but it's slow and heavyweight. It would be much nicer to be able to write that in the language. Unsafe is another option but it has some issues surrounding floating point (see https://github.com/golang/go/issues/26842#issuecomment-1612192931 and the posts after that)

We can make the reflect package approach as fast as we want, if we decide that it's important enough.  I don't think that we should consider performance as a significant factor in deciding whether to add a built-in.

@Nasfame I want to be very clear that we are not going to encourage people to rewrite the millions of line Go code that say if err != nil into if !iszero(err).

@ianlancetaylor i don't think there is no need to rewrite loc's. Just cuz we can write errors.Is(err, nil) doesn't mean we need to rewriteerr==nil.
Its ultimately the preference of the developer. But i do think that iszero(err) is more readable than err==nil .
My primary arguments are

Like i said ==nil can be easily be misplaced to !=nil. Same applies to pointer dereferencing and default values.
Next moving forward we might rollout more builtin types, iszero(), zero, zero() makes it easier to embrace it.

FWIW, in my experience a reflect iszero check is about 50x slower than just x == *new(T). I’d love to see that optimized.

@carlmjohnson kudos!!

Next moving forward we might rollout more builtin types, iszero(), zero, zero() makes it easier to embrace it.

Honestly, to me these sound like arguments for not adopting this proposal.

Does this function only work for comparable types?

Honestly, to me these sound like arguments for not adopting this proposal.

@ianlancetaylor can u clarify this!

is it about the ambiguity on which types can be used with iszero

@ianlancetaylor can u clarify this!

He means adding several new builtins instead of just one is unappealing.

@Nasfame

@ianlancetaylor can u clarify this!

You are suggesting an increasing number of changes to Go: iszero followed by zero.  We try to be careful and slow about language changes.  If one language change works better with another language changes, we need to be even more careful.

Won't this solution eat up more and more memory during runtime?

this is a prototype implementation, it's not concurrent safe either. And of course highly less efficient design for a builtin function.

Next moving forward we might rollout more builtin types, iszero()  zero, zero() makes it easier to embrace it.

@ianlancetaylor @carlmjohnson
I am not suggesting  we rollout - iszero()  zero, zero(), the context refers to new upcoming builtin types.
My apologies, I shouldn't have brought up zero, zero()  in the proposal for iszero ; my bad!

Thanks for the discussion here. I filed #61372 to try to move a concrete plan forward. In terms of that proposal, you can write x == zero instead of iszero(x).

This proposal is a duplicate of a previously discussed proposal, as noted above,
and there is no significant new information to justify reopening the discussion.
The issue has therefore been declined as a duplicate.
— rsc for the proposal review group


MARP-555: add system prompt for edit feature

I think this PR will only fix the edit feature. I tested some cases, and they worked, but I'm not 100% sure. The result depends on the user prompt, and sometimes it doesn't meet our expectations.

I think this PR will only fix the edit feature. I tested some cases, and they worked, but I'm not 100% sure. The result depends on the user prompt, and sometimes it doesn't meet our expectations.

We should check though, that for instance the 'ask' mode still works ... as it provided good results. IF it doesn't, it seem like the wrong location to add the 'chat-gpt' system hint ... then we may rather should do it per action; not globally for all requests.

I prepared new GIF for edit feature

I prepared new GIF for insert feature

insert looks great; please update the tests to reflect the new input/output behavior:
https://github.com/axonivy-market/openai-connector/blob/master/openai-connector-test/src/com/axonivy/connector/openai/mock/json/assist-insert.json
https://github.com/axonivy-market/openai-connector/blob/master/openai-connector-test/src/com/axonivy/connector/openai/mock/json/assist-insert-response.json

Hi @ivy-rew ,
Can you help me take a look at the failed tests? They run fine on my local machine.

Hi @ivy-rew , Can you help me take a look at the failed tests? They run fine on my local machine.

@phhung-axonivy knows what to do -> @IvyTest(enableWebServer=true)


Can't install component suit with Delphi XE4

Hello
i have problems installing the component suite with Delphi XE4. I am able to compile the DragDropDXE4.bpl and dclDragDropDXE4.bpl packages of the DragDropDXE4Group.groupproj but while installing the dclDragDropDXE4.bpl-package I'm getting the following error:
"Package C:\Users\Public\Documents\RAD Studio\11.0\Bpl\dclDragDropDXE4.bpl kann nicht geladen werden.
Das angegebene Modul wurde nicht gefunden."
At the mentioned path the dclDragDropDXE4.bpl file is there.
The same error occurs using the Delphinus package manager.
I tried it on several machines Win7 (64bit) and Win10 (64bit).
Do you have any idea what I am doing wrong?
Thank you for your help!
Regards
Marco

I solved the problem by reading the closed issue "Can't Install XE7 Package".


ci: setup testing workflow

Resolves #5

@oliviergoulet5, do you know why the testing workflow fails? According to our organizations GITHUB_TOKEN permission in settings, it should have read/write to all scopes by default.


Fix PNG memory leak

This is to fix for #861.  I created it on top of @blikblum's very awesome convert-es6 branch that he created for #859 so please merge that first.

Some remarks:

Can you test replacing zlib.deflate by zlib.deflateSync?
The es6 convert branch is a moving target. Currently i'm expanding tests so can continue the conversion
In process of removing deccafeinate artifacts i removed some code that impacts performance and also identified some low hang fruits to reduce memory usage, so when conversion is finished there should be some perf gain

I created a build using deflateSync
You can test by using pdfkit-es@pngsync version

I haven't tried it and honestly the next few weeks are a big release so I doubt I'll get to it soon.  But this looks like a good solution.  I don't see why it wouldn't work.

Probably fixed by #889 . The issue still can occur because png-js decodePixels calls zlib inflate async, but odds are much smaller.


Add warning for large canvases

They don't work at the Move level.

Might not be necessary with https://github.com/banool/aptos-canvas/issues/25.

Now it's a UI restriction, the perf suffers at like more than 800x800 or so.


fix: cursor jumps out of inputs in modal

fixes https://github.com/lonelyplanet/dotcom-profile/issues/407

@thomasthesecond thanks! should be resolved now


#14 Migrate firebase auth to gitlive

I have a feeling that view model and corresponding repository has a room for improvement. This PR is only a migration to gitlive


not buying

yesterday day i have 2.8 profit with 80m coin but now i have 2.8 profit with 130m coin and bot i working but don't buying card for profit
why?

Same issue, went from 1 mil to 200 because the bot isnt buying anything

check config.js

I just tried it 1 hour ago, almost all features work normally.
Try setting config.py before running the bot

No issue

This is my current config:

The bot is just working fine, check your config.py or copy mine, w/e works the best for you. Closing the issue.


Refactor CameraView - implement internal synchronized threads

To avoid access conflicts when acquiring and displaying camera frames, let's implement two internal threads and their synchronization:

one thread acquires camera frames, stores them in a flip-flop buffer and sends an Event to the other thread once frame acquisition is completed;
the other thread waits on the Event. Once signalled, it inserts the last acquired camera frame (extracted from the flip-flop buffer) within the CameraView content.

This might solve bug #25.
That's it!

Completed, tested.


Update documentation on fixing float and currency deprecations in mendix 8

Publish for Beta 3

@ester-nl
You have removed the information about the conversion of the underlying database from this document. Is this no longer relevant, or was this an oversight?
@grootjans, I think you wrote the original - can you look at the changes and tell me if this still works from the data-storage point of view?
Thanks

@ester-nl
You have removed the information about the conversion of the underlying database from this document. Is this no longer relevant, or was this an oversight?
@grootjans, I think you wrote the original - can you look at the changes and tell me if this still works from the data-storage point of view?
Thanks

All of the deprecation error fixes will now happen at once, so fixing the Attributes (and subsequently the domain model and underlying database) will also happen together with all the other deprecation fixes. so yes, that particular sentence is no longer relevant


Question: Best way to suppress fine grained resources

General Issue
Best way to suppress fine grained resources
The Question
I have to use the below code to allow the error:
[Error at /cosmos-back-end/RegisterApi/RegisterUser/ServiceRole/DefaultPolicy/Resource] AwsSolutions-IAM5[Resource::<IACertBucketE6AB2C0F.Arn>/*]: The IAM entity contains wildcard permissions and does not have a cdk-nag rule suppression with evidence for those permission.
Code:
const userFn = new ApiDbFunction(this, "RegisterUser", {
      entry: fileURLToPath(
        new URL("user-fn/adapter/handler.ts", import.meta.url)
      ),
      environment: {
        AD_CLIENT_ID: adClientId,
        AD_CLIENT_SECRET: adClientSecret,
        AD_TENANT_ID: adTenantId,
        STAGE: stage,
        BUCKET_NAME: iaCertBucket.bucketName,
      },
      cluster,
    });
    iaCertBucket.grantPut(userFn);
    const bucketLogicalId = Stack.of(this).getLogicalId(
      iaCertBucket.node.defaultChild as CfnBucket
    );
    NagSuppressions.addResourceSuppressions(
      userFn,
      [
        {
          id: "AwsSolutions-IAM5",
          reason: "AwsSolutions-IAM5",
          appliesTo: [
            "Action::s3:Abort*",
            `Resource::<${bucketLogicalId}.Arn>/*`,
          ],
        },
      ],
      true
    );

Getting the logical id is a little cumbersome. Is there a better way or is this how I must use a fine-grained suppression for a resource?
cdk-nag version
2.14.41
Language
Typescript
Other information
No response

You can use the <IACertBucketE6AB2C0F.Arn>/* that is listed in the error. That value is going to remain consistent within the application.

@dontirun, is that true even if I move the bucket to a different stack?

I don't know the answer to that, but I would imagine that you be able to update the reference when doing that sort of refactoring.
To be clear, there isn't anything wrong with what you're doing if that's what you would like to do.

Got it. Thank you!


Add support for Insecure in cosign verifier

We cannot connect to insecure (plain http) registries with the cosign verifier because cosign does not propagate the insecure flag from their RegistryOptions to the name.Registry scheme, so it stays on HTTPS (when the host is not localhost).
This needs to be adressed on cosign first, see https://github.com/sigstore/cosign/issues/2290.

I'd like to take this on cosign side. ☝

Hi @souleb, the PR^1 on cosign side seems to be merged; what will be the next step ☝

We need to wait for this to be included in a cosign release, then test if it really works and remove the condition where we error out when verifying with insecure.

To complete @stefanprodan comment, we error out here https://github.com/fluxcd/source-controller/blob/main/controllers/ocirepository_controller.go#L405.
Then it's just a matter of passing the insecure option when creating the verifier options.

kindly ping @souleb @stefanprodan ☝

I think this still has not been released. I seems to be targeting 1.14.0.

I think this still has not been released. I seems to be targeting v1.14.0 in cosign.

This is now merged.

kindly ping, I can take care of this one, I think, it will be resolved once we upgrade cosign dep on Flux side.

Now that we've updated Cosign to 2.1 we can map Cosign's AllowHTTP to our insecure flag.


chore(api): Make tests more stable by mocking VTEX API calls

What's the purpose of this pull request?
Make all tests from @faststore/api more stable, by not relying on dynamic data.
How it works?
With this change, no actual HTTP request will be made when the tests suites are executed. Instead, a mocked version of the fetchAPI function will be called by each of our clients. Then the function calls are evaluated using jest.mock utilities.
This should make it so tests don't break if something changes in the account from which we get data (storeframework).
How to test it?
Run yarn test inside packages/api.
base.store Deploy Preview
No need, nothing changed in the actual package's implementation :)

In a future PR, we need to work towards removing these hard-coded mocked API responses too and use snapshots instead

But how would we use snapshots for the API responses? We would have the same problem we currently have, right? If something changes in the storeframework account, the API would return something different and the snapshots would fail.
In the way it is with these mocked API responses, we're testing all the layers apart from the real VTEX's APIs. We're testing the schema, the resolvers, and the logic inside the clients.


switch demo to folium

sadly ipyleaflet doesn't work in binder nor in colab 👎

Oh yeah this is true, you're right.


Return to last opened Contents tab after Delete

Describe your problem
I've almost removed actual pages when removing a couple of drafts. The reason for this is that after deleting a draft, the page refreshes without returning to the Drafts tab. This can be generalized to other tabs, such as in the Settings, too.
Expected behavior
When an operation requires the page to refresh, reload the tab the user was working on, not just the first tab available.
Actual behavior
After a refresh, the first tab is always shown, which causes confusion.
Steps to reproduce the problem
Delete a draft.
Bludit version
3.6.1
Related (but less likely to destroy user data): https://github.com/bludit/bludit/issues/893

Hello, with the new version of Bludit v4.0 rc1, I would like to close the old Github issues. If you feel that your issue is not resolved in the latest version, create a new ticket.

Help and Support use the Forum https://forum.bludit.org
Bugs and new requests here in Github https://github.com/bludit/bludit/issues


add support for session expiry

Add support for specifying session timeout via MqttProperties. This enables one to set the expiry via configuration properties

Good remark, thanks, I adjusted the PR accordingly.


[LGSM] Detect if missing dependencies and warn

Hi,
i installe dGMod server today all worked fine i allowed ports and tried to join buit seteam dont sees the server.
after an hour i tried starting it via srcds_run and it puts out an error:
"error while loading shared libraries: libstdc++.so"
wich google says is fixed by:
"sudo apt-get install lib32stdc++6"
and all worked great again :D
so maybe the GSM should check the srcds if it puts out errors before saying all works [ok]
and while installing gmod check sudo apt-get install lib32stdc++6 ?
with best +DS_DV+

The dependencies are listed in the instructions and the wiki.
http://gameservermanagers.com/lgsm/gmodserver/
https://github.com/dgibbs64/linuxgsm/wiki/Dependencies
There is also ./gmodserver debug that would of identified the issue.
There is also a hidden option that I have recently been working on.
./gmodserver depsdetect
that does something similar what you are suggesting. Once I am happy I will probably write it in to the script to warn admins of missing dependencies.

I am developing this currently. My code works for Ubuntu however needs polish.
Currently is warns of a missing dependency. If the user has sudo it will install it for you.
There are some dependencies that are optional so I will need to deal with them differently.

Speaking of that, most debian users don't have "sudo", as i recently learned the default behaviour is : If you have a root password set, sudo is disabled.
So maybe packages installation instructions should be modified accordingly.

I have ubuntu server running with sudo just fine :)
btw root should be disabled anyway ther is no need if you have an account wich hase rights to use sudo ;)

The way I added it on my branch was to have a package_check_$OS_$ARCH variable that is a simple space-delimited string of package names. Checks the package database to see if they are installed, and reports back. I'm looking at adding per-game (or per-engine) additional prereqs, but it'll at least give a message with the command to paste to install the missing deps.

Just released the feature to master. Please test and let me know. Works with Ubuntu Debian and CentOS. If yum or apt-get are not present then the check will be skipped.

Hey, just had a problem with this.
User I run csgo as does not have sudo access, but the dependencies are installed and the script is giving me the command for debian and not redhat/centos.
Problem:
http://scottsmudger.co.uk/16-02-16_23-38-30.png
Dependencies:
http://scottsmudger.co.uk/16-02-16_23-42-47.png

It checks if apt-get or yum is installed. For some reason it thinks apt-get is installed on your centos server. Can you confirm if apt-get is installed please

Correction if dpkg-query is present then the script assumes Ubuntu or Debian. I may have to add if yim not present as well

I can confirm it is not installed, as YUM would not be available. It isn't installed as a package anyway
sudo yum list installed | grep apt
Didn't return anything

What is the output of:
command -v dpkg-query

/usr/bin/dpkg-query

Please do the same but for apt-get as this should not be installed

@dgibbs64 While you're in there, check that out for mail server management !
https://github.com/dgibbs64/linuxgsm/commit/64686921ddc3239acb4eb992fb3c22dd03ee1fc0#diff-0d55734e68e0cd33f8b206a5f6d70686

command -v apt-get
Doesn't return anything

I am closing this issue as I believe this feature is now stable. However please advise if there are still issues


MasterDetailEditor - list not showing on reload

Hi
First of all, thanks for this fantastic framework.
I would appreciate any pointers about where I may be going wrong.
I am using the MasterDetailEditor as per the documentation.
When I first add a detail record, and close the detail dialog, the correct column is displayed with the new record:

However, when I save that record, and go back into it, the column is now empty:

No errors in the console.
I have confirmed that the record has been correctly created in the database.
Using the console, I have confirmed that getLookup is populating the column names correctly.
As far as I can tell, I have followed the steps in the example to the letter.

Does anyone have any pointers about where I should be looking for my error?
Thanks in advance.

Would you mind adding the snippet of code of how you implemented the Master detail?

See issue https://github.com/volkanceylan/Serenity/issues/1959 and https://github.com/volkanceylan/Serenity/issues/1962
In your xyzDialog.ts try to add
protected updateInterface() {
    super.updateInterface();
    this.form.YoutDetailList.element.triggerHandler('layout');
}

Again :) See tutorial, IncludeColumns and MinSelectLevel


BUG: Hero section not responsive.

Describe the bug

The hero section is not responsive for smaller screen sizes.
Expected behavior

Changing the style so that the section looks good in all types of devices.
Screenshots (Mandatory)

Additional Context (Mandatory )
Please assign me this issue, I have relevant skills to solve this task.
Are you contributing under any open-source program ?

Yes, I am a GSSoC'24 contributor and I want to contribute to this project under it.

Hi ,
This is a good issue, please start working on this.
Regards
Project Mentor (GSSoC' 24)


pings pcmdi db to record a call to pcmdi_metrics

fix #282

@durack1 @gleckler1

@doutriaux1 let's wait till we can check this and make it work - and @chaosphere2112 has the server back up and running..

@durack1 @doutriaux1 Alright, the analytics server has been upgraded to Django 1.8 and refactored to store data in a more compressed form than previously. Currently there's no web-based way to get access to the data, but it's all stored in the DB and I can get at it from the command line on the server.

@chaosphere2112 @doutriaux1 ok, so we're probably at the stage of turning it on and testing things out.. And seeing if @chaosphere2112 can make sense of what we're logging..

@doutriaux1 I'm reluctant to merge until the travis builds are passing fine.. Do you know what is going on here?


No apparent way to fsync a directory

When I try to open a directory with uv_fs_open() and O_RDONLY, I get UV_EISDIR.
However, AFAIK there's no safe way to create a durable file under POSIX without fsyncing a directory that contains the inode.  This means you can't write an ACID database in pure libuv.
Is a uv_fs_fsyncdir() or something like it a possibility?

  It works fine with O_RDONLY, not so fine with O_RDWR.


How to Bind Grid on lookup selection

hello everyone!
I am having problem with binding a grid

here you can see the lookup of enquiry table and each enquiry contains some products

And on that enquiry selection i want to display all products in this grid
anyone have solution then please help

just use build in filtering feature

hey @ga5tan
can you explain it in detail or give a reference with an example ?

there is crapload of examples on demo site.
But this one has extra customization of QuickFilters:
https://serenity.is/demo/BasicSamples/QuickFilterCustomization

hey @ga5tan
the link you sent is for grid.ts
and i have to bind the data on dialog.ts


Improve README

Add more information about the project and how to get started.
Maybe also include a code of conduct or more information about how to contribute.

I would like to do this issue @astanik


Feature/update cruft template

Codecov Report

Merging #1036 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1036   +/-   ##
========================================
  Coverage    88.35%   88.35%
========================================
  Files           15       15
  Lines         2783     2783
  Branches       475      475
========================================
  Hits          2459     2459
  Misses         231      231
  Partials        93       93


Added Distance (Logged - Mileage) stats

This is an improvement to Distance stats to avoid confusing on Mileage and logged distance in TeslaMate as discussed here https://github.com/teslamate-org/teslamate/issues/3727
Now, we can see them separately and its corresponding hint.

Amazing @jheredianet. Thanks so much. I am about to dive more into your dashboards and add them separately.

Just to mention that I can see the feature inside of "Battery Health" of version 1.28.4 but it missing from "Battery Health - LFP".  Thanks, it is a good addition.

Just to mention that I can see the feature inside of "Battery Health" of version 1.28.4 but it missing from "Battery Health - LFP".  Thanks, it is a good addition.

Yeah, that's expected but not ideal.


Mail Connector - IMAP, POP3, SMTP - use of multiple propery files, multiple mail accounts

Hi there,
the MailConnector http://docs.testerra.io/testerra/1.0-RC-23/#_mail_connector
has a property file that allows to specify the Mail account.
1.) Can this propery file include more than one mail account? OK, I could use POP3 and IMAP in parallel, but this is not it.
2.) If not, can I have more than one property file? How could I reference them?
3.)In the ImapMailConnector, I see the reference to the property values and I could override the init method, but I am aksing myself if I am missing soemthing here and this is possible out of the box.
The use case is that the customer orders something and we check if the mail was received and then there is a sales rep, who also get an e-mail about the order, which we would also like to check.
Thanks.
Kind regards.
Christian

You can try to set properties manually and the create a new instance of ImapMailConnector like
PropertyManager.loadProperties("first-account.properties");
ImapMailConnector firstAccountConnector = new ImapMailConnector();
PropertyManager.loadProperties("second-account.properties");
ImapMailConnector secondAccountConnector = new ImapMailConnector();

But the mailconnection.properties must remain complety empty. Otherwise the values of it overwrite your custom property files.

@chrberSTO Does it solved your problem and can we close this issue?

Have not had the time to check, yet. Thank you for the hint, I will check later. Please close.
From: Martin Großmann @.>
Reply to: telekom/testerra @.>
Date: Thursday, 22. April 2021 at 11:25
To: telekom/testerra @.>
Cc: chrberSTO @.>, Mention @.***>
Subject: Re: [telekom/testerra] Mail Connector - IMAP, POP3, SMTP - use of multiple propery files, multiple mail accounts (#19)
@chrberSTO Does it solved your problem and can we close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


from datar.all import * doesn't import trimws

Hello pwwang
I tried to call:
from datar.all import *
and it worked fine, but when I needed trimws I had to import using
from datar.base.string import trimws
If it is a intended functionality please just close the issue

Ah, it's absolutely not intentional. Good catch! Will add it in the next version.

Added at v0.5.1


Document how to get appcast for sourceforge and hockeyapp casks

In https://github.com/caskroom/homebrew-cask/blob/master/doc/cask_language_reference/stanzas/appcast.md we should document all common methods we know.
Currently missing sourceforge:
Sourceforge projects seem to have simple and predictable appcasts, just like github projects: http://downloads.sourceforge.net/projects/{{project_name}}/rss. A more specific rss can be used as needed, pointing to a specific directory structure: http://sourceforge.net/projects/{{project_name}}/rss?path=/{{path_here}}.
And hockeyapp.net:
hockeyapp.net urls are usually of the form https://rink.hockeyapp.net/api/2/apps/STRING. An appcast can be attained by removing api/2/, to get something like https://rink.hockeyapp.net/apps/STRING.
Example:
https://rink.hockeyapp.net/api/2/apps/0172d48cceec171249a8d850fb16276b
https://rink.hockeyapp.net/apps/0172d48cceec171249a8d850fb16276b
Pinging @adidalal.

Node.js/Electron apps normally use an updater.js/auto-update*.js inside their source, but it's a manual process.
ex: gitkraken

But those are only available sometimes, and each electron app is free to implement it they’re own way, it’s too variable.
We should only document common easy methods.

SF appcasts might be broken/not reliable.
https://github.com/caskroom/homebrew-cask/issues/19141 has always thrown an outdated error.

@adidalal We need to check if the other sourceforge casks throw an error as well. That address seems to broad, though, let’s narrow down the appcast to https://sourceforge.net/projects/animgifqlgen/rss?path=/files.


Add myself to the CONTRIBUTORS file [phab:D759]

Created by nghia at 2016-03-07 21:00:31:

fromberger accepted this revision at 2016-03-07 21:47:36

nghia closed via commit 7f69523df90fd6a063d37bebddcbb235208698ea at 2016-03-07 23:14:59


React Native App MacOS process.cwd() got path in node_modules.

the below window is appendFile's content.
error detail
error: bundling failed: Error: Unable to resolve module `./src/app/dao/ConfigurationWithRedux` from `/Users/provoke/Desktop/mobile/app/src/app/conf/configureStore.js`: The module `./src/app/dao/ConfigurationWithRedux` could not be found from `/Users/provoke/Desktop/mobile/app/src/app/conf/configureStore.js`. Indeed, none of these files exist:

  * `/Users/provoke/Desktop/mobile/app/src/app/conf/src/app/dao/ConfigurationWithRedux(.native||.ios.js|.native.js|.js|.ios.json|.native.json|.json)`
  * `/Users/provoke/Desktop/mobile/app/src/app/conf/src/app/dao/ConfigurationWithRedux/index(.native||.ios.js|.native.js|.js|.ios.json|.native.json|.json)`
    at ModuleResolver.resolveDependency (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:158:865)
    at ResolutionRequest.resolveDependency (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:92:16)
    at DependencyGraph.resolveDependency (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/node-haste/DependencyGraph.js:271:4465)
    at dependencies.map.relativePath (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/DeltaBundler/traverseDependencies.js:379:19)
    at Array.map (<anonymous>)
    at resolveDependencies (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/DeltaBundler/traverseDependencies.js:378:16)
    at /Users/provoke/Desktop/mobile/app/node_modules/metro/src/DeltaBundler/traverseDependencies.js:203:33
    at Generator.next (<anonymous>)
    at step (/Users/provoke/Desktop/mobile/app/node_modules/metro/src/DeltaBundler/traverseDependencies.js:296:307)
    at /Users/provoke/Desktop/mobile/app/node_modules/metro/src/DeltaBundler/traverseDependencies.js:296:467

but in windows and linux not which.

Hello.
Please include your babel config, and if this is the development or production build.
Could you clarify what you mean by "but in windows and linux not which"?

Sorry, I mean my project doesn't have this error on windows and linux, , just on macos.
my .babelrc :
{
"presets": [
"react",
"react-native"
],
"plugins": [
[
"babel-plugin-root-import",
{
"rootPathSuffix": "src/app"
}
],
"transform-decorators-legacy"
],
"env": {
"production": {
"plugins": [
[
"babel-plugin-root-import",
{
"rootPathSuffix": "src/app"
}
]
]
}
}
}
And package.json :

This error occurred when starting the project through the XCode run operation, and I tried npm start also this error.

Try specifying it just once
{
  "presets": [
    "react",
    "react-native"
  ],
  "plugins": [
    "transform-decorators-legacy"
  ],
  "env": {
    "production": {
      "plugins": [
        [
          "babel-plugin-root-import",
          {
            "rootPathSuffix": "src/app"
          }
        ]
      ]
    }
  }
}

The readme isn't very clear about this, sorry.


Gracefully handle missing a missing issue/PR when importing a project item

When trying to import a project item where the pull request or issue is missing, this is intended to log a warning, but currently, it causes the entire application to crash.
This fixes it, so a warning is logged and we move on to importing the next project item.
Fixes #227.

@coderabbitai review


Update Minimum Request Version to Mitigate Security Vulnerability

While this repository doesn't have a package-lock.json and developers can choose which version of request to use, there is a CVE vulnerability (CVE-2018-3728) in request->hawk->hoek<EMAIL_ADDRESS>mitigates this by updating hawk which has a version of hoek that no longer has the vulnerability.
This PR bumps the minimum version of request to 2.85.0 so other repos cannot download a lesser, vulnerable version.

Some more details:

https://github.com/request/request/pull/2751/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R38<EMAIL_ADDRESS>actually bumped hawk past v5 which solved the issue
2.85.0 bumps hawk to v7.07

I think I chose 2.85.0 instead of 2.82.0 because codecov chose 2.85.0 to mitigate the issue (https://github.com/codecov/codecov-node/pull/96) :man_shrugging:
I can bump the version up or down per your request!


More skills and integration with classes

This is a mess to say the least and needs a rewrite, i'd rather start a new PR. Also, little to no progress has been done so it's not even a loss.


[CI] Eslint check

We need to add a GitHub action to make eslint check on each PRs.

Closed by #53


Request: Removed PhantomJS dependency

PR Checklist:

[ ] I have run npm test locally and all tests are passing.
No, https tests are failing with cert expired.

PR Description
Phantomjs is an unmaintained project, removed phantomjs dependency from package.json.
Signed-off-by: ossdev07<EMAIL_ADDRESS>

Ping @reconbot
The karma tests were breaking because of no alternative to run test cases which were earlier running through phantomjs.
As I mentioned we can use karma-chrome-launcher as an alternative to phantomjs.
Please, can you reopen this pull request.


Reduce size of decision tree

@raduom https://github.com/kframework/llvm-backend/wiki/Occurrences

I mean it needs to be both easily serializable and easily convertible to a unique string representation. What exactly do you have in mind?

I was looking at the comment you wrote and I though that you can pretty much write it in Haskell, with a couple of comments. I do not really like stringly types in a program, especially if written in Haskell. I would just use a normal Haskell data structure with a Show instance.

Yeah. The recursive structure is clear and the code also looks more intuitive. Some things I would have done differently, and probably will later:

I would have use a record since matching positional arguments can be somewhat confusing for large data structures.
Recursive data structures can be written in fix point notation to be able to abstract over recursion.

However, all this can be done later.


Improve CSS

Fixes #83

Codecov Report

Merging #125 into master will increase coverage by 5.61%.
The diff coverage is n/a.

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


Please Provide Better Error Statement other than "Missing parameters"

First, this is a great tool, but you seem a little bit more work is needed to either catch or at least provide a better description of an error related to parameters.
For instance, I was able to get this error with path that did not contain any parameters. It turns out to be something, but the error provided no guidance whatsoever.

PR?


Banner Hex Color issues

The Banner on the main page, seems to alter the primary color hex code.
After setting the color in the config, it seems to brighten the color when actually generated.
Is there anyway we can prevent it from brightening the color?

Hey there, We used a background color on the banner. so we reduce the opacity from the primary color for the banner. If you want to use a solid color on the banner, then you need to modify the main CSS.


Fix testing with the rust agent

Fix testing with the rust agent (triggered by setting the RUST_TEST environment variable)
This depends on:

https://github.com/keylime/rust-keylime/pull/448

Uses the tpm_ownerpassword when generating the EK

https://github.com/keylime/rust-keylime/pull/432

Add the tpm_ownerpassword to the keylime-agent.conf

One test is still skipped due to https://github.com/keylime/rust-keylime/issues/447
Add a test case for the /keys/verify endpoint for the rust agent

Note that when RUST_TEST=1 is set, the tests are executed first with the python agent, then with the rust agent.
The changes related with the owner password on rust agent are to allow it to access the TPM after the python agent took its onwership and set a password.


Load Consistency

Addresses #2097

If someone who has a better overall knowledge of the codebase knows other places where there are load more let me know.
Sent with GitHawk

Should be used in the base list VC, base list 2 VC, and issues for sure (sorry for short names, on mobile). I think that would cover everything.
Sent with GitHawk

Okay thanks! I’ll go ahead and see if those are using their own or using LoadMoreCell
Sent with GitHawk

@rnystrom I think I nabbed them all now. There was one in BaseViewController, nice catch!


MTL-2513

Summary and Scope
Changes intended for CSM 1.6.1
Fix for CASMTRIAGE-7434 / MTL-2513

https://jira-pro.it.hpe.com:8443/browse/CASMTRIAGE-7434
https://jira-pro.it.hpe.com:8443/browse/MTL-2513

Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] Testing is appropriate and complete, if applicable

Superseded by https://github.com/Cray-HPE/csm/pull/3778


Fixed: Uneven Color for "New" button

Pull Request
Related Issues:
Fixed uneven color with UI
Fixes #154
Description:
Changed color of the button using tailwindcss, and aligned them center for easy usage. Also aligned the form title to the center for readability.
Checklist:

[x] I have tested my changes.
[x] My code follows the project's coding standards.
[ ] I have updated the documentation (if applicable).

Screenshots:
Desktop:

Tablet:

Mobile:

@chandiran-v done


GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python 3.8 PostCommit

retest this please

Run Python 3.8 PostCommit

R: @kennknowles do you have time to take a look at this?


Update readme for 5.0.0

https://github.com/rspec/rspec-rails/releases/tag/v5.0.0
https://rubygems.org/gems/rspec-rails/versions/5.0.0

Please correct me if I'm totally wrong with my understanding of the versioning strategy:
rspec-rails 4.0 is out with Rails 6.0 support
rspec-rails 4.1 supports Rails 6.1
rspec-rails 5.0 is an early version with bleeding edge in-development Rails 7 support
With this in mind it's hard to call 5-0-mainteance a stable release and recommend it to everyone who visits the front repo page.
main tracks changes that will be merged to both the stable 4-1-maintenance and released as 4.1.x, and 5-0-maintenance and released as 5.0.x.
Rails 7-specific changes will go to main, will be merged to 5-0-maintenance, but not 4-1-maintenance.
@JonRowe Is this correct?

This should have been cherry picked from 5-0-maintenance


Make restore poller frequency equal to backup frequency

Pull Request Test Coverage Report for Build 1882

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 88.6%

Totals

Change from base Build 1881:
0.0%

Covered Lines:
8094

Relevant Lines:
8844

💛  - Coveralls


Unable to start the project locally

Does this occur consistently? Yes
Repro steps:
I try to attach to the .NET Functions and it crashes. I've uninstalled and re-installed VSCode, the Azure function extensions etc. The Bundle config that the error says to remove, it automatically regenerated when trying to start the project.
1.Press Start Debugging
2. This error is generated:
Extension bundle configuration should not be present for the function app with pre-compiled functions. Please remove extension bundle configuration from host.json: C:\AzureFunction\bin\Debug\netcoreapp3.1\host.json
Here is the output from the Terminal
Executing task: C:\Program Files\dotnet\dotnet.exe build /property:GenerateFullPaths=true /consoleloggerparameters:NoSummary <
Microsoft (R) Build Engine version 16.6.0+5ff7b0c9e for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Determining projects to restore...
All projects are up-to-date for restore.
AzureFunction -> C:\AzureFunction\bin\Debug\netcoreapp3.1\bin\AzureFunction.dll
Terminal will be reused by tasks, press any key to close it.

Executing task: func host start <

              %%%%%%
             %%%%%%
        @   %%%%%%    @
      @@   %%%%%%      @@
   @@@    %%%%%%%%%%%    @@@
 @@      %%%%%%%%%%        @@
   @@         %%%%       @@
     @@      %%%       @@
       @@    %%      @@
            %%
            %

Azure Functions Core Tools (2.7.2508 Commit hash: 4da36643c32783a832094318afcd679fa9d76455)
Function Runtime Version: 2.0.13351.0
Extension bundle configuration should not be present for the function app with pre-compiled functions. Please remove extension bundle configuration from host.json: C:\AzureFunction\bin\Debug\netcoreapp3.1\host.json
The terminal process terminated with exit code: 1
Action: azureFunctions.pickProcess
Error type: Error
Error Message: Functions host is no longer running.
Version: 0.22.1
OS: win32
OS Release: 10.0.18363
Product: Visual Studio Code
Product Version: 1.45.1
Language: en

Call Stack
extension.bundle.js:334:101651extension.bundle.js:334:101651
fulfilled extension.bundle.js:334:99073

@hardEight how did you create this local project? Also - what do the settings look like in .vscode/settings.json?

it automatically regenerated when trying to start the project.

This shouldn't be happening for C# projects. That's why I'm curious about your settings

Hi Eric,
I followed the directions for developing locally in the Azure Portal. I was
able to create C# function apps previously, this one I was hoping to create
a Powershell App. I assume that may be part of the issue?
Here are the settings you requested.
{
"azureFunctions.deploySubpath": "bin/Release/netcoreapp3.1/publish",
"azureFunctions.projectLanguage": "C#",
"azureFunctions.projectRuntime": "~3",
"debug.internalConsoleOptions": "neverOpen",
"azureFunctions.preDeployTask": "publish"
}
On Mon, Jun 8, 2020 at 4:55 PM Eric Jizba<EMAIL_ADDRESS>wrote:

@hardEight https://github.com/hardEight how did you create this local
project? Also - what do the settings look like in .vscode/settings.json?
it automatically regenerated when trying to start the project.
This shouldn't be happening for C# projects. That's why I'm curious about
your settings
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-azurefunctions/issues/2149#issuecomment-640881520,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBZGD7PGIQ72FAM3UW67V3RVVF3TANCNFSM4NVFXOHQ
.

So I and a college (and someone on stackoverflow) face a similar problem but with python (different laptops, same project, different branch, different python version (3.8.0, 3.8.1) it died on the same day)
Monday evening -> everything worked fine, was able to run/debug and deploy.
Tuesday morning -> Running the function using vscode no longer works, running the function using func host start on the commandline works fine but i'm unable to attach a debugger to it.
(The laptop did crash during the night and was rebooted)
When running it crashes on imports of grpcio inside the azure functions. If these are fixed by installing the packages, it will crash on a protobbuf import.

@hardEight yeah if you were trying to create a PowerShell app, it's weird to me that the projectLanguage is set to "C#". You selected "PowerShell" when creating the project? Double check your user settings (as opposed to workspace settings - see more here) to make sure "projectLanguage" isn't set. If it's set in the user settings, we use that language when creating new projects instead of prompting each time.
@matthijsvdr your issue sounds a bit different. Please file a new issue with the full errors you mentioned

Thanks Eric, after uninstalling and reinstalling .NET 2.something and VS
code and Azure Functions a few times, I seem to have stumbled upon a
working set! Thanks.
On Tue, Jun 16, 2020 at 2:11 PM Eric Jizba<EMAIL_ADDRESS>wrote:

@hardEight https://github.com/hardEight yeah if you were trying to
create a PowerShell app, it's weird to me that the projectLanguage is set
to "C#". You selected "PowerShell" when creating the project? Double check
your user settings (as opposed to workspace settings - see more here
https://code.visualstudio.com/docs/getstarted/settings) to make sure
"projectLanguage" isn't set. If it's set in the user settings, we use
that language when creating new projects instead of prompting each time.
@matthijsvdr https://github.com/matthijsvdr your issue sounds a bit
different. Please file a new issue with the full errors you mentioned
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-azurefunctions/issues/2149#issuecomment-644925438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBZGDZFJCYNG4E5K7SKM4DRW6YUBANCNFSM4NVFXOHQ
.


nable to run the code

Hello, we are very interested in your work and have tried running your code, but we encountered some issues. You mentioned in another repository that it is best to use Julia 1.6, is that the case for this repository as well? Additionally, are there any specific dependency versions required? I am currently unable to run your code, and I am not sure if it's an environment issue or a mistake on my part. Could you please explain which parts of the code I should run step by step? I haven't been able to find a practical README file to understand these basic details.Additionally, I found a file named CRNN-biomass-python-main. May I ask if this file can be used interchangeably with the Julia file? It seems to me that the Python file contains very little content.

Hi @Dionysus7777777 , thanks for your interests. Looks like what you are facing is due to environments or dependency issues. I don't have the original environments I used at hands. I'd recommend consulting Julia documents and Code Assistance like ChatGPT to adapt the code to the Julia/package versions you are using.
You shall start from this file https://github.com/DENG-MIT/Biomass.jl/blob/main/src/crnn_cellulose.jl, which is the main program.
I don't recall if that python version works or not. I'd recommend working with the Julia version.


Support WebDriver > 2

Having to downgrade webdriver but maybe it can be updated to support later versions?
Thanks

Whoops, #144 and #106 cover this.