[Snyk] Upgrade supertest from 6.3.1 to 6.3.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade supertest from 6.3.1 to 6.3.3.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 2 months ago, on 2022-12-07.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Prototype Poisoning SNYK-JS-QS-3153490
482/1000  Why? Proof of Concept exploit, CVSS 7.5
Proof of Concept

Regular Expression Denial of Service (ReDoS) SNYK-JS-COOKIEJAR-3149984
482/1000  Why? Proof of Concept exploit, CVSS 7.5
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: supertest

6.3.3 - 2022-12-07
chore: bump deps  2910f73

v6.3.2...v6.3.3

6.3.2 - 2022-12-02
docs: added maintainer note  e24875d
docs: fixed links  5b59ddc

v6.3.1...v6.3.2

6.3.1 - 2022-10-24
fix: bump superagent (closes #798)  25122e6

v6.3.0...v6.3.1

    from supertest GitHub release notes

Commit messages

Package name: supertest

ffb96df 6.3.3
2910f73 chore: bump deps
fb4f327 6.3.2
e24875d docs: added maintainer note
5b59ddc docs: fixed links

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Coverage: 85.625%. Remained the same when pulling 543801972bc0133021d6cce761158ac3dda899fa on snyk-upgrade-3aebe1f18cf6d25426d84da0ca8aa627 into 488e91b721050bb828440a17bca4aaa3c59ddb25 on main.


Preseason issues

Preseason came, and broke Manaflux with it as expected. Here's what I found for now.
Fixed

Fixed login issues
Fixed Champion.GG
Fixed OP.GG

Known issues

Update design issues: version text is black, button flickers FIXED!
League of Graphs: adds duplicate runes and makes the page invalid

Our new process in Manaflux can now prevent providers from sending invalid runes to League, because Manaflux will now make sure that everything is valid, so that's not a big deal for now.

Manaflux crashes if you have the wait for champion to be locked  feature enabled FIXED!
Manaflux crashes if you enter in ARAM FIXED!
Other providers: new perks may be missing
Data Validator: adding new perks automatically when they're missing doesn't work FIXED!

Came out with the 2.1.7

Data Validator is going to be a lot smarter: RELEASED!
It can now detect if runes are at the wrong place and replace it by a generic one
It can now add the new runes if needed

Overall, those changes should prevent Manaflux from giving invalid rune pages to League.

Update: current issues, what needs to be upgraded, what's coming..

U.GG is coming in the next release

Uses U.GG's own internal API to provide data. However, data is not human readable, so I'll need to ensure that it works 100%.

Data validator doesn't do the job as expected

There's still a lot of buggy perk pages that get injected into League. I'll see what I can do.

There's a lot of missing translations for some languages
You should be able to select the type of keyboard you use, instead of using the default one from the language file
Flu.x doesn't work as expected since preseason came


Add dedicated interactive-only worker nodes

This can be done with a Terraform profile using the worker-gpu image; just requires an extra config for Condor including the ClassAd to make it interactive-only.

Closed by 952f72ab8ae6b3c68641e541d9085eba96f4b14f


Add delete method to the list of allowed methods

Coverage decreased (-0.3%) to 43.016% when pulling 04b8bf72b428af435568c64aef8c2711073a5189 on stgleb:allowed_methods into c244e6fa7e1aad6ea2c54c1015772bb4f62d77e4 on supergiant:master.


Certificate signing issue

On the cloud hosted version, I'm seeing this. Trying to connect to a Postgres instance on AWS RDS.
It's set to be open to <IP_ADDRESS> (btw- you should publish your source IP addresses so we can firewall down to just your IPs).
Looks like the system doesn't like the AWS certs for PG:
2021-06-14 12:27:06.305 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-4c7d-0000-699f-48b802c17c43). Disabled it. 2021-06-14 12:27:06.293 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:27:06.120 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:27:06.113 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-20c5-0000-1438-2d053a3327b6). Disabled it. 2021-06-14 12:27:06.111 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:27:05.959 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.489 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.445 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-215e-0000-eefa-47d977ef775f). Disabled it. 2021-06-14 12:26:49.442 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.429 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.428 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-4ca8-0000-087b-eec7248de3fe). Disabled it. 2021-06-14 12:26:49.423 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-4c7b-0000-81be-0d59f4126ea6). Disabled it. 2021-06-14 12:26:49.394 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-20c5-0000-1438-2d053a3327b6). Disabled it. 2021-06-14 12:26:49.374 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-1fa4-0000-4161-29290d42cd93). Disabled it. 2021-06-14 12:26:49.352 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.351 UTC | SYSTEM | ERROR | Plugin failed to load (instance ID 017a0b12-215e-0000-e16c-5dd08580d5d8). Disabled it. 2021-06-14 12:26:49.316 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-215e-0000-e16c-5dd08580d5d8). 2021-06-14 12:26:49.250 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-4c7d-0000-699f-48b802c17c43). 2021-06-14 12:26:49.216 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-1fc7-0000-21d6-3fc5e923a4a0). 2021-06-14 12:26:49.208 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-215e-0000-eefa-47d977ef775f). 2021-06-14 12:26:49.079 UTC | PLUGIN | ERROR | self signed certificate in certificate chain 2021-06-14 12:26:49.013 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-1f96-0000-ad45-7c97b55d5782). 2021-06-14 12:26:49.004 UTC | SYSTEM | INFO | Plugin unloaded (instance ID 017a0b12-4d08-0000-a00b-2025d5c1ab58).

This was a matter or a poorly worded config option. Fixed in https://github.com/PostHog/postgres-plugin/commit/dbf4b727a85698da86bb4e78497f5a60eaaa11fd


ipv6 support

Thanks for this tool.
I want to use it on a ipv6 env. However I get the following errors in logs.
2021/04/28 16:13:54 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:13:55 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:13:56 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:13:57 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:13:58 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:13:59 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in address
2021/04/28 16:14:00 address 240b:c0e0:202:5e2b:b424:2:0:961d:6379: too many colons in addr

I think a small parsing fix such as enclosing the addr in square brackets i.e. [ ] can fix it.
a new parameter to switch between ipv4 and ipv6 will be needed too.

I can help in testing this out if someone can make the above code changes.

@shishirkh happy to take a pull request!

You can watch out for this PR -> https://github.com/enriclluelles/redis-sentinel-proxy/pull/7

Totally upstream source; fyi @enriclluelles.


Chore remove gen_vector_functions.py

Proposed changes
remove unused code generate gen_vector_functions.py
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Hireme.json bug: Extra fields added does not break the application

Should we leave it like this or do we want to throw an error?

Leave it like this


feat: make access_token optional in /continue response in SDK

Changes proposed in this pull request

Makes the access_token field optional for POST /continue/{id} on the authorization server Open Payments SDK.
Context

Fixes #385
Allows the continue endpoint for an authorization server to be polled for interaction completion.

Should we bring this back?
https://github.com/interledger/open-payments/pull/366/files#diff-3d2563fb312131f782a1f59a1f02993fcd821b57298585402ea2afe30ced5b6cL192-L198


[Feature][Connector-V2][Db2-cdc]Support Db2 cdc

close  #5355

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Hi @NickYoungPeng, Is this PR ready for review?

hi, I also may need to use the db2 CDC feature, may I know what is the status of the pull and why the pull was closed? thanks


Switch primary model to GPT-4 Turbo Preview with GPT-4 as fallback

PR Type:
Enhancement

PR Description:
This PR updates the primary model used by the application to "gpt-4-1106-preview" and sets "gpt-4" as the fallback model. The changes are made in the configuration file.

PR Main Files Walkthrough:
 files:

pr_agent/settings/configuration.toml: The primary model has been changed from 'gpt-4' to 'gpt-4-1106-preview'. The fallback model has been updated from 'gpt-3.5-turbo-16k' to 'gpt-4'.

Preparing PR description...

@KennyDizi We won't switch primary default model that quickly, needs more testing and feedback from the community
It is a good idea to add support for this new model. You need to edit properly:
pr-agent/pr_agent/algo/__init__.py
I don't recommend allowing the maximum context (128K). From my experience, performance go way way down with very large context. i would cap it to 16K

and for the primary model, set:
model="gpt-4" # "gpt-4-1106-preview"

@KennyDizi We won't switch primary default model that quickly, needs more testing and feedback from the community
It is a good idea to add support for this new model. You need to edit properly: pr-agent/pr_agent/algo/__init__.py I don't recommend allowing the maximum context (128K). From my experience, performance go way way down with very large context. i would cap it to 16K

@mrT23 sure I've added support for gpt-4-1106-preview model

@mrT23 let's keep this PR like this and wait for model update as well as feedback from community.

Tested with model  gpt-4-1106-preview, it's much faster than  gpt-4

gpt-3.5 is faster than both. Quality needs to be thoroughly tested to ensure a positive tradeoff when changing the default model.
Support for the model will be added here:
https://github.com/Codium-ai/pr-agent/pull/437


Fix copying symlinks

Pass the dereference option to ncp so that symlinks don't fail to copy.  This is useful when using npm link
I've also fixed how errors are logged - in this instance the error didn't have a stack or message property so all that was being logged was Failed to install vendor modules: undefined which is super-helpful.  Now the app just logs the full Error object

Looks like I can't merge this 😒

I weirdly received no notifications for this... It's just chance that I was talking about it for a new project.

Can a test be added to this for simlinks? It'll avoid regressions later.

Ah, I see. I'm not an admin, just a "contributor". Would someone mind adding me as an admin on this (since it's public anyway)? I can't merge, and I'd like to do some maintenance.

For lack of admin rights on this repo, I merged this fix and some other stuff on my fork. Published as v2.0.0

@spmason @qubyte Made you both admins 😉


Add pyarrow and fastparquet to requirements.txt

Needed these during 0 -> 1 setup today.
Didn't fix version, but this is what was downloaded as of 1/19/2024 ... worth fixing?
pyarrow==14.0.2
fastparquet==2023.10.1

I can confirm that these libraries are necessary and sufficient.


Make tests/conversion a bit faster

Avoiding the 2 go snap installation should shave ~1m15s.

Avoiding the 2 go snap installation should shave ~1m15s.

The savings were actually more like 3-4 minutes but I guess that's mostly dependent on the snap store being slow or fast.
The CI failures are not related.


[feature] Rotation reminder

Option to set a reminder for the currently assigned concierge to assign the next person in the rotation.
Ideas:

This could schedule a DM that's sent to the concierge, OR
Could send a message in the channel mentioning @concierge on some set schedule <-- safer

Have doubts about this. Would rather recommend using /remind Slack command and mentioning @concierge in that message.


Add support for step annotation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Extra specs for RepeatedDescription and Example

I merged #877 before PR feedback was addressed, wanting to get the bugfix released a bit quicker.
@pirj Are these the changed you had in mind?
@lazycoder9 Apologies for my impatience with #877.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the changelog if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@bquorning NP 👍 here is my fault too, I was a little bit busy last days


CIS scan does not run on an rke cluster, security scan pods are not deployed on the cluster

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy a cluster.
Click on run scans.
The scan is stuck in creating state.
No security scan deployements/pods are seen in Cluster --> System project --> workloads

Error in rancher logs:
2020/01/30 18:17:28 [ERROR] ClusterScanController c-cs5d8/cis-1580408233747682027 [cisScanHandler] failed with : error fetching default cis config: resource name may not be empty
2020/01/30 18:17:44 [INFO] cisScanHandler: Create: deploying helm chart
2020/01/30 18:17:44 [ERROR] ClusterScanController c-cs5d8/cis-1580408233747682027 [cisScanHandler] failed with : error fetching default cis config: resource name may not be empty
2020/01/30 18:18:16 [INFO] cisScanHandler: Create: deploying helm chart
2020/01/30 18:18:16 [ERROR] ClusterScanController c-cs5d8/cis-1580408233747682027 [cisScanHandler] failed with : error fetching default cis config: resource name may not be empty
2020/01/30 18:19:20 [INFO] cisScanHandler: Create: deploying helm chart
2020/01/30 18:19:20 [ERROR] ClusterScanController c-cs5d8/cis-1580408233747682027 [cisScanHandler] failed with : error fetching default cis config: resource name may not be empty

Expected Result:
Scan should run on the cluster and report should get generated successfully
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - latest - commit id -4f1aff9b7
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): rke DO/AWS
Kubernetes version (use kubectl version):

1.17.2

Verified on latest master-head - commit id: 48783530d

Run scans on an rke cluster.
Scan is run successfully.
And report is generated with the fails/passed/skipped cases.


Instrumentation with a quickstart sails app fails if request hook intercept data or end events

What happened?
Steps to Reproduce
Quickstart a sails app
sails new test-project
cd test-project
sails lift

Add OTEL instrumentation and the request hook in HttpInstrumentationConfig as:
requestHook: (span, request) => {
    request.on('data', () => console.log('listening to the data event'));
    request.on('end', () => console.log('listening to the end event'));
    request.on('error', () => console.log('listening to the error event'));
},

Sample curl
curl --location 'localhost:1337/test-any-post-route' \
--header 'Content-Type: application/json' \
--header 'Cookie: sails.sid=s%3A3pFy8I0YwEcBNUH7qAuHK67C-awKrjBx.dKNHES%2FiQsDDYqaQXgiXNvlqH2efayaMFazzBK1AD%2FU' \
--data '{
   "sampleData": "321312231321",
}'

Expected Result
The request hook should not impact the POST request. Everything works if I remove the request hook data and end listeners.
Actual Result
The curl request which is a POST request with a sails-generated cookie gets stuck with the following logs, and the client never gets a response.
Additional Details
OpenTelemetry Setup Code
No response
package.json
{
    "@opentelemetry/api": "1.4.1",
    "@opentelemetry/core": "1.15.0",
    "@opentelemetry/exporter-trace-otlp-http": "0.41.0",
    "@opentelemetry/instrumentation": "0.41.0",
    "@opentelemetry/instrumentation-http": "0.41.0",
    "@opentelemetry/otlp-exporter-base": "0.41.0",
    "@opentelemetry/resources": "1.15.0",
    "@opentelemetry/sdk-trace-node": "1.15.0",
    "@opentelemetry/semantic-conventions": "1.15.0"
}

Relevant log output
tail of logs of case when it's stuck

body-parser:json content-type "application/json" +0ms
body-parser:json content-encoding "identity" +0ms
body-parser:json read body +0ms

tail of logs when it's a success
```bash
body-parser:json content-type "application/json" +0ms
body-parser:json content-encoding "identity" +0ms
body-parser:json read body +0ms
body-parser:json parse body +0ms
.
.
.

This helped -> https://github.com/open-telemetry/opentelemetry-js/discussions/3446


Container cannot have a dynamic width

Issue
The width is set explicitly for the DatePicker container component. Because of this, there's no way to have the container's width depend on the width of the placeholder text. Ideally, the placeholder text/container component would just take up as much space as needed to fit the text, with no default width set.
Environment

react-native -v: 0.57.0
node -v: 10.6.0
npm -v: N/A
yarn --version: 1.12.3
target platform: Android | iOS
operating system: macOS Mojave

I got around this by setting the width to undefined on the style prop.


Monitors connected through the dock are blank on Wayland but work fine on X11 (Ubuntu 24.04)

Are you using the latest driver? Yes

Are you using the latest EVDI version? Yes

If you are using a DisplayLink device, have you checked 'troubleshooting' on DisplayLink's website? Yes

Is this issue related to evdi/kernel? Yes

Linux distribution and its version: Ubuntu 24.04
Linux kernel version: 6.8.0-40-generic
Xorg version (if used) 7.7+23ubuntu3, core: 21.1.12-1ubuntu1
Wayland version: libwayland-server0:amd64 1.22.0-2.1build1
Desktop environment in use: gnome 46.0
GPU: Nvidia RTX 3070 Laptop
GPU Driver: nvidia-550 (proprietary)
CPU: AMD Ryzen 7 5800H

I have a Dell D6000 docking station connected to my Lenovo Legion 5 Pro laptop. Everything works (keyboard & mouse through usb, ethernet), but my monitors do not display anything on Wayland. Connecting the cable turns on the monitor but then it goes to standby because it gets 'No signal'.
On X11, everything works fine. On Windows too.

dmesg logs when reconnecting an HDMI 1920x1080@60 monitor to my dock on Wayland, with evdi verbose logging (level 6) (disconnecting, waiting a couple seconds, then reconnecting again): evdi-wayland.log. Reconnection happens between the empty lines I added.

dmesg logs when reconnecting (same thing as above) on X11: evdi-xorg.log. I left some newlines on lines 63-67 to show where I reconnected.

Xrandr output on Xorg, with the monitor connected (DVI-I-1-1): xrandr-xorg.log.

I compared the logs and noticed that on Wayland, after the Edid property is set, it immediately logs evdi_user_framebuffer_destroy, evdi_painter_close, and evdi_driver_close. These logs don't occur on X when connecting the monitor.
On Wayland, the monitor (Acer, id=3, my second monitor is connected directly to my laptop and works fine) is connected properly, shows the correct size, resolution, name, etc:

But the monitor doesn't get a signal.

I also notice that when I use the nouveau drivers instead of the proprietary Nvidia drivers, the monitor does get a signal, but only shows the cursor, the background is black and I can't move windows to the monitor. Either way, using nouveau isn't an option, as I'm planning to use this dock on Hyprland, and play games: both of which aren't great with nouveau. But this info might be useful to determine the cause.
Let me know if you need any other logs.

On Hyprland, reconnecting a monitor has the same behaviour but different logs: evdi-hyprland.log

Seems it happens due to the combination Nvidia + Wayland: I see the same behaviour on KDE + Wayland and on Hyprland when using Nvidia, but when running them on my Intel Integrated GPU it works fine.
Is this the right place to report this? Or should I report it to the nvidia forums?

Still doesn't work with nvidia-560 drivers.

Created a thread in the nvidia forums here: https://forums.developer.nvidia.com/t/displaylink-monitor-doesnt-get-signal-when-connected-through-docking-station-on-nvidia-wayland/304676

@zjeffer
Were you able to resolve it?
I too am facing the same issue (Blank screen),
With
Ubuntu 24.04.1 LTS
Nvidia 550.120
Wayland.
Gnome 46
DisplayLink - displaylink-driver/stable, 6.1.0-17 amd64

However the setup works well,
with X11.

No, still doesn't work. Tried again a few days ago with the latest evdi git commit, no difference. I'm on nvidia-565.77.

Good to hear I'm not alone with this issue :)
Which docking station do you use?

@zjeffer
I am using Alogic Universal Twin HD Pro.


Converted model has issues when importing into xcode

Hello,
I successfully exported a custom onnx model into a coreml model. However, when I import it into xcode I have the following message:

validator error: Layer '133' consumes an input named '132' which is not present in this network.

I checked the spec of the model and it looks fine:

input_names=['ip1', 'ip2', 'ip3' , 'ip4' ]
output_names=['preds']
coreml_model = convert(
    model,
    None,
    image_input_names=input_names,
    image_output_names=output_names,
)
coreml_model.save('mvdepth.mlmodel')

spec = coreml_model.get_spec()

spec -

input {
  name: "ip1"
  type {
    imageType {
      width: 320
      height: 256
      colorSpace: RGB
    }
  }
}
input {
  name: "ip2"
  type {
    imageType {
      width: 320
      height: 256
      colorSpace: RGB
    }
  }
}
input {
  name: "ip3"
  type {
    imageType {
      width: 81920
      height: 3
      colorSpace: GRAYSCALE
    }
  }
}
input {
  name: "ip4"
  type {
    imageType {
      width: 1
      height: 3
      colorSpace: GRAYSCALE
    }
  }
}
output {
  name: "preds"
  type {
    imageType {
      width: 320
      height: 256
      colorSpace: GRAYSCALE
    }
  }
}

Any pointers on where to look to debug this ?

I figured out the issue. I was a torch.cat layer and it wasn't able to add the layer in the exported model


Make tailscale optional

Can we have tailscale as an option during inital setup but not enabled out of the box? I think minimal by default is better. The toggle can enable the extension if users want it.

To be fair, I'm 100% fine w/ tailscale being installed. I just don't like the extension being enabled which adds something to the quick settings dropdown since I don't use it.


Hook the change image button up to the course offering page

@trevorjonesd2l
https://git.dev.d2l/projects/CORE/repos/lp/pull-requests/3193/overview
https://git.dev.d2l/projects/CORE/repos/platformtools/pull-requests/966/overview
Merging this into Trevor's branch
When this is merged in, the "Change image" button won't show up unless you have https://git.dev.d2l/projects/CORE/repos/lp/pull-requests/3193/overview

LGTM


The import priority of init_chat_model for the ollama package

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain.chat_models import init_chat_model
from langchain.chat_models.base import _check_pkg

_check_pkg("langchain_ollama")  # success
_check_pkg("langchain_community")  # success
model = init_chat_model("llama3.1:8b", model_provider="ollama")
print(type(model))  # <class 'langchain_community.chat_models.ollama.ChatOllama'>

Description
When I install langchain_ollama and langchain_community at the same time, it will call langchain_community first.
I think this is unreasonable. I installed langchain_ollama just because I want to use it first.
The current code snippet is as follows. When both are present, langchain_community will override langchain_ollama.
    elif model_provider == "ollama":
        try:
            _check_pkg("langchain_ollama")
            from langchain_ollama import ChatOllama
        except ImportError:
            pass

        # For backwards compatibility
        try:
            _check_pkg("langchain_community")
            from langchain_community.chat_models import ChatOllama
        except ImportError:
            # If both langchain-ollama and langchain-community aren't available, raise
            # an error related to langchain-ollama
            _check_pkg("langchain_ollama")

        return ChatOllama(model=model, **kwargs)

I think the import langchain_community should be placed in the except ImportError section of langchain_ollama (pass part).
System Info
langchain==0.2.12
langchain-community==0.2.10
langchain-core==0.2.27
langchain-ollama==0.1.1
platform==linux
python-version==3.10.12

If you think this change is appropriate, I have created a PR to address this issue.
#24977


added new freva favicon and banner

@eelucio Could you send the png-File once through https://tinypng.com/ in order to reduce the file size and re-upload it again?

On thing, can you update the default favicon. Currently it's the ugly fona logo?

I see that we have favicon.ico and favicon-dev.ico under static_root/
should I replace them with our img/favicon-tmpl.svg ?
I have been looking in the code and there is no mention to those feva logos. Should we remove them alltogether?

here we go. I hope now it goes well


Assistance with custom navbar

Version

react-native-router-flux v3.26.1
react-native v0.26.1

Expected behaviour
I want to render a custom navbar but im afraid i may be doing something wrong
static renderNavigationBar(props){
      return <CommonHeader/>
    }

This doesnt render anything, when i remove it the default navbar is rendered
Any advice would be helpful

fixed, header styling was the issue

Hello @lwhiteley.
I know this is an old issue, but I'm curious how you styling header. I get blank navbar too. Thanks! :beers:

@pewh I'm not sure if you solved this, but as reference for anyone that needs help in the future, here is an example:
 // Default styles from inspecting default nav bar
  const navigationBar = {
    backgroundColor: <Your color>,
    position: "absolute",
    paddingTop: 0,
    top: 0,
    height: 64,
    right: 0,
    left: 0,
    bottomBorderWidth: 0.5,
    borderBottomColor: <Your color>
  }

 // custom component
  static renderNavigationBar(props) {
    return (
      <View style={navigationBar}>
        <Text>{props.title}</Text>
      </View>
    );
  }

@pewh I'm not sure if you solved this, but as reference for anyone that needs help in the future, here is an example:
 // Default styles from inspecting default nav bar
  const navigationBar = {
    backgroundColor: <Your color>,
    position: "absolute",
    paddingTop: 0,
    top: 0,
    height: 64,
    right: 0,
    left: 0,
    bottomBorderWidth: 0.5,
    borderBottomColor: <Your color>
  }

 // custom component
  static renderNavigationBar(props) {
    return (
      <View style={navigationBar}>
        <Text>{props.title}</Text>
      </View>
    );
  }

@jeannei I am having trouble implementing this I have a page that loads a list view do I put this code in there?

@travisday is your scene rendering that component?

@jeannei Yes, I got it sort of working, my custom nav shows up but gets rid of the back button


errcheck, instancetype: Remove pkg/instancetype from exclude_files

/area instancetype
/sig code-quality

What this PR does / why we need it:
pkg/instancetype currently builds successfully without any errcheck errors and so can be removed from exclude_files.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

/retest

Wow, this one was easy 0_o

^_^ I did see a discussion about this somewhere and thought I'd try to fix things up for instancetypes. Feel free to roll this up into another PR if there's already one active.

/retest


remove ddoc whitespace

follow-up to #4354 - a couple of other files in which ddoc insert unnecessary whitespace that looks ugly.

This really should be fixed in dmd, not here.

closing in favor of (hopefully) fixing ddocs
https://issues.dlang.org/show_bug.cgi?id=16110


Registering another KEY_ENTER_COMMAND blocks the menu from closing

Is your feature request related to a problem? Please describe.
I use lexical and your plugin to build a chat like application. To send messages I registered a KEY_ENTER_COMMAND with low priority which blocks further command execution. But now when I add a mention and I try to select a mention item with enter my command gets executed first and prevents the mention plugin from closing the menu and adding the selected mention
Describe the solution you'd like
Raise the command priority to medium
Describe alternatives you've considered
Expose the command priority as a prop

Hi @demsr. Is it possible to return false in your listener? That tells the editor that the command is not completed and other listeners will continue receiving the event, including the mention plugin.

@sodenn but my command isn't supposed to run at this point. When the mention menu is open it should insert the mention and not run my command at all...


updated common.abstract to indicate failing method in error message

Thanks, Saketh! I just wrote a couple of comments / nits. Let me know when you address them and I'll accept.

🙌  This has been bugging me for a while. Thanks for fixing it!


Fix the transmission strategy of the haptic feedback

Right now, both the force and the vibrotactile feedback are sent in a single command per finger using write strict. See
https://github.com/robotology/walking-teleoperation/blob/a3b51def52b3c420d123331a03f924019e0e57f6/modules/HapticGlove_module/src/GloveWearable.cpp#L338-L354 and https://github.com/robotology/walking-teleoperation/blob/a3b51def52b3c420d123331a03f924019e0e57f6/modules/HapticGlove_module/src/GloveWearable.cpp#L368-L385
This is terribly inefficient, requires a huge bandwidth, and it is blocking the module in case of connection issues.
cc @EhsanRanjbari

Is there a way to measure and print the bandwidth?  @S-Dafarra

We were using iftop. I am not sure if there is a way to get it programmatically.

I have some ideas in order to decrease the bandwidth usage by the haptic glove. First, let's start by seeing how I understood the issue so far:

So, HapticGloveModule writes the haptic command to the buffer port, and on the other side, the HapticGloveDevice reads the haptic command from the buffer port.
HapticGloveModule, writes the haptic command using the write() method with the forceStrict = true option which is equivalent to the method writeStrict() (check here).
https://github.com/robotology/walking-teleoperation/blob/a3b51def52b3c420d123331a03f924019e0e57f6/modules/HapticGlove_module/src/GloveWearable.cpp#L352
So, from the figure above, the total bandwidth usage by the haptic glove is bandwidth_1 + bandwidth_2.
My idea is first,

Decrease the number of commands sent in total by NOT sending the zero-value commands: Decrease bandwidth_1
Change the buffering policy at the receiver side to FIFO by calling yarp::os::BufferedPort::setStrict(). The reason is in this case, the command will be stored in the buffer port until the next read call from the reader side. Simply meaning not use the bandwidth_2 until the reader calls it. However, there are drawbacks such as allocating memory in the background and the latency issue if the reading is slower than writing (which I think step 1 will help this): Decrease bandwidth_2

Moreover, I have some questions that I will try to answer:

Do shorter messages use less bandwidth?

Does removing string parts from messages and replacing them with an integer value result in using less bandwidth? or the string parts are the essential part and cannot be removed in any case?

Consider that sending many messages in a short amount of time is much more expensive than sending a single big message. This is because of the so-called overhead.
Moreover, because of the write strict, if one message gets delayed, the sending is blocked, meaning that the whole hapticGloveModule gets stuck. Basically, if for some reason the haptic feedback to the pinky is delayed, the entire retargeting of both hands is blocked.
Hence, a first step could be simply to "merge" all the haptic commands in a single message, thus also allowing the removal of the writeStrict option.

@mebbaid check this issue!

Hi @EhsanRanjbari, that's ok, but consider that what I wrote in the first comment:

Right now, both the force and the vibrotactile feedback are sent in a single command per finger using write strict. See

is still not fixed, i.e., there is still a separate message per finger, and you still need to use force strict

Hi @EhsanRanjbari, that's ok, but consider that what I wrote in the first comment:

Right now, both the force and the vibrotactile feedback are sent in a single command per finger using write strict. See

is still not fixed, i.e., there is still a separate message per finger, and you still need to use force strict.
The idea would be to condense all the haptic commands for a given hand in a single message

Thanks for the suggestion,  now with these commits

wearables:
walking-teleoperation:
I merged all the messages into one message as

"name" type forceValueVector vibrotactileValueVector durationValueVector

"HapticGlove::haptic:HapticFeedback" 0 (0 0 0 0 0) (0 0 0 0 0) (0 0 0 0 0)

where the elements of forceValueVector, vibrotactileValueVector, and durationValueVector correspond to
"r/l_thumb_finger" "r/l_index_finger" "r/l_middle_finger" "r/l_ring_finger" "r/l_pinky_finger"

Note: There is a method called setMotorPosition that is used in Paexo.cpp, which also uses the same thrift message. I think this needs to be considered.
cc @S-Dafarra @mebbaid

There is a method called setMotorPosition that is used in Paexo.cpp, which also uses the same thrift message. I think this needs to be considered.

What about using two different messages, so that Paexo does not need to be touched?

With the above, we tried to replicate the teleoperation setup for testing even though the robot was not available. My labtop, connected to the robot server acted as the robot simulator, and on the other side the teleoperation setup. Both sides connected via vpn and the yarpserver running on the teleoperation side. Here are the output of iftop which we will analyze thanks to @EhsanRanjbari

After modificaiton
Before modificaiton

From the screenshots above it is visible that there is a significant reduction in the bandwidth usage. Almost %64. To be more clear I highlighted the photos below:

After modificaiton
Before modificaiton

From the screenshots above it is visible that there is a significant reduction in the bandwidth usage. Almost %64. To be more clear I highlighted the photos below:
After modificaiton	Before modificaiton

To be honest, I think the two images are not helpful. If you look closely, the traffic you have highlighted is between different machines on the "before" and "after" images. You can also see some traffic toward google. Anyhow, it os worth to test on the robot, so it will be easier to check the traffic flowing between the two networks

the traffic you have highlighted is between different machines on the "before" and "after" images.

I thought the images always show the same traffic namely between iiticubws069 and <IP_ADDRESS> ?

Perhaps @S-Dafarra means that there is traffic for which we don't know the cause since there is multiple machines in that network and also internet (even if the up/down/accum you highlighted are between the same two machines). So on isolation in the check with the robot, we will see only the traffic resulting from the gloves @EhsanRanjbari

Exactly!
Also, the traffic toward <IP_ADDRESS> is basically the same between the two figures. At the same time, I am not even sure what that machine is. It does not seem a machine in the IIT network.
In any case, even if the traffic remains the same, we can avoid using write strict (since we would not need every message to arrive in order), which is an advantage in a delayed network

In any case, even if the traffic remains the same, we can avoid using write strict (since we would not need every message to arrive in order), which is an advantage in a delayed network

I removed the argument true when calling the m_iWearActuatorPort.write(). According to the YARP documentation, this means write strict is avoided.
https://github.com/EhsanRanjbari/walking-teleoperation/blob/957f84545757f10b6a08da2fd1100c26e6dcc2cf/modules/HapticGlove_module/src/GloveWearable.cpp#L417

Testing the SenseGloves with the modified haptic message, @mebbaid and @S-Dafarra noticed a significant delay in the retargeting. After investigation and testing again on February 13th, it turned out that it was related to the failed calibration. More specifically, upon starting the Senseglove, a vibrotactile feedback with a value of 35 is sent to the glove that notifies the operator to start the calibration. Now with the modified haptic message, this is not being done and the user never knows when to start the calibration. Below is the video of the second test the operator started the calibration on his own and it is working fine.
https://github.com/robotology/walking-teleoperation/assets/75568997/d017481e-caeb-4608-9403-308d4c87b0cc

Another point I would like to discuss is that the palm vibrotactile feedback seems to be turned off

I think that's just an initialization value

Another point I would like to discuss is that the palm vibrotactile feedback seems to be turned off

I think that's just an initialization value

yes, but it is not been updated (at least I could not find it) before being written to the port.

With this commit c50fc61 I fixed the vibration feedback when starting the calibration. I tested it in simulation and it works as expected. However, I need to test it on the robot to fully confirm this.
Another point I would like to discuss is that the palm vibrotactile feedback seems to be turned off (https://github.com/EhsanRanjbari/wearables/blob/7094f84275c29e6e4c60312f4754574ab2db790e/devices/HapticGlove/src/HapticGlove.cpp#L177). If this is the case, at this level, this newly modified message does not include the palm vibrotactile value.
cc @mebbaid @S-Dafarra

Today, with @mebbaid we tested the code on the robot and the vibration upon starting the calibration works fine. I undrafted the related PRs:

https://github.com/robotology/walking-teleoperation/pull/139
https://github.com/robotology/wearables/pull/204

Today, with @mebbaid we tested the code on the robot and the vibration upon starting the calibration works fine. I undrafted the related PRs:

Fix Haptic glove transmission - HapticGloveModule #139
Fix Haptic glove transmission - GloveDevice wearables#204

Are they actually ready to be reviewed? I still see a lot of commented code

Are they actually ready to be reviewed? I still see a lot of commented code

Oh my bad, I had to remove them. I will do it now.

I will do it now.

Done! Thanks for the reminder.

The PRs have been merged. I think we can finally close this issue. Thanks a lot @EhsanRanjbari @mebbaid


no read me

why？  package.json??

Sorry for the slow response~
But Thank You so much for send me an issue. now it's available in my next push~ @spiritelf  :grin:


Join Game: Green Path

Green buttons not only the Next buttons should be, but any progress button like upload photo, proof or calculate results. They can blue out after they are not necessary to be used anymore.

done


Challenge http://www.freecodecamp.com/challenges/waypoint-add-placeholder-text-to-a-text-field has an issue

Adding placeholder tag to the  element rather than the  element validates the "Your text field should have a placeholder attribute." checkbox incorrectly.

Could you provide a screenshot? I'm not quite following "Adding a placeholder tag to the element rather than the element...."

Sorry spaced out and wrote the wrong thing. If you add placeholder tag to the img element rather than the text field the waypoint will say it's correct.


Rework Updater logic

Our current Updater implementation works, but it's not very easy to test, and it's not super easy to add an update path. We should rework this (possibly into something similar to Room migrations).

We've made the switch to https://github.com/boswelja/android-migration for our update logic


Context Length seems to be ignored in 0.5.2 but worked fine in 0.4.8

Installation Method
[Describe the method you used to install the project, e.g., git clone, Docker, pip, etc.]
I am using ghcr.io/open-webui/open-webui docker image. The problem is with main version which is 0.5.2, but it was not there with 0.4.8.
Environment

Open WebUI Version: 0.5.2 bug exists, but 0.4.8 worked fine

Ollama (if applicable): 0.5.4

Operating System: Docker container living on Fedora

Browser (if applicable): Google Chrome Version 131.0.6778.204, Firefox 132.0.1

Confirmation:

[ X] I have read and followed all the instructions provided in the README.md.
[X ] I am on the latest version of both Open WebUI and Ollama.
[ ] I have included the browser console logs.
[ ] I have included the Docker container logs.
[X ] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:
The answer gets into account context_length set by me so my AI answers are more consistent, even when slower.
Actual Behavior:
The answers are blazing fast, but lose details from previous messages quickly.
Description
Bug Summary:
When i put my context_length manually to be 32768 it seems to be ignored with new version. It makes my Single User Dungeons with AI less consistent, because it forgets details faster.
Reproduction Details
Steps to Reproduce:

Start new chat with mistral-nemo:12b. It does not matter if i am using local or remote image with ollama.

Ask AI to generate a story adventure and choose name for a character.

Additional Information
In previous version AI needed 2 minutes and 30 seconds to answer my prompts now it's instant. The problem is that it was able to keep details from my prompts for 5-7 messages and now it loses them quickly. I tested it with CPU only ollama version on my laptop and with GPU (RTX4070) version of Ollama on Windows.

Good catch! Fixed on dev and will be released with 0.5.3 shortly!


Scale CLI doesn't create preceeding paths for export commands

Minor inconvenience I'm noticing with the Scale CLI - unlike say Go, it doesn't create the paths on export. For example, I get an error here; coming from Go, I'd probably have expected the output directory to be created:
scale function export local/questions:latest out
Error: failed to stat output path out: stat out: no such file or directory

Fixed in #45


Having Trouble Accessing Tabmon

You probably have gotten this before, but I am trying to use TabMon to monitor the performance of my Tableau Server. When trying to login to "localhost" with the database "TabMon" and the default username and password, tabmon and "password" respectively. I get the following error. I am so excited about using this. I hope I left a detailed enough description of my issue.

The table in my setups Tabmon.countersamples. How did you do the installation?

The "countersamples" table is created the first time you start up the service; if it doesn't exist, then there was some problem starting TabMon.
The most common cause for this is a permissions issue when running the "Start TabMon" shortcut.  Try right-clicking on "Start TabMon" and selecting "Run As Administrator".  Once that's done, if you go into "Services.msc" and look at the list of services, the "TabMon" service should be in a "Running" state and you should then be able to load the workbook.  If it still isn't "Running", then unfortunately there is something else going on which will require digging into the TabMon logs.

Also -- as benedictchan886 mentioned, how did you do the installation?  Did you leave the "Install Postgres for me" box checked?

If jmangue's suggestions do not work can you attach the DBCreation.log file? You should be able to find this file in the temp directory.

Hello,
I just wanted to see if you were able to get TabMon running.
Thanks,
Dan

Closing this. If the issue persists or there are any other issues let me know.

I installed tabmon few minutes back and when trying to access the tabmon workbook, got the error as mentioned in this thread. can you please suggest what the error is?

I also tried to check for dbcreation log.. couldnt find one. there is a tabmon log.
also, when i tried to start & stop tabmon.. a CSV file was generated but most of the counters were null. please suggest.

Have you installed Postgres when you installed Tabmon?
And have you started up postgres? like

cd C:\Postgres\bin

C:\Postgres\bin>pg_ctl start -D "C:\Postgres\data"

Yes.. have installed postgres and I see it is started in services.
<EMAIL_ADDRESS>Regards,
Srujan
From: benedictchan886<EMAIL_ADDRESS>Sent: Monday, January 18, 2016 7:35 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
Have you installed Postgres when you installed Tabmon?
And have you started up postgres? like
cd C:\Postgres\bin
C:\Postgres\bin>pg_ctl start -D "C:\Postgres\data"
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D172700077&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=qYlFX5BaAl9uOUD9N7bah42HVSC1u939-WzRrtMWseg&s=f3Ry1n4zp7tOjAl_KlKSTHrQi0t0keD1mgAVaS3wcJ4&e=.

Let's do this
Do you see something similar?

See if the TabMon database there

C:\Postgres\bin>psql -U tabmon -l
List of databases
Name    |      Owner      | Encoding |          Collate           |           Ctype            |          Access privileges
-----------+-----------------+----------+----------------------------+----------------------------+-------------------------------------
TabMon    | tabmonsuperuser | UTF8     | English_United States.1252 | English_United States.1252 |
postgres  | tabmonsuperuser | UTF8     | English_United States.1252 | English_United States.1252 |
template0 | tabmonsuperuser | UTF8     | English_United States.1252 | English_United States.1252 | =c/tabmonsuperuser                 +
|                 |          |                            |                            | tabmonsuperuser=CTc/tabmonsuperuser
template1 | tabmonsuperuser | UTF8     | English_United States.1252 | English_United States.1252 | =c/tabmonsuperuser                 +
|                 |          |                            |                            | tabmonsuperuser=CTc/tabmonsuperuser
(4 rows)

Let's see what tables inside the TabMon db

C:\Postgres\bin>psql -U tabmon -d TabMon
psql (9.4.4)
WARNING: Console code page (437) differs from Windows code page (1252)
8-bit characters might not work correctly. See psql reference
page "Notes for Windows users" for details.
Type "help" for help.
TabMon=> \d+
List of relations
Schema |         Name          |   Type   | Owner  |    Size    | Description
--------+-----------------------+----------+--------+------------+-------------
public | countersamples        | table    | tabmon | 2771 MB    |
public | countersamples_id_seq | sequence | tabmon | 8192 bytes |
(2 rows)

I was not able to find any relations ☹
<EMAIL_ADDRESS>From: benedictchan886<EMAIL_ADDRESS>Sent: Monday, January 18, 2016 8:20 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
Let's do this
Do you see something similar?

See if the TabMon database there

C:\Postgres\bin>psql -U tabmon -l
List of databases
Name | Owner | Encoding | Collate | Ctype | Access privileges
-----------+-----------------+----------+----------------------------+----------------------------+-------------------------------------
TabMon | tabmonsuperuser | UTF8 | English_United States.1252 | English_United States.1252 |
postgres | tabmonsuperuser | UTF8 | English_United States.1252 | English_United States.1252 |
template0 | tabmonsuperuser | UTF8 | English_United States.1252 | English_United States.1252 | =c/tabmonsuperuser +
| | | | | tabmonsuperuser=CTc/tabmonsuperuser
template1 | tabmonsuperuser | UTF8 | English_United States.1252 | English_United States.1252 | =c/tabmonsuperuser +
| | | | | tabmonsuperuser=CTc/tabmonsuperuser
(4 rows)

Let's see what tables inside the TabMon db

C:\Postgres\bin>psql -U tabmon -d TabMon
psql (9.4.4)
WARNING: Console code page (437) differs from Windows code page (1252)
8-bit characters might not work correctly. See psql reference
page "Notes for Windows users" for details.
Type "help" for help.
TabMon=> \d+
List of relations
Schema | Name | Type | Owner | Size | Description
--------+-----------------------+----------+--------+------------+-------------
public | countersamples | table | tabmon | 2771 MB |
public | countersamples_id_seq | sequence | tabmon | 8192 bytes |
(2 rows)
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D172711008&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=u7lDuWIh_EgeJ5BsvbN00p14JP7NpikASEkENokP8m0&s=asemR8FkOnayrtXN2Z1cg2omcINAtOWGcAbM6jDG7Xg&e=.

There are 3 parts.

Postgres (installed during Tabmon installation) to store the performance data.
Tabmon executable (service), that's the process to go writing into the Postgres
The workbook, connecting to Postgres to show it in the viz.

Your error seems that it cannot find the table that stores the data. So, the previous steps are to show what you really have in your Postgres

I have just executed the tabmon executable as mentioned in the user guide. Why would it not create the tables to store the performance data? What can we do to fix this please?
From: benedictchan886<EMAIL_ADDRESS>Sent: Monday, January 18, 2016 10:44 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
There are 3 parts.

Postgres (installed during Tabmon installation) to store the performance data.
Tabmon executable (service), that's the process to go writing into the Postgres
The workbook, connecting to Postgres to show it in the viz.

Your error seems that it cannot find the table that stores the data. So, the previous steps are to show what you really have in your Postgres
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D172735475&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=-YEOH2-5E2ljI5ViMujrM4UA9iIlxRXLpZ9E__7wt_M&s=hvF1vTQM0qPI8wcCDU6aBcGkYJH1V9d3_qrtoV3l9uM&e=.

Back one step first, do you see the TabMon database and countersamples table in your Postgres?

I couldn’t find the tables in the database.
<EMAIL_ADDRESS><EMAIL_ADDRESS>From: benedictchan886<EMAIL_ADDRESS>Sent: Monday, January 18, 2016 11:27 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
Back one step first, do you see the TabMon database and countersamples table in your Postgres?
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D172742396&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=E1vMMgT2HsVujJqko18zd6OpYMi8I0RUoUW-fb4wSiQ&s=Fx0GkFZN-wyPTE1QZXZyBWLRw9TdGEmTqIxKxqfjuKM&e=.

I earlier tried to install tabmon with "Install and initialize postgres for me" option checked on our tableau server. When i check for the tables in tabmon database after the install is completed, i see no relations displayed.
can you please suggest on how to proceed? I see there is an issue with postgres install. Do we need to uninstall and install again? Will there be any impact to existing postgres tableau database?

Would you please check if there is any error in C:\Temp\DBCreation log?

By the way, thanks for all your support. I was able to uninstall/reinstall successfully.
From: benedictchan886<EMAIL_ADDRESS>Sent: Tuesday, January 19, 2016 4:44 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
Would you please check if there is any error in C:\Temp\DBCreation log?
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D173012037&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=GdwkIAaLJcV-Whia5PCD_AGGwdmLb8JIuM7g-wmwyvw&s=-ZrRH1zizdQN2ls-QM9Tm95bhOrMEW36WK89roBz_r8&e=.

Is it resolved now? Srujan

Yes. It is resolved. I started tabmon yesterday and was able to see results now on the workbook. However, I do not see network related metrics. Checking whats wrong..
From: benedictchan886<EMAIL_ADDRESS>Sent: Tuesday, January 19, 2016 4:51 PM
To: tableau/TabMon
Cc: Yada, Srujan (Oil & Gas, Non-GE)
Subject: Re: [TabMon] Having Trouble Accessing Tabmon (#6)
Is it resolved now? Srujan
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tableau_TabMon_issues_6-23issuecomment-2D173013706&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=z4Hza6crtvFd01FMp6jrLj3S0ANzTNTryD4Bh-GZxJM&m=-UXond7pZIZLa7XK0E0F87uWYpK09nHvzgBYgPBOgKo&s=5Y407e_ZYO4crjOlGxWdVvukXrWlZsB88kuZujFnmdU&e=.

Glad its up and running!
The missing network data may be due to Issue #22 , we have heard of one other user with the same issue.


Fields are editable on confirm subscription screen

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Ski patrol mobile app showcase

Hi @garymeyer. Thank you for the PR! We're in the middle of a big merge in this repo, so I will get to this merge next week. Stay tuned.

Np. Giving an update like this is kind. Thanks.


Contact Delete Override Button Errors Out With Console Navigation

Related to #3319, which fixed the issue for standard navigation within a Lightning app. It appears that if a Lightning app is set up to use console navigation instead of standard navigation, the delete button still produces an error.
Steps to Repeat:

Create a Lightning App with Console navigation. Make sure to add the Contacts tab and enable it for System Administrators.
Create a Contact.
Click the Delete button. You will see an error that says "This page isn't available in Salesforce Lightning Experience or mobile app."

**lurch: add

Tracking W-026342

This has been open for awhile, Do we have a ETA for the fix?

This is unfortunately an issue with core Salesforce.com functionality. Visualforce button overrides are not supported in console apps (https://success.salesforce.com/issues_view?id=a1p3A000000EAa3QAG).
As a workaround for this issue, we need to "trick" the console app into using a custom "delete" button using the following steps:

Setup → Object Manager → Contacts
Click Buttons, Links, and Actions.
Click New Button or Link
Label / Name = Delete 2 (for now so you can distinguish between the other one)
Display Type = Detail Page Button
Behavior = Display in Existing Window With Sidebar
Content Source = Visualforce Page
Content = CON_DeleteContactOverride
Save
Add the button to your page layouts and remove the other Delete button.

Thank you to @bethbrains  and @cdcarter  for this workaround!


Screen recording toggle greyed out

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
1908
Describe the bug
Record Screen toggle is greyed out on the menu.
I want to use it to troubleshoot my setup which died overnight on me, which I suspect to be due to some stuttering issues I've been having recently (game freezes for a second or two after certain button presses) messing up some timing-based followup button presses.
Video

Device model
Oneplus Ace
Android version
13
Screen size
6.7 in
RAM
12.0GB


AttributeError: 'bool' object has no attribute 'startswith'

It seems like after the latest update of essentials, I keep getting this error.

delete the node and recreate it

hello, I tried that but im still getting the same error regarding resize+

i was able to fix it, ty!


Dynamically change URL based off radio list

I want to be able to change the options list in ajaxChosen dynamically as the user, uses the page.
Can someone give me a clue
I have debugged it in the debugger to look at the object structure of the select element.
And I can't figure out where I can access the options.
Some of the Various forms I have tried
$([element]).chosen('options','url',[value]);
$([element]).ajaxChosen('options','url',[value]);
$([element]).data('chosen','options','url',[value]);
$[element]).data('ajaxChosen','options','url',[value]);
$([element]).ajaxChosen().options.url = [value];
$([element]).chosen().options.url = [value];
I for the life of me can't figure this out.
Any help is truly appreciated!

+1 for this, would be very useful!


Circle (turtle.arc) isn't accurate when less than a whole number of degrees

Context here: https://hackclub.slack.com/archives/C04GCH8A91D/p1702967207674719?thread_ts=1702966457.956549&cid=C04GCH8A91D
Screenshot of some lines that should intersect, but the current arc implementation offsets due to accumulated error:

hey! coded a recreation of the arc function using some basic mathematics and i've found it works better than the current one. it can handle small, decimal angles nicely!
unsure if i should make a pull request, so ill just put the code i used + a sample screenshot below in markdown

const WIDTH = 125;
const HEIGHT = 125;

function pointCoord(angle, radius) { // returns the coords of a point provided a radius and an angle to subtend around a circle of that radius
  return [
    (Math.cos(angle) * radius) - radius,
    Math.sin(angle) * radius
  ]
};

function arc(angle, radius, points) { // yeahhhhh the main function
  const arc = createTurtle();
  const angleRad = angle * (Math.PI / 180); // i like degrees more don't harsh it

  let arcAngle = 0;

  for (let i = 1; i <= points; i++) { // draws the arc by just going to each point on the circle
    arcAngle += angleRad / points;
    arc.goTo(pointCoord(arcAngle, radius));
  };

  arc.translate( // makes it so that the arc doesn't move around while its size changes
    [WIDTH / 2, HEIGHT / 2],
    [-radius, 0]
  );

  return arc
};

const arc1 = arc(1.7, 47, 74); // example, feel free to plug in whatever values you want !

drawTurtles([arc1]); // draw it

Great work @ameya-g-git! You should totally contribute this in a PR.
I saw some slight differences between the previous arc function and this implementation.

it takes in a "points" value. It should ideally just figure out how many steps to iterate with.
it doesn't factor in the current turtle position and rotation.

Want to take another pass on this? It'd be awesome if you patch this up.

thanks for the feedback! here's my thoughts

the points value was there to give users the ability to resample based on how many points they want, i might just create some sort of function to determine the default value for the points param then
wdym by factor in the current turtle position and rotation? i was thinking that if the user wanted to transform the arc, they could do so with blot's existing transformation functions, and simply join the turtle once the transformations were complete. should i include parameters to adjust these values?
as for the off-by-one, im not observing this? the right-most point stays stable no matter how i change the parameters, could you elaborate more?

also, since this isnt exactly art, where would i include my file in the PR? thanks again!

Want to try it now Max? I updated the arc implementation to work more like this.


afte unwrap, a string become Array Type

`
let json = JSON(detailArray)
    let string = json.rawString(NSUTF8StringEncoding, options: .PrettyPrinted)

    print(string)

    print(string!)

`
log shows:
Optional("[\n  {\n    "ProductId" : 5629,\n    "SelectedQty" : 2\n  },\n  {\n    "ProductId" : 6700,\n    "SelectedQty" : 1\n  }\n]")
[
{
"ProductId" : 5629,
"SelectedQty" : 2
},
{
"ProductId" : 6700,
"SelectedQty" : 1
}
]
What happen?
I need the stingValue  NOT array!

it's not a array, it's just that print method treat string type differently and printing out \n as actual line breaks.


reg_executing is not cleared in two cases

Steps to reproduce
Case 1:

Run vim --clean
Source the following file:

nnoremap s <Nop>
let @a = 's'
call feedkeys("@aqaq\<Esc>", 't')

A q is added to the buffer and the content of @a is not empty.

Case 2:

Run vim --clean
Source the following file:

call setline(1, 'aaa')
nnoremap s qa
let @a = 'fa'
call feedkeys("@asq\<Esc>", 't')

A q is added to the buffer and the content of @a is not empty.

Not sure if these can really be considered bugs, as the macro in case 1 only seems producible using :let (the keys in case 1 can be typed interactively though), and case 2 seems only reproducible when using feedkeys() with t flag. (Maybe it can be reproduced by typing these keys very quickly?)
Expected behaviour
No q is added to the buffer and the content of @a becomes empty.
Version of Vim
8.2.4651
Environment
Operating system: Arch Linux
Terminal: kitty
Value of $TERM: xterm-kitty
Shell: fish
Logs and stack traces
No response

Case 1 may be fixes by adding a check for clearing reg_executing in the loop in vgetorpeek(). Not sure if this is the best approach. Case 2 is caused by vpeekc() applying the mapping after the macro, causing chars from the mapping to be treated as a part of the macro. This can be hard to fix without breaking other things (e.g. autocommands that use the value of reg_executing(), or invoking :normal inside a macro).

I'm trying to fix this in #10111.

Is there a better problem description for this issue?


[Feature]: Add use_vectorized_reader variable to snowflake_file_format

Use Cases or Problem Statement
I want to create a parquet file format with use_vectorized_reader = true to speed up copying data into Snowflake
Category
category:resource
Object type(s)
resource:file_format
Proposal
Add use_vectorized_reader as a variable.
How much impact is this issue causing?
Low
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hi @deanmorin 👋
File formats are on the remaining objects list and they will be reworked before v1. We'll add this field to the resource.


Add Date or Time beside DNS Queries in the graph

The Home tab displays the graph of DNS Queries vs time. In the web ui, I can hover over the graph and it shows me the value of number of DNS Queries vs the time or date. In your app, if I touch and hold on a certain point in the graph, it only shows the value of DNS Queries (value of the ordinate) at that point. It'd be helpful if you also include the date or time (value of the abscissa).
Also a minor bug that I noticed is in the Settings tab, if I tap on "Access Settings", it opens up the "Allowed Clients" tab and since the list is empty in case of mine, it says, "No hay clientes permitidos". My language is set to English, so in the "Blocked Clients" tab, it says, "No blocked clients". Only the "Access Clients" message is displayed in Spanish.

Hi that issue will be fixed on the next release. Probably tonight or tomorrow. I opened another issue just for it (#46).
The problem with the timestamp on the chart is that I don't receive that information from the API, I only receive a list of numbers, which are the values. But maybe I can calculate it looking at the statistics retention time configuration.

But maybe I can calculate it looking at the statistics retention time configuration.

So, it's the issue with the API then? I guess then I could open an issue in the official repo of AdGuard Home as a feature request.
Alternatively, you probably could use a bit of Web Scraping, I guess. Thought that probably make this complicated. I'm a noob in Flutter so I don't know if my suggestions are feasible.

It would be great if the AdGuard Home people add the timestamps on the response of the API.

Added on v2.5.0-beta.2


Divide Eventwatcher examples into multiple files

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice.
/lgtm

🚀
/approve


Create :feature:trip-planner:state module

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#130
#129  👈
#128
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


POC: Different ways of mondo-building IREE and sandbox.

the iree-integrate branch should supersede this.


Status bar network no longer appears during Parse network activity

I've always seen it before the upgrade to 1.8.0. But now I don't see it.
This is desirable for debugging reasons as well as informing the user of network activity.

Hey @fatuhoku, thanks for reporting. Marking this for the next version, so we actually restore the network activity functionality.


Main Thread Checker: Xcode 9,iOS 11

when in xcode 9 build gpuImage project and run in ios 11,the console show this:
=================================================================
Main Thread Checker: UI API called on a background thread: -[UIView bounds]
PID: 2054, TID: 149860, Thread name: (none), Queue name: com.sunsetlakesoftware.GPUImage.openGLESContextQueue, QoS: 21
Backtrace:
4   SimpleVideoFilter                   0x00000001043cb790 __39-[GPUImageView recalculateViewGeometry]_block_invoke + 56
5   SimpleVideoFilter                   0x00000001043d578c runSynchronouslyOnVideoProcessingQueue + 108
6   SimpleVideoFilter                   0x00000001043cb738 -[GPUImageView recalculateViewGeometry] + 108
7   SimpleVideoFilter                   0x00000001043cc240 __37-[GPUImageView setInputSize:atIndex:]_block_invoke + 312
8   SimpleVideoFilter                   0x00000001043d578c runSynchronouslyOnVideoProcessingQueue + 108
9   SimpleVideoFilter                   0x00000001043cc0e8 -[GPUImageView setInputSize:atIndex:] + 136
10  SimpleVideoFilter                   0x00000001043c6c80 -[GPUImageFilter informTargetsAboutNewFrameAtTime:] + 748
11  SimpleVideoFilter                   0x00000001043c9534 -[GPUImageFilter newFrameReadyAtTime:atIndex:] + 216
12  SimpleVideoFilter                   0x00000001043d1370 -[GPUImageVideoCamera updateTargetsForVideoCameraUsingCacheTextureAtWidth:height:time:] + 1720
13  SimpleVideoFilter                   0x00000001043d1fd8 -[GPUImageVideoCamera processVideoSampleBuffer:] + 2980
14  SimpleVideoFilter                   0x00000001043d285c __74-[GPUImageVideoCamera captureOutput:didOutputSampleBuffer:fromConnection:]_block_invoke + 176
15  libdispatch.dylib                   0x0000000104cdd49c _dispatch_call_block_and_release + 24
16  libdispatch.dylib                   0x0000000104cdd45c _dispatch_client_callout + 16
17  libdispatch.dylib                   0x0000000104cec110 _dispatch_queue_serial_drain + 692
18  libdispatch.dylib                   0x0000000104ce09a4 _dispatch_queue_invoke + 332
19  libdispatch.dylib                   0x0000000104ced104 _dispatch_root_queue_drain_deferred_wlh + 424
20  libdispatch.dylib                   0x0000000104cf4100 _dispatch_workloop_worker_thread + 652
21  libsystem_pthread.dylib             0x00000001864dafe0 _pthread_wqthread + 932
22  libsystem_pthread.dylib             0x00000001864dac30 start_wqthread + 4
i think it warn this funtion:
GPUImageView.m  =====> - (void)recalculateViewGeometry
in line 238:
CGSize currentViewSize = self.bounds.size;
but i don't know how to fix it,can anybody know that?

Choose scheme -> Diagnotics, remove main thread checker, then the warning will disappear. scheme editor
it could close this warning,but I don't know what other harm it might have

I don't think disabling main thread checker should be the way to go. recalculateViewGeometry method dispatches code to the video processing queue (which is not the main thread) where it accesses the bounds property of the UI element, which should be done from the main thread. Although this is relatively harmless, it'd be nice to see it called there. Maybe (since we're certain that this code is run at video queue, not the main queue) dispatch_sync'ing the call to the bounds property to the main queue would fix that (but I haven't tried).

Maybe (since we're certain that this code is run at video queue, not the main queue) dispatch_sync'ing the call to the bounds property to the main queue would fix that (but I haven't tried).

This will lead to a deadlock, e. g. init a GPUImagePicture on the main queue -> dispatch sync to video queue -> dispatch sync to main queue, which is still waiting on that first dispatch. I think a true solution would require a deeper change in the dispatch queue usage of the library.

Have the same problem:
Main Thread Checker: UI API called on a background thread: -[UIView layer]
PID: 412, TID: 54746, Thread name: (none), Queue name: com.sunsetlakesoftware.GPUImage.processingQueue, QoS: 0
Is it fixable?

@dondonut Apparently it's not something we (as users) can fix by ourselves. However, it doesn't seem to cause any problem (other than the log in the console). I don't think it ever will cause harm unless you are repeatedly editing/animating a frame of a GPUImageView.

@canpoyrazoglu No, I think this is a problem, because the GPUImageVideo starts slower

It takes 7 seconds for camera startup with ios 11 on iphone SE. I think that it is a big problem.

This seems to fix it while not breaking anything either: https://github.com/BradLarson/GPUImage/pull/2533/files

@lacyrhoades
hmm! It seems not good for SimpleVideoFilter project!
So!!!
I try to change some code,maybe it will change this problem!

I think it's not good
maybe someone have more good idea.
So i don't PR this Code

I can only think of one solution: to remove all main-queue-specific code from the runSynchronouslyOnVideoProcessingQueue Blocks. Otherwise, you can always create a scenario, where one call is dispatched sync from the main queue to the video processing queue, which itself dispatches sync to the main queue again, resulting in a deadlock.

Check out this changes :-

(void)recalculateViewGeometry;
{
__block CGRect currentBounds;
runOnMainQueueWithoutDeadlocking(^{
currentBounds = self.bounds;
});
runSynchronouslyOnVideoProcessingQueue(^{
CGFloat heightScaling, widthScaling;
  //    CGFloat imageAspectRatio = inputImageSize.width / inputImageSize.height;
  //    CGFloat viewAspectRatio = currentViewSize.width / currentViewSize.height;

  CGRect insetRect = AVMakeRectWithAspectRatioInsideRect(inputImageSize, currentBounds);

  switch(_fillMode)
  {
      case kGPUImageFillModeStretch:
      {
          widthScaling = 1.0;
          heightScaling = 1.0;
      }; break;
      case kGPUImageFillModePreserveAspectRatio:
      {
          widthScaling = insetRect.size.width / currentBounds.size.width;
          heightScaling = insetRect.size.height / currentBounds.size.height;
      }; break;
      case kGPUImageFillModePreserveAspectRatioAndFill:
      {
          //            CGFloat widthHolder = insetRect.size.width / currentBounds.size.width;
          widthScaling = currentBounds.size.height / insetRect.size.height;
          heightScaling = currentBounds.size.width / insetRect.size.width;
      }; break;
  }

  imageVertices[0] = -widthScaling;
  imageVertices[1] = -heightScaling;
  imageVertices[2] = widthScaling;
  imageVertices[3] = -heightScaling;
  imageVertices[4] = -widthScaling;
  imageVertices[5] = heightScaling;
  imageVertices[6] = widthScaling;
  imageVertices[7] = heightScaling;

});

//    static const GLfloat imageVertices[] = {
//        -1.0f, -1.0f,
//        1.0f, -1.0f,
//        -1.0f,  1.0f,
//        1.0f,  1.0f,
//    };
}

It makes unit test failed

__block CGRect currentBounds;
runOnMainQueueWithoutDeadlocking(^{
currentBounds = self.bounds;
});
There is a certain rist to this solution, is not in main thread,  currentBounds = self.bounds; will be excute after dispatch asyc to main thread, the next time we use currentBounds, it may not be changed in main thread, or by the time we use, it changed by the main thread

@JohnJGL  now you can check edited Comment ...

@Sumit-2singh Perfect👍

This seems to fix it while not breaking anything either: https://github.com/BradLarson/GPUImage/pull/2533/files

According to the fix 2533 we change the source code and then rebuild the build.sh file, drag build/Release-iphone folder to your Xcode application project. Then it'd be very good to work!!

@lacyrhoades
hmm! It seems not good for SimpleVideoFilter project!
So!!!
I try to change some code,maybe it will change this problem!

I think it's not good
maybe someone have more good idea.
So i don't PR this Code

This solution works and must be accepted as the correct answer.


tuxpaint: 0.9.28 -> 0.9.31, refactor

Description of changes
Unblocks #276653.
Supersedes and closes #248007.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

After reviewing the docs on by-name, it seems that adding a package called SDL2_Pango is fine. So I'm doing it to match other SDL packages.


Service Stopped & Not Listening

Hi...
Can anyone help please?
On my RDP wrapper for Service Stopped & Not Listening...
After my antivirus cleaned RDP Wrapper by accident...
-Removed Antivirus
-Recopied required files.
-Updated PC
But now shows this after:

My setup is on:
64bit Win10 Pro, updated w/ latest Windows updates.

Try net start termService
Duplicate of https://github.com/stascorp/rdpwrap/issues/1823

Applicable to https://github.com/stascorp/rdpwrap/issues/1883

DAMN you are the MAN!!! Fixed!!


Boku no Natsuyasumi

tonyhax version: v1.3.2
Game name: Boku no Natsuyasumi
Game code: SCPS-10088
Executable hash: Md5 for .cue
a199cbd4917ed5fa752ed47d188276d3
Triggered at: I get the SOFTWARE TERMINATED CONSOLE MAY HAVE BEEN MODIFIED CALL 1-888-780-7690 immediately.
Is it an original disc or a burned copy?: A burned copy

Yay a new anti-piracy game I didn't know about! I'll have a fix in an hour

Fixed in v1.3.3. Please report back with results :)

The anti piracy screen is gone. but now I only get a black screen. Tried to re-burn the CD, but I received the same error...

What I mean by 'orientate your console vertical':

That was it! I also tried a different psone and it worked, but the vertical trick worked for me. Thank you so much!


Plex autoscan integration does not work

Hi there.
I can't get Plex_Autoscan to work when using the included python post processing script. With regular Plex_Autoscan the request included the .mkv file (which I am not keeping) so Plex_Autoscan never actually found that file so it never actually scan the path as expected.
I'm running radarr and sonarr with SMA integrated from these docker containers:
https://hub.docker.com/r/mdhiggins/radarr-sma
https://hub.docker.com/r/mdhiggins/sonarr-sma
I went into each of the containers and modified the /usr/local/bin/sma/sickbeard_mp4_automator/post_process/plex_autoscan.py file to include my URL:
root@daa096dd95b3:/# cat /usr/local/bin/sma/sickbeard_mp4_automator/post_process/plex_autoscan.py
#!/usr/bin/python
import os
import json
import requests

def main():
        print("Plex Autoscan Post Script")
        url = "http://domain.com:3467/111112rdlkhfkdjhfjkdsh"

        files = json.loads(os.environ.get('MH_FILES'))
        if not (files):
        print("Error - Did not find environment variables.")
...

I then configured Plex_Autoscan and SMA as per the notes here: https://github.com/Cloudbox/Cloudbox/wiki/Extra%3A-Sickbeard-MP4-Automator
Is that all that's needed? Do I need to manually call this script? If so, with which settings?
With the way it's currently configured my understanding was that SMA would take care of calling Plex_Autoscan with the path including the mp4 as a manual request but I don't see that request when I look at the autoscan logs.
Thanks!

Check this post here, it worked for me

Updated


Could not use "Spaces" feature with 's3ng' storage driver

Infinite Scale Spaces feature not working when storage is STORAGE_USERS_DRIVER=s3ng.
Other file operations do work using this s3ng setup.
This is how it looks like when I visit spaces page

Spaces must be compatible with Software level storage systems like S3 to support large scale deployments.

Can you please re test on our continuous deployment instance?
we have the spaces feature up and running in PROD with S3NG.
please be aware that you need to have the admin or space admin user role to create spaces.
Log in as admin and check the Admin UI users view. There you can see the user roles.
normally user roles are assigned manually or via OIDC claims.

THANKS! fixed it was having user roles error on my custom OIDC provider side


Expose Image.save_png_to_buffer method

Closes godotengine/godot-proposals#859.
There are other formats of course, but no apparent support within the engine for those yet.
Successfully save to buffer and then save with File:
extends Node2D

func _ready():
	var image = load("res://icon.png").get_data()
	var buffer = image.save_png_to_buffer()
	var file = File.new()
	file.open("res://icon_out.png", File.WRITE)
	file.store_buffer(buffer)
	file.close()

Image.save_png() could be used, yet this is mainly for networking purposes rather than local storage.

Unfortunately, save_png_to_buffer was first introduced in da0457fa29e1ea63f89b1e1d73e72c4dc80a9966 in 4.0, so this method wasn't present in 3.2 to cherry-pick this...
I could probably work on backporting this method to 3.2 specifically, but I need a blessing from @akien-mga to do so. 🙂

Seems fine, including adding the feature to 3.2.

Thanks!


housekeeping: header incorrectly said MS-PL instead of MIT

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
house keeping
What is the current behavior? (You can also link to an open issue here)
Headers incorrectly say MS-PL
What is the new behavior (if this is a feature change)?
Headers state the correct license (MIT)
What might this PR break?
It was a programmatic find and replace

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

I've opened a bug over here https://github.com/onovotny/MSBuildSdkExtras/issues/98 about the weird msbuid.sdk.extras error.

If we need to revert out that commit that updates msbuild.extras then happy to do so.

msbuild.sdk.extras changed a bit in the latest version, you now need to add additional properties to wpf/winforms applications
Support for WPF and Windows Forms requires an opt-in property to enable: Set ExtrasEnableWpfProjectSetup/ExtrasEnableWinFormsProjectSetup to true to include required references and default items.
This is in regards to the recent upgrade to Msbuild.sdk.extras
but the other bug isn't explained by those either though
You may want to revert that until those items can be sorted out by Oren as well.

How are the extras being added? Will fix next week, but it should work as an SDK reference instead of as a PackageReference.

Yeah at the moment it's set as a ProjectReference inside the Directory.Build.Props file.

I'd switch that to set the version in a global.json and then use the Sdk= attribute.
I'll still fix the underlying issue though.

@onovotny About to submit a new PR with the change to the newer format. Seemed to have solved those issues.

See https://github.com/reactiveui/ReactiveUI/pull/1695 implemented your recommendations.


iamb rooms not reachable

iamb rooms all have a canonical alias pointing to 0x.badd.cafe, but it looks like the certificate expired on December 8.
There seems to be a healthy Synapse 1.114.0 running, but the cert expired

Hello, we tried to solve the issue.
This is what we did:
Changed the Matrix homeserver URL and room addresses to use a placeholder domain (matrix.example.com) instead of the one with the expired certificate (0x.badd.cafe). This should be replaced with the actual working Synapse 1.114.0 server address.
You can review changes in this commit: https://github.com/mnmnk43434/ulyssa-iamb-381/commit/587fc0694ae6816c552fddc5bd44ea01b02135f4.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

Hi @thibaultamartin ! Thank you for letting me know. :pray: I've had a lot going on lately and haven't been able to pay attention to the matrix room. The certificate had been updated, but nginx hadn't reloaded it from disk. The homeserver should be reachable now.


Beautyzone - A website for salon

🛠 Issue 409
Closes #409
Description
A website template made using html , css , saas and javascript for salon .Overall it is a responsive salon website with navbar and other services like Home , Services , Price Table , Portfolio , client says ,news and book now  .
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have made this from my own
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
https://user-images.githubusercontent.com/84632701/214952741-33262b2e-baf3-4389-b6c8-bcea0993ff8e.mp4

Looks good to me! 🚀

thankyou


Homework<3>

Drift video:https://video.drift.com/v/abfI8remfMzhUXmxn5zABFcvN8iYWfzU15buZsjpSAc4/
Topics Covered:
Inheritance
Polymorphism
Interface and abstract class
-Exceptions
Approach :
We have implemented a Hospital Management System where the hierarchy is as follows:
-staff extends user
-patient extends user
-doctor extends staff
-nurse extends staff
We have built several classes to perform the following calculations
-Bill paid by the patient and the insurance company
-Insurance amount deducted from staff's salary
-Insurance amount calculation based on the age and smoking activities
Screenshots:

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
16

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
5

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


using with swagger-ui-express, trying to figure out how to handle the promise

Hello,
This tool works great. This issue isn't exactly about the tool. I'm trying to get help using the tool in a work flow.  I added it to my express API and I'm having trouble understanding what i might be missing with handling promise. I'm having trouble with an issue similar to this.
In my case, i need swagger-parser dereference to finish before i call   app.use('/apidocs', swaggerUi.serve, swaggerUi.setup(swaggerSpec));
In the solution above, i have to send a promise to everything that imports app.js. The rabbit hole is getting deep because in this big app, a lot of stuff is importing from app and I'm having to wrap all those things in .then.. calls.
I get the point of promises is to not block functions, but in this use case, I do want to block the app from starting before this is finished but not at the consequence of making everything connected to app a promise. The other thing i though about figuring out how to do is when i do npm run dev' and it watches my files for changes with nodemonsomehow tell it to run theswagger-cli` before starting or restarting the app. I'm not sure if this is possible.
Is there a way around this?
One work around is using swagger-cli instead, but i wan't able to get this working in a way where nodemon would restart the app after the yaml file was being generated. Part of my package.json:
"scripts": {
"dev": "NODE_ENV=development DEBUG=CUSTOM-DEBUG* nodemon --inspect ./bin/www",
"parseSwaggerDocs": "node_modules/swagger-cli/bin/swagger-cli.js bundle ./test/integration_test/openapi/global.yml -o ./openapi.yml -t yaml"
}

You can start your app after dereference() is done like this...
swaggerParser.dereference('my-api.yaml').then(startMyApp);

function startMyApp(swaggerSpec) {
    let app = express();
    app.use('/apidocs', swaggerUi.serve, swaggerUi.setup(swaggerSpec));
    app.listen(8080);
}


feat: updated semantic ado version variable name

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fuzzy local search

Issue: #47465

Problem

Searching to add users is fairly limited and case-sensitive.
Solution

This adds tests and improves search by making matching case-insensitive (for longer searches).
Testing

Create a user, "John Smith", username johns.
Create a project "demo" and try and add the newly created "John Smith" user to the project, try using their second name "Smith", it won't be found.
With this branch, the same instructions, the user will be found.
Also, try creating a user with a DisplayName that contains non-ASCII latin characters, and the same behaviour should also be observed.
Engineering Testing
Manual Testing

Followed the above.
Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

Moved to draft because I think I can come up with an improved index mechanism to reduce the memory for this fix.

@pmatseykanets @raulcabello sorry, dismissed the review by pushing new changes!


Fix a mismatch between Container and Toolbar gutters

Fixes a mismatch between Container and Toolbar gutters when the screen size is larger than md.

[x] I have followed (at least) the PR section of the contributing guide.

Why should it match?

Why should it match?

Without it the content area inside of the Toolbar is narrower than the content area inside of the container. Here is an example (AppBar > Toolbar at the top, then goes Container right below the AppBar):
Before

After

What do you think of reducing the container padding instead?
diff --git a/packages/material-ui/src/Container/Container.js b/packages/material-ui/src/Container/Container.js
index ea24a213b..82716c59c 100644
--- a/packages/material-ui/src/Container/Container.js
+++ b/packages/material-ui/src/Container/Container.js
@@ -17,10 +17,6 @@ export const styles = theme => ({
       paddingLeft: theme.spacing(3),
       paddingRight: theme.spacing(3),
     },
-    [theme.breakpoints.up('md')]: {
-      paddingLeft: theme.spacing(4),
-      paddingRight: theme.spacing(4),
-    },
   },
   /* Styles applied to the root element if `disableGutters={true}`. */
   disableGutters: {

@oliviertassinari that works for me. Just updated the PR.  Let's see how the argos screenshots would look like in this case.

Thanks


Scripting additions do not get installed correctly under a restrictive umask

I don't know what information I can provide to assist in diagnosing this issue. What I'm seeing is this:
I can send a window to a different space with yabai -m window --space 3, but then if I try to focus this space with yabai -m space --focus 3, nothing appears to happen. In fact, I haven't gotten any space command to have any effect: I've tried creating, destroying, focusing, and moving to other displays.
(Consequently, I'm currently using the regular Mission Control hotkeys for switching between spaces, but they are animated and therefore sloooow.)
I have tried running yabai in verbose mode, but nothing in particular shows up in the logs other than that it receives the message.
Please help me help you!

Moving spaces requires the scripting addition to be enabled.

Sorry, this issue was woefully underspecified. I have enabled the scripting addition (as far as I can tell – sudo yabai --install-sa was successful, and sudo yabai --check-sa returns 0.)
I'm running OS X 10.14.6, and yabai-v2.2.2.

When you start yabai, do you get a notification saying that the scripting addition was loaded?

No, that I don't! Furthermore, if I try to manually sudo yabai --load-sa, it tells me that

yabai: scripting-addition could not load, make sure SIP is disabled!

However, csrutil status tells me that

System Integrity Protection status: disabled.

I didn't specifically disable SIP for yabai – I did it at some point before in order to be able to use dtrace. I was fairly certain I disabled it entirely, but maybe I only disabled the parts of it that block dtrace?

Note that sudo is only required for --install-sa and --uninstall-sa.
yabai needs SIP for fs (filesystem) and debug to be disabled.
# from a recovery partition
csrutil enable --without debug --without fs

The problem persists even though I just now rebooted into recovery mode and disabled SIP entirely. I get this notification when starting yabai:

Don't know if this is of any help:
$ sudo yabai --uninstall-sa; echo "Status: $?"
Status: 0

$ sudo yabai --install-sa; echo "Status: $?"
Status: 0

$ yabai --check-sa; echo "Status: $?"
Status: 1

$ sudo yabai --check-sa; echo "Status: $?"
Status: 0

$ yabai --load-sa; echo "Status: $?"
Status: 1

$ sudo yabai --load-sa; echo "Status: $?"
yabai: scripting-addition could not load, make sure SIP is disabled!
Status: 1

Try killing Dock after installing. sudo yabai --install-sa; pkill -x Dock

That did not help. However, I looked into what yabai/src/osax/sa.m is doing, to see if I can figure out where something is going wrong. I noticed it attempts to create directories with permissions 0755. On my end, however, they seem to have been created with 750:
$ sudo stat /Library/ScriptingAdditions/yabai.osax
16777220 34182272 drwxr-x--- 3 root wheel 0 96 "Feb  4 12:23:02 2020" "Feb  4 12:19:25 2020" "Feb  4 12:19:25 2020" "Feb  4 12:19:25 2020" 4096 0 0 /Library/ScriptingAdditions/yabai.osax

which is probably because
$ umask
0007

Indeed, if I set umask 0000 and then try running sudo yabai --install-sa, I am presented with the following notification:

And! Best of all: things now work.

Thank you for your patient assistance. Before closing this issue, I think something should be done about the very non-obvious failure when running on a restrictive umask. Would you prefer it if this was documented in in the installation instructions, or that the --install-sa command itself fixed the umask before running? (Or at least gave a sensible error when it is incorrectly set?)

I think @koekeishiya can fix this quite simply by explicitly setting the umask for the scripting addition when installing it.


Snippets are not respecting trailing spaces

As an example, type =, then tab to produce the <- snippet and the cursor ends up after the - with no trailing space.
The snippet correctly uses $end$ which should make the whitespace get noticed.
<Code Language="R">
        <![CDATA[ <- $end$]]>
 </Code>

@MikhailArkhipov - any update on this bug? Thx.

It is basically formatting. After snippet insertion VS IDE calls format range so snippet would appear according to the selected formatting options rather than how it specified in the snippet file. Formatter trims trailing spaces by default.  It is technically possible to special case standalone operators and suppress formatting in this particular case.

So question is: is this specifically about R operator snippets or is it general request to respect custom formatting with possible trailing spaces in all snippets. The former is doable, the latter is unclear how to handle since reformatting of snippets at insertion is standard VS IDE behavior.

Thanks for explaining the cause of the issue. I've not had the same issue
with C# but perhaps that's just because I haven't needed trailing spaces in
a snippet.
Could there be a way of altering the formatting rules so that it doesn't
get rid of the space after <-  ? It's rare to put a line break right after
that token so it seems to be a reasonable exception for R code
soecifically.
On Mon, 27 Mar 2017, 19:04 Mikhail Arkhipov<EMAIL_ADDRESS>wrote:

So question is: is this specifically about R operator snippets or is it
general request to respect custom formatting with possible trailing spaces
in all snippets. The former is doable, the latter is unclear how to handle
since reformatting of snippets at insertion is standard VS IDE behavior.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/2610#issuecomment-289534992,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADAiy0izofVtA4p_ryW6iuYIcHkcxY3Eks5rp_oTgaJpZM4Klg6G
.

https://github.com/Microsoft/RTVS/commit/7ecfc09d1653ce711bcea3e71b30b8972016edda

Thanks, just re-read your comment and realise what you meant. Your fix is a very sensible balance as presumably it will work for magrittr lines too if somebody has a snippet for that like:
blah %>%
select
The main motivation was specifically for <- though, so great that it works for that


1-click-deployment of fullstack web app

Is your feature request related to a problem? Please describe:
It seems as if there is no real / documented solution for deploying both frontend and backend.
Describe the solution you'd like:
The deploy-button on the top-right allows me to deploy everything with one click, also the server. I can choose from a provider you really know can deal with the deployment triggered by bolt.
Describe alternatives you've considered:

Additional context:
For me it is one of THE most important issues, that I can not just deploy everything at once. This makes your service - as cool as it is - almost unusable for any serious work for me. I can not develop a fullstack app in between 2 different environments and sync between them easily. I love bolt BTW

Thanks for the report and patience during this beta period! Tracking all deployment related bug fixes in issue #499. We do have plans to add additional deployment providers in the future. In the meantime, you can also use StackBlitz.com to push your code into Github or download it, and deploy manually!