Soft link causes the same source file to be edited in two windows at the same time

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.72.2 (Universal)
commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075
date: 2022-10-12T22:16:30.254Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0
OS Version: macOS Monterey 12.5.1

After vscode opens the project in the soft link directory, jumps to the definition in the code file, a new file will be opened, and the path of the new file is displayed as the file in the original directory.
Modifying one of the open files and saving them will trigger another unmodified "same" file window to also save, resulting in a messy file.
Steps to Reproduce:

Create a directory O, and create a soft link directory S to O;
Create project proj under O, add project file main.cpp, and open S/proj in vscode;
Open the file main.cpp through the left resource manager of vscode, add functions foo and bar, and call foo in bar;
Hold down ctrl and click foo in the bar, the expectation is to jump to the definition of foo in the same file, but it actually opens a new tab, and the display path is O/proj/main.cpp;

This works "as designed", we do not resolve symbolic links when you open an editor for performance reasons so you can end up having 2 distinct editors opened pointing to the same file with different buffers.


Printing Query Code Instead of Query Results

First thank you for your work on this.
The plugin only prints the first result of queried blocks.

Figured this out.  You have to scroll to the bottom of the page so the whole thing loads before you export.  When you do this, the export will show all the contents.


Update limits-and-config.md

Updated the Runs to actions and added Interim limits

@MSFTMan : Thanks for your contribution! The author(s) have been notified to review your proposed change. @msftman

@MSFTMan : Thanks for your contribution! The author(s) have been notified to review your proposed change. @msftman


Implement pen functionality for X11

Description
Implements pen with pressure, eraser, and hover.
https://github.com/FreeRDP/FreeRDP/issues/7260

Testing
How to test per the above picture:

have an X11 desktop with a tablet and pen/stylus that has a tip and an eraser (e.g. MPP pens, like microsoft surface)
use xfreerdp to connect to a windows machine with powerpoint, and use the +multitouch option
in powerpoint, pick the black pen and make it as thick as possible

zoom in to 400% to make the pressure sensitivity obvious
draw a line varying the pressure and see the thickness of the line be affected
reduce the line thickness and try writing some words. Ensure that the writing is smooth and nothing skips
hover the pen above the screen and move it around - notice the cursor is a black dot that follows the pen around
flip over to the eraser and hover it over the screen. observe it have the eraser tip cursor like shown here:

try erasing some of your previous lines
switch back and forth with pen and eraser like you normally would writing and observe inking/erasing working as expected

Known issues:

After using the eraser and going from pen down to hovering, the hover icon might switch over to the black dot, but pressing the eraser back into the screen will continue to use the eraser properly. This was somewhat mitigated by trying to force an UPDATE | CANCEL immediately after pen up, but it impacted performance of writing (skips parts if writing fast). There something tricky about managing the state transition with penFlags for using the eraser (PEN_FLAG_INVERTED). See the 2nd commit for additional context.

Exiting the hover state feels a little hacky because X11 does not give you any event to say "pen proximity is away from screen". It simply stops giving you motion events. So exiting the hover state was done whenever any other non-hover input event happened.
In my testing, I had to add pen hover for switching back and forth with the pen tip and eraser tip to work. On a regular windows device like a surface pro, you always see the pen hover cursors before and after any inking or erasing events, so I think it makes sense.
Design notes
Pen behaviour is very similar to multitouch, except while X11 gives explicit events for XI_TouchBegin, XI_TouchUpdate, and XI_TouchEnd, as far as I could see for pen stuff you only get XI_ButtonPress, XI_Motion, and XI_ButtonRelease, and it's up to you to correctly interpret what's going on. Furthermore, a challenge is that pen not only has a active state, but a hovering state. So a complete picture would be PenHoverBegin, PenHoverUpdate, PenHoverEnd, PenDrawBegin, PenDrawUpdate, PenDrawEnd, all while complying with [MS-RDPEI] <IP_ADDRESS> Touch Contact State Transitions

Some helpful tools to see what all the states are before starting coding:

libinput debug-events
xinput test-xi2 --root

Possibly useful observations from xinput:

I see EVENT type 1 (DeviceChanged) every time between switching to a different input device (mouse, touch, pen, keyboard)
Initially only a "ipts 045E:001F Pen Pen (0)" was seen on my Surface Pro 5 (surface linux kernel with ipts), but on first time using the eraser I got a HierarchyChanged event where a "ipts ... Pen Eraser ..."  device was added.

should probably detect HierarchyChanged and re-initialize xf_input_init when that happens

Additional info and notes:

Browsing various X11 touch/stylus implementations, they all seem to rely on the fact that pressure is always the 3rd valuator value. In my implementation, I used this index, but we can also do a string comparison because the valuator name should have "Pressure" in it.
max_pressure must be determined from querying the input devices, so this should also be refreshed whenever devices are added/removed
rdpei has MAX_PEN_CONTACTS = 4 so simultaneous pen contacts are apparently possible, but directly on my Surface Pro 5 in windows 10 this does not seem to work with the 2 pens I have. It may need special devices like Surface Hub or similar.

Also here's a branch for the first commit of the patch on 2.10.0: https://github.com/digitalsignalperson/FreeRDP/tree/2.10.0_archlinux-pendev3

@freerdp-bot test please

@freerdp-bot can you test again seems like TestConnect failed for no reason !

@freerdp-bot come on, test properly please

I just pushed another commit fixing these issues

-Wreturn-type (was missing a return 0)
-Wunused-variable
and removed the temporary comment

https://github.com/digitalsignalperson/FreeRDP/commit/c4e03e0dafeef310f745f8e1076ca8755ed87c8e
Do I need to open a new PR to add it?
I'm not sure why my scan-build said no new issues but the CI build identified them.

@digitalsignalperson please a new pr yes

Hi, I recently tested freerdp-nightly but it seems to recognize my device as a mouse.
$ xsetwacom list
Wacom HID 524D Pen stylus       	id: 9	type: STYLUS
Wacom HID 524D Pen eraser       	id: 15	type: ERASER
Wacom HID 524D Finger touch     	id: 10	type: TOUCH
Wacom One by Wacom M Pen stylus 	id: 16	type: STYLUS
Wacom One by Wacom M Pen eraser 	id: 17	type: ERASER

I think it may be that this part of the code is not compatible with my device name.
https://github.com/FreeRDP/FreeRDP/blob/9c7a541d9c0acb9728cebf344b363f7ea797f920/client/X11/xf_input.c#L145-L165
It should ideally be case-insensitive to match "Pen stylus".

@ichaoX it might work changing strstr to strcasestr. However some other stuff broke here that I'm not sure if it's been fixed: https://github.com/FreeRDP/FreeRDP/pull/9086#issuecomment-1603171372
We should document the current status here: https://github.com/FreeRDP/FreeRDP/issues/7260


PDFView.Confiurator is not being found when signing a release app bundle.

I created a dynamic feature module that uses this library while keeping it in the app base module.
The problem happens when I try to sign an app bundle and dex guard is being used even though I am whitelisting the class as mentioned in the readme file in this repo.
I am getting this error:
Cannot access class 'com.github.barteksc.pdfviewer.PDFView.Configurator'. Check your module classpath for missing or conflicting dependencies
Note: The PDFView class is visible with no issues.
The related rules in proguard file.
-keep class com.shockwave.**
-keepclassmembers class com.shockwave.** { *; }

A glimpse of the build gradle file in app module:
com.github.barteksc:android-pdf-viewer

android {
    buildTypes {
        release {
            proguardFile getDefaultDexGuardFile('dexguard-release.pro')
            proguardFile getDefaultDexGuardFile('dexguard-androidx.pro')
            proguardFile 'proguard-rules.txt'
        }
    }
dynamicFeatures = [":pdf]
}
dependencies {
  api 'com.github.barteksc:android-pdf-viewer:2.8.2'
}

and in the dynamic feature module gradle file:
apply plugin: "com.android.dynamic-feature"
android {
buildTypes {
        release {
            proguardFile 'proguard-rules.txt'
        }
    }
}

dependencies {
    implementation project(":app")
}

I'm having the same issue. Did you find a resolution?

@paulsUsername
sadly not
I had to take the dependency out of the base class to get around this weird error.


Fixes to mia (OMA issue #482)

Fixes to mia (OMA issue #482)

diversity functions - warning that tells that some indices where not calculated
documentation  fix on runRDA()

it only failed here https://github.com/microbiome/mia/actions/runs/9741095621/job/26879566844
It looks like some permission issue in the ubuntu


NC interactions on oxygen

These interactions could make up about 5% of events for a SN burst detection in Super-K. A theoretical calculation of the cross-sections was performed by Langanke et al., arXiv:nucl-th/9511032; experimental data appears to be non-existent.
The main issues here are

the low energy of events (typically gammas with just above 5 MeV)
large uncertainty of the cross-section

and as a result, this interaction is not currently considered in the HK Design Report.

An updated theoretical calculation was published by Suzuki et al. in 2018.
I’ve implemented the CC cross-sections calculated there (not yet on the master branch; I’d like to do some more checks before I push them) based on this follow-up paper by Nakazato et al., but that paper doesn’t make use of the NC cross-sections.


The weekly data shown is incorrect.

The summation shown for weely data is wrong.
The individual view shows all transactions have been made from 19th March. In weekly view, 3 transactions are shown in the past week.

The issue is double stores decimals as fractions. Hence while performing subtraction, it loses precision. Therefore using an alias, perform SUM on credit and debit separately and then use BigDecimal to perform subtraction.

Fixed in dev


why dapr not suport ms sql server by binding

hi
I have a question , why dapr not suport ms sql server by binding, I want to exec T-SQL  in ms sql server by dapr client

No one so far has expressed interest in it. But we welcome contributions if someone wants to make a PR!

Closing this in favor of #3414 which has more detail.


Handle Escaped Quotes in Metadata and Models values

Close #1334
We used the wrong helper function for metadata and model value parsing. I also included a updated unit test that is pretty detailed.
TEST:

Import the seqJson into the SeqN editor.
Verify the seqJson output matches the file  input, which means roundtripping is working.

escaped_metadata.json

@joswig looks good. Here is my test sequence:
C ECHO "Spac"
@METADATA "Key" {    " " : [      1] , "blah" : false  }


Default value of alpha wrong in class initialization.

In the init for class GLM the default value of alpha is set to 0.05 but should be 0.5

oh boy, how did this happen.
Can you make a small PR to fix it?

closed by https://github.com/pavanramkumar/pyglmnet/pull/146


Fix $cookie['expires'] double calculation error

Description
$this->setCookie(
				$cookie['name'],
				$cookie['value'],
				$cookie['expires'], //The expiration time will be double-counted
				$cookie['domain'],
				$cookie['path'],
				'', // prefix
				$cookie['secure'],
				$cookie['httponly']
			);

I don't understand what is happening and how this fixes it. Can you elaborate?

Please run the following code to reproduce the error.
$this->response->setCookie('my_cookie', '123456', DAY);
$redirectResponse = Services::redirectResponse()->withCookies();

$responseCookie = $this->response->getCookie('my_cookie');
$redirectResponseCookie = $redirectResponse->getCookie('my_cookie');

echo $responseCookie['expires'] . '
' . $redirectResponseCookie['expires'];
if($responseCookie['expires'] != $redirectResponseCookie['expires']) {
	echo '
This is a bug.';
}

Please run the following code in the controller to reproduce the error.
$this->response->setCookie('my_cookie', '123456', DAY);
$redirectResponse = Services::redirectResponse()->withCookies();
$responseCookie = $this->response->getCookie('my_cookie');
$redirectResponseCookie = $redirectResponse->getCookie('my_cookie');
echo $responseCookie['expires'] . '
' . $redirectResponseCookie['expires'];
if($responseCookie['expires'] != $redirectResponseCookie['expires']) {
   echo '
This is a bug.';
}
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年7月29日(星期三) 晚上7:11
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [codeigniter4/CodeIgniter4] Fix $cookie['expires'] double calculation error (#3420)
I don't understand what is happening and how this fixes it. Can you elaborate?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Good catch! Thank you.


tunein = MusicService('TuneIn') gives error

File "/home/therol6/PycharmProjects5/sonos/sonos_test3.py", line 5, in
tunein = MusicService('TuneIn')
File "/home/therol6/.local/lib/python3.6/site-packages/soco/music_services/music_service.py", line 354, in init
"No account found for service: '%s'" % service_name)
soco.exceptions.MusicServiceException: No account found for service: 'TuneIn'
This is described in the documentation. But what do i do wrong
Version 0.19
python 3.6

It seems there is a bug. I was troubleshooting it last night. I will post a PR and I workaround as soon as I can.

I think the problem is  MusicService.get_subscribed_services_names() returns an empty list and as such TuneIn and Pandora and other services that normally works will fail with said message. I can reproduce  on 3.7.7  & 0.19

I believe this is now fixed in master. Pleas test it if you have the option and feel free to re-open if the issue remains.
The bug was actually a little a musing. It turned out to be a simple indentation error. Somewhere in the code there is a loop over all the music services that the speaker report that they support. Towards the end was a block stating that there is always an TuneIn service, so it was added no matter what. However, this block was inside the loop over reported music services and had been there since the original implementation. It was just never noticed, because only recently did the speakers stop reporting which services they support, so the loop was never entered.
Oh well. Hopefully should work now.

Still get the following error.
Traceback (most recent call last):
File "sonos_test_music_libary.py", line 221, in
tunein = MusicService('TuneIn')
File "/home/therol6/.local/lib/python3.6/site-packages/soco/music_services/music_service.py", line 354, in init
"No account found for service: '%s'" % service_name)
soco.exceptions.MusicServiceException: No account found for service: 'TuneIn'

@rvhu-cl did you test against the developement code? Because the fix is not yet released.

I can confirm it works with current master:
`Python 3.7.1 (v3.7.1:260ec2c36a, Oct 20 2018, 14:57:15) [MSC v.1915 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

from soco.music_services import MusicService
tunein = MusicService('TuneIn')
print(tunein)
<MusicService 'TuneIn' at 0x226f7b8>

`

Great. It should be out in less than 2 weeks as SoCo 0.20.


Example for facebook-omniauth wrong

When I follow the example (https://github.com/plataformatec/devise/wiki/OmniAuth:-Overview) I get
undefined method `user_omniauth_authorize_path' for #<#Class:0xc3df32c:0xc3de300>
from
<%= link_to "Sign in with Facebook", user_omniauth_authorize_path(:facebook) %>
Which I understand - because where would that route come from after all???

Just to bring up this old issue.. is there a reason the _url helpers aren't included? I want to run the devise controllers over https so it would make sense to generate absolute paths, right? (I'm not using https for the whole application, only certain controllers)

I think they are not available simply because someone haven't implemented them yet.


Enable collection initializer syntax for View

Would be nice to add support for the more structural/declarative style allowed by
collection initializers when creating views, such as:
	var view = new View {
		new View {
			new View()
		},
		new View()
	};

The only reason this currently doesn't work is that View does not implement IEnumerable:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS1922	Cannot initialize type 'View' with a collection initializer because it does not implement 'System.Collections.IEnumerable'	UnitTests	...Terminal.Gui\UnitTests\ViewTests.cs	122	Active

The view could implement that interface explicitly like so:
IEnumerator IEnumerable.GetEnumerator () => Subviews.GetEnumerator ();

And that alone would enable this usage, since the class already provides an Add(View) method, which is
the other thing the compiler looks for in this initialization pattern.

A compromise has been agreed in principle to use params constructor for this use case.  See : https://github.com/gui-cs/Terminal.Gui/pull/2079#issuecomment-1283077623 for full discussion

But he prefers overload your own class. Maybe you can submit a new PR with your params suggestion @tznind .


feat(monitor): mount/unmount syscall support

Purpose of PR?:
This PR adds support for mount and unmount syscalls. after merging the KubeArmor will be able to trace all events of mount/unmount system call. It also makes both the syscalls auditable, so both can be audit at syscall level using an audit policy.
syscalls:
  matchSyscalls:
  - syscall:
    - mount
    - umount2

Fixes #1011
Does this PR introduce a breaking change?
No
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[x] Fixes #1011
[x] New feature (non-breaking change which adds functionality)
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Commit has integration tests

Ensure that we have a document explaining:

Sample mount/umount telemetry events
output of karmor summary with corresponding syscalls
Ginkgo tests.

In KubeArmor, default apparmor profile for k8s workload denies mount/unmount.
@nyrahul  if we want to add automated test for mount/unmount syscall, the possible ways are:

edit native apparmor profile under /etc/apparmor.d after applying the audit policy.
don't modify the profile and just check for the expected alert from DefaultPosture with Action: Block.

for ref., here i'm pasting the telemetry generated during the manual testing.
i'm using the manually edited custom apparmor profile for the testing pod.

apparmor profile
## == Managed by KubeArmor == ##
#include <tunables/global>

## == Dispatcher profile START == ##
profile kubearmor-test-mount-ubuntu-1-deployment-ubuntu-1-container flags=(attach_disconnected,mediate_deleted) {
        ## == PRE START == ##
        #include <abstractions/base>
                                file,
                                network,
                                capability,
                                mount,
                                umount,
        ## == PRE END == ##

        ## == File/Dir START == ##
  ## == File/Dir END == ##
        ## == DISPATCHER START == ##
        ## == DISPATCHER END == ##

  ## == Network START == ##
  ## == Network END == ##

  ## == Capabilities START == ##
  ## == Capabilities END == ##

        ## == Native Policy START == ##

        ## == Native Policy END == ##

        ## == POST START == ##
        /lib/x86_64-linux-gnu/{*,**} rm,

        deny @{PROC}/{*,**^[0-9*],sys/kernel/shm*} wkx,
        deny @{PROC}/sysrq-trigger rwklx,
        deny @{PROC}/mem rwklx,
        deny @{PROC}/kmem rwklx,
        deny @{PROC}/kcore rwklx,

        # deny mount,

        deny /sys/[^f]*/** wklx,
        deny /sys/f[^s]*/** wklx,
        deny /sys/fs/[^c]*/** wklx,
        deny /sys/fs/c[^g]*/** wklx,
        deny /sys/fs/cg[^r]*/** wklx,
        deny /sys/firmware/efi/efivars/** rwklx,
        deny /sys/kernel/security/** rwklx,

        ## == POST END == ##
}
## == Dispatcher profile END == ##

## == FromSource per binary profiles START == ##
## == FromSource per binary profiles END == ##

## == Templates section START == ##

audit policy for mount and umount syscall

policy.yaml
apiVersion: security.kubearmor.com/v1
kind: KubeArmorPolicy
metadata:
  name: audit-all-mount-unmount
  namespace: test-mount
spec:
  severity: 3
  selector:
    matchLabels:
      container: ubuntu-1
  syscalls:
    matchSyscalls:
    - syscall:
      - mount
      - umount2
  action: Audit

telemetry alerts
$ root@ubuntu-1-deployment-868d6fd7bf-4qqkh:/# mount --bind /home /mnt/test

Alert
== Alert / 2023-01-23 10:30:41.707721 ==
ClusterName: default
HostName: hp
NamespaceName: test-mount
PodName: ubuntu-1-deployment-868d6fd7bf-4qqkh
Labels: container=ubuntu-1
ContainerName: ubuntu-1-container
ContainerID: ac3ab371caac7afa227c1229812c47514aa96e17237049d866515386c84c928c
ContainerImage: docker.io/kubearmor/ubuntu-w-utils:0.1@sha256:b4693b003ed1fbf7f5ef2c8b9b3f96fd853c30e1b39549cf98bd772fbd99e260
Type: MatchedPolicy
PolicyName: audit-all-mount-unmount
Severity: 3
Source: /bin/mount --bind /home /mnt/test
Operation: Syscall
Data: syscall=SYS_MOUNT source=/home target=/mnt/test filesystem=none mountflag=0 data=
Result: Passed
HostPID: 62346
HostPPID: 60252
PID: 142
PPID: 125
ParentProcessName: /bin/bash
ProcessName: /bin/mount

root@ubuntu-1-deployment-868d6fd7bf-4qqkh:/# umount /mnt/test

Alert
== Alert / 2023-01-23 10:31:00.718011 ==
ClusterName: default
HostName: hp
NamespaceName: test-mount
PodName: ubuntu-1-deployment-868d6fd7bf-4qqkh
Labels: container=ubuntu-1
ContainerName: ubuntu-1-container
ContainerID: ac3ab371caac7afa227c1229812c47514aa96e17237049d866515386c84c928c
ContainerImage: docker.io/kubearmor/ubuntu-w-utils:0.1@sha256:b4693b003ed1fbf7f5ef2c8b9b3f96fd853c30e1b39549cf98bd772fbd99e260
Type: MatchedPolicy
PolicyName: audit-all-mount-unmount
Severity: 3
Source: /bin/umount /mnt/test
Operation: Syscall
Data: syscall=SYS_UMOUNT2 target=/mnt/test flag=0
Result: Passed
HostPID: 62369
HostPPID: 60252
PID: 143
PPID: 125
ParentProcessName: /bin/bash
ProcessName: /bin/umount

Can you please resolve the conflicts?


Recalbox buildroot

corrected translation of saved values

Hi @gmello77, you must create a PR on the recalbox repository (not on the Aloshi one). You can close this one and make a new PR on the recalbox-buildroot branch of https://github.com/recalbox/recalbox-emulationstation (you should see only your commits in the diff list while creating the PR)
Thanks anyway :)


onError is not called when a network error occurs in query and batch methods

When a network error occurs, a TypeError is thrown and the fetch method passes it to the onError handler, but query doesn't. In order to be able to handle all possible errors, I guess the behaviour should be aligned between the two methods.
Here how the catch looks like in the three methods:
  public async query(query?: OdataQuery) {
    try {
      // ...
    } catch (ex) {
      throw ex;
    } finally {
      this.requests = [];
    }
  }

  public async fetch(query?: OdataQuery) {
    try {
      // ...
    } catch (ex) {
      this.config.onError(this, ex);
      throw ex;
    } finally {
      // ...
    }
  }

  public async batch(query?: OdataQuery) {
    try {
      // ...
    } catch (ex) {
      throw ex;
    } finally {
      // ...
    }
  }

Are you open to a PR that fixes this?
Thanks!

It's not called for network errors, so if you block the API call in your browser's console, fetch will throw a TypeError and the onError won't be called.
  public async query(query?: OdataQuery) {
    try {
      this.config.onStart(this);
      const response: Response[] = await this.getFetch(query); // <------ This throw a TypeError when the API cannot be reached
      const json = await Promise.all(
        response.map(
          async (res) => {
            if (res.status >= 400) {
              this.config.onError(this, res);
              throw res;
            } else if (res.ok && res.json) {
              try {
                this.config.onFinish(this, res);
                const data = await res.json();
                return data[this.config.fragment] || data;
              } catch (ex) {
                return res;
              }
            } else {
              return await res.text();
            }
          },
        ),
      );
      return json.length > 1 ? json : json[0];
    } catch (ex) {
      throw ex;
    } finally {
      this.requests = [];
    }
  }

hi, yeah might be a good improvement. I would accept a PR for this.


Add example print message

In Python, the way to log messages would be:
import logging

logging.debug("")
logging.info("")
logging.error("")

The output of print("") is visible in the log of the development/production server.
Here we have confirmed that the server logs are shown properly in the console and that ELK capture them. I don't think that there is need to show the user how to print messages in Python.


I'd like to a list of quests I meet the requirements for

Is your feature request related to a problem? Please describe.
Quality of life addition to the client
Describe the solution you'd like
A program that checks your stats and then checks what quests are be able to be done.
Additional features
Sort by alphabet and/or difficulty
Maybe clicking on a quest to open a guide?

Duplicate of #1905


Add AWS S3 support for stable/spark-history-server

Signed-off-by: Will Son<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR adds AWS S3 support for stable/spark-history-server chart.
Special notes for your reviewer:
I avoided making breaking changes for other existing users for now and just added enabling S3 functionality on top of the current implementation similar to the GCS one. The detail of its usage is written down in the README.md
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/lgtm


[Question] Code coverage ?

Hi Michael,
Thanks a lot for your library.
I am using it daily and I am about to create my own lib using yours as guidelines / model.
The only thing that I'd love to add is code coverage. Did you try ? If not, where would you begin?
If I manage to get it work, I will open a proper issue and PR so you can see if you want it too.
M.

Hi,
Great idea and no, I've not tried it yet. I guess I would begin here: https://karma-runner.github.io/0.8/config/coverage.html - seems like it would not be too difficult since we already use Karma for the tests.
And yeah, a PR would be most welcome if you get it working!


File search in windows

Hello, I was looking for a better alternative for Alfred on windows. After trying all the available software, I liked Cerebro and Wox Launcher. But an important feature missing in Cerebro is system wide file search. Wox launcher has this feature by using Everything plugin. I suggest same can be done for Cerebro.

in #45 you can find a link @reflog's version of everything plugin that is work-in-progress. So, you can wait some time or help him with development :)


Increment Swift version

In Swift 5.7, a new feature called module aliasing that allows us to have multiple modules with the same name changing these names by an alias. If we want to use this module aliasing feature, the packages of your project's dependencies need to be Swift 5.4 version or over to compile.
So I would like to propose incrementing the Swift version to 5.4

Sorry for my late reply. Thank you!


Add ConfigUpdater interface

To be able to support dynamic configuration providers, an ConfigUpdater interface defines the Update function which can be used to change/extend a loaded static configuration. The Container interface was changed to use the ConfigUpdater interface so the Update function is available in normal runtime callbacks.

lgtm :+1:


[PyTorch] Activation operations

Description
This PR extends the operation-based API (see https://github.com/NVIDIA/TransformerEngine/pull/707) with activation functions like ReLU, GELU, and SwiGLU.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes

Add ReLU, GELU, ReGLU, GEGLU, SwiGLU operations

Checklist:

[x] I have read and followed the contributing guidelines
[ ] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

Merging with approval from @ptrendx and @ksivaman.


resizing images to 256x256

https://github.com/Confusezius/Deep-Metric-Learning-Baselines/blob/dbcd5abe2966c3398b893664bba2d61075b628d9/datasets.py#L397
I believe there's a bug in BaseTripletDataset.transform: during training, images are not resized to 256x256 before taking crops.
Interestingly, this did not seem to affect results by much, at least when I tested it in Online Products with Margin loss...

Thank you for your comment.
I believe that this was by choice. Not normalizing to 256 helps as an extra augmentation.
Nevertheless, thank you for letting us know that it does not change the result much.

That's good to know, thanks!


RTD Fixes

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Tests have been added that prove the fix is effective or that the feature works.
[ ] New and existing tests pass locally with the changes.
[ ] Code follows the style guidelines of this project (PEP8, clean code, linter).
[ ] A self-review of the code has been done.

What is the ticket or issue number?

Pull Request Description

Does this introduce a breaking change?

[ ] Yes
[ ] No

How Has This Been Tested?

LMK what you think about these changes @avanbrunt-cb


Restyling the login component

Fixes  #1381

@iFlameing could you please rebase and add a changelog entry?


Add dry-run ability to servicelog posting

This PR adds a --dry-run parameter to osdctl servicelog post that prints the contents of the servicelog that would be posted, without actually posting it.
With some templates featuring multiple parameters comprising a sentence, I've found a dry-run desirable in order to be able to re-read a servicelog before I actually post it to ensure it makes grammatical sense.

/lgtm

/lgtm
/approve


Pitch Black theme.reg

Ain't that just like Pitch Black theme.reg, minus the credits / areas affected?
(there were no "open-source" documentation before /u/Egg-Tricky's reddit post 2 months ago and mine expanding upon it)
And as .reg it should be way more convenient and clear in scope
(I even shared it with a Violet accent template)
there's nothing dynamic being calculated to require a scripting language at all.

Thanks for the documentation. Ill edit the code and add comments to make this more human readable so people can know what theyre executing on their computers.


Postgres functions

Is it possible to use Postgres functions?

Unfortunately, no. Postlite just acts as a pass through so only SQL supported by SQLite will work.


[v1] Prop aria-describedby did not match

Describe the bug and the expected behavior
Trying out v1 and getting error:

Warning: Prop aria-describedby did not match.
Server: ":R1cop:-form-item-description" Client: ":Rkop:-form-item-description"

Appears that server IDs do not match client IDs.
Conform version
v1.0.0
Steps to Reproduce the Bug or Issue

const MewPasswordForm = () => {
  const fetcher = useFetcher<typeof action>();

  const [form, fields] = useForm({
    lastResult: fetcher.data,
    onValidate({ formData }) {
      return parseWithZod(formData, { schema: FormDataZodSchema });
    },
  });

  const submitting = fetcher.state === 'submitting';

  return (
    <FormProvider context={form.context}>
      <fetcher.Form
        className={formRoot()}
        method="post"
        {...getFormProps(form)}
      >
        {form.errors ? (

{form.errors[0]}

        ) : null}

        <Form.Field input={fields.password}>
          <Form.Item>
            <Form.Label>New Password</Form.Label>
            <Form.Control>
              <Input
                disabled={submitting}
                placeholder=""
                type="password"
                {...getInputProps(fields.password)}
              />
            </Form.Control>
            <Form.Message />
          </Form.Item>
        </Form.Field>

        <Wrap>
          <Button
            alignSelf="flex-start"
            className={css({
              display: 'block',
              width: '100%',
            })}
            disabled={submitting}
            name="intent"
            type="submit"
            value="new-password"
          >
            Update Password
          </Button>
        </Wrap>
      </fetcher.Form>
    </FormProvider>
  );
};

What browsers are you seeing the problem on?
No response
Screenshots or Videos
No response
Additional context
No response

It looks like there needs to be a utility getLabelProps just like there is getInputProps?

All ids generated are just prefixed with an id created using the useId() hook. The id is stable unless the component tree is different as mentioned in the docs:

With server rendering, useId requires an identical component tree on the server and the client. If the trees you render on the server and the client don’t match exactly, the generated IDs won’t match.

I am not sure if Conform has any impact here as the only thing you rendered using Conform is the <FormProvider /> and it is working fine during my tests so far. It would be great if you can share a repo on codesandbox / stackblitz.

I figured out the issue and it was indeed a mistake in my own code. Thanks!

The v1.0.0 is doing something weird.
The moment I add useForm from v1.0.0 to my component tree, a bunch of other components (entirely unrelated to Conform) start breaking with errors like:

Prop id did not match. Server: "accordion::R4lgp::trigger:What makes a good source article?" Client: "accordion::R29gp::trigger:What makes a good source article?"

I have reduced the entire example down to just:
import { Accordion as ArkAccordion } from '@ark-ui/react/accordion';
import { useForm } from '@conform-to/react';
import { useFetcher } from '@remix-run/react';

const Route = () => {
  useFetcher();

  useForm({});

  return <ArkAccordion collapsible />;
};

export default Route;

Please try this solution suggested from the original issue or upgrade to remix v2.5 to see if it is resolved.

Confirming that this is no longer an issue in Remix v2.5. Thank you!


Use remapped ESC to toggle ubiquitous vi mode

Ubiquitous vi mode can be toggled using the Escape key.
However, with CONTROL_L to CONTROL_L (CONTROL_L only to send ESCAPE) enabled, hitting CONTROL_L by itself doesn't toggle ubiquitous vi mode.

You should make new setting.
Please use this private.xml:
https://github.com/tekezo/Files/blob/master/Karabiner/private.ubiq_toggle_control/private.xml
How to use private.xml:
https://pqrs.org/osx/karabiner/document.html#privatexml


Remotefs bak

Implementation of the remote file system for basic docker cp and docker build functionality.

@gupta-ak,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

FYI this should also resolve #42.


react-ts template uses development bundle of React

I apologize for making this an issue when it's more applicable for a google search, but I can't find anything about this.
TL;DR
How do you build with the prod version of React?
When running the vite build command that comes with the react-ts template, React is bundled in development mode, causing a 267kb js file.
Reproduction Steps

yarn init vite-app ts-react --template react-ts
cd ts-react
yarn
yarn build
Look at dist/_assets/ and the size of the JS file.

The file has
/** @license React v16.13.0
 * react-dom.development.js
 *
 * Copyright (c) 2013-present, Facebook, Inc.
 *
 * This source code is licensed under the MIT license found in the
 * LICENSE file in the root directory of this source tree.

What I've Tried

Running NODE_ENV=production && tsc && vite build just to try to be sure. I see this, but even with the setting, it bundles development.

Might be related to https://github.com/vitejs/vite/pull/732

I added an env file with export NODE_ENV='production', then changed the build command to . env/.env.prod && tsc && vite build.
it worked as expected, with the production version of React. It's odd that setting the env during the command with NODE_ENV=production && tsc && vite build didn't work. I even added export before NODE_ENV to be sure I wasn't missing something.

Should no longer be an issue with the latest template which uses official React packages.


Add mod menu button to main menu

This adds a "Mods" button to the main menu. For this to compile it will need a few additional references added; I'm not sure about the best way to do this in a shared repo. Also, this PR includes my previous commit since I'm bad at git.

Compiled and ran, did not see mod button on main menu


Can't train using GPU? The torch version for this environment is '1.10.0cpu', that is, CPU one.

Search before asking

[X] I have searched the MuZero issues and found no similar feature requests.

Description
Hi,
I will appreciate if someone would suggest me that how to train using GPU, the cuda availability check turns out to be 'False' as the torch version for this environment is '1.10.0cpu', that is, the one using CPU (that's my understanding). Please correct me if I am wrong and direct me to the solution.
Thanks and Regards!
Additional context
No response

Which Cuda version do you have? Updating pytorch may work for you.

Hi Dillon,
Thanks for your feedback. I am using my base environment now and it is using GPU now. But now I have another error as follows:
"RuntimeError: module must have its parameters and buffers on device cuda:0 (device_ids[0]) but found one of them on device: cpu"
To overcome this I did the change "self.selfplay_on_gpu = torch.cuda.is_available()" as suggested in one of the same "Issues". By this, the self plays are now far less than the training steps. So no learning!


Sparkhello: Error in spark_compile(jar_name = jar_name, spark_home = spark_home, scalac = scalac_path,  :    failed to discover Spark jars

Hello, sorry to disturb you!
I tried sparkhello on windows but when I run
sparklyr::compile_package_jars()
It ask me to specify spark version and when I specify it(2.1.0), I got error like this:
==> using scalac 2.11.7
==> building against Spark 2.1.0
==> building 'sparkhello-1.5-2.10.jar' ...
Error in spark_compile(jar_name = jar_name, spark_home = spark_home, scalac = scalac_path,  :
failed to discover Spark jars
Can you help me please?

@Vonisoa this is probably caused by missing scalac versions, sparklyr needs 2.10 and 2.11. You could either, manually download them into Documents/scala but I would rather recommend to run download_scalac() which is available in the latest version of sparklyr, once this function is run, it will install the required scalac compilers.

OK Thank you, let's try this and I will tell if if it solve the issue!

Let us know if it doesn't! Thanks for giving this a shot. Cheers.


[Event Request ] "Payment History Line"

Please modify the Event Publisher OnAfterGetUnstrRemitInfo in table 11000002 "Payment History Line".
Change: the variable UnstrRemitInfo to a VAR (call by reference), see example below.
Current:
local procedure OnAfterGetUnstrRemitInfo(var PaymentHistoryLine: Record "Payment History Line"; var DetailLine: Record "Detail Line"; UnstrRemitInfo: Text[140])     begin     end;
New:
local procedure OnAfterGetUnstrRemitInfo(var PaymentHistoryLine: Record "Payment History Line"; var DetailLine: Record "Detail Line"; var UnstrRemitInfo: Text[140])     begin     end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41134.


Update dalek crates, k256, and wasmi for protocol 22

Just update deps for p22
fix #1353
fix #1313

Should we remove dalek from deny.toml? @dmkozh might have context on temporarily having there.

Should we ping dalek to a specific version of 4.1.3 instead of >=4.1.3 ?


Lab 10: Unable to Launch Instance

In terraform/Lab_10_Infrastructure_Security_Compute, terraform apply fails with:
│ Error: 400-CannotParseRequest, Incorrectly formatted request. Please refer to our documentation for help.
│ Suggestion: Please retry or contact support for help with service: Core Instance
│ Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/core_instance
│ API Reference: https://docs.oracle.com/iaas/api/#/en/iaas/20160918/Instance/LaunchInstance
│ Request Target: POST https://iaas.ap-sydney-1.oraclecloud.com/20160918/instances
│ Provider version: 6.18.0, released on 2024-11-13. This provider is 2 Update(s) behind to current.
│ Service: Core Instance
│ Operation Name: LaunchInstance
│ OPC request ID: 2fc21b25f265679ee9937083ae9f6b5e/017CED34F4A19686DFD0BBD2E3FDD43E/3BC51B70132E5CC5EEE5CD4708D2BEE8
│
│
│   with oci_core_instance.IAD-NP-LAB10-1-VM-01,
│   on ashburn_compute.tf line 41, in resource "oci_core_instance" "IAD-NP-LAB10-1-VM-01":
│   41: resource "oci_core_instance" "IAD-NP-LAB10-1-VM-01" {

Added the following line to compute instance resource:
provider = oci.ashburn


Add LTG005

Signify Hue ambiance spot (LTG005)

Device information

Home Assistant Device information

Hue ambiance spot (LTG005)
door Signify Netherlands B.V.
Checklist

[X] I have created a single PR per device. When you have multiple submissions please create separate PR's.
[X] For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
[X] For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Additional info

Thanks @Mariusthvdb

seems to not work:
2024-11-25 16:08:49.696 ERROR (MainThread) [custom_components.powercalc.power_profile.factory] Problem loading model: Model signify Hue ambiance spot (LTG005) not found
2024-11-25 16:08:49.698 ERROR (MainThread) [custom_components.powercalc.sensors.power] light.achterdeur_spot_1: Skipping sensor setup: Model not found in library (manufacturer: Signify Netherlands B.V., model: Hue ambiance spot (LTG005))
2024-11-25 16:08:49.698 ERROR (MainThread) [custom_components.powercalc.sensors.power] light.achterdeur_spot_1: Skipping sensor setup: Model not found in library (manufacturer: Signify Netherlands B.V., model: Hue ambiance spot (LTG005))
2024-11-25 16:08:49.698 ERROR (MainThread) [custom_components.powercalc.power_profile.factory] Problem loading model: Model signify Hue ambiance spot (LTG005) not found
2024-11-25 16:08:49.698 ERROR (MainThread) [custom_components.powercalc.sensors.power] light.achterdeur_spot_2: Skipping sensor setup: Model not found in library (manufacturer: Signify Netherlands B.V., model: Hue ambiance spot (LTG005))
2024-11-25 16:08:49.698 ERROR (MainThread) [custom_components.powercalc.sensors.power] light.achterdeur_spot_2: Skipping sensor setup: Model not found in library (manufacturer: Signify Netherlands B.V., model: Hue ambiance spot (LTG005))

afer installing Master.... must see what went wrong


Member events not firing

This is a problem with discord! (Source: i had the same issue)

You need to check both tick boxes! If it still doesnt work im going to look into it some more. Oh and i checked the code and it should work.

That seems to have fixed it! Thanks


Feature/encryption keys

For some reason Scrutinizer seems to be unable to resolve the the pip dependencies.
This happens even though I added pip install -r requirements.txt to the build's before_commands (and it even seems to have run succesfully) :cry:

Going to work more on this to address the raised issues.


Fix requestExistingInputState response

Description
#47177 attempted to fix this issue by more frequently calling setEditingState. This resulted in races/contention between the framework and platform side about who owned the editing state value, and caused failures on some Espresso based tests.  I'm adding some tests to make sure that behavior is more clearly exercised in https://github.com/flutter/flutter/pull/47464
Related Issues
#47177 Was a different attempt to fix this.
#43959 Was where a lot of this code was originally introduced.
Fixes #47137.
Tests
I added the following tests:
Removed a test that isn't valid - we shouldn't be using setEditingState that way.
Updated a test to use the new property on TextEditingClient.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[x] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

(I will work on a migration guide).

@hixie - this is an example of a breaking change that will be really hard to make soft.
We have an internal customer who implements the interface I want to change.
I could make it a soft break in a really ugly way by adding the new field to a subclass, checking if I have that subclass, and if not doing something to alert the developer that they're going to lose state when the app is backgrounded on Android.
That seems wrong to me. We'd be writing code that's really ugly and hard to understand, e.g.:
if (_currentConnection._client is TextEditingClientWithTextValue) {
  // send state up to platform
} else {
  //  report error with instructions on how to migrate?  Except we'll eventually want to remove the subclass?
}

While the migration is actually fairly simple, and the usage of this interface is very limited.

I'm going to make this a soft migration for the internal client by patching them to have the new value as an override with a temporary interface that we'll delete once this rolls.

xref cl/286430188 and b/146568373

This should wait for the internal CL to land first in case it gets grabbed by a roll soon


AGC functions in documentation aren't in code

Hi,
I was wondering why the documentation mentions agc squelch functionality, but I don't see the squelch functions in the source code, and my code won't compile when trying to call the functions agc_crcf_squelch_...
I'm using version 1.3.0.

I pulled the squelch functionality out of the AGC in order to improve it, but haven't added it back in yet. Is this a critical function for your development? It wouldn't take long to add back in if so. The API will probably change as a result, though.

It's not critical. I was just investigating it for possible use in an application I'm working on. I've actually developed a squelch function more specific to the application, so no hurry. I was mainly wondering if I was somehow out of sync and the functions were really there.

Yeah, I'm the one who's out of sync =)
On Fri, Oct 13, 2017, at 02:48 PM, DavidQrc wrote:

It's not critical. I was just investigating it for possible use in an
application I'm working on. I've actually developed a squelch function
more specific to the application, so no hurry. I was mainly wondering
if I was somehow out of sync and the functions were really there.> — You are receiving this because you commented. Reply to this email
directly, view it on GitHub[1], or mute the thread[2].>

Links:

https://github.com/jgaeddert/liquid-dsp/issues/106#issuecomment-336536949
https://github.com/notifications/unsubscribe-auth/AAPcY82ag_xZjzoDudheyAp3YWEBlaUaks5sr7BvgaJpZM4P0nvH

I see that you added the squelch functions back into AGC in this recent commit:
https://github.com/jgaeddert/liquid-dsp/commit/4f401231d098fb34a48eefb53ae2aad71ffb3118.
Thanks for the updated code!


Increase button menu icon size

Thank you for this library. I will like to increase the icon size of the button menus. It seems to have a fixed menu size. Can the button menu icon size be increased?

Hello.
No, it can't be resized, you could always extend the component or submit a PR.


Investigate and fix session expiration in Binder - returning: "Directory not found" after 5-10 min inactivity

Looks like Binder sessions expires after 10min of inactivity.
From Binder FAQ

How long will my Binder session last?
Binder is meant for interactive and ephemeral interactive coding, meaning that it is ideally suited for relatively short sessions. Binder will automatically shut down user sessions that have more than 10 minutes of inactivity (if you leave your window open, this will be counted as “activity”).
Binder aims to provide at least 12 hours of session time per user session. Beyond that, we cannot guarantee that the session will remain running.

We've switched to colab


Limit the number of processes that are reported

Just been looking at a couple of machines that each had 25,000 defunct nginx processes.
The Scope probe was using about 2 CPUs to report them, and the back-end about the same to render them (and I wasn't even looking at the processes page).
We could just stop reporting processes after 2,000 or 4,000 or whatever seems like a reasonable limit.
Also maybe not report defunct processes?

Also maybe not report defunct processes?

That seems entirely reasonable.

We could just stop reporting processes after 2,000 or 4,000 or whatever seems like a reasonable limit.

How do we decide which 2000 or 4000 processes have to be considered? :thinking:

I meant as a guard against overload, in an error situation - the limit should be higher than you expect to get actual processes.
It might be more obvious to drop the entire set of processes rather than arbitrarily truncating at some number.

Note how this commit may become redundant when we have fixed this on the client side, in which case the querier requests could be lowered back down

"this commit" 404s for me.  And it's in a private repo.


multiresolution with equirectangular images

Is it possible to have a multi-resolution with equirectangular images?
How would you do it?

Yes, it should be possible. The EquirectGeometry constructor accepts a list of level sizes as its argument. You also have to provide an ImageUrlSource that knows how to resolve each level into the respective URL. Something along these lines (disclaimer - I haven't tested this code):
var source = new Marzipano.ImageUrlSource(function(tile) {
  return { url: "http://www.example.com/path/to/equirect" + tile.z + ".jpg" };
});

var geometry = new Marzipano.EquirectGeometry([
  { width: 1024 },
  { width: 2048 },
  { width: 4096 }
]);

However, note that equirectangular geometries can't be tiled - each level must be a full equirectangular image. If you are planning to serve very high-resolution images this way, I strongly suggest you use a cube geometry instead.


Appbundle

@fremartini There has been a larger refactoring in #517 which you should take into account. It will probably give you a few bumps :)
Secondly. I would be mindful of not over-engineering with action files. I can see the value where several workflows have a need to call an action. In cases where an action is only called by one function, it might be just fine to have the steps in the workflow file :)
Thirdly, I am not sure but we should verify that Github Action supports parsing secrets as inputs such that we not leak secret values by accident when parsed as a input.

@fremartini Can you resolve the merge conflicts and then I'll provide my review :)


added aliases for build with batch step

(batch step replaces proposal with custom scenario setup)

:+1: change looks good.

+1


Watering level feature request

Didn't know where to add this, so i put it here.
Some addition like this:
    def getWaterLevel():
try:
	ospiWaterLevel = urllib2.urlopen("http://localhost:" + ospiPort + "/jo?pw=" + ospiApiPasswordHash).read()
except:
	syslog.syslog("Unable to load the OSPi API URL for Water Level. You might have a bad hashed password.")
	sendEmail()

     try:
	data = json.loads(ospiWaterLevel)
except:
	syslog.syslog("Unable to parse OSPi Water Level JSON Output.")
	sendEmail()

waterlevel = data["wl"]
#print "Getting water level" % (data["wl"])
return waterlevel`

But I could have that code all wrong,lol.

I like it! I'll add it in later today or tomorrow

Was my code close? New to Python..

I haven't implemented it yet, but it looks like a clone of the other check functions so it should be pretty close. Just need to add it into the main loop.
I'm also updating the config file for notifications on/off for all these little checks so there's some flexibility in getting notifications for what people care about. (For example, some people may not care about rain sensor)

Your code was mostly there. The print debug was wrong, but I fixed it. I also added in the check loop in the main loop in addition to some options for enable/disable this check and customizing the notification message. Check it out!

Great update!


Scrub clean in gosimple

Clean up two trivial issues found by gosimple.
@mbhinder @variadico @lilirui

LGTM

LGTM


Villages spawned with broken chests.

All blacksmiths in villages have no chests. The chests somehow break when they are generated. The items will lay on the ground at the place where the chest should be, but there is no chest.
Picture:
![Image of the bug]
(https://gyazo.com/112b6948031cadba505188e5b89c6458.png)
Version;
[14:45:42 INFO]: TerrainControl version 2.7.2
[14:45:42 INFO]: Advanced custom world generator.
[14:45:42 INFO]: Authors: Khorn and Olof Cayorion Larsson
[14:43:15 INFO]: This server is running CraftBukkit version git-Spigot-5f38d38-12698ea (MC: 1.8.8) (Implementing API version 1.8.8-R0.1-SNAPSHOT)
[14:43:15 INFO]: Checking version, please wait...
[14:43:16 INFO]: You are running the latest version

Blacksmiths are not spawning with broken chests, but I'm still on 2.7.2-snapshot...

Note: I set villagetype in swamps and forests to true

Experiencing the same issue here. Chests in structures are broken, in villages as well as Desert Temples.

Update: The chests work when TerrainMode is set to Default, so that's what I ended up doing to generate a world without ocean biomes.

ItsHarry's fix wont work for me as I want all features

^ Me neither, turns out ocean temples are spawned in random locations even though there are no ocean biomes when using the "solution" I mentioned above.
If the chests worked, then this plugin would be perfect for me

Still having this issue, I know this is a dead thread but still a issue


[Dashboards] Add extraVolumes and extraVolumeMounts

Description
User can define extra volumes for opensearch-dashboards
Test Results:

Deploy using the following chart values :

extraVolumes:
  - name: my-ca-trust
    hostPath:
      path: "/etc/ssl/certs"

extraVolumeMounts:
  - name: my-ca-trust
    mountPath: /usr/share/opensearch-dashboards/config/certs
    readOnly: true

Check the volumes on the opensearch-dashboards pod

Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

All the linting checks are failed and I think you also need to update the readme.
Thanks.

Hi @rexbut can you update readme files if you have any new config?
Thanks.
https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch/README.md#configuration

Hi @rexbut can you update readme files if you have any new config? Thanks. https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch/README.md#configuration

Hello @peterzhuamazon, I made some changes on the "opensearch-dashboards" chart and there is no "Configuration" section in the README.md
https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch-dashboards/README.md

Hi @rexbut can you update readme files if you have any new config? Thanks. https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch/README.md#configuration

Hi @peterzhuamazon, I made some changes on the "opensearch-dashboards" chart and there is no "Configuration" section in the README.md
https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch-dashboards/README.md

Unfortunately yes there is not. I created an issue #129

So then @TheAlgo can we unblock this PR since there is no dashboards configuration documentation for now.
Thanks.


Cpu vs gpu

** I DON'T THINK WE SHOULD MERGE THIS **
This unfortunately does not really change GPU vs CPU exp and log
Errors accumulate and are excerbated by pow operations

Interesting -- even running the pow operations with float64 we get this difference hey?


Consume the API Proxy in SAP Web IDE

Tutorial URL: https://developers.sap.com/tutorials/hcp-apim-webide-int.html
Hi,
I've created the destinations in Steps 6 and 7.
I'm having issues viewing the catalog service in the WebIDE.
I'm getting the following error in the WebIDE:
"unable to reach the catalog service. please check the service configuration and connect again"
Please help.
Guy

Dear Guy,
As of today SAP APIM integration with WebIDE on CF is a limitation. We are working on this to solve it.
Regards
Shilpa

Thanks, I'll be happy to QA it once resolved.
Guy
Best Regards,
Guy Perets
Principal Banking Solution Architect
Global Consulting Delivery
SAP Israel,  14 Eli Hurvitz St.,  43665 Raanana,  Israel
T +972-9-777-5226  |  M +972-54-745-4880  |  F +972-74-760-1226 | E guy.perets<EMAIL_ADDRESS>From: Shilpa Vij<EMAIL_ADDRESS>Sent: Wednesday, December 9, 2020 6:41 AM
To: SAPDocuments/Tutorials<EMAIL_ADDRESS>Cc: Perets, Guy<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [SAPDocuments/Tutorials] Consume the API Proxy in SAP Web IDE (#6306)
Dear Guy,
As of today SAP APIM integration with WebIDE on CF is a limitation. We are working on this to solve it.
Regards
Shilpa
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Tutorials/issues/6306#issuecomment-741522112, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AR7SY7URHENTKHMIN6F7YULST35UNANCNFSM4URYT3GQ.

@shilpavij If this is a limitation for this tutorial, you should indicate this in the prerequisites, and close this issue. Thanks.

@shilpavij Can you update us on this tutorial, and my question? If the tutorial won't work, we don't want it 😀

@shilpavij Can you update us on this tutorial, and my question? If the tutorial won't work, we don't want it 😀

The tutorial is no longer available.


Options for ways to use TextOverflow

Description
When there is a TextOverflow option set, truncation happens in places where there is a space (or linebreak I guess). This is fine, but if we have a very long word at the end of the string, then that long word will be cut altogether which may cause the text to look weird (considering there is no resizing happening).
For example, if I have this string "Hello Jack, hello Pauline, and finally Hello Fredrikhteshkesteandermastionisk", then if the truncation happens, the last long word will be all replaced with ellipses. This makes the text look weird in some places. In my specific use case, I have a container around the text which is sized with extra space in this case!
Proposed Solution
The solution would have two parts:
First of all, it would be great to put a wrapping setting, to determine what kind of truncation would happen if the overflow symbol is used. One option in the beforementioned example would replace the whole "Fredrikhteshkesteandermastionisk" with ellipses and the other option would use a part of the word in the final string with ellipses at the end, like "Fredrikhteshkestean...".
The other part of the solution which I think could be also reported as a separate bug is regarding the size of the final Text. In the example above, considering the last word is replaced, the size of the Container around the text is not set based on the actual rendered text and seems to consider having a part of that last long word in there. So although the final result is shown as "Hello Jack, hello Pauline, and finally Hello ...", the Container around the text which should wrap around the text is sized bigger like if the text is "Hello Jack, hello Pauline, and finally Hello Fredrikhteshkeste...".
Alternatives
I have not considered anything and for now, I'm living with it hoping that the text our users enter won't have a very long word where truncation happens.
Version

Flutter version: 1.12.13+hotfix.5
auto_size_text version: 2.1.0

The other part of the solution which I think could be also reported as a separate bug is regarding the size of the final Text. In the example above, considering the last word is replaced, the size of the Container around the text is not set based on the actual rendered text and seems to consider having a part of that last long word in there. So although the final result is shown as "Hello Jack, hello Pauline, and finally Hello ...", the Container around the text which should wrap around the text is sized bigger like if the text is "Hello Jack, hello Pauline, and finally Hello Fredrikhteshkeste...".

Note that this is not an enhancement, and I think it is a bug as it makes the auto-size not work correctly.


request for config options to accommodate twitchy mouse

Nexus request

now it doesn't work, am i using i wrong?

I think I know what it is, my very sensetive mouse,
please add an option for mouse not unpausing
-and/or-
an option for ONLY timeout unpause (like skyrim should have been since the release)

fixed in v<IP_ADDRESS>


ENH Use the core ArrayList Searchfilter functionality

No need for the extra dependency after 5.1.0
There's some minor changes needed for docs as well, which should have been done with 2.0.0

Tests pass locally. Gotta figure out what's going on here, but there's no rush since 5.1.0 is still in beta.
Cancelled push CI 'cause we only need to see it in one set.


Enhanced presentation of readme.md

@kevwan Kindly review my pr and merge it.

Thanks! But not going to change like this.


[Question]: LongLLMLingua vs. LLMLingua2 for chatbot history compression

Describe the issue
Hey guys, thanks a lot for your great work on prompt compression, really amazing results!
I have a question regarding a chatbot history compression use-case, may I ask for some intuition of yours on which method might work better for it:

LongLLMLingua using the user's last query as the question, with re-ranking/ordering turned off, and treating each chat message as a separate document
Just using LLMLingua2 to compress the chat history, with the possibility to fine-tune the embedding models on a chat-based dataset

Thank you!

Hi @DomStan, thanks for your interest and support in LLMLingua.
This depends on the overhead you can tolerate.
Generally speaking, for chat scenarios, if the topic is relatively fixed and there's a high requirement for low latency, you might opt for Solution 2. If the topic varies significantly and you can accept online compression, then Solution 1 might be suitable, potentially offering higher performance. However, this will depend on your specific scenario.
Best wishes,


Expose plop.setGenerator()

Make plop file optional, and expose plop.setGenerator() function to allow dynamic creation of generator configs.

In fact, it probably makes sense to expose the other internal plop fields (inquirer, handlebars, addPrompt, etc.)? I can modify the pull request to add these if you like.

yeah, I like the overall idea of being able to use plop without a plopfile.
good idea 👍

It's very handy. I've already manually tweaked the node-plop index to expose the other functions, and have already incorporated some generation logic into my gulp build. And it's all code (and dynamic), so no plopfiles. :-)

@brian428 when you use this approach, do you have to specify absolute paths for the templateFiles and paths, or do you set the plopfilePath later in order to set your context path?

I've got it working just using paths relative from the project root, which is basically the "current directory" while the gulp tasks run. Does that answer your question?

yeah, makes sense. The plopfilePath concept was added so if your ran the plop cli from a subfolder, files would still be created in the correct place. But I suppose when you're using automation in a script, you can always set the cwd in the script if needed.


Unable to download  "Create-Spark-Table-NYCTaxi- Data.ipynb " for tutorial

[Enter feedback here]
Unable to download  notebook "Create-Spark-Table-NYCTaxi- Data.ipynb " for tutorial with error below. Looks no public access permitted.

PublicAccessNotPermitted
Public access is not permitted on this storage account. RequestId:2284df8f-f01e-0093-2c13-f8a728000000 Time:2022-11-14T10:26:36.6567356Z

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

ID: c0f4f337-c2cf-9c67-2de4-051cd8b4f3d9
Version Independent ID: 1831d292-2a2d-f121-9bdf-9604fd7637b6
Content: Tutorial: Train a model by using automated machine learning - Azure Synapse Analytics
Content Source: articles/synapse-analytics/machine-learning/tutorial-automl.md
Service: synapse-analytics
Sub-service: machine-learning
GitHub Login: @NelGson
Microsoft Alias: negust

@koichino
Thanks for your feedback! We will investigate and update as appropriate.

@koichino
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Build Pip Package Failed on Windows v1.8.0 with GPU

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No!!
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): Source
TensorFlow version (use command below): v1.8.0
Python version:  3.6
Bazel version (if compiling from source): CMake
GCC/Compiler version (if compiling from source): VS 2015
CUDA/cuDNN version: 9.1
GPU model and memory: GTX-1070 8GB
Exact command to reproduce:
git pull
git checkout refs/tags/v1.8.0
cd tensorflow\contrib\cmake
mkdir build
cd build
cmake ..  Sorry, this is Japanese System message.
But, problem is seems missing the "tensorflow/python/framework/cpp_shape_inference.pb.h"
"C:\repo\tensorflow\tensorflow\contrib\cmake\build\tf_python_build_pip_package.
vcxproj" (既定のターゲット) (1) ->
"C:\repo\tensorflow\tensorflow\contrib\cmake\build\pywrap_tensorflow_internal.v
cxproj" (既定のターゲット) (2) ->
"C:\repo\tensorflow\tensorflow\contrib\cmake\build\pywrap_tensorflow_internal_s
tatic.vcxproj" (既定のターゲット) (3) ->
"C:\repo\tensorflow\tensorflow\contrib\cmake\build\tf_c_python_api.vcxproj" (既 定
のターゲット) (134) ->
(ClCompile ターゲット) ->
C:\repo\tensorflow\tensorflow\c\python_api.cc(19): fatal error C1083: include
ファイルを開けません。'tensorflow/python/framework/cpp_shape_inference.pb.h':No such file
or directory [C:\repo\tensorflow\tensorflow\contrib\cmake\build\tf_c_python_ap
i.vcxproj]
90 個の警告 (Note: 90's Warning)
1 エラー (Note: Error)

Best regards.

I have similar issue as well, except my errors are in english
"C:\Users\user\Documents\tensorflow\tensorflow\contrib\cmake\buildr1.8\tf_python_build_pip_package.vcxproj" (default ta
rget) (1) ->
"C:\Users\user\Documents\tensorflow\tensorflow\contrib\cmake\buildr1.8\pywrap_tensorflow_internal.vcxproj" (default tar
get) (2) ->
"C:\Users\user\Documents\tensorflow\tensorflow\contrib\cmake\buildr1.8\pywrap_tensorflow_internal_static.vcxproj" (defa
ult target) (3) ->
"C:\Users\user\Documents\tensorflow\tensorflow\contrib\cmake\buildr1.8\tf_c_python_api.vcxproj" (default target) (134)
->
(ClCompile target) ->
  C:\Users\user\Documents\tensorflow\tensorflow\c\python_api.cc(19): fatal error C1083: Cannot open include file: 'tens
orflow/python/framework/cpp_shape_inference.pb.h': No such file or directory [C:\Users\user\Documents\tensorflow\tensor
flow\contrib\cmake\buildr1.8\tf_c_python_api.vcxproj]

    20441 Warning(s)
    1 Error(s)

@gunan could you take a look.

It is possible we have missed a bug with /arch:AVX2.
Otherwise, I think there is an earlier protobuf error we are missing, that results in this file not being generated on your system. Since r1.8 builds just fine in our CI, I am not sure what may be going wrong for you.

I try to build v1.8 branch, but same error is occured.
In v 1.7.0 case, these command is works fine. I can get the Pip Package.
For this reason, I think this compile error was v1.8 related error.

What is generating this file? I can see if that part is working or not.

In v.1.7.0 case. generated file is "tensorflow_gpu-1.7.0-cp36-cp36m-win_amd64.whl"
And, I try some function, it seems work fine.
Sorry, can not attach the file, it is too big for attach.

Is this also the reason why there are no new windows builds for tf-nightly right now?

No, on windows our CI has another issue we are working to solve right now.
On Sun, May 13, 2018 at 12:49 AM Julian Niedermeier <
<EMAIL_ADDRESS>wrote:

Is this also the reason why there are no new windows builds for tf-nightly
right now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/19103#issuecomment-388608200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHlCOXkkXPDmhvewa5QKxXmEOHnBIGemks5tx-WNgaJpZM4TzfMK
.

Same error here. Windows 10. VS 17. Master branch.
cmake .. -G "Visual Studio 15 2017 Win64" ^
 -T host=x64 ^
 -DCMAKE_BUILD_TYPE=RelWithDebInfo ^
 -DSWIG_EXECUTABLE=D:\tf-build\swigwin-3.0.12\swig.exe ^
 -DPYTHON_EXECUTABLE=D:\Program-Files\anaconda\envs\tensorflow\python.exe ^
 -DPYTHON_LIBRARIES=D:\Program-Files\anaconda\envs\tensorflow\libs\python35.lib ^
 -Dtensorflow_BUILD_PYTHON_TESTS=ON

MSBuild /p:Configuration=RelWithDebInfo tf_python_build_pip_package.vcxproj

msg:
d:\tf-build\tensorflow\tensorflow\c\python_api.cc(19): fatal error C1083: Cannot open include file: 'tensorflow/pytho
n/framework/cpp_shape_inference.pb.h': No such file or directory [D:\tf-build\tensorflow\tensorflow\contrib\cmake\tf18b
ld\tf_c_python_api.vcxproj]

Could you share the full log using pastebin?
The error message means a proto file failed to build somewhere earlier in the build.

Looks like it is too big for pastebin. So I attach the log here. Thanks! @gunan
tf.mb.cmake.cpu.vs17.log

Thank you @gunan . Using 'release' did remove all C2471. However C1083 is still there. Please find the attached log.
msbuild.log
Anyway, since I just need the header files to build an user_opr under windows. I use branch r1.7 instead.

Thanks to @Skydes's comment in #18931, I applied the patched line to tensorflow/contrib/cmake/tf_core_framework.cmake and successfully built r1.8.0 wheel with AVX2 support enabled.

hi @yoshimur, were u able to use tf after installing 1.7 version??
I am getting the below error:
Could not find 'cudnn64_7.1.4.dll'. TensorFlow requires that this DLL be installed in a directory that is named in your %PATH% environment variable. Note that installing cuDNN is a separate step from installing CUDA, and this DLL is often found in a different directory from the CUDA DLLs. You may install the necessary DLL by downloading cuDNN 7.1.4 from this URL: https://developer.nvidia.com/cudnn
Could you please guide.

Hi AshimaChawla
I download cudnn-9.1-windows10-x64-v7.1.zip, and extract this zip file.
You can find cuda/bin/cudnn64_7.dll, So copy this file to CUDA bin folder.
In my case, copy it to C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v9.1\bin
This works fine.
Please try it.
Best Regards.
From: AshimaChawla<EMAIL_ADDRESS>Sent: Thursday, June 7, 2018 3:48 AM
To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: Yoshihiro Yoshimura<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] Build Pip Package Failed on Windows v1.8.0 with GPU (#19103)
hi @yoshimur https://github.com/yoshimur , were u able to use tf after installing 1.7 version??
I am getting the below error:
Could not find 'cudnn64_7.1.4.dll'. TensorFlow requires that this DLL be installed in a directory that is named in your %PATH% environment variable. Note that installing cuDNN is a separate step from installing CUDA, and this DLL is often found in a different directory from the CUDA DLLs. You may install the necessary DLL by downloading cuDNN 7.1.4 from this URL: https://developer.nvidia.com/cudnn
Could you please guide.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tensorflow/tensorflow/issues/19103#issuecomment-395172782 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABBUB87Z0c36FNr2ypStCapPLnSp7lwcks5t6CPsgaJpZM4TzfMK .  https://github.com/notifications/beacon/ABBUB2MBOEs6nEAIfCozR-hRVd_WiHTsks5t6CPsgaJpZM4TzfMK.gif

Hi @yoshimur, Thanks very much..
Everything is working fine now..
The only issue I am seeing is when I execute my code, it seem to be bit slower than I used to execute on GPU. Is it something because of Windows ??
Is Ubuntu much faster than that?
/Ashima

Nagging Assignees @gunan, @shivaniag: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I had the same problem (Win10, VS2015, CUDA 9.1, AVX2, with MKL, without gRPC).
I checked tensorflow/python/framework and saw that the cpp_shape_inference.pb.h file was in there.  So I just ran the MSBuild command again and the build completed (WHL file produced)!
Lmk if I should include more info.  This is my first post to a Github issue, plz be nice. :)

Thanks for the tip, this is really helpful.
Could you copy/paste your build output to pastebin for us to be able to see the error message?
@annarev According to the comment by @luranhe could this be a missing dependency in cmake for api generation?

Unfortunately, I don't have the build output from the failed build anymore, I didn't save it anywhere and the command line only saves so much history.  Many apologies.
It definitely failed on tf_c_python_api.vcxproj because of "no such file or directory" of file cpp_shape_inference.pb.h, which clearly existed at the indicated location.


Uncaught TypeError: Cannot read property 'Constructor' of undefined

I get the following error in the console every time I run my app:
Uncaught TypeError: Cannot read property 'Constructor' of undefined
What is causing that and how do I resolve it?

I was having a similar issue when running this through IE.  I normally use Edge and switched to test things in a different browser.  I am using ASP.NET MVC, and in my BundleConfig.cs, I had the following:
     bundles.Add(new ScriptBundle("~/bundles/bootgrid").Include(
               "~/Scripts/jquery.bootgrid.fa.min.js",
               "~/Scripts/jquery.bootgrid.min.js"));

In the jquery.bootgrid.fa.min.js, there's not a lot of code, but looking at it made me think switch the order of include would work.  So I changed the BundleConfig.cs to look like:
     bundles.Add(new ScriptBundle("~/bundles/bootgrid").Include(
               "~/Scripts/jquery.bootgrid.min.js",
               "~/Scripts/jquery.bootgrid.fa.min.js"));

That worked for me here.  No change in functionality with regard to Edge and no more exception in IE.
If you aren't using the bundle config method, check the order of the script links wherever you are doing them.
Hope this works and helps you out.


Typecheck: Destroy trees passed to it in parallel.

Typecheck: Destroy trees passed to it in parallel.
There is no reason for typecheck to keep the trees around.
Avoids destructing all ASTs on a single thread.
Motivation

Potentially speed. It also simplifies an optimization @ngroman is doing to remove package stuff from ASTs prior to CFG running.
Test plan

N/A

I realize now that this defeats our "intentionally leak memory" optimization in CLI mode, so it noticeably regresses typechecking performance!
I think I need to add an option to leak memory inside typecheck.

@froydnj I had to reintroduce leaking memory in CLI mode, so I requested a re-review.


Is the BigQuery dataset public available?

In the paper it was mentioned
The multi-lingual dataset BIGQUERY is a subset of Google’s publicly available BigQuery dataset,
which consists of code (under open-source license) in multiple programming languages.

Just wondering is this subset publicly available?

@enijkamp can you help to public this folder "gs://sfr-tpu-us-east1-research/enijkamp/jaxformer/datasets/bigquery/train/" ?

Maybe just a few sample record files are good.

Hello @enijkamp is it possible to get the access the files under "gs://sfr-tpu-us-east1-research/enijkamp/jaxformer/datasets/bigquery/train/"? I following the code and tried, but the access is forbidden.

+1


Add unit testing for slack package

The slack package currently has no tests, relying on manual testing to verify interaction with Slack.
To reduce manual effort, unit tests can be added that validate the calls to the underlying Slack clients.
References to the slack clients can be replaced with interfaces that can be mocked.
Basic qualities to check:

Block API validation
Correct calls made only once
Handling of events from different starting states
Errors are not swallowed

The above PR doesn't fully address the qualities we want to test, but it refactors as needed to make unit testing possible, and adds some basic tests for receiver functions.

Closing this issue and will open follow ups for specific tests.


Arena - Canvas Display

Problem
Currently, the canvas is hardcoded to have a width and height of 600 pixels. We need the canvas to handle different screen sizes, different arena aspect ratios, and be able to support very large maps.

For this ticket, we're only supporting square arenas.

We're still seeing issues in Safari (#290), but this looks good in Chrome.


Long URLs break when attempting to read/write them as filesystem paths

Describe the bug
In multiple places in the docs (Quickstart for example) it mentions the use of ./archive some-file.txt as a means of ingesting a file with a list of urls in it. There are also examples of using ./bin/archivebox-export-browser-history --firefox which generates a JSON file that users should be able to feed into ./archive as well.
With the latest release it seems that archivebox add would have taken over this behavior but it either isn't supposed to and this functionality was removed or there's a bug somewhere. The existence of the parsers in the archivebox add code path leads me to believe this is a bug and archivebox add should handle these cases.
When running archivebox init there is also a message at the end that states:

To add new links, you can run:
archivebox add ~/some/path/or/url/to/list_of_links.txt

Steps to reproduce

Create a file test_urls.txt which only has the contents https://example.org
Run archivebox add test_urls.txt
Get back an error instead of archiving https://example.org

The same issue happens when passing the output JSON file from running ./bin/export-browser-history.sh --firefox or
Screenshots or log output
# archivebox add test_urls.txt
[i] [2020-08-09 19:50:30] ArchiveBox v0.4.11: archivebox add test_urls.txt
    > /Users/mpeteuil/projects/ArchiveBox/data

[+] [2020-08-09 23:50:39] Adding 1 links to index (crawl depth=0)...
    > Saved verbatim input to sources/1597017039-import.txt
                                                                           0.0% (0/240sec)[X] Error while loading link! [1597017039.360603] test_urls.txt "None"
    > Parsed 0 URLs from input (Failed to parse)
    > Found 0 new URLs not already in index

[*] [2020-08-09 23:50:40] Writing 0 links to main index...
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.sqlite3
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.json
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.html

It also happens when trying the input redirection route, except there is no [X] Error while loading link!:
archivebox add < firefox_history_urls.txt
[i] [2020-08-09 23:00:29] ArchiveBox v0.4.11: archivebox add < /dev/stdin
    > /Users/mpeteuil/projects/ArchiveBox/data

[+] [2020-08-10 03:00:30] Adding 43291 links to index (crawl depth=0)...
    > Saved verbatim input to sources/1597028430-import.txt
    > Parsed 0 URLs from input (Failed to parse)
    > Found 0 new URLs not already in index

[*] [2020-08-10 03:00:34] Writing 0 links to main index...
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.sqlite3
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.json
    √ /Users/mpeteuil/projects/ArchiveBox/data/index.html

Software versions

OS:                        macOS 10.15.6
ArchiveBox version:        87ba82a
Python version:            3.7.8

I finally made my way to the usage docs in the wiki, which have examples of doing this successfully. For example:

archivebox add < urls_to_archive.txt

I'm going to leave this open for the moment because I think a to-do to come of this is that there seems to be some places in the docs that still have references to old methods which are no longer valid.

The docs are indeed out-of-date, thanks for fixing them,
archivebox add --depth=1 ~/Downloads/bookmarks_export.html

is slightly different from:
archivebox add < ~/Downloads/bookmarks_export.html
archivebox add --depth=0 < ~/Downloads/bookmarks_export.html  # these are equivalent

The depth=1 treats the file as a page itself that needs to be archived, then parses it for links and archives those as well.
The depth=0 version just parses stdin as an input file for links, and doesn't archive the bookmarks_export.html file itself as if it's a page (which is probably what you want).

Thanks for the explanation. I didn't realize the --depth option changed how the input file itself was treated depending on the value (0, 1).
I updated the description, but it looks like I still get Parsed 0 URLs from input (Failed to parse) even using input redirection.

--depth alone doesn't change how the input file is treated, note the < pipe is only present in the depth=0 example. This means that it imports all the URLs passed in, but only archives each one at depth=0. In the --depth=1 example the file is not piped in as a list of links, but rather archived as if it were a URL to a page itself, that behavior changes if you were to add a < and pipe the file in instead.
Can you post a snippet of the file you're trying to import? (note the URLs must have a scheme, https://example.com √, example.com X)

Can you post a snippet of the file you're trying to import? (note the URLs must have a scheme, https://example.com √, example.com X)

I would, but I'd like to avoid doing so out of privacy concerns if possible. However, I was able to find the URL which is causing the issue. The problem seems to be that it's just really long. In this instance the URL is 1092 characters, which is causing OSError: [Errno 63] File name too long to be thrown when executing if Path(line).exists() in the generic_txt parser. The error is eventually swallowed by _parse's exception handling , so it's not seen elsewhere.
The good news is that this is reproducible with any sufficiently long and valid URL in a txt file.

Your diagnosis looks right.
Unfortunately, I think this is a much deeper problem than just the generic_txt parser and I can't promise that this is going to get fixed anytime soon.
The "URL == filesystem path" decision was made early on in the design process, and it's a painful choice that has come back to bite me many times, but it's here to stay for the foreseeable future.
URLs don't cleanly map to filesystem paths, and never will (some filesystems are case insensitive!!). On the other hand, there are many benefits to being able to see website paths easily in a file explorer, it's the most durable format of all. There are also many redundant methods to cover you in the rare case of URL => filename conflicts created by wget.
When it does break due to a filesystem:URL mapping failure, I believe it should only be the wget archive method that's broken, the other methods all have hardcoded output paths with no URL fragments in them. If you absolutely need it you can replay a broken wget archive you can always use the wget warc to get the same output independently.

That background definitely helps me understand this better and helps me see that it's not just this one isolated problem.

What we can do for now is catch the exception and skip all attempts to read URL fragment paths, and save long URLs to the index normally up until some ridiculous limit like 65,000 characters. This will still result in broken wget clones, but all the other outputs and the index should work with the long URLs.

That sounds reasonable. One long URL spoiling the whole batch that's being parsed is the main issue at hand here, so I think as long as that's resolved then it's case closed on this one.
Thanks for working through this with me, I appreciate all the help. It's not easy maintaining OSS, but my interactions with this project have been nothing but pleasant 😄

This should be fixed in 2e2b4f8. (going out with the next release)
git remote update
git checkout dev
pip install -e .

cd your/data/dir
archivebox add < ~/path/to/your/links.txt

If you give that a try and still have the issue then comment back here and I can reopen the ticket.


Define Usage Analytics

Description
Evaluate the different channels of usage analytics we would like to monitor including:

[x] #35

Opt in Taqueria usage analytics

VS Code Extension

CLI

[x] #34

Analytics for Taqueria website

Analytics for social media channels

User feedback on documentation and product

Acceptance Criteria

[x] Channels defined and documented
[x] Metrics defined and documented in line with the proposal
[x] Methods of acquiring data defined
[ ] Legal review by @neilmangan

Additional Context and Related Items

Legal review for all activities

Closing: complete


module 'flash_attn' has no attribute 'flash_attn_varlen_qkvpacked_func'

Hi TriDao, thanks for the great work.
I'm trying to train the project to apply flash-attention2. But my Graphics Card is RTX 2080 which is not yet supported flash-attention2, so I downloaded and applied flash-attention1 to train it.
•This is the error I received:
File "/mnt/will1tb/work_space/Pointcept/exp/nusecenes/semseg-v3m1-0-base/code /pointcept /models /point_transformer_v3 / point_v3m1_base.py", line 208, in forward
feature = flash_attn.flash_attn_varlen_qkvpacked_func(
AttributeError: module 'flash_attn' has no attribute 'flash_attn_varlen_qkvpacked_func'
•In https://github.com/Dao-AILab/flash-attention/issues/745#issuecomment-1873780475
I followed and import from 'flash_attn.flash_attn_interface' but it returned: "SyntaxError: invalid syntax."
•So can you help me to reply some questions, please:
1, How do I change from flash-attention 2 to flash-attention 1 while the project is using flash-attention2?
2, Or could you suggest something to me in this case?
My Setup:
• Graphics Card: 1 x RTX 2080 (11GB V-Ram)
• CUDA: 11.8
• Pytorch: 2.2.0
Thank you so much.

pip install flash-attn==1.0.9

Thank you for your prompt reply. @tridao
In my case, I am using Attention1 (because the Graphics Card is RTX 2080 only with Attention1 supported). But the article I want to test run is using Flash-Attention2.
So I want to ask the author: Is there any way to change the code from Flash-Attention 2 back to Flash-Attention1? Or something like that.
Suppose: Rename from flash_attn_varlen_func -> flash_attn_unpadded_func,...

sure you can try that

I appreciate your help, bro!!!


Configuring at IRC could be easier if "help" gave help instead of link to readme.

Web browser isn't always available.

i am +1 for this. i have been looking for the name of the option that will stop zncpush from replying "Ok" when you send messages through it. pretty sure i've seen it before but can't find it now for the life of me. anyone know what its called if i didn't dream it up?


Adding 'edge' dependency used in index.js

This was breaking when building webtasks unless the developer had
already included edge in their package's dependencies. I'm unsure if
this is intentional or accidental so I am submitting this patch to
include it.
Error when building an extension via CLI a0-ext
ERROR in ./~/webtask-tools/lib/index.js
Module not found: Error: Can't resolve 'edge' in '/......./extension-package/node_modules/webtask-tools/lib'
 @ ./~/webtask-tools/lib/index.js 54:13-28
 @ ./webtask.js

This issue was previously raised at: #18 and is a fix.
@ggoodman could you provide some feedback on if this was intentional and if this fix will suffice?

An alternate solution would be to specify in the readme edge is required by the project or required globally

Hey @machuga I think we would be more comfortable with the latter approach or splitting the webtask-tools/cs compiler out into a separate repository entirely. As we add dependencies to this module, we risk making it a catch-all swiss army knife, adding overhead to everyone only interested in small parts of it.

@ggoodman 👍  sounds good.  I looked through the repo a bit more and see that we do this for other packages so I think I have a grip on the reasoning now. I'll submit a PR for the docs to point out some things that should be included by upstream projects.  Thanks!


slides: init at 0.2.0

Motivation for this change
Packages https://github.com/maaslalani/slides: A terminal based presentation tool.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126299 run on x86_64-darwin 1

1 package built:

slides


[ENH] Documentation suggestion: make it clearer what the default values are for compiler directives

Is your feature request related to a problem? Please describe.
The docs for compiler directives is inconsistent in showing the defaults https://cython.readthedocs.io/en/latest/src/userguide/source_files_and_compilation.html#compiler-directives. Some options list the default at the end, some list it somewhere in the middle of the text, and some don't list it at all.
See https://github.com/cython/cython/blob/master/docs/src/userguide/source_files_and_compilation.rst#compiler-directives
I would suggest adopting a style where "default is X" is always a single line at the bottom of the description. Additionally, I would suggest bolding the default option in the list after the name of the directive, like

annotation_typing (True / False)
Uses function argument annotations to determine the type of variables...

I'm happy to make a PR for this but someone will need to point me to where the defaults are defined for the options that don't actually specify that.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context
No response

Defaults are listed here: https://github.com/cython/cython/blob/1d7100d867a8b64b6f661009ec2ed45a4d8d5dba/Cython/Compiler/Options.py#L179
Making things clearer would be good I think


Unit-tests: missing documentation or support for simulating input

I've written a super simple input component that prohibits the user from typing anything that matches some regexp.
I would really like to write a unit test to verify it worked, but can't find any examples, sufficient documentation or figure out how to get the inner input element to fire a change event similar to how it would be fired in the wild.
I'm really hoping for something like enzymejs for react where I can do like this; input.simulate('change', { target: { value: 'Hello' } })
<custom-input sanitize='/[^\d+ ]/g'></custom-input>

class CustomInput {
    ...

    format(value) {
        return value.replace(this.sanitize, '');
    }

    onInput({target}) {
        target.value = this.format(target.value);
    }
}

Of course I hope you'd update the documentation so it might be easier for the next developer with  the same problem, as well as me. If there is something I misunderstand, I'll really appriciate an explanation before you close this issue.
Thanks in advance. ^_^

So this works. ^_^
export function simulateInput(target, value) {
  target.value = value;
  target.dispatchEvent(new InputEvent("input"));
}

This would be a good library https://github.com/PolymerElements/iron-test-helpers/blob/master/mock-interactions.js. Some things are outdated, or specific to polymer 0.5 or 1/2/3. We could create a modernized version of this in @open-wc/testing-helpers.
I would prefer to simulate a keypress instead of setting the value property manually, if possible.

Thanks. I'll look into that. ^_^

imho such an abstraction is only marginally useful nowadays... when we still had to "deal" with keyCodes yeah it made sense... but now I don't see how the syntax could be much improved
this is vanilla
el.dispatchEvent(new KeyboardEvent('keydown', { key: 'PageUp' }));

how would you write it "shorter"?
simulateKey(el, 'PageUp', { type: 'keydown' });

if so then I would rather us vanilla as no one would need to be taught our own simulateKey.
Here is an example where it is used often and I think the tests are quite readable and would not really benefit from a simulateKey https://github.com/ing-bank/lion/blob/master/packages/calendar/test/lion-calendar.test.js#L714

Good point! I wasnt aware that actually works, that fills in the input field too?

Haha! I agree completely with out @daKmoR.
Turns out that it really can be as simple.
It didn't work for me, because I wanted to simulate the change of the input element as nested in another web component and then test the behavior of the parent component that actually change the dom. That is, it seemed to easily break and difficult to make work when I slightly changed implementations but kept behavior.
What I needed to learn was that
(a) events that propagate upwards through shadow boundaries replace the target of the event with the last passed shadow component.
"Bubbling events fired from within shadow DOM are retargeted so that, to any listener external to your component, they appear to come from your component itself."
https://lit-element.polymer-project.org/guide/events
(b) I have to set bubbles and composed to true for the event to pass shadow boundaries
"To make a custom event pass through shadow DOM boundaries, you must set both the composed and bubbles flags to true"
So if I want to simulate the behavior of an input event in the 'wild' I had to set those values too.
Now my tests actually behave as expected regardless of the component nesting. =D
export function simulateInput(target, value) {
  target.value = value;
  target.dispatchEvent(new InputEvent("input", { bubbles: true, composed: true}));
}

Thanks for input! =)