Implementing a PoC for VLS signer

Background
Given the absence of a widely adopted full node for rust-lightning (which we aim to become one of), there exists a notable project called Validating Lightning Signer (VLS) at vls.tech that supports rust-lightning but lacks a compatible project to collaborate with.
From our perspective, we are a small project with significant ambitions. Incorporating the VLS signer as the default signer within Lampo can aid us in enhancing various aspects of our design and ensuring fund security by entrusting responsibility to individuals dedicated to fund safety.
Goal
This SoB project seeks to experiment with Lampo and VLS by implementing the different signers supported by VLS (including the local and remote versions) within Lampo.
The main steps for this SoB project are:

Explore using VLS within a rust-lightning node with the lnrod node.
Implement and potentially extend the common interface of a signer exposed by Lampo to facilitate signer swapping at compile time (e.g., utilizing a Signer trait or leveraging the ldk).
Initially implement the local signer and evaluate its ability to pass Lampo's integration tests.

Due the recent progress that @VanshulB is doing, I will add this to the August release

Closing this as completed.
We have a PoC that vls is possible to use inside lampo architecture with the following PR https://github.com/vincenzopalazzo/lampo.rs/pull/284
we should just write the code a little bit better to be production ready


sVector version 1

@JavierJia

@xixZ the main issue is to change the byte[] frame to ByteBuffer frame.


Update readme.md

To give the basic idea this app deal similar to Instagram. So you can add something like "It allow to follow peoples and post the images to keep connected".Also you can add something like what te h it uses.

I will take this

Published a PR https://github.com/git-anurag-hub/social-media-app/pull/7

I can take this. Thank you.


Address not printed on receipt

If you download the receipt for this transaction: https://opencollective.com/the-archive-project/orders/599934
The address is not shown under the person's name. But when I look at this profile in metabase, the address is there. Addresses should be printed on receipts.
Please note this is an incognito transaction and the user's name or details should not be mentioned publicly in this issue.

Can this be assigned to me?

Thanks on it

This issue will only affect the admins; if the contributor downloads the PDF they'll have the info printed correctly.
The fix must happen in the API, in https://github.com/opencollective/opencollective-api/blob/3fcba43c35025c032a0caffd07348981332bbe82/server/graphql/v2/object/Individual.js#L77 we must allow access in the context of a query that fetches the information form the host. Basically, we need to extend the permission that was implemented with https://github.com/opencollective/opencollective-api/blob/3fcba43c35025c032a0caffd07348981332bbe82/server/graphql/v2/object/Credit.js#L20 to also allow locations.
I'd recommend renaming SEE_ACCOUNT_LEGAL_NAME to SEE_ACCOUNT_LOCATION in this intent.
@yhoungdev Thanks for jumping on it, but it appears this issue will be complex to solve and test for a first contribution. I'd suggest finding another one if that's ok.


Help Yourself: Better Documentation in multiply-local-eth-btc-price

Depends on #795
This PR circles back to the review in https://github.com/keep-network/tbtc/pull/757#discussion_r584823654 to add clarity and standardization to the multiply-local-eth-btc-price script.
Tagging @Shadowfiend for review

I'm going to unmerge this until the unmangle-prettier branch gets merged so that we don't pollute the diff. I'll also mark this as a draft!


Followup #861

Description
Implement comments from https://github.com/opensearch-project/opensearch-migrations/pull/861

Category: Documentation
Why these changes are required? Improve user experience reading documentation
What is the old behavior before changes and new behavior after changes? Just documentation updates

Issues Resolved
MIGRATIONS-1911
Is this a backport? If so, please add backport PR # and/or commits #
Testing
Documentation viewing in Github
Check List

[x] New functionality includes testing

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

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

@AndreKurait Could you please include the context in the pull request of what and why?

@AndreKurait Could you please include the context in the pull request of what and why?

Yes, planning to before I publish this PR. This was created as a draft primarily to view in github how the footnotes are rendered


Automating serialization of pbix file to folder structure

Hi @otykier ,
Is there any way to automate the serialization of pbix file to folder structure done by TE.
So In general we are opening TE through Power BI desktop and then click on save to folder and saving the serialized dataset (w json objects) in repo. So I was wondering if I can use any CLI or PowerShell script to automate serialization such way that when user places pbix file in a folder or sharepoint the automation process saves the folder structure in a different folder.
Appreciate your help and time.
Thanks
Sai

Unfortunately, TE cannot access the data model of a .pbix file directly. You would have to load the .pbix in Power BI Desktop, or use the .pbit file format instead (in Power BI Desktop go to File > Export > Power BI Template).
You can use Tabular Editor's CLI to convert a .pbit file to a folder structure in the following way:
start /wait TabularEditor.exe "myfile.pbit" -F "gitrepo\myfile"

Thank you @otykier


Symbolic Jacobian sparsity

This doesn't work yet, but I quickly tried this out yesterday, thought I might as well put it up to look if we should pursue this.
I think the main advantage is that we don't have to maintain our own code for determining the sparsity, which can more easily get out of sync. It does add some dependencies.
The error I get is with DataInterpolations not supporting ::SymbolicUtils.BasicSymbolic{Real}, which looks like this closed issue: https://github.com/SciML/DataInterpolations.jl/issues/168. DataInterpolations has an extension on Symbolics for this purpose, so perhaps we just need to create a minimal example and file an issue.
ERROR: MethodError: no method matching (::DataInterpolations.LinearInterpolation{Vector{…}, Vector{…}, Float64})(::SymbolicUtils.BasicSymbolic{Real})

Closest candidates are:
  (::DataInterpolations.AbstractInterpolation)(::Symbolics.Num)
   @ DataInterpolationsSymbolicsExt C:\Users\visser_mn\.julia\packages\DataInterpolations\NoIUa\ext\DataInterpolationsSymbolicsExt.jl:15
  (::DataInterpolations.AbstractInterpolation)(::Number)
   @ DataInterpolations C:\Users\visser_mn\.julia\packages\DataInterpolations\NoIUa\src\DataInterpolations.jl:22
  (::DataInterpolations.AbstractInterpolation)(::Number, ::Integer)
   @ DataInterpolations C:\Users\visser_mn\.julia\packages\DataInterpolations\NoIUa\src\DataInterpolations.jl:23
  ...

Stacktrace:
  [1] get_area_and_level(basin::Ribasim.Basin{…}, state_idx::Int64, storage::Symbolics.Num)
    @ Ribasim d:\repo\ribasim\Ribasim\core\src\util.jl:57
  [2] set_current_basin_properties!(basin::Ribasim.Basin{…}, storage::SubArray{…})
    @ Ribasim d:\repo\ribasim\Ribasim\core\src\solve.jl:43
  [3] water_balance!(du::ComponentArrays.ComponentVector{…}, u::ComponentArrays.ComponentVector{…}, p::Ribasim.Parameters{…}, t::Float64)

It could be that there are a bunch more issues of this kind once this is resolved.
If it starts to work but we don't want it, we should perhaps use it to test our manual Jacobian sparsity.

https://github.com/SciML/DataInterpolations.jl/issues/310

I did some digging. I found out these things:

The problem you state above is likely not caused by DataInterpolations.jl, but by SmoothInterpolation.jl, because the problematic type is the output of a function call of that package and the problem goes away when I remove that call. That will be solved by https://github.com/Deltares/Ribasim/pull/1649
Code that branches out is incompatible, for instance in reduction_factor. I made a non-branching version of it
PreallocationTools is not directly compatible with Symbolics, I found a workaround here: https://discourse.julialang.org/t/type-issue-to-get-preallocationtools-diffcache-work-with-symbolics-jacobian-sparsity/88542/3?u=bdekoning
To trace trough the code to obtain the jacobian sparsity, the type parameter T in Parameters and its fields must (probably) be Vector{Symbolics.Num}, so the previous point might be irrelevant. I have not yet thought of a non-intrusive way to achieve this

There seems to be a problem with DataInterpolations after all:
https://github.com/SciML/DataInterpolations.jl/issues/317

One thing to keep in mind here is that some basin dependencies are temporary and not 'active' at initialization. To get a proper sparsity pattern, all nodes should be active

Calculating the Jacobian seems fast, but runtime performance went down a lot. The wrap_forcing function leads to GC and runtime dispatch. One possible way to resolve this that we discussed is to switch the forcing ComponentVector to four separate Vectors. The issue is probably that the ComponentVector does not play well with similar_type used in LazyBufferCache, since the Axis of the components are part of the type.

We should also have a look at https://github.com/adrhill/SparseConnectivityTracer.jl as an alternative, lighter dependency.

SparseConnectivityTracer.jl needs some work: https://github.com/adrhill/SparseConnectivityTracer.jl/issues/145

SparseConnectivityTracer (the unsafe version) now works, only Aqua complains about type piracy on a custom overload, but that should be solved upstream


Add preprocess script section in angular-cli.json

I want to concat css into a single file.
I have to run a separate gulp process.
Thanks for adding it

Hi @claygifford, this is functionality we already have. Have a look at https://github.com/angular/angular-cli/wiki/stories-global-styles for details.
Bear in mind that in development builds we will make a JS file instead of a CSS file for rebuild speed. But on production builds (ng build --prod) you will have the CSS file.

Thanks for the quick reply, that is something different.
I am asking for a
"styles": ["../css-bundles/**"],
Your answer to that almost 2 years ago was no.
Instead simply allow a pre processing script something like....
function requireAll(r) { r.keys().forEach(r); }
requireAll(require.context('../client', true, /.css$/));
Or
gulp-concat-css

https://github.com/angular/angular-cli/issues/3069
Here is your response 2 years ago.

I understand what you were talking about better now, thank you. Yeah I think the stance our stance still the same though, for the same reasons (https://github.com/angular/angular-cli/issues/3069#issuecomment-261941189):

We're not looking at adding glob support to styles, because loading order is relevant. Instead, just make a single file that imports everything in the order you want and put that in the styles array.


Add Quick Start in Readme for Contributors

Included Quick Start
This PR closes #373

Superseded by automation scripts.


Can't download Visual Studio Code 2019

So I heard VS code was the best editor at the moment, however, when I go to download it takes me to the download page only for a split second then redirects me to a blank page that just says not found. I am using google chrome as my browser and I have already tried to download while disabling VPNs. Not sure what to do.

Can't download Visual Studio

+1

+1

Duplicate of #74732 and #74734 and #74737.

+1

Having the same issue

+1

+1

+1

+1

same here

Can be downloaded！

Can confirm it works now!
From: llussy
Sent: Sunday, 2 June 2019 17:29
To: microsoft/vscode
Cc: Steph Wong; Comment
Subject: Re: [microsoft/vscode] Can't download Visual Studio Code 2019(#74736)
Can be downloaded！
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Can confirm it works now!

Can confirm it works now!


[WFRP4e System] FoundryVTT sidebar icons shift 6 places down when collapsing the sidebar

Since 1 or 2 versions, the foundryVTT sidebar icons shift down 6 places when collapsing the sidebar, This makes the sidebar unusable and the situation difficult to recover from. The problem is there with no plugins enabled and in different browsers.
From<EMAIL_ADDRESS>11.315wfrp4e: 7.0.3wfrp4e-altdorf: 4.0.1wfrp4e-archives1: 4.0.2wfrp4e-core: 4.0.2wfrp4e-dotr: 4.0.2wfrp4e-eis: 4.0.1wfrp4e-empire-ruins: 4.0.2wfrp4e-horned-rat: 4.0.2wfrp4e-middenheim: 4.0.2wfrp4e-pbtt: 4.0.2wfrp4e-rnhd: 4.0.1wfrp4e-salzenmund: 4.0.1wfrp4e-starter-set: 4.0.0wfrp4e-ua1: 4.0.1wfrp4e-up-in-arms: 4.0.1wfrp4e-wom: 4.0.1wfrp4e-zoo: 4.0.1Active Modules: LockView, LocknKey, MaterialDeck, _chatcommands, autorotate, background-volume, chat-commander-wfrp4e, colorsettings, combat-tracker-dock, dice-calculator, dice-so-nice, fast-flip, forge-vtt-shared-compendiums-warhammer, foundry-vtt-content-parser, foundryvtt-simple-calendar, fuzzy-foundry, fxmaster, healthEstimate, illandril-token-hud-scale, item-piles, lib-wrapper, michaelghelfi, monks-active-tiles, monks-bloodsplats, monks-combat-marker, monks-player-settings, popout, quick-insert, sequencer, smalltime, socketlib, token-action-hud-core, token-action-hud-wfrp4e, token-hud-art-button, token-mold, tokenmagic, trs-foundryvtt-dice-set, trs-housedivided-dice-set, trs-ruby-dice-set, vtta-tokenizer, wfrp4e-gm-toolkit, wfrp4e-pbth-maps, wfrp4e-unofficial-grimoire, forge-vtt

This issues also shows up on brand new install of the system, e.g., on ForgeVTT


Updated package extensions to associate files

Updated Package.json so that when opening files with related SQL extensions.

Hi @mscodebum, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@mscodebum, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Looks like what I wanted to accomplish this was added in #10427 closing this pull


Define whether OID claim is also 'immutable and cannot be reassigned or reused.'

The definition for the sub claim says:

This value is immutable and cannot be reassigned or reused.

That is a very useful statement! Can the same be said about the oid claim? And if so, can we add that to the documentation for oid?

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

ID: a0cace34-34bf-32f7-c857-e4d71d535a77
Version Independent ID: cdbbc4ae-5894-da21-42d7-928b3674bbd1
Content: Microsoft identity platform ID token reference
Content Source: articles/active-directory/develop/id-tokens.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@ryandle hello, thanks for your feedback, we'll be investigating this and letting you know our updates.

Hey @ryandle can you clarify on what you would need in regards to this? Currently it does say :
The immutable identifier for an object in the Microsoft identity system,
It also says :
This ID uniquely identifies the user across applications - two different applications signing in the same user will receive the same value in the oid claim. The Microsoft Graph will return this ID as the id property for a given user account. Because the oid allows multiple apps to correlate users, the profile scope is required to receive this claim.
So it is dependent on the user id from which if reassigned on the backend will change the value in the OID accordingly.

Thanks @FrankHu-MSFT for taking a look. You are correct that immutability is already covered. But 'reassigned or reused' is not, which is what I am asking for explicit clarification on for oid.
I did not have this same interpretation as you when I read the documentation for oid:

So it is dependent on the user id from which if reassigned on the backend will change the value in the OID accordingly.

In general, as a developer reading through this documentation and trying to decide which user identifier I want to use in my application, there appear to be 3 unique properties of an identifier that I need to consider:

Immutability - can this id ever change for the same user instance?
Reusability - can a new user instance reuse the id from an old (no longer used) instance?
Cross-application uniqueness - is this id the same across all apps for a given user or is it different for each app?

I see 1, 2 & 3 explicitly addressed for sub. Only 1 & 3 are explicitly addressed for oid, and my request in this issue is to add an explicit statement to oid about 2.
(Side note: the documentation for sub says "reassigned or reused" which mean the same thing to me. Is there a subtle difference I am missing?)

Hey @ryandle thank you for that clarification, that definitely clears things up. I'm going to colalborate with my team on this issue to determine best wording. Thanks!

Hey @ryandle i'm following up on this issue.
As OID is a GUID, they cannot be reused.
I will go ahead and put this in the docs as well. Thanks for letting us know about this. That being said, I will be closing this issue by end of day tomorrow. Please let us know if you have anymore questions in regards to this issue.
Thanks,

Frank Hu


Pksunkara/auth

https://github.com/apiaryio/api-blueprint/commit/25081311502c6fca15d92451709c62fec9e8ea8fhttps://github.com/pksunkara

What is this? :disappointed:


[HOLD] Race condition between staging and production deploys

Problem
There is a race condition between staging and production mobile deploys that can cause production mobile deploys to fail.
For some background, when we deploy iOS and Android to staging what we're really doing is:

Creating a production build
Uploading it to the app store as a beta (TestFlight or Internal App Sharing)

Then when we deploy iOS and Android to production, all we're really doing is taking the staging build, submitting it for review + release.
What can happen is:

We have a deploy blocker affecting only one platform (say, web), and it's our last deploy blocker.
The deployer CP's a fix, and as soon as that fix is verified closes the checklist (before the staging deploy from that CP has completed on all platforms, which often takes about an hour)
The production deploy starts before the staging deploy finishes, so when we try to submit the latest version for review + release, Fastlane barfs 🤮 because the build with that version hasn't been uploaded to the app store yet.

Here's a real-world example: https://expensify.slack.com/archives/C07J32337/p1675468753414159
Solution
Let's make it so that a production mobile deploy will wait for the staging mobile deploy of the same version to finish before it begins.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0110288f4cb7c04c91
Upwork Job ID:<PHONE_NUMBER>611951616
Last Price Increase: 2023-02-04

@roryabraham, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.


MAINT: Update readme for write permissions

Closes #41. Closes #25. Closes #36.
Maybe the default should either stop trying to open an issue or include write access as well.

Thanks @HaoZeke , an important fix! @assignUser We should update the workflow with this. I'll be able to do it next week, or you can fix in the meantime if you've got a chance. Thanks!


Support extra installation methods in R

Added support for installing older versions of packages from CRAN
Also added support to direct install from GitHub or Git but with a note that this is only mean to be used during development, to speed up testing if the CRAN package is not yet available
First step towards solving #1350

Isn't #1350 already solved by #1355?

Yes, @bouweandela  was faster than I. This was to be the short term solution
Anyway, it is useful on its own, as will allow us to avoid issues with buggy releases or breaking changes in packages

Tests are failing because a timeout. Are any of you experiencing this in other branches?

Yes, it happened several times.

I likes, but I am not comfortable faffing around with older versions - I mean this could be needed as per the lintr issue, but I would not feel comfortable for users to do that at their leisure and then starting to report issues with unreproducible environments. I also found a way to install straight from the tarball in Archive https://github.com/ESMValGroup/ESMValTool/issues/1350#issuecomment-539960982

I likes, but I am not comfortable faffing around with older versions - I mean this could be needed as per the lintr issue, but I would not feel comfortable for users to do that at their leisure and then starting to report issues with unreproducible environments. I also found a way to install straight from the tarball in Archive #1350 (comment)

I agree, but we need something that allow us to pin versions of R packages if needed, as we do with conda and PyPI when needed. We just need to state clearly that we only pin a version if there is some issue with the newest one on CRAN

@jvegasbsc This pull request seems to contain a new CMORizer as well now. Could you please clean up and let me know when it's ready for review?

@jvegasbsc Any news on this?

Yes, ready to test

Thanks, looks good! @mattiarighi Would you be able to test this?


Add testcase for unicode payment id

I want to merge this change because it is a port of these 3.0 changes #9008

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Please take a look at the failing tests: https://github.com/saleor/saleor/runs/5042011936?check_suite_focus=true

It needs other changes to be ported before it will work on main.


Cookies

Cookie policy for third-party video solution
Cookie-free link possible?
Currently need to accept cookies before being able to view YouTube video


Change the Wired Setup Wiki page so it is correct

Description

The current page says:
"If your headset is not detected, click "Add device manually" and use the IP address <IP_ADDRESS>. Use the hostname displayed on your headset screen."
This is not correct it should be hostname and add .local at the end.

General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR GitHub Wiki

Fwiw there are no checks being done on the hostname, so it could just be whatever. But yea the wiki should probably reflect that fact. I'm closing this tho because the next release will have a built-in wired mode and then we'll get rid of that wiki page anyway.


Changing embedded/imported links to match https, fixes #218

The only links we should worry about are the embedded/imported links, which are changed in this PR. The many other links are either simple links or references in comments thus do not (and shouldn't) be altered.

Looks good! :+1: Making note of our discussion on creating relative paths to links, as opposed to absolute, in the future.


adhearsion-xmpp stops firing in beta1

When I upgraded to beta1, my adhearsion-xmpp no longer received xmpp events.

Confirmed invalid, works perfectly in other applications.


KML: Data handling and GeoJSON layer compatibility

Hi, I have added some functionality to work KML with data:

Inherited options
Geometry feature property
Parser of ExtendedData tags, providing properties
Options to put name, description and ExtendedData/Data/displayName tags into properties
onEachFeature support

Hi @falcacibar this one can't be merged for now, can you fix it please ?

Hey @brunob, it's fixed. check it out.

Thx @falcacibar but iI think there's still a problem, look at the diff. Maybe you should make a new pr.

I have rebased my commits over the last changes, try again please.

This is what I'm searching... @brunob do you intend to accept the changes? Thanks

@falcacibar sorry for the delay, before merge this one could you please provide in the PR an example file & an associated kml file to show the usage ?

Any news about this @falcacibar ?

Sorry @brunob I was very busy this last time, if something can write docs will be great, so I'm gonna explain what does this:
##GeoJSON feature/onEachFeature compatibility
Follows the GeoJSON parse type, adding "feature" property and parsing the properties in the geometry of the KML
##Read KML ExtendedData and place into feature.properties object
Reads the features properties from ExtendedData/Data tag according to the specification in https://developers.google.com/kml/documentation/extendeddata#adding-untyped-namevalue-pairs.

The name of the property is ExtendedData/Data@name
The value of the property is ExtendedData/Data/Value

##Parsing ExtendedData/Data/DisplayName
The specification of KML adds a DisplayName tag to differs the data name with long name of each Data tag

Can read the displayName tag of each ExtendedData/Data tag using the boolean option displayName (default: false) and adding it to properties with a suffix configured in displayName_suffix option (default  '_display_name')
The Name and Description tags stores in a user defined properties by the

Sorry @brunob I was very busy this last time, if something can write docs will be great, so I'm gonna explain what does this:
##GeoJSON feature/onEachFeature compatibility
Follows the GeoJSON parse type, adding "feature" property and parsing the properties in the geometry of the KML
##Read KML ExtendedData and place into feature.properties object
Reads the features properties from ExtendedData/Data tag according to the specification in https://developers.google.com/kml/documentation/extendeddata#adding-untyped-namevalue-pairs.

The name of the property is ExtendedData/Data@name
The value of the property is ExtendedData/Data/Value

##Parsing ExtendedData/Data/DisplayName
The specification of KML adds a DisplayName tag to differs the data name with long name of each Data tag.
This tag can be placed into featuree.properties object with displayName option (default: false) of the KML layer and applying a suffix with the displayName_suffix option (default  '_display_name'), then:

The name of the property is ExtendedData/Data@name + options.displayName_suffix
The value of the property is ExtendedData/Data/DisplayName

Parsing Name and Description tags
The KML specification uses by default Name and  Description tags, to avoid conflicts with possible Name or Description metadata, this can be configurable by the following options:
Name

name_on_props (default: false)
name_prop  (default:  'kml_name')

And in feature.properties will be:

The name of the property is options.name_prop
The value of the property is Name tag

Description

descr_on_props (default: false)
descr_prop (default: 'kml_description')

And in feature.properties will be:

The name of the property is options.descr_prop
The value of the property is Description tag


Blank page when importing package

Hi there, this is just the standard App.js file that is initiliazed to which I imported the Avvvatars package. When I save the code and refresh the page, it turns blank (so no rendering). Note that I'm just importing the package and not using it either.
import logo from './logo.svg';
import './App.css';
import Avvvatars from 'avvvatars-react';

function App() {
  return (

          Edit src/App.js and save to reload.

          Learn React

  );
}

export default App;

I checked whether the package has been properly installed and it's appearing in package.json too.

Hey,
I tried to reproduce this but I don't get the same results, if it's caused by the package you should at least see an error on console.
I'm gonna need a little bit more information on that topic to inspect what's wrong, such as what boilerplate did you use, your package.json content, and if it's possible a codesandbox link to quickly review it.

You're right, this is the error I'm getting from the console
Uncaught TypeError: goober.styled is not a function
    at Object../node_modules/avvvatars-react/dist/avvvatars-react.cjs.development.js (Shape.tsx:18:1)
    at Object.options.factory (react refresh:6:1)
    at __webpack_require__ (bootstrap:24:1)
    at fn (hot module replacement:61:1)
    at Object../node_modules/avvvatars-react/dist/index.js (index.js:7:1)
    at Object.options.factory (react refresh:6:1)
    at __webpack_require__ (bootstrap:24:1)
    at fn (hot module replacement:61:1)
    at Module../src/App.js (logo.svg:34:1)
    at Module.options.factory (react refresh:6:1)

Still can't replicate the issue, which node version and react version have you used? and is there any chance that you can send me a demo on github or codesanbox

Thanks for letting me know, I've managed to reproduce it and fixed it with #7, you can install latest version and let me know if it works.


Update Groups Attempt 3

I have no clue if this is working

@saurabhkakade21 You can merge with all.
We have only 3 grad students :-)


Neo3 neo-gui: failed to vote.

I tried to vote, but execution result was false.

Here return false when checkwitness.

Because cosigner is null.

We will improve neo-gui or create a new one?

We will improve neo-gui or create a new one?

@shargon ,we are creating a new one.

@wanglongfei88 Then we should close this and mark neo-gui as obsolete

@wanglongfei88 Then we should close this and mark neo-gui as obsolete

Let's keep this GUI before we finished the new one which will take some times since it is going to be a complete different tech stack.

Then... we should select the cosigner that you want to use it, because now, cosigners are empty

Then... we should select the cosigner that you want to use it, because now, cosigners are empty

So now we need to provide a place to select the cosigner in this gui?

Yes, I think that the problem is that we need to select the cosigners, but maybe i am wrong

Yes, I think that the problem is that we need to select the cosigners, but maybe i am wrong

This is a transaction sent by one sender which is signer. I do not understand why we need to select the consigner because there is no one cosign this tx with me.
BTW, I have checked the code. When validating a tx, witnesses is matching sender and consigners both. But when checking witness, witnesses is only matching consigners, exclude sender. The consequence is that a tx will pass tx validation but failed on execution.
Is it something you guys design for some purpose or a mistake ?

Could you provide me the raw transaction in order to test it?

my vote tx
{
    "jsonrpc": "2.0",
    "id": 1,
    "result": {
        "hash": "0x5e7c7b581ce61bd210edf6f3d490b7a07c92a43e957ff3f16d099d5a8e5a91b7",
        "size": 246,
        "version": 0,
        "nonce":<PHONE_NUMBER>,
        "sender": "NQhcLXMUShK1EaK7DPd7Z4XMyucWZy2mpP",
        "sys_fee": "600000000",
        "net_fee": "1246270",
        "valid_until_block": 2104342,
        "attributes": [],
        "cosigners": [],
        "script": "IQLxQ1YtbMH6uWDOJplOsDua4fb8+5YbR9AGGTSrLbSSW1HBFDSCO9DWzZp2WQ72qwqGmOB1HSt1UsEEdm90ZRQVyqBCFDEGcNXlo5jhR+Db7ZjPQ2hifVtS",
        "witnesses": [
            {
                "invocation": "QKaQ09+GhPpcYKS5as4bAY8CNEW8ZPEImfxhaQuku+JCkYfc/HhIricK1g+KxggXTLgzBcD9np17FhsQeihbtrc=",
                "verification": "IQJwfeABt+wcPsTBJbDzpd3+v4yx29FjG+6HeACu8N3BrlBoCpBq1A=="
            }
        ],
        "blockhash": "0xac919a07bab819f9b72297e4cbcf9f0e6180d92e4022ea0272c26dd63bb7f202",
        "confirmations": 2,
        "blocktime":<PHONE_NUMBER>919,
        "vmState": "HALT"
    }
}


Readme is missing basic getting started information

goal: improve the main read.me file to point the user how to consume the library
Task

[ ] Add information about how to download the library with: npm and bower
[ ] Add information on how to consume the library, containing

[ ] Full compiled CSS files
[ ] Stand alone components

[ ] Add the information resulting from the tasks above to the "Getting started" page
http://sap.github.io/techne/www/getting-started.html

Note: Temporary solution

Review results
The readme looks good.
The Getting started page, looks ugly as hell, but we can go with that.
Please note:

The tasks are not ticked.
This is missing the "review" label. That should be added when this is ready to review.
This is missing the PR reference as well.

This can be put to done. Thx.


[C++] Add a pivot_longer node

Describe the enhancement requested
Pivoting longer is an operation that is useful when a single row contains multiple observations or measurements.  For example:

time
left_temp
right_temp

1
10
20

2
15
18

The columns left_temp and right_temp are really the combination of a "feature" (location) and a "measurement" (temp).  We can pivot longer to extract these into their own columns:

time
location
temp

1
left
10

1
right
20

2
left
15

2
right
18

The addition of this capability should make it possible to fully bind tidyr's pivot_longer and pivot_wider (the latter can be expressed using a combination of project and aggregate).  However, there is a considerable amount of convenience that could still be added and much of this is captured in the parent issue #34265
Component(s)
C++

It appears that DuckDb has tackled the pivot feature at the same time (and their implementation is much more complete).  They use unnest for pivot longer but it's not clear to me that we could do the same thing here.  It relies on the ability to create a sort of list comprehension out of field references to convert field references to elements into a field reference to a list element  (I'm not sure exactly what this syntax is called).  It's quite impressive :)


Add from_dict classmethod

as discussed on the latest call

Thinking about this again, I'm not sure how this would work
In order for this to be possible, I think the following would probably be necessary:

constructors for Column
a way to generate a given DataFrame from a dict of Columns, which would probably require being able to get the underlying data from the columns

Let's talk about this more next time, but I'm not sure from_dict is the solution

constructors for Column

There should be a lot of functions that one can apply directly to columns, so you can get new columns from ops on existing ones. But agreed - probably also need to be able to construct columns from sequences and arrays for many use cases.

a way to generate a given DataFrame from a dict of Columns, which would probably require being able to get the underlying data from the columns

This one I don't understand - this should be a completely trivial operation, and no access of underlying data should be needed (a dataframe is simply an ordered collection of columns). Or am I missing your point?

Since we agreed in https://github.com/data-apis/dataframe-api/issues/118 that each library can have private methods to access the underlying data of its own columns, you're right that it is indeed trivial - e.g. in pandas we could do
    @classmethod
    def from_dict(cls, data: dict[str, PandasColumn]) -> PandasDataFrame:
        return cls(pd.DataFrame({label: column._series for label, column in data.items()}))

is this one OK, or does anything else need doing?


Platformio VSC IDE Monitor multiple serial/tcp ports at a time:

I was using, normally, two serial ports.
After changing workspace, I only have one and trying to open the second one ( two times or more, because nothing happens ) I get the message:
~No serial monitors are available to start monitoring a port. Only 3 can be opened at one time.
What can I do to resume to, until now, normal behavior.
Thanks in advance
Joaquim Silva

Could you try to use the command mentioned in this comment? https://github.com/microsoft/vscode-serial-monitor/issues/144#issuecomment-1766055678
This should fix your issues. Sometimes VS Code persists the windows even though they aren't visible in the view. Thank you!

https://github.com/microsoft/vscode-serial-monitor/issues/147
Check your terminal tab. The windows that will open later are under the terminal tab. You need to drag them to where you want them.

Thanks all for taking time to answer my post.
Really still only one serial monitor window, on the terminal one

I trying a lot of things ( looking for the windows ) but not succeded.
TIA
Joaquim Silva

The thing is that before two days ago, I was working perfectly with 2 serial monitor windows!

Could you try two additional things to see if it fixes your issue?

Run the View: Reset View Locations. See if this fixes your issue.
If that doesn't work, could you attempt to disable the extension, re-open VS Code, and then re-enable the extension?

I would hope that this fixes your issues. Please let me know if it doesn't.
If you think adding some sort of command from our extension to provide a simple way to bring focus to visible extensions, please let me know by creating a new issue for that. Thanks!

Dear Garret:
Thanks for your interest, in solving the issue.
I followed your suggestions but nothing changed.
I also stopped Malware.
I tried to uninstall Serial Monitor, but not succeeded because it is an Arduino dependency.
But I disable it as per your instructions.
I must tell that this happened after I tried to open a second serial monitor window, when the system was still reopening.
May be this can give you a clue.
Waiting and still trying
Best regards
Joaquim Silva
From: Garrett Campbell @.>
Sent: Monday, November 20, 2023 4:44 PM
To: microsoft/vscode-serial-monitor @.>
Cc: Joaquim @.>; Author @.>
Subject: Re: [microsoft/vscode-serial-monitor] Platformio VSC IDE Monitor multiple serial/tcp ports at a time: (Issue #152)
Could you try two additional things to see if it fixes your issue?

Run the View: Reset View Locations. See if this fixes your issue.
If that doesn't work, could you attempt to disable the extension, re-open VS Code, and then re-enable the extension?

I would hope that this fixes your issues. Please let me know if it doesn't.
If you think adding some sort of command from our extension to provide a simple way to bring focus to visible extensions, please let me know by creating a new issue for that. Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-serial-monitor/issues/152#issuecomment-1819434336, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABI7RT4ALTTXU35ORIN4XWTYFOCD5AVCNFSM6AAAAAA7QRAN76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJZGQZTIMZTGY.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I have refreshed the Extension applyed the command, but the only thing I get is three new attempts, and, after that the same error message: only three serial monitors allowed and only one serial monitor window.
Thanks for your effort in help.
Do you think that, if I reinstall Platformio, this can help? I have not yet tryed it.
Thanks again.
JJDS

@JJDS Could you, after opening all three Serial Monitors, attempt to open all other panes? For example, the terminal pane, etc. etc. Sometimes if you've moved the Serial Monitor pane to somewhere else it will open there again and isn't immediately obvious in the top screen.

Solved!!!
Thanks a lot for your help.
Now I have the 3 serial monitor windows, if I wish so!
Thanks to you, and all of the people, that tried to help.
Best regards
JJDS


Obsolete ICommand & CommandParameter

Description
Obsoletes the ICommand & CommandParameter to prepare for v7 breaking-change.
How Has This Been Tested?
No tests required.
Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.

Thanks, good thinking!

I know this was shipped months ago, but I'm curious as to why the decision was made to remove this. It seems like the power of ICommand is beneficial to be able to disable a button based on the "CanExecute" result.
/cc @henon @ScarletKuro

I know this was shipped months ago, but I'm curious as to why the decision was made to remove this. It seems like the power of ICommand is beneficial to be able to disable a button based on the "CanExecute" result.

https://github.com/MudBlazor/MudBlazor/pull/2108#issuecomment-1582420492


Hovers of the word following ^. doesn't print the type of the function

For example, in this snippet:
meta def foo := let m := rb_map.mk nat nat in m^.insert 0

Hovering over insert in vscode gives no information, but hovering over the ^. prints the type of rb_map.insert

Does your Lean binary include https://github.com/leanprover/lean/commit/339713091f3bbd25a0cb51181c0ec0ed1bb551c2 (merged yesterday)? There was some problem with functions containing implicit parameters.

You're right, just updated and it's fixed. Nice!


Signing assertions compliance report

#1355 #1356
This PR should accommodate the Signing Assertions for submitting a Compliance Report
Changelog:

Updated model to allow classification of signing assertion
Added Compliance Report for signing confirmation
Added Submit button
Added operational script to populate the signing assertions for compliance report
Clicking on Submit shows the modal that contains the new assertions
Status can now be set to submitted
Allowed confirm button in modal to be disabled

 Compliance Report | Declaration confirmation alert when submitting a Compliance Report |5|

 Compliance Report | Submit a Compliance Report |2|


Start to implement parsing and validating request params.

Starting to implement parsing and validating request parameters mechanism.
Introducing a structure to hold the request handler payload used in the processing of incoming requests.
Utilizing a request handler payload in the request handler.

She looks spreading dangles ! :+1: — Approved. :blue_heart:


Issues setting up environment --  Possible [BUG]

Hello!
For the specific case of instance id: matplotlib__matplotlib-17810, the requirements that is installed is: requirements.txt which tries to install git+git://github.com/sphinx-gallery/sphinx-gallery@b41e328#egg=sphinx-gallery and fails. How can I get around this issue?
Thank you!

This might be the result of a possible bug in harness/utils.py:164 where is it currently instance["base_commit"] and I think it should be instance[commit] referencing the commit variable defined in Line 135.
`

Oh that is a great catch, I think you may be absolutely right. I'll push a commit that makes the fix you suggested.
I'm not able to test at this very moment, but I can do it by this coming weekend to verify that the instance installation procedure is no longer failing.

Just run on my side - the set up completed successfully for me. @prabhuteja12, is it working for you now?

Hi John, I didn't run the full suite but tested it out for the specific failing case, for which the fix worked. The code fix you made seems very similar to what i did locally. Thank you for the active development :)

Sweet, closing the issue then - thanks so much @prabhuteja12 for proposing the fix and your kind words 😄 please feel free to post if anything else comes up!


Update favicon

Description

Change default favicon to our logo
Acceptance Criteria

[ ] Favicon displays fine on all kinds of browsers

Implementation details

n/a
Testing notes

n/a

YES! I've been wondering about this haha. LGTM :)


⚠ Bofrost.fr has degraded performance

In 6d166d3, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1026 ms

Resolved: Bofrost.fr performance has improved in cd4f1f6 after 18 minutes.


[Heartbeat Logging] Migrate Messaging from using FIRHeartbeatInfo to FIRHeartbeatLogger

#no-changelog

I had to re-run the messaging CI a few times because the integration tests were flaky. To check if this PR's changes were the cause, I ran triggered the messaging CI on master (#9579) and they flaked there as well.


Add the possibility to create Attribute Functions for Schemas

Problem
Sometimes we need data to be generated and assigned to fields in the database by default. A simple and straightforward example: a customer code that uses a specific format.
Currently, Prisma gives the possibility to do this using middlewares, but I believe that this approach can be cleaner and "direct", being able to have this defined directly in the schema, as it happens today with the attribute function uuid(), used with the @default attribute.
This would literally be very useful, as I currently spend a lot of time making middleware to handle this.
Suggested solution
Enable custom attribute functions to be created.
It could work like this:

Create a JS file with the function to be called by Prisma.
The function would receive as a parameter the DMMF data of the field that is invoking it in the model, because that could base how the default value would be generated.

module.exports = function(fieldData) {
    return `example value of ${fieldData.name}`;
}

In the schema file that is using the attribute function, we define the used functions and the paths to the JS files, like:

functions {
    functionName1 = "../functions/any.js"
    functionName2 = "../functions/any2.js"
}

Finally, we use in a field:

model Customer {
    id  String  @id  @default(functionName1())
   // other fields
}

Functions would obviously be invoked by Prisma before going to the database, like middlewares.
It would be very useful, and would greatly shorten the time it takes to develop the applications.

👋 Hi @devmanbr!
I've been working on my NPM Package Prisma Util, which is all about implementing features that are requested by developers in the community. This issue has attracted my attention and I want to ask if you'd be interested in it being implemented in Prisma Util.
Prisma Util is already capable of enhanced schema parsing, so it should be pretty easy to implement.

Hi, @DavidHancu!
I would love to help directly with this, but unfortunately my time is very limited. I hope there's someone available for this.

Hi @devmanbr, I've just released the proposal for this feature for Prisma Util: https://github.com/DavidHancu/prisma-util/issues/5.
Let me know what your thoughts are.

Seeing your update on that today, @DavidHancu. I'm going to try to make use of the tool in the next project I'm working on, and if any relevant opinion or idea comes up, I'll be happy to let you know.


Helping the user bring the right things based on the weather

If it's raining, bring an umbrella
If it's sunny, bring suncream
If it's cold, bring a coat

I am relatively new to React but would like to work on this issue. Could you elaborate on how I might be able to help? Thanks!


Add Apache README.md

add Apache MXNet (incubating) to the first mention of MXNet? and a statement at the bottom that the project is incubating.

I believe no reader in the world cares about who is incubating what. Happy to change name reference to Apache MXNet, but the parenthetical "(incubating)" is excessive.

I added one reference in the section header: "Implementation in Apache MXNet". Because we refer to MXNet so many times, adding Apache everywhere would be a bit insane and excessive. Also, note for your personal reference that this is a book, not the project itself. It is in reference to and in support of MXNet, but I don't see this book as epxlicitly governed by anyone besides the authors.


variable-length-quantity: wrong test case

The readme says:

The largest integer allowed is 0FFFFFFF, which translates to 4 bytes variable length.

which contradicts this test case "from_bytes":
assert_eq!(&[0xffff_ffff],
           vlq::from_bytes(&[0x8f, 0xff, 0xff, 0xff, 0x7f]).unwrap().as_slice());

I wonder why that is the largest allowed integer. https://en.wikipedia.org/wiki/Variable-length_quantity says:

A variable-length quantity (VLQ) is a universal code that uses an arbitrary number of binary octets (eight-bit bytes) to represent an arbitrarily large integer.

Given this, should it be that the README removes that sentence about the largest integer?

I believe our tests are correct and the README is wrong.


Replacing hardcode RPC URLs with environment variables

This PR contains implementation to replace hardcoded URL to environment variables.
#243

I wasn't sure if we need to also support receiving provider URL via query param. @Pessina let me know what do you think?


When changing the source directory, import maps needs to be in the source for the runtime to work

When changing the source directory, if the import map is still in the root, then this error will happen:
error: Uncaught (in promise) TypeError: Relative import path "typography" not prefixed with / or ./ or ../ and not in import map from "file:///D:/QC%20supplements/Code/Apps/Tr%E1%BA%A5n%20K%E1%BB%B3/Web/tailwind.config.ts"
The runtime still needs the import map in the source directory. However, for the LSP to work, a duplication of it needs to be on the root.
Related bug: https://github.com/denoland/deploy_feedback/issues/590

I can only imagine this would be an issue if you're launching the program from within the source directory. If you're launching it like this: deno run -A src/dev.ts then it should be fine. Are you saying that this isn't working?

yes. I'm on root, and the deno.jsonc in root is like this:
"tasks": {
	"check": "deno fmt --check && deno lint && deno check **/*.ts && deno check **/*.tsx",
	"cli": "echo \"import '\\$fresh/src/dev/cli.ts'\" | deno run --unstable -A -",
	"manifest": "deno task cli manifest $(pwd)",
	"start": "deno run -A --watch=static/,routes/ './Web/dev.ts'",
	"build": "deno run -A './Web/dev.ts' build",
	"preview": "deno run -A ./Web/main.ts",
	"update": "deno run -A -r https://fresh.deno.dev/update ."
},

I only use tasks, not run the full commands directly


Meta issue to track baking more mount features into RB

Feature ideas on the road to make RcloneMount a full-featured rclone mount GUI:

Drop-down selector for mount disk letter
Save/restore last used mount disk letter, per remote
Option to auto-start on user login
Option to start hidden in tray
Double-click on tray icon toggles main window open/close

Autostart: RB can either write its path in the registry under HKCU/**/CurrentControlSet/**/Windows NT/**/Run or create a shortcut under /Users/user/AppData/roaming/Microsoft/Windows/Start Menu/Startup)
Originally proposed here https://github.com/kapitainsky/RcloneBrowser/issues/26
Forked since RB maintainer refused to implement. The fork can be maintained as a set of patches to upstream with option to merge back.

Closing stale issues


gh 2160 Right Inner Joins not working with KeyFunctionMatch

#2160
Problem could be derived down to performing a map.put using the same key hence overwriting the current stored value in the tuple, and the given tuple was given the same reference so it overwrote the value already in the result-list.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@5c8405b). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             develop    #2163   +/-   ##
==========================================
  Coverage           ?   62.84%
  Complexity         ?     2381
==========================================
  Files              ?      744
  Lines              ?    23959
  Branches           ?     2847
==========================================
  Hits               ?    15058
  Misses             ?     7657
  Partials           ?     1244

Impacted Files
Coverage Δ
Complexity Δ

.../gaffer/operation/impl/join/methods/InnerJoin.java
100% <100%> (ø)
7 <0> (?)

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 5c8405b...228874d. Read the comment docs.


Implement per-process Java profiler

Part of  #1115
Pseudo code
package javaprofiler

type Profiler struct {
    // ... fields
}

func New(
    pid int,
    w profiler.ProfileWriter,
    duration time.Duration,
) *Profiler {
    // ...
}

func (p *Profiler) Run(ctx context.Context) error {
    // essentially https://github.com/async-profiler/async-profiler/blob/32601bccd9e49adda9510a2ed79d142ac6ef0ff9/profiler.sh#L364-L380
    // And then convert JFR to pprof and write via profile writer
}

cc @v-thakkar

For this, we can't use the same implementation as the current cpu profiler for other languages as we'll only be able to attach this profiler after metadata gets attached to the process and we know that it's a java process. This means that profiling will start for java processes after the first profiling round(s). So the implementation flow will look something like this:

Find the java process and attach the metadata to it
Start the java profiler for that process with certain parameters (required by async-profiler)
Stop the java profiler in 2 cases: if parca-agent exits or if the process exits

Another way would be to check for each pid if it's a Java process, although this can get quite complicated and may waste some resources. So currently above solution looks best but let me know if there are some other ideas.

Sounds reasonable! Kemal and I were thinking on adding information about the runtime using the new facilities in the Process Information work

Deprecated.
We will use #1 to track the latest implementation.


Chore: create a file structure

create a file structure such that all CSS and JS files Have their own folder

But remember, you have to use tailwindcss cli to convert the input.css to tailwind.css. If you can do that , sure I'll assign it to you

hmm, it's worth trying.

@newtoallofthis123 I'm done with this issue, should I go ahead and create a PR?
or should I wait until the HacktoberFest officially starts.

Wait for October.....


Test summary during watch mode run

Clear and concise description of the problem
The proposal is to show a live-updating summary of the current test run instead of the list of tests that have ran and are running. This would provide a consistent UI that the user can see the progress of the run toward finishing while also outputting the error messages from failed tests as fast as possible. I believe this will both improve workflow of developers and improve the perceived performance of Vitest.
I'm happy to contribute and expand on anything here that you would like.
Suggested solution
Instead of showing the list of all tests suites as they are ran as is the current UI, Vitest could instead show a live-updating summary of the current run of tests. This would look somewhat similar to the Jest watch UI and could show things such as:

Total tests run
Total failures
Total passes
Total skipped
Total tests to run
Total time
Current status (pass or fail overall)

While making this change it will also be important to ensure that:

The UI is interruptible and responsive so that the user can pause execution and change things such as filename pattern
Test failures are printed above the summary as soon as they happen so that the user can scroll up and see the first failed test as soon as possible when running multiple test suites.
Test console output is also included in the correct order along with the test failures so that the messages can be read for debugging

Alternative
I'm aware there is Vitest UI which I am yet to properly check out. On my current project I have had issues getting it going so haven't had a chance to look at it. Although it may improve on the current watch experience it would still be great to improve the default cli experience.
Additional context
I would love to contribute to this and if you have other better ideas or plans that I've missed then let me know and I could help! I've just migrated a project onto Vite that has more than 10k tests and so far I'm really enjoying Vite!
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Is it a duplicate of https://github.com/vitest-dev/vitest/issues/3549 ?

Is it a duplicate of #3549? (but better written)

Yes indeed it does look like it. Are you interested in contribution in the area?

Is it a duplicate of #3549? (but better written)

Yes indeed it does look like it. Are you interested in contribution in the area?

Yes, PR is welcome.

@sheremet-va I'll have a look at this! I have been given the go ahead to spend a little of my day job time on this so I will first have to upskill on how the project works and then hopefully be able to contribute something!

@sheremet-va I'll have a look at this! I have been given the go ahead to spend a little of my day job time on this so I will first have to upskill on how the project works and then hopefully be able to contribute something! Do you have any suggstions or guidance on learning about the codebase?

Renderer that prints test results is located here:
https://github.com/vitest-dev/vitest/blob/8dd5ea5ba10c2a99568ff18f6efff356824cb2cf/packages/vitest/src/node/reporters/renderers/listRenderer.ts#L89
Test summary is printed after all tests finished running, so you will need to figure out a way to print it alongside running tests:
https://github.com/vitest-dev/vitest/blob/8dd5ea5ba10c2a99568ff18f6efff356824cb2cf/packages/vitest/src/node/reporters/base.ts#L210

Test summary during watch mode run

I think this should be based on isTTY instead of --watch/--run.
Even in --run mode this would be really useful, just like Jest does it.

Hi yall! I really would like to work on this, however we've been finding since moving to vitest that our tests are running about 3x slower (from 25min with jest, to 1h 30m with vitest). We have been investigating all the possible options and what not but we may end up moving away from vitest (which I'm sad about) but that means it's hard for me to justify spending the time on this...

Hi yall! I really would like to work on this, however we've been finding since moving to vitest that our tests are running about 3x slower (from 25min with jest, to 1h 30m with vitest). We have been investigating all the possible options and what not but we may end up moving away from vitest (which I'm sad about) but that means it's hard for me to justify spending the time on this...

There is a performance regression in 0.34.0 that might cause this. I am currently working on this.

Just linking what Jest recently did to improve their summary output:

https://github.com/jestjs/jest/pull/15008


create VERSPM Model

Changed model data from Atlanta to RVMPO. VERSPM repo is now ready for testing and expansion of the model run script to include more modules.


JavaScript and TypeScript differences in coloring

Test item #18318

VSCode Version: Code - Insiders 1.9.0-insider (4265f5e35cb3ea8ba7e0cf6b14ec606b6f8647ea, 2017-01-24T10:29:13.076Z)
OS Version: Darwin x64 15.6.0
Extensions: None

Steps to Reproduce:
Atom One Dark Theme 0.3.0
1.8:

1.9:

Many differences, 1.9 has much more red. The source is from our repo.

JavaScript with Atom One Dark Theme 0.3.0
1.8:

1.9:

Marking as important for awareness.

Reloading 1.9 produced the same result.

Also Red's background and tab-rules where more red in 1.8.
1.8:

1.9:

✅  Atom One Dark Theme 0.3.0 Case 1 (TypeScript)
Before
It was matching the theme rule support.class (due to CSS class name matching):

After
It does not match the theme rule support.class which is correct according to TM theme matching rules

Conclusion: No bug here unless TS wants to change its grammar (i.e. current TS grammar + Atom One Dark Theme looks now the same in VSCode, TextMate and Sublime)

✅ Atom One Dark Theme 0.3.0 Case 2 (JavaScript)
Before
It was matching the theme rule variable.meta.function.readwrite.js (due to CSS class name matching):

After
It does not match the theme rule variable.meta.function.readwrite.js which is correct according to TM theme matching rules, but it matches some other rule variable, which is correct.

Conclusion: No bug here unless TS wants to change its grammar (i.e. current JS grammar + Atom One Dark Theme looks now the same in VSCode, TextMate and Sublime)

Red theme background
This is a regression, but it is unrelated to syntax tokens -- created #19344 to track it.


ci: address permissions for trivy-scan to upload

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

:tada: This PR is included in version 3.5.4 :tada:
The release is available on:

GitHub release
v3.5.4

Your semantic-release bot :package::rocket:


url obfuscated because too long creates several issues

blog.adobe.com has a lot of articles where the url is pretty long (they use the article title as page name). This creates several issues in the reporting:

url becomes /en/publish/2024/10/14/... which does not allow to identify the page
url cannot be used as a link
several articles published the same day ends up being the same "url", i.e. all data are grouped

While obfuscating ids or hashes is ok for PII reasons, the length shorting has no valid reason, except maybe aesthetic which should be managed in the consuming application, not in the data.

The length obfuscation is a heuristic, like all others. Of the sites with extra long URL segments, I'd assume blog.adobe.com to be an outlier.

If you don't like the obfuscation, why not just use plainURL?

Well, if I have to change the RUM explorer and Oversight to use plainURL, then URL becomes almost useless no ?
It is not I do not like obfuscation, I do not like the fact it corrupts the data and leads to wrong results. It is not blog.adobe.com specific, but probably a blog site specific issue (business.adobe.com, telegraph.co.uk have the same issue). And that's the ones we have potential stronger need of reporting. Today I cannot share the RUM Explorer with the business users because they do not find their pages.

oh and this solution is completely arbitrary: do we have a case where we are shortening something more than 60 characters that are something else than a long file name ? I checked multiple domains (adobe.com, telegraph.co.uk, bamboohr...), shorten urls are always valid pages.

Made a new proposal: if segment length is great than 60 characters, check if the segment follows the pattern:

contains only characters, digits, dashes and underscores
contains at least 3 dashes or underscores

60 characters would mean at least 4 words, less being suspicious and unexpected.


Enable wait time between retry attempts in Virtual Service configuration

Enable provision of wait time between retry attempts in Virtual Service Configuration
Describe alternatives you've considered
The workaround is to increase the retry attempts.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context
Consider a scenario where the destination server is not reachable due to some temporary network problem or the destination gateway is restarting.
During this time, if my application sends a request to the destination server, I will get an HTTP error.
If I wait for some very small time (few ms) and then retry again then my request may be processed successfully.
Hence, I need an additional field in Istio's VirtualService defining the wait time between different retry attempts.

This is handled automatically. See https://www.envoyproxy.io/docs/envoy/v1.7.0/configuration/http_filters/router_filter#x-envoy-max-retries


ENH: Use 3.11

Now that https://github.com/conda-forge/sleepecg-feedstock/pull/22 is in let's see if it works!
Also sets us up for 1.5 because we'll probably release that soon

Worked for everything but Linux! On Monday I'll look to see where the conflict is coming from

This means we can start shipping TensorFlow now too

Held up by https://github.com/conda-forge/vtk-feedstock/pull/302 (and maybe https://github.com/conda-forge/mayavi-feedstock/pull/75)

Now held up by https://github.com/conda-forge/xorg-libxt-feedstock/issues/14

All green, in it goes!!!


Remember focus per-workspace

Description
Current behaviour seems to be that, when switching workspaces, the focused window on the workspace you're switching to matches (as best it can) the focus of the workspace you're switching from.
i.e. if I'm on workspace 1 with three windows open and focus on the first window, then switch to workspace 2, also with three windows open and switch focus to the third window, then switch back to workspace 1 and focus is on the third window.
It'd be nice if workspaces 'remembered' which window is focused. Or just focus the first window when switching (I did the latter with keybinds but that breaks monocled workspaces out of monocle mode when switching to them).
Apologies if this is mentioned elsewhere - I did search - or possible in configs - I did try!

I just tried a blank config and it definitely doesn't behave that way for me (and never has).

works for me, do you have follow_mouse != 1

Ahh okay, so it works how I expect with follow_mouse = 0. But... I also want focus to follow mouse. Just not on workspace switching. i.e. after switching it should only follow the mouse after the mouse moves.

use 2 to change focus when you click the window

Yeah that's absolutely fair enough - but not what I want.

having to move mouse isn't a solution either since any slight vibration would change the focused window out of nowhere
warping mouse to center of focused window on workspace change doesn't seem like a good idea as well

any slight vibration would change the focused window

I've literally never had that happen in all my years of using WMs!
You've given me a clue though. What I was trying to do before was adding a binding (on the same keys) after switching workspace to focus the first window. But, as mentioned, this popped monocled workspaces out of monocle.
Instead I could just add a binding which move the mouse to top left (or, ideally centre of first window if I can figure that out) after switching.

On digging a bit deeper into the wiki:
mouse_refocus

If disabled and follow_mouse=1 then mouse focus will not switch to the hovered window unless the mouse crosses a window boundary.

Sounds like exactly what I want. But on setting mouse_refocus = false either it's doing nothing or I don't understand what it's doing and it's not what I expected.

After https://github.com/hyprwm/Hyprland/commit/b884544ee63b4c3b3b150ff69cf24c90058c2777 this seems to be fixed for me.

Sounds like exactly what I want. But on setting mouse_refocus = false either it's doing nothing or I don't understand what it's doing and it's not what I expected.

I also have no clue what mouse_refocus is meant to do. Didn't notice any difference in behavior after disabling it.

That definitely fixes it when switching on the same monitor, but when switching to another monitor, no window is focused, which is not good.
Definitely getting there though!
I greatly appreciate the work.

As far as I can tell this all works great now. Gonna close this issue.
Thanks for the work, devs. Greatly appreciated <3


Overlapping of inline codes

You can see one of the problem at https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-client/generating-prisma-client#the-prisma-client-npm-module
Currently it's overlapping

I'm not getting the same overlap: https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-client/generating-prisma-client#generating-prisma-client-in-the-postinstall-hook-of-prismaclient

Thanks @rahul3v for submitting a pull request to fix 💚


TypeScript definitions

Hey @feross, are you open to including TypeScript type definitions in the project?

Sure. My only reservation is who will maintain the types in the future? I don't use TypeScript myself.

Without someone volunteering to maintain them, I'd rather have no definitions than out-of-date ones.

Ok, I am the volunteer, since my stuff depends on safe-buffer.

Worth reconsidering?

Sure, want to send a pull request with the types file and I'll take a look?


Rebase off Dev

Allows DMs to create dynamic paths that are not map or position specific, but rather follow grid paths.
As of now, DMs will need to figure out the Path Code based on the clues until I build the Path Building tool.
Once a path is created (comes with 3 testing paths), the DM can select a token and start the pathing.
The interval is adjustable to help balance latency.
Multiple paths can be assigned to a token for intricate layered pathing.
Comes with full-control menu and macro to call menu at will.
Watches for object changes and removes from path when necessary.
Allows object Lock-in-Place and Following

Hey @sschwarz25 dev is actually a deprecated branch. All branches should be based off of master and have the base set to master in the PR as well. Please adjust accordingly and re-submit your PR. Thanks!


Add support for CoffeeScript sourceMaps (fixes issue #3)

This patch adds support for CoffeeScript source maps. If sourceMap is true then the generated source map will be inlined as a data-uri.
I've tested this in Chrome 28 and FireFox 24 – it works fine in the developer console, but unfortunately their JavaScript engines don't use source maps when generating exception stack traces, so it can still take some work to track down the exact line that an exception is failing on.

Does this feature make the uri a link, or just display?  My just displays:
at Object.<anonymous> (/Users/kimardenmiller/Dropbox/Spokenvote/Git/spokenvote/test/spec/services/voting_service_spec.js:384:22 <- voting_service_spec.coffee:591:21)

Should it be clickable to open that location?


Infinite loading not following window scroll.

Version
2.2.3
Vue.js version
2.5.2
What is expected?
Users scrolls down and content gets loaded
What is actually happening?
Users have to scroll within an "inner div" to trigger infinite loading
How to reproduce this problem?
Visit the below site on mobile:
https://kontinentalist.com/blog
Check it out in the "Latest Stories" section

Sorry for late, I cannot reproduce it, did you fixed it now?

Hi
I have some problem
url http://test.narod.io/buy-apartment
template http://joxi.ru/V2VbQBEcxLpEx2
js http://joxi.ru/D2PzbDGUp4Qjkr
you know how fix?

http://joxi.ru/RmzEpqNH0lXVpr
not helped
i have infinity cycle

@Mikkou did you forgot to add infinite-wrapper attribute on the real scroll parent? In your case, it is the body element.


SVG export: add image size

The SVG root element has no width/height, so they end up drawn at unspecified size
Includes "width" and "height" attributes in root SVG element, including 1x/2x/3x scaling.
Fixes #2291

I tend to agree. It's more intuitive for the SVG to fit to container by default, and it's easy to constrain the size via CSS or editing the SVG file, if needed.

My use-case is that I draw a lot of individual diagrams, and embed them in web-pages as images.
Without size information, these draw at all different scales. One diagram (which was "larger") draws with thin lines and small text, but another (which was "smaller") is drawn with giant lines.
With PNGs, this works as expected: the diagrams have an inherent size, which is used both for PNG export (that's why "2x" is meaningful), and also as displayed in Excalidraw itself!
That size information is lost in SVG export.  My current workaround is: open the SVG, look at the viewport size, and add that explicitly to either the HTML embedding or the SVG itself - either way, this has to be re-done when editing the diagram, because that might change the size.
Anywhere the size is specified with CSS or whatever would still work.


Translation "warmerwijk" is missing

Assigning @marliekeverweij as you oversee the deplay. IMO this is not deplay critical.

https://github.com/quintel/etmodel/commit/523d315de6b86208bcdda46405494fddf79dba9b


Vendorizing all dependencies.

The projects has lots of dependencies which makes it difficult to build and run the examples.
Sometimes the version of the dependencies could vary from what the author has, and that might make development very difficult for others.
Vendorizing all dependencies for easier building/running of code and examples.
This will help in dockerzing project too.

Current coverage is 60.41% (diff: 100%)

Merging #10 into master will not change coverage

@@             master        #10   diff @@
==========================================
  Files            20         20
  Lines           970        970
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            586        586
  Misses          358        358
  Partials         26         26

Powered by Codecov. Last update ebe7283...9e2f17e

Need to use glide instead of checking in all the files.


The cmd+i isn't working to inspect a function, autocomplete isn't working at all

Description
What steps will reproduce the problem?

select any function and type cmd+i
nothing happen
type dot after any function nothing happen

What is the expected output? What do you see instead?
1st case the help should be opened
2nd case the autocomplete panel should be shown
Please provide any additional information below
nothing, so simple like this
Version and main components

Spyder Version: 3.1.3
Python Version: 3.6.0
Qt Versions: 5.6.2, PyQt5 5.6 on Darwin

Dependencies
pyflakes >=0.6.0 :  1.5.0 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
nbconvert >=4.0  :  5.1.1 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
numpy >=1.7      :  1.11.3 (OK)
sphinx >=0.6.6   :  1.5.3 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
jedi =0.9.0      :  0.9.0 (OK)
psutil >=0.3     :  5.2.0 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
sympy >=0.7.3    :  1.0 (OK)
pylint >=0.25    :  1.6.5 (OK)

Things are working as expected in our tests, so I don't know what's happening in your case, sorry.

Closing because I don't know what's happening for you, sorry.


Consensus-Full-Nodes - resources usage issue

Summary of Bug
Hello team! 👋
I want to report to you an issue that we are facing with the consensus-full-nodes, we have been facing issues when trying to run the nodes with less than 20GB of RAM, by the time the nodes have to sync the chain (for example in mocha) the nodes cannot sync with less than this amount of resources. Even if we set the maximum to 20GB they are trying to get all the resources of the server until they get OOM and crash.
This happens if the nodes have to sync either from scratch or since some days ago.
I consider, that they should work with the resources that they have, even if that means that it's going to take longer to sync, but crashing because of that, might be an issue.

I assume that the nodes should work even with 8GB, as we have here
cc: @celestiaorg/devops
Version
v1.3.0
Steps to Reproduce
Start a consensus-full-node and connect it to an existing chain (mocha for example), then, it will have to sync to catch up.
For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Thanks for the issue! A few questions:

Based on the screenshot: do you have any idea on why this is happening for consensus-full-snapshot-0 and not for consensus-full-2 or consensus-full-3? How does conensus-full-snapshot-0 differ from the other consensus nodes? Perhaps a config change?
Does this repro on other chains besides mocha-4?

I think we ran into this before, and becuase of all of the ibc memo spam on mocha, the tx-index will eat gobs of memory. Was the tx-index configured to be on @jrmanes ?

Based on the lines you linked, it looks like the tx indexer is enabled and set to kv which uses goleveldb. If the consensus nodes don't need to index transactions, we can remove those lines because the default is null which disables tx indexing. If we do need tx indexing, we can explore alternative db_backends and/or the psql option.

if this is purely related to the kv indexer, can we perhaps close this issue and open a new one to improve the KV?

we should be able to close this issue once we are able to run v2 in production, as that includes a new version of the kv that should remedy the massive amounts of memory used by the existing kv store

Nina backported https://github.com/celestiaorg/celestia-core/pull/1405 to celestia-core v1.38.0-tm-v0.34.29 which was released in celestia-app v1.13.0. Rachid bumped celestia-node to that release in this PR.
TLDR: we don't need to wait until celestia-app v2 is running in production. As soon as celestia-node cuts a release from main (likely v0.15.0), we can use the lightweight tx status work.


(WebStorm) Moving to core-team-repo

Requested changes completed.

This should be all set on my end.


Publish/Unpublish Network Containers via CNS

Expose 2 APIs to publish and unpublish network containers from CNS.
a.	PublishNetworkContainer
b.	UnpublishNetworkContainer
DNC calls CNS to publish and unpublish the network containers.

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/azp run

/azp run

/azp run

Please close on whether it is ok to use nm agent url here or not?

it looks like you are anyway handling it.

Reopening the PR which looks like was closed accidentally

/azp run

/azp run

/azp run

/azp run


⚠ 366.ru has degraded performance

In f5db1b4, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 9511 ms

Resolved: 366.ru performance has improved in 50c3f12.


unable to bind \cd

fzf_configure_bindings --directory=\cd

won't work but, something like:
fzf_configure_bindings --directory=\ck

works fine

fzf_configure_bindings --directory=\cd works just fine for me. Can you check your terminal settings to make sure it's not being intercepted by your terminal before that key sequence makes it to fish?
If you don't find anything, can you confirm that bind \cd ls works?

Oh yes. It was mapped to halfpagedown. I should've noticed.
I'm using alacritty terminal. Is there a way to use \cd for halfpagescrolldown in normal mode and for fzf in insert mode?

Yes, use bind --mode https://fishshell.com/docs/current/cmds/bind.html. bind it for fzf in insert mode only in your config.fish


Implement DMN 1.3 Group Element

The aim of this ticket is to add the Group node as defined in DMN 1.3 version.
Group
A Group consists of a rounded corner rectangle drawn with a solid dashed line that groups elements together informally.

The Group object is an Artifact that provides a visual mechanism to group elements of a diagram informally.
Groups are often used to highlight certain sections of a Diagram without adding additional constraints for performance.
The highlighted (grouped) section of the Diagram can be separated for reporting and analysis purposes. Groups do not affect the execution of the Decisions.
As an Artifact, a Group is not a DRGElement, and, therefore, cannot be connected to/from an Information Requirement, Knowledge Requirement, or Authority Requirement. It can only be connected to/from an Association.
DataModel changes
Definitions attributes and model associations
artifact: Artifact [0..*] Artifacts include text annotations, groups, and associations among DMN elements.
Group model associations
name: String[0.. 1] The descriptive name of the element.

Will be delivered as part of #577.


ADC Continuous read and LittleFS bug (IDFGH-14079)

Hi
I use an ESP32 Dev Module and I use IDF version: v5.1.4-497-gdc859c1e67-dirty (esp v3.0.3 in Eclipse Sloeber).
My problem is simple : when I use ADC Continuous read and I try to read/write to my LittleFS partition, I crash my program with the error : Guru Meditation Error: Core  1 panic'ed (Cache disabled but cached memory region accessed).
The debug backtrace doesn't help.
I read lot of things about this problem and try to add :
#define CONFIG_ADC_CONTINUOUS_ISR_IRAM_SAFE 1
#define CONFIG_GDMA_ISR_IRAM_SAFE 1

if sdkconfig.h and soc_caps.h files but we no success.
I don't know what to try anymore.
Any ideas to try ?
Thanks
Lionel

Hi @GitHubLionel
Please use menuconfig to enable these two configs and try again.  The linker won't put the function into IRAM if we define the macro by ourselves.

Hi Kainarx
Many thanks for your answer.
Problem, I use Eclipse/Sloeber and I don't know how to use menuconfig in this IDE.
I will ask Sloeber to know that.
Best regards,
Lionel


build against VC runtime: move ld_sprint to port.d

We might also want to replace the ldouble template with constructor call for real_t so it is easier to map it to another type if the host real type doesn't match the target.
This might make longdouble.d obsolete.

May still be relevant in gdc, time will tell.  I am not enthusiastic about the use of native floats that I keep seeing in these kinds of PRs.

I have added the implementation of strtod/f from the C version that sets ERANGE for overflows and underflows.

rebased and conflicts resolved.

Auto-merge toggled on

Thanks


Comments API doesn't work on Excel version 2006

I tried this API on Excel, but got "This operation is not implemented." I use the latest beta API.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 992f7232-6aa0-b170-ff3b-009a273e7586
Version Independent ID: 0fd9c00f-91d6-5036-48c9-e0e2e0690e78
Content: Work with comments using the Excel JavaScript API - Office Add-ins
Content Source: docs/excel/excel-add-ins-comments.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @dsonet,
Thank you for reporting this issue. Are you getting the error for all comment-related APIs? Or is it only happening for specific API calls, like when working with mentions?
If you have a Script Lab snippet to share, that would also help us figure out what's happening.

Hi @AlexJerabek,
var comments = workbook.comments;
"This operation is not implemented."
I still facing this issue when trying to add comments in Excel using Excel Javascript Api. You can see the error attachment image below for more detail. I'm getting the error for all comment-related API. Please help me to resolve this issue. Thanks


cache historical data

Please save downloaded Zips in data folder, if they are older than current year
Maybe add a config where to save them or "false" if you dont want to save at all. If nothing is set, defaults to "save to data folder"
Its not about the bandwidth, but the time improvement.
Example:

extract 20 different legacy futures one by one, without redownloading the entire file
while developing, shorten the time to wait to fetch the data, when you want to test your script again

The package is already using caching via @lru_cache. One the second load you should be able to see a noticable time improvement. Downloaded files are currently loaded into your tmp directory via tempfile. There are no zip files saved in the temp directory as they are loaded as an object directly into extract_text_file_to_dataframe(). I would not recommend loading the data into the data folder, I moved away from that approach as the tempfile approach is much cleaner IMO.

extract 20 different legacy futures one by one, without redownloading the entire file

I think this should already be available via @lru_cache or could you explain what you are experiencing when using for example legacy_report?

while developing, shorten the time to wait to fetch the data, when you want to test your script again

Should not fetch the data again because of @lru_cache.
Have you tried something like this?
def get_cot_data(cot_report_data = pd.DataFrame, contracts):
   ...  # your own implementation
   return df

contracts_1 = (
               "BBG COMMODITY - CHICAGO BOARD OF TRADE",
               "BLOOMBERG COMMODITY INDEX - CHICAGO BOARD OF TRADE"
)

contracts_2 = (
                "FED FUNDS - CHICAGO BOARD OF TRADE",
                "30-DAY FEDERAL FUNDS - CHICAGO BOARD OF TRADE",
            )

df_1 = get_cot_data(legacy_reports(), contracts_1)  # loads longer
df_2 = get_cot_data(legacy_reports(), contracts_2)  # This should make now use of @lru_cache

@lru_cache only works if the parameters are exactly the same
calling 3 times with 3 different Markets is not cached, every function is calculated completely
``
start = time.perf_counter()
df = legacy_report("legacy_fut", "CORN - CHICAGO BOARD OF TRADE")
print(f"\n\tExecution time: {round(time.perf_counter() - start, 3)} sec")
start = time.perf_counter()
df = legacy_report("legacy_fut", "SOYBEANS - CHICAGO BOARD OF TRADE")
print(f"\n\tExecution time: {round(time.perf_counter() - start, 3)} sec")
start = time.perf_counter()
df = legacy_report("legacy_fut", "COCOA - CHICAGO BOARD OF TRADE")
print(f"\n\tExecution time: {round(time.perf_counter() - start, 3)} sec")
``
Execution time: 10.733 sec
Execution time: 10.182 sec

Execution time: 10.193 sec

I have added a convenience function for that problem you have in 0.0.7.
Try this:
from pycot.reports import cot_report, legacy_report

df = cot_report(legacy_report("legacy_fut"), "CORN - CHICAGO BOARD OF TRADE")  # will load the full report (~ 10-15 seconds)
df = cot_report(legacy_report("legacy_fut"), "SOYBEANS - CHICAGO BOARD OF TRADE")  # cached results
df = cot_report(legacy_report("legacy_fut"), "COCOA - CHICAGO BOARD OF TRADE")  # cached results

Perfect thank you


Exception when retrievieng location name from id

Exception when trying to retrieve item name through session.Locations.GetLocationNameFromId(locationName);
Iterating through location ID's does not result in an error. Checked the spoiler log and there were no locations with the same id.
Exception in IL2CPP-to-Managed trampoline, not passing it to il2cpp: System.ArgumentException: An item with the same key has already been added. Key: 257
  at System.Collections.Generic.Dictionary2[TKey,TValue].TryInsert (TKey key, TValue value, System.Collections.Generic.InsertionBehavior behavior) [0x000dd] in <986ed57b9a8f4699a3c59a69eb05944a>:0
  at System.Collections.Generic.Dictionary2[TKey,TValue].Add (TKey key, TValue value) [0x00000] in <986ed57b9a8f4699a3c59a69eb05944a>:0
  at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement] (System.Collections.Generic.IEnumerable1[T] source, System.Func2[T,TResult] keySelector, System.Func2[T,TResult] elementSelector, System.Collections.Generic.IEqualityComparer1[T] comparer) [0x0009e] in <c8122f496f05432baca55ffb7d139a58>:0
  at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement] (System.Collections.Generic.IEnumerable1[T] source, System.Func2[T,TResult] keySelector, System.Func2[T,TResult] elementSelector) [0x00000] in <c8122f496f05432baca55ffb7d139a58>:0
  at Archipelago.MultiClient.Net.Helpers.LocationCheckHelper.GetLocationNameFromId (System.Int32 locationId) [0x0006b] in <17d96186d9fe43bbb3a20afc3fa3c45d>:0
  at TunicRandomizer.TunicArchipelagoClient.TunicArchipelagoClient.Connect (System.String host, System.Int32 port, System.String player, System.String password) [0x0013b] in <69b91c7de2bc4ae291624c6da21b8154>:0
  at TunicRandomizer.Patches.MenuPatches+<>c.<pushDefault_OptionsPatch>b__11_1 () [0x00042] in <69b91c7de2bc4ae291624c6da21b8154>:0
  at (wrapper dynamic-method) UnhollowerRuntimeLib.DelegateSupport.(il2cpp delegate trampoline) System.Void(intptr,UnhollowerBaseLib.Runtime.Il2CppMethodInfo*)

Sorry for the lack of response on my part. Was there a solution which you found or did you otherwise bypass this issue somehow? @Dr4g0nsoul

@Ijwu It was totally my mistake. I used item codes already present in other worlds. Putting an offset to have unique id's solved the issue ^^


Minecraft Docker Compose

Hello here is the mistake I get:
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000
/start: line 10: eula.txt: Permission denied
/start: line 11: eula.txt: Permission denied
ERROR: unable to write eula to /data. Please make sure attached directory is writable by uid=1000

Here is my docker-compose.yml :
version: "2"

services:
  minecraft:
    ports:
      - "25565:25565"

    environment:
      - EULA=TRUE
      - VERSION=1.7.10
      - TYPE=FORGE
      - FORGEVERSION=<IP_ADDRESS>4

    image: itzg/minecraft-server

    container_name: mc

    tty: true
    stdin_open: true
    restart: always

    volumes:
      - ./minecraft:/data
      - ./minecraft/mods:/mods
      - ./minecraft/config:/config

You will need to make sure that the host directory you have provided, ./minecraft is writable by uid 1000. It will either need to be chowned for that user or chmoded with wider write permissions such as 777.

Perfect ! Tanks you !

You’re welcome.


Font selection and new icons

Hi!, I do some refactor of font selection mechanism putting each type in a separate file.
#Maybe this attend issue #81
Added gitlab, googlescholar, matrix and signal social icons, some of them non-existent even in academicons font. (only selected if not defined before)
Maybe this attend issue  #26 #38
I hope that helps.
Have nice day!

Cool, thank you very much! I gave it a quick review and it looks good, but I haven't taken a deep dive yet. I will do it this week (just ping me if you don't hear anything). With this merged I'll fix #73 and do a new release, hopefully this year 🎉
Btw @gcorrad do want to join the team? No duties, we just try to keep this package alive.

Thank you very much Stephan, and yes, I would be proud to participate please let me know any details you find or suggestions.

Thank you very much Stephan, and yes, I would be proud to participate please let me know any details you find or suggestions.

Cool! I invited you to the team. This allows you to push commits and merge PRs. We don't really have a "real" policy, but in general work should be done via PRs and then merged (best if another member takes a look at it, especially if it is a larger change like e.g. this one).
I'll test the code this evening (ping me if I forget it).

Thank you, I accepted it, don't worry, I'll be careful

Thanks I'll be attentive

FYI: I just tagged a new release
cc @ShadowMitia @mefuller

Great - @stephanlachnit is this also being pushed along to CTAN or is additional work needed to update there as well?

Nope, I uploades it, but they'll have to review it first


TypeError: init() takes at least 3 arguments (2 given)

Hi, I clone the code and run it. But it seems that Convolution1D needs to be replaced by Conv1D first. And after the replace, I get the following complaints
conv = Conv1D(filters=num_filters, kernel_size=sz, padding="valid", activation="relu",strides=1)(z)
TypeError: init() takes at least 3 arguments (2 given)
I try removing (z) but the error message wont change. I also try add a "step" argument (=1)  to input_shape but it does not have any effect either.
Any idea on how to fix this? Many thanks.

Hello,
With Keras 2.0.6 and Python 3.6.1 this code runs out of box, provided other necessary packages installed. Hope this helps.


downloading file in progress / failed file throws error

020-05-19 07:58:52,666 INFO     lbry.file.file_manager:126: claim contains a metadata only update to a stream we have
2020-05-19 07:58:52,797 INFO     lbry.blob_exchange.client:159: downloaded 8f1ea93c from <IP_ADDRESS>:5567 at 3.240000MB/s
2020-05-19 07:58:52,810 INFO     lbry.stream.managed_stream:255: write blob 640/1603
2020-05-19 07:58:52,924 INFO     lbry.file.file_manager:157: already have stream for the-bellflower-car-show-2017#9
2020-05-19 07:58:52,925 ERROR    lbry.file.file_manager:241: Unexpected error downloading stream:
Traceback (most recent call last):
File "c:\users\thoma\documents\lbry-sdk\lbry\file\file_manager.py", line 161, in download_from_uri
stream.downloader.node = source_manager.node
AttributeError: 'NoneType' object has no attribute 'downloader'
2020-05-19 07:58:52,928 WARNING  lbry.extras.daemon.daemon:1117: Error downloading the-bellflower-car-show-2017#9: 'NoneType' object has no attribute 'downloader'

I was on an earlier branch, seems to be fine now.


Extension does not work for some searches

I have noticed that when I search some specific cities such as Dallas or Atlanta, your extension does not work.

Hi @PaoloCortezCZ thank you for creating this issue, I have indentified a few occurences where the UI elements are not properly selected (google has a ton of variants of map UI and they also run AB tests which obviously make the html markup a little bit unpredictible. I will try to fix it in the next couple of days and release a new version.

Fixed in ce4323df3528fcb2d09fc6a308c9899a84cf2171


fix.validate_confirmation

If field is required, the code worked fine.
But when the field itself was optional, we had to make the confirmation field as optional (required: false) because if we make it required it will be invalidated when field is not provided.
Example -
@cast_key [
:first_name,
:last_name,
:mobile_number,
:email,
:password,
:password_hash
]
@required_key @cast_key -- [:password, :password_hash]
@primary_key {:id, :binary_id, autogenerate: true}
schema "customers" do
field :email, :string
field :first_name, :string
field :last_name, :string
field :mobile_number, :string
field :password, :string, virtual: true
field :password_hash, :string
timestamps()
end
def changeset(customer , attrs) do
customer
|> cast(attrs, @cast_key)
|> validate_required(@required_key)
|> validate_length(:password, min: 8, max: 26)
|> validate_confirmation(:password, message: "does not match password", required: true)
|> hash_password()
end
Now In this case even if we don't pass password (as its optional ) the validation_confirmation adds error. (Fixing this issue)

@SumitPugalia thanks for the PR!
Quick question: it seems that the behaviour you want to achieve is the one you would get by not passing required: true, no?

@SumitPugalia thanks for the PR!
Quick question: it seems that the behaviour you want to achieve is the one you would get by not passing required: true, no?

If I do so, then validation_confirmation will not add error even if i don't pass confirmation field..
%{password: "password"} will not add error ..

Which is intentional. Confirmation is a UI based feature. You don’t want to
require confirmation for things like API, where the confirmation should not
be sent.
In a browser, if you include the field, it is always sent and therefore it
should always trigger!
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Director of R&D

Which is intentional. Confirmation is a UI based feature. You don’t want to require confirmation for things like API, where the confirmation should not be sent. In a browser, if you include the field, it is always sent and therefore it should always trigger! -- José Valimwww.plataformatec.com.br http://www.plataformatec.com.br/Founder and Director of R&D

Agree 👍
Thanks for that.

Cool. So do you think this issue still applies? It seems removing the
required is the best option.
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Director of R&D

Cool. So do you think this issue still applies? It seems removing the required is the best option. -- José Valimwww.plataformatec.com.br http://www.plataformatec.com.br/Founder and Director of R&D

Yes I can use it as required: false. I was trying to achieve something that cannot occur in real scenario.
I realised confirmation field should always come with the field


When creating multiple rich texts at once, there is a noticeable lag

Jodit Version:  3.24.7
Browser:Chrome
I need to render multiple rich texts at once, but it takes 200ms to render one. So I want to create instances through web workers. However, when I use it, it prompts ‘document is not defined’. Please help me with this issue

hmm, maybe you need to reduce the number of plugins? The editor shouldn't take so long to initialize. In autotests, it takes an average of 70ms