Template Literals documentation appears in Russian even though I have English selected as my language

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
When I view this page with my language set to en-US, it seems to be in Russian. Other pages appear correctly in English.
Here's a screenshot including the address bar and the language dropdown:

The Google Cached version of the page from 4 days ago appears in English.

Argh! That's because someone incorrectly put a Russian translation over the top of the English version. I've reverted it now, so you should see the English text again.
Thanks for reporting this.


x/mobile: reverse binding: undefined

What version of Go are you using (go version)?

$ go version  go1.12.6 linux/amd64
$ gomobile version +e8b3e61 Wed Aug 14 14:30:26 2019 +0000 (android); androidSDK=/home/user/Android/Sdk/platforms/android-29

What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/user/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build422159366=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Get the fields of the Build class.
https://play.golang.org/p/PkV4bxUJf8z

$ gomobile bind -target android -ldflags "-w -s" -bootclasspath /home/user/Android/Sdk/platforms/android-29/android.jar .

The java.lang.Float.MIN_VALUE works. Why? The field type must be Integer? String does not work?
What did you expect to see?
Board String
What did you see instead?

Test1() -> Error: b.BOARD undefined (type "Java/android/os".Build has no field or method BOARD)
Test2() -> Error: undefined: Build.BOARD

The Go project uses its bug tracker only for tacking bugs and for Proposal Process.
For asking questions is recommended stackoverflow and other third-party platforms.
This proposal was published more than 2 years ago:

Static methods and constants
After importing, the resulting packages SomeClass and InnerClass will contain the static methods and static final constants from their Java classes. For example
import "Java/java/lang/Float"
will expose (among others) the constant Float.MIN_VALUE and the function Float.ParseFloat.

Then, the same person answer this:

I'm sorry, but Java fields are not (yet) exposed by reverse bindings.

MIN_VALUE is a field of class Float, and works.
It is contradicting itself because it gives an example how to access Java fields and below it says that it is not possible to access Java fields.
Why do you assign unskilled programmers to do the job?
And then, to solve a problem, they recommend talking to other people (third-party platforms). When they are responsible.
They should be more explicit.
Two years later and still unable to implement a solution.
https://github.com/golang/go/issues/16876

Hello @mvasi90, thank you for filing this bug!
Before proceeding, I'd highly recommend reading through Go's code of conduct https://golang.org/conduct and also I'd recommend, "always assume good intent and be kind", "no is considered temporary, yes is forever".
Most of us are entirely volunteers who happily invest/sacrifice our own time, money and resources to work on the Go project! Regardless of one's position, it can be very discouraging to see not too kind words, so please try to express yourself differently. Everyone is welcome to the Go project and the humility of the folks you saw on that bug is the same humility that doesn't parade their very high proficiency; picture the work it took to get Go supported for mobile devices and how they've made it look easy; that work is only possible because of their amazing talents and skilled work!
I'll assume good intent and that this is perhaps a miscommunication while expressing your frustrations.
With that having been said, let's try to get to the filed issue:
I believe you filed a bug about reverse bindings being undefined and that the message about MIN_VALUE was unrelated?

The java.lang.Float.MIN_VALUE works. Why? The field type must be Integer? String does not work?

Could you please elaborate your question here? I suspect you are directly using that value as a constant and if its constant expression looks like an integer that fits into 256 bits, the compiler will default its type to an integer. Perhaps return or use it as a float32 and you'll be good to go. .String() for integers even in plain Go is undefined, so unlike Java where you'll have an autoboxed/promoted type to Integer and finally java.lang.Integer.toString() method, in Go you'll need to use fmt's "%d" format specifier or strconv for formatting integers.
I am not an x/mobile expert but I shall kindly page @eliasnaur if he has time to reply to your issue.

First: I apologize for not being word perfect in english.
Second: If it seems that I use destructive criticism or I have a strange behavior, it must be due to the lack of tolerance for nonsense.

Most of us are entirely volunteers who happily invest/sacrifice our own time, money and resources to work on the Go project.

Now I understand. I thought it was a serious project.
I am wasting my time researching undocumented things (for more than two years). But I see that what is important and what must be respected is the time of others because they are volunteers.

Regardless of one's position, it can be very discouraging to see not too kind words, so please try to express yourself differently

Okay, I apologize for my severity.

I'll assume good intent and that this is perhaps a miscommunication while expressing your frustrations

Yes, of course. Thank you. I'm working every day (7 days a week), 11-17 hours per day. I'm designing and implementing all (from scratch): my own protocols, advanced security tests, I'm creating SeLinux  policies (domain transition, multi category security, etc) on ArchLinux (which is not fully compatible with existing policies, nor does it have official support), I'm using my own Cluster (with Corosync messaging layer and Pacemaker resource manager) better than Docker swarm, etc etc.
You can imagine how frustrated I am, and what this lost time means to me.
I no longer have patience to see how people learn to walk. (Of course, @eliasnaur  is not "learning to walk", he already has experience. It is a generic comment)
I prefer Go before C / C ++ because it speeds up the work and already ensures typical programmer concerns (pointers, etc.) and I can focus more on my work.

I believe you filed a bug about reverse bindings being undefined and that the message about MIN_VALUE was unrelated?

No. Float.MIN_VALUE are working, but Build.BOARD does not.

I suspect you are directly using that value as a constant and if its constant expression looks like an integer that fits into 256 bits, the compiler will default its type to an integer. Perhaps return or use it as a float32 and you'll be good to go. .String() for integers even in plain Go is undefined, so unlike Java where you'll have an autoboxed/promoted type to Integer and finally java.lang.Integer.toString() method, in Go you'll need to use fmt's "%d" format specifier or strconv for formatting integers.

I want to be kind, I will not answer that. Thanks for your time.
At this step it is not necessary to write this, but I will:
Does not work:
package b

import "Java/android/os/Build"

func Test() {
     _ = Build.BOARD
}

package android.os;

public class Build {
...
  public static final String BOARD = null;
...
}

Note: BOARD field is final, once "instantiated" (initialized), its value cannot be modified. Even if Java (Dalvik) sends it to Go, it doesn't work.
Works:
package b

import "Java/java/lang/Float"

func Test3() {
   _ = Float.MIN_VALUE
}

package java.lang;

public class Float {
...
    public static final float MIN_VALUE = 1.4E-45F;
...
}

What is the difference?

Float.MIN_VALUE type is float and Build.BOARD type is String.
Float class can be instantiated and Build not.

Indeed, if it cannot be instantiated, there is no object. And Go cannot receive "object reference", such as b os.Build. But you should be able to access the constant fields just like Float.MIN_VALUE.

Static methods and constants
After importing, the resulting packages SomeClass and InnerClass will contain the static methods and static final constants from their Java classes. For example
import "Java/java/lang/Float"
will expose (among others) the constant Float.MIN_VALUE and the function Float.ParseFloat.

Then, the same person answer this:

I'm sorry, but Java fields are not (yet) exposed by reverse bindings.

MIN_VALUE is a field of class Float, and works.
It is contradicting itself because it gives an example how to access Java fields and below it says that it is not possible to access Java fields.

The comment above refers to fields that are not constant (final static). PackageInfo.installLocation is not constant and therefore not supported.
You correctly point out that Build.BOARD is final static and therefore should be supported. It's been a while since I worked with gomobile, but my guess is that another restriction is that the field must also be a primitive type to be supported. You and I know that String is immutable and could be treated as a primitive constant, but go mobile doesn't know that (yet).
If you're still interested, I encourage you to take a stab at adding support of constant String fields to gomobile.

Okay. If I had known that the constants depend on the type, I would have started the project with C++. It's not too late, but I lost valuable time. Well, no problem.

If you're still interested, I encourage you to take a stab at adding support of constant String fields to gomobile.

I don't like to leave the projects unfinished, And I am very busy, I am doing the work of an entire company (system administrators, security, database, multiplatform client and server programming, etc etc)
In my spare time I am doing security tests with a ESP32 microcontroller, trying to access eFuse secret keys or bypass ECDSA signature validation.
If I do not find important vulnerabilities, I will begin to develop the second part of current project in this type of microcontrollers.
I think I don't be able to contribute for now. But of course, I appreciate your work. And if you can, please leave documented when something does not work.

I'm having trouble figuring out what the feature request in this issue is: I see some questions in the original post, some discussion of previous proposals, and a comment from @eliasnaur suggesting a specific contribution, but I'm having trouble seeing the scope of that contribution or the component to which it would be applied, and moreover I'm hesitant to tag this issue help wanted due to the fairly hostile tone so far.
If there is a concrete feature request or bug fix to be resolved here, please open a new issue and describe the specific changes that are needed. Thanks.

Hello @mvasi90 ,
It is not related to this issue..
Acutally I want to explore reverse binding.. Can you please let me know which is the best place to start.. I couldn't find any concrete sample or documentation at:
https://github.com/golang/mobile/tree/master/example
Any help will be appreciated..:)

@jay11ca39  This is not a good place to ask that because they don't want to mix things up. At any moment the cleaning lady appears and throws a fight at you.
I strongly recommend that you take time to learn C or Rust for this purpose. You can make your own wrappers manually. x/mobile is interesting but it is not "mature" or stable.
Missing documentation. Volunteers do not specify the limits of existent wrappers. When someone asks something specific they answer incorrectly: For example: treat the Java fields and the constants as two different things. But a constant is a field.

A field may be a class (instance) field, such as java.io.Reader.lock , a static field, such as java.lang.Integer.MAX_VALUE , or an enum constant, such as java.lang.Thread.State.WAITING.
https://docs.oracle.com/javase/tutorial/reflect/member/field.html

Now, if you still want to do it in Golang, you can investigate here:
https://godoc.org/golang.org/x/mobile/cmd/gobind#hdr-Reverse_bindings
https://github.com/golang/go/issues/16876
You can't access some Java fields from Go. I recommend you to invoke setters and getters from Go. If an existent class (Java source) does not have setters and getters you can make a wrapper class in Java an use it from Go. Also you can pass data between Java and Go with interface callback.
For reverse binding the best example is the asset package. Here is an example in which I also incorporate the use of callbacks:
Golang:
package mylib
import (
	"io/ioutil"
	"golang.org/x/mobile/asset"
)
// Send back information to Java from Golang (async)
type Callback interface {
	R(data []byte) // Response: byte slice (array in Java)
        E(error byte)  // Error: single byte (better than string or byte slice)
}

// call from Java Mylib.test(); (before this, you should set the context, read below: java side)
func Test(cb Callback) {
   // go func(){}() is optional, used to run the process in the background with goroutine (asynchronous operations)
   go func(cb Callback) {
      f, err := asset.Open("secret-encrypted.raw")
	if err != nil {
		// call java (callback)
                cb.E(0x01) // or 1 instead 0x01

                // I do not recommend sending compromising information to Java.
                // To make reverse engineering more difficult you can send single byte value as flag.
                // Example: 0x01, 0x02, etc (for every error). And process errors with a switch in Java side.
                // cb.E("Unable to open file") // don't do this
                // cb.E([]byte("Unable to open file")) // or this
		return
	}
	defer f.Close()
	b, err := ioutil.ReadAll(f)
	if err != nil {
                cb.E(0x02) // this is another error. You can process it from Java or better treat them all equally sending every time 0x01 or random number
		return
	}
        // call decrypt function
        // send back to Java decrypted data
        cb.R(decrypted_byte_slice)
   }(cb)
}

Java example (with Fragment):
import mylib.Mylib;
import mylib.Callback;
import go.Seq;

public class MyFragment extends Fragment {
private Context c;
    @Override
    public void onAttach(@NonNull Context context) {
        super.onAttach(context);
        c = context;
    }
    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);
        Seq.setContext(c);
        Mylib.test(new Callback() {
            @Override
            public void r(byte[] data) {
                  // process response
            }

           @Override
           public void e(byte error) {
                  switch(error) {
                         case 0x01:
                               // process error 0x01
                         break;
                          // ...
                   As you can see, you can do background operations. Performing operations on tasks other than the main one adds a little more difficulty when reverse engineering is used. You can set a little random sleep (for example from 1 to 5 seconds) before sending the response to Java. (Android = Java).
If you don't want to call Go from java first and wait for response callback, but directly call Java from Go, if you are calling Java (Android) method you should build the Go lib using --bootclasspath param:
gomobile bind -target android -ldflags "-w -s" -bootclasspath /home/user/Android/Sdk/platforms/android-<your-version>/android.jar .
Or for your own Java/Android package and class you can use the param -classpath /path/to/your/java/classes. Android Studio stores the compiled classes in ProjectName/app/build/intermediates/javac/release/compileReleaseJavaWithJavac/classes. But you can build a simple java package manually with javac. For example make a copy of your package and build it manually (as Java SE desktop) to help gomobile to make wrappers.
More low level example: (you can make your own C wrappers)
https://github.com/golang/mobile/blob/master/asset/asset_android.go
More info:
I know that all developers want to protect their content. Here is an example to verify the Android app signatures with Go:

Check the package of running app.
ioutil.ReadFile(fmt.Sprintf("/proc/%d/cmdline", os.Getpid()))
cmdline contains the  of your app. If package is the same, go to step 2
Note: The native library (Go in this case) have the same PID as the Android app.

Get the apk file path to read and parse the signatures directly from Go
This step can be easy or more complicated.

Complicated if you find the loaded apk path, read it, parse signature (V1, V2 and/or V3) and verify.
Verify does not mean having a condition (if true ok), instead use a master key created with signature as seed.

Get files of all links stored in /proc//fd/. Readlink and check if any file have your package name and if ends with .apk.

Read the apk file and get signatures.
Creating your own implementation of Signature Scheme parser learning this:
https://source.android.com/security/apksigning
https://source.android.com/security/apksigning/v2
https://source.android.com/security/apksigning/v3
Or using any existent library (like apkverifier created by Avast team):
https://github.com/avast/apkverifier

Example:

f, err := os.Open(fmt.Sprintf("/proc/%d/fd", os.Getpid()))
if err == nil {
    fs, err := f.Readdir(-1)
        if err == nil {
            for _, file := range fs {
                // readlink, like linux command
		l, err := os.Readlink(fmt.Sprintf("/proc/%d/fd/%s", os.Getpid(), file.Name()))
		if err == nil {
                    if strings.HasPrefix(l, "/data/app/") && strings.Contains(l, "<your-package-name>") && strings.HasSuffix(l, ".apk") {
                        res, err := apkverifier.Verify(l, nil)
                        if err == nil {
                            for _, cert := range res.SignerCerts {
                                for _, crt := range cert {
                                    // replace this by your function that creates the master key based on the raw signature.
                                    // Important:
                                    // don't break the loop
                                    // don't check if the signature is Okay
                                    // a hacker can keep your signature and add another. Use all signatures to make your master key used to decrypt data (master key must created first on your desktop pc with bash script or go binary, and use it to encrypt important files from your desktop and put on your Android Project)
                                    fmt.Printf("Signature: %v\n", crt.Raw)
                                    // masterKeyCalculator(crt.Raw)
                                }
                            }
                        }
                    break
                }
            }
        }
    }
}

I hope these examples help you.


Refactor flutter dashboard datastore

[ ] Run data analysis process to determine write/read requirements
[ ] find what is writing to TimeSeries entity and delete the code and then delete the entity
[ ] Remove GithubTreeStatusOverride entity
[ ] Copy commit, branch and repo to Task entity
[ ] Break out the CheckList entity relationship with task and rename it to Commit
[ ] Collect branch data from Commit and delete Branch entity
[ ] Add a TTL to GithubBuildStatusUpdate
[ ] Support multiple Task entries for the same repo, branch, commit, task name
[ ] Remove dead code
[ ] Move CocoonConfig out of the data store

@keyonghan @ricardoamador FYI

@godofredoc awesome!


Importers: Review importer classes to ensure logging is scrubbed

This involves doing a pass on all importer classes and verifying that data isn't being logged or sent to Rollbar.  As one specific example, the BehaviorImporter class logs this: https://github.com/studentinsights/studentinsights/blob/master/app/importers/file_importers/behavior_importer.rb#L114
      log("skipping, StudentLocalID not found: #{row[:local_id]}")

Another example is in DibelsRows:
DibelsRow: couldn't parse DIBELS benchmark: #DIV/0!

We should avoid logging any data or values that might contain anything close to personally identifiable information.  If there are particular kinds of validation errors or processing errors, we can add variables to track the count or to track unique kinds of failures and log that count.  For classes that use RecordSyncer, it also has heuristics to alert on significant changes.
For developing new jobs, we can collect failure types or validation types, and if it is really necessary allow options that enable more verbose logging (eg, that would have to be enabled explicitly on the console).

This was done in https://github.com/studentinsights/studentinsights/pull/2710.


Kotlin support

I've started using Kotlin and noticed there is no support for it currently in this plug-in. Based on the Kotlin testing documentation I was able to quickly put together a test runner which uses Gradle by copying and editing code used by the Java test runner. Should these two share code or is best to keep them separate? Happy to raise a PR based on what I have already

@codeinabox I'm interested in Kotlin support for this library. Do you have a branch or repo I can look at with this working?


Nbody single precision fixes

This fixes #39 with the same suggestions as in the ticket, plus a new Travis job added with the same configuration to ensure this doesn't happen again.

I closed the PR by mistake, but nevertheless incorporated the changes onto the masterbranch.


Add the missing instruction to add the repo

Proposed changes

Add the missing instruction to add the repo. This change is only affecting the RHEL 7/8 portion of the doc.
Unreleased project version (optional)

Related issues (optional)

Issue reported by Billy Lieberman. /cc Pierre Antoine

Deploy preview for docsdocker ready!
Built with commit 7f62d13fce7221bb097335c71a2fb274613dbcc8
https://deploy-preview-9911--docsdocker.netlify.com


chore: bump version and changelog

However I dont see the 0.5.2 version deployed on rok.dev.renku.ch yet.
yes this is a bit of a hack because of the way chartpress tags versions - the version in our Chart.yaml is actually the upcoming version. The released one is one semantic version lower.


FS-3797 workflows

Amend workflow to follow expected route from dev->test->uat->prod

[ ] Unit tests and other appropriate tests added or updated
[ ] README and other documentation has been updated / added (if needed)
[X] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

LGTM


Add support for retiring old connections for HTTP clients.

Context
When building an HTTP client, we should be able to configure a time-to-live for all pooled connections, such that an individual connection that is idle can never be older than the configured TTL. This is useful for ensuring that new connections are periodically established to the downstream target system, aiding in scenarios like switching over to newly provisioned IPs when load balancer nodes are being replaced, etc.
Notes
This is not something currently exposed by hyper-util but I think, conceptually, could be easy enough to upstream.

Actually, looking at the new additions to<EMAIL_ADDRESS>we might be able to use the ported "poison pill" behavior to poison a connection after we get a response and determine the connection is old enough.
Essentially:

in HttpClient::send, use capture_connection to get Connected for the connection used for the request
check the response extensions and look for a new type we'll call ConnectionAge (we'll create this as a newtype wrapper around std::time::Instant)
if we can't find it in the response extensions, we'll create one based on the current time and attach it to the connection via Connected::extra
if we do find it, we'll see if the connection is older than the configured TTL, and if so, poison the connection via Connected::poison

A poisoned connection won't be returned to the idle pool, so in effect, we're able to mark these connections as no longer usable, and thereby ensuring they get rolled. This is a little lossy, because we're poisoning them right after they get back into the pool, really... and the connection age is attached after the response arrives, so they must end up a few seconds older in reality than what we'd measure if we tracked the actual moment the connection was created.
All of that said, though... it should be more than satisfactory for our purposes.


Mintfreeos failed even though the CREDIT matches the mint fee required.

6/7/22 freeosgov2 v0.9.28, Iteration = 460300 account: alanappleton
Mint POINTs to FREEOS using mint fee payment only (no freepoints). freeosgov2 parameter dummyfee=0.5000 XPR
User: alanappleton, mint amount=1.0000 POINT, mintfeefree=None, CREDIT=0.5000 XPR
mintfreeos action failed-> Error assertion failure with message: the correct mint fee has not been paid. Expecting mintfreeos action to run successfully as CREDIT matches the dummyfee.
This has regressed as it was fixed in freeosgov2 v0.9.26.

.

11/7/22 Retest. freeosgov2 v0.9.29 alanappleton. Fixed.


Feature/417 leftout comments

Description
Adds detecting multi-line leftover comments as a code smell
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Chore: cleanup/renaming, etc
[ ] RFC
[ ] Test

Notes
I tested with the commit that this file has that has a leftover comment itself (already removed) and it successfully dects the leftover comment.
However, there is an error in the logic. The code gets the correct position of the occurrence of the leftover comment (number of lines below the @@ header hunk, not the actual line number in the file). However it comments the lineNumber under the wrong header hunk (the first one). Thsi needs to be fixed for this to be upgraded from draft PR.
Acceptance

[x] I have read the Contributing guidelines
[x] I have read the Code of Conduct

What can we extract? This is a 600 LOC file! Code seems ok, but I think we do not want blocking commenting of files but rather choosing one?

I totally agree there's a bunch of things to abstract. I'm worrying about making it work first, then I'll worry about abstracting the code.

I upgraded the PR to "ready for review". Changes include:

We now handle the multi-line comments created with "//" on each line case. In addition to the actual multi-line syntax (/* */) case. Image below.
We need to index lineNumbers at 4 instead of 0 because the GitHub UI always adds 3 lines below the @@ header (plus they are indexed at 1 instead of 0, plus the octokit comments.position parameter is the number of lines below the @@ heaer instead of the actual line number in the file).
I changed the name of the file and the function to detectCodeSmells since we're generalizing the capabilities of this feature.

Considerations

Might be missing syntax from some major languages
I agree that the file can be abstracted. I propose we agree on whether this works or not, and then the abstraction step could happen in a separate PR.

CC: @EstebanDalelR


Unexpected reserved word - import

I just did a fresh install in my Sails project. Starting server results in this error.
I'm not sure how much E6 support is in Sails right now if any. Our apps are full E6 compliant or nearly so. I have tried several options using a transpiler. But have not gotten past this one yet.
verbose: Loading app services...
/www/5m/rest-api/node_modules/sails-permissions/api/services/ModelService.js:1
(function (exports, require, module, __filename, __dirname) { import _ from 'l
                                                              ^^^^^^
SyntaxError: Unexpected reserved word
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/www/5m/rest-api/api/services/ModelService.js:4:14)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at /www/5m/rest-api/node_modules/include-all/index.js:129:29
    at Array.forEach (native)

My project dependencies include:
Node 0.12.7
"babel-core": "^6.1.2",
"bcrypt": "^0.8.5",
"ejs": "^2.3.4",
"lodash": "^3.10.1",
"passport": "^0.3.0",
"passport-jwt": "^1.2.1",
"passport-local": "^1.0.0",
"pluralize": "^1.2.1",
"sails": "^0.12.0-rc3",
"sails-auth": "^2.1.1",
"sails-permissions": "^2.1.8",

I'm getting the same error, and it seems to be resulting mostly from https://github.com/tjwebb/sails-permissions/commit/696b56418442efa0a751c71af5f8f0b8fda3f910, though there do seem to be a couple other ES2105 style imports. It seems it might be a good idea to use the old node-style require() syntax for the time being?

+1

add to .babelrc following strings
{
  "presets": ["es2015"]
}

I ran into this issue and guessing it's related. Here's my explanation: https://github.com/tjwebb/sails-permissions/issues/198#issuecomment-177629496


Tin Ore not spawning in worlds.

So me and my brother loaded your pack and made a server and so on, but for some reason Tin didn't spawn at all in our world. I've checked the respective Y levels for its spawn and saw nothing. I'm not sure if this is intentional, because (I think) Immersive Engineering can dig up tin without it being present, but I'm not sure. I just thought I should point this out because I have no clue if it's intentional.
Also, we created this server yesterday. We downloaded and updated the pack and then because there wasn't a server download for 1.0.6 yet when we got it, I just deleted the old server mod folder and dragged the mod folder from instances over to the server. Only thing I can think would be that the configs changed the ore spawn to work.

I assume you found tin?


Interference with custom hightlights

Having both project.nvim and this wonderful plugin erases the custom highlighting I made for trailing whitespaces. Commenting out just one of the two solves the problem (at least in some contexts -- I didn't test for all possible situations). I will try to prepare MWE if you are not aware of this problem already.

I'm not at all sure what you mean by this. You'll have to provide more information

Here is a minimal working example demonstrating the problem. Almost minimal.
To reproduce it, you also need a folder having 1) .git and 2) any file containing trailing whitespaces (I put it in the second code section below)
So, when you enter the buffer, according to settings, you have trailing whitespaces highlighted in red (or brown if the buffer type is markdown). You also receive a notification (right after you entered the buffer) that your CWD is changed to the root of the repository. (Note: for that to happen you must open it outside of the repo in which the file is located). With the notification disappearing, the highlighting also disappears. I guess it is not supposed to clear user highlights.
init.lua
local packer = require 'packer'

packer.init {
  display = {
    open_fn = function ()
      return require('packer.util').float {
        border = 'rounded'
      }
    end,
  },
}

packer.startup(function (use)
  use 'wbthomason/packer.nvim'

  use {
    'rcarriga/nvim-notify',
    config = function ()
      vim.notify = require'notify'
    end
  }

  use {
    'ahmedkhalf/project.nvim',
    config = function()
      require("project_nvim").setup {
        -- manual_mode = true,
        silent_chdir = false,
      }
    end
  }
end)

vim.opt.termguicolors = true

vim.cmd [[
  highlight! Normal guibg=Black guifg=White
  highlight! ExtraWhitespace ctermbg=211 guibg=palevioletred
  au! FileType markdown hi ExtraWhitespace ctermbg=brown guibg=rosybrown

  "" Pattern matching for au is different:
  "" to exclude buffer names with leading underscores, we put
  "" 'negative lookbehind' after the star expression.
  au! BufWinEnter *\(_\)\@<! if len(bufname()) > 0
    \| match ExtraWhitespace /\s\+$/
    \| endif

  au! InsertEnter * if len(bufname()) > 0
    \| match ExtraWhitespace /\s\+\%#\@<!$/
    \| endif

  au! InsertLeave * if len(bufname()) > 0
    \| match ExtraWhitespace /\s\+$/
    \| endif

  au! BufWinLeave *\(_\)\@<! call clearmatches()
]]

file_with_trailing_whitespaces.py
##

##

https://user-images.githubusercontent.com/44115822/172024694-a1c2ae99-7a59-4055-b08f-243d51314ed7.mp4

Your BufWinLeave autocmd assumes that the buffer being removed is the current one but that's not the case here. When a notification buffer is hidden, the matches are being cleared for the current window. You need to pass the window to clear matches for
  au! BufWinLeave *\(_\)\@<! call clearmatches(bufwinid(expand("<abuf>")))

Oh yes, thanks!
I was looking for sth like "clearmatches" in your code but never gave a thought to it that the problem was on my end. Sorry for disturbing you!


github actions update

upload-artifact and download-artifact in v4
Bump actions/download-artifact from 3 to 4
Bump actions/upload-artifact from 3 to 4
Bump geekyeggo/delete-artifact from 2 to 4
-> Possible solution for needed write access token on PRs if implemented  (https://github.com/GeekyEggo/delete-artifact/issues/19)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
633

Relevant Lines:
824

💛  - Coveralls

@s-martin Only solution i see for now is to remove the cleanup stage (at least until the linked ticket is resolved). The artifact get then deleted after 2 days (retention).
This would give us less warnings and saves some runtime (v4 saves up to 90% time on up-/download)

@s-martin Only solution i see for now is to remove the cleanup stage (at least until the linked ticket is resolved). The artifacts get then deleted after 2 days (retention). This would give us less warnings and saves some runtime (v4 saves up to 90% time on up-/download)

I think that’s a reasonable approach


Compatibility with Multimine doesn't work after TreeChop-1.19.2-forge-0.18.3

Hello !
I don't know if you're aware of this issue, but the compatibility with Multimine works well on 0.18.3, and then completely fails on further versions.
I've tested it with all the versions above, I've set the logs/wood blocks on true in the config files but it won't work in survival.
Note : It seems to work fine in Creative though.
I've updated my modpack to version 0.18.3 until then, hope this post helped 👍
Thank you for your mod, it is truly awesome.
latest.log

Hey, thanks for reporting. Careful with the 0.18.3 version - it fixed Multi Mine issues, but it broke modded items that break more than one block at a time. I'll see if I can fix this, but I've had a lot of trouble with Multi Mine in the past, so no promises

As of TreeChop 0.19.1, I got around this issue by disabling MultiMine while chopping. Only for Minecraft 1.19.2 at the moment, will port to 1.20.1 and 1.21 shortly.

If you alternate between 2 wood blocks when almost breaking it, you can cut the tree
https://github.com/user-attachments/assets/9677a864-83e7-46aa-94c0-7cd6b8675304


API to set feature state and promoteId missing

Environment

Xcode version: 13.1 (13A1030d)
iOS version: 15
Devices affected: All
maplibre-gl-native 5.9.0

Not finding the js equivalent of setFeatureState method for MGLMapView and setter promoteId for VectorSource.

Hi @sujithkris. Thanks for reporting. I guess the problem is that feature state is not implemented yet in native, see https://maplibre.org/maplibre-gl-js-docs/style-spec/expressions/#feature-state for the documentation of the supported SDKs.

@wipfli  : Is there any plan in the roadmap to support feature state in native SDKs?


Lab 03: Design Specification Error

Summary
Design Specification Error
Description:
Hello! I am still working through Lab 03 and encountered something. Now, I still don't really understand what a Priority Queue or Comparator is, but I was just submitting stuff into CodeCrunch for fun and obtained the "Design Specification Error" when I submitted all the Java files I had. However, when I removed the Event Comparator class that implements Comparator and submitted the other remaining files, the error disappeared, and there was the usual output telling me that I did not pass the test cases (that is, the submission was still accepted unlike the submission including the Comparator class).
I am just curious about what is going wrong here and what I should be doing instead as I was under the impression that a Comparator class was required for this lab submission. Thank you in advance!

I think you do need a class that implements Comparator. There is an example in the lecture slides that you can follow. Not sure why it says Design Specification Error but maybe you can get the correct output first before submitting.
As for what a Priority Queue is, you can think of it as queue but every element in the queue has some priority value to it. And elements with higher prority value will be taken out first. In this case, you would want to take out events that have a lower time first. But if two or more events have the same time, you would want to put events with the customers that arrived first as higher priority (comes out first).
A PQ class is provided which you can use as your priority queue. The PQ takes in a class that implements Comparator. This class would then have your logic to determine which event has a higher priority.
Hope I explained it well and correct!

I also got a Design Specification Error, and it because I did check instanceof or check class type. Perhaps if you do similar things like this, it will trigger the error. And yes, you need a class that implements Comparator.

I think the design specification error is due to lack of comparator class. You need to include a comparator class and make use of it in your submission. As for the understanding of priority queue and comparator, I felt that the examples and visuals in the lab slides provided by the TAs are extremely helpful for visualization, so you may want to relook into it.

Thank you for the help everyone! I will try out your comments and work on the lab again!

After a terrible midnight marathon with the great assistance above and a proper reading of slides, indeed, a comparator class is necessary, though I think I also found out why I obtained the error --- my comparator class was missing a statement to return 0, so it seems that returning only 1 or -1 is insufficient. This is not a guaranteed diagnosis, but hopefully of any help to anyone who is reading this!


Remove help text test which causes CI hangs

The idea would be to revisit this test once we reach the next stage of our CI evolution. Right now it is unfortunately causing more trouble than it is worth.

Looks good :+1:

:cactus: :cactus: :cactus:


5 - Add Angular JS support

Add Angular JS support

Hello, recently I've worked with fine-uploader from angular and nodejs, and I want download the files (maybe saved into a db) of a user previous upload, the user is identified by an id, so for example, I did the next modifications:

In the fine-uploader.js, I change the reset function, providing a custom URL in the endpoint so the next reset execute a call to our system (an endpoint of REST API):

reset: function(endpoint) {
this.log("Resetting uploader...");

if ( typeof endpoint != 'undefined' ) {
    this._options.session.endpoint = endpoint;
} else {
    this._options.session.endpoint = null;
}

In my directive, I add a custom reset, and I add a broadcast, so for example, from any controller I can reset the directive and then call to download specific images!:

$scope.resetUploader = function(id) {
    $scope.sessionendpoint = '/getimages/' + id;
    uploader.reset($scope.sessionendpoint);
     console.log('uploader', uploader);
};

$scope.$on('resetUploader', function(event, data) {
    $scope.resetUploader(data.id);
});

Finally in controller:

$rootScope.$broadcast('resetUploader', {id: $scope.user_id});

I hope this can help you if you have the same problem !!
Thanks!!

This issue was closed almost 3 years ago. If you have a bug report or feature request, please open up a new issue. Otherwise you can post on Stack Overflow under the fine-uploader tag.


issue#2: Update barcode-scanning sdk

update to com.google.mlkit:barcode-scanning:16.1.4

The issue was fixed in 16.1.1
please check release notes here

The issue was fixed in 16.1.1
please check release notes here


Incorrect aggregation when logging predictions into EvalResult with distributed_backend DP

🐛 Bug
It it possible to log some data in EvalResult like this:
result = pl.EvalResult(checkpoint_on=val_loss)
result.y_hat = y_hat.float()

Suppose I log some value with dimensions (32, 10) - batch size 32 and 10 classes. And I have N samples in total.
If distributed_backend is None, in validation_epoch_end outputs.y_hat.shape is (N, 10)
If distributed_backend is ddp, in validation_epoch_end outputs.y_hat.shape is (N, 10)
But if distributed_backend is dp, in validation_epoch_end outputs.y_hat.shape is (N) - the values are averaged over samples.
Code sample
https://colab.research.google.com/drive/16NQaMTho5gI0URzgh3CNsZvXmHHyynvh?usp=sharing

Sorry about the delay! was wrapping up all the refactors :)
no, you have to log directly with the .log call...
result = pl.EvalResult(checkpoint_on=val_loss)
result.log('my_metric_name', y_hat)


When the Messenger window is inactive, clicking in any in...

Description:
When the Messenger window is inactive, clicking in any input field (either "Type a message" field or "Search for people and groups" field) does not activate the input field, only the window. Another click is required to activate the input field.
It would be OK if upon window activation the message input field would be automatically active, but any way the direct click on the field should work in my opinion.
This is the behaviour that can be seen by clicking on the taskbar button or alt-tabbing into the app.
Steps to reproduce:
Make another window active (any other window), click in the type a message input field of the Messenger window, see no cursor blinking.
App version:
2.0.1 beta
OS:
Windows 10
Works in WhatsApp Web:
Extra:
Keep up the good work!

Yeah this is probably the most annoying issue with this version.  Even more annoying than the right click error thing

Currently on 2.0.5 beta, this issue persists.

2.0.6 beta, issue persists.

Fixed in 2.0.7 (will be released soon): I added an item menu Window > Accept First Click

Freshly updated 2.0.9., still the same issue. It works sometimes, but not always. I haven't noticed a pattern as to when or why it sometimes works yet. The Window menu setting has no effect (behaviour is the same in both settings).

Hi, 2.0.9-stable, issue persists.

I confirm that issue still persists.
I don't see any difference by enabling the "accept first click" choice.
You can reproduce this by clicking some times once, or other times twice at another place and then go back to your app.


Dolby io gets disconnected after 3 hour

Hey There
Describe the bug
On going Dolby io conference disconnects after 3 hour.
Expected Behaviour
Conference should be connected until the Admin ends it.
Steps to Reproduce the Problem

Join Dolby io conference as an admin and Share a PPT
Join the same conference as a listener.
Wait for 3 hour the conference from Operator side disconnects automatically, so as the listener.

Specifications
OS: Windows
Version: 3.5.1
Is this some kind of timeout?
Is this something default with dolby io?
If yes can we increase the time?
Thank You

Yes, this is a timeout that you can change in your Dolby.io Dashboard. Go to your application, then click on Communications APIs, then select the tab SETTINGS, then scroll down to Conferences. There, you can change the maximum duration of the conference.

@FabienLavocat Thank you


HTML and figures

outsource html to files
fix paths (see post of charly)

[x] outsource html to files
[x] fix paths (see post of charly)
[] styling is super random
[] scripts don't work
I will most likely not manage to fix the latter issues


Set --experimental_allow_proto3_optional

By default, the call to protoc doesn't set flags like --experimental_allow_proto3_optional resulting in errors like the following:
path/to/file: This file contains proto3 optional fields, but --experimental_allow_proto3_optional was not set.

I've worked around this using this patch:
From 32a0a0f67482a9206e9b939efafa6ebb6de9ec06 Mon Sep 17 00:00:00 2001
From: Sushain Cherivirala<EMAIL_ADDRESS>Date: Thu, 6 Apr 2023 10:37:57 -0700
Subject: [PATCH] Add --experimental_allow_proto3_optional

---
 buf/internal/plugin.bzl | 1 +
 1 file changed, 1 insertion(+)

diff --git a/buf/internal/plugin.bzl b/buf/internal/plugin.bzl
index 8d684fe..a19d940 100644
--- a/buf/internal/plugin.bzl
+++ b/buf/internal/plugin.bzl
@@ -47,6 +47,7 @@ def protoc_plugin_test(ctx, proto_infos, protoc, plugin, config, files_to_includ

     args = ctx.actions.args()
     args = args.set_param_file_format("multiline")
+    args.add("--experimental_allow_proto3_optional")
     args.add_joined(["--plugin", "protoc-gen-buf-plugin", plugin.short_path], join_with = "=")
     args.add_joined(["--buf-plugin_opt", config], join_with = "=")
     args.add_joined("--descriptor_set_in", deps, join_with = ":", map_each = _short_path)
--
2.40.0

However, ideally this would either be exposed or an option (in the config?) or set based on the protoc version. I've opened a ticket rather than a PR since it's not clear what approach is preferred.

I am not sure if we should have this protoc started compiling without the flag a while ago.

What about providing an option for folks to provide extra parameters to protoc? That would permit usage in contexts where the protoc version is a bit older.

Yeah that could work, would you be able to send a PR for that?

Yep, I'll take care of it in a couple weeks.

@srikrsna-buf https://github.com/bufbuild/rules_buf/pull/50


fetchache breaks Graphql handlers which implements Cache-Control

Issue workflow progress

Progress of the issue based on the Contributor Workflow

[ ] 1. The issue provides a reproduction available on Github, Stackblitz or CodeSandbox

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
When GraphQL Handler uses Cache-control headers then fetchache caches ALL requests under same key and then completely break it because other GraphQL requests get data from cache with data from previous request.
The bug is in computing cachKey in https://github.com/ardatan/whatwg-node/blob/master/packages/fetchache/src/index.ts#L33
It's puerly URL which is in GraphQL handler always same URL.
This bug applies not only to GraphQL but all fetch related handlers where 1 URL is used for different data. typically HTTP POST.
To Reproduce
Steps to reproduce the behavior:

Create a GraphQL endpoint which set Cache-Control header.
Query it in Mesh with 1 unique query - it's cached under the URL
Query it in Mesh with another query - the response you get from cache is invalid

Expected behavior
Cache key should be computed as URL + HTTP body hash
Environment:

"@graphql-mesh/cli": "^0.78.28",
"@graphql-mesh/graphql": "^0.31.20",
"@graphql-mesh/openapi": "^0.33.22",
NodeJS: 16

Additional context
One thing is that cache key is wrongly computed. Other is that would be great to have possibility to enabe/disable/configure it. I'll create a feature request for it.

I don't think HTTP Caching should respect the body. If you disagree, could you share the relevant part of the spec?

There is no spec. This is about designing how cache works.
When 1 request comes and graphql handler performs HTTP POST with body then cache key can't be just endpoint because the endpoint's URL is always same. The another request with different fields will be then recognised as same request as pervious one because it has same URL but this is wrong. It's different request because of different fields (the post body).
Take a look how cache key is computed in apollo's server caching plugins.
https://github.com/apollographql/apollo-server/blob/main/packages/plugin-response-cache/src/ApolloServerPluginResponseCache.ts

Here is the spec;
https://httpwg.org/specs/rfc9111.html
I think the plugin you refer here is response caching plugin more than caching the incoming HTTP responses.

sure. when it's not used anymore let's close this issue as "out of date".
Thanks.


Strong-mode should fill omitted type parameters with type bound instead of dynamic?

Context: I found many usages of class Rectangle<T extends num> with the non-generic raw type Rectangle. Seems like it's treated by strong-mode as Rectangle<dynamic>, while it would seem it could / should be Rectangle<num>:
import 'dart:math';

foo(Rectangle r) => bar(r);
//                      ^
// [error] Unsound implicit cast from Rectangle<dynamic> to Rectangle<num>
bar(Rectangle<num> r) => null;

(tested with 1.16.0-dev.2.0)

cc/ @leafpetersen

This might be an instance of https://github.com/dart-lang/sdk/issues/26265 .  Are you seeing this in the IDE or the command line?  If in the IDE, can you wait until analysis stabilizes, edit the file in a way that causes re-analysis (say insert and remove a space from the middle of the word 'Rectangle'), and see if it fixes itself?

yeah and FYI, the original fix was in https://github.com/dart-lang/sdk/issues/25877

I'm seeing this in command line (dartanalyzer --strong math.dart).
Note: also happens with 1.16.0-dev.5.4, but that fix (https://github.com/dart-lang/sdk/commit/ce806bf30770bb447d077dd5e396b3a6b1906f51) seems to have made it to all 1.16.0-dev.x versions anyway.

If I define my own local Rectangle, it works: could it be that it picks up an old dart:math?
class Rectangle<T extends num> {}
// OK
foo(Rectangle r) => bar(r);
bar(Rectangle<num> r) => null;

@ochafik

Seems like it's treated by strong-mode as Rectangle, while it would seem it could / should be Rectangle

Rectangle is a short form of Rectangle<dynamic>
Q: How (and why) it should be treated asthe other type?

FYI, I'm seeing "no issues found" on your original example. Using a bleeding_edge build.

@jmesserly can you double check? I see the issue with latest dev release, and with bleeding edge analyzer.

Oh sorry! I was hitting the bug where running from the wrong directory, --strong is ignored :(

I can reproduce this using the command-line analyzer, but if I open the same file in my IDE (in a project that enables strong mode) I don't get an error. If I hover over either occurrence of r in the line
foo(Rectangle r) => bar(r);

it reports that r has the static type of Rectangle<num>. (Without strong mode enabled the static type is Rectangle<dynamic> as expected.)
To save time running this down, does anyone know where the substitution of the bounds for the missing type argument is happening?

To save time running this down, does anyone know where the substitution of the bounds for the missing type argument is happening?

TypeSystem.instantiateToBounds is where we handle the difference.
It needs to be called any time we want the "default instantiation behavior". For foo(Rectangle r), IIRC TypeResolverVisitor is the one that resolved parameter types?

IIRC, it should be happening on this line: https://github.com/dart-lang/sdk/blob/fc55cb96a85ba1185b8a7c3966c564a09b8a3105/pkg/analyzer/lib/src/generated/resolver.dart#L9747

Thanks! That's exactly where it should happen. But for reasons I don't understand yet, the type parameter for Rectangle doesn't have a bound (which explains why it isn't being used).

Oh crazy! Shot in the dark: maybe it fails to deserialize it from the SDK summary or something like that?

Thought about that. Konstantin says we're not using summaries anywhere yet. Also, I can see in the debugger where we're resolving and setting the bound (though I haven't been able to find where it's being cleared), which tells me we're not using summaries because we're following the old code path to resolve dart:math.

I believe that this is caused by a bug in the task descriptions: we're not requiring that the bounds for classes in imported libraries be resolved before resolving types in the importing library. But as Konstantin pointed out in #26265, there are also ordering issues even when the types are all defined in the same compilation unit.

Closing this as a duplicate of https://github.com/dart-lang/sdk/issues/26265 since I think that bug captures the issue here.  If there's something additional about this bug that's not captured there, feel free to reopen.


4.y.z snapshots cannot be built with the Dockerfile

After the recent changes to remove hazelcast-all, I see that we have modified the Dockerfile.
That change caused 4.y.z workflows to fail, as they were using hazelcast-all.jar, but the Dockerfile now tries to use hazelcast.jar.
I think we should make this Dockerfile somehow versioned to solve this issue (maybe create a new branch for the 4.y.z and use that branch in the workflow files on the Hazelcast side).
An example failed run: https://github.com/hazelcast/hazelcast/runs/3179147921

FYI: @kwart @frant-hartm

These could be the fixes:

hazelcast/hazelcast#19198
hazelcast/hazelcast#19199
hazelcast/hazelcast#19200
hazelcast/hazelcast#19201


autogenerated-frontdoorhealthprobesetting

Thanks for the contribution, but this resource does not make sense because the API team does not allow use to create Frontdoor sub-resources by ourselves. So let me close this PR first.


LUT-26113 : [File] - NumberFormatException can be thrown when calling getIdFile( ) in some cases

solved with a JsonIgnore anotation


dynamoDB query fails to find any match when using nested attributes in FilterExpression

I tried to find an open issue referring to this problem, but failed to, so I guess I'm not creating a duplicate. Otherwise, I apologize, I'll be happy to continue the discussion in the original issue.
For the record, I'm using version 1.3.3 of moto.
The problem I found is that when querying a table with one as KeyConditionExpression and some nested attribute condition as FilterExpression, moto fails to find any match while using a real table finds it. I guess the best way to illustrate it is with a minimal working example.
So, when executing the following code (using real dynamo, no mock ups)
dydb = boto3.resource("dynamodb")
table = dydb.create_table(
    TableName="TEST_TABLE",
    KeySchema=[{"AttributeName": "Key1", "KeyType": "HASH"},
               {"AttributeName": "Key2", "KeyType": "RANGE"}],
    AttributeDefinitions=[{"AttributeName": "Key1", "AttributeType": "S"},
    					  {"AttributeName": "Key2", "AttributeType": "N"}],
    ProvisionedThroughput={"ReadCapacityUnits": 3, "WriteCapacityUnits": 3})

while table.table_status != u"ACTIVE":
    time.sleep(1)
    table.reload()

table.put_item(Item={"Key1": "foo", "Key2": 1,
                     "details": {"type": "spam"}})
table.put_item(Item={"Key1": "foo", "Key2": 2,
                     "details": {"type": "beans"}})

result = table.query(KeyConditionExpression=Key("Key1").eq("foo"),
                     FilterExpression=Attr("details.type").eq("spam"))

result contains
{u'Count': 1,
 u'Items': [{u'Key1': u'foo',
   u'Key2': Decimal('1'),
   u'details': {u'type': u'spam'}}],
 'ResponseMetadata': {'HTTPHeaders': {'connection': 'keep-alive',
   'content-length': '114',
   'content-type': 'application/x-amz-json-1.0',
   'date': 'Mon, 09 Jul 2018 16:25:06 GMT',
   'server': 'Server',
   'x-amz-crc32': '3511637975',
   'x-amzn-requestid': '1BUUGBN4LOSU58UK7Q31CF3647VV4KQNSO5AEMVJF66Q9ASUAAJG'},
  'HTTPStatusCode': 200,
  'RequestId': '1BUUGBN4LOSU58UK7Q31CF3647VV4KQNSO5AEMVJF66Q9ASUAAJG',
  'RetryAttempts': 0},
 u'ScannedCount': 2}

On the other hand, when mocking dynamoDB executing the code inside a with mock_dynamodb(): block, the query returns zero Items, and the value of result is
{u'ConsumedCapacity': {u'CapacityUnits': 1, u'TableName': u'TEST_TABLE'},
 u'Count': 0,
 u'Items': [],
 'ResponseMetadata': {'HTTPHeaders': {u'Content-Type': u'text/plain',
   u'server': u'amazon.com',
   u'x-amzn-requestid': u'BVAY6HDZ94BNA75MPDC5DIA2NY9Y7PC9BMQQ8Q8GG6WR3VF7KCM5'},
  'HTTPStatusCode': 200,
  'RequestId': u'BVAY6HDZ94BNA75MPDC5DIA2NY9Y7PC9BMQQ8Q8GG6WR3VF7KCM5',
  'RetryAttempts': 0},
 u'ScannedCount': 2}

Am I doing something wrong? Any idea what could be going on?
Thanks!

Going to mark this as a bug

Running into the same problem here. Looks like there's an [incomplete] open PR to fix the issue.

@mgab @geoffblack Can you verify whether this has been fixed already? I can't reproduce it anymore for moto >= 1.3.10

Closing due to inactivity - please open a new issue if the issue still persists


 Should this issue be closed?

Seems from the MVP this is done? Should this issue be closed?

???
func (a *Base) TakeFood(amountOfFood float64) float64 {
	if a.floor == a.tower.currPlatFloor && !a.hasEaten {
		foodTaken := math.Min(a.tower.currPlatFood, amountOfFood)
		a.updateHP(foodTaken)
		a.tower.currPlatFood -= foodTaken
		a.setHasEaten(true)
		a.Log("An agent has taken food", Fields{"floor": a.floor, "amount": foodTaken})
		return foodTaken
	}
	return 0.0
}

The agent example I have did has a.TakeFood(10)
I assumed this was done due to this, sorry

I'm happy to make this change once #62  is closed.

I've created a PR for this https://github.com/SOMAS2021/SOMAS2021/pull/98


Pipeline Details page: Workflow section layout

Mockups
Agent item's side panel in workflow

Specifications

The workflow section in Pipeline Details page won't allow drag and drop of element in this first version and won't allow the dynamic reconnection between agents items either.
In  the pipeline creation wizzard, after the last step, the Workflow section will contain only the starting agent selected in the wizard.
Each agent item in the workflow will have a side panel with the following options:

Remove agent item from workflow: Described in #231
Insert/Add agent item to workflow: Described in #232
Configure agent item in the workflow: Described in #152

The side panel will slightly increase its size on mouse hover event.

@YanetABUC We need to add too a Button "+ Add" below the Agents, that is going to add a new slot for another Agent below, instead in the right, check the images
Ex.

@gorums Please, check this ticket again

Working as expected


Remove all unused SODA assets

This PR removes all unused SODA assets (i.e. images, code, etc.) from the repository and organizes others based on the NWB GUIDE repo structure.

This PR broke the pipeline history for me - confirmed that it still works on main

I've updated the save location from x/SODA/x to x/NWBGUIDE/x. Can you add a new file and see if it saves?

Ahh, I see, so it was only my previous history that was wiped
A new pipeline indeed saved

Note however the "No designated subtitle" - a fix for a future PR


using custom_access_token_hook works in 0.3.0 but not in any newer packages

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
When upgrading from v0.3.0 to any newer version of the ssr package it causes out custom jwt claims function to not work correctly.
I can confirm, the only thing that was changed was this package.  We followed this to modify out JWT's
https://supabase.com/docs/guides/auth/auth-hooks/custom-access-token-hook?queryGroups=language&language=sql
It works fine with v0.3.0 but when we use a later package the session cookie that is saved looks something like this
_base64encoded<what_looks_like_the_jwt_access_key> its not even an object its just literaly that as a string.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Createa project and activate the custom JWT hook and implementation
Create an account and try to login with SSR package v0.3.0
Change to package > 0.3.0 and repear
See cookie saved is not a real cookie

Expected behavior
Should save the session cookie correctly
Screenshots
System information

OS: [macOS]
Browser chrome
"@supabase/ssr": "^0.4.0" || 0.5.1
"@supabase/supabase-js": "^2.45.4"

So are you having an issue with the hook itself, or when trying to parse the session cookie?

I think the biggest issue is not that its an issue with the package but instead an issue with how you guys relay information about switching from v0.3 to v0.5. After loads of research I figured out that you have changed the cookie completely. We have many micro services including ones that are not integrated with supabase at all that just require the access_token and refresh_token from the cookie upgrading our core packages that mints JWT's to version 0..5 will require me to change every microservice we have to use the new cookie with the decoder. This should really be documented in detail somewhere

As you've probably noticed, they break the value into multiple cookies if having one is going to be too big. Are you running into issues with this?

Hi, I need to migrate from 0.3 to 0.5, is there anything I need to be aware of? can I migrate without any news? I am only using the indications in the supabase tutorials to use this dependency, I don't know if there is anything new I should modify.

Hey all,
Thanks for flagging the issue. The Custom Access Token Hook should work on the latest version of SSR. If you are facing issues do feel free to write back or open a ticket via supabase.help referencing this issue.
Thanks


Refactor: remove unnecessary action, refactor safe removal

What it solves
I was browsing the code and found this.
My pr removes unnecessary action that didn't do anything besides calling the correct one. Safes are persisted locally in the middleware (src/logic/safe/store/middleware/safeStorage.ts) It's not needed to call storage functions directly
How this PR fixes it
By using the correct action
Also, move safe reducer types to logic/safe/store folder
How to test it

Remove a safe, remove a default safe
Should work fine and the safe shouldn't be there after reloaded

Issue: If a safe was marked as "default", even when you remove it the system will keep it as default, so the next time you enter into the app it will load it.
Steps:
Copy the base URL of the app https://pr2428--safereact.review.gnosisdev.com/rinkeby/app/#/
Mark a safe as default
Remove it from its settings
Open a new tab and paste the URL
Result:
It will load the safe you removed
Expected:
It should load the "Welcome" page. This is the way is it working on dev right now

The issue with default safe reported above was fixed.
Looks good to me


Add a user setting to disable loading of workspace settings

Workspace settings override user settings, which can cause a fairly unpleasant experience if the repository owner's preferences for VSCode behavior are significantly different from the current users'. I would like a way to configure my VSCode installation such that it will obey my own user configuration, but will not load ${WORKSPACE}/.vscode/settings.json.

@jmillikin-stripe This looks completely opposite of why workspace settings are used. Do you want a way to override workspace settings? Is not it possible to change them in the workspace settings file?

I understand that workspace settings are intended for settings that are reasonable for all users. Unfortunately, there are some repositories whose owners encode more personal preferences into the .vscode/settings.json. I strongly prefer my editor's behavior to be consistent between workspaces, just as way web browsers don't change e.g. key bindings between websites, so I would like a way to completely prevent workspace settings from affecting my editor.
Ideally this would be a user preference useWorkspaceSettings or something like that, which I could toggle off.

But that means you might miss workspace settings which makes sense like insertSpaces etc. So, what you want is to override workspace settings without changing them in workspace settings?

I'd prefer the ability to completely ignore workspace settings.

Oh really?
Sent from myMail for iOS
Wednesday, 31 July 2019, 5:56 pm +1000 from<EMAIL_ADDRESS><EMAIL_ADDRESS>
I'd prefer the ability to completely ignore workspace settings.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .

Hmm I see. Sorry, I do not think we would like to provide that feature to completely ignore workspace settings. As this not a highly requested feature and we do not have plans to implement this I am closing this as out of scope


Nicify Circle's output

Installing wp-coding-standards/wpcs (2.1.0): Downloading (connecting...)Downloading (0%)           Downloading (5%)Downloading (10%)Downloading (15%)Downloading (20%)Downloading (25%)Downloading (30%)Downloading (35%)Downloading (40%)Downloading (45%)Downloading (50%)Downloading (55%)Downloading (60%)Downloading (65%)Downloading (70%)Downloading (75%)Downloading (80%)Downloading (85%)Downloading (90%)Downloading (95%)Downloading (100%)

This was mentioned to the authors of Composer: https://github.com/composer/composer/issues/8112
While they are developing this I'd like to play with sed. How to do that?
Ooops! In my fork :)

quietened it for now


Support for stackage snapshots > lts-8.0

I'd like to use the latest lts (currently 8.4) with ghcjs (see here http://stackoverflow.com/posts/42720347/), and I've been trying to use prepare-ghcjs to do this. There are a couple of changes in the latest ghcjs that need a bit of tweaking:

The inclusion of npm in build.yaml
shelly and haddock-api version bumps for the ghcjs stack.yaml - the haddock-api version required is actually broken, so we need to use something like the config here https://github.com/matchwood/prepare-ghcjs/blob/master/spec-lts/stack.yaml .

I've managed to get that far, but when ghcjs boot runs there is an error compiling base:
GHC/TopHandler.hs:88:29: error:
  Data constructor not in scope: STG_SIG_RST :: CInt

I'm happy to spend a bit of time to hack around and get a working PR but I'd really appreciate some input, as I'm not really sure where to start in terms of how to find out why this error is happening.

I've been poking around, and the issue is that lts-8.0+ use base <IP_ADDRESS> rather than <IP_ADDRESS>, so the patches luite uses have not been applied. The underlying issue is really that https://github.com/ghcjs/ghcjs-boot needs to be updated to support the packages that lts-8.0+ need, as doubtless would be obvious to anyone familiar with this area! I'll close this issue and perhaps start hacking away on ghcjs-boot instead...

Yes, it would be nice to have a branch per resolver on the ghcjs

I'll try to get a ghcjs-boot working for 8.0. As far as I can tell so far most of the work is bumping upper bounds and tweaking the existing patches - does that sound about right?

;) Yes it does.
Start with the base i.e. create a patch with diff, vanila-base-<IP_ADDRESS> -> base-<IP_ADDRESS>, then try to apply it with base-<IP_ADDRESS>... there is a lot of bumping to be done. Aeson is still not useful.

What fun!
I'm not sure what you mean by aeson not being useful though... difficult to get the lts package version in because it is too far ahead?

So I've managed to get something working.
https://github.com/matchwood/ghcjs/tree/lts-8.0 and
https://github.com/matchwood/ghcjs-boot/tree/lts-8.0
It fails a few tests ( https://github.com/ghcjs/ghcjs/issues/571) but I created an sdist tar.gz and tested it with a fresh .stack / .ghcjs and it succesfully installs and compiles reflex and a little test app.
I also hacked together a quick fix for longlinks in the cache .tar archives (https://github.com/matchwood/ghcjs/commit/99fc29dc88725c86ae479bd6c27f4e6089abf052) because they actually broke the installation process.
I'd like to make the bundle for lts-8.0 available - any ideas? I was wondering if perhaps you could host it in the same place as the others? I can make a pr request on the stack docs to refer to it - I'd hate to think of someone else having to scramble up the learning curve I've just been on, particularly when testing something like ghcjs-boot!


Fix link-color var in unraid dark & hotline

Before

After

Imho, it makes it difficult to read... 🤔

@sadstan In Hotline, yes it does. But I didn't set that color, I just fixed the variable 😅. It was doing nothing all this time.
Feel free to suggest a color for it that I can replace this one with, and I'll add that in the PR.

Try the same color as accent-color-hover. I see dark.css has the same ussue

@GilbN Still ugly :P, but readable pretty much.

@GilbN Still ugly :P, but readable pretty much.

Should I add this or change it?

Hm, just set it to white. More readable that way.


Intro content

Add education, projects, LinkedIn and GitHub links to my portfolio.

Didn’t you already merge the first two of the commits in this PR earlier? I suspect that you reused the branch without merging master back into it so the commits show up as new again here.
Did you squash & merge the last PR?

Yes, I have already merged the first two commits and I used rebase and merge for the last PR.
What should I do?


Get PSO Code Tracking Working ER:

Enhancement Request Details
As a end user I want to

Track Optimus Prime usage by calling client INFO API for every DB its importing and sizing
Should be able to see the Optimus prime in the dashboard go/pso-data-reporting

Version
No response
What oracle database version are you seeing the problem on?
No response
Which sql script version ?
No response
Relevant log output
No response

https://critique.corp.google.com/cl/453133282


Fortran optimization flags

Hello,
it seems that the fpm build options you are using do not include optimization flags (see https://github.com/fortran-lang/benchmarks/issues/5#issuecomment-808929152).
Unfortunately, fpm doesn't yet allow custom build profiles, but I do think the latest version (written in Fortran) has a --release flag, which will add at least some optimization flags.

Concerning the Fortran ODE solver interface, you could perhaps borrow some elements from the refactored dop853 code: https://github.com/jacobwilliams/dop853

Thanks for the issue ivan-pi. I think a unified ODE solver interface in F2003 for (at least) Hairer's suite would be very useful, but wasn't a goal for this little project. I had considered mirroring jacob's dop853 for dopri, but decided not to undertake that for this one-off script.
Regarding optimization flags, the fortran program was built with:
-03 -march=native -ffast-math -funroll-all-loops
Thank you very much for taking a look at this repo and providing advice on fpm. Pull requests to the fortran benchmark program are also very welcome. It's got some warts but I would be (pleasantly) surprised to see significant performance improvements.

@lindnemi Could you close this issue? Optimizations flags were used.


Types are broken for moduleResolution: Node16 and NodeNext

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
2.0.2
Wrapper Library
Not Applicable
Wrapper Library Version
None
Public or Confidential Client?
Public, Confidential
Description
reminiscent of #6151
I'm using latest typescript, my server is fully esm.
{
  "type": "module"
}

{
  "compilerOptions": {
    "module": "NodeNext",
    "moduleResolution": "NodeNext"
  }
}

and all the types from msal are broken. This is because typescript doesn't map .mjs to .d.ts naturally. So you get this, if you're looking inside:

Looking at index.d.ts, for esm the imports don't point to any typed file,

If directed explicitly to the d.ts, this is resolved

!NOTE! This is not enough, for this to work all  imports in d.ts files have to explicitly point to d.ts files
I'm not sure how to solve this, but from my personal experience I found that meme extensions are very painful to work with
Error Message
No response
Msal Logs
No response
MSAL Configuration
{}

Relevant Code Snippets
import { CryptoProvider } from "@azure/msal-node"
// or anything, really

Reproduction Steps

create modern esm node project
install @azure/msal-node
??????
profit

Expected Behavior
Typed correctly
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
None (Server)
Regression
@azure/msal-node@~2
Source
External (Customer)

This is due to your moduleResolution: NodeNext which requires relative paths to include file extensions. We do not yet compile for the newer module resolution. Marking this as a feature, for now you may need to go back to moduleResolution: node to get this to work - we also accept PRs if you'd like to take a crack at this yourself.

@tnorling I'll give it a go if you promise that the PR won't sit ignored for 2 months 😅
This is real blocker for us

@tnorling here you go mate, hope this is resolved swiftly 💪🏻


Add white ST-Link/V2 support

According to http://www.diygoodies.org.ua/?p=417 the pinout should be the following:
TCK PA5
TDO PA6
TDI PA7
TMS PB14
NRST PB0
JRST PB1

By white you mean this one?
https://img3.bgxcdn.com/thumb/large/oaupload/banggood/images/47/78/161465dd-84a2-4054-17d2-e43280df1f8c.jpg

Yup the one you gave me. I still have to verify the pinout listed above.

[ ] Make PR for code changes
[ ] Finish the JTAG test wiring board
[ ] Add documentation


Feature/move logic to packages

Related issues

closes #
Short description and why it's useful

Move core code to its own packages that can be released. This is done to clean up the imports and to in a later stage move to an template repository, so that you don't need to clone the core code into your project.
Thinks that still needs to be done are:

[x] Cleanup package.json file so it only contains the decencies that the project needs
[ ] Document the process how to build the production build

Screenshots of visual changes before/after (if there are any)

IMPORTANT NOTICE - Remember to update CHANGELOG.md with a description of your change
Contribution and currently important rules acceptance

[X] I read and followed contribution rules

I have done a small change in the image api logic that needs to be tested. There I have removed the .crop() as it not there any more.
So there we would need to have QA look over.

The fix option for image now work as before.
But I  have found one problem that I need to fix before this can be merged.
When you run yarn build:packages you need to clear it or ts-node should not load the *.d.ts files.
That is something I now try to fix.


Add your company registration number page

For this ticket: https://trello.com/c/nToxBmYI/48-new-one-time-page-build-registered-with-companies-house-page
The page

The reveal

Error messaging

Looking good to me 👍


enable current temperature sensor on Evo2

I modified the custom component to display a Current Temperature sensor for my Evo2 boilers a couple of weeks ago. Was there any reason from not having this sensor enabled for this device ? Else maybe we could bring that change upstream ?
ariston.discover() returns (amongst other things) 'wheType': 6, 'wheModelType': 22
Thanks!

https://github.com/fustom/ariston-remotethermo-home-assistant-v3/issues/224#issuecomment-1874205762

Thanks @fustom! That's what I actually started with, but then I realized I could make this temperature appear in the out-of-the-box device card in the main view of Home Assistant, which is why I decide to go for the patch.
Don't hesitate to close the PR if you think it's not worth it, e.g. because we need to figure out if all Evo2 models actually support it.

Closing to keep my list of opened PR clean!


Unbreak build on FreeBSD

reverted -Wl,--no-gnu-unique -> --no-gnu-unique because of #24

reverted -Wl,--no-gnu-unique -> --no-gnu-unique because of #24

#25 to drop the unused flag


Add reschedule command

The reschedule command takes a name and a datetime. The job is then
rescheduled to run next at that datetime. The command internally allows
for changing the period as well, but this is not exposed in the client
command (yet).
The semantics of rescheduling to a time which is now or in the past is
the same as running execute.
TODO:

[ ] ptime parsing could be improved
[ ] allow changing the period
[ ] review the ASN.1

looks mostly good to me :)

thanks


ui: fix for custom constrained offering params range check

Description
This PR fixes range check in VM deployment wizard while using custom constrained offering.
Existing code was failing while checking user input value in the form for CPU cores and RAM size when minimum and maximum value vary in digit count.
Also, while re-selecting offerings slider position was not restored to minimum value. THis has been fixed with changes.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
Entering a valid value for RAM but getting error,

Selecting custom constrained offering initially, then making changes for CPU, RAM parameters, then selecting a fixed offering and then again selecting custom constrained offering. Though text input shows minimum values but slider is not at the start.

How Has This Been Tested?

UI

@borisstoyanov @andrijapanicsb @anuragaw @rhtyd @svenvogel
kindly consider this for 4.13 milestone and please review changes

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-231

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-232


fix html formatting and add github link

@jerumbaoa   thanks for making your first contribution, I will be merging your Pull Request, also look at the issues so that you can get a chance to win a limited community Swag from me.
If you like the project and Repo please give it a star and Follow me on Github (would be appreciated)


I USE THIS !!! ❤

Owner: @Prajithp
(Deprecated) letsencrypt-cpanel
Notice
I will probably be able to do maintenance and cPanel compatibility for at least a year. As far as I am aware, nobody other than me is using this. If you ARE using this plugin, please feel free to reach out for me.

Why do you really think that someone does not use it?!
The project is currently 173 stars and is the only free project for the Cpanel Plugin. 🆓
So I use it and hopefully you continue to work on it. ❤
Also, I will be glad to don't see the word "Deprecated" at the top of the page at the next visit. 😉

I use it too @Prajithp, please continue to work on it ❤❤❤.
Best Regards,
Arian.

Me too. May we create a wildcard through this?

Hi all, is this project still alive?


Does not work in two or more fields at the same time

What could be causing the problem?
const picker = new WindowDatePicker({   el: '#picker',   inputEl: 'input[datas]',   inputToggle: true,   destroy: true });

Each picker has only one input element.


Cannot capture the correct param

The url is:
.get("/hello/*/:name/*", hello_handler)
The handler:
async fn hello_handler(req: Request<Body>) -> Result<Response<Body>, Infallible> {
    let user = req.param("*").unwrap();
    let name = req.param("name").unwrap();
    Ok(Response::new(Body::from(format!(
        "Hello {:?}, {:?}",
        user, name
    ))))
}

The response should be:
$ curl -X GET -s -i localhost:8080/hello/foo/bar/foobar
HTTP/1.1 200 OK
content-length: 21
date: Sun, 06 Jun 2021 09:09:49 GMT

Hello "foobar", "bar"

But it gives wrong result:
$ curl -X GET -s -i localhost:8080/hello/foo/bar/foobar
HTTP/1.1 200 OK
x-powered-by: Routerify v2.0.1
content-length: 18
date: Sun, 06 Jun 2021 09:14:37 GMT

Hello "", "foobar"

here you are basically saying match everything and a name parameter your syntax is incorrect
Since you say match everything with * it matches everything until the last part which is captured by :name

Sorry, I don't understand what you mean.
The * would be replaced by (.*), and :name would be replaced by ([^/]+):
https://github.com/routerify/routerify/blob/master/src/regex_generator/mod.rs#L20
After matching, the second * would overwrite the first one. So, * should be foobar, and :name should be bar:
https://github.com/routerify/routerify/blob/master/src/route/mod.rs#L127
In my repo router-lite, the result is correct, but I don't figure out why it's different from routerify because I reuse the same codes from routerify.

.* matches everything until the last part that is matched by [^/]+ and the last catchall is basically empty.
https://regexr.com/5uvoo here I made a demo
Am I missing something? this seems to be the expected behaviour

Similarly, I found another issue. Routerify returns a capture value with an unexpected tailed slash.
The url:
.get("/hello/*", hello_handler)
The handler:
async fn hello_handler(req: Request<Body>) -> Result<Response<Body>, RouteError> {
    let user = req.param("*").unwrap();
    Ok(Response::new(Body::from(format!("Hello {:?}", user))))
}

The result:
$ curl -X GET -s -i localhost:8080/hello/foobar
HTTP/1.1 200 OK
x-powered-by: Routerify v2.0.1
content-length: 15
date: Mon, 07 Jun 2021 04:08:59 GMT

Hello "foobar/"

.* matches everything until the last part that is matched by [^/]+ and the last catchall is basically empty.
https://regexr.com/5uvoo here I made a demo
Am I missing something? this seems to be the expected behaviour
You miss the slash between the group.
That is ^(.*)/([^/]+)/(.*)$, not ^(.*)([^/]+)(.*)$


will add voice search

I will add a voice search in the search bar

Sorry to close the issue, but we don't need that feature.
You can still raise new issues.


Added an index for the github repositories.

Also pushed the readthedocs files into a subdirectory.

@mbookman I added a proscriptive section at the top of the index.


How to set the initial size of the ng-Map element?

How can I set the initial size of the ng-Map? ie height and width in pixels? CSS doesn't seem to work.
Also the map doesn't show up in Firefox or Safari but works well on Chrome.....what have I missed?
Thanks

Please create a plunker example for this issue

you can add an attribute default-style="false" to ng-map. Play around with it and if you want to make it fill the container you can add this CSS ng-map {width:100%; height:100%;position: absolute;}.
Experiment on it.

Clonsing for inactivity.

If you are not able to get the map to display the correct height using default-style="false" then try adding a inline style to the map style="height: 500px; display:block; "

problem is it's 'getstyle' function gets the calculated actual height, instead of the specified height style,   which for instance, for height:100%; will be 0 pixels while the page is being built by angular... and so it gets replaced by 300px ! (and will not fill the page!)

Interesting, but is there a work around or solution to this? I want it to fill the entire height of the page.


Fixes for Java properties space and other encoding problems

Fixes #35 and #36.

The specifications of Java properties file is found in this API document: http://docs.oracle.com/javase/8/docs/api/java/util/Properties.html#load-java.io.Reader-
The strategy for escaping characters is deviated from the standard Java escape character sequence.
For a same character, it can be represented in multiple different ways. For example, carriage return code can be represented by \r or \u000a or \u000A. Single/double quote characters does not need to be escaped, thus - you can use either ' " or ' ". We need to pick one of these variations. I decided to follow the spec of Properties#store() method.
http://docs.oracle.com/javase/8/docs/api/java/util/Properties.html#store-java.io.OutputStream-java.lang.String-
http://docs.oracle.com/javase/8/docs/api/java/util/Properties.html#store-java.io.Writer-java.lang.String-
I found each of these specify a part of spec.. so you need to read both. Also, I found the actual behavior does not match the spec. It says:

Characters less than \u0020 and characters greater than \u007E in property keys or values are written as \uxxxx for the appropriate hexadecimal value xxxx.

However, the actual implementation handles following code points differently:
U+0009 -> \t
U+000A -> \n
U+000C -> \f
U+000D -> \r
All other code points, including backspace (U+0008 - "\b" is available in Java language specification as character literal, but not supported by Properties specification) are encoded by \uxxxx format.


jenkins-slave-run script fails when JAVA_ARGS is set with multiple settings

This is due to the quotes used in the script when adding $JAVA_ARGS to the $SLAVE_ARGS
/usr/bin/java '-Xms3078m -Xmx3078m -Dcom.sun.management.jmxremote -Dcom.sun.management.jmxremote.port=3000 -Dcom.sun.management.jmxremote.ssl=false -Dcom.sun.management.jmxremote.authenticate=false' -jar /opt/jenkins-slave/swarm-client-3.17.jar
resulting in following error Invalid initial heap size: -Xms3078m -Xmx3078m
Removing the quotes solves the problem.  PR underway

This was resolved in #939


Restructure UVM test with multiple files (for Cadence flow)

The main objective of this branch is to restructure the testbench_top.sv into multiple files.
In terms of UVM, the main contributions are as follows:

Agent, environment and test files organized into folders.
Implementation of the sequencer class, and package.
Implementation of test_base.

In terms of infrastructure, the main contribution is the implementation of a Makefile to run the Cadence simulation flow.
Next steps:

I haven't tested whether this structure can be used plug-and-play with EDA Playground, so it would be a good idea to investigate that further. Therefore, I suggest keeping this branch separate from the main branch for now.
I couldn't find the RTL specification to align with the UVM process flow. Do you have one, or is it deprecated?

I’m on vacation, posting this from my iPad. It’ll be a few days before I can review.

I'm not able to pull your branch into my local repo. I don't know why that is.  I can't test this code on EDA playground without that.
If there was a spec, it didn't come with the UC Berkeley code I pulled years ago.  I had to intuit the spec by code observation and applying my own experience.


Look into replacement for certain statics

Currently there are a couple statics in use in the project which could cause issues.
Element create event
Currently the static Created event is triggered on every element in the constructor. Ideally we would know which MtaServer instance an element belongs to.
A potential solution would be offering an element factory to be dependency injected, and having that trigger the event.
This does have some implications towards creating new instances of elements (not being able to use a simple new). And this would impact inheriting from element classes as well.
Element ID generation
Currently element IDs are generated using an increment static uint on the ElementIdGenerator class. While for many uses this might be fine, this limits the library to only one range of element IDs per running application. Meaning multiple instances of MtaServer would share the same element ID range.
This also limits a user's ability to sync elements only to specific clients, and being able to re-use element ID ranges between different clients. This could be particularly useful for multi-gamemode servers. Where a client would only ever need to be aware of the elements from that gamemode.
This would likely require several different options for the assigning of element IDs.
If the above solution for element creation is implemented, then this factory could be in charge of assigning element IDs. Ideally it would have some configurable way to set how element IDs are assigned. It's important that this could potentially be null in case the user wants to manually assign IDs. It would be important to check whether or not an ID has been assigned before triggering the created event. Because this might (and probably will) send entity add packets to clients, resulting in errors if the IDs are undefined.
Player join event
Currently the static event OnJoin (which should be renamed to Joined is triggered whenever a player joins the server. Just like with the element create event we would ideally be able to detect this on a per MtaServer basis.
A potential solution would be a PlayerJoined event on the MtaServer class, since instances of the Player class are always created as a result of a client connecting.

Having the MtaServer class as the primary conceptual object for events like PlayerJoined, ElementCreated etc. seems very intuitive. We should watch out that we aren't stacking these big conceptual instances, however. For example, MTA originally stacks "Server" upon "Core" upon "Game" etc. This should be avoided.


plugin: variable not initialized properly

What version of Go are you using (go version)?

$ go version
go version go1.21.0 linux/amd64

Does this issue reproduce with the latest release?

I tried to use regexp on a plugin and it fails. After a little investigation, I saw unicode.Categories is not properly initialized.
Plugin: p1/main.go
package main

import (
	"unicode"
)

func F(s string) *unicode.RangeTable {
	return unicode.Categories[s]
}

func main() {}

main.go:
package main

import (
	"fmt"
	"plugin"
	"unicode"
)

func main() {
	p, err := plugin.Open("p1.so")
	if err != nil {
		panic(err)
	}
	s, err := p.Lookup("F")
	if err != nil {
		panic(err)
	}

	f := s.(func(string) *unicode.RangeTable)
	if f("C") == nil {
		fmt.Println("unicode.Categories not properly initialized")
	} else {
		fmt.Println("unicode.Categories properly initialized")
	}
}

Execution:
$ go build -buildmode=plugin ./p1 && go run main.go unicode.Categories not properly initialized
If I add a use of unicode.Categories in the ./main.go, it is properly initialized. For example:
_ = unicode.Categories["C"].
Note: when unicode.Categories is not properly initialized, It is not nil but an empty map.
What did you expect to see?
The unicode.Categories properly initialized.
What did you see instead?
An empty map.

I observe that if you add this line anywhere inside your top-level main() function, it works:
fmt.Println(len(unicode.Categories))

Or even just add this outside the main function:
var z = unicode.Categories

Hence it appears to be something to do with the triggering (or suppression) of package initialization.  The documentation says:

When a plugin is first opened, the init functions of all packages not already part of the program are called.

You have already imported "unicode" to access the type unicode.RangeTable, but possibly because you're not accessing any values this has suppressed initialization?

Hum, I have not been able to reproduce this behavior with another variable.
If I add  a init function using unicode.Categories in the plugin, there is no reason to remove its initialization.
func init() {
	fmt.Println(len(unicode.Categories))
}

But it prints 0.
$ go build -buildmode=plugin ./p1 && go run main.go
0
unicode.Categories not properly initialized

Yes, what matters is whether the main program initializes unicode (which you can do using "var z = unicode.Categories" in the top-level main.go)
As a complete guess:

Main program thinks: "I'm not using any variables from unicode, I don't need to initialize it"
Plugin thinks: "main program already links with unicode, I don't need to initialize it"

I modified your reproducer to demonstrate a similar issue with the regexp library as reported on golang-nuts:
==> go.mod <==
module example.com

go 1.20

==> main.go <==
package main

import (
	"plugin"
	"regexp/syntax"
)

func main() {
	if syntax.Perl != 212 {
		panic("Unexpected flags")
	}

	p, err := plugin.Open("p1.so")
	if err != nil {
		panic(err)
	}
	s, err := p.Lookup("F")
	if err != nil {
		panic(err)
	}

	f := s.(func())
	f()
}

==> p1/plugin.go <==
package main

import (
	"regexp"
)

func F() {
	_ = regexp.MustCompile(`\w+`)
}

func main() {}

$ go version
go version go1.21.0 darwin/amd64
$ go build -buildmode=plugin ./p1 && go run main.go
panic: regexp: Compile(`\w+`): error parsing regexp: invalid escape sequence: `\w`

goroutine 1 [running]:
regexp.MustCompile({0x107aa6eea, 0x3})
	/usr/local/go/src/regexp/regexp.go:319 +0xb4
example.com/p1.F()
	/Users/brian/tmp/go/62430/p1/plugin.go:8 +0x1f
main.main()
	/Users/brian/tmp/go/62430/main.go:23 +0x103
exit status 2

cc @thanm
I haven't looked into this carefully, but I guess this is related to the map initialization change. My guess (without looking into details) is that as the main program imports the unicode package but not explicitly use the map, it initialize the unicode package but not the map. When the plugin is loaded, it sees the unicode package is already initialized by the main program, it doesn't initialize it again, leaving the map uninitialized. Maybe we need to initialize maps for programs that can use plugins.

Supporting what @cherrymui said, I bisected to e2ca417ee797e1e80d8d395e0f4760416b8346d9
https://go.dev/cl/463395

Thanks for the report. I will take a look.

The regexp case is almost certainly also due to an uninitialized map, in regexp/syntax/perl_groups.go
var perlGroup = map[string]charGroup{
        `\d`: {+1, code1},
        `\D`: {-1, code1},
        `\s`: {+1, code2},
        `\S`: {-1, code2},
        `\w`: {+1, code3},
        `\W`: {-1, code3},
}


Installation issue

I have tried now several times to install rags, but I always get this error message:
(base) kalle@MacBook-Air rags % poetry install --with dev
Installing dependencies from lock file
No dependencies to install or update
Installing the current project: rags (0.0.5)
The current project could not be installed: No file/folder found for package rags
If you do not want to install the current project use --no-root
Any suggestions?

Same problem!

Same problem!

found a solution https://github.com/run-llama/rags/issues/39#issuecomment-1832278349


CaffeFunction doesn't load global average pooling correctly

In case a caffemodel fed to CaffeFunction contains global average pooling layers, these layers have global_pooling: true parameter but no window size values are stored in the caffemodel file.
However, current CaffeFunction implementation (specifically, definition of _stup_pooling in caffe_function.py) doesn't take care of this value (more specifically param.global_pooling should be checked within the code), and eventually generated AveragePooling2D object has 0 in both kw and kh. As the result forward pass cannot be run.
Here is the minimum reproduction code and the actual error I got.
(test caffemodel DenseNet_161.caffemodel can be downloaded from https://github.com/shicai/DenseNet-Caffe )

Chainer: 2.0.2
Cupy: 1.0.2
Python: 3.6.1 (using pyenv, anaconda3-4.4.0)
OS: Ubuntu 16.04 x64
CUDA: 8.0
cuDNN: 5.1

import cupy
import chainer
from chainer.links import caffe

net = caffe.CaffeFunction("/path/to/DenseNet_161.caffemodel")
net.to_gpu(0)
x = cupy.random.random((1, 3, 224, 224)).astype(cupy.float32)
net(inputs={'data': x}, outputs=['fc6'])

Full error message:
% CHAINER_CUDNN=0 python reproduction.py
  File "reproduction.py", line 8, in <module>
    net(inputs={'data': x}, outputs=['fc6'])
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/links/caffe/caffe_function.py", line 208, in __call__
    output_vars = func(*input_vars)
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/links/caffe/caffe_function.py", line 558, in __call__
    return self.func(x, *self.args, **self.kwargs)
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/functions/pooling/average_pooling_2d.py", line 144, in average_pooling_2d
    return AveragePooling2D(ksize, stride, pad, False)(x)
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/function.py", line 200, in __call__
    outputs = self.forward(in_data)
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/function.py", line 327, in forward
    return self.forward_gpu(inputs)
  File "/home/foobar/.pyenv/versions/anaconda3-4.4.0/lib/python3.6/site-packages/chainer/functions/pooling/average_pooling_2d.py", line 41, in forward_gpu
    coeff = 1. / (self.kh * self.kw)
ZeroDivisionError: float division by zero

(CHAINER_CUDNN=0 is set just to avoid an independent problem; this particular caffemodel has too small eps value in BN than required by cuDNN.)

bump