What is the idea with the generated suppressions?

A ton of FP issues were resolved with near zero human intervention - fantastic! However, I didn't find any information yet what the intention of https://github.com/jeremylong/DependencyCheck/tree/generatedSuppressions is.
Should it become part of the "real" base file at some point? Or, more generally, how are we supposed to use the generated file?
Update
By poking through the commits on false-positive-approvals.yml I found #4723 which appears to have kick-started this. #4901 is related.
Based on what I learned from the linked issues and code changes they initiated I conclude that

in order to use the 30+ FP suppressions that were automatically processed in recent weeks I need to set -DsuppressionFiles=https://jeremylong.github.io/DependencyCheck/suppressions/publishedSuppressions.xml  - at least until 8.0 is released.

@jeremylong @aikebah Correct?

@marcelstoer correct. The automated usage of it is scheduled for the 8.x release and until it's out you can add https://jeremylong.github.io/DependencyCheck/suppressions/publishedSuppressions.xml to your own suppressionfile(s) (if any) to benefit from the fast-tracked suppressions resulting from the automation after core committer approval.

Thanks for the confirmation. What triggered this question was the fact that I found those 30+ FP fixes merged but not being part of the 7.2.0, 7.2.1 or 7.3.0 releases - they only went into the generated file AFAIU. Would it make sense to regularly transfer the generated suppressions manually to dependencycheck-base-suppression.xml until 8.0 is out, assuming there will be more 7.x releases before 8.0?

Also see https://github.com/jeremylong/DependencyCheck/pull/5004. I meant to do this on earlier 7.x releases (since we started generating the file).

Sorry to bother again. 8.0 isn't out yet but https://jeremylong.github.io/DependencyCheck/suppressions/publishedSuppressions.xml is now a 404? Is that intentional?
95b86db39eeb21c980f59a8d020f962dc1ae0fe0 from December 4th in the gh-pages branch removed this file.

@jeremylong gut feel is this is an unanticipated side-effect of the release workflow?

@jeremylong thanks for restoring the file 👍


Deleting a project

Does not appear to be a way to delete a project.

Projects can never be deleted, only archived.  Archiving issue is here: https://github.com/projectjellyfish/api/issues/394


text/template/parse: Unexpected Behaviour of ActionNode.Position in text/template - Potential Bug?

Go version
go version go1.21.4 darwin/arm64
Output of go env in your module/workspace:
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/xxx/Library/Caches/go-build'
GOENV='/Users/xxx/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/xxx/projects/gopath/pkg/mod'
GOOS='darwin'
GOPATH='/Users/xxx/projects/gopath'
GOROOT='/opt/homebrew/opt/go/libexec'
GOSUMDB='sum.golang.google.cn'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/opt/go/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.4'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/xxx/projects/xxx/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/mt/w_9vxxsn0ssfxz46n8lbrykh0000gp/T/go-build2022525700=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
https://go.dev/play/p/U8UabjbhJsN
// You can edit this code!
// Click here and start typing.
package main

import (
	"fmt"
	"reflect"
	"text/template"
)

func main() {
	temp := `{{.locale}}`
	tree, err := template.New("test").Parse(temp)
	if err != nil {
		panic(err)
	}

	node := tree.Root.Nodes[0]
	fmt.Println(reflect.TypeOf(node))
	// expect: {{.locale}} 0
	fmt.Println(node.String(), node.Position())
	// expect: {{.locale}}
	fmt.Println(temp[node.Position():])
}

What did you see happen?
I am currently developing a highlighter for text/template scripts. To accomplish this, I require an Abstract Syntax Tree (AST) for the provided template. During this process, I encountered an unexpected behaviour with the Position method of the ActionNode.
Contrary to my expectations, the Position of the ActionNode does not include the delimiter ('{{'). However, the String method does return the full {{.locale}}, inclusive of the delimiter. This discrepancy has prompted me to question whether there is a bug, or if I have misunderstood the purpose of the Position function in the Action interface.
According to the given documentation comment, Position() Pos should represent the 'byte position of the start of node in full original input string'. This, in my understanding, should include the delimiter.
Could you please clarify this discrepancy, or confirm if this is indeed a bug? Thank you for your time and assistance.
What did you expect to see?
temp := `{{.locale}}`
tree, err := template.New("test").Parse(temp)
if err != nil {
  panic(err)
}

node := tree.Root.Nodes[0]
fmt.Println(reflect.TypeOf(node))
// expect: {{.locale}} 0
fmt.Println(node.String(), node.Position())
// expect: {{.locale}}
fmt.Println(temp[node.Position():])

@robpike, @mvdan, can you give this a look?

The action starts inside the delimiter, so the position field is correct. The String method includes the delimiter for ease of use. So yes, they are inconsistent and perhaps should be consistent, but it's more likely to break things than improve them to try to change this. Plus, you'd need to decide what change to make.
I think it can stay as is.

As I said, the action starts inside the delimiter, that's why.

As I said, the action starts inside the delimiter, that's why.

Sorry I may not have made it clear, I mean if there should be a node wraps the action node but contains the delimiter and prefix such as "if" and "range".

It's impractical to change the parse tree given the compatibility promise. You'll just have to code around it, which shouldn't be too inconvenient.

It's impractical to change the parse tree given the compatibility promise. You'll just have to code around it, which shouldn't be too inconvenient.

Got it, thanks.


jQuery Inputmask 5.0.6-beta.15 does not work with wkhtmltopdf (QtWeb Browser)

This Plugin does not work with the QtWeb Browser (http://www.qtweb.net/) which is used by the HTML-to-PDF conversion Software wkhtmltopdf (https://wkhtmltopdf.org/).
Here is a basic test using a 'numeric'-type Inputmask:
<input type="text" data-inputmask="'alias' : 'float'" value="1000000,56">

<script src="https://code.jquery.com/jquery-1.12.4.js" integrity="sha256-Qw82+bXyGq6MydymqBxNPYTaUXXq7c8v3CwiYwLLNXU=" crossorigin="anonymous"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery.inputmask/5.0.6-beta.15/jquery.inputmask.js"></script>
<script>
var aliases = {
	float: {
		alias: 'numeric',
		groupSeparator: '.',
		rightAlign: false,
		removeMaskOnSubmit: true,
		autoUnmask: true,
		radixPoint: ',',
		digits: 2,
		digitsOptional: true,
	}
};

Inputmask.extendAliases(aliases);

$('[data-inputmask]').inputmask();
</script>

Describe the bug:

The mask seems to be applied correctly in all browsers (IE included) except for QtWeb.

OS: Windows 10
Browser: Chrome / QtWeb
Inputmask version: Inputmask 5.0.6-beta.15

@subarachnid ,
wkhtmltopdf  uses a very old version of webkit (qtweb) which probaly does not support the features used.
This should be solable by using polyfills.
I will need to install the qtweb browser to see what is missing.

@RobinHerbots I think you are right, but the strange thing is that QtWeb does not show any JavaScript errors in its developer console. The one thing that I know of is that Function.prototype.bind is not availiable in QtWeb but even when I add a polyfill for that the problem still occurs.

Same problem here! Is there any workaround? does downgrading inputmask version work?

Thanks for the investigation and dedication!! I will definitely have a look at it and incorporate a fix in the next release.
Best regards
Robin

@subarachnid ,
Can you have a try with the latest beta.
Thx,
Robin

Hi @RobinHerbots, thanks so much for fixing the issue. I am 100% confident that it'll work now that you refactored to code to use an array.
Unfortunately QtWeb now shows a new error regarding some regex in the dev-console when using the lastest beta:
var a = e.match(new RegExp("(?<p1>.)\\[(?<p2>[^\\]]*)\\]", "g"));

Could you maybe have another look at this?

#2619

@subarachnid ,
I removed the named capture group from the regex,.  Hopefully this will work in qtweb.


Support for multiple OAuth 2.0 Authorization Servers

Hi again,
Since the last issue I created, I've successfully integrated Microsoft and Google together as OpenID Providers for general login to my app. Now I would like to protect my app's REST API with OAuth 2.0. Individually, I've been able to implement remote and local validation of access tokens with Google and just local validation of Microsoft tokens. However, I cannot implement both Google and Microsoft together.
I'm wondering if there's some clever configuration I can do in my .conf file or if I'm only limited to one OAuth validator.
Here's an example setup that I've tried to give an idea of what I mean by messing with the .conf file.
SetEnvIf Provider "microsoft" op=microsoft
SetEnvIf Provider "google" op=google

<If "env('op') == 'microsoft'">
    OIDCOAuthVerifyJwksUri https://login.microsoftonline.com/common/discovery/v2.0/keys
</If>
<If "env('op') == 'google'">
    OIDCOAuthClientID ${client_id}
    OIDCOAuthClientSecret ${client_secret}
    OIDCOAuthIntrospectionEndpoint https://www.googleapis.com/oauth2/v1/tokeninfo
    OIDCOAuthIntrospectionTokenParamName access_token
</If>

<Location /REST/1.0>
    Authtype oauth20
    <If "env('op') == 'microsoft'">
        Require claim aud:${microsoft_client_id}
    </If>
    <If "env('op') == 'google'">
        Require claim issued_to:${client_id}
    </If>
</Location>

That config doesn't work because it seems I can't put the OIDCOAuth* directives in an If statement though.
I'm not very familiar with Apache configuration, so if there's some way of implementing it, I'd appreciate it greatly.
Thanks,
Anthony

It is not possible to configure multiple Authorization Servers simultaneously.
I would also argue that it is a-typical to have a resource that owned by 2 Resource Owners at the same time.

Alright, fair enough. In that case, I'll just stick to Google for validation.


How to test the nslookup script?

Here's the content of server.txt file under /root/python folder i put in with the nslookup script.
google.com
bbc.co.uk
Is this correct?
If so, the script didn't work.
I tested this script on python 2.6.6 and this error appeared.
Traceback (most recent call last):
File "nslookup_check.py", line 4, in
subprocess.Popen(('nslookup '+server))
File "/usr/lib64/python2.6/subprocess.py", line 642, in init
errread, errwrite)
File "/usr/lib64/python2.6/subprocess.py", line 1238, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
In case of python 3.4
Traceback (most recent call last):
File "nslookup_check.py", line 4, in
subprocess.Popen(('nslookup '+server))
File "/opt/rh/rh-python34/root/usr/lib64/python3.4/subprocess.py", line 858, in init
restore_signals, start_new_session)
File "/opt/rh/rh-python34/root/usr/lib64/python3.4/subprocess.py", line 1456, in _execute_child
raise child_exception_type(errno_num, err_msg)
FileNotFoundError: [Errno 2] No such file or directory: 'nslookup google.com\n'
.

@thinhduckhoi
I too got the same error. This is caused due to
subprocess.Popen(('nslookup '+server))

According to Popen Class, First argument must be sequence, here it is string
It must be
subprocess.Popen(['nslookup', server])

For more information, go through the documentation - https://docs.python.org/2/library/subprocess.html
I have pull request #48  with required changes. Please let me know, If it solves your error.

@munendrasn Thanks you! It worked!


Update README.md to reflect the addition of PriorityClass

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Updated the image on README.md to include PriorityClass, as well as added PriorityClass as a resource that we can simulate.
Which issue(s) this PR fixes:
This PR is related to #27
Special notes for your reviewer:
If there are other changes that I need to make, please tell me and I'll do them.
/label tide/merge-method-squash

/assign @alculquicondor

This PR should be merged after #53.
/hold

Ok, I have closed it for now


sharebot crash

when i type thread amount it simply crashes

can you say more ?

already fixed, please update line of code 115 i guess and change view to share

fixed


Rearrange GPFS docs

ansible/roles/gpfs/README.md now only contains information directly related to the gpfs role, all other information is included in the more general document /docs/storage/gpfs.md

👍


Gastby: prevent layout components from unmounting

https://www.gatsbyjs.com/docs/layout-components/#how-to-prevent-layout-components-from-unmounting

Moved b/c originally opened in wrong repo.


General: Building instructions

Feature Description
General building instructions for Greengrass Nucleus is missing.
Use Case
Building the project manually on my own device would be helpful for understanding the code better.

Have a look at https://github.com/aws-greengrass/aws-greengrass-nucleus/blob/main/DEVELOPING.md. To build, simply run mvn -ntp package -DskipTests. This will build the zip file while skipping validations.

Closing due to inactivity. Please reopen if you still have questions.


Cannot patch new app inside existing tenant

ISSUE TYPE

Bug Report

AS3 BUILD/ VERSION
3.10 build 5

BIGIP VERSION
<IP_ADDRESS>
bigiq 6.1.0

SUMMARY

I'm trying to use the patch method with bigiq to add another app to an existing tenant already containing a previously deployed generic template app.

I get this error when sending the PATCH: InvalidPatchOperationError: path does not exist /tenant_1/myapp2

STEPS TO REPRODUCE

PATCH json looks like this :

{
  "class": "AS3",
  "action": "patch",
  "target": {
    "hostname": "bigip1"
  },
  "patchBody": [
    {
      "path": "/tenant_1/myapp2",
      "op": "add",
      "value": {
        "class": "Application",
        "template": "generic",
        "statsProfile": {
          "class": "Analytics_Profile",
          "collectedStatsInternalLogging": true,
          "collectedStatsExternalLogging": false,
          "capturedTrafficInternalLogging": false,
          "capturedTrafficExternalLogging": false,
          "collectPageLoadTime": true,
          "collectMaxTpsAndThroughput": true,
          "collectClientSideStatistics": true,
          "collectResponseCode": true,
          "collectMethod": true,
          "collectOsAndBrowser": true,
          "collectUrl": true,
          "collectUserAgent": true,
          "collectUserSession": true,
          "sessionCookieSecurity": "ssl-only"
        },
        "vs_myapp2": {
          "class": "Service_HTTP",
          "virtualAddresses": [
            "<IP_ADDRESS>"
          ],
          "snat": "auto",
          "pool": "pool_myapp2"
        },
        "pool_myapp2": {
          "class": "Pool",
          "monitors": [
            "http"
          ],
          "members": [
            {
              "servicePort": 8090,
              "serverAddresses": [
                "<IP_ADDRESS>",
                "<IP_ADDRESS>",
                "<IP_ADDRESS>"
              ]
            }
          ]
        }
      }
    }
  ]
}

GET of existing app in the tenant :

{
    "class": "ADC",
    "schemaVersion": "3.5.0",
    "id": "id_tenant_1_init",
    "target": {
        "address": "<IP_ADDRESS>"
    },
    "tenant_1": {
        "class": "Tenant",
        "defaultRouteDomain": 0,
        "init_app": {
            "class": "Application",
            "template": "generic",
            "vs_init_app": {
                "class": "Service_HTTP",
                "virtualAddresses": [
                    "<IP_ADDRESS>"
                ],
                "virtualPort": 1024,
                "persistenceMethods": [
                    "cookie"
                ],
                "profileHTTP": "basic",
                "layer4": "tcp",
                "profileTCP": "normal",
                "enable": true,
                "maxConnections": 0,
                "snat": "auto",
                "addressStatus": true,
                "mirroring": "none",
                "lastHop": "default",
                "translateClientPort": false,
                "translateServerAddress": true,
                "translateServerPort": true
            },
            "schemaOverlay": "default",
            "enable": true
        },
        "enable": true
    },
    "updateMode": "selective"
}

EXPECTED RESULTS

I want 2 VS in one partition

ACTUAL RESULTS

Error, not working

It works when using the tenant name in the declare url :
/mgmt/shared/appsvcs/declare/tenant_1?async=true
Otherwise, I think it's trying to find the first tenant already configured.
If I do a GET without specifying the tenant /mgmt/shared/appsvcs/declare, it returns the first tenant that I  created via bigiq before tenant_1.

Actually, it only works for one PATCH to add an app.
If I try to patch again to add one more app I get this error :
"response": "customRule111.call(...).then is not a function",
"status": 500
Content of the second patch json :
{
"class": "AS3",
"action": "patch",
"target": {
"hostname": "bigip1"
},
"patchBody": [
{
"path": "/tenant_1/myapp3",
"op": "add",
"value": {
"class": "Application",
"template": "generic",
"statsProfile": {
"class": "Analytics_Profile",
"collectedStatsInternalLogging": true,
"collectedStatsExternalLogging": false,
"capturedTrafficInternalLogging": false,
"capturedTrafficExternalLogging": false,
"collectPageLoadTime": true,
"collectMaxTpsAndThroughput": true,
"collectClientSideStatistics": true,
"collectResponseCode": true,
"collectMethod": true,
"collectOsAndBrowser": true,
"collectUrl": true,
"collectUserAgent": true,
"collectUserSession": true,
"sessionCookieSecurity": "ssl-only"
},
"vs_myapp3": {
"class": "Service_HTTP",
"virtualAddresses": [
"<IP_ADDRESS>"
],
"snat": "auto",
"profileAnalytics": {
"use": "statsProfile"
},
"pool": "pool_myapp3"
},
"pool_myapp3": {
"class": "Pool",
"monitors": [
"http"
],
"members": [
{
"servicePort": 8090,
"serverAddresses": [
"<IP_ADDRESS>"
]
}
]
}
}
}
]
}

I've done more research and it looks like it's when I use service discovery with Azure.
I can only add one appsvc with a service discovery pool inside the existing tenant. After that I cannot any other app (with or without service discovery) inside the tenant with the patch method.
I get a "customRule111.call(...).then is not a function" from the bigiq.
There is no problem without service discovery, I can patch multiple times to add VS in the same tenant.

Currently reviewing patch action under AT-235 internally. We will review if this solves the Service Discovery issue and follow up.

It looks like AT-235 was completed some time ago. Closing this issue. If you are till experiencing a problem, please create a new GitHub issue.


Bug in documentation for ModelSelect2TagWidget

There is a bug in documentation for ModelSelect2TagWidget. It should be     cleaned_values = list(pks) instead of      cleaned_values = list(values)

Hi @pawnhearts,
Thank you for reaching out. Excellent documentation is key for an open-source project. I sincerely appreciate your contribution here. That being said, I am not 100% sure I know where that line is? I'd welcome you to open a PR that changes the lines in question. I will make sure to review it quickly and deploy a new documentation.
Thanks!
Joe

For some reason I can't find that part of documentation in sources. Here is the link https://django-select2.readthedocs.io/en/latest/django_select2.html#django_select2.forms.ModelSelect2TagWidget 4th line from bottom in the example. It should be cleaned_values = list(pks). Otherwise it creates 2 duplicate tags and then fails on optgroups functions.
I was very confused until realized logic in that example is wrong.

Hi @pawnhearts, the documentation is built form Python doc-strings: https://github.com/codingjoe/django-select2/blob/4770f069e5dd6137f725f4e16095cfd4cf9d78b6/django_select2/forms.py#L583-L608

Resolved in #177
Thanks, @pawnhearts, I really appreciate the help!


Repositories must convert arrays to lists or warn if user passes array

Stashing this until it actually comes up. Need feedback from users on best approach.


RuntimeError: Unable to create attribute (Object header message is too large)

Running the files unmodified in Python 3.6.4 on windows I get the following trace:
Traceback (most recent call last):
  File "lstm.py", line 122, in <module>
    train_network()
  File "lstm.py", line 26, in train_network
    train(model, network_input, network_output)
  File "lstm.py", line 119, in train
    model.fit(network_input, network_output, epochs=200, batch_size=64, callbacks=callbacks_list, include_optimizer=False)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\engine\training.py", line 1035, in fit
    validation_steps=validation_steps)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\engine\training_arrays.py", line 217, in fit_loop
    callbacks.on_epoch_end(epoch, epoch_logs)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\callbacks.py", line 79, in on_epoch_end
    callback.on_epoch_end(epoch, logs)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\callbacks.py", line 446, in on_epoch_end
    self.model.save(filepath, overwrite=True)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\engine\network.py", line 1081, in save
    save_model(self, filepath, overwrite, include_optimizer)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\engine\saving.py", line 381, in save_model
    _serialize_model(model, f, include_optimizer)
  File "D:\Programs\Python\Python36\lib\site-packages\keras\engine\saving.py", line 113, in _serialize_model
    layer_group[name] = val
  File "D:\Programs\Python\Python36\lib\site-packages\keras\utils\io_utils.py", line 256, in __setitem__
    self.data.attrs[attr] = val
  File "h5py\_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py\_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "D:\Programs\Python\Python36\lib\site-packages\h5py\_hl\attrs.py", line 95, in __setitem__
    self.create(name, data=value, dtype=base.guess_dtype(value))
  File "D:\Programs\Python\Python36\lib\site-packages\h5py\_hl\attrs.py", line 188, in create
    attr = h5a.create(self._id, self._e(tempname), htype, space)
  File "h5py\_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py\_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py\h5a.pyx", line 47, in h5py.h5a.create
RuntimeError: Unable to create attribute (object header message is too large)

running pip3 show for the libraries:
Name: music21
Version: 5.3.0
---
Name: Keras
Version: 2.2.2
---
Name: tensorflow-gpu
Version: 1.10.0
---
Name: h5py
Version: 2.8.0

Can you tell me what versions you are using? Seems to be a known problem with Keras per: https://github.com/keras-team/keras/issues/6766 but I assume there's a version that works. Thanks!

###USE get_weights AND set_weights TO SAVE AND LOAD MODEL, RESPECTIVELY.
#Assuming that this is your model architecture. However, you may use
#whatever architecture, you want to (big or small; any).
def mymodel():
inputShape= (28, 28, 3);
model= Sequential()
model.add(Conv2D(20, 5, padding="same", input_shape=inputShape))
model.add(Activation('relu'))
model.add(Flatten())
model.add(Dense(500))
model.add(Activation('relu'))
model.add(Dense(2, activation= "softmax"))
return model
model.fit(....)    #paramaters to start training your model
################################################################################
################################################################################
#once your model has been trained, you want to save your model in your PC
#use get_weights() command to get your model weights
weigh= model.get_weights()
#now, use pickle to save your model weights, instead of .h5
#for heavy model architectures, .h5 file is unsupported.
pklfile= "D:/modelweights.pkl"
try:
fpkl= open(pklfile, 'wb')    #Python 3
pickle.dump(weigh, fpkl, protocol= pickle.HIGHEST_PROTOCOL)
fpkl.close()
except:
fpkl= open(pklfile, 'w')    #Python 2
pickle.dump(weigh, fpkl, protocol= pickle.HIGHEST_PROTOCOL)
fpkl.close()
################################################################################
################################################################################
#in future, you may want to load your model back
#use pickle to load model weights
pklfile= "D:/modelweights.pkl"
try:
f= open(pklfile)     #Python 2
weigh= pickle.load(f);
f.close();

except:
f= open(pklfile, 'rb')     #Python 3
weigh= pickle.load(f);
f.close();

restoredmodel= mymodel()
#use set_weights to load the modelweights into the model architecture
restoredmodel.set_weights(weigh)
################################################################################
################################################################################
#now, you can do your testing and evaluation- predictions
y_pred= restoredmodel.predict(X)

open this drive link and read the same above code properly:
https://drive.google.com/open?id=1xzrqP7ExTmJiZqVt0A_G6AT69EbIjEI9tUDLD1twqj8


Icon paths not being correctly handled between environments

Describe the bug
The plugin is not consistent accross environments, it works "fine" (not 100% fine since the GraphQL icon is broken) in local, but not in other environments.
My local environment is Windows, whereas others are Linux. That might be the problem since paths separators change (\ in windows vs / in linux)?
Local display (windows)

Staging display (Linux)

The graphql entry has the following project config file
currLabel: GraphQL
customIcon: null
enabled: true
handle: graphql
icon<EMAIL_ADDRESS>layout: b54f5a17-6a98-4f20-9ca7-195c37575028 # Default
level: 1
newWindow: false
parent: null
prevLabel: GraphQL
prevLevel: 1
prevParent: null
prevUrl: graphql
sortOrder: 18
subnavBehaviour: null
type: craft
url: graphql

The other broken entries have the following project config file
currLabel: Commerce
customIcon: null
enabled: true
handle: commerce
icon: 'C:\path\to\project\vendor\craftcms\commerce\src\icon-mask.svg'
layout: b54f5a17-6a98-4f20-9ca7-195c37575028 # Default
level: 1
newWindow: false
parent: null
prevLabel: Commerce
prevLevel: 1
prevParent: null
prevUrl: commerce
sortOrder: 5
subnavBehaviour: null
type: plugin
url: commerce

And this is how the HTML looks like in other environments

Steps to reproduce

Enable the plugin and leave all values by default

Craft CMS version
<IP_ADDRESS>
Plugin version
4.0.9
Multi-site?
No
Additional context
No response

I can confirm this is due to the path separator differences between Windows and Linux. After manually changing the project config files to be "linux-style" everything is working in the other environments (but it's broken in my environment).
The GraphQL icon being broken in Windows is due to the same reason (@appicons/graphql.svg is no correctly handled because of the path separator being considered "linux-style").
I've fixed it by replacing every DIRECTORY_SEPARATOR occurrency to '/' in the cp-nav/src/models/Navigation.php file and manually changing the project config paths to be linux-style. Appart from that, the project config build has to be changed to build icon paths always in linux-style (I don't know where that happens in the plugin code).

Interesting! I would've totally thought DIRECTORY_SEPARATOR be environment-aware, which is the reason I've always used it. I'll do some testing on Windows to ensure things are properly handled.

Fixed for the next release. To get this fix early, change your verbb/cp-nav requirement in composer.json to:
"require": {
  "verbb/cp-nav": "dev-craft-4 as 4.0.9",
  "...": "..."
}

Then run composer update.

I confirm this is fixed now, thanks!

Fixed in 4.0.10


When page reloads, API is called for input widgets even if not required, which resets data

hello,
The server side filter does not maintain data between pages. Once the we move from one page to another and come back, the selected value is lost.

upgrade to 0.4.11


Scoping secrets to branch/workflow

Currently Secrets defined in a GitHub repo have "repo" level scope. Anyone with write access to a repository can read and use secrets.
However various CD scenarios need to segregate the secrets created for development/Production purposes. Hence scoping of secrets per branch/workflow is a much needed feature.
More about GitHub secrets: https://help.github.com/en/articles/virtual-environments-for-github-actions#creating-and-using-secrets-encrypted-variables


Check out DCompute for GPU integrations

See this post - thanks @kirillseva !

👍

Julia also does something fun with GPUs. Since the language is all JIT'ed, they can just use opencl/cude compiler just in fly.

@kirillseva that's pretty amazing! Will take a look at how they do that.


fix: cloudaccount share_mode not initialized

这个 PR 实现什么功能/修复什么问题:
修正：云账号创建未初始化share_mode字段

是否需要 backport 到之前的 release 分支:
NONE

/area region
/cc @zexi

Codecov Report

Merging #6499 into master will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #6499   +/-   ##
=======================================
  Coverage    7.17%    7.17%
=======================================
  Files         646      646
  Lines      108928   108933    +5
=======================================
+ Hits         7817     7819    +2
- Misses     100510   100514    +4
+ Partials      601      600    -1

Flag
Coverage Δ

#aFlag
7.17% <0.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/cloudaccounts.go
0.69% <0.00%> (-0.01%)
:arrow_down:

pkg/appsrv/response.go
69.11% <0.00%> (+2.94%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix: fail to parse ts code

Codecov Report

Merging #839 (6b080fd) into release/1.0.0 (d4ace51) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release/1.0.0     #839   +/-   ##
==============================================
  Coverage          57.14%   57.14%
==============================================
  Files                  1        1
  Lines                 70       70
  Branches              18       18
==============================================
  Hits                  40       40
  Misses                30       30

Continue to review full report at Codecov.

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


Vulnerability roundup 98: kitty-0.18.3: 1 advisory [9.8]

search, files

[ ] CVE-2020-35605 CVSSv3=9.8 (nixos-20.09)

Scanned versions: nixos-20.09: aa5b9cd16b9.
Cc @Luflosi
Cc @rvolosatovs
Cc @tex

Resolved in #107662 I believe.

Resolved in #107662 I believe.


Update functions.go

to fix int64

thanks @linfangrong  -- have you signed the CLA?

@corylanou

+1, waiting on CLA confirmation.  Should also add change log.

@otoolep yeah, I just signed the CLA


Added more details for noobs like me

Added headings. Added details about setting up Firebase, Rdio js API, and Django secret. Added settings.py changes to get local setup properly.

Thanks for the PR Mike!


Lower right Tweet button/drawer settings no longer working.

Maybe it's just me but something happened today where I cannot hide the lower right Tweet button, even when that setting is on. I've reloaded the extension and toggled the setting but the button will not disappear as it has always done before. Also, the lower right drawer setting to show/hide is not working as well.

Yes, this is a recent change by Twitter. Will address this when I have time, otherwise PR help would be appreciated :)

https://github.com/thomaswang/minimal-twitter/pull/53

Still not working for me. Maybe Twitter changed something again

@D3SOX should be fixed now! https://github.com/thomaswang/minimal-twitter/commit/04912e1c0750e10d075f745a3168f84db8b572b8

I can confirm that it works with the new version. Thank you!


Ledger Nano S stuck on "Open Application" after upgrading

Hi - thank you for this great app
I had v1.0.0 and upgraded to the latest stable 1.02 - the ledger device is stuck on "Open application" (after approving the not genuine app signature)
I've tried uninstalling and re-installing and it yields the same behaviour.
Uninstalling and re-installing v1.0.0 works so it seems that there's a problem with v1.0.2 on the latest ledger nano S
Nano S version is 1.6.1

I can confirm the exact same problem on v1.0.1 and v1.0.2 with the Nano S running v1.6.1.

Please try to update the Ledger to FW 2.0.0

Oops, that was it. Ledger Manager was telling me my Nano was up to date, but only because I was running an older version of Ledger Live. Updating Ledger Live followed by an update to firmware 2.0 resolved the issue, although though I have the same problem as issue #18


[FSDE-20776] Fix sublevel row selection when selectionDisabled is set

The CI pipeline has run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-396/1/. :white_check_mark:

You can check this PRs instance in https://1c06d799aaccece634a81e7aa868f35a38d43b09-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-396-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline has run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-396/4/. :white_check_mark:

You can check this PRs instance in http://fe694db339053abba909de1c3aef887eb4987d71-chi-pr-po-ctl.s3-website-us-east-1.amazonaws.com (external) and in https://nginx-pr-396-ux-chi.kubeodc-test.corp.intranet/ (internal)


Host to correspondances

Hello
Could you add Host directive to correspondences?
Thanks in advance.

Hello @RomanNagornyi,
This has been added in the #9 pull-request, released in the v1.5.0 release.
Thank you for taking time to make this issue ❤

Hello @WilliamDASILVA
Thanks for module ❤


Does SSH task treat exit code as task failure?

The docs speak about stderr but is exit code treated as an error? This is unclear.

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

ID: 8ad03c70-4014-135f-ffd3-245c08b3ce32
Version Independent ID: 42a84b54-9ab7-89b2-00b1-efcfc6ba6eef
Content: SSH Deployment task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/ssh.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@sandersaares Thanks for your question. While we look into changes for the article, consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


go-zero(http优雅关闭问题- proc.AddWrapUpListener)---Goversion 1.17

@kevwan 请问官方这个errwe为nil设计打印是否正确？为啥如此设计？

Yes, It should be , you can make a pr
if e := server.Shutdown(context.Background()); e != nil {
    logx.Error(e)
}

Yes, It should be ：
if e := server.Shutdown(context.Background()); e != nil {
    logx.Error(e)
}

you can make a pr

that is exactly what i thought of!

Yes, It should be ：
if e := server.Shutdown(context.Background()); e != nil {
    logx.Error(e)
}

you can make a pr

that is exactly what i thought of!

PR please!

Yes, It should be ：
if e := server.Shutdown(context.Background()); e != nil {
    logx.Error(e)
}

you can make a pr

that is exactly what i thought of!

PR please!

@kevwan  By the way, i have to add 5 seconds to stop gracefully!

Do you mean logs are lost?

when i have 5 seconds to stop,it will be ok .

by by contrast,i do not have 5 seconds to leave, main goroutine will go away immediately, i cannot release the sentry resource.

i suggest the signal goroutine should not be set in the child goroutine and putting addtional 5 seconds in the main goroutine.


docs: Rewrite help file

First of all, thanks for creating such a great plugin. The plugin always helps me.
Currently colorful-winsep.nvim has a vim help file, but it is too ugly and does not follow the way of vim help file.
I have rewritten the help file to follow the right way. I hope it will help other users when they try to read the document.

#44 correct direction_have sample

😭I'm sorry, I don't quite understand the git specification. I didn't mean to. I didn't know it was wrong until I asked my friends. I'm sorry.

Don't mind. Finally the changes have been included to the plugin. That's all I need.

Don't mind. Finally the changes have been included to the plugin.


Compiler.java:80-83: Implement compilation of high-level...

The puzzle 33-446355f9 from #33 has to be resolved:
https://github.com/objectionary/opeo-maven-plugin/blob/dd5a87dee07533b7fffbecd95738f40fc24011f3/src/main/java/org/eolang/opeo/Compiler.java#L80-L83
The puzzle was created by @volodya-lombrozo on 26-Dec-23.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd the puzzle #62 is still not solved.

@0pdd the puzzle #65 is still not solved; solved: #62.

@0pdd 5 puzzles #74, #75, #76, #77, #78 are still not solved; solved: #62, #65.

@0pdd 5 puzzles #74, #75, #76, #80, #81 are still not solved; solved: #62, #65, #77, #78.

@0pdd 6 puzzles #74, #75, #80, #81, #86, #87 are still not solved; solved: #62, #65, #76, #77, #78.

@0pdd 6 puzzles #74, #75, #80, #81, #87, #89 are still not solved; solved: #62, #65, #76, #77, #78, #86.

@0pdd 5 puzzles #74, #75, #80, #81, #87 are still not solved; solved: #62, #65, #76, #77, #78, #86, #89.

@0pdd 6 puzzles #74, #75, #81, #87, #97, #98 are still not solved; solved: #62, #65, #76, #77, #78, #80, #86, #89.

@0pdd 5 puzzles #74, #75, #81, #87, #97 are still not solved; solved: #62, #65, #76, #77, #78, #80, #86, #89, #98.

@0pdd 5 puzzles #134, #74, #75, #87, #97 are still not solved; solved: #62, #65, #76, #77, #78, #80, #81, #86, #89, #98.

@0pdd 5 puzzles #134, #153, #74, #75, #87 are still not solved; solved: #62, #65, #76, #77, #78, #80, #81, #86, #89, #97, #98.

@0pdd 4 puzzles #134, #74, #75, #87 are still not solved; solved: #153, #62, #65, #76, #77, #78, #80, #81, #86, #89, #97, #98.

@0pdd 3 puzzles #134, #74, #75 are still not solved; solved: #153, #62, #65, #76, #77, #78, #80, #81, #86, #87, #89, #97, #98.

@0pdd 2 puzzles #74, #75 are still not solved; solved: #134, #153, #62, #65, #76, #77, #78, #80, #81, #86, #87, #89, #97, #98.


Should private members be inaccessible from objects of the same type?

Should a class factory not be able to access the object it creates's private methods?  Should a comparison function not be able to access an object of the same type's private members?
from typing import Self, Any

class X:
    _x: int

    def _f(self) -> None:
        ...

    @classmethod
    def factory(cls, x: int) -> Self:
        retval = cls()
        retval._f()  # SLF001 Private member accessed
        return retval

    def __eq__(self, other: Any) -> bool:
        if not isinstance(other, X):
            return False
        return other._x == self._x  # SLF001 Private member accessed

Yes these are false positives. The problem with this is that ruff isn't a static type checker. Static type checkers understand which type cls() returns and they support type narrowing i.e. they understand that the type of other is X after the above isinstance check ... ruff doesn't.
And while we could easily implement some logic to handle the exact cases of your example that logic would fail if e.g. retval was reassigned or the isinstance check was more complicated ... properly handling this requires proper static type checking.

Yes, I considered that it might be difficult.  Shall I close this issue then?

Yeah probably hard to resolve right now. Eventually, I hope!


Added the !path tag.

Closes #1316
Added a short tag on what the PATH is, uses for it and resources on how to modify it.

Was this approved by a core developer?

@Kronifer are you planning on finishing this PR?

No, as im not sure this was explicitly approved.


Better feedback when addon startup fails/times out

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
Test issues reported in https://github.com/Kong/kubernetes-ingress-controller/issues/2574 have been caused by knative failing to start in KTF.
This required @rainest to go to GKE and see the in-cluster logs to investigate the failure. There was no way to infer that knative (or any particular KTF addon) was responsible for the failure.
Proposed Solution
Ensure that feedback on addons startup progress (or lack thereof) is visible in GH Actions logs when KTF is run on CI.
Alternatively: maybe it's time to look at the problem of debugging KTF issues more holistically. Consider adding a mechanism that allows the user to navigate pod logs and other possible sources of failure info.
Additional information
No response
Acceptance Criteria

[ ] https://github.com/Kong/kubernetes-ingress-controller/issues/2574, if reappeared, would be easier to investigate from GH Actions logs.

Closing this one in favor of https://github.com/Kong/kubernetes-testing-framework/issues/306 which is a comprehensive diagnostic tool for KTF environments which will capture these kinds of problems.


:package: Deliver as Jitpack :sloth:

:grey_question: About
Currently, the setup is pretty complex for end-users :

Wihtout having to push to maven central repos and with minimal effort, this could be enhanced with jitpack :

:dart: Action

[ ] Consider document it this way
[ ] Add the markdown badge so people are aware it is available as a jitpack dependency

Thanks for pointing out @adriens! I've update the instructions in readme.
Cheers!


Permissions usecases

Raised for discussion only.
Tests demonstrating various permissions use cases and their caveats:

Permission to request RPC from any service in the cluster
Permission to request RPC from a specific service (not currently possible)
Permission to listen to events from any service in the cluster
Permission to listen to events from a specific service
Permission to dispatch events into the cluster

This was useful for discussion but is not helpful as a PR, so closing.


Create installer package

It would be nice if there was an installer packages, that would create shortcuts in start menu and desktop, put files in appropriate places, detect ffmpeg binaries, and so on.
Edit: (Open for suggestions)
The solution chosed must integrate well with MSBuild process, preferrably as a NuGet package.
ToDo

Install Location (Default to Program files).
Start Menu shortcut
Desktop menu
Optionally Download FFmpeg
Launch after Install
Updating

I'm not familiar with any Installer maker software. Would need to look for a good one and get familiar. So, I will make an installer, but it might take time.
BTW, there is a Chocolatey package. It does not do much, just makes it easier to install/update.
choco install captura -y

Also, you are free to contribute if you wish.

@MathewSachin I recommend Advanced Installer for Visual Studio Projects ;)

Thank you @xxczaki, I will look into it. 👍

This seems to be a very interesting project that might fit captura.

Squirrel.Windows seems really amazing 👍. I just need to figure out how to keep the chocolatey package in sync with automatic updating.

Squirrel installer in master branch. 🎉


Documentation changes v1.4

Pull Request Test Coverage Report for Build 660

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

Totals

Change from base Build 659:
0.0%

Covered Lines:
2597

Relevant Lines:
3461

💛  - Coveralls


More generic encoding for complex return value

In the db_next import, we currently use a custom encoding to return two values (key and value). This is okay for now but not the nices thing you can imagine when developing a rich API via imports.
I tried to use Wasm multiple return values, which is more or less this diff:
diff --git a/README.md b/README.md
index 7653e169..cc3d4985 100644
--- a/README.md
+++ b/README.md
@@ -136,7 +136,7 @@ extern "C" {
     #[cfg(feature = "iterator")]
     fn db_scan(start: u32, end: u32, order: i32) -> u32;
     #[cfg(feature = "iterator")]
-    fn db_next(iterator_id: u32) -> u32;
+    fn db_next(iterator_id: u32) -> [u32; 2];

     fn canonicalize_address(source: u32, destination: u32) -> u32;
     fn humanize_address(source: u32, destination: u32) -> u32;
diff --git a/packages/std/src/imports.rs b/packages/std/src/imports.rs
index 80280311..69ff75d1 100644
--- a/packages/std/src/imports.rs
+++ b/packages/std/src/imports.rs
@@ -30,7 +30,7 @@ extern "C" {
     #[cfg(feature = "iterator")]
     fn db_scan(start_ptr: u32, end_ptr: u32, order: i32) -> u32;
     #[cfg(feature = "iterator")]
-    fn db_next(iterator_id: u32) -> u32;
+    fn db_next(iterator_id: u32) -> [u32; 2];

     fn canonicalize_address(source_ptr: u32, destination_ptr: u32) -> u32;
     fn humanize_address(source_ptr: u32, destination_ptr: u32) -> u32;
@@ -115,23 +115,14 @@ impl Iterator for ExternalIterator {

     fn next(&mut self) -> Option<Self::Item> {
         let next_result = unsafe { db_next(self.iterator_id) };
-        let kv_region_ptr = next_result as *mut Region;
-        let mut kv = unsafe { consume_region(kv_region_ptr) };
-
-        // The KV region uses the format value || key || keylen, where keylen is a fixed size big endian u32 value
-        let keylen = u32::from_be_bytes([
-            kv[kv.len() - 4],
-            kv[kv.len() - 3],
-            kv[kv.len() - 2],
-            kv[kv.len() - 1],
-        ]) as usize;
-        if keylen == 0 {
+        let key_ptr = next_result[0] as *mut Region;
+        let value_ptr = next_result[1] as *mut Region;
+        let key = unsafe { consume_region(key_ptr) };
+        let value = unsafe { consume_region(value_ptr) };
+
+        if key.is_empty() {
             return None;
         }
-
-        kv.truncate(kv.len() - 4);
-        let key = kv.split_off(kv.len() - keylen);
-        let value = kv;
         Some((key, value))
     }
 }
diff --git a/packages/vm/src/context.rs b/packages/vm/src/context.rs
index 0866a02a..c7c3ef16 100644
--- a/packages/vm/src/context.rs
+++ b/packages/vm/src/context.rs
@@ -314,7 +314,7 @@ mod test {
                 "db_write" => Func::new(|_a: u32, _b: u32| {}),
                 "db_remove" => Func::new(|_a: u32| {}),
                 "db_scan" => Func::new(|_a: u32, _b: u32, _c: i32| -> u32 { 0 }),
-                "db_next" => Func::new(|_a: u32| -> u32 { 0 }),
+                "db_next" => Func::new(|_a: u32| -> [u32; 2] { [0, 0] }),
                 "query_chain" => Func::new(|_a: u32| -> u32 { 0 }),
                 "canonicalize_address" => Func::new(|_a: u32, _b: u32| -> u32 { 0 }),
                 "humanize_address" => Func::new(|_a: u32, _b: u32| -> u32 { 0 }),
diff --git a/packages/vm/src/imports.rs b/packages/vm/src/imports.rs
index 4e90b997..4ad04f73 100644
--- a/packages/vm/src/imports.rs
+++ b/packages/vm/src/imports.rs
@@ -196,22 +196,16 @@ pub fn do_scan<S: Storage, Q: Querier>(
 }

 #[cfg(feature = "iterator")]
-pub fn do_next<S: Storage, Q: Querier>(ctx: &mut Ctx, iterator_id: u32) -> VmResult<u32> {
+pub fn do_next<S: Storage, Q: Querier>(ctx: &mut Ctx, iterator_id: u32) -> VmResult<[u32; 2]> {
     let (result, gas_info) =
         with_storage_from_context::<S, Q, _, _>(ctx, |store| Ok(store.next(iterator_id)))?;
     process_gas_info::<S, Q>(ctx, gas_info)?;

     // Empty key will later be treated as _no more element_.
     let (key, value) = result?.unwrap_or_else(|| (Vec::<u8>::new(), Vec::<u8>::new()));
-
-    // Build value || key || keylen
-    let keylen_bytes = to_u32(key.len())?.to_be_bytes();
-    let mut out_data = value;
-    out_data.reserve(key.len() + 4);
-    out_data.extend(key);
-    out_data.extend_from_slice(&keylen_bytes);
-
-    write_to_contract::<S, Q>(ctx, &out_data)
+    let key_ptr = write_to_contract::<S, Q>(ctx, &key)?;
+    let value_ptr = write_to_contract::<S, Q>(ctx, &value)?;
+    Ok([key_ptr, value_ptr])
 }

 #[cfg(test)]
@@ -262,7 +256,7 @@ mod test {
                 "db_write" => Func::new(|_a: u32, _b: u32| {}),
                 "db_remove" => Func::new(|_a: u32| {}),
                 "db_scan" => Func::new(|_a: u32, _b: u32, _c: i32| -> u32 { 0 }),
-                "db_next" => Func::new(|_a: u32| -> u32 { 0 }),
+                "db_next" => Func::new(|_a: u32| -> [u32; 2] { [0, 0] }),
                 "query_chain" => Func::new(|_a: u32| -> u32 { 0 }),
                 "canonicalize_address" => Func::new(|_a: i32, _b: i32| -> u32 { 0 }),
                 "humanize_address" => Func::new(|_a: i32, _b: i32| -> u32 { 0 }),
@@ -998,22 +992,18 @@ mod test {
         let id = do_scan::<MS, MQ>(ctx, 0, 0, Order::Ascending.into()).unwrap();

         // Entry 1
-        let kv_region_ptr = do_next::<MS, MQ>(ctx, id).unwrap();
-        assert_eq!(
-            force_read(ctx, kv_region_ptr),
-            [VALUE1, KEY1, b"\0\0\0\x03"].concat()
-        );
+        let (key_ptr, value_ptr) = do_next::<MS, MQ>(ctx, id).unwrap();
+        assert_eq!(force_read(ctx, key_ptr), KEY1);
+        assert_eq!(force_read(ctx, value_ptr), VALUE1);

         // Entry 2
-        let kv_region_ptr = do_next::<MS, MQ>(ctx, id).unwrap();
-        assert_eq!(
-            force_read(ctx, kv_region_ptr),
-            [VALUE2, KEY2, b"\0\0\0\x04"].concat()
-        );
+        let (key_ptr, value_ptr) = do_next::<MS, MQ>(ctx, id).unwrap();
+        assert_eq!(force_read(ctx, key_ptr), KEY2);
+        assert_eq!(force_read(ctx, value_ptr), VALUE2);

         // End
-        let kv_region_ptr = do_next::<MS, MQ>(ctx, id).unwrap();
-        assert_eq!(force_read(ctx, kv_region_ptr), b"\0\0\0\0");
+        let (key_ptr, _value_ptr) = do_next::<MS, MQ>(ctx, id).unwrap();
+        assert_eq!(force_read(ctx, key_ptr), Vec::<u8>::new());
         // API makes no guarantees for value_ptr in this case
     }

diff --git a/packages/vm/src/instance.rs b/packages/vm/src/instance.rs
index def786c2..1c1c67dd 100644
--- a/packages/vm/src/instance.rs
+++ b/packages/vm/src/instance.rs
@@ -148,11 +148,10 @@ where
                     do_scan::<S, Q>(ctx, start_ptr, end_ptr, order)
                 }),
                 // Get next element of iterator with ID `iterator_id`.
-                // Creates a region containing both key and value and returns its address.
-                // Ownership of the result region is transferred to the contract.
-                // The KV region uses the format value || key || keylen, where keylen is a fixed size big endian u32 value.
-                // An empty key (i.e. KV region ends with \0\0\0\0) means no more element, no matter what the value is.
-                "db_next" => Func::new(move |ctx: &mut Ctx, iterator_id: u32| -> VmResult<u32> {
+                // Creates a key region and a value region and returns their addresses.
+                // Ownership of the result regions is transferred to the contract.
+                // An empty key means no more element, no matter what the value is.
+                "db_next" => Func::new(move |ctx: &mut Ctx, iterator_id: u32| -> VmResult<[u32; 2]> {
                     do_next::<S, Q>(ctx, iterator_id)
                 }),
             },

Returning two pointers makes the code much nicer. However, this fails to link.
---- push_and_reduce stdout ----
thread 'push_and_reduce' panicked at 'called `Result::unwrap()` on an `Err` value: InstantiationErr { msg: "Error instantiating module: LinkError([IncorrectImportSignature { namespace: \"env\", name: \"db_next\", expected: FuncSig { params: [I32, I32], returns: [] }, found: FuncSig { params: [I32], returns: [I32, I32] } }])" }', /projects/cosmwasm/packages/vm/src/testing/instance.rs:129:77

As you can see in expected: FuncSig { params: [I32, I32], returns: [] }, this compiles to a Wasm function with 2 arguments and no return value. It is unclear if multiple return values in imports are supported by the Rust compiler (see e.g. https://github.com/rust-lang/rust/issues/47599).
If we are stuck with FFI types, we might want to use a more generic encoding schema than value || key || keylen, which at least supports more than two values.

Interesting investigation, but I don't think this is much of a pain point - it works, and I don't think we add any more complex FFI types soon.
Happy to revisit this if we have actual use cases that need larger return types (and maybe just leave the next iterator as is anyway to avoid breaking compatibility). But it would be cool to have a better struct if we need to return more.

I looked into Ethereum's RLP but this is too complicated because of the nesting which we do not need. A flat list of binary values is sufficient.

I now realized that the length as a suffix is only needed because we use consume_region which ready the whole region into one Vector. It would be more efficient to make a consume_multi_region which returns a Vec<Vec<u8>>. This avoids all copies of the data as well as the strange length suffix.
So instead of value || key || keylen we do value0_len || value0 || value1_len || value1 || value2_len || value2 || … where length fields are always 4 byte uint32. Then in consume_multi_region we loop over the region, ignore (i.e. leak) the length fields and convert the values into Vectos.

Do not start this before #504 is merged. Otherwise big merge mess ahead.

This is API breaking for the Storage iterator, correct?
I think we should start labeling API Breaking changes, and start moving them to CWIPs. (Not saying we need the full process here, but let's start some process/custom for labeling these. I will make a few API breaking proposals soon as well)

It breaks the contract-VM interface (binary compatibility), not the standard librarie's API (source compatibility). I completely agree moving the result of this long thread into a CWIP.

Do you still want to do this? For me it would be good to get it in the next release as there are quite a few breaking changes already. Or decide not to do it at all.
It would be good to get these apis quite stable soon

Do you still want to do this? For me it would be good to get it in the next release as there are quite a few breaking changes already. Or decide not to do it at all.
It would be good to get these apis quite stable soon

Do you still want to do this?

I think it makes sense to work that out when starting with the crypto APIs, which will touch this whole contract-VM interface a lot.

Do you still want to do this?

I think it makes sense to work that out when starting with the crypto APIs, which will touch this whole contract-VM interface a lot.

Fair enough

Fair enough


Intel aero: config updates

parameter tuning from https://github.com/PX4/Firmware/issues/5955
make sure logging works

Nice, do you have a video / log?

No video, but logs are in #5955:
acro + manual: http://logs.px4.io/plot_app?log=6699704e-c84b-454a-8e0c-eb0b199c4e30
manual + position: http://logs.px4.io/plot_app?log=60252e13-2a75-4a18-aa2c-c6babe43d0ae

Woah, the thing has a big issue with the mag. @jgoppert That explains why LPE is struggling.

Have this magnetometer issue been resolved or improved on the released Intel Aero drone and with improvements in the firmware?
If so, which branch should we use when reflashing PX4 on the Intel Aero and what about the tuned parameters?


display_grade for All Souls

All Souls has a logical rank of 7 = Solemnity.
However, it is never displayed as a Solemnity.
It is called a Commemoration: Commemoration of the Faithful Departed*.
Should commemoration be a display_grade value? Or should it display as nothing?
Commemoration is the same word that is used to refer to Memorials that are reduced to optional memorials in partial form during days of Advent between Dec. 17 and Dec. 24, and during Lent.
But perhaps in the case of All Souls day, usage of the term Commemoration is just a case of homonymity?
English, Italian and Latin all work pretty much the same way in all these cases. However, I have learned that in the Netherlands, Dutch does not have a specific term for Commemoration and they just use optional memorial in the case of Memorials that are reduced in rank during Advent and Lent. However, All Souls should NOT be displayed as an optional memorial. Therefore I am led to think that in the case of All Souls, display_grade should indicate that it should not show anything at all.
But in order to indicate that nothing should be displayed we would probably use an empty value for display_grade.
And yet, we are currently using an empty value for display_grade on all liturgical events to indicate that the property is not needed / used.
In order to indicate that an empty value should actually show as an empty value, we should change all other empty display_grade values to a value of null.

And in case issue #253 is implemented, and All Souls should have an empty display_grade or grade_display, then it should also have a grade_abbr_display with an empty value.

This should have fixed #181 too.


Duplicate "Guide" link in footer

The footer bar contains the "Guide" link to http://babylonjsguide.github.io/ twice in a row.

Helo fabian and thnks for your feedback:) do you want to do a PR?


[Umbrella] Better Docs!

More Tutorials/Guides/Recipes

Integration guide for SQLAlchemy, perhaps also marshmallow?

https://github.com/wileykestner/falcon-sqlalchemy-demo
https://gitlab.com/skosh/falcon-helpers/blob/master/falcon_helpers/sqla/db.py
https://gitlab.com/skosh/falcon-helpers/blob/master/falcon_helpers/middlewares/sqla.py

Integration guide for 2-3 popular configuration frameworks
Integration guide for templating libraries?
Lifecycle management

imlementing close() when assigning to resp.stream

Fast prototyping / evaluation

Read-made docker containers, orchestration templates, and/or cloud images
Recipes (combine with above idea for containers/templates, and/or command-line skeleton app generator

Additional Deployment guides / production concerns (e.g., integration with prometheus)

https://github.com/jmvrbanac/falcon-example

Best practices summary for best performance
Better description of benchmarks (and better benchmarks to begin with)
Tutorial or guide for how to write Cython in your app to use with Falcon
Guide to using CFFI for native extensions in C/Rust
What else?

Maybe ready-made Docker containers and VM appliances to allow people evaluating / prototyping even quicker?

(See also: https://github.com/falconry/falcon/issues/1391)

Maybe ready-made Docker containers, orchestration templates, cloud images, VM appliances to allow people evaluating / prototyping even quicker?

Thanks! Added to the list.

@kgriffs I have been using Falcon at work in Production and love it! I was currently working on a library and stumbled upon the Issues page, and eventually here. Please let me know how I can contribute. Since the list is large, should we go ahead and start taski-fying these?

@bhargavrpatel Welcome! For now we are still brainstorming a bit while we finalize the 2.0 release and wait for development to open on the 3.0 branch. That being said, I think we can start breaking down these ideas into issues soon and sussing out what should go in 3.0 vs. 3.1.

As mentioned by @amitvadhel on Gitter, another popular way to quickstart development that people turn to is Cookiecutter [1].
There is a Falcon recipe to it [2], but it is fairly outdated, we could consider submitting a new one (idea falls under Fast prototyping).

https://github.com/audreyr/cookiecutter
https://github.com/dgarana/cookiecutter-falcon


Less code, more gradle

Utilizing testing tools provided by gradle plugin.

I've updated my CLA.
On Sat, Feb 7, 2015 at 6:44 PM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request.
It looks like this may be your first contribution to a Google open source
project, in which case you'll need to sign a Contributor License Agreement
(CLA) at https://cla.developers.google.com/.
If you've already signed a CLA, it's possible we don't have your GitHub
username or you're using a different email address. Check the information
on your CLA https://cla.developers.google.com/clas or see this help
article on setting the email on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
Once you've done that, please reply here to let us know. If you signed the
CLA as a corporation, please let us know the company's name.
—
Reply to this email directly or view it on GitHub
https://github.com/googlesamples/android-ActivityInstrumentation/pull/1#issuecomment-73368108
.

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


Generate Surface Styles and Generate State Layers not working

When updating a primary color, the Outlined buttons clearly show that the State Layers didn't update to the brown-ish

You seem to have a very old version of the UI kit. The Figma plugin no longer generates read-only style groups, which is what is failing to update in your file.
Instead of M3/read-only/light/, the new structure wants M3/state-layers/light/

You can double click on the color group in the Figma sidebar to rename it.
)

Got it, thank you for the info. The team has decided to switch to MUI, but regardless - thank you for this info


feat: added functions for property escaping

This is partial support for Avansai/properties-file#2
It follows Java implementation, therefore it escapes :=#! in values, even thou it is not needed.
If I've missed some case in the tests, let me know.

Thanks @mdvorak - do you work with @tappleby or do you have use cases for this?
While I agree it's a nice addition, this package was never really built with writing .properties file in mind - I'm not sure if adding this will be sufficient or if we should also include more details in the readme on how to leverage this?
My concern is the lack of clear use cases so far and because writing a simple properties file can do done relatively easily, but if you want to do things like:

Keep comments in the target file
Keep collisions
Keep line breaks
Etc.

Things can become complex quickly. I'm just wondering what would be the best way to offer a real solution for this. I personally don't have a lot of time to tackle this problem on my own right now.

I'll merge this for now and update the doc on potential uses - it might not be covert for all use cases but it's better than what we had! Thanks for the pull request @mdvorak.

If you are curious @mdvorak, I made some tweaks in this commit: https://github.com/Avansai/properties-file/commit/95379d81cf0db14f49ff1fc869378ca8b64619b1
Nothing major, but I decided to split the "escape/unescape functions" into their own files so that tree-shaking can optimize based on use cases.
Thanks again for your contribution and if you ever want to try implementing a full-fledged writer, PRs are always welcome!
Your changes are available in version 2.2.0


add selectionColor as a customizable prop.

Hi @pickeloe5  Thanks for submitting the PR. It had been merged to master and pending for the next release. We noticed that you forget to add the selectionColor props to the typescript definition file, index.d.ts, we had added it as a supplement and have the props added in the example code. Thanks a lot for your contribution.


Render Problem '@attr/colorShadowDark' in current theme.

Please Suggest me what to to.

Im facing same issue adding additional details, this occurs on latest android studio like dolphin.

Temporary fix is downgrating to ChipMunk, but for me i cant downgrade, and its basically studios mistake not owner of library

Hi man Just found some solution:
  <item name="colorShadowDark">@color/black</item>
    <item name="colorShadowLight">@color/white</item>

Addition to above Solve
Add Following color in your color.xml for getting same output in render without error
//Neumorph render issue
#B5FFFFFF
#D89A9A9A
Add following line in Theams


OSX system services shortcuts not  recognized

I'm really enjoying working with VSCode, and appreciate the work going on here.  One issue I'm having (OSX El Capitan) is that VSCode doesn't seem to recognize some of the system wide services keyboard shortcuts I've set up to work with text from any application.
I have some System Services that run scripts, which I activate via a mouse menu, and those work fine.  I have others that I activate via the keyboard. For example, I have a System Service to take selected text and use a shell script to append it to a special file.  I have a mouse menu item that invokes that service via the the keyboard shortcut I've defined:  Cmd-Shift-Alt-Ctrl-A.  This works just fine, but I cannot invoke the same service using the actual keys on the keyboard.
All the shortcuts I'm using have  Cmd-Shift-Alt-Ctrl as the modifiers, so I don't think there are any conflicts with existing keyboard shortcuts.  Other system wide keyboard shortcuts do work, so I'm not sure what the issue is with these particular, OSX System Services related ones.  Is it a problem with the modifiers, or is there some issue currently with OSX system services support?

@unnamedcrewman out of curiosity, does it work if you bind a simpler shortcut that is not yet taken by VS Code?
@alexandrudima fyi

Thanks for the quick response.
I rebound the shortcut to ctrl+cmd+H which is not listed, as far as I can tell, among the default keyboard shortcuts.  My "append selected text to file" script/Service works fine with these keys in Sublime Text, as well as text highlighted in the browser and so on.  However this key-binding still fails to work from VSCode.  I tried several other key binding combos that seemed unique, and all worked except in VSCode.
If it matters, the mouse shortcut I use is simply a PopClip for Mac plugin that runs the system service by name, rather than by invoking the keyboard shortcut.  The PopClip plugin works perfectly, which seems to suggest that the issue is not with the service itself interacting with VSCode, but rather something with the key-binding mechanism instead.

This is most likely caused by us keeping the input textarea empty. I can only imagine that this external tool works by finding the current focused UI element and extracting the selected text from it. So it is more likely that the shortcuts do work, its just that the external tool finds no selected text and there is no visible output in the file.

To investigate if a1bd50f63fd8185a13495ff3d4c67396109e4e71 fixes this too

This is a feature request which does not even appear to work in chrome and sublime text.
Moving it to the backlog

I can see how this is not really a bug, but I also think there is something here that is a bit more than a simple feature request because the conflict seems to be with some built in OSX functionality.
I wanted to clarify a couple of things in case there has been a bit of misunderstanding
@isidorn
I am using features built in to OSX: A system-wide keyboard shortcut (set through a built in OSX mechanism) that invokes an OSX System Service (append selected text to a specific file). The system-wide keyboard shortcut actually does invoke the system service correctly everywhere in the OS.  It works correctly on selected text in Chrome.  It works correctly on selected text in Sublime Text.  It works in every program I use, except for VSCode.
@alexandrudima
I'm not sure what you mean by "external tool", and so I apologize for any confusion.  As I mention above, the issue I'm having is with VSCode not recognizing  system-wide shortcuts, set through a built in OSX mechanism.  I've tried multiple key combos, and none of them work properly in VSCode, even thought they work everywhere else across the OS.  I don't think that the issue is the System Service itself not finding any selected text, because if I actually do use an external tool (instead of the system keyboard shortcut) to invoke the target service, it works perfectly---selected text in VSCode is recognized and appended to the desired file.  In other words, the actual system service can identify and can work with selected text in VSCode.  The problem is that VSCode does not appear to be registering the system-wide keyboard shortcut in OSX that normally invokes the service in every other program.
I have searched the VSCode key-binding file, and used several shortcuts that I could't find there ( e.g. ctrl+cmd+H), and none of them allow me to invoke the system service in VSCode, even though they work in every other program. It's possible that I'm still using a key-binding that conflicts with VSCode, but I don't know how else to confirm that.
I have upgraded to 0.10.8 and the issue still persists.

v1.5.3 and issue persists (as expected, since this discussion has stalled 😄 )
I wanted to bind some system services to global hotkeys, namely so I can switch between vscode, iterm, and chrome using Cmd-1/2/3. They work fine until I invoke the switch to vscode, and then I'm stuck since vscode seems to e.preventDefault() 😉

I think the point is more that the standard macOS system-wide “Services” menu is completely missing (VSC v1.9.0); it’s meant to be under the application menu (i.e. Code → Services). Without it, no services are accessible from VSC and thus any custom (or default?) keyboard shortcuts won’t work.
Compare e.g. with TextMate 2, where the application menu subitems are:

About TextMate
Preferences…
Check for Updates
Services
Hide TextMate
Hide Others
Show All
Quit TextMate

instead for VSC, we have:

About Visual Studio Code
Check for Updates…
Preferences
Hide Visual Studio Code
Hide Others
Show All
Quit Visual Studio Code

I don’t know how complex it is for VSC to support the system-wide Services menu…?

Bummer. I was trying to do the same as @julianlam . Would be nice to see this fixed!

@XanderTigerclaw is headed down the correct path.  The VS application as a whole is not letting any OS Services shortcuts work (the menu doesn't even appear!) while VS has focus.
It is quite unfortunate for me, as I use ~5 different Services on a daily basis. The current workaround is to switch my focus to another application or the Desktop/Finder.  Not a huge problem, but a consistent and mild inconvenience.

Fixed via https://github.com/Microsoft/vscode/issues/26248


[enhancement] Add missing bower.json.

Hey, maintainer(s) of markmalek/Fixed-Header-Table!
We at VersionEye are working hard to keep up the quality of the bower's registry.
We just finished our initial analysis of the quality of the Bower.io registry:
7530 - registered packages, 224 of them doesnt exists anymore;
We analysed 7306  existing packages and  1070 of them don't have bower.json on the master branch ( that's where a Bower client pulls a data ).
Sadly, your library markmalek/Fixed-Header-Table is one of them.
Can you spare 15 minutes to help us to make Bower better?
Just add a new file bower.json and  change attributes.
{
  "name": "markmalek/Fixed-Header-Table",
  "version": "1.0.0",
  "main": "path/to/main.css",
  "description": "please add it",
  "license": "Eclipse",
  "ignore": [
    ".jshintrc",
    "**/*.txt"
  ],
  "dependencies": {
    "<dependency_name>": "<semantic_version>",
    "<dependency_name>": "<Local_folder>",
    "<dependency_name>": "<package>"
  },
  "devDependencies": {
    "<test-framework-name>": "<version>"
  }
}

Read more about bower.json on the official spefication and nodejs semver library has great examples of proper versioning.
NB! Please validate your bower.json with jsonlint before commiting your updates.
Thank you!
Timo,
twitter: @versioneye
email<EMAIL_ADDRESS>VersionEye - no more legacy software!


BD Crashing After Update

I updated better discord, and it keeps crashing. Also, I can't even go to the settings on my plugins.

Yeah, I'm getting this error:

Yeah, I'm getting this error:

This is not the same installer linked in this repository. Get the latest version from here.

As for the main issue, this isn't directly related to the installer itself so it should probably be reopened at https://github.com/rauenzi/BetterDiscordApp/issues, but I can provide a few pointers.

Try wiping your plugins folder.
Try completely reinstalling discord (delete %appdata%/Discord and %localappdata%/discord), then completely reinstsall everything.


fix: Quality selector bg issue in light mode

PR Fixes:

1 Items inside the quality selector in video controls is now visible in light mode
2 Remove unnecessary quotations

Resolves #1276
Before:

After:

Checklist before requesting a review

[X] I have performed a self-review of my code
[X] I assure there is no similar/duplicate pull request regarding same issue

@devsargam @siinghd

Thank u sir.


SinBall - Hrushal

Write a short description about your art piece, such as what your artwork represents and your experience drawing it!
Next, make sure you go through each item in the following checklist. Delete this line after!

[x] I have read the steps to getting a blot
[x] I am submitting art that...

[x] is algorithmically generated (will meaningfully change each time the program is run)
[x] is your own work that is not copied from somewhere else
[x] is drawable through the editor(does not use outside tools)
[x] doesn't call Math.random() (See the documentation on randomness)
[x] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[x] Doesn't overlap lines more than 5 times
[x] Fits in the working area of 125x125mm
[x] Doesn't rely on colour to look good

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:
Lastly, link a picture of your art piece here:

Hey! I really love this! This is very cool (and trippy) to look at :D
There's one small thing though, changing the rotation to something higher (20s-30s) makes the wave count become smaller, is this intended?

Oh yea i forgot to patch that. while i was figuring out how to draw the waves i set a limit on the x that caused this

and now it breaks after ~45 degrees but due to the rotation func

Yep solved the issue

Thank you for fixing it! Do you think you'd be able to work on it a little bit more? One of the first things that come to mind is some sort of sport ball (soccer, volleyball, etc), it'd be really cool to see something like that! You have a great base, the options are limitless :D

my original idea was to make the sine waves move inside the ball like an animation but couldn't do that with blot. and adding geometric patterns wont be hard but i dont have much freetime on my hands rn

Hi there! This is a really cool project, but it needs to be generative! (i.e. it is different every time the code is run) Im going to close this PR for now, but once you are done with your edits please make a new PR and we can review it there! (This is due to a new way we are managing Blot PRs)


Could not find com.google.firebase:firebase-core:9.0.0

Hello
I was following the README instructions using JavaScript. After I installed this plugin this error came up every time I try to build the project.
PS C:\Users\Edcs9\Documents\NativeScript\notifications-test2> tns run android
Executing before-prepare hook from C:\Users\Edcs9\Documents\NativeScript\notifications-test2\hooks\before-prepare\nativescript-dev-android-snapshot.js
Executing before-prepare hook from C:\Users\Edcs9\Documents\NativeScript\notifications-test2\hooks\before-prepare\nativescript-push-notifications.js
Preparing project...

:config phase:  createDefaultIncludeFiles

:config phase:  copyAarDependencies

:config phase:  addAarDependencies
        +adding dependency: C:\Users\Edcs9\Documents\NativeScript\notifications-test2\platforms\android\libs\aar\pushplugin.aar
        +adding dependency: C:\Users\Edcs9\Documents\NativeScript\notifications-test2\platforms\android\libs\aar\widgets-release.aar
google-services plugin could not detect any version for com.google.android.gms or com.google.firebase, default version: 9.0.0 will be used.
please apply google-services plugin at the bottom of the build file.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'notificationstest2'.
> Could not resolve all dependencies for configuration ':_debugCompile'.
   > Could not find com.google.firebase:firebase-core:9.0.0.
     Searched in the following locations:
         https://jcenter.bintray.com/com/google/firebase/firebase-core/9.0.0/firebase-core-9.0.0.pom
         https://jcenter.bintray.com/com/google/firebase/firebase-core/9.0.0/firebase-core-9.0.0.jar
         file:/C:/Users/Edcs9/Documents/NativeScript/notifications-test2/platforms/android/libs/aar/firebase-core-9.0.0.jar
         file:/C:/Users/Edcs9/Documents/NativeScript/notifications-test2/platforms/android/libs/aar/firebase-core.jar
         file:/C:/Program Files (x86)/Android/android-sdk/extras/android/m2repository/com/google/firebase/firebase-core/9.0.0/firebase-core-9.0.0.pom
         file:/C:/Program Files (x86)/Android/android-sdk/extras/android/m2repository/com/google/firebase/firebase-core/9.0.0/firebase-core-9.0.0.jar
     Required by:
         :notificationstest2:unspecified

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

BUILD FAILED

Total time: 6.049 secs
Processing node_modules failed. Error: Command C:\Users\Edcs9\Documents\NativeScript\notifications-test2\platforms\android\gradlew.bat failed with exit code 1

I already put this lines on the build.gradle
classpath "com.google.gms:google-services:3.0.0"
apply plugin: 'com.google.gms.google-services'

Also added the google-services.json on App_Resources/Android folder

UPDATE: I just installed and update the Android Repository and google play on the SDK and tns remove android


Knife on Planks [Bug Report]

I made a iron knife, tried to use on oak planks, and it made the plank into the Oak Debarked Log.
Mod Version: Better With Mods-0.10-1.10.2
Modpack: Proton 1.0.5 (on Curse launcher)

Oh, that's an interesting one. Looks like the changes in the debarking event were getting too greedy and looked for anything the saw can cut, instead of being limited to logs. It's been fixed, I'll probably push an update soon.


Test failure: IdentityUI_ScriptTags_FallbackSourceContent_Matches_CDNContent

This test fails occasionally with the following error:
System.Net.Http.HttpRequestException : The SSL connection could not be established, see inner exception.
---- System.IO.IOException : Authentication failed because the remote party has closed the transport stream.
   at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at System.Net.Http.HttpClient.GetStringAsyncCore(Task`1 getTask)
   at Microsoft.AspNetCore.Identity.Test.IdentityUIScriptsTest.IdentityUI_ScriptTags_FallbackSourceContent_Matches_CDNContent(ScriptTag scriptTag) in /_/test/Identity.Test/IdentityUIScriptsTest.cs:line 71
--- End of stack trace from previous location where exception was thrown ---
----- Inner Stack Trace -----
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ForceAuthentication(Boolean receiveFirst, Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessAuthentication(LazyAsyncResult lazyResult)
   at System.Net.Security.SslStream.BeginAuthenticateAsClient(SslClientAuthenticationOptions sslClientAuthenticationOptions, CancellationToken cancellationToken, AsyncCallback asyncCallback, Object asyncState)
   at System.Net.Security.SslStream.<>c.<AuthenticateAsClientAsync>b__47_0(SslClientAuthenticationOptions arg1, CancellationToken arg2, AsyncCallback callback, Object state)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncImpl[TArg1,TArg2](Func`5 beginMethod, Func`2 endFunction, Action`1 endAction, TArg1 arg1, TArg2 arg2, Object state, TaskCreationOptions creationOptions)
   at System.Threading.Tasks.TaskFactory.FromAsync[TArg1,TArg2](Func`5 beginMethod, Action`1 endMethod, TArg1 arg1, TArg2 arg2, Object state, TaskCreationOptions creationOptions)
   at System.Threading.Tasks.TaskFactory.FromAsync[TArg1,TArg2](Func
...

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on master.
CC @Eilon (because the bot doesn't know who else to pick)
This issue was made automatically. If there is a problem contact ryanbrandenburg.

Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

@mkArtakMSFT can you assign?

@mkArtakMSFT - ping

@javiercn, can you please look into this and see whether this is still an issue or not? Thanks!

Just looked at it. It was a transient network error as this test needs to go and download an external resource to hash it and validate our integrity tags.
We could add a retry here if we have to, but I would do nothing for now until it manifests itself again (if ever)

This will almost certainly fail again if it uses external network. I recommend adding a retry sooner rather than later (but perhaps not critically urgent).

@javiercn , please either add retries or whatever is necessary to reduce the chance of happening this again.

Yes, I will do so once I handle other build issues

Added retries here https://github.com/aspnet/Identity/pull/2054/files#diff-5d5e5a2c35833245c7dc73bf232673abR172


Repo is dead

Trying to add maven dependecy, you'll get an error.
I checked your domains for repo and they are alive, but inaccessible.
Please fix it or just give me the .jar of Shoebill to add it manually.
thx

Please look at this issue: https://github.com/Shoebill/shoebill-runtime/issues/6


Refreshing playground and testset results in an alert

Issue #1015

does it work in firefox ?

Yes it works on Firefox


MMSeqs2 DB slimmer

Hi
this is not an issue but a potential enhancement we discussed with @martin-steinegger.
We have a seed clustering database that is continuously updated with new sequences. The size of the DB is growing quite fast, and eventually, we will have problems storing and distributing it. As we have many redundant sequences in each cluster. We thought that having a module that takes a DB and then filters it based on a criterion similar to --diff from result2msa or result2profile would be very useful to keep only informative sequences in the clusters.
Thanks
Antonio

Thank you very much @milot-mirdita It will be very useful to keep our DBs slim :-)

I added a filterresult module that does basically the same as result2msa but returns as result database. Hope it's useful :)

Awesome! @ChiaraVanni will test it and will we back to you in case we find any problem. Many thanks!

We had a bug that martin fixed yesterday in https://github.com/soedinglab/MMseqs2/commit/18588bb351025f198fdf1f870556d2cc0dcf0a65.

Hi @milot-mirdita
finally, we got our hands on the filterresult, and we have a couple of questions about how to proceed after the filtering.
After running:
mmseqs filterresult seqDB seqDB cluDB cluDB-filt --threads 28

we got few alignment DBs, the index and dbtype files. Looking at the alignment DB files, it seems that they have the cluster DB format, and the number of entries has decreased substantially. Any suggestions on converting the output of filterresult to a cluster DB we can use for updating? Here you can get the DB we are trying to slim down https://ndownloader.figshare.com/files/23066651
Many thanks!
Antonio

You can use filterdb --trim-to-one-column to discard all alignment columns in this database and leave only the identifiers.
Then you can overwrite the dbtype file with:
awk 'BEGIN { printf("%c%c%c%c",6,0,0,0); exit; }' > res.dbtype

To convince MMseqs2 that this database is a cluster result and not an alignment result.

Thanks @milot-mirdita This works perfect.
Last question, if we want to have the associated sequence DB with the subset of sequences in the slimmed down cluster DB. If we use createsubd with the original sequence DB and the slimmer cluster DB will only pick the representative sequences, any tips?
Many thanks!
Antonio

Usually I use something like:
mmseqs createsubdb <(cat db.[0-9]* | tr -d '\000' | cut -f1) db db_sub

Here we use tr to remove the NULL bytes between db entries and then cut to only return the first column (which is not strictly necessary as createsubdb would ignore all other columns anyway. If your database is not split you can drop the .[0-9]* part.

Worked beautifully! Thanks @milot-mirdita

Following this issue, i was confused when i want to select only 10 most divergent seqs in each cluster by the following command:
mmseqs filterresult db db db_clu db_clu_nr -diff 10
I am wondering what's the behavior when the group number is less than 10 (some cluster has 6/7 ...).

filterresult should behave equivalent to HH-suite's hhfilter, with the additional implemented feature that multiple sequence identity thresholds (--qid) for filtering can be used instead of only one.
It should keep all entries, assuming the other filtering parameters are also fulfilled, by default --qsc (Minimum score per column) is also enabled and set to -20.


Need help with laravel config

I have a project structure like at the attached screenshot (root test directory is the app). Laravel (actually lumen) app is located in subdirectory "lumen" and not in the root. So, with the standard laravel receipt I have path problems with vendors and shared tasks.
What is the right way to configure deployer in that case?

With this customized directory structure, I think you have to specify your own paths and tasks. For example
namespace Deployer;

set('shared_dirs', [
   'lumen/storage',
]);

set('shared_files', [
    'lumen/.env',
]);

desc('Installing vendors');
task('deploy:vendors', function () {
    run('cd {{release_path}}/test/lumen && {{bin/composer}} {{composer_options}}');
});

Thank you so much.


preserve boolean dtype in encoding

[x] Closes #4826
[x] Tests added
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Split out of #7654. Follow up to #7719. Needs rebase after #7719 is merged.

Rebased on top of #7719. This is ready for review. It's a one-liner actually :grin:

Thanks, @kmuehlbauer !


Update dependencies

Codecov Report
Patch and project coverage have no change.

Comparison is base (1307656) 94.77% compared to head (603bb00) 94.77%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #62   +/-   ##
=======================================
  Coverage   94.77%   94.77%
=======================================
  Files           5        5
  Lines         306      306
  Branches       70       70
=======================================
  Hits          290      290
  Misses         13       13
  Partials        3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Drain service attempts to create bucket that already exists

It appears that the drain service is attempting to create a bucket that already exists. Here are the relevant logs:
2023-05-26 17:23:49,669 - INFO - Connected to S3 client
2023-05-26 17:23:49,683 - INFO - opni-drain-model bucket does not exist so creating it now
Traceback (most recent call last):
  File "/app/./drain_modules.py", line 11, in <module>
    drain_training_inferencing.main()
  File "/app/drain_training_inferencing.py", line 377, in main
    persistence = FilePersistence("workload_drain_model.bin")
  File "/app/drain3/file_persistence.py", line 26, in __init__
    self.connect_to_s3()
  File "/app/drain3/file_persistence.py", line 56, in connect_to_s3
    self.s3_client.create_bucket(Bucket=S3_BUCKET)
  File "/opt/venv/lib64/python3.9/site-packages/boto3/resources/factory.py", line 520, in do_action
    response = action(self, *args, **kwargs)
  File "/opt/venv/lib64/python3.9/site-packages/boto3/resources/action.py", line 83, in __call__
    response = getattr(parent.meta.client, operation_name)(*args, **params)
  File "/opt/venv/lib64/python3.9/site-packages/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/opt/venv/lib64/python3.9/site-packages/botocore/client.py", line 676, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.BucketAlreadyExists: An error occurred (BucketAlreadyExists) when calling the CreateBucket operation: The requested bucket name is not available. The bucket name can not be an existing collection, and the bucket namespace is shared by all users of the system. Please select a different name and try again.

Restarting the pods does not fix the issue.

This appears to be an issue with Seaweed when nodes are rebooted. The plan is once the Python nats wrapper is updated to include object storage, then that will be used in place of Seaweed.


Initial build of new client for development mode - #3423

@planetf1 @davidradl
Summary of changes:

created a new directory called cra-client, until cutover work is complete
rebuilt existing files with Create React App, so that webpack is optimized to run in development mode with hot reloading
upgraded CRA so it can use SASS styling
updated some files with syntax that did not pass the new linter (more strict JS syntax)
new client uses yarn instead of npm (yarn add vs. npm install, yarn start vs. npm start)
temporarily updated routing in IdentificationContext.js to add a dummy server in the url, until we are able to connect the new client to the Nodejs server and run them concurrently

More iterative updates are needed until the new client can replace the old, but I wanted to merge in my changes so I can work with others on connecting the new UI to the backend.

@jmmadsen I notice that at least one commit has not been signed off. Remember to use git commit -s, please can you amend the commit in git to sign it.


Allow versioning to include git information if not using released version

What is the expected enhancement?
Currently there is no way to distinguish between the master branch dev version and a final release version.  Currently, for example, they would both be labeled as 0.8.0.
It would be nice to have unreleased (dev) versions tagged with the git repo commit that they are pulled from.  This is done, for example in SciPy and NumPy, and it quite easy to implement.

Something I have proposed in another issue related to requirements is to adopt the use of pbr which will handle the dev versioning automatically for us (it just looks at git) and make a lot of other things automatic.
It also should work well with cython (because of #1789 this is something to think about) via: https://copyninja.info/blog/cython_setuptools.html

So in general I think we should not add dependencies unless really needed.  Although I am not sure what the "a lot of other things" you mentioned are, at least in this case, we can follow the SciPy and NumPy people and use a simple git call: https://github.com/scipy/scipy/blob/6db252d0105ba6fe3b23dd10d7a0da17dac0ce53/setup.py#L63
The Cython build in #1789 is already working, and does not need any additional tools (the linked article is three years old).

Well it would handle all the versioning for us, not just dev versions. We actually remove an explicit version string completely and it uses git to determine the version. So if we tag a commit that becomes the version, or it does dev versioning compliant with PEP440 Which removes the need for a version.txt file and an making sure we update everything (it can be manually overridden if necessary). Additionally it simplifies the requirements management in a similar way, just using requirements.txt, so we don't have to worry about forgetting to update it in 2 places. The metadata handling I find a bit simpler because it's done in a config file instead of code (but that's just my personal preference). It also will generate normal files for use like an authors list, a Changelog, and a manifest.in (by just looking at the files tracked in git). There's a high level overview of what it does here: https://docs.openstack.org/pbr/latest/user/features.html
As for the cython article I was just linking that to show it wouldn't break our cython usage by switching. Not that it was a requirement for adding it.

That being said I'm not tied to using pbr we can easily embed something like what you linked from scipy in the setup.py. I just think the benefits pbr provides is worth adding an extra dep (it's why I use it on almost all my personal projects).