Unable to Run Sample

Hello,
I am trying to run the sample but looks like the API is not working with backtrader.
I created a new conda enviroment with python 3.6 in mac and install these packages with pip
alpaca-backtrader-api     0.3
alpaca-trade-api          0.24
backtrader                <IP_ADDRESS>
This line is showing error after I put the key.
store = alpaca_backtrader_api.AlpacaStore(
key_id='',
secret_key='',
paper=True
)

Also, I uncomment alpaca_backtrader_api.AlpacaBroker()  alpaca_backtrader_api.AlpacaData and try to use them directly show authorization error even I have set this in my .bash_profile and source it.
export APCA_API_BASE_URL="https://paper-api.alpaca.markets"
export APCA_API_KEY_ID="mykeyid"
export APCA_API_SECRET_KEY="mysecret"
However, if only using alpaca_trade_api just works to prove that my environment variable setup is good.
import alpaca_trade_api as tradeapi
api = tradeapi.REST()
account = api.get_account()
print(account.status)
Do you have any ideas ?

having the same problem here. have you been able to resolve?

Hi,
The reason @davidkwong's code does not work is because the code cannot read your key_id and secret_key. You will need to either one of the following ways:

Copy your APCA_API_KEY_ID and APCA_API_SECRET_KEY into the store initialization, or
You will need to load those values from the os environment. The way to do this is by the following

import os
...

store = alpaca_backtrader_api.AlpacaStore(
  key_id=os.getenv('APCA_API_KEY_ID'),
  secret_key=os.getenv('APCA_API_SECRET_KEY'),
  paper=True
)

Hope this help @ndenys650 too!

work under fix: https://github.com/alpacahq/alpaca-backtrader-api/pull/22.
it will be merged soon

code has been merged and a new version released. https://github.com/alpacahq/alpaca-backtrader-api/releases/tag/v0.4


Issue with dialogflow connector:  no such type: google.rpc.ErrorInfo

Describe the bug
I am trying to run nlpanalytics with dialogflow, but I am getting an error during validation:
Starting training. Validation training failed: no such type: google.rpc.ErrorInfo
To Reproduce
botium-cli nlpanalytics validate --config ./config/dialogflow.botium.json --train ./convos/db-train --test ./convos/db-test --output ./results/validate-dialogflow.csv --outputPredictions ./results/validate-dialogflow-predictions.csv
My config contains:
    "botium": {
        "Capabilities": {
            "PROJECTNAME": "Botium NLPAnalytics Dialoflow",
            "CONTAINERMODE": "dialogflow",
            "RATELIMIT_USERSAYS_MINTIME": 1000,
            "RATELIMIT_USERSAYS_MAXCONCURRENT": 1,
            "DIALOGFLOW_NLP_PROJECT_ID": "****l",
            "DIALOGFLOW_NLP_CLIENT_EMAIL": "****",
            "DIALOGFLOW_NLP_PRIVATE_KEY": "****"
        }
    }
}

Logs:
I tried to print the stack trace if that helps:
Error: no such type: google.rpc.ErrorInfo
    at Root.lookupType (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/protobufjs/src/namespace.js:382:15)
    at FallbackErrorDecoder.decodeProtobufAny (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:35:32)
    at /home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:50:56
    at Array.map (<anonymous>)
    at FallbackErrorDecoder.decodeRpcStatus (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:50:37)
    at FallbackErrorDecoder.decodeErrorFromBuffer (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:63:46)
    at /home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallback.js:313:57
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  note: 'Exception occurred in retry method that was not classified as transient'
} Error: no such type: google.rpc.ErrorInfo
    at Root.lookupType (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/protobufjs/src/namespace.js:382:15)
    at FallbackErrorDecoder.decodeProtobufAny (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:35:32)
    at /home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:50:56
    at Array.map (<anonymous>)
    at FallbackErrorDecoder.decodeRpcStatus (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:50:37)
    at FallbackErrorDecoder.decodeErrorFromBuffer (/home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallbackError.js:63:46)
    at /home/hala/anaconda3/envs/py3/lib/node_modules/botium-connector-dialogflow/node_modules/google-gax/build/src/fallback.js:313:57
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

Botium Flavour:

Botium CLI

Will have to take a look. Will come back as soon as I have hints.

Will have to take a look. Will come back as soon as I have hints.

I think you are using this repository: https://github.com/codeforequity-at/botium-samples-nlpanalytics ?
I updated the NPM-dependencies there, and I suggest you try to either do a fresh clone, or you remove the package-lock.json file and the node_modules directory and do an npm install again afterwards.
Please give me feedback if this helped.

I think you are using this repository: https://github.com/codeforequity-at/botium-samples-nlpanalytics ?
I updated the NPM-dependencies there, and I suggest you try to either do a fresh clone, or you remove the package-lock.json file and the node_modules directory and do an npm install again afterwards.
Please give me feedback if this helped.

Thanks for your reply.
Yes I am using  https://github.com/codeforequity-at/botium-samples-nlpanalytics
I tried both removing package-lock.json file and the node_modules directory and recloning the repo. But I'm still getting the same error
Starting training.
Validation training failed: no such type: google.rpc.ErrorInfo

Thanks for your reply.
Yes I am using  https://github.com/codeforequity-at/botium-samples-nlpanalytics
I tried both removing package-lock.json file and the node_modules directory and recloning the repo. But I'm still getting the same error
Starting training.
Validation training failed: no such type: google.rpc.ErrorInfo

then maybe a matter of the Node.js version ? what version are you using ?

then maybe a matter of the Node.js version ? what version are you using ?

npm: '6.14.5'

npm: '6.14.5'

I just upgraded node to v15.6.0 and npm to 7.4.0
still getting the same error :(

I just upgraded node to v15.6.0 and npm to 7.4.0
still getting the same error :(

Honestly, no idea. Is there happening anything unusual when running npm install ?
You can try to use the Botium CLI Docker image, everything is pre-installed in there.

Honestly, no idea. Is there happening anything unusual when running npm install ?
You can try to use the Botium CLI Docker image, everything is pre-installed in there.

The issue turned out to be that API calls are not enabled for my dialogflow project.

The issue turned out to be that API calls are not enabled for my dialogflow project.


Best Build Practices: "overly complex" is vague and miscategorized

"Overly complex" is vague and seems untestable. Also in my experience, "complex" tends to be the union of multiple people's "simples", that is complexity arises from supporting the relative simple use cases of many people, so complexity always seems unnecessary from any single person's vantage point.
This also seems mis-categorized under Build Practices.

Our partners at Cyber ITL consider the complexity of code to be a characteristic of the structure of the code, for example related to branch density and number of stack adjusts, function calls, etc. For two examples of code which have the same functionality (i.e, cover the same set of use cases), the code which is structurally less complex will be easier to manage and vet for security.
Sarah - any ideas for this one?

This wording is talking about "spaghetti code".  It's a long-recognized fact of programming that increased complexity makes code harder to review, debug, and maintain, and thus introduces more bugs.  That's why JPL and NASA require limitations to complexity of code that's going into satellites or other critical systems.  We're going to work on refining which complexity indicators correlate strongly with exploit-ability, but it's definitely a build issue - better written code is less complex and easier to read and maintain.

@citl-s and @mrerecich what do you guys think--does this thread need more discussion or can we close it? Thanks!

paging @citl-s and @mrerecich--do we need more discussion?


sum.golang.org: go-get gitern.com/xxx/repo.git 410 gone

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

$ go version
go version go1.14.6 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOENV="/home/user/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/user/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/tmp/trash/go.mod"
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-build897724545=/tmp/go-build -gno-record-gcc-switches"

What did you do?
go get gitern.com/urandom/errors.git

What did you expect to see?
go: downloading gitern.com/urandom/errors.git v0.0.0-20200723020514-49681dcbcc21

What did you see instead?
go: downloading gitern.com/urandom/errors.git v0.0.0-20200723020514-49681dcbcc21
go get gitern.com/urandom/errors.git@master: <EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone
        server response:
        not found<EMAIL_ADDRESS>invalid version: git fetch -f origin refs/heads/*:refs/heads/* refs/tags/*:refs/tags/* in /tmp/gopath/pkg/mod/cache/vcs/c58e8953608705577f340d5d716eb03f031f4f09a3698dde27ce667a4baf661a: exit status 128:
                fatal: unable to connect to gitern.com:
                gitern.com[0: <IP_ADDRESS>]: errno=Connection refused

The repo appears correctly populated in ~/go/pkg/mod/cache, implying that proxy.golang.org is working, plus hitting vcs directly, via GOPRIVATE=gitern.com, works as expected.

You have GOPROXY="https://proxy.golang.org,direct", which means the go command will fall back to direct fetching if proxy.golang.org fails.
This is a strange one. It seems to be failing very consistently. Offhand the only thing I can think is that gitern.com is blocking the Google egress IPs that proxy.golang.org connects from. You may want to ask them if that rings any bells. I'll try to dig a little  more when I get a chance.

cc @hyangah @katiehockman

cc @huumn

I’m not blocking any ips explicitly. If they’re being blocked on my end they’re being blocked by AWS.

All repos are private on gitern which could require additional config for go get?

Would go-get be falling back to direct for the zipball, which I have manually configured via config to use auth, then failing to find an entry in the sumdb?

@arnottcr gitern does not support git over http currently (only ssh). I think go get defaults to http and might need to be told to use ssh.

The .git suffix causes the toolchain to try http and ssh (maybe git:// too), so i can confirm that part is working.
It would be nice to get meta header support for gitern, but that is out of scope here, and I still think there might be a go-get bug.

For private repositories, please see this instruction.
https://golang.org/cmd/go/#hdr-Module_configuration_for_non_public_modules

@arnottcr I just saw you already tried GOPRIVATE and it works. There is no way sum.golang.org can access private repositories (it shouldn't, right? it's private!) GOPRIVATE is the way to opting out of using proxy.golang.org and also sum.golang.org for the specified modules.
I will close this since this is working as intended.

This ux is really confusing. Can we have the toolchain detect these failurse and suggest GOPRIVATE? We use it at work, and I am familiar with what it does, but it was not clear why go get was failing.

No, especially since I still don't know why it's failing. Connection refused isn't a reasonable error to get for a private repository; it can't possibly know whether we can authenticate without accepting the connection. My best guess is still a misconfiguration on the origin side.
At the same time, there is no point spending time diagnosing it if it's just going to fail later, so I'm not going to investigate any further.

I guess my ask is when the module proxy falls back to direct, can we disable sumdb checking? I get that verification is important, but the ux is confusing, and if proxy.golang.org (or whatever module proxy you have configured) cannot reach out, why would we expect the sum.golang.org to do any better.
Since gitern may be making things more confusing, i am happy to repro with a private github repo.

Automatically disabling sumdb lookup would destroy the security model; any attacker that controlled the network could disable sumdb lookup by blocking access to proxy.golang.org.
Again, this is not a general problem with private modules. For example, I believe GitHub returns a warning about authentication that helps avoid confusion. The problem here is that network connectivity between proxy.golang.org and the origin is broken. There may be some room to improve the error message when sumdb fetching fails after a successful proxy download, but that would need to take into account the user's particular proxy/sumdb configuration and should definitely be discussed in a separate issue.

Automatically disabling sumdb lookup would destroy the security model; any attacker that controlled the network could disable sumdb lookup by blocking access to proxy.golang.org.

If you are concerned about this why even have the ,direct option? IIUC, it will always fail, since packages that do not exist in proxy.golang.org will not have entries in sum.golang.org. If we include ,direct it needs to do something, can you walk me through a ux that does?
Furthermore, if you are concerned with network level interception, you likely have larger problems.

Again, this is not a general problem with private modules. For example, I believe GitHub returns a warning about authentication that helps avoid confusion. The problem here is that network connectivity between proxy.golang.org and the origin is broken. There may be some room to improve the error message when sumdb fetching fails after a successful proxy download, but that would need to take into account the user's particular proxy/sumdb configuration and should definitely be discussed in a separate issue.

The ux for github private repos is not better, but this may be the anti-discovery mechanism at work:
[user@localhost ~]$ go get github.com/arnottcr/private
# cd .; git clone -- https://github.com/arnottcr/private /home/user/go/src/github.com/arnottcr/private
Cloning into '/home/user/go/src/github.com/arnottcr/private'...
fatal: could not read Username for 'https://github.com': terminal prompts disabled
package github.com/arnottcr/private: exit status 128

Also the toolchain disallows the use of github.com/owner/repo.git, so it does not directly translate: (really unsure why this is disallowed)
package github.com/arnottcr/private.git: invalid version control suffix in github.com/ path


Methodology documentation

Hello.
Thanks for the package!
Is the methodology documented somewhere? I see there is a reference to  "Korn Ferry Institute" that points to their main website, but cannot see anywhere the methodology used.

Sorry, after posting I saw that something similar was already been asked in https://github.com/BenWiseman/sentiment.ai/discussions/5 It would be good, in any case, to have a proper link to some actual document in the main page, instead of the generic one to the  Korn Ferry Institute.
Cheers!


Bug: Overlay while renaming the file name

Description
While renaming any file, the interface is so transparent that it shows:

File type, for e.g. PNG, and
File size, for e,g. 1.58 MB

This makes it difficult to rename and check what is we typing there.

Steps To Reproduce

Select a file to rename
Choose the "rename" option
Type to rename and there you can see the issue before finishing renaming the file.

Requirements
It should not be so transparent that it overlays on currently writing the names of the file.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2506
Log File
debug.log

Thanks for the report, Are you using the tiles layout?

Looks like Tiles, can you repro in any other layouts as well?

This should be resolved in the next release.

Thanks for the report, Are you using the tiles layout?

Yes


Upgrade Vertx to version 4.4.6

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test camel-knative camel-platform-http-vertx camel-reactive-executor-vertx camel-threadpoolfactory-vertx camel-vertx/camel-vertx-common camel-vertx/camel-vertx-http camel-vertx/camel-vertx-websocket camel-vertx/camel-vertx


What camera intrinsic used for finetuning the gray bg Zero123Plus?

Thanks for the amazing work. I noticed that after finetuning to white bg, the output image has a large scale:

Do you use different intrinsics when finetuning? Would large scale output better for the reconstruction stage?

@cwchenwang Hi, we aimed to strictly follow the camera setting of Zero123++ v1.2 (fov=30) during fine-tuning. We asked the authors of Zero123++ about the object normalization and camera distance in this issue. The original answer was that the object should be normalized into a unit cube (it has been correced to unit sphere), which was an unintentional mistake resulting in larger objects in the rendered image.
This will not influence the reconstruction results in most cases. However, if the shape of the object is close to a cube, it will occupy a very large region in the generated image and make the reconstruction result cropped since it exceeds the [-1, 1] representation range of the triplane. To alleviate this issue temporarily, you can parse a smaller --scale argument in run.py to decrease the size of the reconstructed object. We plan to fix the object normalization issue and provide a new model in the future.

@bluestyle97  thanks a lot for open-sourcing the codebase for fine-tuning the zero123++ models.
I am also facing some issues while rendering the 6 views for a 3d model in blender.
It would be great if you could tell more about the creation of training dataset for zero123++ models:
Specifically the camera_distance and normalization of the 3d model for blender rendering.
I am modifying the blender script from here
but for some objects the views doesn't look like zero123++ (scaling issues, camera distance issue).
I am using this for validation views in blender:
def set_camera_location_validation(camera, view_i):
    # cam_distance = (0.5 / np.tan(np.radians(30/2))) # not sure if this is correct
    cam_distance = 2.0
    azimuths = np.array([30, 90, 150, 210, 270, 330])
    elevations = np.array([20, -10, 20, -10, 20, -10])
    azimuths = np.deg2rad(azimuths)
    elevations = np.deg2rad(elevations)

    x = cam_distance * np.cos(elevations) * np.cos(azimuths)
    y = cam_distance * np.cos(elevations) * np.sin(azimuths)
    z = cam_distance * np.sin(elevations)

    camera.location = x[view_i], y[view_i], z[view_i]

    # adjust orientation
    direction = - camera.location
    rot_quat = direction.to_track_quat('-Z', 'Y')
    camera.rotation_euler = rot_quat.to_euler()
    return camera

normalize_scene function with box_scale=2 # should this be unit cube

Any suggestions would be super helpful
cc: @cwchenwang
Thanks :D

@bluestyle97 thanks a lot for open-sourcing the codebase for fine-tuning the zero123++ models. I am also facing some issues while rendering the 6 views for a 3d model in blender. It would be great if you could tell more about the creation of training dataset for zero123++ models: Specifically the camera_distance and normalization of the 3d model for blender rendering. I am modifying the blender script from here but for some objects the views doesn't look like zero123++ (scaling issues, camera distance issue).
I am using this for validation views in blender:
def set_camera_location_validation(camera, view_i):
    # cam_distance = (0.5 / np.tan(np.radians(30/2))) # not sure if this is correct
    cam_distance = 2.0
    azimuths = np.array([30, 90, 150, 210, 270, 330])
    elevations = np.array([20, -10, 20, -10, 20, -10])
    azimuths = np.deg2rad(azimuths)
    elevations = np.deg2rad(elevations)

    x = cam_distance * np.cos(elevations) * np.cos(azimuths)
    y = cam_distance * np.cos(elevations) * np.sin(azimuths)
    z = cam_distance * np.sin(elevations)

    camera.location = x[view_i], y[view_i], z[view_i]

    # adjust orientation
    direction = - camera.location
    rot_quat = direction.to_track_quat('-Z', 'Y')
    camera.rotation_euler = rot_quat.to_euler()
    return camera

#for normalization using normalize_scene function
normalize_scene(box_scale=2) # this is unit cube normalization , maybe sphere normalization is required

# camera_setup
cam.data.lens = 30 # 24 default for openLRM?

Any suggestions would be super helpful cc: @cwchenwang Thanks :D

Hi, have you found a proper scale to reproduce the results as shown in the InstantMesh? Thanks!

@mengxuyiGit
I think updating these parameters in blender script from openlrm can generate the images that are consistent with zero123++ 6 view images:
# Cam setup
fov = 49.13
cam.data.lens = 49.13
cam_distance = (0.5 / np.tan(np.radians(fov/2)))

# Cube normalization -> Sphere normalization
scale = box_scale / max(bbox_max - bbox_min) -->
scale = box_scale / np.linalg.norm(bbox_max - bbox_min)

# Random normalization
normalize_scene(box_scale=0.8)

@msingh27 Thanks for sharing this! DId you just find this out by trial and error? Because in zero123plus they seem to use the following:
Output views are a fixed set of camera poses:
   Azimuth (relative to input view): 30, 90, 150, 210, 270, 330.
   v1.1 Elevation (absolute): 30, -20, 30, -20, 30, -20.
   v1.2 Elevation (absolute): 20, -10, 20, -10, 20, -10.
   v1.2 Field of View (absolute): 30°.

And since Zero123plus fine-tuning is using the elevation of v.1.2 I would assume FOV is also the one of v.1.2?


Add new parameters to GET_VEHICLE_NUMBER_OF_PASSENGERS.

These parameters are, respectively, "include driver" and "include dead peds." They should be, respectively, false and true to mirror previous behavior. Source: my own reverse-engineering.

This has fixed an issue with Vehicle.Occupants in API v2, where it could crash if a vehicle's occupants were dead.

LGTM, because redundant arguments will just not used in native functions iirc and the commit won't run into ploblems in earlier versions.
We really appreciate this PR. As far as I see the first commit for b2545 in alloc8or's Native DB, the change has really made. Didn't expect some native paramater changes broke compatibility with new versions until this PR was created.


We hope that cesium can support the I3S 3D model of 4490. Please inform us if there is any solution. thank you

hi,We hope that cesium can support the I3S 3D model of 4490. Please inform us if there is any solution. thank you

Thanks for the request @iEdric.
@Tamrat-B What all would be involved in adding support for additional projections for I3S?

I'm closing this issue due to inactivity. If you believe this is still an issue, please feel free to re-open. Thanks!


Menu collection + dropdown item = error

Steps

Go to http://react.semantic-ui.com/collections/menu#size
On size examples, open language dropdown, click outside then try to open it again.

Expected Result
Opening the dropdown. :)
Actual Result
Uncaught TypeError: a._dropdown.blur is not a function(…)
Version
0.56.12
Chrome 54.0.2840.71 (64-bit)

Duplicate #659

Released in<EMAIL_ADDRESS>


makes errors for indexing on columns and rows uniform

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
This cleans up the error checks so they are more useful. For the range checks we say whether its the columns or rows so things like like
matrix[..., min_max] assign columns: accessing element out of range. index 5 out of range; expecting index to be between 1 and 4

The also one place for min_max where I had the max range and value to check in the wrong order and had to flip them.
Intended Effect
Make the error reporting a little nicer
How to Verify
A few additional error tests were added to the lvalue and rvalue checks to try to grab a few extra cases we were missing before
Side Effects
Documentation
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Steve Bronder
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@t4c1 can you take a quick look at this?

@bbbales2 I don't have merge permissions on this repo for some reason can you click the merge button?


Chore/mobile popup

This PR makes the PopUp better on mobile.

The popup is now on the bottom of the screen on mobile
The createOmbul popup is styled for mobile
The system for choosing image for a cmsImage is now styled for mobile (it is a popup)

On landscape on mobile the close button is not visible. And if the pop up has a lot of content maybe a overflow-y: auto is smart, to enable scroll.

On landscape on mobile the close button is not visible. And if the pop up has a lot of content maybe a overflow-y: auto is smart, to enable scroll.

I could not reproduce the problem with the close button. But what I can do is set max-height: 90svh on the popup content

I guess you mean overflow: scroll

@Paulijuz  can you approve this?


[Suggestion] pass method's property descriptor to parameter decorators

It would be very convenient if a parameter decorator could modify the implementation of the function the parameter is declared in. Currently, the suggested way to do this is to have two decorators: a parameter decorator to add metadata to the function, and a method decorator to modify the function based on that metadata (see this example in the TypeScript Handbook, which requires the use of both a "validate" and "required" decorator to implement parameter checking). Not only does this require the inclusion of the reflect-metadata library, it is also quite unwieldy when just a single decorator should suffice.
I originally tried to accomplish this by altering the prototype of the target in the parameter decorator:
function myParamAttribute(target: Object, propertyKey: string | symbol, parameterIndex: number) {
    let method: Function = target[propertyKey];

    target[propertyKey] = function () {
        arguments[parameterIndex] = convert(arguments[parameterIndex]);
        return method.apply(this, arguments);
    }
 }

Unfortunately, the __decorate method that TypeScript emits restores the method's property descriptor after calling the decorators, so my changes were undone. I have found a workaround that works well by redefining the __param function...
var __param = function (paramIndex, decorator) {
    return function (target, key, descriptor) { decorator(target, key, paramIndex, descriptor); }
};

... and then implementing my decorator as such:
function myParamAttribute(target: Object, propertyKey: string | symbol, parameterIndex: number, descriptor?: TypedPropertyDescriptor<Function>) {
    let method = descriptor.value;
    descriptor.value = function () {
        arguments[parameterIndex] = convert(arguments[parameterIndex]);
        return method.apply(this, arguments);
    }
}

This is obviously sub-optimal, as it mucks with TypeScript's internals and is hardly future-proof. So it would be great if the property descriptor were passed to parameter attributes officially.

The decorator proposal is undergoing some changes now. we will not be chaining the TS implementation until the proposal has stabilized and then we can revisit this proposal.


View can be a react component

class MyPresenter extends Presenter {
    view = MyView
}

Neat!  👍

Coverage increased (+0.001%) to 99.789% when pulling e6ff52693e34242f5c0440ee8988a3402c47df4a on presenter-views into 15387f0e96b50f76a2b345157fd61a96ac21a4bc on master.


Fix old timestamp issue

Fix failing tests because of old timestamp

Codecov Report

Merging #148 (1f371b6) into master (349a1b5) will increase coverage by 3.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   93.47%   97.46%   +3.98%
==========================================
  Files           6        6
  Lines         552      552
  Branches      146      146
==========================================
+ Hits          516      538      +22
+ Misses         36       14      -22

Impacted Files
Coverage Δ

contracts/Uniswap/UniswapAnchoredView.sol
98.19% <0.00%> (+15.31%)
:arrow_up:

contracts/OpenOraclePriceData.sol
100.00% <0.00%> (+23.80%)
:arrow_up:

Codecov Report

Merging #148 (1f371b6) into master (349a1b5) will increase coverage by 3.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   93.47%   97.46%   +3.98%
==========================================
  Files           6        6
  Lines         552      552
  Branches      146      146
==========================================
+ Hits          516      538      +22
+ Misses         36       14      -22

Impacted Files
Coverage Δ

contracts/Uniswap/UniswapAnchoredView.sol
98.19% <0.00%> (+15.31%)
:arrow_up:

contracts/OpenOraclePriceData.sol
100.00% <0.00%> (+23.80%)
:arrow_up:


[ScreenRuler]Allow moving the UI by dragging

Microsoft PowerToys version
0.73.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Screen ruler
Steps to reproduce
After running the app and bringing up the ruler, I'm unable to navigate its position, When I need to change its position I exit it and then reopen it, instead of having the ability to move it.
✔ Expected Behavior
being able to change the container box of the screen ruler, some prior versions had the advantage. but it's been disabled!
❌ Actual Behavior
The running screen ruler container is paralyzed in navigation
Other Software
No response

when I try to drag the element that contains ruler options, it doesn't move from its position!

I see, you meant you want to move the UI. I don't recall this ever being allowed to be honest. Anyway, marked the issue as an enhancement. Thanks for opening it.


Bug Report: Preloading Characters doesn't work and songs crash

When Preloading Characters is enabled, on game start, characters don't load!
When I head over to any of FNF' songs when Preloading is enabled, the game just crashes.

True.

already fixed


Figures in quiz M7.04, Question 2

In FUN, We can't have figures in the question. So I had to modify Question 2 of M7.04 Quiz.
Tell me if it's ok for you (captures below)

LGTM except that there was an extra "the" in the instructions "select the which statements". I fixed it in both gitlab and github.

Thanks @ArturoAmorQ , it's corrected


Opening Kaoto's embedded source code editor throws exceptions

Description
The Kaoto's embedded source code editor throws an exception in the vscode-kaoto extension.
This is particularly relevant since this might incur performance penalties.
From the docs

Why all these web workers and why should I care?
Language services create web workers to compute heavy stuff outside of the UI thread. They cost hardly anything in terms of resource overhead and you shouldn't worry too much about them, as long as you get them to work (see above the cross-domain case).

Steps to reproduce

Open a file using the vscode-kaoto extension
Open the chromium dev tools (ctrl + shift + i)
Open Kaoto's embedded source code editor
Notice the warning and errors thrown in the console

Notes
related issue: https://github.com/microsoft/vscode/issues/87282

using web workers in VS Code Webview: https://code.visualstudio.com/api/extension-guides/webview#using-web-workers

obsolete, the integration of Kaoto 2 does not have an embedded editor


Propose for static DSL is actually implemented in terms of generate

The real propose for the static IR generative functions is simply not yet implemented, but as a workaround for common cases, it currently calls generate. However, in the presence of custom generative functions that use custom internal proposals (e.g. a nested particle filter for use in particle MCMC), this will give wrong weights.

Fixed in https://github.com/probcomp/Gen/commit/1383248a4dd241a475ec11d96ad3236f646dc939


desktop on debian stretch

Running latest version at the time of this writing.
stretch-minimal-pine64-bspkernel-0.7.30-140.img.xz
Obtained scripts:
cd
git clone https://github.com/longsleep/build-pine64-image.git
cd build-pine64-image/simpleimage/platform-scripts
Install desktop.  Issue: could not find packages:
./install_desktop.sh mate

Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package ubuntu-mate-core
E: Unable to locate package ubuntu-mate-desktop
E: Unable to locate package ubuntu-mate-lightdm-theme
E: Unable to locate package ubuntu-mate-wallpapers-xenial

I too have the same issue.  any update on this?


have a --cleanup option

While making changes to the configuration, it's easy to end up in a weird state where configuration and cf state mismatch.
I'd like to have a --cleanup option that is going to cleanup "our" mess in CF configs (ip lists and firewall rules).

Fixed via https://github.com/crowdsecurity/cs-cloudflare-bouncer/commit/098596813bc48e9f8bb041e04d173c52421890d6


action_user created with incorrect to= model

Hello,
I have in my Django settings file:
AUTH_USER_MODEL = 'karaage.Person'

If I try to update this model to use django-audit-log, it gives me a migration containing the following field:
('action_user', audit_log.models.fields.LastUserField(related_name='_person_audit_log_entry', editable=False, to='karaage.PersonAuditLogEntry', null=True)),

However, this is incorrect, it should point to karaage.Person, like for the other tables, eg:
('action_user', audit_log.models.fields.LastUserField(to='karaage.Person', null=True, editable=False, related_name='_projectlevel_audit_log_entry')),

This results in errors such as the following one:
  File "/home/brian/tree/karaage/karaage-working/karaage/people/models.py", line 137, in save
    super(Person, self).save(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/django/db/models/base.py", line 589, in save
    force_update=force_update, update_fields=update_fields)
  File "/usr/lib/python3/dist-packages/django/db/models/base.py", line 626, in save_base
    update_fields=update_fields, raw=raw, using=using)
  File "/usr/lib/python3/dist-packages/django/dispatch/dispatcher.py", line 198, in send
    response = receiver(signal=self, sender=sender, **named)
  File "/usr/lib/python3/dist-packages/audit_log/models/managers.py", line 107, in post_save
    self.create_log_entry(instance, created and 'I' or 'U')
  File "/usr/lib/python3/dist-packages/audit_log/models/managers.py", line 102, in create_log_entry
    manager.create(action_type = action_type, **attrs)
  File "/usr/lib/python3/dist-packages/django/db/models/manager.py", line 92, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/django/db/models/query.py", line 372, in create
    obj.save(force_insert=True, using=self.db)
  File "/usr/lib/python3/dist-packages/django/db/models/base.py", line 589, in save
    force_update=force_update, update_fields=update_fields)
  File "/usr/lib/python3/dist-packages/django/db/models/base.py", line 613, in save_base
    update_fields=update_fields)
  File "/usr/lib/python3/dist-packages/django/dispatch/dispatcher.py", line 198, in send
    response = receiver(signal=self, sender=sender, **named)
  File "/usr/lib/python3/dist-packages/django/utils/functional.py", line 17, in _curried
    return _curried_func(*(args + moreargs), **dict(kwargs, **morekwargs))
  File "/usr/lib/python3/dist-packages/audit_log/middleware.py", line 51, in _update_pre_save_info
    setattr(instance, field.name, user)
  File "/usr/lib/python3/dist-packages/django/db/models/fields/related.py", line 597, in __set__
    self.field.rel.to._meta.object_name,
ValueError: Cannot assign "<SimpleLazyObject: <Person: Test User1>>": "PersonAuditLogEntry.action_user" must be a "PersonAuditLogEntry" instance.

Any ideas?
Regards

Oddly enough, e6781837267773bba25338fe33f9198e235bf26b changes the behaviour so that an error is generated instead:
karaage.PersonAuditLogEntry.action_user: (fields.E300) Field defines a relation with model 'karaage.Person', which is either not installed, or is abstract.

Which is odd, I don't see anything in this commit which would have had such a significant change.
Still investigating...

Defining a forward reference from PersonAuditLogEntry to Person here is not as easy you might imagine, for reasons I don't really understand.

Karage.Person does not work.
apps.get_app_config('karaage').get_model("Person") does not work.

If we had a class reference that would work (as this is how copy_fields works), but we don't have a class reference. Only the AUTH_USER_MODEL setting value.

I feel I should contribute here as https://github.com/Atomidata/django-audit-log/pull/16 is where I proposed a fix for this very issue.  The pull request was never merged...

Hi Brian,
I'm not sure that the PersonAuditLogEntry.action_user field should be a foreign key to Person, it kind of makes sense that it should relate to the PersonAuditLogEntry model instead.  Deleting a Person shouldn't cause all related $FooAuditLogEntry to be cascade deleted, should it?
Regards,
Tyson.

Just for the record, I believe that #16 was applied, just not exactly the same way as in the merge request.
@tysonclugg Was going based on what happens for all other classes, where action_user is is a reference to Person. Like it or not. I think consistency is more important here, unless you want to change every action_user.
Cascaded deletes can be disabled by setting on_delete=models.SET_NULL.
Oddly enough, on some circumstances, action_user was being setup with Person not PersonAuditLogEntry, as mentioned above I was getting this error with the latest Master version:
karaage.PersonAuditLogEntry.action_user: (fields.E300) Field defines a relation with model 'karaage.Person', which is either not installed, or is abstract.

This is something I still don't really understand, it should be set to self, which becomes PersonAuditLogEntry not Person. I might try to investigate this in more detail now.

I believe #16 was closed by 529327db618d63dccbce6f682cb27ec99c6997af

It seems to be that under certain circumstances the code introduced by #16 doesn't get AUTH_USER_MODEL value correctly (it gets the default value instead), and as a result, the condition fails, and the self value is never used. Still trying to understand why this is.

Ok, one mystery solved. In audit_log/models/managers.py we have:
from django.conf import settings
from audit_log import settings

i.e. the settings value is being overridden. The audit_log/settings.py file does not pull in the value for the AUTH_USER_MODEL setting, it only defines the DISABLE_AUDIT_LOG setting.
Now I understand why e6781837267773bba25338fe33f9198e235bf26b broke things.

Yesterday I made the following comment, but it accidentally ended up in the wrong place - it ended up in the commit, not here.
In get_logging_fields there is currently the following code:
        #check if the manager has been attached to auth user model
        if [model._meta.app_label, model.__name__] == getattr(settings, 'AUTH_USER_MODEL', 'auth.User').split("."):
            action_user_field = LastUserField(related_name = rel_name, editable = False, to = 'self')

As far as I could tell this issue is fixed with the patch from @ghinch. Can @brianmay confirm?

No, problem still not resolved, just one of the issues I noticed along the way. We need to work out in action_user should point to the Personobject (makes it consistent with established schemas) or the PersonAuditLogEntry object (as per what @tysonclugg  said before). At the moment we set the schema to use PersonAuditLogEntry but pass it a Person value which is broken:
======================================================================
ERROR: test_password_reset_by_self (karaage.tests.test_people.PersonTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/brian/tree/django/karaage/karaage/karaage/tests/test_people.py", line 472, in test_password_reset_by_self
    response = self.client.post(url, form_data, follow=True)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/test/client.py", line 512, in post
    secure=secure, **extra)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/test/client.py", line 313, in post
    secure=secure, **extra)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/test/client.py", line 379, in generic
    return self.request(**r)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/test/client.py", line 466, in request
    six.reraise(*exc_info)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 132, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/utils/decorators.py", line 145, in inner
    return func(*args, **kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/views/decorators/debug.py", line 76, in sensitive_post_parameters_wrapper
    return view(request, *args, **kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/views/decorators/cache.py", line 57, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/contrib/auth/views.py", line 246, in password_reset_confirm
    form.save()
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/contrib/auth/forms.py", line 290, in save
    self.user.save()
  File "/usr/lib/python2.7/dist-packages/model_utils/tracker.py", line 134, in save
    ret = original_save(**kwargs)
  File "/home/brian/tree/django/karaage/karaage/karaage/people/models.py", line 137, in save
    super(Person, self).save(*args, **kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/base.py", line 710, in save
    force_update=force_update, update_fields=update_fields)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/base.py", line 747, in save_base
    update_fields=update_fields, raw=raw, using=using)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/dispatch/dispatcher.py", line 201, in send
    response = receiver(signal=self, sender=sender, **named)
  File "/home/brian/tmp/django-audit-log/audit_log/models/managers.py", line 110, in post_save
    self.create_log_entry(instance, created and 'I' or 'U')
  File "/home/brian/tmp/django-audit-log/audit_log/models/managers.py", line 105, in create_log_entry
    manager.create(action_type = action_type, **attrs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/manager.py", line 127, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/query.py", line 348, in create
    obj.save(force_insert=True, using=self.db)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/base.py", line 710, in save
    force_update=force_update, update_fields=update_fields)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/base.py", line 734, in save_base
    update_fields=update_fields)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/dispatch/dispatcher.py", line 201, in send
    response = receiver(signal=self, sender=sender, **named)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/utils/functional.py", line 17, in _curried
    return _curried_func(*(args + moreargs), **dict(kwargs, **morekwargs))
  File "/home/brian/tmp/django-audit-log/audit_log/middleware.py", line 55, in _update_pre_save_info
    setattr(instance, field.name, user)
  File "/home/brian/tmp/python/local/lib/python2.7/site-packages/django/db/models/fields/related.py", line 627, in __set__
    self.field.rel.to._meta.object_name,
ValueError: Cannot assign "<SimpleLazyObject: <Person: Test User1>>": "PersonAuditLogEntry.action_user" must be a "PersonAuditLogEntry" instance.

----------------------------------------------------------------------
Ran 1 test in 1.695s

Possibly this bug was fixed in c168ef241d967fef0236dff709b6d4dbf6ccd5d2?

No, it's not fixed yet. I was making the unit tests catch this bug as I was starting to fix it. But I've yet to work out a fix.


Bump core testing to v19.0.0rc1

I updated this PR for the stable release, so both the production and testing docker files are changed in this PR.


too many hreflang generation

Hi i can't finde answer on documentation
the plugin create 4(!) hreflang !!
How to fix this ???
 locales: [           {             name: "lang",             code: "ua",             iso: "ua-FR",             file: "ua-FR.js"           },           {             name: "lang2",             code: "fr",             iso: "fr-FR",             file: "fr-FR.js"           }         ],
and i got in header
link data-n-head="ssr" data-hid="alternate-hreflang-ua" rel="alternate" href="http://web.work/" hreflang="ua" link data-n-head="ssr" data-hid="alternate-hreflang-ua-FR" rel="alternate" href="http://web.work/" hreflang="ua-FR" link data-n-head="ssr" data-hid="alternate-hreflang-fr" rel="alternate" href="http://web.work/ua" hreflang="fr" link data-n-head="ssr" data-hid="alternate-hreflang-fr-FR" rel="alternate" href="http://web.work/ua" hreflang="fr-FR"
but i need only
link data-n-head="ssr" data-hid="alternate-hreflang-ua-FR" rel="alternate" href="http://web.work/" hreflang="ua-FR"
link data-n-head="ssr" data-hid="alternate-hreflang-fr-FR" rel="alternate" href="http://web.work/ua" hreflang="fr-FR"

Those are catch-all locales and it works as designed and is explained at https://i18n.nuxtjs.org/seo/#feature-details.

there is no answer there

The documention has a link to https://support.google.com/webmasters/answer/189077 that specifying that's a recommended practice to include those catchall locales (search for "catchall" on that page).

so, 4 hreflang it good ? and "works as designed and is explained at https://i18n.nuxtjs.org/seo/#feature-details." ???

Did you even read my last comment?

yes

i stell can't remove hreflang="en"

What's the issue you are trying to fix? You just don't like it being there? Then don't use $nuxtI18nHead and create your own meta from scratch.
As I said, it's by design and recommended by google.


Add HTTPie

[X] I have checked that this app does not already have a pending request and has not previously been declined.
App Name: HTTPie
Download URL: https://github.com/httpie/httpie/releases/
Website URL: https://httpie.io/
App Summary: HTTPie for Terminal — modern, user-friendly command-line HTTP client for the API era. JSON support, colors, sessions, downloads, plugins & more.

It seems that this issue does not fit in the "Packages that install directly from the official Debian or Ubuntu apt archives will not be accepted."  rule. HTTPie is packaged in Ubuntu https://packages.ubuntu.com/search?keywords=httpie and debian https://packages.debian.org/search?keywords=httpie


00:60 left

This picture says it all:

The issue is utils.ts in secondsToMMSS care of wonderful floating point math. Like the number coming in is something like 59.8 seconds, which is less than 60 so
Math.ceil(seconds % 60)
Since we're modding a floating point, seconds % 60 can yield 0 through 59.999..., so running Math.ceil is bringing it up to 60.
Probably best to run ceiling on the initial seconds before calculating minutes and then use round instead of ceil for calculating secondsForFormatting.
That will also fix the fact that the minute went down to 0 before reaching 59 seconds.

Fixed in 0.0.17


Update README.md

URLS trailing slashes best practice for directories and description of the TAC acronym's meaning

First tiny contribution to the README.md of lfenergy GH :)


Add sample code in Migration Guide

API change check
API changes are not detected in this pull request.


Keep getting killed

I use denseflow to computer tvl1 with images, i.e. enable "input-frames" argument. The process always gets killed by my system:

No flow images generated after a few hours. Is there any way to limit the burden from denseflow so it won't be killed by the system?
By the way, according to the info from "nvidia-smi", there are lots of free space in GPU. Actually, it seems that only one of two GPUs has been used for denseflow, can I set the gpu number for denseflow?

how about host memory? how about reduce the batch size to a smaller number, like 32? https://github.com/open-mmlab/denseflow/blob/032c2fd5548595a2c25082aa257777a8a85517ac/include/dense_flow.h#L95

how about host memory? how about reduce the batch size to a smaller number, like 32?
https://github.com/open-mmlab/denseflow/blob/032c2fd5548595a2c25082aa257777a8a85517ac/include/dense_flow.h#L95

I am not sure what does host memory means. Anyway, I solved this problem by splitting images folders into several small groups. Then do denseflow on each groups which only contains images of about 10 videos (originally there are images of 60 videos in destination directory).
Thanks for your kind attention and suggestion!

the usual memory for cpu, i.e., not the gpu memory

Cpu memory usage is 100%, and after cpu memory get full, the swap memory increase. Finally after swap memory get full, the computer will start to kill denseflow.
I manually changed the value of batch_maxsize to 128 in file dense_flow.h. While in the print log information during executing denseflow still show batch_size=512. Of course, the getting killed problem still exists.
I downloaded the denseflow with setup. So I actually revise the file: app/src/denseflow/include/dense_flow.h. Not sure if this is the correct way. And I use denseflow with some py file in mmaction2 repo

yeah that is the issue. change that line would help

you can mimic the setup script https://github.com/innerlee/setup/blob/master/zzdenseflow.sh#L30-L32 , and remember to set the ENV variable https://github.com/innerlee/setup/blob/master/zzdenseflow.sh#L5

the binary is install in ~/app/bin by default

you can mimic the setup script https://github.com/innerlee/setup/blob/master/zzdenseflow.sh#L30-L32 , and remember to set the ENV variable https://github.com/innerlee/setup/blob/master/zzdenseflow.sh#L5

Do you mean after I change the dense_flow.h file, I need to cmake denseflow again to make the change works?

make and make install is ok
it depends on where you changed the code.
if it was the ~/app/src/denseflow, then make is enough.
for a freshly cloned repo, you need start from the cmake

make and make install is ok
it depends on where you changed the code.
if it was the ~/app/src/denseflow, then make is enough.
for a freshly cloned repo, you need start from the cmake

Cool! That works! Now cpu memory usage gets to a maximum of 40%. Thank you very much.


Inline mode on telegram always responding with error, even when working right

While using my bot on inline mode it always responds with:

@Fer Galván Sorry, MemeBot is having a problem responding right now.

Even when it indeed works. What could be happening?
An example of a requisition in inline mode would be like: "spongebob meme, hello, world, please"
Thanks for your help
My bot, for testing:
https://telegram.me/The_MemeBot

Are you using ngrok by chance? Looking at the Telegram logs, I see this error "Operation returned an invalid status code 'BadGateway'". This looks suspiciously like the error returned by ngrok when it can't forward a message to an app. I wonder if you have an ngrok address set as your bot endpoint, but your bot isn't running locally.

Just for testing, right now is on azure

Does it work on other channels? Is it only having issues on Telegram?

It is only having issues in the inline mode in telegram, it works fine in facebook, skype and web, even in telegram when not called in inline mode but in direct chat

I just solved it, it seems that the problem is that while the user is contructing the inline request the bot can throw a "bad request exception", just returning 'null' for that requests solves the problem


shallow: true not working with dynamic routing

Bug report
When I try to push a new route with the option { shallow: true } it trigger getInitialProps ignoring the option.
The route is a dynamic route (with custom server) it's something like /handling-request/:id/:tab
I need the param tab to get direct to the selected tab, but I also need to update the url when the tab change.
If you have any suggestion/workaround I'm open to new solutions.
That's the code:
handleTabChange = (e, value) => {
	this.setState({ tabIndex: value });
	const { id } = this.props.router.query;

	Router.replace(`/handling-request?id=${id}&tab=${value}`, `/handling-request/${id}/${value}`, {
		shallow: true,
	});
};

(I'm using material-ui-react tabs)
System information

OS: macOS 10.14
Browser: Chrome
Version of Next.js: 7.0.2

I've also checked issues: #4545 #3322

I've solved the issue with a workaround, simply use query params without a custom server integration, just like below:
Router.replace(`/path-to-route?param=${param}&param2=${param2}`, `/path-to-route/${param}/${param2}`, { shallow: true })

And then in the page wrap the component with withRouter
import { withRouter } from 'next/router';

class YourAwesomeComponent extends React.Component {
	render () {
		const { query } = this.props.router
		return <h1> {query.param} {query.param2} </h1>
	}
}

export default withRouter(YourAwesomeComponent)

In my case the usage of a custom route on server.js was unuseful, but what if I need to do custom stuff in my custom route?

https://github.com/zeit/next.js/blob/canary/packages/next-server/lib/router/router.js#L208-L212
https://github.com/zeit/next.js/blob/canary/packages/next-server/lib/router/router.js#L370-L377
Shallow routing works by matching the href's path to a page inside the pages directory to the current page that's rendered. If they're the same page shallow will be used, if not we do a normal router.push/router.replace.
You can read more on how the router works here:
https://github.com/zeit/next.js/issues/2833#issuecomment-414919347


Please rename the table

run_as_root_message says nothing about what this table contains
please rename it

Hi Vlad, thank you for your suggestion.
The changes was applied in the PR #36


TCP proxy with SNI support

Hello,
I have a single server with one Public IP and 10 domains. For each domain I’d like to have a separate docker container as an email server (Postfix + Dovecot). I’d like to achieve this with transparent TCP proxy with SNI support.
I'd like to route traffic from example.com on ports 587 & 143 to one container and traffic for acme.com on ports 587 & 143 to a different container, etc.
Can I achieve this by using host:port? Does anyone know of an example?
Thank you.

I haven't used Fabio in the recent past, but yes, this should be very much possible.
Are the containers registered in Consul? If they are being registered in Consul, the Consul tags should be setup to make the traffic come to each "backend domain".
If not Consul, I think the file backend could be used (but I haven't used the file backend much).
I would like to know too if you are able to solve this.


Support for Cloud database

Sorry to open another issue.
Correct me if I'm wrong, but with this plugin would I need to submit an App Update to a live app in the App Stores to add new targets? Is it possible to use a cloud database to manage targets?

Hi Remi,
At the moment there's no support for the Vuforia cloud database service. This plugin was created for a brochure recognition system that detected certain pre-defined pages.
At the moment if you want to change targets it would require an update on the app store.
At the moment this is out of scope for the current plugin. If you wanted to get around this, you could either create a pull request, or, write some code to check your own server to see if new targets are available and if they are, download them to your device. Once on the device you should them be able to change your call to start Vuforia passing in the new data and XML files.

No problem, I just thought I'd ask before moving forward. Thanks,

You're welcome 😃


Implement responsive design for mobile

fixes #135
LiveURL: https://kiwicomsmart-faq-qrauehifew.now.sh

just two observations:
help not centred:

text area too big:

@jalbertsr they should be resolved now :-)

@git-toni It took a while until I figured out the issue with the 'Back' button... Turns out an invisible part of the help header was overlapping it :man_facepalming:
With that, it seems everything is resolved :-)

when using it on desktop now I see this:

Oh!

At least apparently this only happens when logged out. ( :roll_eyes: :joy_cat: )

Looks like now.sh passed too :-)

And fyi I removed the Sign In buttons. Martin confirmed they have to be removed https://skypicker.slack.com/archives/C9K87QN6Q/p1526643257000054

Ok, it looks really cool, good job man! 👍


Advanced hunting API page missing limit on max query result size in bytes

If a query result is too large (in byte size, not rows), it will return HTTP 400 Bad Request with the message "Query execution has exceeded the allowed result size. Optimize your query by limiting the amount of results and try again". This value is not documented, but it appears to be limited to approximately 130 million bytes or 124MB. Please ensure all limitations with the advanced hunting API query are documented.

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

ID: 53a61a50-f7f2-f09a-a008-759c3afba97b
Version Independent ID: 872c0e23-c626-3e22-efea-d4ef4ad7f32b
Content: Advanced Hunting API
Content Source: microsoft-365/security/defender-endpoint/run-advanced-query-api.md
Product: m365-security
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@MastaDisasta Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5970
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MastaDisasta Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


KAFKA-17108: Earliest pending upload offset

This is the first part of the implementation of KIP-1023
The purpose of this pull request is for the broker to start returning the correct offset when it receives a -6 as a timestamp in a ListOffsets API request.
Added unit tests for the new timestamp.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Should we need to add an OffsetSpec on the client side to make the earliest_pending_upload_offset queryable from command line tools similar to 6e32448?

@kamalcph I can make it as part of a separate change. Will that be ok?

@kamalcph @OmniaGM addressed your comments. PTAL again.

@abhijeetk88
Please rebase this PR with trunk.


Fixed issue about tag capitalization

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closes #18209
@scissorsneedfoodtoo , #19747

@thecodingaviator, thank you for taking the time to create a new PR for this. But it's still not triggering the build for some reason! Maybe it has something to do with the commit history? @raisedadead, do you know what's going on here?


Change midX and midY to be at x and y when frame is 0

Although it is technically not correct, it makes way more sense to me this way.
Also it fixes #375

Makes more sense to me too.


Reduce size by golfing down some pieces 🏌♂ (-43 B)

Summary:
9f6f0fa Move all component diff'ing logic into component diff condition (-6 B)
b547990 Remove dom parameter from diff() (-11 B)
8928b2f Golf createContext (-6 B)
1f7002b Remove unnecessary code from diffChildren (-16 B)
1baa6bb Simplify originalDom check (-4 B)
Total: -23 B
Total: -43 B

Coverage remained the same at 100.0% when pulling 8928b2f55fb1f34d207b9123abc6498090252304 on putt-putt into 3396e3e3f887995522e32bbee57924ed5c9224e7 on master.

The last commit broke two tests regarding hydration behaviour in compat. They test that text nodes are changed on mismatch during hydration. That behaviour is listed in the docs: https://reactjs.org/docs/react-dom.html#hydrate . If we can't find a way around that I'm more than happy to land this PR in master without the last commit :)

I'm looking at it now

Reverted the commit and merged in master. Travis should be green now.

FYI: Just rebased the PR against master


Tags edit proposal

@Valexr ready or some more polishing is needed?

@Valexr ready or some more polishing is needed?

I think a lot will change with time.
For now it's ok.


Added one new programming language

@Ishaan28malik , If everything is done right, then please merge this small change.

@KmrAnish04 yup

Thanks @Ishaan28malik  for considering my contribution.


Renaming a code snippet should update json file name

A code snippet metadata json file name is derived from display_name and used as id for metadata requests.
Updating the display_name does not update its file name, causing issues such as

Not being able to create a new code snippet containing the same display_name as some previous one that had this field already updated.
Being able to see multiple code snippets with same display_name updated after initialization

UI Steps for issue 1 above

Create a code snippet named test (save & close)
Edit snippet name to test1 (save & close)
Create a new code snippet named test again (try to save & close)
Error dialog:

UI Steps for issue 2 above

Create a code snippet named test (save & close)
Edit snippet name to test1 (save & close)
Create a new code snippet named test1 (save & close)

Possible solutions

Rename json file when updating display_name field (preferred for filename readability reasons)
Use a hash (UUID) when generating metadata file name (best practice, but can be hard to navigate through metadata files when needed)

The display_name field has no bearing on the actual filename except (possibly) during creation when name is not provided per https://github.com/elyra-ai/elyra/pull/629.  Following the instance's creation, both display_name and name are distinct and only updates to name will affect the storage location of the instance.
Had the initial instance creation used a display_name that contained spaces, uppercase characters, or other punctuation, the name field would not match even after creation.  It just so happens that in this case, none of the additional criteria that would cause the two properties to differ existed and so it appears that the display_name is what dictates the storage location.
Based on this, it seems like the frontend needs to have awareness of the name field and check its set of instances for conflicts - or let the server check.  Not sure how it would expose name if the user truly wants to rename the file.  Seems like it would need to expose name from the onset - although not necessarily require a value during creation.
I agree that we should introduce an id system property that is a UUID for the instance.  I would prefer, though, that the storage persistence layer be the thing that decides how it's used.   We're using named instances in the filesystem to facilitate better manual administration (per your last comment on UUIDs).


Prevent error snapshotting image with visible Scintilla markers

Change #add: to #nextPut:. Fixes #893.

I should've added a test for this when I changed it, which would have picked up the break. I'll do that.


Link from Readme.md not foun

Link from Readme.md file is not valid - https://getakka.net/articles/Logging - The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

Looks like this didn't get converted when we moved over to DocFX. The link should be https://getakka.net/articles/utilities/logging.html

Thanks. I will leave this issue open for you not to forget to update readme file.


retention: Update to account for the zulipinternal realm.

In https://github.com/zulip/zulip/pull/12823 some changes to the realms
structure have been made, so now both in production and development
cross-realm bots live in the realm with string_id "zulipinternal".
There was a TODO in retention code to eliminate a conditional in a query
that became redundant with this change, and also the zulipinternal realm
should be omitted from the archiving process in archive_messages().

Btw, https://github.com/zulip/zulip/issues/11015 should be updated to mark points 2 and 4 as done.

So, just thinking through the behavior change here -- I think this would make system bot private messages never be removed via retention policy?  I think we'll eventually want them to be removed based on the retention policy of their recipients.

Ah, this got forgotten :see_no_evil: System bot PMs still get removed in another query in that function:
    # Archive cross-realm personal messages to users in the realm:
    # Note: Cross-realm huddle message aren't handled yet, they remain an issue
    # that should be addressed.
    query = """
    INSERT INTO zerver_archivedmessage ({dst_fields}, archive_transaction_id)
        SELECT {src_fields}, {archive_transaction_id}
        FROM zerver_message
        INNER JOIN zerver_recipient ON zerver_recipient.id = zerver_message.recipient_id
        INNER JOIN zerver_userprofile recipient_profile ON recipient_profile.id = zerver_recipient.type_id
        INNER JOIN zerver_userprofile sender_profile ON sender_profile.id = zerver_message.sender_id
        WHERE sender_profile.id IN {cross_realm_bot_ids}
            AND recipient_profile.realm_id = {realm_id}
            AND zerver_recipient.type = {recipient_personal}
            AND zerver_message.date_sent < '{check_date}'
        LIMIT {chunk_size}
    ON CONFLICT (id) DO UPDATE SET archive_transaction_id = {archive_transaction_id}
    RETURNING id
    """
    message_count += run_archiving_in_chunks(query, type=ArchiveTransaction.RETENTION_POLICY_BASED,
                                             realm=realm,
                                             cross_realm_bot_ids=cross_realm_bot_ids,
                                             realm_id=realm.id, recipient_personal=Recipient.PERSONAL,
                                             check_date=check_date.isoformat(), chunk_size=chunk_size)

This change doesn't alter how things work, only eliminates a redundant condition from the query.

Merged as cbdfef28a8b2b7604602e2d6d26fa5077dae51f6 after adding a comment explaining the SYSTEM_BOT_REALM bit.  Thanks @mateuszmandera!


Remove flipv code from glb loader

Once we are able to automatically reconvert affected glb models (or users are easily able to do this themselves), we must reintroduce the change in https://github.com/playcanvas/engine/commit/dcbfd71c36e6ee1b1adbaa3d4c8a12fdcf0f159d (reverted in #4097).

@slimbuck Reapply the commit for v2.0.0?


deparse altered "CREATE TABLE" statement, removed WITHOUT TIME ZONE

Original query (with fingerprint 2c0d1f29c30f5177):
  CREATE TABLE public."AO_563AEE_ACTIVITY_ENTITY" (
    "ACTIVITY_ID" bigint NOT NULL,
    "ACTOR_ID" integer,
    "CONTENT" text,
    "GENERATOR_DISPLAY_NAME" character varying(255),
    "GENERATOR_ID" character varying(450),
    "ICON_ID" integer,
    "ID" character varying(450),
    "ISSUE_KEY" character varying(255),
    "OBJECT_ID" integer,
    "POSTER" character varying(255),
    "PROJECT_KEY" character varying(255),
    "PUBLISHED" timestamp without time zone,
    "TARGET_ID" integer,
    "TITLE" character varying(255),
    "URL" character varying(450),
    "USERNAME" character varying(255),
    "VERB" character varying(450)
  )

Deparsed query (with fingerprint 13401b610a7be108):
CREATE TABLE public.AO_563AEE_ACTIVITY_ENTITY (
 	ACTIVITY_ID bigint NOT NULL,
	ACTOR_ID int,
	CONTENT text,
	GENERATOR_DISPLAY_NAME varchar(255),
	GENERATOR_ID varchar(450),
	ICON_ID int,
	ID varchar(450),
	ISSUE_KEY varchar(255),
	OBJECT_ID int,
	POSTER varchar(255),
	PROJECT_KEY varchar(255),
	PUBLISHED timestamp,
	TARGET_ID int,
	TITLE varchar(255),
	URL varchar(450),
	USERNAME varchar(255),
	VERB varchar(450)
);

On field "PUBLISHED", the type was changed from timestamp without timezone to timestamp.

To reproduce:
const fs = require('fs')
const { parse, deparse } = require('pgsql-parser');

async function main() {
    const stmts = parse(`
        CREATE TABLE public."AO_563AEE_ACTIVITY_ENTITY" (
    "ACTIVITY_ID" bigint NOT NULL,
    "ACTOR_ID" integer,
    "CONTENT" text,
    "GENERATOR_DISPLAY_NAME" character varying(255),
    "GENERATOR_ID" character varying(450),
    "ICON_ID" integer,
    "ID" character varying(450),
    "ISSUE_KEY" character varying(255),
    "OBJECT_ID" integer,
    "POSTER" character varying(255),
    "PROJECT_KEY" character varying(255),
    "PUBLISHED" timestamp without time zone,
    "TARGET_ID" integer,
    "TITLE" character varying(255),
    "URL" character varying(450),
    "USERNAME" character varying(255),
    "VERB" character varying(450)
  )
    `)
    const stmts2 = deparse(stmts[0])
    console.log(stmts2)
}

main()

Schema at https://github.com/prisma/database-schema-examples/tree/main/postgres/jira

what I'm suggesting is that, the deparser is not a bijective function, in other words It's lossy. So I think the fingerprints of two different queries should each be unique, however, once they are passed through the parser/deparser and back, they would only ever produce one query and thus one fingerprint.

Got it, thank you for taking the time to explain.


Shutter control

I would like to control my 2 shutters (rolluiken) in HA. At this moment I'm using KaKu ASUN650 modules, ICS2000 and for each shutter a switch to control it locally. I tried the Integration, but I think it's more for controlling lights?
Could you help me on the way?
Best regards,
Herman van Calker

Right now the lights are the only thing supported, you could try to extend it, the heavy lifting has been done by describing the auth/device polling.
You could checkout: https://developers.home-assistant.io/docs/core/entity and/or https://developers.home-assistant.io/docs/creating_component_index to get started. I'm looking into adding switches myself as the switches are now defined as lights.


How to include tinymce or any other scripts correctly now?

Hello, sorry about that question.
my actions:
npm install --save tinymce
then, in app.module.ts I paste
import 'tinymce';
and then I create files with directive of tinymce and etc.
But after building, it doesnt work:
404 GET http://localhost:3000/themes/modern/theme.js
404 GET http://localhost:3000/plugins/charmap/plugin.js
404 GET http://localhost:3000/plugins/print/plugin.js
404 GET http://localhost:3000/plugins/preview/plugin.js

TinyMCE works only after I add that code in index.html
<script src="//cdn.tinymce.com/4/tinymce.min.js"></script>
According my purposes in future, I dont wanna add that code line in INDEX.HTML. I want to include all code of tinymce to my bundle code of project (without downloading script from internet resources, only localhost use)
How to include plugins like tinymce correctly (without angular-cli)? Again, sorry for that offtopic. I read wiki page about "how to import jquery", but don't get how use it with tinymce.

solved with
https://github.com/tinymce/tinymce/issues/2836#issuecomment-282254616
and
https://www.ephox.com/blog/angular-2-and-tinymce/


[Fix] fix rtl support

Dear utatti,
Thank you for youe awesome library, i have encountered rtl issue and i have read issues #689 #713 #660
so i fixed this issue and now it's fully working as expected in both ltr, rtl
i followed your contribution guide and run tests
Thank Bro and i hope my contribution to this awesome library will be useful.

I liked it very much. Niice


Show the intro video

Overtone should be able to run the intro video from the game (namely, THING2/INTRO.AVI).
As the video uses an old codec and I don't want to mess with any video codecs (yet), I suggest we use an approach similar to the original: just use the external video player to show the video.
So, we'll need a program that extracts the video file from the ISO provided, and runs it via the OS facilities (something like xdg-open/open on *nix, and a similar facility on Windows).

How will you know when the video is done playing?

vlc -f /Volumes/TONE116E/THING2/INTRO.AVI vlc://quit && echo "success" seems to be working and allowing me to chain when done. Thanks to StackOverflow https://stackoverflow.com/a/69476995/1548557

How will you know when the video is done playing?

In the original game, as I see, there's no need to know that the video has ended playing. There was just a Start menu item to start the video, then that's it. The player would see the video, close the player after it ends, and start the game themselves.
Of course, we can improve this. I am thinking about embedding the video player into the game somehow eventually.
Generally, if we want to figure out that the video player has finished playing the video, it gets tricky. In some cases, when we start the player process directly (as in the case you outlined), we can just wait for its finish programmatically. But in some other cases, like when we use open command, I'm not sure it's possible.
Also, while VLC is a great player, we are not going to bundle it with the game :)

Cinepak seems to be a relatively simple video format. I wonder if we could just write a decoder for it.


Name the terminals

Could it be possible to name the terminals so it will be possible to see what they are doing?

@JostCrow That could be possible. What is the best way to implement this? Right click menu and then present a modal for entering a name?

Yes I would do something like that. That way you don't have to name them.

I'll look into adding this and get back to you once I have published it.

Added in latest release (v0.5.0).

Thanks it looks great


planiverse is 404

https://github.com/FuzzJunket/planiverse

How about https://github.com/Oreolek/planiverse ? or is this an old fork?

Hi @SuperSandro2000,
I wouldn't know honestly. I was browsing the list for new stuff when found that dead link.
That being said, this looks more up to date: https://git.mulligrubs.me/planiverse/
On 2020-06-21 15:07 was the last commit

I just went through everything to fix outdated links and I used @lvm's link for Planiverse; thank you!


Feature: Kinesis - list_shards()

Fixes #1635
The StepFunctions-module has a @paginate-decorator - most of this PR involves extracting and reusing this decorator across multiple services.
Changes:

moto/utilities/Paginator now contains a globally useable decorator for pagination
CognitoIDP now uses this decorator, instead of it's own
Logs now uses this decorator, instead of a per-method implementation
And finally, actually implemented the Kinesis::list_shard functionality with pagination

Codecov Report

Merging #3752 (56b95d6) into master (773e9a9) will increase coverage by 0.00%.
The diff coverage is 97.87%.

@@           Coverage Diff           @@
##           master    #3752   +/-   ##
=======================================
  Coverage   95.05%   95.06%
=======================================
  Files         493      495    +2
  Lines       55891    55894    +3
=======================================
+ Hits        53129    53136    +7
+ Misses       2762     2758    -4

Flag
Coverage Δ

servertests
48.92% <31.91%> (+<0.01%)
:arrow_up:

test_responses
32.73% <31.91%> (+<0.01%)
:arrow_up:

unittests
94.96% <97.87%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/stepfunctions/utils.py
100.00% <ø> (+4.65%)
:arrow_up:

moto/utilities/paginator.py
96.66% <96.66%> (ø)

moto/cognitoidp/models.py
88.87% <100.00%> (-0.24%)
:arrow_down:

moto/cognitoidp/responses.py
98.42% <100.00%> (ø)

moto/cognitoidp/utils.py
100.00% <100.00%> (ø)

moto/core/exceptions.py
98.46% <100.00%> (+0.12%)
:arrow_up:

moto/kinesis/models.py
88.74% <100.00%> (+0.22%)
:arrow_up:

moto/kinesis/responses.py
96.66% <100.00%> (+0.33%)
:arrow_up:

moto/kinesis/utils.py
100.00% <100.00%> (+11.11%)
:arrow_up:

moto/logs/models.py
94.39% <100.00%> (-0.18%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


Opnsense 23.7.8 update to wireguard causing script issues.  Gateway not coming up after boot. no internet.

I updated to 23.7.8 and the script is not working again.  Similar to the issue I reported with the 23.7.7 update.  PIA gateway does not come up.  After 10 minutes or so I tried restarting the wireguard service, and the gateway came up as green.  BUT the devices I have running through that gateway do not get any internet access.
23.7.8 includes wireguard 2.5 update. just FYI.
I have rolled back to <IP_ADDRESS> and all is well now.

Hello all,
thanks to PatSloan to write this here....
I have the same Problem now
Is it possible, that you update your Scripts in the next Time?
Many Thanks from Austria :-)

the "script errors" btw are:
[10-Nov-2023 02:45:02 Europe/Berlin] TypeError: explode(): Argument #2 ($string) must be of type string, array given in /usr/local/opnsense/mvc/app/controllers/OPNsense/Wireguard/Api/ClientController.php:79 Stack trace: #0 /usr/local/opnsense/mvc/app/controllers/OPNsense/Wireguard/Api/ClientController.php(79): explode(',', Array) #1 [internal function]: OPNsense\Wireguard\Api\ClientController->setClientAction('45e182d1-f2c3-4...') #2 [internal function]: Phalcon\Dispatcher\AbstractDispatcher->callActionMethod(Object(OPNsense\Wireguard\Api\ClientController), 'setClientAction', Array) #3 [internal function]: Phalcon\Dispatcher\AbstractDispatcher->dispatch() #4 /usr/local/opnsense/www/api.php(24): Phalcon\Mvc\Application->handle('/api/wireguard/...') #5 {main}

Gah!  I saw these posts and ignored them, then updated my opnsense.  I should have paid more attention.  I'm broken too.

I've updated the script now to fix the problems with the new update. Sorry for the delay, been little under the weather.
https://github.com/FingerlessGlov3s/OPNsensePIAWireguard/releases/tag/23.7.8-1

Thanks for the quick update..
I have now installed and the VPN is up and running :-)
Best Regards

Cheers Glov3s! I'll take a look later.
Hope feel better soon!

that fixed my vpn also. thanks
On Tue 14. Nov 2023 at 14.01, westy @.***> wrote:

Cheers Glov3s! I'll take a look later.
Hope feel better soon!
—
Reply to this email directly, view it on GitHub
https://github.com/FingerlessGlov3s/OPNsensePIAWireguard/issues/47#issuecomment-1810074234,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC23RN35DU4BBVX573AU2QTYENMR5AVCNFSM6AAAAAA7E32KX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJQGA3TIMRTGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

Issue has been fixed with https://github.com/FingerlessGlov3s/OPNsensePIAWireguard/releases/tag/23.7.8-1


Optimize script order

The AMP JavaScript should ideally be referenced high up the HTML near the opening <head>.
Also v0.js should be the first script tag to be referenced.

Didn't realise we changed our recommendation there. Wasn't initially last element in head?

Not sure, but start of head is definitely better :)


gelu_accurate: nan gradient with fp16. add an inner cast to float32?

gelu_accurate impl does not cast inputs to float32, so with float16 this leads to finite outputs (because tanh(+inf) = 1) and nan gradients even for moderately large inputs (because it can't represent 256 ** 3). I do understand that half dtype has quite low dynamic range, but it seems that for fixing that gelu(...) has cast to float32. Since it's supposed to be a non-saturating non-linearity, this is somewhat a problem (understanding that large inputs are still not a good idea, some tolerance to them may be good especially during unstable training)
A question is: was gelu_accurate added because it's more accurate? or because it's faster? (it used to be called gelu_fast). Isn't it an approximation? (the original paper says "we can approximate gelu with: ...(gelu_accurate formula)..."
Reusing code from https://github.com/pytorch/fairseq/blob/master/fairseq/modules/gelu.py :
import torch
import math

def gelu_accurate(x):
    if not hasattr(gelu_accurate, "_a"):
        gelu_accurate._a = math.sqrt(2 / math.pi)
    return (
        0.5 * x * (1 + torch.tanh(gelu_accurate._a * (x + 0.044715 * torch.pow(x, 3))))
    )

def gelu(x: torch.Tensor) -> torch.Tensor:
    return torch.nn.functional.gelu(x.float()).type_as(x)

x = torch.tensor([256.0], dtype = torch.float16, device = 'cuda', requires_grad = True)
y1 = gelu_accurate(x)
g1 = torch.autograd.grad(y1, (x,))[0]

y2 = gelu(x)
g2 = torch.autograd.grad(y2, (x,))[0]

print('y1', float(y1), 'g1', float(g1))
# y1 256.0 g1 nan

print('y2', float(y2), 'g2', float(g2))
# y2 256.0 g2 1.0

A question is: was gelu_accurate added because it's more accurate? or because it's faster? (it used to be called gelu_fast). Isn't it an approximation? (the original paper says "we can approximate gelu with: ...(gelu_accurate formula)..."

Nice observation! We don't actually use it anymore (preferring torch.nn.functional.gelu), but it's kept for backward compatibility.

A question is: was gelu_accurate added because it's more accurate? or because it's faster? (it used to be called gelu_fast). Isn't it an approximation? (the original paper says "we can approximate gelu with: ...(gelu_accurate formula)..."

Yeah that was poorly named on my part. I based it on the original README, where it's described as "slower but more accurate" than sigmoid(1.702 * x) * x, but it's still an approximation. Calling it gelu_approx would have been better.

About F.gelu: even without x.float() and then cast-back it seems to work for 256.0 and for torch.finfo(torch.float16).max, so casts may be excessive if the reason was just the dynamic range (and not precision)

In addition, its current impementation cannot be TorschScripted because of tricks with caching sqrt(2/pi) in an attribute of the function


[6.0] Required Validation rule allows empty string.

Laravel Version: 6.18.43
PHP Version: 7.226

Description:
Validation rule required return true also on empty string.
It should not pass, right ?

Steps To Reproduce:

Works fine for me on latest 6.x release:
Route::get('/', function () {
    return dd(\Validator::make(['foo' => ''], ['foo' => 'required'])->passes());
});


Update UG and DG to reflect that Names (Both Person and Group) are not case-sensitive

Fixed by PR #291


Autonomous Builder

https://github.com/FRC1360/Stronghold2016/pull/11
See Autonomous Branch for more info


Implement FromIterator for Vec

As discussed in #36
Panics when the vector can not hold all values supplied by the iterator.
Calling unwrap on the result might not be the best choice here.
It might be better to match the result and have a custom panic message.
Is this the best possible behavior?

bors r+

bors r+


[Question]support for quantization algorithms that are not performed on-the-fly

Reminder

[X] I have read the README and searched the existing issues.

System Info
None
Reproduction
None
Expected behavior
None
Others
Hi, Thank you for the fantastic work on LLaMA Factory! I’ve noticed that the repository supports both quantized models generated by various algorithms and on-the-fly quantization.
I am curious if LLaMA Factory is open to contributions of quantization algorithms that are not performed on-the-fly. We have  open-source AutoRound that serves as a strong alternative to existing methods. We could contribute if it's ok to you.
github
User experience on Finetuning

Sure! We welcome open-source innovations to be integrated into LlamaFactory. Currently we are using the PEFT library to support QLoRA fine-tuning. Feel free to submit a PR and we will review it soon.

Thank you !
I’m currently learning  your code and plan to implement several changes:
Add YAML Configurations: Introduce some new configurations in YAML to support the quantization process.
Export Quantized Models:: export quantized model to GPTQ or AWQ formats to leverage the current pipeline,
Potential Limitations
Model Support Currently, we only support LLMs (Large Language Models). We do not yet have a unified API for multimodal models.
Model Support: Our focus is primarily on 4-bit precision. For 2-bit precision, GPTQ asymmetric kernels have accuracy issues, and symmetric quantization has notably lower performance.
Potential Issue
UI: as the quantization process needs ~20mins for 7B and 3hrs for 70B on cuda, the webui code may need some change, but I am not familiar with this part.
Please let me know if you have any feedback or suggestions.

Hi @wenhuach21 , thanks for your information. It's okay to skip the support for multimodal models for now. Also, since users mostly use 4-bit quantization, it is not necessary to implement the 2-bit quantization. Concerning the Web UI, don't worry about it. We'll take care of the Web UI support so that you can focus on the algorithms.
Best.


Content structure

Refs: https://github.com/metarhia/metarhia.com/issues/68

Landed in https://github.com/metarhia/metarhia.com/commit/5fe50065e5d24b2383183126655ad62704d5e697 and https://github.com/metarhia/metarhia.com/commit/136e0ac5f67b81ad088d04801732258b9e2155ef


Fix compatibility with Python 3

I tried to keep Python 2 compatibility, but I'm not sure if I succeeded.

This change is

I don't use this anymore.


Add opensearch tags to allow automatic search engine addition in browsers

Can we add an opensearch description file following https://developer.mozilla.org/en-US/docs/Web/OpenSearch, this will make it very easy for users to add loogle as a custom search engine in their search bar and make looking things up in loogle quicker for many users.

Nice idea, PRs welcome. Are you interested in preparing one?

It's desired :-)

Ok after some more testing I found that chrome somehow magically autodetects the fact that loogle is a search engine and the right query url, it just needs to be enabled in chrome://settings/searchEngines and given a better shortcut. I'm not sure how this works, but I guess this issue is now less relevant until someone wants this for a browser that doesn't do this automatically (I haven't tested others)


Basic Euclidean Algorithm for GCD

This one is not look like  java code :-1:

this is not java code


Disable ANN001 for arguments with defaults

I would like to proposal a configuration option to disable ANN001 if a function argument has a default value. Consider the following example:
def foo(a: int, b: float, c=False) -> str:
    ...

Checking this file with Ruff reports:
$ ruff check foo.py
foo.py:1:27: ANN001 Missing type annotation for function argument `c`
Found 1 error.

With the option enabled, this function would not report an error.
Additional information:

I searched for existing issues using implicit argument type keywords.
Ruff version: ruff 0.5.1
ruff.toml:

output-format = "concise"

[lint]
preview = true
explicit-preview-rules = true
select = ["ANN001"]

Can be closed as a duplicate of #5958.