refactor suspend/resume

This code is extremely bogus:
https://github.com/ergochat/readline/blob/cd7fbcc32bc9f89efc0bb7baf55abbfe86c9c5ea/utils.go#L60-L83
Instead, resumption should be detected by handling SIGCONT. (Not sure about the story on Windows.)

Related https://github.com/chzyer/readline/pull/48, but don't see if the bogus code is some windows workaround or not, there is specific "supports windows" commit in the PR

Hmm. I tested on Windows and Ctrl-Z doesn't work at all (it leaves the terminal in a broken state, but doesn't suspend the process).
I found a forum thread saying that the Windows equivalent is Ctrl-S? But I'm not sure that this functionality actually exists on Windows.

Ok sorry i know more or less nothing about windows terminals :( just got a little curious why such a function would be needed

Ah, no problem :-) I think e32441ebb791044688b7b3ec7bff99e6b24d9fe1 means "fix compilation on Windows", not "fix the actual functionality on Windows".

Aha :)

hey bro, i see redundant code in this function.

Done in #16


camenduru, i have no words to thank you for this!

Thank you for making my experience with AI such a pleasant one.

thank you all ❤


Bugfix search idp names with punctuation

The encoding used for data-idp-name would cause searches for IdP names containing punctuation, such as apostrophe, to remove valid records when users entered the character.
Punctuation and numerics are no longer considered in searches following this change providing a more flexible end user experience.

Finished in 4.25 seconds (files took 3.33 seconds to load)
229 examples, 0 failures

Coverage report generated for RSpec to /Users/bradleybeddoes/Development/aaf/discovery-service/coverage. 599 / 605 LOC (99.01%) covered.
Running RuboCop...
Inspecting 51 files
...................................................

51 files inspected, no offenses detected

Seems we need to reestablish CI for DS.

https://github.com/ausaccessfed/stockpile/issues/7


timecalculation

please use moment.js or even better https://date-fns.org/ for time calculations for better maintainability

moment.js is used.


Styling~

OMG I should make it all references to the different things in MiA ❤

Ended up using bootstrap so most of the styling comes with the creation of the different components and views


update versions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on blagoev/versions-in-readme at 88.767%

Totals

Change from base Build<PHONE_NUMBER>:
88.8%

Covered Lines:
3232

Relevant Lines:
3641

💛  - Coveralls


Problem when convert cfg model

I met a problem that I cannot run convert.py, can anyone help me with this??

I think there might be an issue with your yolov3.cfg. Can you check it our show it here?

Yes, you are right, I just double check my cfg file. My cfg file was wrong, I just replace my cfg file and it's working now. Thanks


Auger feed rate multipler

"the PID calc does not take into consideration the auger feed rate. I assume the PID calcs were created for a given feed rate which likely does not apply for all appliances, like mine, for instance. If there were a multiplier or divisor that took into account the feed rate of the auger, I think PID tuning would be easier"
"...let's assume you create a setting of 0.5g/sec feed rate. You measure the weight of the pellets with the auger chamber at the bottom of the hopper filled and flush. You run the grill for a couple of hours or so and the program tells you how much (in weight) of pellets that you have used. If you divide that weight in grams by the 0.5g feed rate you will know how long the auger has been run. If you then remove the remaining pellets and weigh them, you will know how many grams of pellets you have actually used, That will give you an averaged feed weight which you can then input and verify. Once that is known, then you can apply your multiplier more accurately."
-FastEddie49 on Discord

Yep, I think this one needs more investigation, given we are all sort of trying to figure out how all of this can be modified to get the proper results out.

I have since come to the conclusion that, in the handful of times I have used the autotune feature of temperature controllers at work, that PIDs don't need to know things like pump output in gallons/min or weights of things being heated. It's not needed to know the specifics of a system ("plant") for PIDs to work with it, the same way that PIDs don't need to be told if the application is a heating or pumping operation. So scratch that idea.
As to the second part of this, I took 2 measurements of the auger output at 20 seconds each and averaged the 2 to come up with 0.8 grams/second output. I set the feed rate in the program accordingly. I then ran a cook of just under 2 hours and did the math as described earlier. My actual pellet feed averaged over the cook time is 0.825, pretty darn close to what I achieved by using the more simple method. I did up my feed rate to 0.85 in the program because I'd rather have an overestimate than an under, but that's personal preference. I can validate that the simple method is accurate enough for most people.

I'm going to close this one out, as I'm probably not going to spend the time to update the PID.  However, with the release of v1.5.0 I welcome others to dive into the controller and hack it to their hearts content. :)


Add configuration option for Redis sentinels

Add configuration option for Redis sentinels; the Redis client can be configured to work with either single Redis server or Redis sentinels.

https://github.com/digitalocean/go-workers2/pull/3


Add rationale dropdown to Chrome extension

Per our small CvC meeting today, we would like to add "Flag for submitter review" as a reason in the Chrome extension.

@marinadistefano I'm assuming this is a "reason" for the "Follow-up" action. If not, please clarify.

@marinadistefano please download the new version 1.9.4, install and verify that the request is resolved. If not, please indicate what is incorrect or If it does then please close this issue.

Installed and tested and works as requested! Closing the ticket.

That’s correct.
On Mon, Aug 29, 2022 at 7:40 PM Larry Babb @.***> wrote:

@marinadistefano https://github.com/marinadistefano I'm assuming this
is a "reason" for the "Follow-up" action. If not, please clarify.
—
Reply to this email directly, view it on GitHub
https://github.com/clingen-data-model/clinvar-scv-curator-ext/issues/37#issuecomment-1230978024,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHFKA6TQBNCAUEQ5JUYEXYLV3VC7ZANCNFSM577CEGOA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

--
*Marina DiStefano, Ph.D., DABMGG, FACMG  *| Assistant Professor
Clinical Lab Director
Geisinger
The Broad Institute
105 Broadway Cambridge, MA, 02139
Email: @.***
@.***
she/her/hers


Make error notifications more discrete

It's important to notify user that the loading is taking time because of errors but it doesn't need to be so big...

Approved by @GuillaumeAmat, closing the issue.


final query

Hi, can pagination work if the Query is final? Thanks

Yeah it works @kiloki-official


add random currencies

add random Currencies() from ISO 4217

Great!

Should probably add an example to the readme as well.

ops! forgot to add an example on the readme. I'll add it asap.

Awesome. Merging.


Navigation Support

Navigation requires sending navigate events to a frame and providing the frame with a NativeView to navigate towards. We should be able to expose this via a module or method that component authors can import eg import { navigateTo, navigateBack } from 'svelte-native'
We can use Nativescript-vue's docs as inspiration
https://nativescript-vue.org/en/docs/routing/manual-routing/

temporary workaround:
import { topmost } from 'tns-core-modules/ui/frame'
let frame = topmost();
//somewhere to mount the component
let dummy = new ElementNode('fragment');
//mount to dummy fragment
let new_page = new SvelteComponent({ target: dummy, props: blah });
//extract the page element and its nativescript view
let native_page = dummy.firstElement().nativeView;
//use nativescripts native navigation to goto the page
frame.navigate({ create: () => native_page })

My slightly modified version. It's basically the same, but I return both the Svelte and the native page, in case it's needed.
import { topmost } from 'tns-core-modules/ui/frame'
import { ElementNode } from 'svelte-native/dom'

export const navigate = (SvelteComponent, props) => {

	// somewhere to mount the component
	let frame = topmost()
	let target = new ElementNode('fragment')

	// mount to target fragment
	props = props || {}
	let new_page = new SvelteComponent({ target, props })

	// extract the page element and its nativescript view
	let native_page = target.firstElement().nativeView

	// use nativescripts native navigation to goto the page
	frame.navigate({ create: () => native_page })

	return { new_page, native_page }

}

implements as "navigate" and "goBack" matching the the docs for Frame.navigate https://docs.nativescript.org/ui/ns-ui-widgets/frame#navigate-by-navigationentry-object
except we have page: SvelteComponentClass props: any and frame: optional different frame that isn't root in the options object


fix: include asset's owners info in response

When assets are sinked to Compass from Meteor, the UUID is not populated for the asset owners. This meant that fetching the asset either by ID or while listing the assets, the owner info (email) was not being returned because of a filter condition on UUID.
Include the owner's information available in the datastore irrespective of whether UUID was populated or not.
Closes #186

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 86.142%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
5010

Relevant Lines:
5816

💛  - Coveralls


Error when importing the dependencies from interactions

In the utils\color.py file, the following code is present:
from enum import Enum
from interactions import Color

class Colors(Color, Enum):
    RED = 0xCE3636
    ORANGE = 0xF48942
    TBGBLAU = 0x0D5EAF
    GREEN = 0x36CE36
    VIOLET = 0x9B59B6
    BLACK = 0x2F3136
    YELLOW = 0xFFA800
    BLUE = 0x0000FF

As of the current state and the imports, there is no text file that could provide interactions. Therefore, the imports or the self-referencing import should be adjusted.

The problem has been fixed and resolved in Merge Request #76. The missing dependency has been corrected. Therefore, the issue can now be considered solved and closed.


[vim] fix issues related to visualBlock mode

fixes:

repeat for changes made with I in block selection mode being applied at the lastSel instead of current cursor
missing text object for `
desired column changing to infinity when moving down at the end of the file
extra fatCursorMarks remaining in the document after using c from block selection mode

Thank you.


Images uploaded to S3 via Parse Server have file extension .bin

I'm not sure where the fault lies here (or if this is a configuration issue on my part), but I have observed the following using a Heroku-hosted Parse Server instance connected to Amazon S3 for image hosting and accessed via a Heroku hosted Parse Dashboard.

Upload a jpg image via the Parse Dashboard. The image is stored on S3 with file extension .jpg.
Upload the same image via the iOS SDK. The image is stored on S3 with file extension .bin.

Images still load correctly in the iOS app, but it is much less convenient when viewing via the dashboard since instead of clicking the image to view it in the web browser, the .bin is downloaded  to the computer, where it must be renamed with the correct extension before it can be viewed.

I think I've got to the bottom of this now. Using
let photoFile = PFFile(data: data)
will indeed result in a .bin file, since the content type isn't known. But
let photoFile = PFFile(data: data, contentType: "image/jpeg")
results in a jpeg file.
I suspect that the previous Parse.com version of the server was doing some kind of MIME type detection so this wasn't an issue.


Fix doc errors around renderAsSvg configuration

Am I the only one that thinks it's bad that we default to render as bitmaps instead of using SVG whenever we can?

Thanks for fixing the docs!

Am I the only one that thinks it's bad that we default to render as bitmaps instead of using SVG whenever we can?

I wouldn't say that rendering to a raster image is 'bad' per se. With a raster image you are sure that the rendering looks the same no matter what pdf viewer and environment. And if you have a lot of vector data, a raster image might take less space.


Can't upload a preset file

The button on both the jobs and presets page (http://hot.geoweb.io/en/jobs/configurations/) doesn't seem to do anything (at least on Firefox). Don't see an error in browser console. Not sure if it's supposed to work yet, but anyway..
Also, is the beta running on an old data set? Tried an export for Dar es Salaam, but it looked quite out of date.

Thanks Paul, see the button issue and will fix. Looks like a firefox issue.  Looking at the minutely update logs it seems they are being applied but will investigate further. What were you expecting to see in Dar es Salaam?

A lot more buildings.. basically, the export from http://hot.geoweb.io/ looks like it would have looked at the end of June/start of July.
From http://export.hotosm.org/:

From http://hot.geoweb.io/:

Paul, thanks for this. Looks like there is an issue with the application of minutely diffs on the dev instance of overpass. Changesets are being fetched correctly but are not being applied. Data is current as at Thu Jun 25 19:50:03 UTC 2015. Will aim to fix this between beta and live release when deploying to live server.

It is correct that the data exported at this moment is still stuck in June? Since it still seems to be, in an export just moments ago.

Will be fixed for live release which should be happening in the next 24hours.


Update contact information

I just noticed that the content on http://heliopython.org/contact/ is:
Contact content goes here. My e-mail is [email@something.com](mailto:email@something.com).

It would be helpful to add an email address or contact person for this effort.
Thanks!
-Nick

@alex-aquila would you be willing to add your email here and mention that you should be emailed to join the mailing list? I'd like to also add a link to the riot channel here.

I think I just did this - let me know if it worked or not!

Closed with PR #11.


Display randomly jumping while printing

Using this snippet, the display should keep printing random numbers, but also shifts the vertical and horizontal in a "pacman-repeat: fashion:
import display
from random import random

while True:
    l = []
    y = 0
    for i in range(8):
        l.append(display.Text(f'{random()}{random()}{random()}', 0, y, display.WHITE))
        y += 50

    display.brightness(0)
    display.show(l)

On the other hand, printing constantly the same data does not seems to trigger any glitch:
from display import *

while True:
    show([
        Text('0.54864070.42984010.3100815', 0, 0, 0xffffff),
        Text('0.35902540.63464520.06198609', 0, 50, 0xffffff),
        Text('0.57318440.86822380.2571942', 0, 100, 0xffffff),
        Text('0.43299210.42563390.09295154', 0, 150, 0xffffff),
        Text('0.067244410.34240930.9991117', 0, 200, 0xffffff),
        Text('0.68130370.072521210.079427', 0, 250, 0xffffff),
        Text('0.79938940.8310890.2462578', 0, 300, 0xffffff),
        Text('0.52144990.59871090.1736188', 0, 350, 0xffffff),
    ])

A shorter reproducer showing a single line of text:
from display import *

while True:
    s = f'{random()}{random()}{random()}'
    show(Text(s, 0, 0, 0xffffff))

By inserting delays between prints, there is less often any glitch happening, sometimes very rarely.
This snippet did produce the glitch occasionally:
from display import *
from random import random
from time import sleep_ms

while True:
    s = f'{random()}{random()}{random()}'
    show(Text(s, 0, 0, 0xffffff))
    print(s)
    sleep_ms(50)

But by replaying the text that did produce a bug, the offset glitch does not show-up anymore:
from display import *
from random import random
from time import sleep_ms

lines = [
    '0.5805330.15940280.6911397',
    '0.79491080.43136680.3676161',
    '0.63313060.0084255930.9829068',
    '0.63552930.8199680.9345885',
    '0.59016860.86261960.8806037',
    '0.34546230.33397380.03462267',
    '0.52931680.4543540.4723445',
    '0.39820990.90908970.5736155',
    '0.012948750.20397440.8955362',
    '0.59366110.9991580.764183',
    '0.93191710.23650120.189626',
    '0.089686760.93767630.04712594',
    '0.8846930.91967950.3160187',
    '0.038520930.65211540.3805225',
    '0.8610210.12769660.08911633',
    '0.14713990.803730.5070182',
    '0.52613640.36443480.9711175',
    '0.0065060850.54428820.1900092',
    '0.33138780.006158710.1245269',
    '0.1975520.010668280.6214256',
    '0.53798060.32947040.3933594',
    '0.589480.043236850.9271793',
    '0.012523890.78625310.6820898',
    '0.96446270.2030720.3585348',
    '0.32869330.64580760.3379695',
    '0.94329910.324850.5810333',
    '0.35280440.21398450.4394406',
    '0.29259660.70331120.5737033',
    '0.11312340.48636320.05542839',
    '0.69707190.2586130.007415295',
    '0.11193860.68077210.931271',
    '0.37418390.31486460.335043',
    '0.51688080.26435510.2860227',
    '0.7012670.17604420.9427845',
    '0.93389740.41557050.8782581',
    '0.95420190.83162290.1003851',
    '0.60233410.60269330.5109658',
    '0.35414360.24549960.6465296',
    '0.65653680.94330510.4313878',
    '0.25421420.92393830.8194826',
    '0.52898790.35168420.1732073',
]

for s in lines:
    show(Text(s, 0, 0, 0xffffff))
    print(s)
    sleep_ms(50)

This suggests that it is not related to the content, but instead by the moment where the glitch happens.
Maybe printing text at the exact same moment it is being flushed to the display?
What happens to the SRAM read address while it is both read and written at the same time?
Should there be some locking mechanism here?

Cross reference: https://github.com/sathibault/vgrs/issues/13

The approach proposed by @sathibault to fix it is to use double-buffering to avoid clashing a same memory region.

Fixed by #210


Javascript Error: Filtering popup hide method fails

Hi, first of all, thanks for this library. It's looking really good.
Here's the problem and how to repro it.

Go to http://mvc6-grid.azurewebsites.net/
Select Ajax Grid
Using Chrome, hit F12 to show developer Tool
Open the Console Tab
Open any filter on the column grid header
Hit the OK or Close buttons on the popup

Result:
The following error is shown:

This also happens with non-ajax grids. On the Quickstart, you can see that this happens briefly but the popup disappears quickly as the page is reloaded.
On my development machine I see the following:

Thanks in advance!

Bug was introduced in v4.1.2, so if your main audience is not mobile users you can safely rollback to v4.1.1 for now.

For anyone else who stumbles into this, I simply updated he mvc-grid.js file with the single line commit you made. Thanks for the quick response!


Account and sign-in does not persist across windows

Type: Bug
Have two or more VSCode windows open - i.e. two windowed instances which have a directoory open each, and one that is just a mix of random files, not all saved to disk.
When VSCode needs to sign in, you have to sign in for each instance, as it doesn't recognise sign in from other instances, and on exit and re-open, you typically have to sign in all instances again.
I am using MS account for sign in and sync.
VS Code version: Code 1.71.2 (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:03:37.738Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2235 CPU @ 3.80GHz (12 x 3792)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.73GB (5.67GB free)

Process Argv
--crash-reporter-id 78b31145-6d3c-4704-b7f4-5a189d012012

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

vscode-openapi
42C
4.13.0

vscode-caniuse
aka
0.5.4

Bookmarks
ale
13.3.1

xml
Dot
2.5.1

EditorConfig
Edi
0.16.4

windows-explorer-context-menu
ele
3.2.1

vscode-guid
hea
1.4.20

rest-client
hum
0.25.1

formate
Mik
1.2.1

csharp
ms-
1.25.0

data-workspace-vscode
ms-
0.3.0

mssql
ms-
1.16.0

sql-bindings-vscode
ms-
0.3.0

sql-database-projects-vscode
ms-
0.19.0

remote-containers
ms-
0.251.0

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.3

vscode-remote-extensionpack
ms-
0.21.0

powershell
ms-
2022.8.5

autodocstring
njp
0.6.1

vscode-yaml
red
1.10.1

code-spell-checker
str
2.8.0

keyoti-changeallendoflinesequence
vs-
0.0.3

vscode-todo-highlight
way
1.0.5

markdown-pdf
yza
1.4.4

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30557514
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
hb751961:30553087
dsvsc012cf:30540253
azure-dev_surveyone:30548225
2144e591:30553903
fc301958:30567733

Can you provide the contents of the Microsoft Authentication output pane:

For each of these instances.
And also anything that shows up in Dev Tools:

ctrl + shift + P
"Dev Tools"

It should persist across windows so I'm suspicious of a bug.

Upon first open of a new Windows session, all instances state the same:
[Info  - 23:18:14.298] Reading sessions from secret storage...
[Error  - 23:18:14.747] Key was not found in SecretStorage.
[Info  - 23:18:14.747] Got 0 stored sessions
[Info  - 23:18:14.783] Getting sessions for the following scopes: email offline_access openid profile
[Trace  - 23:18:14.783] No session found with idtoken scopes... Using fallback scope list of: email offline_access openid profile
[Info  - 23:18:14.783] Got 0 sessions for scopes: email offline_access openid profile

After signing one of them in:
[Info  - 23:20:19.290] Getting sessions for all scopes...
[Info  - 23:20:19.290] Got 0 sessions for all scopes...
[Info  - 23:20:20.470] Getting sessions for the following scopes: email offline_access openid profile
[Trace  - 23:20:20.470] No session found with idtoken scopes... Using fallback scope list of: email offline_access openid profile
[Info  - 23:20:20.470] Got 0 sessions for scopes: email offline_access openid profile
[Info  - 23:20:24.226] Logging in for the following scopes: email offline_access openid profile
[Info  - 23:20:28.149] Exchanging login code for token for scopes: email offline_access openid profile
[Info  - 23:20:28.686] Exchanging login code for token (for scopes: email offline_access openid profile) succeeded!
[Info  - 23:20:28.687] Setting token for scopes: email offline_access openid profile
[Info  - 23:20:28.688] Login successful for scopes: email offline_access openid profile
[Info  - 23:20:28.688] Token available from cache (for scopes email offline_access openid profile), expires in 4083998 milliseconds
[Info  - 23:20:28.691] Getting sessions for the following scopes: email offline_access openid profile
[Info  - 23:20:28.691] Got 1 sessions for scopes: email offline_access openid profile
[Info  - 23:20:28.691] Token available from cache (for scopes email offline_access openid profile), expires in 4083995 milliseconds
[Info  - 23:20:28.692] Getting sessions for the following scopes: email offline_access openid profile
[Info  - 23:20:28.692] Got 1 sessions for scopes: email offline_access openid profile
[Info  - 23:20:28.692] Token available from cache (for scopes email offline_access openid profile), expires in 4083994 milliseconds
[Error  - 23:20:30.53] Error: Not enough memory resources are available to process this command.

Going back to one of the other instances, and opening the Account icon:
[Info  - 23:22:10.198] Getting sessions for all scopes...
[Info  - 23:22:10.198] Got 0 sessions for all scopes...

In Dev Tools, I only see the following console entry after signing in to one of the instances:
INFO Settings Sync: Updated current session ${GUID1}/1:live.com:00037FFEA9CBEFA3/${GUID2}

Across instances, ${GUID1} was the same, but ${GUID2} differed.
In two of the other instances, this was logged in the Dev Tools console:
[[object Object]]Invalid Version: null
$onExtensionRuntimeError @ mainThreadExtensionService.ts:79

TypeError: Invalid Version: null
	at new Ii (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:57:27378)
	at xO (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:57:39339)
	at Object.BO (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:57:39725)
	at zc.checkForPowerShellUpdate (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:85:8754)
	at zc.startLanguageClient (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:85:8586)
	at async zc.startPowerShell (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:85:5586)
	at async zc.start (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:85:316)
	at async cq (c:\Users\${env:UserName}\.vscode\extensions\ms-vscode.powershell-2022.8.5\out\main.js:131:13834)
	at async E._activate (c:\Users\${env:UserName}\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:85:8224)
	at async E._waitForDepsThenActivate (c:\Users\${env:UserName}\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:85:8166)
	at async E._initialize (c:\Users\${env:UserName}\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:85:7530)
$onExtensionRuntimeError @ mainThreadExtensionService.ts:80

/duplicate https://github.com/microsoft/vscode/issues/130893
That issue has some workarounds to try and get you unblocked... but ultimately is a really tricky one for me to lock down so there's not been much progress.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

Thank you for jumping in @TylerLeonhardt, I really appreciate it.
The command outlined in this comment on that issue, and on the troubleshooting docs, has fixed the problem for me for now. As stated on that issue, I will keep an eye on things and reply if things change.


Changed name of Long handed inserter to Long-handed inserter, so it is aligned with the name in game

I merge master in and try to solve the problem again, so it works with the current code.


Feature Request: Custom Policies

Rasa version:
0.15
Python version:
3.6
Operating system (windows, osx, ...):
Ubuntu 18.04
Issue:
Allowing for custom policies. If I have a model, the only way to add that to a list of policies now is to add the model to the policies directory and update the policies/init.py file. Would be great to have a way around this.

You can have a custom policy: https://rasa.com/docs/core/policies/#configuring-polices-using-a-configuration-file


✨ add support for stack overflow and stack exchange

Hi! I added support for Stack Overflow and Stack Exchange, as discussed in https://github.com/jpanther/lynx/discussions/5.
Thanks for developing Lynx!

Thanks, I'm happy to add these in however could you please ensure the icon SVG files conform to the same standard as the rest in the theme? For instance, you must ensure they contain items like fill="currentColor" within the path tag. You can refer to the existing icons as an example.

Thanks for the feedback! I updated my MR with the changes you requested.

@jpanther Hi! This PR has been marked as stale and will be closed automatically soon. Could you please review it?

Thanks, I'll merge this in.


Update README.md

I encounter some Git buffer size limits when trying to download all commits history of the repository, such as:
error: 5815 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF```

therefore the installation is faster and there are not errors when I download only the last version of the repository

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

Hi! Thanks for this. If you could run pre-commit, we can merge:
pip install pre-commit
pre-commit run --all-files

Hello @baberabb. Thanks for your response. I fixed the lining issue. All the workflow works without errors now.


Tutorial?

Make a video tutorial, the main issue with this project is the lack of documentation available, FYI there's already a company called Tetherfi so whenever anyone try to look up on the internet for help those people shows up, I've been trying to make it work on kali using proxychains plus all "instructions" provided  with no luck at all. Method Android + Kali Linux/xfce

Hi,
I'll think about a better way to convey the setup steps, but right now don't have the spare time to record anything useful.
You shouldn't need proxychains, as that is a transparent setup. I'm not even sure if proxychains transparent would work in this case, as TetherFi is an explicit proxy that relies on HTTP CONNECT to be passed through.
You should just be able to export  environment variables - see the Wiki for some common setups.


Rename knative-sandbox to knative-extensions

This is an automated PR to replace  knative-sandbox strings with knative-extensions

This PR fails due to shellcheck tests unrelated to the renaming.
See https://github.com/knative-extensions/eventing-kafka/actions/runs/5758835365/job/15611946836?pr=1371
Please fix the project code such that it passes shellcheck tests in a separate PR, once this is done we can get this PR going

@davidhadas I don't see what's wrong (or why the other PRs go in just fine, like updates of dependencies)

@davidhadas I don't see what's wrong (or why the other PRs go in just fine, like updates of dependencies)

My best guess (and maybe @upodroid can confirm or correct me) is that the productivity team made some changes at some past time. Maybe they added shellcheck tests or maybe they changed the shellcheck configuration. As a result, some repositories became non compliant with respect to the shellcheck tests.
However, these tests are performed only when someone changes a file, so any file not changed since the productivity team made the changes goes unnoticed. Once someone make any change in a file which fail these tests, the tests are performed and the PR fail.
If this is true, I think this process in productivity  should change  - the process should not rely on future PRs failing as it leads to a very long tail of failing PRs and confused developers.
But as I said, this is nothing more than my working theory :)

/override "style / suggester / github_actions"
/override "style / suggester / shell"


Virtual Column Scrolling

We have a need for a pivot table which can be arbitrarily large (millions of rows/columns).  Instead of specifying a giant array of columns, we want to be able to define how many columns there are and what their size is and then provide a cellsAt(x, y, length, height) callback that returns a 2-d array of the resultant values to display.  Looking at the code, it has been designed to only virtualize rows.  Do you have any plans (or would you be interested in consulting?) to add column virtualization support?

Hi Noah,
I have given a little thought to virtualizing columns but nothing too concrete yet. There's interest on my end for adding support because that's something that (I don't think) any of the existing virtual table libraries provide currently.
Given the holiday season I'm not sure what the availability or timeline on my end may be. What did you have in mind? Feel free to shoot me an email if you'd like to chat more in depth.

Closing this issue due to inactivity. @noah79, you can reach back out to me via email if this is still a feature you're looking for.


Adds command context remove, Closes #4043

This PR adds the command context remove. Which remove the context from the .m365rc.json or removes the file if it only contains the context parameter.
Closes #4043

Almost there! Let's do one more adjustment and we'll get it in 🚀

Thank you for your review! 🚀

We're almost there @nicodecleyre. Let's do a few small adjustments and get it merged 🚀

Thank you @waldekmastykarz , PR updated.

Very nice and we're almost there. Let's have another look at the tests so that we can properly use them to maintain the codebase going forward 👍

Thank you! Updated the PR with your remarks

Merged manually. Thank you! 👏


fixes #193: WindowsServiceRecoveryController fails: Failed to change service recovery options

@pianomanjh Any progress on this? We would need this as well.

@eggapauli If you try out the branch/PR and tell us it's good, then we'll merge =)

I updated to use a safe token handle, but did not have time to test today. @eggapauli or @haf, can you check this change?

I just tried it and for me this works, thanks.

FYI, I reverted this commit and am putting it into develop behind all the other develop commits that were well ahead of it.
For future reference, we only merge rebased commits onto develop until they are released.


ci: build inside a builder container

Something like this, @erikgb? We will, of course, need to ditch anything maven from the actions if we select to go this way.

Something like this, @erikgb? We will, of course, need to ditch anything maven from the actions if we select to go this way.

Yes! ❤

Can you remove the maven build from outside the Docker-build?

Can you remove the maven build from outside the Docker-build?

Done. Although I'm still on thin ice when it comes to GitHub actions, so not sure it is correct.

We should also probably test changing pom.xml. That should invalidate the Docker cache from the copy pom.xml statement. I expect that to be the same as the GH Maven cache. We should not have SNAPSHOT-dependencies. If we did, those would be problematic - since they will not invalidate the cache.

We should also probably test changing pom.xml.

I tested locally, first changing the pom and then a source file. The results were as we hope for.

We should also probably test changing pom.xml.

I tested locally, first changing the pom and then a source file. The results were as we hope for. But I'm only using docker build, and nothing fancy.

The GH Docker cache setup is quite aggressive, but this seems to work as we want.

The GH Docker cache setup is quite aggressive, but this seems to work as we want.

There are commits for "change pom" and "change source file" that probably have some action runs.


Updates don't work

The updates as presented in the readme don't work.
If I do:
// Update an existing struct
db.First(&user)
user.Name = "jinzhu 2"
user.Age = 100
db.Save(&user)

The generated update statement will update all columns.
So does this
db.Model(&user).Update("name", "hello")

and this does nothing:
db.Model(&user).Updates(User{Name: "hello", Age: 18})

I should note that all of these would be happening in a Method on user, such as
func (u *User) UpdateStuff(...)

I'm having to fall back on db.Exec("UPDATE ... ")  to get the right behaviour, which kinda defeats the point.

When I debug I found that if the value is nil value, it did not work. Such as the value is 0 or false. Why?


[Feature Request] Integrate Guardrails for Amazon Bedrock to Enhance Application Security

Describe the solution you'd like
I would like to request the integration of Guardrails for Amazon Bedrock into this application. This feature would allow users to leverage the capabilities of Guardrails for Amazon Bedrock directly through the application interface, providing an additional layer of security and privacy controls.
Why the solution needed
Integrating Guardrails for Amazon Bedrock into this application is crucial for enhancing the overall security posture. As our application handles sensitive data and processes critical operations, it is essential to incorporate robust security measures to protect user information and maintain trust. By leveraging the safety filters and privacy controls offered by Guardrails for Amazon Bedrock, we can ensure that our application adheres to the highest standards of security and data protection.
Additional context
For more information about Guardrails for Amazon Bedrock, please refer to the following link:
https://aws.amazon.com/jp/blogs/aws/guardrails-for-amazon-bedrock-now-available-with-new-safety-filters-and-privacy-controls/
This feature will significantly enhance the security capabilities of our application, providing users with peace of mind and demonstrating our commitment to safeguarding their data.
Implementation feasibility
While our technical expertise may be limited, we are willing to actively engage in discussions with the development team to explore potential solutions, determine the most suitable approach, and provide assistance throughout the implementation process. We believe that collaborating closely with the team will enable us to successfully integrate Guardrails for Amazon Bedrock into our application.

[x] Yes
[ ] No

I'd be really appreciated for your willingness. Could you contact with me by linkedin to discuss more further?

I see another ongoing thread about Converse API. Once it's done, I think it would be easier to implement this Guardrail. I think it probably would be something like:

Keep Bedrock Guardrails creation and update to be done on AWS console
Modify the custom bot admin page, add capability to attach optional Guardrail ID and version to custom bot

Change the frontend, add optional dropdown for Guardrail
Add permission to manage and apply Guardrail somewhere (on CDK?)
Modify backend that store custom bot to DynamoDB

Change the get_bedrock_respose method to use guardrail if specified for the custom bot
Not sure how to handle the agent invocation


documentation(Getting started): update 5-working-with-files.md

Small improvement to the documentation

Hey, thank you for your pull requests! I'm closing both this one and the other one without merging as they're actually not improving the grammar of the documentation.
We've got some open issues that we'd be open to contributions on, if you want I can look through and find one that would be good for a first time contributor :)


[hotfix] redefine the 'stage'

The stage we have defined in base_generator and base_preprocessor should be redefined. It should contain '[train, test, predict]', where both 'train' and 'test' need to provide 'x' and 'y' as inputs, and 'predict' only need the 'x' as input.

train, dev and test or train, validation and test? See here


Add ability to store legacy token if requested

no need for this since it's only available for salesforce


fix(11367): parquet writer defaults

WIP: first we need consensus on the desired defaults. Second, decide if the extern parquet's defaults can (or cannot) override the datafusion default.
Which issue does this PR close?
Closes #11367
Rationale for this change
When we switched from using the parquet's ArrowWriter (with options) to the parallelized parquet writer (with it's own options), we ran into unintended behaviors due to different default settings.
Here are the places where the current defaults differ:

setting_name
applied
default, datafusion (ParquetOptions)
default, parquet (ArrowWriterOptions)

data_page_row_count_limit
file
usize::MAX
20_000

column_index_truncate_length
file
None
Some(64)

compression
column default
zstd
uncompressed

dictionary_enabled
column default
None or true †
true

statistics_enabled
column default
None or page †
page

max_statistics_size
column default
None or 4096 †
4096

† For these settings, datafusion has no default (None). However, once datafusion's ParquetOptions are used by the extern parquet (a.k.a. converted to parquet's ArrowWriterOptions) then it uses the extern parquet's defaults. Refer to the newly added tests.
.
Additionally, there are differences in the bloom filter configurations based upon partial definition (a.k.a. leaving some as default, and some as defined):

bloom_filter_on
fpp
ndv
if build from datafusion's ParquetOptions
if build from (arrow-rs) parquet's WriterPropertiesBuilder

false
none
none
None
None

TRUE
none
none
Some(BloomFilterProperties::default)
None

true
SOME
none
Some(BloomFilterProperties: fpp,default_ndv)
None

true
none
SOME
Some(BloomFilterProperties: ndv,default_fpp)
None

What changes are included in this PR?
The first commit is adding tests to define and demonstrate the differences in the defaults.
After discussion and consensus, we'll add other commits (as needed) for implementing the desired changes.
Are these changes tested?
Yes.
Are there any user-facing changes?
No API changes. Only potential future changes to alleviate unintended consequences.

Have opened the upstream PR.
Closing this earlier draft (in our fork).


cherry-pick(controller, api): add label selector to BlockDeviceClaim (#397)

add label selector to BDC api.
add label selector to list method using labels.Selector
include hostname label in the metav1.LabelSelector from BlockDeviceClaim
add test cases for selector in BDC

cherry-pick #397
Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

Codecov Report

Merging #399 into v0.4.x will increase coverage by 0.11%.
The diff coverage is 75%.

@@            Coverage Diff            @@
##           v0.4.x    #399      +/-   ##
=========================================
+ Coverage   50.78%   50.9%   +0.11%
=========================================
  Files          52      52
  Lines        2595    2605      +10
=========================================
+ Hits         1318    1326       +8
- Misses       1168    1169       +1
- Partials      109     110       +1

Impacted Files
Coverage Δ

...er/blockdeviceclaim/blockdeviceclaim_controller.go
40.96% <75%> (+2.5%)
:arrow_up:

Continue to review full report at Codecov.

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


Removing scenes for functions and classes

Right now most of the features in XR Tools need to be added through subscenes. That makes a lot of sense as for many we need a handful of objects properly configured to make the function usable.
However we also define all classes with class_name which makes it possible to see them in the node tree and just add the node. If you do this, you get a node of the correct type, with the script attached, but it doesn't use the related scene.
Most of the scenes are really simple with just a handful of objects. It is fully possible to add the needed nodes in the _ready function of a script and remove the need to have the scenes.
Obviously this is something worth discussing whether we want to do this and to what type of classes to restrict this too. For instance, the scenes meant to be inherited should remain scenes, but we could fully remove the function scenes for instance.
Also this would be a serious breaking change and maybe something we should consider only for Godot 4.

Just talked with a user that's trying to position their player in the scene, and the player leaves a bunch of objects behind. These are all the helper objects under the movement functions (the wind sensor area), and as the MovementProvider isn't a Spatial node, the movement of the parent does nothing for the child.
Switching to script-generated helper objects would fix this problem - as would making MovementProvider extend from Spatial.

After discussing with Malcolm, we're slowly changing this over where we can, there is no point in keeping this issue open.


Locked version of net-ssh due to deploy failure for OpenShift.

This is the same issue we recently encountered with pf.org.

👍  good find and happy to see this fix merged

Looks good


Add a field for arbitrary metadata in annotation

For some, it can be useful to keep arbitrary metadata. Since artifacts.yaml is a yaml file, it could be yaml or anything that works. Something like:
mymodel:
  path: path/to/model
  extra: <.......arbitrary metadata.....>

Some options for the name: extra, custom, info, notes, meta.
$ cat example-gto/artifacts.yaml
awesome:
  description: This model is very cool
  labels:
  - ml
  - cool
  path: models/awesome.pkl
  type: model
  custom: 1


Aeneid textproto comments

There are a few lines that require further investigation as to why it was scanned as defective, I will look into it. Let me know about your preferences for comment wording, and I can apply those across the other files as well.

Just a note that when writing the new pron for diastole comments, I wrote out the word whole, even if elision occurs.

Is there a way to produce a new textproto file (aka running latin_scan) while preserving the old comments, and avoiding the need to reinsert them every time we generate run a new scan?

No such obvious tool exists, so we will perhaps want to make one. I do not
suppose it will be terribly challenging, it's mostly a question of thinking
what the API ought to be (e.g., how does one pass in files etc.).
On Sun, Mar 6, 2022 at 12:10 AM jillianchang @.***>
wrote:

Is there a way to produce a new textproto file (aka running latin_scan)
while preserving the old comments, and avoiding the need to reinsert them
every time we generate run a new scan?
—
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/latin_scansion/pull/97#issuecomment-1059896741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OKFQ5VJYJNSEG535S3U6Q44HANCNFSM5PKZGW5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Is this ready to merge?

After the pronunciation change gets implemented, I think I’m going to have to rerun the engine to generate new textproto files for Aeneid 1 and Aeneid 2, as the change should cause several of the now invalid scans to scan properly. I believe I would have to manually transfer these comments after rerunning? In that case, do you want to merge now or after that is done?

On Apr 2, 2022, at 6:18 PM, Kyle Gorman @.***> wrote:

Is this ready to merge?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

After the pronunciation change gets implemented, I think I’m going to have to rerun the engine to generate new textproto files for Aeneid 1 and Aeneid 2, as the change should cause several of the now invalid scans to scan properly. I believe I would have to manually transfer these comments after rerunning? In that case, do you want to merge now or after that is done?
…
On Apr 2, 2022, at 6:18 PM, Kyle Gorman @.***> wrote:  Is this ready to merge? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

I'm okay with us figuring out how to transfer the comments when we get to that point, so I'll merge.


https://qiskit.org/documentation/install.html needs a fix!

I would call it a doc bug

https://qiskit.org/documentation/install.html

Current text is as follows,

We recommend using Python virtual environments to cleanly separate Qiskit from other applications and improve your experience.
The simplest way to use environments is by using the conda command, included with Anaconda. A Conda environment allows you to specify a specific version of Python and set of libraries. Open a terminal window in the directory where you want to work.
Create a minimal environment with only Python installed in it.
conda create -n name_of_my_env python=3
source activate name_of_my_env

When user abide by the recommendation (above),

Conda would install the latest python version at the time, And Qiskit may or may not be compatible with latest python release. (Current stable version is 3.8 and qiskit isn't compatible yet to python 3.8). Perhaps we can add this piece of information in the html file, notifying the user about the stable python release version. So that new comer's life becomes that much easy!

Documentation doesn't speak about the need to install gcc under new environment that it asks users to create.

there were few issues created which were closed later, but documentation didn't changed. So someone who goes look up in the closed issue will come to know about the fix. But folks who are just starting up with Qiskit may find it difficult!

Does existing documentation need to be updated with new content? -- Yes

index.html

If we decide to notify user about the stable python release version then we can simply add a line informing user about it. But this would require repeated updates. (As soon as supported version changes it should be updated for it as well)

reminding user to install gcc in their new environment should be easy and straight forward approach!

Note: - I want contribute to the community myself, please let me know if I can make these changes myself!

@a07r please do give it a shot. We'll review your work. Thank you.

@derivation , @lzdanski
Current status -
I have opened up a thread in Stack exchange regarding the smartest way possible to make Python version dynamic on documentation -
the same can be accessed at - https://quantumcomputing.stackexchange.com/questions/8720/how-do-i-find-out-which-stable-python-version-qiskit-will-be-compatible-to-at-a
Let me know if you guys have any insights on it, if you guys think it isn't possible at all (and that only with experience one will know which is the stable version) then perhaps I will work towards adding the simple line which reminds user about the need to install GCC on the new anaconda environment.

I'd argue that the documentation is probably fine, it's just new version updates do not happen in a vacuum and can't happen instantly. Python 3.8 was released less than 1 month ago and the entire python ecosystem is working to both make sure code is updated to work with the new version (because new python versions are never 100% compatible) and push out releases with binary wheels that support the new version (because all packages with compiled extensions need to be rebuilt for new versions). For qiskit this is also true, there are several patches to add python 3.8 support to multiple elements already Qiskit/qiskit-terra#3268 Qiskit/qiskit-aer#428 Qiskit/qiskit-ignis#298 but until this week we weren't even in a place where we could install all of qiskit's upstream dependencies on python 3.8 (and some are still missing, but it's not blocking testing). This is just a temporary blip that is happening right now because of a recent new release of python. It doesn't seem worth the effort to try and restructure things when this will likely all be sorted within the next month and won't be a problem again (well at least for another 12-18 months when python 3.9 is released)

I understand/agree, @mtreinish
I brought it up only to double check if we wanted to address (perhaps mainly because documentation reads 3.5 or later) and from user's (especially someone starting afresh) point of view it can be a bummer

@mtreinish @derivation -
There were a few issues reported (see the references above)
Are we still not doing anything about it?

I defer to @mtreinish on this, given his last reply.

@a07r, as Matthew mentioned already, packages are working on providing Python 3.8 support.  As such, there as some packages that have yet to provide wheels for Python 3.8, and thus need to be built from source.  This is not something that we need to correct on our part, and is part of the regular Python update cycle.  It just requires time.  Note also that Qiskit is still working on several issues related to Python 3.8, and we are focused on getting that done.

@nonhermitian / All,
I understood what @mtreinish said in the first place.
Currently the documentation reads Python 3.5 or later is stable version to install qiskit, And someone starting afresh would never know Python 3.8 isn't a supported version yet. And just as we've seen issues ( "pip install qiskit failing" ) being reported will keep getting reported.
I was hoping to get your help in addressing this situation wisely, Cause I simply can't just go and add a line which would let user (new) know that for time being qiskit isn't supported on Python 3.8 for the obvious reason that whenever another new version of python is released we would hit this same bump even then.
And hence I seek your help!

Hi all, this is a tricky docs situation. We don't want to include something that requires constant updates, but we also need to let new users know how to diagnose any installation problems, especially new users, as @a07r has pointed out.
Instead of including specific versions that are causing compatibility issues between Qiskit/Python, this topic really needs to include the process for how to check which version of Python is compatible with Qiskit.
@mtreinish responded with that info in the Stack Overflow post @a07r created, along with instructions for how to check which specific versions of python are supported by the Qiskit package that is installed by pip install qiskit. This was a really useful tip from Matthew that is a good practice for any new user, and it address everyone's concerns.
@derivation and I talked about how we should update this part of the docs. What does everyone think of a change like this?
Current version:
Qiskit supports Python 3.5 or later.
Change to:
Qiskit supports Python 3.5 or later. However, both Python and Qiskit are evolving ecosystems, and sometimes when new releases occur in one or the other, there can be problems with compatibility.
If you're having trouble installing or using Qiskit after updating Python, check the Qiskit Package metadata ( https://pypi.org/project/qiskit) for Programming Language to see which specific versions of Python are supported.


[Enhancement] Update Github Workflows

Jira Link:
Description
update dependency on github actions and add discord-pr-bot
Type of Change

[ ] Bugfix
[x] Enhancement
[ ] New feature
[ ] Breaking change (fix or feature that would cause the existing functionality to not work as expected)

How Has This Been Tested?

[ ] New unit tests added.
[ ] Manual tested.

Checklist:

[x] Using Branch Name Convention

feature/JIRA-ID-SHORT-DESCRIPTION if has a JIRA ticket
enhancement/SHORT-DESCRIPTION if has/has no JIRA ticket and contain enhancement
hotfix/SHORT-DESCRIPTION if the change doesn't need to be tested (urgent)

[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made the documentation for the corresponding changes.

Since the test is failing due to issues with the Jitsuyo and Keisan packages, could you fix those packages first before merging this?


Spelling

Generated by https://github.com/jsoref/spelling f; to maintain your repo, please consider fchurn

@jsoref thanks for the updates! Could you rebase latest master on this PR? I think that will fix the test failures 🤞

@feelepxyz sure, done.


Fix #254

close #254

Thanks for your contribution. :)

Could rebase your code base and push again?  Thanks.

Could rebase your code base and push again? Thanks.

@yang8621  done

I think your patch is OK.  But the full unit test cannot pass. You can also try:
cd test
python3 test_kms_with_cli.py --url https://<ip_addr>:

I think your patch is OK. But the full unit test cannot pass. There should be some existing inside code issues. You can also try:
cd test python3 test_kms_with_cli.py --url https://<ip_addr>:

I found that the issue was caused by ehsm_km_service can't connect to CouchDB.  sometimes Docker container is started but CouchDB is unavailable. To resolve this, I have added a check to the shell script. @yang8621

I think your patch is OK. But the full unit test cannot pass. There should be some existing inside code issues. You can also try:
cd test python3 test_kms_with_cli.py --url https://<ip_addr>:

I found that the issue was caused by ehsm_km_service can't connect to CouchDB. sometimes Docker container is started but CouchDB is unavailable. To resolve this, I have added a check to the shell script. @yang8621

Do you mind sending an email to me<EMAIL_ADDRESS>


Support for Sass >= 1.33

Hello, since sass 1.33.0 (dart-sass), they introduced deprecation warning for division operation with slash.
See : https://sass-lang.com/documentation/breaking-changes/slash-div
I use ContentTools in one of my project, and got a lot of warnings that I can't fix :)
2 options : using css calc() or @use 'sass:math';
Thanks !

@Ei-aaie thanks for the heads up on this - there's not a lot of CSS in the project and SASS provide a nice migration tool by the looks of it so I'll push a fix up this weekend.

I forked the project, it looks like it runs a Ruby sass version, not Dart-sass
The result is that the migration does the job, but the build fails at :
sass:build
Error: Invalid CSS after "...  $number: math": expected "}", was ".div($number, $..."
        on line 45 of external/styles/bourbon/helpers/_str-to-num.scss
        from line 25 of external/styles/bourbon/_bourbon.scss
        from line 5 of src/styles/content-tools.scss

40:       }
41:
42:       @else {
43:         // Move the decimal dot to the left
44:         $divider: $divider * 10;
45:         $number: math.div($number, $divider);
46:       }
47:
48:       $result: $result + $number;
49:     }
50:   }

math lib is dart-sass specific :)
There is a dart-sass grunt lib, but I'm not very familiar with grunt / cofee synthax
https://www.npmjs.com/package/grunt-dart-sass

So I think locally now we use libsass for comping the project, but there's probably still the old ruby sass requirements in the last release of CT. Will look into what's possible, it's likely some of if not all the divisions can be removed to be honest.

After compiling locally, ~50% can be replaced by multiplication without changing any deps, for example :
-        margin-top: -($dialogHeight / 2);
+        margin-left: -($dialogWidth * 0.5);

sass-migrator uses this kind of smart fix if t's possible, and if not, applies math.div

@Ei-aaie thank you for the report and your work looking into the issue - I've pushed up a new release 1.6.15 which removes the reliance on the division operator I believe.
Multiplication would have been a good way to go on thinking about it but in the end I just when we declaring half sizes where convenient.
Let me know if you are still get any warnings in dart-scss now.

@anthonyjb It's almost it ! There are only 2 warnings left :
/external/styles/bourbon/functions/_strip-units.scss 4:12
and
/external/styles/bourbon/functions/_px-to-em.scss 12:12

Ahh that's an external library not part of ct - I'll see if I can remove ideally or upgrade if not.

The version you use is the 4.1.0, and it looks like warning issue is fixed by 7.1.0
https://github.com/thoughtbot/bourbon/releases/tag/v7.1.0

@Ei-aaie if you look at the latest release of bourbon they don't support Dart SASS yet and had to revert the fix for div (https://github.com/thoughtbot/bourbon/issues/1106#issuecomment-1048061338). So for now I'm just going to patch the version I include in CT if I can as they form part of the repo within the external directory. If that doesn't work I'll remove the dependency (but looking at it that's a bigger job so would be another weekend away).

@Ei-aaie ok can we try again, new release 1.6.16 - let me know.

Working like a charm ! Thank you !

@Ei-aaie great thank you for the help with issue.


Scaleway

Hi : ) I was wondering if Lithops is compatible with Scaleway?
Thanks

I see that they offer their own managed kubernetes and they also have an exposed S3 api, so you could try with their managed kubernetes and using the ceph storage backend to communicate with their s3 api, that should work.
I have done something similar with ovh.


base64ToUtf8(): reject invalid characters

RFC4648 states:

Implementations MUST reject the encoded data if it contains characters outside the base alphabet when interpreting base-encoded data, unless the specification referring to this document explicitly states otherwise.

See: https://datatracker.ietf.org/doc/html/rfc4648#section-3.2

Encountered while working on https://github.com/getodk/central/issues/764/


[WIP]: upgrade dapr's version to fix go mod tidy issue in tests/certification #1319

Description
upgrade dapr's version in certification test to fix go mod tidy issue.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1319
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

The CI failure is expected, has to wait for dapr/dapr#3883's merge and bum up Dapr's version in order to make tests pass completely.

@beiwei30 https://github.com/dapr/dapr/pull/3883 is now merged.

https://github.com/dapr/components-contrib/pull/1349 will address the same issue.


Cannot read property 'isARSupportedOnDevice' of null on RN version 0.72.6 in android device

Description
I tried to run this library on RN version 0.72.6 and faced this issue in android devices but working on ios device. can you please help with the solution?

also i downgraded the react native version to 0.71.3 but still getting this type of error.. please guide me how can I solve this issue

@ankittrootech123 sorry bother you, but did you resolve this issue?

No, not yet.
Thanks & regards,
Ankit Limbad
React-Native Developer
TRooTech Business Solutions Pvt. Ltd.
https://www.trootech.com https://www.trootech.com/
On Thu, 18 Jan 2024 at 04:49, hadnet @.***> wrote:

@ankittrootech123 https://github.com/ankittrootech123 sorry bother you,
but did you resolve this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/NativeVision/viro/issues/233#issuecomment-1897396917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCU4JWT6X7BJAAB74TQZMRLYPBMAZAVCNFSM6AAAAAA6VRVOLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJXGM4TMOJRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

it seems okay on RN: 0.72.6 with viro: ^2.40.1
Make sure to import it correctly
import { isARSupportedOnDevice } from "@viro-community/react-viro";
As per Viro docs (IsARSupportedOnDevice) the example code snippet should be updated.


[CHALLENGE SUBMISSION]  3

Please provide the following information for your submission:
What is your name? (First, Last)
Mats, Estensen
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/matsest/AzureEats-Website
What Challenge is this for? (Challenge number between 1 and 5)
3
(Optional) Anything else we should know? e.g., language used, location, blog post?

Already did this as part of challenge 01 :) Haven't run terraform locally as part of these challenges yet, besides what's run with the pre-commit config
Created a storage account in its own resource group
Service principal created and given owner role on my subscription
Keys and secret added as secrets to Github repo
See backend configuration in providers.tf
See github action configuration here

Thanks for participating in the #TerraformOnAzure coding challenges!
We have attached your very special Astronaut Badger background, for you to use in all your upcoming video calls!
And to recognize you for your learning, you can collect your digital Azure Heroes badge on your mobile (you may need to install the Enjin wallet to collect it). Make sure to collect it today!
https://beam.enjin.io/claim/d32599695c8c28cfc24820492080fac0
Keep on learning and sharing!
Greetings from the #TerraformOnAzure team,
Suzanne, Floor, Erik & Nick


Additional DAX editor shortcut keys

These shortcut keys are missing from 'DAX editor' section of this page as of 20 Jan 2021:

Deleting a full line of code – (Ctrl + Shift + K)
Go to the Start and End line of code – (Ctrl + Home ) and (Ctrl + End)
Select line above or below – (Shift + Up arrow key) / (Shift + Down arrow key)

[Enter feedback here]

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

ID: 8df72ff6-206a-243a-a84a-0f6ac418a9b2
Version Independent ID: e6d1ac0d-3785-cb59-da9c-7dcac2876d2d
Content: Keyboard shortcuts in Power BI Desktop - Power BI
Content Source: powerbi-docs/create-reports/desktop-accessibility-keyboard-shortcuts.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @xo9agq46a8k -- thanks! Assigning to the author.

Hi, @xo9agq46a8k -- thanks! Assigning to the author.

Hi @xo9agq46a8k ,
Thank you for your suggestion.  We've made the update and set the article to republish in the next few days.
All the best,
The DOCS team


Added shape and color settings

Added possibility to change shape type and wave color.

I'm not sure when this would get merged, but is there any way I can set the color for now? Sorry for being impatient.

@Alok-Sharma you can checkout @matteosist repo to get their code changes

@SleeplessByte thanks!

@matteosist I've merged your code. Thanks!


Issue 943

Replace MAINTAINER with LABEL in Dockerfile per issue #943.

Codecov Report

Merging #961 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #961      +/-   ##
==========================================
- Coverage   78.49%   78.39%   -0.11%
==========================================
  Files          52       52
  Lines        5572     5572
==========================================
- Hits         4374     4368       -6
- Misses       1198     1204       +6

Impacted Files
Coverage Δ

vent/api/plugins.py
79.09% <0%> (-1.09%)
:arrow_down:

Continue to review full report at Codecov.

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

@twospeeder please add yourself to the AUTHORS file.

addressed by #963  already


[253] Add a "right to work" step to the profile personal details section

Screenshots of UI changes:

 Add "Do you have the right to work in the uk" section to the candidate profile


could not determine kind of name for C.crypto_kx_client_session_keys

My Dockerfile:
FROM s390x/golang
WORKDIR /go/src/app
ADD BusinessLogicServices .
RUN apt-get update
RUN apt-get install -y libsodium-dev
RUN go get -d -v ./...
RUN go run -v ./...
inside my BusineesLogicService there is file crypto.go which imports a golang libsodium library
package main
import (
    "github.com/jamesruan/sodium"
)
and inside "github.com/jamesruan/sodium" there is file exchange.go which contains following
package sodium
// #cgo pkg-config: libsodium
// #include <stdlib.h>
// #include <sodium.h>
import "C"
var (
  cryptoKXPublicKeyBytes  = int(C.crypto_kx_publickeybytes())
  cryptoKXSecretKeyBytes  = int(C.crypto_kx_secretkeybytes())
  cryptoKXSeedBytes       = int(C.crypto_kx_seedbytes())
  cryptoKXSessionKeyBytes = int(C.crypto_kx_sessionkeybytes())
)
output of docker build -t abc .
Step 7/7 : RUN go run -v ./...
---> Running in a805355aaa6a
../github.com/jamesruan/sodium/exchange.go:9:32: could not determine kind of name for C.crypto_kx_publickeybytes
HOST OS = Ubuntu 18.04
docker version:
Client:
Version:	17.12.1-ce
API version:	1.35
Go version:	go1.10.1
Git commit:	7390fc6
Built:	Wed Apr 18 01:22:24 2018
OS/Arch:	linux/s390x
Server:
Engine:
Version:	18.06.1-ce
API version:	1.38 (minimum version 1.12)
Go version:	go1.10.4
Git commit:	e68fc7a
Built:	Mon Oct  1 14:06:53 2018
OS/Arch:	linux/s390x
Experimental:	false
If i run it without docker containers then it works.

The s390x architecture is not tested. Your problem could caused by a legacy version of libsodium. Make sure your libsodium version is at least 1.0.12


move content checksums tsv up one level

the content checksums tsv file should be saved in the genome directory rather than in the fasta asset directory.
We treat it as a byproduct of generating the main genome hash

related to: https://github.com/databio/refgenie/issues/89


Add Corsica default configuration

Since we mostly create API for SPA, we add the default configuration needed for CORS with Corsica.
Closes #10

🦀 Requesting reviewers for this pull request:

@remiprev (contributor with 58 commits in the last 90 days and 104 commits overall)
@olivierbuffon (expert reviewer for the elixir stack)

@garno Just need to rebase when you get the chance and I think it’s ready to merge!


File to memory mapping, why only set data is stored in a DB file?

Ask the EOS data is a file and then config. ini configuration inside the chain size needs to be constantly tuned up. Why not divide into multiple DB stores? What advantages do individual files have?

This leads to the problem that each time a thread suddenly breaks, so many blocks need to be indexed from the new index, so I would like to ask if there is any solution?

There's no advantage of dividing a single block log file into multiple files. Let's do a comparison:
Suppose you have 4 hard disks, each of them has a capacity of 10TB and has a write throughput of 200MB/s.
Option 1):
4 hard disks are mounted to C:, D:, E:, F:, respectively. Then an application which would first fill into C:\log0.txt, then D:\log1.txt, then E:\log2.txt, then F:\log3.txt. You'll notice that at anytime only one of the hard disk is busy, resulting in a write throughput of 200MB/s.
Option 2):
4 hard disks are configured as RAID 0 with a total capacity of 40TB. The application only write into a single file. But in this case, since all the harddisks are operating together, it results in a throughput of 800MB/s.
Option 3):
4 hard disks are configured as RAID 5 with a total capacity of 30TB. The application only write into a single file. But in this case, since all the harddisks are operating together, it results in a throughput of 600MB/s but allowing any one of those to become malfunction.
obviously Option 2 & 3 are better than option 1.


0.2.0 Release should get yanked

We discovered an issue with #8 that introduced the PUSH_CONSTANT enum value to DescriptorType https://github.com/Traverse-Research/rspirv-reflect/blob/85b2db9d04523dd2ee423641d6b04a81d29c4d27/src/lib.rs#L72
However, this DescriptorType doesn't exist in Vulkan (https://www.khronos.org/registry/vulkan/specs/1.2-extensions/man/html/VkDescriptorType.html) which leads to invalid values down stream as this type is meant to be casted/transmuted into the raw Vulkan enum.
A new release will be published shortly that addresses this - and the main fixes were part of #9.

0.2.0 got yanked and replaced with 0.3.0 (because public api change).


Some assets still digested with asset_sync gem - howto

Actually gem working well, created non digested assets in the path.
Problem comes when asset_sync  gem reading rails 4 manifest file which is not include "non-digested" stuff.
solved easily with setting manifest to false in the AssetSync configuration (so it read files after precompile), eg:
AssetSync.configure do |config|
  config.manifest = false
end

:+1


failed to perform copy due to error: cannot start job due to error: cannot scan the path

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.7
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy copy "Z:\xxxxx" "https://xxxxx.blob.core.windows.net/xxxxx" --recursive=true
What problem was encountered?
See above error in title. I can easily get to the share from CMD prompt.
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
Nope

Hi @jradin2001 thanks for reaching out!
Could you please double-check that the quoted path is escaped correctly?

Closed due to inactivity


Getting vulnerable issue<EMAIL_ADDRESS>

Please check below
Using<EMAIL_ADDRESS>--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>deduped
1 high severity vulnerabilities
how to resolve above issue?

Hi 👋 and sorry you are having an issue. Unfortunately I'm not sure off-hand what issue you are getting. Please go ahead and follow our process to report security issues outlined at https://github.com/expressjs/express/blob/master/Security.md and provide as much information as poasible so we can assess.


Improve performance for databases with many partitions

This is built on top of #52, so only look at the second commit in this PR.
Our database has many partitions, causing time rails db:structure:dump to take 5.9 seconds. The changes in this PR get it down to 1.8 seconds (compared to 1.3 seconds without this gem).
This implementation combines a comment and its associated statement into a single string, and builds an array of those comment+statement pairs. Then it drops any array elements that contain a reference to a partition table or index. Combining the comment and statement was a necessary performance optimization, to reduce the number of elements being iterated over and then removed.
I also tried some approaches using pg_query, but struggled to get the test suite to pass
# Build strings for statements until the next comment is reached
strings = []
start = nil
PgQuery.scan(dump).first.tokens.each do |token|
  if token.token == :SQL_COMMENT
    if start != nil
      strings << dump[start..token.start-1]
      start = nil
    end
    strings << dump[token.start..token.end]
  elsif start == nil
    start = token.start
  end
end
if start != nil
  strings << dump[start..]
end

# Combine comments with statements until a semicolon is reached
strings = []
start = nil
PgQuery.scan(dump).first.tokens.each do |token|
  if token.token == :ASCII_59
    strings << dump[start..token.end]
    start = nil
  else
    start ||= token.start
  end
end
if start != nil
  strings << dump[start..]
end

I think this looks like a good refactoring!
@andyatkinson Did you also want to take a look over these changes?
For context, I added Sean and Keiko from the pganalyze team as additional co-maintainers on the repo, since we had some changes that would be useful for pganalyze as well, and that way its more efficient without requiring me to look over everything :)

@lfittl Makes sense, sounds good.
@keiko713 Good point on unintentional side effects. I was reviewing your comment and the code, and wanted to clarify something.
The initial scan for partitioned tables looks for the pattern ?PARTITION OF in the dump. You'd mentioned this scenario:

and if I happen to have a separate, non-partitioned table named customers_cats

If customers_cats wasn't a partition of a table, then it shouldn't get matched on for the partitioned_tables list transformations. Do you agree? I didn't actually try this out, but that was my thought on skimming the code.
For indexes, it looks like they'd be ok too as long as they're not indexes on partitioned tables.
@seanlinsley Through some more ideas in on variations of a partitioned table and primary key structure, if you wanted to test. However, if you're testing this with your database with partitioned tables, then that seems sufficient.
Maybe in the future there could be some different "schema farm" variations of source DBs to dump and test with. 😁


Inconsistent indent

Inconsistent indent in this Python graphml2csv.py: tabs and blanks are mixed with each other. This causes failing to run the code.

I ran into this today as well. The code works fine with Python 2.7 but fails using Python 3.7 with the error below:
$ python3 graphml2csv.py -i air-routes-latest.graphml
  File "graphml2csv.py", line 47
    return '{' + ns + '}' + tag
                              ^
TabError: inconsistent use of tabs and spaces in indentation

It may be that Python 3 tightened up some code indentation rules perhaps?


Getting react-native-fbsdk LoginManager object when remote debugging disabled?

This is so strange. In one file I have:
import {
  GraphRequest,
  GraphRequestManager,
} from 'react-native-fbsdk'

import { FBLoginManager } from 'react-native-facebook-login'

console.log("FBLoginManager:", FBLoginManager)

(I need both because I make graph API requests, but also want the token-caching behavior of react-native-facebook-login)
When I run the app with remote debugging turned on, the app works fine and the log shows me what I expect to see:
FBLoginManager: Object { Events: Object, LoginBehaviors: Object}
    Events: Object
    LoginBehaviors: Object
    getCredentials: fn()
    login: fn()
    loginWithPermissions: fn()
    logout: fn()
    setLoginBehavior: fn()
    __proto__: Object

But if I turn off remote debugging, it shows me the methods of the LoginManager from react-native-fbsdk:
'FBLoginManager:', { setLoginBehavior: { [Function: fn] type: 'remote' },
  setDefaultAudience: { [Function: fn] type: 'remote' },
  getLoginBehavior: { [Function: fn] type: 'remoteAsync' },
  getDefaultAudience: { [Function: fn] type: 'remoteAsync' },
  logInWithReadPermissions: { [Function: fn] type: 'remoteAsync' },
  logInWithPublishPermissions: { [Function: fn] type: 'remoteAsync' },
  logOut: { [Function: fn] type: 'remote' } }

And of course production builds are broken, having the same wrong login manager object as dev builds without remote debugging.
I have no idea why it would be exposing the wrong object, or especially why it would be gated on remote debugging, but I can reliably reproduce these behaviors.
EDIT: this is on iOS

interesting.. might be because the name is the same and it is clashing

almost. in react-native-fbsdk the object is named LoginManager.

oh ill have to look at the code to isolate it, do you ever use their LoginManager in your code?

No, I recently removed it in favor of the FBLoginManager from this package.

are you using android/ios/both?

but its as I suspected for android
https://github.com/facebook/react-native-fbsdk/blob/master/android/src/main/java/com/facebook/reactnative/androidsdk/FBLoginManagerModule.java#L85
The module name in the background is the same

in my case I'm on iOS.

usually its a unified name but just to make it a little clearer.
heres another snippet from their code.
They change the variable ref for the end user but the component is named "FBLoginManager" in the background for IOS as well
https://github.com/facebook/react-native-fbsdk/blob/eea514d8edc8e66f087891aeeb7dde2d5fb0b68d/js/FBLoginManager.js#L24
So some wires being crossed here.

so next steps from here since this is a third party module is to namespace our FBLoginManager to possibly MFBLoginManager
if you want to get ahead of me or @magus,  you can submit a working PR if possible 👍  :)


Garden Valve

Dear ZHA developers:
I bought this device on Ali Express:
https://www.aliexpress.com/item/3256804035783288.html?spm=a2g0o.order_list.0.0.7c961802trDImq
This is the technical information found on zigbee2mqtt website
https://www.zigbee2mqtt.io/devices/QT06.html
ZHA is able to find the device in the Home assistant UI. However, the only entities found are:
Lqi
Rssi
Entities expected such as battery; on/off are not found. I know some python but not sure I can figure out how to develop a bridge.
Just wondering if you have any suggestions to at least find the on/off so I can turn on and off remotely through Home assistant Hub.
Here is the device Signature:
{
"node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4417, maximum_buffer_size=66, maximum_incoming_transfer_size=66, server_mask=10752, maximum_outgoing_transfer_size=66, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)",
"endpoints": {
"1": {
"profile_id": 260,
"device_type": "0x0051",
"in_clusters": [
"0x0000",
"0x0004",
"0x0005",
"0xef00"
],
"out_clusters": [
"0x000a",
"0x0019"
]
}
},
"manufacturer": "_TZE200_sh1btabb",
"model": "TS0601",
"class": "zigpy.device.Device"
}

Duplicate of #1622

The issue was not resolved. I put the solution in #1622

If i was able to get more entities like start time.  Any tips on how to. Make home assistant add, extra attributes.  Its showing in device diagnostic but not on UI

This issue can be closed. Support has been added in ZHA and should be available in HA in next release

Duplicate


Correct way to send a handshake or connect

I'm trying to simulate a charge point and connect to the server. I think the server is working fine but the problem is how i connect to it.
I'm using this to connect
const newSocket = new WebSocket(${ocppJEndpointUrl}/${encodeURIComponent(chargePointIdentity)},['ocpp1.6', 'ocpp1.5']);
so it should look like this
ws://localhost:3000/ABCD with protocols [ocpp1.6,ocpp1.5]
this is the error i get

const err = Error(reason);
^
Error [AbortError]: Client disconnected
at Object.pendingCall.abort (/home/tarek/Desktop/ocpp_csms/server/node_modules/ocpp-rpc/lib/client.js:338:29)
at /home/tarek/Desktop/ocpp_csms/server/node_modules/ocpp-rpc/lib/client.js:440:63
at Array.forEach ()
at RPCServerClient._rejectPendingCalls (/home/tarek/Desktop/ocpp_csms/server/node_modules/ocpp-rpc/lib/client.js:440:48)
at RPCServerClient._handleDisconnect (/home/tarek/Desktop/ocpp_csms/server/node_modules/ocpp-rpc/lib/client.js:458:14)
at WebSocket. (/home/tarek/Desktop/ocpp_csms/server/node_modules/ocpp-rpc/lib/client.js:410:49)
at Object.onceWrapper (node:events:629:26)
at WebSocket.emit (node:events:526:35)
at WebSocket.emit (node:domain:489:12)
at WebSocket.emitClose (/home/tarek/Desktop/ocpp_csms/server/node_modules/ws/lib/websocket.js:258:10)

Hi,
Please can you clarify a few things for me?

Are you using this module to create a central system, a charging station simulator, or both?
Is the error being reported by your client (simulator) or by the server?

Thanks.

I will use it to create a central system but i was just testing things out first and familiarizing myself with this module.
I have a simple ui with a connect button that tries to establish a web-socket connection with the server i created using the example you gave in express
I get this in the browser https://imgur.com/a/bzsPmKk
And I get this in the server https://imgur.com/a/WWyW64S I added some console.log to client.js
if u need any more info please let me know

From your screenshots, it looks like the server is crashing, which is why your client's websocket is disconnecting.
Can we take a look at your server code to see what might be causing that to happen?


Unwind information performance improvements

This commit:

sort the FDEs rather than the whole unwind information, which is a smaller vector. This showed no performance changes in benchmarks;
optimise the unwind info in-place. Most of the performance gains in this commits comes from this changes;

Wrote a benchmark with criterion.rs and saw a ~25% decrease in time spent building and optimising the unwind info.
Running benches/unwind_info.rs (target/release/deps/unwind_info-b3269dc277690031)
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

Gnuplot not found, using plotters backend
unwind info + sorting   time:   [37.697 ms 39.061 ms 40.592 ms]
                        change: [-26.710% -23.315% -19.870%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  3 (3.00%) high mild
  5 (5.00%) high severe

For this test, this node.js binary was used
/home/javierhonduco/.vscode-server/cli/servers/Stable-e170252f762678dec6ca2cc69aba1570769a5d39/server/node: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=e034c66a6dc897492d6ecae6c6b046669f94ad96, with debug_info, not stripped, too many notes (256)

I am not commiting the benchmarks as I bumped into some rough edges with criterion and I am not sure if it's maintained anymore. Might be worth adding some benchmarks later on.
Test Plan
Compared the unwind info before and after this change for a bunch of executables.

Something to note is that this performance win might be greater in memory-constrained hosts or CPU with smaller caches!


Cleaner Server semantics

The Node paradigm doesn't make any sense under PHP - a Server isn't something that can stay up and process more than one request, or produce more than one response; and the implemented model doesn't support serving more than one request to begin with, since both request and response are injected at the time when Server is constructed. For one, a Server can't "listen" for anything.
Here's a greatly simplified Server implementation with semantics that correspond better with the PHP request/response life-cycle.
<?php

namespace xyz; // (ignore this)

use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Message\ResponseInterface;
use Zend\Diactoros\Response;
use RuntimeException;

/**
 * This class implements server semantics, e.g. processing a request and
 * producing a response, and dispatching a response to the client.
 */
class Server
{
    /**
     * @var callable
     */
    private $callback;

    /**
     * Response emitter to use; by default, uses Response\SapiEmitter.
     *
     * @var Response\EmitterInterface
     */
    private $emitter;

    /**
     * Constructor
     *
     * Given a callback, a request, and a response, we can create a server.
     *
     * @param callable $callback function (ServerRequestInterface $req, ResponseInterface $res) : ResponseInterface
     */
    public function __construct(callable $callback)
    {
        $this->callback = $callback;
    }

    /**
     * Retrieve the current response emitter.
     *
     * If none has been registered, lazy-loads a Response\SapiEmitter.
     *
     * @return Response\EmitterInterface
     */
    public function getEmitter()
    {
        if (!$this->emitter) {
            $this->emitter = new Response\SapiEmitter();
        }

        return $this->emitter;
    }

    /**
     * Set alternate response emitter to use.
     *
     * @param Response\EmitterInterface $emitter
     */
    public function setEmitter(Response\EmitterInterface $emitter)
    {
        $this->emitter = $emitter;
    }

    /**
     * Pipe a given server request through the middleware stack and produce a response.
     *
     * @param ServerRequestInterface $request
     *
     * @return ResponseInterface
     */
    public function process(ServerRequestInterface $request)
    {
        $response = new Response();

        $callback = $this->callback;

        ob_start();

        $response = $callback($request, $response);

        if (! $response instanceof ResponseInterface) {
            throw new RuntimeException("middleware returned invalid or no Response\n" . print_r($response, true));
        }

        return $response;
    }

    /**
     * Dispatch a response to the current connected HTTP client.
     *
     * @param ResponseInterface $response
     */
    public function dispatch(ResponseInterface $response)
    {
        $this->getEmitter()->emit($response);
    }
}

Processing a request and dispatching the response isn't a one-liner anymore, but it moves the dependency on Request and Response outside of Server where it belongs, enabling transaction semantics in Server, e.g. provide a Request and get a Response:
$server = new Server(new MiddlewarePipe());

$response = $server->process(ServerRequestFactory::fromGlobals());

$server->dispatch($response);

This is simpler, and should facilitate easier testing as well, since you can now process a request without actually dispatching it - this should help with integration testing on your middleware stack, assertions about expected responses etc. as the response doesn't need to be sent at all anymore. (To me, this was one of the expected benefits of the request/response abstraction, which we didn't get with this.)
I realize this is a BC break, but I think it's worth considering for version 2.0 - Node semantics seem rather inappropriate for PHP, and modeling Server-style transaction semantics is actually quite a bit simpler. (I doubt anyone will fret over the fact that your front controller script needs two lines of code instead of one.)
Note that we throw if the final result isn't ResponseInterface in process() - this was a decision about stricter request/response adherence in our environment, and it's probably stricter than what you'd want in your library.

a Server isn't something that can stay up and process more than one request

I beg to differ - long running processes in PHP are perfectly feasible and not necessarily a crazy idea.
Just because it isn't the canonical way of building PHP apps it should not be dismissed. I've been using long running processes (successfully) for a long time and with great advantages.
Also: has nothing to do with nodejs, it's just what app servers do (since ages). Node.js just made it hip to make the http server listener initialization stuff explicit.

long running processes in PHP are perfectly feasible

Okay, sure, but that's not what this library does.
The listen() method doesn't "listen", by any stretch - it doesn't wait for an incoming request, nor does it process multiple requests in a long-running process, it's nothing like a Node app, nor like React, etc... it simply processes the only request/response it can process, the ones given to the constructor.
With the current library, if you wanted to implement it under React or some other long-running process, you'd have to create a new Server instance for every incoming request.
My point was not at all coined on the feasibility of long-running processes in PHP, but the proposed switch to transaction semantics would actually make a lot more sense in that context, too.

I'm all about semantics, and I prefer these than the current set.

@designermonkey you can simply grab this code and use it now, in place of the standard Server class - we're using it on our project now.
It doesn't matter much (to me) if this is officially included or not - it's a simple class, almost too trivial to bother posting as a separate library. Either way, the important thing to me is middleware compatibility, which we have, so I'm happy either way :-)

:+1:

The main purpose of including Server in this library was to provide an easy-to-use, predictable way to start playing with PSR-7 messages:
$server = Server::createFromRequest(function ($request, $response) {
    $response->getBody()->write('Hello, world!');
    return $response;
}, ServerRequestFactory::createFromGlobals());
$server->listen();

It accomplishes that. The signature of the callable passed as the first argument to createFromRequest() and createServer() is standard middleware, and that also provides an entry point for dispatching PSR-7-based middleware applications. It does not mean, however, that this needs to be the only way to do so. Emitters were added in large part because of a recognition that folks wanted to emit a response without necessarily using the Server semantics.
My point is: I don't disagree that the semantics do not/will not work for everyone's needs. They do, however, fulfill the original goals.
I'd definitely accept a PR with an alternate approach; not one that replaces the current approach, but rather a parallel approach. If anybody wants to do so, feel free to open one and reference this issue; until then, I'm closing it.


configuration

Discussed in https://github.com/ZGIS/semantique/discussions/23

Originally posted by lisahligono April 18, 2023
I would like to ask if it's possible to get the full documentation of the opendatacube and geotiffarchive configuration

Close as duplicate of #10


when use , shows error :Content Security Policy directive: "frame-ancestors 'none'".

when I test the example in
https://dotnet.microsoft.com/platform/try-dotnet?tdsourcetag=s_pctim_aiomsg

it says:
Refused to display 'https://try.dot.net/?fromGist=df44833326fcc575e8169fccb9d41fc7' in a frame because an ancestor violates the following Content Security Policy directive: "frame-ancestors 'none'".
does it mean only private previous user can use it?
then how can I use it? thanks.

Greatly appreciate your reply. Is there any way to apply for being an approved host domains?

or, can u tell me about when to open to other users?

It caused by CSP. U can try to set it.

Could you please tell me why this issue was closed? It doesn't look like it is clear what to do in case of embedding via iframe and what is on road map for that. Greatly appreciate your reply!


Rename getters to be more inline with Golang idioms

Currently there are getters defined that are prefixed with Get e.g.

GetBooleanValue(flag string, defaultValue bool, evalCtx EvaluationContext, options EvaluationOptions) (bool, error)

This goes against Golang convention. Renaming this example to just BooleanValue would be better practice.

Hey @skyerus, can I work on it


allow naming the logging directory

This lets us name the logging directory, instead of forcing it to be named exactly the model we are running. So we can change hyperparams and keep the various versions of the model side-by-side. Also lets us compare loss histories and such in Tensorboard.

Yeah I had that implemented, I didn't notice I took it off. Thanks for adding this!


[11.x] Fixed an issue where migrations wouldn't be logged.

As reported here
https://github.com/laravel/framework/issues/52491

Not sure about this one - some may not want those queries included.

Not sure about this one - some may not want those queries included.

That's unfortunate, sometimes in my current company when we run migrations manually we have to create the log insert query manually so that or would have helped..


Song of Fire and Ice, button update

Changes / Comments
Adjusting buttons, added Journal.
Link to conversation: https://app.roll20.net/forum/post/7032912/a-song-of-ice-and-fire-sheet-need-help-with-roll-template
No attribute name changes

...now I have no idea what the hell just happened.  How did my Mekton stuff end up here?

If you click Commits at the top you can see you merge in the Mekton stuff on 13th December.

Apparently I am having trouble understanding the Github Desktop's interface.
Moving on.  "This branch has conflicts that must be resolved" popped up for me.  Is this a thing for me to fix (I think it is) or can only you fix it.  And, if it is mine to fix, how do I do that?

Greetings @CoalPoweredPuppet
You'll need to fix the merge requests. You could revert the commit for the Mekton Z. Alternatively you could close the request, save your changes as a back ups outside the repo folder, and reset the roll20 fork to the master copy.

I cannot identify what is causing the conflicts.  I will do as you suggested and close this request and try another on (after resetting this thingy-ma-bob)


Support for expand in the fields

Hey!
Many thanks for this awesome library. I think it should be bundled with PocketBase js sdk.
Is it possible to support typing "expand" in the fields function?
await db.collection('coll1').getFullList({
    fields: fields('id', 'name', 'created', 'expand'), // <-- this works but doesn't type.
    expand: expand({ nested: true })
});

Glad you like it!
How exactly does that work? Can you give an example with the database structure, query and result?

It's document in API preview for each collection.

For the example query above PB will return this:
{
    "created": "2023-09-26 19:44:25.101Z",
    "expand": {
        "nested": {
            "collectionId": "garn2zqvrx5etf1",
            "collectionName": "services",
            "created": "2023-09-26 19:44:15.498Z",
            "creator": "nqin9tg31ug9ex7",
            "id": "75or0smdif88hzl",
            "name": "SvcA",
            "updated": "2023-09-26 19:44:15.498Z"
        }
    },
    "id": "x2gyrks4zja6zcj",
    "name": "Dep1"
}

For
		fields: fields('id', 'name', 'created', 'expand.service.name' as any),
		expand: expand({ service: true })
	});

it will only return service name in the expanded object.
We can  also specify expand.nested.expand.field, which works as expected.

This is available in 0.1.0!