Added the command remark.

Week 6 tutorial - don't merge.

Codecov Report

Merging #23 (691b087) into master (761f097) will decrease coverage by 0.57%.
The diff coverage is 63.49%.

@@             Coverage Diff              @@
##             master      #23      +/-   ##
============================================
- Coverage     72.15%   71.58%   -0.58%
- Complexity      399      411      +12
============================================
  Files            70       73       +3
  Lines          1232     1288      +56
  Branches        125      132       +7
============================================
+ Hits            889      922      +33
- Misses          311      330      +19
- Partials         32       36       +4

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
12.00 <0.00> (ø)

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...c/main/java/seedu/address/model/person/Remark.java
62.50% <62.50%> (ø)
4.00 <4.00> (?)

...va/seedu/address/logic/commands/RemarkCommand.java
69.56% <69.56%> (ø)
5.00 <5.00> (?)

...c/main/java/seedu/address/model/person/Person.java
95.74% <75.00%> (-1.88%)
21.00 <1.00> (+1.00)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
97.05% <100.00%> (+0.04%)
12.00 <0.00> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)
5.00 <0.00> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
1.00 <0.00> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
25.00% <100.00%> (+5.00%)
2.00 <1.00> (+1.00)

... and 5 more

Continue to review full report at Codecov.

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


BTable Column width varies when paginating

What you are trying to do. A clear and concise description of what you want to happen.
I want to set BTable columns with qual width.
According to the documentation description for "" on Bootstrap Vue 2.23.0, the property "fixed" is intended for that:
Makes all columns equal width (fixed layout table). Will speed up rendering for large tables. Column widths can be set via CSS or colgroup
The same prop is also present on BTable. So, I presume it should serve the same principle on
Here is my current declaration:
<BTable           thead-tr-class="thead-th"           @row-clicked="selectRowItem"           class="my-4"           hover           bordered           striped           small           :responsive="true"           :sort-by="[{ key: 'model', order: 'asc' }]"           :items="dataTableItems"           :fields="sortTableFields"           :current-page="currentPage"           :per-page="rowsPerPage"           :fixed="true"         />
However, setting fixed as true does not force the columns width to equal size.
Without enforce width by css, is there any other option that can be used? Not sure if this case is a real problem or a side effect of a misconfigured component.
What you have already tried. A clear and concise description of any alternative solutions or features you have considered.
Binding true/false to :fixed
Any error messages you are getting. If any, please list the errors you are getting.
No error messages
Environment (please complete the following information):

OS: [e.g. Windows 11, Pop!_OS]
Mac OS

Package Version: [e.g. 0.8.12]
"bootstrap-vue-next": "^0.23.2",

Npm/Yarn/Pnpm/Bun Version: [e.g<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@VividLemon This is still not working.


add topics

I suggest adding the topics benchmark, weather, climate, numerical-weather-prediction, nwp in the About section.

Thanks for the suggestion.


Update Example in README

What actually works

There doesn't seem to be a nice way to get error messages, or at least it's not obvious to me. I'd assume perhaps I need to walk the AST and find nodes marked with errors or something?

rootNode, comments, positions := php7.Parse(bytes.NewBufferString(`<? echo "Hello world";`), "example.php")
//How do we get a list of errors easily?
//How do we get the position/column?
//for _, e := range parser.GetErrors() {
//	fmt.Println(e)
//}
visitor := visitor.Dumper{
	Writer:    os.Stdout,
	Indent:    "",
	Comments:  comments,
	Positions: positions,
}
rootNode.Walk(visitor)

The non-working example given
src := bytes.NewBufferString(`<? echo "Hello world";`)

parser := php7.NewParser(src, "example.php")
parser.Parse()

for _, e := range parser.GetErrors() {
	fmt.Println(e)
}

visitor := visitor.Dumper{
	Writer:    os.Stdout,
	Indent:    "",
	Comments:  parser.GetComments(),
	Positions: parser.GetPositions(),
}

rootNode := parser.GetRootNode()
rootNode.Walk(visitor)

I limited by goyacc, and I can only save unexpected tokens as errors. When this happens the parser skips the token and try to recover parsing process, so the error is not a part of AST.
AST contains only nodes that can be correctly parsed.

The non-working example given

Are you at the master branch?
Run go get -u github.com/z7zmey/php-parser


Allow users to order by value column

Basically a one line commit to allow users to sort results in the dashboard using the value column:

Thanks @luzfcb for the solution. Thanks @vncscoelho for the bug report.

in a future, maybe replace all money fields (DecimalField) to MoneyField ( http://django-money.readthedocs.io/en/stable/)
This would facilitate calculations if there are expenditures in different currencies

Changes Unknown when pulling 63ad0e710bb9421c5ed67c5664ba5fc6a7801829 on cuducos-allow-sort-by-value into ** on master**.

This would facilitate calculations if there are expenditures in different currencies.

Jarbas doesn't calculate anything so far. It's basically an API to show what the Chamber of Deputies shows — but with enhanced features to facilitate auditioning. So we basically calculate nothing, just show stuff to users. By the way we even replicate some arguably inconsistente data such as year with a different year than issue_date, for example.
Also our database is only in Brazilian Reais — even expenditures made abroad are converted by congresspeople before claiming for the reimbursement. We have rolled out a feature to help users check if the congressperson's converted currency properly in #75.
Therefore I see no use ATM for django-money. It seams so powerful to only help in… sorting fields. But is surely is something to keep in mind ; )

LGTM ;)


(release-1.10): Update go.mod dependencies [SECURITY]

Description of your changes
This PR updates go.mod dependencies to fix the following:

Name
Change
Type
Vulnerability
Severity

golang.org/x/crypto
v0.23.0 -> v0.31.0
go-module
GHSA-v778-237x-gjrc
High

I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] ~Added backport release-x.y labels to auto-backport this PR if necessary.~

How has this code been tested
https://github.com/crossplane-contrib/provider-upjet-azure/actions/runs/12358124366

/test-examples="examples/network/v1beta2/subnet.yaml"

Ignored the required checks, as they are not available in this branch.


"Something Broke"

Hello, I've had this issue in the past and fixed it by modifying my application.yml file.
Issue seems to be back again, no changes made really, it was working fine just a couple days ago. (since July)
lavalink.jar up to date (25th august release)
Red up to date (3.5.13)
It randomly returns the 403 error on some songs, but not all.
Any ideas on what could be causing this ?
Here's my Application.yml:
plugins:
  youtube:
    enabled: true
    allowSearch: true
    allowDirectVideoIds: true
    allowDirectPlaylistIds: true
    clients:
      - MUSIC
      - WEB
      - ANDROID_TESTSUITE
      - TVHTML5EMBEDDED
      - ANDROID_LITE
      - MEDIA_CONNECT
      - IOS
    WEB:
      playback: true
    TVHTML5EMBEDDED:
      playlistLoading: false
      videoLoading: false
      searching: false
lavalink:
  plugins:
  - dependency: dev.lavalink.youtube:youtube-plugin:1.8.3
    repository: https://maven.lavalink.dev/releases
  server:
    bufferDurationMs: 400
    frameBufferDurationMs: 1000
    gc-warnings: true
    password: penis
    playerUpdateInterval: 1
    soundcloudSearchEnabled: true
    sources:
      bandcamp: true
      http: true
      local: true
      soundcloud: true
      twitch: false
      vimeo: true
      youtube: false
    youtubePlaylistLoadLimit: 100
    youtubeSearchEnabled: true
logging:
  file:
    path: ./logs/
  level:
    lavalink: INFO
    root: INFO
  logback:
    rollingpolicy:
      max-history: 15
      max-size: 10MB
metrics:
  prometheus:
    enabled: false
    endpoint: /metrics
sentry:
  dsn: ''
  environment: ''
server:
  address: localhost
  port: 2333

And Here's is my servers log:
2024-11-14 22:20:54.131  INFO 8376 --- [  XNIO-1 task-1] lavalink.server.io.SocketServer          : Connection successfully established from Red-Lavalink/0.11.0
2024-11-14 22:20:54.153  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "configureResuming", "key": "Red-Core-Audio-994072056549363732-gamerz2", "timeout": 60}
2024-11-14 22:20:54.154  WARN 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.WebSocketHandler      : Sending websocket commands to Lavalink has been deprecated and will be removed in API version 4. API version 3 will be removed in Lavalink 5. Please use the new REST endpoints instead.
2024-11-14 22:20:54.156  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "configureResuming", "key": "Red-Core-Audio-887761039226187797-Gamerz", "timeout": 60}
2024-11-14 22:20:54.156  WARN 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.WebSocketHandler      : Sending websocket commands to Lavalink has been deprecated and will be removed in API version 4. API version 3 will be removed in Lavalink 5. Please use the new REST endpoints instead.
2024-11-14 22:21:07.375  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "voiceUpdate", "guildId": "247928585720037386", "sessionId": "29f9b815e3669ed12b10ad2cd8e1903f", "event": {"token": "4caeb494fc8445ed", "guild_id": "247928585720037386", "endpoint": "newark1297.discord.media:443"}}
2024-11-14 22:21:07.944  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "play", "guildId": "247928585720037386", "track": "QAAAmQIAO1N0cmF5IEZyb20gVGhlIFBhdGggLSBOZWVkZnVsIFRoaW5ncyBbT2ZmaWNpYWwgTXVzaWMgVmlkZW9dAARVTkZEAAAAAAADtTgAC0EzVGZmUlBGMDZVAAEAK2h0dHBzOi8vd3d3LnlvdXR1YmUuY29tL3dhdGNoP3Y9QTNUZmZSUEYwNlUAB3lvdXR1YmUAAAAAAAAAAA==", "noReplace": false, "startTime": "0", "pause": false}
2024-11-14 22:21:09.592  INFO 8376 --- [back-1-thread-1] d.l.y.http.YoutubeAccessTokenTracker     : Updating YouTube visitor id succeeded, new one is CgtrX01jWEJjcHV4Zyil_Nq5BjIKCgJDQRIEGgAgODoMCAEg-p76mtfEr5tn, next update will be after 600 seconds.
2024-11-14 22:21:10.575  INFO 8376 --- [back-1-thread-1] c.s.l.c.natives.NativeLibraryLoader      : Native library libmpg123-0: loading with filter com.sedmelluq.discord.lavaplayer.natives.ConnectorNativeLibLoader$$Lambda$873/0x000001683d7eccc0@e624fb1
2024-11-14 22:21:10.596  INFO 8376 --- [back-1-thread-1] c.s.l.c.natives.NativeLibraryLoader      : Native library libmpg123-0: successfully loaded.
2024-11-14 22:21:10.598  INFO 8376 --- [back-1-thread-1] c.s.l.c.natives.NativeLibraryLoader      : Native library connector: loading with filter null
2024-11-14 22:21:10.666  INFO 8376 --- [back-1-thread-1] c.s.l.c.natives.NativeLibraryLoader      : Native library connector: successfully loaded.
2024-11-14 22:21:29.497  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "play", "guildId": "247928585720037386", "track": "QAAAiAIAKlN0cmF5IEZyb20gVGhlIFBhdGggLSBNYXkgWW91IExpdmUgRm9yZXZlcgAEVU5GRAAAAAAAAv2gAAtodEMzR3NQalJWWQABACtodHRwczovL3d3dy55b3V0dWJlLmNvbS93YXRjaD92PWh0QzNHc1BqUlZZAAd5b3V0dWJlAAAAAAAAAAA=", "noReplace": false, "startTime": "0", "pause": false}
2024-11-14 22:21:30.104 ERROR 8376 --- [back-1-thread-1] c.s.d.l.t.p.LocalAudioTrackExecutor      : Error in playback of htC3GsPjRVY

com.sedmelluq.discord.lavaplayer.tools.FriendlyException: Something broke when playing the track.
        at com.sedmelluq.discord.lavaplayer.tools.ExceptionTools.wrapUnfriendlyExceptions(ExceptionTools.java:45) ~[lavaplayer-1.5.4.jar!/:na]
        at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:113) ~[lavaplayer-1.5.4.jar!/:na]
        at com.sedmelluq.discord.lavaplayer.player.DefaultAudioPlayerManager.lambda$executeTrack$1(DefaultAudioPlayerManager.java:324) ~[lavaplayer-1.5.4.jar!/:na]
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) ~[na:na]
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) ~[na:na]
        at java.base/java.lang.Thread.run(Thread.java:1623) ~[na:na]
        Suppressed: com.sedmelluq.discord.lavaplayer.tools.exception.EnvironmentInformation:
  lavaplayer.version: 1.5.4
  os.arch: amd64
  os.name: Windows 10
  os.version: 10.0
  java.vendor: Oracle Corporation
  java.version: 20.0.2
  java.runtime.version: 20.0.2+9-78
  java.vm.version: 20.0.2+9-78
Caused by: java.io.IOException: Invalid status code for player api response: 403
        at com.sedmelluq.discord.lavaplayer.tools.io.HttpClientTools.assertSuccessWithContent(HttpClientTools.java:154) ~[lavaplayer-1.5.4.jar!/:na]
        at dev.lavalink.youtube.clients.skeleton.NonMusicClient.loadJsonResponse(NonMusicClient.java:67) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at dev.lavalink.youtube.clients.skeleton.NonMusicClient.loadTrackInfoFromInnertube(NonMusicClient.java:107) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at dev.lavalink.youtube.clients.skeleton.StreamingNonMusicClient.loadFormats(StreamingNonMusicClient.java:35) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at dev.lavalink.youtube.track.YoutubeAudioTrack.loadBestFormatWithUrl(YoutubeAudioTrack.java:193) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at dev.lavalink.youtube.track.YoutubeAudioTrack.processWithClient(YoutubeAudioTrack.java:119) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at dev.lavalink.youtube.track.YoutubeAudioTrack.process(YoutubeAudioTrack.java:72) ~[jar:file:C:\Lavalink\.\plugins\youtube-plugin-1.8.3.jar!/:na]
        at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:101) ~[lavaplayer-1.5.4.jar!/:na]
        ... 4 common frames omitted

2024-11-14 22:21:30.346  INFO 8376 --- [   XNIO-1 I/O-3] lavalink.server.io.SocketServer          : {"op": "play", "guildId": "247928585720037386", "track": "QAAAjgIAMFN0cmF5IEZyb20gVGhlIFBhdGggLSBJSUkgW09mZmljaWFsIE11c2ljIFZpZGVvXQAEVU5GRAAAAAAAA9CQAAsycmxJYXYyUlNEMAABACtodHRwczovL3d3dy55b3V0dWJlLmNvbS93YXRjaD92PTJybElhdjJSU0QwAAd5b3V0dWJlAAAAAAAAAAA=", "noReplace": false, "startTime": "0", "pause": false}
2024-11-14 22:21:31.051 ERROR 8376 --- [back-1-thread-1] c.s.d.l.t.p.LocalAudioTrackExecutor      : Error in playback of 2rlIav2RSD0

I have the exact same problem, with same lavalink 3.7.13 and youtube-plugin 1.8.3 version or lower...
Also my machine only has ipv4 activated. so no ipv6 rotating. Maybe this is causing the error also.

Since you are both commenting on the Lavalink.jar repository and not in a Red-DiscordBot issue, I'm going to answer this as if you are managing your own Lavalink node. You need to update the yt source plugin to 1.8.3 and take a look at the sources you are using in your application.yml. Currently I'm testing with:
    clients:
      - MUSIC
      - WEB
      - WEBEMBEDDED
      - ANDROID_TESTSUITE
      - TVHTML5EMBEDDED
      - MEDIA_CONNECT
      - IOS

If this message isn't informative for you or you don't understand what to do with this information and you are a Red-DiscordBot user, you should wait until there is a new Red version which will include the changes I'm describing here.

Thanks for the reply to both of you.
Yes I am running a lavalink node on which two discting Redbots connect to.
I was using the following sources:
 clients:
      - MUSIC
      - WEB
      - ANDROID_TESTSUITE
      - TVHTML5EMBEDDED
      - ANDROID_LITE
      - MEDIA_CONNECT
      - IOS

I've upated my application.yml with the same sources as you've indicated to test, and once again you've found the issue @aikaterna.
I am guessing either  ANDROID_LITE or MEDIA_CONNECT was causing the issue?
Or simply adding WEBEMBEDDED fixed it?
I can't really confirm.
I've teste 2-3 albums that were returning 403 on some songs and everything seems to be working fine now.
Thanks once again!


Online doc with i18n support

Is your proposal related to a problem?

N/A
(Write your answer here.)
Describe the solution you'd like
On line doc with i18n support, code as guide, code as doc

(Describe your proposed solution here.)
Describe alternatives you've considered

(Write your answer here.)
Additional context

(Write your answer here.)

https://studygolang.com/articles/26694?fr=sidebar

https://pkg.go.dev/badge/


Possible Bug - get_db_session reference included in {{cookiecutter.project_name}}web.gql.context.pywhen using Tortoise ORM

Awesome project! Really love what you've done here!
I ran into what I think is a bug with the project generation logic. I think it's a simple fix / issue, so I'd be happy to contribute a PR, but wanted to run this by you first.
In the context.py file of the gql package, there's an import for from {{cookiecutter.project_name}}.db.dependencies import get_db_session:
https://github.com/s3rius/FastAPI-template/blob/156883798ab4ec54d97080c77a34d366b8484f95/fastapi_template/template/{{cookiecutter.project_name}}/{{cookiecutter.project_name}}/web/gql/context.py#L16-L18
This import does not appear to be used when using Tortoise ORM, however:
https://github.com/s3rius/FastAPI-template/blob/156883798ab4ec54d97080c77a34d366b8484f95/fastapi_template/template/{{cookiecutter.project_name}}/{{cookiecutter.project_name}}/web/gql/context.py#L38-L42
In fact, with Tortoise ORM, there doesn't appear to be a dependencies.py file at all, see here.
Leaving the import statement throws an error with pytest:
=========================================================================================================== ERRORS ============================================================================================================
________________________________________________________________________________________________ ERROR collecting test session ________________________________________________________________________________________________
/usr/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1014: in _gcd_import
    ???
<frozen importlib._bootstrap>:991: in _find_and_load
    ???
<frozen importlib._bootstrap>:975: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:671: in _load_unlocked
    ???
../../.cache/pypoetry/virtualenvs/gremlinengine--i0qMi2L-py3.8/lib/python3.8/site-packages/_pytest/assertion/rewrite.py:168: in exec_module
    exec(co, module.__dict__)
GremlinEngine/conftest.py:19: in <module>
    from GremlinEngine.web.application import get_app
GremlinEngine/web/application.py:24: in <module>
    from GremlinEngine.web.graphql.router import gql_router
GremlinEngine/web/graphql/router.py:4: in <module>
    from GremlinEngine.web.graphql import dummy, echo, redis
GremlinEngine/web/graphql/dummy/__init__.py:3: in <module>
    from GremlinEngine.web.graphql.dummy.mutation import Mutation
GremlinEngine/web/graphql/dummy/mutation.py:5: in <module>
    from GremlinEngine.web.graphql.context import Context
GremlinEngine/web/graphql/context.py:5: in <module>
    from GremlinEngine.db.dependencies import get_db_session
E   ModuleNotFoundError: No module named 'GremlinEngine.db.dependencies'

Removing the import appears to cause no other issues and resolves the error. Seems like the if/else jinja statement in the gql.context.py template to import dependencies and get_db_session should be updated to also take into account which database is being used? This is my first in-depth foray into standalone Python ORMs (I've been a Django guy, mostly), so sorry if I'm missing something here.

Thank you for using this project! Yeah, that's truly an error. I fixed it in pools issue. Will release soon.
I'll close the issue when upcoming release is merged into master branch.

Awesome! Thanks for the quick response and proposed resolution!

This issue is resolved in 3.3.8. Can you please check? If it's ok, close the issue.

Closing due to inactivity.

Closing due to inactivity.

Sorry just saw this. Thanks for the fix!


[WIP] Containerized image publish

Issue Number: (429)
Summary
Containerize WebApi
Changes

Add Dockerfile to create WebApi image
Add instructions on howto build and deploy

Checklist

[ ] Local Tests Passing?
[ ] Documentation Updates Made?

Are there API Changes?
N/A
Is this a breaking change?
N/A
Anything else?
N/A

Codecov Report

Merging #128 (b1b09a2) into dev (a03b1ad) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #128      +/-   ##
==========================================
- Coverage   85.38%   85.32%   -0.06%
==========================================
  Files          52       48       -4
  Lines        1690     1588     -102
  Branches      176      170       -6
==========================================
- Hits         1443     1355      -88
+ Misses        189      176      -13
+ Partials       58       57       -1

Impacted Files
Coverage Δ

...s/src/Configuration/ServiceCollectionExtensions.cs
100.00% <ø> (ø)

src/CarbonAware.CLI/src/CarbonAwareCLI.cs
62.50% <0.00%> (ø)


Inconsistent behaviour with table valued function and result set limit

System information:

Database: Microsoft SQL Server 2019 (RTM-CU16) (KB5011644) - 15.0.4223.1 (X64)
Apr 11 2022 16:24:07
Copyright (C) 2019 Microsoft Corporation
Express Edition (64-bit) on Linux (Ubuntu 20.04.4 LTS)
DBeaver Version <IP_ADDRESS>205220912

Connection specification:

Driver Microsoft JDBC Driver for SQL Server connected over LAN
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

A table-valued function is restricted by the ResultSet fetchSize:
select 'F' as "Type",Count(*) as "Families",sum(NumMembers) as "Rows" from dbo.fn_Families(0);
returns
Type Families   Rows
F	1280	2501
1280 is the ResultSetFetchSize set in Global Settings. Both numbers are incorrect.
The correct result can be  'forced' - using a union produces the correct result:
select 'Dummy',1,1
union
select 'Function' as "Type",Count(*) as "Families",Sum(NumMembers) as "Rows" from dbo.fn_Families(0);
returns
Type        Families   Rows
Dummy	1             1
Function	6054       12000
These are the correct numbers. It can be seen that in the union query the output of the table-valued function fn_Families is not limited by the ResultSetFetchSize and when the function is called on its own, the result set is incorrect.
Microsoft SSMS gives the correct result for the function call; This puts the the use of table-valued functions in DBeaver into doubt as the results could be unreliable.

@PaulSlater25 I was not able to reproduce this issue. Can you tell me if something in my test is different and could have influenced the result
my function:
CREATE FUNCTION [dbo].table_val
(  @parameter_name AS int

)
RETURNS TABLE
    AS
    RETURN (  select * from dbo.emails )
;

The problem must be in the way that the function is declared:
-- Declaration as required (multiple statements needed inside function – see original method below)
CREATE FUNCTION dbo.fn_Passengers()
RETURNS @Response TABLE(LastName varchar(64),FirstName varchar(64))
BEGIN
INSERT INTO @Response
SELECT LastName,FirstName
FROM txns.Passengers
ORDER BY 1,2;
RETURN;
END
select Count(*) from dbo.fn_Passengers();
-- Returns 1280, the size of the ResultSet fetch size
-- In SSMS returns 12000
select Count(*) from txns.Passengers;
-- Returns 12000, the correct number of rows
-- Simple declaration
CREATE FUNCTION dbo.fn_PassengersToo()
RETURNS TABLE
AS
RETURN (
SELECT LastName,FirstName
FROM txns.Passengers);
select Count(*) from dbo.fn_PassengersToo();
-- Returns 12000
-- This is the original method…..
CREATE function dbo.fn_Families(@argSize tinyint = 0)
returns @Response table(LastName varchar(64),FirstId int,LastId int,NumMembers tinyint)
begin
declare @varLowSize tinyint;
declare @varHighSize tinyint;
if @argSize = 0
begin
set @varLowSize = 0;
set @varHighSize = 99;
end
else
begin
set @varLowSize = @argSize;
set @varHighSize = @argSize;
end;
with Families(LastName,Mini,Maxi,Num) as
(
select distinct p.LastName,
Min(p.Id) over (partition by LastName,StreetAddress order by Id rows between 6 preceding and current row) as "Min",
Max(p.Id) over (partition by LastName,StreetAddress order by Id rows between current row and 6 following) as "Max",
Count(*) over (partition by LastName, StreetAddress) as "Num"
from txns.Passengers p
)
insert into @Response
select LastName,Mini,Maxi,Num
from Families
where Num between @varLowSize and @varHighSize
order by 1,2;
return;
end

@ghost Have you tried enabling use SQL to limit fetch size in data editor properties?

there has been no update on the issue for a long time. If it is still actual - feel free to reopen it


Move-AzureStorageAccount command is not working

I have automated the part when I list the Storage Accounts under a specific Classic Service and when I run the command: "Move-AzureStorageAccount -Validate..." I receive the following error:
Move-AzureStorageAccount : Could not load type 'Microsoft.WindowsAzure.Management.Storage.Models.XrpMigrationValidateStorageResponse' from assembly 'Microsoft.WindowsAzure.Management.Storage,
Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
At line:1 char:13
+             Move-AzureStorageAccount -Validate -StorageAccountName $S ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Move-AzureStorageAccount], TypeLoadException
    + FullyQualifiedErrorId : System.TypeLoadException,Microsoft.WindowsAzure.Commands.ServiceManagement.StorageServices.MoveStorageAccountCommand

Could you guys please help me see what's happening? Is there something missing?

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

ID: 972e651f-7ac9-fd9c-092b-5fb7334f0721
Version Independent ID: f56b5a76-a64d-7f0f-3bf1-307dec38de8e
Content: Migrate to Resource Manager with PowerShell - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/migration-classic-resource-manager-ps.md
Service: virtual-machines-windows
GitHub Login: @tanmaygore
Microsoft Alias: tagore

@jmedina-genetec Thanks for the feedback. We are investigating the issue and will update you shortly.

@jmedina-genetec Whats the version of Azure powershell service management module which you are using?
you can get that from this command from this link  https://docs.microsoft.com/en-us/powershell/azure/servicemanagement/install-azure-ps?view=azuresmps-4.0.0#checking-the-version-of-azure-powershell
If you dont have the latest one, try updating to the latest one and try

Hi @jakaruna-MSFT I have version 5.3.0 using this command:
Get-InstalledModule Azure -AllVersions | Select-Object Name,Version,Path
As far as I know, that's the latest version available, right?

ok. Lets involve azure support now.
Please send your subscription id to this email id<EMAIL_ADDRESS>we will enable a one time free support ticket to get this resolved.
while sending the email include this issue url in the email body and issue description in the subject

Hi @jakaruna-MSFT
I just sent the email you suggested me. Please advise of further actions needed

Hi @jakaruna-MSFThttps://github.com/jakaruna-MSFT
I just sent the email you suggested me. Please advise of further actions needed
From: jakaruna-MSFT<EMAIL_ADDRESS>Sent: April 16, 2020 5:55 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Juan Medina<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Move-AzureStorageAccount command is not working (#51828)
ok. Lets involve azure support now.
Please send your subscription id to this email id azcommunity<EMAIL_ADDRESS>we will enable a one time free support ticket to get this resolved.
while sending the email include this issue url in the email body and issue description in the subject
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51828%23issuecomment-614543583&data=02|01|jmedina%40genetec.com|6a0c7f1ec8fd4930e23308d7e1ec2d8f|7ba8d2fb46604a19802e4d015a17e167|0|0|637226276827144955&sdata=7l76rWdp1n9TIEubYwn8BxAAXBQDZGFwXpIR%2Bpqh3tU%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPCVGRN2QPIVFOZYI4SHF2DRM3IV5ANCNFSM4MCS4UKQ&data=02|01|jmedina%40genetec.com|6a0c7f1ec8fd4930e23308d7e1ec2d8f|7ba8d2fb46604a19802e4d015a17e167|0|0|637226276827154952&sdata=WoDrz5yGJC%2F8HesQB5sZLXUHMLuRRTbALQ0GGeM8Q8s%3D&reserved=0.

Hi @jakaruna-MSFT
I just sent the email you suggested me. Please advise of further actions needed

A free support ticket is enabled and the details are sent to your mail.
I will close this out for now. If you need additional help please open a new issue.
Once your issue is resolved, post the resolution here.
If the resolution needs doc update, then open a new issue by referring to this one


Code has generated pass.pkpass with Zero bytes

Please help me to resolve this.

I've Set the path to my Pass Certificate
I've Set password for certificate
3.I've Set the path to my WWDR Intermediate certificate
4.I've set my passTypeIdentifier
5.I've set my teamIdentifier

What is the point of issue?

Have a look at the comments on #54, which contain some suggestions.

Yes, iI tired that but that not works for me. ButI got the solution from here: https://github.com/tschoffelen/PHP-PKPass/issues/53 . I've just added "echo $file" before the return statement in the create() function.
That's work for me!
But is this the correct way to resolve this issue?

I've updated the class. Can you test if it resolves your issue?

Yes, that works for me! Thank you, buddy 👍  . I've gone through the code , So this would be only change readfile($paths['pkpass']); with echo $file; . Right ?

I also changed some stuff around file names, but that's probably not necessary for it to work correctly.

Ok. Right now this is work correctly. Thank you for your prompt reply & support.You saved my day.I am closing this issue


alunofpb.txt

Add My University ^^

@ImThurner Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

https://www.fpb.edu.br/v2/
https://www.fpb.edu.br/v2/cursos/graduacao/ciencia-da-computacao/

@ImThurner I am sorry for a delay with processing your request.
Pull request merged. Thank you!
For the record: the domain ownership proof is at https://www.fpb.edu.br/v2/blog/news/inscreva-se-nos-projetos-de-extensao-2019-1/


treewide: format files generated by nuget-to-nix with nixfmt and make code generation of nuget-to-nix follow nixfmt rules

Description of changes
This PR is made in anticipation of the upcoming treewide reformat of nixpkgs with the new nixfmt implementation.
The main change is the first commit, that makes code generated by nuget-to-nix be formatted how nixfmt would format it.
Please note, that the current logic is not completely correct, since code generation doesn't handle the case of an empty list correctly. I think this should be addressed before this gets merged.
The second commit is not strictly necessary, since, we're gonna get the total treewide formatting anyways, but I think it should be done to avoid bigger diffs when the files get regenerated next.
The second commit was created by running the following bash command:
find -name "*.nix" -not -name "default.nix" | xargs grep "This file was automatically generated by passthru.fetch-deps" -l | xargs nixfmt

(where the nixfmt is the one provided by the nixfmt-rfc-style package)
I am not sure if the second commit should go into .git-blame-ignore-revs.
I doubt we really care about git blaming these generated lines.

Note: there's another generator script in pkgs/development/compilers/dotnet/update.sh but I won't touch that in this PR.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Have you tried running some fetch-deps to test that they match?  Obviously that's a little tricky since it can update dependencies, but you could at least make sure nixfmt is a no-op afterwards.

Please note, that the current logic is not completely correct, since code generation doesn't handle the case of an empty list correctly. I think this should be addressed before this gets merged.

Is this something that nixfmt is always opinionated about?  It seems like an annoying quirk to deal with for generated expressions.

Here's some small script to test regenerating some deps.nix files:
pkgs=(
    "ArchiSteamFarm"
    "famistudio"
    "galaxy-buds-client"
    "openutau"
    "tone"
    "btcpayserver"
    "nbxplorer"
    "wasabibackend"
    "denaro"
    "pinta"
    "avalonia-ilspy"
)

for pkg in "${pkgs[@]}"; do
    $(nix-build -A $pkg.fetch-deps)
done

At a first glance, the only change seem to be that recently sha256 was changed to hash as the default.
(And empty arrays are also not correct)

Genererated files are generally excempted from formatters

Any idea how we make this happen, particularly so it doesn't complain about newly added files?  see https://github.com/NixOS/nixpkgs/pull/331150#discussion_r1698191795

RFC166 explicitly makes no decision on whether generated files should be formatted:

There are automatically generated files in Nixpkgs, with a potentially different format. This RFC makes no decisions on how to handle such cases, but there are some options:

Exclude them from the CI via some tooling (e.g. treefmt if that is being used)
Format them anyway, either after-the-fact or ideally already in the generator tooling itself

The RFC's implementation tracking issue currently makes no mention of generated files at all.
The CI added in #326407 currently enforces formatting for any new files and also any files that were previously formatted. This is skipped if the PR title contains [skip treewide], perhaps this section could be extended to ignore files that contain some "This file is auto-generated" or "skip formatting" style header? https://github.com/NixOS/nixpkgs/blob/2e62fe69178df754ecc07e4b6ae544b68f8e45a8/.github/workflows/check-nix-format.yml#L75-L80
#322537 also does not address how generated files should be handled. cc @infinisil

I brought this up with the formatting team and we discussed a few ways forward.
In the end the consensus seems to be that generated files should almost always use a format such as JSON or TOML, rather than generating nix code.
This quote from @emilazy nicely summarises the consensus:

you can represent anything that doesn't have bespoke logic in JSON, and by definition generated stuff usually has pretty rote logic. we have a dynamic language so there's not really anything we "need" to output actual code for
I will say, about bespoke formatting: sometimes the most important thing is avoiding nasty Git conflicts
so everything on one line of compact JSON isn't necessarily always best if two PRs are ever going to touch the same file
but you can do less expansion than human-readable source code for sure, like one package per line in JSON is often going to be reasonable
anyway, treating generated files the same as all the others both simplifies our rules/checking infrastructure, and discourages people from generating Nix, so personally I'm in favour of it

This should be a fairly simple change for dotnet. I believe we would have to update two places; the nuget-to-nix script and also the make-nuget-deps function.
Plus replacing any existing deps.nix files, of course. This could either be done in a treewide change, or gradually as passthru.fetch-deps is run in the affected packages. It's probably better to do a treewide switch to json/toml to avoid being affected by later treewide reformatting PRs?
make-nuget-deps could detect whether the sourceFile is nix or json/toml for some transition period. Nix sourceFile support could be deprecated and eventually be removed.
Presumably, even if all deps.nix are removed from nixpkgs, we'd still want to keep support for nix sourceFiles in make-nuget-deps in case out-of-tree projects are relying on the current behaviour.
We'd essentially be doing something like map fetchNuGet (lib.importJSON sourceFile).
@corngood am I missing anything important here?

I'm not necessarily against using JSON for this.  I decided to use JSON when generating metadata for the VMR releases, see e.g. pkgs/development/compilers/dotnet/8/release-info.json.
However, I don't really understand the objection to generating nix.  I also don't see why we wouldn't have format checking for JSON, or whatever other formats end up in the repo.  In that case we'd be back in the same situation.  The RFC says

doing this here just blows up the repository by 40k lines

Is there a reason we'd care about the number of lines?  What is it in absolute/% bytes?  What would it be with JSON?

I'm not necessarily against using JSON for this. I decided to use JSON when generating metadata for the VMR releases, see e.g. pkgs/development/compilers/dotnet/8/release-info.json.
However, I don't really understand the objection to generating nix.

I don't think there's a strong objection to generating nix, however it was noted that importing nix can be slower than importing JSON/TOML and that it is almost always unnecessary to generate nix, since a data-format such as JSON should be able to represent anything generated automatically.
I think the stronger argument is that because generated files can be JSON, there is less motivation to provide a way to avoid formatting generated nix files.

I also don't see why we wouldn't have format checking for JSON, or whatever other formats end up in the repo. In that case we'd be back in the same situation.

It's hard to say whether non-nix files such as JSON would ever be formatted in the future. Currently the view seems to be it's better to not format data files, since we can usually generate them in a bespoke way that minimizes diff-size and merge conflicts.
Quoting @emilazy:

yes, I wouldn't want us to enforce formatting of JSON

If you're able to join the nix formatting matrix room, the discussion started here, and mostly stayed on topic.

doing this here just blows up the repository by 40k lines

Is there a reason we'd care about the number of lines? What is it in absolute/% bytes? What would it be with JSON?

Personally I don't see this as an issue. Potential for merge conflict is far more important than line count. Absolute filesize should be similar with the current formatting vs with nixfmt, I'd expect it to be slightly higher.
When mentioned in the matrix discussion, the formatting team also didn't seem concerned about line count.

I see three potential paths forward:

Make the case again to the formatting team to add an "unformatted file" header comment directive
Format the generated files
Generate JSON/TOML files instead

As codeowner, your opinion probably matters most in deciding which path to take.

Make the case again to the formatting team to add an "unformatted file" header comment directive

This might be the issue to follow: https://github.com/NixOS/nixfmt/issues/91

Format the generated files

This is my preference for now.  When diffing the existing deps files, I almost always use --word-diff.  Having them formatted should work better for diffs.  It seems to be quite common in nixpkgs to generate or modify a file and then run nixfmt on it.

Generate JSON/TOML files instead

I have a couple of changes in mind that make me wary about doing this right now:

adding support for mirrors
adding platform (rid) metadata for filtering

There's also pkgs/development/compilers/dotnet/combine-deps.nix, which rewrites deps files, and would need to be converted.

After having seen the recent discussions of lockfile generation, I do think that this PR should not get merged. I'll leave it as draft, until there's a better solution.

Well, we increase the file size by 9% for nothing.
➜ ls -lah pkgs/applications/audio/galaxy-buds-client/deps.nix
.rw-r--r-- sandro users 33 KB Thu Jul 18 21:51:45 2024  pkgs/applications/audio/galaxy-buds-client/deps.nix

➜ nixfmt pkgs/applications/audio/galaxy-buds-client/deps.nix

➜ ls -lah pkgs/applications/audio/galaxy-buds-client/deps.nix
.rw-r--r-- sandro users 36 KB Sun Aug 18 18:14:45 2024  pkgs/applications/audio/galaxy-buds-client/deps.nix

I was a bit confused why the file size would change at all but, now that I think about it, that might be due to the added newlines.
Your argument however is not sound however because anyone who cares even a little about file sizes will use compression where this increase in newline chars does not matter:
$ gzip -9 < pkgs/applications/audio/galaxy-buds-client/deps.nix | wc -c
10861
$ nixfmt pkgs/applications/audio/galaxy-buds-client/deps.nix
$ gzip -9 < pkgs/applications/audio/galaxy-buds-client/deps.nix | wc -c
10874

That's a whopping 13 bytes difference.
With btrfs' transparent filesystem compression at compress-force=zstd:1, there is no difference at all:
$ sudo compsize --bytes pkgs/applications/audio/galaxy-buds-client/deps.nix
Processed 1 file, 1 regular extents (1 refs), 0 inline.
Type       Perc     Disk Usage   Uncompressed Referenced
TOTAL       33%     12288        36864        36864
zstd        33%     12288        36864        36864
$ nixfmt pkgs/applications/audio/galaxy-buds-client/deps.nix
$ sudo compsize --bytes pkgs/applications/audio/galaxy-buds-client/deps.nix
Processed 1 file, 1 regular extents (1 refs), 0 inline.
Type       Perc     Disk Usage   Uncompressed Referenced
TOTAL       30%     12288        40960        40960
zstd        30%     12288        40960        40960

Git uses gzip on everything it stores and transmits AFAIK.
The only place where this would have an effect that is even measurable would be nixpkgs checkouts in filesystems that do not support transparent compression. Given that such users don't use filesystem compression that IME usually saves you 30-50% of total disk usage in a typical desktop system, I highly doubt they care about something on the order of a 9% increase in a subset of their nixpkgs checkout that amounts to maybe a dozen MiB total at worst.
We should definitely prioritise using JSON or TOML over Nix but we don't need to worry or care about file size increases due to formatting either.
OTOH I also question whether we need auto-generated Nix files to follow formatting rules at all since no human is supposed to read or modify them in the first place.

OTOH I also question whether we need auto-generated Nix files to follow formatting rules at all since no human is supposed to read or modify them in the first place.

It's not that generated files need to be formatted, rather the issue is that allowing them to not be formatted is additional unnecessary complexity.
It's much simpler to say "all nix files must be formatted" rather than "all nix files must be formatted, unless..."

Please continue further discussion on how to move forward on https://github.com/NixOS/nixpkgs/issues/358025


Welcome View shows informations multiple times

VS Code Version: 1.54.0-insider (02/22/2021)
OS Version: Windows 10 1909

Hi
I did find two issues in menus context, one in dropdown menus and one in some Welcome views.
Steps to Reproduce: (Related: #117223, as you can see, its not about VM)

In a folder without launch.json, go to "Run and Debug" view
Select a menu (ex: Run) with some actions disabled
Hover a disabled action
Hovering another action closes dropdown

You will see as well an issue in "Welcome" view which prints four times (Git or Run/Debug). (It may not be related...)

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

https://github.com/microsoft/vscode/issues/117237


[Bug report] v2: broken search result links

Description
I'm using the docs for the first time in months, and am hitting many broken URLs in search results.
For example, when searching for "container", the top ten results are at these nonexistent pages:
https://vuepress.github.io/reference/plugin/container.html
https://vuepress.github.io/reference/default-theme/markdown.html
https://vuepress.github.io/reference/default-theme/config.html
https://vuepress.github.io/reference/default-theme/components.html
In those cases, the pages are now on https://ecosystem.vuejs.press/.
Based on those cases, maybe the solution is to change or redirect all vuepress.github.io/reference/default-theme/* URLs to ecosystem.vuejs.press/themes/default/*, and all vuepress.github.io/reference/plugin/* to ecosystem.vuejs.press/plugins/*. I haven't gone through to see if any slugs don't fit the pattern, or to see what other sections have moved.

algolia recraw will be blocked if the records reduce up to a limit.
so.I guess this is the situation.

Just force refreshed the index
Thanks for reporting! ❤


BUG: too much space given in quick link

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

pls assign this issue to me
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Thanks sir


Bounding box, reduce the margin from 16px to 8px

To win space in small screens we are going to start reducing some spaces, this is the most basic thing.
Context  - https://github.com/CartoDB/design/issues/525

There is a ticket in RT dashboard (waiting to enter the TODO column) about this same issue: https://github.com/CartoDB/cartodb/issues/11102


VsCode doesn't indicate the "currently debugged thread" when multilple threads are stopped during debugging.

VSCode Version: 0.10.12-alpha
OS Version: Ubuntu 14.04
Debugging using Microsoft cpp debugger, but will happen in any "stop-all" style debugger.

Steps to Reproduce:

Compile following C++ code

#include
#include
#include <pthread.h>
using namespace std;
#define NUM_THREADS 5
void *PrintHello(void *threadid)
long tid;
tid = (long)threadid; //set a breakpoint here
cout << "Hello World! Thread ID, " << tid << endl;
pthread_exit(NULL);
}
int main()
{
pthread_t threads[NUM_THREADS];
int rc;
int i;
for (i = 0; i < NUM_THREADS; i++) { //set a breakpoint here
cout << "main() : creating thread, " << i << endl;
rc = pthread_create(&threads[i], NULL,
PrintHello, (void *)i);
if (rc) {
cout << "Error:unable to create thread," << rc << endl;
exit(-1);
}
}
pthread_exit(NULL);
}

Set two breakpoints in the lines with comments "//set a breakpoint here"

press F5 3~5 times

Notice, that if a breakpoint or step causes the threads to switch, the UI does not indicate which thread is currently stepping. Multiple callstacks may be displayed, which can be confusing.

@edumunoz @jacdavis @jrieken @weinand
We on the cpp debugger might be able to contribute a fix for this. It isn't an immediate necessity for us, but it is a little confusing for the user.

PRs are always welcome! If not, I can also look into it for our April release.

@delmyers if I understand correctly, you would like some more UI affordance to show the user which thread is currently stopped. If there are multiple threads, instead of showing 'Paused on breakpoint' we could say 'Breakpoint Thread {name}'. Or you had something else in mind?
Would you like to make a contribution for this or should we look into this?

Having something that says "Thread {name}" might work. Another option would be to have an arrow or special highlighting that indicates the current thread. Also, to auto expand the thread that is currently being debugged (e.g. the last one paused).

Sorry, @isidorn, I forgot to mention you in the last comment I added. Having something that says "Thread {name}" might work. Another option would be to have an arrow or special highlighting that indicates the current thread. Also, to auto expand the thread that is currently being debugged (e.g. the last one paused).

@delmyers you do not have to mention, I am always subscribed to issues I am interested in and I get notificatinos even if you do not mention me.

@isidorn Looking back over the comments, this isn't really about what thread is "currently stopped". This is about which thread is "currently being debugged." For example, you just hit a breakpoint on a particular thread, and are looking at the breakpoint, then the callstack should show you what thread the breakpoint was hit on. Or, you double-click on another callstack to go to source code, the callstack should show you the thread that you last double-clicked on. Does that make sense?

@delmyers now on every call stack changed we are expending the debugged thread and selecting the debugged stack frame in the call stack tree. Please try it out and let us know what you think.
On a side note, if I want to file a bug against a C++ extension I did not find a git hub repo but a mail address. Is that the process for filing bugs right now?

We are tracking issues for the C++ extension here: https://github.com/Microsoft/vscppsamples/issues

@jrieken you also filed a bug that the thread is not expanded on stoped. So giving to you to verify this, since it is in the end the same issue

@isidorn If it's duplicate just tag it as that


fix(nodebuilder/tests): Skip validation for core config in swamp

No need to validate core config in swamp if core client is already passed directly to fx - fixes all swamp tests.

Codecov Report

Merging #2596 (9b1002c) into main (44a0b37) will increase coverage by 0.15%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2596      +/-   ##
==========================================
+ Coverage   51.03%   51.19%   +0.15%
==========================================
  Files         157      157
  Lines       10437    10439       +2
==========================================
+ Hits         5327     5344      +17
+ Misses       4642     4627      -15
  Partials      468      468

see 9 files with indirect coverage changes


Correct fido2 verify command

Restore returning certificate, as it was before the d885ba7fa160e725e080fcffbc75632378950a9f change,
but with an optional parameter to not break the current implementation.
Fixes #96

I had an issue with my GitHub notifications and missed that review request, sorry.  Anyway, thanks for the fix!


Extracting hand detection landmarks

Hi ,
I just wanted to know if there is a way to extract the hand landmarks of an input video per frame and how I can go about doing that.

Hi rohabimbesat, were you able to extract the coordinates of the hand landmarks? I m stuck with the same issue. Can you suggest me any solutions to extract the finger coordinates in any formats like json, list, dict, etc?

yes i used an API for python, you can find instructions here
https://github.com/google/mediapipe/releases/tag/0.8.0
personally, i used json as my file format


[AutoPR advisor/resource-manager] Swaggerfix

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3213

(message created by the CI based on PR content)


“No such host is known” when publishing to azure but working fine in emulator.

Github issues for C# /JS / Java/ Python should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Sample information

Sample type: bot with luis,qna and dispatch bot
Sample language: [dotnetcore or webapi or es6 or nodejs or typescript]
Sample name: NLP with dispatch bot

Describe the bug
LUIS,QnA and Dispatch bot workning fine on emulator but gives an exception when published on azure and tested on facebook messenger.
All the luis,qna and dispatch keys are working fine on emulator and to the other older bot i have.
I have added the exception message to the error text that is why i can see it on messenger.
To Reproduce
Expected behavior
Expected the bot and services to work properly like on emulator.
Screenshots

Additional context
appsettings:
 {
  "Logging": {
    "LogLevel": {
      "Default": "Warning"
    }
  },

  "MicrosoftAppId": "",
  "MicrosoftAppPassword": "",

  "DispatchLuisAppId": DispatchAppId,
  "DispatchLuisAPIKey": DispatchAPIKey,
  "DispatchLuisAPIHostName": "southeastasia",

  "LuisAppId": LuisAppId,
  "LuisAPIKey": LuisAPIKey,
  "LuisAPIHostName": "southeastasia",

  "QnAKnowledgebaseId": QnaMakerKBId,
  "QnAEndpointKey": QnaMAkerEndpointKey,
  "QnAEndpointHostName": QnaMakerEndpointHostName,
  "AllowedHosts": "*"
}

botservice
 public class BotServices : IBotServices
{
    public BotServices(IConfiguration configuration)
    {
        DispatchService = new LuisRecognizer(new LuisApplication(
        configuration["DispatchLuisAppId"],
        configuration["DispatchLuisAPIKey"],
        $"https://{configuration["DispatchLuisAPIHostName"]}.api.cognitive.microsoft.com"),
        new LuisPredictionOptions { IncludeAllIntents = true, IncludeInstanceData = true },
        true);

        LuisService = new LuisRecognizer(new LuisApplication(
        configuration["LuisAppId"],
        configuration["LuisAPIKey"],
        $"https://{configuration["LuisAPIHostName"]}.api.cognitive.microsoft.com"),
        new LuisPredictionOptions { IncludeAllIntents = true, IncludeInstanceData = true },
        true);

        QnaService = new QnAMaker(new QnAMakerEndpoint
        {
            KnowledgeBaseId = configuration["QnAKnowledgebaseId"],
            EndpointKey = configuration["QnAEndpointKey"],
            Host = configuration["QnAEndpointHostName"]
        });
    }

    public LuisRecognizer DispatchService { get; private set; }
    public LuisRecognizer LuisService { get; private set; }
    public QnAMaker QnaService { get; private set; }
}

[bug]

Hi @bnj123 : Can you try and change your LuisAPIHostName to something like:
Australia East - australiaeast.api.cognitive.microsoft.com
West Europe - westeurope.api.cognitive.microsoft.com
West US - westus.api.cognitive.microsoft.com
And see if that helps?

@sgellock This has been a major pain-point for customers. I opened a quick-fix PR for this and I agree with @cleemullins 	that there are better options; they just need to be addressed and implemented. I'm going to leave this ticket up due to [supportability], but will answer the user's SO question.
@bnj123 Feel free to unsubscribe from this issue as I'll be answering you on your SO post. In the meantime, it may help to review the linked PR and attempt @daveta's suggestion while I look into this further.

Digging into this a little further, the error does not appear to be related to format of LuisAPIHostName. Still, this is a major pain-point

Thank you as instructed i will respond in stackoverflow.

@daveta something weird happened. in appsettings i changed hostname to " "DispatchLuisAPIHostName": "southeastasia.api.cognitive.microsoft.com", and in botservice      $"https://{configuration["DispatchLuisAPIHostName"]}"), now it is replying but detecting everything i type as "cancel" intent.

This is my SO question it is a lot detailed and has images thank you
https://stackoverflow.com/questions/57600902/botframework-publish-to-azure-error-no-such-host-is-known

@bnj123 	Sorry. Somehow this one fell by the wayside. You got this working, right? This can be closed?

@bnj123, closing this as issue seems to be resolved.   Please reopen otherwise.


As a Community Member, I want to learn about Paloma chain early so I can participate early!

Closing as shipped.


d3 zoom and slider issue working together

I have an SVG graph which works perfectly with d3 zoom. However, the requirement was to add the slider and control the zooming with the slider as well as the mouse wheel.
The issue I am facing now is I generate a transform object such as:
let sliderScale = zoomIdentityManual.scale(newK); // newK is the new scale based on slider I calculate it.
It is fine again. But when the user starts to zoom by mouse right after slider, the graph jumps around. And I can see why. because the event from the mouse is working completely separate than the transform object I generate manually.
I found this example, (http://bl.ocks.org/bollwyvl/871b7c781b92fd0044f5) and the main difference is the version of d3. It is v3 and I am using v4. as you can see in the slided function:
function slided(d) {
zoom.scale(d3.select(this).property("value"))
.event(svg);
}
looks like it does .event(svg) and that is where makes the current event change. But Version 4 does not have such a method.
All in all, is there any method to do slider and d3 zoom work smoothly together?

Please use Stack Overflow tag d3.js to ask for help. Although I make an effort to assist everyone that asks, I am not always available to provide help promptly or directly. Stack Overflow provides a better collaborative forum for self-help: tens of thousands of D3-related questions have already been asked there, and some answered questions may be relevant to you.
When asking for help, please include a link to a live example that demonstrates the issue, preferably on bl.ocks.org. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


What is missing for release 1.0.0?

Hey 👋
I wanted to ask what is missing  for the 1.0.0 release / what needs to be done before the release can be done.
In some issues you stated that the release will be done in the near future and im happy to hear that! :)
I just wanted to ask so maybe people can help out to get to the 1.0.0 release. At the moment no open issues for the milestone so it's difficult to get a foot into helping out. Or is everything finished and the release is just not done yet?

@DanielBadura Thanks for bringing this up, very valid question indeed! :-)
Most of the planned changes for the upcoming release have been merged already! This is going to be a somewhat major release, see https://github.com/graphp/graph/milestone/7?closed=1 for the full list.
With all these BC breaks here, all components that depend on this core graph component also need to be updated. This means it requires a concentrated effort to update all other components in the @GraPHP organization as well.
If you feel like helping, you're more than welcome to check out all components and make sure they're compatible with the latest beta version from this repository. I'm super happy to accept PRs!
As much as I'd like to commit to specific date, I can only say that we will get to this as soon as time permits. If you need this for a commercial project and you want to help sponsor this release, feel free to reach out and I'm sure we can sort this out.

@DanielBadura Thanks for bringing this up, very valid question indeed! :-)
Most of the planned changes for the upcoming release have been merged already! This is going to be a somewhat major release, see https://github.com/graphp/graph/milestone/7?closed=1 for the full list.
With all these BC breaks here, all components that depend on this core graph component also need to be updated. This means it requires a concentrated effort to update all other components in the @GraPHP organization as well.
If you feel like helping, you're more than welcome to check out all components and make sure they're compatible with the latest beta version from this repository. I'm super happy to accept PRs!
As much as I'd like to commit to specific date, I can only say that we will get to this as soon as time permits. If you need this for a commercial project and you want to help sponsor this release, feel free to reach out and I'm sure we can sort this out.

I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can always reopen this :+1:


fixed file is not a function error

Line 77 was trying to pass filePath into file, but file was already instantiated with filePath.

thanks @th3brink !


fix: match SNI when using TLS listeners with hostname

What type of PR is this?
A fix for TLS listener with hostname handling.
What this PR does / why we need it:
This PR sets filterChainMatch.serverNames when using TLS listeners with a hostname.
Which issue(s) this PR fixes:

hey @michaelbeaumont, for my understanding, is this PR is trying to solve  ?
Listener Protocol: TLS
TLS Mode: Terminate
Route Type: TCPRoute

https://gateway-api.sigs.k8s.io/guides/tls/#clientserver-and-tls
afaik this should be supported today
https://github.com/envoyproxy/gateway/pull/1431
https://gateway.envoyproxy.io/v1.0.0/user/security/tls-termination/

Hey, yeah traffic is terminated properly, but if I set hostname on that listener, there's no filtering of traffic by SNI as specified by the spec. The final commit shows the difference for this listener.

got it makes sense, thanks for clarifying !
is there a way to modify the TCP Listener IR w/o affecting the HTTP Listener IR structure and achieving the same goal ?
Currently the Hostnames field within the HTTP Listener is used for Filter Chain creating with a server_name constaint

@michaelbeaumont This PR looks Great - thank you for addressing the issue!
Just one small thing, could we add multiple hostnames in the gateway API translator and XDS translator tests for tls termination?  This could cover the scenario where a single listener needs to handle multiple SNIs? Thanks!

/retest

Is there any updates for this feature ?

@michaelbeaumont Are you still working on this? If not I can help, based on your previous work.

Updated and merged

@michaelbeaumont 'make lint's is failing because of trailing whitespaces, once that's fixed, LGTM from my end !

@michaelbeaumont can you also fix DCO and force push ?

@zhaohuabing I added another SNI to the test https://github.com/envoyproxy/gateway/pull/2942/files#diff-3481ca7cb1d20b614db1882fa576de2d303d434e519a17d12c2792436d6a0e18R28 actually


deployment.keys..keyFile appears to not work

Given this directory structure
- profiles/base.nix
- profiles/secret
- flake.nix

And base.nix has deployment.keys.secret.keyFile = ./secret, this will produce Failed to upload keys: I/O Error: No such file or directory (os error 2) errors.
Would I be doing something obviously wrong here?

Ohh wow. Just as I send created this issue, it hit me.
First I thought it was revolving relative to the top level directory for some erroneous project.
When using flakes, it is required that all files be checked into the repository, unless we refer to the flake via path: or maybe --impure.
A better solution is to simply use sops-nix.

My apologies for re-opening this issue, but I'm very curious what the recommended approach is now. It's really not obvious--at least from the documentation--how to get keyFile to work in a flake. It doesn't appear to work even when passing --impure! Since flakes-compatibility and secrets management are two of the major selling points for colmena, it's unfortunate they seem to be at odds with one another. Could this issue, and its best-practices workarounds, be clarified somehow in the documentation? I really like the tool, and if there's a way to get around this problem without adding another dependency, it would be nearly perfect!

You need to check it into the repository with git add ./secret.key for example. The big downside is that the secret will be stored in the Nix store of the machine running Colmena, since Nix copies the entire flake into the store during evaluation.

Hi Zhaofeng, thank you for the very quick reply. I see, that makes sense. It's too bad that that's the only way... Well, I won't trouble you with drive-by feature requests, but if a workaround ever comes up, it would be a fantastic capability for the tool to have. Thank you for all your hard work on this project!

Using sops-nix solves it.

The big downside is that the secret will be stored in the Nix store of the machine running Colmena, since Nix copies the entire flake into the store during evaluation.

That sounds like undesirable security issue and this is not even mentioned in the documentation. Please be more explicit or maybe consider removing this. It may create non-obvious security holes (for example if someone is deploying from CI host where other processes may access /nix)

@spinus
This is just a core behavior with nix.
Like pointed out earlier, using sops-nix addressed this well enough. It is what we use for all NixOS machines company wide. This has the added benefit that the repository is going to be enough for manage a machine, without requiring local state to be present.


cudpp_hash_testrig crashes on Win7 x64, CUDA 4.0, GTX 480

What steps will reproduce the problem?
run cudpp_hash_testrig -all

Output:

Using device 0:
GeForce GTX 480; global mem:<PHONE_NUMBER>B; compute v2.0; clock: 1401000 kHz
Basic hash table: Iteration 0
        Space usage: 1.050000
Table size: 1050000 slots
Maximum iterations: 2816 (141.283414 lg (N))
Number of functions: 4
Hash function constants:
<PHONE_NUMBER>,  581869302
<PHONE_NUMBER>,<PHONE_NUMBER>
         545404204,<PHONE_NUMBER>
<PHONE_NUMBER>,  949333985
Num items hashing into a slot:
        (0, 23446)
        (1, 88624)
        (2, 168999)
        (3, 214332)
        (4, 203289)
        (5, 156025)
        (6, 98636)
        (7, 54106)
        (8, 25433)
        (9, 10856)
        (10, 4230)
        (11, 1442)
        (12, 410)
        (13, 130)
        (14, 30)
        (15, 9)
        (16, 3)
Slots assigned to each key: (1, 0) (2, 0) (3, 0) (4, 1000000)
Cuda error: Error occurred during hash table clear.
 in file 'c:/src/cudpp/trunk/src/cudpp_hash/hash_table.cu' in line 179 : invalid
 device pointer.
Press any key to continue . . .

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 28 Jul 2011 at 2:51

Verified fixed.

Original comment by<EMAIL_ADDRESS>on 5 Aug 2011 at 4:56

Changed state: Verified

hey could u post how the error was fixed ?

Original comment by<EMAIL_ADDRESS>on 20 Mar 2012 at 6:08

I don't recall exactly, but you can go look through the checkins into the hash
library between Jul 27 and Aug 4 last year. It was almost certainly a fix we
made in the hash library.

Original comment by<EMAIL_ADDRESS>on 20 Mar 2012 at 6:10


Environment check for SPDK support

Longhorn/longhorn#5738

Some values such as default hugepage size will be discussed later.
cc @innobead @shuo-wu @DamiaSan


Release v1.1.0 of Ceph-CSI driver

[x] Define release process

Release doc would be a good start

[x] Define release deadlines

Code Freeze : 25-May-2019
RC : 04-May -2019
GA: 07- Jun-2019

[x] Define/list roadmap for v1.1.0

Proposed contents for v1.1.0 release
-- Stateless implementation ( RBD, CephFS)
-- Provision cephfs volumes using Ceph manager
-- E2E CI
-- Kubernetes 1.14 base.
The release issues/PRs are labelled with release-1.1.0.

[x] Document features and announce the release.

@rootfs @gman0 @j-griffith @leseb @Madhu-1 @ShyamsundarR Can you share your thoughts or add input ?

we need to make master as the default branch for this one.
@humblec assigning this to you as I see you already working on this one.

we need to make master as the default branch for this one.
@humblec assigning this to you as I see you are already working on this one.

Yep, lets make master as the default branch.
Before we make master as the default branch, I think we have to do or make csiv1.0 and master in parity.

we need to make master as the default branch for this one.
@humblec assigning this to you as I see you are already working on this one.

Yep, lets make master as the default branch.
Before we make master as the default branch, we have to do or make csiv1.0 and master in parity.

https://github.com/ceph/ceph-csi/pull/363 is to bring master and csi v1.0 in parity.

Once above PR is merged, @Madhu-1 would you be changing the default branch to master?

yeah, am planning to make master as the default branch. I will wait till EOD if anyone has any suggestions.

What has been achieved so far:
[x] Master is the default branch now.
[x] Master branch has been filled with missed PRs from CSI v1.0 branch

Progress:

[x] E2E prereq PRs are merged.
[x] Stateless RBD is merged, cephfs is pending/WIP
[x] Ceph Mgr based Volume provisioning is pending/WIP

rbd volume resizing will be included in v1.1?

rbd volume resizing will be included in v1.1?

There is a PR in progress and I would like to get this in next minor release.

[Status update]:

[x] E2E is now available in this Repo!! Its been enabled for all the PRs landing here.
[x] stateless PRs are merged.
[x] updated kube dependency to 1.14.2.
[x] updated all sidecar containers to latest available.

Pending

[x] Ceph Mgr based volume provisioning https://github.com/ceph/ceph-csi/pull/400

@sandromodarelli here is the release planning of https://github.com/ceph/ceph-csi/issues/393. If you wish to see any features or bug fixes, please feel free to mention it there.

https://github.com/ceph/ceph-csi/pulls?q=is%3Apr+is%3Aclosed+label%3Arelease-1.1.0

[status update]
Last item to sort out for the release!
https://github.com/ceph/ceph-csi/pull/400

[status update]
Last item to sort out for the release!
#400
The cephfs container image need to be updated with the minor fixes we require to support cephfs volume provisioning via ceph mgr by ceph-csi. Even though the fixes are available, this image is yet to be updated . We are postponing the release to 12-Jun-2019 to finish this task.

We are still waiting for the upstream ceph container image update to roll out this release. Delay of couple of weeks is expected, however working on other methods to address it, if possible

While we wait for Ceph upstream to cut a v14.2.2 release, I see many other open tickets in the ceph-csi project that have a "release-1.1.0" label. Can we resolve these in the meantime?

@ktdreyer Its other way around :), while we wait, we are getting maximum in. :)

@humblec does this mean, that you generally need the CEPH version 14.2.2 for running v1.1.0?

@humblec does this mean, that you generally need the CEPH version 14.2.2 for running v1.1.0?

@tiwalter yeah! or (latest) nautilus version which support ceph manager based cephfs volume provisioning. The reason being the ceph manager based volume provisioning has to be supported in the cluster and thats what we are trying achive with PR#400 . We are also figuring out if we can get 14.2.1 latest build with required patches in. Any specific queries on this ?

@humblec thanks ! ;)
The problem is, that i have CEPH v12.2.10 and need the feature "Resize Ceph FS volumes" (https://github.com/ceph/ceph-csi/pull/412), that will be available with the newest v1.1.0.
Is there any opportunity to use this feature without the newest CEPH version?
Or is it possible to release for example version 1.0.1 with the new features (like resizing) except the used features of the new CEPH nautilus version?

@humblec thanks ! ;)

YW :)

The problem is, that i have CEPH v12.2.10 and need the feature "Resize Ceph FS volumes" (#412), that will be available with the newest v1.1.0.

I see :).  Valid ask.

Is there any opportunity to use this feature without the newest CEPH version?

The version 1.0 plugin use a different mechanism which use "subdirs".  Ideally if we add resize support for that branch (1.0) it should work too. However the 'resize' sidecar..etc have to be added/qualified too. Let me think some more before confirming.

Or is it possible to release for example version 1.0.1 with the new features (like resizing) except the used features of the new CEPH nautilus version?

We had a thought about it. It duplicate good amount of work between branches. The main reason for 1.1.0 is to  have a base with couple of breaking changes like CephManger based volume provisioning, Stateless implementation..etc . Because thats the way forward or base for future versions and we have to guarentee we will not break in any upcoming releases or future versions.
The existing branches should continue working without disturbance but with available features and not really with new features. Also we have very limited hands here :).. If you could contribute features to 1.0 branch like resize, please feel free. We could get that in or review.  Its maintained, just that we dont have capacity to add or duplicate features. :)

We had a thought about it. It duplicate good amount of work between branches. The main reason for 1.1.0 is to have a base with couple of breaking changes like CephManger based volume provisioning, Stateless implementation..etc . Because thats the way forward or base for future versions and we have to guarentee we will not break in any upcoming releases or future versions.

Yes, i'm absolutely with you ;)

The existing branches should continue working without disturbance but with available features and not really with new features. Also we have very limited hands here :).. If you could contribute features to 1.0 branch like resize, please feel free. We could get that in or review. Its maintained, just that we dont have capacity to add or duplicate features. :)

I also understand this point of view.
Nevertheless, it also impossible to release a new Version "1.0.1" with the resizing functionality (and maybe more features) even if we implement this feature and open a PR for it?
I think, that a lot of people have not the newest CEPH version and an update on a productive system generates a big administrative overhead.
I think it would be creat, if we provide a matrix, which version of the plugin works with which CEPH version.
For example:

csi plugin v1.10.xx for CEPH > v14.2.2
cs plugin v1.0.xx for CEPH >= v12.x and < v14.2.2
Maybe we can set up a CI-Pipeline for CEPH Luminous, Mimic and Nautilus.

I know it will cause duplicated features, but you have now the compatibility with old CEPH versions. But if you define, that all the features for old branches should be developed by the community, it's a win win situation ;) You only have to review and release the versions.

We had a thought about it. It duplicate good amount of work between branches. The main reason for 1.1.0 is to have a base with couple of breaking changes like CephManger based volume provisioning, Stateless implementation..etc . Because thats the way forward or base for future versions and we have to guarentee we will not break in any upcoming releases or future versions.

Yes, i'm absolutely with you ;)

Thanks!

The existing branches should continue working without disturbance but with available features and not really with new features. Also we have very limited hands here :).. If you could contribute features to 1.0 branch like resize, please feel free. We could get that in or review. Its maintained, just that we dont have capacity to add or duplicate features. :)

I also understand this point of view.
Nevertheless, it also impossible to release a new version "1.0.1" with the resizing functionality (and maybe more features) even if we implement this feature and open a PR for it?
I think, that a lot of people don't have the newest CEPH version and an updating a productive system generates a big administrative overhead.

Yeah, true, but we are taking help from "Rook" to simplify this process.

I think it would be creat, if we provide a matrix, which version of the plugin works with which CEPH version.
For example:
* csi plugin v1.10.xx for CEPH >= v14.2.2

* cs plugin v1.0.xx for CEPH >= v12.x and < v14.2.2

Completely agree and this matrix will be available with 1.1 release.

Maybe we can set up a CI-Pipeline for CEPH Luminous, Mimic and Nautilus.
I know it will cause duplicated features, but then you have the compatibility with old CEPH versions. And if you define, that all the features for old branches should be developed by the community, it's a win win situation ;) You only have to review and release the versions.

Sure, that make better deal, but please note that, there are huge number of bug fixes or enhancements went to "master" branch which are missing in 1.0 branch.

[status update]
Apologies for the delay on rolling out this release, but we have a good news that, an upstream ceph container is available now with a tag latest-nautlius-develwhich is very very close to Ceph Nautlius 14.2.2 . It is helpful to unblock our Ceph Manager based PRs or work. We expect the release to be made available soon :
ETA :  08- Jul - 2019

[Status update]
The Rook Master has been updated to use latest-nautlius-devel tagged containers and  #400 is ready to go. We are on track for the release!!

Finally, the release is OUT !!!
Details @ https://github.com/ceph/ceph-csi/releases/tag/v1.1.0
Thanks all for the contribution and making this big release out !
@rootfs @gman0   @Madhu-1 @poornimag @ShyamsundarR @j-griffith @travisn  @leseb @tiwalter @ajarr @batrick @dillaman ...etc!!


cleanup ntl.h

ntl.h is the "public" header that will be used by the users of the library, we should cleanup the included headers to be sure we only expose "private" API and not internals of the library

Indeed, I'll update the issue


full support of harte grammar?

Referencing the paper the notation is: https://ismir2005.ismir.net/proceedings/1080.pdf
<chord> ::= <note> ":" <shorthand> ["("<degree-list>")"]["/"<degree>]
| <note> ":" "("<degree-list>")" ["/"<degree>]
| <note> ["/"<degree>] | "N"
<note> ::= <natural> | <note> <modifier>
<natural> ::= A | B | C | D | E | F | G
<modifier> ::= b | #
<degree-list> ::= ["*"] <degree> ["," <degree-list>]
<degree> ::= <interval> | <modifier> <degree>
<interval> ::= 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9 | 10 | 11 | 12 | 13
<shorthand> ::= maj | min | dim | aug | maj7 | min7 | 7 | dim7 | hdim7
| minmaj7 | maj6 | min6 | 9 | maj9 | min9 | sus4

And looking at the code, the implementation is: https://github.com/sakemin/cog-musicgen-chord/blob/master/audiocraft/modules/btc/utils/chords.py
Does the implementation fully support everything defined in the grammar from the paper? Writing something that generates the text_chords param and want to understand the implementation here better.
Also not sure if you looked at this but a library I found for parsing the Harte grammar: https://jams.readthedocs.io/en/stable/namespaces/chord.html#chord-harte

Found this paper from this year https://www.nature.com/articles/s41597-023-02410-w
They actually built out converters to support a bunch of notations and convert them all to Harte - https://github.com/smashub/choco/tree/main/choco/converters

MusicGen Chord supports a subset of Harte grammar.
Since the model converts chords into 12-pitch-bin-chromas, notations like C/G(C/5) will not reflected, because there is no 'root' idea in chromagram.

Found this paper from this year https://www.nature.com/articles/s41597-023-02410-w
They actually built out converters to support a bunch of notations and convert them all to Harte - https://github.com/smashub/choco/tree/main/choco/converters
It would be really cool and make this easier to use if more common notations were supported, even if they were just converted to Harte. At least one more common one like ABC would make this more interoperable with other tools.

Thanks for sharing it. If this converter is implemented, it would be more nice to use the model. It's not on priority right now, but I will work on it!

Thanks, I think this is the correct BNF for the text_chords field if helpful to anyone reading this:
<progression> ::= <bar> " " <bar>
<bar> ::= <chord> "," <chord>
<chord> ::= <note> ":" <shorthand>
<note> ::= <natural> | <note> <modifier>
<natural> ::= "A" | "B" | "C" | "D" | "E" | "F" | "G"
<modifier> ::= "b" | "#"
<shorthand> ::= "maj" | "min" | "dim" | "aug" | "maj7" | "min7" | "7" | "dim7" | "hdim7"
| "minmaj7" | "maj6" | "min6" | "9" | "maj9" | "min9" | "sus4"

Might be useful to put in the README or something.

Sorry for the late reflection! Now the grammar sheet is added in the README!


style(regular_expression): re-order dependencies in Cargo.toml

Follow-on after #6404. Nit. Group optional dependencies together.

#6675
#6674
#6673
#6672  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

Trivial change, so merging without review.


Close Tabs

Sorry I didnt use the template but this is a simple one.
Add Close Tab After...
I use the Unique function now, but would be great to see that tab go away by itself :)

Be wise. :)
Best


fix(#minor); Uniswap V3; Removed unused fields from the liquidity pool entity - RE: PR 778

Context
There are some fields related to rewards emissions, and the price of the output token which are not applicable to Uniswap V3. There are no rewards emission for Uniswap V3, and because the protocol mints NFTs that represent positions, we do not price the OutputTokens (NFTs).
I am not sure why this happened, but when I force pushed in PR #778 I accidentally push no commits and then it just merged in. This is a redo of that PR.

Should I make the subgraph.yaml and configure.ts ignore global?

interesting, you are right

Should I make the subgraph.yaml and configure.ts ignore global?

let's keep them local for now and up to the subgraph owner to decide. I think there are still a number of subgraphs that don't generate subgraph.yaml using mustache


Update transfer VoIP calls to PSTN and simultaneous ringing tables

Hi,
I find the table for transferring VoIP calls to the PSTN a little bit confusing, it is said that all users are able to make the transfer through Gateway1 based on their voice routing policy, does it mean that the only requirement is to have Gateway1 on their VRP even if they don't have permissions to make the call? I mean:

User3: this user may have Gateway1 in his VRP but it does not have permissions to make an outbound call from Site2 through Gateway1 because the gateway is LBR enabled and in a different site (Site1), so in this case, the transfer would work or fail?
User4: this user is not LBR enabled so it doesn't make sense to have Gateway1 in his VRP, so I guess the transfer would fail always, but in the case Gateway1 is added to his VRP, would it work?

The same doubts for Simultaneous ringing. Would it be possible to update those tables to clarify a little bit how it works? If the only requirement is to have the Gateway on the VRP because what is checked is the permission of the user that is transferred (not the one who is transferring) then I guess the table is fine although it would be great to have a more detailed explanation, if the user who is transferring the call needs to have permissions to make the call through the destination Gateway then I think only the scenario for User2 would work, the rest of them would fail.
Thanks!

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

ID: 5e427e0b-ef18-6947-27a6-eb75ddbdead7
Version Independent ID: baf34a47-7d43-8291-7792-dd0b5dac1931
Content: Plan Location-Based Routing for Direct Routing - Microsoft Teams
Content Source: Teams/location-based-routing-plan.md
Service: msteams
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

@jagonzal01 thank you for reaching out and helping to improve Microsoft Documents. I noticed that the article has been updated recently. Please can you let me know if you still need further assistance.

Hi @lindspea, I see the article has been updated but not the tables that I comment on my post, I think those tables are wrong or at least misleading as people may think that users in different sites can transfer VoIP calls from LBR enabled User1 to the PSTN if they have Gateway1 in their VRP but in fact (at least as far as I know) this is not possible due to users in a different location than a LBR enabled Gateway cannot use it.
In my opinion the table should be similar to this one, where only User2 is allowed to make the transfer:
https://docs.microsoft.com/en-us/microsoftteams/location-based-routing-plan#user-enabled-for-location-based-routing-2

@scanum please can you check with the author if this needs to be updated?

@jagonzal01 Thanks for the feedback! Both tables are in fact correct. In both tables, the caller is the LBR enabled User1 in Site1. User1 is only able to make outbound PSTN calls via Gateway1 in Site1. In both scenarios User1 will be connected to a PSTN user by means of either a transfer or simultaneous ringing. No matter who performs the transfer or simultaneous ringing, it will be only work if the PSTN call they generate is routed out of Gateway1 in Site1. That's why the tables looks like they do.
I'll try to make the tables a bit clearer

Thanks for the answer @JensTrier , I totally agree that User1 can only be transferred to the PSTN through Gateway1 but I disagree that "no matter who performs the transfer or simultaneous ringing" due to as far as I know the user who transfers the call (or makes the simultaneous ringing) needs to have also permissions to use Gateway1, otherwise the transfer or simultaneous ringing will fail. Based on the architecture of the article and the scenarios shown in the tables, the only user that has permissions to make the transfer or simultaneous ringing is User2, all other users will receive an error when they try to do it even if they have Gateway1 in their VRP.
If the point of those tables is to indicate that any user with permissions to use Gateway1, no matter where they are, would be able to make the transfer or simultaneous ringing to the PSTN, then I agree with that point, but if we follow the architecture then in my personal opinion they should indicate that all scenarios except the one for User2 would fail.
Thank you very much for your help, really appreciate it. Best regards.

That is what it means :-)
Whether it fails for User2 to User6 only depends if they have online voice routes that will allow the call to be routed out of Gateway1.
If they have, then it will work.
If they don't, the call will route out some other gateway that User1 is not allowed to use and then the call will fail due to LBR restrictions for User1.
The LBR state of User2 to User6 is not taking into account, since they will not be the resulting party talking to the PSTN. That is User1 in all cases.

Ok, thanks, then everything is clear, my assumption was wrong as I thought LBR was checked for User2 to User6 as well to make the transfer (I know they are not going to talk to the PSTN but I though the authorization process was the same as if they were making a call to the PSTN).

No worries & thanks for taking the time to provide feedback! Please keep it coming.
Feel free to close this issue.


Blender crashes when switching to "Rendered" viewport mode

.blend file: default cube
Steps to reproduce:

Switch render engine to LuxCore
Switch "Device" to "GPU" (optional)
Switch viewport to the "Rendered" mode

Error message from Blender console:
blender.crash.282.txt
blender.crash.282-gpu.txt
blender.crash.283.txt
blender.crash.283-gpu.txt
OS: Ubuntu 20.04
Blender version: both v2.82a and v2.83.0 were tested
LuxCore version: Pre-release build (the f3ace6d one)
Hardware: AMD FX-8350 and AMD Radeon RX Vega 56
Notes:

I got OpenCL support from amdgpu-pro-20.20-1089974-ubuntu-20.04 (link) which is working OK for Cycles.
BlendLuxCore v2.3 is also working nicely with it

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Your fix is correct however it is like your driver is returning multiple binaries for a single device (yes, the OpenCL context in LuxCore has always only a single device associated). It is the only way I can explain the memory corruption with old code.
What is really strange is that clGetProgramInfo() wants to return multiple binaries but than works when reading the cache and only the first one is provided.
I guess the really correct fix would be to add the support for saving/reading multiple binaries from the cache. Anyway, if the current code works, I'm not going to complain.

Thank you! The latest BlendLuxCore build works well for me. I think, the issue can be closed.


Refactor AWS DocumentDB pricing

Currently we support aws_docdb_cluster_instance but we could improve it if we:

add aws_docdb_cluster: this should only have the Backup storage cost component, which should be removed from aws_docdb_cluster_instance and moved into this resource. This cost component should only apply if aws_docdb_cluster.backup_retention_period is more than 1 (the cost component has the backup_storage_gb usage file property too).
add aws_docdb_cluster_snapshot, it only has 1 cost component, which is Backup storage (would be good to call the same method from the above resource to do this instead of copy/pasting the code).
add aws_docdb_subnet_group to the list of free resources

@alikhajeh1 I want to do it

Released in v0.8.3


Test fork behavior in the build pipeline to see what needs to be adjusted

Description

Test fork behavior in the build pipeline to see what needs to be adjusted
Motivation and Context

Test fork behavior in the build pipeline to see what needs to be adjusted

How Has This Been Tested?

That is the test!
Screenshots
Types of changes

[x] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[x] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[x] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[x] I have read the CONTRIBUTING document.
[ ] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

Summary by CodeRabbit

Chores

Updated environment variables for the Azure DevOps pipeline configuration to use hardcoded values for organization and pool name.

Will close this PR, move to ADO pipeline rather than action and retry a new forked PR


On Demand URL Example?

This page seems to show an example of using an on demand url but the code snippet for the example is missing. It would be helpful to see the code, or to have an explanation of how it works.
https://ember-stereo.com/docs/playing-sounds#on-demand-url

The code is the same as the example above it.
And "on demand" is WNYC radio lingo for just a static file, and maybe that should be changed… Just saying that that example is nothing fancy, it's just using a path to an mp3 file for the @identifier value you see in the code sample above it.

Ah! I was misunderstanding what on-demand meant in this context. I thought that it meant something like "after you click the play button we'll retrieve the actual URL over the network and then start to play it". (In my case the files are in a protected S3 bucket and I need to sign a request in order to be able to download them, and I incorrectly assumed that "on-demand" was for that kind of thing.)

One thing that might not be very clear either is that you can pass a promise that returns an identifier (a url or an array of urls to try) into stereo to play, and it will resolve as part of the playing process. In your case that promise would return a signed url with a short expiration time, probably.
It was something that was built in a long ago to get past some some browser autoplay restrictions when the URL wasn't known at page load and it should still work! I haven't used that method lately for the projects that I use stereo on, but it sounds like it might be useful in your case.


Create adapative card based on Json Schema is not working

Problem Description

Go to https://adaptivecards.io/designer/
Click left top corner "New card"
Select from Json Schema
Take to "Create from JSON Schema" page where I copied json schema contents and click create button
Doesn't generate full Adaptive card contents based on json schema.
Card Payload Editor gives adaptive card with null items
Output

Json Schema Input example is below
[   {     "id": "5f4fde50-aa9f-11ea-b5db-sdsdd",     "name": "project-test",     "owner": {       "id": "100",       "username": "John"     },     "project-source": "java"   },   {     "id": "ed02aefe-ecdc-11ec-97dd-dfdfdf",     "name": "Test-Demo",     "owner": {       "id": "200",       "username": "Mike"     },     "project-source": "python"   } ]
Please help me to generate adaptive card based on JSON.
Card JSON
`
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "Container",
            "items": [
                null,
                null
            ]
        }
    ]
}
`

Screenshot

@vn0ytch,
I suggest starting with a blank card and using our designers to design the cards. You can drag and drop elements from the card element pane on the left. Doing so will help you better understand the AdaptiveCards schema. After familiarizing yourself with this, the information found at this link will be invaluable. It delves into the AdaptiveCards' Templating feature. While the JSON Schema you provided can be utilized in templating, it's essential to create an AdaptiveCards template that can process that data. I am closing this issue for now. Please respond to this thread if needed, and refrain from creating a new issue.


November 2021 Luau recap

Suggested new section:
Luau is open-source!
We announced this in early November but it deserves repeating: Luau is now an open-source project! You can use Luau outside of Roblox, subject to MIT License, and - importantly - we accept contributions.
Many changes contributed by community, both Roblox and external, have been merged since we've made Luau open source. Of note are two visible changes that shipped on Roblox platform:

The type error "Expected to return X values, but Y values are returned here" actually had X and Y swapped! This is now fixed.
Luau compiler dutifully computed the length of the string when using # operator on a string literal; this is now fixed and #"foo" compiles to 3.

You might think that C++ is a scary language and you can't contribute to Luau. If so, you'd be happy to know that the contents of https://luau-lang.org, where we host our documentation, is also hosted on GitHub in the same repository (https://github.com/Roblox/luau) and that we'd love the community to contribute improvements to documentation among other changes - see issues in this list that start with "Documentation"!


New Version

The current version of beat saber cannot be downgraded. So that this mod can be usable from now on I would like to ask if you could please make a new version of the mod so that it can be used in version 1.35.0

Have you tried to use this awesome project to get a older and matching version of beatsaber to sideload?
https://github.com/basti564/DownQuest


[AIC-PY][3/n] llama extension+cookbook: remember

[AIC-PY][3/n] llama extension+cookbook: remember

Stack created with Sapling. Best reviewed with ReviewStack.

#224
-> #240

I think this got accidentally merged with the cli-mate cookbook, but please fix forward per comments @jonathanlastmileai

I think this got accidentally merged with the cli-mate cookbook, but please fix forward per comments @jonathanlastmileai

ah strange. re: saving history?

I think this got accidentally merged with the cli-mate cookbook, but please fix forward per comments @jonathanlastmileai

ah strange. re: saving history?

Yes. I think this implementation should leverage the existing previous prompts of the same model for the chat history. It also still needs the prompt.outputs set in run_inference


[Feature Request] Better Code Completion Php

Please add that you can access other files and automatically get suggested Php uses, functions, variables, etc. like with VSCode.

Better suggestion:
Make a plugin:
Selectable language model download with prettifier, import access among files (different languages use different way to import assets) and some code completion suggestions.
Might be a little tough (I'm lying, it will hurt a lot) to make such tool. But if i can be done
Boom, VS code in mobile with 0 competitors