more specific header selection

in my document i want to only generate a table of contents for the headers inside the MAIN tag. outside this main tag is a header for the full page, a header for the table of contents, and another header for an acknowledgements section.
i think the most versatile way to allow this and other complex header management would be to allow css selectors for headers. for example, i could set the option to ["main h2", "main h3", "main h4"]. numbers can keep the same behavior as before for compatibility. it would also allow using any other elements for headers.
i might be able to write a pull request for this later.

markdown-it and its plugins operate on the Markdown level, not on the HTML level. The output is HTML. Hence, while building the table of contents, the structure of the target HTML is not known yet. So there is no querySelectorAll for main h2 and so on.
The plugin basically counts all markdown-it node with token.type === 'heading_open' and takes the level from there. For easier understanding, the level is called h1, h2, etc. in the plugin configuration, but it has nothing to do with a DOM query selector.
What does your input Markdown file look like from which you want to select DOM nodes?

i started using this plugin after a discussion to possibly replace the snap reference manual with markdown (currently it's a word doc, converted to a pdf) i'm trying to keep it mostly matched up, and as one document so it can still be printed to a nice pdf afterwards.
here's my current outline (condensed to the relevant parts):

Snap! Reference Manual

## Table of Contents

[[toc]]

## Acknowledgements

...

<hgroup>

Reference Manual

Version 10.0

<hgroup>

(about the program)

(many more nested headers, regular markdown past this point)

the title, table of contents, and acknowledgements shouldn't be part of the table of contents. only what's in  is the actual manual.


750 - add additional info (for Linux) to README about the limit memor…

…y config
Closes: https://trello.com/c/BeRTxAji/750-troubleshoot-react-docker-container-and-update-readme-once-solve-has-been-identified

Can you add that to readme Philipp?
On Sun, Oct 17, 2021 at 4:26 PM P M @.***> wrote:

@.**** approved this pull request.
In README.md
https://github.com/boxwise/boxtribute/pull/92#discussion_r730376060:

-NB: In case you get out-of-memory related errors, make sure your max memory is at least 4GB in your Docker settings (via Docker Settings UI -> Resources -> Memory) and try again.
+NB: In case you get out-of-memory related errors, make sure your max memory is at least 4GB in your Docker settings (via Docker Settings UI -> Resources -> Memory) and try again.
+In (Linux) Docker there is no UI to set the memory limits globally. In that case, please try to specify the following in the docker-compose.yml for the react container:
+
+- mem_limit: 4G
Yes. However this config key is not supported by docker-compose-yaml of
version >=3, so I also have to put version: "2.4" or similar at the very
top.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/boxwise/boxtribute/pull/92#pullrequestreview-781422007,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABP2BFFFYRAORSSL5J5SY6LUHKCD7ANCNFSM5GEQFO3Q
.


Getting inline URL

Hello,
I’d like to add the inline URL in my custom download page, but there is no way to get just the random part of the string, nor the relative URL. I tried to do a replace on the get URL, but I didn’t find how it can be done (your templating seems a little bit more advanced than basic Go templating, but I didn’t found what is it).
Thanks.

That’s perfect, thanks a lot.

@Br31zh I'm going to rename {{ .UrlRandomToken }} to {{ .URLRandomToken }} due to go linting in master on the backend


build(golang): dumps golang req to 1.16

Testing Brno Cluster Fedora 34 Cluster

Closing as it became irrelevant :)


navigating to case studies from the case studies page

interacting with a company logo on the case studies page should also take you to the case study page

this has been fixed

i did not miss you issue label bot.


Add XPATH to click breadcrumbs

For complex UI, the click breadcrumbs don't have enough detail - can you not include the XPATH of the element that is clicked? Otherwise its sometimes not possible to work out which element was clicked e.g. if there are multiple nested DIVs within a parent div, the breadcrumb could be rendered as DIV > DIV. This doesn't tell you which DIV was clicked.

Originally brought up this issue in https://github.com/getsentry/raven-js/issues/576

@takahashi64 – I'm not sure that we're going to do this or #576 for a few reasons:

Breadcrumbs are intentionally supposed to be lightweight hints, they have character limits, and XPath statements can grow arbitrarily large. Also, the code for generating a full XPath string will likely bloat the library re: filesize.
I've erred on not showing the exact click content for privacy reasons (e.g. in case the links clicked contains sensitive data).

@benvinegar that makes a lot of sense yeah. Is there someway to make Raven extensible/pluggable so I could add custom logging to the sentry reports? For complex UI, it would really help if we knew what the user was doing. Thanks.

We did add breadcrumbCallback, but it doesn't pass the original "object" that created the crumb, so you unfortunately can't construct your own XPATH string.
I think the only workaround presently is to basically instrument the DOM yourself and leave your own custom breadcrumbs.


Incorrect stratum host

You parse string to retrieve the host, but never use it and pass location to stratum
https://github.com/nicehash/nheqminer/blob/master/nheqminer/main.cpp#L423
https://github.com/nicehash/nheqminer/blob/master/nheqminer/main.cpp#L428-L432

Please fix this.

I've just compiled a nheqminer_cpu on Ubuntu 16.04. Have such problems:
./nheqminer_cpu -t 8 -u wallet
[15:20:23][0x00007f121f0f0700] stratum | Connecting to stratum server eu:eu
[15:20:23][0x00007f121f0f0700] stratum | resolve: Service not found
[15:20:23][0x00007f121f0f0700] stratum | Reconnecting in 3 seconds...
./nheqminer_cpu -t 8 -u wallet -l usa
[15:20:56][0x00007fae79cb0700] stratum | Connecting to stratum server usa:usa
[15:20:56][0x00007fae79cb0700] stratum | resolve: Service not found
[15:20:56][0x00007fae79cb0700] stratum | Reconnecting in 3 seconds...
./nheqminer_cpu -t 8 -u wallet -l equihash.eu.nicehash.com:3357
[15:23:24][0x00007f9f15446700] stratum | Connecting to stratum server equihash.eu.nicehash.com:3357:3357
[15:23:24][0x00007f9f15446700] stratum | resolve: Host not found (authoritative)
[15:23:24][0x00007f9f15446700] stratum | Reconnecting in 3 seconds...
./nheqminer_cpu -t 8 -u wallet -l equihash.eu.nicehash.com
[15:23:39][0x00007f3bb85df700] stratum | Connecting to stratum server equihash.eu.nicehash.com:equihash.eu.nicehash.com
[15:23:39][0x00007f3bb85df700] stratum | resolve: Service not found
[15:23:39][0x00007f3bb85df700] stratum | Reconnecting in 3 seconds...
So, it can't be used anyway.

Simple fix is to change line 429 and 432 to host, port, user, password, ...

gumblex, thank you, these changes solved my issue, even i still should use -l with real host and port :)

Fixed.

Can someone Help me connect to the server, europe server gives me the same issue
~ $ cd nheqminer/nheqminer
~/nheqminer/nheqminer $ cd build
~/nheqminer/nheqminer/build $ ./nheqminer -l usa -u 1MhHAo9cWRWvVwGHVZoJC4oRfQ8eSBSwR3.MassimoT -t 1
Kost CPU Miner - https://github.com/kost/nheqminer kost_nheqminer/0.3.8
Thanks to Zcash developers and community, nicehash, tromp and xenoncat. Donate!
BTC:1KHRiwNdFiL4uFUGFEpbG7t2F3pUcttLuX ZEC:t1JBZzdaUUSJDs8q7SUxcCSzakThqtNRtNv
Setting log level to 2
[23:03:05][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:05][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:05][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:08][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:08][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:08][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:11][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:11][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:11][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:14][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:14][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:14][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:15][0x00007fd3bf6a7740] Speed [300 sec]: 0 H/s, 0 Sol/s
[23:03:17][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:17][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:17][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:20][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:20][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:20][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:23][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:23][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:23][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:25][0x00007fd3bf6a7740] Speed [300 sec]: 0 H/s, 0 Sol/s
[23:03:26][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:26][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:26][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:29][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:29][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:29][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...
[23:03:32][0x00007fd3bdb07700] stratum | Connecting to stratum server usa:usa
[23:03:32][0x00007fd3bdb07700] stratum | resolve: Service not found
[23:03:32][0x00007fd3bdb07700] stratum | Reconnecting in 3 seconds...

explain it like i'm 12


[#91211938] Modified cck --auto functionality and added appropriate tests

Signed-off-by: Thulasi Nandakumaran<EMAIL_ADDRESS>

Merged to master with 05ee5809efa261f3ccea6af05307fa7bfba5217c


Set sessions to automatically be published, display in list and engageable

[x] Change default published setting in ctype to published
[x] Investigate and possibly modify COD session view(s)

[x] Proposed session list vs. accepted vs. ?
[x] Contextually filter session view by session track term name
[ ] Make session rows output purdy
[x] Search by title or keyword (views combined filter or search?)

[x] Ensure comments are open by default and allowed by authenticated user
[x] Add comment form and comments to session panel

[x] Publish all sessions live on site and enable comments

Post code update:

[x] Update program page (/drupal-govcon-2015/upcoming_program) to include link to proposed sessions view
[x] Add Proposed Sessions menu item (/drupal-govcon-2015/sessions) under Program main menu item
[x]  Set all published sessions to new status of "Proposed"

Voting on sessions?

@dsheffler Do we also need to look into user voting on sessions? I think COD comes with something OOTB for this. Not sure whether it's intended for participants or only for planners/schedulers.

Someone overrode the feature on the production website to publish sessions automatically. Repeat and export feature override. Feature: cod_session

Comments wasn't being output by the session panel included with COD.

Publish all sessions live on site and enable comments

Kirsten says she'll do this piece.

@jonraedeke Please have a look at the view I've made and make it purdy. You can see it in a local copy or the Acquia dev environment at http://HOSTNAME/drupal-govcon-2015/sessions (you need to change the status of some sessions to "Proposed" or "Accepted". If accepted append /accepted to the view. The view is grouped by session track and filterable by that with the last argument. Eg. http://HOSTNAME/drupal-govcon-2015/sessions/proposed/business-and-strategy
I didn't get to the session tracks view/summary to add to the top like on DrupalConSF

Make session rows output purdy

@timwood, the session track descriptions are as follows:
Code and DevOps
This track is focused on developers and the back-end technologies to deal with today’s and future challenges. With the coming release of Drupal 8, as well as emerging Web technologies, preparation is essential. These sessions will help you learn how to deliver effective solutions to meet these needs.
Business and Strategy
The purpose of this track is two-fold - one angle helps you to understand the advantages of Drupal from an organizational leadership angle and the other is to highlight the great Drupal sites and projects you're working on and to show what's possible. These sessions are business focused, and will be non-technical.
Site Building
Learn how to leverage the flexibility of Drupal to deliver experiences in nearly limitless ways. See how you are able to do this—often without code and relying on the ever-expanding universe of contributed modules.
Front End, UX & Design
This track covers delivering projects focused on the most important audience—the end user. Learn how to leverage Drupal to help you keep abreast of emerging trends in the mobile, componentized Web world. We’ll show you strategies for fostering collaboration between content creators, designers, strategists, and end-users.
Training
These all-day trainings focus on making your team successful in Drupal. As anyone in the field knows, finding knowledgeable practitioners and getting good training is challenging. Here, we show how to bring team members up to speed in Drupal, and where and how to look for outside assistance when internal resources are scant.

Thanks @dsheffler Karl for getting the session track descriptions written. I've added these to the live site. For now they appear on session pages to the right, under the track title.

And there is a taxonomy page for each, which we may replace/redirect elsewhere as part of this task to improve session listings.

The session tracks now appear above the new session view.

Thanks @timwood! I got a start on styling. I haven't added the icons yet, but those will go on the left of each track link. Heads up, you'll need to pull and revert dgc_deploy since I added a few classes to the view.
Sent from my iPhone

On Mar 29, 2015, at 5:13 PM, Tim Wood<EMAIL_ADDRESS>wrote:
The session tracks now appear above the new session view.
—
Reply to this email directly or view it on GitHub.

Looks awesome so far, Jon. What do you have planned for the session description? Can we do the roll-over idea by adding the description to the attached view? Or do you have some other idea?

Is it ok to disable the Proposed Sessions menu item until the styling is done for that page? Without proper spacing and theming, it's honestly a bit difficult to read.

Just an FYI that the page I'm referring to is https://www.drupalgovcon.org/drupal-govcon-2015/sessions.

I'm for leaving up the menu item. I think the page is more useful than no page.

I've pushed up @jonraedeke 's initial styling for the sessions section and re-enabled the Proposed Sessions link in the main menu. https://www.drupalgovcon.org/drupal-govcon-2015/sessions
FYI, when sessions close and we have changed sessions to "Accepted" status we can change the main menu link to point to https://www.drupalgovcon.org/drupal-govcon-2015/sessions/accepted.

@jonraedeke Noticed one bug. If you filter away all results in the sessions view, the no results text shows but is up by the track filter buttons, actually under it. Can you fix this when you work on it next?

Just pushed the styling that includes the session track icons. @timwood I also fixed the empty view result styling bug.

Thanks, Jon. What color will the speaker track be? These look good so far, as well as pop is concerned.

@charlieartist Here's png icons for reference (the site used svg). For the buttons on hover and active, the icons will be on white, so they will pop even more.

@timwood If you have time, can you fix the attached view for selecting session track so it doesn't output an empty div for 'none'. If not, I can try to style away the space it creates at the top of the list.

Something caused the breadcrumbs to get all screwy. I don't see anything obvious changes to the view in the git commits. HEre is a screenshot of what I'm seeing locally.

I cannot reproduce the breadcrumb issue in Acquia dev environment on the latest develop branch.

@jonraedeke Enabling the "hide if empty" setting under no results behavior for the session track field removed the empty nested divs inside the first row result, but didn't get rid of the entire row. With the contents of the first row div gone the extra space seems to be gone too. I still don't understand why it's showing that empty row to begin with.

@timwood Thanks! I think it's fine.


[Docs] Add persistent fixed header to docs

Deploy preview for gestalt ready!
Built with commit 74be5579f128f746ca451f57574661c33ee89546
https://deploy-preview-154--gestalt.netlify.com

Warnings

:warning:

Please add a changelog.md entry for your changes.

  Generated by :no_entry_sign: dangerJS

😍 Love the design, left 1 small comment

@christianvuerings do you mind re-reviewing? I added some more commits last night that tidies it up a bit.

This is amazing!! Only thing I noticed was the main content area feels like it could stretch more to the right edge, right now its not full width and leaves blank space on the right


Restore "language-" prefix for fenced code blocks

As referenced in https://github.com/tightenco/jigsaw/issues/424, the recent change in Markdown parsers from https://github.com/tightenco/jigsaw/pull/416 altered the default behavior for language class names applied to fenced code blocks. The previous parser (Parsedown) followed the Prism convention of prefixing language class names with language-*, whereas the new parser (PHP Markdown) omits the prefix.
This PR restores the previous Jigsaw behavior, to prevent breaking sites that relied on the language- prefix for syntax highlighting.
In addition, since PHP Markdown exposes this setting (and others) through a series of configuration options, this PR makes those settings easily configurable in bootstrap.php. Specifically:

it updates the MarkdownParser binding to use a singleton, and
binds the parser to a markdownParser key on the container

Now, any of the parser's configuration options can be easily set in bootstrap.php. For example, to remove the language- prefix from fenced code block class names, this line can now be added to bootstrap.php:
$container['markdownParser']->code_class_prefix = '';

Likewise, this PR will make it possible to include a custom function for generating IDs (as mentioned here).

Works great, but instead of hyphen you used an underscore 877fa65168f38ce35d86fcc6293a2db276edc216


augur-node changed property need to fix in UI.

Clubhouse Story
description on ch task
Submitter checklist

[ ] Test covering functionality added/fixed in
Check the linked story includes the following:

[ ] Steps to reproduce
[ ] Screenshot (If applicable)

Reviewer Checklist

[ ] Test/lint pass
[ ] Post merge, story marked complete

I'm not entirely clear what this PR is for.  The description on CH is just:
What are the display conditions for Position Close button.
If position has already been closed?
If 0 shares on position?
Need input from @serena

I noticed when Sprinkle did a trading demo and closed an open order. the order 'pending cancellation' didn't get cleared. I found that augur-node changed a property UI needed in order to clear the order.
The comment about 0 shares is unrelated to the code changes. Not sure why open orders would show 0 shares.

corrections made, only change is to listen-to-updates

LGTM


A few ideas.

Hi Dane/Crew,
I just wanted to vent a few ideas I believe would benefit Uncletopia. Firstly, While I was playing on one of the EU servers, the opposing team (RED on Upward) were losing quite badly to which more or less all of them voted to scramble without much say from BLU near the end of the game. I felt that this was a bit unfair to us who wanted to win. To fix this I believe only allowing a scramble to occur in the opening minute or two into the round would benefit the server.
Secondly, I think adding a couple of your favourite community maps would be a lovely addition to Uncletopia.
And lastly, maybe another EU server ;)
I hope this message finds you well Dane/crew, Uncletopia is already perfect how it is,
Kind regards, Daire.

Hi Doire5,
In my opinion, if one team is losing quite badly - more often than not - the teams are unbalanced and should be scrambled. So in the case you provided, it definitely seems like the vote-scramble option was being used appropriately. I do understand there is frustration when a win is cut short by unbalanced teams later on in a round, but it's just the nature of a community server that allows players to leave and join at will. It's always possible for multiple players to leave one team and be replaced with players who are considerably less skilled than the previous ones. So teams are able to become unbalanced quite easily at any point. So, as it stands, I believe the vote-scramble option being available at any point during a round is fine and should remain how it is. I will take your suggestion into account, though. I've been considering slightly changing the rules surrounding vote-scrambling, such as increasing the percentage of players required to pass, or putting scrambling on a longer cooldown to prevent repeated abuse. In the future as well, our admins are planning on implementing a custom statistics tracker that will better aid us in adding plugins that sort players onto teams better, hopefully resulting in balanced teams more consistently than the entirely random process that exists right now.
As for community maps, I feel like adding even one community map, no matter how well-designed it is, will only open the floodgates of people suggesting more and more maps that are not only unfamiliar to the average TF2 player, but come with their own set of inconveniences (such as requiring new players to download the map, possibly giving them longer connection times, and booting them from a full server before they can connect - or experiencing technical issues that are common on Valve-unapproved maps such as frame drops or lighting issues). But overall, I personally feel like the Casual+ / Vanilla Lite "theme" is what makes Uncletopia appealing and unique, and there are many many other quality TF2 community servers that host a wide variety of community-made maps. I feel like if we start branching out to custom maps, Uncletopia may lose a piece of its appeal that we've come to be so successful with.
I hope I've addressed you concerns, and thanks for taking the time to submit feedback. Hope you continue to enjoy Uncletopia!

In response to adding another EU server. We just added one last week, so it will be a while before we add another.
Please reopen this if you have any other questions.


question: does this do nested properties?

Why haven't you looked at the example or tried your self?


added extended chars to helpText

closes #68

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 97.151%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
1807

Relevant Lines:
1860

💛  - Coveralls


In Pro 1.2 LLOS output targets are symbolized correctly but not labeling with the number of observers that can see it

Expected Behavior
Linear Line of Sight output targets should be symbolized and then labeled with the number of observers that can see them (assuming that if it's zero we just leave the label off, even the ones that are symbolized green, meaning they are visible from something, are not labeling)
Current Behavior
Output targets are symbolizing correctly, but are not labeling. Using the test data, there is one target that is visible by three observers, it is properly symbolized as green but is not labeling with a 3. If i apply the output targets .lyr, they symbolize properly, but there is no label. If i apply the output targets .lyrx, they symbolize properly and have the label.

follow up to this... in 1.3 the output still does not symbolize properly, but the opposite is happening with the layer files .... 1. run tool... no label on symbols... 2. use the .lyr for targets, neither target symbols, nor labels show...3. use the .lyrx for targets, symbols show and label properly

still not lableing

Known issue in core geoprocessing as recent as Pro 1.3. Not much we can do for our tools until this one is fixed in core.

BUG-000101471 "Geoprocessing tools in ArcGIS Pro do not honor output layer file (.lyrx) labels for tool results."


need to define uncertainties in spectrum

useful example for aperture photometry given here
could extend thinking to compute error at each pixel

This could be a good task for @tdwilkinson to contribute too if she wants to try adding to the codebase!

Did a first pass at this. (link to commit) Gives reasonable values in the tests I ran.


v16 upgrade

osmosis-labs/osmosis:
tag https://github.com/osmosis-labs/osmosis/tree/v16.0.0
branch https://github.com/osmosis-labs/osmosis/tree/v16.x

osmosis-labs/cosmos-sdk:
tag v0.45.0-rc1.0.20230703010110-ed4eb883f2a6
branch: https://github.com/osmosis-labs/cosmos-sdk/tree/v16.x

wasmd  tag v0.31.0-osmo-v16


Replace text to copypasta so it looks better

Hi, I'm the developer of original page! (roblotwitter.com)
(I apologize for my bad English)
The Japanese text of roblotwitter.com is Japanese copypasta, The Bourbon House.
I used GitHub Copilot to translate to English, but I think the text should be famous English copypasta.
Could you replace the text so it looks more better?

Meaning updating the formatting to closer match the English version on that page you linked, or to a different copypasta entirely?

To a different copypasta entirely.
I don't know English copypasta so much so I used translation, but I think you know more about English copypasta than me.


Integration into vcpkg lead to syntax errors

I'm trying to get this library into vcpkg and while compiling the library through their pipleine I encounter a lot of syntax errors, could be a problem with their pipeline though.
vcpkg issue:
link
The following is the output log (partially in german):
config-x64-windows-out.log

MSVC complex number support is very different from gcc.
MSVC uses _Dcomplex instead of double complex
MSVC
_Dcomplex d = { 1, 2 };

gcc
double complex d = 1 + 2 * _Complex_I;

I've tried converting the code to accommodate the compiler differences, but i still get lots of errors, and not all of them seem to be related to the complex implementation.
My attempts can be found in my fork of the repo

after some googling I might be able to resolve the errors if you would like to move to cpp instead of plain c.
List of Errors still active:

complex numbers cant be multiplied inplace using msvc's implementation due to it being a struct
implicit conversion to comlex not possible and will require a lot of refactoring in the cart2sph.c file for example
some arrays seem to be 0-sized (compiler does not like that)
other arrays have an unknown size

It's a c project. It cannot be compiled with c++ compiler. The code follows c89
standard plus the a few features of c99 (variable-length array and inline comment).
I guess you may need to specify a compilation flag like -std=c99 in the build
pipeline.
On Sat, Dec 26, 2020 at 04:57:48AM -0800, Hendiadyoin1 wrote:

I'm trying to get this library into vcpkg and while compiling the library through their pipleine I encounter a lot of syntax errors, could be a problem with their pipeline though.
vcpkg issue:
link
The following is the output log (partially in german):
config-x64-windows-out.log
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/sunqm/libcint/issues/49

--

I'm not sure about complex number. For other issues, c compiler should work
correctly.
On Sat, Dec 26, 2020 at 10:38:15AM -0800, Hendiadyoin1 wrote:

after some googling I might be able to resolve the errors if you would like to move to cpp instead of plain c.
List of Errors still active:

complex numbers cant be multiplied inplace using msvc's implementation due to it being a struct
implicit conversion to comlex not possible and will require a lot of refactoring in the cart2sph.c file for example
some arrays seem to be 0-sized (compiler does not like that)
other arrays have an unknown size

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/sunqm/libcint/issues/49#issuecomment-751384186

--

even if switch to C99 standard, MSVC's complex number differs from gcc.

So due to that vast incopatability, I've looked into converting the code to cpp and got to the linker stage, but apperantly the implementation of dgemm_(...) is missing.

So due to that vast incopatability, I've looked into converting the code to cpp and got to the linker stage, but apperantly the implementation of dgemm_(...) is missing.

It's a BLAS function. You can use openblas or mkl if available.
On Sun, Jan 03, 2021 at 02:15:46PM -0800, Hendiadyoin1 wrote:

So due to that vast incopatability, I've looked into converting the code to cpp and got to the linker stage, but apperantly the implementation of dgemm_(...) is missing.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/sunqm/libcint/issues/49#issuecomment-753683799

--

It's a BLAS function. You can use openblas or mkl if available.
On Sun, Jan 03, 2021 at 02:15:46PM -0800, Hendiadyoin1 wrote:

So due to that vast incopatability, I've looked into converting the code to cpp and got to the linker stage, but apperantly the implementation of dgemm_(...) is missing.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/sunqm/libcint/issues/49#issuecomment-753683799

--

I've got it to compile under msvc, could you check if everything is still the way it should be?
The repository can be found here

I've got it to compile under msvc, could you check if everything is still the way it should be?
The repository can be found here

The new release 5.* changes the way to handle complex numbers. The real and imaginary parts of a complex number is treated as two number continues in memory.


Remove any mention of Gitpod from the README (issue 11732)

Since the Gitpod contributing work-flow is, from the PDF.js project perspective, completely unsupported we don't want to mention it in such a highly visible way as the README file.

Since Gitpod is a commercial service, we probably want to avoid potentially being seen as advertising and/or endorsing it by mentioning it (somewhat prominently) in the PDF.js README file.

By leaving the actual Gitpod files in the repository, for now, this should thus avoid outright breaking things for any existing users.

Fixes #11732
Fixes #11741

Looks like a good approach to me; thanks!


[HUDI-7507] Adding TimeGenerator support to 0.x branch

Change Logs
Adding TimeGenerator to 0.x branch.
Impact
Adding TimeGenerator to 0.x branch. This will ensure commit timestamps are monotonically increasing.
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Closing this PR as we discussed that we'll go with #11580 instead.


Change the platform delegate to allow us to use a customized path for trace processor binaries

If you look at https://github.com/google/perfetto/blob/main/python/perfetto/trace_processor/api.py#L31C1-L31C18, you see the PLATFORM_DELEGATE is defined as a constant, then it is later provided to the init() function:
def __init__(self,
               trace: Optional[TraceReference] = None,
               addr: Optional[str] = None,
               config: TraceProcessorConfig = TraceProcessorConfig(), # we can also provide a None here, see below
               file_path: Optional[str] = None, # we want to add platform_delegate, see below
               ):
    ...
    self.platform_delegate = PLATFORM_DELEGATE()
     ...

This is not easy for our cases where we have a customized binary path. To make our binary file downloadable from another place, we will have to make changes to both platform.py and api.py.
I want to make the following changes on api.py:
def __init__(self,
               trace: Optional[TraceReference] = None,
               addr: Optional[str] = None,
               config: TraceProcessorConfig = None,
               file_path: Optional[str] = None,
               platform_delegate: Optional[PlatformDelegate] = None,
               ):
    if config is None:
        config = TraceProcessorConfig()
    self.config = config
    if platform_delegate is None:
        platform_delegate = PLATFORM_DELEGATE()
    self.platform_delegate = platform_delegate
    ...

What do you think about this change? Basically we allow users to provide their own version of PlatformDelegate class (this can be a child of the original PlatformDelegate class, with a customized shell path to download the binary though).
I can help contribute the code.

Seems reasonable except I'd actually put in the TraceProcessorConfig object. It was designed exactly for this.

Seems reasonable except I'd actually put in the TraceProcessorConfig object. It was designed exactly for this. Contributions welcome.

Understood. But I still have to call
tp = TraceProcessor(trace=TRACE_FULL_PATH, config=TraceProcessorConfig(bin_path="some_temp_path"))

I would like to make the API look as easy as possible, so somewhere I want to overwrite the default bin path so I don't have to provide that every time.
from my.module.api import TraceProcessor
tp = TraceProcessor(trace=TRACE_FULL_PATH)

Hold on actually. Thought about this a bit more.  In your wrapper module why can't you just do:
import perfetto.trace_processor.api as tp

tp.PLATFORM_DELEGATE = <your custom impl here>

TraceProcessor = tp.TraceProcessor

Platform delegate was designed specifically to allow things like this. It's designed to be overridden in this way and I something we do ourselves.

That's also possible. Thanks for the feedback.
I guess you have 2 ways:

Get customized TraceProcessor and PlatformDelegate classes. So 2 files are modified. Then you can use the simply import like ```
from my.module.api import TraceProcessor
tp = TraceProcessor(trace=TRACE_FULL_PATH)

2. Or you can use the following code, with 1 file (customized PlatformDelegate class) changed but extra line specifying the PLATFORM_DELEGATE.

import perfetto.trace_processor.api as api
api.PLATFORM_DELEGATE =
tp= api.TraceProcessor(trace=TRACE_FULL_PATH)

As I said, I want the API to be as easy as possible, so implementation detail (PlatformDelegate) should not be exposed to our internal users if possible. The current API will support it, but I personally prefer option 1 because API will look better.

Yup you have those two options. I'm not opposed to also having something like you originally proposed upstream but if the current way works, I'd just stick with it for simplicity's sake.


[FEATURE] Add SSL flags usage for PostgreSQL to the docs

Is your feature request related to a problem? Please describe.
Add the documentation for the SSL flags usage for PostgreSQL
Describe the solution you'd like
Clearly explain the usage of the SSL flags and add it to the usage file in the documentation.
Describe alternatives you've considered
If possible also add an example of this into the tutorials section.

Clossing this issue, since this was introduced in v0.21.0


Fixes #83 Implemented Breadcrumbs for Collection/Object hierarch

Fixes #83
@ucsdlib/developers - please review

Coverage increased (+0.06%) to 91.667% when pulling 7b6682dc85b67d6761d412a11306c359e1bfd66a on feature/breadcrumb into fa4b2986fb004ac0c30617c52eeb3fed386c0e53 on develop.

I test it and it looks good for Object view. Some question:

Click on the file set link on the bottom of the object viewer, I see inconsistent breadcrumbs:

From the /catalog search result page, click on an object to the object view, I see no links/breadcrumbs back to the search result. Hyrax has a breadcrumb Back to search results.

Click on a collection, I see breadcrumb My Collections. Maybe it make more sense to have a breadcrumb to All Collections instead of My Collections since the cortex is changed from My Objects now?

@mcritchlow and @lsitu
For #3, It is default collection page breadcrumb which is not in this ticket scope, so I didn't touch it. I think maybe it's better to ask Gabriela, what does she want to see of collection page breadcrumb?
For #2, The following screenshot is the default one from the /catalog search result page . Do you want to have modified breadcrumb:
My Dashboard / Test Collection / Test Object
Or to keep the default breadcrumb?
Back to search results
My understand of this ticket was to use the 1st choice, but let me know if you want to kept old one.

Great questions. Here's my take.

It is true that the scope of the ticket is for what Huawei has in this PR. That said, I can see @gamontoya and other users potentially wanting both since they serve different purposes. That is, the static hierarchical breadcrumb trail + a link back to search results (when available in context). This is actually something that might best be left to usability testing.. what do you think, @gamontoya ?

This is an interesting point, and I suspect we are likely to want to remove ALL instances of "my" and "your" in the UI since we are not building an IR where individuals are uploading content. That said, as Huawei said it wasn't in scope for this ticket, and what's much more important is the end user/public view, for which this is not relevant.

My general take would be to leave the PR as-is and leave any further breadcrumb customization for usability testing, but I'd defer to @gamontoya there.

Thanks @mcritchlow  For #1, File set page breadcrumb, it need to implement in file_sets_controller.rb. But it seems there is no such relationship metadata in file_sets that link back to parent id. It is one way but not both way, which in object there is link data to child files, but not vice verse:

@hweng - thanks for providing this screenshot and clarifying. I would opt for not bringing fileset breadcrumbs in scope for this ticket. This is an entirely new concept, and so I don't think it's technically in scope for what folks are expecting from breadcrumbs. I would vote to leave such a thing for usability testing and/or design.

@gamontoya I agree with @mcritchlow to leave the PR as-is. I just brought up the issues that are related to the breadcrumbs for your reference and I think we can create a new ticket if you want any changes. What do you think?

👍

@mcritchlow @hweng I agree with you all. Leave the PR as it is and we can ask for customization during or after an accessibility phase for DAMS5.


extract archives from downloads

I need to find a tracker where this happen, if someone can help to find an example?

No one requested, did not find any case yet will close for now. Feel free to re-open.


Zooming without a scroll wheel

Martin brought up that not everybody has a trackpad with pinch-scrolling or a mouse with a scroll wheel. All they could do today is enable an option in settings to "use arrow keys to pan and Q/W to zoom", but they'd have to discover this.
How should we handle this? Google Maps and the OSM slippy viewer have +/- icons on the map. We've been trying to consolidate the number of on-map buttons, but maybe we should reconsider here.

Ungap and LTN have these


Add static files and assets to dist folder just by building (add webpack?)

Current you must copy the contents of a folder into the dist for the first build. This is not maintainable. Current we use parcel for building for ease of use, but as far as I know, you cannot copy assets in the src directory to the build directory.
This might mean switching back to webpack (which I would be okay). We orginally used webpack but it was overkill for our project. Now that we have assets and static files, it might be useful

done and merged


mac-8 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-8 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

mac-8 is up and running fine, machine is out of sync with Salt master. Run command “salt-call state.apply” to re-sync Salt master again.
flutter@flutter-devicelab-mac-8 ~ % uptime
11:01  up 21 mins, 3 users, load averages: 1.06 1.26 1.28


Console Spam every Movement i make with Enchants applied.

[04:24:03 ERROR]: Could not pass event PlayerMoveEvent to CrazyEnchantments v<IP_ADDRESS>
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302) ~[PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:78) ~[PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:517) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:502) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.PlayerConnection.a(PlayerConnection.java:273) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.PacketPlayInFlying.a(SourceFile:126) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.PacketPlayInFlying$PacketPlayInPosition.a(SourceFile:57) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.PlayerConnectionUtils$1.run(SourceFile:13) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) [?:1.8.0_144]
at java.util.concurrent.FutureTask.run(Unknown Source) [?:1.8.0_144]
at net.minecraft.server.v1_8_R3.SystemUtils.a(SourceFile:44) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:774) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:378) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:713) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:616) [PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_144]
Caused by: java.lang.NullPointerException
at me.badbones69.crazyenchantments.multisupport.Support$SupportedPlugins.isPluginLoaded(Support.java:241) ~[?:?]
at me.badbones69.crazyenchantments.enchantments.Armor.onMovement(Armor.java:457) ~[?:?]
at sun.reflect.GeneratedMethodAccessor31.invoke(Unknown Source) ~[?:?]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_144]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_144]
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:300) ~[PaperSpigot.jar:git-PaperSpigot-"4c7641d"]
... 16 more

What faction plugin are you using?

FactionsPlus i'm sure

Ah, well CE doesn't support it but that's fine. The problem is how I am handling checking faction plugins. Thanks for the report. It is being fixed now and will be posted in a bit.

Would you recommend a Factions plugin?

I don't know which faction plugins are best but CE does support Factions by MassiveCraft, FactionsUUID, and Legacy Factions.

Ok thank you very much :)

Bug has been fixed and is being posted in a few minutes.

 Console Spam every Movement i make with Enchants applied.


Add tag @mixin \MyClass

Add tag @mixin \MyClass
neild3r/vscode-php-docblocker#120

Resolved in version 2.1.4


Update Readme with instructions  on finding the concourse ip

What
Add instructions to the readme on finding the concourse ip without logging into the console
How this PR should be reviewed
Read the readme and follow the instructions to try logging into concourse and bosh
I'm not sure if this should be merged or if we should think about moving these
instructions to the makefile, but i'm raising the PR for visibility into the
command I used in case it is useful to others
Who should review this PR
Anyone but me

when I was thinking about using jq once, @dcarley mentioned it's not mentioned in README as a dependency...

when I was thinking about using jq once, @dcarley mentioned it's not mentioned in README as a dependency...

Good catch. I think we can use the same trick from: https://github.gds/government-paas/aws-account-wide-terraform/pull/34#discussion_r9128
➜  paas-cf git:(master) ✗ aws ec2 describe-instances --filters 'Name=tag:Name,Values=concourse/0' "Name=key-name,Values=${DEPLOY_ENV}_key_pair" --query 'Reservations[*].Instances[*].PublicIpAddress' --output text
<IP_ADDRESS>

@dcarley @mtekel Agreed that is much better! Have updated it to use the jmespath expression

:+1:
I think it probably would be better in the Makefile. Though, I guess, what we really want is ssh-deployer and ssh-bosh targets, which is a bit more work. What do you think?
Related: https://www.pivotaltracker.com/story/show/113160603

I think it would be better in the Makefile ultimately, but the change to the Readme seems like a step in the right direction

Tested successfully


webpack loader

I am thinking about webpack loader which will recieve configuration object as parameter.
As a result, every time we require('bem-cn') we should receive preconfigured function.

You could configure block in separate module and import it in all components instead originally bem-cn. It may looks like this:
bem.js
import block from 'bem-cn';
export default block.setup(/* custom configuration */);

component.jsx
import block from './bem';

let b = block('popup');

let Popup = React.createClass({
    render() {
        return (
            <div className={b}>
                ...
            </div>
        );
    }
});

export default Popup

I am already using such approach. But for some reasons want to make it configurable through webpack. It is not an issue for current repo but the idea for new one.

But for some reasons

What reasons?

I want to provide an library which would just use defaults but will allow to redefine it on project level.

You should try to use webpack.DefinePlugin instead.


Add title property for Toolbox tools

Property value should be used as alt text for toolbox icons

resolved #540


Multiple reloads making it impossible to work properly

The extension attempts to reload the site being worked on multiple times making it effectively impossible to really work with it. I don't know when exactly this occurs, because it does not happen every time.

same here. has this been abandoned? Mine just kind of flickers.


NullPointerException in CliParser

Describe the bug
When analyzing a multi-module maven project, there's a NullPointerException in CliParser
** Version of dependency-check used **
The problem occurs using version 5.2.1 of the the cli via Jenkins
** Log file **
15:55:52 [DependencyCheck] Exception in thread "main" java.lang.NullPointerException
15:55:52 [DependencyCheck] 	at org.owasp.dependencycheck.CliParser.validatePathExists(CliParser.java:213)
15:55:52 [DependencyCheck] 	at org.owasp.dependencycheck.CliParser.validateArgs(CliParser.java:131)
15:55:52 [DependencyCheck] 	at org.owasp.dependencycheck.CliParser.parse(CliParser.java:90)
15:55:52 [DependencyCheck] 	at org.owasp.dependencycheck.App.run(App.java:112)
15:55:52 [DependencyCheck] 	at org.owasp.dependencycheck.App.main(App.java:80)
Please fix this or add logging, what's wrong.
I tried adding --log owasp.log, but no logfile was written.
Best regards

Can you please provide the exact pipeline command used?  The issue appears to be with the output path (i.e. -o ./reports).  I can see why this might cause an NPE after looking at the code - but I was hoping to know exactly what you had used.

Thanks for your reply!
It's a standard Jenkins job, not a pipeline. When I use the pipeline snippet generator with my configuration, I get
dependencyCheck additionalArguments: '''--format XML --format HTML --prettyPrint --out target''', odcInstallation: '5.x Latest'
where "5.x Latest" is 5.2.1, which was installed by Jenkins with "install from dl.bintray.com".
I do not use option -o. It should be the default value.

Can you try changing the --out target to --out ./target?

Yes, it works with --out ./target.
Thank you for your help!


Paddlepaddle init

Description of changes
having an initial stab at paddlepaddle.
This is far from perfect.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 243583 run on x86_64-linux 1

6 packages failed to build:

python310Packages.paddle_bfloat
python310Packages.paddle_bfloat.dist
python310Packages.paddlepaddle
python310Packages.paddlepaddle.dist
python311Packages.paddle_bfloat
python311Packages.paddle_bfloat.dist

@natsukium hey sorry to bother you, I'm kind of stuck on this package.
The error on ofborg points to the following issue on github https://github.com/PaddlePaddle/Paddle/issues/48681
The issues talks about paddle_gpu though, which doesn't make a lot of sense to me.
I was packaging this to get to paddleOCR eventually, apparently it's one of the best OCRs around.
if this looks too bad, feel free to say so, I'm happy to just close this. I just thought I might ask you first.

Can we build from the source following this instruction?
I think the openssl issue may be solved.
https://www.paddlepaddle.org.cn/documentation/docs/en/install/compile/linux-compile_en.html#compile-on-linux-from-source-code

FYI: I'm working off this branch to get PaddleOCR packaged. See my commits for the progress. At the time of writing, I have PaddleOCR working as a python package through which I got some text extraction using:
from paddleocr import PaddleOCR

ocr = PaddleOCR(use_angle_cls=True, lang='en', show_log=False)
ocr.ocr('./ocr-test-image.jpg', cls=True)

great work, @Avaq!
I look forward to your PR, even a draft is fine.


Error following Run deployment step (of .NET Tutorial - Deploy a microservice to Azure)

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/deploy-microservice-tutorial/deploy
Operating System: Windows
Running command of Run deployment-step results in following error:
error: unable to recognize "deploy-myMicroservice.yaml": no matches for kind "Deployment" in version "apps/v1beta1"
Resolution:
Change apiVersion from apps/v1beta1 to apps/v1
Running same command again will result in new error:
error: error validating "deploy-myMicroservice.yaml": error validating data: ValidationError(Deployment.spec): missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec; if you choose to ignore these errors, turn validation off with --validate=false
Resolution:
In kind: Development, inside spec, append following:
  selector:
    matchLabels:
      app: mymicroservice

Now it should run!
This is how deploy-myMicroservice.yaml should be updated on you page:
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: mymicroservice
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: mymicroservice
    spec:
      containers:
      - name: mymicroservice
        image: [YOUR DOCKER ID]/mymicroservice:latest
        ports:
        - containerPort: 80
        env:
        - name: ASPNETCORE_URLS
          value: http://*:80
  selector:
    matchLabels:
      app: mymicroservice
---
apiVersion: v1
kind: Service
metadata:
  name: mymicroservice
spec:
  type: LoadBalancer
  ports:
  - port: 80
  selector:
    app: mymicroservice

Thanks for the feedback @stjepan. I've sent your suggestion for review.

@mairaw  is there a way open a PR for this Fix.

Thank you - I was stuck and this really helped me complete and understand as well!


description

I think it should say "three.js for visualization" in the description.

Yeah, I think that too)


Bug: dependency resolution conflicts in 0.12.4

Hello, I updated 0.12.2 to 0.12.4 and after it npm install throws an error:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@lexical/code
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@lexical/markdown
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@lexical/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   1 more (@lexical/react)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/lexical
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@lexical/code
npm ERR<EMAIL_ADDRESS>from the root project

All packages were updated to the same version:
package.json
{
  "dependencies": {
    ...
    "@lexical/code": "0.12.4",
    "@lexical/link": "0.12.4",
    "@lexical/list": "0.12.4",
    "@lexical/markdown": "0.12.4",
    "@lexical/react": "0.12.4",
    "@lexical/rich-text": "0.12.4",
    "@lexical/selection": "0.12.4",
    "@lexical/utils": "0.12.4",
    ...
    "lexical": "0.12.4",
}

works with --force ofc.
Maybe related: #5245

sorry, fixed with rm -rf node_modules + remove package-lock.json :\


feat: add node-pre-gyp support

@mapbox/node-pre-gyp is a popular choice for serving prebuilt native node module binaries. These changes add support for downloading prebuilt binaries using it as well as rebuilding in the case that no prebuilt binaries are found.
A skipPrebuilds option was added to give tests the ability to force rebuilds without using any prebuilt binary modules.
node-sqlite3 is used to test for node-pre-gyp support.
resolves #367

Codecov Report

Merging #1095 (c66ea6c) into main (07c4261) will increase coverage by 0.32%.
The diff coverage is 82.50%.

@@            Coverage Diff             @@
##             main    #1095      +/-   ##
==========================================
+ Coverage   76.01%   76.34%   +0.32%
==========================================
  Files          20       21       +1
  Lines         688      727      +39
  Branches      129      137       +8
==========================================
+ Hits          523      555      +32
- Misses        118      121       +3
- Partials       47       51       +4

Files Changed
Coverage Δ

src/types.ts
100.00% <ø> (ø)

src/module-type/node-pre-gyp.ts
76.00% <76.00%> (ø)

src/module-rebuilder.ts
91.66% <92.85%> (+0.14%)
:arrow_up:

src/rebuild.ts
72.82% <100.00%> (+0.29%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

o shoot, I merged before sam's comments loaded :PPP I'll make a new pr to address these


Method eth_maxPriorityFeePerGas not supported

Getting this error when building the transaction the first time at roughly 4:00:00 into the tutorial
Traceback (most recent call last):
File "deploy.py", line 54, in
transaction = Storage.constructor().buildTransaction(
File "/Users/j/Library/Python/3.8/lib/python/site-packages/eth_utils/decorators.py", line 18, in _wrapper
return self.method(obj, *args, **kwargs)
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/contract.py", line 684, in buildTransaction
return fill_transaction_defaults(self.web3, built_transaction)
File "cytoolz/functoolz.pyx", line 250, in cytoolz.functoolz.curry.call
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/_utils/transactions.py", line 121, in fill_transaction_defaults
default_val = default_getter(web3, transaction)
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/_utils/transactions.py", line 71, in
web3.eth.max_priority_fee + (2 * web3.eth.get_block('latest')['baseFeePerGas'])
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/eth.py", line 549, in max_priority_fee
return self._max_priority_fee()
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/module.py", line 57, in caller
result = w3.manager.request_blocking(method_str,
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/manager.py", line 198, in request_blocking
return self.formatted_response(response,
File "/Users/j/Library/Python/3.8/lib/python/site-packages/web3/manager.py", line 171, in formatted_response
raise ValueError(response["error"])
ValueError: {'message': 'Method eth_maxPriorityFeePerGas not supported.', 'code': -32000, 'data': {'stack': 'Error: Method eth_maxPriorityFeePerGas not supported.\n    at GethApiDouble.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/lib/subproviders/geth_api_double.js:70:16)\n    at next (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:136:18)\n    at GethDefaults.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/lib/subproviders/gethdefaults.js:15:12)\n    at next (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:136:18)\n    at SubscriptionSubprovider.FilterSubprovider.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/subproviders/filters.js:89:7)\n    at SubscriptionSubprovider.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/subproviders/subscriptions.js:137:49)\n    at next (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:136:18)\n    at DelayedBlockFilter.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/lib/subproviders/delayedblockfilter.js:31:3)\n    at next (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:136:18)\n    at RequestFunnel.handleRequest (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/lib/subproviders/requestfunnel.js:32:12)\n    at next (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:136:18)\n    at Web3ProviderEngine._handleAsync (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:123:3)\n    at Timeout._onTimeout (/Applications/Ganache.app/Contents/Resources/static/node/node_modules/ganache-core/node_modules/web3-provider-engine/index.js:107:12)\n    at listOnTimeout (internal/timers.js:531:17)\n    at processTimers (internal/timers.js:475:7)', 'name': 'Error'}}

Hello, Please share the .sol and .py file so that we can help you find the solution to your problem.

deploy.py
from solcx import compile_standard, install_solc
import json
from web3 import Web3

install_solc("0.6.6")

with open("./Storage.sol", "r") as file:
    storage_file = file.read()
    print(storage_file)

# Compile the Solidiy Code
compiled_sol = compile_standard(
    {
        "language": "Solidity",
        "sources": {"Storage.sol": {"content": storage_file}},
        "settings": {
            "outputSelection": {
                "*": {"*": ["abi", "metadata", "evm.bytecode", "evm.sourceMap"]}
            }
        },
    },
    solc_version="0.6.6",
)

with open("compiled_code.json", "w") as file:
    json.dump(compiled_sol, file)

# get bytecode
bytecode = compiled_sol["contracts"]["Storage.sol"]["Storage"]["evm"]["bytecode"][
    "object"
]
# get ABI
abi = compiled_sol["contracts"]["Storage.sol"]["Storage"]["abi"]

# For connecting to ganache
w3 = Web3(Web3.HTTPProvider("http://<IP_ADDRESS>:7545"))
chain_id = 1337
my_address = "0x4bd7cD8F349A832AB75a92Ac9Bb0DB08d27D5A86"
private_key = "7bff858e8179ab40091fbf93a7d948dffe80e8ce51d5320a3e47a7f32b41881e"

# Create the contract in python

Storage = w3.eth.contract(abi=abi, bytecode=bytecode)
print(Storage)

# Get Latest transaction count for nonce

nonce = w3.eth.getTransactionCount(my_address)
print(nonce)

# 1. Build a transaction
# 2. Sign a transaction
# 3. Send a transaction
transaction = Storage.constructor().buildTransaction(
    {"chainId": chain_id, "from": my_address, "nonce": nonce}
)

print(transaction)

Storage.sol
//SPDX-License-Identifier: MIT
pragma solidity ^0.6.0;

contract Storage {
    //creates balance
    uint256 public balance;
    //defines Accounts structure
    struct Accounts {
        string name;
        uint256 balance;
    }
    //Generates Array of accounts
    Accounts[] public accounts;

    //Creates mapping for accounts
    mapping(string => uint256) public nameToBalance;

    //stores balance values
    function store(uint256 _balance) public {
        balance = _balance;
    }

    //retrieves balance value
    function retrieve() public view returns (uint256) {
        return balance;
    }

    //adds an account
    function addAccount(string memory _name, uint256 _balance) public {
        accounts.push(Accounts(_name, _balance));
        nameToBalance[_name] = _balance;
    }
}

Try changing the w3 line to
w3 = Web3(Web3.HTTPProvider("http://<IP_ADDRESS>:8545"))

And along with it add this line which when omitted might lead to errors
"gasPrice": w3.eth.gas_price,

In your transaction under the buildTransaction()

The port was correct as 7545 however, adding that gas price line did fix it! Thanks!


jsonschema2csclient does not support $ref to other files

As the schema is loaded without keeping track of its document path, it is not possible to reference schema from other files.

Please fix this issue

Ref: https://github.com/RicoSuter/NJsonSchema/pull/1098


Better errors for a missing self.__WB_MANIFEST

Library Affected:
workbox-webpack-plugin, workbox-build
I've heard from new developers using injectManifest that it can be difficult to trace the error message associated with a missing self.__WB_MANIFEST to an actionable solution.
We should update the error logic at
https://github.com/GoogleChrome/workbox/blob/e0bcb2984ab4881f0f52ae245be999aa452bba18/packages/workbox-webpack-plugin/src/inject-manifest.js#L316
to use the hardcoded error string that workbox-build uses, and then update that string at
https://github.com/GoogleChrome/workbox/blob/e0bcb2984ab4881f0f52ae245be999aa452bba18/packages/workbox-build/src/lib/errors.ts#L70-L71
so that the message includes a short link to a web page that explains the purpose of self.__WB_MANIFEST.

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


luci-app-attendedsysupgrade: enforce correct imagebuilder version

By passing the version_code parameter, we can error out if the imagebuilder that responded to the request isn't the same, mirroring the current behavior of auc.

This is spawned from a recent issue with the Attended Sysupgrade server as can be seen in later entries of https://forum.openwrt.org/t/auc-sysupgrade-does-not-work-for-me/196782, https://github.com/openwrt/asu/issues/848 and with the probable root cause in https://github.com/openwrt/asu/issues/853. Due to the LuCI app not having this check, the incorrect version of the imagebuilder may be used for a request, and still be returned to the user as a valid request, which (I guess?) in worst case could downgrade a user's systems instead.
Adding this check mirrors the behaviour that auc currently has, see image below. The relevant code path for the Attended Sysupgrade code is at https://github.com/openwrt/asu/blob/main/asu/build.py#L72-L79, which enforces this check on the API.

[X] This PR is not from my main or master branch :poop:, but a separate branch :white_check_mark:
[X] Each commit has a valid :black_nib: Signed-off-by<EMAIL_ADDRESS>row (via git commit --signoff)
[X] Each commit and PR title has a valid :memo: <package name>: title first line subject for packages
[ ] Incremented :up: any PKG_VERSION in the Makefile
[X] Tested on: (x86/64, SNAPSHOT r26637-05aec66d53, Firefox) :white_check_mark:
[X] ( Preferred ) Mention: @ the original code author for feedback @aparcar
[X] ( Preferred ) Screenshot or mp4 of changes:
[ ] ( Optional ) Closes: e.g. openwrt/luci#issue-number
[ ] ( Optional ) Depends on: e.g. openwrt/packages#pr-number in sister repo
[X] Description: (describe the changes proposed in this PR)

Dan the man with the big fixes!
Do you ha e any opinion here @aparcar?

Sure let's do it

@systemcrash @aparcar I may have implemented this check incorrectly; is the version_code supposed to be the expected remote revision for the branch you specify (23.05, SNAPSHOT etc.) and not the currently running revision? I'm looking at the API docs at https://sysupgrade.openwrt.org/ui/#/ but I'm actually not sure what the docs mean by It is possible to send the expected revision.
I almost guess it should be the remote revision, since you want to verify that the imagebuilder you get back is for the version you requested to upgrade to. In that case it should not be revision that version_code is set to, but the latest remote revision, which would be the remote_revision variable.

Let's wait to hear from @aparcar who can likely clarify.
It sounds like the revision that you are aiming for, and not the one you
are currently on. Because for all intents and purposes, I don't know of any
openwrt version which actually cares about the version you're currently on.
That might become interesting for stats at some point although nothing
more.

After testing, I'm right now unable to upgrade SNAPSHOT through the LuCI app due to this. Looking at how firmware selector handles the version_code at https://github.com/openwrt/firmware-selector-openwrt-org/blob/master/www/index.js#L106, the #image-code selector is the latest revision (it obviously needs to be cause the firmware selector can't know the current revision cause it's not running on the device itself), so I'm 100% sure it needs to be the remote one, I've created #7166 to hopefully solve it.

is the version_code supposed to be the expected remote revision for the branch you specify

Yes, from the new version to which you are upgrading.  It's in the platform file for the device, for example, open this one and look down at the bottom:
https://sysupgrade.openwrt.org/json/v1/snapshots/targets/x86/64/generic.json


Update scala-java-time to 2.2.2

Updates io.github.cquiroz:scala-java-time from 2.2.0 to 2.2.2.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.cquiroz", artifactId = "scala-java-time" } ]

labels: library-update, semver-patch

@mergifyio update


Deprecation Warning: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0

26 │ @import "breakpoint/settings";
   │         ^^^^^^^^^^^^^^^^^^^^^
   ╵
    node_modules/breakpoint-sass/stylesheets/_breakpoint.scss 26:9  @use

Deprecation Warning: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

Will this package be updated to address this issue? I've attempted to fix the issue myself, but got stuck om some issues I couldn't resolve. In case it helps, I could commit and push what I've done so far. It's mostly about replacing @import with @use and @forward.

I'm also going to need to fix this.
@StanH-Swis can you create a PR, I'd love to review it.

I'll work on this over December.On Nov 28, 2024, at 10:01 AM, Lukas von Blarer @.***> wrote:
I'm also going to need to fix this.
@StanH-Swis can you create a PR? I'd love to review it.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

I've made the PR: https://github.com/at-import/breakpoint/pull/195

@luksak, Can you please check the PR above? It breaks a build task in our project :(

Hey all, I've just published a beta that should work with the current version of Sass. Please check it out and provide feedback; will do a full release after I've done a little more testing.
Thanks @StanH-Swis for your PR; I used it as the basis for my refactor.
https://www.npmjs.com/package/breakpoint-sass/v/4.0.0-beta.1


Selected category pane wrong if no category selected

When you select a class, by default no method category is selected, but the yellow pane above the method list gets set to one of the method categories, not usually the first one - no customer-visible logic.
If you select a category, this pane is updated correctly. But then if you unselect it (no category selected), it reverts to displaying this default category name, although I think the methods in the list are correct (it shows all methods).
Marking this as a P3 since it baffled me as I was trying to find a method that tends to hide.

When selecting classes, if method was selected try to find selector in new class''s method list. If not found, don't select any methods.

Reopening.... behavior is a bit wonky.
If I select a new category, the pan should update to show the category, rather than staying blank, and sometimes it shows just "Category:"  Tests are changing categories and unselecting methods and changing categories and so on.
It's also not reliably showing the template when a category is selected.

Behavior should be more consistent now per commit comment. Needs review.

New issue: when I first select a class, such that no category is selected, the list of methods is displayed, but it doesn't have a scrollbar.  I have to actually select a category to be able to see the methods past the first 10 or so. This is actually very annoying.
Another issue: if you have no categories selected, and you select a method, you can see its category. If you select that category, funny things happen - you may lose the method selection. Sometimes an unrelated method is selected.
Annoying: when you select a class, the categories list is repainted two or three times, with a quite visible flicker.
Previous complaints look fine.

New issue: when I first select a class, such that no category is selected, the list of methods is displayed, but it doesn't have a scrollbar. I have to actually select a category to be able to see the methods past the first 10 or so. This is actually very annoying.

This should be fixed in this commit.
https://github.com/GemTalk/JadeiteForPharo/commit/842840e66b531f81aefdad0bc5ba749b8ff35f68

Annoying: when you select a class, the categories list is repainted two or three times, with a quite visible flicker.

Should be fixed: https://github.com/GemTalk/JadeiteForPharo/commit/65d1ed9bdbf67f281e65483cbf39c8c164c7c1a4

Another issue: if you have no categories selected, and you select a method, you can see its category. If you select that category, funny things happen - you may lose the method selection. Sometimes an unrelated method is selected.

I think this is better as of https://github.com/GemTalk/JadeiteForPharo/commit/debdd5cabe54ddc72a4e7533c783d7ea34f86c45

Found another bug in category selection which selected the wrong method by default. Made the method selection seem random.
https://github.com/GemTalk/JadeiteForPharo/commit/44873e7f4d7f78def45279d694db650ba8e9e7a7


Column/Row Header Nesting in Table Visualization

I would like to have a features some thing like https://github.com/angular-ui/ng-grid/issues/2468, column group header and row group header. Is it possible?

Great idea. Not at the top of our priority list, but definitely something that we'd like to implement.

i too have the same issue. is this issue implemented or is there any alternative for grouping header columns.


feat: redeploy

:tada: This PR is included in version 1.158.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


add / between base and route

Deepkit allows for the following:
@http.controller('/some/base')
export class MyController {
  @http.GET('status')
  async getStatus() {
    // ...
  }
}

Deepkit will generate a route with URL /some/base/status.  However, the generated API spec does not, instead producing /some/basestatus.
Before:

After:

Thank you for your prs!


[giflib] Changed library linkage to static.

The CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS flag prevents giflib from being built with a /GL and /LTCG toolchain.
Tested with this:https://github.com/qis/toolchains/blob/075a467b671cc08b3f94c1f37ef503bc1627f2fe/windows.cmake
Please note that the portfile.cmake file does not explicitly set CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS, but it is set in giflib-5.1.4/CMakeLists.txt line 22 when BUILD_SHARED_LIBS is enabled.
I have found no other support for shared libraries on Windows in the giflib port. All functions are hard-coded as extern or don't have any linkage specifier at all.

I am a little bit against all this static linking you are pushing "everywhere".
It is not something I'd support, because static linking is not always an option. Also, reducing opportunities is not something in my mindset.
What is refraining you from just using your own triplet, instead of forcing the option loosing to everyone? In this case also it is even more disturbing, since the author itself was using this mechanism to create a shared library and we were not even applying it with a patch. I can't understand.

@cenit As I understand it from this discussion, WINDOWS_EXPORT_ALL_SYMBOLS is a hack with multiple drawbacks:

65535 symbol limit
code bloat
reduced performance
incompatible with some optimizations

It would be interesting to know what is true and how many users (and potential users) of vcpkg need /LTCG support or are not OK with this hack.

The main question is: why should we block interested user from exploiting a dynamic link? Just create your own triplet, with static linking for libraries and dynamic for crt, and you should be ok, without limiting others. Or not? Interested users can already do it, without impacting people that does not need it or that require dynamic linking.
For code bloat and whatsoever, with gcc and shared options do something similar from the beginning (since the dll export concept and function visibility is something not native there), and nobody ever complained. Of course it is bloated with respect to selected export, but if it is the author choice (again in this case especially  they are doing so) who are we to block it?!??

Yes, good points. I'm not sure what the exact differences to the Linux ELF model but I'm pretty sure that they are substantial.
By declaring BUILD_SHARED_LIBS the vcpkg already makes a decision. The authors do not declare their libraries as add_library(... SHARED ...), but chose to support this flag (which is not on by default).
Given the drawbacks of WINDOWS_EXPORT_ALL_SYMBOLS I'm not convinced that the authors actually know what they're doing. It could be that they do. Especially when it's a C library like in this case. But I doubt it.

In the contrary, since it is a c library, the symbol proliferation due to templates is not present. And I think the author may know better than us their product...
Again, as I already said, I don’t see any valid point on your side. No strong reason to remove dynamic linking to everybody since with a proper triplet you can have your necessities satisfied without limiting others. My vote would be to revert all your “static linking” recent commits...

@cenit While it would be reasonable to revert the tiff commit, I'd suggest you talk to other vcpkg team members before you revert the changes to the C++ libraries cpr and fmt.
@ras0219-msft What is your take on it?

Of course I can accept community decision, also this one. It is just a trend that I don’t like and I wanted to explain my position before it was too late. Another point: For now no big deal about licenses. But what about if you made only static a library whose licensing required me to open up the source code if static linked? vcpkg is used a lot also in private companies and there might be some serious concern about using it more, if you have to constantly check if libraries become only static, without any message, one day out of the blue, just because you wanted to update...

@cenit Since we have strict no (L)GPL license requirements, I personally wouldn't have touched those. But this is a very good point.
May I suggest adding some guidelines regarding this and similar issues to the documentation? Perhaps in the docs/maintainers/vcpkg_check_linkage.md file? I did not intend to step on anybody's toes.

Yep, I noticed no license damage for now. Not sure if it was intended or by chance, so I raised up my objection and wanted to make it clear, again, before it was too late. I agree with you that maybe some documentation or guideline or whatever might just be enough

@cenit @qis Thanks for opening this discussion!
After considering the significant downsides of WINDOWS_EXPORT_ALL_SYMBOLS, we do not consider it valid shared library support on windows unless the author of the library explicitly wants to opt into it (and even in that case, we are largely against it).
This means for libraries like giflib where we are providing the cmake file, the lack of a .def file and a lack of __declspec() declarations means that the library simply does not support shared builds for windows. A hack that sometimes works is insufficiently high quality for vcpkg to declare support for that configuration.
The above said, it is important to enable users to opt into advanced or custom scenarios assuming they are fully aware of the downsides. In this case, it is as simple as removing the vcpkg_check_linkage() line and adding a -DCMAKE_WINDOWS_EXPORT_ALL_SYMBOLS=ON declaration to the vcpkg_install_cmake() command.

Understood. Just to have it even more clear, does this mean that moving on every library in which we explicitly enable the CMake flag WINDOWS_EXPORT_ALL_SYMBOLS that receives an update should be cleaned up from it?

Yes, for every library that we are providing buildsystem replacements for, we should avoid/remove uses of WINDOWS_EXPORT_ALL_SYMBOLS. If you'd like to do the grep+PR that would be awesome; otherwise we'll put it on our backlog to address once we've sufficiently drained the current PR queue.
I'll note that it (WINDOWS_EXPORT_ALL_SYMBOLS) only "activates" if you ask for a shared build, so we don't even necessarily need to remove it from the build scripts -- we just need to add vcpkg_check_linkage(ONLY_STATIC_LIBRARY) to the top.

You mean this one https://github.com/Microsoft/vcpkg/commit/8feeb8d8134221ae12660d56213c4fe44ee9d0b5 ?
I did it mechanically few hours ago, while thinking for some real algorithms. It was a gift trying to convince you to review that PR, so that we can merge it...


Add windows section for troubleshooting purposes

@JonRowe here is the section I added to troubleshoot windows issues. Thanks.

Thanks @rsanchez79 :)


We need redis authentication

redis authentication is missing

Since 6a429c2, this has been supported.  That does, unfortunately, mean using master instead of 1.2, but the feature has already been implemented.

Thank you so much.


Enable release page.

It would be nice to have the Github Releases page activated using semantic versioning.

Done!


Move some proto definitions to admin package

What changed?
Move some proto type definitions from shared package to admin package.
Additionally added .gitattributes with .gen linguist-generated=true to collapse generated files in PRs by default.

Why?
This is a preparation step before moving out admin/v1 to cadence-idl repository. This is in turn need for WebUI grpc migration.
This change is safe, as types themselves did not change and thus they remain wire compatible.

How did you test it?
Deployed branch to staging2 environment.
Used CLI with grpc transport to issue admin cluster describe command.

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 0181428c-d002-48b2-a921-17788eadbcfd

250 of 284 (88.03%)  changed or added relevant lines in 1 file are covered.
52 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.02%) to 58.003%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/types/mapper/proto/shared.go
250
284
88.03%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

client/history/client.go
2
38.05%

client/history/metricClient.go
2
45.3%

common/membership/hashring.go
2
83.54%

common/task/fifoTaskScheduler.go
2
85.57%

service/history/execution/mutable_state_builder.go
2
69.09%

service/history/handler.go
2
47.6%

service/matching/matcher.go
2
91.46%

service/history/task/fetcher.go
3
91.79%

service/frontend/workflowHandler.go
4
60.28%

Totals

Change from base Build 01814080-1488-4d1e-819f-14f260bf4b9e:
-0.02%

Covered Lines:
83989

Relevant Lines:
144800

💛  - Coveralls

Not at all opposed to this change, but I'm slightly scared that some of the monorepo tooling might expect IDL files to never be moved or renamed. Might I suggest just getting a stamp from @moisesvega  or some folks from the IDL group?

Not at all opposed to this change, but I'm slightly scared that some of the monorepo tooling might expect IDL files to never be moved or renamed. Might I suggest just getting a stamp from @moisesvega or some folks from the IDL group?

ignore me, I didn't know these weren't imported


[Win 7] Installation failed with wrong message about requirements

Today (~16pm KGT) I downloaded dev version of Windows installer from subutai.io. But installation failed.
I am not sure if this is error message or just reminding a user. But it gives an impression that I don't have Internet connection, although I have (could ping successfully Hub).

Installation failed with this message:

But seems my PC config meets those requirements.

Also, I can find any logs in C:/Subutai/bin.

First aid - to run IExplorer and lower security level.

Can not reproduce this error on fresh system. Om user's system: stopped Windows firewall and windows defender services - but it did not help. After that run Internet Explorer (it used to define if Internet connection present and lowered security level) - and Installer started to work. Need some research.

Situation was reproduced when network adapter connected to Internet was disabled in time of download. Installation was finished, adapter enabled, ping worked. But installer showed "No Internet connection". Started to work after refresh page (any) in Internet Explorer (simply first action in Internet Explorer?)


Where are the type 2 diabetes patients in the Metabolic Syndrome Module?

I generated 10,000 patients first using -m Metabolic-Syndrome-Disease and then looked at conditions. While there were many diagnoses related to coronary disease, there were none related to diabetes and no diabetic medications. I then looked at 100 patients using the module Metabolic-Syndrome-Care, and found none. Lastly, I inputted the two Metabolic Syndrome Modules and generated 100 patients but again no diagnoses related to diabetes and no diabetic medications. Here is an example of the code run
java -jar synthea-with-dependencies.jar -p 10000  -m Metabolic Syndrome Disease  --exporter.csv.export true

I noticed on this Wiki [https://github.com/synthetichealth/synthea/wiki/Developer-Setup-and-Running) that the module was metabolic*. That worked completely different than what I tried before, and I retrieved some realistic results

I'm not surprised Metabolic-Syndrome-Disease would not work because it is not an intelligent match algorithm, it only matches on exact/partial strings (case insensitive, but the Dash characters (-) will ruin the match).

I have reviewed all modules and didn't see any short codes or abbreviations. Where is a list of the proper module code to insert in the request?

If memory serves, the match is against the module names, which is not the filename but inside the json as the "name" property. Coincidentally, the filename and the name often match, but not always.

Thanks for the Synthea initiative. I'm an elderly internal medicine physician who teaches machine learning to other physicians and I have been frustrated by the paucity of large healthcare datasets at the patient level. I'll create a few models this week to predict heart disease, diabetes, etc. I appreciate the help.

You are welcome. Let us know how it goes training and running your models. We would be interested in hearing any issues you run into with the data.

I don't like to bother you but I am currently concentrating on the COVID-19
dataset. I emulated the Python exercises you all added to the dataset in my
own Jupyter Lab and everything worked except for the commands that began
with "analysis". For example analysis.outcome_table(inpatient_ids,
survivor_ids, deceased_patients, conditions). The error message was always
"analysis not defined" and in fact I could not find code related to it
Thanks....Bob
On Mon, Sep 14, 2020 at 6:16 PM Jason Walonoski<EMAIL_ADDRESS>wrote:

You are welcome. Let us know how it goes training and running your models.
We would be interested in hearing any issues you run into with the data.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/synthetichealth/synthea/issues/791#issuecomment-692367486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQPL4YOLUFUBW6ZRPWDF7A3SF2P2PANCNFSM4RL7UQZQ
.

--
Robert (Bob) Hoyt MD, FACP, FAMIA, ABPM-CI
Associate Clinical Professor, Department of Internal Medicine
Virginia Commonwealth University
Richmond, VA
CAPT (Ret) USN
InformaticsEducation.org http://InformaticsEducation.org
<EMAIL_ADDRESS>Cell:<PHONE_NUMBER>

Bob, the analysis.py file is here: https://github.com/synthetichealth/module-validation/blob/master/notebooks/analysis.py
Hope that helps.


router.back and router.go can pass arguments

What problem does this feature solve?
Imagine this scenario:
1.I go back from a child page to a parent page
2.The parent page has a list that needs to be refreshed based on the status returned by the child pages
3.I don't want to use pinia to store global state. Because many pages need to return some different information.
What does the proposed API look like?
back(params:Record<string,any>):void
go(delta:number,params:Record<string,any>):void

Duplicate of https://github.com/vuejs/router/issues/1646


Feature request: Add isTraceSampled method to Tracer

Use case
I'd like a attribute in my logs to indicate whether or not a request has been sampled. Something like this:
{
  "level": "ERROR",
  "msg": "something went wrong",
  "trace_id": "1-6419d889-20172e50462fac2c7c481454",
  "trace_sampled": true
}

I can populate the trace_id using Tracer#getRootXrayTraceId , but there's no way to find out if the trace is sampled.
Solution/User Experience
A new isTraceSampled method on the Tracer class.
log({
  level: "ERROR",
  msg: "something went wrong",
  trace_id: tracer.getRootXrayTraceId(),
  trace_sampled: tracer.isTraceSampled(),
})

Alternative solutions
Implement my own isTraceSampled function which parses the _X_AMZN_TRACE_ID header value.
Acknowledgment

[X] This feature request meets Lambda Powertools Tenets
[x] Should this be considered in other Lambda Powertools languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

Return type of the method, considering that the Trace data might be undefined:

boolean | undefined

Where/how this new method should be implemented (I see some common logic with this existing method)

Add a EnvironmentVariableService#getXrayTraceSampled method.
Use that method inside the Tracer#isTraceSampled

Agree on a name for the method that conveys its meaning:

isSampled
isTraceSampled
getTraceSampled
getRootXrayTraceSampled

What type of changes should be made to the documentation to articulate the feature/method

Not sure

Return type of the method, considering that the Trace data might be undefined:

boolean | undefined

I think we could get away with just boolean. If there's no trace, then it's not sampled.

Doesn't matter much to me since if (tracer.isTraceSampled()) { will work either way.

I agree, let's stick to boolean only and assume that a missing value equals to false as it means no tracing can happen.

Where/how this new method should be implemented (I see some common logic with this existing method)

Add a EnvironmentVariableService#getXrayTraceSampled method.
Use that method inside the Tracer#isTraceSampled

I agree with the second point.
Regarding the first one, we are in a good direction but I would try to make an effort to not have duplicate logic between the existing getXrayTraceSampled and new isXrayTraceSampled method.
The current implementation for the former is:
public getXrayTraceId(): string | undefined {
  const xRayTraceId = this.get(this.xRayTraceIdVariable);

  if (xRayTraceId === '') return undefined;

  return xRayTraceId.split(';')[0].replace('Root=', '');
}

Given that both methods now need to access the tracing header, I would consider extracting part of the logic and reuse it for both, kind of like:
private getXrayTraceData: string[] {
  const xRayTraceData = this.get(this.xRayTraceIdVariable);
  if (xRayTraceData === '') return undefined;

  return xRayTraceData.split(';');
}

public getXrayTraceId(): string | undefined {
  const xRayTraceData = this.getXrayTraceData();

  if (xRayTraceData.length === 0) return undefined;

  return xRayTraceData[0].replace('Root=', '');
}

public isXrayTraceSampled(): boolean {
  const xRayTraceData = this.getXrayTraceData();

  // TODO: implement
}

Agree on a name for the method that conveys its meaning:

I feel like isTraceSampled is pretty clear. If you disagree, here are some other options:

isSampled
isTraceSampled
getTraceSampled
getRootXrayTraceSampled

Okay. I agree that isTraceSampled is fairly clear and aligns with the fact that we are expecting a boolean value, so I think we can safely rule out the ones with get.
With that said, I think it would be more consistent to have it called isXrayTraceSampled instead.

What type of changes should be made to the documentation to articulate the feature/method

Not sure

I'm available to help with the editing during the review phase, but the PR for this feature will have to include changes to the corresponding docs as well.
Generally speaking, I think the existing section currently titled "Access AWS X-Ray Root Trace ID" should be changed to something like "Access AWS X-Ray Trace Data", and its content should reflect that:

Now there are two methods: getRootXrayTraceId() and isXrayTraceSampled()
Explain that these two methods can be used to access the X-Ray trace data present in the tracing header
Keep the existing content about getRootXrayTraceId()
Add a new piece that discusses isXrayTraceSampled(), touches on when it can be useful, shows a piece of code (i.e. similar to the one in your OP)

If you agree with the points/direction, let me know and I'll assign the issue to you and you can start working on a PR.

Sounds good to me.

Hi @icholy, I just wanted to check if you were still working on this feature.
If not, please let me know so I can put it back on the backlog.

@dreamorosi I'm on holiday. I'll be back next week.

Working on this now.

@dreamorosi will there be a release soon?

Yes, I'm planning a release this week. It'll include the feature


Mouse flashing in every RetroWrapper version, but not in the vanilla version.

Check 1.3.1 and tell if the issue still occurs.
I would also like more information. Are you using hacks? OS/Java version etc

Java(TM) SE Runtime Environment (build 1.8.0_161-b12) Windows 10 with hacks enabled. Updating doesn't solve the issue. @kcxx

Not getting this issue. Java 1.8.0_161 x64, Windows 10 Home x64.
Build: https://uk.pcpartpicker.com/list/3jrrr7

@HalfOfAKebab
Do you have hacks enabled? If not then I found the issue

What do you mean?

Just tested @kcxx, hacks are the cause of it.

Looks like it only does that while you are moving. JFrame updates are the cause.


Following wiki, always getting 'Some services are not able to be constructed

I have created a .NET Core 6.0 Webapplication and installed the following packages:

OpenCqrs (6.5.0)
OpenCqrs.Caching.Memory (6.5.0)
OpenCqrs.Store.EF (6.5.0)
OpenCqrs.Store.EF.SqlServer (6.5.0)
OpenCqrs.UI (6.5.0)
OpenCqrs.Validation.FluentValidation(6.5.0)

I have followed the wiki Configuration as followed:
In my .net core 6 web project in de Program.cs i have the following configuration:
var connectionString = builder.Configuration.GetConnectionString("DefaultConnection");

builder.Services.AddOpenCqrs(options =>
    {
        options.PublishEvents = true;
        options.SaveCommandData = true;
    }, typeof(CreateOrganisation))
    .AddSqlServerStore(options => { options.ConnectionString = connectionString; })
    .AddEFStore()
    .AddFluentValidation(options => { options.ValidateAllCommands = false; })
    .AddMemoryCache(options => { options.DefaultCacheTime = 60; })
    .AddUI();

var app = builder.Build();

app.UseOpenCqrs().EnsureDomainDbCreated();

When i use migrations update-database i got the following errors:
An error occurred while accessing the Microsoft.Extensions.Hosting services.
Continuing without the application service provider.
Error: Some services are not able to be constructed
(Error while validating the service descriptor 'ServiceType: OpenCqrs.Extensions.IOpenCqrsAppBuilder Lifetime: Transient ImplementationType: OpenCqrs.Extensions.OpenCqrsAppBuilder': Unable to resolve service for type 'Microsoft.AspNetCore.Builder.IApplicationBuilder' while attempting to activate 'OpenCqrs.Extensions.OpenCqrsAppBuilder'.)
(Error while validating the service descriptor 'ServiceType: OpenCqrs.Extensions.IOpenCqrsServiceBuilder Lifetime: Transient ImplementationType: OpenCqrs.Extensions.OpenCqrsServiceBuilder': Unable to resolve service for type 'Microsoft.Extensions.DependencyInjection.IServiceCollection' while attempting to activate 'OpenCqrs.Extensions.OpenCqrsServiceBuilder'.)
(Error while validating the service descriptor 'ServiceType: System.Runtime.Serialization.ISerializable Lifetime: Transient ImplementationType: OpenCqrs.Exceptions.ConcurrencyException': Unable to resolve service for type 'System.Guid' while attempting to activate 'OpenCqrs.Exceptions.ConcurrencyException'.)
(Error while validating the service descriptor 'ServiceType: System.Runtime.Serialization.ISerializable Lifetime: Transient ImplementationType: OpenCqrs.Exceptions.HandlerNotFoundException': Unable to resolve service for type 'System.Type' while attempting to activate 'OpenCqrs.Exceptions.HandlerNotFoundException'.)
(Error while validating the service descriptor 'ServiceType: System.Runtime.Serialization.ISerializable Lifetime: Transient ImplementationType: OpenCqrs.Exceptions.QueryException': Unable to resolve service for type 'System.String' while attempting to activate 'OpenCqrs.Exceptions.QueryException'.)
(Error while validating the service descriptor 'ServiceType: System.Runtime.Serialization.ISerializable Lifetime: Transient ImplementationType: OpenCqrs.Exceptions.ValidationException': Unable to resolve service for type 'System.String' while attempting to activate 'OpenCqrs.Exceptions.ValidationException'.)

What am i missing here?
Thank you for the help

Hi @NathalieSpierings, I'll try to reproduce it.

Okay, the reason is explained here.
For the time being it's possible to use OpenCQRS in .NET 6 by using the old pattern with Startup.cs.

Yes this solution ideed worked for me.
Thank you

Glad it worked!