How to integrate Bootstrap-Native into VueJS correctly?

I spend hours trying to figure out how to get BSN to work in a VueJS environment. It doesn't matter how I include, import or require it, it seems that its never becomes initialized by itself. But BSN sits inside the Vue app (the debugger console prints the funktion on typing app.$bootstrapNative.Collapse)
I know there is a similar issue (#292) which solved it by manually initializing the needed Bootstrap elements. But is this really the way-to-go? If so, the Wiki or Manual should say so, but I couldn't find such information.
Here is my main script which is working, with this weird manual workaround from #292:
// import vue
import Vue from 'vue';

// import bootstrap.native
import BootstrapNative from 'bootstrap.native/dist/bootstrap-native-v4';
Object.defineProperty(Vue.prototype, '$bootstrapNative', { value: BootstrapNative });

window.app = new Vue({
    el: '#app',
    methods: {
        initBootstrapNative() {
            var collapseElements = document.querySelectorAll('[data-toggle="collapse"]');
            if(collapseElements.length) {
                collapseElements.forEach(function(element, index) {
                    new Vue.prototype.$bootstrapNative.Collapse(element);
                });
            }
        },
    },
    mounted() {
        this.initBootstrapNative();
    }
});

@Moongazer the docs and wikies are for the older version, the version we now have on the legacy branch.
However, the dynamic content should be handled similarly.

Hey, thank you very much for the fast response! I've added the current master branch to my project (3.0.0) and tried out different things. Not sure if I do all this right, as I'm quite new to npm, building, ES, Vue and stuff.
This line makes BSN available inside my Vue app, but still nothing is initialized automatically
import BSN from '../../node_modules/bootstrap.native/src/index.js';

But if I added this line to my Vue app, at least the Collapse starts working (Dropdown not, unfortunately)
    mounted() {
        BSN.initCallback();
    }

Just because I've seen selective component imports from other packages like this, I was wondering if it might work: but it seems not. Non of these objects are available in my Vue app.
import {componentsInit, initCallback, Collapse} from '../../node_modules/bootstrap.native/src/index.js';

I find your suggestion for the custom build (custom-index.js) from your post quite complex, is there no other possibility, like other packages do it, e.g. bootstrap-vue with the above syntax of import { componentX, componentY } or so?
Sorry if I might can't be so helpful in giving you productive feedback, as like I said I'm just starting somewhat to understand slowly how all this works together. The last time I was programming with JavaScript was over ~10 years ago, since than it seems like everything had changed (to the better I guess, but its complex to get into it again especially with all the building and package-managing stuff which didn't existed back than as far as I know)^^

@Moongazer TBH I don't know, I'm new to rollup/ES6 myself, I happen to have some friends who helped me with this stuff, perhaps searching through other issues you might find the answers.

Forgot to mention: you can drop some console.log() inside and outside your scope, there might be a type error.

@Moongazer you got any success? I'm curious to know, I want to update the wiki, so please check back :)

If You have installed via npm (from github master, not from registry, as there is no recent version deployed), then You can
import BSN from 'bootstrap.native'

or if You want selective components, then
import Modal from 'bootstrap.native/src/components/modal-native'

because index do not export components separately.

@sookoll did you update your BSN from npm latelly or checked the releases yet?

Hi, same here, trying to use BSN (custom build with only 4 components) with vuejs in production.
Whjat is the best way to access BSN functions inside vuejs component ?
Thanks.

@Eroan what is your environment? You're doin ES6/ES7? If so, try the above and let me know.

No ES6/ES7 for me no, I load vuejs and my components this way :
`
`

@Eroan I don't know what that is :D

@Eroan this is just normal javascript. Download it and use it as You use any other library:
<script src="/assets/js/bootstrap-native.min.js" defer></script>
<script src="/assets/js/vue.min.js" defer></script>
<script src="/assets/js/vue-components.js" defer></script>

In vue-component.js there should be BSN in global scope (window.BSN). On there, You can use
const modal = new BSN.Modal()

Thanks for your answer.
That's the way I did it with previous version yes, I hoped there would be ways to integrate it better in vuejs environment.

@Eroan yes there is, if you're doin ES6/ES7, then you can do this kind of stuff
https://github.com/thednp/bootstrap.native/wiki/How-to-use#es6es7-basic-example

@Moongazer you got any success? I'm curious to know, I want to update the wiki, so please check back :)

Hey, sorry for my late reply. I've made an update to the current master (v3.0.2) and checked in my test-project:
import Vue from 'vue';
import BSN from 'bootstrap.native';

window.app = new Vue({
    el: '#app',
    data: {
        name: ''
    },
    mounted() {
      BSN.initCallback() // this manual call seems to be needed at least to get something to work
    },
});

Without the initCallback() call BSN is not initialized by itself (I don't know if that is even possible, but it would be great if just the import-statement is needed if someone wants to initialize all BSN features).
With BSN.initCallback() in place, the navbar collapse is working, but dropdown-menu is not. No errors are thrown. So it seems it is the same behavior like before (see my comment on 28th March).

You guys suck :)
@Moongazer you stop thinking jQuery.
mounted() {
  Array.from(document.querySelectorAll('[data-toggle="collapse"]')).map(collapse=>new BSN.Collapse(collapse))
  Array.from(document.getElementsByClassName('dropdown')).map(dropmenu=>new BSN.Dropdown(dropmenu))
}

but my method is best, already does that, lots more efficient, and apply to all components
 mounted() {
   BSN.initCallback() // this manual call seems to be needed at least to get something to work
 }

mounted() {
  Array.from(document.querySelectorAll('[data-toggle="collapse"]')).map(collapse=>new BSN.Collapse(collapse))
  Array.from(document.getElementsByClassName('dropdown')).map(dropmenu=>new BSN.Dropdown(dropmenu))
}

This is a shorter code version of my initial post, thanks for that^^

but my method is best, already does that, lots more efficient, and apply to all components
 mounted() {
   BSN.initCallback()
 }

Is this working for you? Like I said, in the current master collapsing works by initializing like this, but dropdown is not (events are not attached to .dropdown-toggle elements).

Nop, they are attached to '[data-toggle="dropdown"]' selector

Как же долго вы разбираетесь в этом =) Вот так я использую bootstrap.native в nuxt

И у меня нет проблем с переходами, и с разрушением Dropdown


Fix definition of Swift version

Currently, to get this working in certain setups I must add a post_install hook in my podfile setting SWIFT_VERSION, when using cocoapods 1.5.3:
post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            if target.name == 'Anchorage' then
                config.build_settings['SWIFT_VERSION'] = '4.2'
            end
        end
    end
end

move swift version from .swift-version to podspec
correctly state the Swift version as 4.2

...unless that .swift-version file is used for other purposes I'm not seeing, of course.

Closing this because now it seems to work ¯_(ツ)_/¯


Add Datelist connector

Datelist - Microsoft Flow Connector
Microsoft flow connector for https://datelist.io
Datelist is an online booking system platform that allows customers taking meetings online.
Triggers

New booking: Triggers when someone books a meeting in the Datelist platform

Actions

Get calendars: List all available calendars in Datelist.io

Questions
I've tried to make our organization name as the Stack owner (datelist.io), but I get the following error:

The request content was invalid and could not be deserialized: 'Could not find member 'stackOwner' on object of type 'ApiPropertiesDefinition'. Path 'properties.stackOwner', line 1, position 384.'.

So, I've put my personal name here "Clarembeau Alexis", do you know what are the accepted values in that field?
Checks done
Please check the following conditions for your PR.

[X] apiDefinition.swagger.json is validated using paconn validate command.
[X] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@aclarembeau Could you please also sign the CLA? Thanks!

Hello, and thank you for the feedbacks.
I've signed the CLA, removed PII and used ACL Software (our company) as a publisher and a stackowner.
Is that good with those modifications?

@aclarembeau I'm just checking in to see how things are going with updating your connector as per the feedback. Feel free to reach out with any questions you may have.

Hello @sriyen-msft , sorry for the delay. I've applied your recommendations; so, it's good on my side.


Error on Processing: this.processPageData is not a function

In n8n v1.67.1 : I installed the n8n-nodes-google-document-ai extension.
Actually i run into another Problem:

This line seems to be not working:
https://github.com/ephraimm/n8n-nodes-google-document-ai/blob/master/nodes/GoogleDocumentAI/GoogleDocumentAI.node.ts#L206
http://localhost:5678/
Internal error
NodeOperationError: Internal error
at Object.execute (/home/node/.n8n/nodes/node_modules/n8n-nodes-google-document-ai/nodes/GoogleDocumentAI/GoogleDocumentAI.node.ts:222:12)
at Workflow.runNode (/usr/local/lib/node_modules/n8n/node_modules/n8n-workflow/dist/Workflow.js:722:19)
at /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/WorkflowExecute.js:704:51
at /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/WorkflowExecute.js:1134:20
this.processPageData is not a function
TypeError: this.processPageData is not a function
at /home/node/.n8n/nodes/node_modules/n8n-nodes-google-document-ai/nodes/GoogleDocumentAI/GoogleDocumentAI.node.ts:206:53
at Array.map ()
at Object.execute (/home/node/.n8n/nodes/node_modules/n8n-nodes-google-document-ai/nodes/GoogleDocumentAI/GoogleDocumentAI.node.ts:206:29)
at Workflow.runNode (/usr/local/lib/node_modules/n8n/node_modules/n8n-workflow/dist/Workflow.js:722:19)
at /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/WorkflowExecute.js:704:51
at /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/WorkflowExecute.js:1134:20

for simplification, the node now uses @google-cloud/vision, review the readme for changes to auth.
The textDetection method on ImageAnnotatorClient returns textAnnotations which is returned as the out put of the node

@ephraimm the authentification works very good now :)!

@ephraimm we tested node-red https://nodered.org/
There are all those modules available + it's a real FOSS Licence. Therefor we switch.


Doesn't work for discord.js v14.9.0

Can't use this, it crashes when executing pagination function with the message:
node_modules@devraelfreeze\discordjs-pagination\src\pagination\pagination.ts:118
initialMessage = await channel.send({
^
TypeError: Cannot read properties of null (reading 'send')

Send me the code you use to get this error please


Correct multipath functional test

Found this bug when adapting to Taproot. Oddly it works with wsh!
https://github.com/wizardsardine/liana/blob/903328ffef8317b1f152633ac671ecd92071d8eb/tests/test_misc.py#L147-L148

Ah no it's not a bug. It's supposed to update the existing PSBT.


add committee progress tests for non genesis case

addresses  #2330

added case for period 1/2 instead of just 0/1
added case for misc_balances state for additional random coverage

Looks good.
I just noticed another issue related to meta.yaml that could possibly be fixed here too, cf. https://github.com/ethereum/eth2.0-specs/issues/2330#issuecomment-820974924

Thanks @michaelsproul. addressed!

Addressed your comments @ralexstokes. Ready for another review


correct example code in javadoc: the method name is attr(), not getAttr()

I found some mistaken in the javadoc of class ChannelHandlerContext:
public void channelRead({@link ChannelHandlerContext} ctx, Object msg) {
Attribute attr = ctx.getAttr(counter);    // this line should be removed
Integer a = ctx.getAttr(counter).get();   // the method should be attr()
And please merge this commit to other branch if need.

@skyao thanks... I squashed the two fixes and updated the commit message.
Cherry-picked into 4.0 (38df32e2a87c56274ffac4f96b827f029bda0c6e) and 4.1 (9b171beb92ce759e0c4c2dd7ddbb5e345a789e83)


Youku server reported error -6001: ccode参数错误

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://v.youku.com/v_show/id_XNDIzMjcyNjc0MA==.html?']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.22621
[debug] exe versions: none
[debug] Proxy map: {}
[youku] XNDIzMjcyNjc0MA: Retrieving cna info
[youku] XNDIzMjcyNjc0MA: Downloading JSON metadata
ERROR: Youku server reported error -6001: ccode参数错误; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youku.py", line 189, in _real_extract
youtube_dl.utils.ExtractorError: Youku server reported error -6001: ccode参数错误; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Description

I tried downloading a random video from youku just to test ytdl. Sadly it didnt work and i dont know what caused the problem

Changed ccode to 5052. Fixed the issue but encountered a new one.


WIP: Add 8.3xx pre-release

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Pushed to a dedicated branch: https://github.com/conda-forge/pythia8-feedstock/tree/8.3xx


Generated server id doesn't match what is advised in the azure artifacts packaging page

Love this tool.  It will help us tremendously, as the auto-authenticate of the maven task requires the root POM to have all upstream registries.  Plus, this will be great for our Docker authentication.
However, the generated ID in the settings.xml is just the name of the artifact stream you are attaching to.  The snippet in the packaging UI has a schema of dev-azure-com-${ORG}-${PACKAGESTREAM}.  Since our engineers will be leveraging the UI a lot, this causes drift with what we have in our poms.xml to support both CI and local building.  Currently we rely on text replacement commands ran immediately after the authenticate task.
Is there a way to specify the name of the ID we will generate?  Or, is there a different standard you are expecting poms to follow so that we can have the repository contain two IDs?  Thank you!

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

ID: 652905ce-af26-dbea-9207-c11cd3bd0064
Version Independent ID: 6558ec51-b1b5-15d8-0d46-1214f4038de1
Content: Maven Authenticate task (for task runners) - Azure Pipelines
Content Source: docs/pipelines/tasks/package/maven-authenticate.md
Product: devops
Technology: devops-cicd
GitHub Login: @chasewilson
Microsoft Alias: phwilson

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.


Log at ERROR level for SeverityError Conditions

I found several cases where a reconcile error occurs, but the log message doesn't stand out because it was at INFO level. Most cases in this code use log.Error when the resource status is being marked unready w/ condition.SeverityError, so this makes it consistent.

These are informative logs, not errors of the operator per se, thats why they are Info messages.


chore(deps): downgrade @types/node from 20 to 18

Drop typings package to align with lowest supported major Node.js version.
Depends on

[ ] #875

I've done this in #875, so this can be closed


Button "is_active()" crash !!!

Hi, I have used for first time gpiozero with this code on Raspberry Pi 3 :
import sys

from gpiozero import Button

input_hw_pin = Button(6, pull_up=False)

try:
    if input_hw_pin.is_active():
        print "Pin High"
    else:
        print "Pin Low"
except:
    print "Unexpected error:", sys.exc_info()[0]

But program carsh with:
exception: Unexpected error: <type 'exceptions.TypeError'>
Has anyone ever had a problem like that ?
debugasm

is_active is a property, not a method, so you need to change if input_hw_pin.is_active(): to if input_hw_pin.is_active: (i.e. remove the parentheses).

You are definitely right. I'm not an expert in Python, I am confused on watching this code:
from gpiozero import LED
from time import sleep

led = LED(17)

while True:
    led.on()
    sleep(1)
    led.off()
    sleep(1)

and I imagined that "is_active" also had a method and not a property.
Thanks very much

The error you should have got is:
TypeError: 'bool' object is not callable

Because is_active is either True or False, and you're trying to call it.


Looks good to me! Will let @SMillerDev merge when he's ✅ as I'm on holiday so fairly unresponsive if this needs reverted.


Add location data to the Pin model

Whatever format it is stored in (a separate model, json, etc.), it should probably be able to be converted to geojson. Client code could use this to render pins on top of datasets as features.

This is a duplicate of #18
I'll go ahead and close this issue.


fix(leap): dont autojump if autojump is off (&& #occurences=1) (#206)

may resolve #206

I think https://github.com/ggandor/leap.nvim/commit/89d878f8399d00fb348ad65b6077b996808234d8 is the correct fix (or at least a bit simpler to follow, if they happen to be equivalent for all possible paths), but thanks anyway!


Fix wrong reference to fastapi-mail and typos in contributing.md

contributing.md talked about fastapi-mail instead of  fastapi-mqtt. That was fixed by changing the text and updating the urls
Also, the ### Before get in to the repo and type section looks a bit weird, i dont understand the purpose. Maybe it had sense in fastapi-mail?

Hi @KrappRamiro, thanks for the PR, I took it from my other module at the initial start, thanks for fixing :) I will approve PR,  @azogue has a comment please take a look, thanks!


e10n not showing up under settings

No triggers during the fight and there's no section for it under the 5.0 raids in the settings. I see the files in the folder though so I'm not sure why it isn't working.

It's in the repository, but I haven't done a release with it yet.

Fixed in https://github.com/quisquous/cactbot/releases/tag/v0.21.10


readme enhancements

integrate codecov
update compatibility notes
add badges

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d3a1a99). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage        ?   67.27%
  Complexity      ?      240
=======================================
  Files           ?       37
  Lines           ?     1858
  Branches        ?      294
=======================================
  Hits            ?     1250
  Misses          ?      490
  Partials        ?      118

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 d3a1a99...f742d11. Read the comment docs.

got approval from security. as long as we are not using codecov github action v1 and bash uploader, we are good.


[HOLD for payment 2023-09-29] [HOLD for payment 2023-09-27] Web - App Fails to Redirect to Announcement Page After Split Bill

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to settings > workspace > create new workspace > members.
Invite members to the workspace.
Click on the three dots and open the announcement room.
Click on the plus sign, select split bill, add the desired number, and click on split bill.
The app should redirect to the announcement page, but the split bill page remains open and the app does not redirect.

Expected Result:
After splitting the bill in the workspace, the app should automatically redirect to the announcement page.
Actual Result:
The app fails to redirect to the announcement page after splitting the bill, and the split bill page remains open.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
screen-capture - 2023-09-18T064009.572.webm
https://github.com/Expensify/App/assets/93399543/8c274d89-0bc6-49f1-a73f-e78fa7f48804
Expensify/Expensify Issue URL:
Issue reported by: @tewodrosGirmaA
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695037437630339
View all open jobs on GitHub

hmm was is split bill even showing up in the + list for an #announce room?
From the looks of it after the split is created it creates a new report in the LHN since it is being split between 2 users (so a manual request) not with the workspace.
I haven't been following much with Manual Requests so I'm not 100% sure that option should be showing or not.

opened a discussion here to see what others think https://expensify.slack.com/archives/C01GTK53T8Q/p1695057805924489

PR to fix has been merged but not yet deployed https://github.com/Expensify/App/pull/27793

i think we can close? if there is payment owed somewhere let me know

@bondydaa I didn't get paid for bug report

(Reporter Role) Hello, @bondydaa I was wondering if I am eligible for payment even though my issue is marked as "Awaiting Payment" and I have not yet received any payment. The issue has been closed and I would appreciate any clarification on this matter. Thank you.

oh doh sorry yes that seems right, lets get a bug zero person involved to handled that, sorry!

@miljakljajic can you handle paying @tewodrosGirmaA for reporting the bug? thanks!

Thank you, handling!

sent contracts to you both

Thank  you @miljakljajic  I accept the offer

Paid and contracts ended

Bonus paid, sorry @aimane-chnaif !


Token transfer impossible - gives insufficient token balans

Hi there,
never used github before, so not sure if Im doing it right!
I tried to transfer tokens, but somehow I got the error 'insufficient token balance'. It worked with two tokens, and some others didnt work (at least hbarbarian and basil). It didnt make any sense because there was sufficient balance.
I tried with hex, which gave me the same error 'you don't have enough tokens'. So I thought maybe it was a hedera issue. However, I now succeeded in transferring all tokens with Dode Cane wallet. So, therefore I guess it must be an issue with Myhbarwallet and hex.
Do say if you need more info.

Hi,
We are currently working on the HTS functionality of MHW, and you should expect a better experience when version 0.6.0 releases sometime in the next week.

The issue is likely in looking up token information, fyi

Hello Chris! Thank you for your answer :)

try next.myhbarwallet.com!
Report any issues you experience here


Fixed nested resources keys escaping

Fixes #234 . Added basic vagrant provision.

@chingor13 Thank you for response. All done.


Ignore namespaces that ignored by sidecar for ambient

(This is used to request new product features, please visit https://github.com/istio/istio/discussions for questions on using Istio)
Describe the feature request
istio has skipped serveral known namespaces, we should skip them for ambient as well.
	constants.KubeSystemNamespace,
	constants.KubePublicNamespace,
	constants.KubeNodeLeaseNamespace,
	constants.LocalPathStorageNamespace)

Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[x] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

Agree, it is better to provide a options to customize.

Can customize this with discovery selector FYI https://github.com/istio/api/pull/3285#discussion_r1697771610


Bumped up the Xtext version to 2.11.0 and the MWE2 to 2.9.0.

Please note, once the changes are merged, then one has to update the Xtext/Xtend tooling version to 2.11 in Eclipse. There were breaking Xtend changes which affect the active annotations. More details about the issue are available here.
Also, I had to provide a workaround for an issue with the contipref library. It does not work with JUnit 4.12 due to this bug. I have tried to get the latest version of this library from Maven, but I had no luck. I was looking for version 2.4.3. I decided to check out the source and provide a jar from a local build. But this project is quite fragile; when I run the Maven build from command line, I get errors, when running the tests from Eclipse I get completely different errors. What is the purpose of this library? Are you storing your benchmark results somewhere for historical comparison or it is used only for assertions regarding the execution times? Although I have no experience with JMH, that seems to be the recommended way to have a proper benchmark. BTW, the owner suggests the same and explicitly declares that to project is not continued anymore.
I have not yet created a dedicated project for the ECL testing, but that is my plan. Do you have any objections?
I look forward to your feedback on this PR.
Signed-off-by: Akos Kitta<EMAIL_ADDRESS>

Thank you for the PR.
We plan to replace contiperf with JMH, but the workaround is okay for now.
What kind of ECL testing project would you like to create/add? The one that Xtext generates (c.b.s.snomed.ecl.tests)?
Our plan with the number of projects right now is to reduce it to a reasonable amount.
For example, it is okay to add the Xtext generated test project to the project set, but that project should contain only grammar, validation, formatting, etc. tests. The evaluation logic should be part of the core SNOMED CT test bundle (c.b.s.snomed.datastore.tests), as it is right now.
Also we plan to merge all bundles that correspond to a single terminology into a single one. For example: merge every *.snomed.* bundle into a single c.b.s.snomed.core bundle (except SCG, ESCG, ECL grammar projects).
I hope this helps, feel free to ping me on Skype if you have further questions.

The evaluation logic should be part of the core SNOMED CT test bundle (c.b.s.snomed.datastore.tests), as it is right now.

Correct. I would like to do exactly what you've described. Probably I do not even need to run the tests with tycho-surefire but maven-surefire. I need neither the OSGi classloader nor the Eclipse platform. Do you have pure maven tests? I did not find them. Maybe I have just missed them. Or is the  Tycho way the preferred one? I am asking because, for instance, the LS does not need Eclipse at all.

Do you have pure maven tests? Or is the Tycho way the preferred one?

We have basic JUnit tests (like the commons.tests˙ bundle), but we let Tycho to run them all. This way, we only configure the tycho-surefire-plugin in the root pom and use eclipse-test-plugin packaging in the actual module. I think you can do the same with the ECL test project, but if it's easier for you to use maven-surefire, just use eclipse-plugin instead of eclipse-test-plugin in the pom, and you're good to go.

Sounds great, once you merge this PR, I start working on the ECL grammar and the tests. Just a short remark; I am glad that the headless Maven build runs so smoothly and fast. Great job!

One more quick question on this PR; how long will it roughly take to get this PR merged? I guess you need to update the development environments, TPs maybe other things and so on before applying the changes onto the develop branch.

how long will it roughly take to get this PR merged?

Our plan is to merge it after the 5.7.0 release, when we start the dev.iteration of 5.8.0-SNAPSHOT. Hopefully tomorrow or Wednesday.

Excellent. Thank you!

5.7.0 is out. Merging develop. Thank you for the contribution, @kittaakos!


Quill realtime editing

How this can be implemented:

Using the transform method of the Quill Delta object (https://github.com/quilljs/delta#transform)
Using websockets, with 2 possible requests that can by sent by the user:

Cursor changed; to notify the server that either selection or the cursor has changed (this can be done using our own implementation of something like https://github.com/reedsy/quill-cursors)
Data changed; to notify the server that the data has changed. Here a timestamp is added, so the server can use operational transform to decide how a following edit can be changed accordingly

The server will notify all users of both changes
If we use Node clustering, we should use some kind of store to store these operations
As said in #77, we will only save edits to the DB after 5 minutes of inactivity. I'd reckon that we also save every edit to some kind of file, so we can see individual edits if we really want to. This would also happen each minute or so, so performance won't be that bad (it should be better than writing each edit to the DB, which is also unnecessary)

What's your 2 cents here, @NULLx76 ?


Openssl can't find libssl.so.1.1 and libcrypto.so.1.1

Hello to all,
when I try to launch openssl I get this error messages
$ openssl version
openssl: /lib64/libssl.so.1.1: version `OPENSSL_1_1_1' not found (required by openssl)
openssl: /lib64/libssl.so.1.1: version `OPENSSL_1_1_0' not found (required by openssl)
openssl: /lib64/libcrypto.so.1.1: version `OPENSSL_1_1_1' not found (required by openssl)
openssl: /lib64/libcrypto.so.1.1: version `OPENSSL_1_1_0' not found (required by openssl)

I tried the solution described in the Git topic below
`OPENSSL_1_1_1' not found (required by openssl) #5845
But it doesn't solved my issue.
My Linux distro is CentOS Linux release 7.9.2009 (Core)
Here is the value of LD_LIBRARY_PATH and the libraries in in /usr/local/lib
$ sudo export -p | grep -i ld_libr
declare -x LD_LIBRARY_PATH="/usr/local/lib"
$ cd /usr/local/lib
$ ls -l
total 0
lrwxrwxrwx 1 root root 13 29 août  11:20 libssl.so.1.0.0 -> libssl.so.1.1
$

Please can you help me ?
Thanks a lot

Oh!  This looks a bit mysterious:

$ cd /usr/local/lib
$ ls -l
total 0
lrwxrwxrwx 1 root root 13 29 août  11:20 libssl.so.1.0.0 -> libssl.so.1.1
$

So there's a symbolic link "libssl.so.1.0.0" pointing at "libssl.so.1.1"....  which doesn't exist?
That's one answer why you're getting errors.
Did you build this OpenSSL installation yourself?

Hello levitte,
thanks for your response.

So there's a symbolic link "libssl.so.1.0.0" pointing at "libssl.so.1.1".... which doesn't exist?

I don't know why I didn't get it !!! May be because I was focused on the error message.
No I didn't installed openssl by myself.
The virtual machine is a docker-engine installed by our dev team. I just have to renew a certificate and I want to check certificate attributes with openssl. When I launched it I've have got the error.
Today I'll try to work arround this symlik issue and post a new comment in this post.

Great 👍
Removing broken symlink => OK
Installing openssl11-libs => OK
$ sudo openssl version
OpenSSL 1.1.1m  14 Dec 2021 (Library: OpenSSL 1.1.1k  FIPS 25 Mar 2021)

$ sudo openssl version
OpenSSL 1.1.1m  14 Dec 2021 (Library: OpenSSL 1.1.1k  FIPS 25 Mar 2021)

This is still quite interesting output as the openssl binary apparently runs against a different openssl library build - the version there is different.

Having same issue on Arch Linux and its derivates. There is after OpenSSL upgrade: openssl 3.0.7-2 currently.

Everyone, sorry for mentioning the issue by mistake.
On Arch, I installed the package openssl-1.1 and the error went away.
After the fix, the version installed on Arch remains 3 as show bellow:
$ sudo openssl version
OpenSSL 3.0.7 1 Nov 2022 (Library: OpenSSL 3.0.7 1 Nov 2022)

Great 👍
Removing broken symlink => OK
Installing openssl11-libs => OK
 $ sudo openssl version OpenSSL 1.1.1m  14 Dec 2021 (Library: OpenSSL 1.1.1k  FIPS 25 Mar 2021)


Wireless mouse battery not detected by the battery module and can't be added manually

I got a wireless mouse with a battery and you can manually look the voltage and stuff with cat just fine in its sysfs folder.
hal@arch /sys/class/power_supply % ls -lh
lrwxrwxrwx root root 0 B Mon Jan 11 04:38:38 2021   hidpp_battery_0@ ⇒ ../../devices/pci0000:00/0000:00:07.1/0000:0c:00.3/usb5/5-3/5-3:1.2/0003:046D:C539.000B/0003:046D:407F.0010/power_supply/hidpp_battery_0

However an unconfigured battery module will detect no batteries and any attempt at telling it to use "hidpp_battery_0" will fail.
This can be worked around with scripting, but would be cool if wireless mice and other peripherals worked out of the box.

Note that this mouse only reports battery voltage and not a fixed percentage so it could definitely be crashy if it was detected (or maybe it is getting detected and failing gracefully)

Note that this mouse only reports battery voltage and not a fixed percentage so it could definitely be crashy if it was detected (or maybe it is getting detected and failing gracefully)

My HID++ 2.0 mouse (Logitech MX Master 2S) has these sysfs interfaces.
status = Charging, Discharging, Unknown
capacity_level = Critical, Low, Normal, Full, Unknown
capacity_level is only available when status is Discharging.
status and capacity_level are unknown when the device goes into low power mode after a short period of inactivity.

My HID++ 2.0 mouse (Logitech MX Master 2S) has these sysfs interfaces.
status = Charging, Discharging, Unknown
capacity_level = Critical, Low, Normal, Full, Unknown
capacity_level is only available when status is Discharging.
status and capacity_level are unknown when the device goes into low power mode after a short period of inactivity.

Similar issue with my Logitech M590 mouse and K780 keyboard connected using the UnifyingReceiver.
Using upower -e | xargs -n1 upower -i I'm able to enumerate existing batteries and see the values reported by the mouse and keyboard, but unable to get them reported by waybar-battery.

On Linux 5.15 rc1 there's support for (at least) my mouse and it now shows proper capacity in sysfs
/sys/class/power_supply/hidpp_battery_0/ cat capacity
68

The module even shows up in waybar but as 0%

Something is being weird here

I own a Logitech g903. It should be detected properly, but instead I get a

Today, waybar showed the correct % of the mouse, but after a while it started going to 0%, 100%, disappear... But it's the first time it has actually shown the percentage.
The kernel sysfs interface is probably weird, some fix got there to 5.16 but it's not fixed completely

Similar issue with my Keychron K8

Arch
Keychron K8 (/sys/class/power_supply/hid-dc:2c:26:26:48:e8-battery)

Same on Fedora. I'm using Keychron K6
cat /sys/class/power_supply/hid-dc:2c:26:ef:0f:4a-battery/capacity
91

the module shows 0 or 100, randomly.

Fixed at some point by what I presume is the kernel. Working perfectly on my system for a while.


Correct calc documentation

Calc works on any field that is present in the struct, since all fields are extracted at the start.

I don’t think this is a correct change? Codegen does not reorder calc to occur after all the other fields are read so I am confused by the comment in the commit. Only when writing can you access all fields of the structure, which is what the documentation says now. You can’t do this:
pub struct A {
    pub a: u8,
    #[br(calc(a + c))] // wrong, `c` does not exist yet
    pub b: u8,
    pub c: u8,
}

It will error:
error[E0425]: cannot find value `c` in this scope
  --> binrw/tests/example.rs:10:23
   |
10 |         #[br(calc(a + c))]
   |                       ^
   |                       |
   |                       a field by this name exists in `Self`
   |                       help: a local variable with a similar name exists: `a`


emoji: Match emoji sequences in markdown.

Fixes #11767.
Previously multi-character emoji sequences weren't matched in the emoji regex, so we'd convert the characters to separate images, breaking the intended display.
This change allows us to match the full emoji sequence, and therefore show the correct image.

Note that this doesn't let us support skin tone modifiers, which is an issue being tracked in #992

@andersk do you have any recommendations for the mypy situation?

current status:

I think this last push fixes the mypy issues (thanks so much @PIG208!)
There are 5 failures in zerver.tests.test_markdown.MarkdownTest currently, which correspond to an actual bug where typing multiple emoji only does the right thing with the first one. I tried a few things but haven't figured it out yet

I believe I've addressed all comments and failing tests now

This lgtm except that I didn't carefully read the discussion in https://github.com/zulip/zulip/pull/26364#discussion_r1297901744 and analyze whether the results for the new tests are correct (I see we do have those tests), and that it needs a PROVISION_VERSION bump after rebasing.

@gnprice can you confirm if the tests look good to you?

bumped PROVISION_VERSION

Merged, huge thanks for doing this! It's very very nice to clear out this old messy issue.

🎉 🎉 🎉 🎉

@gnprice can you confirm if the tests look good to you?

(I see Greg responded "yes" inline, above: https://github.com/zulip/zulip/pull/26364#discussion_r1303588994.)


DX-1023: Add regex for base url

when an invalid url is passed to the client (without https like eu1-flying-whale-434343.upstash.io), the client still tries to call the endpoint but gets invalild URL error. Instead of stopping there, it continues retrying with exponential backoff.
For instance, in ratelimit, there is a 5 second timeout. Since the exponential backoff takes 12 seconds in total, error is never reflected to the console/network and the user simply gets the timeout response.
With this change, client raises an error right away and ratelimit doesn't return the default timeout response.

fixed CI tests in this PR aswell.
Here are the changes:

removed link from cloudflare-workers/package.json to fix cloudflare-workers-local test bdd5a66
updated bun.lockb file to fix nodejs-local test fc93463
bumped node version of vercel deployed tests, which didn't actually solve the issue 7bdd2b2. Updated nodejs versions of sdk-redis and sdk-redis-edge projects at Redis


feat(Add in RSS feed of latest sites)

I have ideas and plans. Would love to get an RSS feed of the latest.

It sounds interesting, let's see if we can do it soon.

@PaulKinlan https://pwa-directory.appspot.com/api/pwa/?format=rss&limit=100

curl https://pwa-directory.appspot.com/api/pwa/?format=rss&limit=100 works great thanks, one thing I noticed though is that if I view it through the web, the appshell hosts the page

I noticed it too. Created #466 sooner today to track it.

We fixed the app shell issue on #467 it should work once the SW updates.
@PaulKinlan does the way we set the content in content:encoded work for you? Please let us know if we should set it up differently.


Get tldr-bot marked as a bot account?

The probot-stale bot we have running has a special bot tag next to it's name. Could we do this for @tldr-bot too?

Yea I saw that, probot-stale is actually an integration. Not an account per-se. And this article mentions its fine to give an user account for bot purposes - https://help.github.com/articles/differences-between-user-and-organization-accounts/#personal-user-accounts


[screengrab] replace double escape with simple escape so command also works on Windows

adb command included double escaped string which doesn't seem to be necessary but caused problems on Windows.
closes #13060

Meh, there probably should be a test for that...
I'll look into this a bit more.

That might be hard to test because that is a shell command that is actually getting run on an Android emulator I believe 🙃

Hm, probably connected to the shell you are using :/

Windows:
C:\Projects
λ echo $foo
$foo

C:\Projects
λ echo \$foo
\$foo

macOS:
MacBook-Pro:FastlaneDemoIonic sujan$ echo $foo

MacBook-Pro:FastlaneDemoIonic sujan$ echo \$foo
$foo

New code now uses different command for Windows.
I also added comment that explains, as I am sure this is not common knowledge.
I checked the codebase (searched for \\$) for similar cases, but didn't find anything relevant.


Yarn Berry Turbo Prune minor incorrectness

What version of Turborepo are you using?
v1.5.1
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
First, congrats to @chris-olszewski on implementing this feature; it is a challenging engineering problem!
I tried to prune my pretty extensive monorepo, and it was so close to supporting --immutable. Here is the diff after running yarn:
-"@types/node@npm:*, @types/node@npm:>=12, @types/node@npm:>=12.12.47, @types/node@npm:>=13.7.0, @types/node@npm:^16.11.59":
+"@types/node@npm:*, @types/node@npm:>=12.12.47, @types/node@npm:>=13.7.0, @types/node@npm:^16.11.59":
  version: 16.11.59
  resolution: "@types/node@npm:16.11.59"
  checksum: b3a2986ef8f8216bb7fd1187dcf2e69aca3e6b6b5fdfd6abf4cf11c44e3cdd587fae875a28dc4ef257e2b369e2c5290642aff6a0a9d702f3385a78f561293259
  languageName: node
  linkType: hard

Where the usages of @types/node >= 12 are as follows:
"@commitlint/load@npm:>6.1.1":
  version: 17.0.0
  resolution: "@commitlint/load@npm:17.0.0"
  dependencies:
    "@commitlint/config-validator": ^17.0.0
    "@commitlint/execute-rule": ^17.0.0
    "@commitlint/resolve-extends": ^17.0.0
    "@commitlint/types": ^17.0.0
    "@types/node": ">=12"
    chalk: ^4.1.0
    cosmiconfig: ^7.0.0
    cosmiconfig-typescript-loader: ^2.0.0
    lodash: ^4.17.19
    resolve-from: ^5.0.0
    typescript: ^4.6.4
  checksum: c35f7c5d7a8e2812a62b2d10f304c4b4ab8754923b0e59c66f3d9ce5ff3ea84502539c905b82fafbe666b02db5e7d818c119764af5d46485532a8bb73dba0661
  languageName: node
  linkType: hard

"meros@npm:^1.1.4":
  version: 1.2.0
  resolution: "meros@npm:1.2.0"
  peerDependencies:
    "@types/node": ">=12"
  peerDependenciesMeta:
    "@types/node":
      optional: true
  checksum: 95ec2dc352ad4ffe7a8572676d0399e8cfe67bb3ebe03ce5b03ebb4d990527994528260ff47a7efedc1348a8ee46bfe782564127bd1566d4790e1189b0a1ed29
  languageName: node
  linkType: hard

It also got a patch slightly incorrect, but you noted that in the feature PR description. The patch does not seem to be related to the @types/node issue.
Expected Behavior
The prune to produce a lockfile that supports the --immutable flag.
To Reproduce
Unfortunately, I can't share the lockfile over here. If you're interested in a proper reproduction it will take some time to isolate the specific issue.
Best of luck!

Thanks for the report. Is there any chance you could get me the entries that reference the other versions of @types/node?

Are the entries for @commitlint/load@npm:>6.1.1" and meros@npm:^1.1.4 that you gave above from the pruned lockfile?

Pre-prune, there are no references to >=12, meros/commitlint post-prune.
I will try and put together a better reproduction if possible tomorrow/this weekend.
Thanks!

Hey @quinnturner, any chance you could get me all of the pre-prune lockfile entries that reference @types/node? I've been trying to repro this and haven't had any luck.

@chris-olszewski I have created a documented reproduction and invited you to the private GitHub repository. Expect an email for approval! The reproduction is on v1.5.6.

@quinnturner, thank you so much for the reproduction. I just put up #2269 which fixes 2/3 changes, the final one comes from the turbo not fully understanding how the resolutions section of package.json impacts yarn.lock. It's doable to fix this, but it'll be a more involved PR.

(Targeting 1.6.1 with this one.)

@chris-olszewski Appreciate the good work! We're also running into the same issue as mentioned above, with both resolutions and peerDependencies. This is currently blocking our transition into Yarn 2, are there any easy workarounds in the meanwhile?

I am closing this ticket now as the issues initially outlined have been resolved except for resolutions. For resolving the resolutions, there's #2791.


[DS-3457] Address tomcat hang when multiple solr shards exist (master)

Port changes for master branch

+1 by review


add archlinux support

I've no idea why there are timeouts on travis :(

I've had that issue for a few weeks now and haven't moved that piece to an allowed failure (or removed it) because I keep telling myself I'll make time to figure out what's going on, but haven't had time.
Thanks for such a complete PR. I'll try and get a new release on the forge soon.

Thanks!


Added zooming functionality

Thanks a lot for contributing to react-easy-crop :beers:
Before submitting the Pull Request, please:

write a clear description of what this Pull Request is trying to achieve
run yarn start and open the demo to see that you didn't break anything

We don't need this for the example.


Introduce a validating webhook for validation of extension resources

How to categorize this issue?

/area control-plane
/area usability
/area ops-productivity
/kind enhancement
/priority 3
What would you like to be added:
Introduce a validating webhook to execute the existing validation of extension resources in https://github.com/gardener/gardener/tree/master/pkg/apis/extensions/validation. The natural place to host it would be the seed-admission-controller.
Why is this needed:
Currently, the validation of extension resources in https://github.com/gardener/gardener/tree/master/pkg/apis/extensions/validation is not executed, so this validation is essentially dead code. It was not seen as high priority so far since the contract is internal to Gardener - extension resources are only created and updated by gardenlet and we could assume this is done correctly.
However, there are edge cases in which gardenlet could create an invalid resource or e.g. attempt to update an immutable field, and without actually validating this could have some negative consequences, see for example this discussion: https://github.com/gardener/gardener-extension-provider-aws/pull/362#discussion_r660405162.
/cc @timebertt @kris94

/assign

/cc @plkokanov

In addition to the extension resources mentioned above, the validating webhook should also take care to validate etcd resources. We had a recent case where if we had such a validation in place, we could have prevented a rather severe issue from happening.
Note that validation of etcd resources is currently missing in the etcd-druid repo, so the webhook would currently have nothing to call. I will open an issue in that repo to request that such validation is introduced.


test-runner now shells one node for each dtslint

It uses the new, local test-worker, which just calls cp.exec(node dtslint) for each dtslint, instead of reusing the same dtslint over and over again.
lodash and reactstrap still run out of memory, but they do that for individual calls to dtslint as well. I think multi-Typescript compiles in ExpectRule may need the same out-of-proc treatment.
Edit: No, types/lodash and types/reactstrap also run out of memory when just compiling with ts 3.4.

Actually, when I think about it, only lodash and react are seeing oomemory crashes on CI. lodash itself runs out of memory and reactstrap depends on react.
So I think the root cause is 3.4+ running out of memory on lodash and reactstrap. This PR isn't really needed.

Filed Microsoft/TypeScript#30973 to track this instead.


Topological Sort

Hi,
I would like to add Topological Sort Algorithm.

Please assign this to me as a GSSOC contibutor.

Please assign me to add Topolgical Sort Algorithm

I'm a contributor for GSSoC '23, I want to work on this issue. Please assign me.

Please assign me for this issue under GSSoC'23  as I would like to add the topological sort algorithm

Hi, I would like to add Topological Sort Algorithm.

@aaryahjolia Please assign this issue


Fix mislabeled header in entrypoint.sh

Both test mode results were labeled "Edit Mode".

Awesome! I've merged your changes.
Thank you for your contribution.


[stable/elasticsearch-exporter] fix non working servicemonitor

Which issue this PR fixes

fixes wrong usage of port for servicemonitor. port only allows the name of the port. targetPort would allow the port number. Without this change, the servicemonitor never gets scraped.

sorry is an duplicate of #11609


Improve page titles

This PR appends the site's name to every page title.
It also provides a means to edit page titles and descriptions for component pages, as discussed with Content Team.

On the PR site About and Get Started titles are "California Design System"
https://site-name-appended-to-titles.pr.designsystem.webstandards.ca.gov/about/
https://site-name-appended-to-titles.pr.designsystem.webstandards.ca.gov/get-started/

@zakiya Yep, that's currently the default if there's no title declared for a page.
This PR is part of a dance with Content Team where they're populating the data and I'm building stuff to display that data at the same time. Some pages just aren't ready yet.

@m-sullivan7 @ashleydraper @LizNobriga @extinah Question for y'all.
When there's no title available for a page, do we still want to offer a default title with the site name appended? Or just use the site name as title?
If no title for page...
page title = site-wide default page title | site name
OR
page title = site name
Let me know if I need to clarify here.

My vote is for 2:

page title = site name

However, if someone else can think of a default title they think is useful, I could be persuaded of 1!

I think option 2 works.
I can't think of a default title that would be appropriate to use for any page.


[Question] How can we prepare table_monitors_config?

Overview
I tried to monitor dbt tests with jaffle_shopw by following the documentation. But, I was not able to upload artifacts due to the lack of the destination table. If I am correct, we have to create the table table_monitors_config a head. But, the documentation doesn't describes table_monitors_config. How can we prepare the table?
Environments

Python 3.8
dbt 1.0.3
elementary 0.3.2.

Error message
09:22:03  Running 2 on-run-end hooks
09:22:33  1 of 2 START hook: jaffle_shop.on-run-end.0..................................... [RUN]
09:22:33  1 of 2 OK hook: jaffle_shop.on-run-end.0........................................ [OK in 0.00s]
09:22:33  2 of 2 START hook: elementary.on-run-end.0...................................... [RUN]
09:22:33  2 of 2 OK hook: elementary.on-run-end.0......................................... [OK in 0.00s]
09:22:33
09:22:33
09:22:33  Finished running 8 view models, 9 incremental models, 2 table models, 3 seeds, 17 tests, 3 hooks in 44.13s.
09:22:33
09:22:33  Completed with 2 errors and 0 warnings:
09:22:33
09:22:33  Runtime Error in model filtered_information_schema_columns (models/edr/metadata_store/filtered_information_schema_columns.sql)
09:22:33    404 Not found: Table sandbox-project:jaffle_shop_elementary.table_monitors_config was not found in location asia-northeast1
09:22:33
09:22:33    (job ID: d0f16aa6-b33d-493f-b0d1-8b934a090682)
09:22:33
09:22:33  Runtime Error in model filtered_information_schema_tables (models/edr/metadata_store/filtered_information_schema_tables.sql)
09:22:33    404 Not found: Table sandbox-project:jaffle_shop_elementary.table_monitors_config was not found in location asia-northeast1
09:22:33
09:22:33    (job ID: 03fc0b75-262f-49f8-8957-96b55268e9ac)

@oravi That's true. I tried out it with BigQuery, not Snowfrake.
I will leave another comment about the details of logs later. But, as far as I am concerned, table_monitors_config  is defined as a dbt source. I don't think it is created by dbt run. Am I correct?
https://github.com/elementary-data/dbt-data-reliability/blob/master/models/edr_sources/sources.yml#L7

@oravi This is the full log of dbt.
https://github.com/yu-iskw/jaffle_shop/blob/048fa29b53fb105b516d5e6634b5325dc315000a/logs/dbt.log

@yu-iskw thanks for sharing it! the sources are being created in the on-run-start hook. They are supposed to be created in the target.database under '<target.schema>_elementary'. According to the log it looks like the on-run-start hook ran successfully. Is there a reason you can think of that it won't be created there or the others models won't have access to it?
In any case, we are doing final tests on BigQuery today and changing the flow of the configuration tables to be simpler to eliminate completely issues like that. So it will be resolved in the next version that should be released in a day or two.

Hi @yu-iskw ! happy to update that we have just released a new version that supports BigQuery and also do not rely on the configuration tables anymore. The configuration tables are now removed and the code basically uses the dbt graph to extract the tests configuration.
Please note that we also updated the documentation so I'd recommend checking them out before upgrading, would love to hear your feedback and see if this resolves this issue.
Thanks!!

@oravi Thank you for sharing. I will give a try the new version. I will let you know how it is later.

Closing the issue, fixed on 0.3.8 and above :)


Create peer_review_PeterM.md

I am going to review you!

Filled this out Mike, I'll let you merge


gradients not supported?

I have this basic svg with the gradient in the rectangle:
<?xml version="1.0" encoding="utf-8"?>
<!-- Generator: Adobe Illustrator 17.0.1, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
<svg version="1.1" id="Слой_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
     width="104px" height="640px" viewBox="0 0 104 640" enable-background="new 0 0 104 640" xml:space="preserve">
<linearGradient id="SVGID_1_" gradientUnits="userSpaceOnUse" x1="52" y1="393.9754" x2="52" y2="1.6092">
    <stop  offset="0.6019" style="stop-color:#B5F4FF"/>
    <stop  offset="0.7487" style="stop-color:#ACF4FF"/>
    <stop  offset="0.9921" style="stop-color:#92F4FF"/>
    <stop  offset="0.9944" style="stop-color:#92F4FF"/>
</linearGradient>
<rect fill="url(#SVGID_1_)" width="104" height="640"/>
</svg>

What I end up with is this:

in PIXI.Graphics, and nothing is displayed.

@makc there is no easy way to fix this as the PIXI.Graphics API itself does not support gradients.

It's 2017, gradients are supported in Pixi now.

It’s 2019, svg consume to much gpu ressouces on mobile devices (5000+ paths)
So im looking for svg to raster with d3js and pixijs combination. Or does pixijs offer a solution for this already?


bugfix: kusion destrory tf resources requires prior state and plan state

What type of PR is this?

/kind bug
What this PR does / why we need it:
kusion destroy TF resources, return success, but still there
Which issue(s) this PR fixes:

Fixes https://github.com/KusionStack/kusion/issues/264
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 6 (83.33%)  changed or added relevant lines in 2 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 72.237%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/engine/runtime/terraform/terraform_runtime.go
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/graph/resource_node.go
5
61.01%

pkg/cmd/apply/options.go
7
71.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4673

Relevant Lines:
6469

💛  - Coveralls


Update xanzy to latest release

Description of your changes
Updated projects api. Added new fields for parameters and observations projects. Updated side for client and controller. Added missing tests.

Fixes https://github.com/crossplane-contrib/provider-gitlab/issues/24
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
manually

LGTM. Thanks a lot @Stanislav2017!


SyntaxError in node version under 8.3

I got a syntax error in nodejs6, said that spreading object is not valid syntax, which is only introduced since node8.3.
May this be fixed in the next version?

@hyurl As stated in the troubleshooting guide, only nodejs versions 8.9.4 and above are supported. However, assuming everything else works on older nodejs versions, backward support could be considered. I'll take a look and keep you posted.

As of better-sqlite3 version 5.2.1, backward support was added for all nodejs versions above v8.0.0.


Monorepoize Dahlia + Calyx

pretty much says it on the bill

Downsides:

We lost all of the nice issue and PR information from the Dahlia repo.

I'm closing this issue for now. We are not changing Dahlia quickly enough at the moment for this to make sense. If we start to again some point in the future, we can reconsider this.


Discord bridge automatically muted newly invited Matrix user

Hi there! Creating an issue for tracking purposes, also posted in the channel.
I ran into an issue where I bridged a room and invited 3 people, one of which got muted on join. The powerlevel for his matrix account is set to -1, and I am unable to edit that manually since the discord bridge bot is the admin of the channel. Running the fixmute command also does not fix the issue.
Edit: here's a log snippet where the user joined:
170337-Jan-20 23:09:11.767 [bot-sdk-Appservice] info: Processing transaction 241
170411-Jan-20 23:09:11.768 [bot-sdk-Appservice] info: Processing event of type m.room.member
170497-Jan-20 23:09:15.930 [bot-sdk-Appservice] info: Processing transaction 242
170571-Jan-20 23:09:15.931 [bot-sdk-Appservice] info: Processing event of type m.room.message
170658-Jan-20 23:09:15.939 [MatrixEventHandler] info: Got new message in !QxhbPcPPmcJhImCyFb:awful.club from @jowj:awful.club!
170778-Jan-20 23:09:15.940 [MatrixEventHandler] info: Handling text event with msgtype m.text...
170868-Jan-20 23:09:16.097 [DiscordPuppet:DiscordEventHandler] info: Received new message!
170952-Jan-20 23:09:16.106 [bot-sdk-MatrixLiteClient] info: (REQ-2) POST http://matrix-synapse:8008/_matrix/client/r0/rooms/!QxhbPcPPmcJhImCyFb%3Aawful.club/receipt/m.read/%24Mv2mf3xvRXBNfWGetHH5zJk2ySkV3hzalEdubZ0_Yy8
171164-Jan-20 23:09:49.054 [bot-sdk-Appservice] info: Processing transaction 243
171238-Jan-20 23:09:49.055 [bot-sdk-Appservice] info: Processing event of type m.room.member
171324:Jan-20 23:09:49.055 [MatrixEventHandler] info: Got new user join event from @me:micahrl.com in !QxhbPcPPmcJhImCyFb:awful.club...
171453-Jan-20 23:09:49.056 [bot-sdk-MatrixLiteClient] info: (REQ-3) GET http://matrix-synapse:8008/_matrix/client/r0/rooms/!QxhbPcPPmcJhImCyFb%3Aawful.club/state/m.room.power_levels/
171629-Jan-20 23:09:49.087 [bot-sdk-MatrixLiteClient] info: (REQ-4) GET http://matrix-synapse:8008/_matrix/client/r0/rooms/!QxhbPcPPmcJhImCyFb%3Aawful.club/state/m.room.power_levels/
171805-Jan-20 23:09:49.095 [bot-sdk-MatrixLiteClient] info: (REQ-5) PUT http://matrix-synapse:8008/_matrix/client/r0/rooms/!QxhbPcPPmcJhImCyFb%3Aawful.club/state/m.room.power_levels/
171981-Jan-20 23:09:49.250 [bot-sdk-Appservice] info: Processing transaction 244
172055-Jan-20 23:09:49.251 [bot-sdk-Appservice] info: Processing event of type m.room.power_levels
172147-Jan-20 23:10:04.984 [bot-sdk-Appservice] info: Processing transaction 245
172221-Jan-20 23:10:04.985 [bot-sdk-Appservice] info: Processing event of type m.room.message
172308-Jan-20 23:10:05.009 [MatrixEventHandler] info: Got new message in !QxhbPcPPmcJhImCyFb:awful.club from @jowj:awful.club!
172428-Jan-20 23:10:05.010 [MatrixEventHandler] info: Handling text event with msgtype m.text...
172518-Jan-20 23:10:05.185 [DiscordPuppet:DiscordEventHandler] info: Received new message!
--

Fixed by https://github.com/Sorunome/mx-puppet-bridge/commit/5a1cf352ddab1ced12adb782a875eb6dfdb771db per Soru!
Side note: For anyone running on older versions of the bridge and can't update for whatever reason, you can run !discord adminme in the channel and then you'll seize admin rights. You can then manually set the pl for the affected user as a work around.

Fixed by https://github.com/Sorunome/mx-puppet-bridge/commit/5a1cf352ddab1ced12adb782a875eb6dfdb771db per Soru!
Side note: For anyone running on older versions of the bridge and can't update for whatever reason, you can run !discord adminme in the channel and then you'll seize admin rights. You can then manually set the pl for the affected user as a work around.


Entering "Å" changes current cell in unexpected way

With a fresh version from trunk I edit exolve.html just to add a line exolve-language: sv Latin after the line with exolve-id.
Then I open the file in Firefox and click in the top left corner of the crossword.
I press the "Å" key on my (Swedish) keyboard. An "Å" is entered as it should be, but then the top right corner becomes the current cell!

I expected the "Å" to just be added without any jumping around, so the middle north cell should be current. (Other "special" letters "Ä" and "Ö" seem to work fine.)

Hi,
I tried to replicate the issue but I haven't been able to, so far. I tried using Firefox v79.0 on Linux using a test puzzle that I have copied below.
Can you please let me know which Firefox version you are using (if older than v79.0 then perhaps try updating?). Perhaps you can share the exolve puzzle specs too.
Thanks!
Viresh
exolve-begin
exolve-id: test-swedish
exolve-title: Test-Swedish
exolve-width: 3
exolve-height: 3
exolve-language: sv Latin
exolve-prelude:
Test entering  Å Ä Ö
exolve-grid:
Å Ä Ö
Å . Ö
Ä Ä Å
exolve-across:
1 aao (3)
3 aaa (3)
exolve-down:
1 aaa (3)
2 ooa (3)
exolve-end

Thanks for looking into this! I've found what it is. The handling of character codes and keyboard codes is a bit strange. When I press the "Å" key on a Swedish keyboard that generates an event with keyCode 219, which is the same as a "[{" key (according to this. But that is what Exolve converts shifted TABs to so it's also handled as that.
I fixed it for myself by using some other replacements for the TAB key – I used:
    key = e.shiftKey ? 34 : 33
that is PageDown and PageUp, which should be safe.

I'm glad that you've figure this out!  I'll fix the code to not convert shift-tab into some character, so that you do not have to hack around that. Will update this bug when I do.
If you've put up a crossword in Swedish on the web using Exolve, I would love to take a look!
Thanks,
Viresh

Thanks! I found it hard to find documentation on how this is supposed to work. For example this question is essentially left unanswered. I rather liked making PageUp/Down work as I did.
Since you asked, I discovered this regarding a crossword originally on paper in a newsletter, now "exolved" here. I have been a bit lazy there because there are pictures to some of the clues, and ideally I should make it so the corresponding image would be shown prominently when looking at one of those clues. (BTW, regarding localization a minor feature request would be to make "a" and "d" for across/down be configurable.)

Thanks for sharing that link--the page looks lovely and makes me happy!
I'll add customization of 'a' and 'd' via exolve-relabel in the next release.

With v0.85 that I checked in a little while ago, the "shift-tab" issue has been fixed.
I did not get to customization of 'a'/'d' yet. Will try to do it asap (feel free to open a bug for it).
Thanks!

With v0.88 that I checked in just now, you can customize the a/d. You can also change every single piece of text (please let me know if I missed something!). You can do something like:
exolve-relabel:   across-letter: q   down-letter: p
Complete details at https://github.com/viresh-ratnakar/exolve#exolve-relabel

Closing out old bugs that I did take some action on. If I haven't fixed every aspect as expected, apologies, but please open a new bug (hard for me to parse out what remains to be fixed).


Missing properties are not reported clearly

I had a reference to a property in my state.yaml:
replication: ${my_data_default_replication}

which I had failed to define in my props file.
The exception message is not very helpful.
[INFO] reading props from ../config/prod.props
java.lang.NullPointerException
at com.devshawn.kafka.gitops.service.ParserService.parseStateFile(ParserService.java:84)
at com.devshawn.kafka.gitops.service.ParserService.parseStateFile(ParserService.java:42)
at com.devshawn.kafka.gitops.StateManager.getAndValidateStateFile(StateManager.java:72)
at com.devshawn.kafka.gitops.cli.ValidateCommand.call(ValidateCommand.java:26)
at com.devshawn.kafka.gitops.cli.ValidateCommand.call(ValidateCommand.java:15)
at picocli.CommandLine.executeUserObject(CommandLine.java:1783)
at picocli.CommandLine.access$900(CommandLine.java:145)
at picocli.CommandLine$RunLast.handle(CommandLine.java:2141)
at picocli.CommandLine$RunLast.handle(CommandLine.java:2108)
at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:1975)
at picocli.CommandLine.execute(CommandLine.java:1904)
at com.devshawn.kafka.gitops.MainCommand.main(MainCommand.java:69)

@tgdavies Which version of kafka-gitops are you using?
The string "reading props from" doesn't appear in its sources anywhere. 🤔
ParserService.java:84 is about extracting the exception root cause and unfortunately offers quite some opportunities to fail with a NullPointerException:
https://github.com/devshawn/kafka-gitops/blob/0.2.15/src/main/java/com/devshawn/kafka/gitops/service/ParserService.java#L84
Would you be able to share your desired state file, the properties file, and the exact command you're running?


Server-api-creation

Added server and staff module to manage endpoint requests.

Added server endpoint and staff modules


Add feature for crazyflie_sim sil pose publisher

Hi there, thank you for this amazing package, since I am using the crazyflie sil especially planning for reciprocal avoidance, here is a way to broadcast the poses in the sim, just a quick hack. thank you :)

Added publisher timer and now the publisher does not interfere with the main loop and also it will have its own pub interval

Hi @matthewoots, thanks for your contribution! I was wondering if/why your use-case is not covered by the existing visualization for rviz (available at https://github.com/IMRCLab/crazyswarm2/blob/main/crazyflie_sim/crazyflie_sim/visualization/rviz.py)? I see that you use poses, rather than TF, but both work fine for rviz. If we want to use pose, it should be part of the visualization folder, and not directly in the simulator.

Thanks for letting me know about it @whoenig and for your inputs
The idea was that the pose of the CF in real hardware was published as name/pose and whereas in the simulator we can only use TransformStamped compared to PoseStamped, so it was more of the sake of consistency and also some planners would be using PoseStamped (though both are very similar)
for state, (_, cf) in zip(states_next, self.cfs.items()):
    cf.setState(state)

for vis in self.visualizations:
    vis.step(self.backend.time(), states_next, states_desired, actions)

Since visualization is in the main loop, are you saying that I should add the timer inside the rviz.py instead, since both used states_next there would be no difference

I think it should be a new visualization module (and potentially we rename rviz to something more descriptive, like ros_tf and yours ros_pose), so it can be easily enabled/disabled
Rather than a timer (which wouldn't operate on sim time), I would add a if-condition in the visualizer callback to only conditionally publish the new states based on t (and the user-setting)

I'm closing this due to inactivity but if you want to work on it again, just reopen it again. This is just to keep the PR list clean


Feature Request: Add Ability to specify multiple PRs using "vsts code pr --id"

When we prep for a code release, we identify a bunch of PRs to complete, merging into our mainline branch.  I'd like to be able to specify a list of PRs using "vsts code pr" so that I can monitor the state of each of the PRs over time.  Ideally, I'd do something like
vsts code pr --id 123,456,789,etc --output table
and be able to see which PRs are completed and which ones still need attention.
thanks
--Bill

Wow thanks for this

With the new azure devops extension, wouldn't az repos pr list with either the source or target branch filter work for your use case? This way, you wouldn't have to even remember IDs. Further, in case you really want to explicitly want to filter out IDs from this result set, you could use JMES path query to filter down on the result set.  Let me know if this still does not suffice your requirement.


Update Development Environment

Update development environment with the latest link checker service and latest versions of the dependencies

This component is deprecated


Add arguments to map functions

Hello,
Not sure if this is possible already or is normally done in some other way, but it would be nice to add variadic arguments to registered Map Functions. For example, currently if I need to filter for a date range, I need to register a function for those specific days, and I need to know which field to use from the row.
If I could make a function that takes start_days_back and end_days_back, and date_position, I could make a more generic reusable function to do the filter that doesn't even need knowledge of the data set, but could just pull the field, compare it to the dates from the current date, and emit the result if it matches.
I think reducers could use the same functionality, but I've not run into that situation yet.

you can use a map object, so that you can "select" the values you want.

Thanks, for others that may come on this, I found a way that you can actually achieve what I was wanting, just not dynamically, using a higher order function:
type RowType map[string]interface{}
func MakeMapper(fields string) gio.MapperId {
	f := func (row []interface{}) error {
		fieldNames := strings.Split(fields, ", ")
		out := make(map[string]interface{})
		for i, f := range row {
			out[fieldNames[i]] = f
		}
		gio.Emit(out)
		return nil
	}
	return gio.RegisterMapper(f)
}

var fields = "event_type, aid, time, cid, ctaid, abhash"
var mapper = MakeMapper(fields)

You could do the same with position args, additional arguments like start_days_back, etc.


Optimize unique_op by repetition rate

My test about unique op:
Input: size:201000k, uniqued size:51000k.
cmp test:
Origin: The initial size of the hash table is 2input_size
My test: The initial size of the hash table is 1input_size(change source code"tensorflow/core/kernels/unique_op.cc").
cmp result:
Origin: spend: 2.517s
My test: spend: 1.709s
I think unique_op should be added a default params of repetition rate, default value is non-repeating(hash table size is 2*N)。Repetition rate will help us do some prediction, like: Adjust the initial size of the hash table to improve performance。
If the user does not know the repetition rate of his data, he should use default value.
If the user knows the repetition rate of his data, he will adjust this args to get a better performance.
Do you think this view makes sense? If it makes sense, I would be happy to discuss & implement this part of the code. ^v^
My test code:
from __future__ import print_function
import random
import tensorflow as tf
import time

size = 20 * 1000000
sum=0
keys, idx = tf.unique(tf.random_uniform(shape=[int(size)], dtype=tf.int64, maxval=int(size/5)), version='v2')

sess = tf.Session()
with tf.Session() as sess:
    for i in range(0,10):
       start = time.time()
       _ = sess.run(keys)
       sum+=(time.time() - start)
    print("avg time:", sum / 10);

I suggest repetition rate is a range of [0, 1), 0 is non-repeating, 1 is completely repetitive.
hash_table_size = 2 - repetition_rate.
If user use default value(non-repeating), hash_table_size is origin(2 * input_size).
If user use default value(my test, 0.75), hash_table_size is (1.25(ps:2-0.75) * N).
In order to solve the problem, we need to find a way to know whether the hash table has been expanded. If there is an expansion, a warning will be issued to notify the user that the wrong usage method may cause performance degradation.

Heuristic hash table size maybe a good idea ~

copy that. @kkimdev

Q: Is this request from a real world use case? If so, do you mind sharing with us?  I'm trying to gauge the importance of this and if it'll be useful more broadly.  Thanks!

Scene: Ps save embedding hash table, and workers get embedding by features.
We need unique_op to filter repeated features(hash key) for decreasing net pressure and decreasing lookup embedding hash table times.
Now we search 2,000,000 features in hash table once(like my test code).
We will search more keys in the future.
Data:The data cannot be given to you due to its sensitivity, but the characteristics of the data are the same as in the example.


[PYTHON-DIST] Distributed xgboost python training API.

see https://github.com/tqchen/xgboost/tree/master/tests/distributed for example.
Running distributed xgboost python code.


[common] TorrentParadise.ml integration

A distributed torrent index (stored on IPFS)
https://torrent-paradise.ml/
Let's do an initial integration with the website first. Ideally integration via IPFS would be a great solution to provide search results in countries that censor torrent websites.
Ideally this index exists as a database file (.sqlite) and diffs of such file that frostwire clients could fetch via IPFS, or better yet via BitTorrent.

Source Code: https://github.com/urbanguacamole/torrent-paradise

Search URI: https://torrent-paradise.ml/api/search?q=<url encoded keywords>
Sample JSON output:
[
  {
    "id":"90d8bef8b5007ed7b3978dda8b536e57438fa32d",
    "text":"Martin Luther King Jr - Beyond Vietnam - A Time to Break Silence",
    "len":18651272,
    "s":0,
    "l":0
  },
  {
    "id":"7697d80cd14cc69139e9a51e466da14f5fb6f4f5",
    "text":"Martin Luther King.pdf",
    "len":26635616,
    "s":0,
    "l":0
  },
  {
    "id":"ce685482f866700a60ce4ebb2e9d3df043c137a9",
    "text":"Martin Luther King - I Have A Dream Speech August 28, 1963",
    "len":30329367,
    "s":1,
    "l":0
  },
  {
    "id":"2f9c8f4c9d295f9438f3fd58269ef0486b854a3b",
    "text":"Martin Luther King 'I have a dream'.avi",
    "len":186640384,
    "s":0,
    "l":0
  },
 ...

torrent-paradise.ml author here
Just a nitpick: there's the centralized version (has a HTTP API) and the IPFS version (no API, slow, you would have to use a library to download the parts of the index you need and then sort the results on your machine since IPFS is all static).
I noticed that in the first post you say stored on IPFS and then give an example with the API -- these are two different services, the API can go down.

Hi there @urbanguacamole glad to see you here, love your initiative. Torrent Search needs to be decentralized.
I was looking at your golang code and I see you use postgresql when you're indexing torrents from the DHT.
I didn't see any special postgresql features being used, saw pretty much 2 tables, one for telemetry, another for the torrents.
I'm wondering how to go about making your crawler configurable to so that I can use a sqlite database file instead which would be a lot more portable, also thinking how to go about creating diffs of such file that could be shared on a torrent feed.
In the end all we'd need to make this index distributed would be either having snapshots of this database synced across all nodes (simple approach), or to split that table into N shards, say 0-255 shards if the current DB is getting too big, and each shard is made up of records that point to torrents whose info hash end on a certain byte 0x00, 0x01, ..., 0xFF, and then FrostWire could use a chatty p2p protocol to ask peers in a group (peers meet over DHT keys) for torrent search results, peers send matched search results from their local copy of the shard.

@urbanguacamole is there a publicly accesible pg_dump snapshot SQL file torrent of the database?

@gubatron I use two tables: torrent and trackerdata. One stores the seed/leech counts for each infohash for each tracker.
Then, I periodically rebuild two materialized views: fresh (stores the highest seed/leech count for each torrent) and search (builds upon fresh, JOINs torrent names from torrent and seed/leech counts from fresh and has a tsvector column (for fulltext search).
The telemetry table is so far write-only, it's there because I might implement machine learned result sorting in the future.

split that table into N shards, say 0-255 shards if the current DB is getting too big, and each shard is made up of records that point to torrents whose info hash end on a certain byte 0x00, 0x01, ..., 0xFF, and then FrostWire could use a chatty p2p protocol to ask peers in a group (peers meet over DHT keys) for torrent search results, peers send matched search results from their local copy of the shard.

What you're describing is essentially the ipfsearch approach. Read the first paragraph here to get the gist of it. Both the inverted index and the index are split up into shards. The client knows which shard to look at by first downloading the meta file. For a visual explanation, open up the Network tab in Devtools on the IPFS version and enter a common search term.
The equivalent to a chatty p2p protocol is so far IPFS -- once a node downloads a piece, it starts providing it to the rest of the network.

is there a publicly accesible pg_dump snapshot SQL file torrent of the database?

I can make you one, what exactly are you looking for? (which tables) There's a .csv of name, infohash, seed, leech that should suffice for most users.

Also, don't use my spider to spider the DHT, it's stupidly slow and not good for the network and clunky. I don't use it myself anymore, I haven't deleted it yet though. Use magnetico (uses SQLite), or just use the magnetico community dumps.

The telemetry table is so far write-only, it's there because I might implement machine learned result sorting in the future.

Wonder what your thoughts are mixing your telemetry data, torrentdata + GPT-3


Force RemoveWires to retain clock and analog wires

This fixes #749 and #767 (#767 has a minimal non-working example that breaks on master).
The change is the simplest possible fix of filtering all clock wires in addition to all analog wires. It would be possible to be more selective, but this would slow compile time with an extra pre-processing scan and provide marginal benefit in emitted verilog.
One interesting thing to note is that the original bug in RemoveWires would also theoretically affect reset wires, but the pass is run after RemoveReset. As a result, this pass ordering is required for correctness. I can add more code to handle the reset case if this ordering constraint is undesirable.

@seldridge, you raised the valid concern about SSA violations in Low FIRRTL. This change is neutral to that concern, so I think the discussion about that issue should continue. Once the dust settles on that, this fix may become irrelevant, but in the short term it's probably useful for getting 1.2.0 out.

@albert-magyar: I'm fine to have the discussion about SSA later, in a new issue. There's the broader notion of Low/Mid really should be checking for SSA, but they aren't and the assumption in this pass that SSA is possible with all register declarations preceding node declarations is unfounded.
However, a quick fix, like what you've shown here (ignoring clocks like with analog wires), makes total sense to me. I do think that a more substantial fix could make it into 1.2.0 if its deemed that there is some real value in stripping every possible wire (and considering how many other issues are in that queue already).
Thanks for getting this together! (I was blocked similarly to @chick with this.)

I added a double compilation test case that fails under master and passes with this change.

Unfortunately, this doesn't work if the wire is not of type Clock but is still cast to a Clock for use by a register, eg.
circuit test :
  module test :
    input ext_clock : Clock
    input in : UInt<1>
    output out : UInt<1>

    wire int_clock : UInt<1>
    reg r : UInt<1>, asClock(int_clock)
    int_clock <= asUInt(ext_clock)
    r <= in
    out <= r

I'm thinking about solution

Resolved by https://github.com/freechipsproject/firrtl/pull/823


VSCode not showing progress from Language Server

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

VS Code Version: 1.82.2 (system setup)
OS Version: Windows 11

Steps to Reproduce:

Make a language server, connect it to VS Code, and send the following messages to the editor

{"jsonrpc":"2.0","method":"window/workDoneProgress/create","params":{"token":"MyCoolAnalysis"}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"begin","message":"1 file","percentage":0,"title":"Analyzing"}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"report","percentage":0}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"report","percentage":100}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"kind":"end"}}}

Expected result: progress shows on the bottom left or somewhere similar
Actual result: no progress

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.82.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I've also tried with version 1.83.0-insider without any configuration to make sure that I didn't disable progress with a setting.
I know that VSCode is still communicating with the server, because the method window/showMessage is working. For example, the following messages are working.
{"jsonrpc":"2.0","method":"window/showMessage","params":{"type":3,"message":"Analyzing"}}

{"jsonrpc":"2.0","method":"window/showMessage","params":{"type":3,"message":"Done analyzing"}}

Due to UI guideline in VS Code the progress is shown as a notification progress to avoid too much flickering. The progress only becomes visible on user action or when it is long running.

Is trigger by the following code
	const progress = await connection.window.createWorkDoneProgress();
	progress.begin('Validating', 0, 'happy coding', true);
	let counter = 1;
	let interval = setInterval(() => {
		if (counter === 101) {
			clearInterval(interval);
			progress.done();
		} else {
			progress.report(counter++);
		}
	}, 1000);
	progress.token.onCancellationRequested(() => {
		progress.done();
		clearInterval(interval);
	});

I am closing the issue since this is the current expected behavior.

Okay, in that case, then part of the initialize method of the client is non-compliant with the protocol. The value of workDoneProgress should be false.
// https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#clientCapabilities
{
	// ...
	"method": "initialize",
	"params": {
		"capabilities": {
			// ...
			"window": {
				// ...
				"workDoneProgress": true
				// This should be false.
				// (It indicates whether the client supports server initiated
				// progress using the `window/workDoneProgress/create` request.)
				// https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#serverInitiatedProgress
			}
		}
	}
}

The client fully supports server initiated progress. It is a choice of VS Code how to present it in the UI. They do as a notification, which from a spec point of view is OK.

I think that there's a misunderstanding. There's no problem with having the progress as a popup notification instead of on the status bar.
My problem is that I don't see any progress at all. I don't see the popup notification on the bottom right. At first I assumed that server-initiated progress was not possible, because it appeared that my 5 messages were following the spec.
Now that I know it's possible, I assume that my 5 messages are not correct or they are missing other message(s).
I am using the npm package vscode-languageserver and sending these 5 messages to it from the server, yet I don't see any progress.
{"jsonrpc":"2.0","method":"window/workDoneProgress/create","params":{"token":"MyCoolAnalysis"}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"begin","message":"1 file","percentage":0,"title":"Analyzing"}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"report","percentage":0}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"cancellable":true,"kind":"report","percentage":100}}}

{"jsonrpc":"2.0","method":"$/progress","params":{"token":"MyCoolAnalysis","value":{"kind":"end"}}}

I traced the communication with the test I did and it looks like this:
[Trace - 9:48:29 AM] Received request 'window/workDoneProgress/create - (3)'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77"
}

[Trace - 9:48:29 AM] Sending response 'window/workDoneProgress/create - (3)'. Processing request took 0ms
No result returned.

[Trace - 9:48:29 AM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "begin",
        "title": "Validating",
        "percentage": 0,
        "message": "happy coding",
        "cancellable": true
    }
}
[Trace - 9:48:30 AM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "report",
        "percentage": 1
    }
}

[Trace - 9:48:31 AM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "report",
        "percentage": 2
    }
}

...

[Trace - 9:50:09 AM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "report",
        "percentage": 100
    }
}

[Trace - 9:50:10 AM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "end"
    }
}

which actually looks like yours.
You could try to debug this be setting a breakpoint in this file vscode-languageclient/lib/common/progressPart.js around here:
            switch (value.kind) {
                case 'begin':
                    this.begin(value);
                    break;
                case 'report':
                    this.report(value);
                    break;
                case 'end':
                    this.done();
                    done && done(this);
                    break;
            }

That code is never called for me because the constructor for ProgressPart is not called. In the file vscode-languageclient/lib/common/progress.js the function ProgressFeature.initialize is called, but it's variable createHandler, which should construct the ProgressPart instance is not called.
It seems that somewhere the handler is supposed to be found by _requestHandlers.get or _pendingRequestHandlers.get or _requestDisposables.get to then be called, but I'm not sure where it is.

I found out how to log the incoming/outgoing requests/notifications for the client. The $/progress notification shows but the window/workDoneProgress/create request doesn't show. In addition a method I made up called fake/method doesn't show. I'm sure that my language server is sending these messages. I switched to TCP and watched the message with Wireshark. The content length is correct. Therefore, they seem to have been skipped by the client before logging.
[Trace - 3:44:40 PM] Sending notification 'workspace/didChangeWatchedFiles'.
Params: {
    "changes": [
        {
            "uri": "(uri omitted here)",
            "type": 2
        }
    ]
}

[Trace - 3:44:41 PM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "cancellable": true,
        "kind": "begin",
        "message": "happy coding",
        "percentage": 0,
        "title": "Validating"
    }
}

[Trace - 3:44:41 PM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "report",
        "percentage": 0
    }
}

[Trace - 3:44:41 PM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "report",
        "percentage": 100
    }
}

[Trace - 3:44:41 PM] Received notification '$/progress'.
Params: {
    "token": "d3b60f14-ab40-49c4-bd1f-ee68ed70dd77",
    "value": {
        "kind": "end"
    }
}

When do you call them. According to the spec you are not allowed to use window/workDoneProgress/create during the initialize request. There is a special progress token passed as a parameter if a server wants to report progress during the initialize request.

No, I create the progress a long time after the initialization handshake. I
do it manually when the user saves. I haven't implemented an analysis after
the client's "initialized" notification yet.
I even tried sandwiching my "window/workDoneProgress/create" between 2
different popup windows "window/showMessage" and both of those messages
were displayed. The progress request was just ignored.
I think that somehow the handler for "window/workDoneProgress/create" is
being removed at some point. I can try to create middleware for it, but the
middleware only works if I create a handler with the client.onRequest
method.
On Fri, Sep 22, 2023, 3:45 AM Dirk Bäumer @.***> wrote:

When do you call them. According to the spec you are not allowed to use
window/workDoneProgress/create during the initialize request. There is a
special progress token passed as a parameter if a server wants to report
progress during the initialize request.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/193138#issuecomment-1731129643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIYXFLMZVDWXHL7CVGRWY3X3VM3DANCNFSM6AAAAAA4YRTF7U
.
You are receiving this because you authored the thread.Message ID:
@.***>

The onRequest method is called and clear is not called.
I don't know what's wrong. For my language server extension. I'm fine with making my own handler in this case. My new handler works similar to the progress of rust-analyzer, which has a 2 status bar items: a minimalistic one with the status and commands and a verbose one with the analysis details. If my users encounter any flickering, they can just hide the verbose one.
Thank you for all the help. I can continue to help in the future if you want me to, in order to prevent his bug from happening to someone else.

Do you every hit the create handler? https://github.com/microsoft/vscode-languageserver-node/blob/a705ba5e4d32b1583ac7f2daa8c71652d8bf1f93/client/src/common/progress.ts#L42

No. That constant createHandler is assigned, but it's not called.
Here's some more information. After I initialize the client, I know that the handler still exists.
async function activate(context)
{
	// ...
	await client.start();
	client._requestHandlers.has('window/workDoneProgress/create');    // true
	client._requestDisposables.has('window/workDoneProgress/create'); // true
}

I actually suspect that the message for some reason never reaches the client. Can you try to set a breakpoint around here: https://insiders.vscode.dev/github/microsoft/vscode-languageserver-node/blob/main/jsonrpc/src/common/connection.ts#L773 to see if the request makes it to the client and whether it is dispatched correctly.

Great idea! That solved my problem. 🤦♂ I can't believe I didn't double check the documentation for a request. Every request must have an id, otherwise it will be considered a notification. My problem was that I didn't have an id. Thus, handleRequest wasn't called.
I guess I was just confused because I didn't see it in the logs.

Super to hear that you could solve the problem.


TypeError: n._handle.getAsyncId is not a function

Describe the bug
I developing  a telegram bot using nodejs. Actually when I tried on local, it is working fine but it is causing bug in stackblitz.
Error:
TypeError: n._handle.getAsyncId is not a function
    at responseKeepAlive (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:6:761706)
    at IncomingMessage.responseOnEnd (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:6:762037)
    at IncomingMessage.EventEmitter.emit (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:6:126312)
    at endReadableNT (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:6:489841)
    at https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:11:125491
    at https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:11:158166
    at _0x31eff0 (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:11:158411)
    at MessagePort._0xe4431a.port1.onmessage (https://github-ej3k7n.jw.staticblitz.com/blitz.84452ea99b8c6a79e19cae2129beaf85d5da007c.js:11:158524)

Link to the blitz that caused the error
https://stackblitz.com/edit/github-ej3k7n
Steps to reproduce
npm install
node index.js
Expected behavior
The server should run without error
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 89.0.4389.90
Major version = 89
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.90 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 72910662,
  "usedJSHeapSize": 57096194,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 84452ea99b8c6a79e19cae2129beaf85d5da007c

Additional context
No response

Hey there 👋 Thanks for reporting this issue. It's indeed a bug. We'll add this to our list and get back to you once it's been fixed.

@Suryaprakash1024 Sorry it took so long, but I think this is no longer an issue. Feel free to file a new one if you find something else off.


Trigger Email SSL issue

I set up a new Trigger email extension configured for mailgun.  My mailgun credentials work with this tool: https://www.smtper.net/
This is what my smtp uri looks like:  smtps://{username}:{passoword}@smtp.mailgun.org:587
when creating a new document this is the response I get:
Error:<PHONE_NUMBER>1056:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:332:
I'm not seeing an issue with outside of the extension, and I am not finding much on my google search.

Hi @jasonspick, have you tried using port 465 instead of 587?

@jasonspick I have the same problem moving from MailGun to Office 365. I was using 465 for MailGun and move to 587 with office. I thinks 587 port seems to be using a ord ssl version so the current version of Trigger Email have error communicate to it. Moving from 587 to 465 will fix your problem.

You can find more info for STMPS and 465 at : https://www.mailgun.com/blog/which-smtp-port-understanding-ports-25-465-587/#:~:text=Port 587%3A,587 as the default port.

We can mark this closed now. I also believe this is a result of poor documentation on Mailgun side and complete understanding on mine.
Hope my experience can help someone else. SO initially setting it to port 465 did nothing. I needed to set up separate SMPT credentials for my domain. The sandbox credentials would not work. But once I put those new credentials in. worked like a charm.

Have separate SMPT account credentials
Use port 465, not 587